Random Field Element

[lambdaclass]

Random Field Element

[ilitteri]

Can I take it?

Sure! The description is not updated yet, follow the guidelines of this comment.

[ilitteri]

Hi, I cannot find IsFielElement trait, should it be included in IsField? or maybe IsField should be renamed to IsFielElement? Sorry for my ignorance, I'm not familiar with rust naming conventions yet.

The description is updated.

[gluax]

UX should be considered here more. Rather than just defining a random method that uses a generic as an argument, we should implement the rand traits for the types.

It is more convenient and better for user experience when using the library.

[ilitteri]

UX should be considered here more. Rather than just defining a random method that uses a generic as an argument, we should implement the rand traits for the types.

It is more convenient and better for user experience when using the library.

Thanks for the suggestion, you're right, we'll update the description!

Will do the same for #26.

[GianfrancoBazzani]

Can I take it?

[GianfrancoBazzani]

Hi, I cannot find IsFielElement trait, should it be included in IsField? or maybe IsField should be renamed to IsFielElement? Sorry for my ignorance, I'm not familiar with rust naming conventions yet.

[ilitteri]

Hi, I cannot find IsFielElement trait, should it be included in IsField? or maybe IsField should be renamed to IsFielElement? Sorry for my ignorance, I'm not familiar with rust naming conventions yet.

IsFieldElement was the former name of FieldElement. You should implement it for FieldElement.

[MauroToscano]

This is done, though the code is not clear on that, we can create another issue to move it around