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

Class name update #1580

Merged
merged 4 commits into from
Sep 20, 2020
Merged

Class name update #1580

merged 4 commits into from
Sep 20, 2020

Conversation

phlptp
Copy link
Member

@phlptp phlptp commented Sep 18, 2020

Summary

If merged this pull request will change the names of some of the classes to start to work towards the agreed upon Naming conventions.

Change helicsFederateGetCoreObject to helicsFederateGetCore

@phlptp phlptp added the Code Improvement code readability or refactoring issues label Sep 18, 2020
github-actions bot and others added 2 commits September 18, 2020 12:33
@phlptp
Copy link
Member Author

phlptp commented Sep 19, 2020

@nightlark can you take a look?

@nightlark nightlark added the v3 breaking change HELICS 3 API changes label Sep 20, 2020
@phlptp phlptp merged commit 532fe4c into helics3 Sep 20, 2020
@phlptp phlptp deleted the class_name_update branch September 20, 2020 12:28
trevorhardy added a commit that referenced this pull request Oct 1, 2020
… v3userguide

* 'v3userguide' of https://github.com/GMLC-TDC/HELICS: (30 commits)
  Moving helics juggle merlin example to user-guide examples
  replacing erroneous files
  Moving the simple orchestration examples to examples directory
  Adding the simple merlin example files here
  no message
  reorg
  added link to examples page
  moving examples folder to user-guide
  moved docs/installation to docs/user-guide/installation, updated .gitignore
  Adding UQ_DAG back
  Revertng orchestration monte carlo use guide again
  Class name update (#1580)
  Command interface (#1569)
  no message
  no message
  organizing the userguide tree
  toc testing
  testing toc
  links to APIs for toc
  moving around top-level docs
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement code readability or refactoring issues v3 breaking change HELICS 3 API changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants