Skip to content

3 scout targets should be updated each time it is saved if not created#5

Merged
jchate6 merged 6 commits intodevfrom
3-scout-targets-should-be-updated-each-time-it-is-saved-if-not-created
Apr 7, 2026
Merged

3 scout targets should be updated each time it is saved if not created#5
jchate6 merged 6 commits intodevfrom
3-scout-targets-should-be-updated-each-time-it-is-saved-if-not-created

Conversation

@jchate6
Copy link
Copy Markdown
Contributor

@jchate6 jchate6 commented Apr 3, 2026

This automatically updates both the orbital elements and the scout data each time a target is created via the SCOUT dataservice

@jchate6 jchate6 requested review from phycodurus and talister April 3, 2026 00:12
@jchate6 jchate6 linked an issue Apr 3, 2026 that may be closed by this pull request
@jchate6 jchate6 moved this to Needs Review in TOM Toolkit Apr 3, 2026
Copy link
Copy Markdown
Collaborator

@talister talister left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me and I've tested it out with the runserver and pre-inserted early versions of a specific (real) Scout object, which then updated properly to the real values. (There's not much genuine updating of Scout right now because Full Moon)
One note before the "V1.0" of tom_jpl: I should be put into the pyproject.toml under the authors and replacing Rachel in the maintainers

…-be-updated-each-time-it-is-saved-if-not-created
@coveralls
Copy link
Copy Markdown

Coverage Report for PR #5

Coverage decreased (-0.04%) to 85.423%

Diff Coverage: 20 of 21 changed lines covered (95.24%)

Uncovered Changes

File Covered Changed %
tom_jpl/jpl.py 15 16 93.75%

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Lost Coverage
tom_jpl/models.py 1 96.3%

Coverage Status
Change from base Build 24040610709: -0.04%
Covered Lines: 545
Relevant Lines: 638

💛 - Coveralls

@jchate6 jchate6 merged commit 84bf882 into dev Apr 7, 2026
20 checks passed
@jchate6 jchate6 deleted the 3-scout-targets-should-be-updated-each-time-it-is-saved-if-not-created branch April 7, 2026 16:38
@github-project-automation github-project-automation bot moved this from Needs Review to Merged (to dev) in TOM Toolkit Apr 7, 2026
@jchate6 jchate6 moved this from Merged (to dev) to Released in TOM Toolkit Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Released

Development

Successfully merging this pull request may close these issues.

Scout Targets should be updated each time it is saved if not created

4 participants