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

[tune](deps): Bump matplotlib from 3.3.4 to 3.4.2 in /python/requirements #13

Closed

Conversation

dependabot[bot]
Copy link

@dependabot dependabot bot commented on behalf of github May 8, 2021

Bumps matplotlib from 3.3.4 to 3.4.2.

Release notes

Sourced from matplotlib's releases.

REL: v3.4.2

This is the second bugfix release of the 3.4.x series.

This release contains several critical bug-fixes:

  • Generate wheels usable on older PyPy7.3.{0,1}
  • Fix compatibility with Python 3.10
  • Add subplot_mosaic Axes in the order the user gave them to us
  • Correctly handle 'none' facecolors in do_3d_projection
  • Ensure that Matplotlib is importable even if there's no HOME
  • Fix CenteredNorm with halfrange
  • Fix bar_label for bars with NaN values
  • Fix clip paths when zoomed such that they are outside the figure
  • Fix creation of RangeSlider with valinit
  • Fix handling of "d" glyph in backend_ps, fixing EPS output
  • Fix handling of datetime coordinates in pcolormesh with Pandas
  • Fix processing of some errorbar arguments
  • Fix removal of shared polar Axes
  • Fix resetting grid visibility
  • Fix subfigure indexing error and tight bbox
  • Fix textbox cursor color
  • Fix TkAgg event loop error on window close
  • Ignore errors for sip with no setapi (Qt4Agg import errors)

REL: v3.4.1

This is the first bugfix release of the 3.4.x series.

This release contains several critical bug-fixes:

  • fix errorbar when specifying fillstyle
  • fix Inkscape cleanup at exit on Windows for tests
  • fix legends of colour-mapped scatter plots
  • fix positioning of annotation fancy arrows
  • fix size and color rendering for 3D scatter plots
  • fix suptitle manual positioning when using constrained layout
  • respect antialiasing settings in cairo backends as well

REL: v3.4.0

Highlights of this release include:

  • Figure and Axes creation / management
    • New subfigure functionality
    • Single-line string notation for subplot_mosaic
    • Changes to behavior of Axes creation methods (gca, add_axes, add_subplot)
    • add_subplot/add_axes gained an axes_class parameter
    • Subplot and subplot2grid can now work with constrained layout
  • Plotting methods
    • axline supports transform parameter
    • New automatic labeling for bar charts
    • A list of hatches can be specified to bar and barh

... (truncated)

Commits
  • e3fe991 REL: v3.4.2
  • 2af33c7 Move API changes to correct file.
  • de6c788 DOC: Update GitHub stats for 3.4.2.
  • 3969286 Merge branch 'v3.4.1-doc' into v3.4.x
  • 990ef2c Merge pull request #20184 from meeseeksmachine/auto-backport-of-pr-20147-on-v...
  • d90b7f0 Merge pull request #20183 from meeseeksmachine/auto-backport-of-pr-20147-on-v...
  • f5a3128 Backport PR #20147: DOC: add example of labelling axes
  • 0327cfb Backport PR #20147: DOC: add example of labelling axes
  • 601d3f5 Merge pull request #20181 from meeseeksmachine/auto-backport-of-pr-20171-on-v...
  • 46343dc Merge pull request #20180 from meeseeksmachine/auto-backport-of-pr-19876-on-v...
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [matplotlib](https://github.com/matplotlib/matplotlib) from 3.3.4 to 3.4.2.
- [Release notes](https://github.com/matplotlib/matplotlib/releases)
- [Commits](matplotlib/matplotlib@v3.3.4...v3.4.2)

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label May 8, 2021
@dependabot @github
Copy link
Author

dependabot bot commented on behalf of github May 17, 2021

Looks like matplotlib is up-to-date now, so this is no longer needed.

@dependabot dependabot bot closed this May 17, 2021
@dependabot dependabot bot deleted the dependabot/pip/python/requirements/matplotlib-3.4.2 branch May 17, 2021 17:09
sven1977 pushed a commit that referenced this pull request Jul 28, 2022
We encountered SIGSEGV when running Python test `python/ray/tests/test_failure_2.py::test_list_named_actors_timeout`. The stack is:

```
#0  0x00007fffed30f393 in std::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(std::string const&) ()
   from /lib64/libstdc++.so.6
#1  0x00007fffee707649 in ray::RayLog::GetLoggerName() () from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#2  0x00007fffee70aa90 in ray::SpdLogMessage::Flush() () from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#3  0x00007fffee70af28 in ray::RayLog::~RayLog() () from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#4  0x00007fffee2b570d in ray::asio::testing::(anonymous namespace)::DelayManager::Init() [clone .constprop.0] ()
   from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#5  0x00007fffedd0d95a in _GLOBAL__sub_I_asio_chaos.cc () from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#6  0x00007ffff7fe282a in call_init.part () from /lib64/ld-linux-x86-64.so.2
#7  0x00007ffff7fe2931 in _dl_init () from /lib64/ld-linux-x86-64.so.2
#8  0x00007ffff7fe674c in dl_open_worker () from /lib64/ld-linux-x86-64.so.2
#9  0x00007ffff7b82e79 in _dl_catch_exception () from /lib64/libc.so.6
#10 0x00007ffff7fe5ffe in _dl_open () from /lib64/ld-linux-x86-64.so.2
#11 0x00007ffff7d5f39c in dlopen_doit () from /lib64/libdl.so.2
#12 0x00007ffff7b82e79 in _dl_catch_exception () from /lib64/libc.so.6
#13 0x00007ffff7b82f13 in _dl_catch_error () from /lib64/libc.so.6
#14 0x00007ffff7d5fb09 in _dlerror_run () from /lib64/libdl.so.2
#15 0x00007ffff7d5f42a in dlopen@@GLIBC_2.2.5 () from /lib64/libdl.so.2
#16 0x00007fffef04d330 in py_dl_open (self=<optimized out>, args=<optimized out>)
    at /tmp/python-build.20220507135524.257789/Python-3.7.11/Modules/_ctypes/callproc.c:1369
```

The root cause is that when loading `_raylet.so`, `static DelayManager _delay_manager` is initialized and `RAY_LOG(ERROR) << "RAY_testing_asio_delay_us is set to " << delay_env;` is executed. However, the static variables declared in `logging.cc` are not initialized yet (in this case, `std::string RayLog::logger_name_ = "ray_log_sink"`).

It's better not to rely on the initialization order of static variables in different compilation units because it's not guaranteed. I propose to change all `RAY_LOG`s to `std::cerr` in `DelayManager::Init()`.

The crash happens in Ant's internal codebase. Not sure why this test case passes in the community version though.

BTW, I've tried different approaches:

1. Using a static local variable in `get_delay_us` and remove the global variable. This doesn't work because `init()` needs to access the variable as well.
2. Defining the global variable as type `std::unique_ptr<DelayManager>` and initialize it in `get_delay_us`. This works but it requires a lock to be thread-safe.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants