More README updates#4
Merged
gigasquid merged 1 commit intogigasquid:improve-clojure-readmefrom Nov 2, 2018
kohr-h:more-clj-readme
Merged
More README updates#4gigasquid merged 1 commit intogigasquid:improve-clojure-readmefrom kohr-h:more-clj-readme
gigasquid merged 1 commit intogigasquid:improve-clojure-readmefrom
kohr-h:more-clj-readme
Conversation
kohr-h
commented
Nov 1, 2018
|
|
||
| ## Getting Involved | ||
|
|
||
| By far the best way to get involved with this project is to install the Clojure MXNet package, run the examples, play around, build new things with it, and get back to the development team with feedback! Your input can not only help to identify current issues, but also guide the future development of the Clojure package by pointing out must-have features that are currently missing, or by identifying usability or performace problems of high impact. |
Author
There was a problem hiding this comment.
I wasn't sure if I should use "we" or "us".
Owner
There was a problem hiding this comment.
I think it sounds fine as is - but us also works
gigasquid
approved these changes
Nov 2, 2018
|
|
||
| ## Getting Involved | ||
|
|
||
| By far the best way to get involved with this project is to install the Clojure MXNet package, run the examples, play around, build new things with it, and get back to the development team with feedback! Your input can not only help to identify current issues, but also guide the future development of the Clojure package by pointing out must-have features that are currently missing, or by identifying usability or performace problems of high impact. |
Owner
There was a problem hiding this comment.
I think it sounds fine as is - but us also works
| There are two main ways of reaching out to other users and the package maintainers: | ||
|
|
||
| - If you have a question or general feedback, or you encountered a problem but are not sure if it's a bug or a misunderstanding, then the *Apache Slack* (channels `#mxnet` and `#mxnet-scala`) is the best place to turn check out. To join, [ask for an invitation](https://mxnet.apache.org/community/contribute.html#slack) at `dev@mxnet.apache.org`. | ||
| - If you found a bug, miss an important feature or want to give feedback directly relevant for development, please head over to the MXNet [GitHub issue page](https://github.com/apache/incubator-mxnet/issues) and create a new issue. If the issue is specific to the Clojure package, consider using a title starting with `[Clojure]` to make it easily discoverable among the many other, mostly generic issues. |
Owner
There was a problem hiding this comment.
Great point about the Clojure title 💯
|
|
||
| ## Getting Involved | ||
|
|
||
| By far the best way to get involved with this project is to install the Clojure MXNet package, run the examples, play around, build new things with it, and get back to the development team with feedback! Your input can not only help to identify current issues, but also guide the future development of the Clojure package by pointing out must-have features that are currently missing, or by identifying usability or performace problems of high impact. |
| Since there is native code involved in the framework, what OS you are running matters. | ||
|
|
||
| The following combinations of operating system and compute device are supported: | ||
| Since the core contains native (compiled) code and is bundled with the language bindings, your hardware and OS matter to the choices to be made during installation. The following combinations of operating system and compute device are supported: |
Owner
|
Looks great. Thanks tons for the improvements |
Author
|
Cool, thanks for the quick reply! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I have made some more edits, in particular the "Getting Involved" section that I've tried to flesh out a bit. Check if you like it -- since you have a better overview of how things work, you may want to change things accordingly.