pyenergyplus / zeppy

To run eppy (or anything else) on multiple nodes in parallel and collect the results.
Mozilla Public License 2.0
3 stars 1 forks source link

Implement using tcp-ip #5

Closed santoshphilip closed 1 year ago

santoshphilip commented 4 years ago

Now we have an ipc version. do it in tcp=ip

as a next step bridge then network gap

orcunkoraliseri commented 1 year ago

Hello,

I am using zeppy with Windows 10. I have encountered zmq.error.ZMQError: Protocol not supported problem.

I found a link which talk about ipc protocol on Windows 10. https://github.com/jina-ai/clip-as-service/issues/39

How can i solve this problem?

Thank you for your great effort with eppy.

santoshphilip commented 1 year ago

@orcunkoraliseri , I am travelling now with bad connectivity. Back early next week. I'll take a closer look once I am back.

I think this is the issue. zeppy is using ipc protocol. This works only on unix zeppy needs to be upgraded to use tcp-ip protocol that works on windows

My recollection is that it is an easy upgrade (maybe) I am glad you are trying to use it. Gives me motivation to upgrade it :-)

orcunkoraliseri commented 1 year ago

@santoshphilip, I'm glad the problem was easy. Unfortunately, I'm a Windows user, but I think I should also start working with Linux now.

I am using Eppy and its side libraries. You are doing a great job.

Thank you. I'll be waiting for the upgrade.

orcunkoraliseri commented 1 year ago

Any progress on the update? If there's anything I can help with, I'd love to hear it.

santoshphilip commented 1 year ago

I am back from my travels. Time to start looking at it. Thanks for the reminder.

I will look at it with an eye to having you help. It often takes longer to collaborate, but the results are much better.

Give me a few days to get back into the code again.

santoshphilip commented 1 year ago

I am running into a bunch of issues running E+ (Did not have them before) This is going to take linger than I anticipated.

orcunkoraliseri commented 1 year ago

I can understand that it may take longer than expected. I think that happens most of the time. I can wait until you complete it. Thank you so much.

santoshphilip commented 1 year ago

You are welcome to keep reminding me :-)

santoshphilip commented 1 year ago

fixed issue #5

:Problem: need a TCP version of ppipes.py :Solution: ptcp is the TCP version of ppipes

santoshphilip commented 1 year ago

@orcunkoraliseri , the coding is done. I need to update the user documentation so that you get a sense of how to try it out

santoshphilip commented 1 year ago

@orcunkoraliseri I made a new release which works with TCP and has updated user documentation to match. Can you test it and confirm that things work.

I am not sure you are seeing these messages once the issue was closed. I am reopening the issue

orcunkoraliseri commented 1 year ago

Thank you again for your effort @santoshphilip. Completing the code made me very happy. You are giving great support to my doctoral study.

What you write comes to me by mail. I didn't see it, so I couldn't answer. You don't need to open the issue again.

Due to the project, I am working on, I have to travel to different cities. I'm away from my computer right now. Can you give me a few days for testing? I'll try it as soon as I get to my computer. I can't wait to try it.

santoshphilip commented 1 year ago

not a problem. Get to it whenever you can.

I am curious about how zeppy helps in your doctoral work. I might have to stop considering zeppy as experimental and treat it as production code (means more robust testing and clearer docstrings and user documentation)

orcunkoraliseri commented 1 year ago

The problem has solved. It works on windows 10 pc. Thank you for your effort @santoshphilip

santoshphilip commented 1 year ago

I was thinking about this. Zeppy is overkill for most projects. Usually runIDFs() from eppy is good enough. How many runs simulations would you be doing ?

As the numbers get larger, zeppy starts to become useful. Of course, not all of this has been coded, but I would find it interesting to so.

Again, if runIDFs() is good enough, you should use that.

santoshphilip commented 1 year ago

edited previous post for better formatting

orcunkoraliseri commented 1 year ago

I need to write a little bit longer to accurately describe what I'm trying to do. Technical details on zeppy are discussed here. So can I take your e-mail and write to you directly in order not to go into unnecessary details under this issue of zeppy? If you want, I would like to consult zeppy and eppy about technical issues in the process. Maybe I can contribute also to zeppy process by giving you an idea. I think the potential of zeppy is huge.

santoshphilip commented 1 year ago

@orcunkoraliseri I have sent an email to your dot edu address. Hit reply to confirm that you got it