Wrong environment detected in unit test under certain conditions

Laravel Version: 5.4.36
PHP Version: 7.0.30 (production), 7.1.16 (local)

Description:
Being very cautious, I implemented a check for the correct environment into the setUp() of my TestCase class.
       if(env('APP_ENV') !== 'testing') {
            $this->fail('Not in testing environment according to APP_ENV. Aborting');
            die(1);
        }

        $status = Artisan::call('config:clear');

        if($status !== 0) {
            $this->fail('Could not clear config cache. Aborting');
            die(1);
        }

        $status = Artisan::call('view:clear');

        if($status !== 0) {
            $this->fail('Could not clear view cache. Aborting');
            die(1);
        }

        $env = app()->environment();

        if($env !== 'testing') {
            $this->fail('App does not detect testing environment via app()->environment(). Detected environment: "' . $env . '". Aborting.');
            die(1);
        }

Glad, I did. The first test fails with the following message in my phpunit log:
<failure type="PHPUnit_Framework_AssertionFailedError">ConfigTest::testConfigCacheClearing
App does not detect testing environment via app()->environment(). Detected environment: "production". Aborting.

[...]/TestCase.php:41
</failure>

This only happens, if I called "php artisan config:clear" immediately prior to the unit test. Subsequent tests do not fail. But this is the lesser important part of the bug. I think, the return value of app()->environment() must never be affected by config:clear.
As you can see, the value of APP_ENV is checked aswell and has the correct value "testing", otherwise the test would fail.
Steps To Reproduce:
Use the code above in your TestCase setUp() and try running the test with and without a prior "php artisan config:clear" via the commandline.

The real problem is you are caching your config on your development box. You should not do this, as it causes problems like what you've described above.

The real problem is you are caching your config on your development box.

I made the same assumption first. That's why I wrote:

This only happens, if I called "php artisan config:clear" immediately prior to the unit test.

So there should be no cached config. Additionally, the unit test itself clears the config cache again, as shown in the above code. Please look more closely at what's happening. How does this fit into your hypothesis, that the config cache was to blame?

It looks, like I need to clarify this further.

This only happens, if I called "php artisan config:clear" immediately prior to the unit test.

This is not the call of "config:clear", that is done via code (I understand your remark concerning the same cycle). I enter this manually in the command line. The command line prints: "Cache cleared successfully.".
After that I manually start the unit test. How could the config cache still exist at this point?

You should not cache your config in development. There is simply no point. You should only cache on production.

I do not cache the config in development, btw. But that's not the point of this bug report, anyway.

I do not cache the config in development, btw. But that's not the point of this bug report, anyway.

So why are you then running Artisan::call('config:clear'); at all?

I resist the urge to go off-topic. At this point, it seems, you are questioning the legitimacy of this issue.
So let me rephrase this, to make things more clear. This can be answered with yes or no:
If the config is not cached before the booting of the framework, should env('APP_ENV') return the same value as config('app.env')?

If the answer is no, then my issue is obsolete. But I would like an explanation in this case.
If the answer is yes, then there is a bug.

...should env('APP_ENV') return the same value as config('app.env')?
It depends on what you have inside config/app.php If you have the default 'env' => env('APP_ENV', 'production'), - then yes.
But you should never call env() outside of the config classes. Not even in your tests. It is only designed to be used inside the config system.
Regardless - on a fresh install of Laravel 5.7 I get testing as the env using your code above (even if I run config:clear before and during the tests).

In the actual system, I only use env() in config files.

Regardless - on a fresh install of Laravel 5.7 I get testing as the env using your code above (even if I run config:clear before and during the tests).

Thanks for this info. In this case, the bug appears less urgent for me. I will upgrade as soon as possible and keep an eye on it. I am all the more curious to what causes it.
As a last note, omitting view:clear can lead to exceptions in the worst case. For example, if a custom blade directive was changed or an upgrade changed something. I had two cases, where unit tests with $this->get('/some/url/') failed, only because the views needed to be recompiled.

For example, if a custom blade directive was changed or an upgrade changed something.

Yeah - fair point 👍

Admittedly, it's a rare case. I told you, I was paranoid. And that got me into that hassle. :)

Accidentally had config cached on my local. Ran tests and it wiped my database. Frustrating as it's not the first time this happened (previously spent a lot of time investigating what could have caused it). So now I added the checks from @LMCom  in my TestCase! 😄 Shouldn't happen again.


constantly getting error

VSCode Version: Code - Insiders 1.3.0-insider (0dba5abc3dcc0b69c04374dee90a2726dcbbdabf, 2016-06-14T07:51:29.842Z)
OS Version: Windows_NT ia32

Steps to Reproduce:
1.it comes randomly anytime

duplicates #7597


Proprietary and confidential notice inside basic_test.cpp

minimal_integration_test's code is open source under the BSD-3-Clause license. This allows anyone to examine, modify and distribute the sources (among other things). However, the source code repository includes a documentation file that has a "Proprietary and confidential" notice:
https://github.com/AndyZe/minimal_integration_test/blob/515bfda90ebec727256f9552168a1bd328dc98c8/test/basic_test.cpp#L4
I suspect this is a mistake since other files do not carry such a notice.
Steps to Reproduce
Open and read: https://github.com/AndyZe/minimal_integration_test/blob/515bfda90ebec727256f9552168a1bd328dc98c8/test/basic_test.cpp#L4
Expected Behavior
This is all open source and publicly usable.
Actual Behavior
There is a "Proprietary and confidential." notice.

Thanks! It should be fixed now.


How to change style of  component when using  component?

How to change style of  component when using  component.
I need to change border radius to make it round.
And I also want to change color of active .

Hello @VitaliiTsurkan there are non-obfuscated classes made available for different pieces of the component that you can reference in your style sheet. Is that what you are looking for?

Hello @tim-steele I'm not sure - I'm new to react, could you provide an example?

override the default style...
.carousel__dot--selected {   background-color: <your fancy custom color > !important; }

You can also pass your own "className" prop to just about any component in Pure React Carousel and it will be appended to the list of classes for that component.

Here, saving the time of whoever stumbles upon this in the future.
<CarouselProvider
...
>
  <DotGroup className="prc-dotGroup" />
<CarouselProvider />

.prc-dotGroup {
  text-align: center;
  padding-top: 5px;
  padding-bottom: 10px;
}
.carousel__dot {
  height: 15px;
  width: 15px;
  margin: 3px;
  border-radius: 50%;
  border: 1px solid black;
}
.carousel__dot--selected {
  background-color: black;
}

How to make  items as image, just like a shopping carousel


Added about us and sponsors

added about us and sponsors section

Please resolve the conflicts and update the PR.


Update email_notifications.py

<EMAIL_ADDRESS>

Coverage decreased (-71.3%) to 0.0% when pulling 3553129c1871c96a830472e390829afb8e0e95c1 on sweetascherri:patch-1 into fa73528f8748fa6e446bbec692d9730a2190471c on django-extensions:master.


Status methods should be pagenated

Can you just rebase and change pagenated to paginated?

@kaizensoze done!


Azure auth options, test framework, tags and resource group actions

May not be able to merge this for a day or two pending new Azure CLI bits in PyPi - but wanted to get it started so people could review.
Lots of things here are not as they should be longer term, including the quality of the tests and some filters, but they get us some functionality to work with as we ramp up

Three azure auth methods now documented in c7n_azure\readme.  Supports service principal auth or passing a raw token (useful for testing).

Some new tag support

Delete resource groups (and filter for empty ones)

VCR tests with automatic rewriting of subscription id's, filtering of auth, etc - so regardless of how you authenticate while debugging the tests will playback in a compatible way.

Cleanup of query/client behaviors, serialization, etc.

Build issue looks to be lack of GCC for compiling scandir package which is required by Azure SDK?  But seems unlikely nothing else in the dep tree required GCC doesn't it?

looks like we'll need to add gcc, using pipdeptree, the reverse dep points to azure-datalake-store
--------------
scandir==1.5
  - pathlib2==2.2.1 [requires: scandir]
    - azure-datalake-store==0.0.19 [requires: pathlib2]
      - azure==3.0.0 [requires: azure-datalake-store~=0.0.18]

Yeah was just odd as I thought the trusty container already had build-essentials

Ok I think the Tabulate/Azure version issue and the GCC issue are resolved, I'll figure out what is going on with c7n_mailer unit tests this weekend.

@kapilt I feel like we need to change the way imports are done in both c7n_mailer and c7n_azure tests so they are all absolute, because the relative stuff is a bit of a mess (which is why my adding a common.py in azure tests broke c7n_mailer).   You have more python experience so I think best to get your feedback before I just go change all those files and add a ton of init.py's  - for now I'll just rename my common.py to azure_common.py

@stefangordon +1 re normalizing imports, mailer testing is depending on particular invocation location which is less than ideal (a couple of different ways of fixing that). in general re imports in tests dir/pkg. ideally we just add init.py to the package and then can do relative imports ... ie from .common import xyz


Disable user

It will be great if there is a way to disable a user. a little bit like "Disallow login form" option but also he will not be displayed on the user list.

it's now available in 1.0.26.


It would be better to list all properties of the entity in the 'Edit Entity' dialog

Storage Explorer Version: 1.20.1
Build Number: 20210715.14
Branch: hotfix/1.20.1
Platform/OS: Windows 10/ Linux Ubuntu 20.04/ MacOS Big Sur 11.4
Architecture: ia32/x64
How Found: Exploratory testing
Regression From: Not a regression
Steps to Reproduce

Expand a storage account -> Tables -> Create a table.
Import an entity that contains more than three properties ('PartitionKey' , 'RowKey' , 'Timestamp').
Click 'Add' -> Only type the value for 'PartitionKey' and 'RowKey' properties (Make sure other properties' value is default).
Select the added entity  -> Click 'Edit'.
Check the contents of the 'Edit Entity' dialog.

Expected Experience
List all properties in the 'Edit Entity' dialog.
Actual Experience
Only list the properties which value is not null.

By design for now, but something we could readdress in the future if a customer requested it.


Update Zendesk tests

Tests must be updated with changes made in dosomething_zendesk_get_ticket_array.

as tests aren't running, these test def aren't :new_moon_with_face:


FF WASI submodule and update the machinery

This commit fast-forwards the WASI submodule and updates the
relevant the machinery. It also updates generate_raw::generate
fn to accept a slice in preparation for multi-module wasi_ephemeral.
Changes introduced by this PR:

change signature of generate_raw::generate from fn generate(&Path) to fn <P: AsRef<Path>>generate(&[P]); this effectively is needed if we want to be able to generate bindings from multi-module snapshots such as ephemeral.
add feature multi-module to generate crate which serves as a workaround for avoiding name clashes for syscalls that are a very present danger introduced in multi-module ephemeral snapshot; this "hack", when the feature is enabled, prepends the Wasm module name to the syscall's name when generating Rust wrappers. This is a temporary solution until we figure out how to handle it better.
escape syscall names; in particular, due to multi-modularity of ephemeral, we can now encounter reserved words used as syscall names. yield is such an example.
add draft implementation for Char8 and USize builtins, and a draft implementation of IntDatatype which currently is used as the underlying representation of Dircookie.

cc @pchickey if you rebase #41 on top of this PR, this should hopefully fix the CI for you, and make your PR less cluttered.
EDIT: forgot to mention that IntDataype is currently generated similarly to how FlagsDatatype. For example:
pub type Dircookie = u64;
pub const DIRCOOKIE_START: Dircookie = 0;

👍
Thanks!


Invalid API key error

I'm getting the following error

Error
Invalid API key. Please see https://openweathermap.org/faq#error401 for more info.

I have created an open weather map account and have used the default key, created a new key, waited several days between account creation and new key creation before using the keys. But I continue to get the same error. I've verified I'm copying and pasting the full key correctly.
Do you have any recommendations?

@shombando sadly i think owm changed their rules on accessing one call api with free api key. Not much i can do :s
I might look at not using one call api in the future, but it will take me time

@farfromrefug thanks for the info. In the meantime, it might be a good idea to remove the pop-up asking the user to enter an API key.

@shombando well still works with old API keys :s

Unfortunately it works not with old api keys. The popup will show anyway. Really annoying.

@topcaser sorry but i tested again with my key and it works fine. At some points they changed api keys and they dont support the single call API. I dont plan on implementing differently, it would requires many http requests which is slow and uses too many resources (not environmentally friendly). Plus now there is Open-Meteo

I've noticed that open-meteo is inaccurate in certain parts of some countries.

I got this error with 3 years old api keys too. Maybe we should remove the API 2 as it does not work as intended any more, and add a clearer warning message for users? (it can be very frustrating to think your api key is invalid :))
FYI I ended up subscribing (for free, and limiting to 1000 calls to be sure it stays free), after 15min it worked, using onecall API 3.0

I will try to do the subscription thing, like you mentioned. I'm assuming this is what your subscription page looks like.

@jacobgonzales20 Exactly!
You can check if it's activated using the api directly meanwhile (for example https://api.openweathermap.org/data/3.0/onecall?lat=SOME_LAT&lon=SOME_LNG&exclude=&appid=YOUR_API_KEY )

@bonswouar but you need to add a credit card for that subscription right?

from my experience yes, you could always add a visa gift card

Right, or you can use a virtual card from some online banking app and cancel it right away (one-use virtual cards aren't accepted though)


Make-An-Audio 2 1D VAE

Hello,
I noticed that in the Make-an-aduio 2 paper, you have not reported the reconstruction loss performance of your trained 1D VAE in comparison with the 2D one. I am wondering if you could get a similar reconstruction performance with the 1D VAE, or the reconstruction performance was inferior yet the generation quality was better overall.
Thank you for your help.

The reconstruction performance is a little worse than 2D VAE but it works better when using with diffusion. Training 1D VAE with 2D discriminators is prone to instability problems, leading to overly smooth results. Because 2D patch GAN is very strong. So we use R1 regularization and set disc_factor=2 to stabilize the training

Thank you very much @Darius-H for your help.
I would appreciate it if you can share the reconstruction loss for the 1d VAE if it is available to you. Looking forward for the full training code and training configuration.

Thank you very much @Darius-H for your help. I would appreciate it if you can share the reconstruction loss for the 1d VAE if it is available to you. Looking forward for the full training code and training configuration.

Make-An-Audio 2 is released in https://github.com/bytedance/Make-An-Audio-2.
The loss figure of 1D VAE:


Updated to VS 2015

Since VS2015 Community available for free, most people might prefer to use it.
I decided to submit a pull request (and maybe convince a fellow programmer to upgrade too? :) )

I've been meaning to do this - thank you. I need to update to 2015 myself, but once I do I'll confirm this and merge.


COIL Crossfade

[x] Read CONTRIBUTING.md.

Coil Image Builder Recipe
Summary of changes:

Converted AsyncImage(model = image...) to use an ImageRequest.Builder allowing for Coil config

Enables more builder usages like .error(Res.drawable...) and .fallback(Res.drawable...)
Could also consider SubcomposeAsyncImage which allows for a composable-scoped to provide a CircularProgressIndicator()

Tested changes:
No regression in loading images, crossfade only happens on first load, afterwards it gets loaded from Coil cache which is near instant

Nice!


Bluetooth: Host: avoid runtime warning in bt_enable

Avoid runtime warning in bt_enable when CONFIG_BT_SETTINGS
and CONFIG_BT_DEVICE_NAME_DYNAMIC is not set.
This warning was intoroduced in
commit d76bba4b5ef85c6b25dfdf9c9809fcead0f4da5c
("Bluetooth: host: Device name handling of invalid length")
Signed-off-by: Martin Tverdal<EMAIL_ADDRESS>

s/wairning/warning in the commit message

Done!


Tilt install fix

Closes: #1369

What changed?
Install tilt directly, rather than using curl | bash

Why?
docker build fails otherwise as it uses (not installed or configured) sudo.

How did you test it?
Tested locally with docker build on Arch Linux & Mac OS, x86_64. I can't test the make targets directly as they refuse to build in a clean chroot, and I don't currently have time to figure out which dependencies are missing.

Release notes

Documentation Changes
Other
I am not familiar enough with your build env to determine which fix is correct: caa497548c5fd1db9eb6493e2f175c550903604c or 0acba92265176c8ccd8ea9870caf52c20a2c673d.

Nice work @morancj !


Update instance image family to slurm-gcp-5-8 version

This updates instance image family to slurm-gcp-5-8 version for hpc-enterprise-slurm, lustre-slurm and hpc-slurm-ubuntu2004 blueprint.
Submission Checklist
Please take the following actions before submitting this pull request.

Fork your PR branch from the Toolkit "develop" branch (not main)
Test all changes with pre-commit in a local branch #
Confirm that "make tests" passes all tests
Add or modify unit tests to cover code changes
Ensure that unit test coverage remains above 80%
Update all applicable documentation
Follow Cloud HPC Toolkit Contribution guidelines #

Thanks for pointing this out. I had only updated for the BPs that had previously failed. After your comment, also modified for the remaining files you mentioned.

I will run all the tests, that should cover it.

/gcbrun


[QUERY] Azure ServiceBus TopicClient Retry Policy

Query/Question
How can we help?
Hi Community,
I have custom Retry Policy class for my Azure TopClient. Class is inheriting from RetryPolicy and  I am overriding the following method:
abstract bool OnShouldRetry(TimeSpan remainingTime, int currentRetryCount, out TimeSpan retryInterval);

From what I have observed, the first argument (remainingTime) is the total time of the operation being performed e.g. if its 20 seconds then the retry will occur only within this timeframe.
Question I have, is their a configuration of setting a timeout per attempt i.e. lets say from the above code the (remainingTime) is 20 seconds and I have a total of 3 retry attempts. When the first retry occurs I want it to timeout within x seconds but within the time frame of (remainigTime )?
This is to reduce each attempts during the timeframe of the (remainingTime)
Environment:

Microsoft.Azure.ServiceBus 5.0.0
.NET runtime version: .NET Framework 4.7.2
IDE and version : Visual Studio 16.7.3

Thank you for your feedback.  Tagging and routing to the team best able to assist.

@kabirhussain There is no configuration setting to override this. To truly modify this, you will have to extend the RetryPolicy class.
hope this helps.
#please-close

@kabirhussain There is no configuration setting to override this. To truly modify this, you will have to extend the RetryPolicy class.
hope this helps.
#please-close

Thank you for the response but i am extending RetryPolicy class as mentioned from the post. Can you please elaborate what you mean by truly modify it as i unable to see how to change this?


Override NvimTree background color

the color changes when I change the cursor position from NvimTree to the editor and the top of the NvimTree does not change color

I am trying to get something like this:

this should do ;)
local overrides = {
    NvimTreeNormal = { bg = 'red' },
    NvimTreeNormalNC = { link = 'NvimTreeNormal' }
}

@rebelot Thanks.


Switch to datahub.core.reversion.NonAtomicRevisionMiddleware

This replaces reversion.middleware.RevisionMiddleware with datahub.core.reversion.NonAtomicRevisionMiddleware which sets atomic = False and lets the atomicity of the transaction be determined by the django logic instead.

I do prefer the mixin or the context manager to be honest, this middleware is way too magic, with lots of assumptions and unnecessary logic for requests that don't need versioning.

Could switch to those if you think it's straightforward enough.


Key exchange

Hello,
I wanted to ask if there is a document that explains the exchange between two smartphones in more detail.
The document for the solution architecture describes very nicely what happens when you want to upload your data, but I'm looking for information that explains the exchange in more detail.
The information how the keys are generated, when they are generated, how they look like and so on would be very interesting.
Is there perhaps also a document here which describes the exchange between mobile phones step by step?
Kind regards
Andreas

Hi @AndiLeni ,
this time a quicker answer :) Google has a very good FAQ page for this information, including details about cryptographics. I hope this is what you're looking for.
Best regards,
TK
Corona Warn-App Open Source Team

@tkowark thanks for the quick response :)
And thanks for teh links.
I thought there might be additional documents from the Corona-Warn-App that explain this function in more detail or in a more specific way.
I will definitely have a look at the document and get back to you if some questions araise.
Kind regards
Andreas

Dear @AndiLeni,
in case you have any additional questions especially with respect to the app and its infrastructure, please open another issue. As @tkowark provided you the link to the respective documents, we will close this issue now.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team


Update sane-doc-reports - fix investigation reports

These are fixes for the investigation reports, so merging ASAP would be awesome (before Sunday).
Related to issues:
https://github.com/demisto/etc/issues/19429
https://github.com/demisto/etc/issues/19306
https://github.com/demisto/etc/issues/19298

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/sane-doc-reports:<IP_ADDRESS>2
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/sane-doc-reports/tags
Get started by pulling the image:
docker pull devdemisto/sane-doc-reports:<IP_ADDRESS>2

Docker Metadata

Image Size: 138.35 MB

Image ID: sha256:5563fff8b101c5847fdb9c90a4b71784aae8fbb9e5ddf1c69df9f3d102b9b809

Created: 2019-10-04T20:43:00.267548347Z

Arch: linux/amd64

Command: ["python3"]

Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=0D96DF4D4110E5C43FBFB17F2D347EA6AA65421D
PYTHON_VERSION=3.7.4
PYTHON_PIP_VERSION=19.2.3
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/309a56c5fd94bd1134053a541cb4657a4e47e09d/get-pip.py
PYTHON_GET_PIP_SHA256=57e3643ff19f018f8a00dfaa6b7e4620e3c1a7a2171fd218425366ec006b3bfe

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:5276f82e4b9b84ff7cfb02a9cacbaeb3d890d0f2
org.opencontainers.image.version:<IP_ADDRESS>2

Thanks @glicht :)

Docker Image Ready - Production
Docker automatic build at CircleCI has deployed your docker image: demisto/sane-doc-reports:<IP_ADDRESS>5
It is available now on docker hub at: https://hub.docker.com/r/demisto/sane-doc-reports/tags
Get started by pulling the image:
docker pull demisto/sane-doc-reports:<IP_ADDRESS>5

Docker Metadata

Image Size: 138.35 MB

Image ID: sha256:b8d804b84597248b2bd1b376e9c2bd6df51827aac58d3409b19c1c1976bfb921

Created: 2019-10-04T23:19:04.546765348Z

Arch: linux/amd64

Command: ["python3"]

Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=0D96DF4D4110E5C43FBFB17F2D347EA6AA65421D
PYTHON_VERSION=3.7.4
PYTHON_PIP_VERSION=19.2.3
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/309a56c5fd94bd1134053a541cb4657a4e47e09d/get-pip.py
PYTHON_GET_PIP_SHA256=57e3643ff19f018f8a00dfaa6b7e4620e3c1a7a2171fd218425366ec006b3bfe

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:1704768c7f1f739bfeb66ff60f8e0dcd73cc6a26
org.opencontainers.image.version:<IP_ADDRESS>5


Bug 2082431: Fix MachineConfig base64 encoding issue on OVN cluster

This PR fixs the issue on OVN cluster, it came to notice that MCO is rendering Machine Config using base64 encoding instead of urlencoding on OVN cluster, this PRs added a check on MachineConfig source, so that the remediation will work properly. BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2082431

/bugzilla refresh

/bugzilla refresh


build: use a fixed google errorprone version

Issue:
"The plugin adds an errorprone configuration that automatically uses the
latest release of error-prone. You can override it to use a specific
version "
see https://github.com/tbroyer/gradle-errorprone-plugin
It appears that by latest version it will use the last locally cached
version. This caused inconsistency in builds.
Also, it introduces additional variables in the build.
e.g. see #1380
Fix:
Specify a fixed version
Will update to the latest version after the internal Artifactory
is updated with the latest version.
Test:
Delete the gradle cache and run the build again and verify in the
console output that the specified version is downloaded.

Yes, this is the same version as published in our internal artifactory.


Inserts a newline before first comment

Ruby version: 2.7
Rubyfmt git sha: acade99e80a18b0dcbec1ba908bb37c9301857bb

Input file
#!/usr/bin/env ruby

puts('ok')

Rubyfmt's output

#!/usr/bin/env ruby
puts('ok')

(that is it starts with an empty line, GitHub UI seems to collapse it.)
This breaks because
with the initial new line, the shebang is no longer acting as a shebang and the file is executed using /bin/sh instead of /usr/bin/env ruby which is just syntax errors forever.
subsequently rerunning rubyfmt on

#!/usr/bin/env ruby
puts('ok')

(that is it starts with an empty line, GitHub UI seems to collapse it.)
gives
#!/usr/bin/env ruby
puts('ok')

(no initial empty line.)
which does use the shebang again, which seems like it would be correct, but is kind of still a change in behaviour from the input.
Found while creating #189

this is a dup of https://github.com/penelopezone/rubyfmt/issues/176


Can't convert Keras model using tflite_convert

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Mac OS Mojave
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): Binary (via Pip)
TensorFlow version (use command below): 1.11.0
Python version: 3.6
Bazel version (if compiling from source): n/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce: tflite_convert --keras_model_file ./mymodel.h5 --output_file ./mymodel.tflite

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
I'm trying to convert a Keras model to TFLite, using the tflite_convert tool. However, I'm getting the below error, using Keras 2.2.4 and Tensorflow 1.11.0. I believe this to be a bug in the tool as this is just a standard H5 file which I've been using in production for months.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.
$ tflite_convert --keras_model_file ./mymodel.h5 --output_file ./mymodel.tflite
2018-10-14 19:11:59.960314: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
Traceback (most recent call last):
  File "~/anaconda3/envs/tf/bin/tflite_convert", line 11, in <module>
    sys.exit(main())
  File "~/anaconda3/envs/tf/lib/python3.6/site-packages/tensorflow/contrib/lite/python/tflite_convert.py", line 401, in main
    app.run(main=run_main, argv=sys.argv[:1])
  File "~/anaconda3/envs/tf/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 125, in run
    _sys.exit(main(argv))
  File "~/anaconda3/envs/tf/lib/python3.6/site-packages/tensorflow/contrib/lite/python/tflite_convert.py", line 397, in run_main
    _convert_model(tflite_flags)
  File "~/anaconda3/envs/tf/lib/python3.6/site-packages/tensorflow/contrib/lite/python/tflite_convert.py", line 100, in _convert_model
    converter = _get_toco_converter(flags)
  File "~/anaconda3/envs/tf/lib/python3.6/site-packages/tensorflow/contrib/lite/python/tflite_convert.py", line 87, in _get_toco_converter
    return converter_fn(**converter_kwargs)
  File "~/anaconda3/envs/tf/lib/python3.6/site-packages/tensorflow/contrib/lite/python/lite.py", line 356, in from_keras_model_file
    keras_model = _keras.models.load_model(model_file)
  File "~/anaconda3/envs/tf/lib/python3.6/site-packages/tensorflow/python/keras/engine/saving.py", line 251, in load_model
    training_config['weighted_metrics'])
KeyError: 'weighted_metrics'

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Added new link for course and free programming playgrounds

What does this PR do?
Add resource(s)
For resources
Added new course link for Data Structure by Jenny's Lectures CS IT and also added new python compiler under free playgrounds.
Why is this valuable (or not)?
It will serve as a beneficial course in free to learn data structure in C Programming language. Also students can run their code for free through the online python compiler of Tutorialspoint.
How do we know it's really free?
The links provided by me are free. Both, the youtube course and the online compiler can be accessed for free and does not require signing up.
For book lists, is it a book? For course lists, is it a course? etc.
I have added a course and a online compiler link.
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Be sure to check the free-programming-books-lint output for linter errors. Alphabetical order and spacing are the most common problems. Once you have made the necessary changes, please update the PR title to include “(changes made)” and a reviewer will take another look as soon as they are available. Thanks!

added in #10988


Vscode suddenly closes when i type a letter.

VSCode Version:1.48.1
OS Version:Windows 10

Steps to Reproduce:
1.Start editing a file which sends an api json response with large data nearly 500MB

Does this issue occur when all extensions are disabled?: Yes

Does this happen with our latest insiders https://code.visualstudio.com/insiders ?


How to customize cells? I need a new style

Hi - I was wondering how you could customize cells and use myself nib file.....Like this picture

#How to implement？
Where I need to change or add something？
thanks！

Checkout the stack overflow question that is similar to yours. Great answer: Custom cell for JSQMessages http://stackoverflow.com/a/35399408/5894123
Let me know if you have more questions and I see what I can do to help. 🕵

Also, see previous questions :smile:
https://github.com/jessesquires/JSQMessagesViewController/issues?utf8=✓&q=[Custom+cells]+in%3Atitle+


HTTP404 when expected to reach a service

Describe the bug
I've deployed an IdentityServer4 demo pod, that works fine & is reachable through Istio's gateway but only when the pod is being set up in the root of the address, so:

apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
name: istio-gateway
spec:
selector:
istio: ingressgateway
servers:

port:
number: 80
name: http
protocol: HTTP
hosts:

"*"

apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
name: istio
spec:
hosts:

"*"
gateways:
istio-gateway
http:
match:

uri:
prefix: /
route:
destination:
host: identity.default.svc.cluster.local

This is working great - i see Identity Server 4's welcome page. As expected. However if i change my VirtualService configuration to this:

apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
name: istio
spec:
hosts:

"*"
gateways:
istio-gateway
http:
match:

uri:
prefix: /identity
route:
destination:
host: identity.default.svc.cluster.local

I get HTTP404.
Expected behavior
i expect to reach the IS4's welcome page when entering http://external-ip/identity address.
Steps to reproduce the bug
Deploy a IS4 demo to a pod (i could give you my container from docker hub, but you'd need my certificate)
Use my istio-gateway & virtual service
Version
istio 1.0.3
kubectl 1.11
Installation
kubectl apply -f istio-demo.yaml
Environment
deployed to AKS, images running using Linux
I apologize if it's a rookie mistake. I've tried various configurations, i went through docs on istio.io and couldn't resolve this.

Does your identity service have the /identity path? If not, I think you have to rewrite the path to /, using https://istio.io/docs/reference/config/istio.networking.v1alpha3/#HTTPRewrite:
rewrite:
      uri: /

@przemekwojcik You are welcome :)


[request] weight to wild card lines

it would be nice if we had a way to add a weight to lines of a wild card instead of copying and pasting that same line multiple times
example:
hot[9]
cold[9]
freezing[3]
burning[3]
on fire
frozen
this could make hot 3 times as likely as freezing and freezing 3 times as likely as frozen
in addition to this, something we can put on the line that is nothing, for the chance to get no selection and control that chance with a weight

You can currently do this inline with jinja2 templates (just be sure to check the enable Jinja2 templates checkbox)
If you want blue 30% of the time, red 60% of the time (0.9 - 0.3), and yellow 10% of the time (1 - 0.9)
e.g.
{% with r = random() %}
    {% for val, weight in [('blue', 0.3), ('red', 0.9), ('yellow', 1)] %}
        {% if r < weight %}
            {% prompt %}{{ r }} I love {{ val }} roses{% endprompt %}
        {% endif %}
    {% endfor %}
{% endwith %}

or
{% set ns = namespace(r=random(), chosen=false, total=0) %}
{% for val, weight in [('blue', 0.3), ('red', 0.6), ('yellow', 0.1)] %}
    {% set ns.total = ns.total + weight %}
    {% if ns.r < ns.total and ns.chosen==false %}
        {% prompt %}I love {{ val }} roses{% endprompt %}
        {% set ns.chosen = 1 %}
    {% endif %}
{% endfor %}

Not sure if it makes sense to do it inside of wildcard files

Does the above suggestion work for you? I know it's more complicated than the basic syntax, but it seems that the use case is quite niche so probably not worth adding more complexity to the regular syntax


Twig Internationalization

Hi @PatrickLouys Thanks for this tutorial, I am able to create a simple web-app. But with the traffic comes from other non-english speaking user.Can you post a simple tutorial n how to integrate Twig with I8n. Thanks!

Hi. I think this would be a little out of scope for the tutorial, I want to keep it as simple as possible while also covering all the important topics.
That being said, I would recommend that you have a look here for the Twig i18n extension. I haven't used it but it looks like it does what you need.
Cheers,
Pat


Reparent button in the right click menu for nodes in scene tree does not work

Godot version
v4.0.dev.calinou [a4c863791]
System information
Windows 10
Issue description
The "Reparent" button in the right click menu for nodes inside of the scene tree does not work, there's no errors that show up in the console. However, the "Reparent to New Node" button does work.
https://user-images.githubusercontent.com/12120644/126707607-2307664d-cb79-4596-9f01-b77b7ed6da0d.mp4
Steps to reproduce

In the scene tree, use the "Reparent" button in the right-click menu for a node
Reparent to another node in the scene
It won't work

Minimal reproduction project
any project with nodes

The same issue occurs in AnimationTrackEditor #50719.

If it's a recent regression, might be related to #50559? See also #50744 due to 3.x cherrypick.

If it's a recent regression, might be related to #50559? See also #50744 due to 3.x cherrypick.

It seems to the node selection popup itself that's broken, see https://github.com/godotengine/godot/issues/50719


Hi,How to run a 3D pipeline ?

Hi,
you can run other pipelines using the corresponding config file (folder "config"). To run the 3D pipeline, you could use for example the following command:
python run_pipeline.py --cfg_file configs/Liquid3d.yml \
                       --dataset_path PATH/TO/DATASET \
                       --ckpt_path checkpoints/Liquid3d/ckpt \
                       --split test

In the section "Datasets" there is a description of how to get the corresponding dataset.
I hope that answers your question.
Best regards,
Lukas


Always install the latest stable helm

use "https://charts.helm.sh/stable" instead
of "https://kubernetes-charts.storage.googleapis.com/"

      from log (+ helm repo add stable https://kubernetes-charts.storage.googleapis.com/
Error: repo "https://kubernetes-charts.storage.googleapis.com/" is no longer available; try "https://charts.helm.sh/stable" instead)

Test name
Status

big-data
passed

scaling
passed

recreate
passed

storage
passed

upgrade-haproxy
passed

haproxy
passed

self-healing
passed

scaling-proxysql
passed

init-deploy
passed

restore-to-encrypted-cluster
passed

upgrade-proxysql
passed

security-context
passed

limits
passed

self-healing-advanced
passed

monitoring-2-0
passed

smart-update
passed

operator-self-healing
passed

affinity
passed

one-pod
passed

auto-tuning
passed

demand-backup
passed

proxysql-sidecar-res-limits
passed

users
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

upgrade-consistency
passed

scheduled-backup
failed

PXC operator docker - perconalab/percona-xtradb-cluster-operator:PR-696-c32af4ca

Test name
Status

upgrade-haproxy
passed

haproxy
passed

scaling
passed

recreate
passed

big-data
passed

storage
passed

init-deploy
passed

self-healing
passed

scaling-proxysql
passed

restore-to-encrypted-cluster
passed

security-context
passed

upgrade-proxysql
passed

limits
passed

demand-backup
passed

smart-update
passed

monitoring-2-0
passed

self-healing-advanced
passed

affinity
passed

one-pod
passed

operator-self-healing
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

scheduled-backup
passed

users
passed

upgrade-consistency
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

demand-backup-encrypted-with-tls
passed


KOCACHE directory is not created if set

$ echo $KOCACHE
/Users/jasonhall/.ko
$ ls $KOCACHE
ls: /Users/jasonhall/.ko: No such file or directory
$ go run ./ build ./test/
... succeeds, takes a few seconds
$ go run ./ build ./test/
... succeeds, takes about the same amount of time
$ ls $KOCACHE
ls: /Users/jasonhall/.ko: No such file or directory

At this point, I would have expected for the KOCACHE directory to be created if I specified it, or at least complain in some way that it was specified but doesn't exist.
If I create it, I also need to export the env var to actually see the cache directory get populated.

hello @jonjohnsonjr, will you take a look at this? I'm asking it because I'm willing to do this 🤩

hello @jonjohnsonjr, will you take a look at this? I'm asking it because I'm willing to do this 🤩


Virtual Apache 2.4 Pagekit Hosting Strange Problem

Hi,
I have 4 Pagekit installations on  virtual hosts on the same Apache server which shares the same ip and different domain names.
When I update one of the installations, the other installations automatically shows the updated version number although they are not upgraded yet.

Can you confirm the versions inside each project? I don't see how can update a project updates the others projects.

I really can't get it, I just upgraded only one of my domains to 1.0.5.
When I go to updates for other domains it says I have 1.0.5 and do not need to upgrade.


Upgraded all codegen related packages

FRONTEND PULL REQUEST
Related Issue
Dependabot tried to upgrade the package @graphql-codegen/cli but was failing due to the typescript packages not being updated simultaneously.
Changes Proposed
Upgrade all the related codegen dependencies as a bulk.
Testing
All regression testing should pass successfully.

Double checked the migration resources since we're bumping a major version.
The main @graphql-codegen/cli --> v5.0.0 package: The change to v.4.0.0 seems to have just dropped support for Node 14. The breaking change from v4 --> v5 seems to be a config change for a subdependency used in their watch mode, which we don't use. We should be fine here.
All the other plugin dependencies seems to have just gotten their version numbers bumped with the main package, so those seem also to be fine.


auto uninstallation?

the channel gets deleted few seconds after getting installed

logs or literally anything could help
вс, 5 мая 2019 г., 9:36 aarongranatto<EMAIL_ADDRESS>
the channel gets deleted few seconds after getting installed
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sudo97/Roku-IPTV/issues/8, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4OCTGMLXHK4WMIE3UC7ZDPTZ567ANCNFSM4HK2QB5Q
.

i have literally nothing, it only uninstall as if I had not installed it I have only this: https://i.imgur.com/mIhyrcx.png (the second picture is what it show me when I convert the file into squashfs)


📖  Document workload object thrashing

Summary
This PR updates the documentation for release 0.25.1 to document the pathology that I realized while thinking about Issue #2634 .
This PR also adds a note to the "known issues" index page to explicitly state its relation with bugs.
Preview at https://mikespreitzer.github.io/kcp-edge-mc/doc-mw-thrash-0251/direct/release-notes/
Related issue(s)
Fixes #

/cc @pdettori
/cc @cwiklik
/cc @eaepstein
/cc @KPRoche
/cc @clubanderson
@vMaroon
@nirrozenbaum


Attempted to dereference garbage pointer 0x10.

In platform of FIR BUGHD show a crash problem that my app Attempted to dereference garbage pointer 0x10. Originated at or in a subcall of _mh_execute_header, but i don't know how to resolved.
KSCrash  libobjc.A.dylib   197d9fbd0                                                                                objc_msgSend           113616
KSCrash  UIKit             18b2d7bbc                                                                                3079100
KSCrash  UIKit             18b014db4                                                                                183732
KSCrash  UIKit             18b0caa98                                                                                928408
KSCrash  UIKit             18b2d8834                                                                                3082292
KSCrash  UIKit             18b0d0d7c                                                                                953724
KSCrash  UIKit             18b0d0ba4                                                                                953252
KSCrash  UIKit             18aff5c20                                                                                56352
KSCrash  libobjc.A.dylib   197da5724                                                                                136996
KSCrash  CoreFoundation    186499074                                                                                _CFAutoreleasePoolPop  36980
KSCrash  CoreFoundation    18656d8a8                                                                                907432
KSCrash  CoreFoundation    1864992d4                                                                                CFRunLoopRunSpecific   37588
KSCrash  GraphicsServices  18fcb76fc                                                                                GSEventRunModal        46844
KSCrash  UIKit             18b05efac                                                                                UIApplicationMain      487340
KSCrash           -WKWebViewJavascriptBridge webView:didFailProvisionalNavigation:withError:
KSCrash  libdyld.dylib     19840aa08                                                                                10760

Needs a repro


internal versus external 3D SDF molecule CIP issue

Messages from:
Internal 3d
INFO:kgcnn.mol.encoder:OneHotEncoder Symbol found ['Cl', 'C', 'Br', 'O', 'N', 'P', 'S']
INFO:kgcnn.mol.encoder:OneHotEncoder Hybridization found [rdkit.Chem.rdchem.HybridizationType.SP3, rdkit.Chem.rdchem.HybridizationType.SP2, rdkit.Chem.rdchem.HybridizationType.SP]
INFO:kgcnn.mol.encoder:OneHotEncoder TotalDegree found [1, 4, 3, 2]
INFO:kgcnn.mol.encoder:OneHotEncoder TotalNumHs found [0, 2, 1, 3]
INFO:kgcnn.mol.encoder:OneHotEncoder CIPCode found [None]
INFO:kgcnn.mol.encoder:OneHotEncoder ChiralityPossible found [None, '1']
External 3D
INFO:kgcnn.mol.encoder:OneHotEncoder Symbol found ['Cl', 'C', 'O', 'P', 'S', 'N', 'Br']
INFO:kgcnn.mol.encoder:OneHotEncoder Hybridization found [rdkit.Chem.rdchem.HybridizationType.SP3, rdkit.Chem.rdchem.HybridizationType.SP2, rdkit.Chem.rdchem.HybridizationType.SP]
INFO:kgcnn.mol.encoder:OneHotEncoder TotalDegree found [1, 4, 3, 2]
INFO:kgcnn.mol.encoder:OneHotEncoder TotalNumHs found [0, 2, 3, 1]
INFO:kgcnn.mol.encoder:OneHotEncoder CIPCode found [None, 'S', 'R']
INFO:kgcnn.mol.encoder:OneHotEncoder ChiralityPossible found [None, '1']
I've also observed that the CIP did not provide same response with internal / external 3D dataset.
do you know what it's like this ?

we need to run more cases but we use 4 different sources of 3D (first line is RDKit)

Sure it is not the other way round? I will look into it. I also wanted to add a compute_charges option.

I am trying to improve the mol-interface and conversion methods. Can you send me the smiles list to reproduce this?

I am not really sure about this part. As it is now:
m = rdkit.Chem.MolFromSmiles(smile)
# rdkit.Chem.RemoveStereochemistry(m)
if sanitize:
    rdkit.Chem.SanitizeMol(m)
if add_hydrogen:
    m = rdkit.Chem.AddHs(m)
m.SetProp("_Name", smile.strip())
if make_conformers:
    rdkit.Chem.AllChem.EmbedMolecule(m, useRandomCoords=True)
if optimize_conformer and make_conformers:
    rdkit.Chem.AllChem.MMFFOptimizeMolecule(m)
    rdkit.Chem.AssignAtomChiralTagsFromStructure(m)
    rdkit.Chem.AssignStereochemistryFrom3D(m)
rdkit.Chem.AssignStereochemistry(m)

Ok the issue is that you need for stability of 3D to addHs during the Embedding and the MMFF optimization step and remove Hs after.
but this is only fix few cases. We always have molecules that can failed the procedure.
I would do something like this
m = rdkit.Chem.MolFromSmiles(smile)
# rdkit.Chem.RemoveStereochemistry(m)
if sanitize:
    rdkit.Chem.SanitizeMol(m)

m.SetProp("_Name", smile.strip())
if make_conformers:
    if add_hydrogen:
        m = rdkit.Chem.AddHs(m)
    rdkit.Chem.AllChem.EmbedMolecule(m, useRandomCoords=True)
    if optimize_conformer:
    rdkit.Chem.AllChem.MMFFOptimizeMolecule(m)
    rdkit.Chem.AssignAtomChiralTagsFromStructure(m)
    rdkit.Chem.AssignStereochemistryFrom3D(m)
    if not add_hydrogen:
        m = rdkit.Chem.RemoveHs(m)

else:
    if add_hydrogen:
        m = rdkit.Chem.AddHs(m)

rdkit.Chem.AssignStereochemistry(m)

to avoid try catch you can also check if the Embeding step work so the function return correct validation
success = rdkit.Chem.AllChem.EmbedMolecule(m, useRandomCoords=True)
if success == -1:
            raise Exception("Embedding failed!")

Okay, thanks, I updated the kgcnn.mol.convert function. Does this help to solve your issue from above?


fix(samples): bug fix, wrong projectId.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕
If you write sample code, please follow the samples format.

@yoshi-kokoro , all the tests for google.retail.v2alpha package succeeded but encountered unknown exception, example:
java.util.concurrent.ExecutionException: com.google.api.gax.rpc.UnknownException: Operation with name "removeFulfillmentPlacesTest" succeeded, but encountered a problem unpacking it.

@yoshi-kokoro , java.lang.RuntimeException: Generated message class "com.google.cloud.retail.v2beta.ImportCompletionDataResponse" missing method "getErrorSamplesList". and java.util.concurrent.ExecutionException: com.google.api.gax.rpc.UnknownException: Operation with name "addLocalInventoriesTest" succeeded, but encountered a problem unpacking it. still persist in Kokoro test for several classes in com.google.cloud.retail.v2beta package


Remove build.sh files moved to common-utils, job-scheduler, and alerting repositories.

Signed-off-by: Marc Handalian<EMAIL_ADDRESS>Description
These build.sh files are taking priority over the latest build.sh scripts in each of these repositories.  Remove these files so the ones in each repository are picked up.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Looks like there are issues when the scripts are using for testing
>           raise AssertionError(_error_message()) from cause
E           AssertionError: Expected call: execute('bash /home/runner/work/opensearch-build/opensearch-build/scripts/components/common-utils/build.sh -v 1.1.0 -p linux -a x64 -s false -o builds')
E           Actual call: execute('bash /home/runner/work/opensearch-build/opensearch-build/scripts/default/opensearch/build.sh -v 1.1.0 -p linux -a x64 -s false -o builds')

The builder expectations have changed for these components, s//g.

Codecov Report

Merging #945 (10fd86b) into main (3675e58) will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #945      +/-   ##
==========================================
- Coverage   92.25%   92.03%   -0.22%
==========================================
  Files          88       88
  Lines        2323     2323
==========================================
- Hits         2143     2138       -5
- Misses        180      185       +5

Impacted Files
Coverage Δ

src/system/temporary_directory.py
82.75% <0.00%> (-13.80%)
:arrow_down:

src/system/os.py
92.30% <0.00%> (-7.70%)
:arrow_down:

Continue to review full report at Codecov.

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

The builder expectations have changed for these components, s//g.

Replaced common-utils with the OpenSearch component for this test.


GetFromHTTPRequest not working

Describe the bug
The GetFromHTTPRequest method is not working on Android when you have a Firebase storage link, which contains query params. I guess it's an issue similar to this one https://github.com/jepiqueau/jeep-sqlite/issues/39.
Error: GetFromHTTPRequest: length=0; index=-3
To Reproduce
Here is an example of one of my databases stored in my Firebase storage project:
https://firebasestorage.googleapis.com/v0/b/ipmacae-app.appspot.com/o/bibles%2FNVI.zip?alt=media&token=3c035e3e-7a5e-4532-bbf2-4e93e5d69b61
To make jeepSqlite work, I need to add a final param called ext with the extension of the database. It works just fine on the web but doesn't work on Android and probably on iOS either.
  async loadDatabaseFromServer(
    database: string,
    url: string,
    ext: 'db' | 'zip',
  ): Promise<void> {
    let databases: string[] = [];
    try {
      const result = await this.sqliteConnection.getDatabaseList();
      databases = result.values ?? [];
    } catch {
      // do nothing
    }
    if (!databases.includes(`${database}SQLite.db`)) {
      await this.sqliteConnection.getFromHTTPRequest(
        `${url}&ext=.${ext}`,
        true,
      );
      const result = await this.sqliteConnection.getDatabaseList();
      databases = result.values ?? [];
    }
    await Toast.show({
      text: `Database ${database} loaded from server. ${databases.values}`,
      duration: 'long',
    });
  }

Expected behavior
It should toast the success message.
Screenshots

Smartphone (please complete the following information):

Device: Pixel 7 emulator
OS: API 34

I found what the problem is, firebase returns a disposition with filename*=utf-8"{fileName}. I tried to fix it by signing the blob just like mentioned here but it still doesn't work.
If it's too overkill to handle this scenario on the plugin, I will try to move to another storage service, e.g. CloudFlare.

@rosostolato i will have a look later to see if i can find a solution for it

@rosostolato fix for Android, iOS and Electron in release 5.7.3-1


Upgrade to h3 v4.2.0

I was a bit surprised to find I needed to relax tests about edgeLength here. Did something change there?
The tests for polygonToCellsExperimental are admittedly not very comprehensive. We should test other modes as well.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

21 of 25 (84.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.1%) to 98.429%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/h3core.js
21
25
84.0%

Totals

Change from base Build<PHONE_NUMBER>9:
-1.1%

Covered Lines:
570

Relevant Lines:
575

💛  - Coveralls


Keine Räume

Hab erst seit gestern den Xiaomi Vacuum Mop Pro, daher auch gut möglich das ich was übersehe.
Bekomme einfach bei den Datenpunkten keine Räume angezeigt.
ich komm nicht drauf woran es liegt...
Danke schon mal für die Hilfe
`

mihome-vacuum.0
2021-04-22 13:56:30.013
debug
(24705) Recievded params for viomi: {"result":[5,1,0,2105,100,0,"0",0,0,7,"0",0,1,1,12,1,0,1,0,0,1],"id":5}

mihome-vacuum.0
2021-04-22 13:56:30.010
debug
(24705) MIIO RECIVE: {"result":[5,1,0,2105,100,0,"0",0,0,7,"0",0,1,1,12,1,0,1,0,0,1],"id":5}

mihome-vacuum.0
2021-04-22 13:56:29.912
debug
(24705) Message= {"id":5,"method":"get_prop","params":["run_state","suction_grade","mode","err_state","battary_life","start_time","order_time","s_time","s_area","v_state","zone_data","repeat_state","r

mihome-vacuum.0
2021-04-22 13:56:29.910
debug
(24705) get params for Viomi

mihome-vacuum.0
2021-04-22 13:56:29.354
debug
(24705) Receive <<< Helo <<<<PHONE_NUMBER>000000198768686081646cffffffffffffffffffffffffffffffff

mihome-vacuum.0
2021-04-22 13:56:09.895
debug
(24705) Recievded params for viomi: {"result":[5,1,0,2105,100,0,"0",0,0,7,"0",0,1,1,12,1,0,1,0,0,1],"id":4}

mihome-vacuum.0
2021-04-22 13:56:09.892
debug
(24705) MIIO RECIVE: {"result":[5,1,0,2105,100,0,"0",0,0,7,"0",0,1,1,12,1,0,1,0,0,1],"id":4}

mihome-vacuum.0
2021-04-22 13:56:09.702
debug
(24705) Message= {"id":4,"method":"get_prop","params":["run_state","suction_grade","mode","err_state","battary_life","start_time","order_time","s_time","s_area","v_state","zone_data","repeat_state","r

mihome-vacuum.0
2021-04-22 13:56:09.700
debug
(24705) get params for Viomi

mihome-vacuum.0
2021-04-22 13:55:49.687
debug
(24705) Recievded params for viomi: {"result":[5,1,0,2105,100,0,"0",0,0,7,"0",0,1,1,12,1,0,1,0,0,1],"id":3}

mihome-vacuum.0
2021-04-22 13:55:49.684
debug
(24705) MIIO RECIVE: {"result":[5,1,0,2105,100,0,"0",0,0,7,"0",0,1,1,12,1,0,1,0,0,1],"id":3}

mihome-vacuum.0
2021-04-22 13:55:49.603
debug
(24705) Message= {"id":3,"method":"get_prop","params":["run_state","suction_grade","mode","err_state","battary_life","start_time","order_time","s_time","s_area","v_state","zone_data","repeat_state","r

mihome-vacuum.0
2021-04-22 13:55:49.599
debug
(24705) get params for Viomi

mihome-vacuum.0
2021-04-22 13:55:29.590
debug
(24705) Recievded params for viomi: {"result":[5,1,0,2105,100,0,"0",0,0,7,"0",0,1,1,12,1,0,1,0,0,1],"id":2}

mihome-vacuum.0
2021-04-22 13:55:29.589
debug
(24705) MIIO RECIVE: {"result":[5,1,0,2105,100,0,"0",0,0,7,"0",0,1,1,12,1,0,1,0,0,1],"id":2}

mihome-vacuum.0
2021-04-22 13:55:29.379
debug
(24705) Message= {"id":2,"method":"get_prop","params":["run_state","suction_grade","mode","err_state","battary_life","start_time","order_time","s_time","s_area","v_state","zone_data","repeat_state","

mihome-vacuum.0
2021-04-22 13:55:29.378
debug
(24705) get params for Viomi

mihome-vacuum.0
2021-04-22 13:55:29.368
debug
(24705) select viomi protocol....

mihome-vacuum.0
2021-04-22 13:55:29.366
debug
(24705) DeviceModel selected to: viomi.vacuum.v8

mihome-vacuum.0
2021-04-22 13:55:29.362
debug
(24705) Get Device data from robot..

mihome-vacuum.0
2021-04-22 13:55:29.361
debug
(24705) Get Device data..0

mihome-vacuum.0
2021-04-22 13:55:29.360
debug
(24705) GETMODELFROMAPI:Data: {"partner_id":"","id":1,"code":0,"message":"ok","result":{"hw_ver":"Linux","fw_ver":"3.5.3_0017","ap":{"ssid":"meins","bssid":"e8:df:70:dd:d8:c0"},"netif":{"localIp":"19

mihome-vacuum.0
2021-04-22 13:55:29.359
debug
(24705) MIIO RECIVE: {"partner_id":"","id":1,"code":0,"message":"ok","result":{"hw_ver":"Linux","fw_ver":"3.5.3_0017","ap":{"ssid":"meins","bssid":"e8:df:70:dd:d8:c0"},"netif":{"localIp":"192.168.170

mihome-vacuum.0
2021-04-22 13:55:29.260
debug
(24705) Message= {"id":1,"method":"miIO.info"}

mihome-vacuum.0
2021-04-22 13:55:29.257
debug
(24705) GETMODELFROMAPI: objModel: {"val":"viomi.vacuum.v8","ack":true,"ts":1619091602762,"q":0,"from":"system.adapter.mihome-vacuum.0","user":"system.user.admin","lc":1619022139092}

mihome-vacuum.0
2021-04-22 13:55:29.250
debug
(24705) MAIN: Connected to device, try to get model..

mihome-vacuum.0
2021-04-22 13:55:29.249
debug
(24705) Receive <<< Helo <<<<PHONE_NUMBER>0000001987686860816430ffffffffffffffffffffffffffffffff

mihome-vacuum.0
2021-04-22 13:55:29.210
debug
(24705) server started on <IP_ADDRESS>:53421

mihome-vacuum.0
2021-04-22 13:55:29.192
debug
(24705) MIIO: Config: ip:<IP_ADDRESS> token: 7645326f354655695576443664746936

mihome-vacuum.0
2021-04-22 13:55:29.190
debug
(24705) Create State for deviceInfowifi_signal

mihome-vacuum.0
2021-04-22 13:55:29.189
debug
(24705) Create State for deviceInfofw_ver

mihome-vacuum.0
2021-04-22 13:55:29.188
debug
(24705) Create State for deviceInfomodel

mihome-vacuum.0
2021-04-22 13:55:29.187
debug
(24705) Create State for deviceInfomac

mihome-vacuum.0
2021-04-22 13:55:29.171
debug
(24705) load Map creator... true

mihome-vacuum.0
2021-04-22 13:55:29.114
info
(24705) check group user admin group admin: false

mihome-vacuum.0
2021-04-22 13:55:29.105
info
(24705) check user admin pw iobroker: true

mihome-vacuum.0
2021-04-22 13:55:28.770
info
(24705) starting. Version 3.1.1 in /opt/iobroker/node_modules/iobroker.mihome-vacuum, node: v12.17.0, js-controller: 3.1.4

mihome-vacuum.0
2021-04-22 13:55:27.594
debug
(24705) statesDB connected

mihome-vacuum.0
2021-04-22 13:55:27.591
debug
(24705) States connected to redis: <IP_ADDRESS>:9000

mihome-vacuum.0
2021-04-22 13:55:27.575
debug
(24705) States create System PubSub Client

mihome-vacuum.0
2021-04-22 13:55:27.569
debug
(24705) States create User PubSub Client

mihome-vacuum.0
2021-04-22 13:55:27.477
debug
(24705) Redis States: Use Redis connection: <IP_ADDRESS>:9000

mihome-vacuum.0
2021-04-22 13:55:27.472
debug
(24705) objectDB connected

mihome-vacuum.0
2021-04-22 13:55:27.458
debug
(24705) Objects connected to redis: <IP_ADDRESS>:9001

mihome-vacuum.0
2021-04-22 13:55:27.415
debug
(24705) Objects client initialize lua scripts

mihome-vacuum.0
2021-04-22 13:55:27.412
debug
(24705) Objects create User PubSub Client

mihome-vacuum.0
2021-04-22 13:55:27.408
debug
(24705) Objects create System PubSub Client

mihome-vacuum.0
2021-04-22 13:55:27.197
debug
(24705) Objects client ready ... initialize now

mihome-vacuum.0
2021-04-22 13:55:27.097
debug
(24705) Redis Objects: Use Redis connection: <IP_ADDRESS>:9001

mihome-vacuum.0
2021-04-22 13:55:21.875
info
(24267) Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason

mihome-vacuum.0
2021-04-22 13:55:21.872
info
(24267) terminating

mihome-vacuum.0
2021-04-22 13:55:21.869
info
(24267) Got terminate signal TERMINATE_YOURSELF

mihome-vacuum.0
2021-04-22 13:40:34.034
warn
(24267) Could not build message without arguments

mihome-vacuum.0
2021-04-22 13:40:34.031
info
(24267) send message: Method:

mihome-vacuum.0
2021-04-22 13:40:19.781
info
(24267) send message: Method: loadRooms

mihome-vacuum.0
2021-04-22 13:40:02.484
info
(24267) check group user admin group admin: false

mihome-vacuum.0
2021-04-22 13:40:02.478
info
(24267) check user admin pw iobroker: true

mihome-vacuum.0
2021-04-22 13:40:02.142
info
(24267) starting. Version 3.1.1 in /opt/iobroker/node_modules/iobroker.mihome-vacuum, node: v12.17.0, js-controller: 3.1.4

mihome-vacuum.0
2021-04-22 13:39:55.070
info
(16792) Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason

mihome-vacuum.0
2021-04-22 13:39:55.068
info
(16792) terminating

mihome-vacuum.0
2021-04-22 13:39:55.064
info
(16792) Got terminate signal TERMINATE_YOURSELF

`

https://github.com/iobroker-community-adapters/ioBroker.mihome-vacuum#supported-devices-and-fetures


vis adapter instantiation is hanging

I am not able to install iobroker.vis
it appears to me, the problem is not in vis but in iobroker.web instantiation.
The process is hanging at

the same problem appears if I try to create a iobroker.web instance.

Systeminfos
Plattform: linux
Betriebssystem: linux
Architektur: x64
CPUs: 2
Geschwindigkeit: 2901 MHz
Modell: Intel(R) Core(TM) i5 CPU M 520 @ 2.40GHz
RAM: 0 MB
System-Betriebszeit: 01:56:48
Node.js: v18.18.2
time:<PHONE_NUMBER>052
timeOffset: --
NPM: 9.8.1
Adapter-Anzahl: 510
Datenträgergröße: 7.8 GB
Freier Festplattenspeicher: 3.0 GB
Aktive Instanzen: 10
Pfad: /opt/iobroker/
Betriebszeit: 01:26:24
aktiv:

Did you checked your computer? Is anything on 100% CPU? Mem issue? any indication of othert issues?

... Oh dear, you are on the right track.
my sandbox iobroker sits on a proxmox server and I had indeed very limited
memory (1Gb). After increasing it to 4Gb instantiation is done within
seconds.
You can close this issue.
Many thanks
Am Do., 16. Nov. 2023 um 09:48 Uhr schrieb Ingo Fischer <
@.***>:

Did you checked your computer? Is anything on 100% CPU? Mem issue? any
indication of othert issues?
—
Reply to this email directly, view it on GitHub
https://github.com/ioBroker/ioBroker.vis/issues/801#issuecomment-1814019220,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFQ5PGCAHCWZ27EA6YMCALDYEXHOPAVCNFSM6AAAAAA7NYVP3OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJUGAYTSMRSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

echarts seems to be a memory-eater - here as well.
And the usage slows down the system performance: Can be demonstrated if this adapter is de-activated (Proxmox performance graphics)

... good to know.
I am currently in a evaluation phase and a iobroker newbie. I am not sure
which graphics engine I will use for my time series. I know grafana for a
long time. I just tried e-charts now, because I thought it might be the
only choice I have to integrate time series charts in vis.
Am Do., 16. Nov. 2023 um 11:50 Uhr schrieb MichaelSchaaf999 <
@.***>:

echarts seems to be a memory-eater - here as well.
And the usage slows down the system performance: Can be demonstrated if
this adapter is de-activated (Proxmox performance graphics)
—
Reply to this email directly, view it on GitHub
https://github.com/ioBroker/ioBroker.vis/issues/801#issuecomment-1814205820,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFQ5PGEVKLJNCUZS5JUWO4TYEXVWBAVCNFSM6AAAAAA7NYVP3OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJUGIYDKOBSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ok thats all interesting ... and yes intertingly enought echarts (and also web?) seems siometimes have issues when beuing uploaded.
(@foxriver76 another case). Please share yiur nodejs versions and such.
When echarts needs too much CPU when running please open an echarts issue with details

@Apollon77 I see the issue of massive RAM consumption on different adater updates (uploads) like vis2, fullcalender as well. Currentl my production system has 8 GByte RAM which are sporadically necessary to update some adapters.
This problem should be tracked somewhere else (js-controller??) but I think it is not a vis specific issue.

Is it during the npm install or during upload?

@foxriver76 on my system during upload

Then controller issue

@foxriver76 Issue created https://github.com/ioBroker/ioBroker.js-controller/issues/2538


[CARBONDATA-3626] Improve performance when load data into carbon table with lots of columns

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[x] Any interfaces changed? No

[x] Any backward compatibility impacted? No

[x] Document update required? No

[x] Testing done Yes
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.  OK

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1234/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1244/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1253/

LGTM.
Thanks for contributing!


changeLocationMarker() null cannot be cast to non-null

Hey, can u please help with this problem? I want first time add marker to map and then just changelocation of this marker. I have something like this in my code
setupMarker(){
  if (oldLocation!= null) {
     await controller.changeLocationMarker(oldLocation: oldLocation, newLocation: geo,markerIcon: icon);
  } else {
     await controller.addMarker(geo, markerIcon: icon);
     oldLocation = geo;
  }
}

if method changeLocationMarker() is called i got this error (only if i add markerIcon parameter)
java.lang.NullPointerException: null cannot be cast to non-null type kotlin.ByteArray
E/AndroidRuntime(20508): 	at hamza.dali.flutter_osm_plugin.FlutterOsmView$changePositionMarker$1.invokeSuspend(FlutterOsmView.kt:591)

ok i will check the issue

i will publish new version contain fix for this
sorry for being late

check our new version 0.54.1

i tried it but this async method never completes
await controller.changeLocationMarker(
            oldLocation: oldGeo, newLocation: newGeo,markerIcon: icon);

is the icon in changeLocationMarker  the same as  in addMarker  ?

yes same, but doesnt work even if dont set markerIcon

can you provide example from your code
because if you check our home example we already use it
and it should works

GeoPoint? markerOrderDest;

setupMarker(GeoPoint geo){ // running periodically
        if (markerOrderDest != null) {
            var oldLocation = GeoPoint(latitude: markerOrderDest!.latitude, longitude: markerOrderDest!.longitude);
            await controller.changeLocationMarker(oldLocation: oldLocation,newLocation: geo, markerIcon: icon);
            print("!!!!!!!never goes here!!!!!!!!!!!!!!");
        } else {
          await controller.addMarker(geo, markerIcon: icon);
        }
        markerOrderDest = geo;
}

if changeLocationMarker  not working properly use removeMarker and addMarker to simplification
it will be great if you have log to understand more the issue

if you can share more code to understand more the issue

Here is the actual code that my colleague Robert wrote about. This code already includes a workaround to remove and add the marker. The problem is that the marker disappears, and only appears after a while. It's not seamless. We would need to fix the changeLocationMarker, so change of position will be fluent.
 void setupOrderMarker() async {
    for (WaypointDto waypoint in widget.userDetail.order!.waypointList) {
      var geo =
          GeoPoint(latitude: waypoint.latitude, longitude: waypoint.longitude);

      var icon = MarkerIcon(
          iconWidget: SvgPicture.asset(
        waypoint.type == WaypointTypeEnum.DESTINATION
            ? "assets/graphics/location_dest_icon.svg"
            : "assets/graphics/location_board_icon.svg",
        height: 136,
      ));

      if (waypoint.type == WaypointTypeEnum.DESTINATION) {
        if (markerOrderDest == null) {
          await controller.addMarker(geo, markerIcon: icon);
        } else if (markerOrderDest!.longitude != geo.longitude ||
            markerOrderDest!.latitude != geo.latitude) {
          var oldLocation = GeoPoint(
              latitude: markerOrderDest!.latitude,
              longitude: markerOrderDest!.longitude);
//await controller.changeLocationMarker(oldLocation: markerOrderDest,newLocation: geo, markerIcon: icon);
// workaround
          await controller.removeMarker(oldLocation);
          await controller.addMarker(geo, markerIcon: icon);
        }
        markerOrderDest = geo;
      } else if (waypoint.type == WaypointTypeEnum.BOARD) {
        if (markerOrderBoard == null) {
          await controller.addMarker(geo, markerIcon: icon);
        } else if (markerOrderBoard!.longitude != geo.longitude ||
            markerOrderBoard!.latitude != geo.latitude) {
          var oldLocation = GeoPoint(
              latitude: markerOrderBoard!.latitude,
              longitude: markerOrderBoard!.longitude);
//await controller.changeLocationMarker(oldLocation: markerOrderDest,newLocation: geo, markerIcon: icon);
// workaround
          await controller.removeMarker(oldLocation);
          await controller.addMarker(geo, markerIcon: icon);
        }
        markerOrderBoard = geo;
      }
    }


PDF417 does not work on iOS

Expected Behaviour
PDF417 barcodes should be decoded
Actual Behaviour
PDF417 barcodes are never detected by the scanner in iOS. Works properly on Android
Reproduce Scenario (including but not limited to)
Any attempt to call
this.barcodeScanner.scan({formats : "PDF417"})

Steps to Reproduce
See sample code
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Plugin version 6.0.6
iOS 10.3.2
Android 7.1.2
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Nexus 5X
Cordova CLI version and cordova platform version
cordova --version
7.0.1
Installed platforms:
android 6.2.3
ios 4.4.0
Plugin version
phonegap-plugin-barcodescanner 6.0.6 "BarcodeScanner"

Sample Code that illustrates the problem
this.barcodeScanner.scan({formats : "PDF417"}).then((barcodeData) => {
      // Success! Barcode data is here
    }, (err) => {
      // An error occurred
      console.error(err);
    });

Logs taken while reproducing problem
Nothing useful is logged to the ionic console when this happens

Might be related to #489

@voodoodrul can you try:
this.barcodeScanner.scan({formats : "PDF_417"}).then((barcodeData) => {
      // Success! Barcode data is here
    }, (err) => {
      // An error occurred
      console.error(err);
    });

@macdonst I should have mentioned, but I did try a few variations of the type string. I tried "PDF417", "PDF_417", lowercase versions, and even adding a few other types before or after. I'll test again on the latest code.

@voodoodrul I'm using this image as a test barcode:
https://en.wikipedia.org/wiki/PDF417#/media/File:Better_Sample_PDF417.png
Using the current version of the barcode scanner it works properly. One thing about the current version on iOS is that is pretty much ignores formats which is something PR #431 should fix.

@macdonst, This is the image example that you asked me for in PR #509

It works correctly on Android but not in iOS.
Thanks

I have same problem works on Android perfect but on IOS only works with  normal text, this is an example of a backpart of costa rican card  ID,  on android we can read it perfect but in iOS something happen, with my id just read 12 chars with my partner card ID we only read 1.
this is an image example

cordova.plugins.barcodeScanner.scan(
function (result) {
alert("We got a barcode\n" +
"Result: " + binaryAgent(result.text) + "\n" +
"Format: " + result.format + "\n" +
"Cancelled: " + result.cancelled);
			console.log(binaryAgent(result.text))
  },
  function (error) {

  },
  {
      preferFrontCamera : false, // iOS and Android
      showFlipCameraButton : true, // iOS and Android
      showTorchButton : true, // iOS and Android
      torchOn: true, // Android, launch with the torch switched on (if available)
      saveHistory: false ,// Android, save scan history (default false)
      prompt : "Ubique la parte detraz de la cédula", // Android
      resultDisplayDuration: 3500, // Android, display scanned text for X ms. 0 suppresses it entirely, default 1500
      formats : "PDF_417", // default: all but PDF_417 and RSS_EXPANDED
      orientation : "landscape", // Android only (portrait|landscape), default unset so it rotates with the device

  }

);

Is binary or text? ios version is giving problems with binary pdf417 codes due the encode, for example the next seq is read from a binary pdf using the android version of the plugin, but in apple code between 128 and 196 are taken  as control signals
19_0_53_152_55_63_167_32_149_156_114_180_66_157_79_141_79_116_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_96_75_246_69_75_210_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_127_181_74_131_32_155_73_98_119_75_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_222_35_153_233_6_208_54_225_53_237_48_21_3_52_148_48_59_147_18_160_209_51_224_3_208_49_233_2_2_48_103_241_255_143_19_149_44_171_183_156_140_162_149_71_171_118_152_117_42_60_157_252_108_236_84_85_157_56_163_106_173_95_64_77_119_54_119_153_249_147_202_65_167_114_163_67_143_65_162_187_166_113_47_156_186_52_176_10_124_98_64_152_64_182_115_171_191_129_144_29_156_219_7_73_33_187_78_165_108_255_45_16_186_106_221_187_12_80_149_37_236_95_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_50_32_160_119_94_108_146_32_95_188_97_139_83_146_166_152_186_159_83_10_120_129_229_144_229_67_97_103_111_81_66_171_82_64_66_112_198_107_139_229_143_207_86_92_120_81_115_98_69_163_187_162_156_57_121_176_205_160_4_118_79_116_179_99_85_125_163_188_157_172_193_133_96_18_91_208_129_86_69_100_127_50_112_188_126_29_251_20_84_243_37_239_93_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_160_0_15_147_18_160_209_51_224_3_208_0_223_0_39_48_4_

#519 also i had this problem

@jasj There should be control chars in the rawText. I encode to base64 before transporting it in order to preserve these since they are critical for decoding AAMVA. AAMVA allows each issuer of a license to specify their own delimiter control char, so anything is fair game there.

@voodoodrul  i didnt get any trouble with iOS using raw text.  and base64 have the only normal chars, so it will work perfectly


chore: fix typos

Will fix the Python actions in another PR since it's related to the docker image release.


workspace/configuration clarification

According to this, rust-analyzer basically requires workspace/configuration to be supported.
According to LSP discussion, assuming I understand the comments correctly:

The client fires a didChangeConfiguration notification
The server then makes a workspace/configuration request

So the client side should look something like this:
result = []
for queried_config in request[ 'params' ][ 'items' ]:
    config_key = queried_config.get( 'section', '' )
    result.append( new_config.get( 'config_key' ) )

The request I got from rust-analyzer was:
2020-05-02 19:33:36,178 - DEBUG - RX: Received message: b'{"jsonrpc":"2.0","id":2,"method":"workspace/configuration","params":{"items":[{"section":"rust-analyzer"}]}}'

Is this aligned with what rust-analyzer expects?
If so, where can I see the RA configuration options?

This is the answer I received in how this should all work (your point 1): https://github.com/microsoft/language-server-protocol/issues/676

Okay, so we're on the same page.
If I receive something like
{"jsonrpc":"2.0","id":2,"method":"workspace/configuration","params":{"items":[{"section":"rust-analyzer"}]}}

what would be possible values for this? To be clear, this isn't asking for complete configuration dictionary?

The spec is super unclear about this, but what VS Code does is that returns all config keys nested under rust-analyzer key:
[Trace - 12:25:53 AM] Received request 'workspace/configuration - (8)'.
Params: {
    "items": [
        {
            "section": "rust-analyzer"
        }
    ]
}

[Trace - 12:25:53 AM] Sending response 'workspace/configuration - (8)'. Processing request took 2ms
Result: [
    {
        "diagnostics": {
            "enable": true
        },
        "lruCapacity": null,
        "files": {
            "watcher": "client",
            "exclude": []
        },
        "notifications": {
            "workspaceLoaded": true,
            "cargoTomlNotFound": true
        },
        "cargo": {
            "noDefaultFeatures": false,
            "allFeatures": true,
            "features": [],
            "loadOutDirsFromCheck": false
        },
        "rustfmt": {
            "extraArgs": [],
            "overrideCommand": null
        },
        "checkOnSave": {
            "enable": true,
            "extraArgs": [],
            "command": "check",
            "overrideCommand": null,
            "allTargets": true
        },
        "inlayHints": {
            "typeHints": true,
            "chainingHints": true,
            "parameterHints": true,
            "maxLength": 20
        },
        "completion": {
            "addCallParenthesis": true,
            "addCallArgumentSnippets": false,
            "postfix": {
                "enable": true
            }
        },
        "callInfo": {
            "full": false
        },
        "updates": {
            "channel": "stable",
            "askBeforeDownload": true
        },
        "serverPath": null,
        "trace": {
            "server": "verbose",
            "extension": false
        },
        "procMacro": {
            "enable": false
        },
        "debug": {
            "engine": "auto",
            "sourceFileMap": {
                "/rustc/<id>": "${env:USERPROFILE}/.rustup/toolchains/<toolchain-id>/lib/rustlib/src/rust"
            }
        },
        "cargo-watch": {
            "enable": true
        }
    }
]

So, it kinda is complete configuration dictionary. It even includes client-side specified default values, which is actively harmful, as we had to duplicate defaults between the server and the client (if we omit defaults on the client, the client fills them with type-based defaults, like false, 0, empty array, etc, instead of fully ommiting them, and this "defaults" override our defaults) .
Also note that, although in VS Code config settings are specified using dotted keys like "rust-analyzer.foo.bar", the response from vscode uses nested structs: rust-analyzer: { foo: { bar: ... } }

If so, where can I see the RA configuration options?

I believe you don't need to know the set of all options, you need to just forward whatever user has written in the config. Unfortunatelly, there's no way to specify "config schema" in the LSP. For VS Code, we list the options here, though this list contains both options specific to typescript extension (ie, which rust-analyzer binary to use) and the options which affect the server. VS Code sends them all to the server, and we just ignore extra ones.

@matklad Thanks for the info, though this raises more questions.

what VS Code does is that returns all config keys nested under rust-analyzer key:

Did I understand you correctly, that VSCode responds with [ config[ 'rust-analyzer' ] ]? If so, then I can implement that.
From what I'm reading, I'm allowed to omit whatever fields the user didn't care about and rust-analyzer will just use its defaults. In case the user didn't care about any configuration, I would only have {} as settings, so setting.get( 'rust-analyzer' ) would return None, which gets serialized as null. The protocol allowed this, but is rust-analyzer okay with that?

I believe you don't need to know the set of all options, you need to just forward whatever user has written in the config.

yeah, sure, but the user needs to know, and we need to test. Where do they find out? Are they expected to read some json file in order to know what to configure?
In fairness, rust-analyser is not the only server that doesn't docuemnt its configuration. Most of the others don't either, so we have to do it for them.
New tool idea: Something which scrapes all theses irritating json files that vscode requires and actually makes human-readable docs out of them. Who's with me?!

The protocol allowed this, but is rust-analyzer okay with that?

Yeah, we should handle this fine (although I didn't test this).

New tool idea: Something which scrapes all theses irritating json files that vscode requires and actually makes human-readable docs out of them. Who's with me?!

This ideally should be a part of the protocol I think. Something like .configurationSchema  in ServerCapabilities or something like that, so that clients can render documentation themselves:

Though, generated docs would also be a nice addition. Generate config from something has been on my todo list for some time... Though, I'd rather make Rust struct than json file a source of truth.

Yeah, we should handle this fine (although I didn't test this).

At the very least, rust-analyzer didn't complain about {"result":[null]}. For the record, I got the answers I came here for. As far as I'm concerned, the issue can be closed.
<rant>

The protocol is so frustrating at times...
Everything we've said here, completely disagrees with the go server (for the curious, check my other opened issue with the same title). Yes, I know rust-analyzer shouldn't be concerned about gopls.
</rant>


fix: new error for unverified login

Description

Issue: #5040
When an unverified user logs in, they are only presented with an authentication error. This change would resolve a new error
"This user has not verified their email address", that will allow the front-end to direct the user to validate their email.
I have not been able to run the translations script. It seems to error due to other missing fields.
Potentially a breaking change if consumers are currently expecting the standard authentication error.

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[ ] Chore (non-breaking change which does not add functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Change to the templates directory (does not affect core functionality)
[ ] Change to the examples directory (does not affect core functionality)
[ ] This change requires a documentation update

Checklist:

[x] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

Here's a recommendation,
as I also require such functionality.
For some users, the option to resend a verification email,
or to handle additional processes, is a quite common feature.
Thank you very much!🙏🙏

Why has this been closed? Looks like a pretty essential feature to me

@OlliePrentice can you please re-open this?


Implement inverse trigonometric functions.

Merge #1534 first.
This PR implements inverse trigonometric functions: arcsin, arccos and arctan.

Please split PR trigonometric functions and unary_math_function_test.

It is already split. The commit list includes unary_math_function_test's commits because the PR is compared to the current master. After #1534 unary_math_function_test will be merged, the list will contain only commits for inverse trigonometric functions.
Before that, please use the following comparison for reviewing.
https://github.com/takagi/chainer/compare/unary-function-test...takagi:inverse-trigonometric

The commit list is now cleaned.

LGTM


Fix blind update logic of emulator

original issue https://github.com/ishikota/PyPokerEngine/issues/46
Fixed logic of blind update in emulator.
before
#initial_ante = 0
#initial_small_blind = 5
#blind_structure = { 2: { "ante": 5, "small_blind": 10 } }
round2_start_state = self.emu.start_new_round(round1_finished_state)
# round2_start_state["small_blind_amount"] => 10
# ...
round3_start_state = self.emu.start_new_round(round2_finished_state)
# round2_start_state["small_blind_amount"] => 5 (it backs to initial blind amount)

Coverage increased (+0.004%) to 97.849% when pulling 65a667aad091fc77fad71f108e75bb4acd537d06 on bug-fix-emulator-blind-structure into 5529ae1252bbb5639f4f235cdd2656d8b1da1b4a on master.


[WIP] Agent provisioning refactor

Adds the ability to supply custom ProvisioningRecord with an invocation method to run post-provisioning configuration (e.g. register NYM for issuer/agent, etc).
Simplify the agent service registration and building pipeline

Moved the agent provisioning to a hosted service IHostedService. This ensures the provisioning will be done during the service startup and also provides tighter integration with the Generic Host improvements coming in NETCore 3.0
Removed a lot of the redundant agent builder classes during configuration
Added agent factory support that allows creating agent instances with a default implementation of IAgentFactory [WIP]

Following our conversation perhaps review the role of the AgentFactory and its distinction in regards to AgentContextProvider.

Following our conversation perhaps review the role of the AgentFactory and its distinction in regards to AgentContextProvider.

I merged them together into one provider. Let me know your thoughts.


Video maximized inside a window first

Description of the new feature / enhancement
When you are watching a video within a window you have the option to enlarge it to a full screen. There should be a middle option of expanding it to the size of the window it's in first.
Scenario when this would be used?
Currently, we have ultrawide monitors. We take advantage of this having several windows open simultaneously and that allows us to watch a video in one window, such a browser for example, and take notes in another window if we want. With the option to maximize the video first to the window in which you are viewing it, you can continue to use the rest of the monitor for other things.
Supporting information
No response

/dup #279

Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!


Use 'active' in sitemap for product

I think it need add the filter by 'active' field in sitemap (django-shop/shop/views/sitemap.py), using something manager:
    def items(self):
        return ProductModel.objects.active()

Although, each site can have own logic for active. If I correct understand, 'get_availability' in BaseProduct can used in manager with method active() (like as above). I hope you develop my idea

The intention of active is to set it to false instead of removing the the product. This is required for SEO purpose, so that products are still reachable through the search engine, even if the have been removed from the catalog.

If the have been removed from the catalog, then how product is displayed by url? I think we need show 404 error page for this case, need not we?

That would be the logical behavior, but the SEO typically recommends to mark that product as gone.
Google for "SEO considerations for discontinued products in ecommerce" and/or listen to Matt Cutts.
Probably the best is to use kind of meta-tag.

Thank you for material about SEO. For different cases we try to display appropriate 404 page. For example, for product is 'Sorry, but product is saled' or for article is 'Article is removed' and so on. Nevertheless I'm agree with you about SEO.

Good point. So want you can do is to replace the url-config for your products as
myshop/urls/product.py:
urlpatterns = [
    ...
    url(r'^(?P<slug>[\w-]+)/?$', ProductRetrieveView.as_view(
        serializer_class=ProductDetailSerializer,
        limit_choices_to=Q(active=True),
    )),
    ...

this would raise a NotFound exception for inactive products. Disclaimer: never tested.

Thanks for code! I will think about how is better. But now I think this issue may be closed.


Export the base form control component

I know this library is for Chakra UI only, but I would like to use other input types as well. For example I would like to extend this library to support @choc-ui/chakra-autocomplete and it would be easier if I had access to BaseProps and modified FormControl.

Also, if either PR for exporting FormControl gets merged, I'll make an other PR to make it possible to pass props to FormLabel, FormErrorMessage, and FormHelperText as well.

I accepted the PR that exports FormControl.
I'm thinking of adding FormLabel, FormErrorMessage, and FormHelperText  to BaseProps but it can complicate things in a bad way.
What is your preferred way to go here? @verekia @wkrasnicki , opinions are very applicated.
Thank you and sorry for the delay.

How bad would adding these props complicate things? Wouldn't passing them as nested objects (like inputProps etc) be sufficient?

I made a PR. What do you guys think of #102?

I'm thinking of adding FormLabel, FormErrorMessage, and FormHelperText to BaseProps but it can complicate things in a bad way.

I'm not sure if my PR is what you meant. Did you have something else in mind?

Thank you for the PR. I was what I had in my mind. It's merged now.

Thank you for the merge! Was the issue reopened by mistake or is there a problem? :)

Yep, we can close it now.


Can We Expose Additional OIDC Claims Resolvers

Hello,
You are already exposing most of the OIDC parameters including username and password which is awesome. We are requesting that ResponseType and ResponseMode also be added to the OIDC claims resolvers to complete the full spectrum of OIDC claims.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4fc94d79-52c2-d03a-9821-32c048ffa020
Version Independent ID: e4303183-d476-6104-fcaf-00e3beb23787
Content: Claim resolvers in custom policies - Azure AD B2C
Content Source: articles/active-directory-b2c/claim-resolver-overview.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@Norrch2
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@Norrch2
Thank you for your feedback and patience. We're working with the product team to investigate your request, and we'll comment back when we've something.

@Norrch2
Since this issue isn't directly related to improving our docs and is along the lines of a product-feedback/feature request, I'd recommend leveraging our User Voice Forum and creating a feature request, so our engineering team can look into implementing this. I've also created an internal feature request, so our engineering team is aware of this as well.
Thank you for your time and patience throughout this issue!

subscribed.


tsbs_generate_data: refactor getting tag keys from simulator

closes #43

Codecov Report

Merging #44 into master will decrease coverage by 0.03%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master      #44      +/-   ##
==========================================
- Coverage   53.68%   53.65%   -0.04%
==========================================
  Files          66       66
  Lines        3172     3174       +2
==========================================
  Hits         1703     1703
- Misses       1457     1459       +2
  Partials       12       12

Impacted Files
Coverage Δ

.../tsbs_generate_data/devops/common_generate_data.go
94.73% <0%> (-5.27%)
:arrow_down:

cmd/tsbs_generate_data/main.go
73.61% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

(Thanks for all these PRs, super helpful)

Made the changes myself to your commit: #51


Added retro programming languages

What programming languages have I added?

Ada
Assembler
COBOL
Fortran
Pascal
Lisp
Oberon

Why did I do this?
There were 3 reasons for this:

Just like that, I was interested and I could do it.
In part, this project compares programming languages and there were not enough “basic” languages so that people for 30+ years would not say “but C would be faster!”)))
This shows that even very old or simply familiar technologies can be combined with new cloud technologies, code assembly in a container or serverless. This will allow you to think differently about digital transformation and migration to the cloud.

That's what inspired me https://aws.amazon.com/blogs/opensource/serverless-cobol-rejuvenating-legacy-code-with-open-source-software/
Thanks to this article, only thanks to it it became possible to do this work.

👋 @kkeker !
Thanks for the PR 🎉
Would you be able to resolve the conflicts and merge main in your branch since there are a lot of unrelated changes (already merged in main) in this PR making it a bit difficult to review
Let me know if you need help of course!