Skip to content

Conversation

satyam-jha
Copy link
Contributor

i have added conda installation in docs

Copy link
Contributor

@mfeurer mfeurer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for your contribution @satyam-jha. However, the docs now contradict themselves. There's now a section "Anaconda installation" that says that Auto-sklearn cannot be installed via Anaconda, and a section which describes how to do this. Could use please make the section "Anaconda installation" two section, one about installing the package via conda-forge, and one in the spirit of the current description? Also, could you please give a link to conda forge docs for people who don't know it? Thanks a lot!

@satyam-jha
Copy link
Contributor Author

i have added "conda forge" section inside the "Anaconda installation" section and added links for both conda forge docs and feedstocks repos link

@franchuterivera franchuterivera self-requested a review June 16, 2021 18:55
@codecov
Copy link

codecov bot commented Jun 16, 2021

Codecov Report

Merging #1144 (8b1a6e1) into master (52dd9a1) will increase coverage by 2.26%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1144      +/-   ##
==========================================
+ Coverage   85.86%   88.12%   +2.26%     
==========================================
  Files         137      138       +1     
  Lines       10625    10866     +241     
==========================================
+ Hits         9123     9576     +453     
+ Misses       1502     1290     -212     
Impacted Files Coverage Δ
...tosklearn/metalearning/metafeatures/metafeature.py 62.13% <0.00%> (-14.57%) ⬇️
autosklearn/experimental/selector.py 66.97% <0.00%> (-7.03%) ⬇️
...eline/components/feature_preprocessing/fast_ica.py 91.30% <0.00%> (-6.53%) ⬇️
...mponents/feature_preprocessing/nystroem_sampler.py 85.29% <0.00%> (-5.89%) ⬇️
...onents/data_preprocessing/rescaling/standardize.py 95.23% <0.00%> (-4.77%) ⬇️
...ents/data_preprocessing/rescaling/robust_scaler.py 96.87% <0.00%> (-3.13%) ⬇️
...ature_preprocessing/select_rates_classification.py 85.91% <0.00%> (-1.41%) ⬇️
...learn/pipeline/components/regression/libsvm_svr.py 90.24% <0.00%> (-1.22%) ⬇️
...data_preprocessing/rescaling/abstract_rescaling.py 91.30% <0.00%> (-1.01%) ⬇️
...omponents/data_preprocessing/data_preprocessing.py 90.09% <0.00%> (-0.23%) ⬇️
... and 50 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 52dd9a1...8b1a6e1. Read the comment docs.

@franchuterivera
Copy link
Contributor

Hello, thanks a lot for this PR. I had some suggestions to make the installation instructions a bit more clear.

I have added some commit suggestion because the instructions did not fully work for me. The problem is that I had conda 4.8.

Additionally, could you please also update the statement in this file about:

Anaconda does not ship *auto-sklearn*, and there are no conda packages for*auto-sklearn*. Thus, it is easiest to install *auto-sklearn* as detailed inthe Section `Installing auto-sklearn`_.

Maybe to something like:

You need to enable conda-forge to install *auto-sklearn* via anaconda. This section explains how to enable conda-forge, to be able to install *auto-sklearn* with `conda install auto-sklearn`. Optionally, you can also install *auto-sklearn* with `pip` as detailed in the Section `Installing auto-sklearn`_. 

eddiebergman and others added 2 commits August 6, 2021 11:25
Co-authored-by: Francisco Rivera Valverde <[email protected]>
minor text changes
@eddiebergman eddiebergman merged commit f0cb1d1 into automl:master Aug 6, 2021
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.

4 participants