add some optional deps to build

[wholmgren]

Checklist

• Used a fork of the feedstock to propose changes
• Bumped the build number (if the version is unchanged)
• Re-rendered with the latest conda-smithy
• Ensured the license file is being packaged.

The pvlib-python license file is not currently packaged in the PyPI releases. Opened pvlib/pvlib-python#579 to resolve for the next release. This PR is for a new build of 0.5.2, so we'll have to skip the license for now.

[conda-forge-linter]

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

[wholmgren]

@conda-forge-admin, please rerender

[wholmgren]

@mikofski would you like to see scipy/pytables/numba/pyephem bundled with the conda package? I'm skipping netcdf/siphon for now because I can't figure out how to include a package from pypi in the build.

[mikofski]

IMO yes include everything in the conda build, or release more than one version, eg "lite" and "complete". Not sure how to include siphon and netCDF4 without vendorizing them. I think in the build script you could use skeleton but seems like a lot of work. Maybe add to docs install instructions to manually install them. I think Unidata has a conda channel, maybe try that?

[wholmgren]

I agree that including everything would be ideal, but it's not worth the effort to me. Just to clarify... do you agree that it's better to include these packages in the conda package than none at all? We're not changing the default source/pypi installation, so there would be an inconsistency there too.

[mikofski]

Sorry, I don't really have a strong opinion either way. I just thought it would be easier, from a user perspective if everything was just installed

[wholmgren]

Ok thanks! I figure that people that try to install with conda might be more likely to expect the full installation. This is at least a step in that direction. I wish I could easily write the recipe as something like Conda install pvlib = pip install pvlib[all] with packages from conda defaults and pypi as needed

…

On Sun, Sep 16, 2018 at 5:05 PM Mark Mikofski ***@***.***> wrote: Sorry, I don't really have a strong opinion either way. I just thought it would be easier, from a user perspective if everything was just installed — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#15 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AELiR2yj6tgFFifS_3vTS84ugkqHUqc6ks5ubudmgaJpZM4WqjuA> .