Skip to content
This repository has been archived by the owner on Aug 31, 2021. It is now read-only.

undefined method A1 for #<Soroban::Sheet:0x007f844c9ffb28> #1

Closed
thirdreplicator opened this issue Sep 2, 2013 · 4 comments
Closed

Comments

@thirdreplicator
Copy link

s.A1 is throwing an error. Am I doing something wrong? Thank you.

$ irb

1.9.3p392 :001 > require 'soroban'
=> true

1.9.3p392 :002 > s = Soroban::Sheet.new()
=> #<Soroban::Sheet:0x007f844c9ffb28 @logger=nil, @cells={}, @changes={}, @bindings={}>

1.9.3p392 :003 > s.A1
NoMethodError: undefined method A1' for #<Soroban::Sheet:0x007f844c9ffb28> from /Users/thirdreplicator/.rvm/gems/ruby-1.9.3-p392/gems/soroban-0.5.4/lib/soroban/sheet.rb:29:inmethod_missing'
from (irb):3
from /Users/thirdreplicator/.rvm/rubies/ruby-1.9.3-p392/bin/irb:16:in `

'

1.9.3p392 :004 > s.A1=2
NameError: uninitialized constant Soroban::Sheet::Set
from /Users/thirdreplicator/.rvm/gems/ruby-1.9.3-p392/gems/soroban-0.5.4/lib/soroban/sheet.rb:17:in block in initialize' from /Users/thirdreplicator/.rvm/gems/ruby-1.9.3-p392/gems/soroban-0.5.4/lib/soroban/sheet.rb:144:inyield'
from /Users/thirdreplicator/.rvm/gems/ruby-1.9.3-p392/gems/soroban-0.5.4/lib/soroban/sheet.rb:144:in default' from /Users/thirdreplicator/.rvm/gems/ruby-1.9.3-p392/gems/soroban-0.5.4/lib/soroban/sheet.rb:144:in_clear'
from /Users/thirdreplicator/.rvm/gems/ruby-1.9.3-p392/gems/soroban-0.5.4/lib/soroban/sheet.rb:140:in _set' from /Users/thirdreplicator/.rvm/gems/ruby-1.9.3-p392/gems/soroban-0.5.4/lib/soroban/sheet.rb:129:in_add'
from /Users/thirdreplicator/.rvm/gems/ruby-1.9.3-p392/gems/soroban-0.5.4/lib/soroban/sheet.rb:27:in method_missing' from (irb):4 from /Users/thirdreplicator/.rvm/rubies/ruby-1.9.3-p392/bin/irb:16:in

'

@kranzky kranzky closed this as completed in fe596e8 Sep 3, 2013
@kranzky
Copy link
Contributor

kranzky commented Sep 3, 2013

Sorry you hit a problem using Soroban @thirdreplicator :(

You can work-around this problem by doing require 'set' prior to requiring Soroban. Alternatively, we've fixed this issue in 0.7.2, so you can update to that and all should be good. Thanks for reporting the issue (we missed it because we use Bundler, and it seems that requires Set and therefore hides this missing dependency).

@kranzky
Copy link
Contributor

kranzky commented Sep 3, 2013

Huh, I see @atlantageek fixed this a week ago in his fork; PR next time please ;)

@atlantageek
Copy link

OK. youre really my first fork. I didnt realize that was the protocol. Do you plan to implement the to_javascript method in the near future?

@kranzky
Copy link
Contributor

kranzky commented Sep 3, 2013

No worries @atlantageek; was just nice to see you'd made the fix. But yes, if you'd issues a pull request I would have accepted it gratefully.

At the moment I personally would love to implement to_javascript, but there's no pressing need to do that at the moment to support what we're using Soroban for, so other work is coming first I'm afraid.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants