-
Notifications
You must be signed in to change notification settings - Fork 35
Gretl submodule #1495
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
base: develop
Are you sure you want to change the base?
Gretl submodule #1495
Changes from all commits
4ccb49e
77b56ab
f7e758b
c19b7ba
4ec0b48
b72a057
ae86ce3
0746ec4
1ce7e8d
2490a51
471dfa5
a928494
449b438
bb271cf
dbe1feb
f5898c5
be0ea51
96fcff9
37c76a7
61fd848
44107f6
c90d23f
0469141
ba58d95
02c56fd
37c7340
ddee2ce
80d7b4d
fce33b1
094c1d0
0474505
5a7c418
1cb451f
fa0e7cf
9e2b7a9
feb37e1
e6ce787
6206f9e
065a335
0b72bd6
c6078f6
457a261
1cc225b
dda8d50
19a6bed
bd16a6b
6458244
ea31d01
40b10a6
b648971
0d0fb7a
1b7366d
450e7b9
3118646
13440ea
b620140
3e1423f
c7a4c66
9a67661
6cbf528
d34c80b
d8caccd
fd371bf
5e3204d
eadb07f
40fab88
1f034c9
6ea2c9e
de421d5
36f578b
84de782
daebe5b
fd09f4d
0cf84c6
027ce61
29f61d5
59acd92
6ebe802
4f41b1e
46f7579
76ec37c
d54a2a3
78bd706
37799e9
7f4fd93
e8bd938
85006e1
63d7e23
b2f0b31
da81c14
d24e8d8
c5bd859
e61ce6a
6e29aec
bbd6384
7247f45
bc5ffc6
da8cf67
53b9915
6c76be1
3e57c1c
55b2bcd
d9c697b
df4aac2
70966fd
33f6146
a21f093
859f3d4
6fc519a
f0566d7
b783fe7
f2e6557
ebdd79f
74b55dc
6f22e78
ac092ae
830a42f
d96dfb2
c2f357a
fa44b88
684dc37
89bf9c7
b1cd821
915d9ae
dad59e5
da54478
1834014
c9050b8
b36dc34
9c07a25
c0874cd
8f97597
eca5b66
ea120a4
94b9360
f24b930
2941c31
0e55b13
baeabc0
b65133f
13714a7
8773991
9a25bad
51a8522
7e43f3a
8755cc2
d860394
a1430b8
8f00d5b
82a8a87
08b4bcc
db6046b
585456f
dc166bb
d42ae3e
5ffe6e6
edcbee7
6a9fa46
0362fee
a000ebf
f205481
a76c437
b8ef7dd
7b4370b
4242eff
1b29101
0bef5fd
d50cff6
a315584
b4e7234
88650f6
524906e
a42ee65
fa4a882
18f4347
d6a7d4e
a6980f7
bbaa967
7e2cd78
678ec06
d127520
d89ec82
f15a351
4252a44
8855f26
37ba752
858dced
5e0c330
d189a6e
13e1b72
2d929c4
46695fe
4e64694
c16a326
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,55 @@ | ||
| # Copyright (c) Lawrence Livermore National Security, LLC and | ||
| # other Smith Project Developers. See the top-level LICENSE file for | ||
| # details. | ||
| # | ||
| # SPDX-License-Identifier: (BSD-3-Clause) | ||
|
|
||
| set(differentiable_numerics_sources | ||
| field_state.cpp | ||
| differentiable_utils.cpp | ||
| differentiable_physics.cpp | ||
| lumped_mass_explicit_newmark_state_advancer.cpp | ||
| ) | ||
|
|
||
| set(differentiable_numerics_headers | ||
| field_state.hpp | ||
| state_advancer.hpp | ||
| timestep_estimator.hpp | ||
| explicit_dynamic_solve.hpp | ||
| lumped_mass_weak_form.hpp | ||
| differentiable_utils.hpp | ||
| differentiable_physics.hpp | ||
| lumped_mass_explicit_newmark_state_advancer.hpp | ||
| ) | ||
|
|
||
| set(differentiable_numerics_depends | ||
| gretl | ||
| smith_state | ||
| smith_boundary_conditions | ||
| smith_contact | ||
| smith_numerics | ||
| smith_physics | ||
| ) | ||
|
|
||
| blt_add_library( | ||
| NAME smith_differentiable_numerics | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think my naming may have caused confusion here. The include dependencies make it clear that differentiable_numerics does not belong in numerics. It combines things in physics, numerics, gretl, etc. to create analogous structures as exist in numerics, but which will be differentiated on the gretl graph. Its not a perfect match to numerics, so perhaps a name change could avoid confusion. differentiable_utilities was another option I was considering. If people have suggestions, I'm happy to change it, but preferably in a follow on PR. |
||
| SOURCES ${differentiable_numerics_sources} | ||
| HEADERS ${differentiable_numerics_headers} | ||
| DEPENDS_ON ${differentiable_numerics_depends} | ||
| ) | ||
|
|
||
| smith_write_unified_header( | ||
| NAME differentiable_numerics | ||
| HEADERS ${differentiable_numerics_headers} | ||
| ) | ||
|
|
||
| install(FILES ${differentiable_numerics_headers} DESTINATION include/smith/differentiable_numerics ) | ||
|
|
||
| install(TARGETS smith_differentiable_numerics | ||
| EXPORT smith-targets | ||
| DESTINATION lib | ||
| ) | ||
|
|
||
| if(SMITH_ENABLE_TESTS) | ||
| add_subdirectory(tests) | ||
| endif() | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to be able to point to a checked out gretl or an installed one like everything else? I think I would prefer an installed one if we allow it at all.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no preference, nor the background to really know the pros and cons. So, up to you! Someone was trying to build with smith_cz and smith with this branch, they were able to get it to work by manually pointing to the gretl.