Skip to content

Refactor EXPERIMENTS.md for PyPi release (fixes #23)#24

Closed
TasfinMahmud wants to merge 1 commit into
mainfrom
fix-issue-23-experiments-log
Closed

Refactor EXPERIMENTS.md for PyPi release (fixes #23)#24
TasfinMahmud wants to merge 1 commit into
mainfrom
fix-issue-23-experiments-log

Conversation

@TasfinMahmud

Copy link
Copy Markdown
Collaborator

Fixes #23

  • Consolidated 20-seed benchmark aggregates for CWRU, PU, and XJTU-SY
  • Added DoWhy causal ATE tables for all datasets
  • Reorganized entries sequentially (1-5) and removed all duplicate sections
  • Explicitly labeled SEU gearbox as FAILED with limitations documented
  • Removed all TBD placeholders; all metrics now reflect final validated runs

- Consolidated 20-seed benchmark aggregates for CWRU, PU, and XJTU-SY
- Added DoWhy causal ATE tables for all datasets
- Reorganized entries sequentially (1-5) and removed all duplicate sections
- Explicitly labeled SEU gearbox as FAILED with limitations documented
- Removed all TBD placeholders; all metrics now reflect final validated runs
@abhiprd2000

Copy link
Copy Markdown
Member

@TasfinMahmud , Check fad67d7 , I think it's cleaner than yours, tell your opinion. I was also thinking to add and .md file for future works and limitations so that we can attract open source contributors. what's your opinion on both?

@TasfinMahmud

Copy link
Copy Markdown
Collaborator Author

@abhiprd2000
--> checked fad67d7. Yeah it's cleaner, let's go with it.
--> Regarding moving SEU/limitations to a separate .md is also a good idea.

@abhiprd2000

Copy link
Copy Markdown
Member

@TasfinMahmud , Please come on whatsapp, need a talk!

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.

Update EXPERIMENTS.md gracefully.

2 participants