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.

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

Mobile compatible #93

Merged
merged 65 commits into from
Aug 6, 2023
Merged

Mobile compatible #93

merged 65 commits into from
Aug 6, 2023

Conversation

sayansil
Copy link
Owner

@sayansil sayansil commented Mar 12, 2023 •
edited by DarkStar1997
Loading

Description and Context

Best Practices

  • My code follows the code style of this project.
  • The code base is in a better state after this PR.
  • The level of testing this PR includes is appropriate.
  • I have considered the impact that this change will cause on application performance.

Git requirements

  • I don't have multiple unrelated changes in the same PR.
  • My branch is in sync with latest production branch.

DarkStar1997
DarkStar1997 previously approved these changes Mar 12, 2023
View reviewed changes
DarkStar1997
DarkStar1997 requested changes Mar 12, 2023
View reviewed changes
Copy link
Collaborator

@DarkStar1997 DarkStar1997 left a comment

Choose a reason for hiding this comment

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

Let's add the CMake presets for mobile compatility to this PR.

@DarkStar1997
Copy link
Collaborator

Need to add workflows to add android build jobs with the newly added presets.

sayansil and others added 25 commits August 6, 2023 17:54
DarkStar1997 and others added 26 commits August 6, 2023 17:55
@DarkStar1997 @sayansil
Fix wrapper functions to return pointers as handlers
625444a
@DarkStar1997 @sayansil
Fix lint
5d98047
@DarkStar1997 @sayansil
Ranges!!!
d4d6256
@DarkStar1997 @sayansil
Embrace modernity
a75e6a3
@DarkStar1997 @sayansil
Store avg world instance in god object
d7346e9
@DarkStar1997 @sayansil
Average plot handler added
74ad481
@DarkStar1997 @sayansil
Inspection now being done elegantly and possibly more efficiently
a8921cc
@DarkStar1997 @sayansil
Move unnecessary query print
eb38f24
@DarkStar1997 @sayansil
Enable link time optimization (LTO)
d9995ae
@DarkStar1997 @sayansil
Remove redundant font files
b22f902
@DarkStar1997 @sayansil
The other guy has a habit of breaking things in production
c5b9109
@DarkStar1997 @sayansil
The platform which gives me the biggest headaches
7b26179
@sayansil
we kill and then think
fc82b8c
@sayansil
Update wrapper
217a674
@DarkStar1997 @sayansil
Get plot attribute list as string
6e8e33c
@DarkStar1997 @sayansil
Extract organism attributes
51bc9b3
@DarkStar1997 @sayansil
Extract plot stats and record them
0e45394
@DarkStar1997 @sayansil
Disable IPO for Debug builds
d25d336
@sayansil
remove non-num from plot attr
9495d8d
@sayansil
Add population in attributes
664e377
@sayansil
Lint fix
9a3a72e
@DarkStar1997 @sayansil
Add default values to missing json entries
2c46b68
@DarkStar1997 @sayansil
Remove mesa dependency
c1a4940
@DarkStar1997 @sayansil
Remove xorg dependency
261198d
@DarkStar1997 @sayansil
Don't build graphs
0199183
@DarkStar1997 @sayansil
Remove dependencies from test as well
0ceb247
@sayansil sayansil merged commit b62dd4e into master Aug 6, 2023
22 checks passed
@sayansil sayansil deleted the mobile-compatible branch August 6, 2023 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DarkStar1997 DarkStar1997 DarkStar1997 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sayansil @DarkStar1997