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

fix 2023 data, update models #136

Merged
merged 1 commit into from
Mar 10, 2023
Merged

fix 2023 data, update models #136

merged 1 commit into from
Mar 10, 2023

Conversation

nkgilley
Copy link
Contributor

This fixes the data issue brought up by @jfacowns in #135.

@kyleskom kyleskom merged commit f82849f into kyleskom:master Mar 10, 2023
@kyleskom
Copy link
Owner

This looks to be breaking the flask app

@kyleskom
Copy link
Owner

The command line as well.

@kyleskom
Copy link
Owner

This might be because of the requirements update. What version of python was the built for?

@kyleskom
Copy link
Owner

Im on 3.9 and getting ERROR: Could not find a version that satisfies the requirement tensorflow==2.11.0 (from versions: none) ERROR: No matching distribution found for tensorflow==2.11.0

@kyleskom
Copy link
Owner

When trying to run command line TypeError: weight_decay is not a valid argument, kwargs should be empty for optimizer_experimental.Optimizer.

@nkgilley
Copy link
Contributor Author

Hey, so on my mac I don't install tensorflow==2.11.0, but rather tensorflow-macos==2.11.0. I'm not sure what the best way to handle this is. I have manually edited the requirements.txt in my local environment. I think once xgboost is updated the errors will go away.

@kyleskom
Copy link
Owner

Oh ya oops. I forgot the m1 Macs have a special tensorflow

@kyleskom
Copy link
Owner

Hmm that didn't seem to fix it.

@kyleskom
Copy link
Owner

A fresh install of tensorflow fixed this. Sorry for confusion.

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.

None yet

2 participants