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

ES: Two access types version #100

Merged
merged 7 commits into from
Jul 10, 2021
Merged

ES: Two access types version #100

merged 7 commits into from
Jul 10, 2021

Conversation

guiomar
Copy link
Contributor

@guiomar guiomar commented Dec 17, 2020

Hi!

I have translated to Spanish the "Two access types version" section that was missing.

@yarikoptic
Copy link
Member

Great, thank you! @con/spanish - could someone assist with the review of added translation? Thank you very much in advance!

@yarikoptic
Copy link
Member

Oh, there aren't members in that team yet, I will issue a call.

@guiomar
Copy link
Contributor Author

guiomar commented Dec 18, 2020

Super happy to be part of the group for future needs :)

Copy link
Contributor

@fabianp fabianp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor suggestions. Otherwise looks good to me

docs/source/i18n/ultimate.es.rst Outdated Show resolved Hide resolved
docs/source/i18n/ultimate.es.rst Show resolved Hide resolved
docs/source/i18n/ultimate.es.rst Show resolved Hide resolved
docs/source/i18n/ultimate.es.rst Outdated Show resolved Hide resolved
@yarikoptic
Copy link
Member

hey @all-contributors please add @guiomar for translation

@allcontributors
Copy link
Contributor

@yarikoptic

I've put up a pull request to add @guiomar! 🎉

@yarikoptic
Copy link
Member

hey @all-contributors please add @fabianp for review

@allcontributors
Copy link
Contributor

@yarikoptic

I've put up a pull request to add @fabianp! 🎉

@yarikoptic
Copy link
Member

hey @all-contributors please add @fabianp for review

@allcontributors
Copy link
Contributor

@yarikoptic

I've put up a pull request to add @fabianp! 🎉

@yarikoptic
Copy link
Member

sorry for all the noise, 3rd time must be a charm:

hey @all-contributors please add @fabianp for review

@allcontributors
Copy link
Contributor

@yarikoptic

I've put up a pull request to add @fabianp! 🎉

guiomar and others added 4 commits January 18, 2021 19:53
Co-authored-by: Fabian Pedregosa <pedregosa@google.com>
Match translation in first and second parts
add Face Masking tool link
Co-authored-by: Julia Guiomar Niso Galán <guiomar.niso@ctb.upm.es>
@@ -63,4 +63,4 @@ Even more data/information preserving approach is to just obscure
facial features in the anatomical images:

- `Obscuring Surface Anatomy in Volumetric Imaging Data <http://link.springer.com/article/10.1007%2Fs12021-012-9160-3>`_
Used for HCP data
Used for HCP data, using the `Face Masking <https://nrg.wustl.edu/software/face-masking/>` tool.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unrelated to the PR but I guess ok to be added . FWIW: URL is still valid

@yarikoptic
Copy link
Member

Dear @fabianp , would you be so kind to give a 2nd pass with the review since there might have been new changes?

Copy link
Contributor

@fabianp fabianp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me, just a minor suggestion


Al aceptar participar, estará haciendo una donación gratuita y generosa para la investigación que podría ayudar a otras personas. Es posible que algunas de las investigaciones realizadas con su información eventualmente produzcan el desarrollo de nuevos métodos para estudiar el cerebro, nuevas formas de diagnóstico, nuevos medicamentos u otros productos comerciales. Si esto ocurriera, no existe ningún plan para proporcionarle parte alguna de las ganancias generadas por dichos productos y usted no tendrá ningún derecho de propiedad sobre ellos.

Desde nuestro conocimiento y mejor entender, los datos que compartimos con otros investigadores o al público en general no contendrán información que pueda identificarlo directamente. Los datos que compartimos con otros científicos o el público en general no tendrán su nombre, solo un código numérico, por lo que la gente no sabrá su nombre o qué datos son suyos. Además, los datos puestos a disposición del público en general no incluirán datos que pensamos que podrían ayudar a las personas que lo conocen a adivinar qué datos son suyos, como sus rasgos faciales o la fecha en que participó en el estudio. Si escribimos un informe o artículo sobre este estudio o compartimos el conjunto de datos del estudio con otras personas, lo haremos de tal manera que usted no pueda ser identificado directamente. Sin embargo, al utilizar datos adicionales vinculados a su nombre (por ejemplo, escáneres cerebrales obtenidos de sus registros médicos), uno podría potencialmente vincular sus imágenes u otra información en nuestra base de datos con usted. Además, una brecha de seguridad (intrusión o ciberataque) podría hacer que alguien pudiera asociarlo a sus datos. Este riesgo es muy bajo porque sus datos se almacenan en una base de datos segura y la información sobre su identidad se almacena de manera separada a los datos en sí, únicamente vinculados a través de un código.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

al público -> el público (wrong first time, right second time)

@CPernet CPernet merged commit 6e8530b into con:master Jul 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants