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

add multi particle support for plugins #794

Conversation

psychocoderHPC
Copy link
Member

This pull changes many names for plugin names and plugin output files -> explicitly mention in the next changelog.

  • change plugin prefix
  • update macro documentation
  • update names to activate plugins in all *.cfg files
  • update tool plotNumericalHeating

This pull request effected all *.cfg files

Please do not merge before #802

@PrometheusPi please also take a look to this pull request

@psychocoderHPC psychocoderHPC added component: plugin in PIConGPU plugin component: tools scripts, python libs and CMake refactoring code change to improve performance or to unify a concept but does not change public API labels Apr 7, 2015
@psychocoderHPC psychocoderHPC added this to the Open Beta milestone Apr 7, 2015
@ax3l
Copy link
Member

ax3l commented Apr 8, 2015

Great pull, thank you!
Nice you updated doc/TBG_macros.cfg, too :)

##
## placeholders must be substituted with the real data name
##
## SPECIES = species name e.g. e (electrons), i (ions)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's write singular <species> and <field> here:

placeholders must be substituted with the real data name (remove the <>)

<species> = species name, e.g., e (electrons), i (ions)
<field>   = field name, e.g., FieldE, FieldB, FieldJ

@psychocoderHPC psychocoderHPC force-pushed the topic-multiSpeciesPluginSupport-rb1 branch 2 times, most recently from 9bb2f25 to 1e9ca79 Compare April 13, 2015 12:32
@psychocoderHPC
Copy link
Member Author

rebased and ready to merge

- change prefix name schema to DATA_PLUGINDESCRIPTION e.g. e_rediation, e_energy, ...
- use template sequence generator to combine plugins and data
change pattern to find energy files
add new plugin names to the macro documentation
update parameter names to activate plugins
- change TBG_macros.cfg documentation
- remove emty line in CountParticles.hpp
@psychocoderHPC psychocoderHPC force-pushed the topic-multiSpeciesPluginSupport-rb1 branch from 1e9ca79 to 9fff5e2 Compare April 13, 2015 15:03
@ax3l
Copy link
Member

ax3l commented Apr 14, 2015

btw: this is a pretty deep change for users, since many output files change. we need to document that in the changelog (comment added in description)

typedef Radiation<PIC_Electrons> RadiationElectrons;
#endif
#endif
/** apply vector component one to component zero
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • if it is vector<dataType,plugin> than you apply el. 0 in el. 1 :)

-> apply the 1st vector component to the 2nd

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls update the year in this file when you change it :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure if you want to add the change commented as a todo above with this pull: #818

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I change the name of PngPlugin in a separate pull

- change documentation for `ApplyDataToPlugin`
- rename type names with pattern `*Anonymous*` to `*Unspecialized*`
@psychocoderHPC
Copy link
Member Author

ready to merge

@ax3l
Copy link
Member

ax3l commented Apr 18, 2015

thank you for the fantastic update!

@ComputationalRadiationPhysics/picongpu-developers @ComputationalRadiationPhysics/picongpu-maintainers take care if you are an early adopter and using dev for production. this merge changes a many options you use in your .cfg files!

ax3l added a commit that referenced this pull request Apr 18, 2015
…upport-rb1

add multi particle support for plugins
@ax3l ax3l merged commit 39a639c into ComputationalRadiationPhysics:dev Apr 18, 2015
@bussmann
Copy link
Member

Amazing! Wonderful work!

@PrometheusPi PrometheusPi mentioned this pull request Apr 29, 2015
19 tasks
@psychocoderHPC psychocoderHPC deleted the topic-multiSpeciesPluginSupport-rb1 branch May 7, 2015 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: plugin in PIConGPU plugin component: tools scripts, python libs and CMake refactoring code change to improve performance or to unify a concept but does not change public API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants