-
Notifications
You must be signed in to change notification settings - Fork 4
New hierarchy for Mapping classes #168
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
Open
Sworzzy
wants to merge
40
commits into
master
Choose a base branch
from
new-mapping-hierarchy-lagarrigue
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 29 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
9440752
fixing git workflow
Sworzzy 4ebd475
fixing the workflow so psydac runs
Sworzzy 6b6a14a
rectifying error of jacobian_inv_eval method
Sworzzy 108e70b
add AnalytciMapping to sympde/topology/__init__.py
Sworzzy b7a7d07
changes mapping class names and files, tests not made yet
Sworzzy 565cd17
deleting prints in base_analytic_mapping
Sworzzy 6dfd682
changing AbstractMapping to BaseMapping to keep domain undefined Mapping
Sworzzy 9674c41
trying to add mapping undefined domain
Sworzzy 053a21f
continuing mapping undefined domain
Sworzzy 718c251
''
Sworzzy 60bc10b
''
Sworzzy ea383d3
''
Sworzzy 4b23aa6
problem comes from BaseMapping doesn't have implemented call method, …
Sworzzy 0a79a2c
trying to solve circular dependencies
Sworzzy 5bf0c33
solving circular dependencies v2
Sworzzy bdff699
trying to fix domain undefined mapping
Sworzzy 3f91d94
deleting a print
Sworzzy b98415d
appropriate method for setting mappings for a Dom
Sworzzy 318fde3
changed the structure of the constructors of BaseMapping and
Sworzzy cb82607
trying to make the new hierearchy work : mapping undefined
Sworzzy 6d06619
repair bug for TerminalExpr with Jacobian : replacing deprecated if s…
Sworzzy 1e534e2
getting rid of the __gypyccel__ and __psydac__ git status inconveniences
Sworzzy 156bc48
test_api_feec_2d and test_2d_multipatch_mapping_maxwell run with doma…
Sworzzy 543d572
removed .DS_Store, repo cleaned
Sworzzy 3158ea0
supported changes made in commit <adding test with numpy constants>
Sworzzy 221cbed
Merge branch 'master' into feature-lagarrigue
Sworzzy b741067
merged branch master into feature-lagarrigue
Sworzzy bd61b53
adapting few changes in sympde tests
Sworzzy e71dd99
test_mapping_evaluation last version
Sworzzy 5a51f54
deleting .DS_Store
Sworzzy 542ac10
changes to make CI pass
Sworzzy 6d06f8b
test_mapping_evaluation.py should work now
Sworzzy dffd49b
use previous Collela Mapping
campospinto 7bd08f6
trailing whitespace solving
Sworzzy 786b1c1
circular imports
Sworzzy 59c11e2
trailing whitespaces
Sworzzy 15e7ad6
trailing whitespace
Sworzzy fc51c95
trailing whitespace
Sworzzy 49dc124
trailing whitespace
Sworzzy 4638b8e
trailing whitespace
Sworzzy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -111,3 +111,6 @@ doc/utils/*.png | |
|
|
||
| # Swap files | ||
| *.swp | ||
|
|
||
| __gpyccel__/* | ||
| __psydac__/* | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sworzzy marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,9 @@ | ||
| from .basic import * | ||
| from .derivatives import * | ||
| from .datatype import * | ||
| from .domain import * | ||
| from .mapping import * | ||
| from .measure import * | ||
| from .space import * | ||
| from .analytical_mapping import * | ||
| from .callable_mapping import * | ||
| from .basic import * | ||
| from .derivatives import * | ||
| from .datatype import * | ||
| from .domain import * | ||
| from .base_mapping import * | ||
| from .base_analytic_mapping import * | ||
| from .analytic_mappings import * | ||
| from .measure import * | ||
| from .space import * |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.