-
Notifications
You must be signed in to change notification settings - Fork 0
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
Nitrogen #92
base: develop
Are you sure you want to change the base?
Nitrogen #92
Conversation
miladagreen
commented
Feb 27, 2024
•
edited
Loading
edited
- ✅ added nitrogen map
- ✅ upgraded bit map
- ✅ added required note on necessary inputs
- ✅ growth stage added
- ✅ *** NEEDS UPDATED BUILD WORKFLOW ***
build is failing due to dockerfile not being updated |
Your review app has been deployed and is available at 'http://172.214.61.236:80'. |
changing the cover crop species doesn't update it in the summary widget |
changing termination date doesn't update the summary widget |
changing species or growth stage doesn't update carbohydrates, holo-cellulose, or lignin in the summary widget |
also, after thinking about it more loading in an example definitely shouldn't auto fill the biomass and nitrogen values. as the api is updated, it may change the output. we only want the example to load in the geometry but always use the api to get the biomass and nitrogen |
loading in an example after already defining a geometry should clear the old geometry |
The User Sampled branch needs a cash crop planting date (found on the Cash Crop tab), and a cover crop termination date (found on the Cover Crop tab). Note that these dates should not have default values. The program has unspecified planting and termination dates on the Location tab. I think these can go away. The planting date on the Location tab is actually the cover crop planting date, which isn't needed for the User Sampled branch. (I'm also unsure why it's needed in the Satellite branch, but I need to study that some more.) |
The User Sampled branch creates a biomass map if a polygon is created. I don't think this should happen – unless someone requested that feature(?) |
On the About screen, a space is needed after "visit": |
After choosing User Sampled and Example: Grass, the program crashes if you click Cover Crop. |
For the User Sampled path, Biomass Map and Nitrogen Map should not be on the menu on the Output screen. |
Now that About appears on the Home screen, I don't think it's needed in the drop-down. |
This may be a question for the group, but I think that Feedback should always be visible rather than on the drop-down. (This is the case for the original program, the Selector, Vegspec, and CC-Econ). Also, Feedback should be sent to the Feedback API rather than to dst-ncalc/discussions. |
… holo-cellulose, or lignin in the summary widget
visithttps://precisionsustainableag.org/.
Your review app has been deployed and is available at 'http://172.214.61.236:80'. |
1 similar comment
Your review app has been deployed and is available at 'http://172.214.61.236:80'. |
…shes if you click Cover Crop.
… clear the old geometry
Your review app has been deployed and is available at 'http://172.214.61.236:80'. |
1 similar comment
Your review app has been deployed and is available at 'http://172.214.61.236:80'. |
Your review app has been deployed and is available at 'http://172.214.61.236:80'. |
1 similar comment
Your review app has been deployed and is available at 'http://172.214.61.236:80'. |
Your review app has been deployed and is available at 'http://51.8.227.85:80'. |
1 similar comment
Your review app has been deployed and is available at 'http://51.8.227.85:80'. |