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.

Sign up for GitHub

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

Jump to bottom

Feature/header #240

Merged
merged 51 commits into from
Jan 12, 2014
Merged

Feature/header #240

merged 51 commits into from
Jan 12, 2014

Conversation

prudhomm
Copy link
Member

@prudhomm prudhomm commented Jan 3, 2014

This branch reorganizes some of the header and split them. We would need now to avoid including

#include <feel/feel.hpp>

which brings the full library in the user scope but rather select only the necessary headers. This requires documentation and in particular an update of the tutorial and the quick reference which would both state properly the needed header file for each feature.

For example ;

  • if a user code uses Pch<N>(mesh) then we need to include #include <feel/feeldiscr/pch.hpp>
  • if a user code uses loadmesh() then we need to include #include <feel/feelfilters/loadmesh.hpp>

@prudhomm
start working on reorganizing headers and splitting them (update issue …
b888348 
…#226)

 - create one file per public feature 
 - split gmsh.hpp, functionspace.hpp
 - move options to feel/options.cpp
 - remove some .cpp files that were dealing with program_options
@prudhomm
move BDF coefficients to bdf class and cleanup
aa65b94 
use TSBase as base class update issue #237
@prudhomm
Merge branch 'develop' into feature/header
982d540 
Conflicts:
	feel/feeldiscr/functionspace.hpp
@prudhomm
rename header for operations(*,+,/,-) in variational formulations
796a935 
update issue #226
@prudhomm
update issue #226
d123dae 
Merge branch 'feature/header' of https://github.com/feelpp/feelpp into feature/header

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
@prudhomm
fix compilation update issue #226
1217667
@prudhomm
update issue #226: work on tutorial to reduce compilation time
052db29
@prudhomm
Merge remote-tracking branch 'origin/develop' into feature/header
eaaa1b6 
Conflicts:
	feel/feeldiscr/functionspace.hpp
	feel/feeldiscr/projector.hpp
@prudhomm
update issue #226: add domain.hpp
b66578e
@prudhomm
call start to update bdf coefficients (update issue #237)
09e5fee
@prudhomm
@StephaneVeys updated test_bdf2 (arguably a simpler version)
f117138
@prudhomm
a+=m was too much
fa9805b 
added more precision
@prudhomm
@StephaneVeys update test to verify bdf implementation
fb18ff3
@prudhomm
fix indentation
c84b808
@prudhomm
update issue #237
912e373 
 - introduce bdf->priorTimes() to initialize bdf, see
 testsuite/feeldiscr/test_bdf2 for an example
 - change behavior of order change, use directly bdf order but requires
 initialization of priorTimes()
@prudhomm
fix compilation (update issue #226)
a5f3a6d
@prudhomm
fix compilation (update issue #226)
e38ffb6
@prudhomm
fix execution of sound
c4ae0d5
@prudhomm
update
5475c75
@prudhomm
update(simpler and use #226) and fix execution
affcb19
@prudhomm
update #define (#226)
e81c083
@prudhomm
fix execution in //
06fa244
@prudhomm
remove LABEL testsuite (will reduce the number of tests by half and f…
6295978 
…ailures

by half) see #88
@prudhomm
fix CrouzeixRaviart interface
cd662b5
@prudhomm
fix benchmark tilted from bit paper
b523b46
@prudhomm
various compilation fixes (see #226 and #88)
1cb3204
@prudhomm
various compilation fixes (see #226, and #88)
c453510
@buildhive
Copy link

Feel++ Consortium » feelpp #135 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@prudhomm
more compilation fixes (see #226)
eda5629
@prudhomm
use new Blocks interface
571760c 
it compiles now, it needs however to be tested
@prudhomm
Copy link
Member Author

prudhomm commented Jan 3, 2014

Note that at this stage doc, testsuite and benchmarks are compiling 100%.

@buildhive
Copy link

Feel++ Consortium » feelpp #138 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@prudhomm
Copy link
Member Author

prudhomm commented Jan 3, 2014

@StephaneVeys at this stage applications/crb compiles (and should run too). The changes were minor. At some point the call to the deprecated constructor of Application should be fixed so that we don't have all these warnings at compilation

@buildhive
Copy link

Feel++ Consortium » feelpp #141 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive
Copy link

Feel++ Consortium » feelpp #142 FAILURE
Looks like there's a problem with this pull request
(what's this?)

prudhomm added a commit that referenced this pull request Jan 12, 2014
@prudhomm
Merge pull request #240 from feelpp/feature/header
a84026d 
Feature/header
@prudhomm prudhomm merged commit a84026d into develop Jan 12, 2014
@buildhive
Copy link

Feel++ Consortium » feelpp #175 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@prudhomm prudhomm deleted the feature/header branch January 12, 2014 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@prudhomm @buildhive