[iCubGenova11] Do not start legs by default

As per the title.
Given that we are using a new stand for the robot with variable height and that not all the legs configurations are now collision free, given the geometry of the stand, we would like to disable the legs by default.

cc @AntonioConsilvio


Portable Mode of Vs Code still have code folder in %appdata%

Vscode version : 1.58.2 (portable
extensions: No extensions were downloaded
The question title is already told the problem. This is the screen recording:
https://user-images.githubusercontent.com/85733811/126873017-4e503ae3-4eec-4458-96d2-608cddde9b09.mp4
Because of some reasons, portable mode of vscode insider doesn't have the problem ...

/duplicate of #93828


Stub for Morphism.agda

Monad morphisms, in two flavours.

As this is clearly still a work in progress, I've marked it as 'draft'.

This module is ready for review, but suggestions on extensions are welcome by me or @iwilare
Cheers!


eas secret:create failed

Build/Submit details page URL
No response
Summary
While running running eas secret:create I'm getting PluginError
Managed or bare?
Managed
Environment
System:
OS: macOS 12.0.1
Shell: 5.8 - /bin/zsh
Binaries:
Node: 16.0.0 - /usr/local/bin/node
Yarn: 1.22.10 - /usr/local/bin/yarn
npm: 7.10.0 - /usr/local/bin/npm
Utilities:
Git: 2.30.1 - /usr/bin/git
npmPackages:
expo: ^42.0.0 => 42.0.5
expo-updates: ~0.8.2 => 0.8.5
react: 16.13.1 => 16.13.1
react-dom: 16.13.1 => 16.13.1
react-native: https://github.com/expo/react-native/archive/sdk-42.0.0.tar.gz => 0.63.2
react-native-web: ~0.13.12 => 0.13.18
npmGlobalPackages:
expo-cli: 5.0.3
Project workflow: managed
Error output
PluginError: Plugin is an unexpected type: object
Code: INVALID_PLUGIN_TYPE
Reproducible demo or steps to reproduce from a blank project
run eas secret:create

What version of eas-cli are you using. Judging by the eas diagnostics output format, you might be on an older version. I am unable to reproduce this on<EMAIL_ADDRESS>

Closing this for inactivity and unability to reproduce in the most recent CLI.

Had the same issue while building after adding expo-build-properties.
"plugins": [       "@notifee/react-native",       "@react-native-firebase/app",         "expo-build-properties",         {           "ios": {             "useFrameworks": "static"           }         }     ]
Found out that you need to add [ ] for multiple plugins:
"plugins": [       "@notifee/react-native",       "@react-native-firebase/app",       [         "expo-build-properties",         {           "ios": {             "useFrameworks": "static"           }         }       ]     ]


Rasa NLU evaluation on a predefined test set

Rasa version: 2.3.0
Python version: 3.6.8
Operating system (windows, osx, ...): Darwin-20.3.0-x86_64-i386-64bit
Issue:
I want to evaluate a trained Rasa nlu model on a predefined test set. To do so, I executed the following command:
rasa test nlu -m models/nlu-20210323-152823 -u train_test_split/test_data.yml
But the command does not generate the intent_report file.
Error (including full traceback):
2021-03-24 12:39:15 INFO     rasa.model  - Loading model models/nlu-20210323-152823.tar.gz...
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/rasa/utils/train_utils.py:437: UserWarning: constrain_similarities is set to `False`. It is recommended to set it to `True` when using cross-entropy loss. It will be set to `True` by default, Rasa Open Source 3.0.0 onwards.
  category=UserWarning,
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/rasa/utils/train_utils.py:410: UserWarning: model_confidence is set to `softmax`. It is recommended to set it to `cosine`. It will be set to `cosine` by default, Rasa Open Source 3.0.0 onwards.
  category=UserWarning,
2021-03-24 12:39:23 INFO     rasa.nlu.test  - Running model for predictions:
100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 [00:00<00:00, 53.57it/s]
2021-03-24 12:39:23 INFO     rasa.nlu.test  - Entity evaluation results:
2021-03-24 12:39:23 INFO     rasa.nlu.test  - Evaluation for entity extractor: DIETClassifier
2021-03-24 12:39:23 WARNING  rasa.test  - No labels to evaluate. Skip evaluation.
2021-03-24 12:39:23 INFO     rasa.nlu.test  - Classification report saved to results/DIETClassifier_report.json.
2021-03-24 12:39:23 INFO     rasa.nlu.test  - Every entity was predicted correctly by the model.
2021-03-24 12:39:23 INFO     rasa.utils.plotting  - Confusion matrix, without normalization:
[[5]]
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/rasa/utils/plotting.py:193: UserWarning: Attempting to set identical left == right == 0 results in singular transformations; automatically expanding.
  axes[0].set(yticks=yticks, xlim=(0, max_xlims[0]), ylim=(min_ylim, max_ylim))
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/rasa/utils/plotting.py:193: UserWarning: Attempting to set identical bottom == top == 0.0 results in singular transformations; automatically expanding.
  axes[0].set(yticks=yticks, xlim=(0, max_xlims[0]), ylim=(min_ylim, max_ylim))
2021-03-24 12:39:24 INFO     rasa.nlu.test  - Evaluation for entity extractor: CRFEntityExtractor
2021-03-24 12:39:24 WARNING  rasa.test  - No labels to evaluate. Skip evaluation.
2021-03-24 12:39:24 INFO     rasa.nlu.test  - Classification report saved to results/CRFEntityExtractor_report.json.
2021-03-24 12:39:24 INFO     rasa.nlu.test  - Incorrect entity predictions saved to results/CRFEntityExtractor_errors.json.
2021-03-24 12:39:25 INFO     rasa.utils.plotting  - Confusion matrix, without normalization:
[[0 0]
 [1 4]]
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/rasa/utils/plotting.py:193: UserWarning: Attempting to set identical left == right == 0 results in singular transformations; automatically expanding.
  axes[0].set(yticks=yticks, xlim=(0, max_xlims[0]), ylim=(min_ylim, max_ylim))

Content of configuration file (config.yml) (if relevant):
language: en

pipeline:
# # No configuration for the NLU pipeline was provided. The following default pipeline was used to train your model.
# # If you'd like to customize it, uncomment and adjust the pipeline.
# # See https://rasa.com/docs/rasa/tuning-your-model for more information.
#   - name: WhitespaceTokenizer
#   - name: RegexFeaturizer
#   - name: LexicalSyntacticFeaturizer
#   - name: CountVectorsFeaturizer
#   - name: CountVectorsFeaturizer
#     analyzer: char_wb
#     min_ngram: 1
#     max_ngram: 4
#   - name: DIETClassifier
#     epochs: 100
#   - name: EntitySynonymMapper
#   - name: ResponseSelector
#     epochs: 100
#   - name: FallbackClassifier
#     threshold: 0.3
#     ambiguity_threshold: 0.1

- name: WhitespaceTokenizer
- name: RegexFeaturizer
- name: LexicalSyntacticFeaturizer
- name: CountVectorsFeaturizer
- name: CountVectorsFeaturizer
  analyzer: char_wb
  min_ngram: 1
  max_ngram: 4
- name: CRFEntityExtractor
  features:
  - - pattern
  max_iterations: 50
  L1_c: 0.1
  L2_c: 0.1
- name: DIETClassifier
  epochs: 100
- name: EntitySynonymMapper
- name: DucklingEntityExtractor
  url: http://localhost:8000
  dimensions:
  - time
- name: ResponseSelector
  epochs: 100
- name: FallbackClassifier
  threshold: 0.5
  ambiguity_threshold: 0.1

# Configuration for Rasa Core.
# https://rasa.com/docs/rasa/core/policies/
policies:
# # No configuration for policies was provided. The following default policies were used to train your model.
# # If you'd like to customize them, uncomment and adjust the policies.
# # See https://rasa.com/docs/rasa/policies for more information.
#   - name: MemoizationPolicy
#   - name: TEDPolicy
#     max_history: 5
#     epochs: 100
#   - name: RulePolicy

- name: MemoizationPolicy
- name: TEDPolicy
  max_history: 5
  epochs: 100
- name: RulePolicy
  core_fallback_threshold: 0.5
  core_fallback_action_name: action_default_fallback

I noticed that there is a warning in the traceback:
2021-03-24 12:39:24 WARNING  rasa.test  - No labels to evaluate. Skip evaluation.
I am not sure how the test file should look like? There is nothing mentioned in the Rasa documentation about that. Did I miss something?

Thanks for raising this issue, @m-vdb will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

Just to check, could you share information from your train_test_split/test_data.yml file? The confusion matrix suggests there's only 5 examples in it.
If your primary interest is evaluating your NLU model (emphasis: just the NLU model, ignoring the TED model) then you might prefer to use the cross-validation command.
rasa test nlu --config config.yml \
          --cross-validation --runs 1 --folds 2 \
          --out gridresults

Does this help?

I have a similar issue as the reporter. I tried the suggested
rasa test nlu --config config.yml \
          --cross-validation --runs 1 --folds 2 \
          --out gridresults

I have few intent definition consisting only 1 example. I also have the nlu data split into different files.
According to -vv debug, there is intent classification through correct models, but without the results and metrics. Any idea into what should I check? Where can be the issue? thx

Can the issue be, that I have empty NLG data (we have moved NLG to a different service and don't have them duplicated inside the original domain.yml)

I have a similar issue as the reporter. I tried the suggested
rasa test nlu --config config.yml \
          --cross-validation --runs 1 --folds 2 \
          --out gridresults

I have few intent definition consisting only 1 example. I also have the nlu data split into different files.
According to -vv debug, there is intent classification through correct models, but without the results and metrics. Any idea into what should I check? Where can be the issue? thx

Nevermind, my custom intent classifier class inherited from Component instead of IntentClassifier.

Just to check, could you share information from your train_test_split/test_data.yml file? The confusion matrix suggests there's only 5 examples in it.
If your primary interest is evaluating your NLU model (emphasis: just the NLU model, ignoring the TED model) then you might prefer to use the cross-validation command.
rasa test nlu --config config.yml \
          --cross-validation --runs 1 --folds 2 \
          --out gridresults

Does this help?

Sorry for the late reply. I was able to fix the issue by adding more training examples.


fix importing with go mod

The repositories github.com/mattn/go-colorable and
github.com/mattn/go-isatty have moved to using go modules; using gopkg.in
to import them breaks:
https://github.com/golang/go/issues/30636
Let's use the real URLs with hashes.
Signed-off-by: Tycho Andersen<EMAIL_ADDRESS>

@cheggaaa can you review this PR?

Currently, progress bar does not work with latest go-runewidth (https://github.com/cheggaaa/pb/issues/142). I hope to fix it soon...

I need this change as well.
If you add the support for Modules, you can pin down the dependency to a specific go-runewidth version:
go get github.com/mattn/go-runewidth@version

I gave up hope with this PR - it has been too long. I forked my own copy of this repo and made the changes myself.
https://github.com/TheBrazenGeek/progbar

Sorry for the late reaction. I could not decide how to do it correctly in order to save both versions of the library and not break backward compatibility.
Please look at my PR #150


Autocomplete not working when typescript.useCodeSnippetsOnMethodSuggest is enabled

When typescript.useCodeSnippetsOnMethodSuggest is enabled and you click Enter the function suggestion you won't get autocomplete option for the parameters.
Example:
When you type console.l and then press Enter the editor will autocomplete to console.log(). The cursor will be inside of the parantheses but you won't get intellisense working when writing now.

What version of VS Code are you using? Do you see this in the latest VS Code insiders build?

Sorry @mjbvz about that.
I am using the version 1.27.1
commit: 5944e81f3c46a3938.....74cfdc

Please test this in the latest VS Code insiders build with all extensions disabled. In insiders, the setting has been renamed to javascript.suggest.insertParametersForFunctionCalls typescript.suggest.insertParametersForFunctionCalls

Thanks. I believe what you want is to see the signature help widget when you hit enter. It's a bug that we don't do this currently


If we enter as a input dataset the taskname of a job, woudl be cool if it fetched the output dataset

As long as it could tell!!

It is done ( along time ago) for the invoke guy.


Printing in the same frame the window closes gives no output

Tested versions

Tested in: 4.2.2 rc2 and 4.2.2 rc3

System information
Godot v4.2.2.rc3 - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3050 6GB Laptop GPU (NVIDIA; <IP_ADDRESS>01) - 13th Gen Intel(R) Core(TM) i5-13450HX (16 Threads)
Issue description
If I try to print anything on the same frame the game window closes in the editor, nothing gets printed. For example, if I run a scene with the following script attached, I would expect to see "I'm exiting the tree" in the output console after pressing ESCAPE:
extends Node

func _process(_delta: float) -> void:
	if Input.is_action_just_pressed("ui_cancel"):
		print("I'm exiting the tree")
		get_tree().quit()

However, upon testing, the output console is empty. If I add an await get_tree().create_timer(0.1).timeout statement after the print line, however, it prints to the output as expected:
extends Node

func _process(_delta: float) -> void:
	if Input.is_action_just_pressed("ui_cancel"):
		print("I'm exiting the tree")
		await get_tree().create_timer(0.1).timeout
		get_tree().quit()

It also works if I add a breakpoint in place of the await get_tree().create_timer(0.1).timeout, so it seems that Godot just doesn't like printing stuff in the same frame the window closes.
I have also tested using the _exit_tree function and the behaviour is the same. The following code does not print anything when I press ESCAPE:
extends Node

func _process(_delta: float) -> void:
	if Input.is_action_just_pressed("ui_cancel"):
		get_tree().quit()

func _exit_tree() -> void:
	print("I'm exiting the tree")

But if I add a breakpoint, it works correctly:
extends Node

func _process(_delta: float) -> void:
	if Input.is_action_just_pressed("ui_cancel"):
		get_tree().quit()

func _exit_tree() -> void:
	print("I'm exiting the tree")
	breakpoint

Steps to reproduce

Create a new scene with a root node of type Node;
Attach the following script to that node:

extends Node

func _process(_delta: float) -> void:
	if Input.is_action_just_pressed("ui_cancel"):
		print("I'm exiting the tree")
		get_tree().quit()

Press play, then press the ESCAPE key;
Check the output console and verify that nothing got printed;
Change the script to the following:

extends Node

func _process(_delta: float) -> void:
	if Input.is_action_just_pressed("ui_cancel"):
		print("I'm exiting the tree")
		await get_tree().create_timer(0.1).timeout
		get_tree().quit()

Press play, then press the ESCAPE key;
Check the output console and verify that "I'm exiting the tree" was printed correctly.

Minimal reproduction project (MRP)
print-on-close-test.zip

This might be related to the debugger-process disconnecting from the game-process before it receives the print message.

This has been reported elsewhere I'm pretty sure, can't find rn

The same happens when the project finishes because of an error. The output reaches the standard output, it shows in the terminal, but not the debug panel.

Yesterday I was having this issue with Godot 4.2.1, now I can't reproduce it with 4.2.2-rc3.
Maybe related: #87493

@berarma weird, I had this issue specifically in 4.2.2 rc3. Did you try the project I provided?

@berarma weird, I had this issue specifically in 4.2.2 rc3. Did you try the project I provided?

No, it was with another project. Maybe something I changed from yesterday to now. I should have tested again with 4.2.1 but I couldn't do more tests today.

I also have this problem, in Godot v4.3.stable.official [77dcf97d8]


Update wording on website homepage

Summary
Resolves #9067

Thanks! I actually liked the version you proposed in the issue with the em dash and fewer filling words in the rest of the sentence:

Tests fail—when they do, Jest provides rich context why. Here are some examples:

Would you mind if we use that?

Codecov Report

Merging #9073 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9073   +/-   ##
=======================================
  Coverage   64.71%   64.71%
=======================================
  Files         277      277
  Lines       11702    11702
  Branches     2872     2872
=======================================
  Hits         7573     7573
  Misses       3514     3514
  Partials      615      615

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 800ba64...e3278da. Read the comment docs.


Fix like? method for server profile

Fixing like? for cases where ov returns integers or strings where those are unexpected.
Description
Fixes like? method for server profile, which failed in cases such as cited in issue #219
Issues Resolved
#219
Check List

[ ] New functionality includes testing.

[X] All tests pass ($ rake test).

[ ] New functionality has been documented in the README if applicable.

[ ] New functionality has been thoroughly documented in the examples (please include helpful comments).

[X] Changes are documented in the CHANGELOG.

@fgbulsoni , I think it would be nice to have a unit test for this case. What do you think?

@fgbulsoni , I think it would be nice to have a unit test for this case. What do you think?

@aalexmonteiro Presto! :sparkles:

Sorry, I didn't get it submitted before the merge, but I think we need to document this behavior clearly in the docs for both the like? and recursive_like? methods, because while I like that it's smart enough to figure out the intent behind the data (ignoring the types), this method is meant to compare equality between 2 data sets. 2 != '2' from a computer's perspective, but who's to say that they are or aren't alike? They're sort-of alike, but they are also different. In this specific case it's useful to ignore those types, but that might not always be the case. It's a bit of a catch-22 I guess, and should be clearly documented.


Set the Artifact.properties using the CI/CD building process (TeamCity)

Hi team,
I Have not found a solution for my request.

I have a task - I need to save some properties about git_commit / build_number _CI_CD/ HW_version in Artifactory.
I found artifact.properties** file in .conan cache folder which is possible to edit only locally. And after editing artifact.properties file I could upload packages manually into Artifactory repository. That is ok,  and working fine.
But how to be with CI/CD building, when packages uploaded automatically ? How to edit this file in this case (like in server side)? Maybe, there are any other possibilities to save the props in packages revision?

One more question, how to read this properties? I mean not only in Artifactory GUI, but in command line. It is needed to check firstly which package to install based on these props values.
For example to see all recipe revisions or package ID, or package revisions I could use conan search command. But what about properties? Are there any commands to represent it in terminal?

Hi @smileNika
This is the conan-extensions repo, which is exclusively for Conan 2.0.
The mechanism of the artifact.properties file is a Conan 1.X, that has been removed in Conan 2.0. In any case, this file shouldn't be edited, it is there mostly for the old way to generate Artifactory BuildInfo (that no longer works)
For Conan 2.0 the command is the properties command in https://github.com/conan-io/conan-extensions/tree/main/extensions/commands/art

@memsharded  Hi. Thank you for your reply. In my task I have to use conan 1.X (1.53 1.54). Then what is the correct way to store all BuildInfo?
How the properties info usually save in Artifactory in 1.X versions of conan? Because as I understand from your words - artifact.properties file is just an output log file which represent what we could manually fulfill in Artifatory. But in a CI continuous building process it is not good idea open Artifactory and try to paste each important peace of information by myself. There should be an automation way continuously store updated building info for each package and represent this info before installation.
One more question: I have seen in 2.0 documentation it is bad way to store multiple package revision for one recipe revision. But in a 1.X - there are  lot of discussion regarding this topic. How to idetify PREV and RREV before installation. So why did you advice not to store few PREV in one RREV? and are there any mechanism how to prevent it towards good way of package storing?

okay. Understod about JFrog - then I need to learn more about its API or CLI aproach.
The question still - is how to check this props when I want to install specific Recipe revision to decide which one revision I need to install.
about package revisions ----> are there any possibilities to handle it (1RREV : 1PREV) in conan 1.53 version? Could I restrict explicitly creation multiple packages for 1 recipe revision?

The question still - is how to check this props when I want to install specific Recipe revision to decide which one revision I need to install.

Conan doesn't check properties to decide to install one revision or the other. The revision used by Conan is resolved with:

By default, if not specified, it will be resolved to latest revision
Users can explicitly require a give requires = "pkg/version#revision", and that will be used
Using lockfiles a specific revision can also be locked

For Conan, the Artifactory properties does not exist in the dependency resolution, so they cannot be used to decide about a specific revision. And most likely they shouldn't, this would be aligned with other package managers for other languages. The version (and revision, which is part of the version) resolution always follow rules based on it.
What you can do in the JFrog platform is use the infrastructure, for example you can tag with certain properties some specific revisions. Then you can use those properties to run a copy of the artifacts with some specific value of that property to another repository. Finally you can point Conan (or other language package manager) to that repository you copied the packages, and Conan will resolve those, because those are the only ones in that repository.

about package revisions ----> are there any possibilities to handle it (1RREV : 1PREV) in conan 1.53 version? Could I restrict explicitly creation multiple packages for 1 recipe revision?

Yes, as long as you don't run conan create indiscriminately (you can check with info the existence of binaries, or you can use --build=missing), and you don't force the build of packages with -build, --build=* or anything like that, having more than 1 prev can be avoided (Recall, the relation is (1rrev-1package_id: 1prev), you cannot drop the package_id from there, it is critical for the model to work). You might need to change the default package_id_modes, as they are too relaxed in Conan 1.X by default, to guarantee correct re-build from source when it is necessary.


sort nlu/story files before training

Proposed changes:

model would always retrain if there's e.g. multiple files in a data/nlu directory - now that should be fixed.
fixes #5405

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

hm ok tests are failing bc of types - is it fine that it's a list now or should i turn it into a set again?


Retry last request

Is there a built-in way to retry the last request?
For instance:
The user was busy sending messages to and fro, and at some time the library responded with an error.
Rather than the user typing the prompt out again, some sort of retry button could ask the library to try again in hopes of a successful response.

@ChinHairSaintClair you can using idea
CompleteText? keepRequest;
  void _translateEngToThai() async {
    final request = CompleteText(
        prompt: translateEngToThai(word: _txtWord.text.toString()),
        maxTokens: 200,
        model: Model.textDavinci3);

    final response = await openAI.onCompletion(request: request);
    setState(() {
      keepRequest = request;
      message = response?.choices.last.text;
    });
  }

  void onRetry() async{
    final response = await openAI.onCompletion(request: keepRequest!);
  }


Custom listenerMiddleware Problem

Hi All,
I'm working on an additional security layer feature for hubot involving two factor authentication with Verisign VIP and am just battling to get my head around how hubot and middleware work.
My middleware code seems to work as expected when chatting directly to hubot (private message) but it's not working correctly when it receives messages from users in a room.
Here is my middleware code:
module.exports = (robot) ->

  robot.listenerMiddleware (context, next, done) ->

    allowed_commands = [ "help", "ping" ]
    # Grab just the command issued, eg "ping" and not "hubot ping"
    command = context.response.message.text.slice(10)

    # Allow if any of our excluded commands are issued otherwise deny
    if command in allowed_commands
      next(done)
    else
      context.response.send "That command is not allowed"
      done()

My aim with the code above is:

if "hubot help" or "hubot ping" is issued, run it as normal...
if anything else is issued to hubot, deny and stop

Testing from a private chat message with hubot:
dave> hubot blah
hubot> That command is not allowed

dave> hubot ping
hubot> pong!

That worked just the way I expected it too: allow certain commands and deny the rest.
 I'm not sure why it's not allowing that command or why it's sending a double response back. I'm using Hubot 2.18.0 and the hubot-hipchat adapter 2.12.0
Unfortunately my coffeescript and coding skills are not great. If anyone could shed some light on this problem or possibly point me in the right direction I'd really appreciate it!
Thanks!
Dave

Managed to solve this one. There were 2 issues.
When I issued something like this to hubot:
dave> hubot blah

In a room, context.response.message.text is:
hubot blah

While in a private message to hubot, context.response.message.text is:
Hubot:  blah

So to grab just the command issued to hubot, I had to check if it was issued from a room first and if so then apply a different slice..
    # Strip the first characters so we're left with just the command issued...
    if context.response.message.user.room?
      # hubot blah
      command = context.response.message.text.slice(6)
    else
      # Hubot:  blah
      command = context.response.message.text.slice(8)

Now it works the same for a room or a private message.
The next issue was where it was returning "That command is not allowed" twice in a room. This was solved by dropping the last done().
For anyone interested, here's the full middleware code I'm using to intercept messages and first check if a user has verified with 2 factor authentication before the command runs. There are any pieces which work together here but this is just the middleware I'm using:
  # Catch all attempted commands and first check that the user is verified before allowing them
  # to run any commands. If not verified, tell them how it's done...

  robot.listenerMiddleware (context, next, done) ->

    name = context.response.message.user.name
    user = robot.brain.userForName(name)

    # Exclude some commands as all users need to be able to use verification funtions
    excluded_commands = [ "status", "users", "help", "ping" ]

    # Verify needs some regex magic...
    regex_verify = /// ^                # begin of line
      verify                            # the word "verify"
      .                                 # followed by a single space
      [0-9.]{6}                         # then exactly 6 digits, can also use [0-9][0-9][0-9][0-9][0-9]
      ///i                              # end

    # Strip the first characters so we're left with just the command issued...
    if context.response.message.user.room?
      # hubot blah
      command = context.response.message.text.slice(6)
    else
      # Hubot:  blah
      command = context.response.message.text.slice(8)

    # Allow if any core commands are issued
    if command in excluded_commands or command.match regex_verify
      next (done)
    else
      # Allow if user is currently verified
      if user.vip_status == "verified"
        next (done)
      else
        # Deny if user is not verified
        if user.vip_status == "never" or user.vip_status == "failed" or user.vip_status == "expired"
          context.response.send "Bot access denied. You need to verify first using two-factor authentication before I can trust you."
          context.response.send "To verify, type: #{robot.name} verify <your 6 digit OTP number>"

Thanks!
Dave

@michaelansel I would really love to see how you guys are handling this with 2 factor authentication! :)

Didn't get a chance to respond right away, but a few comments:

Use robot.respondPattern(/(.*)/).match(context.response.message.text) (or something pretty close to that), not sure it is official API, but I promise it is better than using slice.
You should be calling done when denying access. If you are seeing double responses, figure out why rather than breaking the API (and potentially any other middleware). Adding in receiveMiddleware is one way to get a lot of debugging information (hooks into message processing before listener matching occurs, so you can see if there are double messages coming from the adapter or if you just have multiple listeners matching, which triggers multiple executions of the middleware stack).
Take a look at these docs if you haven't already: https://hubot.github.com/docs/patterns/#restricting-access-to-commands
The 2FA code I wrote is kinda gross (hence no open source yet), but I ended up separating out validation completely like it looks like you have. Currently unwritten is token expiration. I would recommend doing something like robot.auth.hasValidSecondFactor(user) and shoving all the logic in there. Building an API for that is somewhere on my backlog, but maybe just hacking in whatever (like what you have) is fine. I need more hours in my life... 😓

Overall, I think this is a great use of middleware. Good luck!

Thanks for sharing this @justdaver!
(Also, thanks for posting how you solved your problems, since that's always helpful!)

Thanks @cycomachead no problem, much appreciated! When I've completed this project I'll upload all the bits and pieces that work together with it, hopefully someone out there might find it useful :)

Thank you for taking the time to respond to my post and include some great tips @michaelansel :) Really appreciate that. I'm busy ironing out some of the bugs and reworking a few things as you suggested, once done I'll post my solution and see if anyone finds it useful.
Please let us know when you code up a hubot module to help us all find more hours in our lives :laughing:


SW-3717 Create ad-hoc planting sites from admin UI

Testing the geolocation features of the mobile app on real devices requires
visiting a planting site. For development testing by people working remotely,
it's too time-consuming to construct a real set of shapefiles for each person's
location, so people need to be able to create test planting sites near themselves.
Add a section to the organization page in the admin UI to allow creating a
planting site by drawing the boundary on a map. Sites created this way are very
simple (one zone and one subzone) but should be sufficient for test purposes.

Current dependencies on/for this PR:

main

PR #1195   👈

This comment was auto-generated by Graphite.

Updated to expose the GeoJSON (the textarea gets populated automatically when you close the polygon on the map):

@sgrimm merged this pull request with Graphite.


[tests] Uncomment inject-free-vars test.

This is just uncommenting a test that was disabled ~4 years ago. Running the test locally seems to work, so I imagine that this was fixed and never revisited.
I found this by looking at the oldest open issue - https://github.com/webpack/webpack/issues/171

Yeah thanks


WMS ID 11345: Import, Analyze, and Share Your Data + update to existing common lab, data-prep

Add here your WMS ID
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

@NagwangGyamtso Looks like this is a new workshop. Please submit the filled-out Self QA document once the workshop development is finished to<EMAIL_ADDRESS>mailbox for review and update the workshop status to Self QA Complete in WMS. If everything looks good we will approve the PR.


Bail on first error

Would it be possible to return the error bag on the first failure as opposed to running all the rules.

Bail when the first rule in a ruleset for a field fails
example:

rules := govalidator.MapData{
		"email": []string{"required"},
		"password": []string{"required"},
		"some_field": []string{"bail", "required", "in:something,somethingelse"},
	}

Bail when the first rule for the entire validation fails.

opts := govalidator.Options{
		Request:         r        // request object
		Rules:           rules,    // rules map
		Messages:        messages, // custom message map (Optional)
		RequiredDefault: true,     // all the field to be pass the rules
                 BailFirstError: true //returns the error bag with only one item
	}

up


Prepare for v0.0.5 and drop v0.0.1

Dropping v0.0.1 allows the upgrade function to be simplified, as well.

Oh, and I copied v0.0.4 as the base for v0.0.5, but git decided I must have edited v0.0.1 instead. :upside_down_face:


wip

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test integ-pilot-k8s-presubmit-tests-master

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

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

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

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

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

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/e2e-bookInfoTests-trustdomain.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test e2e-bookInfoTests-trustdomain-master

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

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

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/e2e-bookInfoTests-trustdomain.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test e2e-bookInfoTests-trustdomain-master

prow/e2e-bookInfoTests-v1alpha3.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

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

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

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test integ-pilot-k8s-presubmit-tests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/e2e-bookInfoTests-trustdomain.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test e2e-bookInfoTests-trustdomain-master

prow/e2e-bookInfoTests-v1alpha3.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test integ-security-k8s-presubmit-tests-master

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/e2e-bookInfoTests-trustdomain.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test e2e-bookInfoTests-trustdomain-master

prow/e2e-bookInfoTests-v1alpha3.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-pilot-k8s-presubmit-tests-master

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

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

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-trustdomain.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test e2e-bookInfoTests-trustdomain-master

prow/integ-security-k8s-presubmit-tests.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-pilot-k8s-presubmit-tests-master

prow/e2e-bookInfoTests-v1alpha3.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

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

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

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-trustdomain.sh
fb50602ab23a993f0b9073bafb1f9c62fd79b814
link
/test e2e-bookInfoTests-trustdomain-master

prow/integ-pilot-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-pilot-k8s-presubmit-tests-master

prow/e2e-bookInfoTests-v1alpha3.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-security-k8s-presubmit-tests-master

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

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

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-pilot-k8s-presubmit-tests-master

prow/e2e-bookInfoTests-v1alpha3.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-security-k8s-presubmit-tests-master

prow/e2e-bookInfoTests-trustdomain.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test e2e-bookInfoTests-trustdomain-master

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

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

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-pilot-k8s-presubmit-tests-master

prow/e2e-bookInfoTests-v1alpha3.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-security-k8s-presubmit-tests-master

prow/e2e-bookInfoTests-trustdomain.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test e2e-bookInfoTests-trustdomain-master

prow/istio-unit-tests.sh
d801b1480e420ab8a56295d51b5576f2184dea03
link
/test istio-unit-tests-master

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

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

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-pilot-k8s-presubmit-tests-master

prow/e2e-bookInfoTests-v1alpha3.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test integ-security-k8s-presubmit-tests-master

prow/e2e-bookInfoTests-trustdomain.sh
ba4537931a492c6b4caa281a3171ab11a301cf36
link
/test e2e-bookInfoTests-trustdomain-master

prow/istio-unit-tests.sh
d801b1480e420ab8a56295d51b5576f2184dea03
link
/test istio-unit-tests-master

prow/istio-presubmit.sh
d801b1480e420ab8a56295d51b5576f2184dea03
link
/test istio-presubmit-master

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


test: use it() alias; use verbs over leading with "Should"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
24

Relevant Lines:
24

💛  - Coveralls

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

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

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
24

Relevant Lines:
24

💛  - Coveralls


Pods with no eBPF maps attached

What happened:

Sometimes, when starting new pods they are not reachable by other pods. After some debugging I realised that:

There is no eBPF map attached to these pods when executing /opt/cni/bin/aws-eks-na-cli ebpf loaded-ebpfdata | grep Pod, but they do have a mapping when everything works fine.
There is a log entry with the text "Target Pod doesn't belong to the current pod Identifier". The command grep "Target Pod doesn't belong to the current pod Identifier:" network-policy-agent.log | sed -e "s/.*Pod ID\: //" | awk -F "\"" '{print $3}' | sort -n | uniq returns the list of all pods that are hosted in the current instance and are not reachable from other pods (because they don't have a map).
If I open a shell in those affected pods, I realise that they can connect to internet and they have access to all IPs, even to an IP that should be filtered by a network policy attached to the namespace.

Our network policies are composed by:

A generic NetworkPolicy that affects the whole namespace, which has the following rules:

deny all ingress traffic by default
allow all egress traffic going to internet EXCEPT for an specific IP. <-- This is not filtered on the affected pods!

A specific NetworkPolicy that is deployed with the application, which allows access from other services. <-- This is denied on the affected pods!

If we take a look to the PolicyEndpoint resources, they look fine. Seems a problem between the controller and eBPF.
Attach logs

What you expected to happen:

Ingress traffic should be allowed from the specificed pods.
Egress traffic should be filtered to the IPs that uses the except parameter.
There should be a eBPF program attached to all pods that have Network Policies attached.

How to reproduce it (as minimally and precisely as possible):
It is random in our setup, we haven't figured it out yet how to reproduce it.
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version): v1.27
CNI Version: v1.16.0-eksbuild.1
Network Policy Agent Version: v1.0.7
OS (e.g: cat /etc/os-release): Amazon Linux v2
Kernel (e.g. uname -a): 5.10.201-191.748.amzn2.x86_64

@alemuro is the problem persistent, i.e. the eBPF program never gets attached? We do have one known issue that was just fixed by https://github.com/aws/aws-network-policy-agent/pull/179. The short story is that if there are multiple replicas of the same pod on a node, there is a race condition where when one replica is deleted, the eBPF program for the other replica can also be deleted.
If this is a staging environment, you can try the v1.0.8-rc1 release candidate image that we just built. The official v1.0.8 image will be released in the coming weeks.

Got it. If v1.0.8-rc1 does not resolve the issue, you can send an email with the network policy agent logs to<EMAIL_ADDRESS>and we can dig further. Before sending the logs, enable network policy event logs (https://github.com/aws/aws-network-policy-agent?tab=readme-ov-file#enable-policy-event-logs) so the policy decisions can be logged as well

Hello, we've been testing this for the whole day and seems it is fixed.

v1.0.8 release is available - https://github.com/aws/amazon-vpc-cni-k8s/releases/tag/v1.16.3


0.2.17

new release

Pull Request Test Coverage Report for Build 716

7 of 10 (70.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 95.71%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

autokeras/search.py
2
3
66.67%

autokeras/image_supervised.py
1
3
33.33%

Totals

Change from base Build 701:
0.03%

Covered Lines:
1874

Relevant Lines:
1958

💛  - Coveralls


language / translations

Only english supported by default
Implement feature to download language packages? hmm

closing, moving to #182


TimeoutError: [WinError 10060] Deploying a flask application through windows powershell

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp up
Errors:
The command failed with an unexpected error. Here is the traceback:
HTTPSConnectionPool(host='pythonwordtopdf.scm.azurewebsites.net', port=443): Max retries exceeded with url: /api/deployments/latest (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x064CD580>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond'))
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/connection.py", line 169, in _new_conn
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/util/connection.py", line 96, in create_connection
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/util/connection.py", line 86, in create_connection
TimeoutError: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/connectionpool.py", line 699, in urlopen
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/connectionpool.py", line 382, in _make_request
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/connectionpool.py", line 1010, in _validate_conn
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/connection.py", line 353, in connect
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/connection.py", line 181, in _new_conn
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x064CD580>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/adapters.py", line 439, in send
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/connectionpool.py", line 755, in urlopen
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\urllib3/util/retry.py", line 574, in increment
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='pythonwordtopdf.scm.azurewebsites.net', port=443): Max retries exceeded with url: /api/deployments/latest (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x064CD580>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 657, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 712, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/appservice/commands.py", line 37, in _ex_handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 691, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 328, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/appservice/custom.py", line 3873, in webapp_up
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/appservice/custom.py", line 441, in enable_zip_deploy
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/appservice/custom.py", line 3230, in _check_zip_deployment_status
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/api.py", line 76, in get
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/api.py", line 61, in request
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/sessions.py", line 542, in request
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/sessions.py", line 655, in send
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/adapters.py", line 516, in send
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='pythonwordtopdf.scm.azurewebsites.net', port=443): Max retries exceeded with url: /api/deployments/latest (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x064CD580>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond'))

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
git clone https://github.com/Trelaquix/python_word_to_pdf_editor.git
cd into directory
py -3 -m venv .venv
.venv\scripts\activate
pip install -r requirements.txt
az webapp up --sku B1 --name pythonwordtopdf

Expected Behavior
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.28.0

Additional Context

route to service team

Hi, just an update, I managed to fix the deployment issue, however now I'm encountering another issue.
Below are my observations
Http server errors

HttpStatus
HttpSubStatus
Errors
Description

503
0
20
Server Unavailable

Application Logs - Platform logs
Warning 2021-09-15T06:58:39.576
INFO  - docker run -d -p 5568:8000 --name pythonwordpdf_0_34b614a5 -e WEBSITE_SITE_NAME=pythonwordpdf -e WEBSITE_AUTH_ENABLED=False -e WEBSITE_ROLE_INSTANCE_ID=0 -e WEBSITE_HOSTNAME=pythonwordpdf.azurewebsites.net -e WEBSITE_INSTANCE_ID=5656892a738600b748cd8937def16f99a980ac87bf2ea3ad41fb6ecd9e578037 -e HTTP_LOGGING_ENABLED=1 appsvc/python:3.7_20210709.2

Ok 2021-09-15T06:58:42.813
INFO  - Initiating warmup request to container pythonwordpdf_0_34b614a5 for site pythonwordpdf

Ok 2021-09-15T06:58:58.471
INFO  - Waiting for response to warmup request for container pythonwordpdf_0_34b614a5. Elapsed time = 15.6582049 sec

Error 2021-09-15T06:59:08.758
ERROR - Container pythonwordpdf_0_34b614a5 for site pythonwordpdf has exited, failing site start

Error 2021-09-15T06:59:08.766
ERROR - Container pythonwordpdf_0_34b614a5 didn't respond to HTTP pings on port: 8000, failing site start. See container logs for debugging.

Ok 2021-09-15T06:59:08.818
INFO  - Stopping site pythonwordpdf because it failed during startup.

Container Crash Logs Python 3.9
Container pythonwordpdf_0_813adbfd couldn't be started: Logs = 2021-09-14T16:09:27.033876462Z
2021-09-14T16:09:27.033918162Z   _____
2021-09-14T16:09:27.033925562Z   /  _  \ __________ _________   ____
2021-09-14T16:09:27.033929662Z  /  /_\  \___   /  |  \_  __ \_/ __ \
2021-09-14T16:09:27.033933762Z /    |    \/    /|  |  /|  | \/\  ___/
2021-09-14T16:09:27.033937762Z \____|__  /_____ \____/ |__|    \___  >
2021-09-14T16:09:27.033941862Z         \/      \/                  \/
2021-09-14T16:09:27.033946062Z
2021-09-14T16:09:27.033949462Z A P P   S E R V I C E   O N   L I N U X
2021-09-14T16:09:27.033953162Z
2021-09-14T16:09:27.033956462Z Documentation: http://aka.ms/webapp-linux
2021-09-14T16:09:27.033960062Z Python 3.9.0
2021-09-14T16:09:27.033963562Z Note: Any data outside '/home' is not persisted
2021-09-14T16:09:27.330905957Z Starting OpenBSD Secure Shell server: sshd.
2021-09-14T16:09:27.417827231Z App Command Line not configured, will attempt auto-detect
2021-09-14T16:09:27.418772533Z Launching oryx with: create-script -appPath /home/site/wwwroot -output /opt/startup/startup.sh -virtualEnvName antenv -defaultApp /opt/defaultsite
2021-09-14T16:09:27.439611175Z Found build manifest file at '/home/site/wwwroot/oryx-manifest.toml'. Deserializing it...
2021-09-14T16:09:27.442253580Z Build Operation ID: |FEKfGMVdC2s=.75526d0c_
2021-09-14T16:09:27.449813095Z Oryx Version: 0.2.20210708.1, Commit: 6ceb6608673b94827bac111ef5ea01c216f92abb, ReleaseTagName: 20210708.1
2021-09-14T16:09:27.450054096Z Output is compressed. Extracting it...
2021-09-14T16:09:27.451408599Z Extracting '/home/site/wwwroot/output.tar.gz' to directory '/tmp/8d97780cf62f570'...
2021-09-14T16:09:52.855897197Z App path is set to '/tmp/8d97780cf62f570'
2021-09-14T16:09:53.013449311Z Detected an app based on Flask
2021-09-14T16:09:53.013475312Z Generating `gunicorn` command for 'app:app'
2021-09-14T16:09:53.590670664Z Writing output script to '/opt/startup/startup.sh'
2021-09-14T16:09:54.151104384Z Using packages from virtual environment antenv located at /tmp/8d97780cf62f570/antenv.
2021-09-14T16:09:54.153499989Z Updated PYTHONPATH to ':/tmp/8d97780cf62f570/antenv/lib/python3.9/site-packages'
2021-09-14T16:09:56.127481030Z [2021-09-14 16:09:56 +0000] [36] [INFO] Starting gunicorn 20.1.0
2021-09-14T16:09:56.128307532Z [2021-09-14 16:09:56 +0000] [36] [INFO] Listening at: http://<IP_ADDRESS>:8000 (36)
2021-09-14T16:09:56.134008643Z [2021-09-14 16:09:56 +0000] [36] [INFO] Using worker: sync
2021-09-14T16:09:56.143026961Z [2021-09-14 16:09:56 +0000] [37] [INFO] Booting worker with pid: 37
2021-09-14T16:09:59.037479540Z [2021-09-14 16:09:59 +0000] [37] [ERROR] Exception in worker process
2021-09-14T16:09:59.037510640Z Traceback (most recent call last):
2021-09-14T16:09:59.037516640Z   File "/opt/python/3.9.0/lib/python3.9/site-packages/gunicorn/arbiter.py", line 589, in spawn_worker
2021-09-14T16:09:59.037533240Z     worker.init_process()
2021-09-14T16:09:59.037537440Z   File "/opt/python/3.9.0/lib/python3.9/site-packages/gunicorn/workers/base.py", line 134, in init_process
2021-09-14T16:09:59.037541240Z     self.load_wsgi()
2021-09-14T16:09:59.037553340Z   File "/opt/python/3.9.0/lib/python3.9/site-packages/gunicorn/workers/base.py", line 146, in load_wsgi
2021-09-14T16:09:59.037557440Z     self.wsgi = self.app.wsgi()
2021-09-14T16:09:59.037561040Z   File "/opt/python/3.9.0/lib/python3.9/site-packages/gunicorn/app/base.py", line 67, in wsgi
2021-09-14T16:09:59.037564640Z     self.callable = self.load()
2021-09-14T16:09:59.037568640Z   File "/opt/python/3.9.0/lib/python3.9/site-packages/gunicorn/app/wsgiapp.py", line 58, in load
2021-09-14T16:09:59.037572440Z     return self.load_wsgiapp()
2021-09-14T16:09:59.037575840Z   File "/opt/python/3.9.0/lib/python3.9/site-packages/gunicorn/app/wsgiapp.py", line 48, in load_wsgiapp
2021-09-14T16:09:59.037579540Z     return util.import_app(self.app_uri)
2021-09-14T16:09:59.037582940Z   File "/opt/python/3.9.0/lib/python3.9/site-packages/gunicorn/util.py", line 359, in import_app
2021-09-14T16:09:59.037586740Z     mod = importlib.import_module(module)
2021-09-14T16:09:59.037590140Z   File "/opt/python/3.9.0/lib/python3.9/importlib/__init__.py", line 127, in import_module
2021-09-14T16:09:59.037593940Z     return _bootstrap._gcd_import(name[level:], package, level)
2021-09-14T16:09:59.037597440Z   File "", line 1030, in _gcd_import
2021-09-14T16:09:59.037601540Z   File "", line 1007, in _find_and_load
2021-09-14T16:09:59.037605240Z   File "", line 986, in _find_and_load_unlocked
2021-09-14T16:09:59.037608940Z   File "", line 680, in _load_unlocked
2021-09-14T16:09:59.037612640Z   File "", line 790, in exec_module
2021-09-14T16:09:59.037616340Z   File "", line 228, in _call_with_frames_removed
2021-09-14T16:09:59.037620040Z   File "/tmp/8d97780cf62f570/app.py", line 4, in
2021-09-14T16:09:59.037623840Z     from docx2pdf import convert
2021-09-14T16:09:59.037627440Z   File "/tmp/8d97780cf62f570/antenv/lib/python3.9/site-packages/docx2pdf/__init__.py", line 13, in
2021-09-14T16:09:59.037631440Z     __version__ = version(__package__)
2021-09-14T16:09:59.037634940Z   File "/opt/python/3.9.0/lib/python3.9/importlib/metadata.py", line 551, in version
2021-09-14T16:09:59.037638640Z     return distribution(distribution_name).version
2021-09-14T16:09:59.037642940Z   File "/opt/python/3.9.0/lib/python3.9/importlib/metadata.py", line 257, in version
2021-09-14T16:09:59.037646640Z     return self.metadata['Version']
2021-09-14T16:09:59.037654140Z   File "/opt/python/3.9.0/lib/python3.9/importlib/metadata.py", line 245, in metadata
2021-09-14T16:09:59.037658240Z     self.read_text('METADATA')
2021-09-14T16:09:59.037661640Z   File "/opt/python/3.9.0/lib/python3.9/importlib/metadata.py", line 511, in read_text
2021-09-14T16:09:59.037665240Z     return self._path.joinpath(filename).read_text(encoding='utf-8')
2021-09-14T16:09:59.037668840Z AttributeError: 'PosixPath' object has no attribute 'read_text'
2021-09-14T16:09:59.041394548Z [2021-09-14 16:09:59 +0000] [37] [INFO] Worker exiting (pid: 37)
2021-09-14T16:09:59.204186173Z [2021-09-14 16:09:59 +0000] [36] [INFO] Shutting down: Master
2021-09-14T16:09:59.205011874Z [2021-09-14 16:09:59 +0000] [36] [INFO] Reason: Worker failed to boot.

Container Crash Logs Python 3.7
2021-09-15T03:58:23.350757730Z Traceback (most recent call last):
2021-09-15T03:58:23.350764130Z   File "/opt/python/3.7.9/lib/python3.7/site-packages/gunicorn/arbiter.py", line 589, in spawn_worker
2021-09-15T03:58:23.350783030Z     worker.init_process()
2021-09-15T03:58:23.350787430Z   File "/opt/python/3.7.9/lib/python3.7/site-packages/gunicorn/workers/base.py", line 134, in init_process
2021-09-15T03:58:23.350799530Z     self.load_wsgi()
2021-09-15T03:58:23.350803730Z   File "/opt/python/3.7.9/lib/python3.7/site-packages/gunicorn/workers/base.py", line 146, in load_wsgi
2021-09-15T03:58:23.350807630Z     self.wsgi = self.app.wsgi()
2021-09-15T03:58:23.350811230Z   File "/opt/python/3.7.9/lib/python3.7/site-packages/gunicorn/app/base.py", line 67, in wsgi
2021-09-15T03:58:23.350815131Z     self.callable = self.load()
2021-09-15T03:58:23.350818831Z   File "/opt/python/3.7.9/lib/python3.7/site-packages/gunicorn/app/wsgiapp.py", line 58, in load
2021-09-15T03:58:23.350822631Z     return self.load_wsgiapp()
2021-09-15T03:58:23.350826231Z   File "/opt/python/3.7.9/lib/python3.7/site-packages/gunicorn/app/wsgiapp.py", line 48, in load_wsgiapp
2021-09-15T03:58:23.350830131Z     return util.import_app(self.app_uri)
2021-09-15T03:58:23.350833731Z   File "/opt/python/3.7.9/lib/python3.7/site-packages/gunicorn/util.py", line 359, in import_app
2021-09-15T03:58:23.350837631Z     mod = importlib.import_module(module)
2021-09-15T03:58:23.350841331Z   File "/opt/python/3.7.9/lib/python3.7/importlib/__init__.py", line 127, in import_module
2021-09-15T03:58:23.350845031Z     return _bootstrap._gcd_import(name[level:], package, level)
2021-09-15T03:58:23.350848631Z   File "", line 1006, in _gcd_import
2021-09-15T03:58:23.350852931Z   File "", line 983, in _find_and_load
2021-09-15T03:58:23.350856831Z   File "", line 967, in _find_and_load_unlocked
2021-09-15T03:58:23.350860631Z   File "", line 677, in _load_unlocked
2021-09-15T03:58:23.350864531Z   File "", line 728, in exec_module
2021-09-15T03:58:23.350868331Z   File "", line 219, in _call_with_frames_removed
2021-09-15T03:58:23.350872231Z   File "/tmp/8d977fb012810d7/app.py", line 4, in
2021-09-15T03:58:23.350876231Z     from docx2pdf import convert
2021-09-15T03:58:23.350879831Z   File "/tmp/8d977fb012810d7/antenv/lib/python3.7/site-packages/docx2pdf/__init__.py", line 13, in
2021-09-15T03:58:23.350883931Z     __version__ = version(__package__)
2021-09-15T03:58:23.350887531Z   File "/tmp/8d977fb012810d7/antenv/lib/python3.7/site-packages/importlib_metadata/__init__.py", line 994, in version
2021-09-15T03:58:23.350891431Z     return distribution(distribution_name).version
2021-09-15T03:58:23.350896131Z   File "/tmp/8d977fb012810d7/antenv/lib/python3.7/site-packages/importlib_metadata/__init__.py", line 640, in version
2021-09-15T03:58:23.350904131Z     return self.metadata['Version']
2021-09-15T03:58:23.350907931Z   File "/tmp/8d977fb012810d7/antenv/lib/python3.7/site-packages/importlib_metadata/__init__.py", line 618, in metadata
2021-09-15T03:58:23.350912231Z     self.read_text('METADATA')
2021-09-15T03:58:23.350915831Z   File "/tmp/8d977fb012810d7/antenv/lib/python3.7/site-packages/importlib_metadata/__init__.py", line 937, in read_text
2021-09-15T03:58:23.350919631Z     return self._path.joinpath(filename).read_text(encoding='utf-8')
2021-09-15T03:58:23.350923431Z AttributeError: 'PosixPath' object has no attribute 'read_text'
2021-09-15T03:58:23.359289583Z [2021-09-15 03:58:23 +0000] [40] [INFO] Worker exiting (pid: 40)
2021-09-15T03:58:23.512439135Z [2021-09-15 03:58:23 +0000] [37] [INFO] Shutting down: Master
2021-09-15T03:58:23.512519935Z [2021-09-15 03:58:23 +0000] [37] [INFO] Reason: Worker failed to boot.

I'm not sure why azure web app services is asking me to use importlib/metadata (py 3.9) and importlib_metadata (py 3.7) for my flash application even though I did not import it in my app.py file
Here are the dependencies I use
from re import X
from flask import Flask, request, render_template, redirect, url_for
from mailmerge import MailMerge
from docx2pdf import convert
import os
import fitz
import smtplib
from email import encoders
from email.mime.text import MIMEText
from email.mime.multipart import MIMEMultipart
from email.mime.application import MIMEApplication
import sys
import pypyodbc as odbc
import datetime

Update
I realised I was getting the errors for importlib_metadata because I had some codes that were trying to get the current working directory and joining it together with a file.
Initially I tried this:
edited_word_directory = os.path.join(current_dir, "edited_draft" + "." + "docx")
edited_pdf_directory = os.path.join(current_dir, "edited_draft" + "." + "pdf")

After commenting out the code above, my webpage was able to run however it was unable to perform its main role. I figured it was because of the 'os.path.join' so I tried another method
edited_word_directory =  pathlib.Path().resolve() / "edited_draft.docx"
edited_pdf_directory = pathlib.Path().resolve() / "edited_draft.pdf"

The new method did not work too. It produced the following errors:
Application logs
Ok 2021-09-15T10:42:53.0298681
File "/tmp/8d978326cb395af/antenv/lib/python3.7/site-packages/importlib_metadata/__init__.py", line 545, in read_text

Ok 2021-09-15T10:53:12.8666233
from importlib.metadata import version

Error 2021-09-15T10:53:12.8666275
ModuleNotFoundError: No module named 'importlib.metadata'

Platform logs
Container pythonwordpdf05_0_4d0b0cd6 didn't respond to HTTP pings on port: 8000, failing site start
Container pythonwordpdf05_0_4d0b0cd6 for site pythonwordpdf05 has exited, failing site start

Container Crashed Log
2021-09-15T10:34:56.936992871Z   File "/tmp/8d978326cb395af/antenv/lib/python3.7/site-packages/importlib_metadata/__init__.py", line 273, in metadata
2021-09-15T10:34:56.936996771Z     self.read_text('METADATA')
2021-09-15T10:34:56.937000471Z   File "/tmp/8d978326cb395af/antenv/lib/python3.7/site-packages/importlib_metadata/__init__.py", line 545, in read_text
2021-09-15T10:34:56.937004371Z     return self._path.joinpath(filename).read_text(encoding='utf-8')
2021-09-15T10:34:56.937008171Z AttributeError: 'PosixPath' object has no attribute 'read_text'
2021-09-15T10:34:57.001025691Z [2021-09-15 10:34:56 +0000] [40] [INFO] Worker exiting (pid: 40)
2021-09-15T10:34:57.108034188Z [2021-09-15 10:34:57 +0000] [37] [INFO] Shutting down: Master
2021-09-15T10:34:57.114900968Z [2021-09-15 10:34:57 +0000] [37] [INFO] Reason: Worker failed to boot.

@Trelaquix thanks for the updates on this & investigations , the issue is unrelated to CLI. If this still occurs, please open a support case so the feature team, can help assist here. Thanks!

Need to close this since this is not actionable on the CLI team


Travis CI failing

The current Travis build (29) is failing

There were a few people at Mozfest who had the AttributeError: 'DegreeView' object has no attribute 'values' error that interrupts builds of python 3.3 and later

This appears to be a networkx 1 to 2 migration issue. The migration guide seems to recommend using dict(G.degree()) to get the degrees as a dictionary. This should be backwards compatible with networkx 1

Great find @Islast (and our fellow 🐛 finders at MozFest!)

Do you guys want to require networkx version 1.9 to fix the Travis tests first? Or are you planning on migrating to version 2.0 to fix things?

Good question @schae234 ! I think we will definitely want to migrate to networkx version 2.0 sometime. But for now I  we should probably change Travis to use version 1.9. If our Travis builds pass after that we can close this issue and create a new "networkx verison 2.0 compatability issue"

Hi @Islast , that sounds like a good plan! I have a fork thats passing with python version 3.6, but the regression tests fail for other versions of python. I'm still looking into it! :+1:
Do you know if there is a way to get the tests to be more verbose? I'm not familiar with the unittest package

Great job @schae23 ! I believe you use a -v to get "verbose" detail, as in:
 python3 -m unittest discover -v -p '*_tests.py

in the .travis.yml

Some thoughts about why the non-3.6 tests might be failing.

The regression tests compare to files that were generated (using a fixed random seed) in python 3.6
I notice that the corrmat tests don't fail - this is the only file that is created without using networkx
The global and rich club measures both use the random seed, and these tests both fail, but the local measures do not use the random seed and they fail too

@schae234 if you feel like keeping going on this please do! If not, the work you've already done is extremely helpful :hibiscus:
Feel free to PR whenever

Travis builds are now passing as of pull request #49


add tapret DFS path or opret host in psbt bundle CLI command

completing an RGB transfer using the CLI currently requires the user to prepare the PSBT to hold the committment (before psbt bundle for opret or afterwards for tapret), and there is no CLI command to do so (one either needs to use descriptor-wallet for walleting or write one in rust)
this PR attepmts to drop such requirement by adding the close method as an optional parameter (tapret as default, like for the blind command) and, if needed, automatically set tapret DFS path or opret host
this change should allow users to create PSBTs with their wallet software of choice and still be able to use the CLI to transfer RGB assets
PS: opening this on branch v0.8 instead of master as the .set_opret_host method is not available there (yet)

notes:

new version also updates commit_verify and rgb-core dependencies
once this PR is accepted, let's also port this to the master branch, once .set_opret_host will be available (I can open a new PR if you want me to)

@dr-orlovsky could you please let me know if the changes are ok for you?

@dr-orlovsky requested changes applied (with a note), could you please have a look?


Update window position in OnMove method

Fixes problem with saving window position(see #1023 , #1146 )

TODO: test on MacOS


fix: Guide: Rewrite newline character from translations

Hi everyone!
Translations for guides may contain "illegal" backslash characters.
This is just a one line fix

Codecov Report
Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 7.44%. Comparing base (4d9c7fc) to head (14e664b).
Report is 184 commits behind head on develop.

Files
Patch %
Lines

...ackages/smooth_app/lib/helpers/strings_helper.dart
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5365      +/-   ##
==========================================
- Coverage     9.54%   7.44%   -2.11%
==========================================
  Files          325     380      +55
  Lines        16411   19455    +3044
==========================================
- Hits          1567    1448     -119
- Misses       14844   18007    +3163

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

Thanks for your reviews!


api: Enable controlling the HTTP version used to connect to the backend

What this PR does / why we need it:
This PR proposes adding an API to control the HTTP version that will be used when connecting to a backend configured via an HTTPRoute.
kind: BackendTrafficPolicy
spec:
   upstreamHTTPVersion:  SameAsDownstream # or HTTP1/HTTP2/HTTP3

The current behavior is that HTTP/2 is explicitly used if the route is a GRPC route, and it's possible to force HTTP/2-over-cleartext (h2c) by setting the appProtocol field in the backend Kubernetes Service to kubernetes.io/h2c. In all other cases EnvoyGateway generates configuration that forces HTTP/1.1 to the upstream.
The current behavior could be problematic. This proposal changes the defaults so that it becomes harder to exploit this category of issues.
The proposed API would allow configuring Envoy Proxy to use the same HTTP protocol version when connecting to the backend as was used by the downstream client. It would also allow forcing a specified version of the HTTP protocol to be used and upgrade/downgrade the client protocol as required.

hey @liorokman can you raise a GH issue to define the use case ? would like to get input from the community to be understand if this is needed or not

hey @liorokman can you raise a GH issue to define the use case ? would like to get input from the community to be understand if this is needed or not

Opened #2437

/ok-to-test

LGTM. Maybe we need to update the description with an up-to-date example of the API.

Maybe we need to update the description with an up-to-date example of the API.

Done

@guydc @arkodg PTAL


Is there a way to truncate the string that is displayed via "txtContent": "@currentField"?

Category

[X] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Hello,
Is there a way to truncate the string that is displayed via "txtContent": "@currentField"?
In concrete terms, I need to display whatever is found in @currentField without the last 5 characters. The column in question holds a single line of text.
Many thanks,
Santiago.
Thanks for your contribution! Sharing is caring.

Make a new column which is the copy of the @currentField, and apply column-formatting to it.
Use the SPFx field Customizers.

Thanks.
Since (and I've probably should've said this before) my dev skills are very, very basic, I am only capable -so far- to learn from examples. Is there some sample code available anywhere that could be easily adaptable to my needs? Ideally some JSON code like in https://github.com/SharePoint/sp-dev-list-formatting/tree/master/column-samples that is adapted to my basic skills and that I was able to copy and modify for my site.

You can learn the basic TypeScript use 5 mins.
Then, you can follow the document to implement the Field customizer: https://docs.microsoft.com/en-us/sharepoint/dev/spfx/extensions/get-started/building-simple-field-customizer
Finnally, replace the text binding with:
${(event.fieldValue.substr(0,event.fieldValue.length-5))}


How do I connect my controller to the game?

And now its telling me to connect to the toy pad to the system


Convert Uint8Array to Buffer

Should we expose the augment function somehow to allow for converting an Uint8Array to a Buffer without having to do a copy (which I believe new Buffer(uint8array) does now) ?

Published buffer 2.1.8 which exposes Buffer._augment.
Published typedarray-to-buffer. Enjoy!

Thanks for this, so it was really this easy to convert to buffer


colored output of composer

Coverage decreased (-0.05%) to 43.112% when pulling d38a1629537d986ca8618ff0060672fa3dce34c0 on bankiru:composer-colored-ouput into 55e6f896c857bddc58fcc9af48606d2b0b23b740 on Codegyre:master.

This looks good, but there is the question of how we should handle boolean options.  The APIGen tasks use asTextBool; perhaps this could be moved to BaseTask and used consistently?
cc: @DavertMik

This looks good, but there is the question of how we should handle boolean options. The APIGen tasks use asTextBool; perhaps this could be moved to BaseTask and used consistently?

No, I like this PR the way it is and not to be mixed with ApiGen.
Thanks!
@greg-1-anderson can we make coveralls not to fail builds?

I configured Coveralls so that the total coverage must be > 20% (currently 43%) to pass, and a decrease in coverage won't cause a failure unless coverage drops at least 40%. That should effectively stop coveralls from marking tests as failures if coverage goes down -0.05%.


Add default constructor visibility

Part of #37

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (e637117) to head (0e9175b).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #69   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           58        58
  Lines         2736      2735    -1
=========================================
- Hits          2736      2735    -1

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


Update version to 0.2.155

Motivation: Allow updating libc in the standard library. 0.2.154 was broken by https://github.com/rust-lang/libc/issues/3608, I don't think there are more issues now, though I'm sure bors will prove me wrong.
Related: Maybe consider yanking 0.2.154 after this, as consumers of the posix APIs that require posix_spawn_file_actions_t are unsound in that version.

Hope this can be merged and released soon since Nix is broken due to the yanked 0.2.154: https://github.com/nix-rust/nix/pull/2409


Add AWS Kendra Client tool

@vasinov

How hard would it be to add a test with a mock?

Moto doesn't support Kendra. Would a test similar to https://github.com/griptape-ai/griptape-labs/blob/main/tests/unit/test_aws_pricing.py be sufficient?

@vasinov

How hard would it be to add a test with a mock?

Moto doesn't support Kendra. Would a test similar to https://github.com/griptape-ai/griptape-labs/blob/main/tests/unit/test_aws_pricing.py be sufficient?

Yeah, I think that works. Could also use a proper mock like HERE.


ci: license check in github action

Hold this after #7 gets merged? Some files are added/removed there.

Hold this after #7 gets merged? Some files are added/removed there.

Good idea

I will close this PR for too many conflicts, and re-open it


Bower version update to match package.json

Thank you!

Ended up deciding just to pull the bower version field since it's unused and my build script no longer updates -- just merged your first commit and pushed a patch tag


Add a script to measure the bundle size.

This adds a CI that measures the impact of the change against master for the bundle size.

This change is

At the publish level as well as PR-level I think diffing would be useful inside Cloud CI.
I can set that up if people think it's useful.
Also, I can put it in build-npm, that makes sense. Let's first answer whether we want this to just be an automatic check on cloud instead.

Getting the gzip numbers as an FYI upon explicitly calling a script is not so useful since we have bundlephobia.comCloud. However automatically doing it on CI + diffing with master (your PR added x.x KB to the bundle) sounds very useful.

Yeah I was thinking it would be done as you're trying to drop things from the library & measure without waiting for publishing to NPM. But a CI solves that, so I'll do it there.

Okay, updated this so it runs on a CI as a separate thing. Here is an example with me adding two methods (dropping the backends doesn't quite work with CI yet):
https://pantheon.corp.google.com/cloud-build/builds/61efaaed-60d3-4ae5-8308-8243cfd3b472?project=learnjs-174218

The only slight possible issue is this takes a really long time, about the same length as the tests themselves :/

On the last run it's now taking a minute longer than the tests themselves.
Argh.


Athlete Sort in Python

Is your feature request related to a problem? Please describe.
I would like to add this script in this collection I think it will fit right in
Describe the solution you'd like
I will implement it in python I came across this in HackerRank
Additional context
Please give me the opportunity to work on it
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

I can do this in C++
Assign me plz.
LGM_SOC

@Wittty-Panda mention the language

I can do this in C or python.
Assign me plz.
LGM_SOC

@jayant766 implement in c

@Amit366 sure

@Amit366 Please take a look at pr and accept or give me feedback for LMGSOC with appropriate lable


add Compatibility with IE11

The Object.assign function doesn't exist in  IE11. I've added the MDN polyfill if the function is not recognized.
This version of your lib is well working in our product on IE11.

Hi @castorls. As the project uses Babel I've added the Object.assign polyfill to the .babelrc file.
Can you check out the branch object-assign-ie11 (referenced above), build it and let me know if that fixes the issue for you? I'll try and get it tested in IE11 as well.

Hi @ashiguruma
I've tested the object-assign-ie11 branch on IE11 and I get " Expected ')' " in console, but I did not investigate further.

What is the status on this, is this project active?

Hi @m4m4m4 I don't think this is going to be fixed, I personally switched to png backgrounds since they work on IE11 https://www.chartjs.org/docs/latest/general/colors.html

I just built the object-assign-ie11 branch. IE11 works great if you run the built files in the dist folder

Seems like that branch works; can it (or this) please be merged in? Trying to use this on a project now and the lack of IE11 support is an issue.

Can we get a new release in npm with this fix?

@m4m4m4 On it but am having some build troubles but should have a tag out soon.


Hotfix issue/1629

Codecov Report

Merging #1633 (5a8c01d) into main (f0a2b1c) will decrease coverage by 0.28%.
The diff coverage is 92.60%.

:exclamation: Current head 5a8c01d differs from pull request most recent head 72b9508. Consider uploading reports for the commit 72b9508 to get more accurate results

@@            Coverage Diff             @@
##             main    #1633      +/-   ##
==========================================
- Coverage   94.22%   93.94%   -0.29%
==========================================
  Files         235      253      +18
  Lines       42893    47615    +4722
==========================================
+ Hits        40418    44732    +4314
- Misses       2475     2883     +408

Impacted Files
Coverage Δ

common/src/writer.rs
94.11% <ø> (ø)

fastfield_codecs/src/main.rs
0.53% <0.00%> (-0.42%)
:arrow_down:

ownedbytes/src/lib.rs
98.63% <ø> (+1.36%)
:arrow_up:

query-grammar/src/occur.rs
100.00% <ø> (ø)

src/aggregation/bucket/mod.rs
92.98% <ø> (ø)

src/collector/custom_score_top_collector.rs
100.00% <ø> (ø)

src/collector/mod.rs
48.21% <ø> (ø)

src/collector/top_collector.rs
97.60% <ø> (ø)

src/collector/tweak_score_top_collector.rs
100.00% <ø> (ø)

src/core/inverted_index_reader.rs
72.86% <0.00%> (-4.19%)
:arrow_down:

... and 266 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


problem while running script

hello, i runned the script and got this error

You have to set only the ID (number) of the item.

i put the id only and it doesnt work

delete de comma, you only have to write it when you are going for more than 2 items

now it works, thank you so much!


Review is required.

A review of Rust code in this repo is required.

A review is not possible due to trait implementation mismatch errors:
error[E0407]: method `register_channel` is not a member of trait `TransportConfiguration`
  --> src/lib.rs:41:5
   |
41 | /     fn register_channel(&mut self, sender: Sender<Data>) -> Result<()> {
42 | |         // Vec::push() panics when number of elements overflows `usize`
43 | |         if self.channel_pool.len() == std::usize::MAX {
44 | |             return Err(AtMaxVecCapacity);
...  |
47 | |         Ok(())
48 | |     }
   | |_____^ not a member of trait `TransportConfiguration`

error[E0407]: method `register_callback` is not a member of trait `TransportConfiguration`
  --> src/lib.rs:57:5
   |
57 | /     fn register_callback(&mut self, callback: fn(Data) -> bool) -> Result<()> {
58 | |         // Vec::push() panics when number of elements overflows `usize`
59 | |         if self.callback_pool.len() == std::usize::MAX {
60 | |             return Err(AtMaxVecCapacity);
...  |
63 | |         Ok(())
64 | |     }
   | |_____^ not a member of trait `TransportConfiguration`

error[E0407]: method `set_callback_timeout` is not a member of trait `TransportConfiguration`
  --> src/lib.rs:65:5
   |
65 | /     fn set_callback_timeout(&mut self, timeout: u64) {
66 | |         self.callback_timeout = timeout;
67 | |     }
   | |_____^ not a member of trait `TransportConfiguration`

error[E0277]: the trait bound `TCPtransport<Data>: futures_core::stream::Stream` is not satisfied
   --> src/lib.rs:159:36
    |
159 | impl<Id, Pe, Data: 'static, E, PL> Transport<Id, Data, E, PL> for TCPtransport<Data>
    |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `futures_core::stream::Stream` is not implemented for `TCPtransport<Data>`

error: aborting due to 4 previous errors

Those need to be fixed first. A fix for that will not be a part of the review.

Obviously libtransport-tcp should not be reviewed with version of libtransport modified after the last commit in libtransport-tcp - on Friday we were in transition to async/.await framework in Rust nighly for both repos, libtransport has transition completed on Friday while libtransport-tcp got it just today.
Please review libtransport-tcp as of today version.

Compilation succeeds now, thanks! I'll submit a PR with my review since it's going to be awkward reviewing code in multiple commits.


Unable to set permissions for worlds containing numbers or dashes

As title states, this is an issue with Forge/Vanilla worlds as they use the default format of "DIM" plus ID so nether would be "DIM-1".

Yuuup.
Permissions are just stored as a single string. For example:
some.permission only applying on server "survival" gets stored in the file as factions/some.permission.
some.other.permission only applying on server "survival" and in world "nether" gets stored in the file as factions-nether/some.other.permission.
This is the same for temporary permissions, some.temp.perm expiring in a few hours from now is stored as some.temp.perm$1486221311.
This is just the way I decided to store permissions when I wrote the plugin, it made perfect sense to do so at the time, as it meant for really simple config files.
However, the limitations are pretty annoying, I know. I guess I'll have to do the same thing as with meta nodes, and escape these characters when they get stored. Orrr, just change the layout, which is something I'm not so keen on.
Idk, I'll see what I can do.

Ok, that commit should remove all limitations on server / world names.


Changes for Applicants details event

Before creating a pull request make sure that:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/PRL-460
Change description
Changes for Applicants details event
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Closing as the deployment has issues, creating a new PR.


feat: add suspense query hooks

Motivation
Review this pr one by one

~upgrade react-query, upgrade react-query package, align with latest version of react-query for supported api, setup migration guide~ Separated out into https://github.com/lifeomic/one-query/pull/44

First  add useSuspenseAPIQuery hook

Second add useSuspenseInfiniteAPIQuery hook

Third add useSuspenseCombinedAPIQueries hook

@aecorredor I don't think so -- rather, we're just not changing the API of the one-query hooks to take a single object instead of a series of parameters. So, this PR is ready for review.

@jkdowdle one initial piece of feedback: I think we could have separated this PR into two smaller PRs:

one to simply upgrade to react-query v5
one to add support for "suspended" queries.

If it's really easy for you (since they're already broken up by commit), might be worth doing that now before reviews start coming in.

This pr now only includes the new hooks and it targets the branch that includes the breaking change: upgrade


[IR] Create GraphView for read-only view on a subgraph

A read-only view on a graph.
The GraphView is useful for analysis of a subgraph. It can be initialized
with a subset of nodes from a :class:Graph. Creating GraphView does not
change the ownership of the nodes, and so it is possible to create multiple
GraphViews that contain the same nodes.
The graph view can be serialized to ONNX::
    graph_proto = ir.serde.serialize_graph(graph_view)

It can also be used to create a model::
    model = ir.Model(graph_view, ir_version=8)
    model_proto = ir.serde.serialize_model(model)

Example usage:
import onnx

from onnxscript import ir

model = onnx.load("torchscript_model.onnx")
m = ir.serde.deserialize_model(model)
graph = m.graph

nodes = [
    node for node in graph if node.name is not None and node.name.startswith("/model/layers.0")
]

inputs = set()
outputs = set()
for node in nodes:
    inputs.update(node.inputs)
    outputs.update(node.outputs)

graph_inputs = sorted(inputs - outputs, key=lambda val: val.name)
graph_outputs = sorted(outputs - inputs, key=lambda val: val.name)

graph_view = ir.GraphView(graph_inputs, graph_outputs, nodes=nodes)

print(graph_view)

model = ir.Model(graph_view, ir_version=8)
onnx_model = ir.serde.serialize_model(model)
onnx.save(onnx_model, "torchscript_model_layer_0.onnx")

Merging for now. I will create follow-ups if new comments come in


no work in real iphone

I use the following code to download remote image, it is working on Android and IOS Simulator but didn't work on iphone 5s device(downloaded the image file but not save into local file). any one can give me suggest.  I have update to RN0.34 & RNFS 2.0.2rc-2 , it is also happend on RN0.31 & RNFS 1.5.1
       var DownloadFileOptions = {
            fromUrl: url,          // URL to download file from
            toFile: tempImagePath         // Local filesystem path to save the file to
        }
        var result = RNFS.downloadFile(DownloadFileOptions);

        result.promise.then(function (val) {
            Reactotron.log('Success Result:' + JSON.stringify(val));
            var localFilePath = 'file://' + tempImagePath;
            self.setState({
                localImagePath: localFilePath,
                imageLoaded: true,
            });
        }, function (val) {
            Reactotron.log('Error Result:' + JSON.stringify(val));
        }
        ).catch(function (error) {
            Reactotron.error(error.message);
        });

I resolve this issue. on this post:


Eo extension update readme

Related Issue(s): #
N/A This PR addresses issue raised by @lossyrob in STAC/Lobby.

The README here https://github.com/radiantearth/stac-spec/tree/dev/extensions, under the section “Prefixes”, makes it seem like extension prefixes (like eo) are for keys in the properties object of the Item (this is also how https://github.com/radiantearth/stac-spec/blob/dev/extensions/eo/examples/example-landsat8.json is set up). The examples in https://github.com/radiantearth/stac-spec/blob/dev/extensions/eo/README.md make it seem like they go direclty into the body of the Item object. Is this a misplacement in the latter? Is extension Item information explicitly supposed to go into the properties object of the Item?

Proposed Changes:

Modifies examples in Assets Object section of EO extension README to place eo:bands inside of item properties and corrects a linked example JSON.
Updates Prefixes section of extension README with eo:gsd in place of eo:resolution.

PR Checklist:

[x] This PR has no breaking changes.
[x] I have added my changes to the CHANGELOG or a CHANGELOG entry is not required. Documentation changes only.
[x] API only: I have run npm run generate-all to update the generated OpenAPI files. No API changes.

I made some further changes until I realized that this PR - unfortunately - bases on master, not dev. So it's likely that we are working on an outdated example and that's why stac_version, stac_extensions etc are missing. Therefore this PR needs to be done again, basing on dev, sorry.

@m-mohr Sorry about that! I'll walk though it again and will add your changes as well


ERRORS when try to render slots manually with computed and

Vue version
3.2.47
Link to minimal reproduction
https://sfc.vuejs.org/#eNqNUUFu3DAM/Aqhi3fRrHR37bRFgX4gx6oHx5Z3ldiUIFFbFI7/XspaL5JL0ZuoIWeGnEV8815ekxG1aGIfrCeIhpKHqcNzqwVFLR412tm7QLBAMCOsMAY3Q8Vj1ec79t3N/gZIlYtMW2nU2DuMTEsB2jx/qAYzdmmi6sjTGseEPVmH0A3DE4XDERaNkPvltZuSgU8tVCcGs9iqsVHFKNvigszsp44MVwBN1t1eAMuySa48wcDmqPQ8JyJW+9pPtn/lFYssr8mPRhWUOxv1jlo8iLLmae68fIkO+WCbTX0D+E51MZ7/ePVca3Eh8rFWKo59vsdLlC6cFb9kSEh2NtLE+fQc3O9oAhNrkSnY88qS+xX/N5ye+xOZ4QFSNE+To/ghq3dZbFh7bzscd6S/2GkIBhnc2Q6cSPtYdgssH7DMy1uMXyQ3vL3Bz1/smon+lc9gr3A9jS6w+00KLN41tYD61fzZIRkvnTc/pu68bZkP22RPDg0S1DbujQw36o4UIcVKH0MU618DdAJt
Steps to reproduce
click the 'add' button.
What is expected?
the string 'default' change to 'default-add.
What is actually happening?
show errors.
System Info
No response
Any additional comments?
I want to render the slot manually，but it seem to must use the h function。If not，it wont be reactive.

You should modify the binding value of the key.
https://sfc.vuejs.org/#eNqNUk1vnDAQ/SsjX9hVF/tOIW3Vc0851j0QMFknYFv2mKoi/PeO8S7KhxLl5vGbee/Nx8J+OMfnqFjF6tB57RCCwuhgbM19IxkGyW6k0ZOzHmEBrwZYYfB2goLKiq879tNO7gJwkYJEW0gjTWdNIFr00KT6Q9GroY0jFkeqlmaIpkNtDbR9f4v+cIRFGkj5fG7HqOBLA0VJYBJbpalFNkq2KEA1ubFFRRFAnXS3F8CybJIrVRCwOco5dxGR1L53o+4eqcUsS23SoxYZpcxa7NTsxHKT5dQ6/hCsoXFtJuUFoClV2Xb6o8ZTLNkZ0YVKiDB0aRoPgVt/L+jFfTSoJ8VVmMo7b/8G5YlYstMzDkGfs/KlV6ZXXpHF9zlfpb7hTbQ0iZVaue7msyvvKD+i6k8Qg7odLYYXF/BswxvW7GmH4xXpznrsyRuBV7YD7bm5yTPzJO9NrueX4/jGKeHpCX7/IddE9NHWez3DXA7Wk/tNCrTZNSWD6lH9I+hXi2fuW9PbiaylHtOo6+TIGmUQKh2uDATXYkeyjCCdV4ex/gcOix+j

Ricardo is right shapeFlag is not a unique value, so it does not work as a key.

Ricardo is right shapeFlag is not a unique value, so it does not work as a key.
thx.
At first,i just want to set a key to test, but then i forget to correct it, and only think its a bug with computed and slots.
Sorry to interrupt.


Add explicit snapshot date subset to Widget_GroupOverview.

Overview

Resolves #1787
Resolves #1788

Test Notes/Sample Code
load_all()
Widget_GroupOverview(
  dfResults = reportingResults,
  dfMetrics = reportingMetrics,
  dfGroups = reportingGroups
)

Notes:
@jonthegeek - #1788 ties into this issue in {gsm.app}.  Allows you to generate the analysis output table with all groups in addition to only groups flagged for the metric.

@samussiah I think what we really want for that ticket is to show all of the results, but highlight the selected site (similar to what we do with the plots). Is that on your radar?