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

Update protocol mds for field deployment #159

Merged
merged 4 commits into from
Jan 9, 2025

Conversation

ItsUnagii
Copy link
Contributor

@aditya-uw @leewujung revised the protocol mds for deployment - let me know if I should put in anything else. I'm going to go work on detection pipeline usage instructions now

@aditya-uw
Copy link
Contributor

Just moving over a comment from our discussion in #163 about updating the outdated README.md. You can address this after we get all caught up with deployment PRs.

@leewujung
Copy link
Contributor

@ItsUnagii @aditya-uw: I couldn't tell what the latest status of this is based on the above comment related to #163. Are everything updated? If not, @ItsUnagii do you think you could finish it sometime tomorrow? Thanks!

@leewujung
Copy link
Contributor

Also, if I understand this correctly, merging these changes will address #159. Is this correct?

@ItsUnagii
Copy link
Contributor Author

@leewujung in terms of the md files, everything in them is up to date, other than the fact that it refers to ubna_2024.csv. I'm still working on uploading deployment data because of the updates to the field data tests (also we should make a 2025 csv file soon)

@leewujung
Copy link
Contributor

Ok sounds good! I will merge this now. When you are ready to update the 2025 csv file, you could also make it such that in the doc it just refers to field_records/ubna_{YYYY}.csv so you don't have change it.

@leewujung leewujung merged commit 8bd516b into uw-echospace:main Jan 9, 2025
1 check 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.

3 participants