You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2022/03/15 14:01:37 UTC

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

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



##########
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:
       The experimental results and analysis have been put on Apache confluence https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=199536917




-- 
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