Reporter provides unhelpful information / noise if virt-launcher is gone

What happened:
When virt-launcher is gone by the time the reporter is executed, we log several lines of unhelpful logs.
The following was logged after failed [sig-compute]User Access [rfe_id:2919][crit:high][vendor:cnv-qe@redhat.com][level:component] With regular OpenShift user should succeed with admin rights for the project should verify permissions on resources are correct for subresources [test_id:2921]given a vmi (guestosinfo) test due to ERR: Error from server (AlreadyExists): users.user.openshift.io "testuser-kubevirt-test-default3" already exists:
failed to execute command [ip address] on virt-launcher-testvmi-k7pqp-qkkhv, stdout: , stderr: , error: Internal error occurred: error executing command in container: container is not created or running
failed to execute command [ip link] on virt-launcher-testvmi-k7pqp-qkkhv, stdout: , stderr: , error: unable to upgrade connection: container not found ("compute")
failed to execute command [ip route show table all] on virt-launcher-testvmi-k7pqp-qkkhv, stdout: , stderr: , error: unable to upgrade connection: container not found ("compute")
failed to execute command [ip neigh show] on virt-launcher-testvmi-k7pqp-qkkhv, stdout: , stderr: , error: unable to upgrade connection: container not found ("compute")
failed to execute command [bridge -j vlan show] on virt-launcher-testvmi-k7pqp-qkkhv, stdout: , stderr: , error: unable to upgrade connection: container not found ("compute")
failed to execute command [bridge fdb] on virt-launcher-testvmi-k7pqp-qkkhv, stdout: , stderr: , error: unable to upgrade connection: container not found ("compute")
failed to execute command [lspci] on virt-launcher-testvmi-k7pqp-qkkhv, stdout: , stderr: , error: unable to upgrade connection: container not found ("compute")
failed to execute command [env] on virt-launcher-testvmi-k7pqp-qkkhv, stdout: , stderr: , error: unable to upgrade connection: container not found ("compute")
failed to execute command [[ -e /dev/vfio ] && ls -lsh -Z -St /dev/vfio] on virt-launcher-testvmi-k7pqp-qkkhv, stdout: , stderr: , error: unable to upgrade connection: container not found ("compute")
failed to execute command [ip address] on virt-launcher-testvmi-k7pqp-whpjs, stdout: , stderr: , error: pods "virt-launcher-testvmi-k7pqp-whpjs" not found
failed to execute command [ip link] on virt-launcher-testvmi-k7pqp-whpjs, stdout: , stderr: , error: pods "virt-launcher-testvmi-k7pqp-whpjs" not found
failed to execute command [ip route show table all] on virt-launcher-testvmi-k7pqp-whpjs, stdout: , stderr: , error: pods "virt-launcher-testvmi-k7pqp-whpjs" not found
failed to execute command [ip neigh show] on virt-launcher-testvmi-k7pqp-whpjs, stdout: , stderr: , error: pods "virt-launcher-testvmi-k7pqp-whpjs" not found
failed to execute command [bridge -j vlan show] on virt-launcher-testvmi-k7pqp-whpjs, stdout: , stderr: , error: pods "virt-launcher-testvmi-k7pqp-whpjs" not found
failed to execute command [bridge fdb] on virt-launcher-testvmi-k7pqp-whpjs, stdout: , stderr: , error: pods "virt-launcher-testvmi-k7pqp-whpjs" not found
failed to execute command [lspci] on virt-launcher-testvmi-k7pqp-whpjs, stdout: , stderr: , error: pods "virt-launcher-testvmi-k7pqp-whpjs" not found
failed to execute command [env] on virt-launcher-testvmi-k7pqp-whpjs, stdout: , stderr: , error: pods "virt-launcher-testvmi-k7pqp-whpjs" not found
failed to execute command [[ -e /dev/vfio ] && ls -lsh -Z -St /dev/vfio] on virt-launcher-testvmi-k7pqp-whpjs, stdout: , stderr: , error: pods "virt-launcher-testvmi-k7pqp-whpjs" not found
skipping pod virt-launcher-testvmi-56xtv-pmn69, phase is not Running

What you expected to happen:
We should keep the logs clean to allow faster debugging and identification of the issue. Instead of the noise above, we should just log once that the pod is not running and we cannot collect information from it.

We need to fix also the find virt-launcher logic, there is not such thing no-fork anymore, there is monitor instead
failed to execute command [/bin/bash -c pgrep -f "uid dba1fb44-bbef-4ec0-8aea-d568c20fc756.*no-fork"] on virt-handler-827gb, stdout: , stderr: , error: command terminated with exit code 1

bash-4.4# ps -ef | grep  "uid dba1fb44-bbef-4ec0-8aea-d568c20fc756"
root        9809    9798  0 14:27 ?        00:00:00 /usr/bin/virt-launcher-monitor --qemu-timeout 309s --name test-vm --uid dba1fb44-bbef-4ec0-8aea-d568c20fc756 --namespace default --kubevirt-share-dir /var/run/kubevirt --ephemeral-disk-dir /var/run/kubevirt-ephemeral-disks --container-disk-dir /var/run/kubevirt/container-disks --grace-period-seconds 15 --hook-sidecars 0 --ovmf-path /usr/share/OVMF
root        9835    9809  0 14:27 ?        00:00:00 /usr/bin/virt-launcher --qemu-timeout 309s --name test-vm --uid dba1fb44-bbef-4ec0-8aea-d568c20fc756 --namespace default --kubevirt-share-dir /var/run/kubevirt --ephemeral-disk-dir /var/run/kubevirt-ephemeral-disks --container-disk-dir /var/run/kubevirt/container-disks --grace-period-seconds 15 --hook-sidecars 0 --ovmf-path /usr/share/OVMF


Unicode conversion fails

Hi,
The test environment setup:

MacOS Ventura
unixODBC driver manager version 2.3.11
Microsoft ODBC Driver 18 For SQL Server
SQLServer running in Docker (or on a Windows machine)

Database has a single table:
create` table test (col1 `nvarchar(100));

Tested connection to the database with isql (command line tool) and issued the following statement:
insert into test (col1) values (N'Ω');

result is OK

Tested connection with nodejs and issued the same statement as above on the connection:
const statement =  `insert into IR_TSTGIN.dbo.test (col1) values (N'Ω');`
const result = await connection.query(statement);

Statement is executed but the result is not OK. The omega charactered is converted to another symbol.

Ran both statements above (via isql and with nodejs) with odbc debug enabled.

running with isql the log contains a message:

[ODBC][47379][1679320146.155152][SQLAllocHandle.c][395]
Entry:
Handle Type = 2
Input Handle = 0x7f987c815200
UNICODE Using encoding ASCII 'UTF-8' and UNICODE 'UCS-2-INTERNAL'

running through nodejs the log contains the following message:

[ODBC][46268][1679319075.820124][SQLAllocHandle.c][395]
Entry:
Handle Type = 2
Input Handle = 0x7f8a88012600
UNICODE Using encoding ASCII 'US-ASCII' and UNICODE 'UCS-2-INTERNAL'
It seems that the locale settings are not applied when connecting via NodeJS odbc to the driverManager.
The locale settings are:
LANG="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_CTYPE="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_ALL=

Please re-open.

Hi, any solution?

See also #379


Feature/remove session from hooks

Project organization

Related https://trello.com/c/o7jMqms6
Related https://github.com/liteflow-labs/liteflow-js/issues/26
Related https://github.com/liteflow-labs/marketplace-template/pull/32

Description
The goal of this PR is to remove the use of useSession as well as removing the Session Context.
How to test
N/A for now
Checklist

[ ] Update related changelogs
[ ] Update docs if necessary

@antho1404 @cpvdeveloper There are two commits:

The simple swaps of useSession to useWeb3React when possible

Trying to remove the ready and useEagerConnect by introducing a property userHasBeenReconnected that we would pass to templates on the client side. Really not sure of that direction.

Here's the related PR on marketplace-template: https://github.com/liteflow-labs/marketplace-template/pull/32
Take note this was not working this is just the latest version of where I was at.

 Hooks package - Remove session from Hook

closing in favor of https://github.com/liteflow-labs/liteflow-js/pull/90


[5.4] ArrayBoundCheckOpts: introduce a limit for the maximum dominator tree recursion depth

This is a quick fix for a stack overflow in case of very large functions.
TODO: Ideally this algorithm would be implemented as an iterative worklist algorithm.
rdar://77563057
This is a cherry-pick of https://github.com/apple/swift/pull/37315

@swift-ci test macOS


docs: fix external links from md to html

The issue was introduced in 266b2368660be685cb7f614c0d225b8d7186a55f

Oh, thanks, messed up those ones


docs: what's the difference between setState and trySetSate?

what's the difference between setState and trySetSate?

Hey, @zerofront, didn't pick up on this for a while. Sorry!
Too Long; Didn't Read

setState is a React class component built-in feature.
When you pass new state values to it, the component obeys that change and rerenders its elements accordingly. It's as simple as that.

trySetState is a feature shipped in with this manager utility.
It attempts to read your props and find out whether a key of the same name exists, and if it does, only changes the state value if you don't provide a prop of the same name for it. If your prop name is value and your state name is value, then trySetState will attempt to use prop.value to update your state.value.

The Longer Explanation
class Number extends React.Component {
  state = { number: this.props.number || 0 };  // defaults to 0

  trySetState = NumberAutoControlledManager.trySetState;

  incrementNumber = () => {
    this.setState({ // <-- Updates the number state; Ignores the number prop given by the parent.
      number: this.state.number + 1,
    });
  }

  incrementNumberIfNoProp = () => {
    this.trySetState({ // <-- Updates the number state only if the number prop wasn't given.
      number: this.state.number + 1
    });
  }
}

// The number will increment itself without any handlers or values from AppUncontrolled.
const AppUncontrolled =
<Number />
; // Starts at 0

// The number will not increment because AppControlled gave it `123`.
const AppControlled =
<Number number={123} />
; // Starts at 123. Never increment.

The idea of this utility is to help you write components that update themselves if you provide no props for them. You could do the same thing without this utility, but you'll have to write the implementation details yourself. On every component that is uncontrolled.


just --dump should provide recipe order

I was hoping to write something that ran the recipes in a justfile in order (lots of individual set up steps to bootstrap a new machine). I can use just --summary --unsorted, but it would be cleaner to use just --dump --dump-format=json and sort by some recipe order info.

The most straightforward way to do this is probably to just add a new recipe that recursively invokes just with the other recipes you need, in the order they should be invoked in. e.g. something like:

install_dependency_a:
    ...

install_dependency_b:
   ...

bootstrap:
    @just install_dependency_b
    @just install_dependency_a

This requires a bit of manual setup, but it means you can be explicit about what recipes you run, while still having the freedom to add additional recipes to the justfile that won't interfere with the bootstrap process.

I also encountered this. I am building a plugin integrating just into neovim, and the fact that the order of recipes is not preserved with --dump-format=json forces me to manually parse just --unsorted --summary.

What about simply adding a key order to the json? That way we can just sort the recipes by it to get the original recipe order.

just uses Serde for serialization, so the serialization code is automatically derived, not hand written, so it's hard to add additional fields which aren't present on the struct.
Also, there isn't actually a single key which gives the order. It's determined by a weird combination of recipe.import_offsets and recipe.name.offset, which bases order both on the order in which the files were imported, and the order of the recipe in those imported source file. This is the sort which puts them in justfile order:
recipes.sort_by_key(|recipe| (&recipe.import_offsets, recipe.name.offset));

So there isn't a pre-existing order field that we can serialize.

I am not sure I follow what you mean in the second and third paragraph. Wouldn't having --summary super json output simply replicate the same problem?

It depends on what the problem is. It doesn't fix having to call just twice, but it does fix having to manually parse the output of --summary, if it returns JSON and you already have a JSON parser.


add new CRDs as part of upgrade

the crds directory is applied on install but not on upgrade because of potential weirdness with upgrading CRDs. however, this means that new CRDs are also not added as part of helm upgrade. there should be no conflict in doing this, I believe.
Output of helm version: version.BuildInfo{Version:"v3.0.0-beta.3", GitCommit:"5cb923eecbe80d1ad76399aee234717c11931d9a", GitTreeState:"clean", GoVersion:"go1.12.9"}

This was discussed in #6243:

CRDs are not upgraded during helm install or helm upgrade. There is no accepted practice for managing CRD upgrades (again, because they are a global resources, and an upgrade can break things in interesting ways), so we decided not to handle this case. It is up to the operator.

See #5871 for more background on this discussion.

@bacongobbler yeah i saw the discussion. I totally understand the reasoning for not upgrading existing CRDs.
But if I add a new CRD to a chart, why wouldn't we want upgrade to install it? As it sits today:

I have a chart with n CRDs, helm install installs them
if i modify the CRDs and update the chart, helm upgrade does nothing (I get this part)
if I add a new CRD and update the chart, helm upgrade still does nothing <- this is the case I'm concerned about

helm install already has logic for looping through the CRDs. would it be unreasonable to use that same logic in upgrade and only install CRDs that don't yet exist?

fair enough. I think it's safe enough to handle the install case, just not the upgrade case. Marking as a feature enhancement

Any new updates?

No new updates, @thanhtoan1196. But please feel free to contribute.

Bump!

Hello, has there been any progress on this?

@bacongobbler yeah i saw the discussion. I totally understand the reasoning for not upgrading existing CRDs.
But if I add a new CRD to a chart, why wouldn't we want upgrade to install it? As it sits today:

I have a chart with n CRDs, helm install installs them
if i modify the CRDs and update the chart, helm upgrade does nothing (I get this part)
if I add a new CRD and update the chart, helm upgrade still does nothing <- this is the case I'm concerned about

helm install already has logic for looping through the CRDs. would it be unreasonable to use that same logic in upgrade and only install CRDs that don't yet exist?

We're also facing a lot of grief due to helm upgrade not handling scenario #3 here.
My current solution is to iterate all subcharts, pull out their CRD defs and put then as pre-upgrade hooks in my main chart.

If you would consider Flux to manager Helm release, we've implemented CRDs upgrades https://fluxcd.io/docs/components/helm/helmreleases/#crds

Hello, has there been any progress on this?

@bacongobbler yeah i saw the discussion. I totally understand the reasoning for not upgrading existing CRDs.
But if I add a new CRD to a chart, why wouldn't we want upgrade to install it? As it sits today:

I have a chart with n CRDs, helm install installs them
if i modify the CRDs and update the chart, helm upgrade does nothing (I get this part)
if I add a new CRD and update the chart, helm upgrade still does nothing <- this is the case I'm concerned about

helm install already has logic for looping through the CRDs. would it be unreasonable to use that same logic in upgrade and only install CRDs that don't yet exist?

We're also facing a lot of grief due to helm upgrade not handling scenario #3 here. My current solution is to iterate all subcharts, pull out their CRD defs and put them as pre-upgrade hooks in my main chart.

@bacongobbler thanks for linking me to this. Do you know the current status of supporting situation 3? Would upstream accept a PR for this feature enhancement? We have been running into issues with this as well.


Get more customer data from each social network

The only really difficult part here was signing the Instagram request manually.

👍


Latest Ng2-smart-table doesn't have the striped rows

I think css is not get loading properly also i need to change table data with colour LOw,Medium,Hihg

Any updates on this one?

It does seem like at least a part of the styles isn't loading at the moment.


Implement 'Specific neighbourhood' API call #9 AND Implement 'Neighbourhood boundary' API call #10

Thank you for the contribution, the remaining call implementations are slowly going down.


AzureIoTEdgeV2  "Build Module images" task produces expanded deployment in 'undefined' path

Description of issue:
Using default settings for AzureIoTEdgeV2 "Build module images" task:

Results in the creation of expanded deployment in 'undefined' path:

This is in contrast to the behavior of the "Push module images" task, which produces output in "./config" path:

The "Generate Deployment manifest" task allows for specifying the output path:

Suggested Resolution:
Produce expanded deployment from "Build Module images" task in ./config directory similar to the behavior of the "Push module images" task
OR
Allow specific a user-defined Outputh path similar to the behavior of the "Generate Deployment manifest" task

@toolboc It's recommended to use Generate Deployment manifest to generate the final manifest for deployment since it has better flexibility. The file generated in Build/Push Module images is just kept for backward compatibility.
Please let us know if there's scenario that can't be covered by the Generate Deployment manifest task.


[Runtime][PipelineExecutor]Add forwarding queue logic for set input.

[RFC] https://github.com/apache/tvm-rfcs/blob/main/rfcs/0014-pipeline-executor.md
[Issue] https://github.com/apache/tvm/issues/8596
When the set_input function get called, a runtime of pipeline may not yet finish the former computation work then the new set_input call would break the current computation logic, to avoid such issue, we add the forwarding queue logic to guarantee the order of input data consuming.

@masahi, @comaniac , please take a look.


feat(api): publish event when unpublishing representation

Describe your changes
ASB-1742
A change to the PATCH /applications/{id}/representations/{repId}/status endpoint:
When changing the status of a Representation from PUBLISHED to another value:

publish a Update event to the nsip-representations topic containing the Representation details.
also set the unpublishedUpdates flag to false.

Issue ticket number and link
Type of change 🧩

[ ] Bug fix (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
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[ ] I have made corresponding changes to the documentation
[ ] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

The ACs may need to change for this, a change was requested to not un-publish WITHDRAWN or ARCHIVED reps, that was not highlighted in the ticket.

As discussed, will continue to publish events when changing to withdrawn/archived, and leave the decision of what to show/hide on the Front Office
Updated the event payload to only include the status and representation ID fields, rather than the whole representation (+contacts, attachments, etc)


File Form Element - Get the filename to save it into a Grafana variable

Hi everyone,
I'm trying to process an uploaded file with the File Form Element in the plugin, and I need to get the filename (in particular, the extension of the file), because based on that, I need to process the file in different ways (for example, if it is a PDF, I need to convert it to TXT, but if it is a DOCX, I need to convert it to PDF before I convert it to TXT, because the features of the program that I'm using to processes the files, Stirling PDF).
Is there a way to pass the filename to the POST that I'm doing, for example as a query parameter? I tried to troubleshoot this with console.log, but I didn't find the filename in any logged field, :(
Anyone can point me in the right direction?
Thanks in advance for your help.

Finally, the problem was in the Web Server that I'm using to upload the file to (http in Node-RED) from Form Panel.
After I checked the "Accept file uploads" option, req.files appear, where I can see the name of the file, with the encoding, the mimetype and the content of the file:

Thanks!


how to implement promise-based CRUD events?

from what i understand of the docs, the following adds a single event listener for all update events:
db.addEventListener(['updated'], function(event) {

how do i get a callback for the last update event, or a specific one? i'm trying to wrap YDN with something like this:
mystore.save('users', { name: 'joe' }, 24).then(function(result) {
   // handle something here
});

Dispatching events are expensive and won't emit unless requested during database initialisation.
See installable events section http://dev.yathit.com/ydn-db/doc/setup/events.html
Using events for application logic are not encouraged. There should be better alternative.

just noticed the return type of add. can i do it this way:
var req = db.add('users', theObject, theId);
req.done = ...

looks like i should have looked at the doc closer: http://dev.yathit.com/api/ydn/db/request.html
your request already implements a then(success, fail)
sorry


Hi!

Hei Rosty!
I'm trying to start a side business and after multiple discussions came up with the name "overbase", but I just saw that you already created that organization on GitHub 1 year ago. 🙈 Great minds think alike! :)
Any chance you can give me the organisation or do you still have plans for it?
Cheers,
Adi

Hi Adi,
Project is a bit on hold because of war, but definitely I'm planning to continue my work on it.
Regards,
Rostyslav

Any possibility you can continue your work under a different name? I'm open to paying $500 for it.

Without any doubt this name will be used for future project development.

Do you also have a domain name for it?

I see you're a bit attached to this name. If you could explain the product to me, I might help you find a better name. Atm, the code is super minimal, can't make sense of what it's supposed to do.
Have a think about my offer and let me know if you change your mind. I'll have a look around for other names until then.
Cheers and all the best to you and Ukraine!

Thank you for understanding. Best wishes to you and your project.


[Api improvement] More customizable jetpacks

Hi, @aidancbrady.
I've writing addon for mekanism for my server and I encountered a problem:
I've created electrical jetpack, it works on energy, but in main MC gui I see "Hydrogen" string instead of "Energy" or something I want. I think will be better to get the display string from main class of Jetpack and then I can override that method and write my own information.
Thanks!

The Electrical Jetpack is part of IC2, not Mekanism.

And what I need to do with jetpack on other gases?on methane, for example? I too see Hydrogen on main screen.

On my add on, methane jetpack is an upgraded version of hydrogen jetpack.

So, if electrical jetpack already exists in other mod I can't create it in my addon for Mekanism? Non-static jetpack info is a good feature I think.


Add pre-build actions scripts

Description:

Adds missing pre-build action scripts to set the current build configuration
This is needed to fetch the correct keys from the ThirdPartyKeys.plist.

I think this was added in the post build actions, can you check if still there and delete it?

The post-build action is to copy the thirdpartykeys file to the build output.


handle pdu if nil returned

In some cases [1] r.cl.Read() returns a nil interface without any errors, leading to panics when the p.Header() (when p == nil) is called.
This change adds a simple check for nil values.
Example:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0x5c5389]

goroutine 7 [running]:
redacted/vendor/github.com/fiorix/go-smpp/smpp.(*Receiver).handlePDU(0xc0000ec180)
        /home/yekta/.go/src/redacted/vendor/github.com/fiorix/go-smpp/smpp/receiver.go:176 +0x1f9
created by redacted/vendor/github.com/fiorix/go-smpp/smpp.(*Receiver).bindFunc
        /home/yekta/.go/src/redacted/vendor/github.com/fiorix/go-smpp/smpp/receiver.go:132 +0x344
exit status 2

[1] For example, when the connection is abruptly closed by the server.

Why is this not merged yet? If you have a receiver with a bound handler, this panic gets triggered upon a reset connection. The same does not apply to a bound transceiver, but I haven't looked into why.
This seems like a simple enough fix to merge.


Repeat mode

Inconsistently working at the moment, seems to clash with other modes. Opening this draft pull request for visibility and feedback from those more familiar with the code.

Hi @iam6lake , thanks for contributing.
At first glance you might be running into issues with Player.tsx and VideoPlayer.tsx being in different documents. The VideoPlayer is in an iframe and might not always receive key events, so I would implement the keyboard bind inside of Player.tsx as well.
There is also a separate question of whether the actual loop behavior needs to be in VideoPlayer.tsx at all. Player.tsx implements the logical behavior of many of the other play modes, allowing VideoPlayer.tsx to do its own auto-pausing only because there's a need to have the video pause precisely while the subtitle is still on screen.

OK, made those chagnes

Addressed all current comments

Thanks!


LocURI locator is denoted with []

The LocURI has to be denoted with [/] a wrong separate was used in the URI, the file has been changed with correct separator.

The LocURI path modification requested.

#sign-off

@illfated I'm on Dan's team and am helping clean up the backlog, particularly on the easier to resolve PR's

This PR closes the issue https://github.com/MicrosoftDocs/windows-itpro-docs/issues/6082

Closes #6293.

Closes #6293.

Hmm, did you make a typo in the issue number?

Hmm, did you make a typo in the issue number?


How to pre-split the edges in Link Prediction tasks?

A person may be interested in using a fixed split for link prediction tasks (e.g. given with the dataset).
In these cases, it may be useful to allow the user to pass a NumPy array containing the list of the edges to use in the training/validation/test set.

Hi, fixed split has been supported by the corresponding argument in pyg_to_graphs if using standard datasets in pytorch geometric.
For datasets represented by NetworkX, we can simply have 3 graphs (or lists of graphs), corresponding to the train, validation and test set.
Hope that answers?

@RexYing I looked at the three graphs you mentioned but, looking at the implementation of the split function (for link prediction), is not clear (to me) how to create them. This is mainly due to the implementation of the disjoint message passing during the train.
Personally, I think that a support function which takes as input the list of edges in the 2 (or 3) splits, may be a not negligible contribution to the improvement of the framework.

Hi, there are 2 kinds of splits in this context: disjoint split (which is performed on training set), and the train/val/test split.
For disjoint split, it is always randomly performed (and resampled even), in order to avoid the model overfitting to one particular disjoint split.
For train/val/test split, this is where you could simply initialize with GraphDataset(train_graphs); GraphDataset(val_graphs) etc. without the need to care about the disjoint split.
So would it help if we isolate an interface to perform disjoint split only (without splitting train, val, test), so that if we have pre-split datasets, we only need to call disjoint split for the training graphs, and directly use GraphDataset(val_graphs) to construct the validation set?

I think it would be great.
Can this function also deal whit the issue #6 (true negative split)?

function to pre-split the edges supported.


Update algorithms_and _data_structures.md

This Important book was missed from the list...

Hey @shloknangia , to be on the safer side, we can't have PDF links off paid books.
You can add the free books or books distributed by authors for free. If a book is paid and then PDF of that book can not be added in AlgoWiki.
~
Vikesh


Fixes to the KV Storage article

What's changed, or what was fixed?
In the KV Storage: the Web's First Built-in Module article,

Added missing commas/periods.
Made some sentences clearer by adding "that" conjunctions.
Removed redundant "to older browsers" in a sentence.
Replaced "load the demo in Chrome 74 (currently Chrome Dev or Canary)" with "load the demo in Chrome 74 or later".
Added missing "Was this page helpful?" block.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


Doc: fix broken link

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


(PDB-5642) Un-conditionalize log slow PQL warning

Users should not need to opt-in to this warning

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Build new CF buildpacks versions on new release

The aim of this task is to include the CF buildpacks introduced in #67 in the release process. Both buildpacks could be built on the Circle CI automatically whenever a new version is released. The resulting zip files should be attached to the release as an asset.
See: https://github.com/signalfx/signalfx-dotnet-tracing/pull/67

@mateuszrzeszutek can this be closed?

It's not done yet. But if we don't want to waste any time or resources on automatic CF buildpack release (which might not be that necessary) it's fine if we just close it.

Old stale issue.


Feature request: load remote env vars from cmd execution

On some environments, specially with bigger dev teams, is common to use secrets managers like pass, 1pass cli, bitwarden cli and many others.
These secrets managers allows the user to retrieve and read secrets from vaults via a terminal command, for example, for 1pass:
op read op://MyVault/MySecret/secret-field

So if someone or a team have a vault called MyVault, with an entry called MySecret that contains a field called secret-field with value "123", then the command above will return this value to the stdout.
Given that, someone could set the .env file with something like:
# .env
MY_SUPER_SECRET=$(op read "op read op://MyVault/MySecret/secret-field")

Sourcing this file will then issue the 1pass cmd and fill the MY_SUPER_SECRET env with the value "123".
Does makes sense to this library to handle this case?

if maintainers think this makes sense, I can surely open a pr with this feature

1Password et all are viable use-cases... but dotenvy does not parse the .env files with the shell, so any embedded commands are not run.  I avoided doing parsing because I was concerned of the security implications and I feared complexities might arise between different systems, so I concluded that it was best to leave that out of the core library.
Is it possible for you to use System.cmd/3 or System.shell/2  to explicitly run the commands?  For example, if you put your 1Password related variables into a secrets.env file, e.g.
# secrets.env
export MY_SUPER_SECRET=$(op read "op read op://MyVault/MySecret/secret-field");

And then you could source that file using System.shell/2 and read any variables it exported, e.g.
System.shell("source /path/to/secrets.sh")

System.get_env("MY_SUPER_SECRET")

Inside runtime.exs using dotenvy, this might look something like this:
# config/runtime.exs
import Config
import Dotenvy

# Load up 1Password secrets
System.shell("source /path/to/secrets.sh")

source!([".env", System.get_env()])

Would that work for you?

Alternatively, you could write your own implementation of the Dotenvy.parse/3 callback and pass your module as the :parser option...


"Avoid using undefined variables" with short version of list()

PHPMD version: master
PHP Version: 7.4.6
Installation type: composer
Operating system: macOS Catalina 10.15.5
If you define variables like this, no phpmd error occurs:
$array = ['foo' => 'abc', 'bar' => 'def'];
list(
    'foo' => $foo,
    'bar' => $bar
) = $array;
echo $foo;
echo $bar;

But if you use the short syntax of list() with [] = $array, then phpmd will give the following errors:

Avoid using undefined variables such as '$foo' which will lead to PHP notices.
Avoid using undefined variables such as '$bar' which will lead to PHP notices.

$array = ['foo' => 'abc', 'bar' => 'def'];
[
    'foo' => $foo,
    'bar' => $bar
] = $array;
echo $foo;
echo $bar;

As the meaning of the two code pieces is the same, phpmd should also understand the short version syntax.
Addition: it is the same without the usage of the array index inside list() or [].

Duplicates #725 and fixed in "phpmd/phpmd": "dev-master" (e.g. next version).
Thanks.


FloatingPointError: Predicted boxes or scores contain Inf/NaN. Training has diverged.

Hello, did you solve this problem? @junchen14 ，I'm having the same problem．
When my default configuration: "python lazyconfig_train_net.py --num-gpus 1 --config-file configs/mimdet/mimdet_vit_base_mask_rcnn_fpn_sr_0p5_800_ 1333_4xdec_coco_3x.py --num-machines 1".
Training has diverged.", "FloatingPointError: Predicted boxes or scores contain Inf/NaN.
I should follow the complete MAE pre-training weights provided by the author@Yuxin-CV , but I am confused, I did not find the training weights for the decoder in question, where is this decoder? How should I place it?
The details are shown below:
When running: "python lazyconfig_train_net.py --num-gpus 1 --config-file configs/mimdet/mimdet_vit_base_mask_rcnn_fpn_sr_0p5_800_ 1333_4xdec_coco_ 3x.py --num-machines 1", appears:

And I found the definition of encoder and decoder in "mimdet_vit_base_mask_rcnn_fpn_sr_0p5_800_ 1333_4xdec_coco_3x.py --num-machines 1", as shown in the figure:

For the encoder weights, modify them in common.py,

I look forward to your reply, if there is any disturbance, please bear with me, sincerely.

@mike-huangdj   Hello, can you run with a GPU? I see you didn't change the encoder and decoder parameters.


atom.io links broken

Go to the main website http://atom.io
Click on Discussions. Link is broken. (Assume you are a completely new user without any sign-in. Try incognito. )
I tried reporting this in the Contact form. Submitting that form is broken too.

Atom is abandonedware. Due to complications with github being acquired by microsoft, it is unclear if/when this will ever be formally acknowledged. To give an idea of how abandoned this software is:

The discussions section was deleted without notice after getting spammed despite methods existing to fix discussion board spam.
None of the original maintainers/devs have committed to the repo since 2020
There are, as far as I can tell, two official maintainers for this repo these days.
The CI pipeline is broken with no indications of being fixed

This software should not be used. Nearly all of its dependencies are ancient, and nobody is left to pass the torch. It is too big, has been touched by too many hands, and has been left to die for too long to be rescued.
If you would like to explore a de-microsfted text editor based on Electron, try VSCodium, a repackaged binary of VSCode's open source codebase with all the microsoft telemetry stripped out, and which uses an open source extension distribution platform separate again from Microsoft.
I personally find it to be quite adequate.

Really a shame how a team can build such a great software and then its just abandonned

As raer0 pointed out, this is abandonedware, the only difference now is that GitHub are sunsetting Atom, finally. https://github.blog/2022-06-08-sunsetting-atom/


Add context to payloadValidator

Hi,
For most common use cases; one is going to need to check (at least) iss, aud and iat.
Issuer and Audience are likely to be stored in bindings, so to allow this check, I propose passing ctx as a second argument to payloadValidator

Temporary published at<EMAIL_ADDRESS>in the meantime

Hey @cyrilchapon ! Sorry for the late reply! LGTM, and thanks for being interested in this project!

@cyrilchapon version 1.0.6 should contain your change. Please let me know if you have any questions! Thanks again!


common: fix user agent strings

RE: https://github.com/GoogleCloudPlatform/gcloud-common/issues/179
It looks like after modularization, we may have failed to pass UA strings at all. This fixes that, making sure they're sent with every request.
We were also not setting a UA on any calls made with gRPC.

Coverage remained the same at 100.0% when pulling 328765b5ea4b29a5fca5a4a01981f5abddbedcfc on stephenplusplus:spp--user-agents into 494907c390a4eef81cfcbe2947322a2f58d0cd18 on GoogleCloudPlatform:master.

@stephenplusplus Not trying to be a pain in the butt, but could we separate this into 2 different PRs?

All the @google-cloud/common updates.
All the modules upgrading to the latest common + packageJson fields

The way all these changes are lumped together makes testing this locally  .. challenging.

Looks like we're missing packageJson within PubSub/IAM

Added.

Changes Unknown when pulling e2851fc21efb6d564d82162475ff03d6c98e95e0 on stephenplusplus:spp--user-agents into ** on GoogleCloudPlatform:master**.


🚧 Dev ➾ e2e test MAC MINI stress check

🔗 Epic|Feature Link
No response
🚥 Definition of Done

[ ] 🎯 Planning and Design
[ ] 🏎 Dev Implementation Complete
[ ] ♾ PR Merged
[ ] 🏆 Product Acceptance

🧰 Work Description
I have had to restrict the types of e2e tests that can run in the cicd because there seems to be a limit on how much the mac minis can process before they throw random false negatives.
This task is to prove that there is a limit to what can be tested if the mac minis cannot keep up.
The Quickstart Scenario has yet to run to the point of calling for origination since, at this point, typically, there would be a possible cicd failure on a mac mini.
This task will create a PR that will include the entire quickstart scenario in the e2e tests. Observations of the stability will be made to find the limit at which the flakiness becomes untenable for cicd.
From there, we can either increase the mac minis' performance or find another way to test important features in cicd., particularly those that involve a combination of Flextesa with Taquito.
✅ To Do
create a PR with the entire Quickstart scenario E2E test
⚖ Acceptance Criteria
✨ Functional:

[ ] passing tests or report of flakiness
[ ]

🔭 Scope
🙅♂ Out of Scope:

...

tests of specific plugins
Code of Conduct

[X] I agree to follow this project's Code of Conduct

experiment concluded


Fix inconsistent theme on first launch and restart

Tested on Windows 10 version 21H1, should to be tested on Windows 11 to make sure the option Use system setting under Theme Mode works correctly

The method to get the current system theme has been changed, the previously used _theme.GetSystemTheme() returned the Windows mode theme instead of the app mode theme and caused issues when these two values were different (such as the ContentDialog having invisible buttons or text being the wrong color)

The Theme setting will now persist between application restarts

Solves https://github.com/sabihoshi/GenshinLyreMidiPlayer/issues/29

Thank you! Was not meant to merge it without asking you to test it first, but I did test it on my Windows 10 machine, could you check it up as well? I went back to ui:FontIcon since the theme color wasn't changing until you switch pages, and it seems like all that was missing was the font property Segoe MDL2 Assets which was the font I was using before. There's no need to bundle the font since it's in Windows 10 as well!

Hi, I tested the new build on Windows 10 and your solution works perfectly, the icons/buttons are also the correct size now!
The only minor issue I found was when setting the theme to Use system setting it would change back to either Dark or Light after restarting the application, this seems to be fixed by using Settings.Modify() for the AppTheme only when ToggleTheme() is triggered, like this https://github.com/DavG25/GenshinLyreMidiPlayer/commit/49befd632142a444f0cdd002b9d762a98cae9fab
Thank you again!


The solution can't handle large number of disks

Hi! Thank you for this tool. I use it on several servers, but one of then i have a problem.  Perhaps it depends of this issue.
On this server installed about 100 drives.
# time sudo -u zabbix /etc/zabbix/scripts/smartctl-lld.py getverb "hostname"
  Python version:
 3.9.18 (main, Sep  7 2023, 00:00:00)
[GCC 11.4.1 20230605 (Red Hat 11.4.1-2)]

  Sender version:
 zabbix_sender (Zabbix) 6.0.23
Revision 315e9acac58 31 October 2023, compilation time: Oct 31 2023 00:00:00

Copyright (C) 2023 Zabbix SIA
License GPLv2+: GNU GPL version 2 or later <https://www.gnu.org/licenses/>.
This is free software: you are free to change and redistribute it according to
the license. There is NO WARRANTY, to the extent permitted by law.

This product includes software developed by the OpenSSL Project
for use in the OpenSSL Toolkit (http://www.openssl.org/).

Compiled with OpenSSL 3.0.7 1 Nov 2022
Running with OpenSSL 3.0.7 1 Nov 2022

  Config's main settings:
Server=zabbix
ServerActive=zabbix
Timeout=30

/etc/zabbix/scripts/smartctl-lld.py: Could not send anything. Argument list or filepath too long. (HUGEDATA)
  Please report any issues or missing features to:
https://github.com/nobody43/zabbix-smartmontools/issues

Traceback (most recent call last):
  File "/etc/zabbix/scripts/smartctl-lld.py", line 418, in <module>
    processData(senderData, jsonData, AGENT_CONF_PATH, SENDER_WRAPPER_PATH, SENDER_PATH, DELAY, HOST, link)
  File "/etc/zabbix/scripts/sender_wrapper.py", line 163, in processData
    subprocess.Popen(cmd, stdin=subprocess.PIPE, close_fds=(not isWindows()))
  File "/usr/lib64/python3.9/subprocess.py", line 951, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/usr/lib64/python3.9/subprocess.py", line 1837, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
OSError: [Errno 7] Argument list too long: '/bin/python3'

real    1m22.396s
user    0m2.449s
sys     0m1.019s

您的邮件我已收到，谢谢合作！


Literature Basis for the MembershipInference Blackbox Attack?

Hey,
I wasn't able to find any paper or linked literature that describes what is actually implemented in the MembershipInferenceBlackboxAttack. Where does this implementation come from? Can anyone give me some source for that?
Thank's in advance,
Jakob

Hi @blauertee I'll check with the author and come back soon.

Hi @blauertee I have talked with @abigailgold, the attack is based on Shokri et al with a few additions and extensions. Let us know if you have further questions!

Thanks! I know that paper, but I thought that describes MIAs on k shadow models. The Blackbox MIA implemented in art only allows to train an attack model on a single shadow model or the target model directly right? And if so why is that?

Hi @blauertee, the Blackbox MIA in ART can receive any train and test datasets in its fit() method. Those could be derived in a number of ways, including using shadow models. But you are correct that this is assumed to be done external to the attack implementation.
BTW - it may be possible to use ART's extraction attack implementations to create such shadow models.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.10 to before v5.10.102",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=f611e8cf98ec908b9c2c0da6064a660fc6022487",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=6062d1267ff3e5570165c974bb0dc2120c9d363c"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "f611e8cf98ec908b9c2c0da6064a660fc6022487",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "6062d1267ff3e5570165c974bb0dc2120c9d363c",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "lockdep: Correct lock_classes index mapping\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.102 by commit 6062d1267ff3e5570165c974bb0dc2120c9d363c, it was introduced in version v5.10 by commit f611e8cf98ec908b9c2c0da6064a660fc6022487. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000405


Allow space application supporter to access specific environment variable group endpoints.

Issue
Allow space application supporter to access specific environment variable group endpoints.
Context
We are introducing a new role and we want to make sure it has the right access.
Expected result
A space application support should be able access the following endpoints:
GET /v3/environment_variable_groups/:name
Acceptance
A space application supporter would see the same info as a space developer assigned to the same space for these and only these environment variable group endpoints.
Documentation
When I browse to any of these endpoints on v3 docs I can see the Space Application Supporter role in the list of permitted roles with an indication that this role is not fully implemented and the permissions will be changing.

@will-gant and I are working on this one.

LGTM
Bake is passing. I did not deploy changes since this is only an update to tests.


Leave generic types in Gateway (where possible) and use JSON instead

A PR for #49.
Current problems are:

[x] KeyFieldFetcher returns an Object that can be both a list of JsonNodes or a JsonNode - maybe we can use an ArrayNode there, but I don't know if we can later access its elements if we cast back to JsonNode,
[x] There is a problem with marshaling to the JsonNode when the target is a String,
[x] Fix the problem with The QueryKeyArgumentFetcherTest.shouldFetchLongValue test. It fails because the original LongNode is later cast to the IntNode. Strangely, the problem does not arise with double and DoubleNode,
EDIT: The reason is the following: A long is converted from Java's long, i.e., long a = 5L;, into JSON, and then again into Java. However, for JSON it is the same, i.e. there is no differentiation between an int and a long.
Additional question: Is it problematic for GraphQL if it expects a long but receives an int? Probably not as it doesn't have any concept of long.
[x] Check if MirrorValueParser can work with this.resolver.fromString(element.toString()) only,
Preliminary yes.
[x] Fix the serialization error in QuickPropertyDataFetcher.get(),
[x] Check if QuickPropertyDataFetcher can be dropped (as we no longer work with Avro or Proto there),
[x] Immediately convert to JSON in SubscriptionFetcher,
[x] Solve the problem with double quotes - a custom serializer is provided, but it has not been tested yet,
[x] Check if a solution with the generic DataFetcher that works as it did for the primitive types and only uses JsonNode for Avro and Proto makes sense, i.e., if it is maintainable and extensible.

Immediately convert to JSON in SubscriptionFetcher,

This isn't covered yet, is it?

I am closing, for now, not to thwart the implementation of range queries. The issue will be revisited later to solve the problems with SubscritpionFetcher and MultiSubscriptionFetcher.


Repo name doesn't indicate this is a sample

Feedback from HackNYC: the messaging around this repo is unclear. The name would seem to indicate that it is a library, but the README indicates that it is a sample.
If it is a library, we should offer a nuget pkg w/ integration instructions, etc.
Can we include something in our namespacing like Graph.Microsoft.Sample?

Renamed and transferred the sample. This should address the potential confusions you mention.

Thank you for following up, @jamescro!


OGVPlayer: Refused to get unsafe header "Content-Range"

As the title says, when loading a sample with useOgvFallback enabled, I get this error in my console:
Refused to get unsafe header "Content-Range"

As far as I can tell, this is a CORS-related issue. I might also be the first person to have this issue, since I'm running the soundboard frontend locally but using the production API. I guess that when they're running on the same origin, there's no CORS issues.
I think it could be fixed by adding this header: Access-Control-Expose-Headers: Content-Range.
See also this line in stream-file (the library used by ogv.js): https://github.com/brion/stream-file/blob/d1b41ca2992b908d8910d5389b6a663146d5d5b3/src/backend/backend.js#L30

I labelled it with soundboard-3.0, but just realized that the same issue might exist for the main branch.

Hmm. I've tested OGV.js plenty from local to real API and on Safari as well and never ran into this somehow. I'll try to reproduce.


Refactor flutter manifest

WIP
Resolves https://github.com/flutter/flutter/issues/136092
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Closing in favor of https://github.com/flutter/flutter/pull/132985, which will overhaul asset manifest validation and parsing logic.


Wireshark cannot capture 0RTT

Hello, why can't I capture 0-RTT in Wireshark even though I wrote the code like this? The code mainly includes a server and a client. The server listens for 0-RTT using listener, err := tr.ListenEarly(tlsConf, &quic.Config{Allow0RTT: true}), and the client dials using conn, err := tr.DialEarly(context.Background(), server, tlsConfDial, nil). Then, conn.CloseWithError(0x42, "error 0x42 occurred") is executed to close the client, and a subsequent connection is made to verify if 0-RTT is implemented. However, the result is that Wireshark does not capture 0-RTT, and the subsequent connection still goes through the initialization process. I am new to QUIC, so please let me know if there is anything wrong with my practice. I am looking forward to your response. Thank you.
The main code is as follows
server：
	tr := quic.Transport{
		Conn: udpConn,
	}
	listener, err := tr.ListenEarly(tlsConf, &quic.Config{Allow0RTT: true})
	if err != nil {
		fmt.Printf("failed: %s\n", err)
		return
	}
	for {
		conn, err := listener.Accept(context.Background())
		if err != nil {
			fmt.Printf("failed connect: %s\n", err)
			continue
		}
         }

client：
        tr := quic.Transport{
		Conn: udpConn,
	}
	conn, err := tr.DialEarly(context.Background(), server, tlsConfDial, nil)
	if err != nil {
		fmt.Printf("failed: %s\n", err)
		panic(err)
	}
	conn.CloseWithError(0x42, "error 0x42 occurred")
	rconn, err := tr.DialEarly(context.Background(), server, tlsConfDial, nil)
	rconn.CloseWithError(0x42, "error 0x42 occurred")

0-RTT requires resuming a previous connection. Please familiarize yourself with the RFC before asking questions here.


YeeLight Support

List any issues that this PR fixes: fixes #394

This pull request proposes the following changes:

Adds YeeLight support.

Known issues/To do:

N/A

Any news on Yeelight Support, I can´t find it in the device tab.

Hello @SnakePin
Yeelight don't start in the device manager, I changed the IP adress and enable LAN control on the yeelight app.
do you know why it doesn't work ?
2019-05-31 15.02.37.log

It seems like your ip didn't save properly as it says on the log that it
tried to connect to <IP_ADDRESS>, can you backup your old appdata Aurora folder
and then delete it and then try again?
On Fri, May 31, 2019, 16:24 vicdilou<EMAIL_ADDRESS>wrote:

Hello @SnakePin https://github.com/SnakePin
Yeelight don't start in the device manager, I changed the IP adress and
enable LAN control on the yeelight app.
do you know why it doesn't work ?
2019-05-31 15.02.37.log
https://github.com/antonpup/Aurora/files/3241389/2019-05-<IP_ADDRESS>.log
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/antonpup/Aurora/pull/1396?email_source=notifications&email_token=AENCPYCRZCFLYRHYHI4E5UTPYERH3A5CNFSM4GPVIRU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWVGR2I#issuecomment-497707241,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AENCPYBUXOGFRQSBHZ33STDPYERH3ANCNFSM4GPVIRUQ
.


New fixer: anonymous functions spacing

I believe that correct styling should be:
fn() => 1

This is also a deafult (and PSR-12) styling in PHPStorm, but ECS now is trying to change that:
    ---------- begin diff ----------
--- Original
+++ New
@@ -23,7 +23,7 @@
             }
         }

-        fn() => 1;
+        fn () => 1;

         return $this;
     }
    ----------- end diff -----------

Applied checkers:

 * PhpCsFixer\Fixer\FunctionNotation\FunctionDeclarationFixer

It thinks that spaced version is okay:
fn () => 1

We should create some additional rule to ensure that fn() is written without space.

After #52 this stopped work.


Provide a pipeline API

What is a pipeline ?
It's a way to transform resources either at runtime (dev environment) or at startup or at build time (and then the framework doesn't care).
We could offer a pipeline API which would associate a resource to its built equivalent and map it to one single path. And then would transform it or not dependeing on the context.
Example
If we're on a dev environment :

When the user asks for assets/*.css, look for web/styles/*.scss, then send invoke my transform function and return the result.

If we're on a production environment :

Look for every file in web/styles/*.scss, if you fin their equivalent into dist/styles/*.css then it's OK, serve directly those files when assets/*.css is asked. If some are missing, please invoke my transformation function and save the result.

Pretty complicated to explain, not that easy to implement, but very useful.

So the pipeline interface could look like this :
public interface Pipeline {
    public String rawPattern();
    public String distDir();
    public String transformName(String originalName);
    public Buffer transform(Buffer originalContent);
}

In case of sass files for instance :
public class SassPipeline implements Pipeline {
    public String rawPattern() {
        return "web/raw/styles/*.scss";
    }

    public String distDir() {
        return "web/dist/styles":
    }

    public String transformName(String originalName) {
        return originalName.replace(".scss", ".css");
    }

    public Buffer transform(Buffer originalContent) {
       // invoke sass compiler
       return ...
    }
}


SCM: Option for gutter decoration ignore whitespace

The whitespaces are shown as changes in gutter decoration in editors:

I usually use git diff -w in the daily work and think it'll be great if there's an option for ignoring whitespace changes.
I've only found the setting diffEditor.ignoreTrimWhitespace which seems like it is only worked for Diff Editor not for Editors.

Unfortunately this is a technical limitation. These whitespace changes are used for computations in range staging commands. There's no easy way to remove them now.

Did this change recently? The gutter decoration used to follow the diff editor's setting for ignoring whitepace and worked great.
The gutter decorations are pretty useless to me now that they always show whitespace changes. I almost always just see one big vertical line since I often make indentation changes.

@joaomoreno Any idea about the timeline for this issue? It's not a crucial feature, sure, but it'll be nice to have it.

Looks like this was fixed in 98f5bf9b9bea2fe075d3b07fbf0edf5b3576e08b (see #130956) and this issue can be closed :tada:

Making as duplicate of https://github.com/microsoft/vscode/issues/130956


[BUG] New version breaks simple build

Version
Run redhat-actions/buildah-build@v2
/usr/bin/buildah version
Version:         1.20.0
Go Version:      go1.15.2
Image Spec:      1.0.1-dev
Runtime Spec:    1.0.2-dev
CNI Spec:        0.4.0
libcni Version:
image Version:   5.10.5
Describe the bug
Hello,
After the upgrade to 1.20.0 I can't build a simple python.
It seems that docker.io has been removed from the registry.conf and some permission issue is happening
Working build - https://github.com/gfvirga/http-liveness/runs/2242528975?check_suite_focus=true
Can't find python - https://github.com/gfvirga/http-liveness/runs/2250342456?check_suite_focus=true
Tar error - https://github.com/gfvirga/http-liveness/runs/2250371480?check_suite_focus=true
I am using the default agent from github.
Steps to reproduce, workflow links, screenshots
Run redhat-actions/buildah-build@v2
/usr/bin/buildah version
Version:         1.20.0
Go Version:      go1.15.2
Image Spec:      1.0.1-dev
Runtime Spec:    1.0.2-dev
CNI Spec:        0.4.0
libcni Version:
image Version:   5.10.5
Git Commit:
Built:           Thu Jan  1 00:00:00 1970
OS/Arch:         linux/amd64
Performing build from Dockerfile
/usr/bin/buildah bud --arch amd64 -f /home/runner/work/http-liveness/http-liveness/Dockerfile --format docker -t gfvirga/httpliveness:latest /home/runner/work/http-liveness/http-liveness
STEP 1: FROM python
Resolving "python" using unqualified-search registries (/etc/containers/registries.conf)
Getting image source signatures
Copying blob sha256:234b70d0479d7f16d7ee8d04e4ffdacc57d7d14313faf59d332f18b2e9418743
Copying blob sha256:04a31b4508b8e95fb3cb25486c4068185054895b12e0611e386a002ee9c0e07c
Copying blob sha256:6fa07a00e2f029c4b2c7f177a2b696f1b3510040cde4f5bb06ddbca98e7fbf76
Copying blob sha256:5d6f1e8117dbb1c6a57603cb4f321a861a08105a81bcc6b01b0ec2b78c8523a5
Copying blob sha256:004f1eed87df3f75f5e2a1a649fa7edd7f713d1300532fd0909bb39cd48437d7
Copying blob sha256:48c2faf66abec3dce9f54d6722ff592fce6dd4fb58a0d0b72282936c6598a3b3
Copying blob sha256:2eee1b8e253830859988454f07e1e98387cf5e3730e7eedee333aa7fd1a04be2
Copying blob sha256:1823d93d969893b61303752fa11b710c4befae6f15a775fad2c2c0bcb619cf2e
Copying blob sha256:b48672c0140e5d73a8b044e4c1c229f3cc88fd8ff645c7fc6f774c830ae1f8d4
Copying config sha256:587b1bc803b36f65923fe677467bb619b4e464d7f988377bb5c6c0a26eef044d
Writing manifest to image destination
Storing signatures
error creating build container: 2 errors occurred while pulling:
 * Error committing the finished image: error adding layer with blob "sha256:5d6f1e8117dbb1c6a57603cb4f321a861a08105a81bcc6b01b0ec2b78c8523a5": Error processing tar file(exit status 1): operation not permitted
 * Error initializing source docker://quay.io/python:latest: Error reading manifest latest in quay.io/python: StatusCode: 404, <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final/...
level=error msg="exit status 125"
Error: Error: buildah exited with code 125
Resolving "python" using unqualified-search registries (/etc/containers/registries.conf)
Getting image source signatures
Copying blob sha256:234b70d0479d7f16d7ee8d04e4ffdacc57d7d14313faf59d332f18b2e9418743
Copying blob sha256:04a31b4508b8e95fb3cb25486c4068185054895b12e0611e386a002ee9c0e07c
Copying blob sha256:6fa07a00e2f029c4b2c7f177a2b696f1b3510040cde4f5bb06ddbca98e7fbf76
Copying blob sha256:5d6f1e8117dbb1c6a57603cb4f321a861a08105a81bcc6b01b0ec2b78c8523a5
Copying blob sha256:004f1eed87df3f75f5e2a1a649fa7edd7f713d1300532fd0909bb39cd48437d7
Copying blob sha256:48c2faf66abec3dce9f54d6722ff592fce6dd4fb58a0d0b72282936c6598a3b3
Copying blob sha256:2eee1b8e253830859988454f07e1e98387cf5e3730e7eedee333aa7fd1a04be2
Copying blob sha256:1823d93d969893b61303752fa11b710c4befae6f15a775fad2c2c0bcb619cf2e
Copying blob sha256:b48672c0140e5d73a8b044e4c1c229f3cc88fd8ff645c7fc6f774c830ae1f8d4
Copying config sha256:587b1bc803b36f65923fe677467bb619b4e464d7f988377bb5c6c0a26eef044d
Writing manifest to image destination
Storing signatures
error creating build container: 2 errors occurred while pulling:
 * Error committing the finished image: error adding layer with blob "sha256:5d6f1e8117dbb1c6a57603cb4f321a861a08105a81bcc6b01b0ec2b78c8523a5": Error processing tar file(exit status 1): operation not permitted
 * Error initializing source docker://quay.io/python:latest: Error reading manifest latest in quay.io/python: StatusCode: 404, <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final/...
level=error msg="exit status 125"

I'm not sure about the "error reading manifest" but the first error is #45, there's a workaround there while we see if buildah will provide a more complete fix.

That worked! Thanks


Support for extracting multiple archives (& current directory + recursive modes)

Should we not indent console output when [multiple] is not set?
Should we keep the console output as similar to the current implementation as possible when [multiple] is not set?

Rebased & push --force-with-lease.


Messages overlay on IOS devices

Hi,
I am running iterable sdk on react native 0.63.4, and for some ios devices and not constantly I get an "overlay" of a text sourced from getMessages request. At that point in time, I don't initiate the request for getMessages. This request is only called at a later stage when my app is fully loaded.
For example, this might happen while the splash screen is still on and the app isn't fully loaded.

Please advise.

Hi @vladiRozen, Thanks for reporting this.
You can use - Iterable.inAppManager.setAutoDisplayPaused(true) when you want to prevent inapps to be shown in certain screens or time. Make sure to turn it back to false when the app is ready to show the inapp.
Reference:
Hope this helps :)


⚠ Bofrost.lu has degraded performance

In 7d0cbf7, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1532 ms

Resolved: Bofrost.lu performance has improved in 4baa419 after 6 minutes.


WIP - Kafka kerberos authentication support for collector/ingester

Signed-off-by: Ruben Vargas<EMAIL_ADDRESS>Which problem is this PR solving?

Add support for Kerberos authentication to ingester/collector

Short description of the changes

Now that sarama library has support for Kerberos authentication, this PR add support for it on the jaeger side using flags.

I just need to test this.

I've already tested this with a kafka cluster configured with Kerberos. this PR is still using a commit hash, waiting for a new release of Sarama.

Hi I am getting error saying Failed to init storage factory. Kafka client has run out of available brokers to talk to(Is your cluster reachable) ?
I error comes in SASL PlAINTEXT. My producer works fine in Plaintext.
Any help would be great


stopIteration was called without calling startIteration when using ArSceneView.getScene().callOnHierarchy()

I encountered this error for the first time for this piece of code:
public List<CustomAnchorNode> getAllAnchorNodesInScene() {
        List<CustomAnchorNode> anchorNodes = new ArrayList<>();
        arSceneView.getScene().callOnHierarchy(node -> {
            String anchorName = node.getName();
            if (anchorName.startsWith(Enums.NodeNames.ANCHOR.name())) {
                anchorNodes.add((CustomAnchorNode) node);
            }
        });
        return anchorNodes;
    }

The error occurs for this line: arSceneView.getScene().callOnHierarchy
I tried to do some digging here to see if I can understand why this is happening. I came across this. What I don't understand, however, is why would iteratingCounter ever become negative since, according to here, startIterating() is called before stopIterating(). Am I missing something here?
Some more context, the above code for getAllAnchorNodesInScene() is called immediately after I get the view binding for ArSceneView in the OnCreate() function. This is because in my program I am running a background thread where I need to get the distance of my camera to all the anchors. I start this thread immediately, I guess I should start it after a pause of 1 second? I would really appreciate an expert weighing in on this and possible explaining why this error happens in the first place because this is the first time I encountered it.

Some nodes like the Camera node might be added by Sceneform between onCreate and onResume so you can't really rely on callOnHierarchy
Why not keeping a list of your created anchor nodes on your side?

@ThomasGorisse that is actually a good idea. Thank you, I will do that now.


add back table keys header

I can shrink the white space on the bottom of the page but am stuck with the white space at the top of the page, so trying to add in a sort of header to try to reduce this?

This seems fine. idk if we will show it though on the main page but seems useful if we give people these links


Update Travis CI to use python 3.6 and 3.7

Hi 李san,
这里的Location Change Yes/ No的判断，
是只要经过了初期设定画面，即使用户不更改设定直接按Next, 都会判断为Yes是吧？
--->是的，
有Location Change设定的机型，进入Opening画面就会走Location Change Yes流程。
没有Location Change设定的机种会如何处理？
--->没有Location Change设定项的机型会按照Location Change No的流程处理。
| |
Email
|
|
<EMAIL_ADDRESS>|
Signature is customized by Netease Mail Master
On 10/21/2019 11:08, Jack Minardi wrote:
Merged #69 into master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Add color suport to Table component

Is your feature request related to a problem? Please describe.
Right now the Table component doesn't support color=.
Describe the solution you'd like
I want to be able to change the Table color with:
<Table striped color={purple}>...</Table>

It needs to support the following formats:

[ ] Solid
[ ] Row striped
[ ] Column striped
[ ] Hovered

Describe alternatives you've considered
N/A
Additional context
Standard Flowbite documentation: https://flowbite.com/docs/components/tables/#table-colors

This is on flowbite.com so we should have it.


Add list and watch permission for templates to aggregated role

Needed by NHC in order to react on template deletion
ECOPROJECT-1857

/lgtm
/retest


Fix Autoscaler CVEs

Promotes the images built here: https://github.com/aws/eks-anywhere-build-tooling/pull/2085
Issue #, if available:
https://github.com/aws/eks-anywhere-build-tooling/issues/2092
https://github.com/aws/eks-anywhere-build-tooling/issues/2084
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/lgtm

/remove-hold


refactor: telegram bot in one place

We cannot have two instances of the bot in different places.
My idea was to add a service to run a django standalone script, to allow residents to turn on/off the alarm. On the paper it was great but I can't... :no_entry_sign:
telegram_bot_1         | 2020-08-15 16:27:06,370 - telegram.vendor.ptb_urllib3.urllib3.connectionpool - WARNING - Retrying (Retry(total=2, connect=None, read=None, redirect=None)) after connection broken by 'NewConnectionError('<telegram.vendor.ptb_urllib3.urllib3.connection.VerifiedHTTPSConnection object at 0xb47e6358>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution')': /bot1017271700:AAF9IG-AIMHGAuM_eS_l4Ft4F0yVWbdN3H8/getUpdates
telegram_bot_1         | 2020-08-15 16:30:31,381 - telegram.ext.updater - ERROR - Error while getting Updates: Conflict: terminated by other getUpdates request; make sure that only one bot instance is running
^CGracefully stopping... (press Ctrl+C again to force)
Stopping webapp_telegram_bot_1 ... done

It was not this. Just, I was saving the status in the database (it's still the case). By doing so, we are calling a celery Tasks. The issue was that this service was not in the rabbit network, so celery broker was not reachable... I did not receive any error that is weird, I have to intestate on this issue. If I create a task and the broker is not reachable, I have to know it by an error.


doc: fix receiver typo

Current dependencies on/for this PR:

v2-🚀

PR #1   👈

This stack of pull requests is managed by Graphite.


Review Solr autocomplete/autosuggest index requirements

From https://docs.google.com/document/d/1m9k55NIe6ZDVVBrHIiwjVWshcmSxNeAqXUhj8qXCwng/edit
"Huda will start looking at indexing process as well as Solr features for different indexing strategies.  "
Output should be description of and decision regarding how to proceed with indexing.

Possibilities: Solr built-in autosuggest/autocomplete, sematext as used by U Ghent, or any other customizations/configurations

Soundex was also mentioned

Currently pursuing a separate index with a custom analysis field.
Details here: https://docs.google.com/document/d/1qT9gPZrk19hQE62Enk1pFtt6Gk47oMGg-Ucr8Fv4eKQ/edit

Further indexing work can be documented in different GitHub issues


Move Apache.Groovy.2.5 2.5.15 to Apache.Groovy.2 2.5.15

Microsoft Reviewers: Open in CodeFlow

Assuming the move is simply moving and not updating
[Policy] Reset Labels

Automatic Validation ended with:

Executable C:\Windows\SysWOW64\cmd.exe returned exit code: 1

(Automated response - build 881.)


Send task with "DOING" leads to misformed task in Todoist and Logseq

If I try to send a task that has "DOING" in the line, the task name gets corrupt.
See attached example!
image

Hi I don't seem to have this problem.
Does this happen consistently for you?

Yes, happens when I start the line with DOING ...

Hmm, doesn't happen for me though.

Ok, I'll check later and write some infos. Maybe my settings?

Here is how I can reproduce.

Settings (removed api token)
{
"apiToken": "xxx",
"pullDefaultProject": "---",
"pullDefaultAppend": true,
"addParentBlock": "false",
"clearTasks": true,
"sendDefaultProject": "Inbox<PHONE_NUMBER>)",
"sendDefaultDeadline": false,
"sendDefaultLabel": "",
"appendLogseqUri": "Link title",
"appendTodoistUrl": "Link content",
"disabled": false
}

@hkgnp any updates here?

Hi I am still not able to reproduce your issue though.


zigbee2mqtt update crash

Please verify that you have read and understood the guidelines.
yes
A clear and concise description of the issue.
After update, zigbee don't start. With start with cli "zigbee2mqtt" -> this message 👍
Error relocating /usr/lib/zigbee2mqtt/node_modules/unix-dgram/build/Release/unix_dgram.node: _ZN2v820EscapableHandleScopeC1EPNS_7IsolateE: symbol not found
An idea ?
What settings are you currently utilizing?

[X] Default Settings
[ ] Advanced Settings

Which Linux distribution are you employing?
Alpine
If relevant, including screenshots or a code block can be helpful in clarifying the issue.
alpine-zigbee2mqtt:~# zigbee2mqtt
Starting Zigbee2MQTT without watchdog.
[2024-11-23 09:27:12] info:     z2m: Logging to console, syslog
Error: Error relocating /usr/lib/zigbee2mqtt/node_modules/unix-dgram/build/Release/unix_dgram.node: _ZN2v820EscapableHandleScopeC1EPNS_7IsolateE: symbol not found
    at Object..node (node:internal/modules/cjs/loader:1715:18)
    at Module.load (node:internal/modules/cjs/loader:1318:32)
    at Function._load (node:internal/modules/cjs/loader:1128:12)
    at TracingChannel.traceSync (node:diagnostics_channel:315:14)
    at wrapModuleLoad (node:internal/modules/cjs/loader:218:24)
    at Module.require (node:internal/modules/cjs/loader:1340:12)
    at require (node:internal/modules/helpers:141:16)
    at bindings (/usr/lib/zigbee2mqtt/node_modules/bindings/bindings.js:112:48)
    at Object.<anonymous> (/usr/lib/zigbee2mqtt/node_modules/unix-dgram/lib/unix_dgram.js:4:34)
    at Module._compile (node:internal/modules/cjs/loader:1546:14)

Please provide detailed steps to reproduce the issue.
After run
bash -c "$(wget -qO - https://github.com/tteck/Proxmox/raw/main/ct/alpine-zigbee2mqtt.sh)"
or
bash -c "$(wget -qO - https://github.com/community-scripts/ProxmoxVE/raw/main/ct/alpine-zigbee2mqtt.sh)"

i can confirm, also on my alpine it does that.

This looks very much like a bug in Zigbee2MQTT itself, i.e. in the Alpine package.
Can you try the following? -- PLEASE MAKE A BACKUP OF THE WHOLE LXC FIRST! --
apk del zigbee2mqtt
apk add zigbee2mqtt --repository=http://dl-cdn.alpinelinux.org/alpine/edge/community
zigbee2mqtt --version


Label suggestion - Dockerize

The project author can open an issue with this label requesting help to dockerize the project.

issuehub.io is not a source for creating labels to be used on github. It is only a method by which you can search for issues that exist on github. This label currently returns 14 open issues, and as such doesn't seem like a good match for integration to the UI.


Ethereum Sync Error Ledger Live Version 1.8.2 iOS and Win 10.

Ledger Live Version 1.8.2 ios and Windows 10 Ledger Live Version.

tested on Ledger Live 1.8.2
Phone version: iPhone XS
Errorcode: <NSHTTPURLResponse : 0x281cb2ca0>
{ URL: https://exploreres.api.live.edger.co...
Something went wrong. Please retry or contact us

This was a temporary backend incident. Fixed last night.


chore: Refactor Dockerfile

Remove yarn requirement in final image
Commit the yarn release folder as recommended by them
Simplify copys and permission setting
Add extra stage for deps
Remove sharp install as Next.js 15 now includes it by default

Next I want to separate the dependencies for the two projects to reduce the image size. We use Next.js standalone output so that already reduces the dependencies on build. The problem is Nest.js currently requires the whole node_modules folder which includes Next.js dependencies it doesn't need.

I'm going to go ahead and do the above in this branch given that the Dockerfile changes a lot again anyway.


Updated common and query module

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 41.42%. Comparing base (852d282) to head (5da7d30).
Report is 103 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #251      +/-   ##
=============================================
+ Coverage      40.76%   41.42%   +0.66%
- Complexity      1322     1460     +138
=============================================
  Files            257      295      +38
  Lines          10050    11362    +1312
  Branches         982     1112     +130
=============================================
+ Hits            4097     4707     +610
- Misses          5648     6295     +647
- Partials         305      360      +55

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Error when prunning workers from old scheduler states

Hi there, we have been using luigi at work where we developed an extension for managing some data pipelines. This extension used luigi 1.3 and I've been working on a upgrade to 2.5. I think I found a bug when the scheduler loads tasks from previous pickled states. When I submit tasks via luigi.interface.build, I get the following error on the luigid side:
2017-01-18 14:47:18,248 tornado.application[56530] ERROR: Uncaught exception GET /api/add_task (<IP_ADDRESS>)
HTTPServerRequest(protocol='http', host='localhost:8082', method='GET', uri='/api/add_task', version='HTTP/1.1', remote_ip='<IP_ADDRESS>', headers={'Accept-Encoding': 'gzip, deflate', 'Content-Length': '789', 'Content-Type': 'application/x-www-form-urlencoded', 'Host': 'localhost:8082', 'Accept': '*/*', 'Connection': 'keep-alive', 'User-Agent': 'python-requests/2.4.1 CPython/3.4.5 Darwin/15.6.0'})
Traceback (most recent call last):
  File "/Users/thiago/miniconda2/envs/rendang/lib/python3.4/site-packages/tornado/web.py", line 1467, in _execute
    result = method(*self.path_args, **self.path_kwargs)
  File "/Users/thiago/miniconda2/envs/rendang/lib/python3.4/site-packages/luigi/server.py", line 77, in get
    result = getattr(self._scheduler, method)(**arguments)
  File "/Users/thiago/miniconda2/envs/rendang/lib/python3.4/site-packages/luigi/scheduler.py", line 781, in add_task
    if task.batch_id is not None:
AttributeError: 'Task' object has no attribute 'batch_id'

This then triggers a worker prune:
2017-01-18 14:47:18,248 tornado.access[56530] ERROR: 500 GET /api/add_task (<IP_ADDRESS>) 1.18ms
2017-01-18 14:47:38,831 luigi.scheduler[56530] INFO: Starting pruning of task graph
2017-01-18 14:47:38,831 tornado.application[56530] ERROR: Exception in callback <bound method Scheduler.prune of <luigi.scheduler.Scheduler object at 0x1044e0128>>
Traceback (most recent call last):
  File "/Users/thiago/miniconda2/envs/rendang/lib/python3.4/site-packages/tornado/ioloop.py", line 1041, in _run
    return self.callback()
  File "/Users/thiago/miniconda2/envs/rendang/lib/python3.4/site-packages/luigi/scheduler.py", line 680, in prune
    self._prune_workers()
  File "/Users/thiago/miniconda2/envs/rendang/lib/python3.4/site-packages/luigi/scheduler.py", line 692, in _prune_workers
    self._state.inactivate_workers(remove_workers)
  File "/Users/thiago/miniconda2/envs/rendang/lib/python3.4/site-packages/luigi/scheduler.py", line 624, in inactivate_workers
    self._remove_workers_from_tasks(delete_workers)
  File "/Users/thiago/miniconda2/envs/rendang/lib/python3.4/site-packages/luigi/scheduler.py", line 630, in _remove_workers_from_tasks
    task.workers -= workers
TypeError: unsupported operand type(s) for -=: 'set' and 'list'

Deleting state.pickle from my current file system allowed me to run things smoothly again, but this might point to a codepath that is not used very often and maybe some extra defenses need to be put in place.

It's not a bug. We usually put in backward compatibilities but we don't maintain them for ever and ever.

That said, I've for long wanted to just remove the state-file as I find it to cause 95% harm 5% good. What do you think @tgalery, when you did the migration, was it important that state was preserved? In all (2) setups I've worked with, the clients will always rebuild the dependency graphs anyway.

For us, it's not important at all, since we run tasks by deploying a luigi server for a specific purpose and then tearing it down. Should I close the issue, or leave it open so people can manifest their opinion on the matter ?

I think I close it. I've however thought of raising to the mailing group but I got lazy and didnt. :)


useSanctumUser not returning user data immediately after login but only after Hard reload

My flow is like this:
User log's in:
const { login } = useSanctumAuth();
 await login(event.data)

After this I redirect to the main page. Now if you navigate around, and want to go to a certain page that requires authentication and tries to display your username, its not working at first.
TypeError: Cannot read properties of null (reading 'firstName')
This comes from a custom composable useAuthUser I created:
export const useAuthUser = () => {
  const sanctumAuth = useSanctumAuth();
  const sanctumUser = useSanctumUser<UserModel>();

 const userObject = computed(() => {
   return sanctumUser.value === null
    ? null
    : sanctumUser.value.data;
 });

  const isAuthenticated = ref(sanctumAuth.isAuthenticated);

  const firstName = computed(() => {
    if (!unref(userObject)) {
      return null;
    }

    return unref(userObject)?.invoiceAddress?.firstName ?? null;
  });
}

If I use the following in a component:
const auth = useAuthUser();
console.log(auth.firstName.value)

it willl throw null at first, until I do a hard refresh.
Any idea what i'm doing wrong? I'm fairly new to Vue 3, so it could also be related to me doing something wrong using the composition API.

Hey @Scarwolf! I'm not sure if your code might affect the behavior, but it should work fine with the default configuration.
You can check the code of the profile page in the playground, it does almost the same thing you wanted - show identity details only for authenticated users. It is done by using sanctum:auth middleware in the page meta, thus you can guarantee that guests will be prevented from accessing it.
As for hard refresh, it shouldn't be a problem since after successful login, the module refreshes the user identity in the state and propagates it to all references due to reactivity.
So, I suggest using proper middleware for your page and trying to use useSanctumUser composable directly instead of your useAuthUser implementation to make sure that it is not a problem related to the package.
Let me know if it helps.

Closing due to inactivity, feel free to re-open


ZeroDivisionError on number of devices

System Info
Windows 10
Accelerate Version: 0.18.0.dev0

Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
I am having the following error:
per_gpu = module_sizes[""] // (num_devices - 1 if low_zero else num_devices)

ZeroDivisionError: float divmod()

The line around 390 in modeling.py:
max_memory = {i: torch.cuda.mem_get_info(i)[0] for i in range(torch.cuda.device_count())}
produces {0:0} which means that my GPU has 0 memory. The line must be changed to:
max_memory = {i: torch.cuda.mem_get_info(i)[1] for i in range(torch.cuda.device_count())}
to get the correct memory size of a  GPU.
Expected behavior
No error.

Your GPU has no memory available when you execute the code. Using your workaround would still result in an error (out-of-memory error). You have to free GPU memory to be able to use it.

I think there is still a bug.

From the PyTorch official doc:

Returns the global free and total GPU memory occupied for a given device using cudaMemGetInfo.

So the bug is that you have 0 space available on your GPU, maybe because you imported TensorFlow which gobbles everything? Hard to know without seeing the code you run.


Improve experience when writing crate-level documentation

Once already in a documentation block, RA should maybe do stuff like formatting //! even when there's spaces in between, and auto-adding it when you add a new line.

See https://rust-analyzer.github.io/manual.html#on-enter.

Closing, since it's already available (thanks @chetankhilosiya for checking). Enabling it by default is a different story (https://github.com/rust-analyzer/rust-analyzer/pull/6037).


Update Japanese translation

Coverage remained the same at 38.881% when pulling 7b9884c40bc9a3f616f188dbb7ca8abc2c44e4f0 on MMP0:patch-47 into ca271e2ac6c3fa836e74a202653a458522d723c1 on NyaaPantsu:dev.


Problems  with periods in Series name

From<EMAIL_ADDRESS>on July 09, 2013 03:13:25
What steps will reproduce the problem? 1. Search for a series with successive abbreviations, like Avengers A.I. or WildC.A.T.S.
2. No result
3.  What do you see instead? Some better handling of this. It makes these cases harder to deal with manually right now. In the Avengers case, you have to go through all titles that include Avengers and pick it manually (that's a lotta books!) What version of the product are you using? On what operating system? 1.1.19-beta Mac OS Please provide any additional information below.
Original issue: http://code.google.com/p/comictagger/issues/detail?id=30

From<EMAIL_ADDRESS>on July 08, 2013 18:34:43
This is odd.  I think this used to work.  I just did a manual test of the ComicVine API, and searching with and without periods for "Avengers A.I." and "Avengers AI" and it worked both times, but the for some reason fails in the app.  It might have to do with URL escaping of the string.  Gonna look into it soon.  This kind of stuff is highest priority.  Thanks
Status: Accepted

From<EMAIL_ADDRESS>on March 24, 2014 08:00:04
This seems mostly mitigated by fixes on the Comic Vine side.  Closing for now.
Status: Done


Added missing Korean translations

@kimdhoe I’d appreciate if you could review this. :)

Thank you! :)


Implements a benchmark

Codecov Report
Attention: 128 lines in your changes are missing coverage. Please review.

Comparison is base (7c11b33) 71.50% compared to head (47c45b3) 70.72%.

Files
Patch %
Lines

experimental_experiment/torch_bench/dort_bench.py
0.00%
100 Missing :warning:

...perimental_experiment/torch_exp/_aten_functions.py
60.00%
14 Missing :warning:

experimental_experiment/bench_run.py
85.07%
10 Missing :warning:

experimental_experiment/args.py
92.30%
2 Missing :warning:

...imental_experiment/torch_helper/training_helper.py
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
- Coverage   71.50%   70.72%   -0.78%
==========================================
  Files          26       31       +5
  Lines        3850     4062     +212
==========================================
+ Hits         2753     2873     +120
- Misses       1097     1189      +92

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Terminal lacks focus indication

Testing #9695
Through two separate situations, I've been able to get into a state in which the terminal lacks focus indication through the cursor:

Simply disable cursor blinking (related to #9753). When moving focus to the editor and back to the terminal, there is no indication that the terminal has focus at all.
Or, leave the blinking cursor, run the htop command, exit it with Q and the terminal will then lack focus information when moving focus to the editor and back again.

Fixed by https://github.com/Microsoft/vscode/commit/44cd0bee8299f5dcdb40ae31a0f5965ee29cedc9


Allow to set FilterGroups for Build Triggers in CodeBuild Project

Looks like there's the Webhook property defined in the Layer 1 resource. According to the documentation, it should be possible to set FilterGroups as well - https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-codebuild-project-projecttriggers.html#cfn-codebuild-project-projecttriggers-filtergroups
I was trying to fall back to L1 but it's missing there.
const projectResource = project.node.findChild(
      "Resource"
    ) as codebuild.CfnProject;

 projectResource.propertyOverrides.triggers = { // doesn't take `FilterGroups'
    webhook: true
}

Would appreciate hints on how to make it work.

Got it working with this workaround:
    const projectResource = project.node.findChild(
      "Resource"
    ) as codebuild.CfnProject;

    projectResource.addPropertyOverride("Triggers", {
      Webhook: true,
      FilterGroups: [
        [
          {
            Pattern: "PULL_REQUEST_CREATED,PULL_REQUEST_UPDATED",
            Type: "EVENT"
          }
        ]
      ]
    });


Request with tough cookie causes Seg Fault

Hello,
Thank you for your work. The follow code causes a segmentation fault, crashing my app.
request({url:"http://www.google.com", jar:myJar}, funciton(err, resp, body){

	//crashes if cookies are not 'ready' here
	console.log(JSON.stringify(myJar));
	//If output: "mycookie=1234;"   SEGMENTAION FAULT
	//If output: "{_jar...}"        FINE
	request({url:"http://www.google.com", jar:myJar}, funciton(err, resp, body){})
})

Unfortunately, I lost the stack trace, which could only be obtained by running through gdb with a specially compiled version of node.
But it seems that whatever data is getting passed into tough cookie, is causing a problem at this line:
https://github.com/goinstant/tough-cookie/blob/master/lib/cookie.js#L770
The debugger says this: CHECK(object->IsSmi()) failed
Thank you.

Seems like a bug in the one of the unstable versions of node, rather than issue with tough-cookie.
@osxpinot Can you confirm that issue still exists with the stable node 12 or 10 or io.js?


Add additional maven options

Added additional maven options as referenced in issue #9

closing in favor of #13


fix: Typo in history

PR Summary
[comment]: Transaction history has typo

Checklist

[ ] There are breaking changes
[ ] I've added/changed/removed ENV variable(s)
[ ] I checked whether I should update the docs and did so by updating /docs

Description
[comment]: Small typo

@rustairov Thank you for your attention to detail 🧡


[hotfix] Add the feature description of flink-operator in the document

What is the purpose of the change
Improve the feature description of flink-operator in the document
Brief change log
Add Flink Job Autoscaler in _index.md
Verifying this change

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Manually verified the change by running a 4 node cluster with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changes to the CustomResourceDescriptors: (yes / no)
Core observer or reconciler logic that is regularly executed: (yes / no)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks!


Type of ref for Map with generic in values is abnormal.

Vue version
3.3.10
Link to minimal reproduction
https://play.vuejs.org/#eNp9U8lu2zAQ/ZUBL5EBV0aSmyob6OJDA6QtGvdk+aBKY5uJRBJcFAeG/r1DUrYVJIgNSZztzZuFR/ZFqbRzyDKWm0pzZcGgdQqaUuzmBbOmYItC8FZJbeEIGrfTSrbKWayhh62WLVxR/NXnQhTCviiEvwY1zOFYCABeZyBc+w/11IuibDEDYzUXu0L0PmTrRGW5FOAMPtjSYr5aJJMYXUlhLKUsyaPD+1IRLBFIBD4DSfmADD5iQgROEQ2n15yEyDNfrTeLhEDnizFuqXUGZCLP9SZEA6zTNB3lS7uycZg+4YtJJpt0K/WyrPYJyWcs/yOkVDmzT96G7tB698kk+vbDV1OPtfCBIXFU+2cwDNAjvNA/CKWFiNC+WIifysUvVt9TVeeO+olQi8KI3lKkeSfX08u8rkejKtjdXgqE7xILRik9xvsINyOEm1cIv/e84Qrupea0TANI/OezuHO0YSRYbFVDfEkCyGvehQP4OrTJ4tmroftEk6Dl9AbgIjaFQUaNHrQpr8PexvkcjwEj9Yygp74FpNklQ0g4yPnszINNaf+px1u+Sx+NFHRJQo0F86vFG9S/lN9dKis7jaxgZdPI57ugs9rhMDiK2WP19I7+0Ry8jhqlkVh21OmzzZaaNiialw8/8UDns7GVtWvI+wPjHzSycZ5jdPvqRE20R36B7Y9wvelSrszyYFGYU1GeaFi34F8wuunfPij9Qvc2vT2tKev/Ay/CctU=
Steps to reproduce

Create ref with Map that use generic

const reactiveMap = ref(new Map<number, T>());

Try to assign value from crated map in other place that expect this generic:

const test: T = reactiveMap.value.get(1)

You can check typescript error on line 14 in App.vue file in reproduction link.
What is expected?
Code works without any typescript errors.
What is actually happening?
Got error:
 Argument of type 'UnwrapRefSimple<T>[]' is not assignable to parameter of type 'T[]'.
  Type 'UnwrapRefSimple<T>' is not assignable to type 'T'.
    'T' could be instantiated with an arbitrary type which could be unrelated to 'UnwrapRefSimple<T>'

System Info
No response
Any additional comments?
This issue happens on 3.3.10 versions and works well on 3.3.9
I may assume that issue related to this changes: https://github.com/vuejs/core/pull/8960/files#diff-81475b10580b705fb8421b8c430ac63220938646c0e09141be79c85b0ea392ffR499
that was included in lates release.

Values of object types are recursively unwrapped. Due to TypeScriptinability to recognize whether the values in your generic type include the Ref type, it recursively unwraps the values of the object. As a result, an UnwrapRefSimple<T> type is obtained. You can try handling this issue as follows:
  const reactiveMap: Ref<Map<number, T>> = ref(new Map());
  const arr: T[] = []
  arr.push(reactiveMap.value.get(1)) // T

Thanks for the answer.
Yeah, i know that i can workaround this issue by using typecast.
The your version or like this:
const reactiveMap = ref(new Map()) as Ref<Map<number, T>>;
But my point that previously it was worked correct and doesn't required any additional actions in the code. And latest changes bring this side effect so it looks like regression.
This is why i decided to report this issue.

@sinkersan the current type is correct, @Alfred-Skyblue is correct on the explanation.
the same issue is/was present since a long time ago:
function test<T>(): Ref<T> {
  const r = ref<T>(null)
  return r;
}

The solution is, if you're sure T is not a ref you need to cast it
 const r = ref<T>(null) as Ref<T>

Just bear in mind that the type might not be exactly the same, because of the unwrapping.
The reason before was working was because of the type issue where collections were being ignored, but it was fixed recently and not a regression.


Add FID score

It would be nice the add the FID score to evaluate as a metric for generative models.
cc @patrickvonplaten @anton-l

If it is possible I will be happy to develop this metric

Hi, can I work on this metric, please?


Instance Name in "New Tab" view

Feature Request
Display instance name in the header when you "Open instance in new tab".
Feature Information:
When you open and instance with "Manage" it shows the instance name under Section Title, but when you open "In new tab" it does not show what instance you are one.  It would be very useful to just had the same name that is under the section title when in "new Tab" mode.

I confirm:

[x] that I have searched for an existing feature request matching the description.

I also would like to see this implemented. I usually have multiple instances open "in new tab" and after a while I'll need to go through the configuration settings for an instance to find identifiable info and remind myself which instance I am working on. Any new info regarding this feature request?

Yep that would be a very useful feature as I also use multiple tabs for managing different instances. I've allocated slightly different memory settings to be able to tell the difference between them as a workaround, but displaying their names would be ever better!

Yep that would be a very useful feature as I also use multiple tabs for managing different instances. I've allocated slightly different memory settings to be able to tell the difference between them as a workaround, but displaying their names would be ever better!

This is 100% needed. I am using a controller/target deployment and there is no way to tell what instance I am managing. Normally you can tell by looking in the top right-hand corner:

But now that I am using a target/controller setup, there is no information there at all. I don't think this is a 'low priority issue, as it makes it impossible to multitask.

After a year of dealing with not having this, I would give my ex gf's-father's left eyeball for this feature. Between the 4 games for which I run servers, I have a total of 11 instances and not having this features has messed an instance or two up at times. Yes, I got lazy and made an alteration to the wrong instance a time or two. But seriously, throw the instance name in the top-right or left corner and call it good.

It doesn't seem like a very complicated thing to implement. This post was made nearly 2 years ago now, I expected that this would have been something that was implemented a long time ago due to its simplicity.

Also, I forgot to mention this suggestion I made earlier @Zjajo007 which I'm sure you would appreciate: https://archive.cubecoders.com/thread/change-web-browser-tab-title-to-actually-display-the-currently-opened-amp-instance-name/
That in itself would be 10x better than the previously mentioned solution because you can easily tell which instance you're going to manage simply by looking at the tab title. Makes a lot more sense than just seeing 50x of the same titles:

@LeehamElectronics, honestly, an 'Quality of Service' update would make Amp-life simpler. Although with everything in life, as soon you're given one thing that's asked for...you always expect to get more. I feel that this platform is losing steam, which is saddening because it's so awesome.

@LeehamElectronics, honestly, any 'Quality of Service' update would make Amp-life simpler. Although with everything in life, as soon you're given one thing that's asked for...you always expect to get more. I feel that this platform is losing steam, which is saddening because it's so awesome.

What platform? Sorry I don't follow

Implemented in latest build :)

great thanks, when does that become available in nightly builds?

About 3AM tomorrow

About 3AM tomorrow

Just tested it, and upgraded my controller, target, and game server instances to this build. Sadly this feature is not functional in target/controller setups. I tested it by logging into a target ADS directly and I can see that the feature is working but from the controller, it's not. I tested this with and without the branding feature for custom tab titles. Would you be able to test this on a target controller setup? Would also be nice if the branding configuration could remain functional when you patch this, for example, the tab title should display as "CustomTitleName (amp-instance-name)"
Cheers @PhonicUK

This feature is not working in latest build (<IP_ADDRESS>) and would be really helpfull

Right now this isn't possible in the 'New Tab' view - only when managing an instance via the controller in the normal fashion.