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

Cannot terminate the process #27

Closed
B3J4y opened this issue Feb 2, 2024 · 10 comments
Closed

Cannot terminate the process #27

B3J4y opened this issue Feb 2, 2024 · 10 comments
Assignees
Labels
bug Something isn't working joss review related issues related to the review process for publication

Comments

@B3J4y
Copy link
Collaborator

B3J4y commented Feb 2, 2024

After running the toy example, the process did not finish, and I was unable to terminate it properly.

@Flaminietta
Copy link
Collaborator

Flaminietta commented Mar 15, 2024

Dear @B3J4y, in the main branch, latest commit e071d84, the toy example runs and finishes as expected for me.

Screenshot 2024-03-15 at 14 08 41

If you can try it again and confirm that everything is fine, I can close this issue. Thank you!

@B3J4y
Copy link
Collaborator Author

B3J4y commented Jun 27, 2024

Sorry, the problem is still there:
ProMCMA

I left it open for a while to look if there was a calculation which needs some time, but this process doesn't come back.

I use windows.

@mspada
Copy link
Collaborator

mspada commented Jul 2, 2024

@B3J4y I run the test on windows at it works smooth. Just a question: did the figures open on your browser or not? If not it might be a problem with some close ports that do not allow plotly to open the figures and, therefore, not ending the process. Let us know!

@B3J4y
Copy link
Collaborator Author

B3J4y commented Jul 8, 2024

There are 2 figures. I append the open images:
Figure1
Figure2

@mspada
Copy link
Collaborator

mspada commented Jul 8, 2024

@B3J4y Thank you! Therefore, there might be some issue with the saving of some of the output files. When you run it, is it saving some output files in the output directory or not? If yes, what output files is ProMCDA saving?

@B3J4y
Copy link
Collaborator Author

B3J4y commented Jul 9, 2024

You're welcome. It is tough to fix bugs remotely on another machine without having direct access. Hopefully, we can manage this.

Here is an image of the output folder:

Output_files

The red files are new. (red is the color for untracked files in IntelliJ)

@mspada
Copy link
Collaborator

mspada commented Jul 9, 2024

Perfect, thanks! I know this is quite a tough way to try to fix an issue. In any case, it looks like that the issue is related to the figures, probably in the saving of them in the output folder. I'll have a look at it. I'll keep you posted

@mspada
Copy link
Collaborator

mspada commented Jul 10, 2024

@B3J4y I guess I found the issue. It looks like that it is related to kaleido, which is used to save the static image of the plotly plots. In some cases (only on windows and not for everyone) the new version of kaleido (0.2.1), which is used in ProMCDA, is not correctly working. To solve this the community propose to downgrade kaleido to the version 0.1.0.post1 as discussed in https://community.plotly.com/t/static-image-export-hangs-using-kaleido/61519 and plotly/Kaleido#126
Could you please try to downgrade kaleido to the version 0.1.0.post1 and run the test once again?

@B3J4y
Copy link
Collaborator Author

B3J4y commented Jul 23, 2024

@mspada Thank you for finding the solution. I haven't had the time to downgrade Kaleido, and I feel that whether it works or not, your package will be very weak in terms of reusability if it relies on this mechanism for your Windows users. Could you possibly package it into a Docker container and add a Dockerfile to the repository? This could solve the Windows problems and provide users with a system-independent way to run the software.

@Flaminietta
Copy link
Collaborator

Dear @B3J4y, your request is addressed in this PR (ProMCDA is now also dockerized).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working joss review related issues related to the review process for publication
Projects
None yet
Development

No branches or pull requests

3 participants