Python-future: विभाजन का old_div . में गलत रूपांतरण

को निर्मित 30 जन॰ 2019 · 12टिप्पणियाँ · स्रोत: PythonCharmers/python-future

उदाहरण कोड

निम्नलिखित कोड में एक पुरानी शैली का विभाजन है।

(x / 2 * 3.0)

अपेक्षित परिणाम

उपरोक्त कोड को x को असाइन किए गए मनमाने मानों के साथ चलाने का परिणाम निम्नलिखित होना चाहिए, जहां कोष्ठक स्पष्ट रूप से संचालन का क्रम दिखाते हैं (x / 2) * 3.0 । या किसी अन्य क्रम में लिखे गए परिणाम का मूल्यांकन x * (3.0 / 2) रूप में किया जाना चाहिए।

समस्या

एक बार जब हम उपरोक्त कोड पर futurize चलाते हैं, तो fix_division_safe फिक्स्चर द्वारा एक old_div डाला जाता है। हालाँकि जैसा कि नीचे दिए गए अंतर में देखा जा सकता है, फ़ंक्शन को तर्कों के गलत क्रम के साथ बुलाया जाता है।

$ futurize --stage2 src/example.py
RefactoringTool: Refactored src/example.py
--- src/example.py      (original)
+++ src/example.py      (refactored)
@@ -1 +1,3 @@
-(x / 2 * 3.0)
+from __future__ import division
+from past.utils import old_div
+(old_div(x, 2 * 3.0))
RefactoringTool: Files that need to be modified:
RefactoringTool: src/example.py

अपेक्षित रूपांतरण

जैसा कि 'अपेक्षित परिणाम' खंड में पहले ही कहा गया है, सही रूपांतरण होना चाहिए था: (old_div(x, 2) * 3.0)

0.18 bug

स्रोत

wagnerpeer

👍1

सबसे उपयोगी टिप्पणी

मुझे यह समस्या भी है, पायथन 2.7.16 पर फ्यूचराइज़ 0.17.1 चल रहा है। यह उदाहरण समस्या को दिखाता है:

$ echo 'x = 100/10 * 10' | futurize --stage2 -

RefactoringTool: Refactored <stdin>
--- <stdin> (original)
+++ <stdin> (refactored)
@@ -1 +1,3 @@
-x = 100/10 * 10
+from __future__ import division
+from past.utils import old_div
+x = old_div(100,10 * 10)
RefactoringTool: Files that need to be modified:
RefactoringTool: <stdin>

ध्यान दें कि 100/10 * 10 (==100) old_div(100,10 * 10) (==1) में परिवर्तित हो जाता है। यह मेरे लिए एक बड़ी बग की तरह लगता है।

mfripp 19 जून 2019

👍3

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

या किसी अन्य क्रम में लिखे गए परिणाम का मूल्यांकन x * (3.0 / 2) रूप में किया जाना चाहिए।

जबकि गणितीय रूप से समकक्ष, यह पायथन 2 पर बराबर नहीं है। यदि x एक int , तो x / 2 कुछ मामलों में पहले छोटा हो जाएगा, जबकि (3.0 / 2) होगा हमेशा 1.5 रहें और कभी भी छोटा न करें।

हालांकि निश्चित रूप से, यह बग नहीं बदलता है।

QuLogic 31 जन॰ 2019

मैंने भी ऐसा ही सोचा था, यही वजह है कि मैंने उदाहरण चलाने के लिए पायथन 2.7.15 के साथ-साथ पायथन 3.7.1 का उपयोग किया और पुष्टि की कि वे समान व्यवहार दिखाते हैं। यह सुनकर अच्छा लगा कि मेरा पहला इरादा पूरी तरह गलत नहीं था।

शायद यह ओएस से भी जुड़ा है? मैं विंडोज 7 x64 पर उदाहरण चला रहा हूं।

क्या आप एक उदाहरण दे सकते हैं कि किन मामलों में पायथन 2 दूसरे तरीके से अभिव्यक्ति का मूल्यांकन करेगा?

wagnerpeer 31 जन॰ 2019

>>> y = range(10)

>>> [x / 2 * 3.0 for x in y]
[0.0, 0.0, 3.0, 3.0, 6.0, 6.0, 9.0, 9.0, 12.0, 12.0]

>>> [x * (3.0 / 2) for x in y]
[0.0, 1.5, 3.0, 4.5, 6.0, 7.5, 9.0, 10.5, 12.0, 13.5]

(आपका अपेक्षित रूपांतरण निश्चित रूप से सही परिणाम है।)

QuLogic 31 जन॰ 2019

पूरी तरह से मैं क्या उम्मीद करता। हालांकि मैंने एक्स के लिए 1 का उपयोग करके स्पष्ट रूप से परीक्षण किया और परिणामस्वरूप 1.5 मिला।

मैं यह भी उल्लेख करना भूल गया कि यदि आप कोष्ठक का उपयोग नहीं करते हैं तो कोड का एक अलग आउटपुट होता है। इस मामले में फ्यूचराइज सही रूपांतरण देगा।

1 / 2 * 3.0 - -> old_div(1, 2) * 3.0

wagnerpeer 31 जन॰ 2019

मुझे उसी बग का सामना करना पड़ा और मैं अपनी शाखा पर इसे ठीक करने पर काम कर रहा
ऐसा लगता है कि मेरे परिवर्तन इस समस्या को इस प्रकार ठीक करते हैं:

$ cat sample.py          
x =  10
(x / 2 * 3.0)

मूल आउटपुट

$ futurize sample.py 
RefactoringTool: Skipping optional fixer: idioms
RefactoringTool: Skipping optional fixer: ws_comma
RefactoringTool: Refactored sample.py
--- sample.py   (original)
+++ sample.py   (refactored)
@@ -1,2 +1,4 @@
+from __future__ import division
+from past.utils import old_div
 x =  10
-(x / 2 * 3.0)
+(old_div(x, 2 * 3.0))
RefactoringTool: Files that need to be modified:
RefactoringTool: sample.py

नया आउटपुट

$ futurize sample.py
RefactoringTool: Skipping optional fixer: idioms
RefactoringTool: Skipping optional fixer: ws_comma
RefactoringTool: Refactored sample.py
--- sample.py   (original)
+++ sample.py   (refactored)
@@ -1,2 +1,4 @@
+from __future__ import division
+from past.utils import old_div
 x =  10
-(x / 2 * 3.0)
+(old_div(x, 2) * 3.0)
RefactoringTool: Files that need to be modified:
RefactoringTool: sample.py

जब मैं समय देखूंगा तो मैं एक पुल अनुरोध बनाने जा रहा हूं ..

fbessho 17 मार्च 2019

यहाँ और उदाहरण हैं।

मैं यह भी उल्लेख करना भूल गया कि यदि आप कोष्ठक का उपयोग नहीं करते हैं तो कोड का एक अलग आउटपुट होता है। इस मामले में फ्यूचराइज सही रूपांतरण देगा।
1 / 2 * 3.0 - -> old_div(1, 2) * 3.0

आपने जो कहा उसके विपरीत, मैं देख रहा हूं कि 1 / 2 * 3.0 को old_div(1, 2 * 3.0) गलत तरीके से मास्टर पर वर्तमान कोड का उपयोग करके परिवर्तित किया गया है।

$ cat sample2.py
1 / 2 * 3.0
x =  10
x / 2
x / 2.0
x / 2 * 3
x / (2 * 3)
x * 2 / 3

मूल आउटपुट

$ futurize sample2.py
RefactoringTool: Skipping optional fixer: idioms
RefactoringTool: Skipping optional fixer: ws_comma
RefactoringTool: Refactored sample2.py
--- sample2.py  (original)
+++ sample2.py  (refactored)
@@ -1,7 +1,9 @@
-1 / 2 * 3.0
+from __future__ import division
+from past.utils import old_div
+old_div(1, 2 * 3.0)
 x =  10
-x / 2
+old_div(x, 2)
 x / 2.0
-x / 2 * 3
-x / (2 * 3)
-x * 2 / 3
+old_div(x, 2 * 3)
+old_div(x, (2 * 3))
+old_div(x * 2, 3)
RefactoringTool: Files that need to be modified:
RefactoringTool: sample2.py

नया आउटपुट

$ futurize sample2.py
RefactoringTool: Skipping optional fixer: idioms
RefactoringTool: Skipping optional fixer: ws_comma
RefactoringTool: Refactored sample2.py
--- sample2.py  (original)
+++ sample2.py  (refactored)
@@ -1,7 +1,9 @@
-1 / 2 * 3.0
+from __future__ import division
+from past.utils import old_div
+old_div(1, 2) * 3.0
 x =  10
-x / 2
+old_div(x, 2)
 x / 2.0
-x / 2 * 3
-x / (2 * 3)
-x * 2 / 3
+old_div(x, 2) * 3
+old_div(x, (2 * 3))
+old_div(x * 2, 3)
RefactoringTool: Files that need to be modified:
RefactoringTool: sample2.py

fbessho 17 मार्च 2019

PR created (https://github.com/PythonCharmers/python-future/pull/441)

fbessho 17 मार्च 2019

मैंने यह भी देखा है कि old_div तब भी डाला जाता है जब बाएँ और दाएँ दोनों ऑपरेंड तैरते हैं जैसे

1.0/1024.0

हो जाता है

old_div(1.0,1024.0)

मुझे लगता है कि यह रूपांतरण अनावश्यक है क्योंकि विभाजन जहां दोनों ऑपरेंड तैरते हैं, वही परिणाम उत्पन्न करते हैं, दोनों पायथन 2 और पायथन 3 में।

mattgathu 27 मार्च 2019

@mattgathu आप फ्यूचराइज़ और पायथन के किस संस्करण का उपयोग करते हैं? Python 3.7.2 और फ्यूचराइज़ 0.17.1 (PyPI में नवीनतम) के साथ, 1.0/1024.0 बरकरार रहता है जब futurize d.

$ python --version
Python 3.7.2
$ futurize --version
0.17.1

$ cat test.py
1.0 / 1024.0
1   / 1024.0
1.0 / 1024

$ futurize test.py
RefactoringTool: Skipping optional fixer: idioms
RefactoringTool: Skipping optional fixer: ws_comma
RefactoringTool: No files need to be modified.

fbessho 27 मार्च 2019

यहाँ मेरे संस्करण हैं:

$ python -V
Python 2.7.16
$ futurize --version
0.16.0

मैंने 0.17.0 में अपग्रेड किया है और ऐसा लगता है कि यह समस्या नहीं है। धन्यवाद

mattgathu 27 मार्च 2019

👍1

वास्तव में ऐसा लगता है कि v0.17.0 में अनावश्यक रूपांतरणों को छोड़ने के लिए एन्हांसमेंट शामिल है।

https://github.com/PythonCharmers/python-future/releases/tag/v0.17.0

जटिल भावों के बेहतर रूपांतरण का समर्थन करने के लिए fix_division_safe को ठीक करें, और स्पष्ट फ्लोट डिवीजन को छोड़ दें।

fbessho 27 मार्च 2019

👍1

$ echo 'x = 100/10 * 10' | futurize --stage2 -

RefactoringTool: Refactored <stdin>
--- <stdin> (original)
+++ <stdin> (refactored)
@@ -1 +1,3 @@
-x = 100/10 * 10
+from __future__ import division
+from past.utils import old_div
+x = old_div(100,10 * 10)
RefactoringTool: Files that need to be modified:
RefactoringTool: <stdin>

mfripp 19 जून 2019

👍3

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

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

Python-future: विभाजन का old_div . में गलत रूपांतरण

उदाहरण कोड

अपेक्षित परिणाम

समस्या

अपेक्षित रूपांतरण

सबसे उपयोगी टिप्पणी

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

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