Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Naif Kernel Source Changes #5394

Closed
acpaquette opened this issue Jan 2, 2024 · 0 comments · Fixed by #5442
Closed

Naif Kernel Source Changes #5394

acpaquette opened this issue Jan 2, 2024 · 0 comments · Fixed by #5442
Assignees
Labels
bug Something isn't working

Comments

@acpaquette
Copy link
Collaborator

ISIS version(s) affected: 8.0.0+

Description
The sources for some kernel groups within the rclone config file included with ISIS 8.0.0 and up have change reflecting how NAIF archives the kernel groups. Bellow are some of the changes I had to make to get the correct kernels from upstream sources:

 [messenger_naifKernels]
 type = combine
-upstreams = kernels=naif:/pub/naif/pds/data/mess-e_v_h-spice-6-v1.0/messsp_1000/data/kernels/
+upstreams = kernels=naif:/pub/naif/pds/data/mess-e_v_h-spice-6-v1.0/messsp_1000/data/

 [mgs_naifKernels]
 type = combine
-upstreams = kernels=naif:/pub/naif/pds/data/mgs-m-spice-6-v1.0/mgsp_1000/data/kernels/
+upstreams = kernels=naif:/pub/naif/pds/data/mgs-m-spice-6-v1.0/mgsp_1000/data/

 [newhorizons_naifKernels]
 type = combine
-upstreams = kernels=naif:/pub/naif/NEWHORIZONS/kernels/
+upstreams = kernels=naif:/pub/naif/pds/data/nh-j_p_ss-spice-6-v1.0/nhsp_1000/data/

How to reproduce
Attempt to download any of the above kernel sets and compare it to the isis_data area under the asc_data S3 bucket.

Possible Solution
There may be a better way to manage the kernel downloads. I think this is why in the past we have acted as an intermediary between naif and the kernels that are specific to ISIS. This ensures that users are not exposed to this particular issue. We already duplicate the kernels under the S3 bucket so I don't know why that can't be the record/source for all ISIS kernels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging a pull request may close this issue.

3 participants