Skip to content

Updates from AI optics beamtime#294

Closed
canismarko wants to merge 6 commits into
mainfrom
ai-optics
Closed

Updates from AI optics beamtime#294
canismarko wants to merge 6 commits into
mainfrom
ai-optics

Conversation

@canismarko

Copy link
Copy Markdown
Contributor

Things to do before merging:

  • add tests
  • write docs
  • update iconfig_testing.toml
  • flake8, black, and isort

@canismarko
canismarko marked this pull request as ready for review January 29, 2025 15:51
@canismarko

Copy link
Copy Markdown
Contributor Author

Maybe these should just be included in the PR that @yannachen is preparing: #354

@canismarko

Copy link
Copy Markdown
Contributor Author

I don't think this PR is ready to merge, and we have a bunch of work to do still. I'm going to close this PR for now.

@canismarko canismarko closed this Feb 28, 2025
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