GPU Version on Ubuntu

I am not able to build the GPU version on my machine.
I have see the following error:
In file included from /tmp/RtmpzHELwg/LightGBM/lightgbm_r/src/src/treelearner/parallel_tree_learner.h:5:0,
                 from /tmp/RtmpzHELwg/LightGBM/lightgbm_r/src/src/treelearner/voting_parallel_tree_learner.cpp:1:
/tmp/RtmpzHELwg/LightGBM/lightgbm_r/src/src/treelearner/gpu_tree_learner.h:27:34: fatal error: boost/compute/core.hpp: No such file or directory
compilation terminated.

Boost seems to be correctly installed:
-- The C compiler identification is GNU 5.4.0
-- The CXX compiler identification is GNU 5.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Try OpenMP C flag = [-fopenmp]
-- Performing Test OpenMP_FLAG_DETECTED
-- Performing Test OpenMP_FLAG_DETECTED - Success
-- Try OpenMP CXX flag = [-fopenmp]
-- Performing Test OpenMP_FLAG_DETECTED
-- Performing Test OpenMP_FLAG_DETECTED - Success
-- Found OpenMP: -fopenmp
-- Looking for CL_VERSION_2_0
-- Looking for CL_VERSION_2_0 - found
-- Found OpenCL: /usr/lib/x86_64-linux-gnu/libOpenCL.so (found version "2.0")
-- OpenCL include directory:/usr/include
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   filesystem
--   system
-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/RtmpzHELwg/LightGBM/lightgbm_r/src/build

I am on Ubuntu. Have you ever seen this error?

Are you able to compile for CLI with GPU support?

Yes! Just runned this with success:
mkdir build ; cd build
cmake -DUSE_GPU=1 ..
# if you have installed NVIDIA CUDA to a customized location, you should specify paths to OpenCL headers and library like the following:
# cmake -DUSE_GPU=1 -DOpenCL_LIBRARY=/usr/local/cuda/lib64/libOpenCL.so -DOpenCL_INCLUDE_DIR=/usr/local/cuda/include/ ..
make -j4

@dfalbel I think it's missing the compute submodule for unknown reasons.
Can you run interactively the following? https://github.com/Laurae2/lgbdl/blob/master/R/lgb.dl.R#L61-L179
You must do just before running lgb.dl:
debugonce(lgb.dl)

When you hit the following line: https://github.com/Laurae2/lgbdl/blob/master/R/lgb.dl.R#L103
Go in the temporary directory manually using your favorite way (file explorer or command line), and check whether the boost submodule is present (the compute folder).
ping @jameslamb did you modify the installation for LightGBM R package for the compute submodule for GPU, or does it still lie on the same folder without being moved?

Yes! It seems to be:
dani@dfalbel-System-Product-Name:~/LightGBM$ cd /tmp/RtmpLn8Nzh/
dani@dfalbel-System-Product-Name:/tmp/RtmpLn8Nzh$ ls
libloc_169_8f48fc7d1db1bb3e.rds  libloc_174_173b0d0583319ed2.rds  libloc_180_a38f7d5813344af9.rds  LightGBM  temp.sh
dani@dfalbel-System-Product-Name:/tmp/RtmpLn8Nzh$ cd LightGBM
dani@dfalbel-System-Product-Name:/tmp/RtmpLn8Nzh/LightGBM$ ls
build_r.R       CODE_OF_CONDUCT.md  docker  examples  include  pmml            README.md  src   tests        windows
CMakeLists.txt  compute             docs    helpers   LICENSE  python-package  R-package  swig  VERSION.txt
dani@dfalbel-System-Product-Name:/tmp/RtmpLn8Nzh/LightGBM$ cd compute ls
dani@dfalbel-System-Product-Name:/tmp/RtmpLn8Nzh/LightGBM/compute$ ls
cmake  CMakeLists.txt  CONTRIBUTING.md  doc  example  include  index.html  LICENSE_1_0.txt  meta  perf  README.md  test

@dfalbel I pushed b369192 to fix it.


Added nodes not appearing on map after confirmation of creation when launching without Docker

Steps followed : Setup and run the project without Docker as stated in the documentation on Ubuntu 22.04 and Firefox. Use walking, driving and all buses transit modes using commands in the documentation. Launch transition in browser and reload routing module. Add a node with the button on the left.
Expected behavior : Blue point at the clicked position appears on the map.
Observed behavior : Blue point appears when creating the node. When clicking the confirm button, blue point disappears, map flashes one time and blue point does not reappear. Blue point does however exist, as it reappears on page refresh.

Error message is the following :
socket received event transitNode.create
TypeError: Cannot read properties of undefined (reading 'integer_id')
at [project folder]/transition/packages/transition-backend/lib/api/transitObjects.socketRoutes.js:133:55
at Generator.next ()
at fulfilled ([project folder]/transition/packages/transition-backend/lib/api/transitObjects.socketRoutes.js:28:58)
at processTicksAndRejections (node:internal/process/task_queues:96:5)


Save Activity Clips

A good value add here would be to record the activities in a saved CSV and also wait and download the clips/alerts/cropped images to local storage.


.Net Agents - History Management Consistency: ChatCompletionAgent

The direction invocation (no chat) case for ChatCompletionAgent requires the user to manually update the ChatHistory with the agent response.  This isn't required for the streaming case or any usage of OpenAIAssistantAgent.
Even worse, the Termination scenario for IAutoFunctionInvocationFilter and manual function calling cases require complex, non discoverable logic to avoid introducing invalid changes to the ChatHistory.
Having ChatCompletionAgent.InvokeAsync fully manage the history not only is consistent with the rest of the Agent Framework, it solves this advance, non-discoverable cases.

As breaking behavior change, perhaps introduce a setting that allows caller to opt-into an improved pattern?


whitespace matters when not using s-expressions in console

2*2 evaluates to a void variable
but
2 * 2  evaluates to 2 ?

Correct usage is (+ 2 2), it's a lisp ;)

bitch lasagna

@fabulousduck this is by design. 2*2 evaluates to a symbol with name 2*2. 2 * 2 evaluates to just 2 because the rest of the expressions in the line are not parsed. And I think this is actually bug.


DeprecationWarning: tostring() is deprecated. Use tobytes() instead

FYI I'm getting this warning on SimpleITK 1.2.4:
  /usr/local/Caskroom/miniconda/base/envs/torchio/lib/python3.8/site-packages/SimpleITK/SimpleITK.py:3446: DeprecationWarning: tostring() is deprecated. Use tobytes() instead.
    _SimpleITK._SetImageFromArray( z.tostring(), img )

Thank you for the report. I'll be making the recommended fix to the v2.0.0 development branch, so it will be in SimpleITK 2.0rc3.


Heroku Buildpack works except next does not apply correct path

Heroku Buildpack works except next.js does not apply correct path
Need ability to 'rewrite' path to support next.
/.next/dist/bundles/ needs replaced with /_next/<BUILD_ID>/ in the file path.
Folders after /bundles/ needs to persist.

Resolved. The paths are rewritten with the /_next/<BUILD_ID>/ correctly and upload properly.
The replacement only happens for the paths that have /.next/dist/bundles/ in the path. Otherwise they are correctly ignored.


High CPU with multiple CLOSE_WAIT sockets

After a day of running three instances of parcel-tcp2udt on multiple centos 7 systems, the CPU of all are maxed out. Looking at netstat I can see that the python has a growing amount of CLOSE_WAIT sockets and wont close. Once the scripts are stopped and restarted they then close. Is there a way to close these without restarting the scripts?

For some further information, all 3 instances of parcel are running on a different set of ports, and all of them seem to face this issue.
Might also be related to: https://github.com/LabAdvComp/parcel/issues/91


Linker errors for 32-bit msvc builds

I've run into linker errors trying to use os_info::get() in 32-bit Windows (MSVC) builds. I wrote a minimal example that demonstrates the failure, and included an appveyor project so you can see the full output with the error.
Here's the minimal example code, and here's the Appveyor output.
Hope this is helpful!

For a bit more context, the linker error is failing to find __imp__RtlGetVersion. I'm a little bit baffled a) at the name mangling and b) that it can't find RtlGetVersion since the link command includes ntdll.lib, which is supposedly the library that contains that symbol. But I'm not an expert at Windows linking so I'm sort of stretching my limitations here. :)

The dllimport name mangling is not the issue. The issue is actually that the wrong calling convention is being used.
https://github.com/DarkEld3r/os_info/blob/4f7106f8729e169dfbe08027e9d93a6f083ba831/src/windows/winapi.rs#L35-L38
You want to be using extern "system" for Windows API functions, not extern "C".


Ingress SDS Envoy Seg Fault

Running with ingressgateway-sds on 1.1.1
Logs:
[2019-04-15 21:20:42.212][6042][warning][main] [external/envoy/source/server/server.cc:405] caught SIGTERM
[2019-04-15 21:20:42.262][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:81] Caught Segmentation fault, suspect faulting address 0x10
[2019-04-15 21:20:42.262][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:67] Backtrace (use tools/stack_decode.py to get line numbers):
[2019-04-15 21:20:42.262][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #0: __restore_rt [0x7f5776fdb390]
[2019-04-15 21:20:42.265][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #1: Envoy::Config::GrpcMuxImpl::GrpcMuxWatchImpl::~GrpcMuxWatchImpl() [0x9ebccf]
[2019-04-15 21:20:42.267][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #2: Envoy::Config::GrpcMuxImpl::GrpcMuxWatchImpl::~GrpcMuxWatchImpl() [0x9ebd5e]
[2019-04-15 21:20:42.270][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #3: Envoy::Config::GrpcSubscriptionImpl<>::~GrpcSubscriptionImpl() [0x8b6672]
[2019-04-15 21:20:42.272][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #4: Envoy::Secret::SdsApi::~SdsApi() [0x8b1f82]
[2019-04-15 21:20:42.275][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #5: Envoy::Extensions::TransportSockets::Tls::ContextConfigImpl::~ContextConfigImpl() [0x881339]
[2019-04-15 21:20:42.278][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #6: Envoy::Extensions::TransportSockets::Tls::ServerContextConfigImpl::~ServerContextConfigImpl() [0x88265d]
[2019-04-15 21:20:42.280][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #7: Envoy::Extensions::TransportSockets::Tls::ServerSslSocketFactory::~ServerSslSocketFactory() [0x563e3a]
[2019-04-15 21:20:42.283][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #8: Envoy::Server::FilterChainImpl::~FilterChainImpl() [0x8629af]
[2019-04-15 21:20:42.285][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #9: std::array<>::~array() [0x86060f]
[2019-04-15 21:20:42.288][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #10: __gnu_cxx::new_allocator<>::destroy<>() [0x86047c]
[2019-04-15 21:20:42.290][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #11: std::__detail::_Hashtable_alloc<>::_M_deallocate_node() [0x8603df]
[2019-04-15 21:20:42.293][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #12: std::__detail::_Hashtable_alloc<>::_M_deallocate_node() [0x86033e]
[2019-04-15 21:20:42.296][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #13: std::pair<>::~pair() [0x859e83]
[2019-04-15 21:20:42.298][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #14: Envoy::Server::ListenerImpl::~ListenerImpl() [0x85938c]
[2019-04-15 21:20:42.301][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #15: Envoy::Server::ListenerImpl::~ListenerImpl() [0x859733]
[2019-04-15 21:20:42.303][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #16: Envoy::Server::ListenerManagerImpl::~ListenerManagerImpl() [0x860172]
[2019-04-15 21:20:42.305][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #17: Envoy::Server::ListenerManagerImpl::~ListenerManagerImpl() [0x8601ae]
[2019-04-15 21:20:42.308][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #18: Envoy::Server::InstanceImpl::~InstanceImpl() [0x84c514]
[2019-04-15 21:20:42.311][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #19: Envoy::Server::InstanceImpl::~InstanceImpl() [0x84cb2e]
[2019-04-15 21:20:42.313][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #20: Envoy::MainCommonBase::~MainCommonBase() [0x54aa75]
[2019-04-15 21:20:42.316][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #21: main [0x549c3d]
[2019-04-15 21:20:42.316][6042][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #22: [0x7f5776917830]
2019-04-15T21:20:42.503597Z     warn    Epoch 0 terminated with an error: signal: segmentation fault (core dumped)

@jcetkov

@howardjohn Do you have core dump file to debug?

@JimmyCYJ if you tell me how to get one, I can try to get it

Seg fault is in ~GrpcMuxWatchImpl(), this might not be limited to SDS, but all subscription service.

core dump is in /etc/istio/proxy/core.envoy.*
envoy binary is in /usr/local/bin/envoy
Could you provide both two?

I suspect it is related to SDS in some way. We have much much more testing of ingress without sds than ingressgateway-sds so I think this would have came up before otherwise

For testing of ingress without sds, the subscription service is not used.
Envoy uses subscription service to fetch LDS, CDS, RDS, EDS and SDS. GrpcMuxWatchImpl() is in the subscription service stack. My point is this issue might not be limited to SDS, it may affect other ADS. But I need more information to figure out root cause.

@jcetkov Do you have any idea about reproducing this issue? It seems that the seg fault happened when Envoy is terminated.

@JimmyCYJ I was telling @howardjohn I'm unsure this is the cause or result; I see in one cluster the gateway gets restarted ever ~45s and this is tnly interesting part of the logs

It looks the root cause of this is misbehaving citadel:
I can see the ro mount with the certs get updated every minute
oot@istio-ingressgateway-598b5d6fdc-7lmpl:/# ls -l /etc/certs/..data/
total 12
-rw-r--r-- 1 root root 1188 Apr 16 19:38 cert-chain.pem
-rw-r--r-- 1 root root 1679 Apr 16 19:38 key.pem
-rw-r--r-- 1 root root 1054 Apr 16 19:38 root-cert.pem

while citadel constantly logs
2019-04-16T19:16:54.661666Z    info    Refreshing secret heptio-qm/istio.quartermaster-controller, either the leaf certificate is about to expire or the root certificate is outdated
2019-04-16T19:16:54.862303Z    info    Refreshing secret kube-system/istio.deployment-controller, either the leaf certificate is about to expire or the root certificate is outdated

(hundred times a minute)

The problem of certs getting updated every minute is caused by the race condition of two Citadels.
The recommended Citadel deployment is one replica of Citadel. When the Citadel dies, k8s will automatically restart it based on the Citadel deployment. The certificate TTL is 24 hours and the grace period is 12 hours. So one Citadel, as opposed to two Citadels, is a simpler solution and still provide reasonable robustness.

The issue is closed for now as it is confirmed that scaling down Citadel stopped the ingress gateway restarts.

it might be appropriate to modify the helm charts not to allow setting the security replica count via variable, if such case is really not supported and results in hard to trace issues.

@jcetkov That is a good idea. I will have added your comment to the issue https://github.com/istio/istio/issues/13383.


Fix to make it type safe.

Add types to data and post.
Modify to make type checking pass throughout.

Thank you for your contribution!


adds more standart html tags in snippets

hey, found that there are some html tags that would be nice to have around, i know that there is a "default empty" tag that can be used but i thought having more makes it nicer to get going.
hopefully you find this helpful or can give some feedback.
im only starting to work with templ and im amazed by the tooling around it and its LSP, im super excited!
cheers!

Looks good, but why so few? Do you think we should add more?
The built-in HTML support includes a larger set of tag snippets. I'm not sure if it's contextual, in that it's aware of the parent tag etc.

It seems I just added a placeholder for future thinking in this area. I'll merge for now, and we can revisit later.

Thanks for merging!

Looks good, but why so few? Do you think we should add more?

this where just he ones i thought of immediately when starting to work with it. im working on a PoC of my Org so im keeping the perspective of "a fresh new user without knowledge", which also means that in my setup i dont have additional lsp or plugin for html support, so i look at it "what can / should templ lsp provide" vs should not be.
i work with a fairly small neovim setup and since i already have templ installed i naturally concluded i would use its lsp capability (which i love btw, so cool!)  to write the html part. so from that point i would think that there are tags "missing" but than the question is should the templ lsp part also provide complete html support, im not actually sure, maybe you have a thought about it?
(i can see the argument for having all html tags in a separate lsp for html and therefore maybe providing htmx tags?)

Thanks for the feedback!
Do I understand you correctly in the way that you want to incorporate additional html lsp and css lsp into the templ tooling? would that then mean to add htmx tags into the templ lsp?
I have joined the templ slack so we can discuss this on there when it's more about finding the way forward and not strictly tied to a merged PR.
Thank for your time and input!

Yes, I think it's possible to add more tooling into the VS Code extension or LSP by using the sample that Microsoft provide.
So I don't think there's a need to anything htmx specific to the LSP.

Im not sure i understand clearly what you mean. Do you mean to find a way to incorporate existing html and css lsp to Templ to provide that functionality? Im not sure there is an existing htmx lsp out there in the wild but adding that too would be nice as well. I saw your talk at gopherconUK yesterday and that helped to understand how the lsp works, very insightful!
I would look into that as im working with it and find the need to have something like that working within my nvim setup.

Yes, I think that there's a way to incorporate existing LSPs as per https://code.visualstudio.com/api/language-extensions/embedded-languages
Given that templ is basically a way to combine HTML + Go + CSS + JS, then I was wondering if the same approach I've used to proxy to the Go LSP would work, or whether the samples on that page ^ might be easier.
Glad the Gophercon talk had the desired effect. 😁

Okay, i understand! Thanks for clarifying and taking the time to spell it out for me 😄

Also worth noting there is a htmx LSP under development already at github.com/ThePrimeagen/htmx-lsp


The demo crashes in caikit v0.15

The hugging face demo is broken now and don't work with caikit v0.15 because of the change with RuntimeGRPCServer API change
    with RuntimeGRPCServer( TypeError: __init__() got an unexpected keyword argument 'inference_service'
can you please fix it?

Thanks for reporting this.
It should have been clear that the demo app only works with what is specified in requirements.txt, but looking back at the README I am not seeing any specific call-out of the caikit prereq or other library prereqs.  I think I decided that stating that the requirements are prerequisites was redundant and unnecessarily extra wordy.  Does it make sense to say "You must have the required version of caikit (and the other requirements)" before the instruction to install requirements.txt?  Pinned versions are used.  Maybe a few extra words in the README would help others avoid this problem?
Otherwise caikit 0.x upgrades are always assumed to be breaking changes (as opposed to 0.x.y bumps).  I added a new issue for the upgrade work #23.  That's how we generally assign and close the regular maintenance updates vs "crash" bugs.


Execution failed for task ':app:compileDebugJavaWithJavac'. flutter

it gives an error
['FAILURE: Build failed with an exception.
What went wrong:
Execution failed for task ':app:compileDebugJavaWithJavac'.
Could not read previous compilation

when i do flutter run -v
[  +87 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +2 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.

Could you post the full gradle error? The javac task should provide an error pointing to the faulty source.
Either way, this is probably not caused by drift since it's a pure-Dart package. If it's caused by package:sqlite3_flutter_libs for some reason, I can move and re-open the issue in the relevant repository.

Is that the full output of flutter -v? I'd expect it to print more information about the failed gradle invocation.


firestore-send-email: Warn message "message 'undefined' is not a valid object - please add as an object or firestore map, otherwise you may experience unexpected results." logged when using handlebar templates

[REQUIRED] Step 2: Describe your configuration

Extension name: firestore-send-email
Extension version: 0.1.20
Configuration values (redact info where appropriate):

[REQUIRED] Step 3: Describe the problem
When using the trigger email extension with handlebar templates (and no explicit message field), the following warning message is logged each time an email is scheduled:
message 'undefined' is not a valid object - please add as an object or firestore map, otherwise you may experience unexpected results.

Steps to reproduce:
Schedule emails by adding an email as follows to the outbound queue:
await firestore()
  .collection("mail")
  .add({
    to<EMAIL_ADDRESS>    template: {
      name: "templatename",
      data: {
        some: "Data",
      },
    },
  });

Check the Logs of the ext-firestore-send-email-processQueue Cloud function. They emit the following warning message each time an email is sent:
message 'undefined' is not a valid object - please add as an object or firestore map, otherwise you may experience unexpected results.

This does not make sense as the message field is not required to be there when using handlebar templates.
Expected result
No warning is logged that the message field is undefined when using the trigger email extension with handlebar templates.
Actual result
See above.

HI @sceee
Would it be possible to share the template firestore document you are using for this example?

Hi @dackers86 ,
the used templates are all structured like this:
{
html: "<div>...</div>"
subject: "Some subject"
text: "Some text"
}

These are individual fields in the firestore document. Emails are also sent correctly, it's just the annoying warning in the logs.

just reproduced this. I'll need to look into when exactly we need the message field (e.g whether we can have templates but still need a message) and then implement a fix for the next release. Thanks!

The same issue ...log full of this. Luckily - emails were sent.

Same thing here. E-mails not being sent. Good thing is that I did not upgrade it on prod, only on dev project. Is there a way to downgrade a firebase extension??

@dmvvilela in your case it then seems to be something different, probably you need to open another issue for your problem.
This particular issue was "just" about the false warning when using handlebar templates, emails are still being sent as written above.

Hmm, weird thing is as I updated I tested and it was working fine. Then now nothing happens on email document and show that warning on console..
Did you change those time to live new config in some regard?

I'm with version 0.1.19 (different, that @sceee ), but the same result - warning + emails are sent with success.

That version works, @sceee version also works.. 1.22 stopped sending for me. I sent Firebase a request already.

@cabljac i have the same issue after upgrade to version 0.1.22.
 admin
    .firestore()
    .collection("mails")
    .add({
      to: values.email,
      template: {
        data: {
            phoneNumber: '012345',
        },
        name: "test-template",
      },
      createdAt: admin.firestore.FieldValue.serverTimestamp(),
    }),

The bug was introduced  in this commit https://github.com/firebase/extensions/commit/f4bfb28d74208e2cc6c82fb63b655b76ad7ff9d1 i think.

When new email is created using template property, delivery is not send because message property is not defined.

Same here, version 0.1.22 does not send emails with a template.

Have the same problem, everything worked fine till the .22 version update. now no emails are being sent. unless I'm manually adding the document without a template (hence, the problem isn't in the configuration).

@dmvvilela A new release has been added, please update to the latest version.


Port 443 needs to be set after letsencrypt is enabled

If you map port 443 before enabling letsencrypt, the nginx.conf file will be modified to map to a non-existing tls file.
This will prevent nginx from properly reloading, and crash all non-default apps.
So it's easier to first map port 80, enable letsencrypt, then map port 443.
(we need to map port 80 first, otherwise we can't serve a .well-known)

I solved the problem in another way to reduce the number of instructions:

I explicitly exposed port 80 in the Dockerfile (commit) which creates a PORT environment variable with 80 as value.
This PORT environment variable is used by the Plausible server.
The automatic port retrieval for the Dokku proxy uses the EXPOSE statement from the Dockerfile.

No need to configure the dokku proxy anymore, even for SSL.

Perfect! Thanks


Updating the guide (I didn't know where/how to write this)

Being that Luma has been updated for 11.4, may you please update your guide suggesting to download the latest nightly (which fixes DS and GBA launch) please?

I'm assuming that it says to use stable for well, stability. No point in telling people to use risky code if they don't have to. The guide also says thy you don't have to update to .4, but can rather go back to the version you started the guide with.

It seems like we're going to get a new release build tomorrow, so this issue should resolve itself.

I think this issue can be closed now


Bug in report series when some fields have unknown quantities

May 13 19:10:20 v2202105147787154204 poetry[29931]:   File "/root/.cache/pypoetry/virtualenvs/coinbot-jobTb64O-py3.9/lib/python3.9/site-pack
May 13 19:10:20 v2202105147787154204 poetry[29931]:     handler.handle_update(update, self, check, context)
May 13 19:10:20 v2202105147787154204 poetry[29931]:   File "/root/.cache/pypoetry/virtualenvs/coinbot-jobTb64O-py3.9/lib/python3.9/site-pack
May 13 19:10:20 v2202105147787154204 poetry[29931]:     return self.callback(update, context)
May 13 19:10:20 v2202105147787154204 poetry[29931]:   File "/root/coinbot/coinbot/core.py", line 320, in handle_text_message
May 13 19:10:20 v2202105147787154204 poetry[29931]:     self.extract_and_report_series(update, msg)
May 13 19:10:20 v2202105147787154204 poetry[29931]:   File "/root/coinbot/coinbot/core.py", line 353, in extract_and_report_series
May 13 19:10:20 v2202105147787154204 poetry[29931]:     self.report_series(update, coin_df)
May 13 19:10:20 v2202105147787154204 poetry[29931]:   File "/root/coinbot/coinbot/core.py", line 375, in report_series
May 13 19:10:20 v2202105147787154204 poetry[29931]:     amount = large_int_to_readable(row["Amount"] * 1000)
May 13 19:10:20 v2202105147787154204 poetry[29931]:   File "/root/coinbot/coinbot/utils.py", line 47, in large_int_to_readable
May 13 19:10:20 v2202105147787154204 poetry[29931]:     return readable
May 13 19:10:20 v2202105147787154204 poetry[29931]: UnboundLocalError: local variable 'readable' referenced before assignment

Closed by #21


mm_db?

In this line:
https://github.com/mattermost/mattermost-docker/blob/a46068e3057a8a9fdf392a6faa0598f873bbe461/contrib/swarm/docker-stack-traefik.yml#L74
There's a reference to something called "mm_db", but that's only mentioned in that line and in a comment, I don't understand where this thing is coming from, should it just be instead the name of the database container, db? In which case for whatever reason Mattermost isn't able to resolve this hostname, even though I can curl it correctly from within the container using the "mattermost/mattermost-prod-app" image.
The error I'm getting is:
{"level":"error","ts":1571848204.6521745,"caller":"sqlstore/supplier.go:234","msg":"Failed to ping DB","error":"default addr for network 'notable-mattermost-db:5432' unknown","retrying in seconds":10}

In which case for whatever reason Mattermost isn't able to resolve this hostname

Deleting existing Mattermost data/configs fixed this for some reason.

I still get the same error


Vulkan UniformConstant variables only support opaque types

Variables identified with the UniformConstant storage class are used only as handles to refer to opaque resources. Such variables must be typed as OpTypeImage, OpTypeSampler, OpTypeSampledImage, or an array of one of these types.

Glslang tests fail validation with this code.  I think they should not fail (but check)

RuntimeArray must be permitted  (Glslang test spv.nonuniform.frag)
OpTypeAccelerationStructureNV is opaque (from http://htmlpreview.github.io/?https://github.com/KhronosGroup/SPIRV-Registry/blob/master/extensions/NV/SPV_NV_ray_tracing.html )

See https://github.com/KhronosGroup/SPIRV-Tools/pull/2054 for a patch for those two issues.

With #2054 merged, is there anymore work that is needed on this issue?


Inconsistency in buttons

@tobifroe as discussed in our call, the buttons currently are inconsistent in the different plugins. They should all behave like the button in the key features section on the landing page.

@Truthans This should be fixed now.

Perfect, thank!


:b works for buffer, but not :b1 with number

In vim you can use :b1 to go to buff number 1 .. with any number. This does not work in pyvim.

Thanks for reporting.
Normally :b 1 (with a space in between) should already work. I was not aware that it was possible without. I'll fix that as well.
By the way, feel free to keep reporting missing key bindings or commands. Vim has a lot, and implementing all of them is not feasible in the short term. But things that are reported will probably get priority.

Sure, I'll try to use it tomorrow at work and report what I can find.

I made "vi" an alias for "pyvim" now. That forced me to use it all day. :)
Actually, right now, I'm doing all pyvim development in pyvim itself.
That way, every time I need a missing feature, I want to implement it asap.

Yes, I completely agree, this is the best way. I'll try to contribute something during the weekend instead of issues alone :)

I think we should rename :bw to :bd ... from vim's help buffer
:[N]bd[elete][!]			*:bd* *:bdel* *:bdelete* *E516*
:bd[elete][!] [N]
		Unload buffer [N] (default: current buffer) and delete it from
		the buffer list.  If the buffer was changed, this fails,
		unless when [!] is specified, in which case changes are lost.
		The file remains unaffected.  Any windows for this buffer are
		closed.  If buffer [N] is the current buffer, another buffer
		will be displayed instead.  This is the most recent entry in
		the jump list that points into a loaded buffer.
		Actually, the buffer isn't completely deleted, it is removed
		from the buffer list |unlisted-buffer| and option values,
		variables and mappings/abbreviations for the buffer are
		cleared.

:[N]bw[ipeout][!]			*:bw* *:bwipe* *:bwipeout* *E517*
:bw[ipeout][!] {bufname}
:N,Mbw[ipeout][!]
:bw[ipeout][!] N1 N2 ...
		Like |:bdelete|, but really delete the buffer.  Everything
		related to the buffer is lost.  All marks in this buffer
		become invalid, option settings are lost, etc.  Don't use this
		unless you know what you are doing.

I tried :b N in vim, and it doesn't work at all, I guess it never worked and always was without space :bN

I added alias to :bw also work with :bd, there is difference between these in vim, but I'm not sure what to make of the explanation from the help page and how it is supposed to work.
https://github.com/jonathanslenders/pyvim/pull/44

I'm trying to make this work, but from what I've discovered, there would need to be dynamic adding of commands 'b1', 'b2' to the COMMANDS_TO_HANDLERS when adding/deleting buffers. These would need to call function bringing up specific buffer. But what I think would be better is if the command could contain regexp. That way, you could just write into the decorator this:
@cmd('b(%d+)')
def buffer_switch(regexp_groups):
    ...

https://github.com/jonathanslenders/pyvim/pull/50

The change in the pull request works OK, but I'd like to see which buffer I'm switching to since the numbering is dynamic (when you close buffer, others after it get its number). I'd like to add the completer so it would display which buffer are you switching to.
Example:
You'd write :b1 and this would display either No matches, or what is under buffer number 1. But this cannot happen sooner (right after you write :b) this would disable shortcuts :bn :bp and maybe others.
I'll try to figure out how the completer works and try to add this.


Feature/filter catalogue by api owner

Closes #997
Proposed changes

remove unnecessary client console logs from chart.js ( was left when we worked on datatables task )
add 'getApiManagersByName' collection helper for apiBackend

@elnzv I will take cleanup away from this, I made cleanup PR for that.


GRPC failure in istio-pilot-e2e affecting automated tests

https://k8s-gubernator.appspot.com/build/istio-prow/pull/istio_istio/3212/istio-pilot-e2e/2434/
I0206 07:29:33.213] 2018-02-06T07:29:33.168258Z	info	kubectl exec t-67bdf84bfd-lsnp4 --kubeconfig pilot/pkg/kube/config -n istio-test-app-5mlzd -c app -- client -url grpc://d:7070 -count 1
I0206 07:29:35.336] 2018-02-06T07:29:35.336304Z	info	2018/02/06 07:29:35 [0] grpcecho.Echo(message:"request #0" )
I0206 07:29:35.337] 2018/02/06 07:29:35 Error rpc error: code = Unavailable desc = all SubConns are in TransientFailure
I0206 07:29:35.337] command terminated with exit code 1
I0206 07:29:35.338]
I0206 07:29:35.339] 2018-02-06T07:29:35.336370Z	error	client request error command "kubectl exec t-67bdf84bfd-lsnp4 --kubeconfig pilot/pkg/kube/config -n istio-test-app-5mlzd -c app -- client -url grpc://d.istio-test-app-5mlzd:7070 -count 1 " failed: "2018/02/06 07:29:35 [0] grpcecho.Echo(message:"request #0" )\n2018/02/06 07:29:35 Error rpc error: code = Unavailable desc = all SubConns are in TransientFailure\ncommand terminated with exit code 1\n" exit status 1 for grpc://d.istio-test-app-5mlzd:7070 in t
I0206 07:29:38.456] 2018-02-06T07:29:38.456109Z	info	2018/02/06 07:29:37 [0] grpcecho.Echo(message:"request #0" )
I0206 07:29:38.456] 2018/02/06 07:29:37 Error rpc error: code = Unavailable desc = transport is closing
I0206 07:29:38.457] command terminated with exit code 1

I kept hitting the same error in my PR
https://k8s-gubernator.appspot.com/build/istio-prow/pull/istio_istio/3214/istio-pilot-e2e/3188/
Is it disabled?

@nmittler Not sure if this is still failing frequently. Can you update the status of this issue?

I don't think we're seeing this anymore.  Closing for now.


Make file inherit from blob

Bug Report
Problem
console.assert(file instanceof Blob)

should not show a error
What is expected to happen?
File should inherit from Blob and not show a error
What does actually happen?
it logs a error, file is not a instance of Blob
Information
https://github.com/webtorrent/create-torrent/pull/89
Checklist

[x] I searched for existing GitHub issues
[ ] I updated all Cordova tooling to most recent version
[ ] I included all the necessary information above

I'm not a cordova developer but i tried it out a long time ago... it's not really my issue but someone else

The best thing would be if you could extend it...
window.File = class File extends window.File {...}


case sensitivity

Example: In the case that a user has his/hers twitter handle in another case than what is entered in WikiData > Identities > Twitter username the api won't return a match, resulting in that a badge is not visible.
Examples

Twitter username
Wikidata info
match

Landgren
Landgren
:white_check_mark:

BillGates
BillGates
:white_check_mark:

britstakston
Britstakston
:x:

I do not think any code should be updated, to try to solve this, but maybe add some additional info in the Chrome Web Store page, readme and other means where it could be communicated.

Thanks, this will not be solved on our end- when casing in wikipedia/wikidata is wrong- it's more important to update Wikipedia to be correct and this badge can help finding these. There are some known issues with Twitter and casing. For example- my previous twitter username was @landgren which is indistinuishable in some fonts from @Iandgren (with capital I).


Sync upstream TagUI change on accepting www.domain.com as valid [done]

https://github.com/kelaberetiv/TagUI/issues/1046
In this change, if https:// or http:// prefix is not provided, it will be appended as https:// instead of throwing error.
However, at least a www. prefix is required if there is no https:// or http:// prefix, to be accepted as valid URL.

This is added in v1.38, available with pip install rpa --upgrade


What should I do if my player is building outside of this 46000 block

The plugin's stronghold locating breaks when playing at an x or z coord that is roughly over 46,000 or below -46,000
game version 1.17.1
plugin version 9.1.3

I think this was addressed in discord and resolved a month ago. For those that are checking here, essentially there's nothing that can really be done. Play closer to the 0,0 coord.


Include md output: teams app until teams membersettings commands

Related to #3754

Update the documentation to include the md output for the commands under the service:

teams documentation location

This in place for the following command groups:

teams app
teams cache
teams channel
teams chat
teams funsettings
teams guestsettings
teams membersettings

You can reference the commands planner plan and planner bucket. They are already contain the Markdown reponse in their documentation.
For more information what guidelines to follow, be sure to check out the epic issue #3754

I can attempt to work on it.

Can I work on it?

Can I work on it?

Certainly, thank you!


Don't include default_locale in list of fallbacks

Hi everyone,
Is it currently possible to not include the default locale in the list of fallbacks? I'm working on a Rails app and right now I have this in my app config:
config.i18n.fallbacks = {'en-CA' => 'en',
                         'en-US' => 'en',
                         'fr-CA' => 'fr',
                         'ja-JP' => 'ja'}
config.default_locale = 'en-CA'

But the the fallback for 'fr-CA' is actually:
[
    [0] :"fr-CA",
    [1] :fr,
    [2] :"en-CA",
    [3] :en
]

I don't want en-CA or en to be there, because I want i18n to raise a MissingTranslationData exception so that I know where I have translations missing (actionview wraps these in a .translation-missing span which makes it easy to find the missing translations).
Is this currently possible? Or would it require a custom implementation of the Fallbacks class?

@kiramclean a possible solution is to set the default_locale to a dummy/empty locale:
config.i18n.default_locale = :empty

Then create the empty locale file (config/locales/empty.yml):
# This is purposefully empty so that this can be the final fallback locale. Since it's empty, anything hitting this locale would raise a MissingTranslationData exception.
empty:

I tried this and still found that I got an english translation instead of a missing translation exception.

@ngan The problem is I don't actually want the default_locale to be a dummy locale.. if I'm missing a translation in en-CA or en-US, for example, I still want the app to fall back to the actual default I set for those locales, but if I'm missing a translation in, say fr-CA, under no circumstances do I want the fallback to be en.

$  cat config/locales/dummy.yml
dummy:

Then set fallbacks to true (instead of what you have)
config.i18n.fallbacks = true
config.i18n.default_locale = :dummy

Then either, specify all your available_locales:
config.i18n.available_locales = [:dummy, :en, :"en-CA", ...]

or just set config.i18n.enforce_available_locales = false.
Then the fallback logic will be automatic:
>> I18n.fallbacks[:"fr-CA"]
=> [:"fr-CA", :fr, :dummy]

>> I18n.fallbacks[:"en-CA"]
=> [:"en-CA", :en, :dummy]

>> I18n.t(:foo)
=> "translation missing: dummy.foo"

Just a tidbit:
>> I18n.fallbacks[:"en-CA-X1-X2-X3"]
=> [:"en-CA-X1-X2-X3", :"en-CA-X1-X2", :"en-CA-X1", :"en-CA", :en, :dummy]

Thanks everyone. I think @ngan's workaround is great. If someone wants to submit a PR to fix this, then I'd be open to that.


bug in 's _addRefs function

to reproduce:
when there is only one child inside <Form>, an error will come up in the console
        <Form action="/" onSubmit={(submitObject) => {console.log(submitObject)}}>

              <TextField name="firstName" style={{width: "200px"}} label="First Name: " required/>
              <TextField name="lastName" style={{width: "200px"}} label="Last Name: " required/>

            <Button primary type="submit">Submit</Button>

        </Form>

having at least two children will solve this issue
        <Form action="/" onSubmit={(submitObject) => {console.log(submitObject)}}>

              <TextField name="firstName" style={{width: "200px"}} label="First Name: " required/>
              <TextField name="lastName" style={{width: "200px"}} label="Last Name: " required/>

            <Button primary type="submit">Submit</Button>

        </Form>

This has been fixed in the Form PR #953. Thanks for testing.


Question: How to properly type mutation Variables?

What would be the correct way to tell typescript, that Variables is the interface for the arguments of myMutation function?
    interface Variables {
      uuid: string;
      value: string;
    }

    const { mutate: myMutation } = useMutation(myGqlMutation);

I want to avoid using myMutation like this:
 myMutation({
     uuid: '....',
     value: '....',
 } as any); // get rid of any here ...

This is a copy of: https://stackoverflow.com/questions/63355461/mutation-typings-in-vue-apollo-v4-and-composition-api

import LOGIN from './login.graphql'
import {
  LoginMutation,
  LoginMutationVariables
} from '@/generated/graphql-types'

const { mutate: loginMutate, loading: loginLoading } = useMutation<
    LoginMutation,
    LoginMutationVariables
  >(LOGIN)

const result = await loginMutate({})
//   when I try to call the mutation with incorrect variables, I get next error
//   Type '{}' is missing the following properties from type 'Exact<{ username: string; password: string; }>': username, password

Graphql code gen is used to generate muation result + variables.

So, basically you need to pass two generic arguments to 'useMutation' function: mutation result and mutation variables


Add "WITypegroup" URL in space links

Added List WITG URL in space response.
Renamed "Execution" to "Iterations" for display only.
Tests are updated.

@pranavgore09 snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1580-1

Codecov Report

Merging #1580 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1580      +/-   ##
==========================================
+ Coverage   55.94%   55.95%   +<.01%
==========================================
  Files         127      127
  Lines       15513    15515       +2
==========================================
+ Hits         8679     8681       +2
  Misses       6188     6188
  Partials      646      646

Impacted Files
Coverage Δ

controller/space.go
77.53% <100%> (+0.12%)
:arrow_up:

Continue to review full report at Codecov.

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

@pranavgore09 snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1580-2

[test]


video stream error

router.get('/mp4/:name', async ctx => {
    let file = './video/' + ctx.params.name;
    let stat = fs.statSync(file);
    let fileSize = stat.size;
    let range = ctx.header.range;
    if (range) {
        let parts = range.replace(/bytes=/, "").split("-");
        let start = parseInt(parts[0], 10);
        let end = parts[1] ? parseInt(parts[1], 10) : fileSize - 1;
        let chunksize = (end - start) + 1;
        ctx.status = 206;
        ctx.set('Content-Range', `bytes ${start}-${end}/${fileSize}`);
        ctx.set('Accept-Ranges', 'bytes');
        ctx.set('Content-Length', chunksize);
        ctx.set('Content-Type', 'video/mp4');
        ctx.body = fs.createReadStream(file, { start, end });
    } else {
        ctx.status = 200;
        ctx.set('Content-Length', fileSize);
        ctx.set('Content-Type', 'video/mp4');
        ctx.body = fs.createReadStream(file);
    }
});

I used koa to process the video stream and got an unhandled error. The console always outputs this error whenever my video fast forwards.
  Error: read ECONNRESET
      at TCP.onStreamRead (internal/stream_base_commons.js:111:27)

  Error: write ECANCELED
      at WriteWrap.afterWrite [as oncomplete] (net.js:779:14)

  Error: read ECONNRESET
      at TCP.onStreamRead (internal/stream_base_commons.js:111:27)

Error: read ECONNRESET
      at TCP.onStreamRead (internal/stream_base_commons.js:111:27)

This error is always output when the video is fast forwarded. Express does not have this error. I want to know how to solve it.
I have been looking for a solution, but still can't solve it, thanks.

@jonathanong @dead-horse @igoradamenko

@re1oaded, hi! As I understand these error are OK, because connection is being resetted when you fast forwarding video on client. And that's the reason why your pipes broke at that moment.
Try to check #1089 or https://github.com/koajs/koa/wiki/Error-Handling, looks like they have a solution (global error handling). It's just an error suppressing, but I'm not sure that you need any other handling here.

@igoradamenko Thanks, Igor.
Do you mean that these errors will not affect the video playback?
I don't need any other handling for errors unless it affects the normal playback of the video.
Btw, is there any problem with my code?
Opening the video address directly in Firefox cannot be fast forwarded, Chrome is ok.
e.g: http://<IP_ADDRESS>/mp4/video.mp4

@re1oaded, yes, I think it's a mistake in your code.
According to MDN browser should send HTTP range request (with “Range” header) only when it get “Accept-Ranges” header from the server: https://developer.mozilla.org/en-US/docs/Web/HTTP/Range_requests.
As I understand, Chrome always sends “Range” header for video, and you reply to it with stream and “Accept-Ranges” header. But Firefox sends “Range” only when it already got “Accept-Ranges” from the server. So, in Chrome you see correctly working video fast forwarding, but in Firefox it works in a strage way: it loads a chunk in an internal FF buffer, then pauses loading, then when you want to fast forward it loads next chunk. If it's not enough to show you requested position of the video, FF doesn't care and you need to click on progress bar again and again, to force FF to load more chunks.
So, yeah, it doesn't work correctly in FF. The reason is quite simple — due to this chunk-loading algorithm FF can't get “Accept-Ranges” from your server because you send it only when you get “Range” from the browser.
Fix is quite simple — just always set “Accept-Ranges” for videos. I tested it and it works fine in FF now.
Hope it helps!

Oh, and,

Do you mean that these errors will not affect the video playback?

I think it doesn't affect the playback at all, but I don't have enough experience with these things. However I don't see any errors in this behaviour, it sounds logical.

Thanks igor.
In this case, it looks like you can close the issue, doesn't it?


nixos/kubernetes: set k8 home permissions correctly

Description of changes
default home directory permissions are 700 this causes issues for k8s when nix gets rebuilt, but not rebooted, as the permissions for the kubernetes folder should be set to 755 to allow other services (such as etcd) access to the secrets folder.
Permissions are already set to 755 via systemd tmpfiles, so the issue only occurs when nixos is rebuilt and not rebooted (such as via the autoupgrade service).
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

cc @saschagrunert @johanot @offlinehacker @srhb

@srhb or @kirillrdy could either of you merge this? the contributing doc doesn't really state who/how that is supposed to happen :)
I think this one should likely be backported as well

I think this one should likely be backported as well

To backport, add the backport label.

@superherointj I don't have permissions to add labels, but thank you for adding that for me!

@superherointj I don't have permissions to add labels, but thank you for adding that for me!

What a shame!
If you happen to add yourself to:
https://github.com/NixOS/nixpkgs/blob/master/maintainers/maintainer-list.nix
You might be bestowed new super powers.

the contributing doc doesn't really state who/how that is supposed to happen :)

Pinging someone with merge rights and a history of being """involved""", especially once there are approvals is just great, just as you did. :ok_hand:


Adding Clone Project Locally Functionality

This code includes the creation of the Clone Locally menu item button to trigger the flow. It also includes changes to code that trigger the download flow from the VSCode Functions Extension. Finally, it includes docker support files to detect if user has docker and to have a docker prompt.
ADDED FILES:
src/commands/dockersupport/askDockerStep.ts - Prompt pop up UI at the top of VS Code that asks if the user would like to use Docker with their project with yes/no options only. Returns string

src/commands/dockersupport/localDockerSupport.ts - function that checks if certain conditions are met to open the App Function locally with Docker. If Docker is not installed, it points users to the documentation. Returns boolean

src/commands/dockersupport/validateDockerInstalled.ts - checks the user's system if Docker is installed by running a command in the command line. Returns boolean
src/downloadAzureproject/cloneProjectLocally.ts - The function call of the new menu item button "Clone Project Locally…" to trigger the whole download flow
EDITED FILES:
src/tree/SlotTreeItemBase.ts - added a function called getApplicationLanguage() which goes through the app function settings and gets its runtime. Returns string
package.json - Edited to add menu item placement and to update version of extract
package.nls.json - Edited to add menu item, specifically to add the words that go on the button

src/commands/registerCommands.ts - Edited to add menu item, specifically to relate the button to a function call
src/downloadAzureproject/setUpProjectFolder.ts - Edited to handle calls from the extension side and to include call for docker prompt

Closing due to lack of activity.


Update users-default-permissions.md

Adding explicit callout that this will not block users from getting data from Microsoft Graph API.

@barclayn

Can you review this PR?

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


overflow hidden github message on website

Could we just make this the commit SHA ?

text-overflow: ellipsis would work great :)
@oliverbenns are you interested in PR'ing the fix?

Yep will do @samccone when I get a bit of time :)


How to mock value returned by BlobClient.UploadAsync

Query/Question
I wrote as class that is injected a BlobClient dependency and I am writing unit tests to ensure my class behaves as it should. In one instance, my class invokes the UploadAsync and therefore I want to mock the invocation of this method in order to test several scenarios (such as: upload successful, upload failed due to missing blob, etc.).
The UploadAsync method returns an object of type Response<BlobContentInfo> but the properties on the BlobContentInfo class are read-only and furthermore, the class does not have a parameter-less constructor. This means that I can't see a way to mock the value returned by this method.
Code sample
Here's a simplified version of my class:
public class Foo
    private readonly BlobClient _blobClient;

    public Foo(BlobClient blobClient)
    {
        _blobClient = blobClient;
    }

    public async Task Bar(CancellationToken cancellationToken)
    {
        var response = await _blobClient.Upload(...).ConfigureAwait(false);
        ... snip ...
    }
}

Here's an example of a unit test:
[Fact]
public async Task MyUnitTest()
{
    // Arrange
    var cancellationToken = CancellationToken.None;

    var mockBlob = new Mock<BlobClient>(MockBehavior.Strict, new Uri(BLOB_ITEM_URL), (BlobClientOptions)null);
    mockBlob
        .Setup(c => c.UploadAsync(It.IsAny<Stream>(), It.IsAny<BlobHttpHeaders>(), It.IsAny<IDictionary<string, string>>(), It.IsAny<BlobRequestConditions>(), It.IsAny<IProgress<long>>(), It.IsAny<AccessTier>(), It.IsAny<StorageTransferOptions>(), cancellationToken))
        .ReturnsAsync(Response.FromValue(new BlobContentInfo()))
        .Verifiable();

    var foo = new Foo(mockBlob.Object);

    // Act
    await foo.Bar(cancellationToken).ConfigureAwait(false);

    // Assert
    mockBlob.Verify();
}

This unit test does not compile. Visual Studio complains that CS1729 'BlobContentInfo' does not contain a constructor that takes 0 arguments.
Why is this not a Bug or a feature Request?
Maybe there is an alternative that I am not aware of?
Setup (please complete the following information if applicable):

OS: Windows 10
IDE : Visual Studio 2019
Version of the Library used: 12.1

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

There is a factory method available for mocking purposes: https://docs.microsoft.com/en-us/dotnet/api/azure.storage.blobs.models.blobsmodelfactory.blobcontentinfo?view=azure-dotnet

Exactly what I needed! Thanks for the tip.

Any news about Mocking BlobServiceClient?
Thanks


$theme-dark behavior under  components

Current Behavior
<YStack h={100} w={100}>
  <Circle
    h={30}
    w={30}
    backgroundColor={'red'}
    $theme-dark={{ backgroundColor: 'green' }}
  ></Circle>
  <Theme name="dark">
    <Circle
      h={30}
      w={30}
      backgroundColor={'red'}
      $theme-dark={{ backgroundColor: 'green' }}
    ></Circle>
  </Theme>
</YStack>

The above code produces two red circles
Expected Behavior
As mentioned in discord I'm not sure if this is intentional or not, but I'd expect the 2nd circle to be green since its under the "dark" theme.
The produces CSS looks like this:
:root.t_dark ._bg-_themedark_green {
    background-color: green;
}

So removing the :root prefix would make it behave as expected, but I'm not sure what the side-effects would be.
Tamagui Version
latest

Platform (Web, iOS, Android)
Web

Reproduction
See code above

System Info
No response

Can you re-open if still an issue, we did a variety of improvements here that should have fixed


2.4.1 update apparently needs manual reference to zxing.net.mobile in android projects

Hello,
After upgrading zxing.net.mobile.forms to 2.4.1, i got error in the android projects that the namespace android was not found in ZXing.Net.Mobile. I had to manually include the zxing.net.mobile package in the android projects for it to work again. This was not neccesary in earlier versions of the forms library. It automatically had a dependency to the zxing.net.mobile package.

Nope, adding the manual reference to ZXing.Net.Mobile did the trick for me. Does the package reference list any dlls under it in the android project?
Get Outlook for Androidhttps://aka.ms/ghei36

From: sushibear<EMAIL_ADDRESS>Sent: Thursday, April 5, 2018 1:32:20 PM
To: Redth/ZXing.Net.Mobile
Cc: PaulVrugt; Author
Subject: Re: [Redth/ZXing.Net.Mobile] 2.4.1 update apparently needs manual reference to zxing.net.mobile in android projects (#701)
I tried that too but no success. Uninstalled ZXing.Net.Mobile and Zwing.Net.Mobile.Forms then readded both to the Android project. Nothing. An other idea what might be wrong here?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Redth/ZXing.Net.Mobile/issues/701#issuecomment-378905574, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHN3CstdeQkJjCPNUrOYjaIKuwvM0Jwuks5tlgDEgaJpZM4TIAPT.

I added one dll (ZXing.Net.Mobile.Forms.Android) to my references and that worked. But I can not access the namespace ZXing.Net.Mobile.Android.
It works flawless with Version 2.3.1 btw. so I did a roll back. :(

Thanks for reporting this issue!  Unforunately it took me way too long to respond 😭.  Sorry, I apologize!  Recently the source code for this project was completely refactored to modernize it.  Many PR's were included in this effort, and many bugs were hopefully fixed.  Please try out the latest 3.x series of NuGet packages (currently in prerelease).  To try and make the project more maintainable in my spare time going forward, I've decided to close all existing issues to start with a clean slate.  If you're still experiencing this issue on the newest version, please open a new issue with as much detail as possible.  Thank you for your patience and understanding!  Happy scanning!


Fix docs formatting

:tada: This PR is included in version 2.29.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


v0.2.1 docs

Coverage remained the same at 80.933% when pulling cbca0e817a78b16e459c70862ef65c15ebc54e09 on mparke:bump-docs into 073414d853d0f5e5f6d65c44aaa9d330a479c032 on linode:master.


Error: Invalid or corrupt jarfile

I have tried yarn install using node 18 and 20 with the following dependencies on mac and unix
{
  "private": true,
  "devDependencies": {
    "@jnxplus/nx-maven": "0.6.4",
    "@jnxplus/nx-boot-maven": "7.2.3",
    "@nx/workspace": "16.5.3",
    "nx": "16.5.3"
  }
}

I get the following error:
error /Users/jbadeau/git/techx/node_modules/@jnxplus/nx-boot-maven: Command failed.
Exit code: 1
Command: node src/scripts/install-linters.js
Arguments:
Directory: /Users/jbadeau/git/techx/node_modules/@jnxplus/nx-boot-maven
Output:
Error: Invalid or corrupt jarfile /Users/jbadeau/git/techx/node_modules/@jnxplus/tools/linters/ktlint/ktlint
node:internal/errors:867
  const err = new Error(message);
              ^

Error: Command failed: java -jar /Users/jbadeau/git/techx/node_modules/@jnxplus/tools/linters/ktlint/ktlint --version
Error: Invalid or corrupt jarfile /Users/jbadeau/git/techx/node_modules/@jnxplus/tools/linters/ktlint/ktlint

    at checkExecSyncError (node:child_process:885:11)
    at execSync (node:child_process:957:15)
    at isAnotherVersion (/Users/jbadeau/git/techx/node_modules/@jnxplus/common/src/lib/utils/index.js:94:53)
    at /Users/jbadeau/git/techx/node_modules/@jnxplus/common/src/lib/utils/index.js:85:18
    at Generator.next (<anonymous>)
    at /Users/jbadeau/git/techx/node_modules/@jnxplus/common/node_modules/tslib/tslib.js:167:75
    at new Promise (<anonymous>)
    at Object.__awaiter (/Users/jbadeau/git/techx/node_modules/@jnxplus/common/node_modules/tslib/tslib.js:163:16)
    at getKtlintPath (/Users/jbadeau/git/techx/node_modules/@jnxplus/common/src/lib/utils/index.js:67:20) {

I have checked the dir and the jar is not downloaded. Works when the @jnxplus/nx-boot-maven plugin is removed but we use both.

Basically both plugins work alone but not together

Yes we can't use both plugins in the same time. Also we can't add both to to NX.json file.
Nx-maven is still in active development. I want to change how build and serve executor works. I want to add command option so we can know which command we are using with maven. I will remove framework option too. You can check pull request to know more.
Nx-maven is the future but need to fix all mismatch between frameworks.

@jbadeau I will close this as it's not a bug. Nx-maven is more a replacement for nx-boot-maven in the future.

@jbadeau Now Nx-maven is a replacement for nx-boot-maven. Working to update docs


Toyota: log EPS fault from LKA cmd message drop out

The LKA_STATE code for the EPS failing to receive LKA msgs is 3, and on rising edge of the fault it goes to 9 for ~1.2 seconds, then 11 for 0.8 seconds, then rests at 3 (recoverable if EPS gets steering msgs again).
If openpilot were ever to stop sending LKA messages and the EPS status was at 3, openpilot would enable without any steering assist or warnings (not even from car).
f15e3c37c118e841|2022-12-02--23-47-57

Running on the fleet, it's catching valid faults!


Fix snippets inaccessible & unchangeable

1. Why is this change necessary?

The current loading skeleton for snippets is unnecessary because we can just use the loading state of the card
The main isLoading only covers the loading of the main snippet information, but not the loading of the individual snippets

While the main information of the snippet is already loaded, the user can interact with the inputs even when they are still loading

If you try to edit a snippet multiple times without hard refreshing the page, the input will jump back to the previous input and you will not be able to save the new value correctly (because the old snippet state is not correctly cleared beforehand, causing the UI to use the outdated values)

2. What does this change do, exactly?

Changed prepareContent in sw-settings-snippet-detail/index.js to set isLoadingSnippets
Changed onSave in sw-settings-snippet-detail/index.js to correctly reset the snippet data
Changed sw-settings-snippet-detail/sw-settings-snippet-detail.html.twig to remove unnecessary loading skeletons & add isLoadingSnippets check to the snippet inputs to block interaction while loading

3. Describe each step to reproduce the issue or behaviour.

Edit a snippet
Save it
Try to edit the same snippet again

4. Please link to the relevant issues (if any).

Fixes #5371
Fixes #5365

5. Checklist

[X] I have rebased my changes to remove merge conflicts
[X] I have written tests and verified that they fail without my change
[X] I have created a changelog file with all necessary information about my changes
[X] I have written or adjusted the documentation according to my changes
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfill them.

Hey @akf-bw, your PR just got merged. Thank you for the contribution 💙


The disposed engine needs to be the thinengine

Related to a previous PR

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15059/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15059/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15059/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15059/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15059/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15059/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15059/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15059/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15059/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15059/merge/testResults/webgpuplaywright/index.html

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15059/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15059/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15059/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15059/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15059/merge/testResults/webgpuplaywright/index.html


NameError: global name 'FileNotFoundError' is not defined

safety version: 1.0.0
Python version: 2.7.13
Operating System: macOS 10.12.3

Description
Installed fresh safety and run safety check.
Got NameError: global name 'FileNotFoundError' is not defined error.
What I Did
(hic) ➜  HiConversion pip install -U safety
Collecting safety
  Downloading safety-1.0.0-py2.py3-none-any.whl
Requirement already up-to-date: packaging in /Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages (from safety)
Collecting requests (from safety)
  Using cached requests-2.13.0-py2.py3-none-any.whl
Collecting setuptools>=16 (from safety)
  Using cached setuptools-34.3.2-py2.py3-none-any.whl
Requirement already up-to-date: Click>=6.0 in /Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages (from safety)
Requirement already up-to-date: six in /Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages (from packaging->safety)
Collecting pyparsing (from packaging->safety)
  Using cached pyparsing-2.2.0-py2.py3-none-any.whl
Collecting appdirs>=1.4.0 (from setuptools>=16->safety)
  Using cached appdirs-1.4.3-py2.py3-none-any.whl
Installing collected packages: requests, appdirs, setuptools, safety, pyparsing
  Found existing installation: requests 2.9.1
    Uninstalling requests-2.9.1:
      Successfully uninstalled requests-2.9.1
  Found existing installation: appdirs 1.4.0
    Uninstalling appdirs-1.4.0:
      Successfully uninstalled appdirs-1.4.0
  Found existing installation: setuptools 34.2.0
    Uninstalling setuptools-34.2.0:
      Successfully uninstalled setuptools-34.2.0
  Found existing installation: safety 0.6.0
    Uninstalling safety-0.6.0:
      Successfully uninstalled safety-0.6.0
  Found existing installation: pyparsing 2.1.10
    Uninstalling pyparsing-2.1.10:
      Successfully uninstalled pyparsing-2.1.10
Successfully installed appdirs-1.4.3 pyparsing-2.2.0 requests-2.13.0 safety-1.0.0 setuptools-34.3.2
(hic) ➜  HiConversion safety check
class _(tuple):
    '_(rows,)'

    __slots__ = ()

    _fields = ('rows',)

    def __new__(_cls, rows,):
        'Create new instance of _(rows,)'
        return _tuple.__new__(_cls, (rows,))

    @classmethod
    def _make(cls, iterable, new=tuple.__new__, len=len):
        'Make a new _ object from a sequence or iterable'
        result = new(cls, iterable)
        if len(result) != 1:
            raise TypeError('Expected 1 arguments, got %d' % len(result))
        return result

    def __repr__(self):
        'Return a nicely formatted representation string'
        return '_(rows=%r)' % self

    def _asdict(self):
        'Return a new OrderedDict which maps field names to their values'
        return OrderedDict(zip(self._fields, self))

    def _replace(_self, **kwds):
        'Return a new _ object replacing specified fields with new values'
        result = _self._make(map(kwds.pop, ('rows',), _self))
        if kwds:
            raise ValueError('Got unexpected field names: %r' % kwds.keys())
        return result

    def __getnewargs__(self):
        'Return self as a plain tuple.  Used by copy and pickle.'
        return tuple(self)

    __dict__ = _property(_asdict)

    def __getstate__(self):
        'Exclude the OrderedDict from pickling'
        pass

    rows = _property(_itemgetter(0), doc='Alias for field number 0')

Traceback (most recent call last):
  File "/Users/kulapard/virtualenvs/hic/bin/safety", line 11, in <module>
    sys.exit(cli())
  File "/Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages/safety/cli.py", line 41, in check
    click.secho(report(vulns=vulns, full=full_report))
  File "/Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages/safety/formatter.py", line 127, in report
    size = get_terminal_size()
  File "/Users/kulapard/virtualenvs/hic/lib/python2.7/site-packages/safety/formatter.py", line 14, in get_terminal_size
    except (ValueError, FileNotFoundError):
NameError: global name 'FileNotFoundError' is not defined

Thanks for reporting, fixed in https://github.com/pyupio/safety/commit/10d606090eb13d373b8d16142284c302dd76780f and https://github.com/pyupio/safety/commit/ed15d7f100dcc3b7c62b4546e3f9e6ca49742c68


Correct typo in LCD commands, and minor README changes

thanks.. my poll english. so some character is error...


Remove Python runtime.txt to use default fallback

Just for checking if it works...

We've fixed by configuring Netlify to use the latest Xenial image for builds, so I think we can safely close.


How to detect incoming call hang up

ITNOA
Hi, I see Communication blocking and filtering sample but I could not find anyway to detect when incoming call hang up, I see PhoneTriggerType enumeration but some description is very unclear.
I think CallHistoryChanged trigger maybe helpful but after I register background task for this type, my background task never call when I call to my phone or call from my phone or any missed call.
I ask the question in stack overflow but I couldn't found any solution.
Can any body help me?
thanks a lot

Communication blocking and filtering is for detecting the start of a call, not the end. If you are asking a question about UWP in general rather than about a specific sample, you can open a support case with our Developer Support team who can further help with the specific problem you are experiencing. Please use these steps below for further help:

Visit http://aka.ms/storesupport and sign-in with your Microsoft account
Under the “App Development” section choose "Windows 10 universal app development", then choose the appropriate problem type and category to get engaged through a formal support channel.


feat: add edit sign permission

Apologies for sending so many PRs. I was adding Flan support to another mod and realized I can register permission nodes inside Flan using their API. This adds a bespoke permission node for editing signs that can be configured by Flan users.
Additionally because we handle passthrough interaction, we need to check permissions for that block too, unfortunately the Flan methods to do that don't support everything, so we do some manual checks too.

No need to apologize. Thanks so much for the contributions!


Remote Access

when i try to connect from another pc or phone i'm getting this error
Transports.js:28 WebSocket connection to 'ws://<IP_ADDRESS>:3000/patient?signalRTokenHeader=eyJhbGciOiJSUzI1NiIsImtpZCI6ImQ2M2FlNWZiMTExNjlhMTMwMDI2YWFiNGVhOGVkZDNjIiwidHlwIjoiSldUIn0.eyJuYmYiOjE1MTc2NjAwNTQsImV4cCI6MTUxNzc0NjQ1NCwiaXNzIjoiaHR0cDovL2xvY2FsaG9zdDo1MDAxIiwiYXVkIjpbImh0dHA6Ly9sb2NhbGhvc3Q6NTAwMS9yZXNvdXJjZXMiLCJhcGkxIl0sImNsaWVudF9pZCI6ImNsaWVudCIsInN1YiI6IjU3ZDNiOTQ1LTQwZWItNGM4My05MGYyLTIwMDU2YmJmNzAwNyIsImF1dGhfdGltZSI6MTUxNzY2MDA1NCwiaWRwIjoibG9jYWwiLCJzY29wZSI6WyJhcGkxIl0sImFtciI6WyJwd2QiXX0.S6DHS24o60lsFewsvbcyV7j0wxo_vAf4N3ixg4fG_S3rWRMXmrkkHEdHjokVTkXNT1_07zzcehmmWyZdvDCpEsxwhX1mKbZdsZUFT0S8y8IJYsDXJlxw7exZOZOPW3Xp9HeHkfVOGKhDfwzKbHLgBzRpclb8AvMWe-4lAm0snjQTvroRJPSu5ifE8Aqv2qZr5ze_V4tSWX70Ly9ziZyGRGRMDzfyY4mr9Z2yDIYnHOHhBOqlrI4xCp340dZnSaCBLjC7jtI1wSu5nNuQEgQZtC0y5pH5NMi1Sxu7AMaAq4qNY-ivkHiIqVtu7IZrmeJD9HEWEjywVQh62T-Xe3wt2g&id=b1842c02-a9e4-40e7-91de-cdd8f2b244e6' failed: Error during WebSocket handshake: Unexpected response code: 504
Promise @ Transports.js:28
t @ polyfills.js:3
connect @ Transports.js:26
(anonymous) @ HttpConnection.js:54
fulfilled @ HttpConnection.js:6
t.invoke @ polyfills.js:3
onInvoke @ core.js:4629
t.invoke @ polyfills.js:3
r.run @ polyfills.js:3
(anonymous) @ polyfills.js:3
t.invokeTask @ polyfills.js:3
onInvokeTask @ core.js:4620
t.invokeTask @ polyfills.js:3
r.runTask @ polyfills.js:3
o @ polyfills.js:3
e.invokeTask @ polyfills.js:3
p @ polyfills.js:2
v @ polyfills.js:2
Loggers.js:17 Error: Failed to start the connection. undefined

Can you provide some more information about your environment? What operating systems are you using (Windows versions/Linux distros/etc.)? What server is running (Kestrel/IIS/IIS Express)? Are there any logs on the server (from either the Console Window or the Debug Pane in Visual Studio) that indicate exceptions throw on the server? What browser are you testing from (including version)?
The error code '504' indicates a "Gateway Timeout", which means a proxy may be blocking your connection.

from another pc or phone

Just saw this. If you try to connect over localhost on the server, does it work? Could you provide the startup log messages from your server?

yes when i use localhost it works normally however i'm using iisexpressproxy to test it remotely from a phone, note that iisexpressproxy works normally with webApi controllers but with signalr i'm receiving the above error

i was not able to get the logs for it because the connection is not being established however here is what i get from fiddler (
HTTP/1.1 504 Fiddler - Receive Failure
Date: Sat, 03 Feb 2018 19:38:02 GMT
Content-Type: text/html; charset=UTF-8
Connection: close
Cache-Control: no-cache, must-revalidate
Timestamp: 21:38:02.311
[Fiddler] ReadResponse() failed: The server did not return a complete response for this request. Server returned 0 bytes.
)

Try a different transport, I'm guessing your proxy is blocking websocket requests.

@mual1133 What operating system is the server running on? WebSockets in IIS Express require Windows 8/Windows Server 2012 or higher.

i have windows 10

https://docs.microsoft.com/en-us/iis/configuration/system.webserver/websocket

To be clear, when you say you are using "iisexpressproxy" are you referring to this: https://github.com/icflorescu/iisexpress-proxy ?
If so, and it only fails when using that proxy, you'd have to ask them if WebSockets are properly supported and how to configure them.

@anurse yes i was using this and im using typescript too.

Ok, then if it works fine outside iisexpressproxy you'll have to get further assistance from the iisexpressproxy devs. Closing this here as there doesn't appear to be a SignalR issue yet, feel free to reopen if one does arise!


Issues using textfield formatter.

I am trying to use the TextInputFormatter on the Textfield. All i am doing is add a modify the incoming text add a - at the end. It ends up in a infinite loop.  Is there anything wrong in the way I am doing it?

import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

main() {
    runApp(new Home());
}

class MyFormatter extends TextInputFormatter {
    MyFormatter();

    @override
    TextEditingValue formatEditUpdate(TextEditingValue oldValue,  TextEditingValue newValue) {
        var val = newValue.text + "-";
        return new TextEditingValue(text: val, composing: newValue.composing, selection: newValue.selection);
    }
}
class Home extends StatelessWidget {
    @override
      Widget build(BuildContext context) {
        var widget = new MaterialApp(
            home: new Scaffold(
                body: new Center(
                    child: new Container(
                        width: 300.0,
                        child: new TextField(
                        keyboardType:TextInputType.number,
                        inputFormatters: [
                            new MyFormatter()
                        ],
                    )
                    )

                )
            ),
        );

        return widget;
    }
}

I guess you shouldn't add - if there is one already.

Having the same issue. All I want to do is change the text in a formatter and I end up in an endless loop. Somehow when appending text the formatter is called again, however when changing existing text like making it upper or lower case, it does seem to work.

@manujbahl with @Toolenaar's suggestion I'm not able to reproduce.
[✓] Flutter (Channel master, v0.7.6-pre.27, on Mac OS X 10.13.6 17G65, locale en-AT)

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


maxHeapSize default is specified as null but it is actually 512m

Issue type
Wrong or misleading information
Problem description
The documentation for the Test.maxHeapSize parameter states that the default value is null. But this is actually not true since it defaults to 512m. This is actually correctly documented at https://docs.gradle.com/enterprise/test-distribution-agent/#:~:text=For Gradle builds%2C the maximum,to 512 MiB by default
Context (optional)
No response
Page with the problem
https://docs.gradle.org/current/dsl/org.gradle.api.tasks.testing.Test.html#org.gradle.api.tasks.testing.Test:maxHeapSize

Thank you for your interest in Gradle!
This is a valid documentation issue that we will address.

We indeed need to document the value used when the Test.maxHeapSize property value is null.
Note that the other documentation you linked is from Gradle Enterprise Test Distribution. It could be that the default Gradle test runner uses a different value.


ODAVA V4 outparam/inparam is not working in NAV 2018 CU11

Hi i have followed below link for sending outparam and inparam by using ODATA V4.
https://docs.microsoft.com/en-us/dynamics-nav/walkthrough-creating-and-interacting-odata-v4-bound-action#
But i am facing below issue.

can any one please suggest on which build version it is working fine.
Thank you
Ramesh

Hello, please contact your usual support channel for this question, we're unfortunately not able to help.


Why are underscores removed?

Hi there,
Thanks for this amazing library! I'm wondering why the tokenise method replaces underscores with space words = string.strip().replace('_', ' ').split()? This is not parameterized and even with unknown=True and strict=False these are removed. Our use case needs to preserve underscores. Happy to submit a PR if that would be helpful, I just want to understand why this is happening.
Thanks!

Thank you! I am happy you find the package useful! :green_heart:
To be honest, I cannot remember why this is the case. There is a commit from 2017 where I explicitly add the underscore as a word boundary char; I guess because of a dataset I was working with at the time.
At any rate, you are right, underscores should be treated just like any other unknown char; and the unknown and strict flags should indeed have a say in how it is handled. You are welcome to submit a PR if you have the time to! If not, I will take care of it and make a new release next weekend. Thank you for raising this issue!

Done, the latest version handles underscores as any other unknown character. Thank you again for raising this issue! (:

Thanks so much @pavelsof ! Sorry I didn't get around to making the PR myself - you were too quick. It's much appreciated though. Thanks again for the great library.


Clicking row number doesn't select the row in query results grid

Azure Data Studio Version: master

Steps to Reproduce:

Run a query
Click on the row number

Expected: Row is selected
Actual: Only the row number is selected

@anthonydresser could you please take a look at this?  This looks like a regression from the Nov. release.


Missing tesseract file

I've already downloaded and added but still it keeps on popping window missing
I've added like this C:/Program Files (x86)/Tesseract-OCR/
Am i missing anything ?

@MohammedNasheer
you have to write the location C:/Program Files (x86)/Tesseract-OCR/ in this tesseract_path file: https://github.com/Akascape/TEXTEMAGE/blob/main/python version/tesseract_path.txt

thanks for the response @Akascape but I've passed in that file only as per doc still issue persist that's the reason I'm asking

@MohammedNasheer Did you pass the full path to the tesseract.exe? Like this: C:/Program Files (x86)/Tesseract-OCR/tesseract.exe

the issue got resolved after adding the full path thanks @Akascape keep it up the good work


clientRequestValidation not working with camel REST DSL

Description
I have activated the clientRequestValidation and added hibernate-validator extension to rest-json example. When I send an invalid request, I'm not getting a 400 error code. Unless I misunderstood the validation feature described here? If so, any idea how to validate the request in a generic way for all the rest endpoints?
Expected behaviour
I would expect the endpoint to return 400.
Actual behaviour
A 500 is returned.
How to reproduce
Add hibernate extension: mvn quarkus:add-extension -Dextensions="hibernate-validator"
Replace this line https://github.com/apache/camel-quarkus-examples/blob/b1352cd4c0d2a1b7b17ba73c3aca410f516d42f3/rest-json/src/main/java/org/acme/rest/json/Routes.java#L47 with the following:
restConfiguration().bindingMode(RestBindingMode.json).clientRequestValidation(true);

Add a new test in the line https://github.com/apache/camel-quarkus-examples/blob/b1352cd4c0d2a1b7b17ba73c3aca410f516d42f3/rest-json/src/test/java/org/acme/rest/json/RestJsonTest.java#L56 as follows:
/* Test validation */
given()
        .body("{\"name2\": \"Pear\", \"description\": \"Winter fruit\"}")
        .header("Content-Type", "application/json")
        .when()
        .post("/fruits")
        .then()
        .statusCode(400);

Environment
openjdk version "11.0.11" 2021-04-20
macos 11.6 (20G165)
https://github.com/apache/camel-quarkus-examples version: b1352cd4c0d2a1b7b17ba73c3aca410f516d42f3

It sounds like this is not Camel Quarkus specific behavior.
@davsclaus is the REST DSL request validation also expected to cover the situation when the json body cannot be deserialized to the requested Java type (like here due to a misspelled json attribute)?

No the request validation is only to match the Content-Type and Accept HTTP headers that the client send to Camel, and if they is accepted in the Rest DSL, where you can configure what a REST path supports.
This validator does not touch the message body/payload.

And also if there are missing body / query parameters if they were set to be required in the Rest DSL

So its working as designed, to get an error code for invalid parsing json data, then you can use onException and set the http error code you want.

We can likely improved this
https://issues.apache.org/jira/browse/CAMEL-17006

Thanks @davsclaus. I see that the jira issue is resolved. Can I test it in this case?

Yeah its in the upcoming Camel 3.12.0 release

Closing as latest camel-quarkus is upgraded to Camel 3.12


Remove a repeated word in docs/graphql/index.rst

I was just checking out the launch event on YouTube and skimming through the docs a bit :smiling_imp:
(Congrats on launching 1.0 btw)

Good catch 🧐


Allow inversion of modifiers

It's possible at the moment to search for all members that return a Foo: $ juggle Foo; and it's possible to just show the static members returning a Foo: $ juggle static Foo. But there's no way of specifying non-static members.
Juggle should allow inversion of modifiers.
Proposed syntax is to add an operator in front of modifier productions. So for example, $ juggle \!static Foo. (Alternative options for the operator might be ~ or -.)

For many modifiers the behaviour is simple, e.g. !static would filter to members that don't have the static modifier set. Given that approach, would it be confusing that the opposite of !sealed is not the same as non-sealed?
What about negation of annotations, or negation of visibility such as !public? How Juggle handles visibility modifiers requires careful rethought. At the moment protected lists all members with that visibility or more (i.e. protected or public, but not default/package or private). So should !protected mean members with less visibility than protected?
Maybe the easiest option is to not allow negation of visibility modifiers.


Insomnia Cannot Generate Global Key-auth Plugin for DecK

Describe the bug
DecK generated by Insomnia does not contain a global key-auth plugin.
To Reproduce
Steps to reproduce the behavior:

Install the Insomnia IDE and then open it
Create a Design Document
Paste the following OpenAPI to the code area:openapi: 3.0.1
info:
  title: test-openapi
  version: 1.0.0
servers:
  - url: http://localhost
paths:
  /some-path:
    get:
      tags:
        - get
      summary: get response
      description: Returns a single pet
      operationId: get response
      responses:
        '200':
          description: successful operation
          content: {}
components:
  securitySchemes:
    ApiKeyAuth:
      type: apiKey
      in: header
      name: X-API-Key
security:
  - ApiKeyAuth: []

Click on Declarative Config generator and then the following DecK yaml is shown:_format_version: "1.1"
services:
  - name: test_openapi
    protocol: http
    host: test_openapi
    port: 80
    path: /
    plugins: []
    routes:
      - tags:
          - OAS3_import
        name: test_openapi-get_response
        methods:
          - GET
        paths:
          - /some-path$
        strip_path: false
    tags:
      - OAS3_import
upstreams:
  - name: test_openapi
    targets:
      - target: localhost:80
        tags:
          - OAS3_import
    tags:
      - OAS3_import

Expected behavior
The DecK generated should contains key-auth plugin.
Desktop (please complete the following information):

OS: macOS 10.15.7
App Version: v2021.3.0

Additional context
I am not sure if it is an expected behaviour, but if this sounds to be an issue to the team - just IMO, the cause seems to be at https://github.com/Kong/insomnia/blob/5f4c19da35e90ffa2f3ae27e9b309ee48f9d4e16/packages/openapi-2-kong/src/declarative-config/services.ts#L127, as the code does not contain a section to generate global plugins while in contrast the code in k8s folder does.
Thanks for any help.

I verified the same behaviour, it only works at route level, not at root level

+1 there is a bug in the code where plugins are generated. You are calling generateSecurityPlugins only in a loop that iterates over paths array

Hi @jacky96623,
I was trying to debug this issue and I noticed that on openapi-2-kong documentation, it would appear folks need to define the x-kong-plugin-key-auth object on their OpenAPI spec so that Insomnia (or Inso CLI) will return the plugin information on Declarative Config generation.
I've tried that and it seems to do the trick and add the plugin. Check my example section bellow.
BUT, for instance, if we try the example you provided, the generated Kong config on the other hand does mention that plugin without even needing the x-kong-plugin-key-auth part on the OAS:

So perhaps we do have a bug in there or at least we should follow a similar behaviour for Declarative configs.
Tagging @Tieske - any suggestions or ideas what the default or accepted behaviour should be?
Example
Spec:
openapi: 3.0.0
info:
  title: 'test'
  version: 1.0.0
  x-kubernetes-ingress-metadata:
    name: insomnia-api                          # Kubernetes Ingress document name
    annotations:                                # Annotations to automatically apply
      'example': 'example'
x-kong-plugin-custom-global:                    # Global plugin to always be applied
  name: custom-global
  enabled: true
  config:
    key_in_body: false
servers:
  - url: http://one.insomnia.rest/v1
  - url: http://two.insomnia.rest/v2
    x-kong-plugin-custom-server:                # Server plugin to be applied to documents for this server
      name: custom-server
      enabled: true
paths:
  '/path':
    post:
      summary: Some functionality
      responses:
        '200':
          description: successful operation
          content: {}
  '/another-path':
    get:
      summary: Some functionality
      responses:
        '200':
          description: successful operation
          content: {}
      x-kong-plugin-key-auth:                   # Operation plugin on GET /another-path
        name: key-auth
        enabled: true
        config:
          key_names: [api_key, apikey]
          key_in_body: false
          hide_credentials: true

Shows the plugin information in the generated declarative config:

As reference for future us:
Using x-kong-plugin-key-auth is the generic way of adding plugins. This will work with any plugin. So it is different from the user expectations that the generator would take the security directives into account and would then generate the proper key-auth config automatically.
The difference being that using security (the OAS concept) would not require the user to have any knowledge about Kong whatsoever. Whereas using the plugin directive, would require that knowledge.


[#1099]: Add support for multiple events in improved timeline

Preview environment created!
The preview environment has been deployed to the cluster.
Client URL: https://pr-1951--epp.elifesciences.org
Note: If this is a new environment, it can take a few minutes for DNS to update.

Preview environment images deployed!
The preview environment has been updated:
client image tag: preview-1951-887ec9063455074373f7c56b1418eb663d38cc36-20240530.2136
Client URL: https://pr-1951--epp.elifesciences.org
Note: Updated images can take a few minutes to roll out.


[MSYS-814] - Manual cron job deletion

Commit to handle deletion of manually created cron jobs, other than chef
Also fixed an existing issue: If chef created job is not the last job written in crontab, then even after its deletion, message was: save unmodified crontab

Signed-off-by: nimesh-msys<EMAIL_ADDRESS>Description
Cron Job Deletion process: Iteration over each line of crontab and insert the contents(line) into a local string 'crontab'.  When we found "# Chef Name: #{new_resource.name}", this line(the header comment) and the next line(with cron job) will be skipped. Hence 'crontab' string only contains the remaining line of codes. This 'crontab' then completely overwrites crontab file. Thus we were only deleting the jobs which were created by chef (by header search)
Following code changes includes: during iteration if we encounter a 'cron job pattern', which is exactly similar to what user has passed(for deletion), skip that line too. This skipping will be irrespective of resource name, hence the given job could also be deleted which was created manually.
Issues Resolved
MSYS-814
Check List

[ ] New functionality includes tests
[x] All tests pass
[x] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

Also this needs a metric ass-load of tests :D

These code changes may not be required. There are few complexities while handling environment variables. It may also affect cron entries created by chef.
Have sent a request for updating the document including workaround to achieve this.