-
Notifications
You must be signed in to change notification settings - Fork 29
Chem bulk relax #12
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
Open
akberovic
wants to merge
201
commits into
MUSIC-fluid:public_stable
Choose a base branch
from
akberovic:chem-bulk-relax
base: public_stable
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Chem bulk relax #12
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… the thermal shear tensor
We first read in the participant list and determine the event center. We shift the event center to origin. This is useful for asymmetric collisions such as p+Au collisions. Then we find the maximum (x, y) coordinate for the participant nucleon in the event. Finally, the grid size is determined by 2*(x_max + gridOffset) and 2*(y_max + gridOffset) Here the gridOffset = min(3, 5*nucleonWidth). We impose a minimum gridOffset = 3 fm so that when nucleonWidth < 0.6 fm, we still have a large enough grid for the simulations. We tested central Au+Au collision would have a grid size of [-12, 12] fm. and p+Au collision would have a grid size of [-7, 7] In this case, the grid spacing in small collision system will be higher for better accuracy.
Do not use the ${CMAKE_SOURCE_DIR} for the version.h.in file to avoid
confliction when compile MUSIC as an external code package in JETSCAPE
because the variable ${CMAKE_SOURCE_DIR} will be set by the JETSCAPE
directory, not locally from the MUSIC directory.
Simplely omitting it would make the compile work for both JETSCAPE and
MUSIC alone.
Because we improved the stabilitiy of the eos_hotQCD class in the previous commit, the IPGlasma2D test failed because the results is no longer identical to the previous one. We checked that they are still within negligible difference. Now, we update the numbers in the IPGlasma2D test for the latest version of the code.
We use reRunCount to record the times of reRunHydro. When reRunHydro is triggered, we will enlarge the transverse grid by 2 fm in x and y directions. The reRunHydro flag remains effective when the reRunCount < 10.
density The bug was introduced on Aug. 31, 2024 (commit #: d597e4d) when I reformatting the read_in_parameters functions. "eps_switch" needs to be the last one to modify the epsilon_freeze variable so that it is the default parameter. From Aug. 31, 2024 until now, the epsilon_freeze was set to the value of eps_freeze_max (0.18 GeV/fm^3) and user's input of eps_swtich is not effective. Thanks to Xiangyu Wu who pointed out this bug to me!
When the system is too far away from equilibirum, namely 1.5*(RShear + Rbulk) > 1, we will reduce the magnetudes of pi and bulkPi by a factor so that 1.5*(RShear + Rbulk) < 1 - 1e-6.
…tions Adopt codes from Wille_dev branch for checking necessary causal conditions and reduce the viscous stress tensor when the causal conditions are violated.
to be consistent with the resummed scheme and causality conditions
For IP-Glasma initial condition, we impose regulation on the viscous stress tensor (both shear and bulk) at the initial condition level before any evolution when FlagResumTransportCoeff is turned on.
For jet quenching calculations, ideal part of the medium evolution is sufficient. It can save disk space.
with the transverse grid is too small, add 10 more grid points on each side to enlarge the grid size every time
make alpha_preFlow dimensionless
- Add pi_b_chem member to Cell_small class with proper operator overloads - Add bulkPi_chem field to fluidCell struct in data_struct.h - Add piBulkChem_ vector field to Fields class - Update Fields methods to handle chemical bulk viscosity - Initialize piBulkChem_ field in resizeFields and destructor
Add FlagBulkChem in data.h; to make sure Cell::pi_b_chem and Fields::piBulkChem_ are used clear
…p this runs with idx_1d==14
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The calculation of bulk pressure, viscosity and relaxation time of Chemical Equilibration project