Skip to content

Conversation

@AgnieszkaMakulska
Copy link
Contributor

@AgnieszkaMakulska AgnieszkaMakulska commented Jul 2, 2025

Ice as in Shima et al. (2020), with time-dependent freezing

To consider:

  • deposition in per particle substepping and with turb_cond

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 68 out of 69 changed files in this pull request and generated 6 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 68 out of 69 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

}


template <typename real_t, backend_t device>
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a turb_cond option as in cond(), or add a sanity check that deposition doesnt work (yet) with turb_cond

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added a sanity check

}
if (opts_init.ice_switch && opts_init.time_dep_ice_nucl)
{
tmp_device_real_part.add_vector();
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is an additional vector needed? Is it needed even if (allow_sstp_cond && opts_init.exact_sstp_cond), in which case there are more tmp_device_real_part vectors?

@pdziekan pdziekan merged commit 41c117d into igfuw:master Jan 19, 2026
17 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants