Skip to content

close #697: fix radiation types#778

Merged
psychocoderHPC merged 6 commits into
ComputationalRadiationPhysics:devfrom
PrometheusPi:fix_radiationTypes
Mar 26, 2015
Merged

close #697: fix radiation types#778
psychocoderHPC merged 6 commits into
ComputationalRadiationPhysics:devfrom
PrometheusPi:fix_radiationTypes

Conversation

@PrometheusPi
Copy link
Copy Markdown
Member

This pull request replaces the float types used in the radiation plugin by standard PIConGPU types and it renames the vector classes as suggested by @psychocoderHPC in issue #697.

  • numtype1 -> picongpu::float_32
  • numtype2 -> picongpu::float_64
  • vec1 -> vector_32
  • vec2 -> vector_64

ToDo:

  • check SingleParticleRadiationWithLaser

Notice:
@ax3l This pull request adjusts *.param and *.unitless files.

@PrometheusPi PrometheusPi added the refactoring code change to improve performance or to unify a concept but does not change public API label Mar 25, 2015
@PrometheusPi PrometheusPi added this to the Open Beta milestone Mar 25, 2015
@PrometheusPi
Copy link
Copy Markdown
Member Author

Results of the test simulations verify the correctnis of the pull request:

dev:
dev

branch:
branch

@bussmann
Copy link
Copy Markdown
Member

Find the 10 differences.

@PrometheusPi
Copy link
Copy Markdown
Member Author

They are actually identical:

$ diff --report-identical-files 00*/simOutput/totalRad/RadiationElectrons_4000.dat 
Files 001_branch/simOutput/totalRad/RadiationElectrons_4000.dat and 002_dev/simOutput/totalRad/RadiationElectrons_4000.dat are identical

I just like to upload vivid plots. 😄

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we remove this line?

psychocoderHPC added a commit that referenced this pull request Mar 26, 2015
@psychocoderHPC psychocoderHPC merged commit a06b455 into ComputationalRadiationPhysics:dev Mar 26, 2015
@PrometheusPi PrometheusPi deleted the fix_radiationTypes branch March 26, 2015 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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