Apply breduce to reduce snapshot projection time

Nek5000 / NekROM

Model-Order Reduction Framework for Nek5000

13 stars 3 forks source link

Apply breduce to reduce snapshot projection time #217

Closed kent0 closed 7 months ago

kent0 commented 8 months ago

implemented uip (universal inner-product) to replace pv2k and ps2k with p2k. uip will be used to replace various inner-products subroutines to simplify and in the future will be used to form the Gramian and the operators.

Ping-Hsuan commented 8 months ago

@kent0, can you explain to me why changing from lsu to lsu+1? Was the allocation for those variables are not correct? Or it is just for consistency? Thanks!

kent0 commented 8 months ago

@Ping-Hsuan, I still need to modify the breduce subroutine so please wait for that commit before merging.

kent0 commented 7 months ago

@Ping-Hsuan, this PR is good from my side (I think the load balancing modification is unnecessary) so please merge if after review.