ODM2 / ODM2PythonAPI

A set of Python functions that provides data read/write access to an ODM2 database by leveraging SQLAlchemy.
http://odm2.github.io/ODM2PythonAPI/
BSD 3-Clause "New" or "Revised" License
4 stars 13 forks source link

Issue new release? #135

Closed emiliom closed 6 years ago

emiliom commented 6 years ago

Some important enhancements and bug fixes were issued last month, at least on a development branch. These are needed by the ODM2 REST API (https://github.com/ODM2/ODM2RESTfulWebServices/issues/59)

Besides, the last odm2api tagged release was issued 6 months ago. So, it seems like this is a good time to work towards a new release.

@horsburgh, do you agree? Someone from your team would need to take the initiative to merge the relevant dev branch into master. Then, if you'd like our help, Don and I can issue the tagged release.

In the meantime, I've added comments to 3 of the latest issues (#118, #124, #127) to learn about the status of the bugs or enhancements they discussed, and see if any of them can go into the next release.

Thanks!

cc @aufdenkampe

emiliom commented 6 years ago

Let's start compiling here all major new features and major bug fixes, to go into the release notes.

lsetiawan commented 6 years ago

124 was already implemented (see it on the API docs.) can you confirm that it was added in the last 6 months? In that case, we'll add this to the release notes.

It looks like getRelatedSamplingFeatures has been around since April 2016 See https://github.com/ODM2/ODM2PythonAPI/commit/599b6cd8885f7a2956f1b9d1f394e983b9fa5935.

lsetiawan commented 6 years ago

The last release was in May 26, 2017. Here are the major changes that occurred to the API since then.

aufdenkampe commented 6 years ago

On today's BiG CZ call, we decided to:

  1. Tag release as "Release 0.7.0", removing “pre-release”, “alpha” and/or “beta” tags. We're well beyond that, with 6 previous releases and several repos that now depend on this one!

  2. Start using Milestones to organize issues for the next release. We've used them well in other repos, but haven't done a good job with this repo. See https://github.com/ODM2/ODM2PythonAPI/milestones.

aufdenkampe commented 6 years ago

@lsetiawan, do you think you could look at the remaining open issues in these open Milestones and close both issues and Milestones if you think they were completed?

You could even create a new Milestone 0.7, then tag all the fixed issues (closed or still open) as you list above with that Milestone, to keep track of which issues were completed with this release.

emiliom commented 6 years ago

@lsetiawan: I said you were done, but I do have one thing to ask you to help me with issuing the release: could you merge the development branch into master, Friday morning? Feel free to add me as a reviewer on the PR.

@aufdenkampe, regarding this comment:

You could even create a new Milestone 0.7, then tag all the fixed issues (closed or still open) as you list above with that Milestone, to keep track of which issues were completed with this release.

I believe Don mentioned already at the call that we've tracked the issues and bug fixes going into this new release. They will be part of the release notes. So, creating a "restropective" milestone for this release just adds more work w/o the value of having a milestone for tracking issue progress. So, we'll skip it for this release -- which I plan to issue on Friday.

"Milestone 0.6.0" was something I created ages ago, accompanied by an issue (#64) listing potential targets. No one ever commented on my suggested milestone targets, and in the end the milestone wish list did not reflect reality as it happened. I've deleted the milestone, but kept the issue for now to help future assessments of potential to-do's.

emiliom commented 6 years ago

New release is now ready and stamped: https://github.com/ODM2/ODM2PythonAPI/releases/tag/v0.7.0 Request for new conda package is in the books, for Filipe: https://github.com/ODM2/conda-recipes-ODM2/issues/71

Onward and upwards!

There's one issue that I think needs to be discussed soon, before odm2api can be seen as really stable. Currently the package has a packager/folder hierarchy that includes a now-useless "ODM2" folder: https://github.com/ODM2/ODM2PythonAPI/tree/master/odm2api/ODM2

That forces most module import statements to include the useless ODM2 package hierarchy (eg, odm2api.ODM2.services.readService). We'll need to get rid of the ODM2 folder hierarchy, and the sooner we do it the better, to minimize disruptions. I strongly recommend that we do it in the next release, giving ourselves plenty of time for all code that uses odm2api to make the adjustment.

I can discuss this when I'm back from travels, after the 29th.

aufdenkampe commented 6 years ago

@emiliom, thank you for making this release happen, developing helpful release notes, and queuing up a conda package release.

I'm very much a fan of moving forward soon with the next release to remove the useless "ODM2" level of hierarchy, and also changing the default column labels to CamelCase. With release v0.7 minted as is, any software that depends on the old conventions still have handy access to that version, while we can move forward with migrating our actively developed stuff to our widespread conventions.

emiliom commented 6 years ago

I'm reopening this only b/c there are some discussions and notes here that I'd like to capture and move elsewhere. FYI, also intend to go back to very old issues to clean up the repo's issues. But that won't happen until mid February.

lsetiawan commented 6 years ago

I think we should issue a release of v0.7.1 before further optimization work. Here are the changes that happened since the last release (Jan 19, 2018):

aufdenkampe commented 6 years ago

I think that's a great idea.

lsetiawan commented 6 years ago

Following the gitflow workflow, I've created release/0.7.1 branch. This branch is used for preparing the release. ONLY bug fixes, documentation generation, and other release-oriented tasks should go in this branch. So update to the docs mentioned in https://github.com/ODM2/ODM2PythonAPI/issues/146#issuecomment-369310382, should be done in this branch. Thanks. This way further feature development off of the current development branch can still happen and not halted while waiting for the release.

emiliom commented 6 years ago

Thanks, @lsetiawan!

Don and I have discussed this new release. The plan is to create it on Friday, or Monday at the latest.

emiliom commented 6 years ago

The new release 0.7.1 is out: https://github.com/ODM2/ODM2PythonAPI/releases/tag/v0.7.1

A new conda package will be issued in conda-forge early next week.

Thanks @lsetiawan and @Elijahwalkerwest !