Skip to content

Conversation

@kalmarek
Copy link
Owner

@kalmarek kalmarek commented Jul 5, 2024

CC @Khashayar-Neshat

@kalmarek kalmarek force-pushed the mk/make_examples_great_again branch 4 times, most recently from d8e8e54 to 829b6bb Compare July 5, 2024 21:21
@kalmarek
Copy link
Owner Author

kalmarek commented Jul 6, 2024

I'm trying to run the examples through a gh workflow here. For some reason I'm not able to locate the examples folder?
If I ask for pwd I get

 [ Info: /home/runner/work/SymbolicWedderburn.jl/SymbolicWedderburn.jl

however if I want to include(examples/preamble.jl) a totally different path comes out:

ERROR: LoadError: SystemError: opening file "/home/runner/work/_temp/examples/preamble.jl": No such file or directory

any ideas? @blegat

@blegat
Copy link
Collaborator

blegat commented Jul 6, 2024

Use @__DIR__ or pathof(SymbolicWedderburn)

@kalmarek kalmarek force-pushed the mk/make_examples_great_again branch 5 times, most recently from b029c2d to 4f1f281 Compare July 8, 2024 12:39
@kalmarek kalmarek force-pushed the mk/make_examples_great_again branch from 4f1f281 to e86b80a Compare July 8, 2024 12:57
@codecov
Copy link

codecov bot commented Jul 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.45%. Comparing base (8f64019) to head (36a60c8).
Report is 34 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   90.74%   88.45%   -2.29%     
==========================================
  Files          23       24       +1     
  Lines        1329     1343      +14     
==========================================
- Hits         1206     1188      -18     
- Misses        123      155      +32     
Flag Coverage Δ
unittests 88.45% <ø> (-2.29%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kalmarek kalmarek force-pushed the mk/make_examples_great_again branch from 36a60c8 to f02572a Compare July 8, 2024 14:14
@kalmarek kalmarek force-pushed the mk/make_examples_great_again branch from f02572a to 061a062 Compare July 8, 2024 14:46
@kalmarek
Copy link
Owner Author

kalmarek commented Jul 8, 2024

finally include(joinpath(pwd(), ...)) solved it... what a waste of time :P

@kalmarek kalmarek merged commit d9266a7 into master Jul 8, 2024
@kalmarek kalmarek deleted the mk/make_examples_great_again branch July 8, 2024 17:22
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.

3 participants