-
Notifications
You must be signed in to change notification settings - Fork 2
Model with CAR-T cells #7
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
Conversation
…ture paper including forces and diffusion modules
Co-authored-by: Vassil Vassilev <v.g.vassilev@gmail.com>
Co-authored-by: Vassil Vassilev <v.g.vassilev@gmail.com>
Co-authored-by: Vassil Vassilev <v.g.vassilev@gmail.com>
Co-authored-by: Vassil Vassilev <v.g.vassilev@gmail.com>
Co-authored-by: Vassil Vassilev <v.g.vassilev@gmail.com>
use *Simulation::GetActive()->GetResourceManager() just once Co-authored-by: Vassil Vassilev <v.g.vassilev@gmail.com>
Co-authored-by: Vassil Vassilev <v.g.vassilev@gmail.com>
Co-authored-by: Vassil Vassilev <v.g.vassilev@gmail.com>
all fixes into add cart branch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There were too many comments to post at once. Showing the first 10 out of 85. Check the log or trigger a new build to see more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
clang-tidy review says "All clean, LGTM! 👍" |
vgvassilev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks quite good -- as a next step we will need to figure out how to write tests and benchmarks to make sure the model is stable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
|
clang-tidy review says "All clean, LGTM! 👍" |
vgvassilev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm!
No description provided.