Navid comments

Results 10 comments of


                                            Navid

Give an argparse for the model selection as well.

Instead of automatically loading the model hard-coded we can work on making it an argparse. Here's the total error: ``` Traceback (most recent call last): File "predict_batch.py", line 184, in...

Shift from Python 2.7 to Python 3

@duolinwang yeah. I'm thinking of doing it myself. there are some library differences.

billiard.exceptions.WorkerLostError: Worker exited prematurely: signal 6

The solution I found was to handle that exception internally and go on with your life. Like try and except and ignore that exception

Dockerized version

I approved this but then, please address this comment: https://github.com/virattt/ai-hedge-fund/pull/152#issuecomment-2708798096

Disable the unnecessary print() statements

Sure, I can do this. I've already locally done this. How can I make a PR?

environment.py lazy_import() Seems to be Unused

There must be a reason why this was there.

environment.py lazy_import() Seems to be Unused

Seems like it is needed as you can see here in this commit message: ``` removed stupid warning by only importing the necessary ocean env ``` https://github.com/PufferAI/PufferLib/commit/4bcb316d838f1d378503a16cada82268631ee4f2 Is there any...

Client.submit *args is typehinted wrongly

I can take this task up. I use `dask` on a daily basis so I'd like to contribute to the codebase.

Add Semaphore, Condition, Event

Can i work on the `Condition` part, it seems like `Event` has been implemented.

Add `Condition` class

> At a quick glance this seems fine. The CI is hanging on these new tests though so you'll need to take a look at that. > > Once you...