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

Add verbose level to ionization feature #2

Open
n01r opened this issue Dec 18, 2014 · 1 comment
Open

Add verbose level to ionization feature #2

n01r opened this issue Dec 18, 2014 · 1 comment
Assignees
@n01r
Labels
enhancement

Comments

@n01r
Copy link
Owner

n01r commented Dec 18, 2014

Some kind of intelligent verbosity level should be added to the ionization feature. Maybe look at @PrometheusPi 's radiation plugin for help.
@n01r n01r self-assigned this Dec 18, 2014
@ax3l
Copy link

ax3l commented Dec 19, 2014

please consider to simply use the standard PIConGPU log - and add either a class "PLUGIN" or use the class "PHYSICS".

btw: your routine might be a central algorithm/solver, not a plugin on the long hand.

I do not like having multiple verbose/log mechanisms in the code at all,
but @PrometheusPi 's log was older than the generalized log in PIConGPU. creating a "PLUGIN" VERBOSE_LVL might be a good idea to replace it (e.g. in combination with "PLUGIN" + "CRITICAL", "PLUGIN" + "INFO" levels so one can choose).

ax3l added a commit that referenced this issue Jan 15, 2015
@ax3l
Merge pull request #2 from psychocoderHPC/topic-tbgDefaultsExtent
83216da 
Topic tbg defaults extent
psychocoderHPC added a commit that referenced this issue Jan 21, 2015
@psychocoderHPC
Merge pull request #2 from ax3l/topic-cleanupParamFilesChangeLog
4b952f7 
Topic cleanup param files change log
PrometheusPi added a commit that referenced this issue Jan 21, 2015
@PrometheusPi
Merge pull request #2 from ax3l/radiationFilterNamings
32bfc5d 
merge changes from @ax3l
n01r pushed a commit that referenced this issue Apr 7, 2016
@ax3l
Squashed 'thirdParty/cmake-modules/' changes from ad799c9..bef9876
7e15be2 
bef9876 Merge pull request #11 from ComputationalRadiationPhysics/topic-pngwriterVersion
6a72c63 Close #2 PNGwriter Version Support

git-subtree-dir: thirdParty/cmake-modules
git-subtree-split: bef98765e6962ad6b49600c11615f42acc20eb3a
n01r pushed a commit that referenced this issue Apr 7, 2016
@ax3l
Squashed 'thirdParty/cuda_memtest/' changes from 3091fa6..0e27280
bdb099a 
0e27280 Merge pull request #6 from Flamefire/dev
8c8de37 Add space
17550d1 Merge pull request #2 from ComputationalRadiationPhysics/topic-codeCleanup
278e29c Remove Unused Parameter arg
162fa90 Fix (un)singed compares and return types
e843493 CMake: Add Correct Warning and Fix missing -pthread
06419c2 Merge pull request #3 from ComputationalRadiationPhysics/topic-travis
2997e4f Merge pull request #4 from ComputationalRadiationPhysics/fix-headerGuard
e524a78 Add .travis.yml file for CI
c821814 Fix Broken __MEMTEST_H__ Header Guard

git-subtree-dir: thirdParty/cuda_memtest
git-subtree-split: 0e27280cacfd77aec1fce6a7f82236239c02e5d1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@n01r n01r

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ax3l @n01r