Skip to content

Internalize Microsoft.ML.Internal.CpuMath.GeneralUtils #2737

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.

Sign up for GitHub

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

Jump to bottom
Closed
eerhardt opened this issue Feb 26, 2019 · 0 comments · Fixed by #2741
Closed

Internalize Microsoft.ML.Internal.CpuMath.GeneralUtils #2737

eerhardt opened this issue Feb 26, 2019 · 0 comments · Fixed by #2741
Assignees
@codemzs
Labels
API Issues pertaining the friendly API
Milestone
0219

Comments

@eerhardt
Copy link
Member

We have a single type exposed in namespace Microsoft.ML.Internal.CpuMath

namespace Microsoft.ML.Internal.CpuMath {
    public static class GeneralUtils {
        public static int CbitLowZero(uint u);
    }
}

we should make it internal.

Part of #2326

/cc @codemzs @sfilipi
@codemzs codemzs self-assigned this Feb 26, 2019
@codemzs codemzs added the API Issues pertaining the friendly API label Feb 26, 2019
@codemzs codemzs added this to the 0219 milestone Feb 26, 2019
@codemzs codemzs mentioned this issue Feb 26, 2019
Merged
@ghost ghost locked as resolved and limited conversation to collaborators Mar 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@codemzs codemzs

Labels
API Issues pertaining the friendly API
Projects
None yet
Milestone
0219
Development

Successfully merging a pull request may close this issue.

Internalize Microsoft.ML.Internal.CpuMath.GeneralUtils. codemzs/machinelearning
2 participants
@codemzs @eerhardt