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

change requiredness in optical_spectoscopy raman and ellips #308

Open
wants to merge 3 commits into
base: fairmat
Choose a base branch
from

Conversation

RonHildebrandt
Copy link

Just wanted to change a few requirements in optical spectroscoyp, raman and ellipsometry.

I got earlier a problem, with the make nxdlcommand. This was caused by nyaml version 0.0.8. This did not occour for me anymore, if Iused the version 0.1.1.

I changed the requirements txt therefore.

Copy link

@RubelMozumder RubelMozumder left a comment

Choose a reason for hiding this comment

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

Need to upgrade nyaml version.

requirements.txt Outdated
@@ -1,7 +1,7 @@
# Prepare for Documentation
lxml>=4.9.1
pyyaml
nyaml==0.0.8
nyaml==0.1.1

Choose a reason for hiding this comment

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

Suggested change
nyaml==0.1.1
nyaml==0.1.2

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.

2 participants