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.

Sign up for GitHub

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

Jump to bottom

Ensure overhauled package is consistent with old data pulls #49

Open
slucey opened this issue Oct 23, 2023 · 2 comments · Fixed by #51
Open

Ensure overhauled package is consistent with old data pulls #49

slucey opened this issue Oct 23, 2023 · 2 comments · Fixed by #51
Assignees
@slucey
Milestone
Clean up repo

Comments

@slucey
Copy link
Member

slucey commented Oct 23, 2023 •
edited
Loading

Last year the comlandr pull was not the same as the previous year due to the overhauled package. There are two active branches old_way and funWithUnk that are trying to solve this issue. The old_way branch is a band aid for the issue.
@slucey slucey self-assigned this Oct 23, 2023
@slucey slucey added this to the Clean up repo milestone Oct 23, 2023
@slucey slucey mentioned this issue Oct 23, 2023
Open
@slucey
Copy link
Member Author

slucey commented Oct 31, 2023

image

@slucey
Copy link
Member Author

slucey commented Oct 31, 2023

Some "other" area trips are assigned to MAB but otherwise spot on. I'm happy with how the new algorithm is performing (red line)

@slucey slucey linked a pull request Oct 31, 2023 that will close this issue
Fun with unk #51
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slucey slucey

Labels
None yet
Projects
None yet
Milestone
Clean up repo
Development

Successfully merging a pull request may close this issue.

Fun with unk NOAA-EDAB/comlandr
1 participant
@slucey