Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate steppyngstounes and azure #830

Closed
wants to merge 176 commits into from

Conversation

guyer
Copy link
Member

@guyer guyer commented Nov 23, 2021

Integration tests of #777 and #822

This reverts commit 50110f2.
mamba doesn't work right from within a Python 2.7 environment
Troubleshooting how environment has changed from 605cacc
mamba doesn't work from a Py2.7 environment,
but mamba needs to be installed in a Py3k environment for some reason
…ngstounes' into integrate_steppyngstounes_and_azure
@guyer
Copy link
Member Author

guyer commented Nov 23, 2021

  • Substantially slower
  • Trilinos parallel breaks:
     Traceback (most recent call last):
       File "/usr/share/miniconda/envs/myEnvironment/lib/python3.7/doctest.py", line 1337, in __run
         compileflags, 1), test.globs)
       File "<doctest examples.phase.binaryCoupled[93]>", line 7, in <module>
         res = eq.sweep(dt=step.size, solver=solver)
       File "/home/vsts/work/1/s/fipy/terms/term.py", line 214, in sweep
         solver = self._prepareLinearSystem(var=var, solver=solver, boundaryConditions=boundaryConditions, dt=dt)
       File "/home/vsts/work/1/s/fipy/terms/term.py", line 130, in _prepareLinearSystem
         buildExplicitIfOther=self._buildExplcitIfOther)
       File "/home/vsts/work/1/s/fipy/terms/coupledBinaryTerm.py", line 88, in _buildAndAddMatrices
         buildExplicitIfOther=buildExplicitIfOther)
       File "/home/vsts/work/1/s/fipy/terms/binaryTerm.py", line 34, in _buildAndAddMatrices
         buildExplicitIfOther=buildExplicitIfOther)
       File "/home/vsts/work/1/s/fipy/terms/unaryTerm.py", line 67, in _buildAndAddMatrices
         diffusionGeomCoeff=diffusionGeomCoeff)
       File "/home/vsts/work/1/s/fipy/terms/cellTerm.py", line 128, in _buildMatrix
         dt = self._checkDt(dt)
       File "/home/vsts/work/1/s/fipy/terms/transientTerm.py", line 128, in _checkDt
         raise TypeError("`dt` must be a single number, not a " + type(dt).__name__)
     TypeError: `dt` must be a single number, not a ndarray   
    

@guyer
Copy link
Member Author

guyer commented Nov 30, 2021

#777 builds on azure, now

@guyer guyer closed this Nov 30, 2021
@guyer guyer deleted the integrate_steppyngstounes_and_azure branch December 1, 2021 23:53
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.

1 participant