-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
install: reorder methods and add badges #7495
Comments
Sounds like it's mostly about Windows @casperdcl. Would you like to open a PR directly for this? I think you worked on the |
I didn't intend it to be only about Win. In my experience there tends to be different ordering & options for Linux/Win/Mac. Could make a PR but probably needs discussion first from users of different OSs how they prefer/recommend installing (was discussing with @shcheklein...) |
something else include "binary" pyinstaller packages - I think we can do something like: Linux: conda Mac: conda Windows: conda @casperdcl I'm not sure about |
It may be worth it to review past issues related to this, as we recently completely rewrote the installation section. e.g. iterative/dvc.org#701/#734, iterative/dvc.org#660/#701/#734, iterative/dvc.org#656/#695, others?
Related: iterative/dvc.org#65 (That's an old one!) |
I think there are 2 major categories to split up: OS and end-user proficiency. Casual users can't be expected to go through the lengthy "separate conda env with dvc script appended to global |
@casperdcl sounds good.
It seems an obvious though, but my recent experience makes me think differently a bit. And probably, the reason for this that being even an advanced CLI user does not make you immediately an expert in different ways to install things or different environments. Probably they do not fit then into your meaning of advanced though, right? :) and should stick with some default choice. |
Indeed. In this context I was using "advanced" to refer to knowledge of their preferred installation mechanism. |
Shall we include
|
for the record this issue is about |
I think you meant this one is about reordering and iterative/dvc.org#918 about a new option @casperdcl. |
Don't forget brew works on linux! |
also potentially badges (vis iterative/dvc.org#918 (comment)). Added to description here... |
Add |Choco| Add |Brew| Fix style for |Conda| Add |PyPI| Add |Packages| Fixes iterative#3233 Related https://github.com/iterative/dvc.org/issues/904
* docs: add and update badges Add |Choco| Add |Brew| Fix style for |Conda| Add |PyPI| Add |Packages| Fixes #3233 Related https://github.com/iterative/dvc.org/issues/904 * docs: add badge logos * docs: minor badge tidy * docs: add choco install, add OS, update order
Why is conda on top? choco is probably the easiest from cmd line for most cases, no? |
@jorgeorpinel it's a good question. I don't know how popular choco is to be honest ... it's as popular as brew for example? |
I think maybe |
My impression is that it's the equivalent to brew, yes. But much fewer people work on command line in Windows just because (developers mainly, I guess, same as in Mac but there's less). Probably more people use Anaconda Prompt because it's such a popular DS Python disto. But then again Git Bash is probably also quite popular (comes included in a bunch of other tools like Cmder or Git for Windows), and choco would be a better fit in those cases... so yeah not sure. For now I put choco on top in iterative/dvc.org#1195 |
I believe we did take care of this from seeing https://dvc.org/doc/install |
p.s. well except for the badges but do we want those in docs? Maybe in the README |
p.p.s. and the badges do exist in the README now. But the order there is not the same as in docs. Do you think we should reopen this and move it to the core repo @casperdcl ? |
probably could be moved to core - would be a first iteration on updating somewhat neglected readmes. https://github.com/iterative/terraform-provider-iterative/#readme just got an overhaul (iterative/terraform-provider-iterative#433) for reference/review :) |
closing as stale |
LATEST UPDATE: Scroll to #7495 (comment).
Update options including order and badges.
Probably need to split first based on OS:
The text was updated successfully, but these errors were encountered: