search

joaquinanguera / aceR

An R package for processing ACE data
MIT License
3 stars 14 forks source link

Update module-modules-sea.R #6

Closed e-leib closed 5 years ago

e-leib commented 5 years ago

Updated the variable name references to reflect new column name and levels in load-sea-constants-modules.R. Now congruency column (formerly known as num_larger).

monicathieu commented 5 years ago

Ok, these changes look good. I will approve and merge both pull requests!

Note that, in the future, you should be able to (method is slightly different depending on where you init your pull requests from) add subsequent commits back onto the same branch you are PR-ing from so you can have just one PR with multiple changes on it (easier for everyone to track!)

e-leib commented 5 years ago

Great, thank you!

On Tue, Apr 30, 2019 at 1:39 PM Monica Thieu notifications@github.com wrote:

Merged #6 https://github.com/joaquinanguera/aceR/pull/6 into master.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/joaquinanguera/aceR/pull/6#event-2310675363, or mute the thread https://github.com/notifications/unsubscribe-auth/AK72JJLNFBO67H4MDTBJM3LPTCVAFANCNFSM4HJFIVPQ .

-- Elena Leib PhD Student Building Blocks of Cognition Lab bungelab.berkeley.edu University of California, Berkeley