Skip to content

25 new sensors - #26

Open
montse-jara wants to merge 64 commits into
devfrom
25_new_sensors
Open

25 new sensors#26
montse-jara wants to merge 64 commits into
devfrom
25_new_sensors

Conversation

@montse-jara

Copy link
Copy Markdown
Collaborator

This version adds a config file that enables the post camera and post observation scripts to work with a temperature sensor.

@kgarwoodsdzwa kgarwoodsdzwa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can you also make an example csv file with the subject nane/lat/long? like inside the config folder, create like example_subject_post.csv with like a random gps lat long for a random made up 'demo' subject, so that people can see the exact format that file needs to be to work to use post_camera. and if they run it with that example csv, it should post like a dummy subject to earthranger.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

create a folder in /sageranger/sageranger to store these. so /sageranger/sageranger/observation_payloads/camera.json or something like that, may need to update the pyproject or something somewhere to make sure it finds the jsons in that subfolder

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Config Folder now has json files and the example csv

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

same thing as the camera json, create a folder in /sageranger/sageranger to store these. so /sageranger/sageranger/observation_payloads/temp.json or something like that

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

move /config outside the sageranger folder. so high level should be sageranger/config and sageranger/sageranger would be on the same hierarchy.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The config folder has been moved and the toml updated.

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.

2 participants