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.

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

Sauvegarder une PGM #2

Closed
Hagbuck opened this issue Oct 13, 2017 · 7 comments
Closed

Sauvegarder une PGM #2

Hagbuck opened this issue Oct 13, 2017 · 7 comments
Assignees
Labels
Milestone

Comments

@Hagbuck
Copy link
Owner

Hagbuck commented Oct 13, 2017

  • Transformer une structure PGM en chaîne de caractères
  • Enregistrer la chaîne dans un fichier .PGM
@Hagbuck Hagbuck added the To Do label Oct 13, 2017
@Hagbuck Hagbuck added this to the PGM milestone Oct 13, 2017
@Hagbuck Hagbuck self-assigned this Oct 13, 2017
@Hagbuck Hagbuck added Done and removed To Do labels Oct 13, 2017
@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 13, 2017

Fonction de sauvegarde codé.

e__bool PGM_save_image_into_file(PGM_image* pgm, FILE* file); convertie un PGM_image en chaîne
https://github.com/Hagbuck de caractères et sauvegarde la chaîne dans un fichier

@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 19, 2017

Comme l'issue #1 il faut gérer la limite de 70 caractères par ligne.

@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 19, 2017

La sauvegarde d'un PGM prend trop de temps avec la méthode actuelle (1min30 pour 2 images de 512*512).

Par contre la limite de 70 caractères par ligne n'a pas l'air d'avoir d'importance pour la lecture de l'image par la suite.

Donc à coder mais ce n'est plus prioritaire vue que cela fonctionne sans.

@Hagbuck Hagbuck removed the PRIO label Oct 19, 2017
@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 19, 2017

Ce qui est long est la boucle for qui transforme la structure en chaîne de caractères.

for(...)
{
    sprintf(...);
    strcat(...);
}

L'écriture est extrêmement rapide à coté....

@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 19, 2017

En retirant sprintf et en mettant la valeur {'0', '\0'} dans le chaîne de caractère. On tombe à 20 seconde pour les 2 images

Donc il faut trouver le moyen d'optimiser le sprintf

mais également le strcat qui est très / trop long

@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 19, 2017

Avec suppression de strcat en écrivant directement dans le buffer qui contient la chaîne total on atteint 130ms pour les 2 images !

@Hagbuck Hagbuck added Done and removed To Do labels Oct 19, 2017
@Hagbuck
Copy link
Owner Author

Hagbuck commented Oct 19, 2017

L'enregistrement des images avec 70 caractères ou plus ne pose pas de problème à la lecture du PGM. Donc on ByPassed cette règle pour le moment.... À reprendre dans le futur --> #17

@Hagbuck Hagbuck closed this as completed Oct 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant