Skip to content

Conversation

@ovalle15
Copy link
Contributor

@ovalle15 ovalle15 commented Apr 5, 2024

  • Added bulk export with data row identifiers
  • Added bulk delete with data row identifiers
  • Updated composite mask export text

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@ovalle15 ovalle15 requested a review from mina-ebeid April 5, 2024 14:49
@ovalle15 ovalle15 changed the title update composite mask notebook, and data row metadata notebook [SN-96,97] update composite mask notebook, and data row metadata notebook Apr 5, 2024
@paulnoirel
Copy link
Contributor

@ovalle15 Please ensure double or single quotes are consistent in notebooks.

@paulnoirel paulnoirel self-requested a review April 5, 2024 15:40
Copy link
Contributor

@paulnoirel paulnoirel left a comment

Choose a reason for hiding this comment

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

LGTM

@ovalle15
Copy link
Contributor Author

ovalle15 commented Apr 5, 2024

Thank you @paulnoirel

Copy link

@mina-ebeid mina-ebeid left a comment

Choose a reason for hiding this comment

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

Looks good on my end!

@ovalle15 ovalle15 merged commit 3599a77 into develop Apr 5, 2024
@ovalle15 ovalle15 deleted the ao_sn_96_97 branch April 5, 2024 16:45
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.

4 participants