Error when building TensorRT-LLM in the docker from main branch

After changing the git branch from "release/0.5.0" to "main" I'm not able to rebuild the TensorRT-LLM from source.
Steps:

`apt-get update && apt-get -y install git git-lfs
git clone https://github.com/NVIDIA/TensorRT-LLM.git
cd TensorRT-LLM
git submodule update --init --recursive
git lfs install
git lfs pull`

make -C docker build

git checkout main

make -C docker run

python3 ./scripts/build_wheel.py --trt_root /usr/local/tensorrt

The error output:
tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/cutlass_kernels/fpA_intB_gemm/fp16_int8_gemm_fg_scalebias.cu.o
[ 76%] Building CUDA object tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/cutlass_kernels/fpA_intB_gemm/fp16_int8_gemm_fg_scaleonly.cu.o
[ 78%] Building CUDA object tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/cutlass_kernels/fpA_intB_gemm/fp16_int8_gemm_per_col.cu.o
[ 78%] Building CUDA object tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/cutlass_kernels/int8_gemm/int8_gemm_bf16.cu.o
[ 78%] Built target layers_src
[ 78%] Building CUDA object tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/cutlass_kernels/int8_gemm/int8_gemm_fp16.cu.o
[ 78%] Building CUDA object tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/cutlass_kernels/int8_gemm/int8_gemm_fp32.cu.o
[ 79%] Building CUDA object tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/cutlass_kernels/int8_gemm/int8_gemm_int32.cu.o
/code/tensorrt_llm/3rdparty/cutlass/include/cutlass/epilogue/threadblock/default_epilogue_tensor_op.h(244): error: static assertion failed with "ElementOutput needs to be 4 or 8 bit (unsigned) int."
static_assert(platform::is_same<ElementOutput, cutlass::int4b_t>::value ||
^
detected during:
instantiation of class "cutlass::epilogue::threadblock::detail::DefaultIteratorsTensorOp<ElementOutput, int32_t, ElementsPerAccess, ThreadblockShape, WarpShape, InstructionShape, ThreadMap> [with ElementOutput=cutlass::half_t, ElementsPerAccess=8, ThreadblockShape=cutlass::gemm::GemmShape<128, 128, 64>, WarpShape=cutlass::gemm::GemmShape<64, 32, 64>, InstructionShape=cutlass::gemm::GemmShape<8, 8, 16>, ThreadMap=cutlass::epilogue::threadblock::OutputTileOptimalThreadMap<cutlass::epilogue::threadblock::OutputTileShape<128, 8, 2, 1, 1>, cutlass::epilogue::threadblock::OutputTileShape<1, 8, 1, 1, 8>, 256, 8, 16>::CompactedThreadMap]" at line 369
instantiation of class "cutlass::epilogue::threadblock::DefaultEpilogueTensorOp<Shape_, WarpMmaTensorOp_, PartitionsK, OutputOp_, ElementsPerAccess, ScatterD, PermuteDLayout> [with Shape_=cutlass::gemm::GemmShape<128, 128, 64>, WarpMmaTensorOp_=cutlass::gemm::warp::MmaTensorOp<cutlass::gemm::GemmShape<64, 32, 64>, int8_t, cutlass::layout::RowMajorTensorOpMultiplicandCrosswise<8, 64>, int8_t, cutlass::layout::ColumnMajorTensorOpMultiplicandCrosswise<8, 64>, int32_t, cutlass::layout::RowMajor, cutlass::gemm::warp::MmaTensorOpPolicy<cutlass::arch::Mma<cutlass::gemm::GemmShape<8, 8, 16>, 32, int8_t, cutlass::layout::RowMajor, int8_t, cutlass::layout::ColumnMajor, int, cutlass::layout::RowMajor, cutlass::arch::OpMultiplyAddSaturate>, cutlass::MatrixShape<1, 1>>, 1, false, _nv_bool>, PartitionsK=1, OutputOp=cutlass::epilogue::thread::LinearCombination<cutlass::half_t, 8, float, float, cutlass::epilogue::thread::ScaleType::Default, cutlass::FloatRoundStyle::round_to_nearest>, ElementsPerAccess=8, ScatterD=false, PermuteDLayout=cutlass::layout::NoPermute]" at line 327 of /code/tensorrt_llm/3rdparty/cutlass/include/cutlass/gemm/kernel/default_gemm.h
instantiation of class "cutlass::gemm::kernel::DefaultGemm<ElementA, LayoutA, kAlignmentA, ElementB, LayoutB, kAlignmentB, ElementC, cutlass::layout::RowMajor, ElementAccumulator, cutlass::arch::OpClassTensorOp, cutlass::arch::Sm75, ThreadblockShape, WarpShape, InstructionShape, EpilogueOutputOp, ThreadblockSwizzle, 2, SplitKSerial, Operator, SharedMemoryClear, GatherA, GatherB, ScatterD, PermuteDLayout, void> [with ElementA=int8_t, LayoutA=cutlass::layout::RowMajor, kAlignmentA=16, ElementB=int8_t, LayoutB=cutlass::layout::ColumnMajor, kAlignmentB=16, ElementC=cutlass::half_t, ElementAccumulator=int32_t, ThreadblockShape=cutlass::gemm::GemmShape<128, 128, 64>, WarpShape=cutlass::gemm::GemmShape<64, 32, 64>, InstructionShape=cutlass::gemm::GemmShape<8, 8, 16>, EpilogueOutputOp=cutlass::epilogue::thread::LinearCombination<cutlass::half_t, 8, float, float, cutlass::epilogue::thread::ScaleType::Default, cutlass::FloatRoundStyle::round_to_nearest>, ThreadblockSwizzle=cutlass::gemm::threadblock::GemmIdentityThreadblockSwizzle<1>, SplitKSerial=true, Operator=cutlass::arch::OpMultiplyAddSaturate, SharedMemoryClear=cutlass::gemm::SharedMemoryClearOption::kNone, GatherA=false, GatherB=false, ScatterD=false, PermuteDLayout=cutlass::layout::NoPermute]" at line 98 of /code/tensorrt_llm/cpp/tensorrt_llm/kernels/cutlass_kernels/int8_gemm/int8_gemm_template.h
instantiation of "void tensorrt_llm::kernels::cutlass_kernels::genericInt8GemmKernelLauncher<T,arch,ThreadblockShape,WarpShape,Stages>(const int8_t *, const int8_t *, tensorrt_llm::common::QuantMode, const float *, const float *, T *, int, int, int, tensorrt_llm::cutlass_extensions::CutlassGemmConfig, char *, size_t, cudaStream_t, int *) [with T=half, arch=cutlass::arch::Sm75, ThreadblockShape=cutlass::gemm::GemmShape<128, 128, 64>, WarpShape=cutlass::gemm::GemmShape<64, 32, 64>, Stages=2]" at line 197 of /code/tensorrt_llm/cpp/tensorrt_llm/kernels/cutlass_kernels/int8_gemm/int8_gemm_template.h
instantiation of "void tensorrt_llm::kernels::cutlass_kernels::dispatchStages<T, arch, ThreadblockShape, WarpShape, 2, void>::dispatch(const int8_t *, const int8_t *, tensorrt_llm::common::QuantMode, const float *, const float *, T *, int, int, int, tensorrt_llm::cutlass_extensions::CutlassGemmConfig, char *, size_t, cudaStream_t, int *) [with T=half, arch=cutlass::arch::Sm75, ThreadblockShape=cutlass::gemm::GemmShape<128, 128, 64>, WarpShape=cutlass::gemm::GemmShape<64, 32, 64>]" at line 227 of /code/tensorrt_llm/cpp/tensorrt_llm/kernels/cutlass_kernels/int8_gemm/int8_gemm_template.h
instantiation of "void tensorrt_llm::kernels::cutlass_kernels::dispatchGemmConfig<T,arch,ThreadblockShape,WarpShape>(const int8_t *, const int8_t *, tensorrt_llm::common::QuantMode, const float *, const float *, T *, int, int, int, tensorrt_llm::cutlass_extensions::CutlassGemmConfig, char *, size_t, cudaStream_t, int *) [with T=half, arch=cutlass::arch::Sm75, ThreadblockShape=cutlass::gemm::GemmShape<128, 128, 64>, WarpShape=cutlass::gemm::GemmShape<64, 32, 64>]" at line 269 of /code/tensorrt_llm/cpp/tensorrt_llm/kernels/cutlass_kernels/int8_gemm/int8_gemm_template.h
instantiation of "void tensorrt_llm::kernels::cutlass_kernels::dispatchGemmToCutlass<T,arch>(const int8_t *, const int8_t *, tensorrt_llm::common::QuantMode, const float *, const float *, T *, int, int, int, char *, size_t, tensorrt_llm::cutlass_extensions::CutlassGemmConfig, cudaStream_t, int *) [with T=half, arch=cutlass::arch::Sm75]" at line 341 of /code/tensorrt_llm/cpp/tensorrt_llm/kernels/cutlass_kernels/int8_gemm/int8_gemm_template.h
instantiation of "void tensorrt_llm::kernels::cutlass_kernels::CutlassInt8GemmRunner::dispatchToArch(const int8_t *, const int8_t *, tensorrt_llm::common::QuantMode, const float *, const float *, T *, int, int, int, tensorrt_llm::cutlass_extensions::CutlassGemmConfig, char *, size_t, cudaStream_t, int *) [with T=half]" at line 361 of /code/tensorrt_llm/cpp/tensorrt_llm/kernels/cutlass_kernels/int8_gemm/int8_gemm_template.h
12 errors detected in the compilation of "/code/tensorrt_llm/cpp/tensorrt_llm/kernels/cutlass_kernels/int8_gemm/int8_gemm_fp16.cu".
gmake[3]: *** [tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/build.make:2993: tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/cutlass_kernels/int8_gemm/int8_gemm_fp16.cu.o] Error 1
gmake[3]: *** Waiting for unfinished jobs....
[ 79%] Built target common_src
[ 79%] Built target runtime_src
gmake[2]: *** [CMakeFiles/Makefile2:782: tensorrt_llm/kernels/CMakeFiles/kernels_src.dir/all] Error 2
gmake[1]: *** [CMakeFiles/Makefile2:708: tensorrt_llm/CMakeFiles/tensorrt_llm.dir/rule] Error 2
gmake: *** [Makefile:179: tensorrt_llm] Error 2
Traceback (most recent call last):
File "/code/tensorrt_llm/./scripts/build_wheel.py", line 278, in
main(**vars(args))
File "/code/tensorrt_llm/./scripts/build_wheel.py", line 158, in main
build_run(
File "/usr/lib/python3.10/subprocess.py", line 526, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'cmake --build . --config Release --parallel 20 --target tensorrt_llm tensorrt_llm_static nvinfer_plugin_tensorrt_llm th_common  ' returned non-zero exit status 2.`
If I skip step (3) everything works but then I can't use the new features in the main branch.
Thank you.

You may miss one step git submodule update --init --recursive which required for updating cutlass.

Thank you!


Query : How to work with variable width input images?

@Halfish the images of the numeral-sequences you create are all of the same dimensions (36 x 255). How can one go about using variable width images instead? (36 x var-width)
I see that you feed in a complete image at a time to the model (not as 255-strips of 36pix-height of the image -- a sequence of pixel-strips). Can you suggest how this can be modified to incorporate variable length inputs?

Update : I think even you are giving the image in strips just that the model automatically takes care of the splitting into multiple timesteps for you? [step(2) of the model]. However, I'm still unable to figure out my original problem. How do I get this to work for inputs of varying width (not fixed at 255)?
nn.Sequential {
  [input -> (1) -> (2) -> (3) -> (4) -> (5) -> (6) -> (7) -> (8) -> output]
  (1): nn.SplitTable
  (2): nn.Sequencer @ nn.FastLSTM(36 -> 128)          <--- this is the 36pix-height strip?
  (3): nn.Sequencer @ nn.Recursor @ nn.ReLU
  (4): nn.Sequencer @ nn.Recursor @ nn.BatchNormalization
  (5): nn.Sequencer @ nn.Recursor @ nn.Dropout(0.5, busy)
  (6): nn.Sequencer @ nn.Recursor @ nn.Linear(128 -> 12)
  (7): nn.JoinTable
  (8): nn.View(255, 12)
}

I have not tried variable length width yet, since it is a little complex to deal with such situation, and actually this is my first RNN toy project. I will just share some basic ideas as following.

To be able to accept variable inputs, the most simple way is just padding your images with white pixels. I have tried this before and it was easy to implement. However, this may limit your image size with a maximum value H and also waste your computing resources.

Another way to accept variable width inputs is just training the model without batch. By feeding lstm with input samples one by one, you don't need to worry about the sequence length. But this may requires torch to allocate and release memory frequently if no memory-share-mechanism was applied. I don't know how to do it or whether torch has already done it, you need to google it. Besides, training without batch and abandon batch normalization layer means that the model will converge slowly and the loss will decrease with fluctuations.

Finally, as your fore-mentioned, prepare some buckets with different lengths at first, then put the sample into the corresponding bucket(may also need a little padding),  whenever a bucket reach the batch size, you can use this batch bucket to train the model.

So maybe this 'bucket-way' is the most common way to do such thing, considering training sequence with variable lengths is inherently one tedious work. And I heard tensorflow had a better bucket training support with seq-to-seq model.

@Halfish thanks for the advice :smile:  With some more searching around it looks like masking the input is a good idea as the criterion ignores those inputs-values while computing errors/gradients, so I'll be trying that next.
As for the the bucket-index method, I feel I've taken care of all data prep and feeding however can't figure out one detail of the last-layer for the model. model:add(nn.View(decoder.max_steps, decoder.vocab_size)). In case of the fixed dimensional input, we can preset the max_steps and hence it works fine. For the case of variable-length sequences, we would want to model to modify this parameter for each batch of inputs.
I tried something like model:add(nn.View(-1, decoder.vocab_size):setNumInputDims(2)) but it doesn't work as the batchsize and seq-length get multiplied and the output vector is 2-dim instead of being 3-dim as expected. Can you tell me what would be the correct way of doing this?
Addn : I see that you have the same line in a function of the deocder.lua
function decoder_util:outputs2targets(outputs)
    --outputs = nn.View(self.max_steps, self.vocab_size):forward(outputs:double())

Is there some way in which I can remove this View-layer from the model and use it only per-batch somehow as in the decoder?

Maybe you can remove the View layer, and do this dimension adjustment dynamically in the training process? for example, add your code in this line.

@Halfish : thanks for that trick! Updated my codes and am in the process of waiting for results. Will update with my findings.
For any future viewers of this thread...here's what I did.
        -- forward pass
        local outputs = model:forward(inputs)

        -- convert outputs for CTC
        outputs = nn.View(width, vocab_size):forward(outputs:double())

        -- calc CTC loss
        local loss = ctcCriterion:forward(outputs, targets, torch.Tensor(sizes))
        local gradOutput = ctcCriterion:backward(outputs, targets)

        -- convert back for model
        gradOutput = nn.View(-1, vocab_size):forward(gradOutput)

        -- backward pass


Creation of a Campaign Service

creation of a campaigns service and the movement of the work in the member controller to use the campaign service

Another good change, thanks Paul!


seed cu1 and du1-fec1 for blazer cluster

PR adds two nodes for testing in the OpenShift cluster identified as "blazer".  Also some slight semantical updates to the README for the add-workers feature / structure previously defined by @jnunyez .

LGTM. Merging.


Remove ButtonWithTooltip

💬 Description
This isn't used anywhere, easy win!
✅ Checklist

[ ] Tests written
[ ] Browser tested
[ ] Added to documentation
[ ] Added to storybook

 Replace tooltip with tippy


Puzzles about ExecCommand function in client/client_gorums.pb.go

As commented by the function:
ExecCommand sends a command to all replicas and waits for valid signatures
from f+1 replicas

However, the HotStuff paper argues that 2f+1 signatures are needed.
What's the reason for this inconsistency?

This comment is slightly inaccurate. It should really read "...waits for responses from f+1 replicas." Anyway, the ExecCommand function is used by clients to send commands to replicas. The clients do not participate in the consensus protocol, they only submit commands. It is up to the replicas to reach a consensus on which commands should be executed and in which order. Indeed, replicas need 2f + 1 valid signatures from other replicas in order to reach consensus, but a client only needs to receive responses from f + 1 replicas in order to verify that the command has been executed. This is just as described in the first paragraph on "Basic HotStuff" from the paper.
I hope that helps :)

This comment is slightly inaccurate. It should really read "...waits for responses from f+1 replicas." Anyway, the ExecCommand function is used by clients to send commands to replicas. The clients do not participate in the consensus protocol, they only submit commands. It is up to the replicas to reach a consensus on which commands should be executed and in which order. Indeed, replicas need 2f + 1 valid signatures from other replicas in order to reach consensus, but a client only needs to receive responses from f + 1 replicas in order to verify that the command has been executed. This is just as described in the first paragraph on "Basic HotStuff" from the paper.
I hope that helps :)

Thanks so much for your timely reply, that is really helpful and successfully answers my doubts.

This comment is slightly inaccurate. It should really read "...waits for responses from f+1 replicas." Anyway, the ExecCommand function is used by clients to send commands to replicas. The clients do not participate in the consensus protocol, they only submit commands. It is up to the replicas to reach a consensus on which commands should be executed and in which order. Indeed, replicas need 2f + 1 valid signatures from other replicas in order to reach consensus, but a client only needs to receive responses from f + 1 replicas in order to verify that the command has been executed. This is just as described in the first paragraph on "Basic HotStuff" from the paper.
I hope that helps :)

I'm sorry for having another doubt.
As the HotStuff states, the client should only send the command to the primary node. However, it seems that 'ExecCommand' method sends the command to all the replicas.

As the HotStuff states, the client should only send the command to the primary node.

This is simply an implementation detail. You can choose to have the clients send commands to only the primary, or maybe the primary plus and a couple of backups, or simply all replicas. By sending the commands to all replicas, we don't have to worry about a faulty primary, and it makes it easy to change the primary often. However, if you are running with a massive amount of replicas, then it makes more sense to send the commands to a subset of the replicas.

As the HotStuff states, the client should only send the command to the primary node.

This is simply an implementation detail. You can choose to have the clients send commands to only the primary, or maybe the primary plus a couple of backups, or simply all replicas. By sending the commands to all replicas, we don't have to worry about a faulty primary, and it makes it easy to change the primary often. However, if you are running with a massive amount of replicas, then it makes more sense to send the commands to a subset of the replicas.

I see! Thanks again for your help :)


[BUG] Curve Limit Order Bots missing Cancel Functionality on MybOts Page

Curve LimitOrder Bot missing MyBots Page commands to Stop bot. See attached
We need UI for Stopping Curve LimitOrder Bots.

Closing as this is a back-end issue.


Show log component change

Show log should be on right side with divider. Use Clarity Select Boxes component for drop down.
Original screenshot:

Fix:

@tonycgil
Refactor the log action panel as what you mentioned. Will deploy new code in next env update turn.

Dont think we need to say "recent" in the drop down. It makes the dropdown too verbose.

Sure. Removed "recent".


Return an error for queries with invalid, named selectors

Bug report
When querying with an invalid, named selector, yabai acts as if no selector was given at all and returns the focused window/space/display. When given a number instead of a string, yabai returns an error instead.
# returns an error

❯ yabai -m query --windows --window 0
could not locate the selected window.
❯ echo $?
1

# returns the focused window

❯ yabai -m query --window --window asdf
{ ... }
❯ echo $?
0

The behaviour should be consistent so that scripts can test whether a valid selector was provided.
# check if a valid display selector was provided by the user
if ! yabai -m query --displays --display "${1}" > /dev/null 2>&1; then
        >&2 echo "could not locate the selected display."
        exit 1
fi

Fixed on master.


Fails to build Unity.2019.3.0f3 Windows 64

[Exception] ArgumentException: The Assembly Mono.WebBrowser is referenced by System.Windows.Forms ('Assets/Plugins/StandaloneFileBrowser/Plugins/System.Windows.Forms.dll'). But the dll is not allowed to be included or could not be found.

Copying over the following .dll causes the Project to build again

From Unity\Hub\Editor\2019.3.0f3\Editor\Data\MonoBleedingEdge\lib\mono\2.0-api
** Mono.Posix.dll
** Mono.WebBrowser.dll
to StandaloneFileBrowser\Plugins*

However, using the OpenFolderPanel then fails with
NotSupportedException: Specified method is not supported.
  at (wrapper cominterop) Ookii.Dialogs.Interop.FileOpenDialogRCW..ctor()
  at Ookii.Dialogs.VistaFolderBrowserDialog.RunDialog (System.IntPtr hwndOwner) [0x0002f] in <31af777adad04cb788cee0ae18e55ae1>:0
  at System.Windows.Forms.CommonDialog.ShowDialog (System.Windows.Forms.IWin32Window owner) [0x00022] in <75b5cf3ed2c949048fa3d163b7112565>:0
  at (wrapper remoting-invoke-with-check) System.Windows.Forms.CommonDialog.ShowDialog(System.Windows.Forms.IWin32Window)
  at SFB.StandaloneFileBrowserWindows.OpenFolderPanel (System.String title, System.String directory, System.Boolean multiselect) [0x0002c] in <101d639d50c64aaa8014327ad1711585>:0
  at SFB.StandaloneFileBrowser.OpenFolderPanel (System.String title, System.String directory, System.Boolean multiselect) [0x00000] in <101d639d50c64aaa8014327ad1711585>:0
  at Frameworks.IO.FileBrowser.OpenFolder (System.String path) [0x00022] in <c874c53545344e069d363351ca6356d9>:0
  at Presentation.Menus.LoadScene.LoadOpcMenuItem.OnButtonClicked () [0x00000] in <ded431fef5034153ad07e8d2e59335eb>:0
  at Infrastructure.NoxRx.UI.PresenterBindable`1+<>c__DisplayClass6_0[T].<Call>b__0 (T _) [0x00000] in <13aaf08a80ad427588610f476b517342>:0
  at UniRx.Observer+Subscribe`1[T].OnNext (T value) [0x00008] in <101d639d50c64aaa8014327ad1711585>:0
  at UniRx.Operators.RefCountObservable`1+RefCount[T].OnNext (T value) [0x00000] in <101d639d50c64aaa8014327ad1711585>:0
  at UniRx.Subject`1[T].OnNext (T value) [0x00000] in <101d639d50c64aaa8014327ad1711585>:0
  at UniRx.Operators.AsObservableObservable`1+AsObservable[T].OnNext (T value) [0x00000] in <101d639d50c64aaa8014327ad1711585>:0
  at UniRx.Operators.FromEventObservable`1+FromEvent[TDelegate].OnNext () [0x0000b] in <101d639d50c64aaa8014327ad1711585>:0
  at UnityEngine.Events.InvokableCall.Invoke () [0x00010] in <40c0e6fe295245dab933f5d3ac69460a>:0
  at UnityEngine.Events.UnityEvent.Invoke () [0x00022] in <40c0e6fe295245dab933f5d3ac69460a>:0
  at UnityEngine.UI.Button.Press () [0x0001c] in <649dbd3608c64a59abf1700d6987458f>:0
  at UnityEngine.UI.Button.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) [0x00009] in <649dbd3608c64a59abf1700d6987458f>:0
  at UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) [0x00007] in <649dbd3608c64a59abf1700d6987458f>:0
  at UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction`1[T1] functor) [0x00063] in <649dbd3608c64a59abf1700d6987458f>:0
UnityEngine.DebugLogHandler:Internal_LogException(Exception, Object)
UnityEngine.DebugLogHandler:LogException(Exception, Object)
UnityEngine.Logger:LogException(Exception, Object)
UnityEngine.Debug:LogException(Exception)
UnityEngine.EventSystems.ExecuteEvents:Execute(GameObject, BaseEventData, EventFunction`1)
UnityEngine.EventSystems.StandaloneInputModule:ReleaseMouse(PointerEventData, GameObject)
UnityEngine.EventSystems.StandaloneInputModule:ProcessMousePress(MouseButtonEventData)
UnityEngine.EventSystems.StandaloneInputModule:ProcessMouseEvent(Int32)
UnityEngine.EventSystems.StandaloneInputModule:ProcessMouseEvent()
UnityEngine.EventSystems.StandaloneInputModule:Process()
UnityEngine.EventSystems.EventSystem:Update()

while it does work inside of the editor.

Scripting Backend: Mono
Api Compatibility Level: .NET 2.0 Standard

Likely the same as, but should be confirmed. If it is one of the tickets should be closed.:

#69
#71

As the original author of this library seems not be active anymore, is there someone willing to fork this and continue to keep it compatible? If someone knows of an alternative that is still beeing maintained please ping me.

Same issue here

Had the same issue, was able to get it to build by switching my .NET framework from 2.x compatible to 4.x compatible.


Initial MVA root certificates

This PR contains the two current MVA (Mark Verifying Authority) root certificates, as listed here:
https://bimigroup.org/vmc-issuers/
DigiCert PEM confirmed via https://www.digicert.com/kb/digicert-root-certificates.htm
Entrust PEM confirmed via https://www.entrust.com/knowledgebase/ssl/entrust-root-certificates

Hi @EntrustCorporation folks - @ounsworth and @johngray-dev 👋 I'm hoping to get an Entrust source for your root mark certificate pem that's listed on https://bimigroup.org/vmc-issuers (like DigiCert's source here), could you help?

Hi @EntrustCorporation folks - @ounsworth and @johngray-dev 👋 I'm hoping to get an Entrust source for your root mark certificate pem that's listed on https://bimigroup.org/vmc-issuers (like DigiCert's source here), could you help?

Don't worry - found a reference!
https://www.entrust.com/knowledgebase/ssl/entrust-root-certificates
https://web.entrust.com/root-certificates/VMRC1.cer


Schema discovery created unpersistable schema for empty arrays

If all observed values of a document field are [] the generated schema is for ArrayType[NullType] which cannot be persisted or used in any meaningful way.
In the absence of evidence of the type of array elements a more logical behavior would be to allow for overrides of the schema of a subset of fields, e.g., as a JSON string (schema.json in Spark) or, if a default behavior is needed, map to ArrayType[StringType] as opposed to ArrayType[NullType]. The benefits are that this mapping can be persisted and it can represent any Mongo arrays, including heterogeneous ones.

This issues is somehow like https://github.com/Stratio/Spark-MongoDB/issues/114, you should not assume a type from the lack of evidence just because it is convenient.
With the current connector API you have no problem if you want to provide your own schema:
/**
 * A MongoDB baseRelation that can eliminate unneeded columns
 * and filter using selected predicates before producing
 * an RDD containing all matching tuples as Row objects.
 * @param config A Mongo configuration with needed properties for MongoDB
 * @param schemaProvided The optionally provided schema. If not provided,
 *                       it will be inferred from the whole field projection
 *                       of the specified table in Spark SQL statement using
 *                       a sample ratio (as JSON Data Source does).
 * @param sqlContext An existing Spark SQL context.
 */
class MongodbRelation(private val config: Config,
                       schemaProvided: Option[StructType] = None)(
                       @transient val sqlContext: SQLContext) extends BaseRelation

That class will only try to infer the schema if schemaProvided is None :
  /**
   * Default schema to be used in case no schema was provided before.
   * It scans the RDD generated by Spark SQL statement,
   * using specified sample ratio.
   */
  @transient private lazy val lazySchema =
    MongodbSchema(
      new MongodbRDD(sqlContext, config, rddPartitioner),
      config.get[Any](MongodbConfig.SamplingRatio).fold(MongodbConfig.DefaultSamplingRatio)(_.toString.toDouble)).schema()

  override val schema: StructType = schemaProvided.getOrElse(lazySchema)

On the other hand, there are ways of building a modified schema from the inferred which you can use  to provide MongodbRelation with.

That's good to know but are you suggesting that the desired behavior of the library is to produce unwritable schema in this case as opposed to a potentially incorrect but writeable schema? In that case, especially since the exception is rather cryptic, would it not be better to create some best practices and utilities around checking the schema after discovery and, perhaps, offering schema rewriting strategies, e.g., remove the field from the schema (because adding the field later on works well with Spark's schema merging but changing the type of a field does not)?
Perhaps

@ssimeonov The development team behind Spark-MongoDB have been talking about this issue. In spite of being not formally right as I mentioned above we've agreed to change it to StringType. A sweet spot between correctness and convenience.

@ssimeonov A PR has been issued https://github.com/Stratio/Spark-MongoDB/pull/125 with the described change, it will soon merged.
Btw, quite a strange collection, or too low sample ratio, where not a single object has an array with at least one element.


Add license to the project

It'd be better to add a license to the project because having no license specified means no one has the right to use, copy or modify the code.

@pavinduLakshan what do you suggest?

MIT license seems good, IMO.
If you are also +1, I can send a PR :)

+1 for MIT


Athlete Statitics

Think about the graphs for statistics etc.

Further problems with athlete view

ordering of years is ascending but within the comp, it is descending (make the keys strings)
bottom cell has no order but this gets rid of the separator between the snatch and c+j.
Date format -> month, day date. (e.g. Jan, 1st Mon
grade badge takes up ensure column.
"Number time competition" -> "Competed N times."

Ideas for statistics.

Graph for Sinclair totals, get rid of "no total competitions"


High number of i=1 errors after interpolating to standard levels

At present, with our current set of interpolated standard levels, the observational dataset has a large number of i=1 MITprof errors (i.e. "not enough data near standard level"). This issue is particularly noticeable for the XBT and Argo data. Fixing this usually requires a shift in standard levels. We might try the B-SOSE levels. @ShenjieZhouBAS, could you give this a try when you get a chance, please?

@mmazloff: do you foresee any issues with us trying the B-SOSE standard observational levels for SO-WISE?

Hi Dan
I think it should be fine for everything but, perhaps, moorings.
Matt

On May 25, 2021, at 2:41 AM, Dan Jones @.***> wrote:
@mmazloff https://urldefense.com/v3/__https://github.com/mmazloff__;!!Mih3wA!U3ZTMfyV7EKj3CiJc-DmR530EEuR5O1mLFXZTlZrhNw3TaryiEH4bXZ9W-wZAbMsTw$: do you foresee any issues with us trying the B-SOSE standard observational levels for SO-WISE?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://urldefense.com/v3/__https://github.com/so-wise/so-wise-obs/issues/4*issuecomment-847719393__;Iw!!Mih3wA!U3ZTMfyV7EKj3CiJc-DmR530EEuR5O1mLFXZTlZrhNw3TaryiEH4bXZ9W-wIvLL1MQ$, or unsubscribe https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AIEXPOBT3SVGLIKN64Y4WCTTPNWD7ANCNFSM45O6B3JQ__;!!Mih3wA!U3ZTMfyV7EKj3CiJc-DmR530EEuR5O1mLFXZTlZrhNw3TaryiEH4bXZ9W-wYrtc7-w$.


Fix: Require Signature to login + Notifications

This PR fixes #
Changes

Tests

[x] Tested on mobile
[x] Tested on dektop

This PR should be manually tested by @obernardovieira @PauloSousapt , included on reviewers.

fixed 👌


check aio configure option in auto/os/freebsd

try to fix https://github.com/alibaba/tengine/issues/837


Fetching latest records in Batch mode - Apache Seatunnel

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
I have successfully deployed Seatunnel in my system. I'm currently working on synchronizing data between the source and sink in MySQL. I'm running Seatunnel in batch mode, but it's currently retrieving all records from the source and dumping them into the sink table. I need to run this job every 30 minutes, but it should only fetch the latest records from the source, not all the records.
Is there a way to configure Seatunnel to only retrieve the latest records from the source and dump them into the sink table in batch mode?
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Pass the incremental value into sql every time it is scheduled. This can be solved by custom parameters

Hi @liugddx Thanks for your response.
I'm curious to know if the current released versions of this seatunnel supports incremental offline batch synchronization ?

Hi @liugddx Thanks for your response. I'm curious to know if the current released versions of this seatunnel supports incremental offline batch synchronization ?

This scenario requires the use of scheduling capabilities, DolphinScheduler might be a good choice

这个是很常用的功能，希望能添加这方面的支持

DolphinScheduler好像也不支持动态传递sql的where条件的参数

DolphinScheduler 好像也不支持动态传递sql的where条件的参数

SeaTunnel has custom parameters.


Hello! I am an auto generated PR from an update to the swagger Files. Please review my PR and accept it!

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: glennrfisher:x: Travis CI UserTravis CI User seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Use existing translations when using IndifferentAccess and IgnoreUndeclared

When using Hashie::Extensions::Dash::IndifferentAccess and Hashie::Extensions::Dash::PropertyTranslation at the same time, Hashie does not use the translation when accessing properties via IndifferentAccess. This modifies IndifferentAccess to look for a translation first and use it to access the property.

I like it, merging.


updated community blog and videos

Change Logs
updated blog and video content

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

cc @bhasudha please review

Can we please add DOCS prefix to PRs going forward, if docs updaet.


I need help

Issue Type: Feature Request
So i was like omg wait what i cant run all code at a time??????????and i went to report the issue
VS Code version: Code 1.58.2 (c3f126316369cd610563c75b1b1725e0679adfb3, 2021-07-14T22:30:16.440Z)
OS version: Linux x64 5.4.109-26094-g381754fbb430
Restricted Mode: No

/needsMoreInfo including reproducible steps.


Support for React Native 0.71

Describe the bug
Unable to build version 0.20+ successfully on RN 0.71.
Execution failed for task ':stripe_stripe-react-native:compileDebugKotlin'.
> A failure occurred while executing org.jetbrains.kotlin.compilerRunner.GradleCompilerRunnerWithWorkers$GradleKotlinCompilerWorkAction

To Reproduce

Start latest RN.
Install 0.20+ Stripe
npm run android

Expected behavior
Working builds on RN 0.71
Desktop (please complete the following information):

OS: Windows 11
Browser Chrome
Version 0.20+

Additional context
Maybe related to Kotlin version (needs 1.7.0 for RN0.71)

Having the same issue on CI, was working just fine on Thursday but suddenly today this error appeared today without any changes done to our codebase.

Same issue here.
"react-native": "0.70.6",
"@stripe/stripe-react-native": "^0.23.1",

We're getting a similar error too. last Friday we could build 0.23.1 fine. Now that we upgraded to 0.23.2 we're getting errors like:
Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.8.0, expected version is 1.6.0.

Expo 47
React native 0.70.5
"@stripe/stripe-react-native": "^0.23.2"

I have the issue since this morning, i can't build on Android anymore. See my screen for details about the crash

I think this might have to do with a recent release of stripe-android that we're picking up automatically, will look into this today

Don't wanna dilute the conversation but is there a reason that something like this would happen?
I'm not familiar with Gradle or Android but shouldn't the dependencies be locked to specific versions to avoid situations like this one?
Feel free to ignore my comment or mark it as off-topic, I'm just curious about the process in general.

Hey all, I've got a fix on the way but in order to unblock your development, you can use this workaround:
in your android/app/build.gradle file, add this block above your dependencies {*} block:
// workaround for https://github.com/stripe/stripe-react-native/issues/1288
// Remove once @stripe/stripe-react-native is ugpraded
configurations {
    compileClasspath {
      resolutionStrategy.force "com.stripe:stripe-android:20.19.1"
      resolutionStrategy.force "com.stripe:financial-connections:20.19.1"
    }
}

This will lock stripe-android (a dependency of stripe-react-native) to the previous version. Once the fix is out, you can remove this block

@varemenos that's a great question- this should not happen. We soft match the version of stripe-android to the minor version, meaning stripe-react-native automatically picks up bug fixes (as in, we pin stripe-android v20.1.X, so it picks up any new version up to but not including v20.2.0. Usually this works well for us, but it looks like a new feature was added but only released in a patch release, which caused kotlin to shout about missing if else branches.
If you're running into this issue and looking for a workaround, refer to my previous comment. However, we'll publish a fix ASAP to unblock folks. thanks everyone for reporting this so quickly!

@charliecruzan-stripe, just to make sure, do i have to upgrade my kotlin to 1.8.0 even with that block you gave us as workaround ?

Did i do something bad here ? i can make the workaround working ..
i did set up kotlinVersion = "1.8.0" at line 15
and added classpath at line 23

and there is a preview of where i did enter the workaround provided. line 197 to 202

and i'm receiving this message whenever i build
e: file:///Users/michaellabelle/Documents/apps/rabaischocs/node_modules/@stripe/stripe-react-native/android/src/main/java/com/reactnativestripesdk/PaymentLauncherFragment.kt:256:12 'when' expression must be exhaustive, add necessary 'CashAppRedirect' branch or 'else' branch instead
e: file:///Users/michaellabelle/Documents/apps/rabaischocs/node_modules/@stripe/stripe-react-native/android/src/main/java/com/reactnativestripesdk/utils/Mappers.kt:453:3 'when' expression must be exhaustive, add necessary 'CashAppRedirect' branch or 'else' branch instead

UPDATE:: i fixed my problem without the workaround, after i few research inside the stripe source code i realised that the version is fetched while the build from our root build.gradle with the terms stripeVersion. so instead adding the whole block i just added the stripeVersion as follower.
in my android/build.gradle i declared an property inside =>
buildscript {
    ext {
        ***
        kotlinVersion = "1.8.0"
        stripeVersion = "20.19.1"
    }
***
}

thanks @ElSeniorMikael , that would work as well

Released in v0.23.3

Still getting issue in react version 0.70.6 getting build filed error. also tried downgrading the version but this didnt worked

Also still getting the issue with react-native version 0.71.3 with stripe-react-native 0.23.3.

getting error with react-native 0.70.5 with stripe/stripe-react-native 0.26.0

I've managed to fix the error by specifying the kotlinVersion in expo-build-properties plugin.
      [         "expo-build-properties",         {           "android": {             "kotlinVersion": "1.8.0"           }         }       ],
"react-native": "0.71.6"
"@stripe/stripe-react-native": "0.23.3"

Just downgrade your stripe to v0.22.0. works for me

Into android/build.gradle, I just updated my kotlinVersion and kotlin_version from 1.6.21 to 1.8.0.


[AutoRelease] t2-containerservice-2022-09-19-31302(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3182
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=982

issue link:https://github.com/Azure/sdk-release-request/issues/3182


Pluggable services

Make env var parsing more robust and load resources from non template charts.
/cc @sparkprime

Agreed w/propagating the error to the user. Don't like the idea of forcing the content to be resources. Quite clean as Kubernetes API objects, which we do parse. So, leaving that part as it is.
/cc @sparkprime PTAL

Ok I don't agree but I'll add it to #520 so we can get this in now

Thanks, @sparkprime.

Let's include this point the design decision list, since it has IMO a significant impact on chart format, especially for existing charts and for people familiar with original helm. Based on feedback we got from the community, it was also a significant adoption blocker for DM before the helm integration to use something other than the k8s API object format. We were thinking about moving everything to that format before the helm chart option came along.


"sentencepiece.bpe.model" and "dict.txt" in page below seem not available

https://github.com/microsoft/torchscale/blob/main/examples/fairseq/README.md#example-bert-pretraining

Already updated the links. Please try again.

thx


Your example does not work for me.

Hi @waylonflinn,
It all looks very interesting and I would like to see your solution combined with neocortex.
I could not run your example in Chrome on Ubuntu. I have a build in I5 gpu and get:
"Bound framebuffer is not complete."
Error.
The error is in webgl.js:
if( gl.checkFramebufferStatus(gl.FRAMEBUFFER) != gl.FRAMEBUFFER_COMPLETE)
throw new Error("Bound framebuffer is not complete.");
Am I doing something wrong there?

Sorry, the demo page that did not work is: http://waylonflinn.github.io/DeepBeliefSDK/
This example works on windows 10 with I5. I guess some gpu functionality are disabled in chrome on Ubuntu. I will check my settings.
BZW: Please cache the demo database, so it is not loaded every time.

Thanks for the feedback. Caching the database is an excellent idea.
If you're thinking about porting neocortex, you might be interested in this project of mine:
https://github.com/waylonflinn/webnn
It's mission is to create a convolutional neural network that uses weblas in the browser. I'm hoping to return to work on it in the next week or so.

Hi @waylonflinn,
In linux I have this two options:
Native GpuMemoryBuffers: Software only. Hardware acceleration disabled
Rasterization: Software only. Hardware acceleration disabled
Hope this helps to find the problem.

Hi @waylonflinn, it looks like your web example is working on Win with I5.

@MariasStory glad to hear it. I'll close this. :smile:


When hidding left/right arrow background is not white

when datepicker hides the left/right arrow for scrolling through months (by setting "visibility: hidden" on class="prev") the background color changes to #F8F8F8F and not white.  Any idea where to fix this?   Only happens in Firefox.

Setting the row to background-color: white worked.


vue-tsc remove the imported type when emit .vue.d.ts

version
"vue": "^3.4.21" "vue-tsc": "^2.0.10" "typescript": "~5.4.0"
pnpm 8.15.5
node 18.19.0
tsconfig.json
{   "extends"<EMAIL_ADDRESS>  "include": [     "src/**/*.ts",     "src/**/*.d.ts",     "src/**/*.tsx",     "src/**/*.vue",     "auto-imports.d.ts",     "components.d.ts"   ],   "exclude": ["src/**/__tests__/*", "src/App.vue", "src/main.ts"],   "compilerOptions": {     "target": "ES2020",     "useDefineForClassFields": true,     "module": "ES2020",     "moduleResolution": "Node",     "strict": true,     "lib": ["ESNext", "DOM", "DOM.Iterable"],     "skipLibCheck": true,     "baseUrl": ".",     "paths": {       "@/*": ["./src/*"]     }   } }
scripts
"type-emit": "vue-tsc -p tsconfig.json -w --declarationDir dist/types"
$ pnpm type-emit
question

Why do you expect vue-tsc to keep this import? It is not used elsewhere except a shallowRef.

Why do you expect vue-tsc to keep this import? It is not used elsewhere except a shallowRef.
I just try to expose the myColorPickerRef, but it can not emit .vue.d.ts, and no error message output.

Has anyone encountered the same problem as me? This problem appeared after the vue-tsc update.

Why do you expect vue-tsc to keep this import? It is not used elsewhere except a shallowRef.

Can I expose a component instance use defineExpose?

可以使用 defineExpose 暴露一个子组件的实例吗？发现当暴露一个子组件实例时，无法输出 .vue.d.ts.


IGAPP-949: Fix github authentication

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.

Did not read the changelog correctly :man_shrugging:


API always using get handler even when trying to post?

Hi. Sooo I have a very weird issue.
I'm using postman to test my website's api (made using fresh) and I have a route with a get and a post handler. Each request I make is routed to the get handler even though I'm using post to make them.
I tested it by making the get handler respond with _res.method and it was always returning GET even though i was trying to post.

Are you able to share the route file code?

@iuioiua Here.
Also basically what I'm trying to do is make a route that if you POST to you can create a account in the database
import { Handlers } from '$fresh/server.ts';
import { User, connected as dbConnected } from '../../db.ts';
import { hashOf, generateSnowflake, API404 } from '../../util.ts';
import * as uuid from 'npm:uuid';

export const handler: Handlers = {
	GET(_req) {
		return new Response(_req.method);
	},
	POST(_req) {
		if (dbConnected) {
			try {
				const { username, email, password } = _req.json();
			} catch (e) {
				console.error(e);
				return new Response(
					JSON.stringify({
						error: {
							code: 'INVALID_JSON',
							message: 'Invalid JSON Data!',
						},
					}),
					{
						headers: { 'Content-Type': 'application/json' },
						code: 400,
					}
				);
			}
			if (
				typeof username === undefined ||
				typeof email === undefined ||
				typeof password === undefined
			) {
				return new Response(
					JSON.stringify({
						error: {
							code: 'NOT_ENOUGH_DATA',
							message: 'You must provide a Username, Email & Password!',
						},
					}),
					{
						headers: { 'Content-Type': 'application/json' },
						code: 400,
					}
				);
			} else {
				const user = new User({
					_id: generateSnowflake(),
					username: username,
					discriminator: 0,
					email: email,
					hash: hashOf(password),
				});
				user.save();
				return new Response(JSON.stringify({}), {
					headers: { 'Content-Type': 'application/json' },
				});
			}
		}
		return new Response(
			JSON.stringify({
				error: {
					code: 'DB_DISCONNECTED',
					message:
						'(Database Not Connected) Please try again in a few moments!',
				},
			}),
			{
				headers: { 'Content-Type': 'application/json' },
				code: 500,
			}
		);
	},
};

Are you sure that you have Postman configured to make POST requests? Can you share the configuration?

Can you change the GET handler to return new Response(JSON.stringify({ method: req.method, headers: Object. fromEntries(req.headers.entries()) })

@lucacasonato I got:
{"method":"GET","headers":{"accept":"*/*","accept-encoding":"gzip","cache-control":"no-cache","cdn-loop":"cloudflare","cf-connecting-ip":"<IP_ADDRESS>","cf-ipcountry":"US","cf-ray":"7d2832065b610774-IAD","cf-visitor":"{\"scheme\":\"https\"}","cf-warp-tag-id":"34d9a3d1-d316-492a-b448-37793d6dfbe0","connection":"keep-alive","host":"test.disland.uk.to","postman-token":"55c0c5bf-2dc9-494e-8596-7a25ee536804","referer":"http://test.disland.uk.to/api/register","user-agent":"PostmanRuntime/7.32.2","x-forwarded-for":"<IP_ADDRESS>","x-forwarded-proto":"https"}}

(I'm using cloudflare's tunnels because when using postman i cant run it on my own pc so im using their web version where i have to use a domain.)

@lucacasonato

It was a issue with postman :/


Google Analytics Universal Edition

Hi,
I am having problems to install the Google Analytics Tracking ID in Telescope.
I just copied&pasted the Tracking ID UA-1731XXXX-X in the related field in settings in Telescope but Google says: tracking not installed yet.
Is there a way to get it working through a workaround like this? https://github.com/reywood/meteor-iron-router-ga
Why does including my Snippet before the /head>in the main.html not solve it?
Here is the code
(function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
(i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
})(window,document,'script','//www.google-analytics.com/analytics.js','ga');
ga('create', 'UA-1731XXXX-X', 'auto’);
ga('require', 'displayfeatures');
ga('send', 'pageview');
What I am missing?

Should be working now in the latest version by adding your Tracking ID in the admin section.

Hi I'm having the same issue. I just installed Telescope a few days ago so it should be the latest version. I added just my Google Tracking ID into the admin section and in the Google Analytics interface it still is getting reported as "Tracking Not Installed".  Any suggestions?

Did you solve this problem, facing the same problem in the version 0.24. Please update on this.

@sk4blink @jimmibond what if you try the latest version? (0.25.1)


Cannot login nor reset pass phrase and Android app error messages

Cannot login either using the QR code or typing in the phrase words - the app on both windows and android just hang or throw error messages of different kinds! This is ridiculous! Also cannot reset the pass phrase or password as you apparently have no automatic reset in place, I mean come on seriously?! No support or help on hand either, tried Twitter and no reply!

@maxtaco Could you please help? I found your Github name on replies to previous login/reset issues. My username is: pwnedbook

Is it the same problem as me? https://github.com/keybase/client/issues/19818
You didn't give any clues of what problems you were having (so not sure how anyone could help you out)


samples: drivers: led_strip: Correcting document title to 'LED strip'

Other samples do not include 'sample' in the page title, so following that.

Nice catch, thanks!


Completely remove Push type

In #52122 I said I didn't have any immediate plan to attempt to remove it, but, it turns out that nearly all cases we use Push, we only ever pass in an array. The only exception to that rule is noopPush, where its only use can instead just be array?.push(...) (and I sort of expect that to be more efficient than actually going to the effort of calling a noop function through an object).
Whether or not this is desirable is another question; I can understand wanting to constrain uses to a specific operation, but I also like the cleanliness and consistency of just using arrays everywhere.

@jakebailey I'm going through all the ready-to-merge PRs, and this one has been sitting for a while. I'd say let's merge it.


[Snyk] Security upgrade se.swedenconnect.ca:ca-engine from 2.0.1 to 2.1.0

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

keycert/pom.xml

Vulnerabilities that will be fixed with an upgrade:

Issue
Score
Upgrade

Infinite loop SNYK-JAVA-ORGBOUNCYCASTLE-6612984
  600
se.swedenconnect.ca:ca-engine:  2.0.1 -> 2.1.0  No Path Found Proof of Concept

Observable Discrepancy SNYK-JAVA-ORGBOUNCYCASTLE-6613076
  520
se.swedenconnect.ca:ca-engine:  2.0.1 -> 2.1.0  No Path Found Proof of Concept

Allocation of Resources Without Limits or Throttling SNYK-JAVA-ORGBOUNCYCASTLE-6613079
  415
se.swedenconnect.ca:ca-engine:  2.0.1 -> 2.1.0  No Path Found No Known Exploit

Improper Handling of Case Sensitivity SNYK-JAVA-ORGSPRINGFRAMEWORK-8230364
  265
se.swedenconnect.ca:ca-engine:  2.0.1 -> 2.1.0  No Path Found No Known Exploit

Improper Handling of Case Sensitivity SNYK-JAVA-ORGSPRINGFRAMEWORK-8230365
  265
se.swedenconnect.ca:ca-engine:  2.0.1 -> 2.1.0  No Path Found No Known Exploit

[!IMPORTANT]

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

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Allocation of Resources Without Limits or Throttling

Fixed in https://github.com/swedenconnect/signservice/pull/183


Items not detected for my level 72 character

For this savegame: 3.zip,
the Items tab shows:

Having the exact same problem with all my characters I try to mess with. Everything gone.

Same here. SAV file goes through the loading process but no items appear in the Items list.


vcs segment shows incorrect remote branch when tracking remote isn't called "origin"

Tested on master with the default config.

Clone a repo from github.
cd into it.
git remote rename origin source

Expected: There is no remote branch in the vcs prompt (because remote is the same as local).
Actual: vcs prompt shows remote branch "source/master".

Hey @romkatv .
I've been thinking about this, since I implemented the remote name in gitstatus and used that to prepend the branch name if the remote name differs from origin. And that leads to the same error in the gitstatus segment.
And, to be honest, I don't know how if we can fix this. I don't see a proper way to see if a remote is "owned" by the user or not. As far as I know, git makes not difference between a renamed remote or a remote that was just added. All remotes are equal (I might be wrong). But if this is the case, the only thing we could do about that is to check for origin and show the remote name otherwise.

Maybe I'm reading this wrong, but what exactly is wrong?

Maybe I'm reading this wrong, but what exactly is wrong?

Oops, sorry about that. I should've tried my own reproduction instructions before posting them. This should demonstrate the issue:
git remote rename origin my/source

Full setup:
docker run -e LANG=C.UTF-8 -e LC_ALL=C.UTF-8 -e TERM=$TERM -it --rm ubuntu bash -uexc '
  apt update && apt install -y zsh git
  git clone https://github.com/bhilburn/powerlevel9k.git ~/powerlevel9k
  cd ~/powerlevel9k
  git remote rename origin my/source
  echo "
    POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(vcs)
    POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=()
    source ~/powerlevel9k/powerlevel9k.zsh-theme" >~/.zshrc
  zsh -i'

Note that branches also can have slashes in their names, so it's not possible to extract branch name from the symbolic name of a remote reference. That is, if git rev-parse --verify HEAD@{upstream} --symbolic-full-name gives you foo/bar/baz/qux, branch name can be any of qux, baz/qux or bar/baz/qux.

And, to be honest, I don't know how if we can fix this. I don't see a proper way to see if a remote is "owned" by the user or not.

Oh, now I see why you wrote that code with special-casing "origin". You want to distinguish between remotes owned by you and remotes owned by others?
Names of the remotes have no meaning on their own. A user can assign meaning to them but it's not enforced by git and thus won't be shared by other users. A theme shouldn't assume predefined meaning, at least not with default settings, so any mention of "origin" in the code is likely at error.
To distinguish between your and not-your repositories you could look at the remote URL. If the repo is under https://github.com/dritter/ or git@github.com:dritter/, it's yours, otherwise it isn't. This requires a configuration option to specify the ownership predicate.

Names of the remotes have no meaning on their own. A user can assign meaning to them but it's not enforced by git and thus won't be shared by other users. A theme shouldn't assume predefined meaning, at least not with default settings, so any mention of "origin" in the code is likely at error.

Agreed, and it does not show up in code. (only in (old) comments)

And, to be honest, I don't know how if we can fix this. I don't see a proper way to see if a remote is "owned" by the user or not.

Oh, now I see why you wrote that code with special-casing "origin". You want to distinguish between remotes owned by you and remotes owned by others?

I don't think that's the idea. The idea is to show the remote branch if the name differs. So what it should check is, if the tracking branch is refs/remotes/$remote/$localbranchname. If that doesn't exists, it should display the remote branch as $remote/$remotebranchname.
This should(tm) not collide with the issue of not not knowing where the remote ends and where the branch names start since this combination is unique locally, maybe??? (No deep research done just checked my .git/{refs/remotes/,config,FETCH_HEAD})
I don't know how to check this though. Is there a way to get just the remote without the remote branch?
git rev-parse --abbrev-ref HEAD@{upstream} and git rev-parse HEAD@{upstream} --symbolic-full-name always display the branch as well. It's not enough to check if the the output ends with /$localbranchname.
As you said: bar/baz/qux as remotebranch and qux as localbranch could mean the remote is bar with a renames local branch (show remote branch). Or the remote could be bar/baz which would mean the localbranch was not renamed (don't show remote branch).

Names of the remotes have no meaning on their own. A user can assign meaning to them but it's not enforced by git and thus won't be shared by other users. A theme shouldn't assume predefined meaning, at least not with default settings, so any mention of "origin" in the code is likely at error.

Agreed, and it does not show up in code. (only in (old) comments)

And, to be honest, I don't know how if we can fix this. I don't see a proper way to see if a remote is "owned" by the user or not.

Oh, now I see why you wrote that code with special-casing "origin". You want to distinguish between remotes owned by you and remotes owned by others?

I don't think that's the idea. The idea is to show the remote branch if the name differs. So what it should check is, if the tracking branch is refs/remotes/$remote/$localbranchname. If that doesn't exists, it should display the remote branch as $remote/$remotebranchname.

Hm... if @dritter wanted this, he would've written it so:
if [[ -n $VCS_STATUS_REMOTE_BRANCH &&
      $VCS_STATUS_REMOTE_BRANCH != VCS_STATUS_LOCAL_BRANCH ]]; then
  local r="$VCS_STATUS_REMOTE_NAME/$VCS_STATUS_REMOTE_BRANCH"
  segment_content+=("${__P9K_ICONS[GITSTATUS_REMOTE_BRANCH]}$r")
fi

But in fact he has written something much more complex. I don't understand what he's trying to achieve with that code but based on the discussion at https://github.com/bhilburn/powerlevel9k/pull/1229#discussion_r269640779 I though he wants to distinguish between his own remotes (which are usually "origin") and other remotes (he used "ben" as an example).
@dritter It would help if you could clarify.

TBH, my first idea was indeed to distinguish between my branches and foreign ones. But as you, @romkatv , correctly pointed out, this is not possible without the help of the user..
My use case is to get a hint of I am working on a branch that, if pushed, would probably break things (in my case: I have write access to this repo, and I could push to next, which is rarely what I want).
A good alternative to that is @Syphdias suggestion. Could either of you create a PR? That would be nice.

@dritter So there are two ideas here:

The initial one where you could define a lookup to find "owned" remotes. I'd say it could be a new feature. Maybe discussed in a new Issue?
locally differently named branch
Well, the pseudo code from @romkatv looks easy. I think the VCS_* variables are probably output from gitstatus(?) and I didn't find an equivalent for VCS_STATUS_REMOTE_BRANCH with vcs_info but I understand little of its workings. I came up with a simple modification. But I'm not sure if it achieves exaclty what is wanted here. For example it will unfortunatly show no remote if the remote is my/source, upstream/tacking branch is master and the local branch is source/master. I would need a variable with only the remote (without branch name) in it for it to be completely correct...
Tell me how and I'll open a PR ;)

diff --git a/segments/vcs/vcs.p9k b/segments/vcs/vcs.p9k
index de678514..b4214a5c 100644
--- a/segments/vcs/vcs.p9k
+++ b/segments/vcs/vcs.p9k
@@ -209,7 +209,7 @@ function +vi-git-remotebranch() {
   if [[ -n ${remote} \
       && ( \
         "${P9K_VCS_GIT_ALWAYS_SHOW_REMOTE_BRANCH}" == 'true' \
-        || "${remote#*/}" != "${branch_name#heads/}" \
+        || ! "${remote}" =~ "/${branch_name#heads/}$" \
       ) ]]; then
     hook_com[branch]+="${__P9K_ICONS[VCS_REMOTE_BRANCH]}${${remote// /}//\%/%%}"
   fi

Well, the pseudo code from @romkatv looks easy. I think the VCS_* variables are probably output from gitstatus(?) [...]

It was meant to be real code to replace this. I haven't tested it though.

@Syphdias agreed on idea 1. That should be a new feature.
And about your second idea: I wouldn't change the vcs segment.
@romkatv in gitstatus you strip away the remote name. So could we distinguish between the remotes? Coming back to my initial example: if I track a branch next from remote x and name that next locally, is there a difference between VCS_STATUS_REMOTE_BRANCH and VCS_STATUS_LOCAL_BRANCH? I can't check that, currently on my phone..

@dritter I'm not sure I understand the question.
If HEAD is a branch, then VCS_STATUS_LOCAL_BRANCH is the branch name. It's the named marked with an asterisk in the output of git branch.

If the branch is tracking a remote branch, then both VCS_STATUS_REMOTE_NAME and VCS_STATUS_REMOTE_BRANCH are non-empty. VCS_STATUS_REMOTE_NAME is one of the remotes you can see in the output of git remote.

$VCS_STATUS_REMOTE_NAME/$VCS_STATUS_REMOTE_BRANCH is what you see in brackets in the output of git branch -vv.

Or to put it another way, if HEAD is a branch that tracks a remote branch, then the output of git status -sb can be reproduced with "## $VCS_STATUS_LOCAL_BRANCH..$VCS_STATUS_REMOTE_NAME/$VCS_STATUS_REMOTE_BRANCH".

Concatenating strings is easier than splitting, so it's fairly easy to do anything you want with the data gitstatus provides. There is also VCS_STATUS_REMOTE_URL, which you could use for ownership checks, if desired.

@romkatv I didn't even take a look at the gitstatus segment since you reported it for vcs and I just looked at that.

And about your second idea: I wouldn't change the vcs segment.

@dritter Are you saying there is no bug here or just a "won't fix"? The behaviour original intended is no possible as we agreed on. But what is it, it is supposed to do then?
If the intention is to show differently named local branches, it is very straight forward to fix in gitstatus, just copy paste @romkatv's code since it already provides VCS_STATUS_REMOTE_BRANCH that can easily be compared to VCS_STATUS_LOCAL_BRANCH. For vcs I don't know, since I couldn't find a VCS_STATUS_REMOTE_BRANCH-equivalent in vcs_info.
If this is not wanted (currently buggy) behaviour. What should it do? Display the remote ref if there is a/ in the remote name?

@romkatv I didn't even take a look at the gitstatus segment since you reported it for vcs and I just looked at that.

Yeah, it's confusing because we have two parallel threads here. One is the original bug I reported against vcs, and another is about differentiating between remotes in gitstatus that follows from @dritter's comment (https://github.com/bhilburn/powerlevel9k/issues/1209#issuecomment-478402344). All your comments are from the first conversion, while all (or almost all) @dritter's are from the second. I'm replying to both, so I'm confusing everyone.

Haha. Sorry for contributing to the confusion here.
I'll try to bring some light into the dark here:
About vcs segment showing the wrong remote
I think this is more a display issue how we want to display the remote name. In @romkatv screenshot the remote name is mangled with the remote branch, and, if I understood correctly, this is the original issue here. I'll take a deeper look tomorrow.
About remote name in gitstatus segment

... VCS_STATUS_REMOTE_BRANCH that can easily be compared to VCS_STATUS_LOCAL_BRANCH.

I disagree with that one. Local and remote branch might be the same (see my use case below).
Use case
Steps in short:

Create a repo with a second branch
Fork that repo
Clone the fork
Add original repo as remote
Checkout the second branch from the remote (original repo)

To illustrate my use case, I use this repo (bhilburn/powerlevel9k) and Syphdias fork (syphdias/powerlevel9k) as an example (my own fork is a bad example, because I switched the default branch in github). So I cloned syphdias fork (master branch) and added the original repo (bhilburn/powerlevel9k) as remote. After that I checkout bhilburn/next.
If I now do gitstatus_query P9K, $VCS_STATUS_LOCAL_BRANCH and $VCS_STATUS_REMOTE_BRANCH both show next, which is why I started comparing against origin (here is where I went wrong). And this is the loop hole in @romkatv code snippet.
I hope that clears it a bit up, and does not add more confusion.

@dritter, could you describe when and how the right part should be displayed? If there is more than one remote? So for your example:
If you're on master: master→origin/master
If you're on next: next→bhilburn/next
What is not possible imho, is to only show the remote branch for the second thing since we can't know that origin is "your own" (at least not without the detection via url or similar).
PS: Adding to the confusion: my master is closer to upstream next because of an old PR. 😜 And why is specifying harder than programming...

Having slept one night about this problem, I think my use case is an edge case and is not solvable. My use case is about ownership, which git does not know about, hence we need user input for this case. I'll think some more about this and create a new PR, if I got to somewhere..

Seems like the level of confusion is getting boringly low here. Let's bump it up a notch.

After that I checkout bhilburn/next.
If I now do gitstatus_query P9K, $VCS_STATUS_LOCAL_BRANCH and $VCS_STATUS_REMOTE_BRANCH both show next...

If you literally run git checkout bhilburn/next, you'll end up in detached HEAD state and not on any branch. VCS_STATUS_LOCAL_BRANCH and VCS_STATUS_REMOTE_BRANCH will be empty. Symbolic reference refs/remotes/bhilburn/next will resolve to the same commit as HEAD, but it doesn't mean you are on bhilburn/next branch. If your local next branch is up to date with bhilburn/next, symbolic reference refs/heads/next will also resolve to the same commit as HEAD, but this also doesn't mean you are on next branch.
gitstatus has straightforward logic here. VCS_STATUS_LOCAL_BRANCH is empty if and only if the repo is in detached HEAD state. VCS_STATUS_REMOTE_BRANCH and VCS_STATUS_REMOTE_NAME are empty if and only if you are not on a branch with a tracking remote.
P9K has different logic. What it shows as local branch name isn't always a branch. For example, if you run git checkout master && git checkout HEAD~0, you'll be in detached HEAD with P9K showing heads/master. After git checkout HEAD~1 it'll either become master~1 (note the inconsistency with the previous display, which wasn't master~0) or heads/some-other-branch, depending on what other branches you have. However, if you end up on a tagged commit, then local branch name will be different in both cases -- it'll be just the commit hash.
This diversity of local branch name formats and meanings makes it challenging for users to build a consistent model for interpreting what they are seeing. If P9K says I'm on heads/master, it could mean I'm on a branch named heads/master and therefore my commits will move HEAD, or it can mean I'm in a detached HEAD state at the same commit as master branch. There is no way to tell.

If you literally run git checkout bhilburn/next, you'll end up in detached HEAD state and not on any branch.

Yeah. I forgot a -t.. If you do a git checkout -t bhilburn/next, you'll end up in the state I described.

This diversity of local branch name formats and meanings makes it challenging for users to build a consistent model for interpreting what they are seeing.

Full Ack. We should try to make that as consistent as possible. I'll have a look into the vcs segment.


Allow CloudFront continuous deployments

Is your feature request related to a problem
CloudFront allows you to do continuous deployments by creating a staging distribution and routing a portion of traffic to the staging distribution
Describe the solution you'd like
The ability to configure and use continuous deployments as part of the rollout of a change
There are at least two options that can be supported

Only updates to functions & assets
Updates to any config i.e. modifying distribution config, adding functions, etc.

Additional context
https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/continuous-deployment.html
https://aws.amazon.com/blogs/networking-and-content-delivery/use-cloudfront-continuous-deployment-to-safely-validate-cdn-changes/
This will likely include breaking changes, so will probably go out as part of 2.x.x of the module

Blocked by https://github.com/hashicorp/terraform-provider-aws/issues/34511

Workaround created using a bash script to remove the continuous deployment policy from the distribution


hey, Nice script

Can you really install gentoo with this? Let me know how it works. :)

Hi, it's been a long time :)
Yeah i will soon be able to boot, the kernel is compiling for now :+1: , i install grub and test the boot tomorrow on Virtualbox.
I make a simple system first with ext4, later i'll add support for lvm and zfs and encrypted system.

does it take you through the whole install proccess of gentoo?

Sure and more (kernel patched with KSPP, for now, i test only with the last release for the stage3-amd64-systemd (on BIOS with grub and EFI with systemd-boot), after i add the encryption...

So i've finish to add systemd-boot, here's what I did:

Boot on the ISO of archlinux
loadkeys fr
curl -o setup.sh https://raw.githubusercontent.com/szorfein/getch/master/bin/setup.sh
chmod +x setup.sh && ./setup.sh
source ~/.zshrc
getch --username ninja --zoneinfo "Europe/Paris" --language fr_FR --keyboard fr --disk sda

And just wait (at least 4-6 hour for compile the whole system, depend on your machine, the mine sucks).
The script:

Detect EFI or BIOS
Create 4 partition on disk (/boot, / , swap and /home) and mount them at /mnt/gentoo
Download the last stage3-amd64-systemd, verify the checksum
Chroot in the system and update all
Configure the system
Compile the kernel by checking the output of lsmod, for example if he found the module iwlwifi, it add the matching options.
Install and configure the boot loader (GRUB2 or systemd-boot)
Create password
Umount all partitions

After the reboot, you have to configure your network.

what would i put for mine. (loadkeys fr, zoneinfo "Europe/Paris" --language fr_FR --keyboard fr) I'm guessing mine would be different for english what is that?

--keyboard available can be found with: ls /usr/share/kbd/keymaps/**/*.map.gz or /usr/share/keymaps/**/*.map.gz - for me, my keyboard layout is located at /usr/share/keymaps/i386/azerty/fr.map.gz
--zoneinfo at: /usr/share/zoneinfo/Region/City - for me /usr/share/zoneinfo/Europe/Paris
--language can be found in the /etc/locale.gen , just skip the  UTF-8 or .UTF-8 part so fr_FR.UTF-8 UTF-8 become just fr_FR

my friend used the script he's gentoo master so i have hes help :) he said it only gave him 15gb of space he had to regrow it in Arch iso. he also had to emerge newtorkmanager linux-firmware

How do i make only 1 partition /root that's all i use not 4

he also had to emerge newtorkmanager linux-firmware
linux-firmware is alrealy include but not network-manager

getch is just a tool to make a default install, people are free to add their favorite packages after or just before reboot, i will add a message at the end of the install.

How do i make only 1 partition /root that's all i use not 4
The default install make:

/boot or /boot/efi - A must for boot especially the /boot/efi (~= 128M)
/ - (15G)
Swap - (Same size of your memory) You can easily remove it if you don't need.

And if you use the option --username, it create a /home directory.
So 3 partitions minimum but like you friend, you can resize or remove partitions.
But yeah, maybe i will add an option for use the whole space available for the / when --username is not set, just to avoid manual task.

How would i just use /root all space right now what do i need to change?


Craft 3?

Is there any plan to move this over to Craft 3? I was about to start a new plugin today using this as a reference, so I figured I'd ask.
I know it's probably at the bottom of the priority list, but I figured I'd ask.
Thanks!

Hi @bryantAXS,
Did you got any reference for creating new Element Type in CraftCms3 ?

@maddyBoy I did not.

I'm also interested in a Craft 3 version :)


Documentation for OpenAPI plugin

@TobiasWalle do you have time to contribute anything on this? I've tried to get started on it a few times, but it's a bit "heavy".

Sure. This week I won't have much time, but I can help out next week. Where can I get started?

Great! I think this should probably have its own page, like documentation/openapi, I can set that up.

I created https://javalin.io/plugins/openapi
The source file is at https://github.com/javalin/javalin.github.io/blob/master/pages/plugins/openapi.md
If you want to build the website locally you'll need jekyll: https://jekyllrb.com/docs/

Thank you! I will let you know once I think the documentation is ready.

Do you consider this done @TobiasWalle ?

@tipsy The only thing is missing, are the Kotlin examples. I didn't had time to do it yet, but it is very similar to the java examples. There are just some small API differences, so for example, you can omit the OpenApiBuilder. prefix and can just write document() instead of OpenApiBuilder.document().
Otherwise I consider this issues as closed.

If the difference is that small, I think we can live without them for a while. I'll close this now. Thanks again!


Ingestor pay attention to version

At present the Ingestor has no notion of version and so could write updates to ES in the wrong order.
Follows on from https://github.com/wellcometrust/platform/issues/1563, in using the work version and conditionally updating Elasticsearch (it is likely we will need to use external versioning in ES: https://www.elastic.co/blog/elasticsearch-versioning-support).

Duplicate of #1425?

^ closing as dupe.


NameError: name 'basestring' is not defined

It's specified that the libraries are compatible with python 3 but when running:
bitcoin_summary = blockchain.blockexplorer.get_multi_address(...)

an error is thrown:
File "/usr/local/lib/python3.5/dist-packages/blockchain/blockexplorer.py", line 127, in get_multi_address
    if isinstance(addresses, basestring):
NameError: name 'basestring' is not defined

I think the fix is to change all basestring instances to str as basestring is no longer defined in Python3. That may not fix it, but it is a start

A quick test on get_balance indicates that replacing instances of basestring with str works fine. I haven't run it through any unit tests however, so I can't guarantee another function isn't broken.

You can simply fix this by adding this line to the head of the file blockchain/blockexplorer.py:
from future.utils import string_types as basestring
This works, because isinstance also allows a list of types. I think this is the easiest solution for that and it doesn't break on python2

there is already a PR on fixing this issue, please direct further comments there https://github.com/blockchain/api-v1-client-python/pull/78


Callback URL

Hi Alex,
I've installed this plugin in my pybossa installation and created a github application - but I can't work out what to use as the authorization callback URL. Any help you could offer would be much appreciated, as I still don't have a "Sync with GitHub" option on my dashboard in PyBossa (ultimately I'm trying to get project-convert-a-card working).
Thanks,
Gary

Hi Gary,
So, the callback URL would be https://{your-pybossa-domain}/github/oauth-authorized and to get the "Sync with GitHub" link you need to add something like this wherever you want the link to appear:
{% if 'pybossa_github_builder' in plugins %}
    Sync with GitHub
{% endif %}

Note that you will also need the pybossa-z3950 plugin to get the convert-a-card projects working. I'll try to update the docs soon to make this a bit clearer.


Allow to scan small QR Codes

Actually seems the QR code scanner doesn't recognize QR codes smaller then 20x20mm, a possible fix could be using this workaround replacing the vue-qrcode-reader library with that one, see https://github.com/mebjas/html5-qrcode/tree/master/examples/vuejs

+1 on this. The QR codes on most zooz switches are pretty small and I have never been able to get zwavejs to read them.

actually the trick is to take a photo of the qr codes with yopur phone, zoom it and scan the zoomed photo


Track new organisation first IP creation pathway v2.0

WHY:
This PR is version 2 of second step to allowing GA to track first IP creation activity paths. PR #602
First step is PR #595
A new organisation is an organisation with no IPs set up regardless of adding other things e.g location, team members.
We want to follow a new organisation's conversion i.e adding their first IP address and thus making their installation of GovWifi count as an active one.
We need to follow the difference between new and existing organisations through the "create IP" pathway.
IN THIS PR:

Add data layer script with signed_in and hasIPs variables:

Added  a signed_in variable because hasIPs is false when the user is not signed in.

hasIPs is false when the organisation has no IPs

hasIPs is false when the organisation has no IPs (even if locations are added)

hasIPs is true when the organisation had an IP address

Any feedback or suggestions on this approach would be appreciated

UPDATE: Changed signed_in to signedIn to match javascript naming style.


Adding some light documentation

I was trying to get through some of the code on how the map was processed and loaded, like in functions TiledMapLoader and process_loaded_tile_maps, but the code is a little dense and sparsely documented. I'd like to help contribute to this project, and having a little bit more info on how the main processes worked would be a great help.

There is now a readme, and I tried to add comments in the code (check out bevy-0.4-objects branch) but if there are particular things you want to know more about, please let me know


Yannick261

Made changes to deal with Issue #261.
Now, if the model was built (and the data cleaned) through supervised_model_trainer, the pre-dummified columns are used to check for missing variables in the prediction data.  To correctly perform dummification on the prediction data, columns of dtype object are converted to dtype category where the categories are taken from the training data values.  One side effect is that new categories not seen in the training data are set to NaN (and then imputed or dropped) instead of being assigned the base class determined by the dummification (usually the category coming first alphabetically) as before.

Added information about the training data pre-dummified columns and categorical columns to supervised_model_trainer, advanced_supervised_model_trainer, and trained_supervised_model
Modified DataFrameImputer to impute with columns of dtype 'category' (issue #105)
Cast prediction dataframe columns of dtype object to dtype category where the categories were taken from the training set to allow for proper dummification
Added a message to warn the user if new levels of a categorical variable appear in the prediction data
Removed requirement that response variable be included in prediction data

I ran the example scripts (regression, classification, and advanced) to verify that they still work.  I built the following toy data sets
np.random.seed(5077)

df = pd.DataFrame({"id":range(30),
                   "A":np.random.normal(size=30),
                   "B":np.random.normal(size=30),
                   "C":np.random.choice(["red", "blue", "green"], size=30)},
                  columns=["id", "A","B","C"])

df["dot"] = 5*df["A"] - 6*df["B"]
df.loc[df["C"] == 'red', "dot"] += 4
df.loc[df["C"] == 'blue', "dot"] += -6
df.loc[df["C"] == 'green', "dot"] += 2

df["dot2"] = df["dot"] + np.random.normal(size=30, scale=2)

df["num_response"] = df["dot2"] + np.random.normal(size=30, scale = 1)
df["cat_response"] = 'N'
df.loc[df["dot2"] > 0, "cat_response"] = 'Y'

df_regression = df.drop(["dot", "dot2", "cat_response"], axis=1)
df_classification = df.drop(["dot", "dot2", "num_response"], axis=1)

np.random.seed()

and verified the following properties hold, but I would appreciate it if you could double check them (preferably not on the exact same data):

[ ] doesn't raise error if some categories present in the training data are missing in the prediction data (e.g., able to predict a single row of new data)
[ ] does raise error if entire categorical variable is missing entirely in the prediction data (still checks for missing columns)
[ ] individual predictions are independent of the other rows in the prediction data (i..e., dummification is independent of the prediction data)
[ ] performs the desired imputation on new categories not present in the training data

:x: Build healthcareai-py 1.0.545 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/1d64103326 by @yvanhuele)

:x: Build healthcareai-py 1.0.548 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/5890588136 by @yvanhuele)

I checked the four properties and they hold for my dataset. That's great!

:x: Build healthcareai-py 1.0.552 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/6ce60a2338 by @yvanhuele)

:x: Build healthcareai-py 1.0.580 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/9288686690 by @Aylr)

:x: Build healthcareai-py 1.0.599 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/f455138f8d by @Aylr)

:x: Build healthcareai-py 1.0.607 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/fbaf3b5b0c by @yvanhuele)

:x: Build healthcareai-py 1.0.611 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/804724d15f by @Aylr)

:x: Build healthcareai-py 1.0.622 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/33a5eb3765 by @Aylr)

:x: Build healthcareai-py 1.0.627 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/76be0237a0 by @Aylr)

I moved the function get_categorical_levels() to its own file in healthcareai/common and added unit tests to

test that single row predictions work
test that predictions are independent of the other rows in the dataframe
test that an error is raised if a column is fully missing (numeric or categorical)
test that new category levels are imputed correctly
test that get_categorical_levels() behaves as expected

:x: Build healthcareai-py 1.0.695 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/3b98d69235 by @Aylr)

:x: Build healthcareai-py 1.0.699 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/d34c7d2997 by @Aylr)

:x: Build healthcareai-py 1.0.701 failed (commit https://github.com/HealthCatalyst/healthcareai-py/commit/6b0f6bb270 by @Aylr)


GUIs closing when internet is lost

Windows Version
Microsoft Windows [Version 10.0.22631.3527]
WSL Version
WSL version: <IP_ADDRESS> Kernel version: <IP_ADDRESS>-1 WSLg version: 1.0.59 MSRDC version: 1.2.4677 Direct3D version: 1.611.1-81528511 DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp Windows version: 10.0.22631.3527
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version <IP_ADDRESS>-microsoft-standard-WSL2 (root@1c602f52c2e4) (gcc (GCC) 11.2.0, GNU ld (GNU Binutils) 2.37) microsoft/WSL#1 SMP Thu Oct 5 21:02:42 UTC 2023
Distro Version
Ubuntu, Release:        22.04
Other Software
This happens with all the applications that use GUIs. Terminator, Evince, etc.
Repro Steps

Open a programs that uses a GUI.
Open multiple virtual desktops.
Place the GUIs in those desktops, one in each.
Interrupt the internet connection. e.g by shaking slightly the ethernet cable.
All GUIs dissapear.
Open another GUI, e.g. by issuing a command in the terminal. All GUIs will reappear in the current virtual desktop.

Expected Behavior

Open a programs that uses a GUI.
Open multiple virtual desktops.
Place the GUIs in those desktops, one in each.
Interrupt the internet connection. e.g by shaking slightly the ethernet cable.
Internet goes away but nothing else happens.

Actual Behavior

Open a programs that uses a GUI.
Open multiple virtual desktops.
Place the GUIs in those desktops, one in each.
Interrupt the internet connection. e.g by shaking slightly the ethernet cable.
All GUIs dissapear.
Open another GUI, e.g. by issuing a command in the terminal. All GUIs will reappear in the current virtual desktop.

Diagnostic Logs
No response
### Tasks
- [ ] https://github.com/microsoft/wslg/issues/1098

OK, the problem just happened yesterday and I decided to get done with this:
WslLogs-2024-05-25_16-05-36.zip

Ok, there seems to be a solution for this:
https://github.com/microsoft/wslg/issues/1098#issuecomment-2073387788
It's really sad that this issue, been such a big problem, reported by so many people, took around a year to get fixed by a company that is worth tens of billions and has thousands of engineers, in an operating system that is not even free, like linux...

I had the same problem. I am connected to the internet through
usb modem every time i connect/disconnect my google chrome/jupyter lab
disappeared and WSL crashes.
But jupyter lab through
jupyter  lab --no-browser
and opening from Win edge is ok.
################
Then after updating WSL+Kernels, it worked.
#################
I had a special  hand compiled kernel to enable kernel drivers.
I haven't yet tried to compile a new kernel.

(base) C:\WINDOWS\system32>wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.61
MSRDC version: 1.2.5326
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26091.1-240325-1447.ge-release
Windows version: 10.0.22000.160

one more comment
I still have since infinity
WSL crashs when i exceed the RAM

It is strange, because 2.2.3 seems to fix this and you have 2.2.4 WSL @kh-abd-kh I also have 2.2.4 and i haven't seen the issue anymore.

acampove
you are talking about the RAM problem.
It is since infinity working with sage. I can generate big RAM usage . Sometimes just kernel restart.
but more often crashs. When You do big calculation you don't know what to expect.
The Network was solved completely with 2.2.4. No more at all. I am very heavy user  of WSL 8hr+ per day.


Get backups price from API

Closes #279

Coverage decreased (-1.7%) to 87.782% when pulling db32fe12bc2697e93ba4b310400575407600f84c on briansteffens:backups-price into 35b141d57af4f302320e411b2026d27814ec4887 on linode:master.

This should also include the same thing on the Linode -> Backup page.

Coverage increased (+0.04%) to 89.485% when pulling 65e416047118e42151a120aabd133069d65c1736 on briansteffens:backups-price into 35b141d57af4f302320e411b2026d27814ec4887 on linode:master.

Coverage increased (+0.04%) to 89.485% when pulling 3622e0be8348e2b55298a47277b612a67efcd69c on briansteffens:backups-price into 35b141d57af4f302320e411b2026d27814ec4887 on linode:master.

:+1:


Copy edits

Updating "open-source" to "open source" based on https://github.com/cncf/tag-contributor-strategy/pull/339#discussion_r1132601109
Additional copy editing

Good changes, thank you @nate-double-u


Could not download antlr.jar while building android

Description
I created a new react app and tried to build android version by running command : react-native run-android
i get following error :
react-native run-android
Scanning 560 folders for symlinks in /home/moein/AwesomeProject/node_modules (5ms)
Starting JS server...
Building and installing the app on the device (cd android && ./gradlew installDebug)...

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'AwesomeProject'.
> Could not resolve all dependencies for configuration ':classpath'.
   > Could not download antlr.jar (org.antlr:antlr:3.5.2)
      > Could not get resource 'https://jcenter.bintray.com/org/antlr/antlr/3.5.2/antlr-3.5.2.jar'.
         > Could not HEAD 'https://jcenter.bintray.com/org/antlr/antlr/3.5.2/antlr-3.5.2.jar'.
            > Connect to jcenter.bintray.com:443 [jcenter.bintray.com/<IP_ADDRESS>] failed: Connection timed out

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 2 mins 15.115 secs
Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/android-setup.html

I'm also using tor so it's not related to restriction or whatever
Additional Information
react-native-cli: 2.0.1
react-native: 0.43.2

Platform: Android
Development Operating System: Linux Ubuntu 16.04
Dev tools: android sdk version : 24.3.4

I got a step further now i'm getting this error :+1:
react-native run-android
Scanning 560 folders for symlinks in /home/moein/AwesomeProject/node_modules (4ms)
Starting JS server...
Building and installing the app on the device (cd android && ./gradlew installDebug)...
Observed package id 'build-tools;23.0.2' in inconsistent location '/home/moein/android-sdk-linux/build-tools/android-6.0' (Expected '/home/moein/android-sdk-linux/build-tools/23.0.2')
Observed package id 'platforms;android-22' in inconsistent location '/home/moein/android-sdk-linux/platforms/android-5.1' (Expected '/home/moein/android-sdk-linux/platforms/android-22')
Observed package id 'platforms;android-23' in inconsistent location '/home/moein/android-sdk-linux/platforms/android-6.0' (Expected '/home/moein/android-sdk-linux/platforms/android-23')
Observed package id 'build-tools;23.0.2' in inconsistent location '/home/moein/android-sdk-linux/build-tools/android-6.0' (Expected '/home/moein/android-sdk-linux/build-tools/23.0.2')
Observed package id 'platforms;android-22' in inconsistent location '/home/moein/android-sdk-linux/platforms/android-5.1' (Expected '/home/moein/android-sdk-linux/platforms/android-22')
Observed package id 'platforms;android-23' in inconsistent location '/home/moein/android-sdk-linux/platforms/android-6.0' (Expected '/home/moein/android-sdk-linux/platforms/android-23')
Observed package id 'build-tools;23.0.2' in inconsistent location '/home/moein/android-sdk-linux/build-tools/android-6.0' (Expected '/home/moein/android-sdk-linux/build-tools/23.0.2')
Observed package id 'platforms;android-22' in inconsistent location '/home/moein/android-sdk-linux/platforms/android-5.1' (Expected '/home/moein/android-sdk-linux/platforms/android-22')
Observed package id 'platforms;android-23' in inconsistent location '/home/moein/android-sdk-linux/platforms/android-6.0' (Expected '/home/moein/android-sdk-linux/platforms/android-23')
Failed to download any source lists!
Errors during XML parse:
javax.xml.bind.UnmarshalException
 - with linked exception:
[java.net.MalformedURLException]
Additionally, the fallback loader failed to parse the XML.

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':app'.
> Failed to find Build Tools revision 23.0.1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 13.037 secs
Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/android-setup.html

Looks like something wrong with your android sdk, you need to install Build Tools revision 23.0.1.
You may need to reinstall your android sdk, check the link in the error info.

thanks, it is solved now :+1:

I have exact same problem.
i checked android sdk installed correctly.
so can i ask you how to solved this problem?

@MHasanian
I set proxy for npm via this commands
npm config set proxy http://proxy.company.com:8080
npm config set https-proxy http://proxy.company.com:8080


Test and validate the monorepo setup, ensuring that: 1. Backend and frontend run and build independently as well as together. 2. TurboRepo handles parallel builds efficiently across both packages.

Task Overview:
Test and validate the monorepo setup, ensuring that the backend (Node.js) and frontend (Vite) can be built and run independently as well as together. Additionally, ensure that TurboRepo optimizes the build process by handling parallel builds efficiently across both packages.

SMART Criteria
Specific 🎯:
Test and validate the monorepo setup by:

Running backend and frontend projects independently to ensure they can build and execute separately.
Running both projects together to ensure the monorepo can handle combined builds.
Validate that TurboRepo performs parallel builds efficiently, reducing build time across both projects.

Measurable 📏:
Success will be measured by:

Backend and frontend projects building and running independently without issues.
Combined builds working seamlessly, with no conflicts or dependency issues.
TurboRepo improving build times by at least 30% when building both packages in parallel.

Achievable 🚀:
This task is achievable by running independent build and run commands for each project, testing both in isolation and in combination. TurboRepo's parallel execution features will optimize the process for faster builds.
Relevant 🎯:
Testing the monorepo setup is essential for ensuring the workflow supports efficient builds, whether for individual projects or combined efforts. TurboRepo will help optimize build times and parallel execution.
Time-bound ⏳:
This task should be completed in 2 days, allowing sufficient time to test both independent and combined builds and optimize the parallel execution with TurboRepo.

Subtasks 📝

[x] Run and test the backend (Node.js) project independently using pnpm commands.
[x] Run and test the frontend (Vite) project independently using pnpm commands.
[x] Execute combined builds for both backend and frontend to ensure they run together smoothly.
[x] Validate that TurboRepo handles parallel builds efficiently by monitoring build times for both individual and combined projects.
[x] Optimize any build or runtime issues identified during the validation process.

Acceptance Criteria ✅

[x] Backend and frontend projects build and run independently without errors.
[x] Combined builds work seamlessly, and there are no conflicts or issues when both projects run together.
[x] TurboRepo improves parallel build times by at least 30%, with noticeable optimization.

Additional Notes 🗒
Monitor build logs to track any performance improvements using TurboRepo, and ensure that the monorepo is scalable for future packages or projects.

This is done :)


dodge docker-assigned MYSQL_PORT variable

MYSQL_PORT is meaningful in zipkin, basically looking for what's likely port 3306
if you name a container mysql, docker will assign a variable like this: MYSQL_PORT=tcp://<IP_ADDRESS>:3306
Maybe we can make this smarter and automatically unset MYSQL_PORT if it has a non-numeric expression https://github.com/openzipkin/docker-zipkin/blob/master/base/.mysql_profile

Environment variables like this are only set when using links in v1 (legacy / deprecated) docker-compose.yml files
The name of the variable is built from the alias, if it exists. For example --link mysql:mysql_container will set an environment variable MYSQL_CONTAINER_PORT.

Based on these, I'd prefer not introducing code to handle this case, but instead add a note to the README. Sound good?

yep, if we choose to not handle it, we just need to adjust the
docker-compose examples to use a suffix like you mentioned. ex.
mysql_container

The only v1 docker-compose file is https://github.com/openzipkin/docker-zipkin/blob/master/docker-compose-legacy.yml, which calls the mysql container storage, so I think we're good.

this is the one that tripped the issue:
https://github.com/openzipkin/docker-zipkin/blob/master/docker-compose-mysql.yml#L13

But but but. https://docs.docker.com/compose/link-env-deprecated/ says "Environment variables will only be populated if you’re using the legacy version 1 Compose file format.". I'm confused, will reproduce.

ahh.. actually I copied this, but maybe left out the version 2 thing.. will
verify!

On Docker 1.10.2, build c3959b1, docker-compose 1.6.2, build 4d72027, the env var is not set:
$ docker exec query env
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/java/jre/bin:/usr/local/java/bin
HOSTNAME=9024f5de4d16
STORAGE_TYPE=cassandra
TRANSPORT_TYPE=http
CASSANDRA_CONTACT_POINTS=cassandra
JAVA_HOME=/usr/local/java
JRE=/usr/local/java/jre
JAVA_OPTS=
ENV=/etc/shinit.sh
ZIPKIN_REPO=https://jcenter.bintray.com
ZIPKIN_VERSION=1.39.0
HOME=/root

Couldn't reproduce:
$ docker --version
Docker version 1.10.2, build c3959b1
$ docker-compose--version
docker-compose version 1.6.2, build 4d72027
$ docker exec query env
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/java/jre/bin:/usr/local/java/bin
HOSTNAME=ee48fe35a81a
STORAGE_TYPE=mysql
TRANSPORT_TYPE=http
MYSQL_HOST=mysql
CASSANDRA_CONTACT_POINTS=cassandra
JAVA_HOME=/usr/local/java
JRE=/usr/local/java/jre
JAVA_OPTS=
ENV=/etc/shinit.sh
ZIPKIN_REPO=https://jcenter.bintray.com
ZIPKIN_VERSION=1.39.0
HOME=/root

verified.. this is only an issue if using docker-compose version 1.

thanks for looking into it!


use max SMs at the last RS chunk in pipelined overlap

Use 32 SMs at the last ReduceScatter chunk in the split-pipelined overlap

Merged into #162