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

geomeTRIC #6503

Merged
merged 5 commits into from
Aug 16, 2018
Merged

geomeTRIC #6503

merged 5 commits into from
Aug 16, 2018

Conversation

dgasmith
Copy link
Contributor

@dgasmith dgasmith commented Aug 14, 2018

A geometry optimizer for molecular chemistry. @leeping @yudongqiu

@conda-forge-linter
Copy link

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

I wanted to let you know that I linted all conda-recipes in your PR (recipes/geometric) and found some lint.

Here's what I've got...

For recipes/geometric:

  • Failed to even lint the recipe, probably because of a conda-smithy bug 😢. This likely indicates a problem in your meta.yaml, though. To get a traceback to help figure out what's going on, install conda-smithy and run conda smithy recipe-lint . from the recipe directory.

@conda-forge-linter
Copy link

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 (recipes/geometric) and found it was in an excellent condition.

@loriab
Copy link
Contributor

loriab commented Aug 15, 2018

pypa/pip#5679 again, I think.

@dgasmith
Copy link
Contributor Author

@loriab I think: leeping/geomeTRIC#32?


source:
fn: geomeTRIC-{{ version }}.tar.gz
url: https://github.com/leeping/geomeTRIC/archive/{{ version }}.tar.gz
Copy link
Contributor

Choose a reason for hiding this comment

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

Since package/name applies .lower(), you can probably use {% set name = "geomeTRIC" %}, then use the templating in source fields.

But more importantly, yes, give 0.8.8 a try.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

0.8.8 seems to have fixed the issue and I made the suggested changes.

build:
noarch: python
number: 0
script: "{{ PYTHON }} -m pip install . --no-deps --ignore-installed --no-cache-dir -vvv"
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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


extra:
recipe-maintainers:
- leeping
Copy link
Member

Choose a reason for hiding this comment

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

@leeping, are you ok with being a maintainer?

Copy link

Choose a reason for hiding this comment

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

Yes, thank you.

recipe-maintainers:
- leeping
- dgasmith
- yudongqiu
Copy link
Member

@isuruf isuruf Aug 15, 2018

Choose a reason for hiding this comment

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

@yudongqiu, are you ok with being a maintainer?

Copy link

Choose a reason for hiding this comment

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

@yudongqiu Please reply. (I saw a typo above)

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes.

summary: 'Molecular geometry optimizer in the TRIC coordinate system.'

# description: |
# doc_url: http://simplejson.readthedocs.io/
Copy link
Member

Choose a reason for hiding this comment

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

Please remove comments

- setuptools
- pip
run:
- numpy>=1.11
Copy link
Member

Choose a reason for hiding this comment

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

There should be a space, numpy >=1.11, but you could also use pin_compatible() https://conda.io/docs/user-guide/tasks/build-packages/variants.html?highlight=pin_compatible#pinning-at-the-recipe-level

- setuptools
- pip
run:
- {{ pin_compatible('numpy', min_pin='1.11') }}
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's lower_bound='1.11'. min_pin/max_pin expect x.x format. Example. I'm still not sure from docs if pin_compatible doesn't require its target to be in reqs/host, but I'm curious to see the test.

You can change reqs/build to reqs/host, btw.

@loriab loriab merged commit 3f01e3d into conda-forge:master Aug 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

7 participants