Polishment

[hyschive]

Tasks

• Validate hierarchy: similar to _validate_parent_children_relasionship in yt/frontends/gamer/data_structures.py but make it more general
  • Probably embed it in yt_check_grid
  • The grid id should be in range 0 ~ num_grids - 1
  • Root level starts at 0.
  • If there is no parent grid, parent grid id should set to -1.
• Communicate between libyt frontend and simulation frontends
  • Already support loading fields from a simulation frontend
  • Used option: Load all key-value pair in libyt.param_user dictionary to class libytDataset.
• Support dimensionality < 3
  • Polishment and Enhancement #45
• MPI can’t transfer array length bigger than int , so libyt abort when long num_grids exceed size of int
• Update the check at edge when periodic condition. periodic[3] and grid_left_edge[3] and grid_right_edge[3] ( Test GAMER #5 )
• yt inline python script
  • file name is fixed, should we make it changeable in every round?
  • Polishment and Enhancement #45
• Rename confusing function names in yt_get_gridsPtr and yt_add_grids . Change yt_add_grids to yt_commit_grids
• Determine API procedure
  • Option 1 (now): yt_get_gridsPtr -> yt_commit_grids() -> yt_inline -> yt_free_grids()
  • Option 2: Call yt_add_grid() num_grids_local times ->yt_inline
• Input called python function name in every round of inline-analyze
• User should input different fig_basename, otherwise they will be over-written. Solution: Append number of calls to inline script at the end of the fig_basename.
• Do inline-analysis other than yt_inline_ProjectionPlot(). Make it can perform input parameter to function like yt_inline_ProjectionPlot(a,b,c) ...etc.
• Support yt_add_user_parameter_* can add type other than scalar or a 3-dim vector.
  • Polishment and Enhancement #45
• Check the python reference counting

Polishment and Optimization

• Set MPI root rank, for now, it's fixed to rank0.
  • Polishment and Enhancement #45
• Error message format
• Names for input parameter, some might be a little bit confusing
  • Change data_dim to data_dimensions.
• Name of the libyt API