How the embedding matrix is optimized using DNNLinearCombinedClassifier

From the document: https://www.tensorflow.org/api_docs/python/tf/contrib/learn/DNNLinearCombinedClassifier
We know the parameter includes linear_optimizer and dnn_optimizer that can be used to optimize the linear part and dnn part of the weights. However, it seems that the optimization of the weight of embedding matrix is not involved, such as embedding matrix of the wide and deep model. So my question is that is the optimization of the embedding matrix not involved in reality or is there any default way of optimizing the embedding matrix that I do not know
estimator = DNNLinearCombinedClassifier(
# common settings
n_classes=n_classes,
weight_column_name=weight_column_name,
# wide settings
linear_feature_columns=[sparse_feature_a_x_sparse_feature_b],
linear_optimizer=tf.compat.v1.train.FtrlOptimizer(...),
# deep settings
dnn_feature_columns=[sparse_feature_a_emb, sparse_feature_b_emb],
dnn_hidden_units=[1000, 500, 100],
dnn_optimizer=tf.compat.v1.train.AdagradOptimizer(...))

@sjtusmartboy
Sorry, I didn't quite understand your question.
But DNNLinearCombinedClassifier definitely can be used to build and train the Wide and Deep models.

@sjtusmartboy ,
Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version.
Also can you check feedback from @Leslie-Fang regarding the issue.Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


[#51] Parser now supports hexadecimal escape sequences (4-2-stable)

Cherry-picked from #70.

#'d


impromentChange query_statistics_map from default to optional

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Update cardano-ledger dependency

Description
This updates the cardano-ledger dependency to include https://github.com/input-output-hk/cardano-ledger/pull/2871 which is required to resolve this issue https://github.com/input-output-hk/cardano-node/issues/3938.
The change involves using the ListMap type definition of the ShelleyGenesis datatype order to enable lazy streaming as show in https://github.com/input-output-hk/cardano-node/pull/4021
Lazy streaming has favourable performance and memory usage implications.
https://github.com/input-output-hk/cardano-ledger/pull/2890 will need to be merged first as it provides some additional useful functions.
Checklist

Branch

[ ] Commit sequence broadly makes sense
[ ] Commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] If this branch changes Consensus and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[ ] Self-reviewed the diff
[ ] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[ ] Reviewer requested

bors r+


Added configuration guide.

@duftler

PTAL

PTAL

@rbuskens @ewiseblatt is this PR still relevant or should we close it?

@rbuskens will merge this when ready

Friendly ping.

@rbuskens & @ewiseblatt

thanks. forgot about this.
On Fri, Jan 22, 2016 at 6:54 PM, Matt Duftler<EMAIL_ADDRESS>wrote:

Friendly ping.
—
Reply to this email directly or view it on GitHub
https://github.com/spinnaker/spinnaker.github.io/pull/94#issuecomment-174092512
.

@rbuskens PTAL
On Fri, Jan 22, 2016 at 6:55 PM, Matt Duftler<EMAIL_ADDRESS>wrote:

@rbuskens https://github.com/rbuskens & @ewiseblatt
https://github.com/ewiseblatt
—
Reply to this email directly or view it on GitHub
https://github.com/spinnaker/spinnaker.github.io/pull/94#issuecomment-174092550
.

LGTM.

Looks like this document is not being rendered properly on spinnaker.io: http://spinnaker.io/documentation/configuration.html

How are these pages rendered? The page in the github repository seems to
view correctly. I can fix it, but dont know how to identify or verify the
problem.
On Thu, Feb 4, 2016 at 8:09 AM, Matt Duftler<EMAIL_ADDRESS>wrote:

Looks like this document is not being rendered properly on spinnaker.io:
http://spinnaker.io/documentation/configuration.html
—
Reply to this email directly or view it on GitHub
https://github.com/spinnaker/spinnaker.github.io/pull/94#issuecomment-179824028
.


Add a section that contains the most common pitfalls and how to avoid them

Problem Description:
There are common things people should avoid doing
Suggested Solution:
I think if we had a new page for that we could recommend everyone read that page regardless of how you're using glamorous. Here are some things that should be on that page:

[ ] Remember to forward the className if you're wrapping a custom component in glamorous https://github.com/paypal/glamorous/issues/112
[ ] Never use object spread (or Object.assign) when merging styles, use an array instead (object spread is shallow, so pseudo classes/etc. wont merge). Giving an array will make glamorous forward them all to glamor which will merge them. https://github.com/paypal/glamorous/issues/142
[ ] If you're using glamor.css and passing it to className, make sure to call toString() on it https://github.com/paypal/glamorous/issues/140
[ ] Describe order of priority with regard to ...styles -> className -> props as css -> css prop https://github.com/paypal/glamorous/issues/195

I'm sure there are other things that should go on that kind of page. Ideas welcome.

@tkh44 That's a great tip! I'm really interested in seeing flame graphs if you want to share. (Your benchmarks on emotion are 💯. Thank you!)
restyles looks awesome. We should have that in Integrations for sure!

Here is a gif explaining this concept better than anything I could write.

Here you can see that one glamorous component is under 1ms so not bad. The problem is when you nest multiple glamorous elements as you can see. There were ~30 items in that section so it took 30+ms to render that small section of the tree.

@tkh44 Is that in a production build? Because develop & production are different when comparing performance.

This was in production @guidobouman. I resolved the issue by creating https://github.com/emotion-js/emotion. The fundamental problem with glamorous is that it invalidates glamor's WeakMap caching implementation resulting in complete style recalculation on ever render.

@tkh44 would you mind pointing out where that occurs? I'd be interested in learning more about that specifically. Thanks!

Wait... What? I was unaware of this. I'd like to know how that happens and how it can be fixed....... If it weren't you Kye, I'd say you were mistaken...

When the glamor class name is calculated here https://github.com/paypal/glamorous/blob/73cbc5f6a92aa003387a073449f8b3136804ba8c/src/get-glamor-classname.js#L75 it returns a new array every time.
This causes a cache miss in Glamor's WeakMap cache.
https://github.com/threepointone/glamor/blob/master/docs/weakmaps.md#so-how-can-we-use-it-with-glamor
I don't know if there is anything you can do. The same symptom exists in most css-in-js libraries that have a component wrapper, including emotion. I won't pretend to know all the details that go into the performance differences between emotion and glamorous when using dynamic values. The obvious, hand-wavey guess is that emotion is just doing less work to calculate the same style.
I also want to apologize for the tone of my original comment. The statement should have had more clarification.

No worries on the tone. I didn't take it any more than a clarification. I'm just surprised because I was under the impression that we were taking advantage of the cache. I think I understand what's going on and I suppose there's some optimization we could do to ensure that if the styles are the same we use the same array. Thanks for sharing this insight!
I'm not going to personally pursue this any further because I have other more pressing matters WRT performance in my app, but if someone else wants to improve things that'd be pretty great :+1:

This project is now in an unmaintained status. Please see the glamorous README for more information.


Make the reset-local-k3d script more secure

Summary
Makes the reset-local-k3d script check that the cluster creation actually worked and that it's in the correct context. This is so that we don't have any accidents in the future again when the context switching fails.

@joelhoisko I added a commit, I hope the "bash" works on your machine, wanted to make it easier to get out of the problem, as I put myself in it, and then had to recall what we did yesterday over teams :).


Danbooru links are insert into chat incorrectly.

It seems Danbooru is using a subdomain now, Nadeko is pasting links like the following:
https://danbooru.donmai.ushttps//hijiribe.donmai.us/data/image.png
When it should be:
https://hijiribe.donmai.us/data/image.png
or
https://danbooru.donmai.us/data/image.png

Works fine for me, can't reproduce

This appears fixed, was probably something on danbooru's side.

@Kwoth I saw this being able to be reproduced quite frequently today with people using the .hentaibomb goo_girl command.

Ok I'll add some code to check if it's a full url then just in case.

https://danbooru.donmai.ushttps//raikou2.donmai.us
Was another one that showed up.


allow deriving of FromVariant for uninhabitable enums

Fixs #444
What type of error should be returned might need further discussion.

Thanks a lot for this PR! 👍
Out of curiosity, did you stumble upon this in a project of yours? If yes, could you share the context? :)
I'm asking because of this in the issue:

I wonder generally if this is useful -- it basically translates a compile-time error (trait not implemented) to a runtime error (Result::Err or panic if unwrapped). If the uninhabited enum is part of a larger struct/enum for which FromVariant is derived, then it will break the entire conversion.

bors try

Thanks! 🚀
bors r+


Correcting English

Hi all,
Thank you very much for using CV Drone.
The library grew up as a useful one and helped many AR.Drone users.
However, I think I wrote a lot of wrong English in comment.
Of course, it is because of my lack of knowledge of English, but if it is possible, please help me.
Sincerely,
puku0x

Tell me what do you need. English is not my native language. But I can help.

Oh, thank you anfedres.
All comments in source codes and articles in Wiki are written in English.
If you find wrong spellings, strange idioms and grammars in them, please tell me because I am not a native speaker and I am not sure they are understandable for native speakers.
Regards,
puku0x

They are. Don't worry. You are doing great.
anfedres

Thank you.
I deeply appreciate it.