Skip to content

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

Open
0xTiger wants to merge 2 commits into
kiwirm:masterfrom
0xTiger:master
Open

Depend on anvil-parser2 from Pypi instead of Github#18
0xTiger wants to merge 2 commits into
kiwirm:masterfrom
0xTiger:master

Conversation

@0xTiger

@0xTiger 0xTiger commented Aug 3, 2024

Copy link
Copy Markdown

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