Directory CCA/Components/Arches/CoalModels/

Total Files:
23
Deleted Files:
0
Lines of Code:
6886

[root]/CCA/Components/Arches/CoalModels
                    directory in repo fortran (1 files, 37 lines)

Lines of Code

CCA/Components/Arches/CoalModels/ Lines of Code

Developers

Author Changes Lines of Code Lines per Change
Totals 127 (100.0%) 3954 (100.0%) 31.1
u0407585 17 (13.4%) 1471 (37.2%) 86.5
derekhar 30 (23.6%) 979 (24.8%) 32.6
jthornoc 16 (12.6%) 760 (19.2%) 47.5
dsunder 4 (3.1%) 387 (9.8%) 96.7
dav 5 (3.9%) 123 (3.1%) 24.6
allen 39 (30.7%) 118 (3.0%) 3.0
ahumphrey 9 (7.1%) 105 (2.7%) 11.6
jsutherland 2 (1.6%) 5 (0.1%) 2.5
jas 1 (0.8%) 4 (0.1%) 4.0
wyxpuma 2 (1.6%) 2 (0.1%) 1.0
zhouminmin 2 (1.6%) 0 (0.0%) 0.0

Most Recent Commits

jsutherland 2016-12-07 13:38 Rev.: 56073

Clean up numerous compiler warnings

5 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: CharOxidation.cc (+5 -5)
allen 2016-10-14 15:59 Rev.: 55885

fixed boolean chaeck and added modified var mapper for saving vars in the timestep.xml file

2 lines of code changed in 4 files:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+1 -1), FOWYDevol.cc (+1 -1)
zhouminmin 2016-09-15 14:49 Rev.: 55752

Fixed sign error in Thermophoresis model. This will break rt 1GW_RT.ups.

0 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: Thermophoresis.cc (new)
dav 2016-09-06 12:58 Rev.: 55704


The main updates in this commit are to configure and the build system:
A number of bullet-proofing / clean-ups have been added to configure,
and the dependency of Arches on Wasatch has been removed. (Note,
Arches still needs to build TabProps/RadProps in the Wasatch 3P.) I
have tested this on several machines, but it is possible that it will
now catch some peoples' bad configure lines (that 'worked' before but
only accidentally). If your configure stops working because of this
commit, please let me know so I can help you fix it. -Dav

M configure
M configure.ac

- Force bad configure command line arguments to be errors instead of warnings.
This will help avoid users accidentally mis-typing an arg and not noticing.
The autoconf configure system does not normally allow this to happen as
their philosophy is that configure usually calls a sub-configure and passes
args down to it... Thus configure "ignores" bad args under the theory that
a "sub configure" will use them. However, we don't do this and this will
help catch configure command line typos and potentially save developers and
users a lot of headaches.

The bad news is that in order to get this functionality, we have
to hack the autconf'd generated configure script and add a
variable (enable_option_checking=fatal) at the top. I have added
a section to configure.ac with the information about doing this,
but anyone who re-generates configure needs to manually follow the
instructions.

- Configure now will immediately tell you if you have typed in an invalid
file/directory name (bad spelling, doesn't exist) for the --with flags.
Perviously this occurred manually each time (now it is part of the macro)
at the point in the configure script when the library was tested for.

- Fix handling of finding the C/C++ compilers and version checking better.
Configure now determines which compiler (eg: GCC, ICC, XLC) you are
using explicitly and can test for things (like version) appropriately.
The version check may still need to be updated a little.

- Updated the --with handling to know whether the flag was set or not. Previously
we were using a "" (blank) value, which was ambiguous.

- Flag --without as invalid on anything we don't explicitly handle (we
only handle without on a few things such as: --without-fortran or
--without-hypre).

- Allow the external Wasatch 3P to be specified on a single line (instead
of having a separate line for SpatialOps, ExprLib, etc) using
--with-wasatch3p=DIR. This will find all 4 W3P libs with one configure line.

- Got rid of IS_VS (wasn't set/used and don't know what it was for).

- While "no" and no (without quotes) are the same thing to configure, be consistent
and use "no" (with quotes) everywhere.

- Remove the BUILD_WASATCH_FOR_ARCHES logic, as we no longer need Wasatch to
be built for Arches.

- Clean up the left over files from relatively new MPI const test section.

- Fix the HAVE_ACCELERATE

- Update --enable-wasatch_3p to take a list of libs to build (in the case you only
wish to build a subset. Eg: --enable-wasatch_3p=radprops,tabprops
Pass list of libs to the build_wasatch_3p.sh script.

19 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: sub.mk (+19 -19)
allen 2016-08-16 15:35 Rev.: 55648

added debug streams and state vars

6 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+4 -4), FOWYDevol.cc (+2 -2)
allen 2016-08-16 14:33 Rev.: 55646

backed out sim state changes

27 lines of code changed in 20 files:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (+2 -2), CharOxidationSmith.cc (+6 -6), DragModel.cc (+2 -2), EnthalpyShaddix.cc (new), FOWYDevol.cc (+4 -4), KobayashiSarofimDevol.cc (new), MaximumTemperature.cc (new), RichardsFletcherDevol.cc (new), ShaddixHeatTransfer.cc (new), SimpleBirth.cc (new), YamamotoDevol.cc (+2 -2)
allen 2016-08-15 17:29 Rev.: 55641

cleanup of the simulation state - most everything is now private

1 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: ShaddixHeatTransfer.cc (+1 -1)
allen 2016-08-15 17:14 Rev.: 55638

cleanup of the simulation state - most everything is now private

26 lines of code changed in 10 files:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (+2 -2), CharOxidationSmith.cc (+6 -6), DragModel.cc (+2 -2), EnthalpyShaddix.cc (+2 -2), FOWYDevol.cc (+4 -4), KobayashiSarofimDevol.cc (+2 -2), MaximumTemperature.cc (+2 -2), RichardsFletcherDevol.cc (+2 -2), SimpleBirth.cc (+2 -2), YamamotoDevol.cc (+2 -2)
ahumphrey 2016-08-03 13:56 Rev.: 55569

Fix race condition in CharOxidationSmith.
Added copyright header and removed the opening up of std namespace.
Also removed the include of a SpatialOps header that wasn't being used.

86 lines of code changed in 3 files:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+50 -28), CharOxidationSmith.h (new)
derekhar 2016-08-02 12:59 Rev.: 55563

Fixed unnecessary limitation in charOxySmith requiring fields to be saved to disk as specified by the dataArchiver block.

28 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+28 -21)
allen 2016-06-16 11:43 Rev.: 55447

added VisIt dashborad code

23 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: FOWYDevol.cc (+23)
ahumphrey 2016-06-13 11:46 Rev.: 55440

Replace usage of NULL macro (type int) with nullptr (pointer literal of type: nullptr_t).

nullptr is implicitly convertible and comparable to any pointer type or pointer-to-member type, but is not implicitly comparable to integral types, except that it is convertible to bool (explicitly).

19 lines of code changed in 6 files:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (+6 -6), CharOxidationSmith.cc (+6 -6), CoalModelFactory.cc (new), DragModel.cc (+3 -3), FOWYDevol.cc (+3 -3)
derekhar 2016-06-02 13:51 Rev.: 55404

Changed Functor to c++11 lambda.

5 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: SimpleBirth.cc (+5 -52)
derekhar 2016-05-20 16:21 Rev.: 55371

Removing code duplication that was used for functor profiling.

1 lines of code changed in 10 files:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (-173), CharOxidationShaddix.h (+1 -2), EnthalpyShaddix.cc (-118), EnthalpyShaddix.h (new), FOWYDevol.cc (-69), FOWYDevol.h (new), SimpleBirth.cc (-26)
dav 2016-05-16 14:48 Rev.: 55353


Fix the problem with fortran .h files (in Arches) not being generated correctly (on the binary
side) when CUDA is turned on.

Update the make system so that the dependency rules for CUDA enabled files are created
automatically. The user no-longer has to specify that each CUDA_ENABLED_SRCS .cu file
depends on the corresponding .cc file. Updated both Wasatch and Arches to use the
new approach.

M Makefile.in

- Add a 'function' (make-cuda-target) that will create CUDA dependency rules automatically,
instead of having the developer have to list the rules manually. This function must
be called in each sub.mk file that creates a CUDA_ENABLED_SRCS list.
- Remove all .cu and (mostly fortran) .h files from the <bin> side when
"make cleanreally" is called.

M configure
M configure.ac

- Put in check to verify that Uintah and Wasatch 3P are both (or neither) being
built with CUDA.
- Will also add Candera check once I get more info.

A include/sci_defs/wasatch_testdefs.h.in

- Place PoKitt and Candera defines in new (configure generated binary side) wasatch_defs.h
file.

M CCA/Components/Arches/sub.mk

- Fix fortran .h file dependency problem. Note, these dependencies are have been (and
continue to be) manually entered into the sub.mk file. However, previously, they
used multiple lines (eg: Source.$(OBJEXT) was written 5 times. You don't need to
do this, you can just list it once and then put all the files it depends on.

When building without CUDA, the fortran .h files are needed to create
the object files (eg: Source.o). However, if CUDA is turned on, then they are
needed to create the CUDA .cu file (eg: <bin>/Source.cu). Perviously, at least
on some machines (though the problem should have shown up on all machines?), the fortran
.h files were not being generated.

It appears that currently Arches/sub.mk is the only directory that has CUDA files that
are dependent on fortran .h files. If this occurs in other sub-dirs for Arches (as
CUDA is added to the code), then the same approach should be taken in those files.

M CCA/Components/Arches/ChemMix/sub.mk
M CCA/Components/Arches/CoalModels/sub.mk
M CCA/Components/Arches/LagrangianParticles/sub.mk
M CCA/Components/Arches/Operators/sub.mk
M CCA/Components/Arches/ParticleModels/sub.mk
M CCA/Components/Arches/PropertyModels/sub.mk
M CCA/Components/Arches/PropertyModelsV2/sub.mk
M CCA/Components/Arches/SourceTerms/sub.mk
M CCA/Components/Arches/Task/sub.mk
M CCA/Components/Arches/Transport/sub.mk
M CCA/Components/Arches/TransportEqns/sub.mk
M CCA/Components/Arches/Utility/sub.mk
M CCA/Components/Arches/WallHTModels/sub.mk
M CCA/Components/Wasatch/Expressions/BoundaryConditions/sub.mk
M CCA/Components/Wasatch/Expressions/MMS/sub.mk
M CCA/Components/Wasatch/Expressions/PostProcessing/sub.mk
M CCA/Components/Wasatch/Expressions/Turbulence/sub.mk
M CCA/Components/Wasatch/Expressions/sub.mk
M CCA/Components/Wasatch/Operators/sub.mk
M CCA/Components/Wasatch/Transport/sub.mk
M CCA/Components/Wasatch/sub.mk

- Call make system function make-cuda-target to generate CUDA .cu dependencies.

7 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: sub.mk (+7 -44)
wyxpuma 2016-05-06 14:09 Rev.: 55335

to increase the iterations again in order to guarantee converge

1 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (+1 -1)
wyxpuma 2016-05-05 16:05 Rev.: 55327

to increase the iteration number so that a better prediction was achieved

1 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (+1 -1)
dsunder 2016-05-04 16:07 Rev.: 55310

Refactor Array3 to remove iterator to make it possible to merge trunk
into the kokkos_dev branch

387 lines of code changed in 4 files:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (+146 -153), EnthalpyShaddix.cc (+14 -19), FOWYDevol.cc (+129 -135), SimpleBirth.cc (+98 -102)
dav 2016-04-25 16:50 Rev.: 55230


Fix the ability to make just Arches (--enable-arches instead of having to --enable-all-components).

M configure
M configure.ac

- Force the use of CMake 3.1+ if building Wasatch or VisIt).
- Add in RadProps info to W3P check.

M CCA/Components/Wasatch/Transport/sub.mk
M CCA/Components/Wasatch/Expressions/sub.mk
M CCA/Components/Wasatch/Operators/sub.mk
M CCA/Components/Arches/CoalModels/sub.mk
M CCA/Components/Arches/sub.mk

- Remove superfluous white space.
- Alphabetize.
- Fix comment typo (cut/paste error).
- Untabify (as appropriate).

M CCA/Components/Wasatch/sub.mk

- Same as above and:
- Fix the problem where CUDA_ENABLED_SRCS was not set if just building Arches.
- A number of files (eg: BCHelper.cc) were listed twice in the list of sources.

M Core/Grid/UnknownVariable.h

- Remove warning (Exception is now in the Uintah namespace so don't need to qualify it).

2 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: sub.mk (+2 -2)
jthornoc 2016-04-18 13:11 Rev.: 55203

Cleaning up warnings.

212 lines of code changed in 5 files:

  • CCA/Components/Arches/CoalModels: Deposition.cc (+106 -112), PartVel.cc (-6), Thermophoresis.cc (+106 -112)
dav 2016-04-15 22:12 Rev.: 55199

This commit should fix the previous commit with respect to Uintah Variable
types. Additionally, it adds support for Arches files to be pushed through
NVCC as needed.

M CCA/Components/Wasatch/sub.mk

- Added a brief comment on use of CUDA_ENABLED_SRCS.

M CCA/Components/Arches/TransportEqns/CQMOM_Convection.h
M CCA/Components/Arches/TransportEqns/Discretization_new.h
M CCA/Components/Arches/TransportEqns/CQMOMEqn.h
M CCA/Components/Arches/TransportEqns/DQMOMEqn.h
M CCA/Components/Arches/Task/TaskInterface.h

- Minor cleanups:
- Fix warning message.
- Some white space for readability.
- Remove an empty "private:" block.

M CCA/Components/Arches/CoalModels/sub.mk
M CCA/Components/Arches/TransportEqns/sub.mk
M CCA/Components/Arches/Transport/sub.mk
M CCA/Components/Arches/sub.mk
M CCA/Components/Arches/ParticleModels/sub.mk
M CCA/Components/Arches/PropertyModels/sub.mk
M CCA/Components/Arches/ChemMix/sub.mk
M CCA/Components/Arches/PropertyModelsV2/sub.mk
M CCA/Components/Arches/Task/sub.mk
M CCA/Components/Arches/SourceTerms/sub.mk
M CCA/Components/Arches/Operators/sub.mk
M CCA/Components/Arches/WallHTModels/sub.mk
M CCA/Components/Arches/Utility/sub.mk
M CCA/Components/Arches/LagrangianParticles/sub.mk

- Update Arches sub.mk files to correctly specify which files
need CUDA (nvcc) compilation.

M Core/Grid/Variables/CCVariable.h
M Core/Grid/Variables/SFCYVariable.h
M Core/Grid/Variables/NCVariable.h
M Core/Grid/Variables/SFCXVariable.h
M Core/Grid/Variables/SFCZVariable.h
M Core/Grid/Variables/ParticleVariable.h
M Core/Disclosure/TypeDescription.h

- Turns out that the registerMe variable was needed. There was an erroneous comment
in the previous version that implied that it wasn't needed (that the work happened
in the Variable constructor, but this was not the case. I have added a number of
comments to clarify how this works so the next person to dig into it will have an
easier time.

M Core/Disclosure/TypeDescription.cc

- Update file global variables to be designated as such to help avoid confusion.
- Assign to NULL as they are pointers and not integers.


95 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: sub.mk (+95 -21)
allen 2016-04-15 17:00 Rev.: 55196

add in-situ var reference and added comments

33 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+33)
u0407585 2016-03-17 15:37 Rev.: 55063

Changed mass transfer coefficient correction factor to be consistent with derivation.

5 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+4 -4), CharOxidationSmith.h (+1 -1)
jthornoc 2016-03-17 13:15 Rev.: 55055

BUGS:
* Fix a bug with the handoff files. Both heat loss and DQMOM variables
* weren’t being properly setup, resulting in the default value always
* being used.
* Fix a bug wherein the handoff output (showing missed hand-off
* information) wasn’t being
* printed correctly for the velocities.

KOKKOS:
* Cleanup ConvectionHelper/DiscretizationTools to use Dan’s latest
* infrastructure improvements.
* This version of code gives nearly the same answers as the
* non-Kokkoized code. The diffs are understandable from an order of
* operations standpoint.

MODELS:
* For drag, use the magnitude of the difference in velocities
* For drag, use the updated weight rather than the guess. This
* helps stabilize things, BUT it isn’t general enough. —> NEEDS
* WORK<—
* In PartVel, compute the face centered velocities for the
* transport. This should allow us to phase out the
* CCVariable<Vector> velocities.

OTHER
* For MPI_Bcast, send and int rather than an unsigned int to
* remove a warning.
* Consolidated the checkBCs function into BoundaryCondition_new.

528 lines of code changed in 7 files:

  • CCA/Components/Arches/CoalModels: DragModel.cc (+152 -121), DragModel.h (new), EnthalpyShaddix.h (+45 -45), PartVel.cc (+266 -92), PartVel.h (+32 -28)
derekhar 2016-03-16 16:41 Rev.: 55046

Added new functors to the momentum solver. Also, consolidated compiler directives into a new header.

7 lines of code changed in 4 files:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.h (+2 -2), EnthalpyShaddix.h (+2 -3), FOWYDevol.h (+1 -2), SimpleBirth.cc (+2 -3)
jas 2016-03-15 18:26 Rev.: 55036

Eliminate SCIRun namespace usage.

4 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.h (+4 -4)
derekhar 2016-03-15 15:38 Rev.: 55023

Functorized Particle Birth model.

67 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: SimpleBirth.cc (+67 -9)
derekhar 2016-03-04 19:02 Rev.: 54972

The computation of the coal temperature has been functorized. Minor formatting of previous commit.

145 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.h (+145 -145)
derekhar 2016-03-04 15:32 Rev.: 54971

Added a functor for the CharOxidation Source term. Also cleaned up some warnings.

319 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (+48 -1), CharOxidationShaddix.h (+271)
derekhar 2016-03-03 16:17 Rev.: 54970

Functorized the coal devolitalization FOWY model.

174 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: FOWYDevol.cc (+36 -1), FOWYDevol.h (+138 -1)
derekhar 2016-03-03 15:01 Rev.: 54969

In the particle Enthalpy source functor, the function abs( ) was not functioning properly.
Switched abs( ) to std::abs( )

4 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: EnthalpyShaddix.cc (+2 -2), EnthalpyShaddix.h (+2 -2)
derekhar 2016-03-02 18:10 Rev.: 54956

Functorized computation of Coal-Enthalpy source term.

222 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: EnthalpyShaddix.cc (+40 -1), EnthalpyShaddix.h (+182)
u0407585 2016-02-24 17:41 Rev.: 54905

Made root function in char oxidation model a little faster.

19 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+18 -12), CharOxidationSmith.h (+1 -1)
u0407585 2016-02-24 12:08 Rev.: 54893

Moved varlable creation into problem setup. This won't effect regression tests.

23 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+23 -22)
jthornoc 2016-02-17 17:38 Rev.: 54850

pow() -> std::pow()

20 lines of code changed in 4 files:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (+10 -10), EnthalpyShaddix.cc (+7 -7), FOWYDevol.cc (+1 -1), YamamotoDevol.cc (+2 -2)
derekhar 2016-02-17 14:43 Rev.: 54846

Using std::pow( double , int ) causes answers to change when using c++11, changed to pow(double, double)

7 lines of code changed in 4 files:

  • CCA/Components/Arches/CoalModels: CharOxidationShaddix.cc (+1 -1), EnthalpyShaddix.cc (+4 -4), FOWYDevol.cc (+1 -1), YamamotoDevol.cc (+1 -1)
u0407585 2016-02-11 12:55 Rev.: 54819

Initialized temporary variable AreaSum.

2 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+2 -1)
u0407585 2016-02-10 11:12 Rev.: 54809

Fixed another sign error in new char oxidation model.

8 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+8 -8)
u0407585 2016-02-09 15:33 Rev.: 54807

fixed onother sign error in blowing parameter for new char oxidation model.

6 lines of code changed in 1 file:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+6 -8)
u0407585 2016-02-09 01:40 Rev.: 54804

Fixed mass transfer coefficient correction factor, and sign errors in the new char oxidation model.

51 lines of code changed in 2 files:

  • CCA/Components/Arches/CoalModels: CharOxidationSmith.cc (+50 -29), CharOxidationSmith.h (+1 -1)

(3 more)

Generated by StatSVN 0.7.0