search

dilevin / computer-graphics-raster-images

Computer Graphics Assignment about Raster Images
1 stars 7 forks source link

PPM Specification #15

Open LesterLyu opened 4 years ago

LesterLyu commented 4 years ago

Two questions here:

The PPM specification says:

Should we follow this? Since longer line can still be opened.

Also the P2 type seems have the file extension .PGM instead of .PPM. What we should do here?

dilevin commented 4 years ago

  1. The link to the ppm format in the README.md states that the 70 character line length is a recommendation but doesn't need to be followed.

  2. The main.cpp file will send the filename to your write_ppm method so you don't have to worry about it. The code will output the grayscale image as gray.ppm.

LesterLyu commented 4 years ago

Should I close this issue?