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

Loading .nkg file fails on Windows #399

Open
caiw opened this issue Nov 8, 2024 · 8 comments
Open

Loading .nkg file fails on Windows #399

caiw opened this issue Nov 8, 2024 · 8 comments
Assignees
Labels
🪲 bug Something isn't working 📄 .nkg files

Comments

@caiw
Copy link
Member

caiw commented Nov 8, 2024

Discovered when trying to fix #408

@caiw caiw added 🪲 bug Something isn't working 📈 plotting functionality Any issues related to plotting labels Nov 8, 2024
@caiw
Copy link
Member Author

caiw commented Nov 8, 2024

@young-x-skyee I'm here to help with this where I can!

@young-x-skyee
Copy link
Contributor

Thanks a lot, Cai! I'll be in CBU on Monday and post the details about the bug then.

@young-x-skyee
Copy link
Contributor

young-x-skyee commented Nov 13, 2024

When I'm running the script using the old packages, I got this error
image
I don't get this error on Linux machines opening the same file, and neither does @neukym on his Mac.

We have confirmed that the dimensions of both coords and data are the same on the Mac (therefore no error), but for Windows the second dimension of coords is doubled what it should be.

nkg.py reads in the coords using:

 with zf.open(f"/{block_name}/coo-coords.bytes") as f:
                coords: NDArray = frombuffer(f.read(), dtype=int).reshape((3, -1))

Could this not work on windows somehow?

@caiw
Copy link
Member Author

caiw commented Nov 15, 2024

This is super weird. Good job tracking down the error. Do the unit tests run on Windows?

@caiw
Copy link
Member Author

caiw commented Nov 15, 2024

One more Q: this issue suggests the issue arrises in particular when doing expression plots with minimaps - I think that's something I heard from @neukym. Is this actually a key part of causing the error or is it just a coincidence that that's what you were trying to do when you noticed it? (Sorry if it's my misunderstanding!)

@neukym
Copy link
Member

neukym commented Nov 15, 2024

Correct! This latest issue is not related to the minimaps - we found it while trying to fix that problem. We should split this into a new issue.

@caiw
Copy link
Member Author

caiw commented Nov 15, 2024

Ok, this one has details of the .nkg issue so I'll rename it and make a new one for the minimaps (#408)

@caiw caiw changed the title Expression plot minimaps don't work on Windows Loading .nkg file fails on Windows Nov 15, 2024
@caiw caiw removed the 📈 plotting functionality Any issues related to plotting label Nov 15, 2024
@sharonho1215
Copy link
Collaborator

I have this problem too when trying to open this on a windows machine :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪲 bug Something isn't working 📄 .nkg files
Projects
None yet
Development

No branches or pull requests

4 participants