Add only works for two layers of the same size

Hi, so im working on setting up a fast style transfer network in code instead of importing it from a .pb file, so i can start improving it.
However im having some issues trying to setup a residual layer, heres the network.
        styleNet.start
            ->> Convolution(convSize: ConvSize(outputChannels: 32, kernelSize: 9, stride: 1), neuronType: .none, id: "Variable_0")
            ->> InstanceNorm(shiftModifier: "1", scaleModifier: "2", id: "Variable_")
            ->> Neuron( type: .relu)
            ->> Convolution(convSize: ConvSize(outputChannels: 64, kernelSize: 3, stride: 2), neuronType: .none, id: "Variable_3")
            ->> InstanceNorm(shiftModifier: "4", scaleModifier: "5", id: "Variable_")
            ->> Neuron( type: .relu)
            ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 2), neuronType: .none, id: "Variable_6")
            ->> InstanceNorm(shiftModifier: "7", scaleModifier: "8", id: "Variable_")
            ->> Neuron( type: .relu)
            ->> ResidualLayer(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), layers:
                Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_9")
                    ->> InstanceNorm(shiftModifier: "10", scaleModifier: "11", id: "Variable_")
                    ->> Neuron( type: .relu)
                    ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_12")
                    ->> InstanceNorm(shiftModifier: "13", scaleModifier: "14", id: "Variable_"))

When initializing the following error appears: assertion failed: Add works for two layers of the same size: file /Users/Andre/Downloads/Bender-fix-issue-38/Sources/Layers/Add.swift, line 23
I also print the layers in the network.

"PRINTING LAYERS"
": Bender.Start"
": Bender.Convolution"
": Bender.InstanceNorm"
": Bender.Neuron"
": Bender.Convolution"
": Bender.InstanceNorm"
": Bender.Neuron"
": Bender.Convolution"
": Bender.InstanceNorm"
": Bender.Neuron"
": Bender.Dummy"
": Bender.Identity"
": Bender.Add"
assertion failed: Add works for two layers of the same size: file /Users/Andre/Downloads/Bender-fix-issue-38/Sources/Layers/Add.swift, line 23

If you wondering what kind of network im trying to emulate its this one:
https://github.com/lengstrom/fast-style-transfer/blob/master/src/transform.py
def net(image):
    conv1 = _conv_layer(image, 32, 9, 1)
    conv2 = _conv_layer(conv1, 64, 3, 2)
    conv3 = _conv_layer(conv2, 128, 3, 2)
    resid1 = _residual_block(conv3, 3)
    resid2 = _residual_block(resid1, 3)
    resid3 = _residual_block(resid2, 3)
    resid4 = _residual_block(resid3, 3)
    resid5 = _residual_block(resid4, 3)
    conv_t1 = _conv_tranpose_layer(resid5, 64, 3, 2)
    conv_t2 = _conv_tranpose_layer(conv_t1, 32, 3, 2)
    conv_t3 = _conv_layer(conv_t2, 3, 9, 1, relu=False)
    preds = tf.nn.tanh(conv_t3) * 150 + 255./2
    return preds

Note: i have changed instance norm input to allow specific shift/scale modifiers, its a temporary way to just setup the weight ids.

The layers do sanity checks of the sizes. In this case is the Add, which comes from ResidualLayer.
So it seems that what comes before the residual layer and what it yields have different size. I'm trying to figure out why, as it looks good.

A few tips while I tackle it :smile:
You can create your own blocks, as in the file you cite, by extending CompositeLayer and ResidualLayer.
Convolution should not use bias here, as the example you provide uses TF convolutions, which have no bias.
I'm noticing there's a useless convSize in ResidualLayer. Gonna remove it.

It seems to me that the outputsize variable even inside convolution layers isn't mantained, like its reference is set to nill when the network initialize is called.

so Add is not having 2 inputs?

The Add has 2 inputs, the output size of the second input is null.

@AndreJFBico Do you have a repo to take a look?

@dernster Yeah i can provide it, ill update the main post when i have it.
In the mean time this is the network im running right now.
`
styleNet = Network(device: device, inputSize: inputSize, parameterLoader: parameterLoader)
    //We have in total 48 layers and 48 weight variables
    styleNet.start
        ->> Convolution(convSize: ConvSize(outputChannels: 32, kernelSize: 9, stride: 1), neuronType: .none, id: "Variable_0")
        ->> InstanceNorm(shiftModifier: "1", scaleModifier: "2", id: "Variable_")
        ->> Neuron( type: .relu)
        ->> Convolution(convSize: ConvSize(outputChannels: 64, kernelSize: 3, stride: 2), neuronType: .none, id: "Variable_3")
        ->> InstanceNorm(shiftModifier: "4", scaleModifier: "5", id: "Variable_")
        ->> Neuron( type: .relu)
        ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 2), neuronType: .none, id: "Variable_6")
        ->> InstanceNorm(shiftModifier: "7", scaleModifier: "8", id: "Variable_")
        ->> Neuron( type: .relu)
        ->> [Identity(), (
             Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_9")
                ->> InstanceNorm(shiftModifier: "10", scaleModifier: "11", id: "Variable_")
                ->> Neuron( type: .relu)
                ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_12")
                ->> InstanceNorm(shiftModifier: "13", scaleModifier: "14", id: "Variable_")
                ->> Identity())]
        ->> Add()
        ->> [Identity(),(
            Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_15")
                ->> InstanceNorm(shiftModifier: "16", scaleModifier: "17", id: "Variable_")
                ->> Neuron( type: .relu)
                ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_18")
                ->> InstanceNorm(shiftModifier: "19", scaleModifier: "20", id: "Variable_")
                ->> Identity())]
        ->> Add()
        ->> [Identity(),(
            Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_21")
                ->> InstanceNorm( shiftModifier: "22", scaleModifier: "23", id: "Variable_")
                ->> Neuron( type: .relu)
                ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_24")
                ->> InstanceNorm(shiftModifier: "25", scaleModifier: "26", id: "Variable_")
                ->> Identity())]
        ->> Add()
        ->> [Identity(),(
            Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_27")
                ->> InstanceNorm(shiftModifier: "28", scaleModifier: "29", id: "Variable_")
                ->> Neuron( type: .relu)
                ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_30")
                ->> InstanceNorm(shiftModifier: "31", scaleModifier: "32", id: "Variable_")
                ->> Identity())]
        ->> Add()
        ->> [Identity(),(
            Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_33")
                ->> InstanceNorm(shiftModifier: "34", scaleModifier: "35", id: "Variable_")
                ->> Neuron( type: .relu)
                ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, id: "Variable_36")
                ->> InstanceNorm(shiftModifier: "37", scaleModifier: "38", id: "Variable_")
                ->> Identity())]
        ->> Add()
        ->> ConvTranspose(size: ConvSize(outputChannels: 64, kernelSize: 3, stride: 2), id: "Variable_39")
        ->> InstanceNorm(shiftModifier: "40", scaleModifier: "41", id: "Variable_")
        ->> Neuron( type: .relu)
        ->> ConvTranspose(size: ConvSize(outputChannels: 32, kernelSize: 3, stride: 2), id: "Variable_42")
        ->> InstanceNorm(shiftModifier: "43", scaleModifier: "44", id: "Variable_")
        ->> Neuron( type: .relu)
        ->> Convolution(convSize: ConvSize(outputChannels: 3, kernelSize: 9, stride: 1), neuronType: .none, id: "Variable_45")
        ->> InstanceNorm( shiftModifier: "46", scaleModifier: "47", id: "Variable_")
        ->> Neuron(type: .tanh)
        ->> ImageLinearTransform()`

It runs even though the end result is still not working, the way i circumvented the add issue was to add a identity layer next to the instance norm layer, that way the outputSize is passed correctly to the output layer.

If the output is not ok, maybe what you can yield different layers outputs and compare them to the python implementation, to see if they match, in order to be able to find where the error is.
But have you tried saving the protobuf from the python code with benderthon and loading it with bender?

Good suggestion, in terms of trying to import a protobuf, yes i have done that and it works properly, im trying to define the network in code so that i can understand it better and also work with it.

@AndreJFBico Hi! We couldn't reproduce the original issue, could you please provide a repo with it? Additionally, if you can provide an example of a larger input size causing a crash would be helpful.

@AndreJFBico did you figure this out?

Yeah, now we were able to reproduce the error, but we still don't know what is the cause. And we don't have an ETA for this currently.
If you want, you can go ahead and try to find it. We'll check it out when we have time (I hope it's in the following weeks).

Im sorry that i never got that repo ready, its just i started changing the core of bender itself and i no longer had things as they were done originally.
Well i sort of figured it out, however i couldn't make it work with the .pb file straight off the bat.
And since i wanted to experiment with the code itself i converted the network of lengstrom fast style transfer to benders.
        styleNet.start
            ->> Convolution(convSize: ConvSize(outputChannels: 32, kernelSize: 9, stride: 1), neuronType: .none, useBias: false, id: "Variable_0")
            ->> InstanceNorm(shiftModifier: "1", scaleModifier: "2", id: "Variable_")
            ->> Neuron( type: .elu)
            ->> Convolution(convSize: ConvSize(outputChannels: 64, kernelSize: 3, stride: 2), neuronType: .none, useBias: false, id: "Variable_3")
            ->> InstanceNorm(shiftModifier: "4", scaleModifier: "5", id: "Variable_")
            ->> Neuron( type: .elu)
            ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 2), neuronType: .none, useBias: false, id: "Variable_6")
            ->> InstanceNorm(shiftModifier: "7", scaleModifier: "8", id: "Variable_")
            ->> Neuron( type: .elu)
            ->> [Identity(), (
                Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_9")
                    ->> InstanceNorm(shiftModifier: "10", scaleModifier: "11", id: "Variable_")
                    ->> Neuron( type: .elu)
                    ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_12")
                    ->> InstanceNorm(shiftModifier: "13", scaleModifier: "14", id: "Variable_")
                    ->> Identity())]
            ->> Add()
            ->> [Identity(),(
                Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_15")
                    ->> InstanceNorm(shiftModifier: "16", scaleModifier: "17", id: "Variable_")
                    ->> Neuron( type: .elu)
                    ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_18")
                    ->> InstanceNorm(shiftModifier: "19", scaleModifier: "20", id: "Variable_")
                    ->> Identity())]
            ->> Add()
            ->> [Identity(),(
                Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_21")
                    ->> InstanceNorm( shiftModifier: "22", scaleModifier: "23", id: "Variable_")
                    ->> Neuron( type: .elu)
                    ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_24")
                    ->> InstanceNorm(shiftModifier: "25", scaleModifier: "26", id: "Variable_")
                    ->> Identity())]
            ->> Add()
            ->> [Identity(),(
                Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_27")
                    ->> InstanceNorm(shiftModifier: "28", scaleModifier: "29", id: "Variable_")
                    ->> Neuron( type: .elu)
                    ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_30")
                    ->> InstanceNorm(shiftModifier: "31", scaleModifier: "32", id: "Variable_")
                    ->> Identity())]
            ->> Add()
            ->> [Identity(),(
                Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_33")
                    ->> InstanceNorm(shiftModifier: "34", scaleModifier: "35", id: "Variable_")
                    ->> Neuron( type: .elu)
                    ->> Convolution(convSize: ConvSize(outputChannels: 128, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_36")
                    ->> InstanceNorm(shiftModifier: "37", scaleModifier: "38", id: "Variable_")
                    ->> Identity())]
            ->> Add()
            ->> Upscale(size: ConvSize(outputChannels: 0, kernelSize: 0, stride: 2))
            ->> Convolution(convSize: ConvSize(outputChannels: 64, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_39")
            ->> InstanceNorm(shiftModifier: "40", scaleModifier: "41", id: "Variable_")
            ->> Neuron( type: .relu)
            ->> Upscale(size: ConvSize(outputChannels: 0, kernelSize: 0, stride: 2))
            ->> Convolution(convSize: ConvSize(outputChannels: 32, kernelSize: 3, stride: 1), neuronType: .none, useBias: false, id: "Variable_42")
            ->> InstanceNorm(shiftModifier: "43", scaleModifier: "44", id: "Variable_")
            ->> Neuron( type: .relu)
            ->> Convolution(convSize: ConvSize(outputChannels: 3, kernelSize: 9, stride: 1), neuronType: .none, useBias: false, id: "Variable_45")
            ->> InstanceNorm( shiftModifier: "46", scaleModifier: "47", id: "Variable_")
            ->> Neuron(type: .tanh)
            ->> ImageLinearTransform()

Its a one on one conversion from the following file https://github.com/lengstrom/fast-style-transfer/blob/master/src/transform.py
I also changed a bit how the weights are searched but the principle should be the same.
About the add works for two layers of the same size issue, i figured it out that i had to add a identity layer next the instance normalization layer before an add layer, that way the issue disappeared.
I tried looking into the code of how the layers were setup but i found nothing problematic, its just somehow the second incoming node of the add layer looses its convsize atribute, i never really found out why.
If they manage to fix this somehow i bet your conversion should be simple and painless.
Cheers.

Does Bender by chance have notes on how they generated their pb files?

The mnist sample came from the benderthon sample. The style transfer one probably came directly from the repo we were talking about, I don't know why it isn't working now. But we will take a look at this error as soon as we can.

@mdramos I updated benderthon so the sample uses a simpler way to generate the protobuf. Maybe take a look at it.

ok thanks! yea I actually got benderthon working just fine yesterday

I am using Using lengstrom project. While using protobuf file i faced
"Fatal error: Index out of range"
var kernelWidth: Int {
return Int(dim[1].size)
}
What is the Issue ?
test.pb.zip

@mdramos  what changes you have made it to working ?

The issue with your graph seems to be the ExpandDims at the beginning. I am looking into it.

There are two issues with your graph. The first one is fixed with #111 and has to do with the ExpandDims.
The second one is that Mul and Add with scalar are not supported yet. From what I saw they are only used at the end to scale the final result. What you can do there is to cut the graph after the Tanh (when freezing) node and then add a postprocessing layer to do the scaling like this:
Neuron(type: ActivationNeuronType.custom(neuron: MPSCNNNeuronLinear(device: Device.shared, a: 2.0, b: -1)), id: "scale_neuron")

where a and b are the scale and offset.
If there are any other questions please open a new issue.

Working ....
preds= tf.add(tf.nn.tanh(conv_t3)*150, 255./2,name="preds")
how you  calculated a and b as 2 and -1. there is a variation in outimage in python and ios ?
What value we need to put for a and b ?

2 and -1 are an example.
MPSCNNNeuronLinear documentation says it calculates ax + b. So yours would be something like 150 and 255/2


Image deployment hotfix, build image to tagged version

What has changed:

Updated Image deployment to tagged version release
Updated git workflow to release when a tag is created

Waiting for completing test in forked repository

During testing was found that the version of the appVersion needs to be without the "v" prefix.
When deploying a image we are able to set the latest version to the tag, and the deployment image will be pointing to the appVersion tag

LGTM, removing "placeholder" as image tag from default values should now make the chart default to your AppVersion


Mouse Pointer crosshairs: setting to toggle vertical crosshair

Description of the new feature / enhancement
I like to use the crosshairs as a reading ruler, increasing the thickness to 50px is the trick. However, the vertical crosshair is a mild annoyance while using this tool as a reading ruler, I have to stick it to either the right or left side to avoid it.
Having a toggle to show/hide the vertical arm would be nice.
Scenario when this would be used?

as a reading ruler
to check the alignment of things on UI 😅

Supporting information
No response

Hello and thanks for your message. I think this request (use case) is too different from the intention of the crosshairs. If you want a single horizontal line, you might want to try the Screen Ruler. But i predict that will cause issues with scrolling and such...

@Jay-o-Way ,
This is how i use the crosshairs

My Settings:

And its perfect and if the toggle is provided its usefulness will extend beyond its scope.
If the feature/toggle gets added it is great, if it doesn't it's still great. I still will love using it.
Thanks for the awesome tool :)


Build Nektar++ in output dir.

Builds nektar++ in the spack prefix directory for the stage. Avoids users having to use "--keep-stage" to avoid files being removed that are pointed to by nektar++ cmake files (and not copied to output dir).

Does this still respect the ^python argument for building NekPy?

I've amended the boost requirements to include boost-numpy when +python (fixes dependency issues on my machine). I think fixing the install such that "from NekPy import *" works out of the box is an issue for a separate PR (it works if ran in the build_tree directory)

Avoids users having to use "--keep-stage" to avoid files being removed that are pointed to by nektar++ cmake files (and not copied to output dir).

Are you referring to the BLAS/LAPACK binaries which it builds? This can be better solved by modifying he CMake arguments to ensure it will find the ones already installed by Spack and doesn't need to build its own, inefficient ones. This will be part of a pull request I will be raising imminently.

Yes - these libraries were one of the issues. I think we also were having issues with how the Python (NekPy) was being (or was not being) installed. Hopefully this copied build directory approach is a temporary fix until cmake changes are made upstream.

I found I had to install specific python packages e.g. numba during the spack  install step in order to have access to the right versions after the fact.

I found I had to install specific python packages e.g. numba during the spack install step in order to have access to the right versions after the fact.

Not sure I follow what you're saying. Do you mean that you had to modify the package file to install some specific packages during the build step? What do you mean by "the right version"? Of which package? What is "the right version"?

I had to find the version of pyhton that spack installed in the spack path
and install things there. Only possible by having an eagle eye on the build
process
On Fri, 16 Sept 2022, 13:29 Chris MacMackin, @.***>
wrote:

I found I had to install specific python packages e.g. numba during the
spack install step in order to have access to the right versions after the
fact.
Not sure I follow what you're saying. Do you mean that you had to modify
the package file to install some specific packages during the build step?
What do you mean by "the right version"? Of which package? What is "the
right version"?
—
Reply to this email directly, view it on GitHub
https://github.com/ExCALIBUR-NEPTUNE/NESO-Spack/pull/1#issuecomment-1249304767,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADWNA6XZXGN34KZFBEIRPXDV6RR37ANCNFSM6AAAAAAQIS4I7U
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


drivers: nrf_power_clock: Migrate to DTS.

This PR migrates the nrf_power_clock driver to DTS.
This change is needed for proper device power management.

Thank you for pointing that @aescolar!

The changes look ok to me I am just concerned about the stuff we still keep in the Kconfig; could these also be moved to DTS? E.g. the 32kHZ clock source - could it go out of the driver's Kconfig?

This might be done later. Now I have to port all Nordic stuff to DTS in order to move forward with power management.

Codecov Report

Merging #13677 into master will increase coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master   #13677      +/-   ##
==========================================
+ Coverage   52.51%   52.59%   +0.07%
==========================================
  Files         329      329
  Lines       46910    46922      +12
  Branches    10811    10817       +6
==========================================
+ Hits        24637    24678      +41
+ Misses      17340    17310      -30
- Partials     4933     4934       +1

Impacted Files
Coverage Δ

drivers/clock_control/nrf_power_clock.c
50% <ø> (-2.11%)
:arrow_down:

subsys/bluetooth/controller/ll_sw/ctrl.c
54.46% <ø> (ø)
:arrow_up:

.../bluetooth/controller/ll_sw/nordic/lll/lll_clock.c
50% <ø> (ø)
:arrow_up:

subsys/bluetooth/controller/ll_sw/ll.c
73.58% <100%> (ø)
:arrow_up:

subsys/bluetooth/controller/ll_sw/nordic/lll/lll.c
60.12% <100%> (ø)
:arrow_up:

drivers/timer/nrf_rtc_timer.c
92.53% <100%> (+0.6%)
:arrow_up:

boards/posix/nrf52_bsim/bstests_entry.c
57.6% <0%> (-3.27%)
:arrow_down:

lib/os/fdtable.c
42.22% <0%> (-0.48%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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


Server crashed after restarting

Bug Report
Please answer these questions before submitting your issue. Thanks!

What did you do?
If possible, provide a recipe for reproducing the error.

Replica server restarted and crashed after a downtime of the cloud host.

What did you expect to see?

Restart normally.

What did you see instead?

Coredump stack:
Program terminated with signal 6, Aborted.
#0  0x00007f25689a51d7 in raise () from /lib64/libc.so.6
Missing separate debuginfos, use: debuginfo-install glibc-2.17-157.el7_3.1.x86_64 keyutils-libs-1.5.8-3.el7.x86_64 krb5-libs-1.14.1-27.el7_3.x86_64 libcom_err-1.42.9-9.el7.x86_64 libgcc-4.8.5-28.el7_5.1.x86_64 libselinux-2.5-6.el7.x86_64 pcre-8.32-15.el7_2.1.x86_64 zlib-1.2.7-17.el7.x86_64
(gdb) #0  0x00007f25689a51d7 in raise () from /lib64/libc.so.6
#1  0x00007f25689a68c8 in abort () from /lib64/libc.so.6
#2  0x00007f256c4e49fe in dsn_coredump ()
    at /home/wutao1/pegasus-release/rdsn/src/core/core/service_api_c.cpp:76
#3  0x00000000006e9284 in pegasus::server::pegasus_server_impl::cancel_background_work (this=0x19b320800, wait=<optimized out>)
    at /home/wutao1/pegasus-release/src/server/pegasus_server_impl.cpp:1615
#4  0x00007f256c3d0a37 in dsn::replication::replica::close (this=0x4420ff80)
    at /home/wutao1/pegasus-release/rdsn/src/dist/replication/lib/replica.cpp:394
#5  0x00007f256c42ef6c in dsn::replication::replica_stub::close_replica (
    this=0x3236580, r=...)
    at /home/wutao1/pegasus-release/rdsn/src/dist/replication/lib/replica_stub.cpp:1872
#6  0x00007f256c42f1c4 in operator() (__closure=<optimized out>)
    at /home/wutao1/pegasus-release/rdsn/src/dist/replication/lib/replica_stub.cpp:1854
#7  std::_Function_handler<void(), dsn::replication::replica_stub::begin_close_replica(dsn::replication::replica_ptr)::__lambda30>::_M_invoke(const std::_Any_data &) (__functor=...) at /home/wutao1/app/include/c++/4.8.2/functional:2071
#8  0x00007f256c4f6cd9 in dsn::task::exec_internal (this=this@entry=0xc558a2f)
    at /home/wutao1/pegasus-release/rdsn/src/core/core/task.cpp:180
#9  0x00007f256c50aa6d in dsn::task_worker::loop (this=0x2ec9550)
    at /home/wutao1/pegasus-release/rdsn/src/core/core/task_worker.cpp:211
#10 0x00007f256c50ac39 in dsn::task_worker::run_internal (this=0x2ec9550)
    at /home/wutao1/pegasus-release/rdsn/src/core/core/task_worker.cpp:191
#11 0x00007f25692fd600 in std::(anonymous namespace)::execute_native_thread_routine (__p=<optimized out>)
    at /home/qinzuoyan/git.xiaomi/pegasus/toolchain/objdir/../gcc-4.8.2/libstdc++-v3/src/c++11/thread.cc:84
#12 0x00007f2569f70dc5 in start_thread () from /lib64/libpthread.so.0
#13 0x00007f2568a6773d in clone () from /lib64/libc.so.6
(gdb) quit

What version of Pegasus are you using?
pegasus-server-1.11.6-9f4e5ae-glibc2.12-release

https://github.com/apache/incubator-pegasus/blob/v1.11.6/src/server/pegasus_server_impl.cpp#L1613-L1617
void pegasus_server_impl::cancel_background_work(bool wait)
{
    dassert(_db != nullptr, ""); // cash here
    rocksdb::CancelAllBackgroundWork(_db, wait);
}


[bug]: Invoke opens to blank page (Resolved)

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
8GB
What version did you experience this issue on?
33.0.1
What happened?
Steps to reproduce:

Clean install of 3.0.1
Custom install directory: E:\
Chrome Version 115.0.5790.110 (Official Build) (64-bit)
Launched Web UI - Blank.
Reinstalled Web UI - same
Ran Automated fix - same
Lots of googling...
Found: https://stackoverflow.com/questions/23977690/setting-the-value-of-dataurl-exceeded-the-quota
Ran command in Chrome console:
localStorage.clear();

Issue Resolved
Screenshot of Windows Console below (Fixed and could not recreate Chrome Console error output)
Not necessarily a bug, but posting here for Dev's & community awareness.
Screenshots

Additional context
Issue resolved
Contact Details
discord: joehanner

Going to go ahead and close this as it's been resolved. Appreciate you calling it!


Backup codes

It is not possible to know which codes have already been used.
[Images removed]

I removed the images that showed your backup codes, but you need to regenerate them immediately.

I removed the images that showed your backup codes, but you need to regenerate them immediately.

Nice. I regenerated the codes as soon as I opened this issue, thanks for the concern.


Error

http://scratch.mit.edu/projects/21554369
http://scratchv.usa.cc/#21554369
Error: XHR Error
    at XMLHttpRequest.xhr.onerror (http://scratchv.usa.cc/phosphorus.js:225:21)

The request apparently didn't go through. It's an API error.

Hi I no longer work on this since Phosphorus just fixed all of the bugs that this fixed. So please just use Phosphorus :)


New Movies not being updated

Flask server seems to be working, but not updating anything. Is there anything I need to configure to make this work?
INFO:__main__:Successfully connected to server: SubZero
[2024-10-14 03:57:38] Starting webhook server for processing new movies...
INFO:__main__:Starting webhook server for processing new movies...
 * Serving Flask app 'edition-manager'
 * Debug mode: off```

In the Plex server settings, navigate to Webhooks, click on Add Webhook, and enter your Flask server address http://Docker host IP address:8089 and Save Changes. (Non-Plex Pass subscribers do not need to fill this.)
or
In the Plex server settings, navigate to Webhooks, click on Add Webhook, and enter your Flask server address http://IP address of the device running the script:8089 and Save Changes. (Non-Plex Pass subscribers do not need to fill this.)
Note: The add editions for new movies mode requires the server administrator account to be subscribed to Plex Pass in order to use.

Thanks!


Make on Windows link not working

Thanks to @epogrebnyak for reporting.

Hello from a year later! lol
Any idea what this link was supposed to be?

Hm, not sure, but there are a fair number of options now.
I'm not on Windows, so would appreciate testing/recommendations/PR. I think the main options are:

Windows Subsystem for Linux
Cygwin
MinGW

That said, I'm not sure that I'd recommend any of those options where you expect a lot of Windows developers. Either a batch script, powershell, or a Python-based DAG runner (snakemake, luigi, airflow) is probably less headache.

Hey, thanks for such a quick response!
Trying to figure out/proliferate Python best practices at my org of mostly Windows users. I'm currently using Git bash from the Git for Windows suite (which uses MinGW internally) and then, regrettably, steering folks here if they want to add make.
So emphasis on "headache" in your last sentence, haha
I'll give cygwin a shot today, though. Thanks again!

We've been thinking about experimenting with snakemake, which is Python-based so should run without additional installation pain. Would be very curious to look at your snakemake file for inspiration if you go that way!

Hey again.
So cygwin comes even barer than Git Bash, out of the box. You can, however, specify all kinds of packages at the time of install, including make. Problem here, though, is the size of that scrollbar.

So unless you know what you're looking for, it's pretty intimidating.
Couple solutions here:

You can always rerun the setup-XXX.exe to upgrade/add any packages you find out later that you want, which seems pretty painless.
It's possible to just script away your entire setup with a .bat file, once you figure out what works for you.

But it'd probably take awhile to figure needle/haystack all of the good stuff from everything that's available. What I've got now will suffice, I think.

And looking at snakemake, I like the idea, but unless it can natively run my existing makefiles, it feels a bit like this.

@NapsterInBlue What if you were to choco install make --source=cygwin and avoid the huge list? Having a fifteenth package manager on my machine doesn't feel great.

Fixed in #355


spec: add test for basic authentication

Checking on #1362 , the HttpAuthHandlerFactory::CreateDefault does create handlers for basic, negotiate, ntlm and digest . As for #1318 we probably need to pass URLSecurityManager using this method , anyone working on windows can use this as example and implement on brightray side.
/cc @zcbenz , @paulcbetts thoughts ?

@deepak1556 Good find! Let me work on this today

@deepak1556 If you wanted to ditch the server-side, http://httpbin.org's digest auth test will work too

I believe you're right, here's where we end up if you don't override the default URLSecurityManager:
https://code.google.com/p/chromium/codesearch#chromium/src/net/http/url_security_manager.cc&sq=package:chromium&l=24

So, this is a good start, but I think the actual issues here aren't caught by this PR (though with @zcbenz's blessing I still think this is a good test to have just for sanity's sake). Here are the three main things:

Integrated / NTLM authentication is disabled, even though the browser doesn't need any information to send it across. I think I've fixed that here: https://github.com/atom/brightray/pull/104

In network_delegate.cc (https://github.com/atom/brightray/blob/master/browser/network_delegate.cc#L85), when OnAuthRequired fires, we always return AUTH_REQUIRED_RESPONSE_NO_ACTION - this effectively means that at times where Google Chrome would pop a native dialog to ask for the password, we just say "Didn't work" and fail it.

Instead, we probably need to mark all of these requests as AUTH_REQUIRED_RESPONSE_IO_PENDING, then call out to a callback provided by the user (i.e. something like app.setNetworkCredentialCallback, that lets you provide a callback which gives creds). We probably also need a default implementation that drops down a user/pass prompt.

When we are prompted for credentials from a proxy server that uses Basic Auth, we fail similarly. I haven't debugged down to see if this is the same code path or a separate one, but if it was the same one, that'd be super cool.

So, this is a good start, but I think the actual issues here aren't caught by this PR

Yup just sent this PR  for sanity check :) @zcbenz suggested not to rely on external service for tests on one of the commits, so sticking with a local server.

Integrated / NTLM authentication is disabled, even though the browser doesn't need any user interaction to send it across. I think I've fixed that here: atom/brightray#104

:+1:

Instead, we probably need to mark all of these requests as AUTH_REQUIRED_RESPONSE_IO_PENDING, then call out to a callback provided by the user (i.e. something like app.setNetworkCredentialCallback, that lets you provide a callback which gives creds). We probably also need a default implementation that drops down a user/pass prompt.

Yup thats exactly it, but we can just ditch default implementation and leave it to user. We dont have a dialog method designed for prompt yet.

When we are prompted for credentials from a proxy server that uses Basic Auth, we fail similarly.

You probably need to specify the authentication target

This PR looks good to me, for other issues we can do in separate PRs.


Fix doc tests in Metric Conversion bonus exercise under  Advanced Pattern Matching

The Metric.convert/2 function is supposed to always return a float.

But some doc tests (2 in particular) return integers

Good catch! Marking as good first issue.


New version: eza-community.eza version 0.20.4

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/eza-community/eza/releases/download/v0.20.4/eza.exe_x86_64-pc-windows-gnu.zip
Status Code: 200
(Automated message - build 897)


docs(api): define and load liquid methods

Overview
Related to RLIQ-314.
This is a revision to the labware section of the API docs. The new content is in the file new_labware.rst. This PR adds a new section that describes the define_liquid and load_liquid methods. The new section is titled "Well Labels".
Also, as part of the revision, I've added an anchor tag to the "Commands" section of tutorial.rst for a cross reference link.
Sandbox demo link: http://sandbox.docs.opentrons.com/api-label-methods/v2/new_labware.html#well-labels
Test Plan
Testing was through local builds and checked for:

Grammar, style, spelling, element formatting (e.g. code block).
There's a table with complicated elements in it. Those are working.
Checked Sphinx directives and related links connecting text to methods and other docs.

Additional reviews can always help.
Changelog

Updates existing documentation to include description and examples for define_liquid and load_liquid.
Adds an anchor to the "Commands" section in tutorial.rst that's referenced from the new content.

Review requests

Evaluate text for accuracy and clarity.
Evaluate code sample. As a partial code sample (not meant to be complete), is it accurate?

Risk assessment
Low risk. Text change to API docs.

Committed some prose and code edits. We can take one more run through once #12269 merges and any loose ends in RQA-546 are tied up, then :shipit:


Crowd linux

Changes proposed ✍
What

Added config for support URL in the integrations page
Added show/hide config for "Build your own" integrations

Why
LF users need to be redirected to LF's own Jira support link and the build your own integration is not supported by the LF's instance of crowd.dev

 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.

Can we merge this?


Actually making the clang format test properly fail sanity.

Green enough

Sanity is passing, and since this is a high maintenance PR, I'd be really keen to get it in soon.

CLA's are fine.


port to jackson-databind 2.5.x

hi
create a patch for use jackson-databind-2.5.0
--- org.linkedin.util-groovy/src/main/java/org/linkedin/util/json/jackson/MetadataStyleSerializerFactory.java	2014-03-15 02:00:02.000000000 +0100
+++ org.linkedin.util-groovy/src/main/java/org/linkedin/util/json/jackson/MetadataStyleSerializerFactory.java.jackson-databind-2.5.0	2015-04-14 15:52:07.051170519 +0200
@@ -23,6 +23,7 @@
import com.fasterxml.jackson.databind.module.SimpleSerializers;
import com.fasterxml.jackson.databind.ser.BeanSerializerFactory;
import com.fasterxml.jackson.databind.ser.std.BooleanSerializer;
+import com.fasterxml.jackson.databind.ser.std.NumberSerializer;
import com.fasterxml.jackson.databind.ser.std.NumberSerializers;
import com.fasterxml.jackson.databind.ser.std.StringSerializer;
import com.fasterxml.jackson.databind.ser.std.ToStringSerializer;
@@ -53,8 +54,8 @@
new NumberSerializers.IntLikeSerializer();
NumberSerializers.IntegerSerializer integerSerializer =
new NumberSerializers.IntegerSerializer();

NumberSerializers.NumberSerializer numberSerializer =

 new NumberSerializers.NumberSerializer();

NumberSerializer numberSerializer =

 new NumberSerializer();

NumberSerializers.LongSerializer longSerializer =
new NumberSerializers.LongSerializer();
NumberSerializers.FloatSerializer floatSerializer =

regards

Hi

I am not sure I understand the format of the message you sent: it looks like a patch but is not one. Did you want to do a pull request?
can you please provide some details on why to upgrade?

Thanks
Yan

Hi,
thanks for the fast reply. I use and mantain FasterXML jackson packages on Fedora [1].
Practice our policy of packaging, we import only the
latest version of the software, and encourage to do the
same to other realities.
Sorry, for the malformed patch, but for me it's just a suggestion
for your use or just for my.
regards
[1] https://fedoraproject.org/wiki/Overview

I will definitely look into it for a future version (like for example when upgrading to java 8) and bringing all libraries up to date. I do understand your philosophy but it is hard to just upgrade one thing without having massive regression. This project is used by glu and any change anywhere can have impact (for example glu uses json to store its model and depend on json pretty printing to compute a checksum of the model... if the latest version of jackson does it differently (or introduces a bug that is non existent in the version currently in use), then it will break functionalities).
This is why in general, I prefer to keep dependencies as stable as possible and then when a new major version is released, then bring all libraries up to date => that way we can test more deeply all sorts of potential regressions introduced by the upgrade and people are also aware that the release contains potential breaking changes and should be very cautious.

Yes, i are aware that updating can produce compatibility/regression problems related
About, Crockfords json.org library is a very well known root of evil license problems.
Please try to replace it with one of the many free alternatives.
http://www.sonatype.com/people/2012/03/use-json-well-youd-better-not-be-evil/
http://tanguy.ortolo.eu/blog/article46/json-license
Android apparently has a replacement:
https://github.com/icedrobot/android-core/commit/014aa1a70743ae7e6ec0de1555f5b43d43844cf4

https://android.googlesource.com/platform/libcore/+/android-5.1.0_r5/json/src/main/java/org/json

Thank you for the heads up. I created a ticket to keep track of it: https://github.com/pongasoft/utils-misc/issues/7


Nesting Dialogs results in no scroll

When I create nested Dialogs, it removes the scroll on the body after they are all hidden. I tried to use webpackbin but it gives me an error (Unable to update sandbox), maybe I'm using it wrong. Here is the sample code I've made:
class DialogOverDialog extends Component {
  state = {
    open: false,
    openOther: false
  }

  renderOtherDialog () {
    return (
      <Dialog
        open={this.state.openOther}
        onRequestClose={() => this.setState({openOther: false})}
      >
        <DialogTitle>I Should be in a higher layer</DialogTitle>
        <DialogActions>
          <Button onClick={() => this.setState({openOther: false})} primary>Close</Button>
        </DialogActions>
      </Dialog>
    )
  }

  render () {
    return (
      <div>
        <Dialog
          open={this.state.open}
          onRequestClose={() => this.setState({open: false})}
        >
          <DialogTitle>Use Google's location service?</DialogTitle>
          <DialogActions>
            <Button onClick={() => this.setState({open: false})} primary>Disagree</Button>
            <Button onClick={() => this.setState({openOther: true})} primary>Open other dialog</Button>
          </DialogActions>
        </Dialog>
        {this.renderOtherDialog()}
        <button onClick={() => this.setState({open: true})}>OpenDialog</button>
      </div>
    )
  }
}

Material-UI: 1.0.0-alpha.9
React: 15.5.3
Browser: Chromium 57.0.2987.98 Built on Ubuntu , running on Ubuntu 16.10 (64-bit)

@DanielRamosAcosta You shouldn't nest dialogs:
https://material.io/guidelines/components/dialogs.html#dialogs-behavior

Dialog prominence
Dialogs should never be obscured by other elements or appear partially on screen. Dialogs always > retain focus until dismissed or a required action has been taken, such as choosing a setting.

Dialogs should avoid:

Opening dialogs from within a dialog
Containing scrolling content

Ok perfect. But I'll reflect it in the Dialog docs. Thanks!

@DanielRamosAcosta Possibly. There's balance though between that and reproducing the entire spec! :smile:


Filter locations without java executable

Backport of #23643
Fixes: #24439

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

WARN: The milestone of this PR is 7.x backport, but the base branch version is 7.6.2

I've triggered a build for you.


add snappy

https://github.com/google/snappy.git
set(snappy_URL https://github.com/google/snappy.git)
set(snappy_TAG "55924d11095df25ab25c405fadfe93d0a46f82eb")

https://github.com/tensorflow/tensorflow/blob/7f84d88d39f236e5c0cea492a2248782e696c972/tensorflow/contrib/cmake/external/snappy.cmake#L17-L18

Upstream CMake looks clean, and this was already added by @chfast (thanks!) using the tagged upstream release.  I see no need for a hunter-packages fork.   Closing.


Simple sum of two numbers

Added python program to add two numbers.
This can be run by
python3 sum_2_numbers.py

However, is this the right way to create a pull request?

Put in the required comments


Transport: remove redundant error in return from getNodeURL

There are no  error prone cases in getNodeURL.
So the second returning parameter looks redundant.

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

Just a small note on the thinking behind the original implementation: while it doesn't return, in fact, any error at the moment, it's easy to imagine that it might in future, as any getXYZ() method is prone to. So it's more or less a defensive way of coding. (When it would, later, return an error, the calling code should correctly handle it already.)
Of course, an equally valid approach is to return just the value, and when the need to return an error arises, change the method signature — since it's unexported  —, and add the error handling in the calling code. So, in other words, no problems with the patch.
(Small note about commit message again, notice how changes to the estransport sub-package are prefixed with Transport: . It makes it easier to read and discuss the Git log over time.)

I completely agree with the point that MethodName(params) (returningObject, error) is a common case but if you are make it for public methods. But not for private. We can change it any time when refactor the code. Currently it's even a bit confusing (you expect something from the method signature) and pushes you to write other redundant code.

So it's more or less a defensive way of coding.

For public method it make sense. Just in order do not change major version in future.

(When it would, later, return an error, the calling code should correctly handle it already.)

We can easily change the method and parameters then.

(Small note about commit message again, notice how changes to the estransport sub-package are prefixed with Transport: . It makes it easier to read and discuss the Git log over time.)

Thanks! I'll change it.
And thanks for your comment.

Right, as I was saying above, no worries on my side with the patch!

LGTM


error debug

File "main.py", line 104, in
tf.app.run()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 44, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "main.py", line 75, in main
se_model = SEGAN(sess, FLAGS, udevices)
File "/home/kevin/tensorflow_examples/segan/model.py", line 106, in init
self.build_model(args)
File "/home/kevin/tensorflow_examples/segan/model.py", line 122, in build_model
self.build_model_single_gpu(idx)
File "/home/kevin/tensorflow_examples/segan/model.py", line 173, in build_model_single_gpu
do_prelu=do_prelu)
File "/home/kevin/tensorflow_examples/segan/generator.py", line 190, in call
h_i = tf.concat(2, [z, h_i])
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/array_ops.py", line 1047, in concat
dtype=dtypes.int32).get_shape(
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 651, in convert_to_tensor
as_ref=False)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 716, in internal_convert_to_tensor
ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/constant_op.py", line 176, in _constant_tensor_conversion_function
return constant(v, dtype=dtype, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/constant_op.py", line 165, in constant
tensor_util.make_tensor_proto(value, dtype=dtype, shape=shape, verify_shape=verify_shape))
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/tensor_util.py", line 367, in make_tensor_proto
_AssertCompatible(values, dtype)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/tensor_util.py", line 302, in _AssertCompatible
(dtype.name, repr(mismatch), type(mismatch).name))
TypeError: Expected int32, got <tensorflow.python.ops.variables.Variable object at 0x7f1336a42dd0> of type 'Variable' instead.

i got this eroor with tensorflow 0.12.1,could you help me

Have you installed the requirements in requirements.txt? This way you can ensure you have the appropriate software.


Adds error case

See https://github.com/probot/probot/issues/568 for reasoning.
This adds the additional error message as a case.
cc/ @andymckay

Would it be possible to add a test case for this?

Done! Of these scenarios, only 2 of the 5 had tests, so I added tests for all 3.


Add external classname to LinkElment type in slate package for external link.

Hey, I would love to fix this issue, can I pick this up

@kanishq1 The issue was already fixed.
@iFlameing Please remember to link your PRs to the issue, so that the issue gets closed automatically when the PR is merged.


Add inject-Default-SA Transformer

Add an injectDefaultSA transfor function (manifestival.Transformer)
to inject a specific service account as the default-service-account
in config-defaults configMap in payload (release.yaml) being processed
Moves constants, flags etc into a flags package
Updates codebase w.r.t flags package path
Signed-off-by: Nikhil Thomas<EMAIL_ADDRESS>(cherry picked from commit 8de6dccfb15ce72cc4443cc1d0b565ca7efb4e86)

/lgtm


Repo Map - Future Work

Issue
https://aider.chat/2023/10/22/repomap.html

How can we help and contribute back!
How can we help contributing this feature (part1) ?
but it’s also an important foundation for future work on automatically finding all the code which will need to be changed (1).

So that we do not need to add and tell which files to change!
// @hariharpatel1
No response

This is something I really want to have the time to attempt. Though I don't see that soon.
It's essentially a Graph RAG problem, the repomap is good to go, you just need a means of scoring files so that those that are more relevant are selected.

I'm also interested in that materia. Repo-map is great, but I think we can have 2 mayor improvements.

Detailed repo map: maybe including comments or a description for each file, metadata, etc.
Finding EXACTLY (maybe more files but never less) what files are needed to generate a solution and the edit them. I think a good solution here is RAG with a Vector store, but it seems to be a little slow compared to how fast aider is. Maybe this doesn't matter much because genAI models will be faster in the future. I also read in docs something about graph ranking algorithms.

Do know any other ways to solve this? I'm going to look at that Graph Rag algorithm!
Maybe in the discord we can find more details about how to contribute.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


Rust Analyser + Symlink to Apps on MacOs = 🤯

Hello, thank you for awesome template.
I encountered issues using it with rust-analyser, it seems that it tried to analyse everything (ok) and even went through symlinks (not ok). Project could not be compiled and everything I got was:
Blocking waiting for file lock on build directory

I was able to solve the issue by deleting the symlink to Applications.
Is this symlink necessary?

The link is used in macOS builds. Mac users can drag and drop the game into their Applications directory (which seems to be standard for installing mac applications).
Maybe you can tell rust-analyser to ignore the build directory? You are using VS code, right? In that case, it would be awesome if we could add a configuration preventing this problem in .vscode. So others don't have to figure it out again.

If that is not possible, we could try to create the symlink during packaging instead of having it in the repository.

Ok, so here is what I have tried unsuccessfully :(
.vscode/settings.json
{
  "rust-analyzer.files.excludeDirs": ["build"],
  "files.watcherExclude": {
    ...
    "**/build/macos/**": true
  }
}

Oh, just found out that it may be not our fault )))
rust-analyzer/rust-analyzer/issues/7755
So...

I just pushed a commit that removes the symlink and instead creates it only during the macOS build. Could you please verify that your issues with the symlink are now gone? Since you have access to a mac: could you please also check if the latest macOS build is still installable via drag and drop to /Applications?

Both the original issue seems to be gone, and I can install the test app from the .dmg by drag and drop to /Applications.
Thanks for this! I think this issue can be closed now.


Style updates 200815

Looks like it's just the first commit (to SQ1-1j-i.yml) that needs fixing for the tests to pass.

:+1:


[BUG] I got a problem with some custom emojis

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
There is an emoji font I want to use in my game, after days of trying I figured out how to extract the emoji fonts and it seems like I succeed getting them working in the game, however they aren't shown correctly as you guys can see in the image below :

How do I fix this? please help me get these emojis working with the colors, I would really like to have them in my game.

I won't be able to help you here because I wouldn't know myself, sorry.

Acknowledgement of preliminary instructions

[x]  I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
There is an emoji font I want to use in my game, after days of trying I figured out how to extract the emoji fonts and it seems like I succeed getting them working in the game, however they aren't shown correctly as you guys can see in the image below :  How do I fix this? please help me get these emojis working with the colors, I would really like to have them in my game.

what's name emojifont?


[BUG]  app has no network connection after opening it.

Describe the bug
After installing it using sidestore, I set up jit-less and then added the app, which can be opened normally but cannot obtain network connection. It is not just one app, but all apps have no network. I don't know why this happens. When I tried to use version 2.1.1, I found that the network was normal.
Instructions to reproduce
All installed apps have no network
What version of LiveContainer are you using?
2.1.2
Other info
No response

Try Livecontainer 3.0.0 which can be downloaded from Action tab, and check if you allow Livecontainer to use network data.

@boa-z Yes, I used the newly built version of action and it can connect to the Internet.

@boa-z Yes, I used the newly built version of action and it can connect to the Internet.


Fix wrong parse paramaters  uglifyjs commands when using http_proxy configs

PR Title format:
[UNOMI-XXX] Pull request title with JIRA reference

Please add a meaningful description for your change here

Please following this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Format the pull request title like [UNOMI-XXX] - Title of the pull request
[ ] Provide integration tests for your changes, especially if you are changing the behavior of existing code or adding
significant new parts of code.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
Copy the description to the related JIRA issue
[ ] Run mvn clean install -P integration-tests to make sure basic checks pass. A more thorough check will be
performed on your pull request automatically.

If this is your first contribution, you have to read the Contribution Guidelines
If your pull request is about ~20 lines of code you don't need to sign an Individual Contributor License Agreement
if you are unsure please ask on the developers list.
To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[ ] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

Hello @khangtruongbinh thanks for your PR. I don't understand however how the auto clean relates to http_proxy parameters ?
Also could you please create a JIRA in the Unomi JIRA and add it to the title of this PR as is requested in the PR template ?
Regards,
Serge...

Closing because of no activity.


Invalid dates

cluster dates are weird. invalid date and the ticks are showing.

It looks like some of the date formats on the popups are still not displaying correctly.


Added Kadane's Maximum Subarray Sum

Checklist

[x] I've looked at the contribution guidelines.
[x] This pull request is complete and ready for review.

Description

Added Kadane's Maximum Subarray Sum

Hey @emrahu, thanks for the submission! Kadane's Algorithm looks like a great add to the SAC repo.
I think it'd be great to expand the README a little before we merge this PR. Specifcially, it'd be great to add context around the code of Kadane's Algorithm and explaining why and how it works, as opposed to just linking to https://www.geeksforgeeks.org/largest-sum-contiguous-subarray/. Would you mind adding this to the readme?
A good model for what a SAC readme looks like if Linked List
Thanks!


add sonarlint, sonarqube

We applied eslint and prettier on frontend proejct.
We need to do similar work on backend project.
I found sonarlint and sonarqube are about lint.
If there is a formatter on java side like prettier and we can share the setting like .prettierrc, we need to apply it.

@jzi040941
If you don't mind, would you please work on this issue?
As my research, we can apply sonarlint with our individual IDE and then connect it with sonarqube sharing team rule.
Also, we can apply sonarqube on CI phase before merging PR.

ESlint, prettier, valiation on CI phase too.


ENH - Add plausible script

This PR adds the Plausible script so we can start collecting analytics (ethically, so we do not need cookies wording)
Closes #15

Thank you!


Errors in repository avatar rendering from 1.19.4 to 1.20.0

Description
Upgrading from 1.19.4 to 1.20.0 fails complaining error in rendering repository avatars. Here is the messages from logfile:
2023/07/18 09:52:17 routers/web/base.go:110:2() [E] [64b644b1-3] Error whilst rendering repo-avatars 5-8df08b7be0a47f48a5a31bec514c7cc1. Error:
write tcp <IP_ADDRESS>:3000-><IP_ADDRESS>:59989: write: broken pipe
2023/07/18 09:52:17 routers/web/base.go:110:2() [E] [64b644b1-4] Error whilst rendering repo-avatars 14-14a21215b6a0716f80af584b3456c6a8. Error:
write tcp <IP_ADDRESS>:3000-><IP_ADDRESS>:59991: write: broken pipe
2023/07/18 09:52:17 routers/web/base.go:110:2() [E] [64b644b1-5] Error whilst rendering repo-avatars 13-ea6ffdfad98d5ca7e7e670ffce7fae37. Error:
write tcp <IP_ADDRESS>:3000-><IP_ADDRESS>:59992: write: broken pipe

Gitea Version
1.20.0
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
2.25.1
Operating System
Linux
How are you running Gitea?
gitea binary downloaded from ypur site, started with systemd
Database
PostgreSQL

No issues in my case....

Maybe the client closed the browser tab before avatar was finished transferring? I think it may be benign, but we shouldn't really log such errors either.

Whether all the repository avatars are broken or some of them?

UPDATE: errors were due to obsolete directives in app.ini file. I have updated them and now the service works regularly. The log outputs are however confusing, since they signaled me the error I have reported in the first post and I did not realize the true cause of he problem.


video() does not work with Python 3 and OpenCV 3

The newly implemented video() is not compatible with Python 3 and OpenCV 3.
In OpenCV 3, fourcc is a new API cv2.VideoWriter_fourcc(). Could you maybe add an if-else condition for this ?

+1 for fix


Complete Css (Must include another third party CSS Framework)

Please feel free to include others in this responsibility.


Support for Nuxt

Fix #48
Fix #61
support for nuxt, check it's in the browser, if not disable it.

Hey, thank you for your contribution. Very helpful. Just curious - would it not be cleaner to put the conditional around the entire module? This is kind of messy.

Sorry, but can you do something with this PR? Needs the opportunity to use this plugin with SSR?

@Kirill255 will take care soon.

Kirill: try this until then:
https://docs.npmjs.com/cli/install

@FranzSkuffka I solved my problem three days ago. Don't worry.

@Kirill255 can you share your problem & solution?

@FranzSkuffka Sure, you can read about my problem above. And you can try to solve it just using this https://docs.npmjs.com/cli/install

@FranzSkuffka OK, I will do it.

Can you confirm 1.1.16 to work with nuxt?

@FranzSkuffka  I tried the latest version today with nuxt. I am also getting the error - document is not defined. @RouillerRomain  Can you please help me here. I dint understand

by simply adding the 'Separate File For Nuxt Condition' in the plug-in folder and calling it from a page

Can you please give a small example, will be very helpful.

@arihantdaga try creating a file called vue2-medium-editor.client.js in the plugins folder of your nuxt app then including that in your nuxt.config.js
plugins: [
    '~/plugins/vue2-medium-editor.client'
  ]

The contents for that file should be standard as so:
import Vue from 'vue'
import editor from 'vue2-medium-editor'

Vue.component('medium-editor', editor)

The important bit here is the .client in the file name, this tells nuxt to only load this plugin file on the client side where we have the document available.

@basselAhmed Worked great. But i have a doubt, if we register in the client, will it be loaded globally ?
I want to keep the initial package size to minimum and want to load this component only in one page.
How can we that ?

@arihantdaga Yes it'll be loaded globally, I don't think that will be possible with nuxt unless you set your nuxt mode to spa instead of universal, that will disable server-side rendering altogether though.

Ok.

@basselAhmed  I think i found a way. Async Components in vue.
I used this instead -
import Vue from 'vue'
Vue.component('editor', function (resolve){
         require(['vue2-medium-editor'], resolve)
});

And inside my page where i wanted to use editor, simply wrapped it in no-ssr tag.


Move lindera-cil to another repository

Move lindera-cil to another repository.
Currently, the lindera-cli package is managed in the lindera repository as a member of the workspace.
Keeping the lindera repository for library crates only and moving the binary crates like lindera-cli to a separate repository.

Created a repository.
https://github.com/lindera-morphology/lindera-cli


Modifying ID Categories for Medical Insurance Cards & Identification Cards

Describe the feature
The product team should reach a consensus before making changes, but Amanda a case manager from Safe Harbor Easton asked us to change the current categories 'Medicaid Card' and 'Driver's License' to be more general. Her clients won't always have Medicaid as their insurance, some have private insurance. Similarly, her clients sometimes don't have a driver's license and instead only have an identification card or a photo ID. Also, her clients sometimes have multiple health insurances not just one (primary & secondary).
Product Team should answer the following questions before modifying or making additional categories:

Does there need to be a unique category for medicaid cards when applying for benefits OR will a general category for medical insurance cards suffice?
Does there need to be a distinction between primary insurance and secondary insurance cards for benefit applications?
Does there need to be a unique category for driver's license when applying for benefits OR will a general category for identification cards suffice?

User Stories
See description
Link to Figma design
N/A

@crchong1 We decided to change the "Medicaid Card" to "Medical Insurance Card" and "Driver's License" to "Driver's License/Photo ID"


Explore other APIs regarding the same

Explore other APIs regarding the same.

Can you elaborate on the issue?

Can you elaborate on the issue?

Means what other APIs should be explored?

Means what other APIs should be explored?

Sorry! couldn't get back to you, the  APIs for state level... I meant.

Sorry! couldn't get back to you, the  APIs for state level... I meant.


Demo is not working

The demo is not working. :  https://server.payum.forma-pro.com/demo.html
Backend also throws bunch of errors / 404 pages.

demo was fixed, backend as well probably.
Let me know if you find something else not working.
You may need to clear the cache to see the changes.

Thank you.
That was fast. Pay 1 SEK still have some error. Paying USD works.

@harikt what payment method did you select for 1 SEK?

ah see. That's because not all payment methods supports "authorize" operation.

Probably 1 SEK can be removed ? Reason I have tried with stripe, paypal etc. But all gave internal error. Or probably the gateways need to be shown only that are supported.

that's a proof of concept still and it misses a lot of features. For example with this feature implemented https://github.com/Payum/Payum/pull/518 it would be possible to filter out gateways that does not support current request.

:+1:  sure.
I have a question about payum and why twig is used. I haven't checked extensively the code. Where can I ask such questions about the internal decisions ?.

Where can I ask such questions about the internal decisions ?.

https://gitter.im/Payum/Payum


request.local? fails with Dockerized application

In ActiveElasticJob::Rack::SqsMessageConsumer there is a check that the request is local (request.local?).
https://github.com/tawan/active-elastic-job/blob/master/lib/active_elastic_job/rack/sqs_message_consumer.rb#L30-L32
Our application is running inside a docker container, and this check currently fails as the remote_addr and remote_ip resolve to <IP_ADDRESS>.
My suggestion is to make the local check optional. This could be set in an initializer. Or to allow a proc to be set that allows the user to configure their check.
Thanks again @tawan for all the work on this gem. I'll submit more docker related issues as I find them.

@RyanNaughton Thanks for the submission. Sorry for my late reply, I was on the road the recent days.
I'm thinking of introducing an optional environment variable, that overrides the check for a local IP. In your case, would the remote_ip always resolve to <IP_ADDRESS> or does it change between deployments?

@RyanNaughton @tawan instead of making a local check optional, which is a security hole, how about allowing a configuration variable be set that holds the allowed/trusted network. For example (not proposed code, just to give an example):
# Assuming the configuration pattern is implemented.
ActiveElasticJob.configure do |c|
  c.allowed_network = "<IP_ADDRESS>/16"
end

def in_allowed_network?(ip)
  IPAddr.new(ActiveElasticJob.config.allowed_network).include?(ip)
end

# In the wild.
unless request.local? || in_allowed_network?(request.ip)
  m = "Accepts only requests from localhost for job processing".freeze
  return ['403', {CONTENT_TYPE_HEADER_NAME => 'text/plain' }, [ m ]]
end

Sorry for the messy code, just letting the idea flow and typing on my phone.

@RyanNaughton :pager: ping. Any thoughts on PR #30?

@RyanNaughton @tawan I am under the impression we could handle this gracefully. There is no need to check for a custom ip address. The problem is actually forbiding a non-private ip address to deliver requests to the worker. It is just a matter of following RFC1918.
Follow this link: https://github.com/ipaddress-gem/ipaddress/blob/master/lib/ipaddress/ipv4.rb
Go to line 618, there you find an example implementation that would filter the correct ip addresses and avoid forcing the user to configure yet another thing.

@zaaroth checking against default common local networks by default :+1: plus having the ability to configure a custom networks for those special use cases :+1:

@zaaroth I did have a concern with this idea initially, since all EC2 instances have a private IP assigned. I was thinking that an unwanted message could get through from an unauthorized instance :unlock: using this method. However, if I'm not mistaken, Elastic Beanstalk creates a :lock: VPC for the instances to run in correct? If so, this should't be a concern. :relieved:
Other than that, I do like this idea. It's more elegant and like you said no more steps needed to allow a local network -- it's taken care of.
@tawan what do you think? If you like it, I can amend PR #30.

@zaaroth updated PR #30

@mdoza  @zaaroth  Thanks for your input and @mdoza  thank you for your PR.
I took a step back and thought again about why this issue was actually created. The problem only exists if the application is deployed in a Docker environment. I think we got carried away a little be too much when we were thinking about allowing a user configured network. We actually only have to deal with the Docker case. The docs of Docker  make it clear that if not configured otherwise, than the host will have the IP <IP_ADDRESS>. So if we check if the request comes from this particular IP and if the application runs inside a Docker container, then this should be sufficient. See my commit  68711d9.
What do you think?

@tawan sehr gut. Stays within the scope of fixing the original issue. Simple and elegant. :+1:


PoolConfig max_size setter fix

Currently the max_size setter method incorrectly assigns the new value to self.min_idle instead of self.max_size. This
change fixes the issue.

Thank you!


New knobs approach POC

Stage 2 - https://github.com/KonstantinTyukalov/azure-pipelines-agent/pull/2


MODE-1310 Add sequencers to the '3.x' codebase

Added sequencers to the 'modeshape-jcr-redux' module. Very limited testing has been done,
and will follow in subsequent commits.

Merged (after rebasing) into the '3.x' branch.


[HUDI-7373] revert config hoodie.write.handle.missing.cols.with.lossless.type.promotion

Change Logs
hoodie.write.set.null.for.missing.columns was renamed to
hoodie.write.handle.missing.cols.with.lossless.type.promotion. The config only adds the missing columns. The "reverse type promotion" is not controlled by a config.
Impact
Prevent config key change in next release
Risk level (write none, low medium or high below)
none
Documentation Update
https://github.com/apache/hudi/pull/10612
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@nsivabalan azure CI all passing


Feature Requests: URL configuration based on root folder

I have some repos which are from GitHub and some are from internal Git repos and we have our own web interface for those. I have my .gitconfig set to use a different identity based on where the local directory is. I'd like to customize the URL based on the directory as well.
Can this be done with the current version of this extension.
Below is how I configure .gitconfig to use an alternative configuration for my GitHub repos.
[includeIf "gitdir:~/Documents/GitHub/**"]
	path = ~/Documents/GitHub/.gitconfig

Hey Brennan! I am a bit unsure what you are requesting.
If you are looking for a way to customize settings in vscode by workspace folder pattern there might be an extension that will allow you to do that. It is also possible to have custom settings by workspace.
If it has something to do with how git runs it should use any configurations you have. All the extension does is running git as the current user in the same folder where the file can be found. There is no magic beyond that.
I hope that this will help you figuring out a solution to your problem!


Activity feed updates

Need to add activity feed updates to the leaderboard as users add songs. This is distinct from the queue. The queue reflects the order of songs, and there may only be room for 10 or so songs on screen. The activity feeds shows user activity as it happens, and it quickly fades away.
We may need an API route for this, and then need to figure out best way to get data in the frontend.

Regular old polling?
PubSub?
Websockets?

@sbddesign I vote either an API route + timeout polling or websockets. Thoughts? Obv the former is probably fastest since there's already API infra to build upon, but the latter would provider a better, more scalable solution long term.

Regardless of the implementation of this issue, it brings up a relevant question: how do we plan on collecting data on activity (i.e. bids being submitted, users joining host session)? Should we consider creating a new collection called activity storing different update objects? See below. Let's chat about it today.
I propose we create a new Activity schema with 2 initial Update object types:
const ActivitySchema = new Schema<Update>({
  userId: {
    type: String,
    unique: true,
    required: true,
  },
  firstNym: {
    type: String,
    required: true,
  },
  lastNym: {
    type: String,
    required: true,
  },
  avatar: {
    type: String,
    required: true,
  },
});

BidSubmit

{
    "type": "bid",
    "user": User(),
    "song": Song() (optional),
    "timestamp": "2023-01-07T14:37:47.461Z"
}

SessionJoin

{
    "type": "join",
    "user": User(),
    "timestamp": "2023-01-07T14:37:47.461Z"
}

For reference
User
{
  userId: string;
  firstNym: string;
  lastNym: string;
  avatar: string;
  color?: string;
}

Song
{
  duration_ms: number;
  album: { images: { url: string }[]; name: string };
  name: string;
  artists: { name: string }[];
  id: string;
}

I don't think we need a separate collection to store Activity, as there isn't any new data to store.
Using WebSockets: If the server detects a logged in host, bid updates are sent to the host via the websockets.
Using REST: The host/leaderboard page polls the getQueue endpoint on a timer and checks for new bids.

Makes sense. Wasn't sure if we wanted to keep the bid/session join events in a structured way for potential reuse later such as a host view that shows the history of bids and users that joined the session. If that's not something we care about, then your solution is definitely the easier/faster one, which will be much easier to accomplish.

What about when a new user joins the live host session? Do we care about this?

If no new data model(s) are required, and we go with the polling the API option, this becomes more of a frontend issue, so I'll leave that to you two @sbddesign @alexlwn123
However, I'll continue to work on the websockets since I've made progress and that requires backend/API changes.

@bnonni

What about when a new user joins the live host session? Do we care about this?

I don't think so... it lets people lurk without feeling spied on

@alexlwn123
@sbddesign and I discussed this issue and here's the summary:

/leaderboard/queue route is our source of truth for the current queue state
as a result, that route is very important and should be its own, standalone route without added complexity or features
i.e. we think its too important to reuse it for the activity feed updates
additionally, it doesn't exactly return what we want requiring adding new query params and new logic potentially adding unforeseen side affects and threatening the integrity of our "source of truth"
at minimum, we need a new route to handle activity feed updates
proposal:

route name: /activity
method: GET
query params:

hostShortName="atl"
time="1000", where time = amount of milliseconds subtracted for Date.now() establishing filter for Plays query

     // ?time=1000
     const timestampFilter = Date.now() - 1000
     const updates = await Plays.find({ filter: { queueTimestamp: { $gte: timestampFilter } }})

returns: list of Plays who's queueTimestamp is >= the timestampFilter
this allows us to decouple the queue (source of truth) and the activity feed updates while also allowing us to tightly control the window we use to lookback in time

That being said, the further we examine this issue, the more it seems to make sense that we use a separate collection for these updates.
The main rationale is the fact that if we simply reuse the /queue route or create a new /activity route as outlined above, the front end will be receiving a list of Plays which it will have to do some form of diff checking on to figure out the recently added Plays (i.e. compare whats in queue on frontend to what was received by api call). Using the diff check results, frontend can publish new activity updates on screen. Imo, its bad practice to make the frontend do any form of data mutations on the response object. It should just present what was sent back from the API allowing the backend to handle all data processing.
By creating a new collection, we can cater the data object in it to be exactly what we want allowing us to more easily and succinctly return only the min amount of data the frontend needs to fulfill the func req.
Let us know what you think here. At the very least, I'm going to start a new issue for a new route called /activity, and if you agree with our thinking, I'll implement the new collection per the spec I outlined above.

@bnonni

What about when a new user joins the live host session? Do we care about this?

I don't think so... it lets people lurk without feeling spied on

Also, @sbddesign and I were talking about this, and realized that the users are all prepopulated nyms, so publishing a session joined update wouldn't dox or reveal any lurkers. The hosts would probably want to see this data after they close their current PlebFM session. It would be like a timeline feed of what happened when.

PR #72

#78 opened to address new data model and collection


Change release-note.gradle mavenCentral() to mirror

Context
Previously there's a mavenCentral in release-note.gradle's buildscript
block. This PR replaces that mavenCentral with our own repo mirror.

@donat We can't configure a build script's repositories via init script (or at least I don't know how), project.buildscript.repositories {} can't configure a non-project-build-script script.

@oehme Already moved to buildSrc. Anyway, we need some logic to get and set repository mirror for the build.


ensure unique output names to prevent gms staging failures

Current GMS immuno builds are failing because pvacseq outputs 3 sets of 3 files, where each set contains files with identical names. Pvacseq outputs these into 3 subdirectories, but the cwl extracts each file individually, causing naming collisions. Renaming the files could work, as described in the "Rename an output file" section here; however, Cromwell does not support this option. In order to avoid sending each of these 9 files to a tool just to rename, I opted to instead just grab each of the 3 output directories. They do include some additional temp and log files, but they aren't very large and can easily be cleaned up later.

For future reference: there is an open bug report concerning the inability to rename here. The original workaround in this PR failed due to another bug (Cromwell can't handle type Directory?- bug report to be filed soon). The latest commit adds in another workaround by creating a new output directory containing all of the results. Since this directory will always be there, the output can be type Directory, which Cromwell handles just fine.


Release to pypi

Following https://github.com/DiamondLightSource/dodal/pull/164 we should move towards getting the a dodal release on pypi. To do this we will need to:

Get ophyd to do a release so that we can unpin from their GH repo
Confirm that following https://github.com/DiamondLightSource/dodal/wiki/Creating-a-Release auto-pushes a release to pypi

After this is done we can:

Update blueapi to use the pypi release
Remove https://github.com/DiamondLightSource/hyperion/pull/871/files/4bcff919fdbe9830ab7f2b8f356fefd3adcb7d80#r1319872594
Close https://github.com/pypi/support/issues/2676

Ophyd's latest release (1.9.0) should be what we want

We actually need https://github.com/bluesky/ophyd/pull/1148 so need to move to ophyd-async. @coretl when will be the first ophyd-async release?

Now done https://pypi.org/project/dls-dodal/


Remove expansion term from recoil pressure

Description of the problem

Recoil pressure formulation 0.55p_sat + [1/rho]m_dot^2 was accounting 2 time the [1/rho]m_dot^2 because the latter is included in the term  0.55p_sat.

Description of the solution

Remove it, the formulation is now 0.55*p_sat

How Has This Been Tested?

no test has changed

Documentation

no documentation has changed

Future changes

Add test and doc for evaporation model

@hepap can you rebase, then I can merge once CI passes?
Thx

@blaisb  done :)


NOTRUNCATE is applicable only to data files. NONTRUNCATE doesn't affect the log file.

There is a typo in the Arguments paragraph of the documentation:
NOTRUNCATE is applicable only to data files. NONTRUNCATE doesn't affect the log file. => should be NOTRUNCATE

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

ID: 87349f50-5751-68c4-df29-90c1c7c151d9
Version Independent ID: bb82dc63-23ec-eda3-0200-88c4748e050a
Content: DBCC SHRINKDATABASE (Transact-SQL) - SQL Server
Content Source: docs/t-sql/database-console-commands/dbcc-shrinkdatabase-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @pmasl
Microsoft Alias: umajay

@elbertlumenier Thank you for the feedback.
The change has been made.


 IF YOU DO NOT FOLLOW THE FORMAT OR MAKE A GENERIC TOKEN REQUEST YOUR ISSUE WILL BE DELETED WITHOUT COMMENT
YOUR JUSTLIST MUST FOLLOW THE JSON SPECIFICATION
https://github.com/justswaporg/justlists/example.justlists.ts
Checklist

[x] I understand that this is not the place to request a token listing.
[ ] I have tested that my JustList is compatible by pasting the URL into the add a list UI at justswap.org.
[ ] I understand that filing an issue or adding liquidity does not guarantee addition to the justlists website.

Please provide the following information for your token.
JustList URL must be HTTPS.
JustList URL: https://highlink.tech
JustList Name: Highlink
Link to the official homepage of the JustList manager:

Would you let us know how we can add our token itni just swap


Update zebra.js

'items' was undefined inside 'removeKids' function and length is changing during for loop execution

Thank you for paying my attention to the silly code I have written in tree model. Actually there are more problems than you have found. I have just fixed it and committed fixes in GITHUB.


Debug stop in handled try/catch only if you have 1 "if", or in the last "if else" or in the "else"

I have asynchonous generator function with a try/catch, in different "ifs" I call the same method, but if you call in the "else", or if you replace this else for "if else" or if you have only 1 "if" the debug stop.
In this repository you can see the example
@override
  Stream<int> mapEventToState(
    MyEvent event,
  ) async* {
    try {
      // Only debug stop when yield in the try and finally;
      yield currentState + 1;
      if (event == MyEvent.exception1) {
        await userRepository.signIn();
      } else if (event == MyEvent.exception2) {
        await userRepository.signIn();
      } else {
        // here debug stop, also if is "if else" or only 1 "if"
        await userRepository.signIn();
      }
    } on MyException catch (e) {
      // yield currentState + 1;
      print('catch');
    } finally {
      // Only debug stop when yield in the try and finally;
      yield currentState + 1;
      print('finally');
    }
  }
}

@ZichangG I repro'd this with the linked sample. The same exception is thrown in all branches of the if/else if/else but in the else block it's treated as an unhandled exception and causes the debugger to break. This seems unrelated to catchError (which has caused this in the past), since the only difference is being in the else clause instead of if/else if.

Let me assign myself first. I'll come back to this later.

The finally block will be copied over to many places and each of yield will be assigned a unique index. But the token position of 'yield' inside finally block is only recorded once, which leads to a mismatch. The possible solution is to record yield position with the same logic. Will work on a fix later.


Internally importing qiskit.providers.backend.Backend should not raise DeprecationWarning

Fixes #12902

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

4 of 8 (50.0%)  changed or added relevant lines in 2 files are covered.
25 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.03%) to 89.726%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/visualization/gate_map.py
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/expr.rs
1
94.02%

qiskit/visualization/gate_map.py
1
6.91%

crates/qasm2/src/lex.rs
5
92.73%

crates/qasm2/src/parse.rs
18
96.69%

Totals

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

Covered Lines:
67331

Relevant Lines:
75041

💛  - Coveralls

@jakelishman I took a different approach, but also not sure if it is the right one. Have a look and let me know.

Extended the PR because import qiskit.providers.fake_provider had the same issue.


CAPG cluster deletion fails because of route not being deleted

Problem
CAPG cluster deletion fails because of route not being deleted.
Task
Ensure routing rules are deleted in order for CAPG clusters to be cleanly deleted without manual interventation.

I tried adding network and subnetwork information to the kubernetes cloud configuration file. I thought that may fix the issue because routes are managed by the controller manager, but after creating a cluster with the new information, this is still happening
E0919 11:00:42.535258       1 controller.go:317] controller/gcpcluster "msg"="Reconciler error" "error"="googleapi: Error 400: RESOURCE_IN_USE_BY_ANOTHER_RESOURCE - The network resource 'projects/giantswarm-352614/global/networks/jose88-network' is already being used by 'projects/giantswarm-352614/global/routes/jose88-5af6e514-3ecd-41b1-9ca8-07273fa50452'" "name"="jose88" "namespace"="org-giantswarm" "reconciler group"="infrastructure.cluster.x-k8s.io" "reconciler kind"="GCPCluster"

Another option to explore: maybe there is a bug in the cloud provider part of the controller-manager, and we need to move away from the in-tree cloud-controller to the out-of-tree cloud controller, at some point, so it may be worth testing if this happens with the out-of-tree cloud-controller. For that we would need to

create new app to wrap cloud-provider-gcp
add new app to default-apps-gcp
change cluster-gcp to use cloud-provider: external instead of gce

The cloud-provider-gcp component doesn't have docs on how to actually deploy it. There are unanswered questions from the community in both the Github repository and the GCP slack channel on the kubernetes slack. It wouldn't be straight forward to deploy it.

I've checked the SDK and there is no parameter to force the deletion of the network. If we would implement an operator to do this, we will get the same error than CAPG is getting.

I guess another option would be to create a small operator that watches clusters and once they are deleted, this operator would remove the routes from google cloud. Knowing the name of the cluster , I think it would be easy to filter out which routes need to be removed. What do you think @alex-dabija ?

I guess another option would be to create a small operator that watches clusters and once they are deleted, this operator would remove the routes from google cloud. Knowing the name of the cluster , I think it would be easy to filter out which routes need to be removed. What do you think @alex-dabija ?

@calvix is doing the same now for CAPA. Yes, we can definitely go this route for now.


About datasets

Could you please share how to get access to the three datasets?

Thank you for your interest in our work, @ENDLICHERE.

CEFET/RJ-Libras: available under request. (Please, see the corresponding author in the original database paper).
MINDS-Libras: available through this link. Please, see also their  paper.
LIBRAS-UFOP-ISO: also available under request through this link (paper).

Thank you very much!


Cleanup Handling of Targets During JITing.

Until now, special targets during JITing were handled using constant valudes that could be valid addresses on certain architectures. To disambiguate, this patch adds more precise metadata for each target.

Closes https://github.com/iovisor/ubpf/issues/609.

coverage: 78.848% (+0.07%) from 78.777%
when pulling 625c4903b259d54a109138072a1b9b2ee49df0be on hawkinsw:target_refactor
into 0aa26cdd165cebdb678f3138e29162fa172afb81 on iovisor:main.


Cannot load status db when filesystem is readonly

Describe the bug
Cannot load status db when filesystem is readonly
vcpkg list
error writing file: C:\vcpkg\installed\vcpkg\status-new: The data is invalid.

Environment

OS: Windows 10 x64
Compiler: MSVC 2019

To Reproduce
Steps to reproduce the behavior:

Perform vcpkg operation in a writable filesystem
vcpkg install zlib
Make filesystem readonly (you may also deny NTFS write permission)
vcpkg list
Error occur

Expected behavior
No error.
zlib:x64-windows                                   1.2.11-6         A compression library
Additional context
If C:\vcpkg\installed\vcpkg\status is written immediately before vcpkg install exits, instead of lazy write on subsequent load, it won't fail on a readonly filesystem.
vcpkg update and vcpkg upgrade also fail.
If you run vcpkg list once then make readonly, subsequent vcpkg list will succeed.

@ras0219-msft @dan-shaw
Since this scene is too harsh, do we need to consider fixing this issue?

This is a vcpkg bug, but we'll need to fix other issues.

This is by design but could use improvement. See https://github.com/microsoft/vcpkg/issues/13097#issuecomment-763131958.

This is by design but could use improvement. See https://github.com/microsoft/vcpkg/issues/13097#issuecomment-763131958.


[iOS] Theming Customization Improvements

Goal
Simplify the theme configuration for builds of the new openedx mobile app
Slack: #wg-product-mobile
Requirement Definition: Not Drafted
### Tasks
- [ ] https://github.com/openedx/openedx-app-ios/pull/182
- [ ] https://github.com/openedx/openedx-app-ios/issues/223
- [ ] https://github.com/openedx/openedx-app-ios/issues/237
- [ ] https://github.com/openedx/openedx-app-ios/issues/238
- [ ] https://github.com/openedx/openedx-app-ios/pull/311
- [ ] https://github.com/openedx/openedx-app-ios/pull/307
- [ ] https://github.com/openedx/openedx-app-ios/pull/294

Should look into configurable text (title / subtitles / etc)

@saeedbashir is there any work left for theme on iOS?


Define number of town hall meetings

we currently have three:

one after keynote 1 and fellow lightning talks (#10 and #15)
one in the morning of day 2, before keynote 2 (#11)
one in the closing session, after announcement of the idea challenge winners (#17)

Is this done?

We have a little time for this before the day 2 keynote - to recap day 1 I assume. We have 30 minutes scheduled for day 2 in the afternoon. And there is an hour scheduled for "town hall and closing" in the afternoon of day 3.


[PE-D] Not sure of the difference between filter and sort

VS

However, how is the skill proficiency determined if the input for skills are all the same?
Perhaps sort could be included as [Upcoming] in user guide, or an explanation on skill proficiency level could be added

Labels: severity.Low type.FeatureFlaw
original: vanessaxuuan/ped#4

Update documentation regarding skill color indication and number of arguments for sort and filter


error thrown on export

Hi please see the following, can supply pcb if you want

Cheers

Can you try uninstalling and installing this plugin?
If the problem continue after installing again, I want you to share a kicad_pcb file that calls this error.

no go, heres the pcb, it works on everything else just not this.
Application: KiCad x64 on x64
Version: 7.0.6, release build
Libraries:
wxWidgets 3.2.2
FreeType 2.12.1
HarfBuzz 6.0.0
FontConfig 2.14.1
libcurl/7.88.1-DEV Schannel zlib/1.2.13
Platform: Windows 11 (build 22621), 64-bit edition, 64 bit, Little endian, wxMSW
Build Info:
Date: Jul  6 2023 04:56:45
wxWidgets: 3.2.2 (wchar_t,wx containers)
Boost: 1.81.0
OCC: 7.6.3
Curl: 7.88.1-DEV
ngspice: 40
Compiler: Visual C++ 1936 without C++ ABI
Build settings:
KICAD_SPICE=ON
10_pin_atmel_ice_breakout.zip

I could create zip files without any problem.

My environment
OS: Ubuntu: 22.04
KiCad: 7.0.6

Can you try uninstalling and installing this plugin?

Did you do this?

Hey Dean,
Yes I reinstalled it.  Still the same result.
It was working then all of a sudden its not.
I've had very little to do with python, any idea how I would attach a debugger to the python file when it is running.

I also reproduced this

Can you share your project which causes the problem.

Can you share your project which causes the problem?

https://codeberg.org/xtex/micro-carkit.git
Thanks!


Can I translate it into spanish??

Spanish people want to use this in spanish language

Absolutely! If you submit a pr with values-es/strings.xml I'd be happy to accept.

I mad a pr with the file translated


Make FileSystem properties visible

Makes connection, root, and cwd in FileSystem class readonly.
I believe this is the proper scoping. If any other contributor could let me know if that is correct that would be great 😄.

They are obviously not really readonly: https://github.com/trs/ftp-srv/blob/master/src/fs.js#L12. But I'd say that these are indeed not variables you are going to overwrite, so LGTM I guess :-).


Add binary operators

@rjmholt would you be able to review this PR? I think it should resolve #8

+1 it would be nice to be able to build this off of the official repo.

@JamesWTruher are you still able to work on this? (No rush, just wondering if you're still active)

@rjmholt can you merge this as-is? I can do an update PR afterward to fix what you mentioned.

AFAIK I don't have merge privileges

Oh, it looked like you had most of the git history 🙃
Do you know anyone who would?

Maybe @TylerLeonhardt - are you able to merge?

Alas, I do not... but @SteveL-MSFT should!

No wonder there haven't been any updates in 5 years, nobody can push :)

Gentle ping @SteveL-MSFT or anyone else who has permissions - could this get merged?
It would be probably be good if anyone with settings access could add another maintainer (@rjmholt?) to help get this repo back up to date. And/or somebody from the TS side, @amaanq maintains a ton of grammars and might be willing to or might know who to ask.

So this is basically a dead repo?

@JunHaoShih @tgross35, I just shot him an email directly, asking him to assign me write permission to the repo. Hopefully at that point we can begin to unblock a few of these PRs, etc

Might want to check this out, seems more maintained https://github.com/airbus-cert/tree-sitter-powershell

Any luck getting in touch @cjhillbrand?

Might want to check this out, seems more maintained https://github.com/airbus-cert/tree-sitter-powershell

Yeah, I know that one is definitely more active. It just makes sense that the PowerShell syntax in the PowerShell org would be the de facto one, maybe effort can be pooled if a new maintainer can be added here.
Or just archive this and point to that repo... but then you would still need a maintainer to update the readme here 🙂


Enable conditional skipping of components

Since we have the isSkipped method I figured we could start using it. It should be a more general solution to the hypervisor choice problem.
Also make sure virtualbox is skipped when xhyve is found.

Codecov Report

Merging #1332 into master will decrease coverage by 0.5%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master    #1332      +/-   ##
==========================================
- Coverage   93.67%   93.16%   -0.51%
==========================================
  Files          46       46
  Lines        4788     4857      +69
  Branches      533      836     +303
==========================================
+ Hits         4485     4525      +40
- Misses        303      305       +2
- Partials        0       27      +27

Impacted Files
Coverage Δ

browser/pages/selection/controller.js
85.87% <0%> (-1.22%)
:arrow_down:

browser/pages/confirm/controller.js
89.47% <75%> (ø)
:arrow_up:

browser/model/virtualbox.js
91.02% <77.77%> (+0.32%)
:arrow_up:

browser/model/devstudio-autoinstall.js
97.29% <0%> (-2.71%)
:arrow_down:

browser/services/request.js
97.61% <0%> (-2.39%)
:arrow_down:

browser/pages/install/controller.js
91.86% <0%> (-1.75%)
:arrow_down:

browser/model/helpers/downloader.js
95.53% <0%> (-1.68%)
:arrow_down:

browser/model/che.js
95% <0%> (-1.62%)
:arrow_down:

browser/pages/account/controller.js
98.61% <0%> (-1.39%)
:arrow_down:

browser/model/cdk.js
97.6% <0%> (-1.16%)
:arrow_down:

... and 26 more

Continue to review full report at Codecov.

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


Add acceptance tests for supporting raw transactions

More acceptance tests required

Acceptance test added to Quorum-Examples.


implemented load model feature

Added feature and test for loading model parameters via a state_dict
Includes storing model parameters since this shares code with #16
Very much WIP, especially the set_weights() and set_biases() convolution layer method implementations

It's possible I missed a thing here or there, let me know!


Rename pointwise operator CompileCache to PointwiseOperatorCompileCache

Just cosmetic changes, no change in functionality.

@Chillee @bertmaher