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

Depend on anvil-parser2 from Pypi instead of Github #18

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

0xTiger
Copy link

@0xTiger 0xTiger commented Aug 3, 2024

Thanks for making this @ryan-mooore! I really like the visual style of the mapping! :)

This PR updates the dependencies to use anvil-parser2 from pypi so you no longer have to depend on 0xTiger/anvil-parser from github (anvil-parser2 is exactly the same package I've just chosen to publish it on pypi and name it)

This PR also replaces np.mat with np.asmatrix since with newer versions of numpy I get the error:

generate: (INFO) Reading data from /mc-world/...
Traceback (most recent call last):
  File "/topomc/src/convert_world.py", line 139, in <module>
    "dem": create_mat(np.uint8) if args.compress_height_limit else create_mat(np.int16),
  File "/topomc/src/convert_world.py", line 133, in <lambda>
    create_mat = lambda dtype : np.mat(np.zeros((
  File "/topomc/.venv/lib/python3.9/site-packages/numpy/__init__.py", line 397, in __getattr__
    raise AttributeError(
AttributeError: `np.mat` was removed in the NumPy 2.0 release. Use `np.asmatrix` instead.

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.

1 participant