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

ux.plot face missing #1048

Open
rajeeja opened this issue Oct 29, 2024 · 8 comments
Open

ux.plot face missing #1048

rajeeja opened this issue Oct 29, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@rajeeja
Copy link
Contributor

rajeeja commented Oct 29, 2024

Version

main

How did you install UXarray?

Pip

What happened?

#1047

See the notebook, the points and connectivity look fine. ux.plot shows 4 faces instead of 5.

What did you expect to happen?

5 faces

Can you provide a MCVE to repoduce the bug?

No response

@rajeeja rajeeja added the bug Something isn't working label Oct 29, 2024
@github-project-automation github-project-automation bot moved this to 📚 Backlog in UXarray Development Oct 29, 2024
@rajeeja rajeeja changed the title plot face missing ux.plot face missing Oct 29, 2024
@aaronzedwick
Copy link
Member

@philipc2 I looked into this for a bit, I am not sure what is happening. The nodes plot fine with uxds.plot.edges(backend="bokeh", engine="spatialpandas", color="red") * uxds.plot.nodes(backend="bokeh", s=3), but the edges just don't show up. The face_edge_connectivity looks fine to me as well.

@rajeeja
Copy link
Contributor Author

rajeeja commented Oct 29, 2024

Is it something to do with:

  1. Range of values of lat/lon for that face
  2. Missing a value in connectivity

It isn't very obvious.

@philipc2
Copy link
Member

@philipc2 I looked into this for a bit, I am not sure what is happening. The nodes plot fine with uxds.plot.edges(backend="bokeh", engine="spatialpandas", color="red") * uxds.plot.nodes(backend="bokeh", s=3), but the edges just don't show up. The face_edge_connectivity looks fine to me as well.

Can you share a minimal example with the plots you encountered?

@aaronzedwick
Copy link
Member

@philipc2 I looked into this for a bit, I am not sure what is happening. The nodes plot fine with uxds.plot.edges(backend="bokeh", engine="spatialpandas", color="red") * uxds.plot.nodes(backend="bokeh", s=3), but the edges just don't show up. The face_edge_connectivity looks fine to me as well.

Can you share a minimal example with the plots you encountered?

I was messing around with the example notebook in pr #1047 it has the code and displays the error there.

@philipc2
Copy link
Member

@philipc2 I looked into this for a bit, I am not sure what is happening. The nodes plot fine with uxds.plot.edges(backend="bokeh", engine="spatialpandas", color="red") * uxds.plot.nodes(backend="bokeh", s=3), but the edges just don't show up. The face_edge_connectivity looks fine to me as well.

Can you share a minimal example with the plots you encountered?

I was messing around with the example notebook in pr #1047 it has the code and displays the error there.

The plot doesn't show up in the notebook. Can you share a screenshot of it?

@aaronzedwick
Copy link
Member

Image

@philipc2
Copy link
Member

Have you run uxgrid.validate()?

@rajeeja
Copy link
Contributor Author

rajeeja commented Oct 29, 2024

Have you run uxgrid.validate()?
yes, validation is successful

rajeeja added a commit that referenced this issue Oct 31, 2024
o Remove the multipoly example, the old bug still exists, but that can be handled in issue #1048
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 📚 Backlog
Development

No branches or pull requests

3 participants