質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
MacOS(OSX)

MacOSとは、Appleの開発していたGUI(グラフィカルユーザーインターフェース)を採用したオペレーションシステム(OS)です。Macintoshと共に、市場に出てGUIの普及に大きく貢献しました。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

1回答

870閲覧

平均処理時間の計算のエラー

kaeruuuun

総合スコア19

MacOS(OSX)

MacOSとは、Appleの開発していたGUI(グラフィカルユーザーインターフェース)を採用したオペレーションシステム(OS)です。Macintoshと共に、市場に出てGUIの普及に大きく貢献しました。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

0グッド

0クリップ

投稿2021/06/01 01:54

編集2021/06/01 02:32

myBytesにはnで指定した長さの文字列が5000から5000刻みで100000まで入っています.
私が実現したいのはn=5000の時の10回平均処理時間を求めて出力,次にn=10000の時の10回平均処理時間を求めて出力という風にしたいので,結果としてprintは19回されなければいけないのに,数は数えられないですが,大量にprintが出力されました.どこが問題になっているか教えていただきたいです.

python

1import time 2import numpy as np 3 4NA ='ACGT' 5n = 100000 6s = '' 7N=10 8myBytes = {} 9for n in range(5000,100000,5000): 10 myBytes[n] = (s.join([random.choice(DNA) for i in range(n)])).encode() 11 for j in range(N): 12 st1_time = time.time() 13 xR = makeSuffixArrayByInducedSorting(myBytes[n],256,0,False) 14 et1_time = time.time() 15 lap1_time= et1_time-st1_time 16 np.mean(lap1_time) 17 st2_time = time.time() 18 nR = naivelyBuildSA(myBytes[n]) 19 et2_time = time.time() 20 lap2_time = et2_time-st2_time 21 np.mean(lap2_time) 22 print("processing time:{} {} {}".format(np.mean(lap1_time),np.mean(lap2_time), xR==nR))

processing time:0.02840709686279297 0.023591995239257812 True
processing time:0.026887893676757812 0.010459184646606445 True
processing time:0.03901791572570801 0.012247085571289062 True
processing time:0.47066211700439453 3.35798978805542 True
とても早いスピードでこのような結果が出力されます.
ただ,printをfor nのインデントに揃えると
processing time:0.02840709686279297 0.023591995239257812 True
一行しか結果が表示されません.

いろいろ考えてみたのですがインデントをmyBytesに揃えるとn=5000の時の10回の平均が出力されてn=10000の時の10回平均が出力されたのですが,
n=100000単体で出力した時のlap2_timeの値が大体12秒なのに対してこのプログラムを実行して求めるとlap2_timeの値が4秒程度になります.これはプログラムとして間違っているのでしょうか.
結果から見るとlap1_timeは単体での結果とほぼ一致していますが,lap2_timeがずれています.
それともlap2_timeに関してもlap1_timeと同じで1から10まで行いたいので,forを一つにしているのですが,別で回す必要がありますか?

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

printのインデント位置をfor j in range(N):と揃えるのが必要なのではないでしょうか。


現状では、書いたとおりにprintfor j in range(N):のループ内で実行されています。

ただ,printをfor nのインデントに揃えると

こちらも書いたとおりに、ループを抜けきったあとの1度だけ実行されます。

投稿2021/06/01 02:05

maisumakun

総合スコア146018

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

kaeruuuun

2021/06/01 02:15

実現はできたのですが,これだとn=100000単体で実行した結果と繰り返し求めた平均値の差が大きいです.計算として間違っているのでしょうか.
kaeruuuun

2021/06/01 02:16

単体だと12秒程度と出力されて,このプログラムを使うと4秒ぐらいの結果が出ます.
maisumakun

2021/06/01 02:35

とりあえず「大量にprintが出力されました」という問題への対応だけの回答です。計算の中身は見ていないです。
kaeruuuun

2021/06/01 02:38

わかりました.ありがとうございます.
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問