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

added missing cameras: ENTRY for Spirit and Opportunity, SUPERCAM_RMI and LCAM for Perseverance #192

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bk7312
Copy link

@bk7312 bk7312 commented Jan 18, 2024

Added missing cameras to README.md:

  • ENTRY (Entry, Descent, and Landing Camera) for Spirit and Opportunity, only available on sol 1
  • SUPERCAM_RMI (SuperCam Remote Micro Imager) for Perseverance, available on sol 46, 50, 51, and so on
  • LCAM (Lander Vision System Camera) for Perseverance, only available on sol 0

Added LCAM to perseverance_scraper_spec.rb

Added missing camera: ENTRY (Entry, Descent, and Landing Camera) for Spirit and Opportunity, only available on sol 1
@bk7312 bk7312 changed the title added ENTRY camera for Spirit and Opportunity added missing cameras to README.md: ENTRY for Spirit and Opportunity, SUPERCAM_RMI for perseverance Jan 22, 2024
@bk7312 bk7312 changed the title added missing cameras to README.md: ENTRY for Spirit and Opportunity, SUPERCAM_RMI for perseverance added missing cameras to README.md: ENTRY for Spirit and Opportunity, SUPERCAM_RMI for Perseverance Jan 22, 2024
Issue:
- API unable to get photos from Perseverance LCAM (Lander Vision System Camera), LCAM only on Sol 0.
- LCAM created in db/seeds.rb (line 26) but missing in spec/models/perseverance_scraper_spec.rb

Fix:
- Added LCAM to perseverance_scraper_spec.rb
- Added LCAM to README.md

Disclaimer:
- I'm not familiar with the codebase/ruby and this fix is based on searching/comparing where the other Perseverance camera names are being called/used.
@bk7312 bk7312 changed the title added missing cameras to README.md: ENTRY for Spirit and Opportunity, SUPERCAM_RMI for Perseverance added missing cameras: ENTRY for Spirit and Opportunity, SUPERCAM_RMI and LCAM for Perseverance Feb 3, 2024
@bk7312 bk7312 mentioned this pull request Feb 4, 2024
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant