GEOS icon indicating copy to clipboard operation
GEOS copied to clipboard

Feature/andrembcosta/multi resolution solver

Open andrembcosta opened this issue 3 years ago • 6 comments

Creating PR for Multi-Resolution solver

andrembcosta avatar May 26 '22 22:05 andrembcosta

@andrembcosta I think you deleted the integratedTests directory...

CusiniM avatar Jun 01 '22 00:06 CusiniM

Should I re-create it? I think I was having some problems authenticating in the integratedTests repository.

andrembcosta avatar Jun 01 '22 00:06 andrembcosta

Should I re-create it? I think I was having some problems authenticating in the integratedTests repository.

I think I have fixed it but if you have permissions problems let's solver them. Deleting it seems to create quite a few headaches when merging.

CusiniM avatar Jun 01 '22 01:06 CusiniM

Ok, I will try to pull here and see exactly what the error is.

andrembcosta avatar Jun 01 '22 01:06 andrembcosta

fracturedElements should not be empty anymore. I will fix the external set later.

CusiniM avatar Jun 01 '22 01:06 CusiniM

@rrsettgast, @CusiniM

what about merging this simplified version where the meshes of base and patch are identical?

andrembcosta avatar Aug 31 '22 20:08 andrembcosta

To do:

  • there seems to be a type issue with a 2D array in GPU builds; - FIXED
  • verify correct path to apply BCs with multiple bodies;
  • add MR integratedTest and rebaseline PF tests because of change in fieldName.

andrembcosta avatar Nov 20 '22 20:11 andrembcosta

Issue with objectPath in the inputFile.

In the MR solver, we need to specify the displacement BCs to the base problem. Prior to this most recent develop merge, the following FieldSpecification was doing that:

<FieldSpecification name="ypos" objectPath="base/Level0/nodeManager" fieldName="totalDisplacement" component="1" scale="1.0e-7" functionName="f_b" setNames="{ ypos }"/>

now, it seems that it is not finding this objectPath. What worked was to use objectPath="nodeManager", which apparently defaults to MeshBody(0) and gives the right answer.

andrembcosta avatar Nov 20 '22 20:11 andrembcosta