Rcutils: निर्दिष्ट निर्देशिका के आकार की गणना करें (उपनिर्देशिका शामिल करें)

को निर्मित 27 अक्तू॰ 2020 · 13टिप्पणियाँ · स्रोत: ros2/rcutils

rcutils_calculate_directory_size() केवल पुनरावर्ती के बिना निर्देशिका के आकार की गणना का समर्थन करता है।
क्यों rcutils_calculate_directory_size() पुनरावर्ती का समर्थन नहीं करता है?
क्या कोई कारण है?

स्रोत

Barry-Xu-2018

सभी 13 टिप्पणियाँ

मैं कुछ भी नहीं सोचता लेकिन मेरा मानना है कि यह अच्छा होगा यदि यह पुनरावर्ती रूप से काम करे।

rcutils_calculate_directory_size वर्तमान में rosbag2 मेटाडेटा की गणना के लिए उपयोग किया जाता है। निर्देशिका में सभी फाइलों के बाद से, इसे अभी के लिए पुनरावर्ती होने की आवश्यकता नहीं है।

fujitatomoya 27 अक्तू॰ 2020

👍1

rcutils_calculate_directory_size वर्तमान में rosbag2 मेटाडेटा की गणना के लिए उपयोग किया जाता है। निर्देशिका में सभी फाइलों के बाद से, इसे अभी के लिए पुनरावर्ती होने की आवश्यकता नहीं है।

हां। अब केवल metadata_io.cpp rosbag2 में इस फ़ंक्शन को कॉल करें।

Barry-Xu-2018 28 अक्तू॰ 2020

यह स्पष्ट रूप से वर्णन करता है कि यह पुनरावर्ती खोज का समर्थन नहीं करता है। क्या हमें वास्तविक उपयोग के मामलों के साथ पुनरावर्ती होने की आवश्यकता है?

fujitatomoya 28 अक्तू॰ 2020

यह स्पष्ट रूप से वर्णन करता है कि यह पुनरावर्ती खोज का समर्थन नहीं करता है। क्या हमें वास्तविक उपयोग के मामलों के साथ पुनरावर्ती होने की आवश्यकता है?

हां। नीचे के रूप में rosbag2 में वर्तमान कार्यान्वयन

https://github.com/ros2/rosbag2/blob/700817c42f1bbc26ced79aca2186094350c8d1be/rosbag2_storage/src/rosbag2_storage/metadata_io.cpp#L229 -L240

यह केवल निर्दिष्ट पथ में कुल फ़ाइल आकार की गणना करता है। sqlite3 डेटाबेस के लिए (अब rosbag2 इसे केवल स्टोरेज बैक-एंड के रूप में प्रदान करता है), यह ठीक है क्योंकि sqlite3 डेटाबेस एक फ़ाइल है। लेकिन दूसरों के लिए (जैसे कि leveldb), यह डेटा को बचाने के लिए एक निर्देशिका का उपयोग करता है। उपरोक्त कोड डेटाबेस का सही आकार नहीं प्राप्त कर सकते हैं।
मैं रिकर्सिव का समर्थन करने के लिए इस फ़ंक्शन को अपडेट करना चाहता हूं।
यह संशोधन वर्तमान कोड के लिए समस्या का परिचय नहीं देना चाहिए।

Barry-Xu-2018 28 अक्तू॰ 2020

👍1

मैं रिकर्सिव का समर्थन करने के लिए इस फ़ंक्शन को अपडेट करना चाहता हूं।
यह संशोधन वर्तमान कोड के लिए समस्या का परिचय नहीं देना चाहिए।

हाँ, मुझे ऐसा करने के लिए इसे बढ़ाने में कोई समस्या नहीं दिख रही है। यदि कॉलर वास्तव में ऐसा नहीं चाहता है, तो हम फ़ंक्शन में recursive बूलियन पैरामीटर पेश करना चाह सकते हैं, लेकिन अन्यथा यह मुझे अच्छा लगता है।

clalancette 28 अक्तू॰ 2020

👍1

@ क्लालैंसेट
सीसी: @ बैरी-जू-2018

टिप्पणी के लिए धन्यवाद!
वास्तव में हम leveldb को rosbag2 बैकएंड डेटाबेस के रूप में उपयोग करने का प्रयास कर रहे हैं। ऐसा करने के लिए यह वृद्धि आवश्यक होगी।

fujitatomoya 28 अक्तू॰ 2020

@clalancette @fujitatomoya

यदि कॉलर वास्तव में ऐसा नहीं चाहता है, तो हम फ़ंक्शन में एक पुनरावर्ती बूलियन पैरामीटर पेश करना चाह सकते हैं, लेकिन अन्यथा यह मुझे अच्छा लगता है।

यदि rcutils के अपडेट होने के बाद कोई नया पैरामीटर जोड़ा जाता है, तो हमें उसी समय rosbag2 को भी अपडेट करना चाहिए। अन्यथा, rosbag2 बिल्ड विफल रहा।

मैंने लिनक्स प्लेटफॉर्म पर परीक्षण किया है। लेकिन इसे विंडो प्लेटफॉर्म पर टेस्ट करने की जरूरत है।
अब मैं विंडो एनवायरनमेंट तैयार कर रहा हूं।

Barry-Xu-2018 29 अक्तू॰ 2020

यदि rcutils के अपडेट होने के बाद कोई नया पैरामीटर जोड़ा जाता है, तो हमें उसी समय rosbag2 को भी अपडेट करना चाहिए। अन्यथा, rosbag2 बिल्ड विफल रहा।

मैं भी इस दृष्टिकोण को पसंद नहीं करूंगा। हम इंटरफ़ेस रख सकते हैं और पुनरावर्ती खोज करने के लिए कार्यक्षमता बढ़ा सकते हैं। यदि पुनरावर्ती खोज के लिए कोई विशिष्ट उपयोग के मामले नहीं हैं, तो हम पुनरावर्ती मामले के लिए नया फ़ंक्शन जोड़ सकते हैं।

fujitatomoya 29 अक्तू॰ 2020

मैं भी इस दृष्टिकोण को पसंद नहीं करूंगा। हम इंटरफ़ेस रख सकते हैं और पुनरावर्ती खोज करने के लिए कार्यक्षमता बढ़ा सकते हैं। यदि पुनरावर्ती खोज के लिए कोई विशिष्ट उपयोग के मामले नहीं हैं, तो हम पुनरावर्ती मामले के लिए नया फ़ंक्शन जोड़ सकते हैं।

हां। सहमत।

@ क्लालैंसेट

हम नए पैरामीटर के बिना rcutils_calculate_directory_size को कैसे बढ़ाएंगे?
वर्तमान में, rosbag2 एन्हांस्ड rcutils_calculate_directory_size() के साथ सही ढंग से काम कर सकता है।

Barry-Xu-2018 29 अक्तू॰ 2020

हम नए पैरामीटर के बिना rcutils_calculate_directory_size को कैसे बढ़ाएंगे?
वर्तमान में, rosbag2 एन्हांस्ड rcutils_calculate_directory_size() के साथ सही ढंग से काम कर सकता है।

इसके बारे में चिंतित होने का कारण यह है कि rosbag2 निर्देशिका के अंदर "अतिरिक्त" जानकारी संग्रहीत करने के बारे में उपयोगकर्ताओं के बीच कुछ चर्चा है। यदि वे उपनिर्देशिकाओं में बहुत अधिक अतिरिक्त डेटा संग्रहीत करते हैं, तो वह उस आकार को फेंक सकता है जिसकी rosbag2 अपेक्षा कर रहा है।

अगर हम एपीआई को तोड़ना नहीं चाहते हैं, तो हम एक और फ़ंक्शन भी जोड़ सकते हैं जो rcutils_calculate_directory_size_recursive() है।

पिंगिंग @ Karsten1987 अंतर्दृष्टि के लिए कि क्या हमेशा rcutils_calculate_directory_size() में एक पुनरावर्ती खोज करने से rosbag2 के लिए समस्याएँ पैदा होंगी।

clalancette 29 अक्तू॰ 2020

👍1

@ क्लालैंसेट

मैं आपकी चिंताओं को समझता हूं।
मैंने कोड चेक किए हैं। Rosbag2 में केवल नीचे दिए गए कोड इस फ़ंक्शन का उपयोग करते हैं।

https://github.com/ros2/rosbag2/blob/700817c42f1bbc26ced79aca2186094350c8d1be/rosbag2_storage/src/rosbag2_storage/metadata_io.cpp#L229 -L240

कोड बदलने के बाद, मैं डिफ़ॉल्ट sqlite3 स्टोरेज प्लगइन के साथ rosbag2 को भी आजमाता हूं और समस्या नहीं ढूंढता।
बेशक, हमें @ Karsten1987 की राय सुननी चाहिए।

Barry-Xu-2018 29 अक्तू॰ 2020

👍1

@ बैरी-जू-2018

मुझे लगता है कि आप इसे बंद कर सकते हैं।

fujitatomoya 3 दिस॰ 2020

👍1

306 का विलय कर दिया गया। तो इस मुद्दे को बंद करें।

Barry-Xu-2018 3 दिस॰ 2020

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Rcutils: निर्दिष्ट निर्देशिका के आकार की गणना करें (उपनिर्देशिका शामिल करें)

सभी 13 टिप्पणियाँ

306 का विलय कर दिया गया। तो इस मुद्दे को बंद करें।

संबंधित मुद्दों