-
Notifications
You must be signed in to change notification settings - Fork 4
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
feature suggest: include an example which eventually runs by fpm run
#1
Comments
Thanks @nbehrnd. I have updated the README and fpm.toml files. There was a mistake in the README. It must be "prefix" not "perfix." As you mentioned, forimage is a library, not a program. Therefore, I didn't create a main program. However, the examples are in the example directory and can be run using fpm as follows:
Or using response files for specific compilers:
Please note that these changes haven't been merged with the main branch yet. They are in the branch: https://github.com/gha3mi/forimage/tree/gha3mi/issue1. Could you please review these updates? |
@gha3mi I just run the instructions on an instance Linux Debian 13/trixie for an explicit checkout of the branch addressing the issue. fpm seems to have lesser problems. As for running the examples: it is bit difficult to recognize if they were changed. I assumed the time stamps by I perceive this as a small change only; leaning on the side of «too small». (And if request |
Thanks, @nbehrnd, for testing the examples. I'm glad to see that the code can successfully generate, read (import) and write (export) pnm files. The examples demonstrate how to set up a pnm file, save it, import it, and save it again. For instance, in example 3, you can set some data for a ppm file using the following procedure: call ex3%set_pnm(encoding='plain', file_format='ppm', width=4, height=4, max_color=15, comment='example 2', pixels=px) Then, the code saves it to a file using the export procedure: call ex3%export_pnm('pnm_files/example3') Afterward, it is possible to import the file back into the program: call ex3%import_pnm('pnm_files/example3', 'ppm', 'plain') At this point, some processing can be done, although it's not provided in these examples. Finally, after any possible processing, the code exports it again using: call ex3%export_pnm('pnm_files/example3_ex')
call compare_pnm('pnm_files/example3', 'pnm_files/example3_ex', 'ppm', 'plain') In this case, the
|
By now, the intended working is better understood (cf. log of a pristine session). Test/check files: a topic for a bit later. |
Based on the information the landing page provides, I attempted to install
forimage
. By the information collected so far (see log attached below), the fpm-based installation did not include the section[dependencies]
yet. Is--perfix
a flag added just recently to the fpm?Second -- partly because the ford-based documentation still is under construction -- I would like to suggest to extend the set up. While
forimage
aims to be a library to contribute in other projects, can an fpm-based installation be modified in a way that a call byfpm run
executes a small test application? This is inspired by the «first steps» appetizing section of the fpm tutorial here.2023-07-19_forimage.log
The text was updated successfully, but these errors were encountered: