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.

Sign up for GitHub

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

Jump to bottom

Load surfaces #22

Closed
Leguark opened this issue Sep 21, 2020 · 4 comments
Closed

Load surfaces #22

Leguark opened this issue Sep 21, 2020 · 4 comments
Assignees
@MonEstCha

Comments

@Leguark
Copy link
Collaborator

Leguark commented Sep 21, 2020

No description provided.

@MonEstCha
Copy link
Contributor

I thought about writing a generic script for reading in any surface related csv file to a DataFrame and/or xarray, so that the number of columns doesn't matter., i.e. if the csv contains only vertices or vertices with attributes. What do you think?

@Leguark
Copy link
Collaborator Author

Leguark commented Sep 22, 2020

Yes, that would be great. As long as the output is an UnstructuredData object, the more we can do for the user the better :D

@MonEstCha
Copy link
Contributor

Okay, then for examples I create a PointSet class object, if only vertices are listed in the csv, or a TriSurf object, if edges are present, instead of just the DataFrame?

@MonEstCha
Copy link
Contributor

I'm getting warnings with my latest commit, test_surface_reader.test_plot_to_pyvista (line 52) and commented that at the mentioned line as a review

@Leguark Leguark closed this as completed Feb 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MonEstCha MonEstCha

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Leguark @MonEstCha