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

version bump to 0.9 #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shark8me
Copy link

@shark8me shark8me commented Aug 8, 2017

bump version to latest version (0.9) of deeplearning4j

@hansott
Copy link

hansott commented Aug 23, 2017

Hi! Thanks for your pull request! The main developer of this project is currently on vacation. He'll review this when he's back. 🤞

@tolomaus
Copy link

Hi Kiran, unfortunately we have no time to further support this project. Would you be interested to take over the repo? Otherwise I suggest you create a fork.

@hswick
Copy link

hswick commented Aug 30, 2017

Hi all,

I've recently started a new dl4j wrapper for clojure https://github.com/hswick/jutsu.ai, it is still a work in progress and I'm currently planning a slight rewrite but would love all the help I can get.

Currently, 99% of the way finished wrapping nd4j https://github.com/hswick/jutsu.matrix

@shark8me
Copy link
Author

@tolomaus , thanks for letting me know. I don't have the resources to man an independent fork currently though.
@hswick interesting! Could you talk about the motivation for rewriting a new wrapper? is the current implementation insufficient in any manner?

@hswick
Copy link

hswick commented Sep 14, 2017

@shark8me my motivation was that this project didn't look like it was being maintained so rather than learning how this library worked. I felt it would be easier for me to build from scratch. Mainly, I wanted more creative control. Also, my lib(s) are focusing on being more modular, as I have a separate library for nd4j and the word2vec stuff as opposed to packaging it all together in one large code base. If you want to talk more, feel free to DM me on Slack or Discord. Same username.

@joachimdb
Copy link
Collaborator

joachimdb commented Sep 14, 2017

Hi Guys,

I started this project but it is indeed no longer maintained.

It looks like quite some progress was achieved in this fork though, https://github.com/yetanalytics/dl4clj/tree/18ab472fbf36a6d9e93ae37de375b16712af7f7b
Feel free to have a look.

Of course also good luck to your project @hswick!

@FeLungs
Copy link
Contributor

FeLungs commented Sep 14, 2017

@joachimdb I have been working on an update to your original work for a while now but it's still a work in progress. I hope to have a version ready for a PR soon, just finalizing some things here and there. Was staying under the radar until that version was ready but feel to take a look and tell me what you think!

@hswick
Copy link

hswick commented Sep 15, 2017

@FeLungs you have definitely put in a lot of work into this. I will be attending the Conj so hopefully we can chat about this in person.
Will probably steal how you are dealing with dataset iterators because I find them to almost be more complex than the neural nets.
I like that your library is very explicit. However, at a very surface level glance, I can't help but feel it doesn't decrease the complexity of using dl4j very much. Which I feel is most of the benefit of a clojure wrapper. Tough problem, something I am struggling with myself. All about balancing simple vs easy. Looking forward to seeing your progress.

@FeLungs
Copy link
Contributor

FeLungs commented Sep 15, 2017

@hswick wonderful I will see you there, I will be presenting on this work on the 14th (may have the date wrong). I'm still working on my slides and talking points so additional feedback is highly appreciated. dl4j is a very complex library handling a very complex problem and I don't think you can simply that, only make it easier to work with using clojure constructs.

@joachimdb
Copy link
Collaborator

@FeLungs looks quite impressive what you did so far! Unfortunately, I currently can't spare the time, but I hope to find some time in the future to actually play around a bit and maybe start contributing again.

In the mean time: would you like me to figure out how we can regain admin rights to the master repo (I'm not currently working at Engagor anymore) or maybe move things to a new repo?

@FeLungs
Copy link
Contributor

FeLungs commented Sep 15, 2017

@joachimdb Thank you, and I understand. I think moving things to a new repo will be easier and I can set that up on my end.

@oemebamo
Copy link
Contributor

@joachimdb Can easily transfer ownership of this repository to another organisation or user. Just let me know. 👋

@FeLungs
Copy link
Contributor

FeLungs commented Sep 15, 2017

@joachimdb @oemebamo can you transfer ownership of this repo to the yetanalytics organisation https://github.com/yetanalytics thank you!

@joachimdb
Copy link
Collaborator

great, thanks @oemebamo !

@joachimdb
Copy link
Collaborator

and good luck to you of course @FeLungs :)

@FeLungs
Copy link
Contributor

FeLungs commented Sep 15, 2017

Thank you and likewise @joachimdb

@oemebamo
Copy link
Contributor

@FeLungs I can't transfer ownership rights, because: "yetanalytics/dl4clj already exists and You don’t have the permission to create repositories on yetanalytics"
See https://help.github.com/articles/about-repository-transfers/ for details.

@FeLungs
Copy link
Contributor

FeLungs commented Sep 18, 2017

@oemebamo the name is changed on my end. Let me know if you run into any more issues

@oemebamo
Copy link
Contributor

oemebamo commented Sep 18, 2017

@FeLungs Now get this error: "yetanalytics already has a repository in the engagor/dl4clj network and You don’t have the permission to create repositories on yetanalytics"
Maybe because you have a fork of the repo on that organization? The support article mentions that can't be the case either ...

@FeLungs
Copy link
Contributor

FeLungs commented Sep 18, 2017

@oemebamo I was hoping the name change would be enough but you're correct. Let me remove the fork from my organization

@FeLungs
Copy link
Contributor

FeLungs commented Sep 18, 2017

@oemebamo I have deleted the fork from my organization

@oemebamo
Copy link
Contributor

@FeLungs Almost there, the error I get is now "You don’t have the permission to create repositories on yetanalytics" :).

@FeLungs
Copy link
Contributor

FeLungs commented Sep 18, 2017

@oemebamo I need to wait for my boss to get out of a meeting, I do not have the permissions to grant permissions on our organization.

@FeLungs
Copy link
Contributor

FeLungs commented Sep 18, 2017

@oemebamo you should have gotten an invite to a temp group under the yetanalytics org. Let me know if there is anything else I need to do on my end

@oemebamo
Copy link
Contributor

@FeLungs The transfer initiated successfully. "Can take a few minutes". Good luck with the project!

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

Successfully merging this pull request may close these issues.

7 participants