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

Added description in iPQF header #58

Merged
merged 1 commit into from
Jun 25, 2015
Merged

Added description in iPQF header #58

merged 1 commit into from
Jun 25, 2015

Conversation

martifis
Copy link
Contributor

I edited the description in the iPQF.R from the iPQF branch. I just realised that this iPQF.R version I forked from your repository was missing 'the examples and reference' on the iPQF branch, these are only found in the iPQF.R on the master. This confused me a bit and I stayed with the branch.

So far, I only edited the description, but maybe adding some more information about the features used in the details section might be helpful after their naming is solved? Also I could add sth why a combination with MedianPolish might be useful.

lgatto pushed a commit that referenced this pull request Jun 25, 2015
Added description in iPQF header
@lgatto lgatto merged commit ea97ca3 into lgatto:iPQF Jun 25, 2015
@lgatto
Copy link
Owner

lgatto commented Jun 25, 2015

Thanks. I will delete the iPQF branch now to avoid any confusoin. iPQF was a so called feature branch, were I worked on a specific new feature and, once working, got merged into master, which is the main branch, and has now been further improved.
I have copied you improved description and added your paper as in prep.

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.

2 participants