[Docs] Exporting env variables through WSL

Adds a note about exporting the environment variables for the kernel, initrd and UEFI firmware through WSL.

Could we add these to build_support/setup_windows_cross.sh too so they're set automatically for most devs?

Resolves #298 Thanks!

Whooops, sorry @smalis-msft, totally missed your comment before merging this. I agree - this would be good to add to the setup script.


Document expect(.not).(array|object)Containing()'s actual behavior

Summary

The existing documentation for expect.objectContaining(), expect.not.objectContaining(), and to a lesser degree expect.arrayContaining() and expect.not.arrayContaining() doesn't match their behavior. This has resulted in a number of issues (see #11126, #10462, #10186) and attempts to align the behavior with the documentation (see #10508, #10766, #10708), with varying degrees of success.
In this PR, I documented the actual behavior of these asymmetric matchers on master. As it turns out, the actual behavior of expect.objectContaining() is quite strange. Unfortunately, I don't know what the behavior should be, so I'm submitting this PR as-is rather than opening issues.
Documentation/behavior conflicts and edge cases:

expect.objectContaining(), expect.not.objectContaining() are not recursive
expect.objectContaining() can match primitive values

expect.objectContaining() boxes truthy primitives and doesn't box falsy primitives

expect.arrayContaining([]) can match non-arrays (and in fact matches anything)
expect.not.arrayContaining([]) matches nothing

I recommend not merging this PR and instead deciding what expect.objectContaining() should actually do. I'd be happy to update this to reflect the intended behavior, or someone else could submit a new PR.
This PR would close #11126 and close #10462.
Test plan

Several tests were added to cover expect.objectContaining() and expect.not.objectContaining()'s behavior with primitive received values.

Codecov Report

Merging #11157 (da3ad5c) into master (1d8f955) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11157      +/-   ##
==========================================
- Coverage   64.20%   64.19%   -0.02%
==========================================
  Files         307      307
  Lines       13371    13371
  Branches     3261     3261
==========================================
- Hits         8585     8583       -2
- Misses       4082     4083       +1
- Partials      704      705       +1

Impacted Files
Coverage Δ

packages/expect/src/utils.ts
94.83% <0.00%> (-1.30%)
:arrow_down:

Continue to review full report at Codecov.

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

Hey @ninevra! I'd love to land this - any particular reason it's a draft? 🙂

I don't want this to hold up the release, so swapping milestone. I'll probably have time to deal with this before next major (which will be this summer after node 17 is EOL)


Support for plain HTML files as model instances

Model instances can currently only be written in JSON or Markdown formats, but there are use cases where straight HTML would be preferable.

The Markdown processor in Statik already actually supports embedding plain HTML into the .md file. This should be good enough to fulfil this use case. See the examples/blog project in the repo.


dulwich: bump minver to 0.21.5

Closes https://github.com/iterative/dvc/issues/9262

Codecov Report
Patch coverage: 100.00% and project coverage change: -1.05 :warning:

Comparison is base (ed83c57) 81.23% compared to head (a371708) 80.18%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #239      +/-   ##
==========================================
- Coverage   81.23%   80.18%   -1.05%
==========================================
  Files          27       27
  Lines        4012     4013       +1
  Branches      708      708
==========================================
- Hits         3259     3218      -41
- Misses        630      660      +30
- Partials      123      135      +12

Impacted Files
Coverage Δ

src/scmrepo/git/backend/dulwich/__init__.py
80.48% <100.00%> (-1.07%)
:arrow_down:

... and 6 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add support for tokio console

Tokio console support can be enabled by passing --tokio-console flag to maker
or taker binary.
When enabled, it is possible to see spawned tasks via tokio-console CLI util.
See more: https://docs.rs/tokio-console/

screenshots of tokio-console running:

bors r+

Do we really need this merged in master? I think this tool is more useful locally and can be added on demand if needed. I'm afraid that with additional dependencies and features we might be adding more problems.

Do we really need this merged in master? I think this tool is more useful locally and can be added on demand if needed. I'm afraid that with additional dependencies and features we might be adding more problems.

this is meant to be used locally, it is not enabled by default and there is no intention of enabling it in production - this just adds one more tool into our toolbox for debugging issues locally.
Otherwise, if we e.g. had it on a feature branch somewhere, it would require us to recompile the whole workspace from scratch every time a person wants to use it (and then back again).

If it is really just for locally I propose to hide it behind a feature flag and enable that feature flag by default for our aliases dev-maker and dev-taker.


Add GetLeaderIP method to LeaderEngine

What does this PR do?
add GetLeaderIP method to LeaderEngine, allowing the leader election engine to inspect its kubernetes service endpoints and return the leader's IP if found.
Motivation
Will be used by the cluster check followers to redirect requests to leader.
Split from https://github.com/DataDog/datadog-agent/pull/2696 to ease review.

Codecov Report

Merging #2706 into master will increase coverage by 0.02%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master    #2706      +/-   ##
==========================================
+ Coverage   56.67%   56.69%   +0.02%
==========================================
  Files         380      381       +1
  Lines       24126    24136      +10
==========================================
+ Hits        13673    13685      +12
+ Misses       9538     9536       -2
  Partials      915      915

Impacted Files
Coverage Δ

pkg/util/kubernetes/apiserver/endpoints.go
80% <80%> (ø)

pkg/forwarder/worker.go
92.5% <0%> (-1.25%)
:arrow_down:

pkg/logs/auditor/auditor.go
71.83% <0%> (+0.7%)
:arrow_up:

pkg/forwarder/transaction.go
86.32% <0%> (+3.41%)
:arrow_up:


Fix boundary issues

This should fix 2 of the issues Brian noticed in his QA/QC of the CDD/HDD tool (error while using the place boundary and the oversimplification)

 Tools: CDD/HDD

Closing this. Will address error handling in a new PR along with the State boundary issue.


macOS 15 Dock Tile privileges

macOS Sequoia extends background task management to also affect Dock tile extensions.
I think if you want the end user to not be able to turn off the Privileges app Dock functionality you will have to supply a com.apple.servicemanagement profile via MDM.
The wiki documentation on managing Privileges via MDM profile should therefore be ammended to include this information and perhaps an example profile.

Is there any official documentation from Apple for this? Privileges does not even show up in the Login Items so I wonder how people could deactivate the Dock Tile plugin.

I see. Ok, thanks for pointing that out. We're currently working on the documentation for Privileges 2, and while it wouldn't be a problem for a user to disable the Dock Tile plugin for Privileges 2 (because it doesn't contain much functionality anymore), we'll still mention it. Thanks a lot!


Test Examples

Love the example you have presented here, it's a very valuable resource for me as this is my first time working with riverpod. I was wondering if you plan on adding some example tests; would be interested in seeing how that's structured and what it looks like to have a mock repo/api client.
Thanks for taking the time to build this repo out!

In what ways are you thinking about simplifying it?

Given a feature, one could simplify its file system structure like so:

controllers
providers
repositories
models
etc.

So that we don't need the tedious data / presentation / domain subfoldering, which make no sense imho.
Also, anything related to the data layer deserves an ad-hoc folder, away from lib. This enables extracting the data layer even to another package, if necessary.


Add support for the 'tap' event

On mobile, flip doesn't respond well to touches because it only listens for the click event.
On mobile browsers, there is a 300ms delay between the user tapping the device and the click event firing. This allows the browser to figure out whether the user was double-tapping to zoom etc, but it makes the app feel unresponsive.
To fix it, we can listen for the tap event. This is not available in jQuery by default, but can be added by using e.g. jQuery Touch Events from Ben Major, or jQuery Mobile, or other solutions I haven't tested with yet.

I added support for the tap event. It's in version v1.0.8 on download/flip. I'd be happy to make a pull request if you would accept it.


Updates to your README!

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/main/CONTRIBUTING.md.
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[x] Something that does not neatly fit into the binary options above

Something that does not neatly fit into the binary options above

[x] My suggested edits are not about an existing topic or collection, or at least not a single one
[x] My suggested edits are not about curating a new topic or collection, or at least not a single one
[x] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Hey Github Explore! Thought your README could use some help.
MacOS users get an extension build error when bundling and there are no instructions for resolving this. I added these instructions as well as a quick video walkthrough at the end. It's hosted on replayable.io, a company that's sponsoring my contributions.
If you accept my PR. We'll have our bot check your readme everyday for changes and notify us in case we need to update the instructions and video so its always up to date. We respond to changes real quick (usually within a day or two).
Let me know what you think or if you have any questions.
I believe these changes will help your users especially newbies.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Sorry, but we're going to pass on this.
I can't reproduce the problem myself and in fact don't even see a transitive dependency on rdiscount in our generated Gemfile.lock. If it recurs and is widespread, I would prefer fixing the problem as directly as possible (upgrading the gem, committing local bundler config) to hosting a video walkthrough in the README.
Thanks though!

I can't reproduce the problem myself

Are you sure about that @smashwilson? I'm using a freshly installed MacOS Monterey installation in a Parallels VM. Ruby version is set to 3.1.2 via RBenv.
Here's the upper portion of the error log I received
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

    current directory: /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/rdiscount-2.1.7/ext
/Users/replayable/.rbenv/versions/3.1.2/bin/ruby -I /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0 -r
./siteconf20221217-1559-5est0v.rb extconf.rb
checking for random()... yes
checking for srandom()... yes
checking for rand()... yes
checking for srand()... yes
checking size of unsigned long... 8
checking size of unsigned int... 4
checking size of unsigned int... 4
creating Makefile

current directory: /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/rdiscount-2.1.7/ext
make DESTDIR\= clean

current directory: /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/rdiscount-2.1.7/ext
make DESTDIR\=
compiling Csio.c
compiling amalloc.c
compiling basename.c
compiling css.c
css.c:81:12: warning: implicit conversion loses integer precision: 'unsigned long' to 'int' [-Wshorten-64-to-32]
        written = fwrite(res, 1, size, f);
                ~ ^~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
compiling docheader.c
compiling dumptree.c
dumptree.c:23:29: warning: returning 'const char [11]' from a function with result type 'char *' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    case WHITESPACE: return "whitespace";
                            ^~~~~~~~~~~~
dumptree.c:24:29: warning: returning 'const char [5]' from a function with result type 'char *' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    case CODE      : return "code";
                            ^~~~~~
dumptree.c:25:29: warning: returning 'const char [6]' from a function with result type 'char *' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    case QUOTE     : return "quote";
                            ^~~~~~~
dumptree.c:26:29: warning: returning 'const char [7]' from a function with result type 'char *' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    case MARKUP    : return "markup";

And here's the lower portion:
rdiscount.c:28:7: warning: initializing 'char *' with an expression of type 'const char [7]' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    { "strict", MKD_STRICT },
      ^~~~~~~~
rdiscount.c:29:7: warning: initializing 'char *' with an expression of type 'const char [9]' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    { "autolink", MKD_AUTOLINK },
      ^~~~~~~~~~
rdiscount.c:30:7: warning: initializing 'char *' with an expression of type 'const char [9]' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    { "safelink", MKD_SAFELINK },
      ^~~~~~~~~~
rdiscount.c:31:7: warning: initializing 'char *' with an expression of type 'const char [20]' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    { "no_pseudo_protocols", MKD_NO_EXT },
      ^~~~~~~~~~~~~~~~~~~~~
rdiscount.c:32:7: warning: initializing 'char *' with an expression of type 'const char [15]' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    { "no_superscript", MKD_NOSUPERSCRIPT },
      ^~~~~~~~~~~~~~~~
rdiscount.c:33:7: warning: initializing 'char *' with an expression of type 'const char [17]' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
    { "no_strikethrough", MKD_NOSTRIKETHROUGH },
      ^~~~~~~~~~~~~~~~~~
rdiscount.c:50:17: error: implicit declaration of function 'rb_rdiscount__get_flags' is invalid in C99
[-Werror,-Wimplicit-function-declaration]
    int flags = rb_rdiscount__get_flags(self);
                ^
rdiscount.c:63:48: warning: implicit conversion loses integer precision: 'long' to 'int' [-Wshorten-64-to-32]
    MMIOT *doc = mkd_string(RSTRING_PTR(text), RSTRING_LEN(text), flags);
                 ~~~~~~~~~~                    ^~~~~~~~~~~~~~~~~
/Users/replayable/.rbenv/versions/3.1.2/include/ruby-3.1.0/ruby/internal/core/rstring.h:52:27: note: expanded from macro
'RSTRING_LEN'
#define RSTRING_LEN       RSTRING_LEN
                          ^
rdiscount.c:93:17: error: implicit declaration of function 'rb_rdiscount__get_flags' is invalid in C99
[-Werror,-Wimplicit-function-declaration]
    int flags = rb_rdiscount__get_flags(self);
                ^
rdiscount.c:102:48: warning: implicit conversion loses integer precision: 'long' to 'int' [-Wshorten-64-to-32]
    MMIOT *doc = mkd_string(RSTRING_PTR(text), RSTRING_LEN(text), flags);
                 ~~~~~~~~~~                    ^~~~~~~~~~~~~~~~~
/Users/replayable/.rbenv/versions/3.1.2/include/ruby-3.1.0/ruby/internal/core/rstring.h:52:27: note: expanded from macro
'RSTRING_LEN'
#define RSTRING_LEN       RSTRING_LEN
                          ^
14 warnings and 2 errors generated.
make: *** [rdiscount.o] Error 1

make failed, exit code 2

Gem files will remain installed in /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/rdiscount-2.1.7 for inspection.
Results logged to
/Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/extensions/x86_64-darwin-21/3.1.0/rdiscount-2.1.7/gem_make.out

  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/builder.rb:95:in `run'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/builder.rb:44:in `block in make'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/builder.rb:36:in `each'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/builder.rb:36:in `make'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/ext_conf_builder.rb:63:in `block in build'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/tempfile.rb:317:in `open'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/ext_conf_builder.rb:26:in `build'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/builder.rb:161:in `build_extension'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/builder.rb:195:in `block in build_extensions'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/builder.rb:192:in `each'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/ext/builder.rb:192:in `build_extensions'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/3.1.0/rubygems/installer.rb:853:in `build_extensions'
/Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/rubygems_gem_installer.rb:72:in
`build_extensions'
/Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/rubygems_gem_installer.rb:28:in
`install'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/source/rubygems.rb:207:in `install'
/Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/installer/gem_installer.rb:54:in
`install'
/Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/installer/gem_installer.rb:16:in
`install_from_spec'
/Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/installer/parallel_installer.rb:186:in
`do_install'
/Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/installer/parallel_installer.rb:177:in
`block in worker_pool'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/worker.rb:62:in `apply_func'
/Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/worker.rb:57:in `block in
process_queue'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/worker.rb:54:in `loop'
  /Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/worker.rb:54:in `process_queue'
/Users/replayable/.rbenv/versions/3.1.2/lib/ruby/gems/3.1.0/gems/bundler-2.3.21/lib/bundler/worker.rb:91:in `block (2 levels) in
create_threads'

An error occurred while installing rdiscount (2.1.7), and Bundler cannot continue.

In Gemfile:
  github-pages was resolved to 39, which depends on
    rdiscount


Trouble with conditional loading

Hello,
I fail to have the conditional loading to work with system.buildStatic.
I've been following this documentation : http://jspm.io/0.17-beta-guide/conditional-loading.html
My use case is that I want to switch a configuration file when ever a certain parameter is passed to my build.
I'm working with jspm 0.17 beta with typescript and the typescript plugin
So here is what I have :
My jspm configuration looks like that
SystemJS.config({
  transpiler: "plugin-typescript",
  typescriptOptions: {
    "tsconfig": true
  },
  packages: {
    "app": {
      "main": "main",
      "defaultExtension": "ts",
      "map": {
        "authConfig":{
          "skipConfiguration": "./session/auth.config-travis.ts",
          "~skipConfiguration": "./session/auth.config.ts"
        }
      }
    },

  },

});

From my understanding whenever I import "authConfig" system will resolve the path to ./session/auth.config.ts if the skipConfiguration param is not set or false else ./session/auth.config-travis.ts
My import looks like this:
import {authConfig} from "authConfig";

And when I run the app, it actually works.
Then I want to build the app, and I use buildStatic in a gulp task :
 static bundle(done) {
        let builder = new Builder("./", "jspm.browser.js");

        builder.loadConfig("jspm.config.js")
            .then(() => {
                builder.buildStatic("app", "./.tmp/app.js", { minify: true, skipConfiguration: true, browser:true, sourceMaps: false, runtime:false})
                    .then(() => {return done()})
                    .catch((ex) => {done(new Error(ex));
                });
        });
    }

My jspm.browser.js looks like this :
SystemJS.config({
  baseURL: "/",
  paths: {
    "github:*": "jspm_packages/github/*",
    "npm:*": "jspm_packages/npm/*"
  }
});

this fails with this error :
Error: TypeError: No resolution found at all for condition app/#:authConfig
I feel the issue is a path issue, but, I can't figure it out.
Your insight would be greatly appreciated.
EDIT:
Forgot to mention the folder structure of my app :
/
   |_app/
      |_session
         |_ here are the ts files using the import and the actual files to import.
   jspm.config.js
   jspm.browser.js
   package.json
   ...

I could be wrong but based on my reading of 0.15.16 code support for arbitrary conditions isn't supported? definition of processTraceOpts() in builder.js#332 indicates only "browser", "node" and "production" flags (and their ~ opposites) are set without the ability to define any others.


Replace secret workqueue

fixes #518

This change is

Coverage increased (+0.4%) to 44.569% when pulling f28142ae8e6aa3428280651db02f01f37f2a79a8 on aledbf:refactoring-secrets-sync into 7c635a8c8367a54401a90710edcbbe3762da2d59 on kubernetes:master.


onReset: TypeError null is not an object (evaluating 'handler.ptrElement.classList')

Hi
Because of the setTimeout just below, it may happen that handler is already null when trying to reset it.
It would be safer to check that ptrElement is not null before changing classList and style.
https://github.com/BoxFactura/pulltorefresh.js/blob/c674942b1f171129cef3fd0343acd446e300abfe/dist/index.umd.js#L71-L78

Could you provide that change back in code? If it passes our test-suite I think it'll be fine, thank you!

https://github.com/BoxFactura/pulltorefresh.js/pull/103

@pateketrueke, I saw your commit https://github.com/BoxFactura/pulltorefresh.js/commit/3b4e7561ea51ab6bc91bd2b1250ccaaddc074e27
But it seems to be not yet shipped npm package(https://www.npmjs.com/package/pulltorefreshjs)
So I still seen Cannot read property 'classList' of null error in sentry.
Could you please publish this commit?

Could you please publish this commit?

Yeah, no problem.


Feature/cors

Intent
To add functionality that tests the XHR instance when a network error occurs and gives an educated guess whether the XHR request was terminated by CORS issue.
Solution
Provided responseURL remains empty, but readyState is 4 we assume is is blocked request due to cross browser origin issue.
Tests

New unit test to cover this scenario automatically
Tested manually against Safari, Chrome, FF, Edge

Notes
While reproducing the mentioned scenario in IE11, the request wasn't blocked and the 302 was appropriately processed. Nevertheless the content wasn't flipped as expected.

Just started reviewing :) :octocat: Sent from GH.

We have verified this approach works as well in IE, but @vbence86 has come up with a more robust way to catch the CORS redirect issue.
Based on browser observed behaviour, on CORS redirect errors, xhr.readyState jumps from 1 to 4 directly, which could probably be used to identify this case. @vbence86 is going to do some more thorough testing and send a PR with those changes if satisfactory.


Added annotate-always option

You can see it here. This would make the comparing benchmark run even on pull requests because it wouldn't need the github token anymore

@rhysd Could you please the PRs a look?

Updated.

@rhysd Can I get this reviewed please?

Any update on this @rhysd?

@rhysd Is there anything blocking this?


Add proc_macro impls

Add relevant trait implementations for proc_macro/proc_macro2 behind a feature gate.
This is superior to an external crate because an external crate can't provide these implementations on the native types, and would need to add another layer of wrapping.
Thoughts?

I'm not sure I understand. Is this to use chumsky to parse rustc token streams, or...?

Yes, this would be allowing chumsky to be used in the creation of a proc macro to parse input

Update: With the recent additions of Stream and nested_in, this actually already works. Just call Stream::from_iter(ts.into_iter) and use nested_in to split up TokenTree::Group


feat: Vulnerability Scanning

Motivation & Context:
reason/motivation behind the PR, context of what problem it solves
This PR solves the issue #issue
Description:
describe the change briefly
Types of Changes:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

PR Checklist:

[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s

@dependabot rebase


Tweak missing tile source message in the TileSet editor

This also renames the Tiles tab in the TileSet bottom panel to Tile Sources to make it more explicit (and avoid using the same name as the Tiles tab in the TileMap editor).

See https://github.com/godotengine/godot-proposals/issues/3874#issuecomment-2283019896.

cc @KaijuKoder2
Preview

Thanks!

That would do it! Thanks!


switch in firefox...

Version
0.1.1
Android or iOS, which browser?
Firefox, mmmh, seems not supported...?! Any suggestions how to solve this would be appreciated though.
Your Codes
When running the switch demo I do not get the switch image, but a html checkmark, see attached image.
Expected behavior and actual behavior.
Expected: normal switch.
Actual: html checkmark.
Steps to reproduce the problem
Run switch demo in firefox.

Sorry to tell you Vux currently does not support Firefox because few people use it.

Thanks for your swift reply. Firefox covers 16%...:
http://www.w3schools.com/browsers/browsers_stats.asp
Any idea what could be causing the issue?

https://github.com/weui/weui/issues/392

Few pepole use Firefox on mobile in China actually...

Yes, which means the % is even higher outside of China!
So it is a weui issue. Thanks.

so we are sorry for this.
Closed the issue, If you have any other problem, we are so happy to make a  discussion together

Cool, thanks.

@airyland I am firefox user..

@graysheeep PC & Mobile?


Getting Response Headers

Hi,
now i am using the below code to get request body. but how i can get the response header.
$request = $this->github->getAuthenticatedRequest($method, $url, $token,$data);
 $data = $this->github->getResponse($request);

i am try to use this in a web app i am developing for that i need to read the header for the current request

Hmm, we don't provide anything like that. Your best bet if/until we do would be to extend and add a method similar to sendRequest (such as getRawResponse) that returns the unparsed request.

@shadowhand ok if i add and incase if i update latest source from this repo and that update done by me will be removed.. how i can ?

and i can see that not sendRequest function parses the response
[getResponse](https://github.com/thephpleague/oauth2-client/blob/b3242ca3761e8789d642f385473ebe2dc6b498b4/src/Provider/AbstractProvider.php#L615-L623) dose it

Extend the your provider, don't modify the class.

i am using a Github Provider By This https://github.com/thephpleague/oauth2-github
do i need to extend that github class ???

Correct. :)

@shadowhand
Hi,
After extending i am unable to use OAuth Login for Github
I am getting below error
An uncaught Exception was encountered
Type: InvalidArgumentException
Message: Required option not passed: "access_token"
Filename: C:\xampp\htdocs\gitsync\vendor\league\oauth2-client\src\Token\AccessToken.php
Line Number: 58

The below is my extended class code
class Github_oauth extends League\OAuth2\Client\Provider\Github{

   /* public function __construct(array $options = [], array $collaborators = []){
        parent::__construct($options,$collaborators);
    }*/

    public function getResponse(Psr\Http\Message\RequestInterface $request){
        $response = $this->sendRequest($request);
        $pagenation = $this->getPagination($response);
        $parsed = $this->parseResponse($response);
        $this->checkResponse($response, $parsed);
        return array('pagenation' => $pagenation,'data' => $parsed);
    }

    public static function getPagination($response) {
        $header = $response->getheaders();
        $pagination = array();
        if(isset($header['Link'])){
            foreach($header['Link'] as $link){
                $l = explode(',', $link);
                foreach($l as $ll){
                    preg_match('/<(.*)>; rel="(.*)"/i', trim($ll, ','), $match);
                    if (3 === count($match)) { $pagination[$match[2]] = $match[1]; }
                }

            }
        }
        return $pagination;
    }

}

Sorry i fixed it thanks :)


Update GH Secret Key

:tada: This PR is included in version 1.5.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature] Prevent breaking ender chests with a non silk touch pickaxe

Hi! I was wondering if it would be possible to implement a feature that prevents breaking ender chests with a non silk touch pickaxe (I always accidentally use my fortune pickaxe instead and end up with 8 obsidian). Thanks!

Yes, sounds good, i'll implement that in the next days


previous_version and next_version: unintuitive behaviour for related objects

How to reproduce.

Create a Versionable object having relation (m2m or o2m) with another Versionable. Save this first version.
Create a new version of this model.
Using the latest version call on it the previous_version() method to get back the initially created version.
Navigate the relation of that origin version and see that you don't get anything back as you would expect.

Possible solution:
Add an optional as_of parameter to previous_version() and next_version(), so that the object returned will propagate this given as_of time to the queries generated for related objects.  If no as_of time is given, set the object's query_time to it's version_end_date minus 1 microsecond (or to None if it's the current object)
In the case that the given as_of lies outside of the version's validity period, an Exception will be raised.
However:
How common is the use case that we ask for a previous version and know it's expected validity period?  Perhaps previous_version / next_version should not have an additional as_of parameter - they should just set the returned object's query_time to it's version_end_date minus 1 microsecond (or to None if it's the current object).  The caller can then adjust the object's as_of time themselves, if needed.

Good points.

Code for this was merged into master.


Run pnpm install everytime there is a change on package.json

Whenever there is a change on package.json, pnpm install must be ran. We need to make this process smoother.

We can leverage git hooks to achieve that.


Implement TRY_CAST

Hi @andygrove
This change adds the TRY_CAST alongside CAST so that the choice on how to handle data casting errors can be differentiated.
See: https://docs.microsoft.com/en-us/sql/t-sql/functions/try-cast-transact-sql?view=sql-server-ver15
Also fixed a clippy error that must have been added recently.

Pull Request Test Coverage Report for Build 662664782

22 of 23 (95.65%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 90.048%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/parser.rs
13
14
92.86%

Totals

Change from base Build 552534083:
0.03%

Covered Lines:
5429

Relevant Lines:
6029

💛  - Coveralls

How does the release process work for this library?

How does the release process work for this library?

I'm embarrassed to say that I don't really know anymore. @Dandandan released a version more recently than me so maybe he or @maxcountryman can help with that.

@andygrove I think we can let you off the hook for not knowing given the work you have done building DataFusion and Ballista  😁

@seddonm1 @andygrove
thanks a lot, I will have a look at this PR later.
The release process is documented here: https://github.com/ballista-compute/sqlparser-rs/blob/main/docs/releasing.md
Basically fully automated after doing a version bump.

Thanks @Dandandan

Happy to help with a release. As @Dandandan mentioned, it should be automated once we've pushed a tag.

@Dandandan are you able to review and hopefully release? This will allow us to add the CastOptions https://github.com/apache/arrow/pull/9682 to DataFusion - hopefully before the next major release.

I think this looks good @seddonm1 ! I think we are good to go, do you want to address the SAFE_CAST question?

@seddonm1 will be released as 0.9.0 , thanks!


[Bug Report] This happens when another thing should have

Server Software and Version:
13:11:01 [INFO] Linux 4.15.0-122-generic [x64],
13:11:01 [INFO] Java 1.8.0_272 [x64],
13:11:01 [INFO] BungeeCord git:BungeeCord-Bootstrap:1.16-R0.4-SNAPSHOT:9c078b7:1539,

Hello,
first of all, please refer to this issue: https://github.com/ME1312/SubServers-2/issues/56#issue-738556062
So basically, this plugin somehow does not work with SubServer, but I cannot find some easy reason why.
It works fine when used with bungeecord itself, but it only shows "Cannot connect to the server" when the bungeecord is running through SubServers.
Any idea?

I set the MOTD using the standard API, ServerPing#setDescriptionComponent, nothing out of the ordinary. I have no idea what is causing your issue and I do not know of or use SubServer. If MiniMOTD is working without SubServer but not with SubServer, I can only assume SubServer is causing the problem. I test MiniMOTD with Waterfall, not BungeeCord, so it may be worth giving that a try, however I don't see how it would make a difference to be honest.

I wouldn't exactly call AdapterComponent standard API. Anyway, after further investigation, the problem seems to be that your plugin is submitting a blank description object to BungeeCord when running alongside SubServers:

ping() placed at the end of the event queue
done() placed at the equivalent of here in BungeeCord's code
Also, it seems to be running a duplicate event, which goes unused:

when running alongside SubServers

The implementation of the description component is irrelevant. MiniMOTD emits proper json and sets the description using the standard bungeecord api, as mentioned before.
Not sure what you are on about with this duplicate event either, the listener is only registered once.
In my opinion the most likely suspects in order here are

SubServers
Bungeecord
The adventure bungeecord platform adapter

If this issue is reproducible without SubServers I will look into it further. In that case, please open a new issue.


Grouping variable to display meaningful errorbars

Hi Pierre,
I was wondering whether you foresee any method which would make me able to automatically group observations into "individuals", in order to have correct errorbars when plotting for eg SEMs.
Imagine that I have 3 repeated observation per individual, per condition. The default input method of gramm doesn't allow me to automatically average these 3 repeated observations, does it?
Cheers,
Romain

Salut Romain,
To get what you want the only way is to provide gramm with the per-individual per-condition means. The good news is that if your data is already organized for gramm use (as a table or as something that is very close to a table) you can get a new table containing the desired means in one line of code using Matlab's rowfun() using the 'GroupingVariables' argument.
For some representations however, you could use the 'group'argument in gramm, for example if you wanted to represent per-individual per-condition averages directly (i.e. not making CIs over subjects) but didn't want to have different colors/facets/etc per individual.
I hope that helps !
Pierre


Bug a simple add library?

I have simle html5 page:

        <video id="video" width="320" height="240" autoplay></video>
        <canvas id="barcode" width="320" height="240" style="display:none"></canvas>

I add library in head:

After adding the library I stopped working component video, just a black screen.

Please take a look at the examples available in /examples or open up the source code on the examples page.
Nevertheless, the following code shows the most basic usage of quagga using a camera:
<!DOCTYPE html>

        <title>Example</title>

            <video id="video" width="320" height="240" autoplay></video>

Thank you, I saw examples.
Look, here's my example, and it works: https://jsfiddle.net/pkochubey/98sqs3oc/.
But if you insert QuaggaJS it no longer works: https://jsfiddle.net/pkochubey/qx8cw6ec/.

Can you explain what you are trying to do? Judging by your examples, you are simply trying to get the quagga working in jsfiddle, right? But when I look at your second link, you are requesting the camera stream outside of quagga, which is not the way it works. Quagga itself requests the camera if it is initialized to have a LiveStream as an input (see example above).
Or try this adaption of your fiddle: https://jsfiddle.net/qx8cw6ec/1/

Ok. I understand.


Deduplicate custom return types with macros

What is the expected enhancement?
The custom return type definitions in src/iterators.rs are pyclass struct wrappers around a rust native container (like a Vec or HashMap) which we use to avoid the copy and overhead of going from rust->python for large return types. Most of the code there is copy and pasted boilerplate to fulfill the interface for exposing an Python Sequence or Python Mapping (and the auxiliary methods for a dict) and the only updates are the type information for the most part. This feel like the kind of thing that should be easy to simplify by just having a custom macro that expands the definition based on the input types and names.

This was done in https://github.com/Qiskit/retworkx/pull/354


bestaudio not so "best" after all?

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.02.08. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2019.02.08

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[x] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

Description of your issue, suggested solution and other information
I'm not entirely sure if I should classity thiss issue as a bug so I decided to mark it as 'other'.
https://i.imgur.com/fRNJnz4.png
When using -f bestaudio parameter youtube-dl will use 171 format (it's new). However, as it shows on Spek's output, it's just 128k with nice and hard cut after 15kHz, compared to juicier 251 format.
You can clasify this a bug from user's perspective, but it's not a bug when you look at format output: 151k is more than 141k. On the other hand format description says completely different story.

So, as I understand with just closing issue and leaving no comment, this is indended behavior?

@CatPlanet
Firstly, next time please do not post images if you can post text. It is
impossible to copy and paste commands posted in an image.
Secondly, I agree with your assertion: 251 is of higher quality than 171
because the format of 251 is opus @160k, while the format of 171 is
vorbis@128k. Note that 251 has higher rated bitrate (160k) than that of
171 (128k). The fact that 251 has lower actual bitrate is precisely
because opus is meant to be a replacement for vorbis.
It provides better compression with no apparent loss in quality.
As we can see, the youtube-dl way of detecting best formats using the actual
bitrate is only a general heuristic. It does not always yield the best format.
However, I don't think changing the behaviour of youtube-dl is a good idea
because different users have different requirements. There isn't a simple way
to satisfy all users, e.g. I prefer free formats over patent-encumbered
formats (so vp8, vp9 > h264), as long as they have no worse resolution and
rated bitrate.
Therefore, I think we should follow the unix philosophy,
keeping youtube-dl minimal, and instead modularly create a script which takes a
list of formats and print the best format based on user configuration.
This can be done because youtube-dl can dump the list of formats as json.
Actually, this is something I want to do for a long time.


Fix typo

a Azure -> an Azure

@changeworld : Thanks for your contribution! The author, @CarlRabeler, has been notified to review your proposed change.

#sign-off


Add all used Cacert.org certificates

This PR all know certificates used by cacert.org to dodo.
I was not able to find their unpublished 2008 roots & sub-roots.

@nsapa I've only added the current CACert root to this repository, since the old root and the subordinates all chain to it.  It may take a few days for our Ops team to deploy this to Dodo.


Not working at all as of 25 minutes ago

I'm assuming Tumblr changed something as they said this was the deadline for NSFW stuff... All I can get it to do now is evaluate tumblr post URLs, then stop... never finds downloads to get... Stopped right at Midnight Eastern Time.

Yeah, seems to be working for me as well, as of 11:30 am CET..

Not really sure what happened. I trashed my (portable) install of it and started over and it's able to find the downloads again... I guess I just jumped to (an incorrect) conclusion when it broke mid-download at the turn of the hour on announced date...
I don't see a way for me to close this issue myself, so if a mod can close this, that'd be great. Sorry and thanks :)

You can close issues as the opener.

Totally missed the button at the bottom.


Disabling Fabs android Ripple effect

Issue Description
Cannot disable the android ripple effect for FABs.
npm, react-native, react and native-base version version  used
"react": "16.4.1",
"react-native": "0.56.0",
"native-base": "^2.8.0",

Expected behaviour
No ripple effect or changed color of android ripple effect.
Actual behaviour
Cannot change the FABs android ripple color.
Steps to reproduce
Tried changing the style and containerStyle of FAB to change the androidRippleEffect.

Only on android

I'm also seeing this issue. Is there a way to disable this behavior? I didn't see anything in the documentation.

I'm also trying to remove the ripple effect on the container of the FAB with no success. It should be applied only to the circle.

No update so far! Any workarounds @SupriyaKalghatgi  or anyone

@YoonjaeYoo  thanks for the workaround, it does the trick for me

@YoonjaeYoo That does not give the desired output
When Fab is toggled from active to inactive, the button shrink and retains back to usual size

@SupriyaKalghatgi  yes it's not perfect, but it's at least "pretty-enough" compared to a squarre overflowing , so it's indeed not a fix, but a "better-than-nothing workaround"

Fixed with commit https://github.com/GeekyAnts/NativeBase/commit/56597c45df1b75e1658b1e049d38e08dbc954e3e

@sharathprabhal  thanks a lot, can't wait for the bugfix release :)

itemRippleColor="@null" doesn't seem to work but rippleColor="@null" seem to somewhat help with the issue


Is the Samples link broken?

I just published https://github.com/microsoft/windows-rs/releases/tag/0.48.0 and the links should now be working.


Sets linker flags for building libosrm.so shared library, resolves #172

For https://github.com/Project-OSRM/node-osrm/issues/172.
At the moment we only set linker flags for osrm-* binaries.

Failing because of: https://travis-ci.org/Project-OSRM/node-osrm/jobs/169404996#L1226

Apparently clang does not understand rt

I wonder if this could be the bug I found and fixed in https://github.com/Project-OSRM/osrm-backend/pull/3130 whereby the libosrm.pc has -l rt instead of -lrt. @daniel-j-h you can pass --verbose to npm install to get the linking line to show up which should help diagnose.

Current coverage is 0.00% (diff: 100%)

No coverage report found for master at 02153df.

Powered by Codecov. Last update 02153df...cdc5df0

The problem @daniel-j-h ran into here was unrelated to this PR and was also impacting master. Found and fixing the bug (which I introduced) at https://github.com/Project-OSRM/osrm-backend/pull/3163.


Logout could return a boolean instead of a string result

The current logout() method returns a Result<String>, which in my test environment weirdly even says "Welcome". While posting to the /logout endpoint should actually return ["Your token has been cleared"] (https://github.com/saltstack/salt/blob/develop/salt/netapi/rest_cherrypy/app.py#L1481), I guess we could change the return type of the logout() to boolean (in case the string matches the above).

I found out that logout() is actually broken since we don't POST: passing null as data will create a GET request, which is also the reason why "Welcome" is returned. I just fixed this in a branch with dcc30c25765dbd89bbe2ab05553676f6e2403ffe. I will further change the method signature of logout() to return a boolean and then make a pull request.

Fixed with #71.


Use restore-state-none feature in optional dependency critical-section

This should provide a slightly user-friendlier error message in the case that a user enables a different restore-state in critical-section.
Right now it's possible to e.g. enable the std feature on critical-section; cargo will not complain but rustc will complain about the return types of the CriticalSection::enter implementation.

Thanks!


Documentation in the Auth setup is broken.

function init() {
gapi.load('auth2', function() { // Ready. });
}
should be
function init() {
gapi.load('auth2', function() { /* Ready.*/ });
}

This error is on the Google Sign-In JavaScript client reference page in the Auth Setup section.
Ideally the example would demonstrate what to do in the Ready block.
function init() {
  gapi.load('auth2', function() {
    /* Ready. Make a call to gapi.auth2.init or some other API */
  });
}

Thanks.
This will be fixed soon with internal change 253834992.

Fixed. Changes are live!


Send event to Snowplow on initialize

Migrated from: https://github.com/snowplow/snowplow/issues/1506

This was done


ControlledOnBitString behavior when bitstring and control register are not the same length not documented

Describe the bug
This may not be a bug, but I wanted to capture that the behavior for ControlledOnBitString is not clear when the control register it is applied to is longer than the control bitstring (the first argument) itself.
(ControlledOnBitString(address, ApplyPauliFromBitString(PauliX, true, _, _)))
(addressRegister!, (values, targetRegister));
To Reproduce
See gist here, run the sample in binder:
open Microsoft.Quantum.Diagnostics;

operation ControllingStringCorrectLength() : Unit {
    let address = [true, false, true];
    using((addressRegister, target) = (Qubit[3], Qubit())){
        // Prepare address register to address value
        ApplyPauliFromBitString(PauliX, true, address, addressRegister);
        // Control on the same address, should put target qubit in superposition.
        (ControlledOnBitString(address, H))
            (addressRegister, target);
        DumpMachine();
        ResetAll(addressRegister+[target]);
    }
}

operation ControllingStringShort() : Unit {
    let address = [true, false];
    using((addressRegister, target) = (Qubit[3], Qubit())){
        // Prepare address register to address value
        ApplyPauliFromBitString(PauliX, true, address, addressRegister);
        // Control on the same address, should put target qubit in superposition.
        (ControlledOnBitString(address, H))
            (addressRegister, target);
        DumpMachine();
        ResetAll(addressRegister+[target]);
    }
}

Expected behavior
I guess I am not sure what I should expect in this case? That is part of why I am filing this :P
Screenshots
Screenshot of the attached

System information

OS: Windows 10

Thanks for the report, and for the reproduction, I really appreciate having the minimal working example of the issue! In any case, the issue is at the very least a problem of the documentation not being more clear. Given that, I can see four distinct resolutions:

Keep the current behaivor of ControlledOnBitString, but clarify in the API documentation.
Make ControlledOnBitString pad the control array on the right by zeros instead of ones, bringing ControlledOnBitString in line with the little-endian representation used elsewhere. This would be a breaking change.
Make ControlledOnBitString not control on qubits past the length of the control pattern. This would be a breaking change.
Make ControlledOnBitString fail when the control pattern and register are of different lengths. This would be a breaking change.

I'm personally partial to the second option, myself, as that would have the interpretation of leaving qubits without a control pattern uncontrolled, but I'd love to get other input before deciding on that. Any thoughts, @msoeken?

@cgranade, I don't completely understand when you say

as that would have the interpretation of leaving qubits without a control pattern uncontrolled.

Wouldn't they be controlled on false with the second option, but uncontrolled with the third option? If we accept a breaking change, then I'd favour the third option and always take as length the length of the shortest array, similar to many implementations of zip.

Sorry @msoeken, I apparently can't count. Yes, the third option, you're quite right. In any case, I think that even though that would be a breaking change, it would be the most consistent with my expectation from Zip and other functions, as you say.


Update case-study-creating-impossible-perspectives-for-holotour.md - Added correct GitHub author to metadata

Added correct GitHub author to metadata

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/case-study-creating-impossible-perspectives-for-holotour.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


GroupBy with concatMap duplicates items

Hi there!
I need to use groupBy with concatMap to ensure sequentally groups processing.

Hello, thanks for the report - the behavior seems surprising to me
It is caused by this line: https://github.com/monix/monix/blob/master/monix-reactive/shared/src/main/scala/monix/reactive/internal/operators/GroupByOperator.scala#L72
It seems to be intended by the current implementation but I'm not sure about retrying in this scenario.
If downstream returns Stop it probably handled the message and decided to stop. If it failed and it needs a retry logic then it's probably better to do it in the downstream. Any thoughts @alexandru ?
In the meantime, can you remove take from concatMap(_.dump("O").take(10).sum) in your real use case?

Unfortunately, I need to somehow switch between GroupObservable. In the real case i tried using takeUntilEval and other take....
I found only one correctly case:
.concatMap { group =>
   group.timeoutOnSlowDownstreamTo(10.millis, Observable.empty).sum
}

But it doesn't suit for me.

Why do you need to switch?
If you'd like to emit intermediary sums then maybe that will work for you:
bufferTumbling(10).map(_.sum)

The buffer size is determined dynamically and depends on the external side effect. So I use takeUntilEval to switch.

bufferWithSelector ? :D
Note that we might fix the behavior so takeUntilEval is OK but it can take some time so I'm throwing ideas that could unblock you right now

Thanks for the ideas) I have considered buffering options, but in my case I can't wait for the buffer to fill up. I need to start calculating immediately

I tried modify GroupByOperator, but if I remove retryOnNext, then on some launches the elements are lost instead of duplicating

then on some launches the elements are lost instead of duplicating

Maybe it will be solved if we synchronize the access to cache instead of using atomics but it might come with a performance impact, hard to say without measuring


Add clone "-g" option that creates needed Gerrit remotes

This requires a distgit and patches entry in ~/.rdopkg/rdoinfo/rdo.yml, for
example:

project: sample
conf: core
distgit: ssh://rpmfactory.beta.rdoproject.org:29418/%(project)s-distgit
patches: ssh://rpmfactory.beta.rdoproject.org:29418/%(project)s
maintainers:

<EMAIL_ADDRESS>

So if I get it right, this creates two redundant remotes:
origin == gerrit-origin
patches == gerrit-patches

What exactly are they needed for?
#49 uses patches remote.

Yes, if you start from scratch these branches will be identical. The idea behind this is to be able to setup a branch that is used with Gerrit, while still being able to use a branch for example with the repository located on Github. So it's simply to allow different remotes.

@yac if any other concerns left, let's discuss it on the phone

@yac there will be some redundancy as we're adding extensions needed for the new workflow with softwarefactory while keeping and not breaking the old workflow, so we can run them in parallel

Fair enough.


Implemented Volume encryption support

Proposed Changes

Implemented Volume encryption support

Merging it already, considering itss dependency on the deployment that Lukas would like to test now.


Fix slow performance for confusion matrix based metrics

What does this PR do?
Fixes #1267
Fixes #1277
After the refactor the underlying _bincount function that does a lot of the computations for confusion matrix based metrics was changed. I did some testing at the time and everything seemed to be fine. However, for large inputs the
implementation is really slow.
Here is a direct comparison in colab:
https://colab.research.google.com/drive/18tGZj_dPria6NSwVOIgwPXO8mJBr21kc?usp=sharing
The results:

The TLDR:

Regardless of batch size the new implementation is approx 2x slower on cpu
For batch size 100 the new implementation is approx ~4x FASTER on gpu
For batch size 10.000 the new implementation is approx ~30x SLOWER on gpu

The simple fix currently in this PR is to change it back to the old implementation. The alternative is that we have something like this in the code:
def _bincount(x, minlength=minlength):
    ...
    if len(x) > N or not x.is_cuda:
        return torch.bincount(x, minlength=minlenght)
    else:
        z = torch.zeros(minlength, device=x.device, dtype=x.dtype)
        return z.index_add_(0, x, torch.ones_like(x))

where we have to set N based on some experimentation.
@Borda, @justusschock opinions?
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

I would be open for changing this conditionally in the future, but this would require more experimenting :)

do you want to open an issue for it so we won't forget? :otter:


CMS_PLACEHOLDER_CONF 'limits' appears to have gotten broken between 3.3.3 and 3.4.1

I have a minimal template like this:
{% load cms_tags sekizai_tags %}
<html>
  <head>
      <title>{% page_attribute 'page_title' %}</title>
      {% render_block 'css' %}
  </head>
  <body>
      {% cms_toolbar %}
      {% placeholder mike-mcgowan-test-placeholder %}
      {% render_block 'js' %}
  </body>
</html>

And a corresponding CMS placeholder configuration like this:
    'mike-mcgowan-test-placeholder': {
        'plugins': ['TextPlugin',],
        'limits': {
            'TextPlugin': 1,
        },
    },

If I try to add just one TextPlugin to a new CMS page, I get "This placeholder already has the maximum number (1) of allowed Text plugins."
When I look at my network panel, I see two requests, the first returns a redirect to the second, and the second returns a 400:
http://<IP_ADDRESS>:8000/en/admin/cms/page/add-plugin/?placeholder_id=2941&plugin_type=TextPlugin&plugin_language=en
http://<IP_ADDRESS>:8000/en/admin/cms/page/add-plugin/?delete-on-cancel&placeholder_id=2941&plugin_language=en&plugin=14495&plugin_type=TextPlugin

I am using Python 2.7.6, Django==1.8.15, Django-CMS==3.4.1, djangocms-text-ckeditor==3.3.0.

Hello @mikemcgowan,
Does this only happen with the text plugin?
Would you mind checking with a different plugin type?

Infact it seems to be okay with LinkPlugin and FilePlugin. Apologies. I should have raised this in the plugin-specific repo. Closing.


Create checkmarx-scans.yml to run Checkmarx on developer.sailpoint.com

We decided to use the existing template instead of this change. Closing the PR.


Sometimes the worker needs to be restarted before jobs will appear in the rabbitmq queue

This seems to occur more frequently after a job fails or is left hanging. Need more information.

Since this occurs when a response does not get sent to Rodan (at least in most cases), I've been trying to add a dependency to send a response in cases of errors, for example when slurm kills the job itself due to an OOM or timeout.


Properly handle requests that do multiple writes (e.g. by aiohttp 3)

The aiohttp client in version 3 does (at least) two writes if the request has a body: one for the headers, a second for the body. This was done in aio-libs/aiohttp#2126 and is documented in the "Are there changes in behavior for the user?" section of the description. Check out the diff of the write_headers function.
With mocket, this is not handled correctly. The body is lost. Would it be possible to implement proper handling of this scenario?
Example test failures: https://travis-ci.org/sjaensch/aiobravado/jobs/343058521
Test 1 source
Test 2 source

Thank you very much for the contribution. I'll have a look at that as soon as possible.

Hi @sjaensch, I did pip install -U aiohttp ending up with a virtualenv using version aiohttp-3.0.2 and I was able to run all the Mocket tests. Also, if I look at the two failing tests in your code, I don't see you actually enabled Mocket as I do at this line:
https://github.com/mindflayer/python-mocket/blob/master/tests/tests35/test_http_aiohttp.py#L60

@mindflayer Mocket is enabled with this fixture: https://github.com/sjaensch/aiobravado/blob/dfacf2655053f765d832f50adff4659a79a3063d/tests/functional/conftest.py#L74
There are several other tests that use Mocket and continue to pass. Note that the two tests in question also pass if you use aiohttp<3.0. The fact that Mocket tests pass with aiohttp 3.0.2 tells me that this case is not covered by your tests. :)
All of your aiohttp tests do simple GET requests. Make sure you do a request that contains a body, so typically a POST request with some data. The aiobravado tests that do GET requests are still passing as well with aiohttp 3.0. As I explained in the description, aiohttp 3 does two network writes: One for the request headers, one for the request body. GET requests usually do not have a request body.

Hi @sjaensch, could you try to run your tests using this branch (https://github.com/mindflayer/python-mocket/tree/body-via-send)?

@mindflayer awesome! Both tests pass if I use that branch, well done!

Here you are: https://pypi.python.org/pypi/mocket/2.2.0

Here you are: https://pypi.python.org/pypi/mocket/2.2.0


$data parameter required

When sending an invite the only think we know from the invitation are the requester, the requester's team and the invitee.
When sending an invitation email we require additional data in the email like product, version etc.
The easiest way to do this is via an additional data parameter which can be stored as a json object in additional field that will need to be added to the invitation table.

Added and pushed

@iantasker, let me know when you're happy with this in data exchange so I can close the issue.

@drjonnicholson i have gone through this and it looks good.


Make title link on nav

We are using Link component which is a React component can you follow these steps
1- Import this at the top of the pages
import {Link} from "react-router-dom";
2- replace the h1 content with these
<Link to="/">
   <h1 className="text-4xl text-white ml-auto mr-auto font-Jolly-Lodger">
        404 Pages
      </h1>
</Link>

3- open a new pull request

Sorry, I haven't used react in a bit, I mainly use svelte.


Direct Method SetConfiguredEndpoints not working with nodes defined by BrowsePath

Describe the bug
Calling directmethod SetConfiguredEndpoints with the nodes defined with BrowsePath instead of Id yield 500.
To Reproduce
Steps to reproduce the behavior:

Go to the azure portal
Invoke direct method SetConfiguredEndpoints
with the payload：
{
"endpoints": [
{
"EndpointUrl": "opc.tcp://xxxx.com:4897/Softing/dataFEEDSIS/Server",
"UseSecurity": false,
"OpcNodes": [
{
"BrowsePath": [
"Objects",
"Server",
"LocalTime"
]
}
]
}
]
}
Get 500 (See error)
Change the payload to
{
"endpoints": [
{
"EndpointUrl": "opc.tcp://xxxx.com:4897/Softing/dataFEEDSIS/Server",
"UseSecurity": false,
"OpcNodes": [
{
"ExpandedNodeId": "i=17634 [Server_LocalTime]"
}
]
}
]
}
Got 200 as reponse

Expected behavior
Endpoints together with the nodes are correctly updated.
Screenshots

Environment (please complete the following information):

OS: linux
OPcpublisher Version 2.9.9

Please use 2.9.10, this was fixed there - I just tested it.


Spoiler popup/notification for new chapters skipping

Why/User Benefit/User Problem
Sometimes chapters might be uploaded but it skipped a few chapters. For example Ch 10 is uploaded but only Ch 1-5 were uploaded before and Ch 6-9 is missing. I read the new chapter and was super confused because I didnt notice the chapter number. Think this will be especially helpful for the people who use the start reading button.
What/Requirements
Have a spoiler warning like how MD does it when there is a gap between chapters.
EX: https://mangadex.org/chapter/710017

upstream code that can be used
https://github.com/tachiyomiorg/tachiyomi/pull/3745
https://github.com/tachiyomiorg/tachiyomi/pull/3755

this was added in 2.6.2


[Format] Editor Config

Description
This adds an .editorconfig to the project.
Review checklist

[x] PR is split into meaningful commits for the ease of reviewing
[-] Description contains clear instructions on how to test the feature (where applicable)
[-] Tests have been written
[x] Appropriate labels have been applied

Codecov Report

Merging #90 (358fa98) into main (2c30773) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              main     #90   +/-   ##
=======================================
  Coverage     90.9%   90.9%
  Complexity      55      55
=======================================
  Files           16      16
  Lines          111     111
  Branches        18      18
=======================================
  Hits           101     101
  Misses           5       5
  Partials         5       5


Pipeline failure log

Need to output the log when the pipeline fails. Currently we only output the results which makes it difficult to debug issues

Note: the main things behind this is due to --rm flag in the docker-run.sh so that the container disappeared and logs were gone when the pipeline running inside the container failed.

Need to extract the full gst_launch log from the container


e2e tests fail half of the times

It seems that almost 50% of e2e fails at "cypress run". It happens randomly on either chrome or firefox tests on many (if not most) PRs at least once.
Can we improve this?

Hi @muhlemmer and @eliobischof maybe we should revert Cypress to  "cypress": "^12.2.0" and check again. Would Cypress 13 have some performance issues in lower resourced machines and affect some of our current tests randomly?

If it is not too much work, feel free to give it a try and open a PR. We will see how it runs and take it from there.

Sure @muhlemmer I'll prepare a PR this evening reverting to the latest known working cypress version.

We've upgraded to paid runners and reconfigured in https://github.com/zitadel/zitadel/pull/6916. At first look it seems to be the proper solution. Will analyze and close this issue if everything stays ok.

e2e seem stable. Closing


Should be able to drag database table to the code editor, to instead of typing table name

SQL Operations Studio Version:
Version 0.26.7
Commit 723c7cb3bfd80d87fc8a1141632089cd4808495c
Date 2018-02-16T21:34:47.034Z
Shell 1.7.11
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64
Steps to Reproduce:

dupe #296


Trying the common/-web way without React, still needs React to be installed

Hey, I'm currently trying the non-react way to set it up with Svelte, only installed:
  "dependencies": {
    "@effect/schema": "0.62.6",
    "@evolu/common": "3.1.3",
    "@evolu/common-web": "5.0.3",
    "effect": "2.3.5"
  }

But when starting the serve process it stops with:
✘ [ERROR] Could not resolve "react-dom"

    node_modules/.pnpm/@evolu+common-web@5.0.3_@evolu+common@3.1.3_@sqlite.org+sqlite-wasm@3.45.1-build1/node_modules/@evolu/common-web/dist/PlatformLive.js:5:26:
      5 │ import { flushSync } from "react-dom";
        ╵                           ~~~~~~~~~~~

  You can mark the path "react-dom" as external to exclude it from the bundle, which will remove
  this error and leave the unresolved path in the bundle.

Not knowing anything yet, does this functionality needs to be in common-web? or can it be moved to the react lib?

Ups, that's a bug. Thank you for reporting that. @evolu/common-web should don't depend on react-dom. I will fix it soon.


release for scala-native 0.5

See https://github.com/orgs/typelevel/discussions/135
Also I'm completely out of clue:
https://github.com/milessabin/shapeless/actions/runs/8838312727/job/24528052841
[error] Caused by: java.net.ProtocolException: Server redirected too many  times (20)

Some discussion here: https://discord.com/channels/632277896739946517/920140670394589224/1235965389024923679
Are we publishing to the wrong sonatype destination somehow?

@joroKr21 I think I've probably resolved the published problem, do you want to give it another try?

2.4.0-M1 is on Sonatype. Any ETA for 2.4.0 final? Needed for Twiddles which is needed for Skunk.

2.4.0-M1 is a dud. The only reason to have 2.4.0 would be cross-compiling with Scala 3 but this is hard to achieve.
Regarding releasing for Scala Native 0.5 I'm fine with it. I don't think we should wait for the rest of the typelevel ecosystem because Shapeless is one of the slow-moving projects that is close to the root of the dependency tree.
I think we should do two releases though:

All changes merged until now
Bump of Scala Native (and also drop Scala 2.11)

@joroKr21 Agreed with the above.

Sounds like a plan. Note that most other projects have been first releasing a patch release with all changes to date using 0.4 and then bumping the minor version and releasing against 0.5. Following that model would mean shapeless 2.13.11 against native 0.4 and then same source tree as shapeless 2.14.0 against 0.5.

https://github.com/milessabin/shapeless/releases/tag/v2.3.12


How do you browse to next page programmatically?

I'm trying to hijack the <enter> key, so that pressing enter in a <input type="text"> shifts focus to the next question.
This works fine when there are multiple questions on the same page, but fails when trying to turn the page, or trigger submit.  This appears to be because of some strange deferred validation, stemming from this code:

// validate 'required', checking value in Model (not View)
validReq = !( n.enabled && n.inputType !== 'hidden' && n.required && getDataNodeObj().getVal()[ 0 ].length === 0 );
> _— https://github.com/enketo/enketo-core/blob/master/src/js/Form.js#L2011_

It seems like the value of `getDataNodeObj().getVal()[ 0 ]` is the **_previous_** value of the field, whereas `n.val` is up-to-date.  Can you give any more background on what the comment means, specifically:

1. why is it important to check the Model and not the View?  And
2. how do I get the Model to update before submit/next triggers the validation?

For reference, here's my code:

```javascript
$('.enketo form').find('input').on('keydown', function(e) {
  if(e.keyCode !== 13) {
    return;
  }
  var $thisQuestion = $(this).closest('.question');

  // If there's another question on the current page, focus on that
  if($thisQuestion.attr('role') !== 'page') {
    var $nextQuestion = $thisQuestion.find('~ .question');
    if($nextQuestion.length) {
      return $nextQuestion.first().trigger('focus');
    }
  }

  // TODO here I've tried a number of things including `validate`, `change`,
  // `blur`, and also doing nothing at all.
  $thisQuestion.trigger('change');

  // If there's no question on the current page, try to go to next
  // page, or submit the form.
  var enketoContainer = $thisQuestion.closest('.enketo');
  var next = enketoContainer.find('.btn.next-page:enabled:not(.disabled)');
  if(next.length) {
    next.trigger('click');
  } else {
    enketoContainer.find('.btn.submit')).trigger('click');
  }
});

Hmm.

is the previous value of the field

It shouldn't be. Value is set here: https://github.com/enketo/enketo-core/blob/master/src/js/Form.js#L2001 (the function name should perhaps be validateAndUpdate or something).
Maybe there is a bug?
What if the question is a select minimal, map-based geo input etc? I think both blur and fakeblur might be interesting but am not sure. Or maybe valuechange.enketo if it doesn't fire too often.

As I'm triggering a validate event, I don't think line 2001 is executed: rather the validation occurs at https://github.com/enketo/enketo-core/blob/master/src/js/Form.js#L1997 - i.e. without a call to _dataNodeObj.setVal().
I think either:

I'm firing the wrong type of event; or
setVal() should be called prior to the validate() call on line 1997

I think I may have lost you. Pressing enter will fire the change event (I think) which sets the value in the model via validateInput. Is your handler executing before the change event is fired by the browser maybe?

Apologies - didn't realise I'd posted that.  There is no change event fired, and I was mistakenly triggering it on the .question instead of on the <input>.  I've changed to trigger it on the <input>, and everything seems fine now.  I'm not sure why the change event is not triggered automatically, but perhaps because of the call to event.preventDefault().

ah yes, that must be it, together with apparently getting precedence


Upgrade JGit

See if tests pass, test lots of other things, update pom.xml, test what happens to the rest of us when that pom.xml change happens

Updated!
The release() method was gone from the ObjectReader class, but there was a close() method introduced, so I replaced release() with that.


Limit pytest version

Fix https://github.com/huggingface/optimum-intel/actions/runs/8879625147

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


add qualifiedNameOf for nested members

My usecase is querying ElasticSearch where nested case classes have been inserted.
I'm really not sure about the implementation as I'm not too familiar with macros. Tbh I'm surprised I got it to work at all.. But the test passes ;)

hey @dwickern, any feedback on this one?

@dwickern ?

hi @dwickern , any chance this PR is getting merged?

@an-tex another use case is use it with Spark Datasets to get a nested column name.
@dwickern could it be merged?

hi @oleksandr-yatsuk . i've tried to reach @dwickern already even on a few other channels with no success :( in the meantime i'm publishing my builds from my own fork to
Global / resolvers += "antex public" at "https://mymavenrepo.com/repo/zeKhQjbzBED1vIds46Kj/"
val libraryDependencies += "com.github.dwickern" %% "scala-nameof" % "3.0.0-antex"

feel free to use those


Selenide closes alert automatically in IE 11

Whenever I run tests in IE11, Selenide automatically closes all alerts in IE11 without any explicit configuration. I used to run test scripts with NativeEvents = false for IE11.
Any idea, why it behaves like this ?

Selenide suppresses all alert and confirm dialogs if you configured selenide.dismissModalDialogs system property. Otherwise, alerts should work as usual.

I didn't set any configuration. I'm using the default setting.
I tried to explicitly set dismissModalDialogs to false, but still it's
behaving the same. Alerts are dismissed automatically
Regards,
Lesly Arun Franco
On 25-Mar-2016 4:29 am, "Andrei Solntsev"<EMAIL_ADDRESS>wrote:

Selenide suppresses all alert and confirm dialogs if you configured
selenide.dismissModalDialogs system property. Otherwise, alerts should
work as usual.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/codeborne/selenide/issues/307#issuecomment-201065142

@leslyarun sorry for the long delay.
I havn't experimented with this issue - partly because I don't like Windows. :)
Is the problem still actual?

@leslyarun do you mean alert for BasicAuth?

It seems no feedback here. Closing the issue.


Explain tunnel timeouts

We'll use exitcode 124 to indicate a timeout.
See: https://github.com/aptible/dynamic-uno/pull/12
cc @fancyremarker

It turns out that I had also broken tunnels in #218, which we fortunately never released (this would have been caught by the integration tests); that was never released, though, but it's another reason to merge this fix.

Codecov Report

Merging #221 into master will decrease coverage by 0.1%.
The diff coverage is 88%.

@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
- Coverage   90.47%   90.36%   -0.11%
==========================================
  Files          43       43
  Lines        1449     1464      +15
==========================================
+ Hits         1311     1323      +12
- Misses        138      141       +3

Impacted Files
Coverage Δ

spec/aptible/cli/helpers/tunnel_spec.rb
100% <100%> (ø)
:arrow_up:

lib/aptible/cli/helpers/tunnel.rb
90% <75%> (-4.55%)
:arrow_down:

lib/aptible/cli/subcommands/logs.rb
100% <0%> (ø)
:arrow_up:

lib/aptible/cli/helpers/app.rb
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


multiple modals appended to their parent element in reversed order

When multiple modals are open at the same time, they are attached to their parent element in order, which is opposite to the order in which they were opened. This behavior first appeared in version 1.0.
If a custom window template is specified for the modals, this forces us to specify z-indexes for modals (using the ugly ng-style="{'z-index': 1050 + index*10}" in window template).
Before version 1.0, it was possible to define a single z-index for modals in a CSS file and the natural order of html displayed the modals in a correct order.
Furthermore, there seems to be a bug in the indexing system. Let's say multiple modals are open at the same time. When I close any modal other than the one on the top and then open a new modal, the newly opened modal has the same index & z-index as the last open modal, i.e. breaking the z-index calculation. All of which worked before version 1.0 with the natural order of modals.
Is closing modals in the middle of modal stack unsupported? Can we get the non-reversed order of modals back?
Angular: 1.4.7
UIBS: 1.2.4
Bootstrap: 3.3.6

@anakreon please be respectful of our time and use the issue template we've put in place.  The reason we have it is that 1) in makes the investigative portion of our jobs that much quicker (and thus, able to react to YOUR needs faster), and more importantly 2) it forces you to really think about your problem and requires to to reproduce it in a sandboxed environment.
I'm going to close this until you can create a minimal reproduction.  Feel free to use the plunker link from our demo site to get you started.


Virtual Host not working on port's other than 8123

Problem/Motivation
When going to https://****duckdns.org:8123 all is good but when going to https://****duckdns.org:4865 or another port to my pi's domain I get this error on chrome

This site can’t be reached zeak.duckdns.org refused to connect.
Try:
Checking the connection
Checking the proxy and the firewall
ERR_CONNECTION_REFUSED

Additionally, under the Hosts section, I have this code:
  "hosts": [
    {
      "name": "laundrycam",
      "ip": "<IP_ADDRESS>"
    }

and it does nothing same error as above.
Expected behavior
The expected behavior is that the urls would resolve to the correct ip's
Actual behavior
errors out
Steps to reproduce
n/a
Proposed changes
I have no idea the only workaround I've been able to use the direct IP addresses.

Looks like this issue goes back 2 years https://github.com/hassio-addons/addon-pi-hole/issues/6

I have the same failure output as in #6
i try to access my pi2-Hassio via zerotier.
Access via zerotier to HA via port 8123 works perfect, but to the pihole-port 4865 it shows this
Failed Host Check: 192.168.xx.xx vs 192.168.y.y, , hassio.local, pi.hole, localhost, hassio, hassio.local
while 192.168.xx.xx is the zerotier ip and 192.168.y.y the local Hassio-IP
no further log entries at HassOS.
System:
RasPI2
HassOS 2.12
Hass.io supervisor 167
piHole-Addon 3.0.2
zerotier-Addon 0.3.0

Next version of the add-on uses Ingress, which makes this no longer an issue.


[influxdb-exporter] Add influxdb-exporter chart for Proxmox metric server

Description of the change
Add influxdb-exporter chart to the library. It is an exporter for metrics in the InfluxDB format used since 0.9.0. It collects metrics in the line protocol via a HTTP API, transforms them and exposes them for consumption by Prometheus.
Benefits
Import Proxmox metrics from the integrated metric server (in influxdb format) into Prometheus with a simple ServiceMonitor.

(Chart json: here)
Possible drawbacks
It needs a LoadBalancer or NodePort for the UDP service.

n/a

fixes #

n/a

[x] Title of the PR starts with chart name (e.g. [home-assistant])
[x] Chart version bumped in Chart.yaml according to semver.
[x] Chart artifacthub.io/changes changelog annotation has been updated in Chart.yaml. See Artifact Hub documentation for more info.
[x] Variables have been documented in the values.yaml file.

@onedr0p something went wrong with the merge, the README.md in the master branch is missing the last update (description and app args). Should be as in the PR list of changes and on the merged branch

Looks like you added it manually to the README, you'll need to add that custom config to charts/stable/influxdb-exporter/README_CONFIG.md.gotmpl, l


Chrome console (if selected iphone) shows [Error from backend: Error: Bad state: Not connected to an application.] and no VM

I used chrome on windows to run the app. The problem happens in device mode on chrome and for any iphone. But it works fine if I choose other phones like galaxy S5.
Steps to Reproduce

flutter create sample_project && cd sample_project
flutter run -d chrome
In developer console select device mode then select any apple device (like iphone x)
Refresh the page

Expected results:
Connecting to VM to be able to hot reload & stuff
Actual results:
No connection to VM and error : Not connected to an application

flutter run --verbose -d chrome (I added comments)
[ +423 ms] executing: [D:\flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[ +411 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[   +3 ms] 18116933e77adc82f80866c928266a5b4f1ed645
[   +3 ms] executing: [D:\flutter/] git tag --points-at 18116933e77adc82f80866c928266a5b4f1ed645
[ +356 ms] Exit code 0 from: git tag --points-at 18116933e77adc82f80866c928266a5b4f1ed645
[   +3 ms] 2.5.3
[  +51 ms] executing: [D:\flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[ +240 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[   +2 ms] origin/stable
[   +1 ms] executing: [D:\flutter/] git ls-remote --get-url origin
[ +209 ms] Exit code 0 from: git ls-remote --get-url origin
[   +2 ms] https://github.com/flutter/flutter.git
[ +639 ms] executing: [D:\flutter/] git rev-parse --abbrev-ref HEAD
[ +255 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[   +2 ms] stable
[ +404 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[  +26 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'WindowsUwpEngineArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[   +3 ms] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[ +508 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[   +3 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[  +14 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'WindowsUwpEngineArtifacts' is not required, skipping update.
[   +3 ms] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[   +3 ms] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[   +3 ms] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[   +7 ms] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[   +3 ms] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[   +3 ms] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[ +449 ms] Skipping pub get: version match.
[ +649 ms] Generating D:\dvp\my_project\android\app\src\main\java\io\flutter\plugins\GeneratedPluginRegistrant.java
[ +674 ms] Launching lib\main.dart on Chrome in debug mode...
[ +738 ms] Updating assets
[ +431 ms] Waiting for connection from debug service on Chrome...
[ +527 ms] <- reset
[  +49 ms] D:\flutter\bin\cache\dart-sdk\bin\dart.exe --disable-dart-dev
D:\flutter\bin\cache\artifacts\engine\windows-x64\frontend_server.dart.snapshot --sdk-root D:\flutter\bin\cache\flutter_web_sdk/ --incremental
--target=dartdevc --debugger-module-names --experimental-emit-debug-metadata -DFLUTTER_WEB_AUTO_DETECT=true --output-dill
C:\Users\WinUs\AppData\Local\Temp\flutter_tools.e2ffa870\flutter_tool.e48c0928\app.dill --libraries-spec
file:///D:/flutter/bin/cache/flutter_web_sdk/libraries.json --packages D:\dvp\my_project\.dart_tool\package_config.json
-Ddart.vm.profile=false -Ddart.vm.product=false --enable-asserts --track-widget-creation --filesystem-root
C:\Users\WinUs\AppData\Local\Temp\flutter_tools.e2ffa870\flutter_tools.9e806a58 --filesystem-scheme org-dartlang-app --initialize-from-dill
build\b1b715402d823b7fd5c2b68d2edcb2ce.cache.dill.track.dill --platform file:///D:/flutter/bin/cache/flutter_web_sdk/kernel/flutter_ddc_sdk_sound.dill
--sound-null-safety
[  +76 ms] <- compile org-dartlang-app:/web_entrypoint.dart
[+88423 ms] Waiting for connection from debug service on Chrome... (completed in 89.0s)
[  +12 ms] Synced 26.2MB.
[   +7 ms] <- accept
[  +10 ms] Caching compiled dill
[+1930 ms] [CHROME]:
[  +10 ms] [CHROME]:DevTools listening on ws://<IP_ADDRESS>:13477/devtools/browser/81354596-dada-4e25-a842-19f82fad6df3
[+3028 ms] DwdsInjector: Received request for entrypoint at http://localhost:13460/main_module.bootstrap.js
[  +89 ms] MetadataProvider: Loading debug metadata...
[ +158 ms] MetadataProvider: Loaded debug metadata (sound null safety)
[  +61 ms] DwdsInjector: Injected debugging metadata for entrypoint at http://localhost:13460/main_module.bootstrap.js
[+18189 ms] ChromeProxyService: Initializing expression compiler for main_module.bootstrap.js with sound null safety: true
[ +361 ms] DevHandler: Debug service listening on ws://<IP_ADDRESS>:13501/XrVwyEdQQd8=/ws

[ +127 ms] This app is linked to the debug service: ws://<IP_ADDRESS>:13501/XrVwyEdQQd8=/ws
[  +62 ms] Debug service listening on ws://<IP_ADDRESS>:13501/XrVwyEdQQd8=/ws
[   +9 ms]  Running with sound null safety
[  +28 ms]   To hot restart changes while running, press "r" or "R".
[   +4 ms] For a more detailed help message, press "h". To quit, press "q".
[  +12 ms] An Observatory debugger and profiler on Chrome is available at: http://<IP_ADDRESS>:13501/XrVwyEdQQd8=
[+3423 ms] DevTools activation throttled until 2021-10-27 20:54:23.252308.
[+6458 ms] The Flutter DevTools debugger and profiler on Chrome is available at: http://<IP_ADDRESS>:9101?uri=http://<IP_ADDRESS>:13501/XrVwyEdQQd8=

##################################### Now I select iphone & press refresh

[+46449 ms] DwdsInjector: Received request for entrypoint at http://localhost:13460/main_module.bootstrap.js
[  +17 ms] MetadataProvider: Loading debug metadata...
[  +89 ms] MetadataProvider: Loaded debug metadata (sound null safety)
[  +37 ms] DwdsInjector: Injected debugging metadata for entrypoint at http://localhost:13460/main_module.bootstrap.js

##################################### Chrome console error is shown like 40 times

##################################### Hot restart

[+49398 ms] Performing hot restart...
[  +27 ms] Initializing file store
[  +62 ms] Skipping target: gen_localizations
[  +27 ms] Skipping target: gen_dart_plugin_registrant
[  +10 ms] Skipping target: _composite
[  +25 ms] complete
[ +255 ms] Scanned through 522 files in 230ms
[   +6 ms] Waiting for connection from debug service on Chrome...
[  +12 ms] <- recompile org-dartlang-app:/web_entrypoint.dart 2a83f41f-f8ec-48f7-a6b5-7cee9597b5cd
[   +3 ms] <- 2a83f41f-f8ec-48f7-a6b5-7cee9597b5cd
[  +80 ms] Waiting for connection from debug service on Chrome... (completed in 87ms)
[   +5 ms] Synced 0.0MB.
[   +4 ms] <- accept
[  +19 ms] DwdsVmClient: Attempting a hot restart
[   +9 ms] DwdsVmClient: Attempting to disable breakpoints and resume the isolate

##################################### Nothing happens & no restart performed

Error in chrome console
client.js:24134 Error from backend:

Error: Bad state: Not connected to an application.

 Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at D:\AndroidSdk
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] Connected device (2 available)
    • Chrome (web)      • chrome      • web-javascript • Google Chrome 95.0.4638.54
    • Edge (web)        • edge        • web-javascript • Microsoft Edge 95.0.1020.30

! Doctor found issues in 1 category.

I have also same issue.

Found a cause: we use window.navigator.userAgent in the injected client to detect the browser it is running in. The value changes in iPhone simulation mode to Safari:)
Working on a fix.


Can't run/Can't Install/Not smart enough to check closed issues

Pip install hiddeneye_reborn but it won't work, its still in development

Opening this back as main "issue" for those spamming me with questions
New issues with same context will be referencing this one and people making those will be banned permanently.


Added code to find out if the diff is backward compatible

If a client has generated model for the apis with [@JsonIgnoreProperties(ignoreUnknown = true) ] then any fields added in the response is backward compatible whereas removal/change of type in the existing fields is backward incompatible.
Any existing api/operation removal causes the diff to be backward incompatible
Any removal in the request field is backward incompatible where as any addition in the request field which is not a required field is backward compatible.
TODO: Add more test cases for BackwardCompatibilityTest

Added SchemaDiffCache so that same refs are not compared more than once


feat: add gptlint

📥 Pull Request
📘 Description
This PR is an example of how to add gptlint to AgentOps.
🔄 Related Issue (if applicable)
https://github.com/gptlint/gptlint/issues/17
🎯 Goal
This PR ads a single gptlint rule aimed at enforcing correct spelling and grammar.
🔍 Additional Context
Previous confo w/ Alex.
🧪 Testing

Maybe I'm just running it wrong, but it's not catching English errors I've intentionally introduced:

```
npx gptlint
Ignoring invalid source file: /Users/reibs/Projects/agentops-ai/agentops/examples/langchain_examples.ipynb
Ignoring source file with 2 long lines: /Users/reibs/Projects/agentops-ai/agentops/tests/openai_handlers/_test_gpt_role_function.py (a long line has at least 1024 characters, and a file can have at most 1 long line which will be truncated)
Linter tasks { numFiles: 38, numRules: 1, numTasks: 0, numTasksCached: 38 }
Linter stats; total cost $0.00 {
  model: 'gpt-4-turbo',
  weakModel: 'gpt-3.5-turbo',
  numModelCalls: 0,
  numModelCallsCached: 38,
  numPromptTokens: 0,
  numCompletionTokens: 0,
  numTotalTokens: 0,
  lintDuration: '1s'
}
no lint errors 🎉


Review 3rd party feedback services for integration with docs

We can't offer real-time support, and an LLM is not going to work with the complexity of the API, but could a third party widget on the docs site help? We do have email and slack for support, but those are relatively high friction. Imagining something like:

Was this helpful? Yes No
Anything else we should know? ______________
Include an email if you'd like a response: ______________

And it gets sent to our slack or email, and there's no software for us to maintain.

Annie: we just want thumbs up / down.
Next step: evaluate offerings and find the cheapest that offers this.

Annie suggested I loop in Sharon. Might pick this up next week.


connections marked as 'established' with half-duplex UDP connectivity

Inbound connections get marked as established when first receiving a heartbeat frame over UDP, which are the only frames that get transmitted prior to the connection being fully established. By contrast, outbound connections get marked as established when receiving the ConnectionEstablished message over TCP, which is sent by the peer when it first receives a heartbeat.
Consequently, when UDP is blocked in the opposite direction of TCP connection establishment, then the connection still gets marked as 'established' by both peers.

To reproduce...
host1$ weave launch
host1$ iptables -I FORWARD -d <host2-ip> -p udp -j DROP
host1$ docker logs -f weave
host2$ weave launch $HOST1 <host1-ip>
host2$ docker logs -f weave

Observe that both logs show the connection as getting established. But pinging between a pair of containers does not work.
I reckon 'established' should mean "we know our packets make it to the other end". That fits in well with route calculations, which only take into account symmetric connections. In future we could actually get more clever here and work with asymmetric connections.


Clock drift issues

Our software has issues due to clock drifts between EC2 instances using this AMI.  We see it more in region: ap-northeast-2.
Relevant issues:

#249
#321

I'm going to close this as a duplicate of #249 and keep looking into it there. Can you add details of how much drift you're observing?


FIX: create autoscaler policy with agnostic information

Changed the optimizer code to create policies with the structure as in the following example:
 policies:
      - autoscaling:
          type: seaclouds.policies.autoscaling.AutoScalerPolicy
          metric: seaclouds.metrics.requestPerNode
          autoscaler.resizeDownStabilizationDelay: 120000
          metricLowerBound: N1
          metricUpperBound: N2
          minPoolSize: N3
          maxPoolSize: N4

Current coverage is 37.97%

Merging #267 into master will increase coverage by +1.24% as of 89773d8

@@            master    #267   diff @@
======================================
  Files          329     331     +2
  Stmts        12064   12292   +228
  Branches      1548    1573    +25
  Methods          0       0
======================================
+ Hit           4432    4668   +236
+ Partial        449     442     -7
+ Missed        7183    7182     -1

Review entire Coverage Diff as of 89773d8

Powered by Codecov. Updated on successful CI builds.

@perezp , thanks a lot for this.
Could you provide an example of a generated ADP with policy?

@kiuby88
Here it is an example of ADP for an input AAM of a three-tier app www+ws+db
https://gist.github.com/perezp/704d148cb2e65b3c2fff

:+1:  merge!!!!!!!!

Thank you @kiuby88


[Bug]: Differentiation between version missing and module missing needs to be handled

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Its impossible to know that you need to just republish the existing valid package for the suspended states to go away,It gives Import CTA with missing module error after first import. Its very misleading
It gives Import CTA with missing module error after first import. Its very misleading
Steps To Reproduce

add relevant package
import an app which consists instances out of these packages
You will get suspended states , But the CTA will be Import Package with messages missing modules/packages which is incorrect. It should have guided the user to just plain refresh

Public Sample App
No response
Environment
Production
Severity
Medium (Frustrating UX)
Issue video log
No response
Version
Cloud/Release

@pedro-santos-rodrigues  Need direction


Co-author white-paper on MLxAI - RAGVar

Co-author white paper with David Beaumont on the product of the MLxAI code-a-thon - RAGVar. This is a potentially useful bit of software that we want to write a little bit of a paper on to serve as a demonstration of our initial work.

I believe this has been shelved for now.