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

#305502 : Osc plugin controller #6095

Closed
wants to merge 1 commit into from

Conversation

NozBead
Copy link

@NozBead NozBead commented May 17, 2020

Resolves : #305502

After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

  • I signed CLA
  • I made sure the code in the PR follows the coding rules
  • I made sure the code compiles on my machine
  • I made sure there are no unnecessary changes in the code
  • I made sure the title of the PR reflects the core meaning of the issue you are solving
  • I made sure the commit message(s) contain a description and answer the question "Why do those changes fix that particular issue?" or "Why are those changes really necessary as improvements?"
  • I made sure the commit message title starts with "fix #424242:" if there is a related issue
  • I created the test (mtest, vtest, script test) to verify the changes I made

@Jojo-Schmitz
Copy link
Contributor

Jojo-Schmitz commented May 18, 2020

Please squash those commits into just one and change the commit title to start with "fix #305502:", for the the corresponding issue to automatically get marked fixed once this PR got merged.
Are you Enzo Pain on musescore.org?

In case you want this included in MuseScore 3.x, please rebase to that branch.

@NozBead
Copy link
Author

NozBead commented May 18, 2020

Please squash those commits into just one and change the commit title to start with "fix #305502:", for the the corresponding issue to automatically get marked fixed once this PR got merged.
Are you Enzo Pain on musescore.org?

In case you want this included in MuseScore 3.x, please rebase to that branch.

Yes i'm Enzo Pain on musescore.org. I'm fixing my mistakes and rebasing to the MuseScore 3.x branch.

@NozBead NozBead force-pushed the osc-plugin-controller branch 9 times, most recently from d6f9689 to 174d9bd Compare May 18, 2020 15:30
@NozBead NozBead changed the base branch from master to 3.x May 18, 2020 15:31
@NozBead NozBead force-pushed the osc-plugin-controller branch 2 times, most recently from 38ab4f4 to 174d9bd Compare May 18, 2020 15:35
@NozBead NozBead changed the base branch from 3.x to master May 18, 2020 15:35
@NozBead NozBead changed the base branch from master to 3.x May 18, 2020 15:40
@NozBead NozBead changed the base branch from 3.x to master May 18, 2020 15:51
@NozBead NozBead changed the base branch from master to 3.x May 18, 2020 16:57
@Jojo-Schmitz
Copy link
Contributor

That was a bit of a rough ride ;-)
And now the tests fail for some reason, or don't run at all.

@NozBead
Copy link
Author

NozBead commented May 18, 2020

I'm still a bit new to git :D !
Yes they aren't running at all , i'm trying to figure it out :(
appveyor was unable to build non-mergeable pull request

@NozBead NozBead closed this May 18, 2020
@NozBead NozBead reopened this May 18, 2020
@Jojo-Schmitz
Copy link
Contributor

That seems to have done the trick

Copy link
Contributor

@ecstrema ecstrema left a comment

Choose a reason for hiding this comment

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

You added some functionality in third party libraries. I know that we tend to refrain from doing that as much as possible, however, if they are cloned into the repo, it's probably because we still need to modify them sometimes. The others will know more about it.
Also, you where is the documentation for using what you just did with osc? How could i find what to do if I wanted to call a plugin?

thirdparty/ofqf/qosctypes.h Outdated Show resolved Hide resolved
mscore/plugin/mscorePlugins.cpp Outdated Show resolved Hide resolved
Changes to respect musescore coding rules

switched == nullptr to !, removed empty lines, fixed indent

thirdparty code formating

indention changes
@Moutard3
Copy link

Moutard3 commented Mar 4, 2021

Will this be merged? Looking forward to it :)

@Jojo-Schmitz
Copy link
Contributor

Jojo-Schmitz commented Mar 4, 2021

I guess it may need to get ported to the master branch, for inclusion in MuseScore 4

Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 2, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095 (I can't  seem to attribute it to @NozBead)
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 2, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095 (I can't  seem to attribute it to @NozBead)
@NozBead NozBead closed this Sep 2, 2021
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 3, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095 (I can't  seem to attribute it to @NozBead)
@Jojo-Schmitz
Copy link
Contributor

In order for me to be able to attribute this to you in my #9000, please tell me your email address (via PM, if need be)

Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 4, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
(I can't attribute s commit to @NozBead)
@NozBead
Copy link
Author

NozBead commented Sep 4, 2021

In order for me to be able to attribute this to you in my #9000, please tell me your email address (via PM, if need be)

nozbead@outlook.com

@Jojo-Schmitz
Copy link
Contributor

Thanks, done

Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 4, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 9, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 10, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 23, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 24, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 26, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 29, 2021
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 12, 2022
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Mar 5, 2023
After a lot of plugin projects I felt blocked by the lack of network communication between a plugin and an exterior app so i imagined a way of doing that kind of things.

Duplicate of musescore#6095, fix an MSVC compiler warning it introduced.
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

4 participants