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

Fade-in articulation in GP4 and GP5 formats #988

Merged
merged 1 commit into from
Jul 6, 2014

Conversation

jpirie
Copy link
Contributor

@jpirie jpirie commented Jun 18, 2014

Adds support for importing the 'fade in' articulation for GP4 and GP5 formats. GP6 support has yet to be created. (Note that the fade out symbol is Guitar Pro 6 specific, I'll bundle that in with the fade in articulation support for GP6 in a future pull request.)

@@ -1658,6 +1658,12 @@ void OveToMScore::convertArticulation(
cr->add(a);
break;
}
case OVE::Articulation_Fade :{
Copy link
Contributor

Choose a reason for hiding this comment

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

If you know nothing about the ove format, I wouldn't change the code in importove.cpp, ove.cpp and ove.h

@jpirie
Copy link
Contributor Author

jpirie commented Jun 19, 2014

Thanks for the comments! I've updated the PR with another commit which takes your comments into account.

@@ -3531,4 +3531,4 @@
"wiggleVIbratoLargestSlower",
"wiggleVibratoLargestSlowest"
]
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be great if you can let the bravura files untouched.

@euwbah
Copy link

euwbah commented Jun 21, 2014

Yay! Finally some proper dynamics in playback!

@lasconic
Copy link
Contributor

@euwbah not sure what you expect, but this PR has nothing to do about playback. It "just" imports the fadein symbols from guitar pro files.

@euwbah
Copy link

euwbah commented Jun 21, 2014

Oh okay.. i thought musescore would finally have dynamics on a single note XD

@lasconic
Copy link
Contributor

Looks good now. Could you squash all these commits in one?

@jpirie
Copy link
Contributor Author

jpirie commented Jun 23, 2014

Yeah sure, no problem.

@jpirie
Copy link
Contributor Author

jpirie commented Jun 30, 2014

Does travis just need a kick on this? I can't see any actual tests failing either in that report or on my end.

@lasconic
Copy link
Contributor

I ran travis again.

@jpirie
Copy link
Contributor Author

jpirie commented Jun 30, 2014

Thanks! Gave me a diff that time, must have got lost in that previous run somehow. All should be well now. :-)

lasconic added a commit that referenced this pull request Jul 6, 2014
Fade-in articulation in GP4 and GP5 formats
@lasconic lasconic merged commit c63936e into musescore:master Jul 6, 2014
@liufeigit
Copy link

hello ,Can you give me .ove/.ovex file format document,I will development staff ,thank you !!!

@Jojo-Schmitz
Copy link
Contributor

@liufeigit better request that via the musescore.org forum or the musescore developers' mailing list

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