docs: reorganize developer docs

I think this might be a modest improvement in organization.

Just to clarify the main change is that the architecture, "adding an executor tutorial", and contributing guide pages are given their own folder. From a user's perspective, they are now hidden under one layer of indirection. The side bar reads "getting started guide", "api reference", and "developer docs" where before it read "getting started guide", "api reference", "contributors and developers", "programming guide", and "tutorial: adding an executor". I think it makes a bit more sense to be organized in the new way but I don't think it's a big deal.

It worked fine for me in my local build. I'll leave the PR up until next week though in case anyone else wants to comment.


Consider supporting TensorFlow/CNTK

Please consider providing a runtime to TensofFlow/CNTK
See TensorFlow repo issue here
https://github.com/tensorflow/tensorflow/issues/16745
See CNTK repo issue here
https://github.com/Microsoft/CNTK/issues/2918

Sorry, this isn't in our road map.
How would you use TensorFlow and Q# together?

Running tensor flow on a quantum computer/simulator


add initialHtml value as a property in the constructor object as initialValue

Version
2.5.2
Development Environment
Mac
Current Behavior
Currently the initialValue property in the object constructor gets the text in Markdown mode, The problem was that I was saving the content in html format, and when I wanted to edit a content with the editor again and set the value as the initial value, I had so many problems, now I'm using the initial value of empty string and then used the setHtml function, it would be nice to have initialHtml value as a property in the constructor object
 initialValue: this.initialValue,
      initialHtml: this.initialValue,

@masoudtahmasebi93 This is what we've been discussing and we decided not to add an option. If you want to set html as it is, you have to use setHtml() API.

@seonim-ryu may I see the argument to know why?


PromiseKit requires Swift libraries to be embedded in the binary

I have an Objective-C codebase. I would prefer not to include Swift libraries because of the impact on size of the app. Those few MBs don't hurt too much, but I'd rather not have them.
I don't know of any magical way to conditionally link Swift, but if there's one, it would be awesome.
A little bit clunkier but more straightforward solution would be adding a new framework, let's say PromiseKitObjC and then statically link it to the main PromiseKit. The outcome would be two frameworks where PromiseKit includes both Swift and Objective-C whereas PromiseKitObjC would only include the Objective-C version.
I know this adds some complexity, but it should be doable (and maintainable) if there's no Swift code included in the Objective-C implementation and if there are no plans to unify them/get rid of one of them.
I could do this if you find this suggestion useful.

The ObjC implementation is not ObjC it is Swift, so it is impossible not to require the swift runtime.
If you want a pure ObjC PromiseKit, we still maintain PromiseKit 1.x.

Oh, I should've checked. Thanks!


FR: Allow tapping Istio control plane traffic

Contact Details
<EMAIL_ADDRESS>Is your feature request related to a problem? Please describe.
I am working on Corsha's CAST project, which is an API security tool for analyzing Kubernetes API traffic for authentication vulnerabilities such as reused credentials, traffic from outside the service mesh (Istio) using Istio-signed certificates, etc.
https://github.com/corshatech/cast
We are interested in seeing both:

App-to-App traffic within an Istio service mesh
Istio’s control plane traffic such as CSRs

I found this closed issue to ensure that Istio is supported by Kubeshark, but there isn't much info in the ticket: https://github.com/kubeshark/kubeshark/issues/1398.
Running Istio's Bookinfo demo application, starting a Kubeshark tap (kubeshark tap --tls), then generating traffic to the app, I have successfully observed app-to-app traffic within Istio:

However, I have been unable to capture any Istio control plane traffic so far. (The Istio control plane manages and configures the proxies to route traffic, so I have had Kubeshark tapping traffic while the app is being set up, torn down, or otherwise modified, but with no success.)
I filtered out the health-checking and readiness-probe traffic by applying this filter in the Kubeshark UI:
!dns and request.path != "/-/ready" and request.path != "/healthz/ready" and request.path != "/stats/prometheus" and request.path != "/-/healthy" and request.path != "/ready" and request.path != "/health" and request.path != "/api/health" and request.path != "/metrics" and request.path != "/kiali/healthz" and request.path != "/api/v1/query_range" and request.path != "/app-health/loki/readyz" and dst.name != "jaeger-collector". Then, with Kubeshark already tapping all pods in all namespaces within the k8s cluster (kubeshark tap --tls), I spun up, modified, and tore down the demo application, all the while observing no traffic at all (other than the traffic from rendering the webpage from the demo app).

I would love to know if y'all have any suggestions for me on how to proceed.
Original Thread
No response
Describe the solution you'd like to see
I would like to know definitively whether this feature gap (the ability to observe Istio control plane traffic through Kubeshark) is known and/or intended. If not, I would love to hear any suggestions that y'all might have for me to successfully capture this traffic data.
Provide additional context
More details on our project and what we are overall trying to achieve: https://github.com/corshatech/cast#cast
Some info on how we use Kubeshark: https://github.com/corshatech/cast#kubeshark
Code of Conduct

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

@katieatcorsha thanks for your detailed note! Kubeshark deals with Envoy/Istio in a much different way than it does a cluster without Envoy.  We never tried to capture Istio control plane traffic. It doesn't mean that we can't, we simply didn't put any focus on that. I'd love to better understand your use case and see if we can address it. Feel free to reach out to me on Slack to continue this discussion.


Adding official deCONZ Discord url

As support option.

Hi @Mimiix,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


Improve TestAssertions

This fixes #274.
Changes:

added assert_email_sent/0 that asserts any email was sent
added refute_email_sent/0 (macro) that asserts no email was sent ("alias" of assert_no_email_sent/0)
added refute_email_sent/1 (macro)

Besides implementing the simple pattern matching with refute_email_sent(pattern) - which requires exact matching on inner data structures - I took the opportunity to also add a refute_email_sent(attributes) version that creates a pattern (a Map) that automatically converts single values to single element lists while formatting the addresses (to<EMAIL_ADDRESS>becomes to: [{"", "email@example.com"}]).
Some notes:

mix format changed several lines
added @specs
added a stricter pattern matching on assert_email_not_sent/1 (since it pins the email variable, it only works with actual Email structs - any other value can be a false positive - so it seems clearer if it accepts only Email structs)
added more test cases for assert_no_email_sent/0

Didn't know that refute_x actually return false when success. Learned something new.

Thanks again @13k !

No problem, thanks for the work on swoosh :)
refute_receive specifically returns false. I'm not sure about others. Looks like refute returns whatever the expression evaluated to.


How to send Ecto Metrics

Hi Folks,
I am able use statix for http metrics but I wanted to know if there is a way I can configure it to  capture ecto metrics as well.
Thanks

Hi @lexmag,
Any update on this one please?


 The installation of LightGBM was run (apparently) successful, but so far I never could fit a single model. My R always crashed. How can I figure out the likely cause and fix it?

Reproducible examples
require(lightgbm)
require(methods)

# We load in the agaricus dataset
# In this example, we are aiming to predict whether a mushroom is edible
data(agaricus.train, package = "lightgbm")
data(agaricus.test, package = "lightgbm")
train <- agaricus.train
test <- agaricus.test

# The loaded data is stored in sparseMatrix, and label is a numeric vector in {0,1}
class(train$label)
class(train$data)

print("Training lightgbm with sparseMatrix")
bst <- lightgbm(data = train$data,
                label = train$label,
                num_leaves = 4,
                learning_rate = 1,
                nrounds = 2,
                objective = "binary")

Thanks for reporting this! I can help work through it. I've never observed this before with the R package, but we have seen it on the Python side (see Question 10 in the LightGBM FAQs).
Are you using gcc to compile LightGBM? gcc comes with libomp bundled already, so if you are using that AND did something like brew install libomp, it can lead to this conflict at run time.
One fix suggested in dmlc/xgboost#1715, specifically this comment:, is to remove the version(s) of libomp known to Homebrew.
brew uninstall libiomp clang-omp

Could you try that, then reinstall LightGBM, and let me know if it works for you? If that works I will add it to our FAQs.

@danielmarcelino are you still facing this issue? I'd be happy to help if you can answer the questions in my comment above.
Thanks!

No problem! We are here to help YOU, just want to know if it is still an issue which other people might be facing.
I'm glad you got it working on Linux, but I am worried we have some problem on Mac OS (see #2350 for example). Could you tell me how you are installing lightgbm on your Mac? Like show me the exact code you are running? Namely, I'm curious about whether you are using gcc or clang.

Ah! Only after a month since creating the issue I managed to take a closer look at the provided screenshot. And seems that I found the root cause of the crash.
Refer to https://lightgbm.readthedocs.io/en/latest/FAQ.html#lightgbm-crashes-randomly-with-the-error-like-initializing-libiomp5-dylib-but-found-libomp-dylib-already-initialized.


conda install of latest glib=2.51.0 fails

i use the following command to install the latest glib:
$ conda create -n glib-test -c defaults -c conda-forge python=2 glib=2.51.0

The installation fails with the error message below. The former version glib=2.48.0 works.
Is this maybe related to https://github.com/conda/conda-build/issues/1482?
Current conda install:

               platform : linux-64
          conda version : 4.2.13
       conda is private : False
      conda-env version : 4.2.13
    conda-build version : 2.0.11
         python version : 2.7.12.final.0
       requests version : 2.7.0
       root environment : /home/pingu/anaconda  (writable)
    default environment : /home/pingu/anaconda
       envs directories : /home/pingu/.conda/envs
                          /home/pingu/anaconda/envs
          package cache : /home/pingu/.conda/envs/.pkgs
                          /home/pingu/anaconda/pkgs
           channel URLs : https://repo.continuum.io/pkgs/free/linux-64
                          https://repo.continuum.io/pkgs/free/noarch
                          https://repo.continuum.io/pkgs/pro/linux-64
                          https://repo.continuum.io/pkgs/pro/noarch
                          https://conda.anaconda.org/birdhouse/linux-64
                          https://conda.anaconda.org/birdhouse/noarch
                          https://conda.anaconda.org/conda-forge/linux-64
                          https://conda.anaconda.org/conda-forge/noarch
            config file : /home/pingu/.condarc
           offline mode : False

`$ /home/pingu/anaconda/bin/conda create -n glib-test -c defaults -c conda-forge python=2 glib=2.51.0`

    Traceback (most recent call last):
      File "/home/pingu/anaconda/lib/python2.7/site-packages/conda/exceptions.py", line 479, in conda_exception_handler
        return_value = func(*args, **kwargs)
      File "/home/pingu/anaconda/lib/python2.7/site-packages/conda/cli/main.py", line 145, in _main
        exit_code = args.func(args, p)
      File "/home/pingu/anaconda/lib/python2.7/site-packages/conda/cli/main_create.py", line 68, in execute
        install(args, parser, 'create')
      File "/home/pingu/anaconda/lib/python2.7/site-packages/conda/cli/install.py", line 405, in install
        execute_actions(actions, index, verbose=not context.quiet)
      File "/home/pingu/anaconda/lib/python2.7/site-packages/conda/plan.py", line 643, in execute_actions
        inst.execute_instructions(plan, index, verbose)
      File "/home/pingu/anaconda/lib/python2.7/site-packages/conda/instructions.py", line 135, in execute_instructions
        cmd(state, arg)
      File "/home/pingu/anaconda/lib/python2.7/site-packages/conda/instructions.py", line 58, in EXTRACT_CMD
        extract(arg)
      File "/home/pingu/anaconda/lib/python2.7/site-packages/conda/install.py", line 810, in extract
        t.extractall(path=temp_path)
      File "/home/pingu/anaconda/lib/python2.7/tarfile.py", line 2079, in extractall
        self.extract(tarinfo, path)
      File "/home/pingu/anaconda/lib/python2.7/tarfile.py", line 2116, in extract
        self._extract_member(tarinfo, os.path.join(path, tarinfo.name))
      File "/home/pingu/anaconda/lib/python2.7/tarfile.py", line 2184, in _extract_member
        os.makedirs(upperdirs)
      File "/home/pingu/anaconda/lib/python2.7/os.py", line 150, in makedirs
        makedirs(head, mode)
      File "/home/pingu/anaconda/lib/python2.7/os.py", line 157, in makedirs
        mkdir(name, mode)
    OSError: [Errno 36] File name too long: '/home/pingu/.conda/envs/.pkgs/glib-2.51.0-1.tmp/share/gdb/auto-load/feedstock_root/build_artefacts/rec
ipe_root_1481139809452/_b_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeho
ld_placehold_placehold_placehold_placehold_placehold_placehold_placehold_'

Confirmed. My miniconda install is in my encrypted home folder.
Current conda install:

               platform : linux-64
          conda version : 4.2.13
       conda is private : False
      conda-env version : 4.2.13
    conda-build version : not installed
         python version : 3.5.2.final.0
       requests version : 2.12.1

The latest glib, failing here, was built with conda-build 2 while the previous one, that is working, was not.
It is not related to https://github.com/conda/conda-build/issues/1482 as that is by design.
I cannot reproduce that locally on my Linux machine. See
> conda create -n glib-test -c defaults -c conda-forge python=2 glib=2.51.0
Fetching package metadata .........
Solving package specifications: ..........

Package plan for installation in environment /home/filipe/miniconda3/envs/glib-test:

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    libffi-3.2.1               |                0          36 KB  defaults
    libiconv-1.14              |                0         2.0 MB  defaults
    zlib-1.2.8                 |                3         101 KB  defaults
    pcre-8.39                  |                0         553 KB  defaults
    python-2.7.12              |                1        12.1 MB  defaults
    glib-2.51.0                |                1         6.0 MB  conda-forge
    ------------------------------------------------------------
                                           Total:        20.7 MB

The following NEW packages will be INSTALLED:

    gettext:  0.19.7-1 conda-forge
    glib:     2.51.0-1 conda-forge
    libffi:   3.2.1-0  defaults
    libiconv: 1.14-0   defaults
    openssl:  1.0.2j-0 defaults
    pcre:     8.39-0   defaults
    python:   2.7.12-1 defaults
    readline: 6.2-2    defaults
    sqlite:   3.13.0-0 defaults
    tk:       8.5.18-0 defaults
    zlib:     1.2.8-3  defaults

Proceed ([y]/n)? y

Pruning fetched packages from the cache ...
Fetching packages ...
libffi-3.2.1-0 100% |######################################| Time: 0:00:00 663.46 kB/s
libiconv-1.14- 100% |######################################| Time: 0:00:01   1.24 MB/s
zlib-1.2.8-3.t 100% |######################################| Time: 0:00:00 829.46 kB/s
pcre-8.39-0.ta 100% |######################################| Time: 0:00:00   1.19 MB/s
python-2.7.12- 100% |######################################| Time: 0:00:10   1.23 MB/s
glib-2.51.0-1. 100% |######################################| Time: 0:00:06 905.95 kB/s
Extracting packages ...
[      COMPLETE      ]|#########################################################| 100%
Linking packages ...
[      COMPLETE      ]|#########################################################| 100%
#
# To activate this environment, use:
# > source activate glib-test
#
# To deactivate this environment, use:
# > source deactivate glib-test
#

> source activate glib-test
(glib-test)
> conda info
Current conda install:

               platform : linux-64
          conda version : 4.2.13
       conda is private : False
      conda-env version : 4.2.13
    conda-build version : 2.0.11
         python version : 3.5.2.final.0
       requests version : 2.12.1
       root environment : /home/filipe/miniconda3  (writable)
    default environment : /home/filipe/miniconda3/envs/glib-test
       envs directories : /home/filipe/miniconda3/envs
          package cache : /home/filipe/miniconda3/pkgs
           channel URLs : https://conda.anaconda.org/conda-forge/linux-64
                          https://conda.anaconda.org/conda-forge/noarch
                          https://repo.continuum.io/pkgs/free/linux-64
                          https://repo.continuum.io/pkgs/free/noarch
                          https://repo.continuum.io/pkgs/pro/linux-64
                          https://repo.continuum.io/pkgs/pro/noarch
            config file : /home/filipe/.condarc
           offline mode : False

> glib-compile-resources -h
Usage:
  glib-compile-resources [OPTION…] FILE

Compile a resource specification into a resource file.
Resource specification files have the extension .gresource.xml,
and the resource file have the extension called .gresource.

Help Options:
  -h, --help                  Show help options

Application Options:
  --version                   Show program version and exit
  --target=FILE               name of the output file
  --sourcedir=DIRECTORY       The directories where files are to be read from (default to current directory)
  --generate                  Generate output in the format selected for by the target filename extension
  --generate-header           Generate source header
  --generate-source           Generate sourcecode used to link in the resource file into your code
  --generate-dependencies     Generate dependency list
  --dependency-file=FILE      name of the dependency file to generate
  --manual-register           Don’t automatically create and register resource
  --internal                  Don’t export functions; declare them G_GNUC_INTERNAL
  --c-name                    C identifier name used for the generated source code

@msarahan can you take a quick look?

additional info: my conda installation is on an encrypted home folder. But i had no troubles with this before.

+1, this is a regression.

I think this is probably related to https://github.com/conda/conda-build/issues/1482
That behavior is certainly by design, but that doesn't mean that it can't cause problems.  In this case, I think that we can fix the issue by changing how we put files into the tarball.  We really shouldn't be including the full file path in the tarball - just a relative path.  All of that prefix junk is unavoidable at build time, but it should not be there at run time.  Something has broken.  I'll see about a fix today.

That behavior is certainly by design, but that doesn't mean that it can't cause problems.

Ah. Missed that part. :+1:
@gwvr and @cehbrecht can you live with the previous version for now? Not sure how long it will take a proper solution to be implemented and applied here... And I am not eager to revert back to conda-build 1 :confused:

@ocefpaf I guess we'll have to live with it for now!
I only encountered this issue because I'd done a
conda update --all

I don't need the latest version of glib.

Thanks @gwvr!
Until we find a proper solution let's keep this issue open.

@ocefpaf

can you live with the previous version for now?

yes ... currently no problem, just need to pin it in my environment.

Sorry, I took a closer look at this, and I don't think it's a new issue in conda-build 2.  It is certainly caused by the longer prefix, but it's glib's build that is ultimately the culprit here in how it captures the environment in which it was built.
The folder structure for most of the stuff is fine, but there is just that one folder: share/gdb/auto-load/feedstock_root/build_artefacts/recipe_root_1481139809452/_b_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_ - this folder needs to be manually compensated for somehow in your build.sh script.  If you don't need that gdb stuff, I recommend just deleting share/gdb - that's the quickest fix.

@cehbrecht and @gwvr can you test the latest builds when they make into the channel? I removed the gdb directory as suggested by @msarahan. Hopefully that won't be a problem for us.

@ocefpaf, @msarahan I've just installed 2.51.0-2, no obvious issues. Many thanks.

Thanks @gwvr for the quick test!

@ocefpaf
the latest version works also on my site. Thanks :)

Awesome. Thanks for the feedback @cehbrecht!
PS: @cehbrecht and @gwvr consider adding yourself as maintainers on the feedstocks you are interested. See https://github.com/conda-forge/staged-recipes#10-what-does-being-a-conda-forge-feedstock-maintainer-entail


The AUC Result

May I know why the AUC score in the Kaggle dataset and my own dataset is so strange that never runs above 0.5 ?
Thanks a lot~
from sklearn.metrics import roc_auc_score

[1] train-result=0.0130, valid-result=0.0551 [33.9 s]
[2] train-result=0.2280, valid-result=0.2518 [35.6 s]
[3] train-result=0.2539, valid-result=0.2593 [33.7 s]
[4] train-result=0.2590, valid-result=0.2629 [34.0 s]
[5] train-result=0.2559, valid-result=0.2561 [36.0 s]
[6] train-result=0.2588, valid-result=0.2594 [34.4 s]
[7] train-result=0.2579, valid-result=0.2587 [33.2 s]
[8] train-result=0.2640, valid-result=0.2626 [33.6 s]
[9] train-result=0.2671, valid-result=0.2638 [34.8 s]
[10] train-result=0.2619, valid-result=0.2601 [35.0 s]
[11] train-result=0.2667, valid-result=0.2609 [36.3 s]
[12] train-result=0.2664, valid-result=0.2627 [37.6 s]
[13] train-result=0.2687, valid-result=0.2650 [34.1 s]
[14] train-result=0.2724, valid-result=0.2692 [33.9 s]
[15] train-result=0.2689, valid-result=0.2663 [34.0 s]
[16] train-result=0.2710, valid-result=0.2671 [33.2 s]
[17] train-result=0.2677, valid-result=0.2655 [33.1 s]
[18] train-result=0.2694, valid-result=0.2674 [34.5 s]

what's happended? I had the same problem

what's happended? I had the same problem

what's happended? I had the same problem

i see, metric is different.

what's happended? I had the same problem

i see, metric is different.

what's happended? I had the same problem

i see, metric is different.

是啥原因？

我是健翔，您的邮件我收到了。


HostKeyAlgorithms: add rsa-sha2-256 and rsa-sha2-512 for ssh-rsa

Related to https://github.com/trzsz/trzsz-ssh/issues/93

Steps to reproduce:

Change /etc/ssh/sshd_config on the ssh server, add a configuration:HostKeyAlgorithms rsa-sha2-512,rsa-sha2-256

Restart the ssh server service ssh restart.
Delete the known host items of the server in ~/.ssh/known_hosts on the client.
Log in to the server using go ssh client, which should add a ssh-rsa item to ~/.ssh/known_hosts.
Log in to the server using go ssh client again, an error will occur:client offered: [ssh-rsa], server offered: [rsa-sha2-512 rsa-sha2-256]

Great debugging and fix, thank you @lonnywong, this is excellent!
One minor nit: should the 3 resulting rsa entries get returned in the opposite order, so that the most-secure algorithm gets listed first? I'm pondering this since ssh.ClientConfig.HostKeyAlgorithms is a sorted list by preference, first match between client and server wins. In other words, instead of expanding the rsa key's algorithms to "ssh-rsa", "rsa-sha2-256", "rsa-sha2-512", should it be ordered like "rsa-sha2-512", "rsa-sha2-256", "ssh-rsa" ?
To be clear, aside from how we expand ssh-rsa knownhosts entries, we can keep all other ordering as-is: that is, the returned order just matches the order of the entries in the knownhosts file.
Separately, CI is failing because of something with golint, probably because the Go version I have in the GitHub Actions config is old. I'll try bumping that version on my end momentarily; or if that fails I'll just finally remove golint since it is deprecated/unmaintained.

update: CI is now fixed on main branch, if you rebase your PR against latest main commit it should hopefully work now 👍

@evanelias Now, the order is rsa-sha2-512, rsa-sha2-256, ssh-rsa.

Thanks, looks perfect! Merging momentarily and tagging a new version.


Patch Release v0.8.1

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Breakage on null body/struct

...writing tests for param validation in Ringpop and discovered I can't send and verify an error on a null body being sent across the wire.
# sends join without required params, regrets it

node_modules/tchannel/node_modules/thriftify/specs/astruct.js:129
        var value = struct[field.name];
                          ^
TypeError: Cannot read property 'app' of null
    at AStruct.uglify (node_modules/tchannel/node_modules/thriftify/specs/astruct.js:129:27)
    at AStruct.toBuffer (node_modules/tchannel/node_modules/thriftify/specs/astruct.js:164:29)
    at TChannelAsThrift.stringify [as _stringify] (node_modules/tchannel/as/thrift.js:345:28)
    at TChannelAsThrift.send (node_modules/tchannel/as/thrift.js:186:32)
    at onIdentified (ringpop/server/index.js:373:31)
    at DefinedEvent.onIdentified [as listener] (node_modules/tchannel/peer.js:220:9)
    at DefinedEvent.emit (node_modules/tchannel/lib/event_emitter.js:86:14)
    at TChannelConnection.onOutIdentified (node_modules/tchannel/connection.js:367:26)
    at DefinedEvent.onOutIdentified [as listener] node_modules/tchannel/connection.js:347:14)
    at DefinedEvent.emit (node_modules/tchannel/lib/event_emitter.js:86:14)

Relatively accurate pseudo-test case:
var tchannelAsThrift = tchannel.TChannelAsThrift({
  source: <some-source>
});
var request = tchannel.request({
  host: <some-host>,
  hasNoParent: true,
  serviceName: 'ringpop',
  retryLimit: 1,
  trace: false,
  headers: {
    'cn': 'ringpop'
  }
});
tchannelAsThrift.send(request, endpoint, head, null, onSend);

I believe an error, albeit a more informative error, is expected for this case. This is not an operational error, since you are responsible for constructing a valid body for the corresponding Thrift definition.

Agreed. Passing in null for a struct should be an exception.

I see why you'd say that, but isn't the conversion of body to a struct somewhat of a underlying implementation detail to tchannel/as/thrift? As a client, I should be able to send a null/empty body if I wish... shouldn't I?

For an outgoing request; you should be able to send null; because that's zero arguments.
So I guess it makes sense to take null for a struct as long as it has zero required fields.

I think tchannel might need to make this a little more explicit.
As a developer building on top of tchannel (as thrift), I'm not thinking that the body is being packed as a Thrift struct (even though the tchannel thrift docs say so). I'm thinking that tchannel wants me to pack the primitives into a body object.  I dunno. Maybe my expectations are wrong. Should I expect that there is client-side Thrift validation going on too?

We could alter thriftify to treat null as an empty struct. If there are any required fields on the struct, thriftify would still throw an exception.
The thriftify request handler does bypass the application handler and respond with bad request frames if a request does not meet or exceed the thrift spec’s requirements.

@jwolski
There are two parts; parsing & serializing; If you pass an invalid data structure to the thrift serializer it will throw an exception.
If tchannel/as/thrift parses an invalid data structure it will warn() and send back a BadRequest.

Maybe my expectations are wrong. Should I expect that there is client-side Thrift validation going on too?

The serialization is thrift validation; so we do thrift validation in the client; you must make a valid outgoing request as per the thrift IDL or we will throw a programmer exception.


aum math has small rounding error

change 30.44 to 30.5

submitted for review, to be tested in 4.0.2 (260)


Several issues

I've been testing jsonic for a few minutes and discovered several issues:

JSON file can contain just a number or string:

JSON.parse('0')
// -> 0
JSON.parse('"a string"')
// -> 'a string'

jsonic cannot parse this, but should be compatible to JSON.

Hexadecimal number support ?

{ 0x12: 0x14 }
// -> { '18': 20 }
jsonic('{0x12:0x14}')
{ '0x12': '0x14' }

The result is not what I expected (same result as in JS). Hexadecimal numbers could be supported.
And invalid numbers like 0g12 could cause a parse error.

Strings without quote

I don't really like that way string don't require quotes. It can confuse and it's not much work to add two quotes.
Example:
var name = 'John Doe';
var o = { name: name }

I don't need to tell what the result of js and jsonic is.
And { name: John Joe } is not even valid JS.
Better: disable that by default, and require quotes.
If the programmer badly wants the short syntax, it could be turned on optionally.

these are now mostly fixed


Update Az content to match latest for AzRM 6.13, + other edits.

OPS Build status updates of commit a56c9a4:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/azps-1.0.0/overview.md
:warning:Warning
View (azps-1.0.0)
Details

docs-conceptual/azps-1.0.0/authenticate-azureps.md
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/azureps-vm-tutorial.yml
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/context-persistence.md
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/formatting-output.md
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/migrate-from-azurerm-to-az.md
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/migration-az-0.3.0.md
:white_check_mark:Succeeded

docs-conceptual/azps-1.0.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/uninstall-az-ps.md
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/using-experimental-modules.md
:white_check_mark:Succeeded
View (azps-1.0.0)

docs-conceptual/azps-1.0.0/overview.md

Line 23: [Warning] Invalid file link:(~/docs-conceptual/azps-1.0.0/release-notes-azureps.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


utils.select_proxy raises an exception when URL has no hostname

URLs sometimes have no hostname (e.g. file URLs, or unix socket URLs). The parsed URL hostname attribute returns None in this case, making utils.select_proxy complain about concat'ing a str with None.

@chipaca keep in mind that whatever adapter you're using will have to deal with older versions of requests and if it's not meant to touch the network should not be inheriting from the HTTPAdapter.

@sigmavirus24 I'm using https://github.com/msabramo/requests-unixsocket/ which does inherit from HTTPAdapter. I'm not sure what you mean, though; where does "touching the network" become a factor?

If you're using a file:// URI, you're never going to talk to something over the network using HTTP (for example). I'll talk to @msabramo about fixing unixsocket as this problem will persist otherwise for several people who will not get this fix.

I'm curious how you saw this exception there. Looking at unixsocket's code, it overrides the method which is the one place where select_proxy is used in requests.

it's used in one more place: HTTPAdapter.request_url.
On Tue, Apr 26, 2016 at 3:56 PM, Ian Cordasco<EMAIL_ADDRESS>wrote:

I'm curious how you saw this exception there. Looking at unixsocket's
code, it overrides the method which is the one place where select_proxy
is used in requests
https://github.com/kennethreitz/requests/blob/60aee145b96a4cf13a7daefd8578d538fe8459a8/requests/adapters.py#L250
.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/kennethreitz/requests/issues/3131#issuecomment-214772376

--
John Lenton<EMAIL_ADDRESS>::: http://chipaca.com

As to where I'm using it, I'm using it to connect to a unix socket :-)
I packaged httpie as a snap (originally for snappy ubuntu core systems, now on ubuntu classic itself), and added support for unix sockets so that it could talk to the snappy daemon. That's on https://github.com/chipaca/httpie-snap fwiw.

Ah so @msabramo should be overriding that too if he continues to subclass the adapter.

I didn't know about requests-unixsocket! We must catalogue these extensions somewhere.

Hmmm, but don't unixsocket URLs always have a "hostname", because the hostname is a URL-encoded path to the socket on the filesystem. E.g.:
[38] > /Users/marca/dev/git-repos/requests/requests/utils.py(592)select_proxy()
-> proxy = proxies.get(urlparts.scheme+'://'+urlparts.hostname)
(Pdb++) urlparts
ParseResult(scheme='http+unix', netloc='%2Ftmp%2Ftest_requests.132820227_21127_b4f4da9f', path='/path/to/page', params='', query='', fragment='')
(Pdb++) urlparts.scheme
'http+unix'
(Pdb++) urlparts.hostname
'%2ftmp%2ftest_requests.132820227_21127_b4f4da9f'


Gateway default flow selection

Is your feature request related to a problem? Please describe.
Add support to select default flow element of the gateway
Describe the solution you'd like
Display default outgoing gateway route and selector for it

Improved selection, but still need to be selected as is in Flowable. TBD by #100


cjs/esm/esbuild pain

Current repo (initial commit) is in a state to begin testing.
Unfortunately, there is a nasty web of incompatibilities that is present and some incompatibilities are preventing this.
Basically, because napi-rs generates cjs, but aegir et all expects esm, we're in a world of hurt.

during the build:rs step, napi-rs writes out a file that requires a .node native binding - this is currently written to src/napi.cjs

typescript type checking breaks in this file - this is "fixed" by adding a // @ts-nocheck in a post-build step (currently scripts/postbuild.sh)
aegir needs ts/js source to live in src/, so the cjs file is written to this non-default (from napi-rs perspective) subdirectory. This breaks the brittle napi-rs generated code for local builds (which expects a .node file within the same directory as the cjs). - this is "fixed" by altering the require statements in a post-build step (currently scripts/postbuild.sh)

during the build:rs step, napi-rs writes out a type declaration file - this is currently written to src/napi.d.ts

typescript doesn't nicely use the types when importing import * as napi from './napi.cjs' - this is "fixed" by importing the types and code separately and manually typecasting the code (see src/index.ts#L8-10)

during the build:ts step, esbuild is adding an export statement to the .cjs file (wtf?!) so aegir test currently blows up

cc @achingbrain

This was fixed by hacking the napi-rs generated js file from cjs to esm. (sed is your friend)
yarn test now runs properly


uniq output mistake

The output of sort example.txt | uniq isn't quite right. Given the input file, it should be:
a
b
c
d

@drhr thanks. I fixed


How do you yield* with async/await?

Hi there, I was looking to move my CLI tool org-labels over to async/await as an experiment, using node+v8-5.2 & --harmony_async_await.
I've noticed there does not immediately appear to be a replacement for yield*, as with the co library.
That is, it is unclear what you need to do to wait for all awaits to resolve within an async function...

await Promise.all(arrayOfPromises);

@kittens Given the following:
async function a() {
  await x;
  await y;
}

async function b() {
  await a(); // previously `yield* a()`
}

do you mean await Promise.all(a())? Will the async function with multiple awaits return an array?
Always or only when multiple exist/are run?

Similar issue: https://github.com/tc39/ecmascript-asyncawait/issues/61

Async functions with multiple awaits still only return a single promise, which is resolved once the function has finished. await makes it wait for a given promise to resolve before the function is continued.
With multiple awaits, it'll wait until the first promise is resolved, then the second promise, then the third promise, etc, until the end of the function has been reached. At that point, the promise that the function as a whole returned is resolved.
There is nothing wrong here?

Because an async function that awaits on promises itself returns a promise that is resolved when the async function return, await works for awaiting the result of other async functions too out of the box.

Hmm, ok. Looks/sounds like this is the default behavior. In a bit of hindsight that makes sense.
That being said I am running into problems related to this, but not really this issue.


Reimplementation of ccdmodel according to the linkml model template

Started a new clone of the linkml model template in order to use its publish to PyPI functionality. See Issue #69
Then wiped all files from this branch and copied the linkml model template based content back in. Finally added functionality from the main branch of this repo back in.
A few things to keep an eye on:

AUTHORS file. I'm not sure what created this or how it is used. But it's "just me" now.
the PyPI metadata in setup.cfg. Note that re-running MakeConfig parses model/CONFIG.yaml and might overwrite some of the files I'm mentioning here.
ChangeLog (generated by the Makefile?) vs CHANGELOG.md
taking advantage of the established mike docs practices. see esp. mkdocs.yml
manual edits to MAINTAINERS.md

Just to check: apart from https://github.com/cancerDHC/ccdhmodel/issues/77, are there any other issues blocking this PR?

Just to check: apart from #77, are there any other issues blocking this PR?

I'll double check tomorrow

I haven't disabled pbr yet.
I don't know what would happen to the documentation if this was merged into main

I haven't tried running the Makefile yet, but I will do so and leave further comments on this PR if I come across anything of concern. But based on your demo, it looks like everything is in order @turbomam, and this PR is good to go.🚀

Commented pbr out all mentions of pbr in setup.py. Didn't add any documentation per se. I have rerun make pypi and ChangeLog hasn't been regenerated. There are a log of residual ChangeLog* files in from_template

I deliberately didn't regenerate the CRDCH model here, as I'd like to do that in another PR. But make regen-google-sheets should now be working again if you'd like to test that out or put it back into this.

@gaurav I like what you've done with this PR.  I have some suspicion that the documents generation & publishing or the virtual environments may need some tweaking after this merge, but I'm confident that we can easily resolve anything like that.
I think I have some verbiage about different virtual environments and the potential use of tox in the MAINTAINERS doc that could be pared down after we merge. We might be able to make the notes about initial pushes to PyPI vs subsequent pushes less convoluted, too.

Thanks, @turbomam! I'll go ahead and merge this PR. I've opened an issue (https://github.com/cancerDHC/ccdhmodel/issues/103) for covering the improvements to the MAINTAINERS document you've suggested.


Cond expr - labels embedded in cond expr appear at the end on restart

Ref:

I guess the root cause is #935

Dupl: https://github.com/atlasmap/atlasmap/issues/990


How To Add achievements

Describe your problem here.

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No,Just Trying To add A new achievements

you probably are doing it right, but from the screenshot it looks like you forgot to close an array and caused a syntax error


Break after attach to process freezes Visual Studio

I launched a node process using node debug <scriptname> and was able to attach to it fine, however after selecting Debug | Break VS hung. Attaching to VS I can see the UI thread is blocked at the below callstack
 	WindowsBase.dll!System.Windows.Threading.DispatcherSynchronizationContext.Wait(System.IntPtr[] waitHandles, bool waitAll, int millisecondsTimeout)	Unknown
 	mscorlib.dll!System.Threading.SynchronizationContext.InvokeWaitMethodHelper(System.Threading.SynchronizationContext syncContext, System.IntPtr[] waitHandles, bool waitAll, int millisecondsTimeout)	Unknown
 	[Native to Managed Transition]
 	[Managed to Native Transition]
 	mscorlib.dll!System.Threading.Monitor.Wait(object obj, int millisecondsTimeout, bool exitContext)	Unknown
 	mscorlib.dll!System.Threading.Monitor.Wait(object obj, int millisecondsTimeout)	Unknown
 	mscorlib.dll!System.Threading.ManualResetEventSlim.Wait(int millisecondsTimeout, System.Threading.CancellationToken cancellationToken)	Unknown
 	mscorlib.dll!System.Threading.Tasks.Task.SpinThenBlockingWait(int millisecondsTimeout, System.Threading.CancellationToken cancellationToken)	Unknown
 	mscorlib.dll!System.Threading.Tasks.Task.InternalWait(int millisecondsTimeout, System.Threading.CancellationToken cancellationToken)	Unknown
 	mscorlib.dll!System.Threading.Tasks.Task.Wait(int millisecondsTimeout, System.Threading.CancellationToken cancellationToken)	Unknown
>	Microsoft.NodejsTools.dll!Microsoft.NodejsTools.Debugger.DebugEngine.AD7Engine.CauseBreak() Line 682	C#

The offending code in AD7Engine.cs is the `BreakAllAsync().Wait() below:
        public int CauseBreak() {
            DebugWriteCommand("CauseBreak");
            AssertMainThread();

            _process.BreakAllAsync().Wait();

            return VSConstants.S_OK;
        }

Note this is with Node 0.12.7 and NTVS 1.1 RC2.

I'm unable to repo this exact bug, but did keep hitting #425. I'm also assuming that you actually want node --debug <scriptname> instead of launching the node debugger itself. I tried both setups though, and neither hit this hang.
Closing as norepo. Please add some step by step instructions if this problem is still occurring and we can investigate further.
Thanks,


change PolicyAuditMode config option through CLI doesn't affect datapath

Bug report
General Information

Cilium version v1.8.4
Kernel version 4.14.67
Orchestration system version in use K8S v1.17.3

How to reproduce the issue

create some test pods on node1 and a deny-all CiliumNetworkPolicy enforcing on them, all traffic accessing test pods should be denied
https://docs.cilium.io/en/v1.8/policy/language/#ingress-egress-default-deny
execute the following command in cilium-agent on node1 to enable PolicyAuditMode on this node, all traffic accessing test pods are expected to be allowed

cilium config PolicyAuditMode=enable

cilium config | grep PolicyAuditMode displays the expected output, but traffic to test pods are denied

PolicyAuditMode          Enabled

cc @ArthurChiao

It seems that this is due to CLI config change only triggers a RegenerationWithoutDatapath while PolicyAuditMode requires BPF recompilation:
https://github.com/cilium/cilium/blob/73be2c15ced2d54aecdfa462b38d2aac2e6f631f/daemon/cmd/policy.go#L67-L76

@jaffcheng good catch, you're right. Will you submit a patch to ensure that specifically when changing PolicyAuditMode it regenerates the datapath?

By the way, toggle on/off global PolicyVerdictNotification (cilium config PolicyVerdictNotification=false) is also broken due to the same reason as @jaffcheng described above.
AND, toggle off endpoint-level PolicyVerdictNotification is broken due to bpf compiling errors, a patch is submitted for review: https://github.com/cilium/cilium/pull/13921, @joestringer
Thanks!

Thanks for checking! I'll send a PR.


fix typeahead link

typeahead link went to bootstrap, now goes to typeahead github repo

I thought the same ting too, but he is using bootstrap2.3's typeahead function not the typeahead js


Fix TimeIntervalCollection.removeInterval

Reported on the forum: https://groups.google.com/forum/?hl=en#!topic/cesium-dev/SGcYfC6lPLk
The missing update in the while loop caused all intervals after an overlapped one to be removed from the collection.  This properly halts the loop once it reaches an end time outside the removed interval.
@mramato confirmed this was left out by mistake when the code was ported

Thanks @hpinkos.  I agree, no idea how this went unnoticed for so long.


Admin Panel Configuration Page is showing blank

I am bit confused that in admin panel configuration page is showing 'No Result'. What should be there? I am not understanding if I did any error so that its not coming. Please help.

Hi @Rituparna007
It is normal. For now there is no configuration loaded in DB.


[simulation] do not update CSL IE in retransmission

A retransmission frame should not have its CSL IE updated which will leads to incorrect MIC.
For the CI test case failure, it is caused by racing condition in test.

SSED sets CSL channel to 12 which is different from the Thread operating channel(channel 11).
Leader sends ping request at channel 12 during sample time.
Leader receive Enh-ACK from SSED on channel 12.
Leaded change back to channel 11 when this Enh-ACK is set to posix host and MAC state is back to idle.
SSED sends ping response on channel 11 to leader.

A racing condition is step 5 comes before step 4. So simulation radio will reject the ping response as it is currently on channel 12 while the receiving frame is on channel 11.
SSED will do a retransmission again with CSL IE updated. This breaks the integrity of MIC. Leader will drop the retransmitted frame. Thus ping fail.

A follow up PR is #6297


feat: Add minFocusDistance prop to CameraDevice

What
Adds the minFocusDistance prop to the CameraDevice to expose how close the device can focus to.

Changes

Tested on

Related issues

Resolves https://github.com/mrousavy/react-native-vision-camera/issues/2246

whoops wrong branch


Pulls from forks fail due to secret not being shared

Pulls from forks, like #229, fail testing because testing depends on a secret and that secret isn't available in the source repository. There must be some way that people are recommended to handle this?

I was reading a couple of posts about this issue, it seems that there is a way in which we can modify the Actions workflow to grant access to Secrets for pull requests, but there are security implications with the approach:
https://github.com/actions/add-to-project/issues/163
https://securitylab.github.com/research/github-actions-preventing-pwn-requests/
In the first link, a folk who seems to work for GitHub, acknowledge that they don't support a functionality for this issue, although the post was about a year and a half ago.
Another idea might be to specify in the Actions workflow that the actual connection to IDT API is only needed when we are running the tests in the parent SBOL-utilities repository, and mock the API call for any fork of it. In this way we would solve the issue of sharing the Secret and also avoid the hassle of needing to have an IDT account for any person who would like to contribute to the repository and properly run Actions in its fork. I don't know if this is possible, but I would be happy to give it a try.
Let me know your thoughts, thanks.

Thanks for looking into this. I've followed your link and read, and I agree that granting access to secrets is a bad idea.
I also notice that it won't actually solve the problem of testing in a fork, which we want to be able to do before the pull request starts.
Put those together, and I find that I really like the idea of using the mock API call for testing.
In fact, we could have the mock call run all the time, and then repeat it with the real call only when the attempt to read the secret succeeds.
That will mean that we don't poke IDT's API for routine testing, but still check it once per day.

Thanks for your response @jakebeal
I have a doubt: If we have the mock call running all the time, what can we do to make the real call once per day?
Is it by modifying the Actions workflow file or is there an easier way to do it?

I am thinking that we have one test that calls twice: one with the mock call, and once with the real call (but only if the JSON file exists)

Ohhh that sounds great to me, I will try to implement it then.


Add enum support for flags

This might be somewhat controversial.
I love the ability to use enums directly when checking things, such as the recent ability to use enums for authorize(). This adds a similar capability to flagging/unflagging/checking flags on models.
The caveat is that BackedEnum was introduced in PHP 8.1, which would theoretically be a breaking change here. While it would be ideal to use Illuminate\Support\enum_value, it's marked as @internal for reasons not immediately clear.
Thanks for your consideration!

Could you rebase against main to solve the conflicts?

Should be good now @freekmurze!

Seems like the tests fail on L9. Could you take a look at that? I would be ok with bumping the minimum version to L10.

Looks like it's just from php 8.0 being in the matrix. I've removed that, assuming that's the direction that it's headed.

Seems good! Thank you for this


Error: Text content does not match server-rendered HTML. (and 2 other errors on start up of GitPod)

Unhandled Runtime Error
Error: Text content does not match server-rendered HTML.

Warning: Text content did not match. Server: "connection" Client: "Connection to Deep"

See more info here: https://nextjs.org/docs/messages/react-hydration-error

Component Stack
h2
eval
ChakraComponent
Heading2
div
eval
ChakraComponent
CardHeader2
div
eval
ChakraComponent
Card2
Connection
div
eval
ChakraComponent
eval
VStack
div
eval
ChakraComponent
Page
CustomI18nProvider
ProviderCore
TokenProvider
LocalStoreProvider
CookiesStoreProviderCore
CookiesProvider
CookiesStoreProvider
QueryStoreProvider
CapacitorStoreProvider
EnvironmentProvider
ColorModeProvider
ThemeProvider
ThemeProvider
ChakraProvider
ChakraProvider2
Provider
App
Call Stack
checkForUnmatchedText
node_modules/react-dom/cjs/react-dom.development.js (9647:0)
diffHydratedProperties
node_modules/react-dom/cjs/react-dom.development.js (10310:0)
hydrateInstance
node_modules/react-dom/cjs/react-dom.development.js (11306:0)
prepareToHydrateHostInstance
node_modules/react-dom/cjs/react-dom.development.js (12564:0)
completeWork
node_modules/react-dom/cjs/react-dom.development.js (22181:0)
completeUnitOfWork
node_modules/react-dom/cjs/react-dom.development.js (26596:0)
performUnitOfWork
node_modules/react-dom/cjs/react-dom.development.js (26568:0)
workLoopSync
node_modules/react-dom/cjs/react-dom.development.js (26466:0)
renderRootSync
node_modules/react-dom/cjs/react-dom.development.js (26434:0)
performConcurrentWorkOnRoot
node_modules/react-dom/cjs/react-dom.development.js (25738:0)
workLoop
node_modules/scheduler/cjs/scheduler.development.js (266:0)
flushWork
node_modules/scheduler/cjs/scheduler.development.js (239:0)
MessagePort.performWorkUntilDeadline
node_modules/scheduler/cjs/scheduler.development.js (533:0)

Other errors (at the same time)

Unhandled Runtime Error
Error: Hydration failed because the initial UI does not match what was rendered on the server.

Warning: Text content did not match. Server: "connection" Client: "Connection to Deep"

See more info here: https://nextjs.org/docs/messages/react-hydration-error

Component Stack
h2
eval
ChakraComponent
Heading2
div
eval
ChakraComponent
CardHeader2
div
eval
ChakraComponent
Card2
Connection
div
eval
ChakraComponent
eval
VStack
div
eval
ChakraComponent
Page
CustomI18nProvider
ProviderCore
TokenProvider
LocalStoreProvider
CookiesStoreProviderCore
CookiesProvider
CookiesStoreProvider
QueryStoreProvider
CapacitorStoreProvider
EnvironmentProvider
ColorModeProvider
ThemeProvider
ThemeProvider
ChakraProvider
ChakraProvider2
Provider
App
Call Stack
throwOnHydrationMismatch
node_modules/react-dom/cjs/react-dom.development.js (12507:0)
tryToClaimNextHydratableInstance
node_modules/react-dom/cjs/react-dom.development.js (12520:0)
updateHostComponent
node_modules/react-dom/cjs/react-dom.development.js (19902:0)
beginWork
node_modules/react-dom/cjs/react-dom.development.js (21618:0)
beginWork$1
node_modules/react-dom/cjs/react-dom.development.js (27426:0)
performUnitOfWork
node_modules/react-dom/cjs/react-dom.development.js (26557:0)
workLoopSync
node_modules/react-dom/cjs/react-dom.development.js (26466:0)
renderRootSync
node_modules/react-dom/cjs/react-dom.development.js (26434:0)
performConcurrentWorkOnRoot
node_modules/react-dom/cjs/react-dom.development.js (25738:0)
workLoop
node_modules/scheduler/cjs/scheduler.development.js (266:0)
flushWork
node_modules/scheduler/cjs/scheduler.development.js (239:0)
MessagePort.performWorkUntilDeadline
node_modules/scheduler/cjs/scheduler.development.js (533:0)

Unhandled Runtime Error
Error: There was an error while hydrating. Because the error happened outside of a Suspense boundary, the entire root will switch to client rendering.

Component Stack
h2
eval
ChakraComponent
Heading2
div
eval
ChakraComponent
CardHeader2
div
eval
ChakraComponent
Card2
Connection
div
eval
ChakraComponent
eval
VStack
div
eval
ChakraComponent
Page
CustomI18nProvider
ProviderCore
TokenProvider
LocalStoreProvider
CookiesStoreProviderCore
CookiesProvider
CookiesStoreProvider
QueryStoreProvider
CapacitorStoreProvider
EnvironmentProvider
ColorModeProvider
ThemeProvider
ThemeProvider
ChakraProvider
ChakraProvider2
Provider
App
Call Stack
updateHostRoot
node_modules/react-dom/cjs/react-dom.development.js (19849:0)
beginWork
node_modules/react-dom/cjs/react-dom.development.js (21615:0)
beginWork$1
node_modules/react-dom/cjs/react-dom.development.js (27426:0)
performUnitOfWork
node_modules/react-dom/cjs/react-dom.development.js (26557:0)
workLoopSync
node_modules/react-dom/cjs/react-dom.development.js (26466:0)
renderRootSync
node_modules/react-dom/cjs/react-dom.development.js (26434:0)
recoverFromConcurrentError
node_modules/react-dom/cjs/react-dom.development.js (25850:0)
performConcurrentWorkOnRoot
node_modules/react-dom/cjs/react-dom.development.js (25750:0)
workLoop
node_modules/scheduler/cjs/scheduler.development.js (266:0)
flushWork
node_modules/scheduler/cjs/scheduler.development.js (239:0)
MessagePort.performWorkUntilDeadline
node_modules/scheduler/cjs/scheduler.development.js (533:0)

Workaround
Restart SDK.

https://nextjs.org/docs/messages/react-hydration-error
https://react.i18next.com/latest/ssr
These links may be useful for fix.


Release 1.13

Prepare for 1.8 schema
Bump version to 1.13

Ok, so roll back the 1.8.0-next changes. When we're comfortable with the changes we would release 1.13.1, or 1.14?

I guess we could release 1.13.1. Not sure what the best approach is there, doesn't matter too much.


Added health check and upload download methods made public

test_create_session_from_server_info_file_with_wrong_password:
def test_create_session_from_server_info_file_with_wrong_password(
    tmp_path: Path,
) -> None:
    # ...
    session = _BaseSession.create_from_server_info_file(
        server_info_filepath=str(server_info_file), cleanup_on_exit=False
    )
    assert session.check_health() == HealthCheckService.Status.NOT_SERVING.name

Now we are raising an exception in this scenario. So the test is failing. The raise behaviour is the only viable design alternative so the test needs to be updated with this PR.

test_create_session_from_server_info_file_with_wrong_password:
def test_create_session_from_server_info_file_with_wrong_password(
    tmp_path: Path,
) -> None:
    # ...
    session = _BaseSession.create_from_server_info_file(
        server_info_filepath=str(server_info_file), cleanup_on_exit=False
    )
    assert session.check_health() == HealthCheckService.Status.NOT_SERVING.name

Now we are raising an exception in this scenario. So the test is failing. The raise behaviour is the only viable design alternative so the test needs to be updated with this PR.
See uses of with pytest.raises(RuntimeError): in tests/

Right, Thank you.


probem with History

youtube-dl -v 'http://www.history.com/shows/engineering-disasters/videos/raging-rocket'
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-v', 'http://www.history.com/shows/engineering-disasters/videos/raging-rocket']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.05.20
[debug] Python version 3.4.1 -<EMAIL_ADDRESS>[debug] exe versions: ffmpeg 2.2.14, ffprobe 2.2.14, rtmpdump 2.4
[debug] Proxy map: {}
[History] raging-rocket: Downloading webpage
[ThePlatform] dX9JRjjp0Mnv: Downloading XML
[ThePlatform] dX9JRjjp0Mnv: Downloading webpage
Traceback (most recent call last):
  File "/usr/bin/youtube-dl", line 6, in <module>
    youtube_dl.main()
  File "/usr/lib64/python3.4/site-packages/youtube_dl/__init__.py", line 405, in main
    _real_main(argv)
  File "/usr/lib64/python3.4/site-packages/youtube_dl/__init__.py", line 395, in _real_main
    retcode = ydl.download(all_urls)
  File "/usr/lib64/python3.4/site-packages/youtube_dl/YoutubeDL.py", line 1486, in download
    res = self.extract_info(url)
  File "/usr/lib64/python3.4/site-packages/youtube_dl/YoutubeDL.py", line 660, in extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/usr/lib64/python3.4/site-packages/youtube_dl/YoutubeDL.py", line 713, in process_ie_result
    extra_info=extra_info)
  File "/usr/lib64/python3.4/site-packages/youtube_dl/YoutubeDL.py", line 649, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib64/python3.4/site-packages/youtube_dl/extractor/common.py", line 273, in extract
    return self._real_extract(url)
  File "/usr/lib64/python3.4/site-packages/youtube_dl/extractor/theplatform.py", line 173, in _real_extract
    for f in switch.findall(_x('smil:video')):
AttributeError: 'NoneType' object has no attribute 'findall'

This video can be viewed when you log in with your TV provider that is currently not supported.

Obviously, that would be a great feature, :) but no traceback (and more informative error messages) might be worthwhile in the short term (there are probably a lot of sites that require logins for at least some videos, and the extractors ought to recognize when it is required).

i added support only for DirecTV provider, it can be used with AENetwork extractors and National Geographic.


Footer Button?

How can you add the footer "Close" button as seen in the docs?

@garrettmac You can pass components via actions prop
<PopupDialog
  actions=[
    <Button
      title="CLOSE"
    >
    <Button>
  ]
>
</PopupDialog>


Differential code coverage

Hi! Can istanbul count coverage relative to previous report or different git branch?
I can`t find this feature but I think it was very useful when old code just begin covered with tests or new features must be covered with tests and diff-coverage can show regression (or progression) about this.
Thank you.

sure, it s very import feature for a lot of team, however,  one year pass

+1


Symlink read break out

Found a new filesystem access control bypass.
You can simply create a symlink within the module's folder that points to a file outside of the root folder and read said symlink.
e.g:
import fs from 'fs';
import path from 'path';

export function run () {
  const srcPath = path.resolve('./my-symlink');
  const dstPath = '/some/folder/outside/root/mysecret.txt';
  fs.symlinkSync(dstPath, srcPath);
  const content = fs.readFileSync(srcPath, { encoding: 'utf8' });
  console.log('CONTENT=', content);
}

Having though about this problem and its various solutions along the years I believe that you'll have troubles securing Hagana due to how it implements the access control.
Since you wrap or override existing Node.js APIs, you will always be chasing down lesser known APIs or OS behaviours that allow an attacker to get read / write access to the file system, network or shell.
I would probably advise you to consider a more syscall based approach that would always work, no matter which Node.js, V8 or libuv API is called.
TypeScript takes a similar approach to you I believe and has (or had until recently) the same symlink vulnerability.

Fixed in v0.0.15

Sure, perfection doesn't exist. So 99% coverage is better than nothing. That's absolutely true.
However, if Hagana were to be used more widely, Hagana specific bypasses would be included in every supply chain attack just in case the consuming application were to use Hagana.

This is true, but my hope is that there won't be a generally available bypass for Hagana. It will take a determined attacked to find a bypass for Hagana and include it in their attack.


Bottleneck retrained model predicts one class after applying transforms and opimize_for_inference.

I have asked the same question on stackoverflow
https://stackoverflow.com/questions/50243492/inception-v3-retrained-model-incorrect-predictions-after-using-optimize-for-infe/50263837#50263837
and opencv help
http://answers.opencv.org/question/191168/inception-v3-retrained-model-incorrect-predictions-after-using-optimize_for_inference-and-graph_transform-tools/
I can really use some suggestions:

I followed the tensorflow for poets tutorial to retrain inception V3 for 3-class image task

https://codelabs.developers.google.com/codelabs/tensorflow-for-poets/#0

The network fits the data well and i'm able to validate the frozen model using tensorflow python which predicts unseen samples at 97 % accuracy

The frozen model has to be integrated to a legacy application which uses opencv::DNN module. To get the forzen model ready I applied

optimize_for_inference
graph_transforms with " remove_nodes(op=PlaceholderWithDefault) strip_unused_nodes(type=float, shape="1,299,299,3") sort_by_execution_order" flags

Loaded transformed model with opencv and run some more validation before deployment. The transformed version could not match (not even close) the trained original model.  It mostly predicts a single class.

I used opencv blobFromImage function with swapRB set to true.
I tried to normalize input by subtracting ImageNet mean as well as my training set mean. Unsuccessful in both cases.
I just have no idea what is causing for the model to be so different after its transformed.
Thanks for your time

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Updated question with platform etc info

This was primarily due to the preprocessing and batch normalization that happens during training.To normalize an input at inference time please use 127.5 for RGB mean values and set the scale factor to 1/127.5 when calling opencv::dnn::blobFromImage. You might want to swap RB channels depending how you did the training.


Access toggles-prefs from an artifactory

Hi Erik.
Do you have any plans to upload toggles-prefs and toggles-coroutines to an artifactory? Meanwhile, do you think it will be an ok approach if we just make the middle layer that talks to the Toggles app ourselves?

Hi! @dmitry-ryadnenko-izettle
It would be super fun to upload to an artifactory! And it is something I've been planning but it hasn't been a priority as I wasn't sure if the artifacts would be used so I've been holding off on it. I'll look into setting it up.. :) Thanks for reaching out!
Regarding writing your own middle layer that is the whole purpose of the ":toggles:core" module - to facilitate such use cases. The toggles app is written to cater to any and all libraries that want to talk to it.
Do you think you have need for the "prefs" version of the library? I'm kind of thinking that the flow version of the library might be the one that better fits current android development patterns and hence I'm contemplating not releasing the prefs version of the library at all.

Thank you for the answer Erik! I think we would need the prefs one, we have a synchronous middle layer right now, you know :)

Thanks for the feedback. I'll make a prefs library release as well then. :)

Give it a squeeze and don't hesitate to provide feedback:
repositories {
    maven("https://s01.oss.sonatype.org/content/repositories/snapshots")
}

dependencies {
    implementation("se.eelde.toggles:toggles-flow:0.0.1-SNAPSHOT")
    implementation("se.eelde.toggles:toggles-prefs:0.0.1-SNAPSHOT")
}

Thank you Erik a lot! We'll check this out ❤

I'm giving the libs a few spins myself and will push to central if nothing arises.

@dmitry-ryadnenko-izettle Think I've done my testing now. :) How are you guys getting along?

Hi Erik, we have this pretty high on our list of our priorities, but there were even more pressing issues on us right now. I expect that we'll start on this pretty soon, in 2-4 weeks, but I'm not 100% sure.

@dmitry-ryadnenko-izettle I'll push current version to maven soon then and feel free to feedback if/when you get around to. :) It should ease testing if nothing else.

That sounds great, thank you Erik 🙏

https://search.maven.org/search?q=se.eelde.toggles


Move parts of packcheck into haskell

This is not a good move.

Packcheck is supposed to be universal and hence the better choice is bash
If we are to write packcheck in haskell, we have to make sure to have compiled executables for all types of OS and all types of architectures. This is a maintainence overhead for a project that only a few folks use.


Parent defined rules not working in child forms

Hi there,
Are the parent rules supposed to be inherited by the child class? I'm assuming so since the properties theirselves are. However that's not working for me.
Thanks

Hello!
I saw no need to inherit the parent rules, because the save and validation methods of the child class call the methods of the parent class, so the parent rules affect the parent properties already and the getErrors method return all the errors in the two classes when it is called.

Hello!
Then I may be missing something, because the client-side validation of the child class doesn't work for inherited properties if the rules are setup in the parent class...

You are right in that. The client side validation doesn't work, because the parent rules are not inherited by default. I will think about it and fix the issue.

Hi!
Has any solution been defined for this problem?
Force the merge with the parent is the best solution?
Thanks!


Dataset Access

Hello,
I'm reaching out regarding the full release and safe release dataset, specifically inquiring about the process to access it.
It appears that the Google Forms mentioned in your readme is locked behind an authorization access. I'm assuming that i am not the only user in this case.
Given this situation, could someone kindly provide guidance on how individuals interested in accessing the dataset should proceed? Are we expected to reconstruct the dataset independently, or is there an alternative method i may have missed for accessing the data?
Any assistance or clarification on this matter would be greatly appreciated.
Thank you!

Hello.
Thank you for bringing this to our attention, it is not the intended behavior. We have updated the permissions on the Google Forms. The knowledge graph construction code is provided to construct the dataset on your own with the latest results. The form is intended to grant access to the snapshot of the dataset: either a safe release with all data sources under the release license, or to the full dataset with appropriate licenses.
Link to the form: https://forms.gle/3HdKRtvW7ce9PKpw6


use xunit exception assertions in grpc integration test sample

Fixes #26937
This test fails to build when using Visual Studio Code or a CodeSpace. This change makes use of xUnit's exception assertions that are already included in the .csproj to conduct the same test.

 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: glennmusa sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the fix!


#solution to error with executalbe "from_unionbed_to_ratio_per_window_CC0"

Hi!
I found an issue with the file from_unionbed_to_ratio_per_window_CC0 that comes with the downloaded version 3.
The file seems to be crashed and do not run even after giving permissions with chmod.
I solved this by deleting this file and compiling it with make.
Regards

Thank you for reporting,
it will be helpful to other users if they encounter the same issue.
The binary file may not be compatible with all systems, that's why Make
file is provided.
чт, 23 февр. 2023 г. в 14:03, ferrojm @.***>:

Hi!
I found an issue with the file from_unionbed_to_ratio_per_window_CC0
that comes with the downloaded version 3.
The file seems to be crashed and do not run even after giving permissions
with chmod.
I solved this by deleting this file and compiling it with make.
Regards
—
Reply to this email directly, view it on GitHub
https://github.com/timnat/DifCover/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEGJB62FGTD7LLB6F76TJC3WY6YADANCNFSM6AAAAAAVGALF7E
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Develop

Initial release of the module.

Looks pretty straight forward to me - this is ready for a beta/first release.


[GetTopDirections] - update request with city filtration

Follow the link: https://github.com/ita-social-projects/OoS-Frontend/issues/1627

please don't merge this till tomorrow :) might refactor the popular workshops endpoint model on backend


re-think configuration file design to allow capability specific configs

Closed by https://github.com/deislabs/spiderlightning/pull/222


Bug Fix.

Coverage remained the same at 45.8% when pulling 0b031fde9e868ad48bb7616f6db7e7c03cb1d145 on yuye/bug-fix-testing into 0ebc6b500007486f33c9c9362703c3831c371d1f on authndev.

Coverage remained the same at 45.8% when pulling 0b031fde9e868ad48bb7616f6db7e7c03cb1d145 on yuye/bug-fix-testing into 0ebc6b500007486f33c9c9362703c3831c371d1f on authndev.

Coverage remained the same at 45.8% when pulling 0b031fde9e868ad48bb7616f6db7e7c03cb1d145 on yuye/bug-fix-testing into 0ebc6b500007486f33c9c9362703c3831c371d1f on authndev.


Working with 3D

I'm working with 3D volumes, base model is ResNet3D50 (all convolutions are Conv3D) and wants to predict x, y, z co-ordinates. Can you please help with that? Thanks.

Currently I'm using this with the 3D model:-
outcoords, heatmaps = self.net(vol)
outcoords = outcoords[:,0,:] #to loose extra dim in between
euc_losses = dsntnn.euclidean_losses(outcoords, coordinates) # Per-location euclidean losses
reg_losses = dsntnn.js_reg_losses(heatmaps, coordinates, sigma_t=1.0) # Per-location regularization losses
loss = dsntnn.average_loss(euc_losses + reg_losses) # Combine losses into an overall loss
and commented out line 200 in the dsntnn init (it was complaining as left is 2D tensor, right is 1D)
Is this the correct method to deal with 3D?
Thanks.

Instead of commenting out the assert in dsntnn, you should change the shape of the tensors to match each other. So remove the "lose extra dim in between" line, and instead unsqueeze the target so it is of shape B x 1 x 3 (ie coordinates = coordinates.unsqueeze(1)). This is the intended way of using the library.
I'm also assuming that somewhere in self.net you are using flat_softmax and dsnt.

Thanks. I replaced "loose extra dim in between" with unsqueeze and uncommented that assert.
Forward function of the self.net contains the following:-
x = self.model.features(x) #ignoring logits
unnormalized_heatmaps = self.hm_conv(x) # 2. Use a 1x1 conv to get one unnormalized heatmap per location
heatmaps = dsntnn.flat_softmax(unnormalized_heatmaps) # 3. Normalize the heatmaps
coords = dsntnn.dsnt(heatmaps) # 4. Calculate the coordinates
return coords, heatmaps
If something is wrong let me know.
Thanks.

That looks pretty reasonable to me.

Thank you so much.
One problem I'm encountering now is that after some epochs, model parameters are becoming NaN. Which results in NaN in the output.
Is somehow the loss calculation are to blame?
Do you have any suggestions to deal with this issue?
Thanks.

That's a fairly generic problem that can be caused by many different things. If you don't know about it already, torch.autograd.detect_anomaly could help you find more info (https://pytorch.org/docs/stable/autograd.html#torch.autograd.detect_anomaly). If you're worried that somehow dsntnn is to blame, that's relatively easy to check---assert that there are no NaNs/infs in the output of self.hm_conv, and assert that there are no NaNs/infs in the loss.

Thank you so much for your help.
In the problem I'm working on, I have a segmentation mask. And the idea is if the predicted coordinate is anywhere within the region (the region which is set to 1 in the mask), its correct (so no loss basically). But when it is outside of the region, then we need to try to pull it in the region. For the same, I was thinking if I send an inverted mask to the average_loss function. What do you think? Or do you have any other better idea maybe?
Sorry for bothering you and I really appreciate and thankful for all your help.
Thanks.

I think that for your problem you would probably be better served using a loss of zero for those points rather than excluding them from the loss calculation. A point that's within your segmentation region is "good" and should lower the loss, not be ignored.
Off the top of my head, I would think about implementing it this way:

Use euclidean_losses and js_reg_losses to get two loss tensors (let's call them elosses and jlosses)
Add them together to get a single loss tensor: losses = elosses + jlosses
Create a mask which is 0 for points that are within the segmentation mask, and 1 elsewhere
Apply the mask to zero out losses for the "good" points: masked_losses = losses * mask
Average the losses to get the combined loss: loss = losses.mean()


how to perform click progrmetically

when i perform click programetically sub menu buttons are not visible properly in screen

Hello, @123pavan123. If this question still current of interest for you, please provide with more specific info of what you're trying to achieve. Until then this issue is considered as closed.


No module named shark

I cd to web, then paste "python index.py", press enter, and i get this error:
(deck@steamdeck web)$ python index.py
Traceback (most recent call last):
File "index.py", line 8, in
from models.stable_diffusion.main import stable_diff_inf
File "/home/deck/git/automatic1111/Shark/SHARK/web/models/stable_diffusion/main.py", line 6, in
from models.stable_diffusion.cache_objects import model_cache
File "/home/deck/git/automatic1111/Shark/SHARK/web/models/stable_diffusion/cache_objects.py", line 10, in
from models.stable_diffusion.opt_params import get_unet, get_vae, get_clip
File "/home/deck/git/automatic1111/Shark/SHARK/web/models/stable_diffusion/opt_params.py", line 2, in
from models.stable_diffusion.model_wrappers import (
File "/home/deck/git/automatic1111/Shark/SHARK/web/models/stable_diffusion/model_wrappers.py", line 3, in
from models.stable_diffusion.utils import compile_through_fx
File "/home/deck/git/automatic1111/Shark/SHARK/web/models/stable_diffusion/utils.py", line 3, in
from shark.shark_inference import SharkInference
ModuleNotFoundError: No module named 'shark'
(1)(deck@steamdeck web)$ ^C
(130)(deck@steamdeck web)$

we haven't tested on steamdeck. What GPU does it have ?  Also the checkout path seems funny. Do you have a venv created ?

Here's the gpu specs
https://www.techpowerup.com/gpu-specs/steam-deck-gpu.c3897#:~:text=Recommended Gaming Resolutions%3A&text=The Steam Deck GPU is,the device supports DirectX 12.
Pretty much a custom 6000 RDNA2 APU, but with 8 CU's

What does vulkaninfo report?

What does vulkaninfo report?

https://pastebin.com/6xgbHnEs

Ah it uses RADV ? That would be a lot of work to get the driver to work. Can the AMD drivers work?

Lets close this issue. It is unlikely we can support on Steamdeck without official drivers for vulkan.


Update aws-java-sdk-dynamodb to 1.12.233

Updates com.amazonaws:aws-java-sdk-dynamodb from 1.11.1034 to 1.12.233.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #300.


Add support for go modules

not strictly necessary, but makes life easier for anyone depending on it.  Feel free to reject if you're not interested

@raidancampbell Thanks for the PR!


[csrng, dv] Fix the assertion fail issue

Turn off the assertions that fired unexpectedly

Signed-off-by: Muqing Liu<EMAIL_ADDRESS>

Thanks Steve for explanation. Agree and I think should be fine to disable them.


Flip the order of the 'moveend' and 'viewreset' events in map (fixes #5170)

Flipping the order of the moveend and viewreset events in the map fixes #5170, but I'm not sure if this might have any side effects.
In particular, I don't know if this will be a regression of https://github.com/Leaflet/Leaflet/pull/4193

Closing in favour of #5250.


Fix endpoint discovery

The original implementation of endpoint discovery had drifted (on Juju2 and Juju3), and the required observer was not started.
Correct the library, and some error handling for data in the charm itself.

@rbarry82 There is a library fetch issue in CI
Not sure how to solve it.

I guess, for one, may as well update the action to the latest version.
But I can't see the secrets for your repo to know whether charmcraft_credentials exists at all (or whether it's set). We have a shared secret among all of our repos, but in general, following these docs to get a token and using it as a secret will help it pass. We set the TTL to the highest possible value so it basically never expires (the default is 30 days, which is too short to be meaningful in CI).
The docs for the action are easy enough. I'm guessing from the output (charmcraft couldn't find the token) that the secret may exist, but expired or is empty.

Check libraries and publish in CI will fail due to this PR being done from fork. Acknowledging this checks failing and merging this PR without those checks after approval.

Acknowledging CI failing in:

Check libraries
Publish charm


Backwards compat modules

This adds in the long-suffering upgrade tests from #471. They do still need to be refactored, but they will be useful to the devs running on trunk.
This also makes the subfolders in the dtests into modules. This will fix the recent problem where some modules of tests didn't run at all because of hidden errors during test module loading. Here is the list of tests that weren't running; they ran successfully in this CassCI job:
http://cassci.datastax.com/job/scratch_mambocab-dtest_trunk/lastCompletedBuild/testReport/

Is this ready for review, would you like me to review, and did you see the travis complaints?

@ptnapoleon yes, this is ready for review, and you'd be a good person for it.
Just saw the linter problems -- I thought they'd be more extensive. There are few enough that it'll be worth the effort now; I'll fix those up.

This is pre-refactor, just to confirm? So if all we're doing here is modulizing, and making a note in the contributing doc, then +1.

Yes, this is pre-refactor; it

re-adds the upgrade tests
makes folders into modules and a note in CONTRIBUTING
makes a few fixup changes to the upgrade tests, in particular fixing imports, linter errors, and an incorrect use of the ccm cluster-creating api.

+1


SSL_CTX_set0_tmp_dh_pkey and SSL_set0_tmp_dh_pkey erroneously free provided key in error path

When the functions SSL_CTX_set0_tmp_dh_pkey(3) and SSL_set0_tmp_dh_pkey(3) return an error, they MUST NOT free the provided key.  Existing code in OpenSSL and reasonable user expectations do not expect a failing call to take ownership of and free the key.

The docs only say that they take ownership on success, so this does look like a bug (The apps are also written with this in mind)
NOTE: the calls to these 2 functions within ssl3_ctrl() will need to change to free the key on error instead of just returning.

Fix in #17209


Fix random integers

This was broken by returning invalid values due to use the wrong type in the output array
The statistical properties of the distribution were not currently tested, now they are

Jenkins, test this please.

Jenkins CI test (for commit 7c12eb10150edc386f8eed1a9c5ae98cb3b5a69c, target branch master) succeeded!


[CFEngine] CFEngine 3.19.0 released

https://cfengine.com/blog/2021/cfengine-3-19-released-collaboration/
https://groups.google.com/g/help-cfengine/c/o6oF-HlpcNc

@captn3m0 thank you for fixing related fields, LGTM

Thank you!

@nickanderson Thank you for your contributions 🎉 💯


Problem creating bridged network

resource "libvirt_network" "vm_network" {
  name = "vm_network"
  domain = "xx.xx.com"
  mode = "bridge"
  bridge = "br3"
}

gives the following libvirt error
* libvirt_network.vm_network: Error defining libvirt network: [Code-27] [Domain-19] XML error: The <nat> element can only be used when <forward> 'mode' is 'nat' in network vm_network -   <network>
      <name>vm_network</name>
      <domain name="london.zelotus.com"></domain>
      <bridge name="br3" stp="on"></bridge>
      <forward mode="bridge">
          <nat>
              <port start="1024" end="65535"></port>
          </nat>
      </forward>
  </network>

I've been looking at the source but sadly do not know go very well ( at all ) so haven't got very far.
I am assuming i need to pass an empty NAT block but can't work out the format ?
Documentation is not very forthcoming on NAT, so I'll volunteer to update or add some examples.
Thanks

I now understand why this is happening.
The default XML for a network includes
<nat>
      <port start='1024' end='65535'/>
</nat>

Which won't work for bridge connection.
#93 is an extremely naive attempt by me to fix this. It does cause the network to be created but terraform crashes out. This is not beyond my current ability in go or terraform to diagnose.
And its new years eve.

@deasmi  so your fix got merged. can we close the issue?

@MalloZup Please do, sorry missed your reply

@deasmi thx, no pb
@flavio @moio  i would propose to close this . ( pinging moio since this is ref. on sumaform)


fix: disable http agent keep alive for postinstall

What kind of change does this PR introduce?
follow up https://github.com/supabase/cli/pull/1791
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 58.295%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
6005

Relevant Lines:
10301

💛  - Coveralls


UI - as User - Project Limits is visible

ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

4.15

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

In the old UI, a user was not able to see the limits per project although this data is available in the API:
(user) 🐱 > sync
Discovered 269 APIs
(user) 🐱 > list projects
{
  "count": 1,
  "project": [
    {
      "cpuavailable": "40",
      "cpulimit": "40",
      "cputotal": 0,
      "created": "2021-10-20T08:26:43+0000",
      "displaytext": "test",
      "domain": "ROOT",
      "domainid": "df7fc865-2ff9-11ec-b4a5-1e00180003a4",
      "id": "b92f02aa-3f7d-45a9-8741-16f5632a2191",
      "ipavailable": "16",
      "iplimit": "20",
      "iptotal": 0,
      "memoryavailable": "40960",
      "memorylimit": "40960",
      "memorytotal": 0,
      "name": "test",
      "networkavailable": "20",
      "networklimit": "20",
      "networktotal": 0,
      "owner": [
        {
          "account": "ACSUser"
        }
      ],
      "primarystorageavailable": "200",
      "primarystoragelimit": "200",
      "primarystoragetotal": 0,
      "projectaccountname": "PrjAcct-test-1",
      "secondarystorageavailable": "400.0",
      "secondarystoragelimit": "400",
      "secondarystoragetotal": 0,
      "snapshotavailable": "20",
      "snapshotlimit": "20",
      "snapshottotal": 0,
      "state": "Active",
      "tags": [],
      "templateavailable": "20",
      "templatelimit": "20",
      "templatetotal": 0,
      "vmavailable": "20",
      "vmlimit": "20",
      "vmrunning": 0,
      "vmstopped": 0,
      "vmtotal": 0,
      "volumeavailable": "20",
      "volumelimit": "20",
      "volumetotal": 0,
      "vpcavailable": "20",
      "vpclimit": "20",
      "vpctotal": 0
    }
  ]
}
(user) 🐱 >

Due to feature parity with this I suggest we introduce this change in the new ui as well, open for discussion.

@Pearl1594 @weizhouapache I have created the issue: https://github.com/apache/cloudstack/issues/5606


Upgrade Vitamin-R to v 2.40

Upgrade Vitamin-R to v 2.40

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Thank you for the contribution. It needed some fixes, so they were made in 216e23e7367e09c456ed302f9bdd8c04df08712e. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


fix: Add top level entities in extracted subpath references

Description
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/ok-to-test sha=9044ea6

Tested ok on DP


Fixed NullpointerException 330

Fixed NullpointerException in https://github.com/opencast/opencast/pull/5810

Otherwise looks good, but since this is basically my patch, somebody else should probably review this.


Feature: Zombies / Show Purple Tiger Progess

It would be nice it if showed the number of Purple Tiger camos you have at the top like it does Mystic Gold / Opal / Afterlife / Nebula.

Most of the time, it is pretty easy to get Mystic Gold right after Purple Tiger, but sometimes (cough 9MM) it is a bit more of a pain to get all 3 after Purple Tiger.

I feel like the camos up top are more for milestones where Red/Purple Tiger is not really a milestone camo

Purple Tiger definitely is. Purple Tiger requires 2k critical kills, then the specials + Mystic Gold requires up to an additional 900 kills (usually not critical). The Special ones also have special requirements. Half of the assault rifles require parasite or vermin kills so if you are on the wrong map, you just cannot do them at all. Or mangler kills which are not on the new map.
So Purple Tiger / Mystic Gold are the first two milestones to track progress. There are many cases where you cannot just get Mystic Gold in the same match as Purple Tiger.


[FEATURE] Add Splash Screen and App Logo

Detailed description
the App schould have a proper splash screen and logo added
Context
basic setup
Possible implementation
using the cyberworld logo
Additional information
No response
Code of Conduct

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

Hey @ATH42.
are you going to work on this issue?

yes, i'll do it on a seperate branch.
but if there is anyone else that wants to take this, i am happy to leave it for them.

yes, i'll do it on a seperate branch.
but if there is anyone else that wants to take this, i am happy to leave it for them.

okay.


Dependencies in meta

Otherwise users of the role will not get the dependencies

No, dependencies in meta are not meant for this.
Needed dependencies are now documented in the README


Feature request: Language specific URLs

Hey! I use codehost all of the time, but 100% of the time I use it for highlighting Rust code. Would it be possible to have a subdomain or url path that pre-fills the language selector? Something like rust.godbolt.org.
Having something like https://rust.codehost.wavebeem.com/ or https://codehost.wavebeem.com/rust (or really anything my browser history can remember and autofill!) would be awesome.
Thanks again for a super useful tool!

I definitely won't do subdomains, but I might do something with URLs. I'd probably do https://codehost.wavebeem.com/?lang=rust
I was also thinking it might be good to remember the language you picked last, since I feel like most people are gonna be highlighting the same languages a lot...

Actually I think it could be good to remember your last theme and language. I've been wanting to add more themes too I just haven't worked on the site more than a couple times.

That would work too! I also usually change the theme, so either having it as part of a query parameter or a saved state would work perfect for me at least :)

the site now remembers your settings!
there's also a new theme (miasma), i updated the themes a bit, and fixed an issue with bold text being inherited

Thanks @wavebeem, it works great!


Requested Feature : Include Item Ids on SVG

TODO:
Pull Item ID List for DU-LUA.DEV
Combined into Recipe.Json

This is being pushed to master now