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

ValueError: invalid literal for int() with base 10 #475

Closed
ghost opened this issue Mar 13, 2020 · 3 comments
Closed

ValueError: invalid literal for int() with base 10 #475

ghost opened this issue Mar 13, 2020 · 3 comments
Labels
Duplicate Duplicate of another issue Support Questions that needs answering with no code changes needed or that only require a one time change

Comments

@ghost
Copy link

ghost commented Mar 13, 2020

arch linux kde here, whipper 0.9.0-1

config file, see #472

$ whipper cd rip
Traceback (most recent call last):
  File "/usr/bin/whipper", line 11, in <module>
    load_entry_point('whipper==0.9.0', 'console_scripts', 'whipper')()
  File "/usr/lib/python3.8/site-packages/whipper/command/main.py", line 42, in main
    cmd = Whipper(sys.argv[1:], os.path.basename(sys.argv[0]), None)
  File "/usr/lib/python3.8/site-packages/whipper/command/basecommand.py", line 114, in __init__
    self.cmd = self.subcommands[self.options.remainder[0]](
  File "/usr/lib/python3.8/site-packages/whipper/command/basecommand.py", line 114, in __init__
    self.cmd = self.subcommands[self.options.remainder[0]](
  File "/usr/lib/python3.8/site-packages/whipper/command/basecommand.py", line 60, in __init__
    self.add_arguments()
  File "/usr/lib/python3.8/site-packages/whipper/command/cd.py", line 241, in add_arguments
    default_offset = config.Config().getReadOffset(*info)
  File "/usr/lib/python3.8/site-packages/whipper/common/config.py", line 103, in getReadOffset
    return int(self._parser.get(section, 'read_offset'))
ValueError: invalid literal for int() with base 10: '6\t\t\t; drive read offset in positive/negative frames (no leading +)'

hp drive offset definitely is +6

@lonemadmax
Copy link
Contributor

The error says it's trying to convert to an integer the string 6 ; drive read offset in positive/negative frames (no leading +), which certainly is not an integer.

From the comments in #472 it seems you don't have the unreleased fix dca9fcb. I'd remove inline comments from the config file and try again.

@ghost
Copy link
Author

ghost commented Mar 19, 2020

working fine now, thanks for your help.

@JoeLametta JoeLametta added Duplicate Duplicate of another issue Support Questions that needs answering with no code changes needed or that only require a one time change labels Mar 22, 2020
@JoeLametta
Copy link
Collaborator

Related to #461.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Duplicate Duplicate of another issue Support Questions that needs answering with no code changes needed or that only require a one time change
Projects
None yet
Development

No branches or pull requests

2 participants