Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

change duration compute methods #49

Merged
merged 8 commits into from
Jan 19, 2018
Merged

change duration compute methods #49

Changes from 2 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
26f5b2d
change duration compute methods
Jan 11, 2018
78cce58
modify time calculation method and print mean loss,acc of every pass
Jan 15, 2018
3713d12
modify time calculation method and print mean loss,acc of every pass
Jan 15, 2018
8d5b1aa
delete redundant line and change judge iter
Jan 15, 2018
6e9b36b
Merge branch 'master' of github.com:dzhwinter/benchmark
Jan 15, 2018
0157ed8
Merge branch 'master' of github.com:dzhwinter/benchmark
Jan 17, 2018
c838e11
modify GPUPlace to CUDAPlace
Jan 17, 2018
ba60b28
move line 173 to line 212
Jan 17, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 2 additions & 5 deletions fluid/resnet50.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ def run_benchmark(model, args):
buf_size=5120),
batch_size=args.batch_size)

place = core.CPUPlace() if args.device == 'CPU' else core.CUDAPlace(0)
place = core.CPUPlace() if args.device == 'CPU' else core.GPUPlace(0)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I remember CUDAPlace is used in the latest PaddlePaddle. Could you refer to the source codes to confirm this.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

exe = fluid.Executor(place)
exe.run(fluid.default_startup_program())

Expand All @@ -209,15 +209,12 @@ def run_benchmark(model, args):
label = np.array(map(lambda x: x[1], data)).astype('int64')
label = label.reshape([-1, 1])

iter = 0
im_num = 0
start_time = time.time()
for pass_id in range(args.pass_num):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://github.com/ccmteorljh/benchmark/blob/6e9b36b2cd318f308b80c984617f79f6eccdce5e/fluid/resnet50.py#L170 Maybe moving L170 before L213 is better.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

every_pass_loss = []
every_pass_acc = []
accuracy.reset(exe)
if iter == args.iterations:
iter = 0
iter = 0
for batch_id, data in enumerate(train_reader()):
if iter < args.skip_batch_num:
iter += 1
Expand Down