lab-cosmo / i-pi-dev_archive

Development version of i-PI

21 stars 12 forks source link

Let's keep the repo clean... #152

Closed grhawk closed 7 years ago

grhawk commented 7 years ago

Branches

Simply behind master

Since there is nothing to loose deleting them, I would propose to delete them and recreate the useful ones when needed...

[x] newdep
[x] constrain
[x] mts-venkat
[x] dynma
[x] style
[x] script-interface
[x] forces-copy
[x] wte
[x] constraints
[x] mds

Easy to merge

(should just be merged, no threats)

[x] geop

Easy to merge with small revisions

(mostly due to the advances on the master)

[x] newdep-context: improve the readability when defining many dependent quantities.

ceriottm commented 7 years ago

Actually this was a useful and important attempt to clean up the depend mechanism. Basically the idea was that the whole dset/dget machinery was very cumbersome and one should have a way to access the depend machinery within the depend object. I think it would be useful to revive it.

On 9 November 2016 at 22:57, Riccardo Petraglia notifications@github.com wrote:

newdep: Merged ~6 month ago... now ~270 commit behind master

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/epfl-cosmo/i-pi-dev/issues/152, or mute the thread https://github.com/notifications/unsubscribe-auth/ABESZ-tNbEIhM0mQfqM1KM2rkdnxBtu8ks5q8kHOgaJpZM4KuCBE .

grhawk commented 7 years ago

You mean the newdep-context? The newdep should have already been merged...

ceriottm commented 7 years ago

Oh. The idea was that we had to "deprecate" the dset/dget route and modify all of the code. We can discuss that on Monday

On 11 November 2016 at 12:27, Riccardo Petraglia notifications@github.com wrote:

You mean the newdep-context? The newdep should have already been merged...

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/epfl-cosmo/i-pi-dev/issues/152#issuecomment-259938229, or mute the thread https://github.com/notifications/unsubscribe-auth/ABESZyabRk360OK8QKa-RI-r-Hi6jGtRks5q9FErgaJpZM4KuCBE .

ceriottm commented 7 years ago

Time to get rid of some stale branches. Now that we merged in the new depend mechanism, newdep_context can go - I will delete it right now as this is quite a clear-cut case. Other proposals for stale branch removal should be posted here to see if there are objections.

ceriottm commented 7 years ago

Also, I would remove bias and all the half-assed REMD branches, unless they contain valuable stuff. @grhawk as far as I am concerned they can all go so if no-one objects and you find no leftover pieces of code worth looting, let's wipe 'em all out

grhawk commented 7 years ago

Branches mentioned here have been deleted...