Skip to content

Tests fix - #1

Open
domupls wants to merge 5 commits into
mainfrom
tests-fix
Open

Tests fix#1
domupls wants to merge 5 commits into
mainfrom
tests-fix

Conversation

@domupls

@domupls domupls commented Feb 12, 2025

Copy link
Copy Markdown
Collaborator

Ahoj, psala jsem email, ale nevím, jestli dorazil, tak zkusím ještě takhle pingnout přes pull-request.
Testy jsou opravené a runtests.jl jsem přidala. Zároveň jsem překopala projekt, aby více využíval multiple dispatch.

@domupls
domupls requested a review from masenka31 February 12, 2025 08:45
@masenka31

Copy link
Copy Markdown

Review in process :D

@masenka31

Copy link
Copy Markdown

Ahoj, projekt stale nesplnuje nutne pozadavky:

  1. Testy nefunguji: Je potreba udelat spravny soubor runtests.jl (chybne jmeno souboru). Navic chybi Project.toml, takze testy nejde spustit.

Example usage a output, ktery dostanu:

~/Project_sidlodo1/ > julia

julia > ]

(@v1.10) pkg> activate .
    Activating project at `~/Project_sidlodo1`

(SurvivalPrediction) pkg> test
    ERROR: Package SurvivalPrediction did not provide a `test/runtests.jl` file
  1. examples slozka obsahuje vlastni Project.toml a v examples se pouziva balicek pomoci using SurvivalPrediction (zadne include!).

  2. Spatne pouzivani modules. Cela pointa multiple dispatch je v tom, ze pak pouzivam jednu funkci a jen menim jeji parametry a julia vi, kterou metodu pouzit. Pokud musim specifikovat modul, je to spatne. V commitu je, jak by mel vypadat funkcni example.

@domupls

domupls commented Feb 13, 2025

Copy link
Copy Markdown
Collaborator Author

@masenka31 Snad už to teď bude fungovat 😅 s tím importováním modulů jsem hodně strugglovala, nějak mi nedošlo, že vlastně mohu exportovat ty fce z importovaných modulů

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.

2 participants