Standalone Binary Install

Not everyone will want to install SBCL and Quicklisp to run Coleslaw. Those users should just need a config file and the binary dropped on a server. We'll need new install instructions for that case too.
Let's get a standalone binary going with buildapp. How do we support plugins though? :-/

For this request, would it be better to start using Roswell instead? One could simply write a Roswell script that provides the base quicklisp script needed to deploy (I think Roswell already assumes or install quicklisp when you install it)

@rvedam I haven't used Roswell at all but open to it. If someone submits the PR (w/ the documentation) I would merge it. Don't worry about shrinking the executable size.
One way to use Coleslaw with gh-pages is to install ccl or another implementation on travis and run the post-recieve-hook as a shell script. We should add an walkthrough to the documentation

I just did it. See the PR https://github.com/kingcons/coleslaw/pull/146

Updates on this: I know some ppl don't like roswell for whatever reason, so I will make sure there is a corresponding command from REPL too. (wip)

I could update the Travis script to let Roswell prepare a standalone binary and upload it to releases.


Test build 5 4

Summary of Changes

Update to generate.py, to use rocm 5.4 and to install rocm (including migraphx) from repository instead of building migraphx from source code.  This is a merge of sample changes made by Chris Austen, with the current main branch.

This PR is not ready for merge.  It is created to facilitate debugging.
Changes:

changes URLs of apt and yum repositories for rocm
functions build_migraphx, install_migraphx_dev and install_migraphx_prod have been largely disabled by removing the code that performs builds.  What's left just installs from repository with apt or yum.  These functions can probably be merged if it can be kept compatible with the Python structure.
Ubuntu version is reset from 18.04 to 20.04  I'm not sure which is correct.
The changes were originally set to test the prerelease rocm version at http://repo.radeon.com/rocm/apt/.apt_5.4/   .  Since then, the public release at http://repo.radeon.com/rocm/apt/.apt_5.4/ was made.  The latter is included here but commented out.

Does it work?

The change was shown to work on server rocm-rome-6.amd.com with the .apt_5.4 repository.  I tested with the three migraphx scripts.
The change does not work on Brian's home tower.  The symptom is that within the Docker container, the user home directory is owned by root, not the user (use ls -l /home to see  this).  The command line used to start Docker was ./amdinfer run --dev --group-add video   --network=host  With this ownership problem, the server can't be built so it can't be tested.  This problem was thought to have to do with needing to give the user access to group video, but adding the command line switch --group-add video did not help.
The Docker change works, but rocm doesn't work,  on server rocm-rome-6.amd.com with the http://repo.radeon.com/rocm/apt/.apt_5.4/ repository.  The docker image runs and the server can be built, but when I tested the Resnet50 script I got the following errors:
amdinfer-dev:amdinfer$ amdinfer start gRPC server starting at port 50051 HTTP server starting at port 8998 MIOpen(HIP): Error [Compile] 'hiprtcCompileProgram(prog.get(), c_options.size(), c_options.data())' static_kernel_gridwise_convolution_forward_implicit_gemm_v4r4_xdlops_nchw_kcyx_nkhw_padded_gemm.cpp: HIPRTC_ERROR_COMPILATION (6) MIOpen(HIP): Error [BuildHip] HIPRTC status = HIPRTC_ERROR_COMPILATION (6), source file: static_kernel_gridwise_convolution_forward_implicit_gemm_v4r4_xdlops_nchw_kcyx_nkhw_padded_gemm.cpp MIOpen(HIP): Warning [BuildHip] /tmp/comgr-396113/input/static_kernel_gridwise_convolution_forward_implicit_gemm_v4r4_xdlops_nchw_kcyx_nkhw_padded_gemm.cpp:7:15: error: unknown type name 'FLOAT'      const FLOAT* const __restrict__ p_in_global,            ^ /tmp/comgr-396113/input/static_kernel_gridwise_convolution_forward_implicit_gemm_v4r4_xdlops_nchw_kcyx_nkhw_padded_gemm.cpp:8:15: error: unknown type name 'FLOAT'      const FLOAT* const __restrict__ p_wei_global,            ^ /tmp/comgr-396113/input/static_kernel_gridwise_convolution_forward_implicit_gemm_v4r4_xdlops_nchw_kcyx_nkhw_padded_gemm.cpp:9:9: error: unknown type name 'FLOAT'      FLOAT* const __restrict__ p_out_global)      ^ /tmp/comgr-396113/input/static_kernel_gridwise_convolution_forward_implicit_gemm_v4r4_xdlops_nchw_kcyx_nkhw_padded_gemm.cpp:149:13: error: use of undeclared identifier 'FLOAT'          FLOAT,       // Input data type          ^ 4 errors generated when compiling for gfx908. MIOpen Error: /long_pathname_so_that_rpms_can_package_the_debug_info/data/driver/MLOpen/src/hipoc/hipoc_program.cpp:304: Code object build failed. Source: static_kernel_gridwise_convolution_forward_implicit_gemm_v4r4_xdlops_nchw_kcyx_nkhw_padded_gemm.cpp MIGraphX Error: /long_pathname_so_that_rpms_can_package_the_debug_info/data/driver/AMDMIGraphX/src/targets/gpu/convolution.cpp:83: compute: MIOpen Convolution: running convolution using find_2.0 failed [amdinfer] [error] Failed to call function

The machine rocm-rome-6 runs Ubuntu 18.04.  My tower runs Ubuntu 20.04.
generate.txt

Updated from .apt_5.4 Rocm version to the official 5.4 release.  This was shown to work correctly for Chris Austen on rocm-rome-6; still troubleshooting Brian's issues.

We've identified the problem causing the long string of messages above (MIOpen problem).  The fix to this bug is scheduled to be released with Rocm 5.4.1.  There are two workarounds possible:

The error doesn't happen when loading a *.onnx model, only when loading a compiled *.mxr file.  Therefore, you can avoid it by deleting *.mxr files before each worker run.  Recall that the migraphx worker creates a *.mxr after compiling a *.onnx file, so this would have to be repeated each time the worker restarts or loads a different model.  Loading will be extremely slow and this is suitable only for checking out the code.
Revert to building migraphx from source (but with a newer commit of the source code if we like) with a build flag to avoid the bugged feature.  I'll have to find out what commit was used for the official repo version 5.4.  Going back to building migraphx from source gives up the primary motivation for this PR.

Instead of either workaround, I advocate letting this PR wait for Rocm 5.4.1 unless we have a critical need.

@bpickrel #97 will update the base OS but the migraphx changes will be a separate PR based on what the decision is about your findings above. If ROCm needs to be updated to 5.4, then your migraphx worker will also need to be updated to handle deleting the .mxr (or not creating it). If we continue to use a source build, then you'd need to find the commit to use. Or if we're waiting to 5.4.1, then the PR would wait and it would need to be tested internally until 5.4.1 is live.


update(circleci): multiple ci updates

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
What this PR does / why we need it:

Use cimg/go instead of deprecated circleci/golang images.
moreover, properly install goreleaser from deb repository
Finally, bump buildx version

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 07c221f8c5474d3e43c04b0d7d5770df43c8a973


Consider writing a detekt-internal rule for RuleNameIsTooLongAndThereforeShouldNotGetReleased

Context
Looking at the new rule DestructuringDeclarationWithTooManyEntries in context, it really stands out:

@Suppress("DestructuringDeclarationWithTooManyEntries") is 64 characters long, so it might be too long for some code styles. :)
Expected Behavior of the rule
Detekt on the detekt codebase should fail PRs with long names.
Each name should be easily remembered and easy to type for a better user (dev) experience.

I don't know about this, if there's a specific issue that this is causing we can take a look and consider renaming the rule, but in general I think contributors aim for succinct rule names where possible, and maintainers consider the implications of proposed new rule names before accepting contributions.
This particular rule name doesn't seem excessive to me, and truncating or shortening it may then trigger feedback that the rule name is not clear.
I'll leave it open for other feedback.

I prefer to have a clear name than a shorter name too. If the @Suppress is too long you can add an alias to the rule and use it instead.
I could also consider a rename if something clearly better is proposed but I don't think we need a rule for that.

It's really long, if you try to use it in a sentence or even try to talk about it (e.g. in documentation, or comments) you'll very quickly run into problems with understandability or style. Really, I challenge you to talk about this rule in a voice call 🤓.
For example compare these two paragraphs:

We use MagicNumber and ForbiddenImport to get a consistently readable and hopefully less buggy application. MagicNumber flags things that would be confusing to read and forces us to think about making a name for a thing; and ForbiddenImport documents our choices of libraries and conventions.

We can try to improve your style, Detekt's new DestructuringDeclarationWithTooManyEntries, UseRequire and UseCheckOrError will give you hints wherever we're using confusing structures.  DestructuringDeclarationWithTooManyEntries will flag too many component usages, as the order can change and break or code, let's prefer simpler x.val calls; the other Use... checks will flag unnecessarily used and thrown exceptions.

One thing to note is that I was trying to write this naturally and for the others I typed out the rule names, while this long one I had to copy-paste. Anther thing is that I can't write a nicely formed flowing sentence/paragraph because the "word" is just too long and wraps strangely.

Additionally when suppressed there's way less space for explanations:
@Suppress(
    "MagicNumber", // It's obvious what 1337 is, isn't it?
    "DestructuringDeclarationWithTooManyEntries", // In this case it's still acceptable, Vector4 structure will never change.
)

To suggest a better name: I would remove the filler words beating around the bush and use an English word that depicts the same "with too many entries" concept: LengthyDestructuring. Note that "Destructuring Declaration" feels in itself redundant similar to "DVD Disk", because a destructuring is always at a declaration (be it local variable or lambda parameter). Yes, there are other types of declarations, but "destructuring" implies declaration.

I think the point here is that I think there should be more challenging in the naming to find a better set of names that can be used by developer in speech and easily remembered. I would draw the line at most 4 words and a hard limit of 30 on top of this. Of course if there's no better alternative this can be suppressed as usual, but calling it out automatically and maybe documenting how to create more succinct and clear names in context would help to move the ecosystem forward. Naming is hard, yes, but also very important as public names are used in contexts you wouldn't imagine.

If the @Suppress is too long you can add an alias to the rule and use it instead.

+1 for adding LengthyDestructuring as an alias for this rule 👍

I'm closing this issue as won't do. I mean, we don't want a "max lenght" in our names. But I agree with many of the things that you pointed our @TWiStErRob. I would consider an issue to fix the name of DestructuringDeclarationWithTooManyEntries and change it for another one.  Lately there are some issues were we lack the option to deprecate a rule so if we implement it we could use it to fix this kind of things too.

Fair enough, feel free to move this as a discussion too. Thanks for listening.


Prototype of S3 ConsumerInterceptor

Description
Prototype to intercept consumer records to apply claim-check via AWS S3 before records reach sink task.
Test Steps
The connector configuration file e2e/connect-config.json is prepared for LocalStack. The threshold for the full Kafka message (as bytes) is 16 byte. Each Kafka message w/ a payload exceeding 16 Bytes will produce a S3 object with an UUID as key. The key is added to the Kafka message header s3 and send to EventBridge. The modified payload is null currently which should/must be corrected (not a tombstone).
Before send messages to the connector one has to create the S3 bucket example in LocalStack:
$ awslocal s3api create-bucket --bucket example

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue
#125
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

A screenshot of my manual test:


style: Fix jscs formatting issues on core.js

whatever tool you used (jscs?) broke the code.
please do not use that tool on beaker.

@scottdraves what part of the code broke?
The PR's goal was to update the syntax to follow javascript best practices (JSHint) and style (as per the google style guide).
It should have ideally not impacted the functionality, however since it seems like it did can you point out what broke and I can investigate. We can reasonably deduce that there is some other code that is relying on some global state mutation or "undefined" return that I can most likely fix.
Thanks

if you edit a notebook and then close it, a dialog comes up to ask you if you want to save.
the dialog was broken because the quotes around some html were mangled.

k thanks I will update


Grammatical Corrections

Few grammatical corrections

@AyushyaChitransh
Cello project use gerrit for code contribution: http://cello.readthedocs.io/en/latest/CONTRIBUTING.
Thanks!


cms decrypt failures

OpenSSL fails to fetch ciphers from engine:
OPENSSL_CONF=gost.conf openssl cms -decrypt -debug_decrypt -in P1323565.1.025-2019K.pdf.cms.kuznyechik-ctracpkm.agree -out out -inform DER -inkey Recipient_G2012_256_TK26_A.key -recip Recipient_G2012_256_TK26_A.crt -originator Originator_G2012_256_TK26_A.crt

The output is
Error decrypting CMS structure
00713D2FDF7F0000:error::CMS routines:cms_get0_signed:content type not signed data:crypto/cms/cms_sd.c:35:
00713D2FDF7F0000:error::digital envelope routines:evp_generic_fetch:fetch failed:crypto/evp/evp_fetch.c:319:Global default library context, Algorithm (kuznyechik-ctr-acpkm), Properties (<null>)
00713D2FDF7F0000:error::CMS routines:cms_EncryptedContent_init_bio:unknown cipher:crypto/cms/cms_enc.c:62:

The diagnostics about the content type seems not relevant at all, and the cipher is not fetched. It's a CMS-specific bug: enc command works well.

@slontis , could you please take a look?

I think I would need all the info you supply as input to the command in order to figure out what rabbit hole the engine code is supposed to go down..

Sure. Assuming you are able to build the gost-engine (if not, please let me know), I use the commands from the gost-engine folder:
./openssl_wrap.sh cms -verify -in tcl_tests/tc26_cms/signed_a111.pem -inform PEM -noverify

./openssl_wrap.sh cms -digest_verify -in tcl_tests/tc26_cms/hashed_a321.pem -inform PEM -out hashed_a321.out

./openssl_wrap.sh cms -decrypt -debug_decrypt -in tcl_tests/tc26_cms/encrypted_keyagree_a211.pem -inform PEM -inkey tcl_tests/tc26_cms/recipient512_key.pem -recip tcl_tests/tc26_cms/recipient512_cert.pem -originator tcl_tests/tc26_cms/sender512_cert.pem

./openssl_wrap.sh cms -decrypt -in tcl_tests/tc26_cms/encrypted_keytrans_a231.pem -inform PEM -inkey tcl_tests/tc26_cms/recipient256_key.pem

The openssl_wrap.sh script:
#!/bin/bash

OPENSSLDIR=`pwd`/../openssl
export LD_LIBRARY_PATH=$OPENSSLDIR
OPENSSL_CONF=`pwd`/engine.conf $GDB $OPENSSLDIR/apps/openssl $@

engine.conf file:
openssl_conf = openssl_def
[openssl_def]
engines = engine_section

[engine_section]
gost = gost_section

[gost_section]
engine_id = gost
dynamic_path = /path/to/gost.so
default_algorithms = ALL
CRYPT_PARAMS = id-Gost28147-89-CryptoPro-A-ParamSet

[req]
distinguished_name = req_dn
[ req_dn ]
C = RU
L = Moscow
CN=Test srv gost2012_256:XA
O=Cryptocom
OU=OpenSSL CA
emailAddress =<EMAIL_ADDRESS>

I will try to look at this tomorrow.. it is in my queue :)

The problem is in this section of code, that forces the use of a fetched implementation, unconditionally:
https://github.com/openssl/openssl/blob/c7dfb2abe5655ad1de7dfbdf5a9a6b0164f429cf/crypto/cms/cms_enc.c#L48-L71

Would this fix it on line 66?
if (fetched_ciph == NULL)
fetched_ciph = cipher

And then one line 71
if (fetched_ciph != cipher)
EVP_CIPHER_free(fetched_ciph);

The easier way is this, at line 61:
            if (fetched_cipher != NULL)
                cipher = fetched_cipher;

Then you don't need to check when freeing fetched_cipher

... and use cipher rather than fetched_cipher in the EVP_CipherInit_ex caall

Thanks @levitte
@beldmit try this out if you like as I will be going offline soon.. (its nearly 10pm here and I need to go and eat :))
You may submit a PR if it fixes it,  or I will do it tomorrow..

It seems it didn't help :(

Try this patch, I believe it's closer to the original intent of the code:
diff --git a/crypto/cms/cms_enc.c b/crypto/cms/cms_enc.c
index e25453ec9c..b366e260bb 100644
--- a/crypto/cms/cms_enc.c
+++ b/crypto/cms/cms_enc.c
@@ -54,16 +54,20 @@ BIO *cms_EncryptedContent_init_bio(CMS_EncryptedContentInfo *ec,
             ec->cipher = NULL;
     } else {
         cipher = EVP_get_cipherbyobj(calg->algorithm);
-    }
-    if (cipher != NULL) {
-        fetched_ciph = EVP_CIPHER_fetch(cms_ctx->libctx, EVP_CIPHER_name(cipher),
-                                        cms_ctx->propq);
-        if (fetched_ciph == NULL) {
+
+        if (cipher != NULL) {
+            fetched_ciph = EVP_CIPHER_fetch(cms_ctx->libctx,
+                                            EVP_CIPHER_name(cipher),
+                                            cms_ctx->propq);
+            if (fetched_ciph != NULL)
+                cipher = fetched_ciph;
+        }
+        if (cipher == NULL) {
             CMSerr(CMS_F_CMS_ENCRYPTEDCONTENT_INIT_BIO, CMS_R_UNKNOWN_CIPHER);
             goto err;
         }
     }
-    if (EVP_CipherInit_ex(ctx, fetched_ciph, NULL, NULL, NULL, enc) <= 0) {
+    if (EVP_CipherInit_ex(ctx, cipher, NULL, NULL, NULL, enc) <= 0) {
         CMSerr(CMS_F_CMS_ENCRYPTEDCONTENT_INIT_BIO,
                CMS_R_CIPHER_INITIALISATION_ERROR);
         goto err;

It makes the situation much better: only 633 failed tests instead of 710 :)

Decryption failures seem to be fixed at least partially.
These two still persists:
./openssl_wrap.sh cms -verify -in tcl_tests/tc26_cms/signed_a111.pem -inform PEM -noverify

./openssl_wrap.sh cms -digest_verify -in tcl_tests/tc26_cms/hashed_a321.pem -inform PEM -out hashed_a321.out

Should I move them to a separate issue?

Should I move them to a separate issue?

Nah...


Add support for REGISTRY parameter

On Wed, Dec 16, 2015 at 04:03:20AM -0800, Michal Fojtik wrote:

https://github.com/openshift/sti-perl/pull/64
-- Commit Summary --

Add support for REGISTRY parameter

No problem with it. I only wonder why you used

if [[ ! -z "${REGISTRY}" ]]; then

instead of

if [[ -n "${REGISTRY}" ]]; then

-- Petr

@ppisar yeah, that is better, I will change it. Thanks


webpack && bootstrap

webpack 'bootstrap-sass! ./ src / theme / bootstrap.config.js' ，  it is doing what?

That line is controlling how bootstrap-sass is loaded into the site. You can decide what bootstrap modules are included and excluded from the build to minimize size. Those options are specified in bootstrap.config.js. Personally, I opted to upgrade from bootstrap-sass-loader to bootstrap-loader which uses a .bootstraprc config file in the root.

Anyone else getting '[object Object]' instead of the actual style in development? Stuck with this bug

Such is the way css style embed html , which is the loss of the cache means, but how to link incorporated ?

This is just available in development mode as far as I know. It writes the style inline with the html helps to avoid the "flash of unstyled content" when doing a full refresh in development (see the helpers/Html.js file).

How to engage in the production environment ?

@jaraquistain would you mind sharing your implementation? I'm trying to do the same thing.

@josephdburdick sure, here are all the relevant parts of my webpack/dev.config.js and webpack/prod.config.js:
entry: {
    "main": [
      "bootstrap-loader/extractStyles",
      "./src/client.js"
    ]
  }

module: {
    loaders: [
      ...
      { test: /\.scss$/, loader: ExtractTextPlugin.extract("style", "css?importLoaders=2&sourceMap!postcss-loader!sass?outputStyle=expanded&sourceMap=true&sourceMapContents=true&includePaths[]=node_modules") },
      { test: /\.css$/, loader: ExtractTextPlugin.extract('style-loader', 'css-loader')}
    ]
  }

postcss: function() {
    return [autoprefixer({ browsers: ['last 2 versions'] })]
  }

I think that's all the pieces of it and they should be the same for prod/dev


Tidied modules in archive.py

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (d13a6fe) 15.79% compared to head (829ef5b) 8.66%.
Report is 5 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             main     #32      +/-   ##
=========================================
- Coverage   15.79%   8.66%   -7.14%
=========================================
  Files           8       8
  Lines        1076    1073       -3
=========================================
- Hits          170      93      -77
- Misses        906     980      +74

Files
Coverage Δ

bigplanet/__init__.py
100.00% <100.00%> (ø)

bigplanet/archive.py
9.32% <100.00%> (ø)

bigplanet/bigplanet.py
25.49% <100.00%> (ø)

bigplanet/extract.py
10.16% <100.00%> (ø)

bigplanet/filter.py
12.39% <100.00%> (ø)

bigplanet/process.py
3.48% <ø> (ø)

bigplanet/read.py
6.14% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


BEACH_ASSET_PROXY_RESOLVER

IP address of a DNS server to use for resolving domains when proxying assets. Set this to <IP_ADDRESS> when using Local Beach.

Merge https://github.com/flownative/docker-nginx/pull/14 before this one!


riak_pb_kv_codec.erl type dict() undefined while compiling

I cloned the Repo and tried it with make in the terminal but it throws this error:
/Library/WebServer/Documents/editionf/accounting-erlang/ChicagoBoss/deps/riak_pb/src/riak_pb_kv_codec.erl:52: type dict() undefined
repsonse of kerl_status:
Available builds:
18.0,18.0
----------
Available installations:
18.0 /Library/WebServer/Documents/editionf/accounting-erlang/ChicagoBoss
----------
The current active installation is:
/Library/WebServer/Documents/editionf/accounting-erlang/ChicagoBoss

System: Mac-OS 10.10

riak_pb doesn't work with Erlang 18
https://github.com/basho/riak_pb/issues/124

How does one do a make without having to compile riak_pb on Erlang 18, which does not work?

you can fork boss_db, comment riak as dependency and use link to your version of boss_db in CB dependency list.

latest master compiles on Erlang 18. Support for riak dropped for now

I have same error when run make in master branch and when checkout to v0.8.19

you cannot use riak_pb on erlang >=18

Someday wil be possible use? @danikp With new release of Riak, etc


Custom Serving Runtime creation

Description

Closes #1080
How Has This Been Tested?

Create a new serving runtime

Navigate to /servingRuntimes/addServingRuntime
Select 'Start from scratch' or upload the file
Create a new resource

Bad identation YAML

Navigate to /servingRuntimes/addServingRuntime
Select 'Start from scratch' or upload the file
Add an indentation error
Check that it fails and a new alert is displayed

Bad resource error

Navigate to /servingRuntimes/addServingRuntime
Select 'Start from scratch' or upload the file
Add other resource rather than a ServingRuntime
Check that it fails and a new alert is displayed

Test Impact

Tests not implemented still, waiting on integration from previous PR
Request review criteria:

[X] The commits have meaningful messages (squashes happen on merge by the bot).
[X] Included any necessary screenshots or gifs if it was a UI change.
[X] Included tags to the UX team if it was a UI/UX change.
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has manually tested the changes and verified that the changes work
[X] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

lgtm!


Required Help to Integration MS Bot framework + Node JS + Heroku

Hello Team
Please provide me guidelines to do the above mentioned combination that i am using Microsoft bot framework in node js language where i am planning to deploy it in heroku but i am facing the issue that keep on getting same "\api\message" that is only greeting message none other messages i am getting :-(

Can you share your code?  I've had Heroku working before. The only thing I had to make sure of was that my package.json was setup to properly start my bot and that I had my appId and appPassword for the connector properly set. I am planning to do a guide for deploying to Heroku but not sure when

Hi @vignestheo, can you share your code?

Closing for age

Would be nice @Stevenic If you could demonstrate or document a guide through for Microsoft bot framework on Heroku. Not a lot to find about this build.


JMAP.js: parseStructure() differs from spec

In the spec is shown an example of parseStructure().
I see the same function in the JMAP.js but it very differs. Could you please say which one version is better and newer?

There's no one "right" answer here, hence the spec only giving the code as a starting point, however the version in JMAP.js is the one we currently use at Fastmail in our UI.


Incorrect indentation when treating cluster as an ordered array

Before fix:

After fix:

Fixed in next release

Hi Jim,
The test case for issue #31 is missing from branch master, but "Test Github Issues.vi" references it.


How to use string variables in datatable in if statement? - user found solution

Hello,
I'm trying to use variables in datatable in if-statement.
Ex."size" in datatable.('S'=small, 'L'=large )
What I want to do is
if size equals to "S"
	echo This is SMALL
else
	echo This is LARGE

I have tried various ways, but not worked.
If anyone know solutions, please share your solutions.
Thank you.

Hello,
After checking a log of js file. I found a solution.
if "`size`" equals to "S"
	echo This is SMALL
else
	echo This is LARGE

I hope this help somebody in future.
kensoh, My prayers with you and family.

Thank you for that gem3

Thank you @gem3... Sorry I was away the past few months - https://github.com/tebelorg/RPA-Python/issues/144
Yes that's the syntax for comparing a variable as a string in conditions.

Adding on, you can use double or single quotes in TagUI -
size = 'S'

if '`size`' equals to 'S'
    echo This is SMALL
else
    echo This is LARGE

I personally prefer single quote because I don't need to press shift key each time I type the quote. This tiny time-saving each time a quote is typed will be compounded info significant time savings over a long period of time.

Helloo kensoh,
I have to press shift key to type double or single quotes with a keybord in my language.
this is a keybord layout problem with my language.
btw are you ok now?
I don't know whole your story, but I'm glad to know you are back.

Oh I see.. Then there is no difference using double or single quotes, I understand now...
Thank you..... My mum has just been discharged to go home, after 6 months in hospital. Condition is still bad but we are trying to do what we can to help her recover and not be dependent on oxygen supply through a hole in her throat. It looks like going to be a long journey ahead.
Plan to send 18-month old baby to childcare failed. On first day got a couple of accidents already. Now trying to hire a helper to help with housework, cooking and taking care of baby. So that I can start work again. It's tough being sandwiched with the demands of taking care of a young baby and my mum's illness. But there's no other option than facing it and marching on.

Hello kensoh,
It's hard to find words for you.
But I like the story of "塞翁失马，焉知非福".
"There is no unstoppable rain."

Thank you @gem3 I appreciate your kind words of encouragement in these dark times 🤞


No Audio Transmitted

Hello
I was previously running an old version of sip-ua(0.2.2) with it working perfectly on custom application.
I have upgraded to use the current master branch of the project and all is going well except audio transmission
Audio not transmitting on both iOS and Android.
I have noticed below and I am suspecting that is where the issue is arising from?? not sure.
`
flutter: [2021-04-28 21:13:30.403] Level.debug invite_client.dart:82 ::: Timer B expired for transaction z9hG4bK1578273830000000
flutter: [2021-04-28 21:13:38.855] Level.debug invite_client.dart:71 ::: Timer M expired for transaction z9hG4bK1578273830000000
flutter: [2021-04-28 21:14:22.566] Level.debug non_invite_server.dart:35 ::: Timer J expired for transaction z9hG4bK9cf4.f7237cc6.0
`
If there are any insights on how to solve problem, your assistance would be appreciated.
Call Flow:
CLIENT(Dart-SIPUA) --- wss ---> OPENSIPS --- sip ---> FREESWITCH ---> END_USER
Full Call Trace:
https://pastebin.com/yhbu44FW

Hi there.  I am a freelancer  and I need  assistance with this plugin.
I have been provided with the websocket uri,  sipURI,  password  and sorts. And when I use it then there is no connection,
Could  you perhaps show or at least tell me what I should expect in the debug output if it had connected or not


reconfirmable ignored

The short story:
I activated config.reconfirmable = true, but when I update a user, it updates the email without copying email to unconfirmed_email and it doesn't email anything. Basically, it does not respect config.reconfirmable = true. I restarted the server, and was trying to debug for two days.
The details:
I am not able to make devise send reconfirmation emails. It sends sign up confirmation emails just fine, but when I update the user's email, it does not work for me.
I reverted to Devise's own registration controller by removing all my custom code to debug this
# registrations_controller.rb
class Users::RegistrationsController < Devise::RegistrationsController
end

and even suspended my own user edit view, so it's now using the basic Devise Edit form.
I have this in devise.rb initializer:
config.reconfirmable = true
config.allow_unconfirmed_access_for = 2.days and confirmation_sent_at = Time.now

My database fields:
# user.rb
field :confirmation_token,                 type: String
field :confirmed_at,                       type: Time
field :confirmation_sent_at,               type: Time
field :unconfirmed_email,                  type: String

and I have these Devise settings:
devise :database_authenticatable, :registerable,
       :recoverable, :rememberable, :trackable,
       :confirmable,
       :omniauthable, :omniauth_providers => [:facebook]

My routes are set as follows:
devise_for :users,
  controllers: {
    omniauth_callbacks: 'users/omniauth_callbacks',
    registrations: 'users/registrations',
    sessions: 'users/sessions'
  }

The unconfirmed_email never changes, regardless of how often I update the email. The email gets updated actually, which is not logical if it was not confirmed.
What am I missing? How do I debug this?

What Devise version are you using? Can you please provide a sample application that reproduces the error?

I'm using devise 3.5.6

@whoteaches Are you still having this issue? Could you provide a sample application only with rails and devise that reproduces the error?

@whoteaches I'm closing this issue. Feel free to reopen when you can evidence the error with sample rails app.
Best

I'm having the same problem on devise (3.5.8) and also after upgrading to devise (4.1.0)

Rolled back to 3.5.2 which we had previously and no problem there.

Thanks, good to know. I'm reopening this issue. If anyone knows how to reproduce feel free to share with us.

Having this same issue after upgrading to 4.1.0 from 4.0.1.
I'll try to put a small PoC Rails app together this week to isolate the issue.

@shaneog That's weird the release 4.1.0 the only thing that changed about the reconfirmable config is its default value (now is true). Did you have news about this issue?

Hi @ulissesalmeida - the error is definitely occurring for me, and only with 4.1.x (and not 4.0.3), but it's to do with something unique in my controllers I suspect.

@ulissesalmeida After spending most of today digging into this issue (it only happens in my test (rspec) environment, in development and production it's fine 😩 ), it turns out that someone better than me has fixed it in #4125. 👍


remove: unused functions

What changes have been made
This pr contains the removal of unused functions and tests within instascale.
Including:

checkExistingMachineSet
getPodResourcesWithReplicas
MachineClientOrDie
NewClientBuilder
GetListOfPodResourcesFromOneGenericItem
getRestConfig
And their related unit tests

Verification steps

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

thanks for cleaning up the functions. I would like to think most functions are deprecated and similar functionality exists in the modified codebase.
/LGTM


M1 Apple Silicon, run_tests fails

Error:
#9 9.370 Collecting qrcode[pil]~=6.1
#9 9.410   Downloading qrcode-6.1-py2.py3-none-any.whl (31 kB)
#9 9.515 ERROR: Could not find a version that satisfies the requirement aries-askar~=0.2.4 (from versions: 0.1.0, 0.1.2)
#9 9.515 ERROR: No matching distribution found for aries-askar~=0.2.4
#9 9.654 WARNING: You are using pip version 21.1.3; however, version 21.3.1 is available.
#9 9.654 You should consider upgrading via the '/usr/local/bin/python -m pip install --upgrade pip' command.
------
executor failed running [/bin/sh -c pip3 install --no-cache-dir         -r requirements.txt     -r requirements.askar.txt       -r requirements.bbs.txt         -r requirements.dev.txt]: exit code: 1

run_tests fails on my new mac with an M1. Has anyone had any success developing with Apple Silicon?

PR, #1697 gets tests running locally for me.


query Control Flow from Function missing from predefined node groups

According to the list of predefined RACK queries, there should be a query called query Control Flow from Function - however, there doesn't seem to be a corresponding nodegroup in RACK/nodegroups/queries.

We're working on a way to automatically update the predefined RACK queries wiki page. Stay tuned.

The wiki page is now auto generated.


Add 4/5 of the new macs

See https://github.com/servo/saltfs/issues/674#issuecomment-303591568.
Test that the 4 completed new builders are actually working, and free up a bit of capacity, while I finish up with mac9.
Also format these lines to have less icky diffs as we add and remove others.

This change is

You can r=me after it passes :-)

[ FAIL ] Buildbot master config lint check failed:

         ./buildbot/master/files/config/master.cfg:21:14: E124 closing bracket does not match visual indentation

oh gosh dangit it passed flake8 :/ fixing

That works for me. Can either r+ or squash and r+, whichever you prefer.

@bors-servo r=larsbergstrom

:pushpin: Commit 21b277f has been approved by larsbergstrom

:hourglass: Testing commit 21b277f802bd8cab773d9dbdf1b42db5ef81ce29 with merge d8f1e02cf1d494f2d93b1235224b5a33bd006959...


Core.js not found, but it's actually there in node modules.  Meteor client is fine. Ionic is broken.

The errors occurs when importing the meteor-client.js.  When loading ecmascript-runtime-client, it suggested: The core-js npm package could not  be found in your node_modules directory.  Please run the following command to install it: meteor npm install --save core-js.
Actually, I did install it on both meteor's node_modules and ionic's node_modules.I launched the meteor client, it is fine. While in ionic, the compile process goes well, but with the error displayed in the browser.  I was stuck there for 3 days, I deep dived into src. I am still not sure whether it came from webpack or meteor-client-bundler or anything else.   I surfed and found no any answer in online resources.
The error stack:
Error: Cannot find module 'core-js/modules/es7.object.get-own-property-descriptors'
at makeMissingError (http://localhost:8100/build/vendor.js:125256:14)
at Module.require (http://localhost:8100/build/vendor.js:125275:19)
at _require (http://localhost:8100/build/vendor.js:125291:25)
at modern.js (http://localhost:8100/build/vendor.js:128255:15)
at fileEvaluate (http://localhost:8100/build/vendor.js:125369:9)
at Module.require (http://localhost:8100/build/vendor.js:125272:16)
at _require (http://localhost:8100/build/vendor.js:125291:25)
at http://localhost:8100/build/vendor.js:128275:18
at Object. (http://localhost:8100/build/vendor.js:128279:3)
at Object.defineProperty.value (http://localhost:8100/build/vendor.js:214805:30)

Could you reproduce a repo?

I made this repo:  https://github.com/dc165015/issue
Thanks a lot!

The repo looks empty, could you check?

I tried either to make a node_modules dir for api or link the node_modules dir of project root folder.

You must have a node_modules symlink in your Meteor project folder. That's the problem.

@dc165015 I had the same issue. I'm using Ionic 4, Angular 6 and Meteor 1.7. I have a node_modules symlink and I installed core-js in it. That didn't work.
I finally was able to fix it by adding "npmPackages": [   	"core-js",   	"<EMAIL_ADDRESS> ] to meteor-client.config.json
After adding "core-js" I got another error which I fixed by adding "<EMAIL_ADDRESS>I also have this "generateNodeModules": true in case that makes a difference.
I don't know if this is the best way to fix it but it's working for me and it doesn't seem like a crazy hack workaround.

@jimenezff  THANK YOU VERY MUCH! You saved my days ...

@dc165015 I'm glad I was able to help :)
Now I have to figure out why my build fails when I add "@types/meteor" to tsconfig.json
I get a bunch of [ng] node_modules/@types/meteor/mongo.d.ts(344,9): error TS1005: ',' expected.

I bumped into this error ever. This is caused by the mess when you mix " meteor npm install " and " npm install".
I found 1 difference is they have different version. I can tell the other differences for there is no detailed doc.
You have to either use meteor npm install or use npm install  to avoid the inconsistence.
To fix this, you may remove node_modules and package-lock.json, then try again.

@KeithGillette  this meteor-client-bundler is a black-box, no detailed doc and very tricky. It did trap me days and days...

Please note, core-js v3.x is not compatible anymore with meteor here: https://github.com/meteor/meteor/blob/3051150f2f5ae953f391802e73682fba613b3d46/packages/ecmascript-runtime-client/modern.js
You need to install v2.6.5.
I updated my package.json to:
"core-js": "~2.6.5"
All works after that.

I had the same problem also with the proposed fixes.
I could solve it by upgrading the project to meteor 1.8
You can do this by running the following command in this folder:
meteor update --release 1.8


Support S3 endpoint

Description:
Config file in sample notebook has property of output_path for protobuf profiles, but I can’t find that config for json and specify minio endpoint url.

S3 private endpoint is another use case: https://docs.aws.amazon.com/vpc/latest/privatelink/vpc-endpoints-s3.html
Reference code:
https://github.com/whylabs/whylogs/blob/c31494addb9576a48b6847c7899c6830417adb12/src/whylogs/app/writers.py#L365
smart_open + minio: https://github.com/RaRe-Technologies/smart_open/issues/416
transport_params = {
    'resource_kwargs': {
        'endpoint_url': 'http://localhost:9000',
        'aws_access_key_id':'minioadmin',
        'aws_secret_access_key':'minioadmin',
    }
}
with open('s3://data-test-bucket/test.txt', 'wb', transport_params=transport_params) as fout:
    fout.write(b'Yay I work on S3 on minio!')

We'll have to support transport_params somehow with smart_open

This is one of the great feature!
Supporting s3 object store is most helpfull.

From Slack:

I want to give following parameters which are not supported by current whylogs.
endpoint_url
region_name
Certification file path of s3(minio) server
Certification file path of client
Intermediate certificate file path

Fixed in #277


Pretrained models

thanks a lot for sharing yr code.I spent many time to use my device to train.Can you publish your pretrained models.

@lhr18716032695 do you have the pretrained models?thank you very much!

@MathiasGruber where can i download the pretrained models? i need your help,thank you !

I've updated the readme with links to pretrained networks. I've not gotten them to completely converge though, as I'm also somewhat limited on computational resources.


Print top-level tasks with their main line at the bottom

This makes it more obvious when it completes. Also fixes build to say
it's completed.

Hooray!


[BUG]: multi machines, multi cards training is stucked while "Launch model Parallel"

🐛 Describe the bug

Environment
pytorch==1.11, cuda==11.3, colossalai==0.3.0
transformers==4.28.1

May I take a look at your script commands?

🐛 Describe the bug

Environment
pytorch==1.11, cuda==11.3, colossalai==0.3.0 transformers==4.28.1


Starting the CN5G docker error

I am getting this error when trying to run oai-cn5g.
Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/home/jschroedl/oai-cn5g-fed/docker-compose/conf/config.yaml" to rootfs at "/openair-nrf/etc/config.yaml": mount /home/jschroedl/oai-cn5g-fed/docker-compose/conf/config.yaml:/openair-nrf/etc/config.yaml (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type  Incorrect/Unsupported executing command docker-compose -f docker-compose-basic-nrf.yaml up -d
Somewhere in the setup a file is not copied to the correct location.

Hi there!
Sorry for the late response, this error means that the config.yaml file is not located at the corresponding directory.
are you sure that you have the config.yaml at /home/jschroedl/oai-cn5g-fed/docker-compose/conf/?
Also:

Check that you have the corresponding directories and also permissions needed for running docker-compose
Review also docker-compose-basic-nrf.yaml

I've seen that OAI have been changing stuff in the repository (links, manuals) I don't know if they've also modified some configuration files.


Fixing issue 101: "}" symbol breaks the template inside of a string

Fixes issue #101 by implementing a ~20-line parser (no regular expression can be used solve this problem).

Thank you for your contribution. Release 1.0.7 is on it's way.


Installation documentation note

Installation instructions from README:
npm install cash-dom --save-dev
Why save-dev? Is it not meant to use in final production build? As it is front-end dom manipulation library, it is not devDependency.

right, fixed 👍


PHP code should not require mapping decodeSegmentsJson() across OrderedStation objects

This issue doesn't exactly duplicate #2, but will become unnecessary if #2 is completed.
Currently when the ORM (Respect\Relational) creates an OrderedStation object it populates the segment_xxcolumns auto-magically, however we have to manually call the decodeSegmentsJson() function on the object to decode the segment_xx columns into the geojson_xx columns.
The object, when instantiated, should automatically do this decoding, so that the controller never has do anything to "prepare" the object to actually be usable. Normally this would be done by using a constructor, but it seems that what the ORM is doing is creating the object, then manually populating the properties, which means that decoding during __construct() will not work, since the object doesn't contain data.
One possible workaround is to use the __call() method to intercept the call to set the property, but this doesn't seem to be working either.
Need to investigate if respect\relational has some sort of "object is ready" callback we can use, or if we can override the property somehow to decode itself whenever called.

It appears that PHP must manually call the decode_json() method on the strings returned to it from Postgres, so we need to take apart our entity classes to see how we can try and get this decode to happen manually. (We want it to happen every time an object is fetched from the DB, the controller code should not have to map this method across all results!)

Resolved when we moved the DB logic into the entities. Now the entity manually maps this during it's fetch() methods. This is as good as we're going to get.


Questionnaire Electronic Transition of Care (David Gutknecht - openmedical AG)

ch.fhir.ig.ch-etoc#1.0.0 /Questionnaire-QuestionnaireEtoc.html
diagnosisList.primarydiagnosis -> diagnosisList.primaryDiagnosis
to be consistent with
diagnosisList.primaryDiagnosis.item
Also same remark for diagnosisList.secondaryDiagnosis
There is also a tipping error on
attachtment.title -> attachment.title
David Gutknecht - openmedical AG

typo and camelCase corrected


init: ported Get_String_Encoder primitive function

Since level-2 libraries use certain string functions, we have to port their primitive functions into HestiaKERNEL. Hence, let's do this.
This patch ports Get_String_Encoder primitive function into init/ directory.

skip review since it's a port and is not affecting the production implementations.


Proposal for issue templates

Starting with the OpenCue templates as a starting point.
https://github.com/AcademySoftwareFoundation/OpenCue/tree/master/.github

This is a great start!
Since users may be unaware of C++ compiler info (like when using pre-built wheels), or if they're working with C++ only python version may not be relevant, maybe we should add an "(if applicable)" to those in the bug template just to let people know we're not too stressed about it.
Also, how much do we worry about classifying feature vs. enhancement? I feel like sometimes to users the differentiation between these may be a little tough to work out - we often identify these in triage (like adding descriptive test to Markers where some research and discussion was warranted).

I was cribbing off OpenCue, which itself is cribbed from somewhere else :)
Worth considering starting with just Bug and Feature, since those are the two that github supports by default (https://docs.github.com/en/github/building-a-strong-community/configuring-issue-templates-for-your-repository#creating-issue-templates)
With respect to asking users to supply compiler information ~ what actually happens if you try to run pip install on a machine that does not have a compiler toolchain installed by default? Windows/MSVC or Mac/clang are obvious examples. If pip install can somehow wrangle a toolchain, then I don't think we have to ask, but if it is a prerequisite, then we'd probably end up asking the question as a follow up.
Since I suspect the latter, why don't we modify the template to say something like:
on Linux, type gcc -v, and paste the results here
on Windows, type cl, and paste the results here.
If you are unable to determine your compiler via these commands, please indicate that here.

Since users may be unaware of C++ compiler info (like when using pre-built wheels)

That suggests more; the templates could be broken up by scenario

pip and a wheel
C++ only
building locally for Python


APNG Disassembler

APNG Disassembler version？

The embedded modified APNG disassembler is based on APNG disassembler version 2.6.
https://github.com/Benny-/apng2webp/tree/master/apng2webp_dependencies/apngdisraw
Is this the answer you where looking for? (Please close the issue if so)


combine NWL scripts into single script with different modes

Instead of different small scripts that call different NWL capability, use argparse to define different modes that will launch different capability.

Has this been addressed?

I think this was addressed in #100


be a part of zsh-users zsh-completion ?

i don't know if you're aware of https://github.com/zsh-users/zsh-completions/tree/master/src

My github has been in limbo for a few years now, but I'm back from the dad. Let me see if I can get this added to trunk.


refactor: rename new attribute on oci_tarball

Followup to #441. I think 'command' is too vague, as it might seem to apply to the creation of the tarball under 'bazel build', and also doesn't say anything about what purpose the command has. Also improve the docs around this pair of attributes.
Note: not a breaking change as the 'command' attribute wasn't released yet.

Thank you for grabbing!


special.factorial test failure with Python 3.9

With Python 3.9a3
_____________________________________________________________________ TestFactorialFunctions.test_mixed_nan_inputs ______________________________________________________________________
scipy/special/tests/test_basic.py:1835: in test_mixed_nan_inputs
    result = special.factorial(x, exact=True)
scipy/special/_basic.py:2354: in factorial
    val = math.factorial(un[0])
E   DeprecationWarning: Using factorial() with floats is deprecated

@rlucas7 looks like our abstract discussion of whether to ban floats in this case has become less abstract.

@person142 yeah, that's going to be fun to fix :)
I guess we should start with a deprecation warning?
I'm assuming we want to also deprecate float inputs-IIRC we both wanted to deprecate but it was messy...
for ref: https://github.com/scipy/scipy/pull/11254
I think the warning or value error was getting consumed in the build and ignored by cython:
https://github.com/scipy/scipy/pull/11254#discussion_r365528511
I'd be +1 for adding deprecation warning for floats in the upcoming release.

It seems to only deprecated in Python 3.9a3 at the moment, so we may have a while?

@andyfaff that is true, it doesn’t look like 3.9 is released until October 2020 so that deprecation won’t go in effect until after the next scipy release. I looked around and couldn’t find the deprecation policy for the math library, not sure how long we have to deprecate, I’d guess until 4.0 python release.
Sincerely,
-Lucas Roberts

On Feb 24, 2020, at 11:13 PM, Andrew Nelson<EMAIL_ADDRESS>wrote:

It seems to only deprecated in Python 3.9a3 at the moment, so we may have a while?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

sorry, but i'm here just to complain, why keep changing the position of this function from scipy.misc.common -> scipy.misc -> scipy.special ?

scipy.misc is being emptied

How does one install scipy on Python 3.9 release candidate 2 on Ubuntu?

This was a duplicate of a couple of other issues. Closing this; remaining open issue is gh-13122 and PR (with discussion) is gh-13121.


Which version do you need to use for this? 1.0.0 or 1.1.0?

Basically title

for the zelda one*

1.1.0

1.1.0

I haved tried it on both 1.1.0 and 1.0.0 and they work on 1.0.0 but not 1.1.0 (Specifically on Ryujinx 1.1.760)

someone was able to get it working on 1.1.0? in the previous version if it worked

I've tested it, and others have reported it working... It's most likely a user error. Please read Emu FAQ's from the emu makers on how to install properly.


Enforce broadcasting

Related issues: #23

This will break piping:
@. cube |> sqrt |> abs
# ERROR: MethodError: no method matching length(::DataCube)

@. abs(sqrt(cube))
#openEO DataCube
#   collection: SENTINEL2_L2A
#  bands: ["B02", "B04", "B08"]
#   spatial extent: BoundingBox{Float64}(5.14, 51.17, 5.17, 51.19)
#   temporal extent: ("2021-02-01", "2021-02-10")
#  license: proprietary
#   connection: https://openeo.dataspace.copernicus.eu/openeo/


win-15 is low on disk space for local Temp directory.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
win-15 is low on disk space for local Temp directory.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Removing old directories in C:\users\swarming\AppData\Local\Temp\ to free up disk space.


openrazer-daemon: make build src name independent

Motivation for this change
Cherry-picked from #153386
Source names are kind of an implementation detail, and should probably avoid hard-coding conventions.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

change was made in e2c79b3e4a2d9e9ef4072417be10a8faf888d081


Add bigint-backed Node

Motivation
https://github.com/ChainSafe/lodestar/issues/2885
One strategy for lowering memory footprint of our merkle tree is to replace the many Uint8Arrays of the tree with BigInts.
Description
Add an additional Node/Branch/Leaf implementation that is backed by BigInt instead of Uint8Array.
Explore arising issues.
Notes / issues:

The implementation is built along side the existing implementation, and is not currently exported. It also isn't integrated into the subtree-generating code (eg: sutbreeFillToDepth). This is because the bigint-buffer and bigint-hash libraries don't work in the browser, but rely entirely on native bindings! We need to retain browser compatibility in this library to facilitate proof consumption and other browser use-cases.

Resolving this may take several forms:

improve bigint-* library browser compatibility
selectively export or use Uint8Array-backed Nodes vs BigInt-backed Nodes
Resolve all issues with BigInt-backed Nodes and fully remove Uint8Array-backed Nodes

The bigint-hash library provides a hasher that can natively return a BigInt, however it does not currently accept a BigInt (or multiple BigInts) as input. Consequently, BigInt inputs must be converted/copied into a Buffer before hashing. This causes significant re-work and should be improved.

Nice work to start! We really only want to use bigint for NodeJS when the native bindings for conversions are available. Otherwise it's too slow in browser to convert back and forth. We could use a switching pattern like in chainsafe/bls to alternate between both approaches under a flag or some global thing that then we can enable in lodestar land

@wemeetagain  it looks like we need to import node definition from nodeBigint in tree.ts

|#|master|8 numbers|bigint|

closing in favor of #52


add release 1.3.0 info

OPS Build status updates of commit edbd92a:
:clock10: Preparing: average preparing time is 42 sec(s)

OPS Build status updates of commit edbd92a:
:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in http://SiteHelp and include the URL of publishing report as well as below information: One or more errors occurred..

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.

OPS Build status updates of commit 0e5b86f:
:clock10: Preparing: average preparing time is 42 sec(s)

OPS Build status updates of commit 0e5b86f:
:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in http://SiteHelp and include the URL of publishing report as well as below information: One or more errors occurred..

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.


Cancel_return issues

Hi,
I have intergrated mini cart into my website, i have 2 drop down options with add to cart buttons for each, once customers have added what they would like they click checkout and it goes through to paypal all fine and dandy, however when i try to test the cancel and return to feature it doesn't seem to work, it just returns them to the standard URL for my website, any ideas why?
My site is www.randomrallies.co.uk
Here is the code im using:

T-Shirt Options
No T-Shirts Thanks £0.00
1 T-Shirt £10.00
2 T-Shirt £20.00
3 T-Shirt £30.00
4 T-Shirt £40.00
5 T-Shirt £50.00

Sizes e.g 2 x L 1 x M 1 x S

Team Entry Options
2 Person Team £130.00
3 Person Team £155.00
4 Person Team £180.00
5 Person Team £205.00

I have figured out the problem, and it was nothing to do with minicart, just my plain dumbness! Firefox had chached the site, and was therefore not going to the updated link, but the old one! My bad!!


refactor: Refactor exception control file structure

Pull Request Test Coverage Report for Build #154
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.568%

Totals

Change from base Build #152:
0.0%

Covered Lines:
369

Relevant Lines:
458

💛  - Coveralls


[TS][TypeError] JSONValue doesn't accept optional keys

https://www.typescriptlang.org/play?#code/JYOwLgpgTgZghgYwgAgFIGUDyA5TAjAKwgTGQG8BYAKGVuQG0APALmQGcwpQBzAXVYw4AanAA2AVwgBuagF9qoSLEQpB2AIJQocAJ7IIjSCAAmbZJu06APGpESIAPnJzqYHQAdVWbHcnIAvOycPMgAPsgg4gC2eNBhyHgA9omiEHAg8Wr4RCSZ3ha6MlQK4NDwSMhROujQAG7ASABKEGzuiSBsEAAiLQhc7mDA7c40yCxBXCDc1LQ6APysHJPTVPJUbp6V1XUNEM2t7Z0BI7QgcFEQi8FTM8jGvf2D7QtbNVD1TS1tHd0PwANDEAuYpUGDiEAkQFbABi4IQAAoAJQCby+FCUUYIQ6kKrqADucGAkGM+2+nVYVTeHz2X0OKECGLoEXOl2QAHJIBw2QAaW60e5sPr-J4gViMpljVgclpgHl8ug6Vjg+4wUAQYzyta0Na3KAQMDiKAZXEEonq0l0uRAA
I am trying to use h3 to return some responses from a service (in my case, it's contentful). Of course some values from the service can be undefined, because in the CMS they are not required, but JSONValue (more specifically, JSONObject) doesn't accept undefined values.
interface JSONObject {
    [x: symbol]: JSONValue;
}

This fixes errors, but I'm not sure it's correct.
@antfu cc

@pi0 sorry for bothering you again on this, would you mind releasing 0.7.6 when you have some spare time? Thank you 💚


[PREVIEW] ISPN-11303 Scaling without state transfer

https://issues.redhat.com/browse/ISPN-11304

Separate module: anchored-keys
Works on top of an INVALIDATION_SYNC cache
Keep a mapping of keys -> nodes in a replicated cache
When inserting an unmapped key, map it to the newest member
When updating an existing key, keep it on the same node

TODO

Use cache topology instead of JGroups view to determine current writer
Add support for operations other than put and get
Optimize storage, currently the full address of the owner is stored in
the anchor cache (20 bytes when serialized)
Support deserializing configuration from JSON

I've merged this as-is since it lives in its own module.

I won't close the Jira.


wsk project deploy fails if ~/.wskprops files defines a namespace

This is based on work related https://github.com/apache/incubator-openwhisk-catalog/pull/268.
To reproduce, create a ~/.wskprops file like this, for example:
> cat ~/.wskprops
APIHOST=localhost
AUTH=23bc46b1-71f6-4ed5-8c54-816aa4f8c502:123zO3xZCLrMN6v2BKK1dXYFpXlPkccOFqm12CdAsMgRU4VrNZ9lyGVCGuMDGIwP
INSECURE_SSL=true
NAMESPACE=guest

Then try to run wsk to project deploy. The cli is picking up the NAMESPACE property and trying to install actions in the wrong namespace. This will fail since the key on the command line does not match. AFAIK there is no way to override the namespace property from the command line.
Info: The API host is [<IP_ADDRESS>], from wskdeploy command line.
Info: The auth key is set, from wskdeploy command line.
Info: The namespace is [guest], from .wskprops.

which later leads to errors like:
Error: Deployment did not complete sucessfully. Run `wskdeploy undeploy` to remove partially deployed assets.
servicedeployer.go [1660]: [ERROR_WHISK_CLIENT_ERROR]: Error code: 147: The supplied authentication is not authorized to access 'guest/watson-textToSpeech'. (code 702f9dfbfc752ee58b0546d6a15d6cb2): HTTP Response Status: 403 Forbidden HTTP Response Body: {"code":"702f9dfbfc752ee58b0546d6a15d6cb2","error":"The supplied authentication is not authorized to access 'guest/watson-textToSpeech'."}

This is related to #809, lets add support to read env. variable WSK_CONFIG_FILE in addition to what we have today with --config, here is the precedence order (highest to lowest):

--config
env. variable WSK_CONFIG_FILE
$HOME/wskprops as a default

@pritidesai I'd like to resolve this issue to land a patch before the release I'm going to push next week. Do you think we can fix this before then? Do you have any bandwidth to work on it?


Docs are missing

I just recently started to using IBM Mobile first SDK for Cordova and Mobile First Server. When I began my research on the topic the docs had much more rich information in various topics. The documentation for MVVM Pattern is missing, as well as the API specification (I know there is one but the previous one was much more detailed), also the section for hybrid UI is missing; just to mention a few.
Where they moved to other site or link? Are you going to put them back?
Thanks

Never mind. I got confused with other site. 👍


Specify installation for Linux, Mac OSX, and WIndows

We don't differentiate well between operation systems. http://cookiecutter-django.readthedocs.org/en/latest/developing-locally.html is for Linux, but doesn't do much to help Mac OSX or Windows users.

Alongside this, probably want to define a requirements.rpm and requirements.brew for install_os_dependencies.sh

If you do:
pip install unixpackage

and then:
unixpackage install -r requirements.apt

It should find and try to install the relevant equivalent packages on RPM systems, Arch and Brew.
This doesn't help Windows users though.

I can see three ways to improve our support for Windows:

Create our own chocolatey package to add those binaries/libraries that need separate installers for Windows. Right now I can only think of psycopg but this will future proof us. Windows 10 has built-in support for external package repositories (and it integrates with chocolatey), and for Windows 7/8 users they can install chocolatey to download the package.

Advocate the use of docker as our compatibility later for Windows. This has its own challenges; in that docker is rapidly updating and especially on Windows a lot of things are still in early beta or can cause unexpected errors. Advantage to this approach is that it takes away a lot of the management that we have to do to support Windows; plus there are promising IDE integration options for docker like the work being done at Jetbrains.

Complete the documentation - adding a FAQ section for Windows to cover most issues; and let the end user decide which way to move forward. I personally think this is the best way forward as we don't end up advocating one or other (less opinionated, more choice).

Thoughts?

option 3.

I have a fourth option:

Encourage contributors to submit pull requests for links to a community supported fork for OS-specific flavors. I can see this being useful for other Linux distributions (and of course, Windows). If we do this we have to come up with some guidelines to ensure compatibility with the core version of the django cookiecutter.

option 3

Considering the growing support for Linux on Windows, does it make sense to take Windows off the list?

The WSL is still not installed or enabled by default and if you do not have
admin access, or Windows Professional the option is not available to you.
I would caution against removing it from the list.
On Fri, Sep 22, 2017 at 12:53 AM Daniel Roy Greenfeld <
<EMAIL_ADDRESS>wrote:

Considering the growing support for Linux on Windows, does it make sense
to take Windows off the list?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pydanny/cookiecutter-django/issues/431#issuecomment-331292467,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkz6OWxm02wLwMymVR2KJu6LE6myfbCks5sktrJgaJpZM4GvPB_
.


Match connection string variable to test-resources override for communication samples

The smoke tests are failing because the env variable override in test-resources.json injects a value for COMMUNICATION_CONNECTION_STRING, but most of the samples call for process.env["CONNECTION_STRING"]. This PR updates all samples and README docs to reference the longer, scoped connection string environment variable.
https://github.com/Azure/azure-sdk/issues/1879

@RezaJooyandeh @DominikMe can one of you guys please take a look and at these changes?

/azp run js - smoke-tests

@benbp does this affect the test pipelines that run the test in livemode?
CC: @turalf and @minnieliu

/azp run js - communication-administration - tests

/azp run js - communication-chat - tests

@benbp does this affect the test pipelines that run the test in livemode?
CC: @turalf and @minnieliu
@RezaJooyandeh Triggering the live test pipeline manually still passed successfully.

@minnieliu @RezaJooyandeh now that the test is moving beyond the initial error, I'm seeing a 404 from the communication-sms sendSms.js test with the url that ends up getting constructed from test-resources.json: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=574949&view=logs&j=f3570516-c8fe-5ce9-92b2-b737df25d5ea&t=f7e12054-28e8-5a56-a94b-45551d60467c&l=1222
Do either of you have an idea of the issue here?

@DominikMe @0rland0Wats0n what was the decision for sendSms here? Are we disabling it for now till we have static resource setup?

@minnieliu @RezaJooyandeh now that the test is moving beyond the initial error, I'm seeing a 404 from the communication-sms sendSms.js test with the url that ends up getting constructed from test-resources.json: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=574949&view=logs&j=f3570516-c8fe-5ce9-92b2-b737df25d5ea&t=f7e12054-28e8-5a56-a94b-45551d60467c&l=1222
Do either of you have an idea of the issue here?

@turalf @DominikMe @0rland0Wats0n what was the decision for sendSms here? Are we disabling it for now till we have static resource setup?

@minnieliu @RezaJooyandeh now that the test is moving beyond the initial error, I'm seeing a 404 from the communication-sms sendSms.js test with the url that ends up getting constructed from test-resources.json: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=574949&view=logs&j=f3570516-c8fe-5ce9-92b2-b737df25d5ea&t=f7e12054-28e8-5a56-a94b-45551d60467c&l=1222
Do either of you have an idea of the issue here?

Yes, for now it should be skipped. We will be creating telephony resources on demand as well in a test subscription. (I have started an email thread on this)

/azp run js - smoke-tests

The linux smoke tests are getting timed out, but the specific issues this PR addresses are fixed (https://dev.azure.com/azure-sdk/internal/_build/results?buildId=576644&view=logs&j=f3570516-c8fe-5ce9-92b2-b737df25d5ea&t=f7e12054-28e8-5a56-a94b-45551d60467c).

/check-enforcer override


PEP8

😘

Hello @untzag! Thanks for submitting the PR.

In the file microcontroller.py, following are the PEP8 issues :

Line 29:80: E501 line too long (99 > 79 characters)

🙄 pep8speaks shouldn't be tripping on the 99 characters thing, since we explicitly allow that many characters in the yaml...
I move that we ignore for now and hope that it works itself out
perhaps it is confused because the yaml is not in master?


added password support

Thank you for the pull request, it means a lot to me when others are willing to invest their time into making Directory Lister better.  However, I will not be merging this PR as password protected listings is outside of the scope of Directory Lister, as I mentioned previously.
That being said there's a lot that could be done to improve this PR.  For example, please use a single pull request for all changes pertaining to a single logical feature (i.e. adding password protection) instead of submitting multiple pull requests (132, 133, 134).  Additionally, providing a detailed summary of the proposed changes you're making and why you're making them helps the maintainer(s) (in this case me) understand those changes.  Lastly, I'll be reviewing your submission in more detail and making some comments on the code directly with the intention of providing feedback you can use going forward to help you improve.
Thanks again for your contributions and please don't let this stop you from making future contributions to Directory Lister or any other project for that matter.

Hmmm, how to make multiple file editions to single PR? When i edit it i
only have option to merge
El 8 oct. 2016 23:30, "Chris Kankiewicz"<EMAIL_ADDRESS>escribió:

Thank you for the pull request, it means a lot to me when others are
willing to invest their time into making Directory Lister better. However,
I will not be merging this PR as password protected listings is outside of
the scope of Directory Lister, as I mentioned previously
https://github.com/DirectoryLister/DirectoryLister/issues/131#issuecomment-252432347
.
That being said there's a lot that could be done to improve this PR. For
example, please use a single pull request for all changes pertaining to a
single logical feature (i.e. adding password protection) instead of
submitting multiple pull requests (132
https://github.com/DirectoryLister/DirectoryLister/pull/132, 133
https://github.com/DirectoryLister/DirectoryLister/pull/133, 134
https://github.com/DirectoryLister/DirectoryLister/pull/134).
Additionally, providing a detailed summary of the proposed changes you're
making and why you're making them helps the maintainer(s) (in this case me)
understand those changes. Lastly, I'll be reviewing your submission in more
detail and making some comments on the code directly with the intention of
providing feedback you can use going forward to help you improve.
Thanks again for your contributions and please don't let this stop you
from making future contributions to Directory Lister or any other project
for that matter.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DirectoryLister/DirectoryLister/pull/132#issuecomment-252449499,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AP7zRUxV_GOO4cRtONd8k6iy7ENiWxtIks5qyAt9gaJpZM4KRzOv
.

You have to use the git command line client or GitHub Desktop client.
It sounds like you don't have much experience with git, if you'd like to learn more please check out the Try Git tutorial or for a more in-depth overview check out the Pro Git Book.

Very thanks for reply with my ignorance about github :P, i will setup it.
Thanks for reply
El 9 oct. 2016 1:03, "Chris Kankiewicz"<EMAIL_ADDRESS>escribió:

You have to use the git https://git-scm.com/ command line client or GitHub
Desktop https://desktop.github.com client.
It sounds like you don't have much experience with git, if you'd like to
learn more please check out the Try Git https://try.github.io/ tutorial
or for a more in-depth overview check out the Pro Git Book
https://git-scm.com/book/en/v2.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DirectoryLister/DirectoryLister/pull/132#issuecomment-252453260,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AP7zRdSetlNsNFL63SY8OdtJx2S98QlTks5qyCFMgaJpZM4KRzOv
.


Reflect the Layer Folders Structure

Hello @jwa107
Can you please, oh please, add an option where the plugin extract the PSD Layer Folders and Sub-folders structure the same way we arranged it in our Iconography.psd file ?
So that it reflects our work, and make it easy for developers to understand which icons belongs to which view ?
Example :
Iconography.psd (Main Folder)
Home (SubFolder)
├ Search.png
├ Menu.png
├ Pencil.png
└ Flag.png
Login (SubFolder)
├ Facebook.png
├ Twitter.png
├ LinkedIn.png
└ E-Mail.png
I'm sure everyone needs this feature very badly.

i did it here :

Hello Apauriche!
Thank you I saw the commit and it sounds great.
But I tested it (I ticked "Create Sub folders"), and it doesn't create the folder structure as expected... Maybe did I missed something?

Hi Isaac,
Here some screen of the photoshop structure :
[image: Images intégrées 1]
Here the folder structure created :
[image: Images intégrées 2]
I will remove the "@2x" suffix i think this is the problem you get ?
2016-04-25 14:07 GMT+02:00 Isaac Slama<EMAIL_ADDRESS>
Hello Apauriche!
Thank you I saw the commit and it sounds great.
But I tested it (I ticked "Create Sub folders"), and it doesn't create the
folder structure as expected... Maybe did I missed something?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/jwa107/Photoshop-Export-Layers-to-Files-Fast/issues/81#issuecomment-214284344

Here it go i pushed it in the subfolders branch.
2016-04-25 14:44 GMT+02:00 Anthony Pauriche<EMAIL_ADDRESS>
Hi Isaac,
Here some screen of the photoshop structure :
[image: Images intégrées 1]
Here the folder structure created :
[image: Images intégrées 2]
I will remove the "@2x" suffix i think this is the problem you get ?
2016-04-25 14:07 GMT+02:00 Isaac Slama<EMAIL_ADDRESS>
Hello Apauriche!
Thank you I saw the commit and it sounds great.
But I tested it (I ticked "Create Sub folders"), and it doesn't create
the folder structure as expected... Maybe did I missed something?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/jwa107/Photoshop-Export-Layers-to-Files-Fast/issues/81#issuecomment-214284344

Hello have you tried ?
2016-04-25 14:51 GMT+02:00 Anthony Pauriche<EMAIL_ADDRESS>
Here it go i pushed it in the subfolders branch.
2016-04-25 14:44 GMT+02:00 Anthony Pauriche<EMAIL_ADDRESS>
Hi Isaac,
Here some screen of the photoshop structure :
[image: Images intégrées 1]
Here the folder structure created :
[image: Images intégrées 2]
I will remove the "@2x" suffix i think this is the problem you get ?
2016-04-25 14:07 GMT+02:00 Isaac Slama<EMAIL_ADDRESS>
Hello Apauriche!
Thank you I saw the commit and it sounds great.
But I tested it (I ticked "Create Sub folders"), and it doesn't create
the folder structure as expected... Maybe did I missed something?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/jwa107/Photoshop-Export-Layers-to-Files-Fast/issues/81#issuecomment-214284344

Done.


Add bundler instructions for rollup

Requires a pinned version since newer Rollup versions has discontinued IE8 support.
Requires commonjs plugins see (https://github.com/alphagov/govuk-design-system/blob/1a883f66a50a25615a85f7ccdf9016695dd06518/lib/metalsmith.js#L131)

Let's re-open if we get any user feedback that this is needed.


[WIP] Enh vizu

The idea is to be able to obtain detailed plots like this:

Nice addition!
For the documentation, Keras uses its own markup parser. I'm not sure if the numpy like syntax will be nicely parsed. You can modify the doc like this:
"""
# Parameters
    verbose: boolean, optional (default False). If true, then more layer parameters will be reported in the generated plot.
...
"""

@tboquet: Thanks for the suggestion. I've integrated it.

Any news for this that would be a really nice feature !

Closing outdated PR. If you still care about the content of the PR, please submit a new PR to master, updated for the Keras 2.0 API.


Add a math function "cut" to support precision cut of double value

Add a function cut(x, d) to cut the precision of a double value to a specific decimal. This can help us to do the checksum over the double value columns.

Is "cut" the common way to refer to this in ieee754 parlance?

Why not call this floor()?

round_down seems like a better name.

@martint  from ieee754 parlance, it should be classified as "Directed roundings - round  toward 0". So I think the name round_down makes more sense, as @electrum suggested. Let me know your thoughts

@martint feel free to assign back to me to review when you've settled on the name


Investigate Basic-Auth errors on staging

Our staging API will not allow requests from the staging site, even though the origin in the same. As a result we can't mimic behaviour on production, because all API requests are disallowed.
This is mysterious, but of limited interest. The easiest way to deal with this may be to disable Basic-Auth on the API using nginx.

I'm going to time box this to 30 mins, by debugging direct on the staging server.

To reproduce:
200:
curl -s 'http://staging.openprescribing.net/' -H 'Authorization: Basic

dGVjaDpVZFByPm00Vw==' -o /dev/null -D -
403:
curl -s 'http://staging.openprescribing.net/api/1.0/org_location/?p=1'

-H 'Authorization: Basic dGVjaDpVZFByPm00Vw==' -o /dev/null -D -

It seems this is nothing to do with Basic Auth, as we continue to see the problem when that is switched off.  A new data point is that if we switch Cloudflare to "development mode" (i.e. turn off caching) we get the same behaviour in the production flavour:
$ curl -s 'https://openprescribing.net/api/1.0/org_code/?org_type=CCG&q=boggle'  -H 'Authorization: Basic dGVjaDpVZFByPm00Vw==' -o /dev/null -D -

HTTP/1.1 403 Forbidden
Date: Mon, 06 Jun 2016 09:18:19 GMT
Content-Type: application/json
Transfer-Encoding: chunked
Connection: keep-alive
Set-Cookie: __cfduid=d8a5083cf4a2527af8354bc60d5beb9061465204699; expires=Tue, 06-Jun-17 09:18:19 GMT; path=/; domain=.openprescribing.net; HttpOnly
Vary: Accept, Cookie
Allow: OPTIONS, GET
Server: cloudflare-nginx
CF-RAY: 2aeac6bd68690b81-LHR

So it was something to do with Basic Auth; cloaked by the fact that you apparently get a 403 if you send nginx auth headers when there's no auth required.
Closing this as per @annapowellsmith's preference to just not have HTTP auth for the staging server.


test(backend): 新增db_monitor测试 #3816

 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.leifxu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Use https://grpc.io consistently as the canonical URL

This PR changes instances of http://www.grpc.io and http://grpc.io in use in various places of repository to the canonical nice URL https://grpc.io.

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Documentation fixes — Basic sanity & build passed. Other tests ignored.


Update status_module.conf

Status bar entries have text up against the icon:

Probably should look like:

Can you update the individual status modules? Putting a space there for all modules reduces flexibility

Can you share the config used?
It doesn't seem to happen on the main branch

set -g default-terminal screen-256color
set -g @test_config "tmux"
set -g mouse on

run "$CATPPUCCIN_TMUX_PLUGIN"

set -g status-right-length 100
set -g status-right "#{E:@catppuccin_status_directory}"
set -ag status-right "#{E:@catppuccin_status_date_time}"

If this is a bug it could/should be fixed by using the @catppuccin_status_middle_separator options/setting it's default

Can you share the config used? It doesn't seem to happen on the main branch
set -g default-terminal screen-256color
set -g @test_config "tmux"
set -g mouse on

run "$CATPPUCCIN_TMUX_PLUGIN"

set -g status-right-length 100
set -g status-right "#{E:@catppuccin_status_directory}"
set -ag status-right "#{E:@catppuccin_status_date_time}"

If this is a bug it could/should be fixed by using the @catppuccin_status_middle_separator options/setting it's default

You're right, I was setting the text with no space in my config:
set -g @catppuccin_directory_text "#{pane_current_path}"
set -g @catppuccin_date_time_text "%H:%M"

and your default config puts the space in those variables. Not sure if you want to be enable folks to set the text of those variables and still have space from the icon, but this was mostly my fault.

People should still be able to set the text and have space. Does this not work?
set -g @catppuccin_directory_text " #{pane_current_path}"
set -g @catppuccin_date_time_text " %H:%M"

People should still be able to set the text and have space. Does this not work?
set -g @catppuccin_directory_text " #{pane_current_path}"
set -g @catppuccin_date_time_text " %H:%M"

That does work, it's just in previous versions when I set the text without space, there would still be space between the icon and text, so I missed this in my testing.


Config for text message [Enchancement]

Would it be possible to add a config, so you can disable the green text message that pops in the chat once you join a world? I tried disabling it with a kubejs script, but I had no luck.

Added in 2.3.17


Change public political naming

Trello https://trello.com/c/LHR7GodT/1234-release-history-mode
The terminology used in the application has diverged since the initial implementation of the feature. This changes the public routes where we show the word "political" to instead reference "gets history mode" as is used in the UI.
Fixing this doesn't really resolve any huge problems but my logic is that this will reduce scatterings of confusion where things don't reflect naming - and since this feature isn't released yet it is still relatively pliable.
Obviously these could diverge again and we could end up with the naming not matching again, and likely we wouldn't fix it then as it would cause bc breaks for analytics and such. But since we're not in that situation and naming changes are very infrequent this seems a good course for now.

I'm not sure i'm overly keen on 'gets history mode' in general as I think it looks a bit icky in places but I acknowledge it's a tricky concept to name! But yay for consistency! 🎉

Yup, I know what you mean. I'm happy that it's a big improvement on the ambiguity of "political", so yay for iteration.


Asynchronous deserialization with Streams

The xml lib has very good Stream support.
It would be great if the generated code would also make use of that.
This would probably improve performance as the data could be deserialized asynchronously as it arrives.

Hi @Leptopoda, thank you very much for your feature suggestion. This is something that I have considered in the past, however it does add quite a bit of complexity to deserialization. There are other more straightforward features that I would like implementing before reviewing support for streams.

Sad to hear that you are no longer planning to implement this.
Can you please let us know what your reasoning is? Is it impossible or are you just no longer considering this in scope for your package?
I don't think that I'd be working on this in the near future but are you open for a PR implementing this?

Hi @Leptopoda, hope you're well! It has been over a year since the feature request was submitted and it has not garnered any further discussion. I also believe that there is a significant amount of complexity involved in supporting streams in the source code generated by xml_serializable. I am open to discussing this feature request in more detail especially regarding the ways that support for streams could be implemented.


Resource punkt not found.

I am getting the below error and unable to test it. I am trying this on my mac
(Resume-Matcher) ➜  Resume-Matcher git:(main) ✗ streamlit run streamlit_app.py
You can now view your Streamlit app in your browser.
Local URL: http://localhost:8501
Network URL: http://<IP_ADDRESS>:8501
[nltk_data] Error loading punkt: <urlopen error [SSL:
[nltk_data]     CERTIFICATE_VERIFY_FAILED] certificate verify failed:
[nltk_data]     unable to get local issuer certificate (_ssl.c:1002)>
2023-07-27 16:49:19.424 Uncaught app exception
Traceback (most recent call last):
File "/Users/kas772/Surendra/Resume-Matcher/lib/python3.11/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 552, in _run_script
exec(code, module.dict)
File "/Users/kas772/Surendra/Resume-Matcher/streamlit_app.py", line 160, in
annotated_text(create_annotated_text(
^^^^^^^^^^^^^^^^^^^^^^
File "/Users/kas772/Surendra/Resume-Matcher/streamlit_app.py", line 86, in create_annotated_text
tokens = nltk.word_tokenize(input_string)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/kas772/Surendra/Resume-Matcher/lib/python3.11/site-packages/nltk/tokenize/init.py", line 129, in word_tokenize
sentences = [text] if preserve_line else sent_tokenize(text, language)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/kas772/Surendra/Resume-Matcher/lib/python3.11/site-packages/nltk/tokenize/init.py", line 106, in sent_tokenize
tokenizer = load(f"tokenizers/punkt/{language}.pickle")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/kas772/Surendra/Resume-Matcher/lib/python3.11/site-packages/nltk/data.py", line 750, in load
opened_resource = _open(resource_url)
^^^^^^^^^^^^^^^^^^^
File "/Users/kas772/Surendra/Resume-Matcher/lib/python3.11/site-packages/nltk/data.py", line 876, in open
return find(path, path + [""]).open()
^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/kas772/Surendra/Resume-Matcher/lib/python3.11/site-packages/nltk/data.py", line 583, in find
raise LookupError(resource_not_found)
LookupError:

Resource punkt not found.
Please use the NLTK Downloader to obtain the resource:

import nltk
nltk.download('punkt')

For more information see: https://www.nltk.org/data.html
Attempted to load tokenizers/punkt/PY3/english.pickle
Searched in:
- '/Users/kas772/nltk_data'
- '/Users/kas772/Surendra/Resume-Matcher/nltk_data'
- '/Users/kas772/Surendra/Resume-Matcher/share/nltk_data'
- '/Users/kas772/Surendra/Resume-Matcher/lib/nltk_data'
- '/usr/share/nltk_data'
- '/usr/local/share/nltk_data'
- '/usr/lib/nltk_data'
- '/usr/local/lib/nltk_data'
- ''

Hi @surenganne
Punkt is missing in your system.
Activate your venv and via terminal do:
>>> import nltk
>>> nltk.download('punkt')
```

(Resume-Matcher) ➜  Resume-Matcher git:(main) ✗ python3
Python 3.11.3 (v3.11.3:f3909b8bc8, Apr  4 2023, 20:12:10) [Clang 13.0.0 (clang-13<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.

import nltk
nltk.download('punkt')
[nltk_data] Error loading punkt: <urlopen error [SSL:
[nltk_data]     CERTIFICATE_VERIFY_FAILED] certificate verify failed:
[nltk_data]     unable to get local issuer certificate (_ssl.c:1002)>
False

Hi, I think you have not installed the packages.
After activating the virtual environment, have you done pip install -r requirements.txt

(Resume-Matcher) ➜ Resume-Matcher git:(main) ✗ python3 Python 3.11.3 (v3.11.3:f3909b8bc8, Apr 4 2023, 20:12:10) [Clang 13.0.0 (clang-13<IP_ADDRESS>)] on darwin Type "help", "copyright", "credits" or "license" for more information.

import nltk
nltk.download('punkt')
[nltk_data] Error loading punkt: <urlopen error [SSL:
[nltk_data]     CERTIFICATE_VERIFY_FAILED] certificate verify failed:
[nltk_data]     unable to get local issuer certificate (_ssl.c:1002)>
False

Are you behind some proxy?

Try this
import nltk
import ssl

try:
    _create_unverified_https_context = ssl._create_unverified_context
except AttributeError:
    pass
else:
    ssl._create_default_https_context = _create_unverified_https_context

nltk.download()

Ref

I took this route and it worked. Thanks!
If the issue still persists, then there is a workaround over https://github.com/gunthercox/ChatterBot/issues/930#issuecomment-322111087
Note: This workaround disabled SSL checks.


btn group

a group of buttons that can have one selected at a time. comes in both raised and flat depending on the classes sent in

Maybe it is better to rename directive 'btn-style' to'btn-class', and 'selection-style' to 'selection-class'

To me the Button Group is similar to a group of radio buttons. How about implementing it in a more semantic way, using , , etc?

i was planning on using it like that but isnt it more semantic to use buttons in a button group?
Also the styles we have for buttons, labels and radio are a bit tricky to combine so i just went with the less painful one

It is tricky and more painful. Since the user is choosing something between some options, that seems to me more like what radio 'buttons' do. But this is trivial.

How about adding a test for this?

I did not add any tests since the component only does one function that simply sets the selected item to whatever you send in or click on, there is no major thing to test