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

Updates url for basicModel_neutral_lbs_10_207_0_v1.0.0.pkl #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Updates url for basicModel_neutral_lbs_10_207_0_v1.0.0.pkl #46

wants to merge 1 commit into from

Conversation

drewtu2
Copy link

@drewtu2 drewtu2 commented Dec 20, 2024

  • The model was removed from the repository which causes this to break. Technically, the maintainers want people to download it through their site, so the proper fix is to remove this logic all together, however, in the short term, this fixes this 404 error that will be thrown when attempting to run.

Addresses
#33
#38

@drewtu2
Updates url for basicModel_neutral_lbs_10_207_0_v1.0.0.pkl
38b6f70 
- The model was removed from the repository which causes this to break. Technically, the
maintainers want people to download it through their site, so the proper fix is to remove
this logic all together, however, in the short term, this fixes this 404 error that will
be thrown when attempting to run.
@drewtu2 drewtu2 mentioned this pull request Dec 20, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@drewtu2