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

AutoML WIP #1281

Closed
wants to merge 2 commits into from
Closed

AutoML WIP #1281

wants to merge 2 commits into from

Conversation

bshaffer
Copy link
Contributor

@bshaffer bshaffer commented Sep 7, 2018

We are still waiting on a PHP Namespace fix so that all instances of Automl => AutoMl.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 7, 2018
@jdpedrie
Copy link
Contributor

jdpedrie commented Sep 7, 2018

Could you be sure to remove the artman-genfiles directory?

@jdpedrie
Copy link
Contributor

jdpedrie commented Oct 18, 2018

@bshaffer any update on the namespace fix?

@jdpedrie jdpedrie added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 18, 2018
@alietors
Copy link

Any update on this? It will be very helpful

@jdpedrie jdpedrie mentioned this pull request Dec 13, 2018
@jdpedrie jdpedrie closed this Dec 13, 2018
@bshaffer
Copy link
Contributor Author

Sorry for being late to respond here.
We found out the casing AutoMl is in the proto itself, and decided it was best to not change it.

@bshaffer bshaffer deleted the add-automl branch December 13, 2018 18:28
@jdpedrie
Copy link
Contributor

@bshaffer It looks like googleapis was updated earlier this week with the AutoMl PHP Namespace. However, now there's an inconsistency between that and the GAPIC configuration which is again blocking any release. :)

@bshaffer
Copy link
Contributor Author

If we update the GAPIC configuration to be consistent with the protos, then we'll be good to go?

@jdpedrie
Copy link
Contributor

Yep! I think @dwsupplee or @tmatsuo might be working on that already.

@dwsupplee
Copy link
Contributor

Yeah, we are on it - also working on getting the AutoML sub-repo set up which we will need before we can release as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement. do not merge Indicates a pull request not ready for merge, due to either quality or timing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants