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

Bug located in plcint, pldisp, plnsol, plesol, plorb and plvect #2193

Closed
4 tasks done
clatapie opened this issue Jul 19, 2023 · 5 comments
Closed
4 tasks done

Bug located in plcint, pldisp, plnsol, plesol, plorb and plvect #2193

clatapie opened this issue Jul 19, 2023 · 5 comments

Comments

@clatapie
Copy link
Contributor

🤓 Before submitting the issue

🔍 Description of the bug

The previous functions need to be defined in another way as nothing is displayed for now.

🕵️ Steps To Reproduce

Steps:

  • First I did...
  • ...

And I ran the following code:

from ansys.mapdl.core import launch_mapdl
mapdl = launch_mapdl()

# Insert your code here

💻 Which Operating System are you using?

None

🐍 Which Python version are you using?

None

📝 PyMAPDL Report

Show the Report!

# PASTE HERE THE OUTPUT OF `python -c "from ansys.mapdl import core as pymapdl; print(pymapdl.Report())"` here

📝 Installed packages

Show the installed packages!

# PASTE HERE THE OUTPUT OF `python -m pip freeze` here

📝 Logger output file

Show the logger output file.

# PASTE HERE THE CONTENT OF THE LOGGER OUTPUT FILE.

@mikerife
Copy link

Hi @clatapie I'd not consider this a bug as those plot commands need to be preceeded by a "/show" command to redirect the plots from the terminal (which does not exist) to a file type. Mike

@clatapie
Copy link
Contributor Author

Hi @mikerife
Thank you for your prompt feedback! I didn't about that but it is exactly what I was looking for.
I will use it.

@germa89
Copy link
Collaborator

germa89 commented Jul 24, 2023

Hi @clatapie I'd not consider this a bug as those plot commands need to be preceeded by a "/show" command to redirect the plots from the terminal (which does not exist) to a file type. Mike

I wonder if we should made PyMAPDL automatically issue /SHOW. We could have something like a default image type:

mapdl.default_image_file = "PNG"
mapdl._use_vtk=False # we should also do a property `mapdl.use_vtk`

mapdl.pldisp()

what do you think @mikerife , will it be (sort of) consistent (convenient?) with MAPDL users?

@mikerife
Copy link

@germa89 @clatapie I think this might be a hard programming task as MAPDL does not seem to have a way to get the status of the display (neither *get nor /pstatus). So I guess you will need to track the status of any prior /show somehow....??? I do think defaulting to png would be convinient.

@germa89
Copy link
Collaborator

germa89 commented Jul 24, 2023

@germa89 @clatapie I think this might be a hard programming task as MAPDL does not seem to have a way to get the status of the display (neither *get nor /pstatus). So I guess you will need to track the status of any prior /show somehow....??? I do think defaulting to png would be convinient.

We could track it ... as everything pass by mapdl.run. I will draft a PR soon.

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

No branches or pull requests

3 participants