Skip to content

Conversation

@noemifrisina
Copy link
Collaborator

Set up some tools for checking that the structure/datasets/attributes/whatever are correct

Non-exhaustive list

  • Correct definition (and spelling)
  • Dependency tree
  • Units
  • Correct data type (floats/strings/...) in datasets
  • module_offset calculations

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Sep 6, 2021

This pull request introduces 1 alert when merging 718e37e into 8862915 - view on LGTM.com

new alerts:

  • 1 for Unnecessary delete statement in function

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Sep 6, 2021

This pull request introduces 1 alert when merging b1d0085 into 8862915 - view on LGTM.com

new alerts:

  • 1 for Unnecessary delete statement in function

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Sep 6, 2021

This pull request introduces 1 alert when merging 8ace170 into 8862915 - view on LGTM.com

new alerts:

  • 1 for Unnecessary delete statement in function

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Sep 7, 2021

This pull request introduces 1 alert when merging 26a5282 into 8862915 - view on LGTM.com

new alerts:

  • 1 for Unnecessary delete statement in function

@noemifrisina noemifrisina marked this pull request as draft October 4, 2021 15:48
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