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.

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

Python3 uploading issue with bytes/str #39

Closed
anntzer opened this issue Nov 3, 2015 · 1 comment
Closed

Python3 uploading issue with bytes/str #39

anntzer opened this issue Nov 3, 2015 · 1 comment

Comments

@anntzer
Copy link

anntzer commented Nov 3, 2015

$ echo 'for _ in range(int(1e8)): pass' > foo.py && python3 -mvmprof --web foo.py
<elided WARNINGS regarding unreadable symbols>
/home/antony/.local/lib/python3.5/site-packages/vmprof/addrspace.py:170: UserWarning: there are 136 broken profiles out of 1091, data will be unreliable
  warnings.warn("there are %d broken profiles out of %d, data will be unreliable" % (skipped, len(filtered_profiles)))
Compiling and uploading to http://vmprof.com...
Traceback (most recent call last):
  File "/usr/lib/python3.5/runpy.py", line 170, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/antony/.local/lib/python3.5/site-packages/vmprof/__main__.py", line 67, in <module>
    main()
  File "/home/antony/.local/lib/python3.5/site-packages/vmprof/__main__.py", line 64, in main
    show_stats(prof_file.name, output_mode, args)
  File "/home/antony/.local/lib/python3.5/site-packages/vmprof/__main__.py", line 25, in show_stats
    upload_stats(stats, args)
  File "/home/antony/.local/lib/python3.5/site-packages/vmprof/__main__.py", line 36, in upload_stats
    res = vmprof.upload.upload(stats, name, argv, host, auth)
  File "/home/antony/.local/lib/python3.5/site-packages/vmprof/upload.py", line 35, in upload
    return "http://" + host + "/#/" + val[1:-1]
TypeError: Can't convert 'bytes' object to str implicitly

Note that the file is actually uploaded correctly.

Arch Linux standard packages (Python 3.5.0).

@fijal
Copy link
Member

fijal commented Nov 7, 2015

should be fixed, thanks for reporting

@fijal fijal closed this as completed Nov 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants