docs: Document installation on EKS using Helm and Operator and integration with AWS services for auth, monitoring and logging

Description
This documents installation and integration on AWS when installing using either the Helm Chart or the Operator.

Installation on EKS clusters using Helm or Operator
Integration with AWS CloudWatch Logs for logging
integration with Amazon Managed Service for Prometheus for monitoring metrics
Using AWS Cognito as auth provider (through the built-in oidc provider)

Which issue(s) does this PR fix

Related to https://issues.redhat.com/browse/RHIDP-1065 (Epic)

Related to https://issues.redhat.com/browse/RHIDP-1081 (Story covering the Helm Chart)
Related to https://issues.redhat.com/browse/RHIDP-1068 (Story covering the Operator)

PR acceptance criteria
Please make sure that the following steps are complete:

[x] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[x] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Requires an AWS account.
Follow the documentation in this PR to try it out

@rm3l Keep in mind that for RHDH you will also need to create PullSecret and configure helm chart to use to in order to download images from RH container registry

@rm3l Keep in mind that for RHDH you will also need to create PullSecret and configure helm chart to use to in order to download images from RH container registry

@kadel I've added those instructions in a separate document for downstream RHDH. This GDoc can be used later on as a starting point by the docs team. See https://docs.google.com/document/d/1MWPIaehSAIH-B_peV4UgHuuER-3ZgSbLSLQxomDRfuE/edit
PTAL. Thanks.

/hold
I'll update this PR accordingly after the docs team merges the downstream version. We are iterating on the corresponding doc.

@rm3l is this aligned with downstream docs, can we merge it?

@rm3l is this aligned with downstream docs, can we merge it?

@kadel Yes, this is aligned with downstream docs and can be merged. Thanks!

/override "PR Docker Build"
Doc-only PR

/override "PR Docker Build"
Doc-only PR

@rm3l I'm assuming this is outdated because we already have doc in https://docs.redhat.com/en/documentation/red_hat_developer_hub/1.2/html-single/installing_red_hat_developer_hub_on_amazon_elastic_kubernetes_service/index for installation and all three linked jIRAs are closed.


[GLUTEN-3134][CH]Bug fix left join field not match

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #3134)
How was this patch tested?
UT
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Run Gluten Clickhouse CI


Fix some typos in the codebase

I have used the SwiftTypoDetector tool that I built recently, to try to find some typos in the .swift files of the project.
Given the tool is pretty new, it spat out a lot of false positives, but I managed to find and a fix some typos.
Let me know if I'm not following some convention, and I'll do my best to stick to them!

@swift-ci test


Goal: main view

[x] description
[ ] #93
[x] issuer
[ ] #94
[ ] #95
[x] createdAt
[x] state
[ ] #96
[ ] #97
[ ] #92
[ ] #98
[x] i'am reporter/owner styles

Figma https://www.figma.com/file/OZhSt60CA5rinmbr5aTfSP/Taskany?node-id=47%3A457

So, I've made smaller issues to finish this big one. Currently this one can be closed.


I want to add google's bert as a new feature

I have tested Google's BERT as a text classifier, it performances very well in my company. So, I want to migrate BERT's power to RASA NLU. I'm workding on it recently, does anyone else interest?

Can you explain further and perhaps share some part of your work ?
Do you apply Bert on Intent classification or Entity extraction or something else?

I have tested Google's BERT as a text classifier, it performances very well in my company. So, I want to migrate BERT's power to RASA NLU. I'm workding on it recently, does anyone else interest?

Can you explain further and perhaps share some part of your work ?
Do you apply Bert on Intent classification or Entity extraction or something else?

I apply Bert on Intent classification, work language is chinese

did u append the word vector of each token from Bert to the Featurization ?

Hi @winstarwang we're currently playing around with it ourselves actually, thanks for the suggestion. There's a branch called bert_embeddings which you can play around with if you like. One of the major concerns we have at the moment is that at prediction time, using BERT takes quite a long time

did u append the word vector of each token from Bert to the Featurization ?

yes

Hi @winstarwang we're currently playing around with it ourselves actually, thanks for the suggestion. There's a branch called bert_embeddings which you can play around with if you like. One of the major concerns we have at the moment is that at prediction time, using BERT takes quite a long time

Thanks very much! I will try it later

Hi @akelad, do you have in plan playing around with LASER from Facebook too? Table on linked article suggest that LASER has better performance than BERT at zero-shot transfer.

@winstarwang We've added a BERT component in our component suite if you want to check it out.
https://pypi.org/project/innatis/

@blakeandrewwood thanks very much,I will try it later.

awesome work @blakeandrewwood!
Hey @prograwertyk we've looked into it briefly but haven't tried it yet, definitely worth doing so though, thanks for the suggestion :)

@blakeandrewwood can you remove the dependence on "bert_tfhub_module_handle", I try it failed with the inaccessible

Any update on this? I looked at the bert_embeddings branch and it seems like some of the BERT code has changed or the code in that branch references functions that aren't in BERTs code.

yeah it's based off of my/rasas fork of the repo. Honestly right now we're not convinced we should be adding it to the repository given inference time is very long and performance improvement isn't huge. But we're still running a few more experiments and will let you know

Thanks I will take a look at the fork!

I would also suggest looking at this repo: https://github.com/hanxiao/bert-as-service
as there is a significant speedup according to this thread: https://github.com/google-research/bert/issues/18

yeah we looked into that repo, it seems to require an insane amount of computing power though, so for us it was simpler to just use the existing code

For people wondering what the numbers are for inference time. I ran a few tests on one of our models:
Non-BERT model: ~0.04 seconds per request
model with BERT features: ~13 seconds per request

I have made a clone of rasa-nlu-0.14.4 and add bert feature. If someone has interest with it, you can go to https://github.com/winstarwang/rasa_nlu_bert

For people wondering what the numbers are for inference time. I ran a few tests on one of our models:
Non-BERT model: ~0.04 seconds per request
model with BERT features: ~13 seconds per request

do you use the presist() and load() ?

@winstarwang no I didn't, which would explain the 13 seconds! I looked over your code and I will set it up and run a few tests and get back with the results.
Thanks!

For people wondering what the numbers are for inference time. I ran a few tests on one of our models:
Non-BERT model: ~0.04 seconds per request
model with BERT features: ~13 seconds per request

which environment did you run this? have seen bert standalone executing around 0.5 seconds on cpu.

I have made a clone of rasa-nlu-0.14.4 and add bert feature. If someone has interest with it, you can go to https://github.com/winstarwang/rasa_nlu_bert

Hi @winstarwang I looked at your repo and set it up..Can you please advise as to how should I go about training the bert module in your repo.I just trained google bert model using this linkhttps://blog.insightdatascience.com/using-bert-for-state-of-the-art-pre-training-for-natural-language-processing-1d87142c29e7 and it workd fine. But how should I train it when it is integrated with RASA?Can I provide the model directly path(already trained using google bert) somewhere?Please advise

yeah it's based off of my/rasas fork of the repo. Honestly right now we're not convinced we should be adding it to the repository given inference time is very long and performance improvement isn't huge. But we're still running a few more experiments and will let you know

hi @akelad ..I just trained a model using google bert and and another model using RASA.I used the exactly same dataset for training in both the models.I can defnitely tell you that google bert is working like a charm and predicting intents perfectly whereas RASA is going haywire on many of the sentences that I give it for predicting intents. Although I am in the process of integrating bert with RASA and then will repeat the tests again.

@anubhavpnp what kind of dataset is this?

yeah it's based off of my/rasas fork of the repo. Honestly right now we're not convinced we should be adding it to the repository given inference time is very long and performance improvement isn't huge. But we're still running a few more experiments and will let you know

hi @akelad ..I just trained a model using google bert and and another model using RASA.I used the exactly same dataset for training in both the models.I can defnitely tell you that google bert is working like a charm and predicting intents perfectly whereas RASA is going haywire on many of the sentences that I give it for predicting intents(I used tensorflow embedding pipeline in RASA). Although I am in the process of integrating bert with RASA and then will repeat the tests again.

Bert is way way better than other models when i have tried it standalone.

@anubhavpnp what kind of dataset is this?

@akelad Its a dataset containing about 14500 sentences which are divided into 7 classes. i.e. 2.2 k sentences in each dataset of a class. I used the same dataset in both RASA and standalone google bert(according to the format of each) and then gave sample sentences to predict the classes. The formation of the sentence was very different then the training datasets.

I have made a clone of rasa-nlu-0.14.4 and add bert feature. If someone has interest with it, you can go to https://github.com/winstarwang/rasa_nlu_bert

hi @winstarwang ..I have used your branch but I am getting below error when I try to train RASA nlu
2019-05-09 16:53:10 INFO     rasa_nlu.model  - Starting to train component tokenizer_bert
2019-05-09 16:53:13 INFO     rasa_nlu.model  - Finished training component.
2019-05-09 16:53:13 INFO     rasa_nlu.model  - Starting to train component ner_crf
Traceback (most recent call last):
File "/usr/local/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/local/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/usr/local/lib/python3.6/site-packages/rasa_nlu/train.py", line 184, in
num_threads=cmdline_args.num_threads)
File "/usr/local/lib/python3.6/site-packages/rasa_nlu/train.py", line 154, in do_train
interpreter = trainer.train(training_data, **kwargs)
File "/usr/local/lib/python3.6/site-packages/rasa_nlu/model.py", line 196, in train
**context)
File "/usr/local/lib/python3.6/site-packages/rasa_nlu/extractors/crf_entity_extractor.py", line 140, in train
dataset = self._create_dataset(filtered_entity_examples)
File "/usr/local/lib/python3.6/site-packages/rasa_nlu/extractors/crf_entity_extractor.py", line 149, in _create_dataset
dataset.append(self._from_json_to_crf(example, entity_offsets))
File "/usr/local/lib/python3.6/site-packages/rasa_nlu/extractors/crf_entity_extractor.py", line 453, in _from_json_to_crf
ents = self._bilou_tags_from_offsets(tokens, entity_offsets)
File "/usr/local/lib/python3.6/site-packages/rasa_nlu/extractors/crf_entity_extractor.py", line 473, in _bilou_tags_from_offsets
starts = {token.offset: i for i, token in enumerate(tokens)}
File "/usr/local/lib/python3.6/site-packages/rasa_nlu/extractors/crf_entity_extractor.py", line 473, in
starts = {token.offset: i for i, token in enumerate(tokens)}
AttributeError: 'str' object has no attribute 'offset'
Makefile:24: recipe for target 'train-nlu' failed
make: *** [train-nlu] Error 1
Below is the content of nlu_config.yml
language: "en"
pipeline:

name: "tokenizer_bert"
name: "ner_crf"
name: "ner_synonyms"
name: "intent_featurizer_bert"
name: "intent_classifier_tensorflow_bert"
I am using the RASA starter nlu project at https://github.com/RasaHQ/starter-pack-rasa-nlu
I built the RASA nlu project using your branch and installed it in my local and then used project at above link. Can you please help.If I replace bert_tokeniser with the standard whitespace_tokeniser, it works

I have made a clone of rasa-nlu-0.14.4 and add bert feature. If someone has interest with it, you can go to https://github.com/winstarwang/rasa_nlu_bert

hi @winstarwang ..I notice that you have not handled offsets in the bert_tokenizer because of which the error is coming..Any idea of how to handle that?

@anubhavpnp https://github.com/GaoQ1/rasa_nlu_gq/blob/master/rasa_nlu_gao/featurizers/bert_vectors_featurizer.py  I already added it.

Thanks a lot @GaoQ1 ! ..I will try it

@anubhavpnp https://github.com/GaoQ1/rasa_nlu_gq/blob/master/rasa_nlu_gao/featurizers/bert_vectors_featurizer.py I already added it.

hi @GaoQ1 ...Couple of questions:

I suppose bilstm_crf_entity_extractor.py is the file that does entity extraction utilizing google bert.But  I could not find any parameter in defaults section which specifies the fine tuned google bert model that we want to utilize.
I already have intent classification working for google bert. I only want to utilize entity extraction. Can I just copy this file to extractors folder of my RASA project and mention this file in the pipeline in nlu_config file.
I will not be using this on a GPU. I will be using a simple vm for it. Would it work properly?

Thanks in advance

@akelad Is your branch still usable ? I have the same issue as https://github.com/RasaHQ/rasa/issues/3674. The import of create_features from bert.extract_features doesn't work, as it doesn't find it.
The package to install is only bert-tensorflow, isn't it ?

Hi @GaoQ1
I have seen your implementation of an EmbeddingBERTClassifier. Am I able to use it with rasa version >= 1.0 ? What if I want to use a pretrained BERT model?
Any help in which version is currently working would be appreciated since I am going to start some experiments into the direction of domain specific fine-tuning on a german BERT.
Regards

I have made a clone of rasa-nlu-0.14.4 and add bert feature. If someone has interest with it, you can go to https://github.com/winstarwang/rasa_nlu_bert

Hi @winstarwang ..Your repo was quite useful..Do you have any plans of modifying it so that it is compatible with rasa x ?Your code was working for me with old RASA versions. But not with rasax anymore.PLease do make the modification..That would be really great

@anubhavpnp https://github.com/GaoQ1/rasa_nlu_gq/blob/master/rasa_nlu_gao/featurizers/bert_vectors_featurizer.py I already added it.

Hi @GaoQ1 ..Couple of questions:

I only need to use entity extraction feature as my intent classification after integrating with bert is working fine. As I see in you repo, https://github.com/GaoQ1/rasa_nlu_gq/blob/master/rasa_nlu_gao/extractors/bilstm_crf_entity_extractor.py is the file that is doing the entity extraction.
Will the above file require whitespace delimited tokens.In that case, can I use the whitespace tokeniser for RASA repo.The reason I am asking this question is bert expects entity tagging in BILOU format and does not accept the format in which we tag entities in RASA training files. Does your entity extractor handle that conversion(RASA format to BERT format)?
Please do help as I need entity extraction with bert very badly. All the git repos that I have explored expect data in the format of BILOU which is very different from which RASA expects. PLease do revert.

Hi,
My purpose is to run an intent classification exercise on a set of user responses by using RASA NLU. I had earlier used pretrained_embeddings_spacy but now want to try out with BERT embeddings given its obvious advantages. After installing the requirements, I used the following configuration as mentioned here
https://github.com/Revmaker/innatis
I encounter the following errors when I try to train rasa nlu using 'rasa train nlu' command

ImportError: cannot import name 'create_argument_parser'
Exception: Failed to find component class for 'innatis.classifiers.BertIntentClassifier'. Unknown component name. Check your configured pipeline and make sure
the mentioned component is not misspelled. If you are creating your own component, make sure it is either listed as part of the component_classes in rasa.nl u.registry.py or is a proper name of a class in a module.

How do i get rid of these errors and exceptions?
Is innatis.classifiers.BertIntentClassifier a valid component that can be used to train a RASA model now?

Hi,
My purpose is to run an intent classification exercise on a set of user responses by using RASA NLU. I had earlier used pretrained_embeddings_spacy but now want to try out with BERT embeddings given its obvious advantages. After installing the requirements, I used the following configuration as mentioned here
https://github.com/Revmaker/innatis
I encounter the following errors when I try to train rasa nlu using 'rasa train nlu' command

ImportError: cannot import name 'create_argument_parser'
Exception: Failed to find component class for 'innatis.classifiers.BertIntentClassifier'. Unknown component name. Check your configured pipeline and make sure
the mentioned component is not misspelled. If you are creating your own component, make sure it is either listed as part of the component_classes in rasa.nl u.registry.py or is a proper name of a class in a module.

How do i get rid of these errors and exceptions?
Is innatis.classifiers.BertIntentClassifier a valid component that can be used to train a RASA model now?

I have used this repo and it works fine
https://github.com/winstarwang/rasa_nlu_bert

I have made a clone of rasa-nlu-0.14.4 and add bert feature. If someone has interest with it, you can go to https://github.com/winstarwang/rasa_nlu_bert

hi @winstarwang ..I notice that you have not handled offsets in the bert_tokenizer because of which the error is coming..Any idea of how to handle that?

I am so sorry, I didn't get back to you in time because of working on others.
Here is my bert_rasa config file content, hope it will be helpful
language: "zh"
pipeline:

name: "tokenizer_bert"
vocab_file: "data/chinese_L-12_H-768_A-12/vocab.txt"
do_lower_case: True
max_seq_length: 64

name: "intent_featurizer_bert"

name: "intent_classifier_tensorflow_bert"
bert config
The config json file corresponding to the pre-trained BERT model.
This specifies the model architecture.
"bert_config_file": data/chinese_L-12_H-768_A-12/bert_config.json
The vocabulary file that the BERT model was trained on.
"vocab_file": data/chinese_L-12_H-768_A-12/vocab.txt
The output directory where the model checkpoints will be written.
"output_dir": data/chinese_L-12_H-768_A-12/output
Initial checkpoint (usually from a pre-trained BERT model).
"init_checkpoint": data/chinese_L-12_H-768_A-12/bert_model.ckpt
Whether to lower case the input text. Should be True for uncased
models and False for cased models.
"do_lower_case": True
training parameters
Total batch size for training.
"train_batch_size": 32
Total batch size for eval.
"eval_batch_size": 8
Total batch size for predict.
"predict_batch_size": 8
The initial learning rate for Adam.
"learning_rate": 0.0005
The maximum total input sequence length after WordPiece tokenization.
Sequences longer than this will be truncated, and sequences shorter
than this will be padded.
"max_seq_length": 64
The num to labels
"num_labels": 990
number of epochs
"epochs": 10
Proportion of training to perform linear learning rate warmup for.
E.g., 0.1 = 10% of training.
"warmup_proportion": 0.1
How often to save the model checkpoint.
"save_checkpoints_steps": 1000
How many steps to make in each estimator call.
"iterations_per_loop": 1000

I have made a clone of rasa-nlu-0.14.4 and add bert feature. If someone has interest with it, you can go to https://github.com/winstarwang/rasa_nlu_bert

Hi @winstarwang ..Your repo was quite useful..Do you have any plans of modifying it so that it is compatible with rasa x ?Your code was working for me with old RASA versions. But not with rasax anymore.PLease do make the modification..That would be really great

I will try to merge the BERT codes to newest RASA version recently.

@winstarwang I really liked your implementation and I implemented it. I also stumbled upon: https://github.com/JulianGerhard21/bert_spacy_rasa which simplifies a lot of the configurations you have to do. You should check it out and see if it's worth implementing your solution or the solution in bert_spacy_rasa.

We added support for BERT and other known language models in our latest release 1.8.0.
In order to use the pre-trained word embeddings from BERT, you need to include the following components in your pipeline:
pipeline:
  - name: HFTransformersNLP
    # Name of the language model to use
    model_name: "bert"
- name: "LanguageModelTokenizer"
- name: "LanguageModelFeaturizer"

You can read more about those components on our docs (https://rasa.com/docs/rasa/nlu/components/).


Bad API even worse than thought: danger_connect_async_without_providing_domain_for_certificate_verification_and_server_name_indication

The choice of whether to send SNI should be separated from the choice of whether to validate the certificate for the (SNI) host name.
In particular, a server might use the SNI to route a connection to a backend, and/or it might fail to do any routing if SNI isn't provided, so SNI is needed in some cases even when certificate validation isn't done.
Also, I imagine some users might want to avoid putting SNI in the client hello but still want to validate the certificate for a particular host name, e.g. if they're trying to hide the hostname from the plaintext part of the TLS connection.
Some users of this API are already doing the wrong thing. For example, this one is disabling SNI when only disabling certificate verification is intended: https://github.com/hyperium/hyper-tls/blob/da69ab8b9897f3b2bce25e5afce02ff94b750985/src/client.rs#L93-L100
Also, IMO, the scope of this crate should be "connect Tokio and rust-native-tls" together. tokio-tls should have a way of accepting a configuration that is configured using the rust-native-tls APIs, and that should be the way configuration of SNI, certificate validation, etc. is done, without any such APIs in tokio-tls itself.
Therefore, I propose we remove this API.
See the related discussion at https://github.com/quininer/tokio-rustls/pull/14#issuecomment-325548177

/cc @carllerche

cc @sfackler, thoughts?

Removing the API from here would require making some changes to native-tls and openssl's APIs but I'm not opposed to splitting these up.

This will be complex on OSX since SNI and hostname verification both use the same method: https://developer.apple.com/documentation/security/1393047-sslsetpeerdomainname.

Oh right, they're also intermingled in SChannel: https://msdn.microsoft.com/en-us/library/windows/desktop/aa375924(v=vs.85).aspx.

See SCH_CRED_NO_SERVERNAME_CHECK and SCH_CRED_SNI_CREDENTIAL. My hypothesis is that they work like this:

Neither flag set - SNI with server name checking when name is provided.
SCH_CRED_NO_SERVERNAME_CHECK = no SNI and no server name checking.
SCH_CRED_NO_SERVERNAME_CHECK | SCH_CRED_SNI_CREDENTIAL = SNI with no server name checking.

However, I've not found any evidence that this hypothesis is true; it's just speculation after 10 minutes of Googling.
Maybe macOS has something similar.
Regardless, I think that all of this should be handled outside tokio-tls and inside rust-native-tls instead.

I'm confused - the method here directly forwards to the equivalent method in native-tls: https://github.com/tokio-rs/tokio-tls/blob/master/src/lib.rs#L193-L203.


[cxx-interop] Initializing a Swift.Array from CxxConvertibleToCollection should not copy the collection

This makes the CxxConvertibleToCollection parameter shared, preventing the unnecessary copy of the C++ value.
rdar://110110376

@swift-ci please smoke test

You can test this by having a std::vector of type who's copy constructor logs how many copies have been made, and then tracking that now we don't have any additional copies of that type stored in the vector?

@hyp thanks for the suggestion!
I added a slightly simpler test that keeps track of how many times the collection value was copied.

@swift-ci please smoke test

@swift-ci please smoke test


Clickhouse connection no working due to ClickHouseLZ4Stream

Description
I did update my dbeaver to latest version (Version <IP_ADDRESS>312122044) and connections to clickhouse are not working.
Could not initialize class ru.yandex.clickhouse.response.ClickHouseLZ4Stream
MySQL connections are working fine.
How can I fix it?
DBeaver Version
Community Version <IP_ADDRESS>312122044
Operating System
Windows 10
Database and driver
Clickhouse
com.clickhouse - 0.3.2-patch11
Steps to reproduce
Just to try to connect to database.
Additional context
No response

Hello @pablolopez2022
What was the previous DBeaver version?
Could you provide the log files? You can find instructions here: https://dbeaver.com/docs/wiki/Log-files/.
Also, looks like you are using the old (legacy) Clickhouse driver version.
Please try to change your driver to non-legacy.

If you want to continue to use the legacy driver version, then go to your Connection Settings -> Driver Settings -> Libraries -> Add Artifact.
and add this string maven:/ru.yandex.clickhouse:clickhouse-jdbc:RELEASE[0.2.6]
and remove the downloaded one.

Ok, I can reproduce this issue.

Same problem with Ubuntu 22.04. It worked well before the update.
DBeaver Version <IP_ADDRESS>312241705
Error message:

Could not initialize class ru.yandex.clickhouse.response.ClickHouseLZ4Stream

Hello,
I'm using 23.3.1 version.
How can I change the driver to non-legacy?
[image: image.png]
On Mon, Dec 25, 2023 at 1:45 AM Anastasiya @.***> wrote:

Hello @pablolopez2022 https://github.com/pablolopez2022
What was the previous DBeaver version?
Could you provide the log files? You can find instructions here:
https://dbeaver.com/docs/wiki/Log-files/.
Also, looks like you are using the old (legacy) Clickhouse driver version.
Please try to change your driver to non-legacy.
2023-12-25.09_35_27-Window.png (view on web)
https://github.com/dbeaver/dbeaver/assets/45152336/b52e2a3a-999c-46d6-a1d5-07cea6ea92c8
—
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/22241#issuecomment-1868827829,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZYSBHOXBFKNJOBG7N2RERLYLEVLBAVCNFSM6AAAAABBAHJEZWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYHAZDOOBSHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello @pablolopez2022
You can use step one from this message: https://github.com/dbeaver/dbeaver/issues/22241#issuecomment-1868827829
And then choose the non-legacy driver.

Or, if you want to keep using the legacy driver, you can add ai as an artifact, but turn on the dependency loading setting:


Project using kits and variants doesn't work in 1.13.40

Brief Issue Summary
Project using kits and variants doesn't work in 1.13.40

Create a CMake project without presets, in which case it should fall back to kits and variants.
Open project in VS Code and notice that the tool bar doesn't switch to kits and variants view, and none of the configure / build related commands work.
Multi-root with kits and variants doesn't work as well, not sure if that's also caused by this though.

CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

This is the project that repro the issue:
azure-sphere-samples/Samples/HelloWorld/HelloWorld_HighLevelApp at main · Azure/azure-sphere-samples (github.com)

We are going to add a rename or delete event listener for CMakeLists.txt . We can add the presets files to the event listener as well (for the cases where "cmake.useCMakePresets" is set to "auto".
Look at onDidRenameFiles events from here : https://github.com/microsoft/vscode-cmake-tools/pull/2946/files

Ah yeah I was using an old version of that sample... So the actual steps:

Remove presets file from the project so it uses kits/variants.
Install Azure Sphere SDK: https://aka.ms/AzureSphereSDKDownload/Windows
Install Azure Sphere extension in VS Code
Open the project

I believe the reason this issue repros is that the Azure Sphere extension calls the following commands on activation, and causes a deadlock:

cmake.setKitByName
cmake.launchTargetPath
cmake.hideLaunchCommand
cmake.hideDebugCommand

We can add the presets files to the event listener as well (for the cases where "cmake.useCMakePresets" is set to "auto".

I remember I wrote this? I'm probably wrong though since that's a long time ago.

By following the steps, I am able to see the kit menu in the status bar (in both situations where Azure Sphere extension is installed or not).
however, after installing your extension, I see this and I am not able to configure correctly:

I used another project with a CMakeLists.txt as below and I am able to configure without setting the kits:
CMAKE_MINIMUM_REQUIRED(VERSION 3.20)
PROJECT(CMakeSample C)

azsphere_configure_tools(TOOLS_REVISION "22.11")
azsphere_configure_api(TARGET_API_SET "15")

@bobbrow The issue repro when I disable the Azure Sphere extension and enable it again, then the status bar starts showing presets options.
A sample settings.json including Azure commands:
{
    "cmake.buildToolArgs": ["-v"],
    "cmake.buildDirectory": "${workspaceRoot}/out/${buildType}-${command:azuresphere.AzureSphereTargetApiSet}",
    "cmake.configureSettings": {
        "CMAKE_TOOLCHAIN_FILE": "${command:azuresphere.AzureSphereSdkDir}/CMakeFiles/AzureSphereToolchain.cmake",
        "AZURE_SPHERE_TARGET_API_SET": "latest-lts"
    },
    "C_Cpp.default.configurationProvider": "ms-vscode.cmake-tools"
}

and a sample CMakeLists.txt :
CMAKE_MINIMUM_REQUIRED(VERSION 3.20)
PROJECT(CMakeSample C)

azsphere_configure_tools(TOOLS_REVISION "22.11")
azsphere_configure_api(TARGET_API_SET "15")

# Create executable
ADD_EXECUTABLE(${PROJECT_NAME} main.c)

Call stack:
expandStringHelper (d:\CMakeTools\src\expand.ts:237)
expandString (d:\CMakeTools\src\expand.ts:113)
<anonymous> (d:\CMakeTools\src\drivers\cmakeDriver.ts:674)
await (Unknown Source:0)
doRefreshExpansions (d:\CMakeTools\src\drivers\cmakeDriver.ts:662)
_refreshExpansions (d:\CMakeTools\src\drivers\cmakeDriver.ts:666)
_baseInit (d:\CMakeTools\src\drivers\cmakeDriver.ts:1833)
processTicksAndRejections (internal/process/task_queues:96)
await (Unknown Source:0)
createDerived (d:\CMakeTools\src\drivers\cmakeDriver.ts:1849)
create (d:\CMakeTools\src\drivers\cmakeFileApiDriver.ts:67)
startNewCMakeDriver (d:\CMakeTools\src\cmakeProject.ts:778)
<anonymous> (d:\CMakeTools\src\cmakeProject.ts:1085)
await (Unknown Source:0)
<anonymous> (d:\CMakeTools\src\strand.ts:41)
Promise.then (Unknown Source:0)
_enqueue (d:\CMakeTools\src\strand.ts:18)
<anonymous> (d:\CMakeTools\src\strand.ts:39)
execute (d:\CMakeTools\src\strand.ts:38)
getCMakeDriverInstance (d:\CMakeTools\src\cmakeProject.ts:1070)
get binaryDir (d:\CMakeTools\src\cmakeProject.ts:2498)
checkBuildDirectories (d:\CMakeTools\src\projectController.ts:223)
createCMakeProjectsForWorkspaceFolder (d:\CMakeTools\src\projectController.ts:215)
await (Unknown Source:0)
addFolder (d:\CMakeTools\src\projectController.ts:272)
loadAllProjects (d:\CMakeTools\src\projectController.ts:199)
init (d:\CMakeTools\src\extension.ts:168)
create (d:\CMakeTools\src\extension.ts:212)
setup (d:\CMakeTools\src\extension.ts:1603)
activate (d:\CMakeTools\src\extension.ts:1793)

Need to check what's changed from 1.12

I think the difference is in projectController.ts, line 215. Can we move checkBuildDirectories to after activation, or don't wait for it?


Update virtual-network-network-interface.md

Following should be clearly stated: "Each NIC can be assigned to only one subnet."

@asudbring
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@asudbring sorry for the late reply. I missed this. I accepted your suggestions. Can you please sign this off?

#please-close


Fail to load native extension when using custom git source and having an existing cache

This issue may look like to #4106 but as stated, I decided to open a new issue, here is the workflow to reproduce the bug. I've reproduced it with version on the 1.13 branch and 1.14 branch

Create a new rails project

rails new bundler-test
cd bundler-test

Add the following gem in your Gemfile:

gem 'ox', git: 'https://github.com/hiptest/ox.git', branch: 'add_tolerant_mode_to_builder'  # Fast XML reader/writer

Run bundler to freeze deps:

bundle install

Run bundler in deployment mode:

bundle install --without development:test --path vendor/bundle --binstubs vendor/bundle/bin -j4 --deployment

Here the .so file is built correctly, as a static library:
└> ls -ld vendor/bundle/ruby/2.3.0/bundler/gems/ox-8211624ae976/ext/ox/ox.so
-rwxr-xr-x 1 leo leo 997064 May  9 15:39 vendor/bundle/ruby/2.3.0/bundler/gems/ox-8211624ae976/ext/ox/ox.so
┌[leo☮soulou7]-(~/Scalingo/SampleProjects/sample-ruby-ox)-[git://master ✗]-
└> ldd vendor/bundle/ruby/2.3.0/bundler/gems/ox-8211624ae976/ext/ox/ox.so
	linux-vdso.so.1 (0x00007ffc8cb4e000)
	libpthread.so.0 => /usr/lib/libpthread.so.0 (0x00007f840811d000)
	libgmp.so.10 => /usr/lib/libgmp.so.10 (0x00007f8407e8a000)
	libdl.so.2 => /usr/lib/libdl.so.2 (0x00007f8407c86000)
	libcrypt.so.1 => /usr/lib/libcrypt.so.1 (0x00007f8407a4e000)
	libm.so.6 => /usr/lib/libm.so.6 (0x00007f840773b000)
	libc.so.6 => /usr/lib/libc.so.6 (0x00007f8407395000)
	/usr/lib64/ld-linux-x86-64.so.2 (0x0000558cbbd93000)

The environment car be loaded correctly:
└> bundle exec rake -P
rake about
...

RE-Run bundler

bundle install --without development:test --path vendor/bundle --binstubs vendor/bundle/bin -j4 --deployment

The file is now different as well as its dependencies
└> ls -ld vendor/bundle/ruby/2.3.0/bundler/gems/ox-8211624ae976/ext/ox/ox.so
-rwxr-xr-x 1 leo leo 1003058 May  9 15:41 vendor/bundle/ruby/2.3.0/bundler/gems/ox-8211624ae976/ext/ox/ox.so
┌[leo☮soulou7]-(~/Scalingo/SampleProjects/sample-ruby-ox)-[git://master ✗]-
└> ldd vendor/bundle/ruby/2.3.0/bundler/gems/ox-8211624ae976/ext/ox/ox.so
	linux-vdso.so.1 (0x00007ffc31be3000)
	libruby.so.2.3 => not found
	libpthread.so.0 => /usr/lib/libpthread.so.0 (0x00007ff54c54c000)
	libc.so.6 => /usr/lib/libc.so.6 (0x00007ff54c1a8000)
	/usr/lib64/ld-linux-x86-64.so.2 (0x000055cf19eb6000)

And obviously the environment does not load anymore:
└> bundle exec rake -P
rake aborted!
LoadError: libruby.so.2.3: cannot open shared object file: No such file or directory - sample-ruby-ox/vendor/bundle/ruby/2.3.0/bundler/gems/ox-8211624ae976/ext/ox/ox.so
sample-ruby-ox/vendor/bundle/ruby/2.3.0/bundler/gems/ox-8211624ae976/lib/ox.rb:79:in `require'
sample-ruby-ox/vendor/bundle/ruby/2.3.0/bundler/gems/ox-8211624ae976/lib/ox.rb:79:in `<top (required)>'
sample-ruby-ox/config/application.rb:7:in `<top (required)>'
sample-ruby-ox/Rakefile:4:in `require_relative'

So the bundle install operation is not repeatable here, it looks like a bug from bundler to me.

Thanks for reporting this issue. I can't recreate this issue using the steps but can you please attach the output of bundle env so we can get some information about your environment.

Here it is:
└> bundle env
Environment

    Bundler   1.13.6
    Rubygems  2.5.2
    Ruby      2.3.3p222 (2016-11-21 revision 56859) [x86_64-linux]
    Git       2.12.2

Bundler settings

    frozen
      Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "1"
    path
      Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "vendor/bundle"
    bin
      Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "vendor/bundle/bin"
    jobs
      Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "4"
    without
      Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "development:test"
    disable_shared_gems
      Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "true"

Gemfile

    source 'https://rubygems.org'

    git_source(:github) do |repo_name|
      repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?("/")
      "https://github.com/#{repo_name}.git"
    end

    # Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
    gem 'rails', '~> 5.0.2'
    # Use sqlite3 as the database for Active Record
    gem 'sqlite3'
    # Use Puma as the app server
    gem 'puma', '~> 3.0'
    # Use SCSS for stylesheets
    gem 'sass-rails', '~> 5.0'
    # Use Uglifier as compressor for JavaScript assets
    gem 'uglifier', '>= 1.3.0'
    # Use CoffeeScript for .coffee assets and views
    gem 'coffee-rails', '~> 4.2'
    # See https://github.com/rails/execjs#readme for more supported runtimes
    # gem 'therubyracer', platforms: :ruby

    # Use jquery as the JavaScript library
    gem 'jquery-rails'
    # Turbolinks makes navigating your web application faster. Read more: https://github.com/turbolinks/turbolinks
    gem 'turbolinks', '~> 5'
    # Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
    gem 'jbuilder', '~> 2.5'
    # Use Redis adapter to run Action Cable in production
    # gem 'redis', '~> 3.0'
    # Use ActiveModel has_secure_password
    # gem 'bcrypt', '~> 3.1.7'

    # Use Capistrano for deployment
    # gem 'capistrano-rails', group: :development

    group :development, :test do
      # Call 'byebug' anywhere in the code to stop execution and get a debugger console
      gem 'byebug', platform: :mri
    end

    group :development do
      # Access an IRB console on exception pages or by using <%= console %> anywhere in the code.
      gem 'web-console', '>= 3.3.0'
      gem 'listen', '~> 3.0.5'
      # Spring speeds up development by keeping your application running in the background.
LoadError: libruby.so.2.3: cannot open shared object file: No such file or directory - sample-ruby-ox/vendor/bundle/ruby/2.4.0/bundler/gems/ox-8211624ae976/ext/ox/ox.so

└> bundle env

Environment
Bundler   1.14.6
Rubygems  2.6.11
Ruby      2.4.1p111 (2017-03-22 revision 58053) [x86_64-linux]
Git       2.12.2
Platform  x86_64-linux

Bundler settings
bin
  Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "vendor/bundle/bin"
jobs
  Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "4"
frozen
  Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "1"
path
  Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "vendor/bundle"
without
  Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "development:test"
disable_shared_gems
  Set for your local app (/run/mount/crypt/Scalingo/SampleProjects/sample-ruby-ox/.bundle/config): "true"

Gemfile
Gemfile
source 'https://rubygems.org'

git_source(:github) do |repo_name|
  repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?("/")
  "https://github.com/#{repo_name}.git"
end

# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
gem 'rails', '~> 5.0.2'
# Use sqlite3 as the database for Active Record
gem 'sqlite3'
# Use Puma as the app server
gem 'puma', '~> 3.0'
# Use SCSS for stylesheets
gem 'sass-rails', '~> 5.0'
# Use Uglifier as compressor for JavaScript assets
gem 'uglifier', '>= 1.3.0'
# Use CoffeeScript for .coffee assets and views
gem 'coffee-rails', '~> 4.2'
# See https://github.com/rails/execjs#readme for more supported runtimes
# gem 'therubyracer', platforms: :ruby

# Use jquery as the JavaScript library
gem 'jquery-rails'
# Turbolinks makes navigating your web application faster. Read more: https://github.com/turbolinks/turbolinks
gem 'turbolinks', '~> 5'
# Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
gem 'jbuilder', '~> 2.5'
# Use Redis adapter to run Action Cable in production
# gem 'redis', '~> 3.0'
# Use ActiveModel has_secure_password
# gem 'bcrypt', '~> 3.1.7'

# Use Capistrano for deployment
# gem 'capistrano-rails', group: :development

group :development, :test do
  # Call 'byebug' anywhere in the code to stop execution and get a debugger console
  gem 'byebug', platform: :mri
end

group :development do
  # Access an IRB console on exception pages or by using <%= console %> anywhere in the code.
  gem 'web-console', '>= 3.3.0'
  gem 'listen', '~> 3.0.5'
  # Spring speeds up development by keeping your application running in the background.  It seems that with RVM it works correctly but it's not the case with rbenv
Actually I did manage to reproduce this behavior in two different environments:

With rbenv on my local workstation
In Scalingo deployment environment (docker container based on ubuntu trusty, ruby deployed with heroku buildpack https://github.com/heroku/heroku-buildpack-ruby

I can't reproduce this on OS X, vendor/bundle/ruby/2.4.0/bundler/gems/ox-8211624ae976/ext/ox/ox.bundle is present still after installing a second time

Thanks, i can reproduce this problem on linux/ubuntu.

This does not reproduce for me if i use the master branch instead.

Hi @colby-swandale, you mean the master branch of the gem itself, or the master branch of bundler?

the master branch of the gem (sorry that wasn't clear)

I confirm that on master of github.com/hiptest/ox, I can't reproduce the problem.

Does this mean the problem is related to the ox gem build script
What could lead to such behavior?

Between master and the branch I'm giving in the Gemfile, there is only one commit:
https://github.com/hiptest/ox/commit/8211624ae9764c5fc147e1450850fde46f339ab1

How could such a commit change the behavior?

The branch is 1 commit ahead of master but it's also 9 commits behind master, so there might be a commit that addresses some build issue. Sorry but, Im not very familiar with native extensions in ruby (something i need to read up on more to really give you an informative answer). @segiddins do you have any thoughts about this?

I was actually speaking about being 1 commit ahead of master in the fork. I've actually tested hiptest/ox#master and hiptest/ox/#add_tolerant_mode_to_builder, it worked correctly with the former, but failed with the add_tolerant_mode_to_builder branch.

https://github.com/hiptest/ox/commit/8211624ae9764c5fc147e1450850fde46f339ab1 added ext/ox/ox.so to git, which probably was unintentional

My bad :-( Thanks for figuring it out.

Oh! just seen it.. It seems so obvious now. Thanks @segiddins for find this information.  Closing the issue, thanks again for your help

No worries! I only saw it after you pointed out the issue had to be in that commit (and it also explains why I didn't see anything on macOS)


What is Update in Display pane for?

Since updates happen immediately, why do we need an Update button?

If you move the map (pan or zoom), it rebins the binned data.

Ah I see. I'd suggest renaming to "Update Bins" and disabling the button if you are not in a heatmap view mode. If there's a use case outside updating bins we could go back on that but seems most clear for now.

Refactored in PR #99.

"Update bins" seems to work how I'd expect now - thanks.


Enhance Japanese language detection

Pull Request
Context
Some Languages are harder to detect than others, this miss-detection leads to bad tokenization making some words or even documents completely unsearchable. Japanese is the main Language affected and can be detected as Chinese which has a completely different way of tokenization.
A first iteration has been implemented for v1.1.0 but is an insufficient enhancement to make Japanese work. This first implementation was detecting the Language during the indexing to avoid bad detections during the search.
Unfortunately, some documents (shorter ones) can be wrongly detected as Chinese running bad tokenization for these documents and making possible the detection of Chinese during the search because it has been detected during the indexing.
For instance, a Japanese document {"id": 1, "name": "東京スカパラダイスオーケストラ"} is detected as Japanese during indexing, during the search the query 東京 will be detected as Japanese because only Japanese documents have been detected during indexing despite the fact that v1.0.2 would detect it as Chinese.
However if in the dataset there is at least one document containing a field with only Kanjis like:
A document with only 1 field containing only Kanjis:
{
 "id":4,
 "name": "東京特許許可局"
}

A document with 1 field containing only Kanjis and 1 field containing several Japanese characters:
{
 "id":105,
 "name": "東京特許許可局",
 "desc": "日経平均株価は26日 に約8カ月ぶりに2万4000円の心理的な節目を上回った。株高を支える材料のひとつは、自民党総裁選で3選を決めた安倍晋三首相の経済政策への期待だ。恩恵が見込まれるとされる人材サービスや建設株の一角が買われている。ただ思惑が先行して資金が集まっている面 は否めない。実際に政策効果を取り込む企業はどこか、なお未知数だ。"
}

Then, in both cases, the field name will be detected as Chinese during indexing allowing the search to detect Chinese in queries. Therefore,  the query 東京 will be detected as Chinese and only the two last documents will be retrieved by Meilisearch.
Technical Approach
The current PR partially fixes these issues by:

Adding a check over potential miss-detections and rerunning the extraction of the document forcing the tokenization over the main Languages detected in it.
Adding a filtering threshold during the search over Languages that have been marginally detected in documents

Limits
This PR introduces 2 arbitrary thresholds:

during the indexing, a Language is considered miss-detected if the number of detected tokens of this Language is under 10% of the tokens detected in the same Script (Japanese and Chinese are 2 different Languages sharing the "same" script "CJK").
during the search, a Language is considered marginal if less than 5% of documents are detected as this Language.

This PR only partially fixes these issues:

✅ the query 東京 now find Japanese documents if less than 5% of documents are detected as Chinese.
✅ the document with the id 105 containing the Japanese field desc but the miss-detected field name is now completely detected and tokenized as Japanese and is found with the query 東京.
❌ the document with the id 4 no longer breaks the search Language detection but continues to be detected as a Chinese document and can't be found during the search.

Related issue
Fixes #3565

Hey @dureuill,
I'd like your POV on this PR, thank you if you have the time to look at it

Hello @ManyTheFish, thanks for thinking of me to review your PR!
Before I dive into the code, a question: did we consider the option of having some documents left as "language-ambiguous" and tokenized as many time as necessary in each of the candidate languages? Similarly, for search queries that could match several languages, tokenize them in all of these languages and return results for all the resulting queries?
Such a change could cause structural change and so be a large undertaking. Still, if it has been discussed somewhere, I'd love to be pointed to that discussion.
Meanwhile, the change proposed here appears like a nice heuristic to prevent the misdetection of languages using frequency.
I'm not 100% sure of what the scope of this sentence is:

rerunning the extraction of the document forcing the tokenization over the main Languages detected

can you maybe rephrase a bit?

Hello @dureuill, I added a section in Technical Approach detailing the indexing process and giving an example at the end. Let me know if you need more! 😄

Thank you Many it is indeed clearer with the added explanation!

bors merge


Send stop signal specified in image config.

Fixes https://github.com/kubernetes-incubator/cri-containerd/issues/61.
Stop container with the stop signal specified in image config if not empty.
I built a docker image with STOPSIGNAL SIGHUP to verify the change, here is the cri-containerd log output:
I0620 21:46:21.320098   20386 container_stop.go:94] Stop container "26d64fe32ccef5663de3725279398e19224ecc7a8a91f45b77ecf1524b249015" with signal hangup

Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

CRI Validation test result (skipped host port test):
Summarizing 5 Failures:

[Fail] [k8s.io] Security Context runtime should support container with security context [It] runtime should support RunAsUserName [security context]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/container.go:326

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support portforward [Conformance]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:231

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support attach [Conformance]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:183

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support exec [Conformance]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:127

[Fail] [k8s.io] Security Context runtime should support container with security context [It] runtime should support RunAsUser [security context]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/container.go:326

Ran 35 of 36 Specs in 28.683 seconds
FAIL! -- 30 Passed | 5 Failed | 0 Pending | 1 Skipped --- FAIL: TestE2ECRI (28.68s)
FAIL

Ginkgo ran 1 suite in 28.752698801s
Test Suite Failed
exit status 1


change owners

Update with people that are actually working on the project: we avoid spamming on other teammates mail and get relevant people on the review.

/verify-owners


Fix credentials not being returned along with the request ID

Client expects credentials to be returned as a promise, but it can't match the response to the request due to response ID not being set:
if (packet.rid !== undefined) { // this is always false
        const request = this.requests.get(packet.rid)
        if (request != null) {
          this.requests.delete(packet.rid)
          if (packet.type === 'error') {
            request.reject(new SignalingError('server-error', packet.message))
          } else {
            request.resolve(packet)
          }
        }
      }

This causes TURN to be completely broken since no credentials are available.
Tested locally by hardcoding some creds:
func (c *CredentialsClient) GetCredentials(ctx context.Context) (*Credentials, error) {
	return &Credentials{
		URL:        "someURL",
		Username:   "bsusername",
		Credential: "bspassword",
		Lifetime:   123,
	}, nil
}

Happy to make adjustments as I have no experience in Go :)

Thanks for your contribution! LGTM. Waiting for @erikdubbelboer to check it out.

@koenbollen thank you for your suggestions, applied them both :)

Thanks @remvst! Nice find.


⚠ Bofrost.at has degraded performance

In 8932097, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1348 ms

Resolved: Bofrost.at performance has improved in 92ac513 after 13 minutes.


Updated notifier/notifier.scpt to use /usr/bin/tail, in case users have coreutils installed and GNU tail in their PATH

Fixes #37
GNU tail doesn't support the -r flag, so that's why it was failing for me and running the "on error" section:
try
	set tailLog to do shell script "tail -r ~/Library/Logs/com.github.domt4.homebrew-autoupdate/*.out | grep -E -m1 'taps|date'"
on error
	set tailLog to "Logs not found; cannot show summary!"
end try

Can be fixed by using /usr/bin/tail -r, which is the default tail that comes with macOS.
Before:

After:
cp notifier/notifier.scpt /usr/local/Homebrew/Library/Taps/homebrew/homebrew-autoupdate/notifier/brew-autoupdate.app/Contents/Resources/Scripts/main.scpt
launchctl start com.github.domt4.homebrew-autoupdate

I also just noticed there's a bug with the log parsing. grep -E -m1 'taps|date'" is matching the or list them with brew outdated. line in the logs:
assimp
aom
ansible
adwaita-icon-theme
==> Outdated Formulae
Updated 1 tap (homebrew/cask).
Sat Oct  8 13:25:11 NZDT 2022
or list them with brew outdated.
You can upgrade them with brew upgrade
You have 178 outdated formulae and 13 outdated casks installed.

virtualbox
rescuetime
qbittorrent
postico
playonmac
...

Also confirming this fixes the specific scenario of GNU coreutils installed (tested locally), great work @ndbroadbent. Can the PR be expedited?

Alternatively, if this is not a desirable change perhaps using tac rather than tail -r could be considered. The same reversed output of the log is created by tac natively whether using native MacOS binaries or GNU Coreutils.

@troyfisher Thanks for the tip about tac, I didn't know about that! I will update to use that


Not generating slug of the category while creating new category.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
v2.0.0-BETA-1
Issue Description
While creating a new category and mentioning the category name still it is not generating a slug.
Preconditions

Admin should be logged in.

Steps to reproduce

Go to 'Admin panel'
Click on the 'Create category button' inside the catalog/category
Entered category name
Scroll down to 'Slug'
See the error

Expected Result
It should be generated automatically while entering the category name.
Actual Result
Not generating we have to enter manually. Here
Screenshots
No response
Additional context
No response

Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


Cross compiler for linux-s390x

Referred PR - https://github.com/conda-forge/ctng-compilers-feedstock/pull/38
Opted binutils version 2.33.1 for the below fix
https://patches-gcc.linaro.org/patch/9830/

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@isuruf @beckermr : Please let me know if you have any comments/ suggestions . Thank you !

@isuruf @beckermr : Please let me know if you have any comments/ suggestions . Thank you !

We'll have to force a downgrade to conda-build.
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=263215&view=logs&j=d1321064-f6c3-56b7-0172-6d994d01c836&t=39022612-2533-571e-7b41-94814e7d38bb&l=29440

We'll have to force a downgrade to conda-build.
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=263215&view=logs&j=d1321064-f6c3-56b7-0172-6d994d01c836&t=39022612-2533-571e-7b41-94814e7d38bb&l=29440

@conda-forge-admin restart ci

@conda-forge-admin restart ci

Thanks

Thanks


(#188) add information about approver user to user model

https://github.com/howtohireme/give-me-poc/issues/188
Description
Add approver_id column to users.
Also add relation between user and approved user.
When we call Approve operation, add current_user as approver to applicant.
And in user management show action display who has approved user.
Testing steps
Explain how to test your PR manually

gup
rails db:migrate
Click button to activate user in User management
View information about the approving user in the approval line (action show User controller)

Features PR url
Checklist
Make sure that all steps a checked before merge

[ ] RSpec tests are passing on CI
[ ] Cucumber features are passing localy
[ ] bin/cop -a does not return any warnings
[ ] Tested manually

Screenshots
Provide screenshots of implemented functionality

Don't forget to check checkboxes in PR description.
Soon we'll have a bot, that does not allow to merge PR without checked check-list.


Not compatible with terraform 0.12

closed in favour of https://github.com/cloudposse/terraform-aws-iam-account-settings/pull/8


feat: use npm to get/run bun

Instead of using platform-specific methods of fetching bun, swap to npm. This requires adding it to devDependencies.
I've also moved the actual bun invocations to npm scripts, with a pre-script for bun install.

I'm planning to tweak this a little further tomorrow before merging.


Bad nominal value for state variable in Asynchronous induction machine

Reported by johan_ylikiiskila on 2 Apr 2015 09:35 UTC
In the MSL example Modelica.Electrical.Machines.Examples.AsynchronousInductionMachines.AIMC_DOL the variable aimc.i_0_s is chosen as state variable in multiple (3) tools.
In the above mentioned example, this state has a nominal value of 1.0, which I assume is the default value, while the state is of the order of magnitude 1e-6 or smaller. This causes seemingly random (at least tool-specific) fluctuations which makes it impossible to make a good comparison of state trajectories between tools.
By somehow modifying the nominal value to the smaller 1e-6 (in either the example model or component model) this can be avoided.
NOTE: the same problem is present in Modelica.Magnetic.FundamentalWave.Examples.BasicMachines.AIMS_Start and the state varaible aimsE.i_0_s.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1693

Comment by ahaumer on 7 Apr 2015 17:26 UTC
A model should cover all possbile use cases.
There are a few use cases where the zero sequence current is of importance, and in these cases the nominal value isn't that small.
In many other cases the zero sequence current vanishes, but for consistency reasons it has to be included in the set of equations. In these cases the nominal value should be set to a small number.
Without burden the user with the choice of a meaningful nominal value for the zero sequence current, I see no chance to set this nominal value in a consistent way for all possible use cases.
If the states for your comparison are choosen manually, just omit the zero sequence current.
To be honest, I have no idea how to solve your problem in a meaningful way.
Another solution for your example is to change the nominal value of the zero sequence current with a modifier, or to use an alias variable with a small nominal value which is set equal to i_0_s.

Comment by johan_ylikiiskila on 4 Sep 2015 08:14 UTC
I agree that changing the nominal value in the component model is a really bad idea, but as long as the specific examples are not used in a case where the zero sequence current is of importance it should be possible to add modifiers in the Examples (and by that saying that the zero sequence current is of no importance in that particular experiment)?
Another note, the same behaviour have also been found in the example models:

Electrical.Machines.Examples.AsynchronousInductionMachines.AIMC_withLosses
Electrical.Machines.Examples.AsynchronousInductionMachines.AIMC_Initialize

Comment by otter on 13 Dec 2015 18:00 UTC
I tried in Dymola for Examples.AsynchronousInductionMachines.AIMC_DOL
nominal=1     -> aimc.i_0_s in the order of 1e-9
nominal=1e-9  -> aimc.i_0_s in the order of 1e-13
nominal=1e-13 -> aimc.i_0_s in the order of 1e-17, simulation is extremly slow

You are right, it seems the value of this variable is about zero and the numerical solution is small but random around zero.
Since it seem to not help to provide a better nominal value, I do not know what to do.

Comment by hansolsson on 13 Dec 2015 18:23 UTC
To me the problem is that the comparison routine should be improved, and the model is fine and should not be changed.
With a relative tolerance and a nominal value we allow an error (|x|+|xnominal|)*tolerance as error in the solution.
To me it is clear that we should use the same tolerance for the result comparison as well, so if the fluctuations are a fraction of the nominal value I don't see that there is a problem (unless there is more to this). I assume the comparison routine often avoids this problem by taking the maximum for |x| over the entire trajectory - and is normally saved by |x| being large somewhere.

Comment by ahaumer on 18 Dec 2015 12:24 UTC
The only solution we agreed upon the web-meeting 2015-12-14:
Please remove the zero-sequence currents from the reference results.

Comment by leo.gall on 26 Jan 2016 08:33 UTC
Replying to [comment:5 ahaumer]:

The only solution we agreed upon the web-meeting 2015-12-14:
Please remove the zero-sequence currents from the reference results.

Implemented in r9004 (updated reference results, based on MSL 3.2.1+build.4).
Please let me know if I should have missed any of those signals.


Feature request: Force-saving to yml file through command

A reload command along with this as well, if possible. I want to edit the holograms through the yml file

These features already exist; use the /fancyholograms command


Add state grades to historical data

Is there a way to access states' letter grades through time? We would like to compare time-series data across states in our analysis.
Thanks for your work collating all of this!

See https://github.com/COVID19Tracking/covid-tracking-api/issues/11.
For now I had to cook up https://github.com/uva-bi-sdad/COVID_Tracking_Clean so you can get that data at data/daily. I wrote the up the clean up process there and the CRON scheduler. I am not affiliated with the project. However, that should get you what you want in the meanwhile while the feature request is processed.


[Bugfix] Bilinear resize bug fix from PR #2777

@FrozenGene Please review.

Thanks @Laurawly generally looks good to me and could solve the issues mentioned. However, could we add one unit testing of resize bilinear to compare MXNet forward? I think it will be more nice.

@FrozenGene Does the test look good to you or more tests are needed?

@FrozenGene Mxnet gives me no "scale_height/width" parameter error locally. I think maybe it depends on the version of Mxnet.

@Laurawly I use MXNet 1.4 and also get the same error. Seems that it is MXNet's doc issue. I think we test height width is enough currently.

@Laurawly I use MXNet 1.4 and also get the same error. Seems that it is MXNet's doc issue. I think we test height width is enough currently. However, could we construct another testing? i.e. currently , we are data -> resize bilinear , data is variable. We could add data -> add -> resize bilinear. i.e. resize bilinear accepts one op's output. Then we could cover all conditions of input.

I confirmed with mxnet developers and confirmed that it's a bug. Mxnet 1.5 solves this issue. If we could upgrade ci to use mxnet 1.5 (pre-release) problem will be solved.

@FrozenGene @tqchen Is it possible to update the CI to use mxnet 1.5? Or I remove the test for now?

I personally suggest remove the test of scale_height / scale_width. Because current stable version of MXNet doesn’t have it. Also want to hear @tqchen’s comment.

Let us remove the test

@FrozenGene Test removed.

Please kick the CI again.

Thanks everyone. this is now merged


Sync eng/common directory with azure-sdk-tools for PR 2290

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/2290  See eng/common workflow

Error not related to the eng/common updates. Skipping.

/check-enforcer override


avoid collisions

when using kfactory==0.20.5

avoiding colisions worked
however in latest kfactory==0.20.6 it doesn't

@sebastian-goeldi

i can confirm this issue is coming from autotaper
When i use auto_taper=False i get:

this feature will be in implemented in kfactory

Yeah, but it's going to take time :D.
What you can do in the meantime is add
box = gf.kdb.Box()
for taper in auto_tapers:
    box += auto_taper.ports["o1"].trans.disp.to_p()
    box += auto_taper.ports["o2"].trans.disp.to_p()
# call to route_bundle function with bboces = other_boxes + [box]


StyleBox "hover_pressed" missing from Button but used by CheckBox

Godot version
v4.0.rc2.mono.official [d2699dc7a]
System information
Windows 10
Issue description
Button does not have a "hover_pressed" StyleBox (which I've wanted for a while), but CheckBox does, and will try to use the parent Button's hover_pressed state, making the Button become invisible when hovering over a pressed CheckBox.
Hover:

Hover while Pressed:

Steps to reproduce

Create a CheckBox
Toggle it on
Hover your mouse over it

Minimal reproduction project
N/A

I don't understand the problem you describe, and the steps are not enough to demonstrate any incorrect behavior. Please provide an actual MRP with a scene and a theme that demonstrate the issue clearly.

The issue is that when you hover over a CheckBox while it is toggled, the underlying Button becomes invisible because Button does not have a "hover_pressed" StyleBox state.

There is no underlying Button. CheckBox is a Button.

Either way, the StyleBox for it vanishes when you hover it while it is pressed.

Can you provide an MRP? Nothing vanishes with the default theme, or your repro steps are incomplete.

Added. Toggle the CheckBox on, then hovering at all causes the underlying Button to become invisible. The custom theme only modifies Button, not CheckBox.

Okay, I see what you mean. Theme properties are inherited alongside the class hierarchy. Since CheckBox is a Button, it takes custom styles for the Button from your theme over its default look from the default theme. However, since your theme does not define any styles for the hover_pressed state, and indeed normal Buttons don't have it, the default theme is used. You need to add a hover_pressed definition for the CheckBox, or for even the Button in your theme.


unit test fails in i386 platform

See https://bugs.debian.org/860207, it has the following error during build in i386 platform.

...
   dh_auto_test -a
	make -j4 test
make[1]: Entering directory '/«PKGBUILDDIR»'
cc -g -O2 -fdebug-prefix-map=/«PKGBUILDDIR»=. -fstack-protector-strong -Wformat -Werror=format-security -Wall -Wextra -g -std=c99 -O3 -pedantic  -Isrc -o test/fzytest test/fzytest.c src/match.o src/choices.o -lpthread
./test/fzytest
test "test_exact_scores" failed
   assert(match("a", "*a") == SCORE_GAP_LEADING) was false
   at test/fzytest.c:81

14 tests, 80 assertions, 1 failures
Makefile:28: recipe for target 'check' failed
make[1]: *** [check] Error 1

Hi @czchen. Thanks so much for including fzy to debian. Please let me know if there's anything I can do to aid in package maintenance.
This issue should be fixed in Version 0.9, which I've just released. The issue was simply that I should not have been doing == to test floats. I now test with a tolerance, which should fix the architectures it was failing on (verified in a debian i386 qemu). The test suite has also had a major overhaul in this release and now gives better errors.


Fix publishXXXPublicationToMavenLocal is not invoked

Use ProjectEvaluationListener rather than BuildListener to handle bintray extension block.
It would be too late for BuildListener.projectsEvaluate to add task dependencies.

Thanks you for this @hkhc!

When a version including this fix will be published?

Note that this means gradle-bintray-plugin now races with any other afterEvaluate blocks that attempt to provide it configuration.

For me this manifested as #170, i.e. the plugin just silently starts failing, and it's not at all obvious why.


is running ok download?

How can I tell if any download is in progress?  except for queueEnded.  Is there a function like OkDownloader.with().IsRunning() ?

@lingochamp


New in Chrome 83 post

What's changed, or what was fixed?

adds New in Chrome 83 post

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2020/05/nic83.md#L8 - Common typo found: 'chrome' Should it be 'Chrome'?
src/content/en/updates/images/2020/05/forms-focus-ring.gif - Try to keep files below 500K, was 601K


artifacts/parameters between workflows

Hi, all:
There are two workflow templates ta and tb,
argo template create ta.yaml
argo template create tb.yaml

and I want to use workflowTemplateRef to trigger them
(We are using Argo Workflow v 2.9.5).
But I want to use workflow ta's output(parameters or artifacts) in workflow tb, which means I want send ta's output to tb, Is there any way to do that?
Thanks a lot!!!

you can pass parameters and artifacts from one template to another template using arguments
Here the example.  Please refer workflow-template-inner-steps
https://github.com/argoproj/argo/blob/master/examples/workflow-template/templates.yaml

@sarabala1979 thanks for your reply,
I still have a question, for the following WorkflowTemplate ta, how can I send the parameter hello-param to another template(e.g. tb)
---
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: workflow-template-whalesay-outputs
spec:
  entrypoint: whalesay-outputs
  templates:
  - name: whalesay-outputs
    container:
      image: docker/whalesay:latest
      command: [sh, -c]
      args: ["sleep 1; echo -n hello world > /tmp/hello_world.txt"]
    outputs:
      parameters:
      - name: hello-param
        valueFrom:
          default: "Foobar"   # Default value to use if retrieving valueFrom fails. If not provided workflow will fail instead
          path: /tmp/hello_world.txt

just you can refer `{{steps..output.parameters.}}'
Here the examble
---
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: workflow-template-whalesay-template
spec:
  entrypoint: whalesay-template
  templates:
    - name: whalesay-template
      inputs:
        parameters:
          - name: message
      container:
        image: docker/whalesay
        command: [cowsay]
        args: ["{{inputs.parameters.message}}"]
---
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: workflow-template-whalesay-outputs
spec:
  entrypoint: step
  templates:
  - name: step
    steps:
    - - name: generate
        template: whalesay-outputs
    - - name: pass-to-wft
        templateRef:
          name: workflow-template-whalesay-template
          template: whalesay-template
        arguments:
          parameters:
            - name: message
              value: "{{steps.generate.outputs.parameters.hello-param}}"
  - name: whalesay-outputs
    container:
      image: docker/whalesay:latest
      command: [sh, -c]
      args: ["sleep 1; echo -n hello world > /tmp/hello_world.txt"]
    outputs:
      parameters:
      - name: hello-param
        valueFrom:
          default: "Foobar"   # Default value to use if retrieving valueFrom fails. If not provided workflow will fail instead
          path: /tmp/hello_world.txt

@sarabala1979  thanks, I get the point why my workflow failed.
pass ta's parameters to tb need to create a new WorkflowTemplate tc,
then follow your answer, it works


(fix) Copy/Paste shortcut in layout.py

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Proposal on adding Copy/Paste shortcut to the layout.py
Tests performed by the developer:
Tips for QA testing:

Oh well


Add option 'icons' to change default icon classes

Second attempt :)

In your earlier PR you mentioned that this is hard to override in CSS - can you tell me a bit more about that?

OK, I understand the change.

I think that everything must be on their own place. Those icons styles already have theirs.

I understand that everything has its place.
In this case, the styles are defined in CSS so why is it a hack if the override/redefinition is also in CSS?

Any thoughts on when this will be available. This is a major pain point for anyone working with bootstrap, angular, but using a different font, such as FontAwesome. I ran into this issue today.

I'm was waiting for a response from OP.
I'm struggling to understand how putting 12 lines of formatted css and then ignoring it is a "is a major pain point".
I don't use font awesome so I assume I am missing something, but simply telling me it is "hacky" or "painful" does not help me understand WHY it is a pain point and, as s result, I have no confidence that this change is the best change to make.
Can you help me understand?

Forcing overrides of glyphicon to try and swap them to any other icon font isn't an obvious solution outright supported by the code - that's what makes it a hack.
It being ugly is attributed to needing to know the icon's font codes (which not all of them publish for some reason) and your configuration is not longer in one place, making code less maintainable (especially if you're not styling anything else from the element).
It's useful because you can put custom classes on the buttons of specific pickers in addition to swapping them out. A very simple/silly example: maybe you already have a class you use everywhere for changing font. You want to apply this to the right arrow, but not the left, and only on one specific picker.
Easiest and most maintainable way is to simply apply that class to the right arrow in the config like this PR. The alternative solution would be to wrap the specific picker in something you can easily identify via id or whatnot to hit with a CSS selector. Then, you need to create a new CSS entry that is an exact duplicate of your already existing class. If you change the class, it no longer changes both.
Simply giving the option to swap out the classes in the directive makes everything easier, more powerful, and more maintainable.

Thanks for the info. I sort of understand what you are getting at, but it is not very compelling yet. Simply stating something is easier, more powerful, and more maintainable does not make it true. Please support your claims with evidence.
Taking your argument to the logical conclusion, there should be a configuration option to set every class on every element in the html - and also for the hover color, and the highlight color, etc, etc - that is adding configuration complexity for very little value.
There is a cost to testing, maintaining, documenting, and evolving code - it is not free. Every single line of code and logic branch of functionality should (must?) be tested - the value delivered to the end user must exceed the cost of adding and forever maintaining the feature.
I'm not exactly opposed to this type of change, its just that the arguments thus far are not supported by actual evidence of it being a better solution - it sounds to me more like opinion than fact (hopefully it's not just me being difficult),
One could argue the the definition of the arrows is in CSS, to put it any place else is confusing. If I want change the way something looks, I change the CSS. If I want to change the way something behaves when I change the configuration/code.
Finally, I will point out that there are no similar configuration options - all configuration options control the behavior of the directive, not the appearance. This is by design. Appearance settings, in general, do not belong in JavaScript.
Why didn't you allow the user to completely override the HTML (one configuration option with high value) rather then pick and choose little parts of the HTML to change via configuration?

Why didn't you allow the user to completely override the HTML (one configuration option with high value) rather then pick and choose little parts of the HTML to change via configuration?

That I actually think is really cool, if the entire thing was a template somewhere people could edit. That's usually much more complex than only one or two common swap-outs, but could make for some really cool customization.
I can see your choice to want to keep visual changes completely out of the directive itself, though. Likewise, maybe I'm just used to seeing this elsewhere, and justifying it in my mind.


Question: only continuous variables (no category)

Is it possible to use SAINT for the tabular data, which contains only continuous variables, without categorical?
We need to pass to SAINT model two parameters: x_categ and x_cont
Do I need to pass some torch.empy tensor as x_categ?
What to pass as "categories" parameter to the SAINT model? Empty tuple?

Hi, it is possible to train the model only on continuous data. Please check how the function (link) is outputting for openml datasets- 42178 and 42724 (they have no categorical features either), you can use the same format for your dataset.


Image_picker camera temporary null size file

When I use image_picher to read images from the camera and cancel the request without taking the photo, the plugin correctly returns a null value, but a temporary file with zero size is created in my folder anyway, which I cannot delete because the returned value is null.

Hi @kikoba
can you please provide your flutter doctor -v and flutter run --verbose?
Also, to better address the issue, would be helpful if you could post a minimal code sample to reproduce the problem
Is this happening on iOS or Android?
Thank you

[√] Flutter (Channel stable, v1.12.13+hotfix.5, on Microsoft Windows [Versione 10.0.18362.778], locale it-IT)
• Flutter version 1.12.13+hotfix.5 at D:\SW\Flutter_SDK
• Framework revision 27321ebbad (5 months ago), 2019-12-10 18:15:01 -0800
• Engine revision 2994f7e1e6
• Dart version 2.7.0
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at D:\Users\Ketty\AppData\Local\Android\Sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.3
• ANDROID_HOME = D:\Users\Ketty\AppData\Local\Android\Sdk
• Java binary at: D:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
• All Android licenses accepted.
[√] Android Studio (version 3.6)
• Android Studio at D:\Program Files\Android\Android Studio
• Flutter plugin version 45.1.1
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
[√] Connected device (1 available)
• Android SDK built for x86 • emulator-5554 • android-x86 • Android 5.0.2 (API 21) (emulator)
• No issues found!

import 'dart:io';
import 'package:flutter/material.dart';
import 'package:image_picker/image_picker.dart';
void main() => runApp(MyApp());
class MyApp extends StatelessWidget {
// This widget is the root of your application.
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'Flutter Demo',
theme: ThemeData(
    primarySwatch: Colors.blue,
  ),
  home: MyHomePage(title: 'Flutter Demo Home Page'),
);

}
}
class MyHomePage extends StatefulWidget {
MyHomePage({Key key, this.title, }) : super(key: key);
final String title;
@override
_MyHomePageState createState() => _MyHomePageState();
}
class _MyHomePageState extends State {
Future imageFile;
//Open
pickImageFrom(ImageSource source) {
setState(() {
imageFile = ImagePicker.pickImage(source: source);
});
}
Widget showImage() {
return FutureBuilder(
future: imageFile,
builder: (BuildContext context, AsyncSnapshot snapshot) {
if (snapshot.connectionState == ConnectionState.done &&
snapshot.data != null) {
return Image.file(
snapshot.data,
width: 300,
height: 300,
);
} else if (snapshot.error != null) {
return const Text(
'Error Picking Image',
textAlign: TextAlign.center,
);
} else {
return const Text(
'No Image Selected',
textAlign: TextAlign.center,
);
}
},
);
}
@override
Widget build(BuildContext context) {
return Scaffold(
appBar: AppBar(
title: Text(widget.title),
),
body: Center(
child: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: [
showImage(),
RaisedButton(
child: Text("Select Image from Camera"),
onPressed: () {
pickImageFrom(ImageSource.camera);
},
),
],
),
),
);
}
}

import 'dart:io';
import 'package:flutter/material.dart';
import 'package:image_picker/image_picker.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title, }) : super(key: key);

   final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {

Future<File> imageFile;

 //Open
  pickImageFrom(ImageSource source) {
    setState(() {
      imageFile = ImagePicker.pickImage(source: source);
    });
  }

  Widget showImage() {
    return FutureBuilder<File>(
      future: imageFile,
      builder: (BuildContext context, AsyncSnapshot<File> snapshot) {
        if (snapshot.connectionState == ConnectionState.done &&
            snapshot.data != null) {
          return Image.file(
            snapshot.data,
            width: 300,
            height: 300,
          );
        } else if (snapshot.error != null) {
          return const Text(
            'Error Picking Image',
            textAlign: TextAlign.center,
          );
        } else {
          return const Text(
            'No Image Selected',
            textAlign: TextAlign.center,
          );
        }
      },
    );
  }
   @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            showImage(),
            RaisedButton(
              child: Text("Select Image from Camera"),
              onPressed: () {
                pickImageFrom(ImageSource.camera);
              },
            ),
          ],
        ),
      ),
    );
  }
}

This file is created upon entry into the camera but is not deleted, if you enter several times many files with different names are created.

This is issue isn't reproducible using image_picker 0.6.6+1 on latest dev channel

flutter doctor -v
[✓] Flutter (Channel dev, 1.19.0-1.0.pre, on Linux, locale en_US.UTF-8)
    • Flutter version 1.19.0-1.0.pre at /home/tesser/Code/flutter_dev
    • Framework revision 456d80b9dd (3 days ago), 2020-05-11 11:45:03 -0400
    • Engine revision d96f962ca2
    • Dart version 2.9.0 (build 2.9.0-7.0.dev 092ed38a87)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /home/tesser/Code/SDK
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /home/tesser/Code/SDK
    • Java binary at: /home/tesser/Code/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Android Studio (version 3.6)
    • Android Studio at /home/tesser/Code/android-studio
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.45.0)
    • VS Code at /usr/share/code
    • Flutter extension version 3.10.1

[✓] Connected device (1 available)
    • SM M305F • 32003c30dc19668f • android-arm64 • Android 10 (API 29)

• No issues found!

I noticed you 5 months old flutter setup and probably older version of image picker
Please upgrade to latest stable and use the latest Image_picker and try reproduce the issue
Thank you

I have this problem with:

image_picker 0.6.6+1

is true I use old flutter version the new for me non stable. But I now try to update.

I update flutter, the emulator API 21 same problem, with API 28 is all ok
[√] Flutter (Channel stable, v1.17.1, on Microsoft Windows [Versione 10.0.18362.836], locale it-IT)
    • Flutter version 1.17.1 at D:\SW\Flutter_SDK
    • Framework revision f7a6a7906b (2 days ago), 2020-05-12 18:39:00 -0700
    • Engine revision 6bc433c6b6
    • Dart version 2.8.2

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at D:\Users\Ketty\AppData\Local\Android\Sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = D:\Users\Ketty\AppData\Local\Android\Sdk
    • Java binary at: D:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    • All Android licenses accepted.

[√] Android Studio (version 3.6)
    • Android Studio at D:\Program Files\Android\Android Studio
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] Connected device (1 available)
    • AOSP on IA Emulator • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!```

with new flutter any app test with image_picker when load from camera the emulator (API 21) crashing con error:
Emulator: Process finished with exit code -1073741819 (0xC0000005)


Add Plesk product

Hi @sibprogrammer, thank you for this issue.
Can you provide more information about Plesk ?

Full and short name of product : Let us know the long, full name and, if it has one, the short name (for example, PowerShell and pwsh).
Does this product have LTS versions? What are the intervals between each LTS version? : For example, a current release is a release that occurs between LTS releases. Current releases can contain critical fixes.
Does this product have a documented release / EOL policy? : For example, Python have this one : https://devguide.python.org/versions/.
What is the website for the product and for its version information? : For example: https://github.com/PowerShell/PowerShell/tags
Additional context: Add any other context or screenshots about the product which you want us to add endoflife.date

Sorry @sibprogrammer, did not see it was a PR (thought it was an issue 😅). I removed my comment.


redisDriver adds the setKeyExpire method to set the expiration time of the key. After the key expires, it will no longer be available. The unit is in seconds.

storage.setKeyExpire(key, seconds)
Set the expiration time of the key. After the key expires, it will no longer be available. The unit is in seconds.
Note: Only for redisDriver.
await storage.setKeyExpire("foo:bar", 1); // 1s

Test file path: test/drivers/redis.test.ts.
Before testing, you need to modify the ip, password, and port in the file.

@rbndelrio @manniL @benjamincanac @Atinux

Thanks for the PR dear @rdzhaoxin (please keep doing it again!). Supporting ttl is a good idea however it is driver dependent.
Please see my comment at https://github.com/unjs/unstorage/issues/13#issuecomment-1448906670