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

create and use data_models/terms #768

Merged
merged 4 commits into from
Jun 8, 2022

Conversation

marscher
Copy link
Contributor

@marscher marscher commented May 31, 2022

Changes

split single pass schema into a common terms schema and reference it in SinglePass- and MultiPass-Weld.

Related Issues

Closes #582

Checks

  • updated CHANGELOG.rst
  • updated tests
  • updated doc/
  • update example/tutorial notebooks
  • update manifest file

@codecov
Copy link

codecov bot commented May 31, 2022

Codecov Report

Merging #768 (992dda4) into master (ceaa4c5) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #768      +/-   ##
==========================================
- Coverage   96.83%   96.78%   -0.05%     
==========================================
  Files          88       88              
  Lines        6009     6009              
==========================================
- Hits         5819     5816       -3     
- Misses        190      193       +3     
Impacted Files Coverage Δ
weldx/asdf/util.py 91.19% <0.00%> (-0.95%) ⬇️

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 ceaa4c5...992dda4. Read the comment docs.

@github-actions
Copy link

github-actions bot commented May 31, 2022

Unit Test Results

2 152 tests  ±0   2 152 ✔️ ±0   2m 54s ⏱️ -9s
       1 suites ±0          0 💤 ±0 
       1 files   ±0          0 ±0 

Results for commit 992dda4. ± Comparison against base commit ceaa4c5.

♻️ This comment has been updated with latest results.

@marscher marscher requested a review from vhirtham June 2, 2022 12:39
@marscher marscher marked this pull request as ready for review June 3, 2022 09:29
Copy link
Collaborator

@vhirtham vhirtham left a comment

Choose a reason for hiding this comment

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

LGTM

@marscher marscher merged commit e36d678 into BAMWelDX:master Jun 8, 2022
@marscher marscher deleted the schema_datamodels_terms branch June 8, 2022 11:45
@CagtayFabry
Copy link
Member

looks nice, thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

split single_pass_weld schema into sub-references
3 participants