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

Rename QTCBS to QTB11 #235

Open
PDuckworth opened this issue Feb 19, 2016 · 5 comments
Open

Rename QTCBS to QTB11 #235

PDuckworth opened this issue Feb 19, 2016 · 5 comments

Comments

@PDuckworth
Copy link
Member

As per here or here, QTCBS is already defined as QTCB11.

Similar issue may exist with QTCC also. @yianni @cdondrup

@PDuckworth PDuckworth changed the title Rename QTCBS to QTBC11 Rename QTCBS to QTB11 Feb 19, 2016
@cdondrup
Copy link
Member

Yes it does also exist for QTCC. As I said, if this really is an issue, please feel free to refactor and also refactor strands_hri as this is using it heavily. I have no time to do this so please don't break my code just because of a naming convention.

@marc-hanheide
Copy link
Member

@PDuckworth you are right, but as this is mostly a documentation issue rather than a code issue, I suggest to simply update the documentation and point out the equivalence there.

@marc-hanheide
Copy link
Member

Or, alternatively introduce aliases in QSRlib to avoid breaking existing code.

@gatsoulis
Copy link
Contributor

@marc-hanheide Not sure what you mean by aliases, but if you are referring to dictionary mapping it might be tricky as this name is unique and tightly integrated to the retrieval and parsing of the QSRs returned also.

@cdondrup Don't worry about breaking code. Will not actually refactor but do one of the following solutions I see viable and quick:

  • Copy the current QTC?S and rename them, and documentation will only refer to the renamed one, hence leaving STRANDS code unbroken but not publicising it outside this naming. Only issue I see that any change made will mean to actually copy the files to the other.
  • Might be more elegant to subclass QTC?S from the proper naming conventions and simply overwrite the name. Will have to check the inheritance dependencies to decide how quickly this can be done.

@PDuckworth @cdondrup What about QTCCS and QTCBCS? To what should they be renamed?

@cdondrup
Copy link
Member

The versions used are QTCB11, QTCC21, and then these are combined into QTCBC which would then be QTCB11C21 to be exact but I defined that as QTCBC in my paper. It's up to you how you want to name them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants