Support multiple content types

I had errors of Unimplemented, Content with not exactly 1 option. and I thought I will quickly fix this.
The change was done as part of https://github.com/golemcloud/golem/pull/1049

CLI can do serde_json::from_str(a_yaml_file) and use the the REST API with content-type as json itself. So this is not for golem-cli to work with latest changes that support YAML, but rather make this client generation successful without compromising the details in the spec (refer the other PR). That is, in the PR, golem-service.yaml has details of API-definition import that says it supports both xml and json.

With this new spec, the client generation failed with an error mentioned above. This PR is just enough reasonable changes to get over this problem. Currently it supports multiple content type for 1 single endpoint  handling only yaml or json. If anyone needs more , they can add. Hopefully this scenario is not going to come often.

The changes are tested and now works: the branch is in use in PR https://github.com/golemcloud/golem/pull/1049

TODO

[ ] Cleanup

Part of #1035


Interactive Tutorial - Deploying an App

In this module the deployment is created however Port is empty.
I the next module the deployment is created for you, and then there is a Port assigned
how does that happen?

I think --port=8080 is missing, right?
Just trying to follow the tutorial in my own k8s deployment.

/kind support


Roll Flutter Engine from 4c868ee85616 to 6925ce32a67e (1 revision)

https://github.com/flutter/engine/compare/4c868ee85616...6925ce32a67e
2024-08-03<EMAIL_ADDRESS>Roll Skia from 60c660ecdf0a to 8f0e1486b887 (1 revision) (flutter/engine#54331)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Editor can edit the document using a GUI

[ ] Editor can bold text using a GUI
[ ] Editor can italise text using a GUI
[ ] Editor can set different heading for text using a GUI
[ ] Editor can create a table using a GUI

This could be closely dependent on using the codemirror

This could be closely dependent on using the codemirror and depended on #228


Save button

A save button for the rendered iframe would be wonderful.

Do you mean save as file and download?

Yes

That sounds useful :P Will add this later.

Mabe we can use jszip and file-saver .

Hi @egoist, is this issue still open?


Local cache key is not matching correctly on release branches

Checking through the release branch CI and its clear that the release branches are pretty much never getting cached runs
The reason appears to be that the cache keys dont match correctly
In pre/postsubmit (ie pr/land) the keys are eg

instance_name: instances/release/v1.24
instance_name: instances/v1.24

release tags get yet another - these runs should all be using the same cache ideally

cc @mum4k


VS 2015 Glass fixes

Host was mistakenly taking a hard dependency on the existance of a shell dll. This moves the dependency to be a soft dependency.
On the machine I was running things at least, there was a different NDK, and the .apk files where dropping to a different place than when AndroidTest.cmd expected. This made AndroidTest.cmd a bit more forgiving.

@chuckries please review


Shortcut Ctrl+Alt+Q used elsewhere

Ctrl+Alt+Q shortcut is used by Atom's Reflow Selection under the edit menu which seems to override open-in-browser setting.

Can confirm, happens to mee too.


HTTP/2 priority fixes to match common browser patterns

Now that HTTP/2 priorities are no longer crashing, we did some short experiments activating in production.  However, that resulted in some customer-side latency problems which I have been digging into through the past week.  I have been concentrating on exercising ATS via a continuous streaming site.  I have been taking advantage for browser developer hooks to simulate slower network speeds.
I found a paper which gave a nice overview of browser priority strategies https://www.researchgate.net/publication/324514529_HTTP2_Prioritization_and_its_Impact_on_Web_Performance . Chrome does a chain of priorities (first come, first served).  The session window limit seems to be hit first.  Firefox uses priority holder nodes to set up 4 or 5 buckets.  Safari seems to do a naive round robin approach where all priority nodes are direct children of the root.  My testing has concentrated on Firefox and Chrome.
The PR adds debug messages to dump the state of the priority tree as JSON output which has been very useful for tracking down these issues.
This PR has fixes for two problems.

For Chrome, I would occasionally see the priority chain be broken.  Originally, the new direct descendent would be a shadow node.  Then I took out the shadow node addition unless the parent_id was in the future and it would just be that new stream as the direct root descendent.  Since the point value of the new node was lower than the others, it would unfairly get a higher priority of the bandwidth (based on the weighted-fair queuing logic) starving out the older nodes and violating Chrome's first come first served policy.  The problem is that the parent of the new node got finished and removed from the priority tree before the stream id request appeared.  I saw this mostly with a high bandwidth client.  I addressed this issue by adding a small circular buffer of ancestors, so if the parent is missing, we can look to see where that node originally plugged in.  It isn't perfect since we are not keeping a complete history, but once I made this change, I no longer saw the problem.  I think in practice you will only have one or two parents missing before the next stream request comes.

For Firefox, we are inserting nodes in the priority tree for priority frames, but we were treating them as shadow nodes.  So once their last direct descendent was deleted, the priority node was also deleted losing the buckets for future requests on the session.  This is not desirable for the Firefox strategy since we are relying on the priority direct root descendants as weight buckets.  So rather than using the fact that there is no associated Http2Stream node to identify a priority node as a shadow (true for shadows and priority frames), I added another flag to explicitly mark a node as a shadow.

I am still a bit unclear on the use of shadow nodes.  I have commented out a couple of the tests until we reach a conclusion on what the shadows should be.  I did some searching based on our original behavior of creating shadow nodes for unknown parent and found  this discussion https://github.com/http2/http2-spec/issues/764 where Lukasa declares that the right way to do this is to make the new stream for the missing parent and do default priority processing. But in their example, they have stream 6 depending on stream 7 (presumably a future stream). I looked into how nghttp2 does it (which is declared to be the right way) and indeed they create a new node for the missing parent with default weight and root it at the root node.  Based on that discussion, I am assuming that shadow nodes are only to represent nodes that will come in the future (e.g. parent id > new stream id).
We are still in the process of testing these changes, but I wanted to get more eyes on this.  Particularly since I was not involved in the original priority code development.
I also need to augment the unit tests to model the chrome and firefox scenarios.  I hope to test some android apps.  It would also be good to test some iOS apps, but I don't have access to an iOS device.

[approve ci autest]

I'll deeply take a look later. At first glance, this approach looks good.

For Chrome...The problem is that the parent of the new node got finished and removed from the priority tree before the stream id request appeared. I saw this mostly with a high bandwidth client.

This could happen. We have to remove the finished node to keep the tree depth shallow ( 256 is current maximum depth ). Tracking ancestors looks reasonable.
Anyway, we should add this scenario to the unit test (Http2DependencyTree_Chrome_51).

For Firefox, we are inserting nodes in the priority tree for priority frames, but we were treating them as shadow nodes. So once their last direct descendent was deleted, the priority node was also deleted losing the buckets for future requests on the session.

I agree we should leave the node even if the children and queue is empty. The shadow flag make sense.

Based on that discussion, I am assuming that shadow nodes are only to represent nodes that will come in the future (e.g. parent id > new stream id).

Practically, this assumption could be true. But theoretically, this could be false. Because below case looks allowed.

client skip stream 1 ( client can't use this stream in the future )
client open stream 3 depends on stream 1

Another thing I noticed in the code about shadow nodes,  when we add a node and see there is already a shadow node in place, we don't update the shadow node's parent and wait based on the new stream criteria.

Thanks for the comments.  I'll take a pass through and update the PR.  And update/add tests.
I had a chance to run my test client on my Android phone via a box running these changes.  And the android app is doing a naive bushy priority tree like Safari.
It would be interesting to get some priority tries from iOS apps.
I assume that the Priority strategies could be unique by App, but I assume there are only a handful of priority strategies in the while based on the standard set of app communication libraries like okhttp.

Pushed a new version to address @masaori335's comments, reactivate shadow tests, and add new tests.
I'm guessing there will be more tweaks moving forward as we see more browser and app priority schemes, but these changes move us in the right direction.

@shinrich This will need a new PR against the 7.1.x branch please


Local deployment issues

To ease the deployment of the tool locally, we can make the following improvements:

Provide a docker file
Specify the versions of the tools used for installing/deploying (e.g. npm and node)
Provide exemplary curl commands

I added a Dockerfile to the root directory of the repository and wrote the instructions on how to use it in README. Dockerfile currently works only in Linux and Windows operating systems and does not work in macOS because of the Puppeteer related issues.
I also added recommended Node.js and npm versions that can be used during the deployment of the service as a note in README.
Finally, I detailed the existing curl and fetch API example in README with real data to make it more understandable.


Add code for (part of) Thunderbird's autodiscovery mechanism

This only adds the code, but doesn't yet make use of it from the setup Activity.

Any ETA on when this code will end up in a release? It has been merged quite a while ago, but so far is only on master and not officially released yet.

@kbabioch AFAIK this code is just a part of Thunderbird discovery so even on master the discovery is not completely wired and wouldn't work as-is.


SDL2_ttf: 2.0.15 -> 2.0.18

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg eval

Have any of the downstream consumers been tested with this change?

I've just tried mepo and flare and both doesn't seem to have any problem.

We found one new package with a problem: https://github.com/NixOS/nixpkgs/pull/157455#issuecomment-1031183256


Notify assignee when reminder time is reached

Details will be added as soon as someone wants to take over the exercise

I want to try doing this

Go for it!
On May 10, 2017 13:12, "Ben Adrian Sarmiento"<EMAIL_ADDRESS>wrote:

I want to try doing this
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prooph/proophessor-do/issues/130#issuecomment-300375700,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYEvKDH-iAeKnt34_CvCazA1T5-XXHYks5r4UcpgaJpZM4M-D0n
.

Go for it!
On May 10, 2017 13:12, "Ben Adrian Sarmiento"<EMAIL_ADDRESS>wrote:
I want to try doing this
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prooph/proophessor-do/issues/130#issuecomment-300375700,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYEvKDH-iAeKnt34_CvCazA1T5-XXHYks5r4UcpgaJpZM4M-D0n
.

Hello guys,
I'm interested to give a try to that exercice. Is there additional instructions somewhere ?

@shulard sorry, I need to check the current implementation first. We have the script send_reminder_notification.php. Looks like we did not close the issue.
Let me check that at the weekend.

Oh ok 👍
No problem, I'll take the next one. The idea to learn by doing on that kind of stuff is really awesome 😄. Let me know !

@shulard I checked the history. Indeed, the feature was already added: https://github.com/prooph/proophessor-do/commit/a0208f7a2906d6ab8ed53d797ec69dfe241adec6
It was bonus task of the original reminder feature.
I cleaned up the list of open tasks. At the moment only github login is left.
Let me know if you want to take this one (complex) or try another one. I'll add new tasks the next days. But need to collect some new ideas first ;)


Update the data model and API logic to include a new "title" field for the UserInput entity

Related User Story
#120
Related Tickets
#148 #162 #160
Acceptance Criteria

The UserInput data model includes a "title" field.
The related API endpoints accept and return the "title" field for UserInput objects.
The database schema is updated to store the "title" field for UserInput records.
The backend services and data access layer handle the "title" field correctly.
The API documentation is updated

@zhiyi15 This is the original ticket for updating the userinput schema. I just assigned it to you.


update/unpin/enable python37/38/39

Make sure that this package works with python versions 3.7, 3.8, 3.9
Summary:

updated the dev environment to use Python39
fixed tests that failed with Python>37
one test is now skipped on CI as it would hang (see #195)

merging  this in order to enable testing ilastik with all deps with py373839


ToCode Solution: תרגול קבצים

01 is ok.
02 works but you could have solved this without creating line1 and line2 outside of the 'while' loop. That would create a more readable solution. Would you like to give it a try or shall I close this exercise?

Giving it a try !
On Wed, Dec 21, 2016 at 12:00 PM, Gabor Szabo<EMAIL_ADDRESS>wrote:

01 is ok.
02 works but you could have solved this without creating line1 and line2
outside of the 'while' loop. That would create a more readable solution.
Would you like to give it a try or shall I close this exercise?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tocodeil/python-course-examples/pull/126#issuecomment-268482885,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXJ9X631U1CL53Ed89pnqFDX6kYV8gDkks5rKPikgaJpZM4LSwY2
.

Is that what you meant ?
On Wed, Dec 21, 2016 at 12:42 PM, anat shapira<EMAIL_ADDRESS>wrote:

Giving it a try !
On Wed, Dec 21, 2016 at 12:00 PM, Gabor Szabo<EMAIL_ADDRESS>wrote:

01 is ok.
02 works but you could have solved this without creating line1 and line2
outside of the 'while' loop. That would create a more readable solution.
Would you like to give it a try or shall I close this exercise?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tocodeil/python-course-examples/pull/126#issuecomment-268482885,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXJ9X631U1CL53Ed89pnqFDX6kYV8gDkks5rKPikgaJpZM4LSwY2
.

almost. You don't even need the two lines at the top
line1 = " "
line2 = " "

though that would require further changes to the loop inside.

like this ?
"""
ex02.py
merge 2 files into a third file
"""
import sys
continue_reading = 1
(_, file1, file2, file3) = sys.argv
with open(file1,'r') as f1:
with open(file2,'r') as f2:
with open(file3,'w') as f3:
while continue_reading == 1:
line1 = f1.readline()
line2 = f2.readline()
f3.write(line1)
f3.write(line2)
if line1 == "" and line2 == "":
continue_reading = 0
On Wed, Dec 21, 2016 at 1:02 PM, Gabor Szabo<EMAIL_ADDRESS>wrote:

almost. You don't even need the two lines at the top
line1 = " "
line2 = " "
though that would require further changes to the loop inside.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tocodeil/python-course-examples/pull/126#issuecomment-268495703,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXJ9X5dBkCKLGWq5bzaWUt4sHtDqK_r0ks5rKQdbgaJpZM4LSwY2
.

yes, except that you don't need another variable 'continue_reading'.
Instead you can have an infinite loop while True
and then call break when when you are done with the loop.

👍
On Wed, Dec 21, 2016 at 1:17 PM, Gabor Szabo<EMAIL_ADDRESS>wrote:

yes, except that you don't need another variable 'continue_reading'.
Instead you can have an infinite loop while True
and then call break when when you are done with the loop.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tocodeil/python-course-examples/pull/126#issuecomment-268498421,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXJ9X5JIHpGx-atNH46j0nwxjX1UCA55ks5rKQrjgaJpZM4LSwY2
.


feat(templates): adding external templates

Fix for #2086
Types of changes
What types of changes does your code introduce to Graphback?

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] Other (please specify)

Checklist

[x] I have read the CONTRIBUTING document.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
I've created a community-templates.json file in the root directory. Here people can specify their templates which they want to appear in the CLI. Then in the starterTemplates.ts file I import these templates as externalTemplates and then create the externalTemplatesArray which I merge with the already existing allTemplates array.
I didn't know how to go about testing this so I haven't done that. Will write documentation for this once the feature is completed and tested. I also had to add
{
  "compilerOptions": {
    "resolveJsonModule": true,
    "esModuleInterop": true
  }
}

in the tsconfig.json in order to import json files.
Please let me know if you have any feedback or suggestions. Thanks.

Hey @wtrocki! I've tried to split the templates into categories in the CLI by adding an external property ( just like the disabled one already present ). I then choose the templates which get loaded into the displayedTemplates variable based on whether this property is set to true or not. Please let me know if you have any other suggestions.
Thanks!
P.S. I've let a sample template still remain in community-templates.json. Will remove that once I write the docs.

Really good work. I will verify that tommorow

Thanks a lot! Will look into why the build is failing once we've decided on the way to show the templates.
Thanks for reviewing! :)

Graphback supported templates:
-
-
_

Community templates
-
_
_
``

Like new display style. Would review PR today.

Hey! My build keeps failing each time because of the way things are imported in some files which aren't even relevant to this PR. Am I doing something wrong? Should I go and fix all these imports? Please let me know. Thanks.
For example:
github and request were imported in starterTemplates.ts but I had to change the way they were imported otherwise the build would keep failing. Similarly for DataLoader in the CRUDServices.ts file.
@wtrocki @craicoverflow

I took a look into it but there seem to be some issues with the lining etc.
I will check that this week and merge! Thank you so much!

@RinkiyaKeDad I'm worried that we cannot accept change due to numerous issues.
I think we should

Keep the file close to the code and keep it as .ts file rather than json (Using json forces us to change compilation rules that we do not want to do.
Add information into contributing guide how to add new community template

I have done some work on https://github.com/aerogear/graphback/pull/2226 so feel free to change and based on this. If you require any help or advice give us shout.
You can ignore website builds. They will be gone once we rebase this and update dependencies.

@RinkiyaKeDad are you ok to continue working on this. We might actually need it to push our own templates at some point

@wtrocki yeah definitely. I've written a draft documentation. What are the next steps?

We need templates in TS file like current templates.
JSON outside package will not work when publishing package.

Oh I was under the impression that we would be merging the PR you opened so I didn't change from json to a TS file in this branch. I've done that now. Please have a look and let me know the further steps. Thanks a lot!

See my comment here: https://github.com/aerogear/graphback/pull/2215#issuecomment-739413263

Got it. So should I also bring over the other changes you made in your PR to this one?

Like we talking about 5 minutes changes. You can simply start from scratch and just reapply docs and some of the changes from this PR. Closing this PR feel free to open new one.


Warnings and issues when baking in 2019.1.3f1

I am getting these warnings repeatedly, when the bake completes the navmesh is not viewable and does not appear to have baked correctly.
Internal: JobTempAlloc has allocations that are more than 4 frames old - this is not allowed and likely a leak
To Debug, enable the define: TLA_DEBUG_STACK_LEAK in ThreadsafeLinearAllocator.cpp. This will output the callstacks of the leaked allocations
Internal: deleting an allocation that is older than its permitted lifetime of 4 frames (age = 14)
Internal: deleting an allocation that is older than its permitted lifetime of 4 frames (age = 15)

was using the components incorrectly, I was putting a navmesh surface on every mesh instead of creating one mesh for all surfaces.


collides on character bug

Okay, situation goes like this; spawn a player character in a scene, give it movement, switch scene via the door, switch back (both via this.scene.start() and then you can no longer walk over the intial spawn tile. I've ran a test with collides: false on the player char, and it lets me, which leads me to think that theres some weird ghost being left in place or something?
Img for clarity:

Thanks for telling 👍  I will take a look at this.

Can you provide some code around it, that helps me to reproduce the error? A simple this.scene.start() seems to work but this will reset the characters position.


How to run demos?

Could you please also provide a demo python file? I would like to run the model on my own test images.

Hi there,
Thank you for your attention on our work!
You can run demo by using op.py.
If you want to evaluate our model on the custom dataset, then you might need to add some lines to load it in dataselector.py
It could be done easily, since there are some reference codes regarding to how we load our dataset in the same python file.

Could you please also provide a demo python file? I would like to run the model on my own test images.

Hi, I have the same question, could you tell me how to inference on my own images?


[Snyk] Security upgrade cookie from 0.5.0 to 0.7.0

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

package.json
yarn.lock

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Cross-site Scripting (XSS) SNYK-JS-COOKIE-8163060
  59

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.


Problem with "subfolders" and blacklist channels

Hello, so i saw that divided was now dead and so i added the "subfolders" and i wrote this line (check pic)

so that it download to the folder that already existed before the new changes except even with the change i've made, he use the default line which is

"subfolders": [ "{{fileType}}" ],

So i was wondering if i've messed up somewhere or something? Also i think the Blacklist channel setting doesn't work because i've added some channel to the blacklist and the program still download them

Ok i've found my mistake. It's because in here https://github.com/get-got/discord-downloader-go/wiki/Settings i saw the subfolders settings in the channels brackets but i didn't saw it was also outside and so if you put the subfolders settings in the channel bracket it will only apply to the channels brackets and not the whole if you use Category or Server  and so it will result in the default setting for subfolders "subfolders": [ "{{fileType}}" ],
So for people who will or have made the same mistake as me, DO NOT PUT the settings of the subfolders in the Server, category, channel bracket. Just put it outside and you are good to go
I still have a problem with the blacklist channel

The 403's and 404's aren't something I can help. It may have cached thumbnails on your Discord but if the source has since 403/404'd then there's nothing more the bot can do.
I'll look into the history since ranges. This has been a long time issue that I've tried to fix several times. The latest fixes on that seemed to resolve it for most people but I may need to add more checks.
The subfoldering should be working when placed in sources (server/category/channel). I can't really determine the issue without seeing the settings setup and a description of what did and didn't work for that setup. Same goes for the blacklisting issue.

Alright i see for the 403/404
Here is my settings setup:

the subfolders was in the channel where the "divideby" was and before it was working even tho at some point it started ignoring "dividedby" idk why.. But yeah thats my settings

That's not how you use "all". "all" is basically it's own source config to apply to everything, like the groups of configs within the [ ] of servers/categories/channels. If you wanted to download absolutely everything the bot has access to, you'd use "all" within the global scope, not within servers/categories/channels, then you'd use "allBlacklistChannels" as a separate line outside of "all", not within it.
I'm not sure what you're trying to do, since you're trying to blacklist channels within a config of specific channels?
If you're trying to block channels for the server config, add a list like [ "1", "2" ] to the server group called "serverBlacklist" and the same for categories called "categoryBlacklist". Also don't have "channels" and "channel" present with each other, delete the blank "channel" line.

Given what you had said about subfoldering before, I'm thinking you're misunderstanding how these settings groups work.
You seem to be trying to put things in the "channels" section thinking that will make those settings apply to every "channel" the bot encounters, thinking that also applies to servers and categories. That's at least what I'm gathering from this setup and what you said about subfoldering.
Each { } group is an individual configuration. "servers" and "categories" and "channels" and "users" are all the same thing. They're lists of separate source configurations but they follow the same structures. The only difference between them are specifiers for "channel"/"channels", "server"/"servers"/"serverBlacklist", etc. "all" is also the exact same as this, it's just a fallback configuration for anything the bot encounters that doesn't belong to an established "servers"/"categories"/"channels"/"users" config. For most (or maybe all, idk) settings in the source config groups, there's global equivalents that act as defaults. When you don't have them specified within "servers"/"categories"/"channels"/"users", they'll use those values. That's why your subfoldering started applying to servers and categories properly once you moved it to the global scope/area.
Also don't have "channel" and "channels", "server" and "servers", "category" and "categories" present together within one config. It may not be causing issues since you have it blank but still, it should only be the IDs you're actually targeting.
This may help explain more, here's a dummy copy of your settings showing how they should be structured:
{
	"subfolders": [ "{{serverName}}", "{{channelName}}", "{{fileType}}" ],
	"allowCommands": false,
	"sendErrorMessages": false,
	"updatePresence": false,
	"reactWhenDownloaded": false,

	// these settings will be used for ALL messages UNLESS that message belongs to a separate config below.
	// In that case, these settings aren't used, only that configs settings.
	"all": {
		"destination": "non-configured downloads",
		"ignoreBots": true,
		"saveOtherFiles": false,
		"filters": {
			"blockedUsers": [ "1", "2" ]
		},
		"subfolders": [ "{{serverName}}", "{{categoryName}}", "{{channelName}}", "{{fileType}}" ]
	},
	"allBlacklistChannels": [ "5", "6" ],

	"servers": [
		{
			"servers": [ "1", "2" ],
			"serverBlacklist": [ "11", "22" ],
			"destination": "servers"
		}
	],

	"categories": [
		{
			"servers": [ "1", "2" ],
			"categoryBlacklist": [ "11", "22" ],
			"destination": "categories",
			"filters": {
				"blockedUsers": [ "1", "2" ]
			}
		},
		{
			"servers": [ "3", "4" ],
			"categoryBlacklist": [ "33", "44" ],
			"destination": "categories 2"
		}
	],

	"channels": [
		{
			"channels": [ "1", "2" ],
			"destination": "channels",
			"subfolders": [ "{{fileType}}" ],
			"saveVideos": false
		},
		{
			"channels": [ "3", "4" ],
			"destination": "other channels",
			"reactWhenDownloaded": true
		},
		{
			"channels": [ "1", "2" ],
			"destination": "channels 3",
			"filters": {
				"blockedUsers": [ "1", "2" ]
			}
		}
	]
}

yeah basically i was putting channels and channel - servers and server - categories and category when i needed them but then i saw that sometimes i was switching between them often so i decided to let them in the config and fill the blank when i needed them but yeah i added them and i had no issue previously.
For the blacklist, basically my goal is that sometimes i take a whole category but there will be one or two channels that i didn't want to download and i wanted the bot to ignore them when i tried to download the whole category without them if not it would be adding all the channels i wanted which was annoying. And basically i thought that "serverBlacklist" and "categoryBlacklist" was only for servers IDs and categories IDs that's why i thought "allBlacklistChannels" was the one to block the channels IDs i didn't wanted to use.
I will change some of the stuff then, thanks


[feat]: Memory inclusion feature added.

Docs link: https://docs.mem0.ai/features/selective-memory#memory-inclusion

opened a better PR #1894


support 256x256 images

Dear developer,
Thanks for your perfect project.
Does the model support 256x256 or 512x512 inputs?

Have you managed to figure out what input does model supports?


kubectl -n kube-system and kubectl --all-namespaces, differ in behavior

What keywords did you search in Kubernetes issues before filing this one? (If you have found any duplicates, you should instead reply there.): kube-system all-namespaces

Is this a BUG REPORT or FEATURE REQUEST? (choose one): Bug?

What you expected to happen:
Both to exhibit the same behavior, whether -n is used or --all-namespaces is used
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:

@steward-yu should this be fixed to have universal behavior?

In my opinion, you should follow command line rules: -n is kubectl command options, but --all-namespaces response to kubectl <commandname>. In other word, -n and --all-namespaces  are use in different level. @naisanza

I agree with @stewart-yu.
Having ... kubectl --all-namespaces create ... makes no sense, while kubectl -n some-namespace create ... does.

@naisanza I'd be happy to know why you disagree? :-)

the flags must come after the command to which they apply. --all-namespaces belongs to the get command. -n and --namespace are global options
/close


Port IncludeExtensions

NOTE: Must be an andculture employee to work on this issue due to requiring access to private resources.
From the LMS project, port over the IncludeExtensions static class used for building out include property strings. While it has unit tests, please review during the port and see if it requires more.
Once done, we will want to refactor this and other projects to leverage the new extension method approach.

Transferred this since the classes don't have any requirement for EF, the only dependency is on the Include model which also makes sense to live in this Core repository.


General way to construct a pod template in a deployment spec

Our operators should start the deployments.  Every operator should generate a deployment that mounts the application config.  Hopefully, we can build a generate function for this that can be reused for the other deployments

Hey team! Please add your planning poker estimate with Zenhub @richscott @kannon92 @Sharpz7

Please add your planning poker estimate with Zenhub @dejanzele


Formatter bug

This code:
object[foo: "foo",
  bar: "bar"
]

After first format, gets formatted to: (note the extra comma after "bar")
object[foo: "foo",
  bar: "bar",
]

After second format:
Syntax Error: expected named argument, not ]
Crystal version:
Crystal 0.23.1 (2017-07-13) LLVM 4.0.1

This is a bug about the parser because trailing comma in index notation should be parsed also. I fixed this in #4784. Thank you for reporting 👍


Enable full testing suite

This applies the commit fixing https://github.com/racket/racket/issues/2137 as a patch so that the full Racket test suite can be run when building the conda package.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Interestingly, the testing issue goes away with their latest master, so some additional recent change apparently contributed to the fix. Given that this isn't a major issue, I'll just wait until their next release to include the additional tests here.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Alright, it looks like the fix for the test failure that shows up here wasn't actually included in the 7.0 release, It's just in master currently. The easiest/simplest solution is to wait for another release.


Add a Country Option to Email Sign Up

Currently only the Email Address is all that is required to be submitted on the email sign up form on the home and video page. We are working to include a required Select Country in the sign up form to enable email regionalisation for our marketing communications.
UI will mock this up, but the existing backpack component will need to be modified.
PENDING: MOCKUPS

https://github.com/lonelyplanet/backpack-ui/issues/222


Set version 3.0.0-RC5

@roman-mazur could you release, please?

Have you guys tested with snapshot this time?

yes)

released


Sync external Device with D435

Required Info

Camera Model
D435

Firmware Version
05.10.13

Operating System & Version
Win 10

Platform
PC

SDK Version
2.25.0

Language
python

Segment
others

Issue Description
Hello,
i have a passive hw sync between two D435 cameras. Now i need to sync a external device to the D435 cameras, but the external device require a 5v pulse. So my question is, how can i shift the 1.8v  level pulse from the hw sync between the two D435 to the required 5V TTL pulse?

Another user did this 1.8 to 5v conversion with a 'voltage shifter'.
https://github.com/IntelRealSense/librealsense/issues/4574

Hi ArniBarni
There is a lot of breakout boards in the market, i.e. https://www.digikey.com/product-detail/en/sparkfun-electronics/BOB-11771/1568-1208-ND/5673794?WT.srch=1&gclid=EAIaIQobChMIyZbYi8TW5QIVleNkCh3eOArMEAkYBCABEgLcAvD_BwE
If you can do some simple HW job, you should be able to DIY.
Thanks
**Please be informed that we will close this ticket in seven (7) days if we have not heard back from you.


fix(core): ensure setTimeout delay value in machine action not overflowing

Closes #
📝 Description
The actions function would be executed immediately if the delay of actions was specified to exceed the max range.
This PR fixes this issue by set the maximum value for setTimeout if exceeded value set.
⛳ Current behavior (updates)
If set the exceeded value, setTimeout's handler is executed immediately.
(strictly, the overflow results in faster execution than specified.)
🚀 New behavior
If set the exceeded max value, setTimeout's handler is executed after max delay time.
💣 Is this a breaking change (Yes/No):
No
📝 Additional Information
setTimeout’s timeout value is defined as long type(interface), and long type’s range is [−2147483648,<PHONE_NUMBER>] (Web IDL).
If it is exceeded, it will be overflowing and set value as 0 in many case. (converting algorithm)
As a result, the handler of setTimeout will be executed immediately.

Thanks for this PR. I understand the intention behind this and it make sense.
I really believe this is something that should be handled outside the machine core logic.
Anyone building a machine on top of Zag should handle this use case by configuring their delays to not exceed the overflow.

@segunadebayo
Thank you for your comment.
I understand that you don't want to include this kind of logic in the machine core, so I will try to handle it on the user side.


fix: types via JSDoc

It seemed from "files" and tsconfig.json we wanted to include type definitions. This adds a setup similar to the one used in podium-lib. hubro.d.ts and fixup.js lets us extend for instance the FastifyReply and FastifyRequest interfaces should we need to.

With npm link I can type a route function in userland like this.
/**
 * @param {import('hubro/request').default} request
 * @param {import('hubro/response').default} response
 */
export async function GET(request, response) {

}

Thanks a lot!
Yes we do want to provide typings. Main focus to provide that have not been the main focus yet so thanks a lot for this.


Add Code Institue (Ireland)

https://codeinstitute.net/
Ireland
1st Floor, Block 1, Blackrock Business Park, Carysfort Ave, Blackrock, Co Dublin Email<EMAIL_ADDRESS>Phone: 353 1 539 79 73

@Immobilise Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Add a line to docs

This is a very small PR, but one that I hope saves others.

Adding an explicit warning to users that Flux ignores JSON files.

@xtellurian Can you rebase onto the latest head of master, and git push --force-with-lease please? There shouldn't be any conflicts. (Usually I would just do this and push to the source branch of a PR, but you've created this from your master, so I'm reluctant to do so in this case.)


Add ability to filter test results list (/scenario/items) by environment

Is your feature request related to a problem? Please describe.
Add ability to filter test results list (/scenario/items) by environment.
Describe the solution you'd like
Add an option via a text field or pulldown or additional API endpoint parameter to select from the existing environments for the specific test scenario.  That way the graph showing the average response times can be compared for a single environment only, or all environments if none was specfiied.

Hi, I like the idea of adding a filter for the scenario view, although it includes at least two endpoints - one for the listing and the other for the charts(scenario trends). And most likely new endpoint is going to be needed for the list of environments.

👀
https://user-images.githubusercontent.com/13610612/235139742-b67a3c6f-918e-4d9e-875c-aae2c259b87a.mov

That looks great to me! When do you expect to merge it in to main?

I need to add some tests, but around next week.

Hi @ludeknovy,
Finally got around to upgrading to 4.8.5 today. Just wanted to thank again for adding this environment filter!

Hi @slash-zero You're welcome. Feel free to share another improvement idea you might have.


Slow when switching from a large file to a small one

Every time I switch from a large file (20,000+ lines) to another file, VS Code freezes for 3-5 seconds (even longer if I'm debugging).
I tried disabling all the extensions and enabling them one by one, and it seems like BracketPair is causing the issue.
About Visual Studio Code:

Version 1.16.0
Shell 1.7.3
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

I now only color the first 5000 lines


ed25519 key derivation

This PR introduces derivation package which provides functions for ed25519 key derivation as described in SLIP-0010 in Go. This is compatible with ed25519 key derivation used in hardware wallets (like Ledger, Trezor).
Derivation path (m/44'/148'/x') for Stellar has been initially agreed in https://github.com/stellar/stellar-protocol/issues/61 and will become SEP soon (this PR will be updated with correct SEP number). EDIT: SEP is here: https://github.com/stellar/stellar-protocol/pull/63
This PR and SEP will allow generating hierarchical deterministic Stellar accounts from a single seed (which can be generated using mnemonic code, like in BIP-0039). This should improve key storage (hardware wallets and storing master seed as 12-24 words instead of long strkey-encoded strings) and moving keys between wallets and apps.
@stanford-scs you are probably the main person who should check it out.

Yeah, I don't the the name neither but could find anything better.

why don't we call the package slip0010 ?

why don't we call the package slip0010 ?

The problem with this name is that slip0010 describes methods for other curves and we only implement one. But let's consider this when moving this outside exp.


Failed to import - d3

Failing module

GitHub: https://github.com/d3/d3
npm: https://npmjs.com/package/d3

import {schemeObservable10} from "https://esm.sh/d3";

Error message
After onload I got this:
Uncaught SyntaxError: The requested module 'https://esm.sh/d3' does not provide an export named 'schemeObservable10' (at test.html:4:9)

Additional info
The built version of d3 currently resolves the d3-scale-chromatic dependency to 3.0.0, but the latest release of d3-scale-chromatic is 3.1.0.
<EMAIL_ADDRESS>
esm.sh version: N/A
Browser version: N/A

Unfortunately this is still an issue and extends to other packages that pull in d3.
For d3 itself a workaround appears to be to append a cache breaker, e.g. https://esm.sh/d3@7.9.0?foo. In another case I was able to circumvent the cached dependency by adding ?bundle-deps: https://esm.sh/gh/observablehq/plot@v0.6.14?bundle-deps&keep-names

Same here, running:
import * as Plot from "https://esm.sh/<EMAIL_ADDRESS>Gives:
Uncaught SyntaxError: The requested module<EMAIL_ADDRESS>does not provide an export named 'schemeObservable10' (at schemes.js:51:3)

Thanks @mootari for the workaround:
import * as Plot from "https://esm.sh/@observablehq/plot@0.6.16?bundle-deps";


Correct display of colors in a dark theme

When using a dark theme, many colors are displayed quite differently from how they actually look, for example, yellow in a dark theme, and how yellow really is:

I know that color inversion is used in a dark theme, but it might be better to just change the default colors, because it's very difficult to find the color you want, for example, I haven't picked up yellow yet (the photo was taken in the editor).

At the same time, the standard colors look quite good in a dark theme, so for comparison, the colors in a light theme are the same without inversion in a dark one, and they are the same but the way Excalidraw displays them:

The default colors look more saturated.

Black color becomes white in dark theme


SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes

Laravel Version: 5.4
Voyager Version: 0.11.9
PHP Version: 5.6
Database Driver & Version: mysql

Description:
Receiving the following errors in console when attempting to install Voyager:
[Illuminate\Database\QueryException]
SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes (SQL: alter table translations add unique translations_table_name_column_name_foreign_key_locale_unique(table_name, column_name, forei   gn_key, locale))
[Doctrine\DBAL\Driver\PDOException]
SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes
[PDOException]
SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes
Steps To Reproduce:
php artisan voyager:install
I have also tried adding Schema::defaultStringLength(191); to AppServiceProvider::boot(), which did not correct the issue.

This is the only solution that worked: https://github.com/the-control-group/voyager/issues/901#issuecomment-291470960

Scratch that, all you have to do is set the mysql engine to InnoDB ROW_FORMAT=DYNAMIC

After adding Schema::defaultStringLength(191); to AppServiceProvider::boot(), You need to change
'engine' => 'InnoDB', in config/database.php in mysql.
It is working for me !

Please don't common on very old closed issues.


Overview / drawing of VM, clusters, multus networks, and how NFs connect to them

Vish taking the lead

I added a free5gc-packages repository for our various free5gc kpt packages we will need for the workshop. This is an additional upstream "external" (read only) repository that will need to be registered with Porch along with nephio-packages.

I'm going to try to answer some of the @vishwanathj questions with the current implementation of the provision scripts

NOTE: Given that this is on-progress any of this values can changed.

How many VMs are we going to be using? At the moment, discussions are to have a single VM

The terraform PR indicates one single instance deployed in GCE

What would the VM operating system, version and architecture?

I suggest to use Ubuntu 20.04 LTS (Focal) but KinD images and Terraform scripts use Ubuntu 22.04 LTS (Jammy), so maybe we can adjust this. The CPU arch is x86_64 (IMHO dealing with other architectures should be out of the scope for now)

What would be the configurations (CPU, RAM, Storage, networking) of the VM?

For the CI tasks, it's used 2vCPUs, 7 GB of RAM and 14 GB of SSD space. One single network interface.

How many KIND clusters are we going to be creating?

The dynamic list provision 3 KinD clusters.

What would be a version of KIND that is stable that we would want to deploy?

We're using the latest stable version

$ kind --version
kind version 0.17.0
$ kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.3", GitCommit:"434bfd82814af038ad94d62ebe59b133fcb50506", GitTreeState:"clean", BuildDate:"2022-10-12T10:57:26Z", GoVersion:"go1.19.2", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.3", GitCommit:"434bfd82814af038ad94d62ebe59b133fcb50506", GitTreeState:"clean", BuildDate:"2022-10-25T19:35:11Z", GoVersion:"go1.19.2", Compiler:"gc", Platform:"linux/amd64"}

How many control plane and worker nodes will each of the KIND cluster have?

Those are deployed as single node cluster. This definition can be modified but it's important to consider extra cpu and/or memory resources.

What CNI plugin is needed on each of the KIND clusters?

It's using the KinD default CNI kindnetd. This can be changed as needed.

What would be the networking requirements for the KIND clusters to talk to each other?

Working on that

A few comments on this:

There will be one VM per participant.
The original PR had 3 clusters but I just merged Eric's PR for 4 clusters which matches his design.
If we can get away with a single node per kind cluster, all the better.
@electrocucaracha does the cluster inter-connectivity approach you implemented work with Multus?

Oh, and participant VMs are now planned as 8 core 32GB RAM. But the smaller the better for cost reasons. Once we have it all working we can see how small of one we can get away with. The terraform config right now uses Ubuntu 22.04.

* @electrocucaracha does the cluster inter-connectivity approach you implemented work with Multus?

No, that approach only creates a container and docker network to interconnect nodes, maybe something similar can be created for Multus Network Definitions.

I am working on a solution for the cluster interconnectivity.

A draft PR by @vjayaramrh  to update README is at https://github.com/nephio-project/one-summit-22-workshop/pull/55/files
A PR by @debeaueric that adds a diagram is at https://github.com/nephio-project/one-summit-22-workshop/pull/50


mupen64plus: merge mupen64plus-video-glide64mk2

mupen64plus-video-glide64mk2 is actually already provided with mupen64plus source bundle since version 2.0.0, so it's redundant to have another formula, which is outdated anyways.

mupen64plus-video-glide64mk2

formula removed
(automatic) version bump (2.0 to 2.5)

inreplace added to correctly set -mmacosx-version-min, otherwise it'll link against libstdc++ on recent OSes, giving warnings on dependencies with libc++.
homepage updated, desc added

Rebased onto the master.

Rebased onto the master.

Thanks!

Thanks!


The fix for #125 breaks the Ball example

The fix for #125 (08e01d7e38a4fa0cbc08081191e6f3073b5c1e5a) added Q._generateCollisionPoints(this) to Sprite.update and now I get a cannot read property 'length' of undefined error on quintus_sprites.js:312.
From the Quintus site, it looks like the Ball example doesn't need collision detection and it might be wasteful to add p.points just to make Q._generateCollisionPoints(this) happy, so maybe it'd be better just check for p.pointsbefore calling it?
I'm very new to this stuff so I don't really know how to fix it.

:+1:

Bug is still here.
And http://www.html5quintus.com/quintus/examples/ball/ uses old version of quintus_sprites.js


Bugfix for subtraction of a vector from a scalar

The result of constant - vector operation was equal to vector - constant.
I have changed the implementation so that constant - vector == -(constant - vector) and added a couple of tests.
The changed operator is only used in LogisticCostFunction.cs (the usage is fixed) and Softmax.cs (it seems that it was not correct before).

Chris - does this fix issues with the cost functions?

Hey Seth, the logistic cost function would need some changes to suit.


Remove obsolete cached keyword from documentation

According to changelogs the cached keyword has been deprecated for conv layers, but it was still in the docs.

Removed the example from docs
Some minor rewording

@abieler Thank you for the rewording. #184 already exists for update docs. I will pass the reword to #184.

nice, thanks. will close PR

@abieler @CarloLucibello Thank you for your suggestions. PR or some comments are welcome.


Problem with update accessories in iOS since I update the add-on to 2.4.0

Problem/Motivation
Since I update this add-on to 2.4.0, the update of accessories under iOS is very very long or impossible.
Expected behavior

Same speed of the add-on 2.3.0

Actual behavior

error during the update in iOS

Can you help me ?

I also submitted a ticket about this. It appears that certain types of accessories are the potential cause of the problem.
https://github.com/hassio-addons/addon-homebridge/issues/84
For me, it was the climate accessory for my Nest. When I excluded that from my HomeBridge setup, the accessories magically started getting updates again.

that's the same for me, it's the climate accessory ! but I use broadlink custom component to control my air cooler and radiator and I discover a problem with my radiator, I solve it and that work now.
thanks for your answer

Yeah, it seems we'll just have to wait for the next update to see if they fix the climate thing. It's kind of annoying, but at least I can control my Nest through the Nest app in the meantime? 🤷♂

Have you tried adding the nest to hassio and then exporting it to homebridge as a device instead of using homebridge-nest plugin?

@CroVlado
My nest is added straight into Home Assistant. I then have this add-on.

I’m not using the nest plugin. I have the entity in Home Assistant setup and working properly. This addon then adds two accessories in HomeKit, one for the fan, and one for the climate controls.
The problem is that since this update, both accessories in HomeKit never finish updating. In the Home app is basically times out trying to get the current state.

I got ya. Have you tried replacing that with homebridge-nest and seeing if it creates the same issues?
I use homebridge-nest until the whole HomeKit stuff comes along better in HA. It works well for me.

For me it’s a full on crash when I add BroadlinkRM Homebridge plug-in
When I go into the logs I see that it’s broadlink plugin that is not fully loading. When I remove that plug in and the corresponding entry on the config file things a back to normal.

I'm having the same issue here, but with my Xiaomi Air Purifier

Hi mates,
I was trying to find a solution to the  problem as this was working fine before. I think it’s more to do with my HomePod.
As a process of elimination:
I removed my HomePod
Setup the Apple TV 4 as the hub
Then I restarted Homebridge
Checked on both home network and on 4G
Re added HomePod
Updated HomePod to 11.3
Re tried homebridge
Now it seems to work and has been stable for past 5 days.
Hope this helps.


test: data-driven scope testing

Closes partially #540.
Summary of Changes

Data-driven creation of scope tests
Clean up of creation of grammar and formatter tests

:tada: This PR is included in version 0.2.0 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


A model results inconsistent output values depending on input batch size

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): python:3.7.5-slim docker image, macOS Catalina 10.15.2
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): pip 19.3.1
TensorFlow version (use command below): 2.1.0
Python version: 3.7.5(docker), 3.7.4(macOs)
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the current behavior
The predict method of tf.keras.applications.mobilenet.MobileNet, gives different output values when the batch sizes are different even though all input value are the same.
This problem does not occur with Tensorflow 2.0.0 and occurs with 2.1.0
Describe the expected behavior
The output value of an input does not change as it is in another batch with different size.
Code to reproduce the issue
import numpy as np
import tensorflow as tf

def test_mobile_net():
    four_black_images = np.zeros((4, 224, 224, 3), dtype='uint8')
    one_black_image = np.zeros((1, 224, 224, 3), dtype='uint8')

    four_input = tf.keras.applications.mobilenet.preprocess_input(four_black_images)
    one_input = tf.keras.applications.mobilenet.preprocess_input(one_black_image)

    # below assert statement passes, meaning all input batches have the same input values
    for i in range(4):
        assert (four_input[i] == one_input[0]).all()

    model = tf.keras.applications.mobilenet.MobileNet(input_shape=(224, 224, 3), include_top=False, pooling='avg')

    one_image_result = model.predict(one_input)
    four_images_result = model.predict(four_input)

    # bellow assert statement passes
    # There is no inter-differences between  output values within the same batch
    for i in range(4):
        for j in range(4):
            assert (four_images_result[i] == four_images_result[j]).all()

    # Bellow assert statement fails with Tensorflow 2.1.0 but passes with 2.0.0
    assert (four_images_result[0] == one_image_result[0]).all()

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
I also have tested with batch size with 4 and 7, wondering if the output result is only different when the batch has size 1, but it also gave inconsistent results.
So the output value changes as batch size changes with version 2.1.0.

I have tried on colab with TF version 2.0 and i am not seeing any issue. However with TF version 2.1 i am seeing AssertionError: .Please, find the gist here. Thanks!

@donguklim I agree. AssertionError:  is reproducible with tf-nightly but not with TF2.0. However, the inconsistency is in 5th decimal when I applied tf.reduce_sum for the two data. So I would say the inconsistency could be due to precision related. Please check the gist here.
Output of your code with tf.reduce_sum as follows. You can check the results differ only in the 5th decimal. Thanks
Four tf.Tensor(76.64975, shape=(), dtype=float32)
Single: tf.Tensor(76.64976, shape=(), dtype=float32)
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
<ipython-input-8-646cdf461c0a> in <module>()
     29     print('Single:',tf.reduce_sum(one_image_result[0]))
     30     assert (four_images_result[0] == one_image_result[0]).all()
---> 31 test_mobile_net()

<ipython-input-8-646cdf461c0a> in test_mobile_net()
     28     print("Four",tf.reduce_sum(four_images_result[0]))
     29     print('Single:',tf.reduce_sum(one_image_result[0]))
---> 30     assert (four_images_result[0] == one_image_result[0]).all()
     31 test_mobile_net()

AssertionError:

I agree with  @jvishnuvardhan that the numerical difference is precision related and very insignificant. Rather than use "==", you can use np.allclose() which will return True.
Closing this bug since there is nothing need to be addressed here.


Editing and Reading Inverted

With the 1.0.0 update released Aug 2019, if you are in a decompiled file it will say "Editing" on discord, while if you are in a normal file that is supposed to say "Editing", it says "Reading

Fixed in the 1.0.1 release


Block cyborgs from receiving criminal status icons

About the PR
Prevents cyborgs from seeing criminal status icons.
Why / Balance
Put briefly, cyborgs aren't part of Security and don't need criminal status icons. I argue that giving them this information encourages poor cyborg play.
https://discord.com/channels/310555209753690112/1218078327504506960
Technical details
ShowSecurityIconsSystem makes use of the existing CanDisplayStatusIconsEvent to allow another system to cancel the function before it adds the rest of the status icons. SiliconLawSystem responds to this event and cancels if the user is bound by silicon laws.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl: Krunk

remove: Cyborgs no longer see criminal status icons.

good change, validhunting borgs are exhausting. might cause problems if sec borgs are ever added though


Mac rls

please do a mac release. thanks

Well since VB.net is for Windows this would need project would need to be redone in JAVA.


AO-12116: [transaction setting] Rename transaction filtering settings

Rename the transaction filtering setting name to keep it consistent among all language agents.
Update version number to v1.6.1

Codecov Report

Merging #101 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #101      +/-   ##
==========================================
- Coverage   82.25%   82.23%   -0.03%
==========================================
  Files          34       34
  Lines        3793     3793
==========================================
- Hits         3120     3119       -1
  Misses        573      573
- Partials      100      101       +1

Impacted Files
Coverage Δ

v1/ao/internal/config/config.go
87.75% <100%> (ø)
:arrow_up:

v1/ao/internal/reporter/reporter_grpc.go
67.12% <0%> (-0.18%)
:arrow_down:

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 56ece1d...e83d54a. Read the comment docs.


Fb vmware controller enhancements

This change is

Codecov Report

Merging #566 into fb_vmware_controller_enhancements will not impact coverage.

@@                         Coverage Diff                          @@
##             fb_vmware_controller_enhancements     #566   +/-   ##
====================================================================
  Coverage                                58.15%   58.15%
  Complexity                               18686    18686
====================================================================
  Files                                     1932     1932
  Lines                                    99435    99435
  Branches                                 10099    10099
====================================================================
  Hits                                     57830    57830
  Misses                                   38527    38527
  Partials                                  3078     3078

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 3e76472...6bedb18. Read the comment docs.


Defining customMappingText before usage

Fixes #578  Defining customMappingText before usage in the user provided config folder section.

Thank you so much @andrewreeman for opening the PR. will get it in as fast as possible

Thank you so much @andrewreeman for opening the PR. will get it in as fast as possible


Add description to some of the main fields in DnsEvents table

Add description to some of the main fields in DnsEvents table:

ClientIP, Computer, IPAddresses, Name, QueryType, SubType, TimeGenerated

@EdB-MSFT
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @tduarte14
Thanks for your contribution to the documentation.
These docs are automatically generated via an API. This means that the changes must be made in the manifest file for the specific resource log, otherwise, they will be overwritten when the automation is next run.
I have forwarded your changes to the developers responsible. Once they add them to the manifest file, they will be reflected in the generated docs.
#please-close


Debug terminal

You can open any godot editor version thru the terminal to have the debug output in that window, cannot do that thru gvm so maybe add a toggle to open it with the terminal?

has this stopped being updated?

Not at all, but I work on this 100% for free on my free time. The last two months have been very busy between trade shows, events, wrapping up projects and moving to a different office so I haven't been able to work on it. I hope this weekend I'll finally be able to get back to it and release an update.

Unfortunately there is no easy way to launch a console using OS.execute(), it will hide text based binaries by default
As a workaround you can launch gvm from a console and when you double click on any project it you'll get debug output
You can create a launcher saving something like the following as gvm-console.cmd
@echo off
cmd /k gvm.exe

If anyone finds a way to do this feel free to send me a PR


[MAC Manager] Enhance UTs, exception handling, and comments

This PR proposes the following changes:

Unit Test - it used MockMvc, Moked Repositories & mokito. It removed Mocked Controller & Mocked Service.

Exceptions - it added more exceptions such as MacAddressFullException, MacRepositoryTransactionErrorException,etc.

Comment - it added google doc format comments to all methods in Service and Repositories.

@kimeunju108 Can we address those minor comments and move on?


functional feature layer widget

@dmartine Code looks good to me. Can you review and merge it if it's OK?


transfer chat-drive to wechaty-plugin ?

When I walk through the chat-drive source code,  I find that file is saved in Message Event Listener, which can be more easier. All of this can be a plugin for wechaty. @huan how do you think about it ?

Yes, this should be a plugin of Wechaty, and also a Vorpal command too!
For example:
import {
  ChatDrive
}                                  from 'chat-drive'

wechaty.use(ChatDrive({
 ...
}))

Please feel free to send PR to this repo to improve it if you have any good ideas!


Viewgroup

Re-write our view management to use real tabs.

fix https://github.com/dart-atom/dartlang/issues/526
use the tabs control from atom
when there's just one tab in a group, don't show the tabs control, just the content
when there are no tabs, hide the tabbed container completely

fix https://github.com/dart-atom/dartlang/issues/435
fix https://github.com/dart-atom/dartlang/issues/394


Update the swarm notebook example

Why are these changes needed?

Related issue number

Checks

[ ] I've included any doc changes needed for https://ag2ai.github.io/ag2/. See https://ag2ai.github.io/ag2/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Thanks for the comments, I've resolved and updated as well. Will also update the blog post, so it can be reviewed again and merged after that.

Is there an obvious benefit of the new swarm programming interface vs. the old one? If so, can we talk about them in doc/blogpost?

Is there an obvious benefit of the new swarm programming interface vs. the old one? If so, can we talk about them in doc/blogpost?

Yes, will update that


Dashboard goes to error page instead of login when it's lost auth

Steps:

Leave Rancher running on the dashboard until it times out
Refresh page

Results:  Goes to the Error page instead of taking you back to Login page.

Rancher version master-head (07/14/2020) 691feb01a k8s HA install
dashboard-index -> latest
Tested with an expired Dashboard UI cookie. As described in this issue.
Checked also removing the Dashboard UI cookie from Rancher.
Refreshing the browser takes the session back to login /dashboard/auth/login


Support bootstrap env from file

https://github.com/awslabs/amazon-eks-ami/issues/844
Description of changes:
Support providing environment variables to bootstrap.sh via an optional file (/etc/profile.d/bootstrap-env.sh) that is sourced if it exists. This solves the problem of no variable persistence between cloud-init steps which currently stops MNGs providing bootstrap overrides. To support adding KUBELET_EXTRA_ARGS to MNGs if  KUBELET_EXTRA_ARGS is set before the args are processed it will be persisted to  KUBELET_EXTRA_ARGS_ENV which will be appended to any value pass in via the args.
From non-MNG nodes this functionality can be used to add dynamic node labels based on instance metadata amongst other things.
This pattern has been in use for while with no issues, but this official change will make it safer as it removes the need to modify bootstrap.sh.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@cartermckinnon could you take a look at this please?

The alternative I've been thinking of is that the bootstrap script supports accepting two filePaths : one that points to the kubelet configuration, and one to the container runtime configuration. The latter now exists for containerd.
If we were to re-write the bootstrapping in a different language / as a binary, it might be easier to continue to honor these configuration files rather than honoring all bash environment variables as defined in the original bootstrap script.
I'm making an assumption here that overriding kubelet configuration, and container runtime configuration are the only use cases. I don't think most folks are looking to override the API Server endpoint / Cluster certificate etc. Having said that, if someone's looking to do that, they should probably just bootstrap the whole thing themselves.

@suket22 I missed your second reply. Many people are making other changes such as adding custom CAs to the OS and adding OCI credentials to kubelet. The configuration options in Bottlerocket are a good starting point, but remember that it has init containers for the real edge cases; say dynamic node labels based on the subnet the node starts in.

That said, I think you'll have to offer a switch between the new binary and old shell bootstrapping; so I don't think it matters if more work is done to the current logic as long as there is a way to do the same with the new binary

Agreed, I just don't know how we can offer the switch via the Managed Nodegroups API. I don't think we're going to be adding a new EKS API parameter for this. I think the best CX would be for us to switch over to a newer binary without anyone with an existing nodegroup having to change anything.

Many people are making other changes such as adding custom CAs to the OS and adding OCI credentials to kubelet

Fair point. I'd assume this is very niche though right? I think as a managed service we need to draw the line somewhere. At this level of customization, it probably makes sense to take complete ownership of the bootstrapping (like how custom AMIs + MNG does it).

Playing devil's advocate, there's no way enforce backwards compatibility regardless :(
If someone's using Managed Nodegroups and is editing the /etc/eks/bootstrap.sh file on disk directly in their MIME part, all their edits become useless if Managed Nodegroups starts calling a different binary.
So maybe a switch is something we'd need regardless.

Closing along with #875 .


Error launching Chromium: Passthrough is not supported, GL is swiftshader

[21581:259:0706/131446.110056:ERROR:gpu_init.cc(441)] Passthrough is not supported, GL is swiftshader
Big Sur: 11.4 (20F71)
Puppeteer: 10.1.0
Node: 16.0.0
launch: {
  headless: false,
  args: [
    '--remote-debugging-port=9222',
    '--no-sandbox',
    '--no-zygote',
    '--disable-gpu',
    '--disable-setuid-sandbox',
    '--disable-dev-shm-usage',
    '--disable-breakpad',
    '--enable-logging',
    '--no-first-run',
    '--start-maximized',
    '--disable-default-apps'
  ],
  browserContext: 'incognito'
}

I thought this was the main problem I was facing when updating to 10.1.0, but this is a Chromium error, but didn't crash Chromium nor affect the puppeteer launch. In my case it was something else (host/port problem).

So how did you solve it in the end？I have the same problem as you

So how did you solve it in the end？I have the same problem as you


Videa.hu - HTTP 403 error after long waiting

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.11.24
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar issues including closed ones

Verbose log

/home/<username>/.local/bin/youtube-dl "https://videa.hu/videok/zene/yesterday-klip-koncert-qytlviqASyguiKEt" --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://videa.hu/videok/zene/yesterday-klip-koncert-qytlviqASyguiKEt', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2020.11.24
[debug] Python version 3.8.5 (CPython) - Linux-5.4.0-54-generic-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4, rtmpdump 2.4
[debug] Proxy map: {}
[Videa] qytlviqASyguiKEt: Downloading XML
ERROR: Unable to download XML: HTTP Error 403: Forbidden (caused by <HTTPError 403: 'Forbidden'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/home/<username>/.local/lib/python3.8/site-packages/youtube_dl/extractor/common.py", line 632, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/home/<username>/.local/lib/python3.8/site-packages/youtube_dl/YoutubeDL.py", line 2248, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/usr/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.8/urllib/request.py", line 563, in error
    result = self._call_chain(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 755, in http_error_302
    return self.parent.open(new, timeout=req.timeout)
  File "/usr/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/usr/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.8/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)

Description

Downloading from videa.hu doesn't work. For the first try, youtube-dl hangs for 1-2 minutes after "Downloading XML", then fails with the error above. For the further tries, it fails immediately with the same error.

tried with both youtube-dl and youtube-dlc
youtube-dl gives failure and youtube-dlc does download the video.


Added Input.getMaxPointers(), fixes #4977 and update working from source link in README

Something to keep in mind, some platforms using the desktop backend support more than one cursor.
I don't know if that's physically handled by LibGDX atm, but it's possible.

I was careful to check that the returned values are always correct, they're pretty much hardwired into all of the Input implementations I encountered. Take a look at the examples given in #4977 to get an idea of what I mean.

Everything else seems fine to me, I guess we wait for the merge now.
There must be a reason not to support multiple cursors I'm unaware of.

Looks good. Can you please run the LibGDX formatter provider, on only the lines you have changed.

I tried importing the eclipse-formatter.xml file into InteliJ, but now it wants to indent everything with a tab followed by  a space.
Are there any specific things wrong with the formatting? Installing and learning eclipse is a bit of a stretch to format 60 lines of code, I can go trough them manually.

I use the formatter in intellij and its fine.
You need a space between method name and parenthesis.
getMaxPointers() should be getMaxPointers ()


Pyglet+blender

any way to run it without making blender crash?

What platform etc?

Hey @1C0D, if you could explain your problem a little more we will be able to help you out, if your problem has already been solved feel free to close this issue.


Reloading

backend live reloading built out, still need to manually click refresh on browser

too late no one reviewed it now completely built out live reloading


aws provider

what

aws provider

why

Crazy debugging issues in new modules

references
N/A

Another PR by @mcalhoun will fix this


[BUG] Insomnia Startup Error

Insomnia Version: 7.0.3
Operating System: MacOS Mojave 10.14.5

Details
My Insomnia app keeps throwing this error and it does not open. Can someone please help.

same question,there is no fix for 2 days?

The app couldn't run by this bug. Anyone  know which version is effected ?

I have installed Insomnia-7.0.3 version. The problem is resolved It seems

Same here. Updating to 7.0.3 version solved problem.

Same issue here. Fixed by updating to 7.0.3

I still get the same error after updating to 7.0.3

FYI I just fixed this on my mac. Can't tell you why this happened but this is how I fixed it.
In my case it looks like the php5 directory was moved or something. If you look in /usr/local/ my php5 was in php5-5.6.19-20160310-141555 so I just created a php5 symlink to php5-5.6.19-20160310-141555 i.e. ln -s php5-5.6.19-20160310-141555 php5

Today i try to reinstall via brew without any success, you can find my configuration at #1754

Same problem, won't start. I just love Insomnia, signed up for the team plan, so bummed I'm going to have to find something else now.

Hi folks, I'm going to go ahead and close this one since it's been a while, but feel free to report a new issue if you are still getting this error in one of our latest Insomnia versions


Question: Why checkoutLineItemsReplace?

What is the main reason why checkoutLineItemsAdd, checkoutLineItemsRemove and checkoutLineItemsUpdate is deprecated in favour for checkoutLineItemsReplace.
Does checkoutLineItemsReplace replace all line items that are currently on the server?

Yes, checkoutLineItemsReplace will replace all line items.
Mutating previously created/updated line items is not always possible, so the -Add -Remove and -Update mutations were sometimes unable to find previously added or updated line items.
The -Replace mutation provides a consistent interface for modifying line items on a checkout.


Preserve connection specification on reconnect

Hi,
In our app we call Vanity.connect! with a connection spec, so that we can use a custom adapter and programmatically determined options, e.g.:
Vanity.connect!(
  adapter: :logging_redis_adapter,
  url: OurApp.config.redis_url
)

However, when connections are established like this, Vanity.reconnect! does not preserve the connection params, and the new connection falls back to the default redis adapter. I've added a failing test to reflect this.
Assuming the intended behaviour is for these values to be preserved, I would be very happy to contribute a fix, but I'd appreciate some feedback on what approach might be best. I notice that currently the collecting option is saved to the configuration on connection. Could this be expanded to save the full connection params? Configuration#connection_params could be modified to be writeable, falling back to params parsed from a file as it currently does.
What do you think?

Hi @urbanautomaton, some more context might be helpful here - where does reconnect! get called from?

Hi Phillip, thanks for the response. In our app, reconnect! gets called in an after-fork hook called by Unicorn when it forks a worker process, so our vanity setup looks like this:
# config/initializers/vanity.rb
Vanity.connect!(
  adapter: :logging_redis_adapter,
  url: OurApp.config.redis_url
)

# config/unicorn.rb
after_fork do |server, worker|
  defined?(Vanity) && Vanity.reconnect!
end

The hook performs basically the same function as the Phusion Passenger hook in the existing Rails integration.
The result is that the master process boots, loading the vanity initializer, which creates the connection correctly. But each forked worker process calls reconnect! and ends up with the default connection spec, { adapter: 'redis' }.
We can work around this with an explicit disconnect!/connect! instead of using reconnect!, but it looks like the Phusion Passenger hook would cause the same issue we experienced.

Thanks for the context - I'd probably encourage the workaround of disconnect! / connect! for a couple of reasons:

the passenger hook is for an outdated version of passenger, I actually meant to remove it in the 1.0 release, but forgot. Going forward the idea would be to keep that complexity out Vanity itself but document common patterns and provide useful APIs - a PR to this effect would be great!
the collecting option from the connection params is also a bit of a odd duckling - it may be a good configuration parameter, but it's not a connection setting and is considered "legacy".

What do  you think about that approach?

Thanks very much @phillbaker, and apologies for the lack of response until now. 👍


applyFilterAsync creates new filter with the same name instead of applying existing one

Hello,
While applying YEAR(Order Date) filter in Superstore Customers dashboard using applyFilterAsync method of Worksheet object new filter with the same name is created instead of applying values to existing one.

Bartosz

Hi @bartoszZamojcin
Good news here. This has been fixed in 2021.4.3. I was able to reproduce the problem in 2021.3, but it works in 2021.4.3.
John

Hello @johnDance
good to hear it, but currently we are limited to version 2021.1.4 and problem still exists. We would appreciate it if you can propose any workaround how to apply original filter, as our internal investigation leads us nowhere.
Bartosz

@bartoszZamojcinRoche
Would you be able to update to 2021.1.n? If so, we can see if we could backport the change. If not, we'll have to get with the filter team to see if there is some way to combine the filters.
Let us know.
Thank you,
John

@johnDance
We already use version 2021.1.4 so if you could backport the solution and deal with the issue that way it will be great.
Bartosz

Filter team is unable to backport to 2021.1. Sorry.