(vagrant) Update to match new filenames

Description
Looks like Vagrant have changed the filename and the links on the page being scraped.
Motivation and Context
Fixes #1973
How Has this Been Tested?
Ran the update.ps1 file and checked the amended .nuspec and chocolateyInstall.ps1
Screenshot (if appropriate, usually isn't needed):
N/A
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] I have updated the package description and it is less than 4000 characters.
[x] All files are up to date with the latest Contributing Guidelines
[x] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

@pauby thanks for getting this fixed up!


Parameter and Argument names should support snake case

Implements #1047
This workflow should be valid now:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: spec-arg-snake-case-
spec:
  entrypoint: whalesay
  arguments:
    artifacts:
    - name: __kubectl
      http:
        url: https://storage.googleapis.com/kubernetes-release/release/v1.8.0/bin/linux/amd64/kubectl
    parameters:
    - name: my_snake_case_param
      value: "hello world"
  templates:
  - name: whalesay
    inputs:
      artifacts:
      - name: __kubectl
        path: /usr/local/bin/kubectl
        mode: 0755
    container:
      image: docker/whalesay:latest
      command: [sh, -c]
      args: ["cowsay {{workflow.parameters.my_snake_case_param}} | tee /tmp/hello_world.txt && ls  /usr/local/bin/kubectl"]

Looks like this fix is not released yet =(

Is there any reason not to support spaces as well?


Crash on Death

Hi! I've noticed that my game has been crashing if I die near a functioning Sprinkler block.
This is happening on Forge, 1.20.1, Forge version 47.2.0 with Java 17.0.2.

I think this is the exact Stack Trace...

trace-resume.txt

full-trace.log

I also noticed one more thing. After I respawn and the game crashes, if I teleport back to the location of the Sprinkler that caused the crash, it gives one last splash of water and stops until I trigger it again.
I'm a programmer, but I don't know much about Minecraft modding, but if I had to guess, I'd say there's a loop running that causes the splash of water, but when I respawn there's no longer the Sprinkler block, which causes the crash or freeze.

interesting, Ill investigate this tomorrow. thanks for the report

Just to add: If I'm playing on a Forge server with the same mods installed with everything of the test scenario, and I repeat the steps, the Server doesn't crash, only my Client.
I did the test twice just to confirm, and indeed, the server continues to run normally.

Okay, so the thing that is most likely happening here is that when the player is for any reason quickly moved away from a sprinkler, be it by death or teleportation, the chunk the sprinkler is in is unloaded before the client-side logic has finished running (spawning splash particles and whatnot). Deploying a fix shortly

I see! I'm glad it's fixed! I'll test the new update as soon as possible!


design graphics to illustrate core OLS concepts.

One of the core concepts of OLS is that by consciously and intentionally designing open behaviours into our work, we're more likely to be able to be effective.
Currently we use these slides: https://docs.google.com/presentation/d/1fXnQvrt0uBXrYwUlvePSx6N7m7DnKLD1hCRTxos-QOk/edit#slide=id.p16
see slides 3, 7, 16, 18, 19
multiple people can make prs on this issues

Hi @yochannah . Are we to redesign just the slides noted or the entire presentation?

@yochannah could you explain further on what you mean by graphics? Like poster designs?

great questions!
@Preciousutos  The slides here are to illustrate concepts - but think of making a graphic, rather than just the slide. Something we can re-use everywhere that illustrates the concept.
@jilaga yes, but probably not as big. Try to aim for things that would still fit onto a slide, or a webpage. They can be more complex than a logo, though!

Thank you @yochannah ❤

Okay @yochannah. Should we use the current branding guide or can we stretch a little bit away from it?

Hello @yochannah . Should we stick to the exact same copy that's already on the slides?

@yochannah I noticed some typo and punctuation errors on some of the slides

@jilaga go wild, use your creativity!
@stormztee little tweaks are okay, and creative interpretations. IF you're not sure, ask in Slack?
@Preciousutos heheh, don't worry about it too much, but definitely don't replicate our oopsies!

Alright @yochannah

@yochannah am working on this but not sure its the track. I need to know if I can change anything.
say the name of the Organization, the design and the structure.
Core Values.pdf

Alright @yochannah

@Ismael-KG @yochannah here are some redesign ideas I came up with for the pages highlighted above. Feedback would be appreciated, thanks.

@yochannah I reread the description of this issue again and I am now a little confused, do we redesign the whole slides or just the numbers you highlighted,
Also I just realized that Ishould have made a pr with the ideas I came up with 😬

Hello, @yochannah @Ismael-KG. Here's my take on the redesign of the highlighted slides. Looking forward to your feedback; thanks.

@Ismael-KG @yochannah here are some graphics I came up with to illustrate the concepts in the slide. I look forward to the feedback.

So much great work! Just a few simple comments for each:

@niyolynate, try to simplify the slides – for example, your third slide with the core values should be either the tree or the blurb on the left, but not both. Your use of trees is very creative, and I hope they become a feature somehow!
@jilaga, it would have been nice to see some use of green. That said, your design is nice and simple, very slick.
@stormztee, those are very clear slides! The green you use for text is different to the logo's, and it doesn't contrast well enough with the white background, which is needed for legibility (check this site).
@AdeolaAdekoya, your slides are also clear, and use green in a clever way: three different tints/tones and one of them (I think) being the same as the current logo. I would just ask that you check the [contrast rate(https://contrast-ratio.com/#hsla(200%2C0%25%2C7%25%2C.7)-on-white) is good enough so that the green-on-white text is easily legible.

Thanks for the feedback @Ismael-KG .. Utilized the contrast checker and adjusted the text to contrast to make it more legible. Here are the updates.

This is a very useful too @Ismael-KG will definitely be using it in the future. Thank you☺

This is my illustration.
hope any guidance is highly welcome.

Finally made it here!!!

@yochannah @Ismael-KG Here is one part of my submission for the redesign. Please review 🥺
I went for this Dark Mode because I naturally lean towards light mode. For me it's easier on the eyes and make for good contrast too. But that is subjective

@yochannah @Ismael-KG  Yes, It's me again. My second submission for consideration; the light mode

For the light mode, I'd use a different background gradient from the light mode. There is a contrast with hoe the logo appears, it is bigger on the dark mode than the light. Which is better?
The last slide is also quite different in arrangement. I had the two concept sketched down so I decided to put them separately on each mode. Which is better?

Thanks for the feedback @Ismael-KG .. Utilized the contrast checker and adjusted the text to make it more legible. Here are the updates.

Babes @AdeolaAdekoya I like this. It is quite refreshing and clean. Well done 😊😊

Hello, @yochannah @Ismael-KG. Here's my take on the redesign of the highlighted slides. Looking forward to your feedback; thanks.

Love the simplicity of the design 😍😍 Very clean too 🥺

Hi everyone,  @yochannah @Ismael-KG. Here's my take on the redesign of the highlighted slides. Looking forward to your reviews. Thank you

So much great work! Just a few simple comments for each:

@niyolynate, try to simplify the slides – for example, your third slide with the core values should be either the tree or the blurb on the left, but not both. Your use of trees is very creative, and I hope they become a feature somehow!
@jilaga, it would have been nice to see some use of green. That said, your design is nice and simple, very slick.
@stormztee, those are very clear slides! The green you use for text is different to the logo's, and it doesn't contrast well enough with the white background, which is needed for legibility (check this site).
@AdeolaAdekoya, your slides are also clear, and use green in a clever way: three different tints/tones and one of them (I think) being the same as the current logo. I would just ask that you check the [contrast rate(https://contrast-ratio.com/#hsla(200%2C0%25%2C7%25%2C.7)-on-white) is good enough so that the green-on-white text is easily legible.

So much great work! Just a few simple comments for each:

@niyolynate, try to simplify the slides – for example, your third slide with the core values should be either the tree or the blurb on the left, but not both. Your use of trees is very creative, and I hope they become a feature somehow!
@jilaga, it would have been nice to see some use of green. That said, your design is nice and simple, very slick.
@stormztee, those are very clear slides! The green you use for text is different to the logo's, and it doesn't contrast well enough with the white background, which is needed for legibility (check this site).
@AdeolaAdekoya, your slides are also clear, and use green in a clever way: three different tints/tones and one of them (I think) being the same as the current logo. I would just ask that you check the [contrast rate(https://contrast-ratio.com/#hsla(200%2C0%25%2C7%25%2C.7)-on-white) is good enough so that the green-on-white text is easily legible.

Thank you @Ismael-KG I would take that to effect and adjust the design.

Thanks to those who've made reviewed submissions! Just some comments on the completely new ones:

@Bisola15 it's great to see both light and dark mode options! I'd be curious to hear why light mode slides should prevail – dark mode is definitely easier on the eyes! I just wanted to comment how the use of gradients (which looks great!) is clearer on the light than dark background. Was this on purpose or a result of the light/dark background?
@Bharlikiss These are great! Very simple and clear. There's just a change in font size that can be a bit confusing to the audience as you change from slide to slide. And the last slide switches between black and green text too frequently. Switching to green to highlight words is really useful to focus one's attentions, but too much of it and it breaks up the text!

Thank you @Ismael-KG for the reviews. I have adjusted the font size and also reduced the highlighted words on the last slide to avoid the break. Here are the updates.

Thanks to those who've made reviewed submissions! Just some comments on the completely new ones:

@Bisola15 it's great to see both light and dark mode options! I'd be curious to hear why light mode slides should prevail – dark mode is definitely easier on the eyes! I just wanted to comment how the use of gradients (which looks great!) is clearer on the light than dark background. Was this on purpose or a result of the light/dark background?
@Bharlikiss These are great! Very simple and clear. There's just a change in font size that can be a bit confusing to the audience as you change from slide to slide. And the last slide switches between black and green text too frequently. Switching to green to highlight words is really useful to focus one's attentions, but too much of it and it breaks up the text!

Thank you so much for that comment @Ismael-KG 😊  Personally, I would go for dark mode. I had just done the light mode as an option to present what it looks like. So the gradient patterns are not the same for the both modes. I had tried my hands on two different patterns, and for the dark mode I reduced the opacity a lot so that it would blend finely with the green background. I may have overdone it a bit.

This what the background in the dark mode looks like with increased Opacity, Lemme know what you think @Ismael-KG

@Ismael-KG  Ok thanks for the comment. am going to try that

@Ismael-KG @yochannah here are my redesign ideas for the pages highlighted above. Feedback would be highly appreciated, thanks.

I would really love to participate on this task, @yochannah

I would really love to participate on this task, @yochannah

Hi @Emmzybrown, yes you can. She stated earlier that multiple people can contribute.

Hey @Olakid645, thanks for your suggestions! Two comments:

Are these meant to be different styles to choose from? The five slides are great but very unique.
I think I can quite confidently say the last one is a bit too "corporate" in its colours and general theme. The first two seem most coherent with other things we see on the OLS brand 😊

Hi all, I made some illustrations for the core values of OLS and they are refrenced above. I would really love some feedback or suggestions on improvements if any, Thanks.

Hello guys, below are some illustrations I made for the core values of OLS. I would love to get some reviews and suggestions on any improvement please.

Redesigned the image on the intro slide of OLS.
i surrounded "Hello" with tons of life sciences to give a new person an idea of what the organization is about.

I could only attempt 2slides

Hello guys, below are some illustrations I made for the core values of OLS. I would love to get some reviews and suggestions on any improvement please.

I really like this. welldone😊

Hello guys, below are some illustrations I made for the core values of OLS. I would love to get some reviews and suggestions on any improvement please.

Hey @Emmzybrown This is a nice concept. The contrasts of the green texts though, what do you think about it? I think the white background works better than the green one for the contrast. What do you think?

I could only attempt 2slides

This is sweet!! 😍😍 I like the simplicity. Did you make the illustrations?

 Redesigned the image on the intro slide of OLS.
i surrounded "Hello" with tons of life sciences to give a new person an idea of what the organization is about.

I like the colours and the way they blend with the black background 😍😍 and "POP" very nice tbh. Always a sucker for dark modes.

I could only attempt 2slides

This is sweet!! 😍😍 I like the simplicity. Did you make the illustrations?>

No I didn't. I downloaded and edited them

Hi lovely people. 👋
Thanks for letting your creativity run wild!
We're sliding in to close this issue, as OLS has a new brand identity. Your involvement in the redesign is appreciated!
All the best! 🚀🎨


x/mobile: Failed to upload app to the appstore when using SDK generated by gomobile

This issue is related to #15657
When trying to archive https://github.com/AdrianaPineda/iOSGOMobileSampleApp to upload it to the app store we are getting errors related to missing keys in the Info.plist file.
The following screenshot shows the errors we're getting:

To reproduce the issue follow the instructions in the repository, archive and upload to the appstore. You should get an error in the middle of the process.

@crawshaw @eliasnaur hi, any chance to look at this? I know you are busy with the 1.7 release but I'd be nice if you could take a look at this since is blocking our release.
Thanks in advance

Is this a problem with all gomobile (bind?) apps or just the ones that use "super" frameworks as described in #15657 ? In other words, can you reproduce this issue with the gomobile examples?

yes this is for bind and frameworks, gomobile apps (and examples) work fine.

So just to be perfectly clear, the gomobile bind example "golang.org/x/mobile/example/bind/ios/bind.xcodeproj" works and can be archived and uploaded without errors? In that case, what makes adding a gomobile bind framework to an application different from adding it to another framework? In other words, why are Info.plist suddenly needed and why is it a problem that x86_64 is included?

no, wait, I know that the examples of apps work, we haven't tried the bind example. We will test it and report results later.

@eliasnaur we tested the bind example and it works (except for some issues with the icons but we don't believe that's gonna be a problem).
So yes the problem happens when you have a framework embedded.

Thank you for testing. I don't have time myself in the near future to dig into what is necessary from the gomobile side to make this use case work. If you or someone else are up for the challenge, a good first step is comparing a working sub-framework with a framework generated by gomobile bind and work out what additional work gomobile needs to do.
The Info.plist errors seem easy, but I'm still puzzled by why the x86_64 architecture is a problem. x86_64 is not supported by iOS hardware, but is needed for the emulator. In other words: how do other sub-frameworks work on the emulator?

@eliasnaur I'm going to look at this with @dcu
About the unsupported arch x86_64, I've seen this with other frameworks when I try to upload to the app store. I can remove the arch from the framework with a script (when generating the archive) with no problem.

Thanks @AdrianaPineda! I'm sorry for my absence on this, it has been busy around here, and I'm out on vacation until the end of July. But @eliasnaur can review any CLs you have.

As an update, we are trying a different approach to generate our umbrella framework. We will let you know if it works

after generating the umbrella framework in a different way the issue is gone. Thanks @eliasnaur your input was very helpful


"Old Reddit Broken Link Fixer" no longer works

As the title says, links have been showing the backslashes in them again for me in the last few weeks.

I am not able to reproduce, can you give an example url?


test: scenarios-feature_branch_happy_path (2024-07-22 22:24:34.397259+00:00)

automatic test cleanup


RC5: ion-slides misses some Swiper functionality

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.0.0-rc.5
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request
Current behavior:
As getSlider() is deprecated and not all of Swiper's api's are implemented in RC5, some functionality has been lost.
Expected behavior:
I'd like to be able to use all of Swiper's api's, as I'm currently unable to upgrade to RC5.
Steps to reproduce:
Example Swiper implementation that was possible in RC4 but is not possible in RC5, using <Swiper>.params.control:
Demo:
http://idangero.us/swiper/demos/23-thumbs-gallery.html
Source:
https://github.com/nolimits4web/Swiper/blob/5a1b28a84e9504e49975c8f6a85bf90183f13fea/demos/23-thumbs-gallery-loop.html
Related code:
  @ViewChild('sliderGrid') public sliderGrid: Slides;
  @ViewChild('sliderHeader') public sliderHeader: Slides;

  // This works in RC4 to sync two sliders
  public ngAfterViewInit() {
    const header: any = this.sliderHeader.getSlider();
    const grid: any = this.sliderGrid.getSlider();
    header.params.control = grid;
    grid.params.control = header;
  }

  // This works in RC4 to check if it's currently animating
  if ((<any>this.sliderHeader.getSlider()).animating) {
    return;
  }

Ionic info:
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.5
Ionic CLI Version: 2.1.18
Ionic App Lib Version: 2.1.9
Ionic App Scripts Version: 1.0.0
ios-deploy version: 1.9.0
ios-sim version: 5.0.13
OS: macOS Sierra
Node Version: v6.9.1
Xcode version: Xcode 8.2.1 Build version 8C1002

PS: Can we please get up to date angular compatibility (and zone, rxjs) :)
Really great work on the treeshaking!

for example my app uses onlyExternal option. is there a way to use it in rc5 ?

I've submitted a PR for the control functionality that was lost in RC5 (#9983). I suggest anyone that misses other Slides functionality does the same :).

I will close this issue as the missing control this was about has returned in the latest nightly, and the Ionic team is actively working on getting any missing options back to work (e.g. https://github.com/driftyco/ionic/commit/2d26edb679ddf990f6b330c7c9840d8fd9117e4b).
Anyone who is missing other options in the new slides, please create a separate issue or even better, create a PR ;).


TypeScript tests fail with ParseError: 'import' and 'export' may appear only with 'sourceType: module'

Attempting to set up cypress-cucumber-preprocessor with add-typescript-to-cypress, I receive the following error when running a .feature test in Cypress whose step definitions are in a steps.ts file containing an import statement:
Error running plugin
The following error was thrown by a plugin. We've stopped running your tests because a plugin crashed.

...

ParseError: 'import' and 'export' may appear only with 'sourceType: module'

Running a separate spec.ts file containing an import using only @bahmutov/add-typescript-to-cypress in cypress/plugins/index.js works fine, but as soon as I add 	on('file:preprocessor', cucumber()); TypeScript tests containing import fail with the same error.
I see other open issues related to that same error but for ES6, not TypeScript (targeting ES5), so I am opening this case.
Versions:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>MacOS X 10.13.5

can you try using require instead of import?

I tried require, but then I just get another parsing error from our use of TypeScript decorators:
 Error running plugin
The following error was thrown by a plugin. We've stopped running your tests because a plugin crashed.

Error: Parsing file /path/to/file: Unexpected character '@' (X:Y)

that's not good. any chance for a simple reproduction?
I would love to make this work with typescript. import might be tricky but everthing else should work.

Do you want a Github repo, @lgandecki? Is there a sample base project on which to create a reproduction?

Yeah github repo would be great, the simplest thing you can create that replicates the problem so I can jump in and see what’s going on

Please see my reproduction in my cypress-cucumber-typescript. The add-typescript-to-cypress preprocessor works fine with TypeScript syntax, but as far as I can tell, cypress-cucumber-preprocessor ignores its output and attempts to interpret .ts files as JavaScript instead of transpiling them first.

Thanks, I will take a look Friday/Saturday :)

ok.. I spent a couple hours on this, and it is much more work than I initially thought... I will have to pass on it for now. Maybe once we/I need to use typescript I will be able to spend more time to get this to work.
If we were able to use babel7 with cypress (possibly soon), then we can use the plugin-transform-typescript in browserify to remove types from our output js file. At this point I can't think of a quick fix here. :-(
Thanks for the reproduction and sorry couldn't help.
If someone would like to pick it up - looks like the compile function in index.js produces a file with, possibly, valid typescript code. We would need to transpile it to javascript before passing it back to browserify..

Thanks for investigating, @lgandecki. Until TypeScript is actually supported, perhaps the README should be updated to remove TypeScript instructions.

Hi @KeithGillette,
we ran into the same problem during evaluation of cypress + gherkin due to this and #52, #51 and #44 (which can be overcome by using a beforeStep-Hook) i have created https://www.npmjs.com/package/gherkin-to-mocha which is a webpack loader which transforms gherkin files into a mocha test suite. By using https://github.com/cypress-io/cypress-webpack-preprocessor you can have feature files and typescript steps/hooks.
In the end we have decided against cypress because there is no support for Firefox and IE... anyway you may find the loader useful for your project.

@sloosch that looks really cool :-)
I need to evaluate it a bit closer.
Right now #52 is fixed, #51 should be finalized this week (we want to make sure that we get it right the first time around so we don't mess up peoples tests changing the tags API in the future), and having #52 done gets us a step closer towards using typescript.
About.. #44 - could you elaborate a bit? what do you mean by overcoming it with beforeStep-hook?
Do you have a way of showing the separate steps in Cypress? Without resetting state between them?

@lgandecki
by having beforeStep/beforeScenario you can keep the state by calling
Cypress.Cookies.preserveOnce() in beforeStep and overwrite  Cypress.LocalStorage.clear with a noop in beforeScenario.
Of course this is a hack and we are all waiting for the user-managable state PR https://github.com/cypress-io/cypress/issues/686 :) But it will make your manager happy to see the steps when you are trying to argue against established solutions like cucumber-jvm + selenium.

@sloosch yeah, that sounds like a workable hack :-) I might try it.
Thanks for the input.
Btw, TypeScript works with 1.2.0!
I've added instructions to the readme in 1.2.1
@KeithGillette These are the only changes You'd have to make in your repro to get your example to work:
diff --git a/cypress/plugins/index.js b/cypress/plugins/index.js
index a63eb04..bde8a9a 100644
--- a/cypress/plugins/index.js
+++ b/cypress/plugins/index.js
@@ -9,14 +9,13 @@
 // ***********************************************************

 const cucumber = require('cypress-cucumber-preprocessor').default;
-const cypressTypeScriptPreprocessor = require('./cy-ts-preprocessor');
+const browserify = require('@cypress/browserify-preprocessor');

-// This function is called when a project is opened or re-opened (e.g. due to
-// the project's config changing)
-
-module.exports = (on, config) => {
-  // `on` is used to hook into various events Cypress emits
-  // `config` is the resolved Cypress config
-  on('file:preprocessor', cypressTypeScriptPreprocessor);
-  // on('file:preprocessor', cucumber());
+module.exports = (on) => {
+  on('file:preprocessor', file => {
+      browserify.defaultOptions.browserifyOptions.plugin.push(['tsify'])
+      return cucumber(
+        browserify.defaultOptions
+      )(file);
+  });
 };
diff --git a/cypress/support/step_definitions/facebook.ts b/cypress/support/step_definitions/facebook.ts
index 6c7dbe5..7c01c24 100644
--- a/cypress/support/step_definitions/facebook.ts
+++ b/cypress/support/step_definitions/facebook.ts
@@ -1,4 +1,4 @@
-const {given} = require('cypress-cucumber-preprocessor');
+declare const given;

 const url = 'https://facebook.com';

diff --git a/package.json b/package.json
index 2a979be..bfc37e2 100644
--- a/package.json
+++ b/package.json
@@ -25,7 +25,8 @@
     "@bahmutov/add-typescript-to-cypress": "^2.0.0",
     "@types/node": "^10.3.4",
     "cypress": "^3.0.1",
-    "cypress-cucumber-preprocessor": "^1.0.0",
+    "cypress-cucumber-preprocessor": "^1.2.1",
+    "tsify": "^4.0.0",
     "typescript": "^2.9.2"
   }
 }

Thanks, @sloosch & @lgandecki!
I confirm that the diffs above allow use of some TypeScript with cypress-cucumber-preprocessor. However, I notice that one can't use of TypeScript standard import or export statements. Any remedy for either of those shortcomings?

wow, what an ugly hack!! why would you want to infer 'given' instead of importing the type ?
import and export make the most sense in typescript, solving the problem reported here makes me want to think 🤔 that I don't want to use cypress, ever.


feat(NXP-000): Adding special variant to cozyMarkers

Description
Adding special variant to cozyMarkers:

This variant is needed for the favorite icon coloring:

the print in description is for android or iOS? is there any difference in the design?

the print in description is for android or iOS? is there any difference in the design?

There should be no difference! The print is on Android


Automatic Annotation Inference Failing

My actions before raising this issue

I have been trying to get the automatic annotations up and running following "https://github.com/openvinotoolkit/cvat/blob/develop/cvat/apps/documentation/installation_automatic_annotation.md"
Apparently model is loaded successfully, when I open Cvat Automatic Annotation Option appears and runs but doesn't generate inference.

Following Error is Generated:
Error: Inference status for the task 34 is failed. requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: http://nuclio:8070/api/function_invocations
Expected Behaviour
Should give proposals/bounding boxes around person.
Current Behaviour
Failing
Steps to Reproduce (for bugs)
I am using Ubuntu 20.04
Current Cvat version
Nuctl version --> Client version:
"Label: 1.5.16, Git commit: ae43a6a560c2bec42d7ccfdf6e8e11a1e3cc3774, OS: linux, Arch: amd64, Go version: go1.14.3
Follow the instructions from "https://github.com/openvinotoolkit/cvat/blob/develop/cvat/apps/documentation/installation_automatic_annotation.md"
And error might be reproduced.
I am attaching some log info in text file with respective [commands.](url
Error500_Debug_Logs.txt
)

Duplicate of #2189

Yes! But I don't see the solution,  I have uploaded the required logs there too.

@ZahraAnam Could you please attach logs from nuclio dashboard container as well?

Sure I can do that, umm but am scared I don't know how to get those logs,
can you guide please.
On Mon, Apr 12, 2021 at 9:22 AM Andrey Zhavoronkov @.***>
wrote:

@ZahraAnam https://github.com/ZahraAnam Could you please attach logs
from nuclio dashboard container as well?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openvinotoolkit/cvat/issues/3033#issuecomment-817555806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIIJTCW2X52DFFPGNP6HVLTTIKNTJANCNFSM42B2ADJA
.

--
Best Regards,
Anam Zahra,
PhD Fellow MPI,
Email- @.***

Has this issue been solved? Because I have encountered the same error:
Error: Inference status for the task 12 is failed. requests.exceptions.HTTPError: 504 Server Error: Gateway Timeout for url:
Kindly help!

I am afraid that its not resolved yet
…
On Fri., Jul. 16, 2021, 3:23 p.m. Ashwin George @.***> wrote: Has this issue been solved? Because I have encountered the same error: Error: Inference status for the task 12 is failed. requests.exceptions.HTTPError: 504 Server Error: Gateway Timeout for url: Kindly help! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3033 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIIJTCWCDB3R6FFR5UL26FLTYAXFDANCNFSM42B2ADJA .

Oh! I deployed the yolo-v3 tensorflow model on a remote Linux server.
nuctl version: "Label: 1.6.11, Git commit: da21b912502a41681a6cddb550b56a7ec304ef28, OS: linux, Arch: amd64, Go version: go1.14.3"
But inferencing throws me the error below:
Error: Inference status for the task 12 is failed. requests.exceptions.HTTPError: 504 Server Error: Gateway Timeout for url: http://nuclio:8070/api/function_invocations
It would be great if someone could help!

Yeah! The issue wasn't resolved so I switched to uploading annotations
rather than getting annotations from CVAT
On Fri., Jul. 16, 2021, 3:30 p.m. Ashwin George @.***>
wrote:

I am afraid that its not resolved yet
… <#m_-7195058484065640807_>
On Fri., Jul. 16, 2021, 3:23 p.m. Ashwin George @.***> wrote: Has this
issue been solved? Because I have encountered the same error: Error:
Inference status for the task 12 is failed. requests.exceptions.HTTPError:
504 Server Error: Gateway Timeout for url: Kindly help! — You are receiving
this because you were mentioned. Reply to this email directly, view it on
GitHub <#3033 (comment)
https://github.com/openvinotoolkit/cvat/issues/3033#issuecomment-881447113>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIIJTCWCDB3R6FFR5UL26FLTYAXFDANCNFSM42B2ADJA
.
Oh! I deployed the yolo-v3 tensorflow model on a remote Linux server.
nuctl version: "Label: 1.6.11, Git commit:
da21b912502a41681a6cddb550b56a7ec304ef28, OS: linux, Arch: amd64, Go
version: go1.14.3"
But inferencing throws me the error below:
Error: Inference status for the task 12 is failed.
requests.exceptions.HTTPError: 504 Server Error: Gateway Timeout for url:
http://nuclio:8070/api/function_invocations
It would be great if someone could help!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openvinotoolkit/cvat/issues/3033#issuecomment-881451912,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIIJTCQJDZDEG4EBXLO4IT3TYAX73ANCNFSM42B2ADJA
.

Yeah! The issue wasn't resolved so I switched to uploading annotations rather than getting annotations from CVAT On Fri., Jul. 16, 2021, 3:30 p.m. Ashwin George @.> wrote:
…
I am afraid that its not resolved yet … <#m_-7195058484065640807_> On Fri., Jul. 16, 2021, 3:23 p.m. Ashwin George @.> wrote: Has this issue been solved? Because I have encountered the same error: Error: Inference status for the task 12 is failed. requests.exceptions.HTTPError: 504 Server Error: Gateway Timeout for url: Kindly help! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3033 (comment) <#3033 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIIJTCWCDB3R6FFR5UL26FLTYAXFDANCNFSM42B2ADJA . Oh! I deployed the yolo-v3 tensorflow model on a remote Linux server. nuctl version: "Label: 1.6.11, Git commit: da21b912502a41681a6cddb550b56a7ec304ef28, OS: linux, Arch: amd64, Go version: go1.14.3" But inferencing throws me the error below: Error: Inference status for the task 12 is failed. requests.exceptions.HTTPError: 504 Server Error: Gateway Timeout for url: http://nuclio:8070/api/function_invocations It would be great if someone could help! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3033 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIIJTCQJDZDEG4EBXLO4IT3TYAX73ANCNFSM42B2ADJA .

It's strange! Automated annotation worked for me when I did the same on the local machine.

Close as dublicate


How to remove newlines from expected value in equal assertion?

I have a subchart test that has this basic test:
suite: Service test suite
templates:
  - templates/deployment.tpl
tests:
  - it: Test service deployment
    set:
      global.image.repository: test-repo.com/
      global.image.tag: v99.99.99
    asserts:
      - notFailedTemplate: {}
      - equal:
          path: spec.template.spec.imagePullSecrets[0].name
          value: regcred
      - equal:
          path: .spec.template.spec.containers[0].image
          value: test-repo.com/service:v99.99.99

This tests the deployment's image repo and tag templating:
image: {{ required "Image repo required." .Values.global.image.repository }}service:{{ required "Image tag required" .Values.global.image.tag }}

I call this subchart's test from the umbrella parent chart like this:
helm unittest . --debug -d

The subchart's test always fails because a newline is included in the equal's expected content.
time="2024-04-15T17:00:05-05:00" level=debug msg="expected content: test-repo.com/service:v99.99.99\n" validator=equal
time="2024-04-15T17:00:05-05:00" level=debug msg="actual content: test-repo.com/service:v99.99.99" validator=equal

How do I avoid this newline in the expected assertion?

Closing as I redacted info from the above output for my service name. Where my test included <name>-service and the actual is <name>_service.
Tests work as expected!


[SIP] Improve pivot chart with automatic update and much better export

Please make sure you are familiar with the SIP process documented
here. The SIP will be numbered by a committer upon acceptance.
[SIP] Proposal for ...
Motivation
Improve Pivot 2: we only use this chart, and we come from saiku (+ mondrian) which has been discontinued. So here is some of our experience.
Proposed Changes
Update the pivot chart automatically:
Add an automatic update option to the pivot chart and switch it on by default please. It is a much better user experience to see the results of each change that the user makes to the query right away. The "update chart" button and "your chart is not up to date" banner make you perform extra clicks, add visual clutter, and makes you explore the data in less incremental steps because you'd want to avoid to click on update chart after each and every change you make.
In case the current behaviour would have been chosen in order to limit the number of queries executed in the background: by making it optional, and/or giving a suggestion based on the time the query takes (which is monitored already) to switch the setting off if the dataset doesn't have subsecond latency.
Everybody uses Microsoft Excel: make export to excel nice and easy
Export to excel should be easy. It currently requires 3 clicks "..." > download > "export to pivoted csv". The excel current export is not the pivoted version of the data but the unaggregated source data. So a user has to create the pivot table again in excel, so the only alternative is to use the csv file: which is not meant for normal computer users. The only know excel, so please stop forcing this strange file onto the user. Also Excel doesn't play nice with CSV files, it requires your computer to be correctly configured (wich decimal sign is used on the system?). Finally, if there are multiple attributes in the rows, they are concatenated in the exported CSV file into one column. So the data is not well preserved. A code starting with zeroes is turned into a number, so that the leading zeroes are lost,...
Our users have been used to exporting to excel simply by pressing a single button in Saiku.
So please offer an export to excel:

which doesn't require 3 clicks,
gives the user the pivoted data that he just built and expects, and
preserves the pivoted data as its displayed in the browser: so no concatenation of the row columns, no conversion of numerical strings to numbers wich causes it to lose leading zeroes, ...

If it must be an open standard, use the libre office spreadsheet format, that will already be a huge improvement over CSV.
The Apache Kylin project has incorporated Mondrian into its code base to support XMLA and support pivoting form excel, because the recognised that everybody uses excel, power bi, etc. (For people who have read the Mondrian in Action book, Julian Hyde is among the contributors of the mdx-kylin repo on github) However, kylin is not end-user oriented, and we are currently not planning to start using xmla in excel directly.
New or Changed Public Interfaces

New dependencies

Migration Plan and Compatibility

Rejected Alternatives

Hi there. While we feel your pain and appreciate your willingness to participate in the improvement of Superset, I don't think this qualifies as a SIP. A SIP should be an implementation plan, rather than a group of feature requests. If you want to open these feature requests as GitHub Discussions, you're more than welcome to do so. If you want to implement these changes or propose their design in technical detail, please revise this Issue and we can revisit it.

Hi,
No I do not; that technical detail is not for me, but I can summarise it though.
Automatic chart update: this one is easy

add a setting,
if the setting is on, and the query has been changed, then run it instead of showing an "your chart is not up to date" banner.
test, release, enjoy

Better Export:

create an export to a modern file format (Excel/LibreOffice format) which normal people can use, and that contains the pivoted data exactly like the user sees it in front of him. This is simply "What You See Is What You Get" which is not a new principle.
add an easy to find button for this export so that it is not hidden 3 levels deep in a menu, surrounded by many export options the user probably doesn't want to use because they are not WYSIWYG.
test, release, enjoy

If you are just going to close this, then you will need to keep closing such requests, because they will keep coming. You know more then I do, so open the correct topics.
There are discussions and issues started in 2022 which you can easily find. If you look at them, you can even see that the export to pivoted csv has gotten worse: row attribute columns were not concatenated in the past.

https://github.com/apache/superset/discussions/19705
https://github.com/apache/superset/issues/23374
https://github.com/apache/superset/issues/21854
https://github.com/apache/superset/issues/21337

Fix the root problem: make a new export, and don't invest time in fixing something that is broken: CSV is not a good export format.


add reset to console options

From<EMAIL_ADDRESS>on 2010-09-12T18:07:47Z
Add the IPython %reset functionality to the options of the console.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=353

From pierre.raybaut on 2010-09-22T15:07:37Z
Labels: -Type-Defect -Priority-Medium Type-Enhancement Priority-Low

From<EMAIL_ADDRESS>on 2010-09-12T16:08:20Z
a very "smart" reset would offer to "suspend" the current variables for the next command and then restore them

From pierre.raybaut on 2011-05-16T01:10:55Z
Labels: Cat-Console

From pierre.raybaut on 2011-10-29T04:35:41Z
Status: WontFix


Fix bug in pyarrow and hdfs

Small change to fix error in pyarrow + hdfs.
dd.read_parquet("hdfs://some-hdfs-hostname/" + filename)
would raise TypeError: update_hdfs_options() got an unexpected keyword argument 'host'

[ ] Tests added / passed
[x] Passes flake8 dask

Good spot; does this mean we are not running HDFS/arrow tests? @jcrist


pipe start throw exception,and rs2_get_librealsense_exception_type return 720

My camera info is here:

I use vs2015 in win10,and librealsense SDK is 2.0
Issue Description
code is like this:
rs2::pipeline *temp_pipe = new rs2::pipeline(ctx);
rs2::config temp_cfg;
temp_cfg.enable_stream(RS2_STREAM_DEPTH, 1280, 720, RS2_FORMAT_Z16, 30);
temp_cfg.enable_stream(RS2_STREAM_COLOR, 1280, 720, RS2_FORMAT_BGR8, 30);
temp_cfg.enable_device(temp_sn);
temp_pipe->start(temp_cfg);  //this line throw a exception and rs2_get_librealsense_exception_type  return 720
when comment all temp_cfg.enable_stream line,temp_pipe->start(temp_cfg)  success,why?
What should i do to fix this problem?

@barcelona-delpy hello,
What is the actual camera type that you're using - is it D430 or D435(i) ?  Can you attach a screenshot ?
In the above screenshot the camera is recognized as D430, a model w/o RGB sensor. If that's correct then the following line in the code is not applicable

temp_cfg.enable_stream(RS2_STREAM_COLOR, 1280, 720, RS2_FORMAT_BGR8, 30);

You can list the supported streaming resolutions and formats with rs-enumerate-devices -m

Hi @barcelona-delpy,
Please provide us with following information about the configuration you have experienced issues with:
RealSense SDK 2.0 build version
Camera type (D430/ D435/ D435i/etc.)
Thank you,
David

@barcelona-delpy hello, do you have the requested data for analysis?
Please update


boot.initrd.luks: add input_leds module

To get working caps lock lights already at stage 1, the input_leds module needs to be loaded.
Closes #12456.
/cc @IreneKnapp
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval

Thanks so much for fixing this! :)


Update modules to reflect DSL2

Updated modules.md to reflect DSL2 with a warning about the end of life of DSL1. Also added basic information about what DSL means. This is related to issue https://github.com/nf-core/nf-co.re/issues/1631

overwrite https://github.com/nf-core/nf-co.re/pull/1715


Documentation: explain "ticking"

I suggest adding some documentation on the concept of ticking.

Why and how it is used
How it is detected in common usage

This could be a section on the Getting Started guide (see #65)

Yes, covering this in getting started :)

Added some notes about ticking in first steps page, closing this issue as complete.


feat(avm): new public inputs witgen

Please read contributing guidelines and remove this line.

#10179  👈 (View in Graphite)
master

This stack of pull requests is managed by Graphite. Learn more about stacking.


Update 01-routing.md

Made the word "route" plural on line 9.

This has come up numerous times. I agree with you because I'm American. But the original author of this project is from somewhere where "route" is plural

Thanks for the clarification, @benmccann.


Add NixOps

It would be great to build virtual machines for each of my devices locally before completely building the configuration, enabling me to test things out.
Bonus points for infrastructure that allows me to easily deploy to servers and other things. This may require some changes in configuration for my server setup.

https://github.com/lovesegfault/nix-config/blob/master/nix/deploy.nix

https://github.com/lovesegfault/nix-config/blob/master/nix/deploy.nix

https://justinas.org/nixos-in-the-cloud-step-by-step-part-1

https://justinas.org/nixos-in-the-cloud-step-by-step-part-1

this is kind of overkill right now, and there are much better services available for use than those linked above.


Removed unsafe

Original from #20:

This pr resolves https://github.com/ClueLang/Clue/issues/15.
A lot of things needed to change but for the better.
First of all, I needed to add the lazy_static dependency, I used it to instantiate and hold all the data.
Then, I implemented a struct containing all the data and getters for it, that struct has even a convenient function for adding things to the output code.
I wrapped that struct inside a RwLock, if in the future the need arises (the other option was Mutex).
I created lib.rs that contains mods and macros and removed the arg macro as it is useless at this point (there's no more need for unsafe)
I added a configuration file for clippy because it was angry that EnvData::set_data had 10 arguments, 3 more of the max he tolerated.

the pr works, but it seems to have somehow re-enabled the snake case forcing so now it spams a ton of warnings every time I try to compile

I'll try a quick fix

Done


Add Dark Theme

This PR closes #29
I added a dark theme via flutters built in mechanism for this. The app will switch theme according to the phones system settings for dark themes.
I removed some hard coded color references for text which would otherwise break in dark mode and changed some colors to the accent color so they would look nice in both modes.

Screenshots

Thank you very much for your contribution!! 😊
I will properly check this out on the weekend and then maybe directly do a release!

Waouh it's beautiful !!
Thank you @SeineEloquenz !


Enhanced 'MenuButton' component

I was unable to figure out how to use the  MenuButton component.  It seems to have a very different API than what I would expect for a wrapped up control.  In particular, it requires a Model be provided, rather just dropping in data for its items.
I understand that our Menu API is extremely mixed, but it would be nice if this control were a little simpler to drop in.
(Also could use a toolbox example of this)

Update -- I see now that I need to actually place and render the menu myself.  So the button really doesn't give me too much, other than a consistent way to render itself, and the little click handler to open the menu!

The comment kind of says it all:
Convenience Button preconfigured for use as a trigger for a dropdown menu operation.

We have a bunch of 'preconfigured' buttons like this, across desktop and mobile, which are simply there to offer a consistent way to render and a click handler (e.g. FeedbackButton, OptionsButton, LogoutButton)

Yes, my bad for not remembering that and wasting time.  But the other three are really appropriate to be just simple triggers and can stay like that.  There is no other code needed.
Applications really need something like a MenuButton that takes arbitrary menu items and does the work for you of placement etc.!  This is similar to our SplitButton project on Desktop, and could perhaps be folded in together in terms of adding something with consistent naming.

Definitely agree that a component that does the work of creating and placing the menu for you would be much better - I'll see what can be done!

Is this all specific to mobile?  Note https://github.com/xh/hoist-react/pull/1800 for earlier discussion from last spring about building a desktop MenuButton (as a more general / first-order add to support notion of a SplitButton).

Is this all specific to mobile?  Note https://github.com/xh/hoist-react/pull/1800 for earlier discussion from last spring about building a desktop MenuButton (as a more general / first-order add to support notion of a SplitButton).

Tom and I discussed this in-detail, and I gave him a few minor suggestions.  Some big points we agreed on:

We are going to have to live with the bifurcation between mobile/desktop menus and menuitems for a while, maybe long-term.  But this change will really make a big step in the right direction.

We need general popover like support on mobile to support apps like the big client mobile app we are currently working on. This is a seperate (but critical) project, and Tom is going to make a ticket for that.

Tom and I discussed this in-detail, and I gave him a few minor suggestions.  Some big points we agreed on:

We are going to have to live with the bifurcation between mobile/desktop menus and menuitems for a while, maybe long-term.  But this change will really make a big step in the right direction.

We need general popover like support on mobile to support apps like the big client mobile app we are currently working on. This is a seperate (but critical) project, and Tom is going to make a ticket for that.

To answer your question ATM, we think it would be good to keep MenuButton API similar to what we think might be a SplitButton API, but no plans to make this cross-platform at this time.

To answer your question ATM, we think it would be good to keep MenuButton API similar to what we think might be a SplitButton API, but no plans to make this cross-platform at this time.


WSL - bash: the command is repeated in PROMPT_COMMAND in nested shells

Regarding https://docs.microsoft.com/en-us/windows/terminal/tutorials/new-tab-same-directory#bash: It is suggested to put the snippet in .bashrc that is evaluated on start of every bash process, even nested ones. That causes repeated appending to PROMPT_COMMAND variable.
Who about suggesting to use .bash_profile config file?

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

ID: 439baddf-3ec9-a46e-1b6e-3ec47cbd1d4c
Version Independent ID: 439baddf-3ec9-a46e-1b6e-3ec47cbd1d4c
Content: Opening a tab/pane in the same directory
Content Source: TerminalDocs/tutorials/new-tab-same-directory.md
Product: dev-environment
Technology: windows-terminal
GitHub Login: @zadjii-msft
Microsoft Alias: migrie

Thanks @happylynx. Seems like a good recommendation to me. Updated in https://github.com/MicrosoftDocs/terminal/commit/2562bf53f5b13a224c703fe6645ff6f397f1d242


Specifying target of Global results in error.

When I run this command:
python3 sumologic_backup.py -c sumologic-backup.initial.cfg -t Global -o backups -z sumo-backup.zip

I receive this error:
Traceback (most recent call last):
  File "sumologic_backup.py", line 516, in <module>
Step-001: - Authenticating
Step-002: - Creating Supporting directories
Step-003: - Discovering content targets
    main()
  File "sumologic_backup.py", line 342, in main
    _content_manifest = create_content_map(source)
  File "sumologic_backup.py", line 250, in create_content_map
    uid_parent = content_list['parentId']
KeyError: 'parentId'

Personal backup works fine.   Only a Global target fails.
User account has Administrator rights in Sumo and can view all searches.

I confirmed this fixed the issue, thank you!


docs(ast): enforce doc comments on generated ASTBuilder methods

Part of https://github.com/oxc-project/backlog/issues/130

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6714
#6713  👈
#6712
#6711
#6710
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite

Merge activity

Oct 20, 10:01 PM EDT: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..


The program seems to lose the path of my steam libraryfolders.vdf

libraryfolders lost!

Today I deleted all save files in "My game" folder including "Titan Quest ", "Titan Quest - Immortal Throne" and "Titan Quest Backup".
After that, I restarted my game to rebuild save files.
Unfortunately, I find that TQVaultAE not working....
It indicated that can't find a part of this path"X:\Grim Dawn\Steam\SteamApps\libraryfolders.vdf"
By the way, my Grim Dawn is installed in X:\Grim Dawn.(It's not downloaded from steam)

My steam libraryfolders is not in that path at all!(It's "E:\Steam\steamapps\libraryfolders.vdf")
Then I copy my libraryfolders.vdf(file in real path) to the false path the messagebox mentioned(I created new folder "Steam" and "SteamApps")
it worked again...
Can anyone tell me how to fix this problem?

I have debugged the code,and find the the program get a wrong steam path from a registry function
Maybe my registry is a mess or the function is not perfect
If there is no better idea to solve this problem, I'd like to copy the libraryfolders.vdf temporarily.

This is a known problem people have with the auto-detection of the path. To fix it, tell TQVaultAE where it can find the game files manually: See The F.A.Q., section "Error Loading Resources. This may be caused by a bad language or game path setting."

Will close as a duplicate of https://github.com/EtienneLamoureux/TQVaultAE/issues/80
Fell free to keep commenting for that problem in the main issue.


SecRule with nolog is still logging.

Describe the bug
We have created a SecRule to allow a certain type of request, however despite the rule working the below is still logged into the nginx error log, this is happening in DetectionOnly mode, so it's hard to know the rule we have added has actually worked without setting SecRuleEngine to On, again where it logs however is not blocked.
Logs and dumps
2021/09/02 08:21:12 [info] 45611#45611: *691956 ModSecurity: Warning. Matched "Operator `Rx' with parameter `(?:^|[\\/])\.\.(?:[\\/]|$)' against variable `ARGS:reference.refAddress1' (Value: `..' ) [file "/opt/openresty/site/modsecurity/coreruleset/REQUEST-930-APPLICATION-ATTACK-LFI.conf"] [line "52"] [id "930110"] [rev ""] [msg "Path Traversal Attack (/../)"] [data "Matched Data: .. found within ARGS:reference.refAddress1: .."] [severity "2"] [ver "OWASP_CRS/3.3.0"] [maturity "0"] [accuracy "0"] [hostname "<IP_ADDRESS>"] [uri "/Applicants/Reference_Edit"] [unique_id "25764-c64035d41d54aeec7c1c707399424d62"] [ref "o0,2v236,2"], client: <IP_ADDRESS>, server: example.com, request: "POST /Applicants/Reference_Edit?refid=925A65C4654A926053FA05568BA67265 HTTP/1.1", host: "example.com"
2021/09/02 08:21:12 [info] 45611#45611: *691956 ModSecurity: Warning. Matched "Operator `Ge' with parameter `5' against variable `TX:ANOMALY_SCORE' (Value: `5' ) [file "/opt/openresty/site/modsecurity/coreruleset/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "80"] [id "949110"] [rev ""] [msg "Inbound Anomaly Score Exceeded (Total Score: 5)"] [data ""] [severity "2"] [ver "OWASP_CRS/3.3.0"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"] [hostname "<IP_ADDRESS>"] [uri "/Applicants/Reference_Edit"] [unique_id "25764-c64035d41d54aeec7c1c707399424d62"] [ref ""], client: <IP_ADDRESS>, server: example.com, request: "POST /Applicants/Reference_Edit?refid=925A65C4654A926053FA05568BA67265 HTTP/1.1", host: "example.com"

SecRule ARGS_NAMES "@contains reference.refAddress1" "id:7, phase:2, t:none, nolog, pass, ctl:ruleEngine=Off"

To Reproduce
curl --resolve example.com:443:<IP_ADDRESS> https://example.com/Applicants/Reference_Edit?refid=1234 -d 'reference.refAddress1=..'

Expected behavior
I would expect for this request to be allowed, but also respect nolog
Server (please complete the following information):

modsecurity-nginx-v1.0.1 / ModSecurity v3.0.4
nginx version: openresty/<IP_ADDRESS>
CentOS Linux release 7.9.2009 (Core)

Rule Set (please complete the following information):

OWASP CRS 3.3.0

Additional context

Depends on where you have added your own rule. If the CRS is being loaded and processed before your rule, it would match before your rule and will be logged. Also, I think instead of pass, you need to use allow so stop further processing of the rules if that is what you want.
Or you need to use SecRuleUpdateTargetById
SecRuleUpdateTargetById 930110 !ARGS:reference.refAddress1

Hi @ledzepp4eva
Just to add one extra note of clarification here, when you say:

I would expect for this request to be allowed, but also respect nolog

The nolog action means 'do not perform the logging for the current rule'.  It does not mean 'perform no logging for this transaction'.
The log excerpt that you have included is consistent with this.  Rules 930110 and 949110 have been logged but rule 7 (which has the nolog action) has not been.

Was there anything more on this?  If not, I will proceed to close this issue.

Apologies. Feel free to close it and thank you all for the feedback.


Sinatra::Namespace does not prefix redirect() nor url()

I would expect the redirects to be prefixed like the routes.
namespace '/admin' do
    get '/a' do
        redirect '/b'  # is not redirecting to '/admin/b' but to '/b'
    end

    get '/a' do
        redirect to('/b')  # is not redirecting to '/admin/b' but to '/b'
    end
 end

:prefixed_redirects raises an error
namespace '/admin' do
    enable :prefixed_redirects
    get '/somewhere'
 end

 # => may not set prefixed_redirects (ArgumentError)

I don't know where this should be fixed
module Sinatra::Namespace::NamespacedMethods
    prefixed :to
end

didn't help either.

ok, I digged into it. It seems the following should do the trick:
module Sinatra::Namespace::InstanceMethods
  def redirect(uri, *args)
    prefix = @namespace.instance_variable_get("@pattern")
    super("#{prefix}#{uri}", *args)
  end
end

But I guess you would organize it some other way to fit into your code.

same thing for url()
module Sinatra::Namespace::InstanceMethods
  def url(uri, *args)
    prefix = @namespace.instance_variable_get("@pattern")
    super("#{prefix}#{uri}", *args)
  end
end

ok, so it seems that if we use redirect to() we could have the choice:
module Sinatra::Namespace
  module InstanceMethods
    def url(uri, *args)
<EMAIL_ADDRESS>*args)
    end

    def to(uri, *args)
<EMAIL_ADDRESS>*args)
    end
  end

  module NamespacedMethods
    def pattern
      @pattern
    end
  end
end

and then we could use it like this:
class App < Sinatra::Base
  register Sinatra::Namespace

  namespace '/admin' do
    get '/a' do
      "A"
    end

    get '/c' do
      redirect to('/a')   # goes to /admin/a
    end
  end

  namespace  '/badmin' do
    get '/b' do
      "B: #{url('/d')}"
    end

    get '/c' do
      redirect '/admin/a'  # goes to /admin/a
    end
  end
end

Thanks for this! I'll see about adding some specs to cover the cases you've identified and adding support for them. Any other weirdness you've noticed with using namespaces? (Excluding what's covered in #28.)

This needs an (actual) failing test and patch before it can be fixed.

@zzak I added a method called "redirect_to" in the #180 that allows redirects within the namespace.

This can be closed by #180 which included specs for redirect in a namespace


Minor Grafana Dashboard Cleanup

Minor dashboards cleanup after running against large clusters.

https://github.com/oracle/coherence-operator/pull/270


Databricks stream processing

https://docs.databricks.com/_static/notebooks/structured-streaming-python.html

use this data for streams - https://github.com/j143/nyc-taxirides-stream-feeder


Add documentation

Pull Request Test Coverage Report for Build 21

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 19:
0.0%

Covered Lines:
800

Relevant Lines:
800

💛  - Coveralls


adding Footer Emoji

Description
changed :love: to ❤

...
...
...

Related issue(s)

#264

Hey @Souvikns! Thanks for taking the time! The :love: word was like that on purpose to make it look more code-ish 😄
Let's wait for others but I'd not change it.

tbh I always thought it is broken but never had time to jump in to fix 😆 this is why I wrote @Souvikns to go ahead and fix it

I also think :love: was intentional

Actually, if it wasn't intentional, it wouldn't have the  tag wrapping it 😄

@fmvilas you think we do not believe you it was intentional? 😄 It's just that nobody inspects the website to double-check if what they see is a bug or not 😄 at least not me 😆
how about we add a tooltip to it 🤣

Hahaha! That would be fun. Let's do it!

Yeah, I also had a hunch that it was intentional. You can say I was just trying to see how it looks. 😊

I guess we are closing this one?


Doc Feedback:Suggest rewriting of this document

Location : https://apim.docs.wso2.com/en/latest/Learn/APIController/building-a-cicd-pipeline/
Document is all over the place and confusing, suggest a rewrite on the documentation with proper syntaxes and examples. A user who is new to WSO2 or even an existing user finds the document very confusing.

@nadeesha5814 , the updated content can be found in the following URL:

https://apim.docs.wso2.com/en/latest/learn/api-controller/ci-cd-with-wso2-api-management/
https://apim.docs.wso2.com/en/latest/learn/api-controller/building-jenkins-ci-cd-pipeline-for-dev-first-approach/


Update all labels with Weight to contain unit - "Weight [kg]

https://sd-vxnaid.atlassian.net/browse/VRVU-167
cc @jdolkowski  @pwargulak  @druchniewicz

sharing a clearn PR.


Request soon after premature close can result in database connection leak

A client can cause a premature close. For example, if you request an OData feed using curl, then kill curl as the response is being received, then there may be a premature close. (The OData feed may have to be on the larger side for this to work.) A premature close logs an error, but by itself, it doesn't seem to cause an issue. For example, you can take the following steps:

Run select count(*) from pg_stat_activity where state <> 'idle'. The count should be 1 (corresponding to this query itself).
Request an OData feed and cause a premature close.
Immediately run the count query again. The count should be 2.
Wait a few seconds, then run the query again. The count should be back to 1.

However, if something (either a request or a worker) attempts to acquire a database connection soon after the premature close, things can go wrong. The OData feed uses a Slonik stream, and it looks like if the connection pool gives out the same connection that the stream was using, that becomes an issue. To see this, specify the Slonik interceptors option to createPool():
interceptors: [{
  afterPoolConnection: ({ connectionId }) => {
    console.log('using connection', connectionId);
  },
  beforePoolConnectionRelease: ({ connectionId }) => {
    console.log('releasing connection', connectionId);
  }
}]

Now take the following steps:

Request an OData feed and cause a premature close.
Immediately after, send another request. (You may find it easiest to send the same request, but any request should reproduce.)
If the stream's connection has been given out, the request will not receive a response.

You may have to try a few times to see this behavior. You'll see logs that look like this:
using connection HSEiX93XRJK1gDyV70K_Cg-22
releasing connection HSEiX93XRJK1gDyV70K_Cg-22
using connection HSEiX93XRJK1gDyV70K_Cg-22
releasing connection HSEiX93XRJK1gDyV70K_Cg-22
using connection HSEiX93XRJK1gDyV70K_Cg-22
releasing connection HSEiX93XRJK1gDyV70K_Cg-22
using connection HSEiX93XRJK1gDyV70K_Cg-22
releasing connection HSEiX93XRJK1gDyV70K_Cg-22
using connection HSEiX93XRJK1gDyV70K_Cg-27
using connection HSEiX93XRJK1gDyV70K_Cg-29
releasing connection HSEiX93XRJK1gDyV70K_Cg-29
::1 - - [14/May/2022:14:51:29 +0000] "GET /v1/projects/1/forms/foo.svc/Submissions HTTP/1.1" 200 -
releasing connection HSEiX93XRJK1gDyV70K_Cg-27
Error [ERR_STREAM_PREMATURE_CLOSE]: Premature close
    at ServerResponse.onclose (internal/streams/end-of-stream.js:121:38)
    at ServerResponse.emit (events.js:412:35)
    at ServerResponse.emit (domain.js:470:12)
    at Socket.onServerResponseClose (_http_server.js:221:44)
    at Socket.emit (events.js:412:35)
    at Socket.emit (domain.js:470:12)
    at TCP.<anonymous> (net.js:675:12)
using connection HSEiX93XRJK1gDyV70K_Cg-27

In the logs above, HSEiX93XRJK1gDyV70K_Cg-27 is the ID of the connection that stream() used. The connection is given out to the next request, which seems to be unable to use it.
You may have to try the steps above several times before reproducing this behavior. Once you do, run the count query from above. The count is now 2 and as far as I can tell, never goes back down, indicating a connection leak. Even if you kill the second request, the count remains at 2.
If you repeat this process enough times such that the count reaches 11, the connection pool will be full. At that point, Problem responses will be returned with the message Completely unhandled exception: timeout exceeded when trying to connect.
I believe that this is an upstream issue and have filed an issue with Slonik.
(By the way, I think that the latest version of Slonik may pass slightly different information to the interceptor methods, so I'm not sure that the interceptor above would work as-is in the latest version.)

https://github.com/gajus/slonik/issues/347

Closed by #604.


Mbed library Lpc11u68 support

Can you add support for this device in the library?
Thank you

The LPC11U68 is already supported platform.
https://developer.mbed.org/platforms/LPCXpresso11U68/

Sorry, my fault, I thought this was the RPC library github. I was try to compile the RPC library (https://developer.mbed.org/cookbook/RPC-Interface-Library) for LPC11U68 giving me errors in the parse_pins.cpp file of the library. And reading that file, my platform is not added.
Thank you.

ARM Internal Ref: IOTMORF-189


Optimized code in injector and added a possibility for method chaining

We use a fork of your great library with some patched code which I now committed in this PR. All tests are green.

Optimized code in injector (more concise and faster for some methods).
Added a possibility to chain method calls using cascade operator.

Thanks for accepting this PR in advance. I would be happy if you could release a new version of this lib.

Thanks for this, the code looks good.  I'll have a bit of time to merge this in tomorrow and do a new release.
Is this test still passing?
https://github.com/jonsamwell/flutter_simple_dependency_injection/blob/master/test/flutter_simple_dependency_injection_test.dart#L199

Yes, test is still passing of course. Many thanks too!
Jon Samwell<EMAIL_ADDRESS>schrieb am Fr., 23. Okt. 2020, 10:40:

Thanks for this, the code looks good. I'll have a bit of time to merge
this in tomorrow and do a new release.
Is this test still passing?
https://github.com/jonsamwell/flutter_simple_dependency_injection/blob/master/test/flutter_simple_dependency_injection_test.dart#L199
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jonsamwell/flutter_simple_dependency_injection/pull/15#issuecomment-715193478,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF3KZS6DRLWKZLOIGWAJPDSME6QPANCNFSM4SZYMHJQ
.

@jonsamwell I've updated this pull request. The Injector class has a factory constructor now. An Injector instance can be instantiated more shorter. "Injector.getInjector()" is marked as deprecated as discussed here https://github.com/jonsamwell/flutter_simple_dependency_injection/issues/16
I've updated CHANGELOG too.
Looking forward for a new release!

@ova2 many thanks for this PR.  Released as version 1.0.4

@jonsamwell Many thanks for release too.


Uploading folders using the file manager

Is your feature request related to a problem? Please describe.
I cannot upload any folders to the file manager, and SFTP doesn't work.
Describe the solution you'd like
To be able to upload folders to the file manager.
Describe alternatives you've considered
There are no alternatives to this.

What are the permissions of the server folder/files?
PufferPanel needs to own these, and tampering with them manually frequently leads to breakages like this.

I have installed pufferpanel and it has created the files, but I wish to upload my own folders to the file manager. It should own the server files, but I want to replace them with my own.

The SFTP implementation is reliable, any breakages are likely from tampering with files, running the panel in an unsupported manner or rarely an unsupported client.
For support issues, join the discord.

Just to make it clear, the web file manager does not support folders and probably will not any time soon. Browsers are not good at file handling in the best of cases and completely insane in the worst. Uploading a folder is completely unsupported by browsers and even letting a user select a folder is only somewhat possible in non-standard ways bringing along tons of headaches.
The built in SFTP exists because browsers are bad at being file managers and it usually works perfectly fine so long as no firewall is blocking the port (5657 by default) and you copy the credentials from the panel correctly.

Just to make it clear, the web file manager does not support folders and probably will not any time soon. Browsers are not good at file handling in the best of cases and completely insane in the worst. Uploading a folder is completely unsupported by browsers and even letting a user select a folder is only somewhat possible in non-standard ways bringing along tons of headaches.
The built in SFTP exists because browsers are bad at being file managers and it usually works perfectly fine so long as no firewall is blocking the port (5657 by default) and you copy the credentials from the panel correctly.

I tried to use SFTP, but I get an error that my password is invalid. I use my panel password.


Bug when trying to make an analysis no longer public

Found on qa.clinepidb.org. I logged in, went to the "My Analyses" page, and tried to remove a public analysis I had made previously by clicking the checked box under the "Public" column. There was no response to clicking the checked box. If I clicked the empty box on another analysis, I would see the expected pop-up to make an analysis public.
I was able to delete the public analysis, but we want to make sure users can remove an analysis from the public list without deleting it entirely

Works nicely, thanks!


feat: add kms_master_key_id to alarm baseline and config-baseline module

In this PR I added possibility to use KMS key for encryption of an SNS Topics used by modules alarm-baseline and config-baseline.
Set alarm_sns_topic_kms_master_key_id parameter with value of either KMS key ID or it's Alias name.
Set config_sns_topic_kms_master_key_id parameter with value of either KMS key ID or it's Alias name.
Default values:
alarm_sns_topic_kms_master_key_id  = null
config_sns_topic_kms_master_key_id = null

@mknapcok Thanks for sending a PR, I agree that enabling SNS topic encryption is a way to go. I left some review comments, could you take a look?

@mknapcok would you be able to update the READMEs and add in an example of the creation of the kms keys you're passing in for the sns_topics with policies?

For config-baseline it may be beneficial to use a multi-region kms key, but not available in terraform just yet. https://github.com/hashicorp/terraform-provider-aws/issues/19896

I'm closing this PR in favor of #216 for now. We'll continue working on this issue with the multi region CMK after it's implemented in terraform-aws-provider.


Document/Add more test cases to WalletLib

Right now there are a good amount of test cases for the Wallet Library but it could use more test cases for better code coverage.

Document our current suite of test in order to determine what test cases we have covered

Use this information to reveal any states that our tests are not covering

Include tests that use common solidity exploits to ensure our Wallet is resistant to them.

Expand the tests to include all of these cases

Speak with Chris or Josh for help on the WalletLibraries or testing framework.  We are happy to help. :)

I'm on it :)

Completed.


[Feature]: add support for roller blinds (probably equal to curtain / cover) support through Gateways (sub-devices)

EDIT: I have this working using a fork of localtuya and shared my setup-details here
I'd like to ask for roller-blind support to be added to tuya-local.
The roller-blind-motors used are branded by enjoy Motors and connect wirelessly (RF 433) to a Gateway (GW03 / GWE) as sub-devices.
Currently I am having issues adding the Gateway with the following error:
Unable to connect to your device with those details. It could be an intermittent issue, or they may be incorrect. and a  log entry like this
However I tried the pytuya_test.py which I found on another thread on Github (tuya local) script using host, device_id, local_key, cid and protocol 3.3 to receive the following results (so I guess it should work somehow):
Information on the devices:
Gateway:
{
  "result": {
    "active_time":<PHONE_NUMBER>,
    "category": "wg2",
    "category_name": "Gateway",
    "create_time":<PHONE_NUMBER>,
    "gateway_id": "",
    "icon": "smart/icon/ay1521165954459sVl6M/5bd48aa970b762f6bf4687af1f52a2da.png",
    "id": "bf4XXXXXXXXXXXXXXXX4vw",
    "ip": "XXX.XXX.XXX.XXX",
    "lat": "00.0000",
    "local_key": "XXXXXXXXXXX",
    "lon": "0.0000",
    "model": "GW02,GW03",
    "name": "Gateway ",
    "online": true,
    "owner_id": "XXXXXXXX",
    "product_id": "q3rrd31ntit0m2fw",
    "product_name": "Gateway.e ",
    "sub": false,
    "time_zone": "+02:00",
    "update_time":<PHONE_NUMBER>,
    "uuid": "a96XXXXXXXXXX0f8"
  },
}

Motor (subdevice => true, cid === node_id => 0CXXXX ):
{
  "result": {
    "active_time":<PHONE_NUMBER>,
    "category": "cl",
    "category_name": "Curtain",
    "create_time":<PHONE_NUMBER>,
    "gateway_id": "bf4XXXXXXXXXXXXXXXX4vw",
    "icon": "smart/icon/ay1521165954459sVl6M/95ef6a9e597c0700f6898944e7132ed3.png",
    "id": "bf7XXXXXXXXXXXXXXXXynd",
    "ip": "",
    "lat": "00.0000",
    "local_key": "7c4ba6e0d1577749",
    "lon": "0.0000",
    "model": "GW Motor",
    "name": "TESTROOM",
    "node_id": "0CXXXX",
    "online": true,
    "owner_id": "XXXXXXXX",
    "product_id": "slrxhmozdm3qq9cl",
    "product_name": "Roller blind",
    "sub": true,
    "time_zone": "+02:00",
    "update_time":<PHONE_NUMBER>,
    "uuid": "bf7XXXXXXXXXXXXXXXXynd"
  },
}

Motor Device Specification:
{
  "result": {
    "category": "cl",
    "functions": [
      {
        "code": "control",
        "lang_config": {
          "close": "DOWN",
          "open": "UP",
          "stop": "STOP"
        },
        "name": "Curtain control",
        "type": "Enum",
        "values": "{\"range\":[\"open\",\"stop\",\"close\",\"continue\"]}"
      },
      {
        "code": "percent_control",
        "lang_config": {
          "unit": "percentage"
        },
        "name": "Favorite position",
        "type": "Integer",
        "values": "{\"unit\":\"%\",\"min\":0,\"max\":100,\"scale\":0,\"step\":1}"
      }
    ],
    "status": [
      {
        "code": "control",
        "lang_config": {
          "close": "DOWN",
          "open": "UP",
          "stop": "STOP"
        },
        "name": "Curtain control",
        "type": "Enum",
        "values": "{\"range\":[\"open\",\"stop\",\"close\",\"continue\"]}"
      },
      {
        "code": "percent_control",
        "lang_config": {
          "unit": "percentage"
        },
        "name": "Favorite position",
        "type": "Integer",
        "values": "{\"unit\":\"%\",\"min\":0,\"max\":100,\"scale\":0,\"step\":1}"
      }
    ]
  },
}

...and from the debugger:
code: control, type enum, values:
{
  "range": [
    "open",
    "stop",
    "close",
    "continue"
  ]
}

code: percent_control, type Integer, values:
{
  "unit": "%",
  "min": 0,
  "max": 100,
  "scale": 0,
  "step": 1
}

Roller blinds should be cover devices, the same as curtains or garage doors that are already supported. However, the underlying tuya library does not currently support gateways and sub devices, so that will be a blocking point for this particular device for now.

The details given do not include any local protocol details, so this issue will be closed with the addition of general subdevice support.


Any plan to support s390x

Since 1.6.0 alpha 1, s390x is included into Kubernetes official build, any plan for cube-prometheus to support s390x?

This is not on our roadmap, but contributions are welcome. Realistically almost all components are statically compiled go programs that do not use c libraries, so it should technically be possible to cross compile all the components.

I found image based on busybox:uclibc, while there is no busybox:uclibc on s390x, is it to use busybox:glibc to replace busybox:uclibc? Any specific reason to busybox:uclibc except the image size?

Which image exactly are you talking about, in specific cases it is better to address these things directly in the upstream repository that the images are released from, rather than out of this repository. Feel free to leave status reports here in form of a check list and then open issues on the particular repository when you have questions like those.


Reorganize Eth1-specs Repo & Add Network Upgrades Info

Two big changes in this PR (I know...):

Change the top-level README & folder structure to make it clear that more than network upgrades spec are tracked here (specifically, the JSON RPC spec);
Clarify the CFI process by including some information from a previously opened PR.

Given that https://github.com/ethereum/pm/issues/295 and https://github.com/ethereum/eth1.0-specs/issues/23 are still opened, I did not clarify where CFI is being tracked, or whether it resets every upgrade. Based on the outcome of these two issues, more changes will be necessary, but I think merging this is a net improvement.

@poojaranjan addressed the bulk of your comments in the last commit. Still two open threads to discuss.
Re: the diagram, if we have an updated one, yes, we should add it We can also add it in a subsequent PR if we don't have one now.

@timbeiko
I'm fine with making the edits later on. Comments resolved.
Diagram: I have got the updated diagram here. One major change from the earlier version, CFI approved is moved to CFI phase. Looks like it was mistakenly shown in the Devnet phase. If you'd rather like to have only one CFI instead of applied and approved, I can make the edits, let me know.

Thanks @poojaranjan just added the diagram in a separate commit 😄 !


[KYUUBI #3420] Spark sql engine register self to Discover Server with Spark Web UI uri

Why are the changes needed?

see detail in KYUUBI #3436
Splitting it out of a giant PR https://github.com/apache/kyuubi/pull/4002 to perfect
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

cc @pan3793 @turboFei @zhaohehuhu

Codecov Report

Merging #4459 (69292ab) into master (cc4ec5a) will decrease coverage by 0.05%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##             master    #4459      +/-   ##
============================================
- Coverage     53.26%   53.22%   -0.05%
  Complexity       13       13
============================================
  Files           569      569
  Lines         31182    31186       +4
  Branches       4211     4213       +2
============================================
- Hits          16610    16599      -11
- Misses        12995    13007      +12
- Partials       1577     1580       +3

Impacted Files
Coverage Δ

...ubi/engine/spark/SparkTBinaryFrontendService.scala
81.60% <80.00%> (-0.32%)
:arrow_down:

...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala
59.09% <0.00%> (-7.58%)
:arrow_down:

.../kyuubi/server/mysql/constant/MySQLErrorCode.scala
13.84% <0.00%> (-6.16%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLCommandHandler.scala
77.77% <0.00%> (-4.05%)
:arrow_down:

...apache/kyuubi/engine/JpsApplicationOperation.scala
77.41% <0.00%> (-3.23%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLGenericPackets.scala
76.59% <0.00%> (-2.13%)
:arrow_down:

...apache/kyuubi/service/TBinaryFrontendService.scala
49.46% <0.00%> (+1.07%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Gentle ping @turboFei, would you have other suggestions?

https://github.com/apache/kyuubi/pull/4541
how do you think about this? @zwangsheng

Thanks, merged to master


Output results to a panel

I'd love if Sublime Linter could output errors and warnings to a panel at the bottom of the window, a bit like how SublimeGit displays commit output.

What are the chances of this being possible?

Would need to be implemented in https://github.com/SublimeLinter/SublimeLinter3


Control Typography memory usage

CSharpMath uses a lot of memory via Typography:

Possible improvements would be some combination of loading glyphs lazily and storing them more efficiently.
Can this be improved from within CSharpMath or should this be an issue in https://github.com/LayoutFarm/Typography ?

I think  some objects could be unloaded/dispose
eg. List could be unload after you create a glyph outline from it.
but this I have to test it first

please visit and test 'Trimmable features' again, see=> https://github.com/LayoutFarm/Typography/issues/187

So there are two ways forward with the Typography submodule:

Drop LayoutFarm/Typography#192 and update
Merge latest Typography into LayoutFarm/Typography#192 and resolve conflicts (and do them again for future updates)

@prepare @charlesroddie Thoughts?

I don't believe CSharpMath is affected much by the NRT PR LayoutFarm/Typography#192. That PR makes it easier for Typography to control NRE bugs and to do further development without having to worry about NREs. However it's not worth forking Typography just for that. If the @prepare prefers to do more work to implement features safely that is up to him!
CSharpMath should depend on Typography master branch if possible, through the submodule for now, and through a nuget package when Typography stabilizes.
Note that it can't use the current state of Typography master as UWP support is broken by the netstandard2.1 change.

Hello

Note that it can't use the current state of Typography master as UWP support is broken by the netstandard2.1 change.

I move back to netstandard 2.0, please check again :)
https://github.com/LayoutFarm/Typography/commit/0899ddfb4294325bc3f9bc06909bd3bee8323eca

@charlesroddie How the NRT PR affects CSharpMath can be seen through the difference between using null-oblivious and null-explicit APIs. By dropping the current submodule reference to LayoutFarm/Typography#192, all Typography APIs currently is use will start ignoring accidental nulls.

@prepare @charlesroddie Seems that LayoutFarm/Typography#192 will never be merged. Should we drop it and update or should we resolve merge conflicts?

That is a shame if true. @prepare it would be good to have your thoughts on that PR even if in an initial form. Maintaining a branch or fork of typography for use in CSharpMath would require a lot of work on @Happypig375 's part, and is unfeasible to maintain indefinitely IMO. So if it will really never be merged then dropping and accepting a less safe typography library for use in CSharpMath is inevitable.

@Happypig375 👍
I want to confirm that the https://github.com/LayoutFarm/Typography/pull/192 is NOT your waste.
But I can't merge it directly now because of this https://github.com/LayoutFarm/Typography/pull/192#issuecomment-635820167 too.

By the way,
The Typography+PixelFarm is now under refactoring-code-arrangement phase again,
It will be released this week.


Use vscode user to install gems.

Updated the line to install the Rails gem to use the vscode user. This
brings it in line with the optional line to install node packages. Also
updated the example line for additional gems.

This pull request appears, initially, to address the immediate concern that I raised in #188. The directory where the gems are installed appear to be owned by the vscode user and it looks like the ruby-lsp gem is able to be installed when the VS Code extension is installed. This is all the desired behavior.
I'm not sure what, if any, tests are necessary to cover this. I will continue exploring this change manually, but could use some guidance if I should be update the tests and how.
However, there are still fundamental issues that remain unaddressed:

Should this devcontainer even exist? This devcontainer only does two things: install two gems and include a compose file to stand up a Postgres container. Both of these are trivial tasks that can be done on top of the existing Ruby container image by a developer. This configuration also neglects Rails projects that use other database engines. Out of the box, Rails can also be configured for SQLite, MySQL, Oracle, and more. Postgres is (from my understanding) a more common database for Rails projects, but it's not the only option. Minimizing the supported devcontainers could be beneficial to the core team, especially when the value added by this devcontainer is minimal.
Why does the base Ruby container install RVM? This was discussed in 188 and referred to devcontainers/images #572. It would make a lot more sense to install the requested Ruby version as a core OS version and use the Ruby feature to allow the user to install other Ruby versions using a version manager of their choosing. This is a worthwhile change to investigate.

@microsoft-github-policy-service agree

@samruddhikhandale This is ready for your review. I've created a basic Rails application and used this devcontainer to run an existing Rails application without error. Do let me know if you want the tests to be updated - I suspect the test would be to check who owns the gems directory for the default rvm version and ensure it's "vscode" and not "root", but I'm not sure if that is the right test. If you'd like to see a specific test, let me know and I can look at see what it would take to implement.

@samruddhikhandale I've added the tests.
I left out the webdriver test. Honestly, I'm not sure why webdriver is being installed where it is. I know it's common, but it's not a dependency of Rails as far as I can see. If a project is using it, I'd expect it in the gemfile or however the project is managing dependencies, I don't want to remove it from the container setup because I feel that could be a breaking change for someone out there and isn't related. So I ended up adding two tests to verify that (1) Rails is installed and (2) the user can install gems.
I did run these tests on the devcontainer without the other changes. The Rails test passes and the check for the user's ability to install gems fails.
Let me know if there's anything else you'd like to see.

@samruddhikhandale
My solution here is starting to feel a bit hacky.
I fixed the issue with the failing tests in GitHub Actions. I think I was running into a similar problem that the fixTestProjectFolderPrivs function is designed to work around. In my local builds, the vscode user was owning the directories. However, in the GitHub Actions builds, it was owned by 1000. I didn't want to force the ownership of the directories, so I started to look for another solution.
I noticed that the rvm group didn't have write permission to the gems directories, even though it should. Since the vscode user is indeed in the rvm group, making sure the group had the right permissions made sense. So even though it wasn't necessary for local builds to work, I also wanted a solution that would pass the tests.
We could get away with just running the rvm fix-permissions command and not running the gem install as the vscode user, but it seems like we shouldn't be installing gems as the root user anyway.
So, at this point, the end result of the devcontainer build process is a devcontainer where a user can install gems. I still have lots of questions about the utility and viability of this devcontainer and if installing rvm (and, apparently, rbenv) is appropriate. But those are longer term discussions and decisions that go beyond me. The important thing is that anyone who picks up this devcontainer shouldn't have broken VS Code extensions and should be able to run a Rails project out-of-the-box.
Let me know if there are any other questions or concerns and I'll do my best to address them.


Approximate conformal Bayes for classifiers

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the new behavior?

Implemented an approximate version of Fong & Holmes (2021) for classification
Added an example script on 2 moons example

Other information

Currently, the user is required to manually construct a dataloader which concatenates the training data followed by the test data plugging in Y = 0 then Y = 1 (a grid of size 2). This is shown for example in lines 52-64 in examples_2moons_CB.py. Would it be possible to have the user pass in a train_data_loader and test_data_loader, then we construct this combined data loader together with the grid within conformal_set?
Still need to add unit tests for conformal_set but will depend on how we handle the above.

Thank you!

@edfong Thanks for the PR!
About your question, yes, it should be possible using DataLoader.from_inputs_loaders. Suppose that you have arrays of inputs train_inputs and test_inputs. Then something like the following should work:
from fortuna.data import DataLoader, InputsLoader
data_loader = DataLoader.from_inputs_loaders(
    inputs_loaders=[
        InputsLoader.from_array_inputs(train_inputs, shuffle=True),
        InputsLoader.from_array_inputs(test_inputs, shuffle=True),
    ],
    targets=[0, 1]
)

If, instead, you have already a train_data_loader and a test_data_loader instead of array of inputs, you can just do
from fortuna.data import DataLoader
data_loader = DataLoader.from_inputs_loaders(
    inputs_loaders=[train_data_loader.to_inputs_loader(), test_data_loader.to_inputs_loader()],
    targets=[0, 1]
)

Would this work?

Thank you @gianlucadetommaso!
Apologies I wasn't clear enough in my earlier message, but this is close. What I was looking for is to have a Dataloader that concatenates: (training inputs, training targets) + (test inputs, 0) + (test inputs, 1). Is there a way to combine a Dataloader with Inputs loader (with targets (0,1))?
Is there also a way to manually make a Dataloader output the data in order so I can check my implementation?
If this is problematic, I could have the user pass in train_data and test inputs, but ideally passing in a train_data_loader and a test_data_loader.

Perhaps you are looking for .to_array_data? Given your concatenated data_loader, you can do
data_loader.to_array_data()

This will return a tuple of arrays of inputs and targets. This should help you check your implementation.


Add better error handling if select field cannot retrieve data

Add better error handling if select field cannot retrieve data
:heavy_check_mark: Checklist

[ ] Added tests for new functionality and regression tests for bug fixes
[x] Added changeset (run yarn changeset in the root)
[ ] Screenshots of before and after attached (for UI changes)
[ ] Added or updated documentation (if applicable)

Is there anything I may do to assist you in getting this out? I believe we are going to see this more often, now that SelectFieldFromAPI is being incorporated into more scaffolder templates.


Scale bar grayed out for many elements

I'm not sure if something else is interfering with the addon but the scale bar is grayed out and inaccessible for castbar, minimap, action bars, and chat, so I can basically only resize the unit frames. This happens with no other addons active so I'm not sure what could be causing it if the scale feature should be usable?

The new Version (v0.4.0) opens up more customization options. Some are still deactivated while they are worked on.