Module Infrastructure - Githubissues

Foggalong commented 1 month ago

I've written the initial build file and checked that deployment works as intended, the rest just needs neatening up into a more formal package.

[x] Add build file
[x] Switched out in-repo examples for test scripts
[x] Add on-push workflow lints and tests
[x] Update readme with install instructions
[x] Write proper documentation for file formats
[x] Fix inconsistency with $\mathbf{w}$ vs $w$ in docs
[x] Tag an initial alpha release :eyes:

Foggalong commented 1 month ago

Fixed in 0c47d36 and v0.0.1

Foggalong commented 1 month ago

@sakurita17 You'll be pleased to hear this can be installed normally as a module normally now! No more import sys; sys.path.insert(1, '../../') hacks needed 😅

sakurita17 commented 1 month ago

Thanks Josh,

From: Josh Fogg @.> Sent: Monday, 22 July 2024 09:45 To: Foggalong/RobustOCS @.> Cc: Jaime Ortiz @.>; Mention @.> Subject: Re: [Foggalong/RobustOCS] Module Infrastructure (Issue #19)

@sakurita17https://github.com/sakurita17 You'll be pleased to hear this can be installed normally as a module normally now! No more import sys; sys.path.insert(1, '../../') hacks needed 😅

— Reply to this email directly, view it on GitHubhttps://github.com/Foggalong/RobustOCS/issues/19#issuecomment-2243141236, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS36HVKFEAJA2UCBUC4IYNLZNULHRAVCNFSM6AAAAABLDEKEPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBTGE2DCMRTGY. You are receiving this because you were mentioned.Message ID: @.***>

sakurita17 commented 1 month ago

Hey Josh, I'm encountering an issue while trying to load the .txt files. I've already imported the necessary module, but I still get this error. Any idea what might be causing this problem? [cid:710c8a9f-ce50-4f91-ae68-277d39811669] Thanks for your help! 🙂 Jaime

From: Josh Fogg @.> Sent: Monday, 22 July 2024 09:45 To: Foggalong/RobustOCS @.> Cc: Jaime Ortiz @.>; Mention @.> Subject: Re: [Foggalong/RobustOCS] Module Infrastructure (Issue #19)

@sakurita17https://github.com/sakurita17 You'll be pleased to hear this can be installed normally as a module normally now! No more import sys; sys.path.insert(1, '../../') hacks needed 😅

— Reply to this email directly, view it on GitHubhttps://github.com/Foggalong/RobustOCS/issues/19#issuecomment-2243141236, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS36HVKFEAJA2UCBUC4IYNLZNULHRAVCNFSM6AAAAABLDEKEPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBTGE2DCMRTGY. You are receiving this because you were mentioned.Message ID: @.***>

Foggalong commented 1 month ago

Hey @sakurita17, the error message you sent didn't come through to GitHub. Would you be able to post it there directly?

Foggalong / RobustOCS

Module Infrastructure #19