Skip to content

Bug fixes #98

Merged
eyoung55 merged 34 commits into
NatLabRockies:devfrom
arswalid:domain_restructuing
Jul 10, 2025
Merged

Bug fixes #98
eyoung55 merged 34 commits into
NatLabRockies:devfrom
arswalid:domain_restructuing

Conversation

@arswalid
Copy link
Copy Markdown
Collaborator

@arswalid arswalid commented Jun 25, 2025

This PR also includes #88 and #96 (comment)

Items fixed:

  • creation of pytest for all input files tested with meh only and full sim with 1 processor and 8 processors
  • Fix each input file to work
  • removal of input files from tracked files
  • fix to structural mesh, if coarse mesh and not enough elements on all cores the allmin or allmax shouldn't fail
  • edit to input schema to have rows in default set to zero, any panel case require explicit value of rows in both directions and allow cylinder cases to work
  • some quantities are proper to panels, if statement checks if num_panels > 0, this only works if previous bullet is set otherwise if statement is always true
  • numpy_list in cylinder 2d and 3d is set to rand number just to bypass call In mesh manger, numpy list should only be used/called for required cases
  • add solvers and pc to list of options
  • why do we have test_heated and heated.yaml inputs
  • time_varying_inflow_bc needed in input schema (removed from input schema)
  • applying Amin amax to flow manager
  • changing the test all file to set l_char to 4 if using the duramat input
  • adding inputs to gitignore

input files fixed

  • 2d panels
  • 3d panels
  • 2d cylinder
  • 3d cylinder
  • flag 2d
  • inflow_input.yaml
  • test_heatedpanels2d.yaml
  • single_heliostat.yaml
  • heated_panels2d.yaml

list still in progress
I need reviews as the PR progresses

@brookeslawski
Copy link
Copy Markdown
Collaborator

I tried running with stream_rows = 0 and span_rows = 0 and got the following error:
image

Comment thread input/panels3d.yaml Outdated
mesh_only: False
# input_mesh_dir: output/panels3d/mesh
structural_analysis: True
structural_analysis: False
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What's the reason for turning off the structural solve for panels3d?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

no specific reason, probably forgot it, ill change structure to true

hmin_local = np.inf

# collect the minimum hmin from all processes
print(hmin_local)
Copy link
Copy Markdown
Collaborator

@brookeslawski brookeslawski Jun 30, 2025

Choose a reason for hiding this comment

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

Should we remove these print statements when merging into dev? Or add the flag if verbose_flag == True: before printing?

Comment thread pvade_main.py
local_def_max = np.amax(np.sum(u_reshaped**2, axis=1))
else:
local_def_max = -np.inf # So it doesn't interfere in max
print(
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Same comment as above - do we want to remove these print statements or move them into a verbose_flag == True block?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I agree, we need to set most prints to show up ally if verbose, We need to determine which prints should always remain and which should be verbose only.

@arswalid
Copy link
Copy Markdown
Collaborator Author

arswalid commented Jun 30, 2025

I tried running with stream_rows = 0 and span_rows = 0 and got the following error: image

you need to define numpy_pt_total_array in your domain creation file

@brookeslawski
Copy link
Copy Markdown
Collaborator

I tried running with stream_rows = 0 and span_rows = 0 and got the following error: image

you need to define numpy_pt_total_array in your domain creation file

This error will be addressed in PR #89

@eyoung55 eyoung55 marked this pull request as ready for review July 10, 2025 20:09
@eyoung55 eyoung55 merged commit c0313e6 into NatLabRockies:dev Jul 10, 2025
3 checks passed
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.

3 participants