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

Depedency: matplotlib version bump #1072

Merged
merged 3 commits into from
Jan 8, 2024

Conversation

taylorfturner
Copy link
Contributor

No description provided.

Copy link
Contributor

@rxm7706 rxm7706 left a comment

Choose a reason for hiding this comment

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

matplotlib>=3.3.3,<3.9.0 might be better - to accommodate patches.
3.8.2 was released last friday.

Copy link
Contributor

@rxm7706 rxm7706 left a comment

Choose a reason for hiding this comment

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

@taylorfturner That still wouldn't pick up 3.8.2 -- did you mean to do <=3.8.2 or <3.9.0

@rxm7706
Copy link
Contributor

rxm7706 commented Nov 29, 2023

hi @taylorfturner,
Did you get a chance to consider using <3.9.0, the current pin will not pick up 3.8.2 or any patch releases in the 3.8 series.
Thank You.

@taylorfturner
Copy link
Contributor Author

hi @taylorfturner, Did you get a chance to consider using <3.9.0, the current pin will not pick up 3.8.2 or any patch releases in the 3.8 series. Thank You.

Hey @rxm7706 -- haven't take a look in-depth at this yet. At AWS re:Invent this week as well so the team's response will probably be delayed.

@taylorfturner taylorfturner force-pushed the dep_matplot_version_bump branch from bbe6204 to 7ed1c48 Compare December 12, 2023 19:10
@taylorfturner taylorfturner force-pushed the dep_matplot_version_bump branch from 7ed1c48 to 534410e Compare December 12, 2023 19:11
@taylorfturner
Copy link
Contributor Author

Updated to 3.9.0 @rxm7706

@micdavis micdavis enabled auto-merge (squash) December 12, 2023 19:28
@micdavis micdavis merged commit 93268f8 into capitalone:dev Jan 8, 2024
5 checks passed
@taylorfturner
Copy link
Contributor Author

@rxm7706 merged FYI -- thanks

micdavis pushed a commit that referenced this pull request Jan 11, 2024
* Feature: added parquet sampling (#1070)

* parquet sampling function developed in data_utils.py; Added sample_nrows argument in ParquetData class; Added test_len_sampled_data in test_parquet_data.py

* resolved conflict with dev, added more tests

* fixed sample empty column bug

* fixed comments in data_utils.py, including:
1. added type of return in sample_parquet function;
2. changed variable names in sample_parquet function to more descriptive names (select -> sample_index, out -> sample_df);
3. created convert_unicode_col_to_utf8 function to reduce repeating code in sample_parquet and read_parquet_df functions

* 1. renamed variable names in covert_unicode_col_to_utf8 function (data_utils.py) to be more descriptive (types -> input_column_types, col -> iter_column), other part unchanged

2. test_parquet_data.py, move import statement to the top of file

3. test_parquet_data.py, merged all tests about parquet sample feature to their original tests

* checked the datatype and input file path before and after reload with sampling option enabled

* test

* delete test edit in avro_data.py, updated fastavro version in  requirment.txt

* remove fastavro.reader type

* change fastavro version back to original

* 1. sample_parquet function description
2. test_len_data method keep one sample length test
3. remove sampling test in test_specifying_data_type
4. remove sampling test in test_reload_data

* Depedency: `matplotlib` version bump  (#1072)

* bump tag matplotlib

* bumpt to most recent

* 3.9.0 update

* Bump actions/setup-python from 4 to 5 (#1078)

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Taylor Turner <[email protected]>

* Make _assimilate_histogram not use self (#1071)

Co-authored-by: Taylor Turner <[email protected]>

* version bump

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: WML <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Junho Lee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants