-
Notifications
You must be signed in to change notification settings - Fork 34
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
Feature Request: Add support for pytorch #5
Comments
We will do the following steps for version 1.2.0:
Added to the first post. |
@fsschneider this list is correct at the time of merging my pull request. All ticks are correct. |
Hey, is this still not resolved? and open for contributions? |
Hey, We are definitely open for contributions if you want to help us solve this issue. |
Hi! I'm looking forward to using this suite in PyTorch, and I'm happy to help a bit with the porting. What's holding back the Tolstoi RNN task? I see there is both net_char_rnn and also a Tolstoi dataset, but no corresponding task. Is there any particular challenge I am missing? What is your procedure to test for equivalence between the TensorFlow and PyTorch implementations? |
Hi @tvogels, great to here it! There is actually a PR that introduces the Tolstoi RNN task to PyTorch: #40. I am currently working on a refactoring of DeepOBS for a new release. This will then include the open PRs, and a full PyTorch implementation. If you want to use DeepOBS for PyTorch, I would advise using the pre-release (or directly installing from the |
Any updates? Will we have pytorch support anytime soon or has this become stale? |
Hi @pratt3000, the DeepOBS package is no longer maintained. I highly recommend using the AlgoPerf benchmark suite instead (I helped develop it). In my opinion, it is better in pretty much every aspect. You can keep using DeepOBS, as is (we are also doing it for first prototypes). If you want to use PyTorch, I advise using the pre-release or directly installing it from the |
(This is a test issue. As mentioned in the responses to #3 and #4 , there is a development branch that supports pytorch. find it here: https://github.com/abahde/DeepOBS)
Expected behavior
I would appreciate if DeepOBS had built-in support for my pytorch optimizers.
This is relevant because a lot of optimizer research happens on pytorch.
Proposed approach:
Maybe @abahde could send a pull request when he's finished implementing it. Then @fsschneider can accept the pull request, handle the merging and we have success!
To Do
We will do the following steps for version 1.2.0:
The text was updated successfully, but these errors were encountered: