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 Organizations to main: #288

Merged
merged 5 commits into from Feb 19, 2018
Merged

Add Organizations to main: #288

merged 5 commits into from Feb 19, 2018

Conversation

omazapa
Copy link
Contributor

@omazapa omazapa commented Feb 5, 2018

Hello guys,
added
Jozef Stefan Institute (JSI)
University of Nebraska Lincoln (UNL)

Addes Participating Organizations to projects:
CERN, JSI -> DD4HEP
CMS -> HahRd - 3DClustering
UNL -> ROOT - RootPackageManager

minor fixes in some projects.

Cheers
Omar.

organization: LLR
organization:
- LLR
- CMS
Copy link
Contributor

Choose a reason for hiding this comment

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

CMS is not an organization but a project. Please move it to the project section.

project: HAhRD
project:
- CMS
- HAhRD
Copy link
Member

Choose a reason for hiding this comment

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

A proposal can be supported by multiple organizations, but only one project. Can you choose either CMS or HAhRD?

organisation: CERN
organisation:
- CERN
- JSI
Copy link

Choose a reason for hiding this comment

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

Please remove JSI from this proposal

@@ -0,0 +1,11 @@
---
Copy link

Choose a reason for hiding this comment

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

And since JSI is not associated with DD4heo this file can also go

Copy link

Choose a reason for hiding this comment

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

This file is superfluous since no project is associated with JSI

@oshadura
Copy link
Contributor

@omazapa any news about fate of this PR? I would like to fix my proposal (I am from University of Nebraska, and not from CERN), but I found that you already added UNL here

Josef Stefan Institute (JSI)
University of Nebraska Lincoln (UNL)

Add Participating Organizations to projects:
CERN, JSI -> DD4HEP
CMS -> HahRd - 3DClustering
UNL -> ROOT - RootPackageManager

Change project titles:

Old: Machine Learning Project - Convolutional Deep Neural Networks on GPUs for Particle Physics Applications
New: Convolutional Deep Neural Networks on GPUs for Particle Physics Applications (DONE)

Old: Optimization of the ultra-fast detector simulation package FALCON and multi-objective regression  
New: FALCON - optimize fast detector simulation package and multi-objective regression (DONE)
@omazapa
Copy link
Contributor Author

omazapa commented Feb 15, 2018

Hi @oshadura

code updated, with the changes.

Cheers,
Omar.

@@ -4,7 +4,7 @@ layout: gsoc_proposal
project:
- DD4hep
year: 2018
organization:
Copy link

Choose a reason for hiding this comment

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

This is wrong, and it has to be with a "z"

@@ -0,0 +1,11 @@
---
Copy link

Choose a reason for hiding this comment

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

This file is superfluous since no project is associated with JSI

@etejedor
Copy link
Contributor

@omazapa can you check @petricm comments? Thanks!

@omazapa
Copy link
Contributor Author

omazapa commented Feb 16, 2018

Hi @etejedor
done!
Cheers.

@etejedor
Copy link
Contributor

Thanks @omazapa for the changes.
@hegner are we happy with this one? There seems to be a failing check but it is not clear to me what the problem is.

@hegner hegner merged commit 610564c into HSF:master Feb 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants