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

Don't install unused pandas and numpy #3034

Merged
merged 2 commits into from
Jan 21, 2021
Merged

Conversation

Twixes
Copy link
Collaborator

@Twixes Twixes commented Jan 21, 2021

Changes

With #2997 out of the way we can speed up builds by not including these two.

@timgl timgl temporarily deployed to posthog-no-numpy-pandas-3yzc3a January 21, 2021 12:07 Inactive
vine==1.3.0 # via amqp, celery
whitenoise==5.0.1 # via -r requirements.in
zipp==3.1.0 # via importlib-metadata
aioch==0.0.2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting that the format changed. Makes it a bit harder to read

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, yeah, dunno why!

@macobo
Copy link
Contributor

macobo commented Jan 21, 2021

Oh did not notice some tests were red. Carry on then :)

@Twixes Twixes temporarily deployed to posthog-no-numpy-pandas-3yzc3a January 21, 2021 12:26 Inactive
@Twixes Twixes merged commit 481e9dd into master Jan 21, 2021
@Twixes Twixes deleted the no-numpy-pandas-in-requirements branch January 21, 2021 12:47
EDsCODE added a commit that referenced this pull request Jan 21, 2021
* master:
  Add handing off event ingestion to plugin server (#2898)
  Plugin unique url with different private tokens & improve privacy (#3033)
  Delete omni_person.proto (#3014)
  Don't install unused pandas and numpy (#3034)
  2248 remove pandas numpy (#2997)
  Exclude health from ip block (#3027)
  Allow filtering by unseen recordings (#3000)
  Don't include materialized columns in kafka table (#3026)
  Standardize created at and by table columns (#2961)
  Add user param for clickhouse if provided (#3025)
  Setup logging to system.text_log in dev environment (#3024)
  feat: set event root logger severity using the DJANGO_LOG_LEVEL env variable (#3016)
  feat: allow setting Sentry environment by using the SENTRY_ENVIRONMENT env variable (#3015)
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