Fix Jira Bug I7-2460

https://inform7.atlassian.net/jira/software/c/projects/I7/issues/I7-2460?filter=updatedrecently
This allows the sensible error messages from "remove" to trigger rather than giving strange "you can't see any such thing" errors.

The first version of this didn't work right and broke a bunch of stuff, due to some important things I overlooked when trying to convert it from an extension to a patch (which is an error-prone process).
The new version doesn't break anything.  It fixes the error message for the single-item case ("REMOVE CAT FROM BOX" where the cat is in the room but not in the box), which is by far the most common case, and at least means that the error message in Standard Rules gets printed sometimes (rather than never ever being used, which is the situation without any patch).
The multi-item case ("REMOVE CAT AND DOG FROM BOX") is too hairy to handle at this time and still gives "You can't see any such thing".  This is largely due to the amount of special casing of ##Remove in the I6 parser, and the inaccessibility of the look-forward code to any token other than multiinside and multiexcept.  (I may approach it again after I've figured out how to make the look-forward code accessible to the "deciding whether all includes" activity.)


Clean up the config

Not tested.
I love trailing whitespace.

would prefer tested before merge, and would need a version update or something to warn people that the configs won't be cross compatible.

How about supporting both configs for a couple of releases while showing deprecation warnings?

that'd work -- just needs to be a big label warning :)

I admire your effort to clean up the config, but there's a reason why I put all the config comments in one block at the top.
Config comments aren't parsed by the yaml parser, so if you interlace comments and settings, most of the comments will be deleted if you ever update a settings from console triggering a re-save of the config.
snakeyaml is at least smart enough to keep any existing comments at the top of a file when saving. Any comments before the first setting will be preserved, while any other comments will be removed.
I will add there could be a way of preserving the interlaced comments that I'm not aware of, I was just never able to figure out a way to do it. It's easy enough to test, just run the plugin, change a value with ep config set and then ep config save from console. Observe the resulting saved file.

most of the comments will be deleted if you ever update a settings from console

The purpose of which is?
I don't see any benefits over simply editing the config file itself. If you want dynamic reloading either periodically check if the config file has changed or add a /config_reload command.

It's a pretty big feature class to have live editing of configs in a minecraft mod. Surprisingly difficult to do right.

There actually is a config reload command.
And I don't entirely remember the reason for changing settings through console, but @ProgrammerDan may have specifically requested it.

Surprisingly difficult to do right.

Did I do it right??

Does it work? You did it right :P

@iykHvfIvfR Do you intent to modify or alter approach based on comments here? Or, something else?

@ProgrammerDan Oh yea, I'm just being slow. I'm gonna remove whatever functionality is responsible for editing the config file and make my changes backward-compatible.

No worries, thanks for the update.


Move br(count) attribute to desugar to binrw::helpers::count

Unlike the count attribute, which is limited to supporting Vec<T>, the count helper function works for any type which implements FromIterator<T>.
Upsides:

Less magic implementation, more type safe
Adds support for more types

Possible pitfalls:

Perf regression due to the lack of "fake specialization" provided for Vec<u8>
Breaking some existing use-case maybe?

FWIW, if there's a measurable perf regression, this sounds exactly like the kind of use-cases where autoref specialization could be used to recover the lost perf.


Cannot read property 'success' of undefined

After adding one more stack to an application with ~25 stacks I got the following error in deployment summary:
/foo/eu-west-1/bar/bar.yml/eu-west-1                      bar-eu-west-1-dev            FAILED     DEPENDENCIES_FAILED  50.5s     Cannot read property 'success' of undefined
That new stack had one new dependency. This was resolved by removing the only dependency from that new stack.
Also if only that new stack is deployed even with dependencies there are no problems. Or if the target of the dependency is deployed with dependencies.
The detailed error:
2020-10-09 17:17:59 +0300 [error] - /foo/eu-west-1/bar/bar.yml/eu-west-1 - An error occurred while waiting dependencies to complete TypeError: Cannot read property 'success' of undefined
    at /packages/takomo/1.5.1/node_modules/@takomo/stacks-commands/dist/stacks/deploy/wait.js:13:75
    at Array.find (<anonymous>)
    at Object.exports.waitForDependenciesToComplete (/packages/takomo/1.5.1/node_modules/@takomo/stacks-commands/dist/stacks/deploy/wait.js:13:60)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)

It seems that the array of dependencies contains one item that is undefined. The wait code assumes each is an object.
Takomo version 1.5.1. Installed with Volta install
This error was originally reported for 2.8.0. I also tested this with 2.8.0 but was not able to get the application working even without the new stack. But the error was the same and maybe might be related to swallowing of an error somewhere else. It might be related to Takomo using different Node version compared to the application.

Edited the issue after finding a workaround for 1.5.1. On 2.8.0 there might be some other issue too so I only mention it separately.

Some additional questions:

Did you run deploy with or without the -y or --yes option?
Are you using parameter resolvers in your stacks?

With ‘-y’. Parameter resolvers are in use. So are command and custom hooks.

I just released version 2.9.0. There's now some more logging in place that hopefully helps to reveal the root cause of the issue. Once you have time, please try again with the 2.9.0 version and --log trace.

Thank you. I tried the new 2.9.0. This stack configuration works with 1.5.1.
I noticed there's a new logging for stack deployment order. That shows that all stacks that fail due to reported error are in the list before their dependencies. There's one stack that fails due to it's dependencies failing. Please find the complete list below with notes. Stack names have been changed.
  - /foo/eu-west-1/b/b.yml/eu-west-1
  - /foo/eu-west-1/c/c.yml/eu-west-1
  - /foo/eu-west-1/a/a.yml/eu-west-1
  - /foo/eu-west-1/d/d.yml/eu-west-1
  - /foo/us-east-1/b/b.yml/us-east-1
  - /foo/us-east-1/x/x.yml/us-east-1
  - /foo/eu-north-1/e/e.yml/eu-north-1 -- eu-north-1/a/a.yml
  - /foo/eu-west-1/f/f.yml/eu-west-1 -- eu-west-1/p/p.yml (NOTE: dependency below) FAILED Cannot read property 'success' of undefined
  - /foo/eu-west-1/g/g.yml/eu-west-1 --> us-east-1/x/x.yml
  - /foo/eu-west-1/h/h.yml/eu-west-1 -->   - us-east-1/x/x.yml, eu-west-1/g/g.yml
  - /foo/eu-west-1/i/i.yml/eu-west-1 -->   - us-east-1/x/x.yml, eu-west-1/g/g.yml
  - /foo/eu-west-1/j/j.yml/eu-west-1 -->   - eu-west-1/h/h.yml, eu-west-1/p/p.yml, us-east-1/t/t.yml, eu-west-1/r/r.yml, eu-west-1/q/q.yml (NOTE: dependency below) FAILED Cannot read property 'success' of undefined
  - /foo/eu-west-1/k/k.yml/eu-west-1 -->   - eu-west-1/p/p.yml, eu-west-1/h/h.yml, eu-west-1/i/i.yml, eu-west-1/q/q.yml (NOTE: dependency below) FAILED Cannot read property 'success' of undefined
  - /foo/eu-west-1/l/l.yml/eu-west-1 -->  - eu-west-1/p/p.yml, eu-west-1/h/h.yml, eu-west-1/q/q.yml (NOTE: dependency below) FAILED Cannot read property 'success' of undefined
  - /foo/eu-west-1/m/m.yml/eu-west-1 -->   - eu-west-1/p/p.yml, us-east-1/t/t.yml, eu-west-1/i/i.yml, eu-west-1/h/h.yml, eu-west-1/a/a.yml, eu-west-1/r/r.yml, eu-west-1/q/q.yml (NOTE: dependency below) FAILED Dependencies failed
  - /foo/eu-west-1/n/n.yml/eu-west-1 -->    - eu-west-1/p/p.yml, us-east-1/t/t.yml, eu-west-1/i/i.yml, eu-west-1/h/h.yml, eu-west-1/a/a.yml, eu-west-1/r/r.yml, eu-west-1/q/q.yml (NOTE: dependency below) FAILED Cannot read property 'success' of undefined
  - /foo/eu-west-1/o/o.yml/eu-west-1 -->   - eu-west-1/c/c.yml, eu-west-1/h/h.yml
  - /foo/us-east-1/g/g.yml/us-east-1 --> us-east-1/x/x.yml
  - /foo/eu-west-1/p/p.yml/eu-west-1 -->   - us-east-1/g/g.yml
  - /foo/eu-west-1/q/q.yml/eu-west-1 -->   - eu-west-1/p/p.yml, eu-west-1/h/h.yml
  - /foo/eu-west-1/r/r.yml/eu-west-1 --> eu-west-1/p/p.yml, eu-west-1/h/h.yml
  - /foo/eu-west-1/s/s.yml/eu-west-1 -->   - eu-west-1/p/p.yml, eu-west-1/c/c.yml, eu-west-1/h/h.yml, eu-west-1/r/r.yml, eu-west-1/q/q.yml
  - /foo/us-east-1/t/t.yml/us-east-1 -->   - eu-west-1/p/p.yml,  - eu-west-1/c/c.yml

If I leave the -y out and choose to review each stack manually I got the following error:
2020-10-10 08:06:37 +0300 [debug] - /events/eu-north-1/a/a.yml - Change set creation failed with reason: No updates are to be performed.
2020-10-10 08:06:37 +0300 [error] - /events/eu-north-1/a/a.yml/eu-north-1 - Failed to create change set Error: No updates are to be performed.
    at CloudFormationClient.waitUntilChangeSetIsReady (/packages/takomo/2.9.0/node_modules/@takomo/aws-clients/dist/cloudformation.js:173:23)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async Object.exports.reviewChanges (/packages/takomo/2.9.0/node_modules/@takomo/stacks-commands/dist/stacks/deploy/review.js:54:27)
    at async Object.exports.prepareCloudFormationTemplate (/packages/takomo/2.9.0/node_modules/@takomo/stacks-commands/dist/stacks/deploy/template.js:121:20)
    at async Object.exports.executeDeployContext (/packages/takomo/2.9.0/node_modules/@takomo/stacks-commands/dist/stacks/deploy/execute-deploy-context.js:84:27)
    at async Object.exports.handle (/packages/takomo/2.9.0/node_modules/@takomo/cli/dist/common.js:243:24)

Stack path                                                     Stack name                        Status     Reason                    Time  Message
-------------------------------------------------------------  --------------------------------  ---------  ------------------------  ----  -------------------------------
/events/eu-north-1/a/a.yml/eu-north-1    a-eu-north-1-de   FAILED     CREATE_CHANGE_SET_FAILED  6.9s  No updates are to be performed.
/events/eu-west-1/b/b.yml/eu-west-1                          b-eu-west-1-devmika              CANCELLED  CANCELLED                 0ms   Cancelled

Just for reference, the TypeError on 2.9.0 is:
2020-10-10 08:08:09 +0300 [error] - /events/eu-west-1/n/n.yml/eu-west-1 - An error occurred while waiting dependencies to complete TypeError: Cannot read property 'success' of undefined
    at /packages/takomo/2.9.0/node_modules/@takomo/stacks-commands/dist/stacks/deploy/wait.js:13:75
    at Array.find (<anonymous>)
    at Object.exports.waitForDependenciesToComplete (/packages/takomo/2.9.0/node_modules/@takomo/stacks-commands/dist/stacks/deploy/wait.js:13:60)
    at async Promise.all (index 16)
    at async executeStacksInParallel (/packages/takomo/2.9.0/node_modules/@takomo/stacks-commands/dist/stacks/deploy/execute-deploy-context.js:32:21)
    at async Object.exports.handle (/packages/takomo/2.9.0/node_modules/@takomo/cli/dist/common.js:243:24)

Would there be something of interest in the logs?
I could try to create a simplified version of the stack that I could share. It would take some time.
Maybe there could be some logging where the list of dependencies is updated to catch the writer of undefined?
It might be a fun and useful feature to get the dependency graph out in visual format such as https://en.wikipedia.org/wiki/DOT_(graph_description_language)
Unfortunately I have limited access to the project in the following days, so it will take some time before I can try to work on this.

Here are all the lines with the new "Dependency in stack does not exists" errors from the log. All the stack names have been changed manually so there could be errors here and in earlier comment.
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/p/p.yml/eu-west-1' in stack /events/eu-west-1/f/f.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/p/p.yml/eu-west-1' in stack /events/eu-west-1/j/j.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/us-east-1/t/t.yml/us-east-1' in stack /events/eu-west-1/j/j.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/r/r.yml/eu-west-1' in stack /events/eu-west-1/j/j.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/q/q.yml/eu-west-1' in stack /events/eu-west-1/j/j.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/p/p.yml/eu-west-1' in stack /events/eu-west-1/k/k.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/q/q.yml/eu-west-1' in stack /events/eu-west-1/k/k.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/p/p.yml/eu-west-1' in stack /events/eu-west-1/l/l.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/q/q.yml/eu-west-1' in stack /events/eu-west-1/l/l.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/p/p.yml/eu-west-1' in stack /events/eu-west-1/n/n.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/us-east-1/t/t.yml/us-east-1' in stack /events/eu-west-1/n/n.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/r/r.yml/eu-west-1' in stack /events/eu-west-1/n/n.yml/eu-west-1 does not exists
2020-10-10 08:07:10 +0300 [error] - Dependency '/events/eu-west-1/q/q.yml/eu-west-1' in stack /events/eu-west-1/n/n.yml/eu-west-1 does not exists

The error that you got in version 2.9.0 when reviewing the stack was fixed in #112 and included in version 2.10.0. Stacks with SAM transformations produce different message when there is no changes to perform and this was not handled properly. I don't think that this fixes the problem you have with dependencies, though. I'll continue to investigate the issue.
2.10.0 also has a new CLI command to print stacks dependency graph in DOT format. You need to paste the output to some tool that can draw the graph, e.g. https://dreampuf.github.io/GraphvizOnline

Thank you very much.
Now when running without -y and by choosing to review each stack individually I get the same error as with -y
Here's the dependency graph output. The letters are most likely different than in earlier snippet:
{
  "/h/eu-north-1/a/a.yml/eu-north-1" -> "/h/eu-north-1/b/b.yml/eu-north-1"
  "/h/eu-north-1/b/b.yml/eu-north-1"
  "/h/eu-west-1/c/c.yml/eu-west-1" -> "/h/us-east-1/h/h.yml/us-east-1"
  "/h/eu-west-1/c/c.yml/eu-west-1" -> "/h/eu-west-1/s/s.yml/eu-west-1"
  "/h/eu-west-1/d/d.yml/eu-west-1" -> "/h/eu-west-1/e/e.yml/eu-west-1"
  "/h/eu-west-1/e/e.yml/eu-west-1" -> "/h/us-east-1/s/s.yml/us-east-1"
  "/h/eu-west-1/s/s.yml/eu-west-1" -> "/h/us-east-1/h/h.yml/us-east-1"
  "/h/eu-west-1/g/g.yml/eu-west-1"
  "/h/eu-west-1/h/h.yml/eu-west-1" -> "/h/eu-west-1/l/l.yml/eu-west-1"
  "/h/eu-west-1/h/h.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/eu-west-1/i/i.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/eu-west-1/i/i.yml/eu-west-1" -> "/h/eu-west-1/e/e.yml/eu-west-1"
  "/h/eu-west-1/i/i.yml/eu-west-1" -> "/h/us-east-1/i/i.yml/us-east-1"
  "/h/eu-west-1/i/i.yml/eu-west-1" -> "/h/eu-west-1/r/r.yml/eu-west-1"
  "/h/eu-west-1/i/i.yml/eu-west-1" -> "/h/eu-west-1/k/k.yml/eu-west-1"
  "/h/eu-west-1/j/j.yml/eu-west-1" -> "/h/eu-west-1/e/e.yml/eu-west-1"
  "/h/eu-west-1/j/j.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/eu-west-1/j/j.yml/eu-west-1" -> "/h/eu-west-1/h/h.yml/eu-west-1"
  "/h/eu-west-1/j/j.yml/eu-west-1" -> "/h/eu-west-1/k/k.yml/eu-west-1"
  "/h/eu-west-1/k/k.yml/eu-west-1" -> "/h/eu-west-1/e/e.yml/eu-west-1"
  "/h/eu-west-1/k/k.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/eu-west-1/l/l.yml/eu-west-1"
  "/h/eu-west-1/m/m.yml/eu-west-1" -> "/h/eu-west-1/e/e.yml/eu-west-1"
  "/h/eu-west-1/m/m.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/eu-west-1/m/m.yml/eu-west-1" -> "/h/eu-west-1/k/k.yml/eu-west-1"
  "/h/eu-west-1/n/n.yml/eu-west-1" -> "/h/eu-west-1/l/l.yml/eu-west-1"
  "/h/eu-west-1/n/n.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/eu-west-1/n/n.yml/eu-west-1" -> "/h/eu-west-1/b/b.yml/eu-west-1"
  "/h/eu-west-1/n/n.yml/eu-west-1" -> "/h/eu-west-1/i/i.yml/eu-west-1"
  "/h/eu-west-1/n/n.yml/eu-west-1" -> "/h/eu-west-1/h/h.yml/eu-west-1"
  "/h/eu-west-1/b/b.yml/eu-west-1"
  "/h/eu-west-1/o/o.yml/eu-west-1" -> "/h/eu-west-1/e/e.yml/eu-west-1"
  "/h/eu-west-1/o/o.yml/eu-west-1" -> "/h/us-east-1/i/i.yml/us-east-1"
  "/h/eu-west-1/o/o.yml/eu-west-1" -> "/h/eu-west-1/h/h.yml/eu-west-1"
  "/h/eu-west-1/o/o.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/eu-west-1/o/o.yml/eu-west-1" -> "/h/eu-west-1/b/b.yml/eu-west-1"
  "/h/eu-west-1/o/o.yml/eu-west-1" -> "/h/eu-west-1/r/r.yml/eu-west-1"
  "/h/eu-west-1/o/o.yml/eu-west-1" -> "/h/eu-west-1/k/k.yml/eu-west-1"
  "/h/eu-west-1/p/p.yml/eu-west-1" -> "/h/eu-west-1/l/l.yml/eu-west-1"
  "/h/eu-west-1/p/p.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/eu-west-1/j/j.yml/eu-west-1" -> "/h/eu-west-1/e/e.yml/eu-west-1"
  "/h/eu-west-1/j/j.yml/eu-west-1" -> "/h/eu-west-1/l/l.yml/eu-west-1"
  "/h/eu-west-1/j/j.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/eu-west-1/j/j.yml/eu-west-1" -> "/h/eu-west-1/r/r.yml/eu-west-1"
  "/h/eu-west-1/j/j.yml/eu-west-1" -> "/h/eu-west-1/k/k.yml/eu-west-1"
  "/h/eu-west-1/q/q.yml/eu-west-1"
  "/h/eu-west-1/r/r.yml/eu-west-1" -> "/h/eu-west-1/e/e.yml/eu-west-1"
  "/h/eu-west-1/r/r.yml/eu-west-1" -> "/h/eu-west-1/c/c.yml/eu-west-1"
  "/h/us-east-1/s/s.yml/us-east-1" -> "/h/us-east-1/h/h.yml/us-east-1"
  "/h/us-east-1/g/g.yml/us-east-1"
  "/h/us-east-1/h/h.yml/us-east-1"
  "/h/us-east-1/i/i.yml/us-east-1" -> "/h/eu-west-1/e/e.yml/eu-west-1"
  "/h/us-east-1/i/i.yml/us-east-1" -> "/h/eu-west-1/l/l.yml/eu-west-1"
}

That's one impressive dependency graph =) . It helped me to reproduce the problem. Let's see how to fix it.

Excellent! I verified that the fix works and the application that worked with 1.5.1 works now with 2.10.1 as well. Thank you.

Cool. The logic that sorted the stacks in deployment order was flawed. The test cases were not complex enough to catch the bug but now, thanks to your feedback, the sorting works also in bigger setups.


Fix more headers.

Attempt 2.
Staging: https://geeknoid.github.io/istio.github.io/docs/setup/kubernetes/sidecar-injection.html

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
Assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Updated. PTAL.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ayj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
Assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Problem with f11

Hi guys, is possible use hotkeys with f11?

@guiaparicio I am not sure, you can test it.

hotkeys('f11', function(event, handler) {
event.preventDefault();
alert('you pressed F11!');
});
Not work =(
fullscreen browser is enable

hotkeys('f11', function(event, handler) {
    event.preventDefault();
    alert('you pressed F11!');
});

Not work =(
fullscreen browser is enable

@guiaparicio it is workable.
maybe u are using macOS, f11 is blocked by system, cc
https://social.technet.microsoft.com/Forums/windows/en-US/784b2bbe-353f-412e-ac9a-193d81f306b6/remote-desktop-for-mac-f11-key-not-working-on-macbook-pro-touchbar?forum=winRDc


Update pyproject.toml linking to repository and issues

This will help on repository discoverability within PyPI.
Followed official Python guidelines from here.

Thank you!


Updates to README, profile, & ONBOARDING files

Description
Updated files with new link to COMMUNICATIONS file, now in the community repo, along with additional information regarding how to connect with others on Slack, forum, and others.
Issues Resolved
n/a
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

edits resolved. thanks @dblock

@krisfreedain Thanks for this update!


Feature request: Embed Tweets

New feature request - Embedded tweets
It would be useful if there was a feature to embed tweets, the same way you can use a custom component to embed YouTube videos.
Currently I have to make them as image screen shoots of the tweet and set a link on the the image to the tweet. This works, but is not as handy or nice. Not super critical, I can work with the image + link too.  I would just prefer to embed a tweet, like below. This does does not work on GitHub either, but the code Twitter gives to embed a tweet is below:
FlexColorScheme V5-dev.2 is here 💙With an even cooler config/code gen app https://t.co/3cgTNvoahHGet the #Flutter package here https://t.co/oDheabOXIv Official v5 release coming when doc update is done.This thread is still relevant and has more info:https://t.co/dS8jPAB53q pic.twitter.com/gxdCAjuJxl— Mike Rydstrom 💙 (@RydMike) April 5, 2022
And it should end up looking like the image plus link version below. Well almost, this URL shows what the above should look like:
https://publish.twitter.com/?query=https%3A%2F%2Ftwitter.com%2FRydMike%2Fstatus%2F1511166732496429058&widget=Tweet
But close enough to this version:

this would be a nice feature. As you say, we could simply add a custom Twitter component

Cool, works well great!
Tested it here: https://docs.page/rydmike/docs_page_test/more_intro
Using:
<Tweet id="1517005624814493696" width="550" align="right" cards="show" theme="dark"/>

Would of course be useful if there was a way to link the theme mode of the tweet light/dark setting to a prop that is tied to effective light/dark mode used on the site.
Also I did not see any difference when applying align prop, it always seemed to be centered.

@cabljac one more observation, if I set align to "center", the width prop did not work, it, works with left though, so using that for now. Still lays out as center, and animates out to the max 550 width that I used.


Restangular compatibility

Hi!
I was wondering, is there a Restangular integration example?
Thanks!

It is working, it just happens that locally runs so fast that probably you won't be able to see it.


Updated ESP-IDF and get thrown a esp_http_client error (IDFGH-2042)

Compiling .pio/build/esp32dev/esp_http_client/lib/http_utils.o
/Users/ashleywilkie/.platformio/packages/framework-espidf/components/esp_http_client/esp_http_client.c: In function 'esp_http_client_init':
/Users/ashleywilkie/.platformio/packages/framework-espidf/components/esp_http_client/esp_http_client.c:510:15: error: 'esp_http_client_config_t {aka const struct }' has no member named 'use_global_ca_store'
if (config->use_global_ca_store == true) {
^
/Users/ashleywilkie/.platformio/packages/framework-espidf/components/esp_http_client/esp_http_client.c:516:15: error: 'esp_http_client_config_t {aka const struct }' has no member named 'client_cert_pem'
if (config->client_cert_pem) {
^
/Users/ashleywilkie/.platformio/packages/framework-espidf/components/esp_http_client/esp_http_client.c:517:59: error: 'esp_http_client_config_t {aka const struct }' has no member named 'client_cert_pem'
esp_transport_ssl_set_client_cert_data(ssl, config->client_cert_pem, strlen(config->client_cert_pem));
^
/Users/ashleywilkie/.platformio/packages/framework-espidf/components/esp_http_client/esp_http_client.c:517:91: error: 'esp_http_client_config_t {aka const struct }' has no member named 'client_cert_pem'
esp_transport_ssl_set_client_cert_data(ssl, config->client_cert_pem, strlen(config->client_cert_pem));
^
/Users/ashleywilkie/.platformio/packages/framework-espidf/components/esp_http_client/esp_http_client.c:520:15: error: 'esp_http_client_config_t {aka const struct }' has no member named 'client_key_pem'
if (config->client_key_pem) {
^
/Users/ashleywilkie/.platformio/packages/framework-espidf/components/esp_http_client/esp_http_client.c:521:58: error: 'esp_http_client_config_t {aka const struct }' has no member named 'client_key_pem'
esp_transport_ssl_set_client_key_data(ssl, config->client_key_pem, strlen(config->client_key_pem));
^
/Users/ashleywilkie/.platformio/packages/framework-espidf/components/esp_http_client/esp_http_client.c:521:89: error: 'esp_http_client_config_t {aka const struct }' has no member named 'client_key_pem'
esp_transport_ssl_set_client_key_data(ssl, config->client_key_pem, strlen(config->client_key_pem));
^
Compiling .pio/build/esp32dev/jsmn/src/jsmn.o
*** [.pio/build/esp32dev/esp_http_client/esp_http_client.o] Error 1

@awilkie1 Thanks for reporting this. Would you please help provide more details as suggested in the issue template? Information like elf, sdk configuration, backtrace, log outputs, commit ID, hardware and etc. would help us debug further. Thanks.

@awilkie1 Did you also update git submodules? Can you please share output for git submodule command?

So I did a platformIO update, which intern did an esp-idf update.
this then added a new sdkconfig file to my project src file. it had a sdkconfig.h file already. Could this be conflicting ??
git submodule gives me no output.

@awilkie1 We will need more information to comment or provide any help. I am not sure about which ESP-IDF version did PlatformIO upgrade to, but do you get same issue if you download latest ESP-IDF v3.3 from either github repository or from release artifacts?

@mahavirj we are using the IDF with the ADF framework. however, the IDF seemed to update and now has caused issues.
the idf is now on 3.30300.0

SOLVED.
Issues arose due to the EDP-IDF not compatible with the ADF! need to put "platform =<EMAIL_ADDRESS>in the platformio.ini file.
Thanks for your help!


Merit Order Capacities Chart doesn't scale properly

The chart doesn't scale properly when there are no oil and diesel plants present. As shown below

However, when oil-fired power plants /diesel generators are added, then the chart works fine:

Seems a little odd to this within the February deploy. Will pick this up, but if it's not of high priority I'm going to do this tomorrow.


Adding Structured output option to OpenAIGenerator

Proposed Changes:
Structured outputs are still a beta feature of OpenAI but they're super useful and I think we should provide an experimental extension to the already existent OpenAIGenerator. This PR adds this feature which calls .parse
How did you test it?
TBD
Notes for the reviewer
Can I have some help: I would like the model to default to gpt-4o-mini in the init...
Checklist

I have read the contributors guidelines and the code of conduct
I have updated the related issue with new insights and changes
I added unit tests and updated the docstrings
I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
I documented my code
I ran pre-commit hooks and fixed any issue

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

16 of 34 (47.06%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 93.53%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

haystack_experimental/components/generators/openai.py
14
32
43.75%

Totals

Change from base Build<PHONE_NUMBER>0:
-0.4%

Covered Lines:
3368

Relevant Lines:
3601

💛  - Coveralls


Add dynamic yaml example to MissingAnnotation component

Hey, I just made a Pull Request!
The MissingAnnotationEmptyState component will now be aware of the entity being viewed and use the details of the current entity to generate the YAML snippet that includes the annotations helper on it.

Closes #18264
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Not stale

Still waiting on review

Still waiting on review

Still waiting on review

@Rugvip can you review this?


Cleaning up the README a bit

Sorry but we're still supporting Ruby 1.8.7 so I can't merge it in since it has 1.9-only syntax

Argh, well that sucks. I forget people support 18 still, I've since moved mine all to 1.9 only! Anyways, thanks for your time to read it :)

@solnic yeah, we can drop 1.8 support once 1.9 is really solid across the primary ruby implementations. At the moment, I wouldn't use anything other than YARV in production for 1.9, since JRuby and RBX are still lagging.
Plus there's a real lack of tooling in 1.9 due to their choice of removing the ability to access the AST at runtime. That was a horrible choice IMHO and you can see how it's hurt the ruby community, very few new metrics or other kinds of tools have been ported to 1.9 in the last few years. I like being able to do things to my own code to get a birds eye view of how it's doing, and heckle has become an indispensable part of my toolkit.
Tools like mutant will help replace heckle, but that's still pretty new. The benefits that heckle brings to the development process are critical IMHO, and the reason why veritas is so solid, and I would not be willing to trade that just so we can use fancier syntax.

@solnic I almost forgot the main benefit we get from 1.8.7. It's a stable reference, and still (much to my dismay) is by far the most popular version of ruby in current use.
In theory rubyspec should make sure that all implementations of ruby behave the same, but in practice the semantics are all over the place, especially at the edges. If we limited the test matrix, we would lose some reference points and possibly write code to an implementation (with it's quirks) rather than the specification.
With that said, I do often code for one version of ruby and then test against the others as I'm getting ready to push. I often find implementation specific bugs; when writing veritas it was as high as 1-2 per week. I report these, and because I can confirm the code works on all of the supported rubies minus, say, JRuby in 1.9 mode, I can state with some confidence the implementation is buggy. Depending on the bug, sometimes we will work-around it, and other times the behaviour will be marked as pending until the implementation is fixed. The main thing is we don't let it to chance.
For a small project I guess it doesn't matter. Most gems are only used by a small handful of people, and even with a few hundred users, chances are most people aren't going to hit implementation specific bugs. Across DataMapper, and associated projects, there are over a thousand committers and several times more users. People find implementation bugs regularly, and I prefer to deal with those up-front as part of the development process and CI using (travis-ci)[http://travis-ci.org/#!/solnic/virtus].


Refactor fabric data in database

This PR separates the fabric data from the part data by introducing a new mapping YAML file which contains all information about the parts and devices. The new mapping will can also allow to easily generate a complete database some day by iterating over each entry.
Some scripts and makefiles got updated to handle the new location of the fabric.
Additionally, fixed two minor issue in Python.
Two patches in symbiflow-arch-defs are required: https://github.com/dnltz/symbiflow-arch-defs/commits/WIP/dnltz/refactory_database
Needs to be rebased after #1534 was merged.
Closes #1475

Overall this looks like a good first iteration.  Couple changes that I feel are needed going forward. I'll also kick off the full CI on this version to see if anything breaks.

Overall this looks like a good first iteration.  Couple changes that I feel are needed going forward. I'll also kick off the full CI on this version to see if anything breaks.

@litghost - Pushed a new, rebased version.

Moved the mapping files into the database (https://github.com/SymbiFlow/prjxray-db/pull/15) and changed the pathes to the DB variables.
Added an "export LC_ALL=C" to the utils/environment instead for replacing character which sets the localization settings to default.
Undid the settings renaming.

@litghost - Pushed a new, rebased version.

Moved the mapping files into the database (https://github.com/SymbiFlow/prjxray-db/pull/15) and changed the pathes to the DB variables.
Added an "export LC_ALL=C" to the utils/environment instead for replacing character which sets the localization settings to default.
Undid the settings renaming.

Looks like this is failing due to the Cleaning out current database?
========================================
Cleaning out current database
----------------------------------------
rm -f zynq7/*.csv zynq7/*.db zynq7/*.json zynq7/*.yaml zynq7/*.fasm
rm -f zynq7/timings/*.sdf
rm -rf zynq7/xc7*/
----------------------------------------

========================================
Running Database build
----------------------------------------
make --dry-run
source settings/zynq7.sh && make -C fuzzers
make[1]: Entering directory '/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/fuzzers'
.//run_fuzzer.py 000-init-db
Traceback (most recent call last):
  File ".//run_fuzzer.py", line 703, in <module>
    sys.exit(main(sys.argv))
  File ".//run_fuzzer.py", line 455, in main
    part = get_part_information(db_root, os.environ['XRAY_PART'])
  File "/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/prjxray/util.py", line 43, in get_part_information
    assert os.path.isfile(filename), \
AssertionError: Mapping file /tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/database/zynq7/mapping/parts.yaml does not exists
Makefile:87: recipe for target '000-init-db/run.xc7z020clg484-1.ok' failed
make[1]: *** [000-init-db/run.xc7z020clg484-1.ok] Error 1
make[1]: Leaving directory '/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/fuzzers'
Makefile:170: recipe for target 'db-zynq7' failed
make: *** [db-zynq7] Error 2

[ID: 1790215] Build finished after 135 secs, exit value: 1

Warning: Permanently added 'localhost' (ECDSA) to the list of known hosts.
[16:20:24] Collecting build artifacts from build VM
Build script failed with exit code: 1

Looks like this is failing due to the Cleaning out current database?
========================================
Cleaning out current database
----------------------------------------
rm -f zynq7/*.csv zynq7/*.db zynq7/*.json zynq7/*.yaml zynq7/*.fasm
rm -f zynq7/timings/*.sdf
rm -rf zynq7/xc7*/
----------------------------------------

========================================
Running Database build
----------------------------------------
make --dry-run
source settings/zynq7.sh && make -C fuzzers
make[1]: Entering directory '/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/fuzzers'
.//run_fuzzer.py 000-init-db
Traceback (most recent call last):
  File ".//run_fuzzer.py", line 703, in <module>
    sys.exit(main(sys.argv))
  File ".//run_fuzzer.py", line 455, in main
    part = get_part_information(db_root, os.environ['XRAY_PART'])
  File "/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/prjxray/util.py", line 43, in get_part_information
    assert os.path.isfile(filename), \
AssertionError: Mapping file /tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/database/zynq7/mapping/parts.yaml does not exists
Makefile:87: recipe for target '000-init-db/run.xc7z020clg484-1.ok' failed
make[1]: *** [000-init-db/run.xc7z020clg484-1.ok] Error 1
make[1]: Leaving directory '/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/fuzzers'
Makefile:170: recipe for target 'db-zynq7' failed
make: *** [db-zynq7] Error 2

[ID: 1790215] Build finished after 135 secs, exit value: 1

Warning: Permanently added 'localhost' (ECDSA) to the list of known hosts.
[16:20:24] Collecting build artifacts from build VM
Build script failed with exit code: 1

Looks like this is failing due to the Cleaning out current database?
========================================
Cleaning out current database
----------------------------------------
rm -f zynq7/*.csv zynq7/*.db zynq7/*.json zynq7/*.yaml zynq7/*.fasm
rm -f zynq7/timings/*.sdf
rm -rf zynq7/xc7*/
----------------------------------------

========================================
Running Database build
----------------------------------------
make --dry-run
source settings/zynq7.sh && make -C fuzzers
make[1]: Entering directory '/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/fuzzers'
.//run_fuzzer.py 000-init-db
Traceback (most recent call last):
  File ".//run_fuzzer.py", line 703, in <module>
    sys.exit(main(sys.argv))
  File ".//run_fuzzer.py", line 455, in main
    part = get_part_information(db_root, os.environ['XRAY_PART'])
  File "/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/prjxray/util.py", line 43, in get_part_information
    assert os.path.isfile(filename), \
AssertionError: Mapping file /tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/database/zynq7/mapping/parts.yaml does not exists
Makefile:87: recipe for target '000-init-db/run.xc7z020clg484-1.ok' failed
make[1]: *** [000-init-db/run.xc7z020clg484-1.ok] Error 1
make[1]: Leaving directory '/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/fuzzers'
Makefile:170: recipe for target 'db-zynq7' failed
make: *** [db-zynq7] Error 2

[ID: 1790215] Build finished after 135 secs, exit value: 1

Warning: Permanently added 'localhost' (ECDSA) to the list of known hosts.
[16:20:24] Collecting build artifacts from build VM
Build script failed with exit code: 1

@dnltz hasn't added the YAML files to this PR yet, which is why it failed.

Looks like this is failing due to the Cleaning out current database?
========================================
Cleaning out current database
----------------------------------------
rm -f zynq7/*.csv zynq7/*.db zynq7/*.json zynq7/*.yaml zynq7/*.fasm
rm -f zynq7/timings/*.sdf
rm -rf zynq7/xc7*/
----------------------------------------

========================================
Running Database build
----------------------------------------
make --dry-run
source settings/zynq7.sh && make -C fuzzers
make[1]: Entering directory '/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/fuzzers'
.//run_fuzzer.py 000-init-db
Traceback (most recent call last):
  File ".//run_fuzzer.py", line 703, in <module>
    sys.exit(main(sys.argv))
  File ".//run_fuzzer.py", line 455, in main
    part = get_part_information(db_root, os.environ['XRAY_PART'])
  File "/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/prjxray/util.py", line 43, in get_part_information
    assert os.path.isfile(filename), \
AssertionError: Mapping file /tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/database/zynq7/mapping/parts.yaml does not exists
Makefile:87: recipe for target '000-init-db/run.xc7z020clg484-1.ok' failed
make[1]: *** [000-init-db/run.xc7z020clg484-1.ok] Error 1
make[1]: Leaving directory '/tmpfs/src/github/symbiflow-prjxray-presubmit-db-zynq7/fuzzers'
Makefile:170: recipe for target 'db-zynq7' failed
make: *** [db-zynq7] Error 2

[ID: 1790215] Build finished after 135 secs, exit value: 1

Warning: Permanently added 'localhost' (ECDSA) to the list of known hosts.
[16:20:24] Collecting build artifacts from build VM
Build script failed with exit code: 1

@dnltz hasn't added the YAML files to this PR yet, which is why it failed.

@litghost @mithro - Sorry for the chaos! My understand was the database is cloned completely from prjxray-db. However, I added exceptions to the gitignore file for the yaml files and placed them in database//mapping and settings/.
Since the files are in the project now, kokoro should run :)

@litghost @mithro - Sorry for the chaos! My understand was the database is cloned completely from prjxray-db. However, I added exceptions to the gitignore file for the yaml files and placed them in database//mapping and settings/.
Since the files are in the project now, kokoro should run :)

Initial CI run looks good, just need to waiting for it to finish to see if we have any suprise.

Initial CI run looks good, just need to waiting for it to finish to see if we have any suprise.


Using Guice for dependency injection

In the current implementation we are reading API URLs from a json file and injecting it by hand in the service implementation. Guice will be best fit for this task

Finished implementing the integration.


Add support for batch creation with unique resources

The current batch method (which creates a RefBatch) takes references and clones if needed. In some cases, the user knows that the data is unique (e.g. he just created it), so he'd like to pass it by value and avoid the cache lookup.
I wonder if we can do it in a generic way separately for each resource:
enum MaybeUnique<'a, T> {
    Unique(T),
    Reference(&'a T),
}
trait ToMaybeUnique<T> {
    fn to_maybe_unique<'a>(self) -> MaybeUnique<'a, T>;
}
impl<T> ToMaybeUnique<T> for T {...}
impl<'a, T> ToMaybeUnique<T> for &'a T {...}

Note that we better do nothing than harm the ergonomics. A couple of saved allocations at the initialization stage may not be worth the hit.

Radically fixed in #757


Allow external config values

Was trying to use vault and/or another api/function in order to retrieve the values outside of local.settings.json and/or configuration settings variables w/in azure functions.  I assumed this might cause an issue, as I tried using this issue on how to override configuration startup config builder in order to add additional providers:
https://github.com/Azure/azure-functions-host/issues/4464
So, when that didn't work, I thought I could use an singleton class (one that doesn't rely on dotnet DI) in order to do instance = instance ?? new SingletonConfig(); which it could at least ensure that the property values i was pointing to in this class, would at least ensure it would call that external endpoint, retrieve those values, set the properties, etc.  Sadly, wasn't able to have any luck.  Not sure if there is a way to this already?  I saw there was this ticket, that was then closed
https://github.com/JamesRandall/FunctionMonkey/issues/73
Anyways, I appreciate any help and/or feedback that you (or the community) could give.  Best regar and thanks!

Closing this issue.  Wasn't truly thinking on this when i asked this question.  at the end, you would still have to have the funcitonmonkey.compiler in order to get the expected files that will work in azure functions.


mu-editor: init at 1.1.0-alpha.2

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

package request: #73975

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3165


[Instant wallet Balance] Balance doesn't update when the swap is claimed

Balance will only update when we get 1 conf, same for transactions list. This is because we are broadcasting via boltz lowball node and with lwk we are using blockstream server which is not connected to the lowball network. The have different mempools so they wont pick up the same 0 confs.


typo in powershell command

Good morning,
There seems to be a slight error in a powershell command listed in the article, as seen on the below screenshot :

The command should read Set-CsOnlineDialInConferencingTenantSettings
Typing the command without setting being plural results in the following error :

Thank you !
Cheers

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f93a7606-45b2-2032-c7b0-d79b07e65399
Version Independent ID: 39c20b9e-713f-bf2b-9c41-3945d64d07b1
Content: Manage the Audio Conferencing settings for my organization
Content Source: Skype/SfbOnline/audio-conferencing-in-office-365/manage-the-audio-conferencing-settings-for-my-organization.md
Service: skype-for-business-online
GitHub Login: @tonysmit
Microsoft Alias: tonysmit

@guillaumecouture I have fixed the type for the scrip so that it runs properly. Thank you again for catching this.


Update link to the network-mode page

The present link is a 404.
https://tanstack.com/query/v4/docs/guides/network-mode

I'm glad for everyone who tries to fix the broken links. What I would really like to have is relative links. linking to https://tanstack.com/query/v4/ will break if we go to v5!
I'm not entirely sure why the relative links are broken. It seems that going to ../guides/network-mode works. I'm not sure why though.
If you want, please replace all links that go to ./ with ../ + the respective slug. Also, there are some absolute links (those that start with /) that do not work and would need to be replaced.
If you want to contribute that, go ahead. If not, I'll close this PR. Thank you

I'll make a more complete PR. Just like you I could not completely understand why the relative link with ./Network-mode does not work.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

merged to Sandhya master


Fix create mount pod

fix #229

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@42ba742). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #230   +/-   ##
=========================================
  Coverage          ?   90.63%
=========================================
  Files             ?       16
  Lines             ?     1346
  Branches          ?        0
=========================================
  Hits              ?     1220
  Misses            ?       96
  Partials          ?       30

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 42ba742...10dbf2a. Read the comment docs.


Real classes always used in test instead of mocks

I've been reading about this library all day but have been unable to get an implementation working. Whenever I run my MainActivityTest it will always create a new instance of MainPresenter and not use my Mock. What am I doing wrong? I feel I have copied your example almost exactly.
The test file is here and you can see the rest in the repository.
Sorry for opening an issue. I thought it would be quicker than a SO question.

I managed to get it working by adding the DaggerMock library as a module and adding
androidTestCompile project(':daggermock')
to my build.gradle. Surprised no one else has this issue so there must have been a problem with my implementation

I already fixed this error but it wasn't available yet in version 0.6.4. I have release version 0.6.5 now, you can use it instead of including the module directly.

Confirmed. Thanks @fabioCollini


Access to DockerHub 'Contributed Repositories'

enhancement:  It's great to see the new "private repo view", but I would like to be able to access containers from an organization. These show up under "Contributed Repositories" in Docker Hub. It would be great to get that feature on the roadmap.

@ptepper Thanks for this suggestion. In 0.6 we didn't have support for organizations within Docker Hub. We will support it in the future - for sure.


Profile Page Not Working

The profile page is not working despite of logging. I have logged in with admin id and pw and the profile page is not at all working. It says page not found. Below I have attached some of the screenshots of it.

Please look to it if there is any typo in code.

Before accessing the profile page you need to create at least one session and one semester, that's why you're getting 404 error

I just fixed the 404 error, it should work fine now


fix typos in example

subj

Hello! You need to pass an array of containers, not a map. So your changes would cause an error, nevertheless, thanks for your PR.


Add an Optional Correlation ID to Experience Event

What are the schemas that are affected by the issue
https://github.com/adobe/xdm/blob/master/schemas/context/experienceevent.schema.json
What are examples of products that are impacted by the issue
Target, Analytics, ECID, Ad Cloud, Platform
Summary
The larger Experience Edge Project has a couple work streams which include Unified JavaScript and Unified Data Collection on a common Domain.  ( For details of that project see: https://wiki.corp.adobe.com/display/DMSArchitecture/Experience+Edge ) As this new unified JS library is sending data to Adobe rather than it's acquired solutions, it has an opportunity to fix a long standing issue of not being able to tie events (such as hits or page views) together into a single grouping.  For example in SPAs and mobile the concept of page loads or page views are fluid and there may not be a single point in time page load in the traditional sence. Another example is the A4T case where we do crazy gymanstics both client side and server side to tie a Personalization view to an Analytics Page View.  This this partcilar case the JS library is likely to send mutiple commuincations and mutiple events and yet the reporting is likley to want to group these together.  Ex: Top of page an ECID is requested and and authenticated state of the user is recorded, subsequently as personalization request for content is made, lastly a page view or page bottom call is made to record the visit and record which personalized experience or activity was visible.  Likely 3 commuincations or beacons were sent and yet there may be a desire to "stitch" these or group them together.   THis is just one of many examples.
To address these and other use cases, we are proposing an optional grouping or Correlation ID be added to the Experience Event that consumers of the ID can use to stitch or group events together both with Adobe and Non-Adobe products whenever two or more events should be correlated together.
For more Details see: https://wiki.corp.adobe.com/display/Activation/Universal+Event+Correlation

After receiving alpha customer feedback, we have decided to rename this field to something less confusing and more specific.  "Stitch ID" primarily to drive users toward it's original purpose of stitching hits together similar to the A4T type use cases and less about session or attribution type use cases.


execute it from jupyter notebook

hi, great work on implementing this algo.
I have a 2D dataset that i would like to cluster via denclue. However it seems that there are some issues while calling the algo. Is there any example on calling denclue?
Thanks!

What is your specific issue. I'm also trying to cluster lat/lon paris but encountered issue #3 .


Fun video window render.

In Vulkan API we can set the place on window where will be displayed vulkan render ! Cool !
https://www.youtube.com/watch?v=tjr691_oGns&feature=youtu.be
https://www.youtube.com/watch?v=3nZpfC3yXw0

Its amazing !
https://www.youtube.com/watch?v=RmFk9f5F5yo


removed an extra dnt check that always returned a warning whether col…

removed an extra dnt check that always returned a warning whether collectDnt was set or not

Thanks for creating this PR. What is the issue it solves? Does is show the DNT warning for people without DNT?

Thanks for creating this PR. What is the issue it solves? Does is show the DNT warning for people without DNT?

At the moment the attribute data-collect-dnt="true" on the script element has no effect.
Line 396-417 looks as follows:
// Don't track when Do Not Track is set to true
/** if ignorednt /
if (!collectDnt && doNotTrack in nav && nav[doNotTrack] == "1")
return warn(
notSending +
"when " +
doNotTrack +
" is enabled. See " +
docsUrl +
"/dnt"
);
/ else /
if (doNotTrack in nav && nav[doNotTrack] == "1")
return warn(
notSending +
"when " +
doNotTrack +
" is enabled. See " +
docsUrl +
"/dnt"
);
/ endif **/
The second if-statement will therefore always return a warning as long as nav.doNotTrack is 1, without checking if collectDnt was set.
Only the first if-statement should be needed, otherwise data-collect-dnt="true" will have no effect.

At the moment the attribute data-collect-dnt="true" on the script element has no effect.
Line 396-417 looks as follows:
// Don't track when Do Not Track is set to true
/** if ignorednt /
if (!collectDnt && doNotTrack in nav && nav[doNotTrack] == "1")
return warn(
notSending +
"when " +
doNotTrack +
" is enabled. See " +
docsUrl +
"/dnt"
);
/ else /
if (doNotTrack in nav && nav[doNotTrack] == "1")
return warn(
notSending +
"when " +
doNotTrack +
" is enabled. See " +
docsUrl +
"/dnt"
);
/ endif **/
The second if-statement will therefore always return a warning as long as nav.doNotTrack is 1, without checking if collectDnt was set.
Only the first if-statement should be needed, otherwise data-collect-dnt="true" will have no effect.

Do you have a live example of that? When I test I only get the warning when DNT is enabled. Do you see something else in your console?
This is a JSfiddle in a browser with DoNotTrack:

And this one without DoNotTrack:

I think something you should know. When you see /** if ignorednt / it will only be added to scripts that we export that support de ignore do not track feature. The second warn is for our light script. It doesn't have that ignore feature.
If this PR is based purely on our code, I think this explains it :)

Do you have a live example of that? When I test I only get the warning when DNT is enabled. Do you see something else in your console?
This is a JSfiddle in a browser with DoNotTrack:

And this one without DoNotTrack:

I think something you should know. When you see /** if ignorednt / it will only be added to scripts that we export that support de ignore do not track feature. The second warn is for our light script. It doesn't have that ignore feature.
If this PR is based purely on our code, I think this explains it :)

Yeah, I was just reading the code and not thinking of script exports. Sorry, my bad. :)

Yeah, I was just reading the code and not thinking of script exports. Sorry, my bad. :)

No worries, thanks for taking to time to improve our script! Appreciated.

No worries, thanks for taking to time to improve our script! Appreciated.


[Feature] Enabling Wayland over on Linux

Before Requesting

[X] I found no existing issue matching my feature request

Describe the feature you'd like!
A toggle in the BD settings to enable Wayland over on Linux, this could be done by adding the --enable-features=UseOzonePlatform and --ozone-platform=wayland flags to the Discord client.
From my testing, along with Discord's embedded Electron now bumped to version 22, It's stable enough to be used.
Anything else?
No response

There is currently no reasonable way of doing this. BD loads after Discord was already started, so in order to set such parameters we would have to trigger a relaunch of the application.
On top of this if somebody were to enable this option on a non-Wayland environment, Discord would no longer open at all, so the option would have to be turned off by modifying a settings file. So we'd have to implement it with added complexity to safe-guard against this.
Wayland support is also still an experimental electron feature, even if it's relatively stable in day-to-day use. But really the step of automatically selecting the appropriate render platform is something that should be implemented on chromium's end once it's a stable feature in their books.
The much simpler way which you'll have to resort to for the time being is adjusting the DesktopFile entry and adding these flags yourself in the Exec line there. Plasma and Gnome have graphical settings dialogs for this, for sway or alike you'll have to modify it somewhere in /usr/share/applications either directly in place or by copying it to ~/.local/share/applications and modifying it there. For more details on this refer to the Arch wiki

@QbDesu Would using the command line api in electron be insufficient to apply these options?

Going to close this as out of scope.


Update pg-xnamath-internals.md

Updated blog links to include AVX2 and ARM64 content

@walbourn : Thanks for your contribution! The author(s) have been notified to review your proposed change.


fixed GP interpolations

I made an effort to go through and fix other Stheno functions since gp_fast is no longer using Stheno, but I did not test them all.

It looks like the CI tests failed because you need to add KernelFunctions to dependancies.
Doing Pkg.add("KernelFunctions") should add it.  Then, it'll want you to add a compat line to the Project.toml specifying the version that it's know to work with.  Then commit and push the updated Project.toml, and there's a good chance it'll pass.

Yup, that worked!


Search Blogs  error

Please, replace the name of procedure "Search Posts By Term" to other name "SearchPostsBy Category" inside the function SearchPostsByCategory. (file SqlDataProvider_CMD.vb string 147 )
The function must be:
Public Overrides Function SearchPostsByCategory(moduleId As Int32, blogID As Int32, displayLocale As String, userId As Int32, userIsAdmin As Boolean, categories As String, searchText As String, searchTitle As Boolean, searchContents As Boolean, published As Int32, limitToLocale As String, endDate As Date, authorUserId As Int32, pageIndex As Int32, pageSize As Int32, orderBy As String) As IDataReader             Return CType(SqlHelper.ExecuteReader(ConnectionString, DatabaseOwner & ObjectQualifier & ModuleQualifier & "SearchPostsByCategory", moduleId, blogID, displayLocale, userId, userIsAdmin, categories, searchText, searchTitle, searchContents, published, GetNull(limitToLocale), GetNull(endDate), authorUserId, pageIndex, pageSize, orderBy), IDataReader)                     End Function
Thank to Author for the source code!

Done


fix: block global commands from dm

This is a PR for slash-command branch.
The Global Commands are available in DMs, so it should be blocked to avoid unintended issue when the i.GuildID is empty.

An individual app's global commands are also available in DMs if that app has a bot that shares a mutual guild with the user.
https://discord.com/developers/docs/interactions/application-commands#registering-a-command

If you have the plan to handle this or your code is under developing about this, feel free to reject this PR 🚀

Great point, I totally forgot about DMs! I think eventually it would be cool to respond to them, but for now you're totally right; makes sense to block them.


Update SDK for compose extension bot activity preview

Update SDK for compose extension response to support bot activity preview.

@robin-liao Is this still needed?

@robin-liao Is this still needed?

No. Closed it & deleted branch.


Rework map settings

Somehow "Starting area size" is only disabled when both "Enemy Bases" and "Angel's enemy multiplier" disabled??
Can we add a new Map Type: Sea Block? When type "Island" is selected, setting "Water" gets replaced with "Island Size" and "Coverage" is disabled.
Can we completely hide Enemy Expansion, Evolution? Setting them to nil causes error
default_enable_all_autoplace_controls doesn't seem to work? Maybe because the preset doesn't include any autoplace controls, just had the default ones.
Spawn worms only in a ring around starter area
Past the worms maybe get back to normal terrain generation. No ores though.
Map Generator Settings requests

Auto hide Resource tab if all autoplace controls are disabled. Ideally, this would apply to all tabs actually.
Ability to hide each setting. Particularly: water, moisture, terrain type, peaceful mode, enemy expansion, enemy evolution, pollution.
Keep starting area size active, even when Enemy bases is disabled. This setting affects more than just enemy bases.
Ability to add new named parameters / rename parameters. Like how map type Island replaces water with setting Island size. Can't be disabled and only Scale is enabled, not Coverage.

https://forums.factorio.com/viewtopic.php?f=28&t=92553

Would it be possible to force the research que on and maybe disable cliffs? A quick search revealed the following mod ixuPatchMapPreset.

Would it be possible to force the research que on and maybe disable cliffs? A quick search revealed the following mod ixuPatchMapPreset.

I've got a separate issue logged already for research queue #13
For cliffs, I don't want to disable them entirely. They should respect the map start up settings - like how worm spawning now does.

I've got a separate issue logged already for research queue #13
For cliffs, I don't want to disable them entirely. They should respect the map start up settings - like how worm spawning now does.

Somehow missed that one, great!
Yeah probably makes sense to leave players the option if they want to play with cliffs. It just doesn't make much of a difference in SeaBlock imo. There are so few cliffs, they could probably be disabled by default and hardly anyone would notice.

Somehow missed that one, great!
Yeah probably makes sense to leave players the option if they want to play with cliffs. It just doesn't make much of a difference in SeaBlock imo. There are so few cliffs, they could probably be disabled by default and hardly anyone would notice.

Idea: Make an Islands map type. Show popup message at start of game if cargo ships mod isn't enabled. Disable landfill.
Actually, this might need to be a separate mod.


Checked for emptiness of the list

Insert JIRA Issue number and update link
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] The layers in the kustomize folder have been updated accordingly.
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

Hi @Aakanksha0311 . What's the status of this PR please? Do we want to merge it or close it?
cheers,
Rob

Hi @Aakanksha0311 . What's the status of this PR please? Do we want to merge it or close it?
cheers, Rob

We can close it. Thanks for the mention.


web page down?

Short description
web page down?
...
What is current behavior
...
What is desired behaviour
...
Link to jsfidle/codepan with sample code
...
Steps to reproduce or sample
...

fixed. hosting issues. moved to cloud


New test for grep exercise

Check that "match entire lines" flag is correctly implemented.
As an alternative to this PR, you could wait until the https://github.com/exercism/x-common/pull/369 PR is merged, then re-create the grep exercise tests from that data. But I figured since I'd already written the F# version of the test, it costs almost nothing to do a PR in case that's faster than re-creating the tests from the grep.json file in x-common.

OTOH, the https://github.com/exercism/x-common/pull/369 PR now contains several new exercises, which I have not yet implemented in F#. So it might be better to close this PR and just wait for exercism/x-common#369 to be merged, then re-create the F# tests.

@rmunn https://github.com/exercism/x-common/pull/369/files has been merged, so we'll probably should be using that. I'll close this issue and created a new issue to update the exercise: https://github.com/exercism/xfsharp/issues/216.


TE-3.7 Base Hierarchical NHG Update

Tracks the effort to implement https://github.com/openconfig/featureprofiles/pull/2231 once it's merged.

fixed by https://github.com/openconfig/featureprofiles/pull/2604


Feat next setup

Set up the initial Next application structure with TailwindCSS. Configured basic settings and dependencies.
Implemented default styling and layout.

Hey I've just had a look, would you be able to sync your fork with Reece's most recent pr? There might be a couple conflicts with that and this one :)

Hey I've just had a look, would you be able to sync your fork with Reece's most recent pr? There might be a couple conflicts with that and this one :)

I'll get this done today.

I've added current api route and layout to client side.

Looks good!


Feature Request: Notification on song change like for Spotify

Apple Music/iTunes has a notification when the song changes:

Could we have a setting in preferences that would enable this for Spotify as well?

There is also another app for notifciations, written in swift: https://github.com/nahive/spotify-notify
It would be great if you could implement this feature into SpotMenu


Fails to build with llvm-project repository override

Click to expand!
Issue Type
Build/Install
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
2.9
Custom Code
No
OS Platform and Distribution
Linux Ubuntu 22.04
Mobile device
No response
Python version
3.10
Bazel version
5.3.0
GCC/Compiler version
11.3
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Following instructions from tensorflow/compiler/mlir/README.md with ["X86", "AArch64", "ARM"] in targets.bzl results in a build failure with:

cannot load '@llvm-project//:vars.bzl': no such file and referenced by '//tensorflow/compiler/mlir/tensorflow:tensorflow_test_passes'

Standalone code to reproduce the issue
LLVM_SRC=/home/ubuntu/src/llvm-project
LLVM_BAZEL_OVERLAY=/home/ubuntu/src/llvm-overlay
mkdir -p ${LLVM_BAZEL_OVERLAY}
python3 ${LLVM_SRC}/utils/bazel/overlay_directories.py \
    --src ${LLVM_SRC} \
    --overlay ${LLVM_SRC}/utils/bazel/llvm-project-overlay/ \
    --target ${LLVM_BAZEL_OVERLAY}
touch ${LLVM_BAZEL_OVERLAY}/BUILD.bazel ${LLVM_BAZEL_OVERLAY}/WORKSPACE
echo 'llvm_targets = ["X86", "AArch64", "ARM"]' > ${LLVM_BAZEL_OVERLAY}/llvm/targets.bzl

Relevant log output
INFO: Repository stablehlo instantiated at:
  /home/ubuntu/src/tensorflow/WORKSPACE:15:14: in <toplevel>
  /home/ubuntu/src/tensorflow/tensorflow/workspace2.bzl:960:28: in workspace
  /home/ubuntu/src/tensorflow/tensorflow/workspace2.bzl:81:14: in _initialize_third_party
  /home/ubuntu/src/tensorflow/third_party/stablehlo/workspace.bzl:11:20: in repo
  /home/ubuntu/src/tensorflow/third_party/repo.bzl:136:21: in tf_http_archive
Repository rule _tf_http_archive defined at:
  /home/ubuntu/src/tensorflow/third_party/repo.bzl:89:35: in <toplevel>
INFO: Repository nsync instantiated at:
  /home/ubuntu/src/tensorflow/WORKSPACE:15:14: in <toplevel>
  /home/ubuntu/src/tensorflow/tensorflow/workspace2.bzl:967:21: in workspace
  /home/ubuntu/src/tensorflow/tensorflow/workspace2.bzl:470:20: in _tf_repositories
  /home/ubuntu/src/tensorflow/third_party/repo.bzl:136:21: in tf_http_archive
Repository rule _tf_http_archive defined at:
  /home/ubuntu/src/tensorflow/third_party/repo.bzl:89:35: in <toplevel>
ERROR: /home/ubuntu/src/tensorflow/tensorflow/compiler/mlir/tensorflow/BUILD:1455:11: error loading package '@llvm-project//llvm': at /home/ubuntu/.cache/bazel/_bazel_ubuntu/b813f517b143a3c1665dd902035fe00f/external/llvm-project/llvm/config.bzl:8:5: cannot load '@llvm-project//:vars.bzl': no such file and referenced by '//tensorflow/compiler/mlir/tensorflow:tensorflow_test_passes'
ERROR: Analysis of target '//tensorflow/compiler/mlir:tf-opt' failed; build aborted:
INFO: Elapsed time: 59.508s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (84 packages loaded, 255 targets configured)
    currently loading: @llvm-project//mlir ... (3 packages)
    Fetching https://storage.googleapis.com/.../github.com/openxla/stablehlo/archive/51f005f0a8ff6e28f535adfec4de936cb4097aa4.zip
    Fetching https://storage.googleapis.com/mirror.tensorflow.org/github.com/google/nsync/archive/1.25.0.tar.gz

This happens because vars.bzl is created by the implementation function of llvm_configure() which is not called in the case of a repository override. I think the instructions should be modified to override llvm-raw instead which only requires creating a WORKSPACE and BUILD.bazel in the LLVM repo and does not need to create an overlay. I'm happy to provide a patch for that.

Hi, @RoboTux
Apologize for the delay and It seems like there is some issue with the local_repository rule in bazel is for external bazel repositories only. To use a non-bazel external repository, we need to use new_local_repository which takes build_file as an argument.
Bazel has the new_local_repository check this link https://bazel.build/reference/be/workspace#new_local_repository which allows you to use a local directory as a repo.
You will need to change the tf_http_archive(name = "llvm-project",...) section in tensorflow/workspace.bzl into new_local_repository, and add necessary BUILD files for llvm, mlir, and mlir/tests directory in your local llvm directory. you can check this discussion and stack overflow answer, I hope it will help you to resolve your issue.
If issue still persists please let us know ? or Could you please confirm if this issue is resolved for you ? Please feel free to close the issue if it is resolved ? Thank you!

Any reason why we don't override llvm-raw instead of llvm-project? When I tried it seemed to work fine and avoid the issues mentionned here. The patch I have is to change the instructions to override llvm-raw instead of llvm-project.

Hi, dude! Did u solve this problem? I ran into the same problem as you.
I followed the steps given in the readme, and roll back the version of llvm-project to corresponding version.
When I start build
 bazel clean --expunge

bazel build --override_repository=llvm-project=$LLVM_BAZEL_OVERLAY \
  -c opt tensorflow/compiler/mlir:tf-opt

bazel build --override_repository=llvm-project=$LLVM_BAZEL_OVERLAY \
  -c opt tensorflow/compiler/mlir:tf-mlir-translate

I got Error output below
ERROR: /private/var/tmp/_bazel_isolateya/af219aaee912b96bcf0876417404d886/external/llvm-project/mlir/BUILD.bazel:5904:11: error loading package '@llvm-project//llvm': at /private/var/tmp/_bazel_isolateya/af219aaee912b96bcf0876417404d886/external/llvm-project/llvm/config.bzl:8:5: cannot load '@llvm-project//:vars.bzl': no such file and referenced by '@llvm-project//mlir:Transforms'
ERROR: Analysis of target '//tensorflow/compiler/mlir:tf-opt' failed; build aborted:
INFO: Elapsed time: 38.684s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (122 packages loaded, 463 targets \
configured)
    currently loading: @com_google_protobuf// ... (4 packages)
    Fetching https://storage.googleapis.com/.../nsync/archive/1.25.0.tar.gz
    Fetching https://storage.googleapis.com/.../archive/v2.0.6.tar.gz
    Fetching https://storage.googleapis.com/...65082ee350509af1d113344d.tar.gz

ERROR: /Volumes/back/mlir_project/tensorflow/tensorflow/compiler/mlir/BUILD:219:13: error loading package '@llvm-project//llvm': at /private/var/tmp/_bazel_isolateya/af219aaee912b96bcf0876417404d886/external/llvm-project/llvm/config.bzl:8:5: cannot load '@llvm-project//:vars.bzl': no such file and referenced by '//tensorflow/compiler/mlir:tf-mlir-translate'
ERROR: Analysis of target '//tensorflow/compiler/mlir:tf-mlir-translate' failed; build aborted: Analysis failed
INFO: Elapsed time: 27.395s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (54 packages loaded, 247 targets configured)
    currently loading: @llvm-project//llvm
    Fetching @local_config_cc; Building xcode-locator
    Fetching https://storage.googleapis.com/.../abseil-cpp/archive/273292d1cfc0a94a65082ee350509af1d113344d.tar.gz

Standalone code to reproduce the issue
LLVM_SRC=/Volumes/back/mlir_project/llvm-project
LLVM_BAZEL_OVERLAY=${LLVM_SRC}/bazel # Note: this can be anywhere
mkdir -p ${LLVM_BAZEL_OVERLAY}
# This will symlink your LLVM sources with the BUILD files to be usable by Bazel.
python ${LLVM_SRC}/utils/bazel/overlay_directories.py \
    --src ${LLVM_SRC} \
    --overlay ${LLVM_SRC}/utils/bazel/llvm-project-overlay/ \
    --target ${LLVM_BAZEL_OVERLAY}
touch ${LLVM_BAZEL_OVERLAY}/BUILD.bazel ${LLVM_BAZEL_OVERLAY}/WORKSPACE
echo 'llvm_targets = ["X86","NVPTX","AMDGPU","AArch64","ARM"]' > ${LLVM_BAZEL_OVERLAY}/llvm/targets.bzl

Environment
bazel 5.3.0
openjdk <IP_ADDRESS>
python 3.9.16
llvm 15.0.5
OS mac(M2)

Hi, @isolateFeng
Apologize for the delayed response and May I know have you tried this workaround, If yes is it resolving your issue or not ? I see @RoboTux has submitted one PR 59942 to solve this issue so we'll wait to merge that PR, meanwhile if you did not follow above workaround could you please give it try ? Thank you!

Hi, @gaikwadrahul8
Sorry for replying so late. I did try this, but I'm not sure if I tried it correctly.
I modified the tensorflow/workspace.bzl(https://github.com/tensorflow/tensorflow/blob/master/tensorflow/workspace2.bzl), added the following content to it, but it did't work
tf_http_archive(
      name = "llvm-project",
      urls = [
          /Volumes/back/mlir_project/llvm-project
      ],
      build_file = str(Label("//third_party/llvm:llvm.BUILD")),
  )

I just don't know how to modify the  tensorflow/workspace.bzl  correctly, for I don't know anything about bazel

I saw RoboTux's  PR https://github.com/tensorflow/tensorflow/pull/59942 had  failed merge test, maybe I'll try it (qwq
or could someone give me an alternative method that I can try

I saw RoboTux's PR #59942 had failed merge test, maybe I'll try it (qwq
or could someone give me an alternative method that I can try

I don't think the failure is related to the patch since it fails to find patchelf and the patch only changes the documentation which does not require patchelf to build obviously. I'd be interested to know if the new instructions suggested in the patch fail to work for you.
Best regards,
Thomas

Hi, Thomas
I'm sorry to inform you that it(PR https://github.com/tensorflow/tensorflow/pull/59942) didn't work  for me ：(
it returns me the ERROR below
INFO: Found applicable config definition build:short_logs in file /Volumes/back/mlir_project/tensorflow/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /Volumes/back/mlir_project/tensorflow/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:macos in file /Volumes/back/mlir_project/tensorflow/.bazelrc: --apple_platform_type=macos --copt=-DGRPC_BAZEL_BUILD --copt=-w --define=PREFIX=/usr --define=LIBDIR=$(PREFIX)/lib --define=INCLUDEDIR=$(PREFIX)/include --define=PROTOBUF_INCLUDE_PATH=$(PREFIX)/include --cxxopt=-std=c++17 --host_cxxopt=-std=c++17
ERROR: no such package '@llvm-raw//utils/bazel': The repository's path is "llvm-raw" (absolute: "/Volumes/back/mlir_project/llvm-project/bazel") but it does not exist or is not a directory.
INFO: Elapsed time: 2.574s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)

I will try to try other ways I can find, thinks a lot
Best regards,
Tingfeng

I used the command git checkout 31c39439a894 in the llvm-project main folder ( download at 22:30 3.7.2023)

I used the command git checkout 31c39439a894 in the llvm-project main folder ( download at 22:30 3.7.2023)

What was the bazel command that you ran? Note: there is a mistake in my patch, the bazel build line should be LLVM_SRC not LLVM_BAZEL_OVERLAY

There have been too many things in the course recently, I sincerely apologize for my late answer
at the first I used
LLVM_SRC=/Volumes/back/mlir_project/llvm-project
touch ${LLVM_SRC}/BUILD.bazel ${LLVM_SRC}/WORKSPACE

# build
bazel build --override_repository=llvm-raw=$LLVM_BAZEL_OVERLAY \
-c opt tensorflow/compiler/mlir:tf-opt

I was so impetuous that I didn't notice the problem with the bazel build line
just now I tried
LLVM_SRC=/Volumes/back/mlir_project/llvm-project
touch ${LLVM_SRC}/BUILD.bazel ${LLVM_SRC}/WORKSPACE

# build
bazel build --override_repository=llvm-raw=$LLVM_SRC \
-c opt tensorflow/compiler/mlir:tf-opt

I get Error below
ERROR: /private/var/tmp/_bazel_isolateya/af219aaee912b96bcf0876417404d886/external/com_google_protobuf/BUILD.bazel:459:10: Linking external/com_google_protobuf/protoc failed: (Exit 1): cc_wrapper.sh failed: error executing command external/local_config_cc/cc_wrapper.sh @bazel-out/darwin_arm64-opt-exec-50AE0418/bin/external/com_google_protobuf/protoc-2.params
clang: error: invalid linker name in argument '-fuse-ld=-debugger-tuning=lldb'

I did install llvm, lldb --version will output lldb version 15.0.5
the full build info
INFO: Found applicable config definition build:short_logs in file /Volumes/back/mlir_project/tensorflow/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /Volumes/back/mlir_project/tensorflow/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:macos in file /Volumes/back/mlir_project/tensorflow/.bazelrc: --apple_platform_type=macos --copt=-DGRPC_BAZEL_BUILD --copt=-w --define=PREFIX=/usr --define=LIBDIR=$(PREFIX)/lib --define=INCLUDEDIR=$(PREFIX)/include --define=PROTOBUF_INCLUDE_PATH=$(PREFIX)/include --cxxopt=-std=c++17 --host_cxxopt=-std=c++17
INFO: Analyzed target //tensorflow/compiler/mlir:tf-opt (369 packages loaded, 25834 targets configured).
INFO: Found 1 target...
ERROR: /private/var/tmp/_bazel_isolateya/af219aaee912b96bcf0876417404d886/external/com_google_protobuf/BUILD.bazel:459:10: Linking external/com_google_protobuf/protoc failed: (Exit 1): cc_wrapper.sh failed: error executing command external/local_config_cc/cc_wrapper.sh @bazel-out/darwin_arm64-opt-exec-50AE0418/bin/external/com_google_protobuf/protoc-2.params
clang: error: invalid linker name in argument '-fuse-ld=-debugger-tuning=lldb'
Target //tensorflow/compiler/mlir:tf-opt failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 447.115s, Critical Path: 66.04s
INFO: 6768 processes: 4338 internal, 2430 local.
FAILED: Build did NOT complete successfully

Thank you for all your help
Best regards,
Tingfeng


New LogBox won't disable warnings with 'console.disableYellowBox'

Description
After enabling the experimental/unstable new logbox with
require('react-native').unstable_enableLogBox();
the warnings won't be disabled when setting
console.disableYellowBox = true;
The reason for this is that there many warnings generated even by RN itself and I'd like to ave them hidden
React Native version:
$ react-native info
info Fetching system and libraries information...
System:
OS: macOS Mojave 10.14.6
CPU: (8) x64 Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz
Memory: 715.46 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 11.12.0 - /usr/local/bin/node
Yarn: 1.7.0 - ~/.npm-packages/bin/yarn
npm: 6.13.6 - ~/.npm-packages/bin/npm
Watchman: 4.7.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.8.4 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
Android SDK:
API Levels: 28, 29
Build Tools: 28.0.2, 28.0.3, 29.0.3
Android NDK: 20.1.5948944
IDEs:
Android Studio: Not Found
Xcode: 11.2.1/11B500 - /usr/bin/xcodebuild
Languages:
Python: 2.7.16 - /usr/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.11.0 => 16.11.0
react-native: 0.62.0 => 0.62.0
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce

Enable the unstable api: require('react-native').unstable_enableLogBox();
Try to disable the warnings: console.disableYellowBox = true;

Expected Results
The Yellow box (small toast in the new version) should not be there
Snack, code example, screenshot, or link to a repository:
n/a
Thank you 😻

Here is a temporary workaround in case someone needs:
import { YellowBox } from "react-native";
YellowBox.ignoreWarnings([""]);

That disables all warnings that start with an empty string, in other words, all warnings.

It looks like console.disableYellowBox has been deprecated in favor of LogBox.ignoreAllLogs(). Can you try using that API instead?

@hramos I tried doing the same, but apparently, LogBox isn't exported yet, if I try to import it import { LogBox } from 'react-native' it comes back undefined.

For some reason YellowBox.ignoreWarnings(['']); completely turns off my error box, when I raise an exception I see a white screen, if I uncomment this line, the error is shown with the new log box as expected.

Will this be available in react-native 0.63, that we can import LogBox and disable just the yellow warnings?

Hi fellas, any luck on this, I also can't seem to import LogBox from 'react-native' ???

Hi fellas, any luck on this, I also can't seem to import LogBox from 'react-native' ???

Are you using react-native 0.63? The docs indicate that this should work: https://reactnative.dev/docs/debugging#logbox

@nicoburns Yh currently using 0.63.1 and trying to ignore/disable some warnings but can't seem to import them

ReactNative.LogBox.ignoreAllLogs(true);
In React-Native 0.63, this will prevent annoying yellow warning box in screen.

ReactNative.LogBox.ignoreAllLogs(true);
In React-Native 0.63, this will prevent annoying yellow warning box in screen.
You can add above line in root app function.

Can confirm that this works on 0.63^
import {LogBox} from 'react-native'
LogBox.ignoreAllLogs(true)

ReactNative.LogBox.ignoreAllLogs(true);
In React-Native 0.63, this will prevent annoying yellow warning box in screen.
You can add above line in root app function.

Can confirm that this works on 0.63^
import {LogBox} from 'react-native'
LogBox.ignoreAllLogs(true)

yeah,  my current React-Native version is 0.63.2

import {LogBox} from 'react-native'
LogBox.ignoreAllLogs(true)

Does not work for me on 0.63.2
The import does not work.

I'm on RN 0.63.3, and cannot import LogBox from 'react-native'.

ReactNative.LogBox.ignoreAllLogs(true);
In React-Native 0.63, this will prevent annoying yellow warning box in screen.
You can add above line in root app function.

Can confirm that this works on 0.63^
import {LogBox} from 'react-native'
LogBox.ignoreAllLogs(true)

anyway to disable specific log like RCTBridge required dispatch_sync .... ?

anyway to disable specific log like RCTBridge required dispatch_sync .... ?

Here is an example of how to ignore warning "Non-serializable values were found in the navigation state":
LogBox.ignoreLogs([
  "Non-serializable values were found in the navigation state",
  // name of the error/warning here, or a regex here
]);

Guys, be careful where the error occurs, more precisely in which screen because the warning may appear after you have ignored it.
I don't understand why it's not global, but for me that was the problem.

This error is shown in console, even with ignoreLogs specific, global, etc. even with ignoreAllLogs(true)
VirtualizedLists should never be nested inside plain ScrollViews

Calling LogBox.ignoreAllLogs(), LogBox.ignoreAllLogs(true), and LogBox.ignoreLogs('') doesn't seem to work in react-native 0.64.3. I tried making the calls globally at the entry of my app code.

@bpringe How did you replace console.warn? I have tons of useless warnings being logged to my console regarding packages I have no control over and it's making it impossible to debug usefully using the console.

@kingdomstrategies I'm using ClojureScript but you can probably understand what I did even if you aren't familiar with ClojureScript.
(defonce warn js/console.warn)
(set! js/console.warn
      (fn [& args]
        (when-not (str/includes? (first args) "React Components must start with an uppercase letter.")
          (apply warn args))))

Basically, create a new variable and assign console.warn to it to store the original function, then redefined console.warn to a new function that only delegates to console.warn if the message isn't one you want to filter out.
Let me know if you need more help with a pure JS/TS solution.

LogBox.ignoreLogs is not working

Temporary solution.
ignoreWarnings.js
import { LogBox } from "react-native";

if (__DEV__) {
  const ignoreWarns = [
    "EventEmitter.removeListener",
    "[fuego-swr-keys-from-collection-path]",
    "Setting a timer for a long period of time",
    "ViewPropTypes will be removed from React Native",
    "AsyncStorage has been extracted from react-native",
    "exported from 'deprecated-react-native-prop-types'.",
    "Non-serializable values were found in the navigation state.",
    "VirtualizedLists should never be nested inside plain ScrollViews",
  ];

  const warn = console.warn;
  console.warn = (...arg) => {
    for (const warning of ignoreWarns) {
      if (arg[0].startsWith(warning)) {
        return;
      }
    }
    warn(...arg);
  };

  LogBox.ignoreLogs(ignoreWarns);
}

App.js
// import at the very top of everything.
import "../ignoreWarnings";

Sad that the actual LogBox doesn't work correctly and we have to hack it through!

Temporary solution.
ignoreWarnings.js
import { LogBox } from "react-native";

if (__DEV__) {
  const ignoreWarns = [
    "EventEmitter.removeListener",
    "[fuego-swr-keys-from-collection-path]",
    "Setting a timer for a long period of time",
    "ViewPropTypes will be removed from React Native",
    "AsyncStorage has been extracted from react-native",
    "exported from 'deprecated-react-native-prop-types'.",
    "Non-serializable values were found in the navigation state.",
    "VirtualizedLists should never be nested inside plain ScrollViews",
  ];

  const warn = console.warn;
  console.warn = (...arg) => {
    for (const warning of ignoreWarns) {
      if (arg[0].startsWith(warning)) {
        return;
      }
    }
    warn(...arg);
  };

  LogBox.ignoreLogs(ignoreWarns);
}

App.js
// import at the very top of everything.
import "../ignoreWarnings";

This worked for me, you're a legend for this

Recommended Solution
LogBox added two new methods for controlling logs:

LogBox.ignoreLogs(["pattern to ignore"]): Use this when you want to ignore a specific list of warnings (such as when a library is causing a warning that you cannot fix). It is not recommended to use this for warnings that you do have control of fixing.
LogBox.ignoreAllLogs(): Replaces console.disableYellowBox. Use this to temporarily disable all warnings in development. It is not recommended to check this in as it will disable warnings for your entire app.

Also, many of the the warnings in the above Temporary Solution from @SohelIslamImran should not be ignored. These warnings include deprecation warnings which should be fixed so that your code is ready to upgrade to a future versions that removes the deprecations. Ignoring these warnings will result in your code throwing later.

But LogBox.ignoreLogs(["pattern to ignore"]) doesn't work. @rickhanlonii

But LogBox.ignoreLogs(["pattern to ignore"]) doesn't work. @rickhanlonii
x2

@SohelIslamImran @kuriel-trivu do you have a repro I can look at? We have tests for this behavior here so I need to look at the code in order to see what's going on.

@kingdomstrategies I'm using ClojureScript but you can probably understand what I did even if you aren't familiar with ClojureScript.
(defonce warn js/console.warn)
(set! js/console.warn
      (fn [& args]
        (when-not (str/includes? (first args) "React Components must start with an uppercase letter.")
          (apply warn args))))

Basically, create a new variable and assign console.warn to it to store the original function, then redefine console.warn to a new function that only delegates to console.warn if the message isn't one you want to filter out.
Let me know if you need more help with a pure JS/TS solution.

Thank you very much for this suggestion. Worked right way.
IgnoreLogs is still not working in my cljs react native app with react native version 0.71.2

How to make it ignore 'VirtualizedLists should never be nested' and not send it to the logs?

@bmitioglov You can try the above-proposed solution/hack. https://github.com/facebook/react-native/issues/28453#issuecomment-1152412853
But If that doesn't work, it means the warning is coming from a library you are using and the LogBox cant reach it for some reason.

I suspect it's not working, since hermes doesn't allow to monkey-patch console. This temporary didn't work for me neither.

Great, worked like a charm
Thanks @Dror-Bar :)


No support for filters hosted on GitLab

Observation:
Regolith doesn't support the downloading of filters that are hosted on a GitLab instance (both public and private repositories
Problem:
This seems to be caused by the construction of the download URL, which is done as follows:
url := fmt.Sprintf("%s//%s?ref=%s", f.Url, f.Id, repoVersion)
Proposed solution:
Might be a bit tricky, because just including a check like:
if url contains 'gitlab': ...
is not really the way to go. More in-depth analysis is required.

Seems like a lot of issues can be fixed by forcing the protocol
https://github.com/hashicorp/go-getter#forced-protocol

It might've been fixed in https://github.com/Bedrock-OSS/regolith/commit/dc0734428f1e8b3d0cc4b9e4bb11312f55288c8a
Please let me know whether it works on this commit.

Closing due to inactivity


In TypeScript code, never bind JSDoc normally, just set parent pointers

Fixes #16554
The fixes to #14752 (#14997) #16358 (#16413), and #16359 (#16416) were overly complex and specialized.
The only reason we needed to bind JSDoc in TypeScript code was to set parent pointers. This adds a function to do only that. (I started binding JSDoc in #15856 and should have just done this PR in the first place.)
Since we will now never reach a JSDoc node in the normal binder in TypeScript code, we can remove a bunch of checks for isInJavaScriptFile (undoing the previous specialized fixes to the above issues).

please port this to release-2.4


Fix issue with OS.execute always silencing stderr

https://github.com/godotengine/godot/blob/8d117b214f2bcd14015532fdfbed9b3f059da0e6/core/bind/core_bind.cpp#L452
This function calls the bottom function
https://github.com/godotengine/godot/blob/8d117b214f2bcd14015532fdfbed9b3f059da0e6/drivers/unix/os_unix.cpp#L312
Issue here is that OS.execute doesn't actually send out a value to stderr. The default value for stderr is false, which will invoke this bottom line and effectively always silence stderr.
https://github.com/godotengine/godot/blob/8d117b214f2bcd14015532fdfbed9b3f059da0e6/drivers/unix/os_unix.cpp#L315
Some programs, such as FFmpeg, as is my use case for my current game, will print out to stderr with valuable and vital information. (I don't know why FFmpeg prints to err instead of out, don't blame me)
The only workaround I was able to achieve at the moment was to have Godot open a shell and have that shell then run my command, though this only seems to work for me on Linux.
The fix I provide allows users the option to have stderr not get silenced which I would very much appreciate.

It looks like AppVeyor pulled the object file from cache instead of rebuilding it for some reason.
I pulled the SubSage:patch-1 branch and successfully built it on Windows with MSVC 2015.

/core/bind/core_bind.h:217:6: error: default argument missing for parameter 5 of 'int _OS::execute(const String&, const Vector<String>&, bool, Array, bool)'
  int execute(const String &p_path, const Vector<String> &p_arguments, bool p_blocking, Array p_output = Array(), bool p_read_stderr);
      ^~~~~~~

Please squash commits together, see PR workflow.


Add set-elem navigator

This is the set analogue to map-key. Used to change the value of a particular element in a set.
Open question as to what the behavior should be if the element does not exist in the set.

Implemented in master.


Meitrack Commands

I try to implement the commands on meitrack devices
in the Message.js
I add

Message.commands = {
GET_PICTURE : '@@O48,863835020683608,D00,151021062113_C1E19.jpg,0*DB',
};

in the Tracker.js
I add

Tracker.prototype.getPicture = function(message) {
this.connection.write(Message.commands.GET_PICTURE);
};

in the commands.js
I add

tracker.getPicture(function(err, data) {
if (err) {
console.log('GET PICTURE ERROR: ', err);
} else {
console.log('GET PICTURE : ', data);
}
});

I can build without errors but when I try to use and send the commands
MEITRACK_GPRS_Protocol

Example

GPRS Sending @@O48,353358017784062,D00,0215080432_C2E03.jpg,0*DB\r\n
GPRS Reply The example cannot be displayed because of hexadecimal characters.
I take by example the nodejs-meiligao
I'm not sure if I forget something.
please help me

@sherome, it seems you are trying to use quite different protocol. This nodejs module implements Meiligao protocol, when you are trying to use Meitrack one.

now I had problems transmission commands to gps device
Can you help me ? Gps device does not return data

@sherome could you tell me your device model?

T333
http://www.meitrackusa.com/trackers/gps-vehicle-trackers/t333-3g

@sherome unfortunately, I can't help you - your device does not support this protocol. It is not intended to work with this module.

I adapted your module. I can receive data


EventErrorPayload has not attr  code

    payload = EventErrorPayload(**payload_data)  | TypeError: __init__() got an unexpected keyword argument 'code'
@dataclass
class EventErrorPayload(EventPayloadBase):
data: str
sees:    /wechaty_puppet/schemas/event.py

Can you give more info about the error?

token type
event_name
more context info

So I can help you fix this problem.

tokenType: puppet_paimon_********
receive  event <EventErrorPayload(code=36, ...)
receive error info <{'code': 36, 'msg': 'The text (贾玲) is banned'}>
EventErrorPayload has no attr "code" or msg.
code status 36 sometime 31.  you should debug code status  from where.

now,  I directly edit event.py,  add  some fields(code, msg, data.....) avoid exception
/Users/hiro/Library/Caches/pypoetry/virtualenvs/dichat-Yf_2revS-py3.8/lib/python3.8/site-packages/wechaty_puppet/schemas/event.py

This problem has been fixed by the new commit for wechaty-puppet. So you can upgrade the wechaty-puppet package to avoid this issue.


Add Zerotier One Package/Justfile

Describe the package
I would really like to see Zerotier support added to the image. In fact, I was planning to make a pull request myself, however I didn't know how you all want that done. There are two ways I see that it could be added. First, I can just add the package to the package list and create a justfile to enable the service. Second, much along the lines of ollama, I could add a justfile which creates a podlet. I really like that second option, however there are some issues/questions I see.

The default directory is /var/lib/zerotier-one, I do not know if you would like that moved somewhere else
Zerotier has an accompanying zerotier-cli which interacts with the zerotier socket. If the above directory is changed, the justfile would probably need to create a proxy script which passes everything into the podlet, or changes the location of the socket through environment variables, if that is even possible. If the directory isn't changed, is there a way to install only the cli, and would it even work? I know the ollama justfile requires you to install the cli through brew, but Zerotier doesn't have a cli-only downlod (as far as I know).
There would be no support for global override, I believe. I have only used this feature once, and did not seem to work through a container, however that could be user error on my end

Image
Ubuntu

Pull requests are welcome.

Pull requests are welcome.

Well, yes, that is what I was planning to do. I just wanted clarification on how you want those issues handled.

Package or container

And if you want it in a container:

Location of data
Cli

I'll write a pr using a podlet and a proxy cli script to see if that's accepted, unless you all have a different way of wanting that handled

If needs to run a rootful container, that would give me pause but can be setup in a more secure manner. I run rootful quadlets utilizing unprivileged namespaces.
If it can run as rootless container a quadlet would be awesome to see.
I would lean against the layered approach unless you have something that it will specifically tie into.
For the cli tool, see if there is a scriptable way to install that. Having a window pop up for accept is not the end of the world.


As a user, I want to be able to add new tags to the system

Tag additions handled in a separate view to course addition


Fix meta description so that embeds are properly formatted [H]

Currently we have a placeholder description that will show up if someone copy/pastes the URL into Discord (and likely also in Twitter, Slack, etc).

We should audit this and craft the right meta data so that the project logo and description show up correctly.
Side note: the title is also repeating needlessly on the homepage.

The home page title should be OpenFGA | Fine Grained Authorization

Note that in Docusaurus, each page's title in metadata will be " | ".  This is not something that is customizable at this moment.
As such, the home page title should be
The home page title should be Fine Grained Authorization | OpenFGA instead.