Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

create stitch.rspec for row-wise merges #175

Open
thomased opened this issue Nov 15, 2019 · 4 comments
Open

create stitch.rspec for row-wise merges #175

thomased opened this issue Nov 15, 2019 · 4 comments

Comments

@thomased
Copy link
Collaborator

Wondering if merge.rspec() could be tweaked to allow for row- or column-wise merges (probably via a new argument, since by = specifies the column to be used for merging). I'm thinking of UV-VIS + NIR merges, where you've got separate measures of spectral regions separates but want to row-merge them into one full range spec. Would need to think about how to handle spectral gaps etc., but shouldn't be too tough.

@Bisaloo
Copy link
Collaborator

Bisaloo commented Nov 15, 2019

I think it would be better to use another function since it would differ a lot from merge.data.frame, which is the function most users will be familiar with.

I was thinking of using rbind.rspec but it's not ideal either since it may discard rows in case of wavelength range overlap. Maybe the most reasonable option is a completely new function such as stitch.rspec.

@thomased
Copy link
Collaborator Author

Yeah fair enough. merge and stitch — I could see something like that working.

@Bisaloo
Copy link
Collaborator

Bisaloo commented Nov 18, 2019

combine could be another option.

@thomased thomased changed the title merge.rspec() tweak for row-vs-col merges create stitch.rspec for row-wise merges Jun 4, 2020
@thomased thomased changed the title create stitch.rspec for row-wise merges create stitch.rspec for row-wise merges Jun 4, 2020
@thomased thomased changed the title create stitch.rspec for row-wise merges create stitch.rspec for col-wise merges Jun 4, 2020
@thomased thomased changed the title create stitch.rspec for col-wise merges create stitch.rspec for row-wise merges Jun 4, 2020
@Azem1hyseni
Copy link

I'm so interesting to be part of your group in rock filer foundation , i have one small office in Kosovo but we have large impact in development and more .

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

3 participants