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

Import Tyria2D from Tyria3DLibrary/dev1.2 #24

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

njibhu
Copy link
Owner

@njibhu njibhu commented Oct 9, 2020

Import the work done on Tyria2D on the dev1.2 branch of Tyria3DLibrary.

  • Work is incomplete, many viewers are not working at all.
  • The work was done against a slightly similar API. Some data in the context might not be compatible.
    The DataViewer output in particular should be double-checked.

Code went through some linting/formatting and some fixes after import.

Comment on lines +1 to +18
/*
Copyright © Tyria3DLibrary project contributors

This file is part of the Tyria 3D Library.

Tyria 3D Library is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation, either version 3 of the License, or
(at your option) any later version.

Tyria 3D Library is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.

You should have received a copy of the GNU General Public License
along with the Tyria 3D Library. If not, see <http://www.gnu.org/licenses/>.
*/
Copy link
Owner Author

Choose a reason for hiding this comment

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

Copyright notice should be updated or removed

@njibhu njibhu force-pushed the master branch 4 times, most recently from 0cec9f6 to ff07381 Compare March 16, 2021 21:19
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.

1 participant