Skip to content

(hopefully, maybe, probably not) Final tweaks and cleanup#48

Merged
Eoghan-Perkins merged 6 commits intomainfrom
final-tweaks-and-cleanup
Mar 29, 2026
Merged

(hopefully, maybe, probably not) Final tweaks and cleanup#48
Eoghan-Perkins merged 6 commits intomainfrom
final-tweaks-and-cleanup

Conversation

@Eoghan-Perkins
Copy link
Copy Markdown
Collaborator

@Eoghan-Perkins Eoghan-Perkins commented Mar 28, 2026

In addition to some code cleanup and comments, the following last-minute features were implemented:

  • auto-deletion of job output/assets after 24 hours
  • .env file that controls some things (go look at it)
  • Timebox validation checks for parameters that exceed 3 hours in length as discussed with Natalie
  • made the switch to cloud-optimized-geotiff output being sent to the frontend
  • fixed linter, linter still ignores NFGDA algorithm code
  • added end-to-end integration test that runs on pull requests

@Eoghan-Perkins
Copy link
Copy Markdown
Collaborator Author

@robertmcabee Made a very slight change to the API routes in app.py in /backend: "/APIs/" -> "/apis/" and I think I edited the frontend where necessary to accommodate, would appreciate your eyes on that though

@Eoghan-Perkins Eoghan-Perkins merged commit 855e3cf into main Mar 29, 2026
2 checks passed
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