fields are statically typed. Make sure all fields are of the correct type

Describe what happened
Trying to create a NotificationDestination using Pulumi Golang, but seeing this error.
using latest version
github.com/pulumi/pulumi-newrelic/sdk/v5 v5.31.0
error: 1 error occurred:
    * 403 response returned: NOTICE: fields are statically typed. Make sure all fields are of the correct type

Sample program
notificationDestination, err := newrelic.NewNotificationDestination(ctx, destinationName, &newrelic.NotificationDestinationArgs{
	AccountId: pulumi.String(accountId),
	Name:      pulumi.String(destinationName),
	Type:      pulumi.String("WEBHOOK"),
	Properties: newrelic.NotificationDestinationPropertyArray{
		&newrelic.NotificationDestinationPropertyArgs{
			Key:   pulumi.String("url"),
			Value: pulumi.String(webhookUrl),
		},
	},
})

Log output
No response
Affected Resource(s)
No response
Output of pulumi about
some fields are masked
CLI
Version      3.131.0
Go Version   go1.23.0
Go Compiler  gc

Plugins
KIND      NAME        VERSION
resource  archive     0.0.6
resource  aws         6.29.0
resource  aws-native  0.13.0
resource  cloudflare  5.16.0
resource  cloudflare  3.5.0
resource  command     0.11.1
resource  fastly      8.4.1
language  go          unknown
resource  kubernetes  4.9.1
resource  newrelic    5.31.0
resource  random      4.2.0
resource  tailscale   0.13.3

Host
OS       darwin
Version  14.6.1
Arch     arm64

This project is written in go: executable='/opt/homebrew/bin/go' version='go version go1.22.2 darwin/arm64'

Current Stack: xxxxxxxxxxx

TYPE                       URN
pulumi:pulumi:Stack        urn:pulumi:newrelic::xxxxxxxx::pulumi:pulumi:Stack::xxxxxxxx-newrelic
pulumi:providers:newrelic  urn:pulumi:newrelic::xxxxxxxx::pulumi:providers:newrelic::default_5_31_0
pulumi:providers:newrelic  urn:pulumi:newrelic::xxxxxxxx::pulumi:providers:newrelic::default_5_17_0

Found no pending operations associated with newrelic

Backend
Name           work.local
URL            s3://xxxxxxxx
User           raywonkari
Organizations
Token type     personal

Dependencies:
NAME                                        VERSION
github.com/aws/aws-sdk-go-v2                v1.21.2
github.com/aws/aws-sdk-go-v2/config         v1.15.15
github.com/aws/aws-sdk-go-v2/service/ecs    v1.9.0
github.com/pulumi/pulumi-aws-native/sdk     v0.13.0
github.com/pulumi/pulumi-random/sdk/v4      v4.2.0
github.com/pulumi/pulumi/sdk/v3             v3.129.0
github.com/stretchr/testify                 v1.9.0
github.com/aws/aws-sdk-go                   v1.44.298
github.com/chartmogul/chartmogul-go/v3      v3.1.1
github.com/docker/docker                    v24.0.7+incompatible
github.com/go-git/go-git/v5                 v5.12.0
github.com/opencontainers/image-spec        v1.0.2
github.com/pulumi/pulumi-archive/sdk        v0.0.6
github.com/pulumi/pulumi-aws/sdk/v6         v6.29.0
github.com/pulumi/pulumi-cloudflare/sdk/v5  v5.16.0
github.com/pulumi/pulumi-command/sdk        v0.11.1
github.com/pulumi/pulumi-fastly/sdk/v8      v8.4.1
github.com/pulumi/pulumi-kubernetes/sdk/v4  v4.9.1
github.com/pulumi/pulumi-newrelic/sdk/v5    v5.31.0
github.com/pulumi/pulumi-tailscale/sdk      v0.13.3
github.com/tdewolff/minify/v2               v2.20.16
github.com/pulumi/pulumi-cloudflare/sdk/v3  v3.5.0

Pulumi locates its logs in /var/folders/m_/y8fbgmsd693cgpl3lh22hqth0000gn/T/ by default

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @raywonkari, thanks for reporting the issue and sorry you've hit this.
The pulumi newrelic provider uses the terraform newrelic provider under the hood and this issue seems to be a problem with the TF provider. I've reported it here https://github.com/newrelic/terraform-provider-newrelic/issues/2746 to the newrelic team.
The newrelic team has the expertise to handle this. As soon as that issue is addressed we'll pick it up in the pulumi provider as well.

@raywonkari, a user came up with a workaround for the issue in the TF issue thread: https://github.com/newrelic/terraform-provider-newrelic/issues/2746#issuecomment-2405731337
Can you please try it and see if it works for you? You'd need to remove the account ID parameter from the NotificationDestination resource and specify it on the provider resource instead.


#1490 add all coco users to h2 in-mem for UI demo

See #1490
Added coco users to UI H2 db
Signed-off-by: Nigel Jones<EMAIL_ADDRESS>

closing - typo with file committed


Update jackson version to 2.9.8

See #137

Coverage remained the same at 51.765% when pulling 2801276a6491b3786b0426a17c6141f9f2b46990 on yumaoka:jackson-ver into d48150ea19aea9ed20ebdcc0b40aab0823066118 on IBM-Cloud:master.


Have an issue on IP detection step.

Hi, I have an issue when the library tries to detect IP.
I tried it on another machine with the same result also tried different domain names.
It always somewhy stuck on this step.
Command:
./cloudbrute -d target_domain -k target -m storage -t 80 -T 10 -w "./data/storage_small.txt"

2:43PM INF Detect config path: config/config.yaml
2:43PM INF Detect provider path: config/modules
2:43PM INF Initialized scan config
2:43PM WRN IP detection failed
2:43PM FTL Source detection failed as well use -c . error="no provider found in HTML"

This output happened when Cloudflare firewall detected.
12:13AM INF Detect config path: config/config.yaml
12:13AM INF Detect provider path: config/modules
12:13AM INF Initialized scan config
12:13AM ERR IP detection failed error="CloudFlare detected target is behind proxy"
12:13AM WRN IP detection failed
12:13AM FTL Source detection failed as well use -c . error="no provider found in HTML"

This is expected behavior , because of CloudFlare engine can't detect the IP in your case also HTML source scanning failed you have to force engine to go with your selected provider using -c option .


Uneven top and bottom padding in the inverse heard

Hello
There is a big inconsistency in the application of padding in the inverse header, most visible in HTML attachments, like this one or that one.
We suggest solving it by making the padding from the top the same as the padding from the bottom. You could do it the other way around, but that would look like less visually balanced.
There is a bunch of weird !important stuff in the HTML so I'm not sure what the right solution is, but see attached for what it should look like when addressed.

HTML attachment with even spacing

I think the problem is to do with the way the component is being used on this page. By default the title component (the white text rather than the blue box) has a margin top and margin bottom, but it looks like an option has been set on the title to turn off the bottom margin. Using this option on the title component should remove the top margin.
Side note, the option in question is currently listed under 'using Design System template' rather than the thing I was immediately looking for 'change top margin'. Might be worth rewording this and moving to be next to the margin_bottom option at some point to make it more discoverable.

There is a PR opened for this issue here.


Vim9: crash when interrupting job callback which encounters error

Describe the bug
In Vim9 script, interrupting a job callback which encounters an error might make Vim crash.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9
    def Func(...l: any)
        try
            echohl ModeMsg
        catch
        endtry
        timer_start(0, () => win_execute(id, 'invalid'))
    enddef
    var id = popup_create('', {})
    job_start(['/bin/sh', '-c', 'locate /'], {out_cb: () => Func()})
EOF
)

E492 is raised:
E492: Not an editor command: invalid

This is expected.
Now, keep pressing C-c to interrupt the errors.  Sometimes, Vim ends up crashing.  If it doesn't crash, you might need to re-run the shell command.
Expected behavior
Vim doesn't crash.
Environment

Vim version: 8.2 Included patches: 1-2374
OS: Ubuntu 16.04.7 LTS
Terminal: xterm(363)

Additional context
If the code is re-written in Vim script legacy, pressing C-c doesn't seem to make Vim crash:
vim -Nu NONE -S <(cat <<'EOF'
    fu Func(...)
        try
            echohl ModeMsg
        catch
        endtry
        call timer_start(0, {-> win_execute(s:id, 'invalid')})
    endfu
    let s:id = popup_create('', {})
    call job_start(['/bin/sh', '-c', 'locate /'], {'out_cb': {-> Func()}})
EOF
)

backtrace
valgrind log
asan log

Tricky to reproduce.  Or write a test for when it gets fixed.
Could you try to make it crash reliably?

Tricky to reproduce.  Or write a test for when it gets fixed.
Could you try to make it crash reliably?

I've just noticed that it was hard, if not impossible, to make it crash in xterm, unless TERM is changed.    I guess it means that some terminal options are involved.  I'll start from there, and see what I can get.

I've just noticed that it was hard, if not impossible, to make it crash in xterm, unless TERM is changed.    I guess it means that some terminal options are involved.  I'll start from there, and see what I can get.

I guess it means that some terminal options are involved.

That won't help for a test, but it seems that clearing t_TI (--cmd 'set t_TI=') makes it easier to crash Vim by pressing C-c interactively.  I'll keeping looking for something which can be automated.

I guess it means that some terminal options are involved.

That won't help for a test, but it seems that clearing t_TI (--cmd 'set t_TI=') makes it easier to crash Vim by pressing C-c interactively.  I'll keeping looking for something which can be automated.

This crashes almost consistently in xterm on my machine; but it still needs a manual interrupt:
vim -Nu NONE --cmd 'set t_TI=' -S <(cat <<'EOF'
    vim9
    def Func(...l: any)
        try
            echohl ModeMsg
        catch
        endtry
        timer_start(0, () => win_execute(id, 'invalid'))
    enddef
    var id = popup_create('', {})
    job_start('locate ' .. $HOME, {out_cb: Func})
EOF
)

This crashes almost consistently in xterm on my machine; but it still needs a manual interrupt:
vim -Nu NONE --cmd 'set t_TI=' -S <(cat <<'EOF'
    vim9
    def Func(...l: any)
        try
            echohl ModeMsg
        catch
        endtry
        timer_start(0, () => win_execute(id, 'invalid'))
    enddef
    var id = popup_create('', {})
    job_start('locate ' .. $HOME, {out_cb: Func})
EOF
)

@brammool  wrote:

Tricky to reproduce. Or write a test for when it gets fixed.
Could you try to make it crash reliably?

On my system, it's easy to reproduce after running the command
in bug description and pressing CTRL-C once or sometimes
multiple times till it crashes.  I get the same stack trace as @lacygoill .
I don't have time now, but I can try later to automate it with a test
if that's possible.
excp is NULL at ex_eval.c:670 which causes the segfault:
 667     void
 668 catch_exception(except_T *excp)
 669 {
!670     excp->caught = caught_stack;

I use vim-8.2.2377 (huge, without gui) in xfce4-terminal on xubuntu-18.04.5.

@brammool  wrote:

Tricky to reproduce. Or write a test for when it gets fixed.
Could you try to make it crash reliably?

On my system, it's easy to reproduce after running the command
in bug description and pressing CTRL-C once or sometimes
multiple times till it crashes.  I get the same stack trace as @lacygoill .
I don't have time now, but I can try later to automate it with a test
if that's possible.
excp is NULL at ex_eval.c:670 which causes the segfault:
 667     void
 668 catch_exception(except_T *excp)
 669 {
!670     excp->caught = caught_stack;

I use vim-8.2.2377 (huge, without gui) in xfce4-terminal on xubuntu-18.04.5.

This crashes almost consistently in xterm on my machine; but it still needs a manual interrupt:
vim -Nu NONE --cmd 'set t_TI=' -S <(cat <<'EOF'
    vim9
    def Func(...l: any)
        try
            echohl ModeMsg
        catch
        endtry
        timer_start(0, () => win_execute(id, 'invalid'))
    enddef
    var id = popup_create('', {})
    job_start('locate ' .. $HOME, {out_cb: Func})
EOF
)

Thanks, now I can reproduce it, although not reliably.
Looks like the problem is that when a timer is called nearly all state
concerning exceptions is saved and restored, but not the need_rethrow
flag.  Let's see if saving and restoring need_rethrow helps.
--
hundred-and-one symptoms of being an internet addict:
192.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

This crashes almost consistently in xterm on my machine; but it still needs a manual interrupt:
vim -Nu NONE --cmd 'set t_TI=' -S <(cat <<'EOF'
    vim9
    def Func(...l: any)
        try
            echohl ModeMsg
        catch
        endtry
        timer_start(0, () => win_execute(id, 'invalid'))
    enddef
    var id = popup_create('', {})
    job_start('locate ' .. $HOME, {out_cb: Func})
EOF
)

Thanks, now I can reproduce it, although not reliably.
Looks like the problem is that when a timer is called nearly all state
concerning exceptions is saved and restored, but not the need_rethrow
flag.  Let's see if saving and restoring need_rethrow helps.
--
hundred-and-one symptoms of being an internet addict:
192.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Make incorrect spec watcher suggest better specs

At minimum we could have a hash of the most common ones: http://logs.glob.uno/?a=search&c=mozilla%23servo&q=w3.org%2FTR&ss=24+Mar+2013&se=24+Feb+2015

Additionally make it notice dev.w3.org/html5/.


fix: prefixed all the internal variables with _ to prevent collisions with user defined variables

fixes: https://github.com/Decathlon/tzatziki/issues/25
we need to write something in the release note for developers using those internals variables!

Codecov Report

Merging #27 (6473bfc) into main (7d34c17) will decrease coverage by 0.04%.
The diff coverage is 91.30%.

@@             Coverage Diff              @@
##               main      #27      +/-   ##
============================================
- Coverage     70.85%   70.81%   -0.05%
  Complexity      691      691
============================================
  Files            35       35
  Lines          2711     2714       +3
  Branches        291      291
============================================
+ Hits           1921     1922       +1
- Misses          639      641       +2
  Partials        151      151

Impacted Files
Coverage Δ

...n/java/com/decathlon/tzatziki/steps/HttpSteps.java
85.76% <81.81%> (-0.65%)
:arrow_down:

...java/com/decathlon/tzatziki/steps/ObjectSteps.java
73.24% <100.00%> (ø)

.../java/com/decathlon/tzatziki/steps/KafkaSteps.java
86.90% <100.00%> (ø)

...java/com/decathlon/tzatziki/steps/SpringSteps.java
90.62% <100.00%> (+0.30%)
: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 7d34c17...6473bfc. Read the comment docs.


[cmake error] Could not find a package configuration file provided by "hip"

Describe the bug
Doing cmake , while met a error:
(base) loong@home:~/Downloads/rocRAND/build$ CXX=hipcc cmake -DBUILD_BENCHMARK=ON ../.
-- The CXX compiler identification is Clang 15.0.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/hipcc - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Setting build type to 'Release' as none was specified.
-- Found Git: /usr/bin/git (found version "2.34.1")
-- Performing Test COMPILER_HAS_TARGET_ID_gfx803
-- Performing Test COMPILER_HAS_TARGET_ID_gfx803 - Success
-- Performing Test COMPILER_HAS_TARGET_ID_gfx900_xnack_off
-- Performing Test COMPILER_HAS_TARGET_ID_gfx900_xnack_off - Success
-- Performing Test COMPILER_HAS_TARGET_ID_gfx906_xnack_off
-- Performing Test COMPILER_HAS_TARGET_ID_gfx906_xnack_off - Success
-- Performing Test COMPILER_HAS_TARGET_ID_gfx908_xnack_off
-- Performing Test COMPILER_HAS_TARGET_ID_gfx908_xnack_off - Success
-- Performing Test COMPILER_HAS_TARGET_ID_gfx90a_xnack_off
-- Performing Test COMPILER_HAS_TARGET_ID_gfx90a_xnack_off - Success
-- Performing Test COMPILER_HAS_TARGET_ID_gfx90a_xnack_on
-- Performing Test COMPILER_HAS_TARGET_ID_gfx90a_xnack_on - Success
-- Performing Test COMPILER_HAS_TARGET_ID_gfx1030
-- Performing Test COMPILER_HAS_TARGET_ID_gfx1030 - Success
-- Performing Test COMPILER_HAS_TARGET_ID_gfx1100
-- Performing Test COMPILER_HAS_TARGET_ID_gfx1100 - Success
-- Performing Test COMPILER_HAS_TARGET_ID_gfx1101
-- Performing Test COMPILER_HAS_TARGET_ID_gfx1101 - Success
-- Performing Test COMPILER_HAS_TARGET_ID_gfx1102
-- Performing Test COMPILER_HAS_TARGET_ID_gfx1102 - Success
Looking in ...
CMake Error at cmake/VerifyCompiler.cmake:35 (find_package):
  Could not find a package configuration file provided by "hip" with any of
  the following names:

    hipConfig.cmake
    hip-config.cmake

  Add the installation prefix of "hip" to CMAKE_PREFIX_PATH or set "hip_DIR"
  to a directory containing one of the above files.  If "hip" provides a
  separate development package or SDK, be sure it has been installed.
Call Stack (most recent call first):
  CMakeLists.txt:111 (include)

-- Configuring incomplete, errors occurred!
See also "/home/loong/Downloads/rocRAND/build/CMakeFiles/CMakeOutput.log".

To Reproduce

cd rocRAND; mkdir build; cd build
CXX=hipcc cmake -DBUILD_BENCHMARK=ON ../.

Hi, thanks for the report. It looks like rocRAND cannot find the HIP package. You can solve this by specifying the ROCm installation prefix to CMake with -D CMAKE_PREFIX_PATH=<rocm_path>. To find out where ROCm is installed, you can run hipconfig. By default, it is installed in /opt/rocm. Alternatively, you can set the ROCM_PATH environment variable to the ROCm path, or include it in PATH. This is indeed lacking from the documentation and we shall update it.


TP: Queuing Servers for updates - Queues up all the servers

I'm submitting a ...

[X] bug report
[ ] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[X] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:

When Queue Server Updates is performed
All the servers other than edges and mids get the wait icon next to it.
Expected / new behavior:

Only Mids and Edges should be queued for updates and the wait/pending icon should be displayed next to only edge and mid servers
Minimal reproduction of the problem with instructions:

Anything else:

What do you mean by "Queue Server Updates"? There are a few different ways to do that. What exactly did you do?

What do you mean by "Queue Server Updates"? There are a few different ways to do that. What exactly did you do?

She queued server updates at the cdn level which will queue ALL servers in the CDN even though only Edges / Mids are affected by a "queue". This won't hurt anything but it is kind of silly to set the upd_pending flag = true on anything besides an Edge* / Mid*
IMO, 2 things should be fixed:

POST api/1.4/cdns/2/queue_update should only update Edge* / Mid* servers
In TP, the "updates pending" column of the servers table should probably be N/A if the server is not Edge* / Mid*

I think other server types can meaningfully have pending updates, although I can't quite remember for sure.

I think other server types can meaningfully have pending updates, although I can't quite remember for sure.

hmmm. i don't think so...but i guess i'm not 100% sure...


how to play video with react-player from spesific path

I 'm using react-player for playing videos.It plays videos from urls(like youtube url) succesfully. But I'm trying a video stored in a spesific path like this "E:\video\xxx.mp4" . But video is not loading-
Video Player should work with videos stored whre in a folder path.
I think need to stream but how? I have tried a few way but I could not achieve. Can somebody help me?
Environment

URL attempting to play:  "E:\video\xxx.mp4"
Browser: Chrome
Operating system: windows

See https://github.com/cookpete/react-player/issues/823#issuecomment-604073841


Errors when trying to re-install

This might be considered a user error, because what I did was install code-debug a second time, rather that uninstalling and reinstalling, or updating. I didn't read far enough in the installation instructions. :-)
However, I would suggest that code-debug (or perhaps VS Code itself) should check if the user is trying to reinstall, and give some more user-friendly error handling.
Attached is the output I got on my xterm, when I started Code, then tried to reinstall code-debug. Note that the first few lines, down to and including "bash: no job control in this shell" happen every time I start Code. That's probably a separate issue, but if someone knows how to fix that, please let me know.
InstallErrors.txt

Um seems like a vscode bug. Try manually deleting the folder /home/jan/.vscode/extensions/webfreak.debug-0.3.1
If that works then report it as issue to Microsoft/vscode. Extensions don't handle anything about installation and deinstallation except they add some binaries. IIRC they can't call any code there

Not sure what you mean by "If that works..." If I delete .../webfreak.debug-0.3.1 then start Code, Code still shows Debug 0.3.5 as an installed extension. If I click README in the Debug entry (in the installed extension list), Code displays something in the web browser and also shows in the Code window: "Info / Would you like to install 'Debug'? / Install Now / Cancel" If I click Install Now, it appears to install and gives about the same error messages as I reported before.
Do you want me to report this to Microsoft/vscode?
P.S. Thanks for developing this extension. I think it will be quite pleasant to use after it is polished up a bit more.

yeah I think this is more an issue in vscode if its throwing that from the application and not from an extension. Don't forget to post clear steps to reproduce it there


⚠ Bofrost.de has degraded performance

In 945bd6f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1422 ms

Resolved: Bofrost.de performance has improved in a70ad54 after 9 minutes.


Available mode

All done, it should work with the current version of the HA driver

Doing some tests now with the new version...

Done, ready to be reviewed and merged


Update consulting-services.md

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

@yijenj : Thanks for your contribution! The author, @ellacroi, has been notified to review your proposed change.

#sign-off


if i set kubeProxyReplacement=strict , pod connected to k8s api server(u can curl -k https://<IP_ADDRESS>:443 in pod) failed

so what cause this problem？i found if i manully add iptables rule( iptables -t nat -I POSTROUTING -s <IP_ADDRESS>/16 -j MASQURADE), it conneted to <IP_ADDRESS> success。

@liuyixian123 When you created this issue, you should've been prompted with a template. Can you fill in those details? At the very least, can you write a proper one-line description of the problem in the title, and provide details regarding cilium version, sysdump, etc?


copyright note added

copyright note added to all jupyter notebook sample files and to rust src files

Looks good but could you update the copyright to 2021-2022

updated


Restoring from a checkpoint failed. Missing optimizer error.

Using DorefaQuantizer, but believe problem existing for other quantizers in clipped_linear.py.
Issue seems to be that the optimizer must be loaded first before a call to line 223 that requires the optimizer. Currently load optimizer happens near the end at line 247 in the function load_checkpoint.
Here is the trace dump.
Log file for this run: classifier_compression/logs/2019.08.27-131259/2019.08.27-131259.log
Traceback (most recent call last):
File "compress_classifier.py", line 204, in
main()
File "compress_classifier.py", line 73, in main
app = ClassifierCompressorSampleApp(args, script_dir=os.path.dirname(file))
File "compress_classifier.py", line 162, in init
super().init(args, script_dir)
File “\distiller/apputils/image_classifier.py", line 68, in init
self.start_epoch, self.ending_epoch) = _init_learner(args)
File “distiller/apputils/image_classifier.py", line 377, in _init_learner
model, args.resumed_checkpoint_path, model_device=args.device)
File "distiller/apputils/checkpoint.py", line 223, in load_checkpoint
quantizer = qmd['type'](model, **qmd['params'])
TypeError: init() missing 1 required positional argument: 'optimizer'

Hi @wfong443 ,
This is a known bug which we are fixing in #309 , soon it will be pushed and you'll be able to resume any Quantization-Aware Training compression scheduler.
Thanks,
Lev


When adding a DHCP static reservation in Kia the DNS service needs restarting/reloading

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[ yes] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[yes ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
When I added a DHCP reservation I found I couldn't ping the client.  It appears as if a reload/restart of unbound dns was required.
To Reproduce
Steps to reproduce the behavior:
1.) add dhcp reservation in Kia
2.) try to ping host via DNS
3.) host times out
Expected behavior
1.) add dhcp reservation in Kia
2.) try to ping host via DNS
3.) host responds
Describe alternatives you considered
A clear and concise description of any alternative solutions or workaround you considered.
Screenshots
If applicable, add screenshots to help explain your problem.
Relevant log files
If applicable, information from log files supporting your claim.
Additional context
Add any other context about the problem here.
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 23.7.8 (amd64).
Intel® Xeon™ E3-1225V5 3.3Ghz Quad Core
Network Intel® I210-AT

Working as discussed/intended.

Surely if you add a lease and it doesn't work it isn't as intended ?  If you need to restart DNS, it should tell you at least ?

Docs PR is welcome. Behaviour for static DHCP  leases to DNS services is consistent across the software. Since the two services do not belong together no restart is forced for the other.

Lots of the other pages have notices saying 'Don't forget to restart the service'.  Can this be added please ?

@simonmcnair will do, better for consistency indeed. Let me close this ticket while adding the message

sorry, it's not that you need to apply them.  You need to restart whichever DNS service you use.

The generic apply message is no problem to add (as it was missing), trying to explain dependancies is not something we're considering. The documentation is a better place for that, as @fichtner already mentioned, PR's are welcome.

  When you do something you expect it to take affect.  If it doesn't and you have to troubleshoot and figure stuff out when a warning or notice (without having to look at documentation) would have helped, then you should be asking yourself why.  If adding a DHCP lease reservation requires DNS to be restarted in order for name resolution to work.  Why wouldn't you tell someone ?

Your response isn’t great either, but it’s besides the point. Why do you expect a particular service restarting warrants restarting an auxiliary service?

From a usability perspective if you change something and hit apply, or continue you expect it to take effect.
If something additional is required to make it happen, not telling someone that it is needed is a poor experience.
I can see your point though.  In this specific case once the target has been rebooted and has its DHCP address, if configured, it'll update DNS anyway.

This is non-negotiable for apply doing auxiliary reloads of mildly related services. We are to this day fighting side effects stemming from this historic approach of „gluing everything together“.

But this is new, right? I know from a fact that I didn't have to restart unbound when adding a static DHCP entry. Since the last big update I have to restart it every time now. And as @simonmcnair says, it's hard to figure out what the problem is if your dns record is in the DHCP list and not reachable -> Which has worked before in previous updates.
Would it be possible to add a checkbox in unbound with gives us the option do decide whether it restarts or not when applying dhcp entries?


Firebase pause and restart

Hello,
I am using two firebase objects, one for stream and other for update data. I am doing a Ping to detect if the internet connection is good. And when I detect a bad connection I use the function pauseFirebase to pause. And I restart just when connection is good again.
But sometimes, after a while, firebase stops to receive the stream and to update. I think that maybe it is not restarting for some reason.
What is the return of pauseFirebase? Could I use the result to know if firebase restarted?
Is there any other way to know if it is connected? And if it is not connected, is there any way to restart?
I just try to update when wifi is connected and the connection is good.

No extra ping or stop/start is necessary, just call httpConnected() from Firebase Data object which gives the precise status of server connection.
The stream resuming is done appropriately when timed out (heart beat data or keep-alive event data from server was missed more than 40 seconds).
Please don't use pauseFirebase for stream Firebase Data object and let the internal flow operate normally.

The internet connection will be managed (resume) internally as you call Firebase.reconnectWiFi(true);.
Do not use reconnectWiFi(true) with the WiFiManager or code that also handles the WiFi reconnection because it will interfere to each other.


Not Found<EMAIL_ADDRESS>

+1

Seems like an npm issue.
This package does exists: https://www.npmjs.com/package/@sindresorhus/is
You might be able to get support from npm team, but most likely it'll be back online soon.


Add blocksize to task name

[x] Tests added / passed
[x] Passes black dask / flake8 dask

Closes #6814
I am still thinking about how to add tests. I think this only happens on a distributed cluster.

Thank you @jsignell for looking into it!

Thanks @jsignell for fixing this and @nils-braun for raising


Should diff always be empty when Immutable.is returns true

I am having trouble tracking down the cause of a diff. Two immutable objects return true when passed into Immutable.is while diff return a total replace of the object in question. Should this behaviour be possible?

Could you please provide a simple test case that reproduces the issue?

Trying to repo but finding it difficult. Code in question (using transit-immutable):
      const payload = response.payload;

      console.log(diff(
        fromJSON(payload).get('dataValue'),
        mockState.get('dataValue')
      )); // not empty
      console.log(Immutable.is(
        fromJSON(payload).get('dataValue'),
        mockState.get('dataValue')
      )); //true :(

This should not happen? Will post back here if I can find simple example

Putting some logs into the function primativeTypeDiff
function primitiveTypeDiff(a, b) {
  console.log(a); // Set { "dataValueId1" }
  console.log(b); // Set { "dataValueId1" }
  console.log((0, _immutable.is)(a, b)); // true
  console.log(a === b); // false

it seem there are some cases where you are using js equality when we need Immutable.is ?

Here is the test
import { Set } from 'immutable';
import assert from 'assert';
import diff from 'immutable-diff';

const setDiff = diff(Set(['1']), Set(['1']));
assert.equal(setDiff.size, 0, 'there should not be a diff');

@juanplopes this seems like a bug.

@des-des v0.4.3 was published to NPM and should fix this issue. Could you test it please?

@pedroteixeira could you publish v0.4.3 to NPM?


[FEATURE] Setup backend - Firebase Cloud Functions

Overview
Add Firebase Functions to retrieve/update data from Firebase Realtime Database:

Install Firebase Functions as an npm package. Reference: https://firebase.google.com/docs/web/setup
Install Firebase CLI. Reference: https://firebase.google.com/docs/cli
Do firebase init functions and select CICAPI project (cicapi-b3398). Use TypeScript when asked for language. Reference: https://firebase.google.com/docs/functions/typescript.
Firebase Realtime Database Reference: https://firebase.google.com/docs/database

Scrapping Firebase Cloud Functions, will use Firebase API for Node.js to directly interface with Realtime Database (with event listeners) in a pub/sub pattern.

Integrated updated UI for members in this same issue.


Update head and tail to work for all Foldable

Currently Head and Tail only work for Array, but it is possible to implement them for all Foldable.

Sample head implementation:
const f = S.reduce(b => a => S.maybe(S.Just(a))(_ => b)(b))(S.Nothing)
Sample, but incomplete tail implementation:
const tail = xs => {
  let skipped = false;
  return Z.reduce ((xs, x) => {
    if (skipped) {
      return Z.append (x, xs);
    } else {
      skipped = true;
      return xs;
    }
  }, [], xs);
};

Here's a complete tail implementation:
// tail :: (Foldable f, Applicative g, Monoid g) => TypeRep g -> f a -> Maybe (g a)
const tail = tr =>
  S.reduce(b => a =>
    S.maybe
      (S.Just(Z.empty(tr)))
      (x => S.Just(Z.concat(x, Z.of(tr, a))))
      (b)
  )(S.Nothing);

[tail(Array)([]), tail(Array)([1]), tail(Array)([1, 2]), tail(Array)([1, 2, 3])]
// => [Nothing, Just ([]), Just ([2]), Just ([2, 3])]

@Bradcomp and @masaeedu, would one of you like to add these functions to sanctuary-type-classes?

@syves has expressed interest in working on these generalizations. While we're at it, let's generalize S.last and S.init.


[ Vieja Escuela] HP Tokens and character sheets doesnt match itselfs

Good Night, I have this problem with the tokens and the character sheet , specifically with the HP attribute , look for example an X character have 12 of max HP and it actual HP is 10 in it respective character sheet, but with it default token appears the reverse , like it have 12 actual life and  10 as the max , and it's really confusing trying to play like that , but if I change it in the token , happens the same on the character sheet.  I apologize your team if I am annoying with this request because it's a simplicity , but my players and me will be thankful

This seems like an issue of mislabeled fields on lines 63 and 64:

<input class="sheet-underlined" type="number" name="attr_HP" step="1">
Max

<input class="sheet-underlined" type="number" name="attr_HP_max" min="1" step="1">
Actual

@elfonochasis you seem to have made recent changes to this sheet. Would you be willing to fix?

Sorry mate, Vieja Escuela is not mine. I've created Nuestro Ultimo Verano sheet.


Enhancement: Platform Extension for CloudFormation to be orchestrated by Ansible

Enhancement Summary
Instead of using AWS cli in the Platform Extension job's execute shell to run Cloudformation templates, we can utilize Ansible to orchestrate it and have a flexible way of launching user's templates without locking them with predefined 'CF parameters'. For now let's just focus on CloudFormation.
[Update]
Attached the sample screenshot from my Jenkins instance.
I just tested the whole suggestion working using our:

forked platform management
forked chef adop extension
adop-docker-compose - A project that has a custom jenkins-slave container with ansible installed.
Jenkins-slave

Terminologies

Playbook - the Ansible file or script. It's like the cookbook of Chef.
Ansible CF module - cloudformation_module

Requirements to accomplish this:

Update the jenkins-slave  - yum install ansible python-boto in the jenkins-slave.
Update the platform extension specification project - Have a standardized name for the playbook like cf-runner.yml. Users will place the cf-runner.yml in the aws directory.

service
|__aws
   |__ service.template
   |__ cf-runner.yml

Users customized their own cf-runner.yml - so they can add their own Cloudformation parameters if they have to.  The example below runs the adop chef server extension's service.template using the ansible cloudformation module.

- hosts: localhost
  connection: local
  tasks:
    - name: Launch cloudformation for Chef extension
      cloudformation:
        stack_name: "ansible-cloudformation"
        state: "present"
        region: "{{ lookup('env','AWS_REGION') }}"
        disable_rollback: true
        template: "service.template"
        # User defined parameters
        template_parameters:
          KeyName: "{{ lookup('env','AWS_KEYPAIR') }}"
          InstanceType: "t2.large"
          EnvironmentName: "ChefServer"
          EnvironmentSubnet: "{{ lookup('env','AWS_SUBNET_ID') }}"
          VPCId: "{{ lookup('env','AWS_VPC_ID') }}"
          InboundCIDR: "<IP_ADDRESS>/0"
        tags:
          Stack: "chef-stack"

      # Register the cloudformation output facts into a variable
      register: cf_out

    # Print the output to stdout
    - debug: var=cf_out

    - name: Copy all the output facts into a file
      copy: content={{ cf_out }} dest=./cf_out.json

    # Get the important data and create a file for it.
    # In this example, we want to save the ip address of the provisioned IP so we can parse it to a variable
    # for modifying the configuration file that will be copied over the ADOP nginx proxy container.
    - name: Copy the ip address attribute of the output facts into a file
      copy: content={{ cf_out["stack_outputs"]["EC2InstancePrivateIp"] }} dest=./instance_ip.txt

Jenkins execute shell now will have something like this..

# Provision any EC2 instances in the AWS folder
if [ -d ${WORKSPACE}/service/aws ]; then

    if [ -f ${WORKSPACE}/service/aws/service.template ] && [ -f ${WORKSPACE}/service/aws/cf-runner.yml ]; then

        echo "#######################################"
        echo "Adding EC2 platform extension on AWS..."

        cd service/aws/
        ansible-playbook cf-runner.yml
        if [[ $? -gt 0 ]]; then exit 1; fi
        cd -

        if [ -f ${WORKSPACE}/service/aws/ec2-extension.conf ]; then

            echo "#######################################"
            echo "Adding EC2 instance to NGINX config using xip.io..."

            export SERVICE_NAME="EC2-Service-Extension-${BUILD_NUMBER}"
            cp ${WORKSPACE}/service/aws/ec2-extension.conf ec2-extension.conf
            NODE_IP=$(cat ${WORKSPACE}/service/aws/instance_ip.txt)

            ## Add nginx configuration
            sed -i "s/###EC2_SERVICE_NAME###/${SERVICE_NAME}/" ec2-extension.conf
            sed -i "s/###EC2_HOST_IP###/${NODE_IP}/" ec2-extension.conf
            docker cp ec2-extension.conf proxy:/etc/nginx/sites-enabled/${SERVICE_NAME}.conf

            ## Reload nginx
            docker exec proxy /usr/sbin/nginx -s reload

            ## Don't stop jenkins to exit with error if nginx reload has failed.
            if [[ $? -gt 0 ]]; then
              echo "An error has been encountered while reloading nginx. There might be some upstreams that are not reachable."
              echo "Please run 'docker exec proxy /usr/sbin/nginx -s reload' to debug nginx."
            fi

            echo "You can check that your EC2 instance has been succesfully proxied by accessing the following URL: ${SERVICE_NAME}.${PUBLIC_IP}.xip.io"
        else
            echo "INFO: /service/aws/ec2-extension.conf not found"
        fi

    else
        echo "INFO: /service/aws/service.template or /service/aws/cf-runner.yml not found."
    fi
fi

Console Output

Interesting stuff!
Could this be a way to launch new services via Ansible instead of specifically just being for AWS? i.e. it'd fall under service/ansible in the platform extension (and therefore be an addition to the spec).
I think I'd want this method to exist alongside the existing one instead of replacing it, just for compatibility, too.

@nickdgriffin the new way to launch new services, hopefully yes, there is a lot of ansible module that we can use to faciliate all of the extensions. E.g, for docker, the container module or line in file to modify nginx configurations..
For making it compatible with the old style, we can create a wrapper script for all of those old aws cli commands/scripts to make  the execute shell as neat as possible.

I think in that case I could see this as "service/ansible" then instead of changing the existing "service/aws". I like the idea of having different ways to launch the same thing if it gives people the option of choosing their orchestration/deployment tool depending on what they are familiar with and what their use case is.

@nickdgriffin I agree with the "service/ansible" idea! :)
FYI - @paoloxide @arcyteodoroacn

@nickdgriffin agree, as tomorrow someone could try to change on Chef or Puppet or whatever tool :)
@bzon Would you have a time to implement it as "service/ansible"? Feel free to ask for any assistance on this, happy to help.

@anton-kasperovich I would be happy to.
Let's plan this @paoloxide @arcyteodoroacn

I'm closing this issue at least for the meantime because I won't be able to look it because of a long term project priority commitments.


Update version of k8s client

Fixes https://github.com/dotnet/tye/issues/942.
Let's see if tests pass 😄

First resolve the package conflict 😄

@JunTaoLuo quick approval here?


[$1000] mweb- Chat - When clicking on the emoji, the cursor does not appear.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open any chat
Press the emoji picker button on main composer
Pick any emoji
Verify the emoji added to the main composer without additional space
Send a message
Edit the message
Press the emoji picker button on edit composer
Pick any emoji
Verify the emoji added to the edit composer without additional space
Verify that no errors appear in the JS console

Expected Result:
When clicking on the emoji, the cursor appears.
Actual Result:
When clicking on the emoji, the cursor does not appear.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/233796292-102f5be2-a550-458e-998a-2c64255224ea.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01df09511a4ee1d073
Upwork Job ID:<PHONE_NUMBER>994423296
Last Price Increase: 2023-04-24

Triggered auto assignment to @kevinksullivan (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Current assignee @kevinksullivan is eligible for the External assigner, not assigning anyone new.

Not overdue

I have tested this, I think the issue is just that we dont focus the composed on mWeb Safari after doing this.

updated the OP

Nice, I agree, added it there and closing this one


Script to launch selenium with php

$selenium = new Selenium(array('browser' => 'chrome', 'insider' => false));
$selenium->run();

please update readme too with an example.
Thanks

Thank you @compojoom @810


use variant replace StatusOr for getJsonStringFromMessage

Signed-off-by: wbpcode<EMAIL_ADDRESS>Commit Message: use variant replace StatusOr for getJsonStringFromMessage
Additional Description: Using absl::variant replace StatusOr as the return type of getJsonStringMessage. Because absl::variant is more friendly to the move semantics. By this way, we can reduce uncessary string copies.
Risk Level: Low.
Testing: Unit.
Docs Changes: N/A.
Release Notes: N/A.
Platform Specific Features: N/A.

/retest

Hi @wbpcode -- the perf improvement seems like a reasonable motivation. Do you have a microbenchmark? absl::variant may have its own overhead :)
I'm also going to check with the absl team to see if there are plans to even out this gap with absl::StatusOr, as StatusOr usage seems easier to read.
Related: would you consider making a helper class that had a similar API to StatusOr, but wrapped absl::variant underneath for the move semantics, so this would be more of a drop-in at call-sites?

I have no a benchmark result now. I just discover this problem by chance. So I create this simple PR as a quick fix. But I do not consider the overhead of variant self.
It would be easy to create a move friendly StatusOr based on the ProtobufUtil::StatusOr.

getJsonStringFromMessageOrErrorSpeedTest/0/5120    90108909 ns     90109301 ns            7
getJsonStringFromMessageOrErrorSpeedTest/0/10240  182810518 ns    182808230 ns            4
getJsonStringFromMessageOrErrorSpeedTest/0/40960  718936721 ns    718937866 ns            1
getJsonStringFromMessageOrErrorSpeedTest/0/81920<PHONE_NUMBER> ns<PHONE_NUMBER> ns            1
getJsonStringFromMessageOrErrorSpeedTest/1/5120    87497827 ns     87498099 ns            8
getJsonStringFromMessageOrErrorSpeedTest/1/10240  176938797 ns    176939029 ns            4
getJsonStringFromMessageOrErrorSpeedTest/1/40960  677867321 ns    677848951 ns            1
getJsonStringFromMessageOrErrorSpeedTest/1/81920<PHONE_NUMBER> ns<PHONE_NUMBER> ns            1

Some simple benchmark results.  getJsonStringFromMessageOrErrorSpeedTest/0/* use the old ProtobufUtil::StatusOr. getJsonStringFromMessageOrErrorSpeedTest/1/* use the new ProtoStatusOr.
The new ProtoStatusOr does impove the performance. But the improvement is very minor and hard to get stable result. This is because that the main overhead of getJsonStringFromMessage is come from the proto serialization.

I didn't add the benchmark to the PR. Our test set is getting bigger and this benchmark test doesn't seem to be of much value at the moment.
This PR can bring a slight performance improvement, and there is no negative impact. But considering that there is really not much improvement, It is up to the community to decide whether to merge this PR.

Yeah I guess I'd push back on this as it's a bunch of new code for a relatively minor speed improvement.
I'd guess that StatusOr being moveable doesn't make much difference if you are not generating error strings. And mostly I assume we are less concerned with performance for errors.
Instead it might be worthwhile to open an issue in https://github.com/protocolbuffers/protobuf/issues and ask them to make their StatusOr moveable for the benefit of all :)

Get it. Thanks @jmarantz for your kind reviews. 🌷 I will close this PR and create new issue to the protobuf.

https://github.com/protocolbuffers/protobuf/issues/9827


Rom sensor ctrl

Add a sensor_ctrl driver to configure the peripheral from OTP.
Configure sensor_ctrl during ROM bootup.
Addresses: #23150, #21871.

The CI test failure is unrelated to this test:
t="//sw/device/tests:spi_device_sleep_test_fpga_cw310_rom_with_fake_keys"
# On this branch and on master:
BITSTREAM=1459afe788072bd95e185225f2b0179b7aed9065 bazel test $t  # Fail
BITSTREAM=73c42574957fad3c99aee16cce2bfcbb720e0d04 bazel test $t  # Pass


Character Encoding

Fixes #11
modify the ColumnNameSubheader and bytesToString functions to make proper use of ability to read different character encodings for sas7bdat files
remove SessionEncoding from metadata, this appears to not be reliable and a better definition of these bytes is required
adds @FriedEgg to POM as contributor
increments version to 2.0.3
adjusts the release notes as DBCS only now supported and not previously

Thank you! We released version 2.0.2.


uncrustify: 0.71.0 -> 0.72.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/uncrustify/versions.
meta.description for uncrustify is: "Source code beautifier for C, C++, C#, ObjectiveC, D, Java, Pawn and VALA"
meta.homepage for uncrustify is: "http://uncrustify.sourceforge.net/"
meta.changelog for uncrustify is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.72.0 with grep in /nix/store/8765r4mga62c4njbx9p6i39kxy1r6ar6-uncrustify-0.72.0

directory tree listing: https://gist.github.com/0e3586c2aeb05f2ee9bb5d7f411411c8

du listing: https://gist.github.com/cd671a95d7817dc4036a0fec88133d7d

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
uncrustify

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8765r4mga62c4njbx9p6i39kxy1r6ar6-uncrustify-0.72.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A uncrustify https://github.com/r-ryantm/nixpkgs/archive/1a78afcabf1b33392840c898e56f9e4e5354f3e0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8765r4mga62c4njbx9p6i39kxy1r6ar6-uncrustify-0.72.0
ls -la /nix/store/8765r4mga62c4njbx9p6i39kxy1r6ar6-uncrustify-0.72.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

uncrustify

Maintainer pings
cc @bjornfor for testing.

Result of nixpkgs-review pr 115624 run on x86_64-linux 1

1 package built:

uncrustify


invalid syntax to run coverage with nosetests

the way virtualenv of pyenv prevents me from using coverage + nosetests. Please check out the following steps to reproduce the problem:
$ mkdir py-virtualenv
$ cd py-virtualenv/
$ pyenv virtualenv test
New python executable in /home/vagrant/.pyenv/versions/2.7.10/envs/test/bin/python2.7
Also creating executable in /home/vagrant/.pyenv/versions/2.7.10/envs/test/bin/python
Installing setuptools, pip, wheel...done.
Ignoring indexes: https://pypi.python.org/simple
Requirement already satisfied (use --upgrade to upgrade): setuptools in /home/vagrant/.pyenv/versions/2.7.10/envs/test/lib/python2.7/site-packages
Requirement already satisfied (use --upgrade to upgrade): pip in /home/vagrant/.pyenv/versions/2.7.10/envs/test/lib/python2.7/site-packages
$ pyenv activate test
pyenv-virtualenv: deactivate
pyenv-virtualenv: activate test
pyenv-virtualenv: prompt changing will be removed from future release. configure `export PYENV_VIRTUALENV_DISABLE_PROMPT=1' to simulate the behavior.
(test) $ pip install nosetests
Collecting nosetests
  Could not find a version that satisfies the requirement nosetests (from versions: )
No matching distribution found for nosetests
You are using pip version 7.1.2, however version 8.0.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
(test) $ pip install nose
Collecting nose
  Using cached nose-1.3.7-py2-none-any.whl
Installing collected packages: nose
Successfully installed nose-1.3.7
You are using pip version 7.1.2, however version 8.0.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
(test) $ pip install coverage
Collecting coverage
Installing collected packages: coverage
Successfully installed coverage-4.0.3
You are using pip version 7.1.2, however version 8.0.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
(test) $ nosetests .

----------------------------------------------------------------------
Ran 0 tests in 0.002s

OK
(test) $ coverage run --branch --source=. `which nosetests` -v
Coverage.py warning: No data was collected.
Traceback (most recent call last):
  File "/home/vagrant/.pyenv/versions/test/bin/coverage", line 11, in <module>
    sys.exit(main())
  File "/home/vagrant/.pyenv/versions/2.7.10/envs/test/lib/python2.7/site-packages/coverage/cmdline.py", line 741, in main
    status = CoverageScript().command_line(argv)
  File "/home/vagrant/.pyenv/versions/2.7.10/envs/test/lib/python2.7/site-packages/coverage/cmdline.py", line 481, in command_line
    return self.do_run(options, args)
  File "/home/vagrant/.pyenv/versions/2.7.10/envs/test/lib/python2.7/site-packages/coverage/cmdline.py", line 620, in do_run
    self.run_python_file(filename, args)
  File "/home/vagrant/.pyenv/versions/2.7.10/envs/test/lib/python2.7/site-packages/coverage/execfile.py", line 173, in run_python_file
    code = make_code_from_py(filename)
  File "/home/vagrant/.pyenv/versions/2.7.10/envs/test/lib/python2.7/site-packages/coverage/execfile.py", line 212, in make_code_from_py
    code = compile_unicode(source, filename, "exec")
  File "/home/vagrant/.pyenv/versions/2.7.10/envs/test/lib/python2.7/site-packages/coverage/phystokens.py", line 287, in compile_unicode
    code = compile(source, filename, mode)
  File "/home/vagrant/.pyenv/shims/nosetests", line 3
    [ -n "$PYENV_DEBUG" ] && set -x
                      ^
SyntaxError: invalid syntax
(test) $

As far as I know, coverage expects nosetests to be python script instead of bash script shimmed by pyenv (as specified from https://github.com/yyuu/pyenv/wiki/Understanding-binstubs#pyenv
I'm not sure if this is the problem from pyenv or coverage.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/30217767-invalid-syntax-to-run-coverage-with-nosetests?utm_campaign=plugin&utm_content=tracker%2F335155&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F335155&utm_medium=issues&utm_source=github).

related: https://bitbucket.org/ned/coveragepy/issues/174/make-it-easier-to-run-python-scripts-under

I'm no longer using nosetests. Need more information.

we're going to use Docker for working with different Python versions. We don't need this support any more. Thank you.


Updating Destination Host

The destination host lead to Microsoft DevTunnels Website, should have been lead to the actual devtunnels domain. Updated if from the following sigma rule.
https://github.com/SigmaHQ/sigma/blob/master/rules/windows/dns_query/dns_query_win_devtunnels_communication.yml

Thanks for the PR @deFr0ggy
DevTunnels isn't an RMM and it's corresponding  will be deleted along with others that were added accidentally.
Also keep in mind the detections are generated automatically from the YAMLs. So as a general thing in case of issues in the Sigma rules, try to fix it in the YAML source for it to propagate.
Thanks.


Add typing to RadioButton component

I'm not sure how the labelStyle props can be used so the comment doc for that prop may be wrong.

@nhanaa @ncovercash  The proptype for labelStyle is object - is there a good library that can generically type CSS style objects?

@JohnC-80 thanks for the insight; @nhanaa you should be able to reference the type CSSProperties from react


NO MERGE, TESTING- Initial commit for SQ scans.

NO MERGE
SonarQube dotnet scan.

Needs more work. We need to compile locally without storage dependencies first, and then test.


RFC: Graphcache support changing __typename component of the cache

Summary
The format for keys in the graphcache index is <__typename>:<id>. The id component is a bit ambiguous, as different apis will use different identifiers. Consequently, graphcache allows specifying a keys field on the options object that allows users to override how that id portion is computed for the key.
However, there is no way to do the same for the __typename component. This makes sense -- it is a standard part of the GQL api and there's little reason to override.
Unfortunately, the current caching model has no internal notion of inheritance (or unions). What this means is that if we have a resolver that returns Car whose two inhabitants are FourWheeler and TwoWheeler, a FourWheeler with id 123 will be cached as FourWheeler:123.
Now let's say we have a mutation that unceremoniously rips the back two wheels off our car and returns it to the frontend. The id remains 123 and the entity is still a Car, but the underlying __typename would change to TwoWheeler. Graphcache will see this and insert a brand new cache entry called TwoWheeler:123, and not update the status of FourWheeler:123.
In a perfect world, this feature request would be to include better handling of that (maybe by generating multiple cache keys for each type and then having a separate type of link that joins them? not sure -- out of scope here).  However, there is a much simpler solution that handles the 95 percentile of cases (at least, that I personally encounter).
Proposed Solution
Even if graphcache isn't really able to understand what the relationship between elements of a common inheritance structure is, the programmer does. In the same way that we are allowed to use the keys option on graphcache to re-key (really re-id) a record, we could also expose a typenames option that let's us "re-type" a record.
I don't know if this is a perfect solution. It really only works if cache keys are write-only (i.e. never parsed). As far as I can tell, this is the case, but I only gave a cursory glance.
For what it's worth, I've implemented this naive solution for our codebase, and so far it seems to be working exactly as expected. However I would love to hear the thoughts of the maintainers on whether this is a feasible (and safe!) solution.
Requirements
For me personally, the primary requirement is that I can use mutations that safely re-type an entity of a given inheritance chain (or union) and have the cache keep the link between it and previous versions.
POC
https://gist.github.com/zwade/64b96dea8b483720be32eba51713a8bc

We've got some issues/discussions floating around with this explanation, I believe, but to keep it short, I think, it's very unlikely for us to implement this, partly because it pushes complexity in an unreasonable direction for Graphcache to not be able to make assumptions (or rather, even less assumptions) about the data and data structure, which then in turn blocks other features,
and, in part; because it encourages patterns that I (at least personally) wouldn't want to encourage, in terms of the mental model of how to model and design a schema.
To put it short, GraphQL actually also doesn't have a notion of inheritance, and rather it explicitly has a model of interfaces that can be chained. This is actually a bigger different than it seems. While unions and interfaces are often combined to model inheritance, I think it often leaves out some convenient affordances that GraphQL provides.
Let's suppose, we've got an interface CarLike and two implementors, TwoWheeled and FourWheeled. FourWheeled and TwoWheeled are further an implementor of TwoWheeledLike.
A field could now either return FourWheeled, an interface, or a union of concrete object types.
The existence here of concrete object types is important. To put it differently, ultimately, a result must of course always resolve to concrete object types, i.e. one ambiguous object type.
However, let's say you would enable FourWheeled to turn into TwoWheeled. Now, in your cache implementation, you're basically "downleveling" the type and all references to it can legally change to TwoWheeled.
However, what if the type turns from TwoWheeled into FourWheeled?
This is still legal. After all, we can "complete" the type to fit the "extended" object.
If we allowed this then we'd also have to assume that either:

all references to the "upleveled" FourWheeled type are now invalid where TwoWheeled is used and have to be invalidated
or, all references are still kept but there's no guarantee they'll match types, so matching in the future may fail
or, all references are deleted no matter how they change, which brings you back to the problem you describe

In short, in GraphQL’s type system of interfaces and unions being returned by fields, even if everyone included unambiguous schema information on the client-side (which we wouldn't ask people to have to do of course 😅 hence, it's optional) there's still some ambiguity in wanting objects to "change" type.
This is partially solved by letting objects have a globally unique ID (like Relay enforces). But that's a requirement for the API, and, I suppose, we could allow this at least in the future, since that's a smaller change and switch.
However, the above limitations still apply. There's no way of knowing whether this is legal, and there's no way for a user to specify that only some relations are affected if an object changes types.
How does one get around this? — and this is something that I think is underrated.
I'd recommend Edge types for this. An Edge type can have a unique type and can reference an interface on the server-side schema. However, what it also allows us to do is to change what's referenced globally, wherever this Edge is used.
So, let's say you want an object to "change type". If that object is already written, you can switch every matching edge over to this new type, especially if it has a matching id field.
This is such a subtle but powerful change in schema modelling, because you detach your constraint of how types can change "one away". Now, instead of pointing at TwoWheeledLike, for example, your schema could use a TwoWheeledLikeEdge type and a FourWheeledLikeEdge type. If a TwoWheeled object is replaced or turns into a FourWheeled like object, you can switch the matching TwoWheeledLikeEdge over without having to actually care about the "legality" of it, and vice versa.
So, in short, I often recommend people to look into edge types when getting into scenarios where links between objects "switch" between different types, or where the relation between objects needs to be addressed globally

Hi! Thank you so much for the in-depth response. I appreciate that the team has spent a lot of time thinking about this.
If you're able, I'd like to follow up to make sure I understand the nature of the Edge suggestion.
Let's say for the sake of this discussion that we have the following (naive) GQL schema
interface Car {
  id: ID!
  color: String!
  numWheels: Int!
}

type TwoWheeler implements Car { ... }
type FourWheeler implements Car { ... }

type query {
  myCars: [Car!]!
}

type mutation {
  truncateCar(id: ID!): Car!
}

And then we convert it to an edge-oriented schema as such:
# `Car`, `TwoWheeler`, and `FourWheeler` unchanged

type CarEdge {
  id: ID! # Same as the node's id
  node: Car!
}

type query {
  myCars: [CarEdge!]!
}

type mutation {
  truncateCar(id: ID!): CarEdge!
}

I can understand how this would solve my direct problem -- even if the type of the Car changes as a result of truncateCar, the CarEdge would serve as a pointer to the car that can forcibly invalidate any query that returns it when the type changes.
In other words, switching to this model would allow for safe caching in the following sequence of queries (assuming I have one FourWheeler with id "1":
query getMyCars {
  myCars {
    id
    node {
      id
      color
      numWheels
    }
  }
}

# Some time later

mutation sliceAndDice {
  truncateCar(id: "1") {
    id
    node {
      id
      numWheels
    }
  }
}

Once the mutation is called, the linkage that previously existed from CarEdge:1 to FourWheeler:1 would be updated to TwoWheeler:1. The first query would recognize that its gone stale, and re-execute.
This is correct behavior, but it requires refetching all of the data from the query. In this particular example I've specifically only fetched data off of the Car interface. Since the edge in question is itself a Car edge, my desired outcome would be for the mutation to simply update the cached records in place.
However, since the caching key of the record changes, this seems like a non-starter (regardless of schema architecture). Is there an alternate handling of edges (or of the caching system) that bypasses this problem?
Thanks again so much for the help, and for your work with URQL!

Awesome, yeah this makes a lot of sense. I think you're right and that it would be worthwhile for us to reevaluate our schema to allow for more modular updates like the ones you're suggesting.
I am a bit curious however, if the pattern of "make the common data query-able on the edge," or the more generic form of "model inheritance as a multi-level object" is a common pattern? It's not one that I've come across, but I am also far less exposed to others' GQL schemas 😅
Otherwise thank you so much again for the help. For now I think i'll do a manual cache invalidation on the offending mutations (fortunately there aren't many), and then re-evaluate how we're modeling our schema moving forward!

Just chiming in here because I've come up against this issue, unfortunately it would be very hard for me to implement 'Edge' types on the Graphql provider that I am using. Thanks for your in depth explanation though.
The suggestion of either global IDs or cache enumeration would work for my use case with that latter I think be the more flexible of the two.


Another Lissajous Table contribution

I would like to submit my contribution for the Lissajous Table coding challenge.
I plan on adding the ability to change the number of circles shown on the screen!
I would love it if this would pass and the gh-page gets listed under the Coding Challenge!
Thanks in advance!

Thanks for sharing!


readSafeJSONString returns null

This code for decoding a token is directly taken from the demo on the website:
  var a = currentToken.split(".");
  var uHeader = b64utos(a[0]);
  var uClaim = b64utos(a[1]);
  var pHeader = KJUR.jws.JWS.readSafeJSONString(uHeader);
  var pClaim = KJUR.jws.JWS.readSafeJSONString(uClaim);

Problem here is that readSafeJSONString always returns null for a valid json (checked in json lint).  I am using it with
  'vendor/jsrsasign/jsrsasign-4.1.4-all-min.js',
  'vendor/jsjws/jws-3.0.min.js'

Any ideas?

Thank you for the report. I've fixed them in the latest 3.2.2 release. Thanks.


[WIP] Update response verification for ingress SDS test.

https://github.com/istio/istio/issues/16998

@JimmyCYJ: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
4ac9eaa74dd6ad0a201b3864ca40364920efce29
link
/test integ-new-install-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
4ac9eaa74dd6ad0a201b3864ca40364920efce29
link
/test integ-new-install-k8s-tests_istio

pilot-multicluster-e2e_istio
4ac9eaa74dd6ad0a201b3864ca40364920efce29
link
/test pilot-multicluster-e2e_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #18291


Fixing Git URL for bamboo_boy requirement

Submitting a change to the URL for the bamboo_boy module based on the discussion in #1245

:+1:
Looks good!

Thanks :)


add simple pyramid benchmark

This seems to be missing a run_benchmark.py to run it as part of pyperformance.


module should fail on unsupported platforms

This module should fail on any platform for which support is not expressed in metadata.json.

Possibility related to #410


min-widthis not changed for the formulas \begin{equation}...\end{equation}

For \begin{equation}...\end{equation} the size of the container is not changed regardless of the value containerWidth
\begin{equation}
\begin{aligned}
\mathbb{P}\left[X_{n+1}=i_{n+1} | B\right]=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{1}\right] \mathbb{P}\left[A_{1} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{2}\right] \mathbb{P}\left[A_{2} | B\right] \\=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=i_{n}\right\}\right] \mathbb{P}\left[X_{n}=i_{n} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=-i_{n}\right\}\right] \mathbb{P}\left[X_{n}=-i_{n} | B\right]
\end{aligned}
\end{equation}

But for this formula everything works
\begin{aligned}
\mathbb{P}\left[X_{n+1}=i_{n+1} | B\right]=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{1}\right] \mathbb{P}\left[A_{1} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{2}\right] \mathbb{P}\left[A_{2} | B\right] \\=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=i_{n}\right\}\right] \mathbb{P}\left[X_{n}=i_{n} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=-i_{n}\right\}\right] \mathbb{P}\left[X_{n}=-i_{n} | B\right]
\end{aligned}

I have also tried to set docTeX.options.OutputJax.minwidth = ..
But I see that this value is re attributed https://github.com/mathjax/MathJax-src/blob/master/ts/output/svg.ts#L265
Please, tell me how I can change the container's width in this case.

Thank you so much for your reply.
I use the tags: "ams" option. In the previous screenshot, the numbering is not visible since it is hiding.
I want to clarify in the example below, it turns out that the min-width for the formulas  \begin {equation} ... \end {equation} exceeds the width of the container
because of this, part of the formula is hidden.
Is there a way to reduce the min-width for the formulas \begin{equation} ... \end{equation}?

\begin{equation}
\begin{aligned}
\mathbb{P}\left[X_{n+1}=i_{n+1} | B\right]=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{1}\right] \mathbb{P}\left[A_{1} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{2}\right] \mathbb{P}\left[A_{2} | B\right] \\=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=i_{n}\right\}\right] \mathbb{P}\left[X_{n}=i_{n} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=-i_{n}\right\}\right] \mathbb{P}\left[X_{n}=-i_{n} | B\right]
\end{aligned}
\end{equation}

\begin{equation*}
\begin{aligned}
\mathbb{P}\left[X_{n+1}=i_{n+1} | B\right]=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{1}\right] \mathbb{P}\left[A_{1} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{2}\right] \mathbb{P}\left[A_{2} | B\right] \\=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=i_{n}\right\}\right] \mathbb{P}\left[X_{n}=i_{n} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=-i_{n}\right\}\right] \mathbb{P}\left[X_{n}=-i_{n} | B\right]
\end{aligned}
\end{equation*}

Equations that are too wide for the container do extend past the right-hand side of the container.  Automatic line breaking can help in some cases but (as mentioned) that is not yet ported to version 3, and it is not very effective with tables (which is how the aligned environment is implemented), so it would not be very helpful here even so.
As I mentioned, the minimum width is the actual minimum required size for the expression and its label (when the equation is centered), and it can not be reduced without damaging the rendering.  The second example you give doe not contradict that because it has no label.  If the container we a bit less wide, that one would extend beyond the right-hand side as well.
The only option to get a reduced is to reduce the size of the font surrounding the expression.  You could do that by adding a span or div that has CSS that reduces the font size.  E.g.,
<div style="font-size: 90%">
\begin{equation}
\begin{aligned}
\mathbb{P}\left[X_{n+1}=i_{n+1} | B\right]=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{1}\right] \mathbb{P}\left[A_{1} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap A_{2}\right] \mathbb{P}\left[A_{2} | B\right] \\=& \mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=i_{n}\right\}\right] \mathbb{P}\left[X_{n}=i_{n} | B\right] \\ &+\mathbb{P}\left[X_{n+1}=i_{n+1} | B \cap\left\{X_{n}=-i_{n}\right\}\right] \mathbb{P}\left[X_{n}=-i_{n} | B\right]
\end{aligned}
\end{equation}
</div>

but this has to be done by hand (MathJax doesn't have an automatic way to do that).
Alternatively, you could arrange for the div to have overflow: scroll so that it can be scrolled to the right or left to expose the rest of the equation.


How to integrate this library with parse server fileadapter

I would like to integrate parse server with this library. Could you please guide me regarding this.
I am referring to configuring parse file Adapters

I asked same question in parse server question link . I would like to know can integrate this library with  parse fileadapter.js . This feature will help many people who want host parse in bluemix.

@gowridev no, this library is solely for consuming the standard Object Storage service offering on bluemix


How to reference "team" from the JSON structure passed by practiceLobbyUpdate

Really sorry if this is a dumb question but I need to reference "members" from the JSON structure passed by practiceLobbyUpdate. The structure looks like this:
{"lobby_id":{"low":-4303139,"high":5681462,"unsigned":true},"me
mbers":[{"id":{"low":163589911,"high":17825793,"unsigned":true},"hero_id":null,"
team":4,"name":"BG-Bot","slot":0,"party_id":null,"meta_level":null,"meta_xp":nul ...
For lobby_id, I have already found a solution in the closed issues which works fine:
var parsed_lobby_id = lobby.lobby_id + "";
However, when I try to reference "members" with:
var parsed_members_id = lobby.members[0].id + "";
I get a: TypeError: Cannot read property '0' of undefined
What am I doing wrong?

Try doing a JSON.stringify() to see exactly what is happening. It's possible you are looking at the array when that value is not populated.
I'm going to hazard a guess that it would be lobby.members[0]['id'] or lobby['members'][0][''id] but I always screw up objects... leaving it to a game of trial and error.

"" + lobby.members[0].id should work normally. Try to put some prints in your code or step through it to see what happens

Here's my code:

  		Dota2.on('practiceLobbyUpdate', function (lobby) {

  			util.log(JSON.stringify(lobby));

  			var parsed_lobby_id = lobby.lobby_id + "";
  			util.log(parsed_lobby_id);

  			var parsed_members_nicks = "" + lobby.members[0].id;
  			util.log(parsed_members_nicks);

  		});

And here's the error.

E:\Node-Dota2\Node-Dota2\node-dota2-master\test\test.js:238
var parsed_members_nicks = "" + lobby.me
mbers[0].id;
    ^

TypeError: Cannot read property 'id' of undefined
at Dota2Client. (E:\Node-Dota2\Node-Dota2\node-dota2-master\test
test.js:238:54)
at emitOne (events.js:77:13)
at Dota2Client.emit (events.js:169:7)
at Dota2Client.handleSubscribedType (E:\Node-Dota2\Node-Dota2\node_modules\d
ota2\handlers\cache.js:18:18)

I suspect that the reason for this is that the 1st lobbyUpdate object directly after creating a lobby comes with "members" being empty:
e.g.:

{"lobby_id":{"low":513459630,"high":5681463,"unsigned":true},"m
embers":[],"left_members":[],"leader_id":{"low":163589911,"high":17825793,"unsig...

But I may be wrong, I'm really bad at coding :)
What do you guys think?

Well, yea... If it's not set it will crash unless you check for the value before using it.

if (lobby.members[0] != null) {
var parsed_members_nicks = "" + lobby.members[0].id;
util.log(parsed_members_nicks);
}
Now it works. I'm so dumb...
Thanks guys.


Debug failed when i am using react-geocode library.

I have updated this lib to v0.1.0. Please update to latest version.


boundarySupplementaryItems

if you change layout dynamically and those layouts have headers than the content offset of a collection after animation and correct drawing suddenly jumps to incorrect value.
do know how to resolve this issue or its normal behaviour:)?
as an example I've made small changes to SimpleGridViewController
override func createLayout() -> UICollectionViewLayout {
        let itemSize = NSCollectionLayoutSize(
            widthDimension: .fractionalWidth(gridItemSize.rawValue),
            heightDimension: .fractionalHeight(1.0)
        )
        let item = NSCollectionLayoutItem(layoutSize: itemSize)
        item.contentInsets = NSDirectionalEdgeInsets(top: 5, leading: 5, bottom: 5, trailing: 5)
        let groupSize = NSCollectionLayoutSize(
            widthDimension: .fractionalWidth(1.0),
            heightDimension: .fractionalWidth(gridItemSize.rawValue)
        )
        let group = NSCollectionLayoutGroup.horizontal(layoutSize: groupSize, subitems: [item])
        let section = NSCollectionLayoutSection(group: group)

        let headerSize = NSCollectionLayoutSize(widthDimension: .fractionalWidth(1.0), heightDimension: .estimated(44))
        let headerElement = NSCollectionLayoutBoundarySupplementaryItem(layoutSize: headerSize, elementKind: UICollectionView.elementKindSectionHeader, alignment: .top)
        section.boundarySupplementaryItems = [headerElement]

        let layout = UICollectionViewCompositionalLayout(section: section)
        return layout
    }

have found the reason of that issue:
.estimated(44) makes this mess
use absolute value : )

Great! Glad to hear!


Wemos Lolin32 & iPhone app lost connection

Hi, I'm trying example code of  “BLE_notify” on “Wemos-Lolin32” and simple BLE app of iPhone by core Bluetooth.
I can connect each other. But these are disconnected a few seconds later.
I also tried communicate with Android phone by “Light Blue” app.  It’s stable.
And I also use “Red Bear BLE NANO” with the simple BLE app of iPhone. It’ also stable.
Only ESP32 with iPhone situation is disconnect automatically.
Does anyone know some solution for this problem?

Hi, there is one thing that can help you with this issue.
https://github.com/asterics/esp32_mouse_keyboard/issues/4#issuecomment-398695065

You dont have to use all security options, from what ive found with help other users this should be enough, at least for ble hid application:
setAccessPermissions(ESP_GATT_PERM_READ_ENCRYPTED | ESP_GATT_PERM_WRITE_ENCRYPTED);
Also there can be some connection interval requirements (i dont have iPhone brick so i cant test it).

@chegewara Hello I've read most code of this repo and most your discussion about connection with iOS, and checked every place I can call setAccessPermissions but it just doesn't work with my iPhone6. The connection was successful but nothing happened when I send key strokes. I guess the reason is here  chapter 9.2. I can't find any document about esp32 using SSP yet, I'll try the password method and tell you the result.

@wxjcug If you have read this issue https://github.com/asterics/esp32_mouse_keyboard/issues/4 then you can see that all users confirmed it is working now with this solution. Last post is confirmation that iPhone 6S is working too.

@Weijian-Espressif Thank you for that, I've switched to the ble_hid_device_demo and it works very well with my iPhone.

I will try to borrow iPhone and will work on this library, but its not possible before next week.

@chegewara Thank you very much!

Hi @SuzukiYushin
according to this document its important to setup properly advertising data (10.4-10.6). I know there is no issue with discovering and connecting esp32, but try to change BLEAdvertising.cpp with this code https://gist.github.com/chegewara/192d3a4b267592c828f2a0d25f9b8eb0
Unfortunately i dont have access to iPhone(only 30 minutes today and maybe some time tomorrow).
Also change your code according to this:
BLEDevice::init("MyESP");

// Create the BLE Server
pServer = BLEDevice::createServer();
BLEAdvertising* pAdvertising = pServer->getAdvertising(); /// <--- call this right after createServer()
pAdvertising->setMinInterval(0x100);   //try also 0x20
pAdvertising->setMaxInterval(0x100);   // try also 0x20

@SuzukiYushin Im sorry, but i could not make it work yet. Good news is that code based on my hid library which is in this repo works I tested it personally on iPhone):
https://github.com/asterics/esp32_mouse_keyboard
That means i can find issue and fix it, but dont know when, because i dont have access to iPhone anymore.

@chegewara Thank you very much.
Do you mean replace ESP32 libraries of Arduino to your "components/nkolban_BLE" folder?

@SuzukiYushin I just figured it out, this is not hid device code. What app on iPhone are you using to connect?

@chegewara Thank you for quick response.  I'm using own app that most simple code of core bluetooth. Also using LightBlue.

Is there any chance that you could try to build app with esp-idf instead of arduino? Arduino-esp32 is slightly behind esp-idf and this can be core bt stack issue.

@chegewara OK. I will try.

@chegewara I tried with LightBlue. It was disconnected as soon as I connected.

@SuzukiYushin please test our gatt_security_demo in ESP-IDF, it works well with iPhones.

FYI I had difficulties keeping a BLE connection until I upgraded to esp-idf v3.1 at minimum.    That helped me save time by not trying to debug something that was already fixed.
If arduino integration is lagging, one idea is to do the next most meaningful thing in your life and come back to Arduino BLE later.

@pctj101 I see. I think it's better way too.

@Weijian-Espressif Thanks. I will try.

@SuzukiYushin Could you help share if any updates for this issue? Thanks.

@Alvin1Zhang I think this issue can be closed, since its problem with arduino and 3rd party library (Kolban's ble library). We can raise new issue in Kolban's repo or esp32-arduino.

Thanks.


Plotting into dev

Small changes:

Fixed issue where if there is no entry for the current month, the previous month wouldn't show on the Running Totals monthly plot
Added cron job for populate_db to django-crontab
Changed font for the Ploutos navbar brand :)
Updated README (to be completed when everything merged so can add screenshots)
Updated requirements.txt
Formatting of dnanexus_queries.py

Hello @rklocke! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file Ploutos/scripts/DNAnexus_queries.py:

Line 137:40: E231 missing whitespace after ':'
Line 420:9: E122 continuation line missing indentation or outdented
Line 429:9: E122 continuation line missing indentation or outdented
Line 797:9: E128 continuation line under-indented for visual indent
Line 799:9: E128 continuation line under-indented for visual indent
Line 1282:71: E502 the backslash is redundant between brackets
Line 1325:71: E502 the backslash is redundant between brackets


Add parameter in fluentd parser for remaining another field even reserve_data is false

Is your feature request related to a problem? Please describe.
We can use the fluentd parser to parse a field to JSON and also we could set the reserve_data as false to remove all other fields if the parsing success. However, we might need to create some exception to remain another field for other purpose.
In my scenario, I am using the gsub function in record_modifier to create a new field "taskDef", then use the parser to parse the filed "log" to JSON. If I set the reserve_data as false, it will delete my new created field "taskDef". That's why I would like to request for a new feature request.
<filter **firelens**>
  @type record_modifier
  <record>
    taskDef ${record["ecs_task_definition"].gsub(/:.*/, '')}
  </record>
</filter>

<filter **firelens**>
  @type parser
  key_name log
  reserve_data false
  emit_invalid_record_to_error false
  <parse>
  @type json
  </parse>
</filter>

Describe the solution you'd like
I would like to add a new parameter to add exception for another filed. For example "exception_field taskDef", it will let the parser not remove the taskDef even the reserve_data is false.
Describe alternatives you've considered
In my scenario, I have tried to use the parser first, and then use the record_modifier gsub.
However, the "reserve_data false" will remove all other the field but the "log" field, then the gsub function will return null error as it is looking at the field "ecs_task_definition"
<filter **firelens**>
  @type parser
  key_name log
  reserve_data false
  emit_invalid_record_to_error false
  <parse>
  @type json
  </parse>
</filter>

<filter **firelens**>
  @type record_modifier
  <record>
    taskDef ${record["ecs_task_definition"].gsub(/:.*/, '')}
  </record>
</filter>

Additional context
No response

@chikinchoi I'm not 100% sure but I think you can set reserve_data true and add remove_key_name_field true to the parse to accomplish what you want. As I understand the parse doc page here, reserve_data should only determine if the field in key_name should be kept after successful parse (in your case log) so I'm not sure I understand whats going wrong.
Also, I think you have remove_keys and whitelist_keys options available so you could try something like parse json -> add taskDef -> drop/allow specific fields

@chikinchoi Please let us know whether @renegaderyu's idea satisfies your requirements.

@ashie @renegaderyu 's idea is good. Thank you very much!

The issue seems resovled.


Enable TravisCI tests

This is a Linux only change. It affects mostly just the configuration of TravisCI builds. Also included is the ability to compile Google Test locally to the project build folder. TravisCI builds are now running tests, and passing. As such, I'm going to go ahead and merge this.


MGMT-19174: Add rpms-signature-scan to Konflux pipeline

Taks will become mandatory to release, starting November 1st

/lgtm


Update Scrawl-canvas to latest version 8.10.1

Updating to latest version of SC.
Interesting to compare libraries across browsers - it might just be my local machine (MacBook Pro circa 2019) but the differences on latest versions of browsers (on 8k boxes) seem surprising:

Library
Chrome
Firefox
Safari

Pixi
60
43
24

Scrawl-canvas
51
60
32

P5
15
4
34

Mesh
47
30
18

ZRender
13
4
22

Two
23
21
16

Konva
23
7
16

CanvasKit
17
19
19

Paper
16
6
14

Easel
11
4
22

Pencil
12
3
19

Pts
12
4
11

Fabric
9
4
7

SVG
10
7
8

Three
8
6
4

DOM
12
1
9

Thank you!
I'd like to add the table to the ReadMe file (and mention you there) if that's ok with you

By all means feel free to add the table to the readme. It may encourage users and maintainers of other canvas libraries to review the code for their respective tests to see if they can identify/implement faster (more efficient) solutions, update the library dependencies to latest versions, etc.
Best wishes,
Rik


Uncaught Error: "Volume" must be an number between 0.0 and 1.0

i'm getting the same error someone else had with this library, i tried the same soluction (modifying the code in the build) and yet it did not work, i'm a bit confused because i did EXACTLY the same as the example

@PYTERODACTYL Sorry for the late response. Can you post a snippet of your code?


RPC call got an EOF error after shutdown the service

When we use this code( https://github.com/micro/go-micro/blob/master/function.go#L55L57) tells the server to wait for requests to finish before exiting, but when all sync.WaitGroup is done.

server and client conn't finish close socket
client con't refresh server nodes

That cause some error below when the server ends:
2018/10/10 18:20:04 error, {"id":"go.micro.client.transport","code":500,"detail":"EOF","status":"Internal Server Error"}
2018/10/10 18:20:04 error, {"id":"go.micro.client.transport","code":500,"detail":"EOF","status":"Internal Server Error"}
2018/10/10 18:20:04 error, {"id":"go.micro.client","code":500,"detail":"connection error: dial tcp <IP_ADDRESS>:53306: connect: connection refused","status":"Internal Server Error"}
2018/10/10 18:20:04 error, {"id":"go.micro.client","code":500,"detail":"connection error: dial tcp <IP_ADDRESS>:53306: connect: connection refused","status":"Internal Server Error"}

And I add a sleep time to fix it.
https://github.com/imneov/go-micro/compare/master...imneov:patch-1

You're seeing this error due to a connection being closed by the server when the client is maintaining a connection pool and because other connection attempts are being made because of node caching by the selector. A 1 second sleep is a hack to solve the problem but ideally we should just be able to deal with these errors at a higher level e.g with retries.

No response in 10 days. Closing.


Documentation about 1.3.7 features

Codecov Report

Merging #230 into master will decrease coverage by 0.29%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master    #230     +/-   ##
========================================
- Coverage    75.7%   75.4%   -0.3%
========================================
  Files           5       5
  Lines         671     671
========================================
- Hits          508     506      -2
- Misses        141     145      +4
+ Partials       22      20      -2

Flag
Coverage Δ

#unittests
75.4% <ø> (-0.3%)
:arrow_down:

Impacted Files
Coverage Δ

selenoid.go
74.78% <0%> (-0.58%)
:arrow_down:

Continue to review full report at Codecov.

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


Default export not set to styled global

I'm trying to use styled-components in browser without any bundler, so I just copied the CDN link from the docs. It loads fine, but when I want to actually use it I get the message styled.<element> is not defined. I found out that when I use styled.default.<element> it works, however, this breaks my syntax highlighting and who knows else that I haven't run into yet.
I still have limited knowledge of ES6 modules, but it appears that the styled object is not properly exported as default and set as the styled global as used in the docs.
I made a screenshot of the FF Dev Edition console here ->

Sidenote: we use ReactJS.NET to transpile JSX to JS on the fly but given I get the same results directly from the browser console I do not think this is the culprit. Adding styled = styled.default; to the very top of the document works too, though obviously this is quite a terrrible hack.

This isn't really fixable since the library has both a default and named exports.

I don't have any issue with the workaround I'm using at the moment since we're making a strictly contained product so I'm actually fine with the situation as is. However, I do think the team needs to decide on what your recommendation in this situation is and, more importantly, reflect it in the docs.

Going to address this for 4.0


Pretrained Model

Hi, thanks for sharing , I wanted to test your model I would really appreciate if you could share the pretrained weights. thank you so much.
<EMAIL_ADDRESS>

Here is pretrained weights
https://drive.google.com/file/d/19bmuol6CbSqL3pj9SBzUL6UhrxC3XYbC/view

Here is pretrained weights
https://drive.google.com/file/d/19bmuol6CbSqL3pj9SBzUL6UhrxC3XYbC/view

I couldn't extract it, Probably during compression had an error. Can u compress and share it again, or u can upload pth file directly , Thank you too much

Here is pretrained weights
https://drive.google.com/file/d/19bmuol6CbSqL3pj9SBzUL6UhrxC3XYbC/view

I couldn't extract it, Probably during compression had an error. Can u compress and share it again, or u can upload pth file directly , Thank you too much

If i remember correctly, i misplaced .pth with .zip . You should able to load .zip file without extract it.

Here is pretrained weights
https://drive.google.com/file/d/19bmuol6CbSqL3pj9SBzUL6UhrxC3XYbC/view

I couldn't extract it, Probably during compression had an error. Can u compress and share it again, or u can upload pth file directly , Thank you too much

If i remember correctly, i misplaced .pth with .zip . You should able to load .zip file without extract it.

Sorry , i didnt realize, thanks


Could not resolve type id '' into a subtype of [simple type, class com.atlassian.confluence.api.model.people.Person]

Describe the bug
When I attempt to create a new page, I receive an HTTP 440, which is failing with a "Could not resolve type id '' into a subtype of [simple type, class com.atlassian.confluence.api.model.people.Person]" error.
I've been digging into this a bit. I suspect that this has something to do with my credentials being managed via an external identity provider (in my case Google).
To Reproduce
I'm running code which is identical to https://github.com/Virtomize/confluence-go-api/blob/master/examples/content/content.go#L39
Expected behavior
I'd expect the page to be created.
Release version
v1.2.7

See my PR, this was failing because the Confluence API did not like the empty export_view. I added omitempty to all of the optional Content fields required by the POST/create method.

PR merged


Support TLS Expiry alerts

This is an amazing project, I had it up and running within minutes of your post on Reddit, thank you so much!
It would be awesome if you could add TLS monitoring alongside the "uptime" monitoring.
Although LetsEncrypt does a great job of auto-renewing certs, it would be great to monitor the following metrics as well:

Days until certificate expires
Whether the certificate is valid or invalid

I've been bitten by this so many times in the past!

The ability to import a trusted root CA (for internal CA's) would be a welcome addition to the above.

Adding to this domain expiration would also be great.

The ability to import a trusted root CA (for internal CA's) would be a welcome addition to the above.

Hi I detailed how you can do this in https://github.com/louislam/uptime-kuma/issues/21#issuecomment-881757886 using an environment variable.

@tmarly nope, not wrong, this is working well in the latest version.

Is there a way to disable these alerts? I don't want to be notified when the certificate is going to expire in 14 days, I just need outage notifications.

See #844, config is currently not implemented yet.


WIP: 1-cockpit-11: [no-test] Update translations from Fedora Weblate

Update translations from Fedora Weblate

[ ] po-refresh

po-refresh in progress on 1-cockpit-11.
Log: https://logs.cockpit-project.org/logs/po-refresh-911-20200318-164949/