Skip to content

Add Proper License to com.jme3.anim files #1359

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.

Sign up for GitHub

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

Jump to bottom
Merged
stephengold merged 10 commits into from May 9, 2020
Merged

Add Proper License to com.jme3.anim files #1359

stephengold merged 10 commits into from May 9, 2020

Conversation

@ghost
Copy link

@ghost ghost commented Apr 22, 2020

In order to chip away at this issue, #1001
I've added the proper jME license to the Anim files written by @Nehon (or at least the ones that he committed to, excluding the ones by pspeed which already have the new licenses) .
Now in the previous discussion on that issue, it seemed as if @stephengold was attempting to contact nehon to get his permission. If Nehon replies, then we'll follow from there, but if he doesn't, I say that since he contributed as an engine leader, it would be fine to go ahead and add the licenses.

ItsMike54 added 3 commits April 22, 2020 12:24
Update Weights.java
c8c7026
Update SkinningControl.java
28daacb
Update SeparateJointModelTransform.java (#2)
abba5d4 
* Update SeparateJointModelTransform.java

* Update MorphControl.java

* Update MatrixJointModelTransform.java

* Update Joint.java

* Update ArmatureMask.java

* Update Armature.java

* Update AnimationMask.java

* Update AnimTrack.java

* Update AnimComposer.java

* Update AnimClip.java

* Update Weighted.java

* Update Primitives.java

* Update JointModelTransform.java

* Update HasLocalTransform.java

* Update AnimMigrationUtils.java

* Update ContainsTweens.java

* Update LinearBlendSpace.java

* Update ClipAction.java

* Update BlendableAction.java

* Update BlendSpace.java

* Update BlendAction.java

* Update BaseAction.java

* Update Action.java
stephengold
stephengold reviewed Apr 22, 2020
View reviewed changes
stephengold
stephengold reviewed Apr 22, 2020
View reviewed changes
@ghost
Copy link
Author

ghost commented Apr 22, 2020

@stephengold , simply remove those headers then? Or try to find their author?

stephengold
stephengold reviewed Apr 22, 2020
View reviewed changes
stephengold
stephengold reviewed Apr 22, 2020
View reviewed changes
stephengold
stephengold reviewed Apr 22, 2020
View reviewed changes
@stephengold
Copy link
Member

stephengold commented Apr 22, 2020

@stephengold , simply remove those headers then? Or try to find their author?

I don't know what to do with these files. Their style matches Nehon's, so I'm 90% sure he wrote them. But he's the only person who knows for sure, and as I said, he hasn't responded to my inquiries. Perhaps someone else will have better luck contacting him.

@ghost
Copy link
Author

ghost commented Apr 22, 2020 •
edited by ghost
Loading

Well assumptions bite me in the ass, and I'll remove those right now.
Edit: Done

Update AnimTrack.java (#3)
9f84a13 
* Update AnimTrack.java

* Update ArmatureMask.java

* Update MatrixJointModelTransform.java

* Update SeparateJointModelTransform.java

* Update Weights.java

* Update Action.java

* Update BaseAction.java

* Update ContainsTweens.java

* Update SkinningControl.java
ItsMike54 added 2 commits April 24, 2020 12:45
Update AnimTrack.java (#6)
a2f5c7a 
* Update AnimTrack.java

* Update BlendAction.java

* Update BlendSpace.java

* Update BlendableAction.java
Update AnimMigrationUtils.java (#7)
936f09d 
* Update AnimMigrationUtils.java

* Update HasLocalTransform.java

* Update JointModelTransform.java

* Update Primitives.java

* Update Weighted.java

* Update ClipAction.java

* Update AnimInterpolator.java

* Update AnimInterpolators.java

* Update FrameInterpolator.java
@ghost
Copy link
Author

ghost commented Apr 24, 2020

@stephengold , I went ahead and reviewed the files, removing all that Nehon didn't expressly state his authorship in. I did add licenses to some that I missed previously that he did state authorship of.

@stephengold
Copy link
Member

stephengold commented Apr 24, 2020

Please take another look at LinearBlendSpace.java

@ghost
Copy link
Author

ghost commented Apr 24, 2020

Obviously I can't seem to make a perfect PR just yet, but LinearBlendSpace.java is fixed.

@stephengold
Copy link
Member

stephengold commented May 8, 2020

As far as I can tell, this PR just needs the one-line change to AnimTrack.java and then it will be ready to merge.

@ItsMike54 have you abandoned this PR?

@ghost
Copy link
Author

ghost commented May 9, 2020 •
edited by ghost
Loading

@stephengold Over the past week I've been trying to make progress on a game and I don't have much time to do so, so I abandoned this PR for that time.
But since it was brought up again, I reverted the AnimTrack.java interface back to its original state before I modified it.

@stephengold
Copy link
Member

stephengold commented May 9, 2020

It's not back to its original state.

@ghost
Copy link
Author

ghost commented May 9, 2020

@stephengold , I deleted the AnimTrack.java from my branch and uploaded the AnimTrack.java interface that is currently on Master jME branch. That file is completely unmodified, and I don't know what else to say about it.

@stephengold stephengold merged commit f808187 into jMonkeyEngine:master May 9, 2020
@stephengold stephengold added this to the v3.4.0 milestone Mar 13, 2021
@stephengold stephengold added the Documentation Issues that affect the Wiki, Javadoc or any other form of documentation label Mar 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Documentation Issues that affect the Wiki, Javadoc or any other form of documentation

Projects

None yet

Milestone

v3.4.0

Development

Successfully merging this pull request may close these issues.

1 participant

@stephengold