Skip to content

Added some information about bitstype in doc #16242

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

Closed
wants to merge 1 commit into from

Conversation

Skylion007
Copy link

I was really confused when attempting to use SharedArrays the first time because of this minor distinction between Types and Immutables which are effectively structs.

I was really confused when attempting to use SharedArrays the first time because of this minor distinction between Types and Immutables which are effectively structs.
@Skylion007 Skylion007 changed the title Added some information about bitstype Added some information about bitstype in doc May 7, 2016
@@ -562,6 +562,10 @@ arrays, a shared array is accessible only from those participating
workers specified by the ``pids`` named argument (and the creating
process too, if it is on the same host).

Note: If you want to store a Type like object, it must be immutable to count
as a [bitstype](http://docs.julialang.org/en/release-0.4/manual/types/).
Copy link
Contributor

Choose a reason for hiding this comment

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

The document is wrong here, it should be isbits not bitstype. Also, I believe you are using the markdown syntax instead of rst cross reference and you are linking the 0.4 doc here.

Copy link
Member

@quinnj quinnj May 19, 2016

Choose a reason for hiding this comment

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

@Skylion007, to clarify, the wording should be something along the lines of Note: only objects where "isbits(obj) == true" are supported. Then link to docs.julialang.org/en/latest/... instead of release-0.4. We can the backport the docs change to 0.4.

@kshyatt kshyatt added the docs This change adds or pertains to documentation label May 7, 2016
@ViralBShah ViralBShah closed this Apr 30, 2020
@ViralBShah ViralBShah reopened this Apr 30, 2020
@ViralBShah
Copy link
Member

ViralBShah commented Apr 30, 2020

Ouch - shouldn't have closed and reopened. This is a nice update. I'll get it in and close after #35650 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants