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
airbnb fixes and improvements #2600
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont think this what Bressler meant when he created the issue.
@noamzbr if it is, lets discuss.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Nadav-Barak actually airbnb is used in the latter way in the mon quickstarts - first train data and preds are loaded, then production data and pred. It makes sense to use it like that for the quickstarts, though I agree it's not the format used in other datasets. Agree with @JKL98ISR that download times are critical here and so we want to avoid redundant downloads.
@JKL98ISR - did you run this in the monitoring onboarding flow? How did it look like?
looks good @noamzbr. tested all possibilities |
No description provided.