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

Updating deprecated alias 'np.bool8' -> 'np.bool_' #3258

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sebastianbrzustowicz
Copy link

@sebastianbrzustowicz sebastianbrzustowicz commented Feb 23, 2024

Description

Part of code is outdated. I updated one small issue with old alias in numpy. Basically I changed "np.bool8" -> "np.bool_" in 3 places, because otherwise there is a warning when using gym repository. Warning information:

C:\Users\Administrator\AppData\Local\Programs\Python\Python311\Lib\site-packages\gym\utils\passive_env_checker.py:233: DeprecationWarning: np.bool8 is a deprecated alias for np.bool_. (Deprecated NumPy 1.24)

Screenshots

My code:

Zrzut ekranu 2024-02-23 050824

Warning:

Zrzut ekranu 2024-02-23 050704

After changes

I took the following steps to check the results of the change:

  1. Uninstalled gym
    pip uninstall gym

  2. Installed gym from my forked repo with new branch
    pip install git+https://github.com/sebastianbrzustowicz/[email protected]_

There is no longer any redundant warning.

Hope I helped a little bit.

@pseudo-rnd-thoughts
Copy link
Contributor

Hey, we just launched gymnasium, a fork of Gym by the maintainers where all maintenance and improvements will happen moving forward. Therefore, we are not updating or maintaining this repo anymore

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.

2 participants