Skip to content

Upgrade infastructure and improve rendering #23

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
merged 62 commits into from
Dec 19, 2024
Merged

Upgrade infastructure and improve rendering #23

merged 62 commits into from
Dec 19, 2024

Conversation

da-luce
Copy link
Owner

@da-luce da-luce commented Dec 19, 2024

  • Embed data (fixes running from outside build dir)
  • Add constellations and improve rendering
  • Fix moon phase
  • Testing
  • Code coverage
  • Formatting

da-luce and others added 30 commits December 16, 2024 15:39
@da-luce
add unit testing framework
7493975
@da-luce
write first coord unit test
da84e77
@da-luce
add codecov action
b90f4fd
@da-luce
change meson install
25ecc5f
@da-luce
fix meson setup
112ea61
@da-luce
install code cov tools
2a4ef4a
@da-luce
fix codecov file path
bd8b7f3
@da-luce
fix codecov token, maybe
ad2972c
@da-luce
fail if codecov updload fails
9962e7c
@da-luce
add julian date test
8a00cb9
@da-luce
begin work on embedding BSC5 into source
9cc99ef
@da-luce
parse data and names via embedded
b2475f8
@da-luce
embed constellations
9c597ea
@da-luce
include last constellation in rendering
9084382
@da-luce
stop tracking bsc5
73502a3
@da-luce
rename data files
0433944
@da-luce
remove bsc5
264d1a0
@da-luce
update screenshot
199c8c5
@da-luce
update meson minimum version and add vscode settings
de0f4e0
@da-luce
Merge pull request #14 from da-luce/infra
8e8ea23 
Unit tests and code coverage
@da-luce
merge dev into bsc5_fix
0bd52ab
@da-luce
Merge pull request #15 from da-luce/bsc5_fix
68b38d4 
Embed bsc5
@da-luce
add install script
fdc3070
@da-luce
fix bsc5 install location
ecc9ec1
@da-luce
try vim xxd instead
f733109
@da-luce
remove -n option from xxd
d7d5637
@da-luce
clean readme and check for xxd
f20b9f8
@da-luce
add dependency check
89dfae5
@da-luce
remove array from install script
716670e
@da-luce
Merge pull request #16 from da-luce/fix_install
ec04c0c 
Generalize data embedding
da-luce and others added 26 commits December 18, 2024 11:56
@da-luce
move unity to third_party, add codecov ignore
1291e0a
@da-luce
add all constellation figures
f192c33
@da-luce
clip constellations
b5a76a4
@da-luce
update help for constellations
b885ca6
@da-luce
move unity back into test dir
c3d20b1
@da-luce
add codecov app
5f44958
@da-luce
Merge pull request #18 from da-luce/fix_coverage
172160c 
Add codecov ignore and app
@da-luce
Merge pull request #19 from da-luce/constell
300e8bb 
Improve constellations
@da-luce
add features to readme
f974156
@da-luce
Merge pull request #20 from da-luce/moon_fix
d373b02 
Fix moon phase
@da-luce
simple unicode tweak
c763e21
@da-luce
Merge pull request #21 from da-luce/fix_unicode
1ef9d44 
Remove broken unicode character
@da-luce
use clang format
d53b864
@da-luce
add clang-format-ignore
eda1c75
@da-luce
fix clang ignore
073dd0e
@da-luce
add build to ignore list
e29d249
@da-luce
add hokey format script becase clang-format sucks
ec86d9e
@da-luce
generalize format script
7747905
@da-luce
correct exclude paths
2941aaf
@da-luce
use local format file... maybe
5ba59c1
@da-luce
add version check
cd9d00d
@da-luce
disable format for weird line
14ce48f
@da-luce
note weird vscode format on save bug
dc5228d
@da-luce
Merge pull request #22 from da-luce/format
e95cc7a 
Add formatting
@da-luce
remove codecov status thingy
225e435
@da-luce
add gif
9ceca9c
@codecov Codecov
Copy link

codecov bot commented Dec 19, 2024

Codecov Report

Attention: Patch coverage is 21.60612% with 410 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/core.c 0.00% 93 Missing ⚠️
src/main.c 0.00% 82 Missing ⚠️
src/core_render.c 0.00% 58 Missing ⚠️
src/drawing.c 0.00% 53 Missing ⚠️
src/parse_BSC5.c 0.00% 28 Missing ⚠️
src/astro.c 29.72% 26 Missing ⚠️
src/coord.c 13.33% 26 Missing ⚠️
src/core_position.c 0.00% 19 Missing ⚠️
src/term.c 0.00% 11 Missing ⚠️
src/bit.c 0.00% 7 Missing ⚠️
... and 1 more
Files with missing lines Coverage Δ
test/astro_test.c 100.00% <100.00%> (ø)
test/coord_test.c 100.00% <100.00%> (ø)
src/bit.c 0.00% <0.00%> (ø)
src/stopwatch.c 0.00% <0.00%> (ø)
src/term.c 0.00% <0.00%> (ø)
src/core_position.c 0.00% <0.00%> (ø)
src/astro.c 10.97% <29.72%> (ø)
src/coord.c 9.25% <13.33%> (ø)
src/parse_BSC5.c 0.00% <0.00%> (ø)
src/drawing.c 0.00% <0.00%> (ø)
... and 3 more

@da-luce da-luce merged commit 8116469 into main Dec 19, 2024
3 checks passed
da-luce added a commit that referenced this pull request Dec 21, 2024
@da-luce
Merge pull request #23 from da-luce/dev
c28fbb5 
Upgrade infastructure and improve rendering
da-luce added a commit that referenced this pull request Dec 21, 2024
@da-luce
Merge pull request #23 from da-luce/dev
6187c2b 
Upgrade infastructure and improve rendering
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@da-luce