[ENH] Coverage for the networks module

[TonyBagnall]

Describe the feature or idea you want to propose

networks has a lot of parameters that are not covered in testing, it would be good if they were esp the ae ones @aadya940

Describe your proposed solution

add tests for network parameters

Describe alternatives you've considered, if relevant

No response

Additional context

No response

[Nikitas100]

Hello, I would like to contribute by adding tests for the network parameters. I found three test files related to ae in the networks module. Are these the parameters that require additional coverage, or are there other specific areas I should focus on? Any further details or guidance would be appreciated.

[MatthewMiddlehurst]

While this does not cover every single case and item we want tested, an east place to start is our test coverage https://app.codecov.io/gh/aeon-toolkit/aeon

[Nikitas100]

Hi @MatthewMiddlehurst ,

I noticed that the tests don't fully cover 10-11 files in the networks module. Would you prefer I start with specific files or parameters, or should I gradually work on covering all uncovered areas?

Thanks!

[aadya940]

@Nikitas100 you can start with any file, typically divide the tasks to 1 network per pull request. :)

[lucifer4073]

Hi @Nikitas100,

There seems to be a decent number of files which need a higher test coverage. It might be a good idea to collaborate on addressing this issue together.

[lucifer4073]

Hi @Nikitas100 ,

If you are currently not working on this issue, I will be proceeding with the rest of the files.

[kavya-r30]

@lucifer4073 I would like to collaborate together on addressing this issue.

[Cyril-Meyer]

Duplicated with #1233 ?

[MatthewMiddlehurst]

Good catch, I have closed that one as this seems more recently active. @hadifawaz1999 is it fine to work on the clustering ones now?

[hadifawaz1999]

@MatthewMiddlehurst this morning I reviewed all the PRs for the test coverage stuff and asked for some changes, let me know what you think because other than my requests it's all good, i prefer having one test file for each network as its becoming less ans less common parameters, so once we have covered everything we can remove the common one that does a lot of if and else

[MatthewMiddlehurst]

Is this complete? Can still open PRs at least.