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

Expose a variant of connect that takes offset and length as argument #6

Closed
reynir opened this issue Mar 8, 2023 · 11 comments · Fixed by #13
Closed

Expose a variant of connect that takes offset and length as argument #6

reynir opened this issue Mar 8, 2023 · 11 comments · Fixed by #13
Assignees
Labels

Comments

@reynir
Copy link
Owner

reynir commented Mar 8, 2023

The current connect interface has the nice property that it is difficult to create overlapping partitions. However, in the context of Mirage the Functoria API doesn't have a concept of pairs. It is not clear to me how or if the current interface can be integrated into the Mirage tool.

Implement a variant of connect (could be named connect') that takes and offset and a length and returns a single partition.

@0xrotense
Copy link

can i work on this?

do i write the code to this file https://github.com/reynir/mirage-block-partition/blob/main/lib/mirage_block_partition.ml ? or as a comment here ?

@reynir
Copy link
Owner Author

reynir commented Mar 8, 2023

Yes, you may work on this. It would be great if you could submit it as a pull request. You need to update lib/mirage_block_partition.ml but as well the interface file lib/mirage_block_partition.mli to expose the new function.

0xrotense added a commit to 0xrotense/mirage-block-partition that referenced this issue Mar 8, 2023
@AryanGodara
Copy link

@reynir , are there more issues coming for mirage-block-partition, or can everyone try this issue first?

@reynir
Copy link
Owner Author

reynir commented Mar 17, 2023

@reynir , are there more issues coming for mirage-block-partition, or can everyone try this issue first?

If you like you can work on this issue

@AryanGodara
Copy link

@reynir , are there more issues coming for mirage-block-partition, or can everyone try this issue first?

If you like you can work on this issue

Thanks @reynir , I'll go open a PR, when ready!

@reynir
Copy link
Owner Author

reynir commented Mar 17, 2023

@AryanGodara I would look at the definitions of connect, subpartition and partition as they do something similar.

@AryanGodara
Copy link

@AryanGodara I would look at the definitions of connect, subpartition and partition as they do something similar.

Got it!
Thanks @reynir , I'll start work on this first thing tomorrow morning!

@0xrotense
Copy link

@reynir , I just updated the PR, kindly review and advise further. Thanks

@0xrotense
Copy link

@reynir please check my new comment on my PR, to advise further. thanks

@reynir
Copy link
Owner Author

reynir commented Apr 5, 2023

@0x0god You are welcome to keep working on this. Note however that the Outreachy contribution period is over and further contributions will not be considered for you application. I will as well have less time to help and guide you. So expect to be more self-reliant if you want to continue working on this.

@0xrotense
Copy link

Thanks,...Yes, I understand the outreachy contribution period is over :)

I'm doing this personally to improve my OCaml skills as I develop a passion for OCaml and also to contribute more to the opensource community. :)

@reynir reynir linked a pull request Mar 20, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants