ckloi / Scripting-Languages

0 stars 1 forks source link

Element values cannot be NA #47

Closed Ajfaust closed 7 years ago

Ajfaust commented 7 years ago

I talked to the professor after class about this. We cannot have NA for any value in our column 1 for bintree. Columns 2 and 3 can, as they are "pointer" values.

Also, if we want to go with time efficiency for bintree, I want to suggest using the code I have in ajfaust-2nd. It's a bit more compact and easy to follow in my opinion. Again, this is for time efficiency, so it will not be space efficient. The code is still for an S3 class, but converting it shouldn't be an issue.

I want us all to be on the same page, so if we can't agree on who's code to use, we should get together and plan it out.

ckloi commented 7 years ago

What value should we set for column 1?

Ajfaust commented 7 years ago

Well, depends. If we want to maximize space by replacing popped nodes, then I'm not really sure. If we don't care about space though, we just leave it and modify the pointers when a row gets popped