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

Additional small changes #89

Merged
merged 8 commits into from
Aug 26, 2024
Merged

Additional small changes #89

merged 8 commits into from
Aug 26, 2024

Conversation

nmoust
Copy link
Collaborator

@nmoust nmoust commented Aug 26, 2024

This PR includes the following changes:

IRI Before After Comment
0000080 fourier transform infrared spectroscopy Fourier transform infrared spectroscopy Changed f to F in Fourier
0007024 rate of reaction reaction rate Interchange of preferred and alternate labels. This fits better other similar concepts like reaction name and reaction type
00000017 powdered photocatalyst powdered catalyst Make concept more general. Definition also changed (photocatalyst --> catalyst
0000122 - - Added "sample pretreatment" as Alternate label of "sample pre-treatment"

nmoust and others added 5 commits June 17, 2024 16:03
Changes mostly focusing on generalizing pre-existing photocatalysis concepts (+ fixing some typos).
This PR includes some changes in already existing concepts (typos, generalizing concepts etc.)
@nmoust nmoust requested a review from dalito August 26, 2024 13:03
vocabularies/voc4cat/0000181.ttl Outdated Show resolved Hide resolved
vocabularies/voc4cat/0000017.ttl Outdated Show resolved Hide resolved
nmoust and others added 2 commits August 26, 2024 16:51
Changes included:
1) Deleted the second sentence in the "powdered catalyst" concept (0000017).
2) Deleted the w3.org link from concepts 0000181 and 0005009.
3) Generalized concepts 0000038 and 0000054.
4) Edited the preferred label of concepts 0000064 and 0000065.
@dalito dalito merged commit c32c504 into nfdi4cat:main Aug 26, 2024
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