Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

119 improve learningtest scripts #123

Merged
merged 1 commit into from
Jan 12, 2024
Merged

119 improve learningtest scripts #123

merged 1 commit into from
Jan 12, 2024

Conversation

marcboulle
Copy link
Collaborator

Amelioration principales

  • fiabilisation de l'ordre des resultats en triant par nom de repertoire dans les ApplyCommand
  • capture effective des FATAL ERROR dans les errors
  • amelioration du script de comparaison pour detecter qu'il n'y des differences que dans le fichier err.txt et dans les rapport json, en meme nombre
  • supprimer les message "The test has not been launched" si test non lance
  • applyCommandAll conditionne par la variable d'environnement KhiopsCompleteTests pour ne la lancer que les les repertoire necessaires

Reporting de applyCommandAll errors sous forme de fichier csv:

  • Status: DONE ou vide
  • Tool: TestKhiops|TestCoclustering|TestKNITransfer
  • Family: famille de test
  • Test: jeu de test
  • Warning: warnings ou vide
  • Warning number
  • Errors: errors ou vide
  • Errors: nombre d'erreurs si > 0
  • FATAL ERROR: FATAL ERROR ou vide
  • Files with errors: liste des extension de fichiers comportant des erreurs (plus err.txt de facon specifique)
  • Note: info complementaires (ex: 'errors only in err.txt')

@marcboulle
Improve LearningTest scripts
2294a00 
Amelioration principales
- fiabilisation de l'ordre des resultats en triant par nom de repertoire dans les ApplyCommand
- capture effective des FATAL ERROR dans les errors
- amelioration du script de comparaison pour detecter qu'il n'y des differences que dans le fichier err.txt et dans les rapport json, en meme nombre
- supprimer les message "The test has not been launched" si test non lance
- applyCommandAll conditionne par la variable d'environnement KhiopsCompleteTests pour ne la lancer que les les repertoire necessaires

Reporting de applyCommandAll errors sous forme de fichier csv:
- Status: DONE ou vide
- Tool: TestKhiops|TestCoclustering|TestKNITransfer
- Family: famille de test
- Test: jeu de test
- Warning: warnings ou vide
- Warning number
- Errors: errors ou vide
- Errors: nombre d'erreurs si > 0
- FATAL ERROR: FATAL ERROR ou vide
- Files with errors: liste des extension de fichiers comportant des erreurs (plus err.txt de facon specifique)
- Note: info complementaires (ex: 'errors only in err.txt')
@marcboulle marcboulle added the Type/Enhancement New feature or request label Jan 12, 2024
@marcboulle marcboulle linked an issue Jan 12, 2024 that may be closed by this pull request
Improve LearningTest scripts #119
Closed
@folmos-at-orange folmos-at-orange changed the base branch from dev to release-10-2-0 January 12, 2024 12:52
@marcboulle marcboulle merged commit cfbf4e1 into release-10-2-0 Jan 12, 2024
26 checks passed
@marcboulle marcboulle deleted the 119-improve-learningtest-scripts branch January 12, 2024 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@folmos-at-orange folmos-at-orange folmos-at-orange approved these changes

Assignees
No one assigned
Labels
Type/Enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Improve LearningTest scripts
2 participants
@marcboulle @folmos-at-orange