[GitHub] [iotdb] HTHou commented on a change in pull request #5118: [IOTDB-2609] A new lossy encoding method based on frequency domain

[GitBox <gi...@apache.org>]

HTHou commented on a change in pull request #5118:
URL: https://github.com/apache/iotdb/pull/5118#discussion_r824362777



##########
File path: docs/UserGuide/Data-Concept/Encoding.md
##########
@@ -54,6 +54,12 @@ Usage restrictions: When using GORILLA to encode INT32 data, you need to ensure
 
 DICTIONARY encoding is lossless. It is suitable for TEXT data with low cardinality (i.e. low number of distinct values). It is not recommended to use it for high-cardinality data. 
 
+* FREQ
+
+FREQ encoding is lossy. It transforms the time sequence to the frequency domain and only reserve part of the frequency components with high energy. It is more suitable for sequence with obvious periodicity.
+
+> There are two parameters of FREQ encoding in the configuration file: `freq_snr` defines the signal-noise-ratio (SNR), `freq_block_size` defines the data size in a time-frequency transformation.

Review comment:
       How do these two parameters affect the encoding performance and compression ratio?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org