feat: add jamietsao/random-winner

https://github.com/aquaproj/aqua-registry/pull/7788 jamietsao/random-winner: Wrote a silly program to select a random winner from my team
$ aqua g -i jamietsao/random-winner

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well. Please confirm if this package works well yourself as much as possible.
Command and output
$ randowm-winner --help
Usage of /Users/user/.local/share/aquaproj-aqua/pkgs/github_release/github.com/jamietsao/random-winner/v0.1/random-winner-v0.1-darwin-amd64.zip/random-winner:
  -names string
        Comma separated list of names

Example:
$ random-winner -names alice,bob,carol,dave
First to 1000 wins!
[ alice _________________________________________________🐌  ]
[   bob ___________________________________________________🐌]
[ carol ________________________________________________🐌   ]
[  dave __________________________________________________🐌 ]

  FINAL RESULTS
------------------
   bob  1000
  dave   989
 alice   974
 carol   959

 TOTAL  3922

The winner is ....... BOB!

Reference

README.md

https://github.com/jamietsao/random-winner/blob/master/README.md

Thanks!

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.93.0


Remove missed use of deprecated SecurityGroupId in verifier test

This was missed in https://github.com/openshift/osdctl/pull/592

/retest

/lgtm
/approve


Fixes #2112 - custom password reset emails

Notes:

The main work here is supporting custom reCAPTCHA site keys in the /auth/resetpassword endpoint
Also added support for sendEmail = false in the /auth/resetpassword endpoint
Added type to PasswordChangeRequest which can be invite or reset
Moved the "Custom Welcome Emails" docs to simply "Custom Emails", because there is a lot of overlap between the two
Updated the "Custom Emails" bot, which is now driven by PasswordChangeRequest rather than ProjectMembership

coverage: 94.053% (+0.005%) from 94.048% when pulling 4ab63c5919ac2826f5df842b66ccbc4b2ee2b317 on cody-custom-password-reset-emails into 9a0e69082373d12f34321ba0db1c93a7273a18a3 on main.


Readme update for sample/asset.

We should also add the LRI badge in the README while we are at it, and remove the assets directory and references to it.

Let's delete the assets folder while we are at it.


Support for retriable and timeout configuration

This PR provides ActiveGraphQL with:

Support of retry strategy based on Retriable.
Support of requests timeout configuration.

Having these changes in place, an ActiveGraphQL model could be configured doing something like:
class Model < ActiveGraphQL::Model
   configure url: 'http://some.api.domain',
             timeout: 0.1,
             retriable: { tries: 10,
                          base_interval: 0.5,
                          max_interval: 1 }
end

Why not simply pass the options given to retriable to the Retriable.retriable block? Maybe I want to use some options that you have not added to retriable_supported_options (like multiplier).
The same could be argued for http options ( http: { timeout: 0.1 } to easily support all available options of the httparty gem such as http_proxyaddr or the authentication ones.

Yes @menno it makes sense, it will be much more flexible and understandable.
I will change it 👍 .

Great, looks good 👍


Feature: Anvil shell completions

Component
Anvil
Describe the feature you would like
anvil shell completions
both forge and cast have this already.
usr:~ $ anvil bash completions > /usr/local/etc/bash_completion.d/anvil

Additional context
part of https://github.com/foundry-rs/foundry/issues/2396

thanks @gakonst and @mattsse !


Revert "fix(deps): update dependency esbuild to v0.22.0"

For some reason, the new bundler version does not inline the @actions/core and other dependencies: https://github.com/EricCrosson/install-github-release-binary/blob/ada7367625c7b059571215da2fed6b72bda146e6/dist/index.js#L30-L31
In the previous release, the code looks like this: https://github.com/EricCrosson/install-github-release-binary/blob/a3c4ee9462fa1f8ad5e76478cc587633c394f79c/dist/index.js#L69-L79
This causes an error when running the latest version of this action:
Error: Cannot find module '@actions/core'
Require stack:
- /home/runner/work/_actions/EricCrosson/install-github-release-binary/v2/dist/index.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1145:15)
    at Module._load (node:internal/modules/cjs/loader:986:27)
    at Module.require (node:internal/modules/cjs/loader:1233:19)
    at require (node:internal/modules/helpers:179:18)
    at Object.<anonymous> (/home/runner/work/_actions/EricCrosson/install-github-release-binary/v2/dist/index.js:30:21)
    at Module._compile (node:internal/modules/cjs/loader:1358:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)
    at Module.load (node:internal/modules/cjs/loader:1208:32)
    at Module._load (node:internal/modules/cjs/loader:1024:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:174:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/home/runner/work/_actions/EricCrosson/install-github-release-binary/v2/dist/index.js'
  ]
}

This reverts commit 4453ccd2aad5b7b579c1c2f20b70a35b977ad26f to un-break workflows that call this action using a major version label (such as here)

From the 0.22 Changelog

With this release, esbuild will now omit packages from the bundle by default when the platform is node (i.e. the previous behavior of --packages=external is now the default in this case). Note that your dependencies must now be present on the file system when your bundle is run. If you don't want this behavior, you can do --packages=bundle to allow packages to be included in the bundle (i.e. the previous default behavior). Note that --packages=bundle doesn't mean all packages are bundled, just that packages are allowed to be bundled. You can still exclude individual packages from the bundle using --external: even when --packages=bundle is present.

That seems likely to be the issue

Thanks @bitgopatmcl!


Add client implementation

I've pulled in clap for argument handling, chrono for time pretty-printing, and hex to parse ed25519 public keys.
If a public key is specific on the command line, the client performs all of the required validation. However, it does not yet support request chaining as described in ECOSYSTEM.MD
Let me know if you have any questions about how I've implemented things.

I didn't see this until just now (note to self: update Github notification settings!).
Thanks a bunch I'll take a look right away.

Merged!
There's a nit or two, I'll comment on those separately and we can evolve the code from there.


Unsubscribe and re-subscribe to different joint state topics at runtime

[x] I am at the right place and my issue is directly related to ROS#. General technical questions I would post e.g. at ROS Answers or Stack Overflow. For library-specific questions I would look for help in the corresponding library forums.
[X] I have thoroughly read the Contributing Guideline and writing this issue is the right thing to do in my case.

I have a question!

[X] I searched the Wiki, open and closed issues for an answer. I tried my best to find the answer by myself without success. I believe that the discussion we will have in this issue, and the solutions we might find, will help me, and likely other community members who have a similar problem.

Here is my question:
I want to unsubscribe from my current joint state topic and subscribe to an other joint state topic at runtime with the same robot game object. Is there a way to this at runtime?
So far I wait unitl I press a button to subscribe to a joint state topic. But I couldn't figure out how to unsubscribe and re-subscribe. I tried to remove/destroy the comoponents and to add them again with the different topic name, but that didn't work either.  Also, I tried to disable the JointStateSubscriber varibale (SetSubscribeJointStates(false)) and enable it again with a different topic at the JointStateSubscriber.cs component.
Those are the scripts components that are added at runtime:

JointStatePatcher.cs
JointStateSubscriber.cs

Relevant Code:
1  RosSharpObject.AddComponent<RosSharp.RosBridgeClient.JointStatePatcher>();
2  // Set Urdf Robot gameobject
3  RosSharpObject.GetComponent<RosSharp.RosBridgeClient.JointStatePatcher>().UrdfRobot = robot_hologram.GetComponent<UrdfRobot>();
4 // Set subscribing value for joint states --> adds JointStateSubscriber.cs to game object
5  RosSharpObject.GetComponent<RosSharp.RosBridgeClient.JointStatePatcher>().SetSubscribeJointStates(true);
6  RosSharpObject.GetComponent<RosSharp.RosBridgeClient.JointStateSubscriber>().Topic = TopicDropdown.options[TopicDropdown.value].text;

My OS is: Windows 10
My Unity Version is: 2020.3.16f1
My ROS Distribution is: Noetic
My Build Target Platform is: UWP

Hi @Gescheld
you will have to modify UnitySubscriber.
This basic version just subscribes on Start().
You have to add e.g. the following functionality in e.g. Update():
if your doChangeSubscription condition is satisfied: unsubscribe and resubscribe

Hi @MartinBischoff,
Thank you for the fast answer. I don't have the opportunity to try this at the moment, but I will test it next time :)


Add Auto Configurations

NOTE: Implementation on this issue can be started only after Creating App Skeleton issue has been closed.
Add the auto configurations as seen in the nexmo-spring-boot-autoconfigure sub-module(The vonage-spring-boot-autoconfigure folder should already exist). The filenames and configurations should be the same but written in java.
Also, make sure the pom.xml exists(It should be created in issue #2) The make sure all of the configurations exist from the corresponding nexmo spring boot starter pom file for the auto configurations sub-modules then modify the file as indicated below:
pom.xml Modifications

[ ]  Remove any Kotlin plugins or dependencies

[ ]  Make sure the <parent> property is under the <modelVersion> property and specifies the correct parent dependency

[ ]  artifact version should be 1.0.0

[ ]  update assertj-core dependency to the latest version

[ ]  update the jacoco-maven-plugin dependency to the latest version

[ ] Change the license name in the<license> property from MIT to Apache 2.0 and make sure the URL points to the correct LICENSE.md file in this repo.

@yallen011 I'd like to take this on once prerequisite issue is done.

@j4ckofalltrades if you would like, you can take on the prerequisite since no movement have been made towards it. Let me know if you want it.


Mat.calc_hist example

I am trying to create a histogram for an image. My rust ability is not that good.  Can you please give and example.
I plan to use cv for a project. I plan to ask many questions so I would be happy to work with you to improve this project.
    let channels = 1 as i32;
    let hist_size = 256;
    let r1 = 32.0 as f32;
    let ranges: *const f32 = &r1;
    let par = &ranges;
    let tst = mat.calc_hist(&channels, Mat::new(), 2, &hist_size, &par );

new code, this does not crash, but I an still working on it
        let channels = &1 as *const i32;
        let hist_size: [i32; 2] = [64, 32];
        let sr = [0.0, 256.0].as_ptr();
        let hr = [0.0, 1024.0].as_ptr();
        let range = [sr, hr].as_ptr();
        let tst = mat.calc_hist(channels, Mat::new(), 2, hist_size.as_ptr(), range );
        println!("{:?}", tst);

See hs_hist example. Code snippet as below:
    let hbins = 30;
    let sbins = 32;
    let hist_size = [hbins, sbins];

    let hranges = [0.0, 180.0];
    let sranges = [0.0, 256.0];
    let ranges = [hranges.as_ptr(), sranges.as_ptr()];

    let channels = [0, 1];

    let hist = hsv.calc_hist(
        channels.as_ptr(),
        Mat::new(),
        2,
        hist_size.as_ptr(),
        ranges.as_ptr(),
    );

P.S. Apologies for the late reply. I was traveling.


vuex2..0   mapActions

when i use vuex2.0   -- mapActions
the code like this https://vuex.vuejs.org/zh-cn/actions.html
...mapActions([
'increment' // 映射 this.increment() 为 this.$store.dispatch('increment')
]),
vue-cli  return a error

node build/dev-server.js

(node:5264) DeprecationWarning: loaderUtils.parseQuery() received a non-string value which can be problematic, see https://github.com/webpack/loader-utils/issues/56
parseQuery() will be replaced with getOptions() in the next major version of loader-utils.
ERROR  Failed to compile with 1 errors                                                                                17:21:56
This dependency was not found in node_modules:

babel-runtime/helpers/extends in ./~/.npminstall/babel-loader/6.3.2/babel-loader/lib!./~/.npminstall/vue-loader/11.1.0/vue-loader/lib/selector.js?type=script&index=0!./src/components/Hello.vue

Did you forget to run npm install --save for it?

Listening at http://localhost:8080

Update vue-loader


Migrate nomad backend plugin to new backend system

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

hey @lavanya-sainik-ericsson :wave:
Looks like you'll need to update the api-reports here so that we can see what's happened!
yarn buiild:api-reports should generate you some new markdown files that will need pushing to this branch in order to make the build green :tada:

Hi benjdlambert,
Thanks for your comment. I have generated the api-report. Please have a look and let me know if any more comments / corrections needed.

Hey benjdlambert,
Thanks for your comment. Done that change :) Please have a look and let me know if any more comments / corrections needed.

Hey benjdlambert,
Thanks  :) Please have a look again and let me know if any more comments / corrections needed.


Proposal to swap out forward and backslash characters

Out of 64 characters used in Hatnotation, there are 28 punctuation characters in use from a range of 32 possible characters in the string library of which 4 are currently not used in the base64library library index. Because one of the 28 characters already in use is causing a rendering issue (bug), namely the single backslash character \, I propose to use two of the unused punctuation characters, namely the closing curly bracket } to replace the single backslash \ and the opening curly bracket { to replace the forward backslash /. This change will affect zero-indexed value 59 (111011, which represents 60 th value) for the backslash character and index value 50 (110010, which represents the 51 st value) for the forward slash character.

Closed with PR: https://github.com/hatgit/hatnotation/pull/5


Add Javascript target

🗒  Notes

Initial implementation of the JS target

Still has some work left to do (you'll see a number of TODO()s)
Wanted to lay the initial ground work to allow for integration of additional targets

Dropped discoverServices from public API

This will be handled automatically on connect
Might re-introduce function on the public API later

🔧  Features

Observations remain active across connections

Any subscribers of observes will remain active on connection drop and continue to pick up new characteristic changes upon reconnect (without having to re-subscribe)

Peripheral.connect can be invoked from Event.Disconnected

Allows for simple reconnect logic

LGTM, pending sorting out that whole issue under discussion re holding direct references to the underlying system objects.
Lots of debug console logging which is immensely useful but will need to be removed at some point, recommend parking a ticket or issue to clean them out before going full production.

Good thinking. Logged under #6.

Performed the discussed refactor, sans proper platform type caching.
Basically, every read/write will do a lookup for the platform type (e.g. characteristic or descriptor). In other words, all characteristic and descriptors objects on the public API will be lazy (and always lazy)...for now.
Decided it's best to go with the simple inefficient approach at first. In a majority of cases the lookup will be very fast (as most users will only have a few characteristics/descriptors).
I figured it'd be better to optimize later after all the platforms are up and running so we can see how the API settles.
I'll do some updates to the MacOS branch to see how this refactor effects that API and make sure it doesn't kill performance in Native threading land; then I'll re-request review.

I'll do some updates to the MacOS branch to see how this refactor effects that API and make sure it doesn't kill performance in Native threading land; then I'll re-request review.

Was a bit more work than expected, so I'm going to postpone testing that platform for the time being and focus on it again when I start putting a PR up for it.


Add MaxMind GeoLite db?

Would it be possible to add the MaxMind GeoLite db as a backup/alternate solution in addition to the web-based GeoIP lookups?

Although it would be a simple proxy to another package (geoip2? as an optional dependency), I think it makes sense to provide such geocoder class with an interface similar to the others. Reverse geocoding would not be implemented, of course.
But are there any geocoders in geopy which support online lookup for IP addresses? I know about maxmind API service, but it's not currently implemented in geopy. In that case it would be nice to have both implemented (as two different geocoder classes).

On the second thought I think there's no point in having it in geopy. geoip works specifically with IP addresses, while geocoders work with points of interest. These two applications are different.
I'm closing this, and I would suggest to use other libraries for working with geoip instead.


Include capture attribute

input type="file" accept="video/*" capture
http://www.w3.org/TR/html-media-capture/
https://dev.opera.com/articles/media-capture-in-mobile-browsers/

Duplicate of #1115.


Refactor Value.Record and Value.RecordUpdate to have Dict as parameter instead of List

Terms

THIS SOFTWARE IS CONTRIBUTED SUBJECT TO THE TERMS OF THE TERMS OF THE CCLA DATED 2017-11-07 WITH FINOS/LINUX FOUNDATION (FORMERLY THE SYMPHONY SOFTWARE FOUNDATION CCLA).
  THIS SOFTWARE MAY BE REDISTRIBUTED TO OTHERS ONLY BY EFFECTIVELY USING THIS OR ANOTHER EQUIVALENT DISCLAIMER IN ADDITION TO ANY OTHER REQUIRED LICENSE TERMS.

@bekand There are merge conflicts in the PR.


Response Body Rules are Not Executed

Hi,
Response Body Phase rules are not executed by ModSecurity, although content-type haeder of response is matched. Specifically, when I check the debug log, it says
[162029056252.586970] [/student.xml] [4] Starting phase RESPONSE_BODY. (SecRules 4)
[162029056252.586970] [/student.xml] [5] Response Content-Type is . It is not marked to be inspected.

I guess, it couldn't reach out the content-type header. I sent the GET request using Postman, and I can see the response headers contain a valid Content-Type header.
Debug log: https://gist.github.com/frkngksl/ae0e2744f04cde6e9801a313f92c46a7
Conf file: https://gist.github.com/frkngksl/e51083e894affc856822b763d7fb743b
Example Rule: any phase 4 rule actually. But for example:
SecRule RESPONSE_CONTENT_TYPE "^.*/xml"\
    "id:875324,\
    phase:4,\
    deny,\
    t:none,\
    msg:'POST without Content-Length or Transfer-Encoding headers',\
    log,\
    setvar:tx.schemafile='/var/www/student.xsd',\
    exec:/var/www/check_schema.lua"

To reproduce:
1- Put an XML file to Apache Web Directory
2- Send a GET request to that file

Hi @frkngksl ,
Yes, that is what appears to be happening.
I was not able to reproduce what you have reported in my 3.0.4 test environment.
If you'd like to share a .pcap file -- or other format that preserves all bytes  -- of a minimal reproduction, I could investigate further.
If you prefer not to do that, one thing you could try to do on your end is narrow down whether it is the header name that is not being found or if the header name is being found but something is going wrong with the associated value of the header.
For example, you could confirm whether the former is the case with something like:
SecRule RESPONSE_HEADERS_NAMES "@rx content-type" "id:1003,phase:3,pass,log,auditlog,t:none,t:lowercase"

Was there anything further on this?
There are no known parsing -- or related -- bugs that would cause what you have reported.  Without additional information there really isn't anything that can be done with this item.


bug: #if, #ifdef, and #ifndef branches in C/C++ have the same color

Neovim version (nvim -v)
v0.10.0
Terminal / multiplexer
Kitty
Describe the bug
Branches of conditional directives, such as #if, #ifdef, or #ifndef in C and C++ appear as if they both were true.
This is how it looks in Rosé Pine theme:

And this is how it should look:

Repro
No response

I think I've figured out what the problem is. In #155, linking @lsp.type.comment was reset to {}, because by default it was either Comment, or @comment, and it made some TODO comments look like normal comments.
However, in most themes @lsp.type.comment is set to @comment, and, unfortunately, it causes the same problems, but I've never experienced them, because I've always used that plugin (https://github.com/folke/todo-comments.nvim.git).
As a fix for C and C++, this can be done (in config):
require("rose-pine").setup({
    highlight_groups = {
<EMAIL_ADDRESS>= { link = "@comment" },
<EMAIL_ADDRESS>= { link = "@comment" }
    }
})

Because it doesn't break TODO comments in C and C++:


Merge main into live

Please don't squash-merge this PR.

closing and reopening for a fresh build


revert to non-libuv worker

Immediate benefit is that it fixes https://github.com/supabase/pg_net/issues/66, added a test for proving it.
Also planning to add mitmproxy's pathod for testing against other http faillures like disconnects.
Will refine other things along the way.

This one should close all the pending bugs.
For https://github.com/supabase/pg_net/issues/59, the only relevant remark left was the hardcoded database name, which is now fixed. The other remarks are no longer relevant since libuv is gone, malloc is no longer used, etc.
For https://github.com/supabase/pg_net/issues/42, I wasn't able to reproduce locally but I did saw a case in production where after a Server returned nothing (no headers, no data) response the worker stopped working. I've added a test with pathod that proves we keep working if that happens.
https://github.com/supabase/pg_net/issues/26, is no longer reproducible but I've corrected a FIXME in the code where curl_slist_free_all wasn't being used.
The other bugs should be clear with the commits.


DGOSS: Honor ${GOSS_FILE} when copying back from container to host

Checklist

[ ] make test-all (UNIX) passes. CI will also test this
[ ] unit and/or integration tests are included (if applicable)
[ ] documentation is changed or added (if applicable)

Description of change

While testing docker images with a custom GOSS_FILE setting, I found dgoss was copying the modified file back to goss.yaml instead of ${GOSS_FILE}.
Additionally the images I was testing were derived from a minimal Alpine linux image, which basically just has /bin/sh and /bin/ash as symlinks to busybox, so I also changed the shell spec so dgoss would run there and didn't encounter any POSIX compatibility issues.

Goss_file change makes sense.
For the shebang are you sure sh supports double brackets? I am fairly certain it doesn't, but haven't used pure posix-sh in ages.

I would have expected an immediate failure if there was unsupported syntax but everything ran fine for me with the shebang change.
Looking at https://busybox.net/downloads/BusyBox.html they do list [[ as a supported function. And on https://www.busybox.net/ it appears to have been included since 1.26

test: make [ and [[ forms individually selectable

I don't see Busybox version mentioned on the Alpine release notes prior to 3.9, but Alpine 3.9 included Busybox 1.29
https://alpinelinux.org/posts/Alpine-3.9.0-released.html
I  don't have Docker available at the moment to test the Alpine images but installing Busybox 1.30.2 and testing it seems to work as expected
4/25/2020 4:26:32 PM Justin : ~
> busybox cat --help
BusyBox v1.32.0-FRP-3445-g10e14d5eb (2020-04-11 10:50:43 BST) multi-call binary

Usage: cat [-nbvteA] [FILE]...

Print FILEs to stdout

        -n      Number output lines
        -b      Number nonempty lines
        -v      Show nonprinting characters as ^x or M-x
        -t      ...and tabs as ^I
        -e      ...and end lines with $
        -A      Same as -vte
4/25/2020 4:26:34 PM Justin : ~
> busybox sh
C:/Users/Justin # [[ "foo" == "foo" ]] && echo foo
foo
C:/Users/Justin # [[ "foo" != "foo" ]] && echo foo
C:/Users/Justin # ls .gitconfig*
.gitconfig
C:/Users/Justin # [[ -f .gitconfig ]] && echo foo
foo
C:/Users/Justin # [[ -f .gitconfig2 ]] && echo foo
C:/Users/Justin #

Running the script with /bin/sh through https://www.shellcheck.net/ or shellcheck cli it prints the following:
In extras/dgoss/dgoss line 23:
    if [[ $id ]];then
       ^-- SC2039: In POSIX sh, [[ ]] is undefined.

https://github.com/koalaman/shellcheck/wiki/SC2039
Some sh implementations are more than just POSIX, but not all:
$ docker run -it --rm alpine:3.9 sh -c '[[ "foo" ]] && echo "works"'
works
$ docker run -it --rm debian:jessie sh -c '[[ "foo" ]] && echo "works"'
sh: 1: [[: not found

fair point, reverted the shebang change.

Awesome, thanks for the contribution!


Error [object Object] on node_modules/cypress-visual-regression/dist/command.js:33

Hi, I'm having an error, but the logs are not helpful.
Snapshots files are in the base folder, diffs are not generated.
Cypress version 5.0.0.
Node version 10.x

Error: [object Object]
    at Context.eval (http://localhost:36223/__cypress/tests?p=cypress/support/index.js:507:17)
From previous event:
    at Context.thenFn (http://localhost:36223/__cypress/runner/cypress_runner.js:154868:26)
    at Context.then (http://localhost:36223/__cypress/runner/cypress_runner.js:155308:21)
    at Context.<anonymous> (http://localhost:36223/__cypress/runner/cypress_runner.js:168808:21)
    at http://localhost:36223/__cypress/runner/cypress_runner.js:168241:15
From previous event:
    at runCommand (http://localhost:36223/__cypress/runner/cypress_runner.js:168220:8)
    at next (http://localhost:36223/__cypress/runner/cypress_runner.js:168366:14)
    at http://localhost:36223/__cypress/runner/cypress_runner.js:168394:16
From previous event:
    at next (http://localhost:36223/__cypress/runner/cypress_runner.js:168366:34)
From previous event:
    at http://localhost:36223/__cypress/runner/cypress_runner.js:168407:37
From previous event:
    at run (http://localhost:36223/__cypress/runner/cypress_runner.js:168400:19)
    at $Cy.cy.<computed> [as window] (http://localhost:36223/__cypress/runner/cypress_runner.js:168852:11)
    at Context.runnable.fn (http://localhost:36223/__cypress/runner/cypress_runner.js:169078:21)
    at callFn (http://localhost:36223/__cypress/runner/cypress_runner.js:103957:21)
    at Test.../driver/node_modules/mocha/lib/runnable.js.Runnable.run (http://localhost:36223/__cypress/runner/cypress_runner.js:103944:7)
    at http://localhost:36223/__cypress/runner/cypress_runner.js:174727:28
From previous event:
    at Object.onRunnableRun (http://localhost:36223/__cypress/runner/cypress_runner.js:174715:20)
    at $Cypress.action (http://localhost:36223/__cypress/runner/cypress_runner.js:165051:61)
    at Test.Runnable.run (http://localhost:36223/__cypress/runner/cypress_runner.js:173050:13)
    at Runner.../driver/node_modules/mocha/lib/runner.js.Runner.runTest (http://localhost:36223/__cypress/runner/cypress_runner.js:104616:10)
    at http://localhost:36223/__cypress/runner/cypress_runner.js:104742:12
    at next (http://localhost:36223/__cypress/runner/cypress_runner.js:104525:14)
    at http://localhost:36223/__cypress/runner/cypress_runner.js:104535:7
    at next (http://localhost:36223/__cypress/runner/cypress_runner.js:104437:14)
    at http://localhost:36223/__cypress/runner/cypress_runner.js:104503:5
    at timeslice (http://localhost:36223/__cypress/runner/cypress_runner.js:98429:27)
logError @ cypress_runner.js:199987
(anonymous) @ cypress_runner.js:199628
emit @ cypress_runner.js:51075
(anonymous) @ cypress_runner.js:183967
emit @ cypress_runner.js:51075
emit @ cypress_runner.js:184010
onPrint @ cypress_runner.js:182829
_onPrintClick @ cypress_runner.js:182834
(anonymous) @ cypress_runner.js:184221
executeAction @ cypress_runner.js:48940
n @ cypress_runner.js:48940
ca @ cypress_runner.js:59472
ja @ cypress_runner.js:59473
ka @ cypress_runner.js:59473
wa @ cypress_runner.js:59475
Aa @ cypress_runner.js:59476
ya @ cypress_runner.js:59476
Da @ cypress_runner.js:59479
Ad @ cypress_runner.js:59542
Gi @ cypress_runner.js:59708
Kb @ cypress_runner.js:59497
Dd @ cypress_runner.js:59544
(anonymous) @ cypress_runner.js:59709
../../node_modules/scheduler/cjs/scheduler.production.min.js.exports.unstable_runWithPriority @ cypress_runner.js:63856
Ii @ cypress_runner.js:59709
Cd @ cypress_runner.js:59543

cypress.json:
{
  "projectId": "3paxvy",
  "viewportWidth": 1920,
  "viewportHeight": 1080,
  "defaultCommandTimeout": 6000,
  "experimentalComponentTesting": true,
  "componentFolder": "tests/visual",
  "nodeVersion": "system",
  "env": {
    "failSilently": false,
    "type": "actual"
  },
  "screenshotsFolder": "cypress/snapshots/actual",
  "trashAssetsBeforeRuns": true
}

Found the issue, it happens when the filename contains newlines and/or tabs.

Hi, I stumbled across another error and also found no helpful error in the log. After investigating a bit I found out that reject(new Error(...)) does not work while reject(...) leads to a error message in the log. The reject is found in plugin.js [parseImage].
Not sure why rejecting an Error leads leads to a corrupt error message in the log. maybe here a similar situation is responsible for the wrong "Error [object Object]" log.

I had the same exception, and my filenames are fine

@fabriciols @gabbersepp it also does this if you don't have the base files in snapchost/base

@skjnldsv yes, so I think that by sanitizing the file names it will actually display this in the log since it won't be able to find the file: Error: [object Object].
My guess is that it's blowing up somewhere else, causing the error to be caught somewhere where it's not being handled correct with new Error(...).

Can you fix the broken build for your PR @skjnldsv. I'm curious to see the results of this.
https://travis-ci.org/github/mjhea0/cypress-visual-regression/builds/721099784

Can you fix the broken build for your PR @skjnldsv. I'm curious to see the results of this.

I have no ideas on how to fix this. My pr properly include the required deps, If the tests uses different ones, maybe the travis workflow should be adapted to use the dependencies from cypress-visual-regression ?

After investigating a bit I found out that reject(new Error(...)) does not work while reject(...) leads to a error message in the log. .

@gabbersepp reject needs to take an error object. It won't work if you pass a string to it. My guess is that it's blowing up somewhere else and sending a non-error object to reject which causes it to display Error: [object Object].

Can you fix the broken build for your PR @skjnldsv. I'm curious to see the results of this.

I have no ideas on how to fix this. My pr properly include the required deps, If the tests uses different ones, maybe the travis workflow should be adapted to use the dependencies from cypress-visual-regression ?

Ok. I'll look into it.

changes: https://github.com/mjhea0/cypress-visual-regression/pull/53/commits/24dc7260c58305851c631ba322f0b2e8f53eec15
@skjnldsv
do you want to incorporate into your pr?
also, i'd like to add a few quick test cases. what's an example of a filename on your end that's causing it to break?

also, i'd like to add a few quick test cases. what's an example of a filename on your end that's causing it to break?

Thanks :)
here you go:
https://github.com/nextcloud/nextcloud-vue/blob/27c0c29302e57e3a9a03b2e9547a9995ca39faa3/tests/visual/components/AppSidebar/AppSidebar.visual.js#L71-L76
					const fileName = `AppSidebar.vue
						 - starred_${star === null ? 'null' : star ? 'true' : 'false'}
						 - compact_${comp ? 'true' : 'false'}
						 - header_${head ? 'image' : 'none'}
						 - secondary_${second ? 'button' : 'none'}
					`

@skjnldsv
is this correct?

AppSidebar.vue-starred_null
AppSidebar.vue-compact_false
AppSidebar.vue-header_none
AppSidebar.vue-secondary_none

@skjnldsv none of those names break the test. I'm going to hold off on merging https://github.com/mjhea0/cypress-visual-regression/pull/56 for now.

@skjnldsv none of those names break the test. I'm going to hold off on merging https://github.com/mjhea0/cypress-visual-regression/pull/56 for now.

No, you need breaklines in the string.

					const fileName = `AppSidebar.vue
						 - starred_${star === null ? 'null' : star ? 'true' : 'false'}
						 - compact_${comp ? 'true' : 'false'}
						 - header_${head ? 'image' : 'none'}
						 - secondary_${second ? 'button' : 'none'}
					`

This is only one string. This is using the new es6 multi line template string feature.


how do you get any infomation about migrations?

Discussed in https://github.com/drizzle-team/drizzle-orm/discussions/285

Originally posted by toddgeist March 19, 2023
the migrate from "drizzle-orm/better-sqlite3/migrator" works, but there is no result. There also doesn't seem to be any way to see if there are migrations that need to be performed.
How can I log what the migrator just did?  or get any info about migrations.
Thanks
Todd

You can create your own logger.
Example code
Output:


Modify Makefile to work correctly.

these problems fixed:

target dependencies.
new targets to build/run server application.
install dependencies by godep on the container.
execute tools in GOPATH.
help target.

thank you


Support for vacuums?

Any chance there could be support for the Mi robot vacuums?

Yes - it is possible to be supported. But what is your device's model? Isn't it supported by xiaomi_miio?

rockrobot ......
Its accessible trough Miio....

indeed not in list Xiaomi MioT
greets Mupsje

It seems that this device uses miio to communicate instead of MIoT. So it is just ok to use Xiaomi Miio. Although it is also possible to be supported by Xiaomi MIoT, it provides less features than Miio because it is not the device's native protocol.

This is great, thank you.  This app is wonderful tool for anyone trying to get the api token.  https://github.com/Maxmudjon/Get_MiHome_devices_token


Reimplement sharedDistinctUntilChanged() as RxJava operator

PR refactors sharedDistinctUntilChanged() (used to be named distinctUntilChanged()) so that it now actually is a proper RxJava operator, rather than a combination of switchMap/just/never.
This results in a better memory and cpu consumption and pretty much minimal required amount of code to do the job implementing RxJava protocols.
The only problem is that I'd like to implement Operator Fusion, but looks like interfaces required for that are internal in RxJava:

io.reactivex.internal.fuseable.QueueDisposable
io.reactivex.internal.fuseable.QueueFuseable
io.reactivex.internal.fuseable.SimpleQueue

@akarnokd @vanniktech would really appreciate your review

Does it appear around queuing operators a lot? If not, don't bother with operator fusion.

Does it appear around queuing operators a lot? If not, don't bother with operator fusion.

Fair, given that it's an Observable operator. I've updated the comment.
Does it otherwise look good, @akarnokd?

Also should we raise a ticket to expose fusion to public API?
It seems that your RxJava2Extensions rely quite a lot on internal RxJava apis which is not great

gonna merge then, can address future feedback in separate PRs

Thanks for review, everyone, really appreciate it


Support tenants for taggings

@seuros Let me know if I should be tagging (pun intended) other contributors to get attention to this PR.
I'm opening up this PR to add tenant capabilities to acts-as-taggable-on. Specifically this would be helpful for multi-tenant applications that will want to use a tenant column to track ownership.
I know there's an owner concept already in this gem, but that doesn't handle the case where you want to track the user who tagged the object as well as the tenant_id for partitioning purposes.
We use this gem (via a fork right now) across millions of rows and this helps keep performance sane. However, we'd love to get this change merged back into the root repo for future compatibility. It's also being introduced here in a way that will not impact existing usage syntax.
Would love to get your thoughts on the acceptability on this idea.

Hi @lunaru
I this PR should be a documented opt in feature and not hardcoded into the code.
the developper will have to require a file before having the tenant feature.
require "acts_as_taggable_on/tenant_column"
I dont want to cause issue with existing codebases since it contain a feature migration and will change the sql queries.
this feature will have a rake task to generate the migration.

@seuros thanks for the review --
The code is actually already built so that anyone not explicitly asking for tenants (via call to acts_as_taggable_tenant {tenant_column}) will not have any queries altered. This is accomplished by checking against the existence of the tenant_column value before create. The only read-query that's new is for_tenant (and related scopes) and that will not be called by existing codebases anyway. So it's already an opt-in feature and it's fully transparent to anyone not using tenants.
As for the migration, because this gem abides by the Engines rules, the migrations will be generated. It would difficult to put it into a separate rake since the modus operandi of this gem is to utilize the Engine standard. However, I've now modified the README file to indicate that it's fully optional, so existing projects can completely ignore the generated migration. New projects can choose to keep it or discard it as they see fit.
TL;DR: This PR will not impact existing production code. The new migration can be discarded with no problems.
Let me know what you think of this approach

That sound good .
Let have some approval from the community for the PR and i will merge it.

👍 often times the owner of a tag has a parent owner, for example a Post can have tags, and the Post can belong to an Account, and being able to specify a tenant (Account) for querying will help performance. Will also help querying for all tags belonging to a particular Account.

👍 to this one! Was just about to monkey patch this library to add this feature.

@pablo-co Glad you found this! Feel free to use the fork at https://github.com/reamaze/acts-as-taggable-on (branch name is tenants-2) if you need this immediately while this awaits merging

@pablo-co @seuros Any further thoughts on this? Would love to see it get incorporated.

@pablo-co @seuros @mbleigh This is a fantastic PR. Any plan to finish review and merge soon or any other thoughts?

Hey this is great, any plan to merge?

This is a fantastic feature, any plan to merge it soon? @lunaru is your fork safe for production usage (seems out of sync from mbleigh:master)? Thanks!

@apuntovanini Yes it’s safe for production usage.
Why it hasn’t been merged after all this time is a mystery. It’s clearly a needed feature for any multi-tenant system and it had no negative implications for existing production deployments.
@pablo-co ping one more time.

@lunaru I will test it this weekend and have it merged. Sorry for delay.

Okay

released

Are there any plans in making tenant as a polymorphic association? would help more with joining records than a simple string column


fix: replace fs.existsSync() with Bun.file().exists()

[x] Run tests

@the-pesar Thanks. Merge now.


Collection of ideas for new functions

[ ] More string utilities (e.g. regex functions)
[x] Formatting functions (echo 3.1415 | map format "{:.2f}")

String

[ ] search and match based on regex
[ ] Add regex support for replace input
[ ] Add function endwith a pattern

Arithmetic

[ ] Add mod, pow,  div and log  etc. functions

Filesystem

[ ] Add function mv to move and remove files
[ ] More functions to filter files based on file mode bits and ownship

@leeleeee Sounds great. Especially the mv function, which will be really useful (we should add cp as well)

@sharkdp What do you think if we can add du or something like this?

@sharkdp What do you think if we can add du or something like this?

In which sense? Like a filesize :: Path -> Int function?

By the way, I just realized that the coreutils du commands works very nicely with functools:
> du | filter -c1 greater_than 8000 | map -c2 abspath
8340	/tmp/demo/folder/subdirectory
8408	/tmp/demo/folder
8480	/tmp/demo

Yeah, more precisely it is filesize :: Path -> String.
$ find . | map basename | filter startwith test_ | map filesize mb
test_file1 1MB
test_file2 5.2MB

It is fantastic that linux commands are work very well with these functions. 👍

filesize is now supported.

[ ] fst and snd for array-index access (alias for index 1 and index 2)
[ ] replicate
[ ] reverse

Does split currently work with variable size of whitespace between words? I was unable to, if not, does it make sens to add it? Something like below by default
➜  ~ echo "a b         ccc " | ~/repos/shell-functools/ft/map split | map at 2
ccc

Does split currently work with variable size of whitespace between words?

It does not (yet). We could either provide a split_whitespace function which would simply call .split() without argument and/or provide a split_regex function which could be called with map split_regex '\s+'.

The reverse function, do you mean on array level, or on string level?

I think array-reverse can be useful. Not sure if we need a string-reverse function.

Hey @sharkdp, what about creating a max as well as min function in shell-functools? I can work on something if there is interest.
max :: Array -> Int
find . -name cnt.txt | xargs cat | max

@guilhermeleobas Sounds great!

These days, new unix/linux CLI tools are coming out supporting JSON outputs. Would it make sense to create a json function that can parse JSON outputs?

These days, new unix/linux CLI tools are coming out supporting JSON outputs. Would it make sense to create a json function that can parse JSON outputs?

I've been thinking about JSON support in the past, so this sounds interesting. What would this function do, exactly?

I can imagine there would a command named json which could be used like this:
$ curl http://echo.jsontest.com/key1/value1 | json key1

The curl command outputs {"key1": "value1"}. The whole command would output value1. The argument to json command would specify the path to the value you want. It could support the full json path spec or just a small part of it.
And then there would also a function called json which could be used like this:
$ curl <url that returns json in each line> | map json <json path>

In this case, the assumption is that the curl outputs a json-encoded string in each line, which is also a proposed standard format called jsonlines. And the map will transform each line with the json function.

Do you know jq? I think this could probably play well in combination with shell-functools.

@sharkdp jq seems to cover my use cases. Cool, thanks.

More functions that should be implemented:

[ ] div/divide
[ ] sum/product (working on arrays)
[ ] matches => similar to starts_width/ends_with

Offering
For README.md:
### Usage of `lambda`

The `lambda` *function* allows you to create custom functions using a
definition string to Python's *lambda* function.
``` bash
> seq 3 | map lambda 'x : x+x'
2
4
6

> echo -e 'a:b\n2:4' | map split ':' | map lambda 'x, y : x+y'
ab
6

> seq -3 3 | map lambda 'x : abs(x)'
3
2
1
0
1
2
3

> echo -e "uppercase\nLOWERCASE\ncAMELcASE" | map lambda "x : x.swapcase()"
UPPERCASE
lowercase
CamelCase

For ft/ft/functions.py:
@register("lambda")
@typed(None, None)
def ft_lambda(definition, inp):
    _lambda = 'lambda ' + definition.value
    if type(inp) == list:
        args = list(map(lambda v: v.value, inp))
        return [TypedValue(str((eval(_lambda))(*args)), T_STRING)]
    else:
        return (eval(_lambda))(inp)

Offering
For README.md:
### Usage of `shell`

The `shell` *function* allows you to create custom functions using arbitrary
*shell* commands.  If the command string contains pairs curly braces `{}`, the
arguments to the function are formatted into it, otherwise they are appended
to it.
``` bash
> /usr/bin/ls -1 ft/ft/*.py | map shell 'grep -H -c def' | map split ':' | map format '{} has {} functions'
ft/ft/command.py has 14 functions
ft/ft/error.py has 1 functions
ft/ft/functions.py has 60 functions
ft/ft/__init__.py has 0 functions
ft/ft/internal.py has 4 functions
ft/ft/termcolor.py has 2 functions
ft/ft/test_command.py has 5 functions
ft/ft/types.py has 21 functions
ft/ft/version.py has 0 functions

> echo -e 'a\tb\n2\t4' | map shell 'echo "{} and {}"'
a and b
2 and 4

Total lines of text
find | filter is_file | map shell 'cat {} | wc -l' | foldl1 add
5040

For ft/ft/functions.py:
@register("shell")
@typed(None, None)
def shell(command, inp):
    if type(inp) == list:
        args = list(map(lambda v: str(v.value), inp))
    else:
        args = [str(inp)]

    command = command.value
    if '{}' in command:
        command = command.format(*args)
    else:
        command += ' '+' '.join(args)

    result = os.popen(command).read().rstrip('\r\n')

    if type(inp) == list:
        return [TypedValue(result, T_STRING)]
    else:
        return result

Offering
For README.md:
### Usage of `regex`

The `regex` *function* permits filtering based on Python supported regular
expressions.  For case-insensitive filtering, use the `(?i:...)` syntax:
``` bash
> /usr/bin/ls | filter regex '(?i:[a-c]+)'
LICENSE
make
path
README.md

For ft/ft/functions.py:
@register("regex")
@typed(T_STRING, T_BOOL)
def contains(pattern, inp):
    pattern = dynamic_cast(T_STRING, pattern).value
    return re.search(pattern, inp)

hi! can u expand on that? like what would these do or what would be their input be like.

[ ]  div/divide
[ ]  sum/product (working on arrays)
- @ sharkdp at https://github.com/sharkdp/shell-functools/issues/7#issuecomment-493706062


Prevent test from changing golden schema

When we add the extra test entity to the current JpaTransactionManagerRule by calling jpaRule.withEntityClass(TestEntity.class) and jpaRule.withProperty(Environment.HBM2DDL_AUTO, "update"), the rule would override the golden database scheme with the schema derived from the current entity class(This is an expected behavior by enabling HBM2DDL_AUTO). This behavior prevents us from detecting breaking changes in ORM classes.
This PR fixed the issue by adding HibernateSchemaExporter to export the DDL script for given extra entity class, and make JpaTransactionManagerRule execute the DDL script to create the corresponding table for the extra entity when initializing the database. This PR also simplified the code when adding extra entity class for testing. For now, you don't need to(and shouldn't) call jpaRule.withProperty(Environment.HBM2DDL_AUTO, "update").

This change is

the table name was not compatible with our name convention because of the override from HBM2DDL_AUTO.

Why not just change the table name?  It's explicitly specified in the Entity annotation for TestEntity.
Regarding removal of withProperty(), we might just have always done that in cases where withEntityClass() is called.

if we want to test some change to the golden schema and need an extra entity,

We don't really have an example of this case and it's difficult for me to envision one.  I expect you'd typically want to test only real classes against the golden schema.
On balance, I'm not opposed to this change.  I don't think it was necessary, but it's also not detrimental.

I think you are right, we probably won't gain as much as we thought before. Though, this is also a trivial change so I didn't spend too much time on it.


TODO: add warning on the setup leader connect guardians screen for only doing the correct number

Todo: this should be enforced on the backend instead, the additional connection should just be ignored

Fixed and Closed


Add healthchecks for dbsync to check current tip

This is the new dbsync healtlh check:
#!/bin/bash
export PGPASSWORD=${POSTGRES_PASSWORD}
[[ $(( $(date +%s) - $(date --date="$(psql -qt -d ${POSTGRES_DB} -U ${POSTGRES_USER} -h ${POSTGRES_HOST} -c 'select time from block order by id desc limit 1;')" +%s) )) -lt 3600 ]] || return 1

The psql health check is added, ideally (low-priority future task) we should try and use curl http://<IP_ADDRESS>:8080 instead - which will also capture if dbsync is starting (while postgres is on tip) or doing ledger revalidation, while not really adding new blocks yet.
The problem encountered when doing so with official image as-is was that we didnt have basic utils (grep/sed/awk) to work with output of above URL for checking prometheus metrics


feat: skip tls when making connection

Add option to skip tls when making nats connection

Will close this for now as it's old and out dated after no response


Recipe-App Tasks

Hello 🤓 We are a team of 2 🚀

Victor Peralta
Mateo Villagómez

Our link project 📝

Hi Team,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Required Changes ♻

[ ] Kindly assign all tasks to team members. There is one task that is not assigned to anyone.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Many git processes can be spawned at startup

From https://github.com/Microsoft/vscode/issues/23773

Open a large git repo in vscode (here, chromium)
Quickly after opening vscode, make an edit, save the file

Repeat making an edit and saving several times

Git processes are spawned without limit:

.gitignore is the file I was editing. I think another one is spawned each time I save the file. My computer is slow and my tunes start skipping (my main motivation for investigating).
They run for a very long time. I close that vscode window, and they are still running a minute or more later.
Only reproes if I do this right after opening the vscode window. I can't repro it later.

I have taken some steps to mitigate this problem with the release of 1.7.0. I got better performance results in my environment.
Is this still an issue for you with the latest version?


Use only milliseconds for begin, end, and derived columns

Timestamps are a PITA across datasets and platforms and require careful conversions and tests. In ElPaCo we've made an early choice to convert all timestamps in incoming data to milliseconds counted from the beginning of the source recording. Reasons:

milliseconds offer the right level of precision (seconds don't: people are sensitive to timing differences on the order of 0.1s)
this makes them also useful as a key part of the UID, the utterance-level unique identifier we use throughout
milliseconds are simple integers that are easy to manipulate for all sorts of time-related measures (duration, length of overlap, etc.)
milliseconds are the 'industry standard' for work on the timing of turn-taking (FTO)

My proposal is to make sure there is a simple and robust function to handle the timestamp to millisecond conversion at the earliest moment of import, and perhaps some tests to check that timestamp data at later stages are milliseconds. And that all subsequent levels of data processing and data dumping in both scikit-talk and talkr deal with timing data only in milliseconds. This will continue to save us a lot of headaches.

Agree. That function is in 'dataloader.py'.


try out template repo.

-Here this template repository https://github.com/innovolve-ai/template-electron-vuex-vuetify is simple vuetify template but we need this repo https://github.com/governance-foundation/knowledge-fund/tree/master


remove ! $template->getLoader() check in Twig::__construct

as getLoader() on TwigEnvironment will call \LogicException when loader not defined

This patch is contrary to what we're trying to do; we want to inject a loader if none is present. The more appropriate solution is to wrap the getLoader() statement in a try/catch block:
try {
    $template->getLoader();
} catch (LogicException $e) {
    $template->setLoader($this->getDefaultLoader());
}

I'll make that change on merge.


Resolving a list of Unions

Hi @helfer @stubailo ,
I just found out how to implement a Resolver for a single Union (#172), but how about resolving a list of Unions?
In relation to the example provided in #172: how would I resolve a list of Characters? ([Character]!)
For me this is a practical issue. I am using this in a real-life case with an AvailableSearchFilters Union List, which can contain for example a GeoPointDistanceFilter and a PriceRangeFilter.
Of course I'm willing to update the docs if you can point me to the solution.
Kind regards,
Sander

Just return a list where each item implements one of the union types. Or return a list of promises that resolve to one of those types. I believe the resolveType method will still apply to each individual item, but if that doesn't work, you can try defining ofType on each of the implementing types.

Hi @helfer,
Thanks for pointing that out.
I got it to work with a list of interfaces using __resolveType, but not with the list of unions.
I cannot find any documentation on ofType. Can you give me an example, or point me to the documentation?
Kind regards,
Sander

Addendum
So when resolving a list of NodeInterfaces, this works:
  NodeInterface: {
    __resolveType(root:Object, context:Object, info:Object) {
      if(root._graphQLType === 'Offer'){
        return 'Offer';
      }

      return null;
    },
  },

But when resolving a list of SearchFilterUnions, this specific resolver isn't called. The array is available on the parent resolver though.
SearchFilterUnion: {
    __resolveType(root:Object, context:Object, info:Object) {
      if( root.offerCategoryFilter) {
        return 'OfferCategorySearchFilter'
      }
      if( root.geoBoundingBoxFilter) {
        return 'GeoBoundingBoxSearchFilter'
      }
      if( root.geoDistanceFilter) {
        return 'GeoDistanceSearchFilter'
      }
      if( root.priceRangeFilter) {
        return 'PriceRangeSearchFilter'
      }
      return null
    }
  },

Example array logged from parent:
[ offerCategoryFilter: { id: '6ca46120-8a42-11e6-8692-553a5a80fc35' },
  geoBoundingBoxFilter: { topLeft: { lat: 86.5, lon: 86.5 },
    bottomRight: { lat: 5.5, lon: 5.5 } },
  geoDistanceFilter: { location: { lat: 5.5, lon: 5.5 },
    distance: 15000,
    units: 'km' },
  priceRangeFilter: { min: 6, max: 44 } ]

Phew... I've found the issue.
Do not name array elements like this, as named array elements seem not to be passed to the resolver:
[ offerCategoryFilter: { id: '6ca46120-8a42-11e6-8692-553a5a80fc35' },
  geoBoundingBoxFilter: { topLeft: { lat: 86.5, lon: 86.5 },
    bottomRight: { lat: 5.5, lon: 5.5 } },
  geoDistanceFilter: { location: { lat: 5.5, lon: 5.5 },
    distance: 15000,
    units: 'km' },
  priceRangeFilter: { min: 6, max: 44 } ]

Instead, use a default array like this, you can pass the ofType to resolve the Union type:
[ { id: '6ca46120-8a42-11e6-8692-553a5a80fc35',
    ofType: 'offerCategoryFilter' },
  { topLeft: { lat: 86.5, lon: 86.5 },
    bottomRight: { lat: 5.5, lon: 5.5 },
    ofType: 'geoBoundingBoxFilter' },
  { location: { lat: 5.5, lon: 5.5 },
    distance: 15000,
    units: 'km',
    ofType: 'geoDistanceFilter' },
  { min: 6, max: 44, ofType: 'priceRangeFilter' } ]

Now you can resolve the union type like this:
SearchFilterUnion: {
    __resolveType(root:Object, context:Object, info:Object) {
      if( root.ofType === 'offerCategoryFilter') {
        return 'OfferCategorySearchFilter'
      }
      if( root.ofType === 'geoBoundingBoxFilter') {
        return 'GeoBoundingBoxSearchFilter'
      }
      if( root.ofType === 'geoDistanceFilter') {
        return 'GeoDistanceSearchFilter'
      }
      if( root.ofType === 'priceRangeFilter') {
        return 'PriceRangeSearchFilter'
      }
      return null
    }
  },

I was surprised the resolver skips named array elements. Is this intended behaviour? Took me a while to figure this out :)

@sandervanhooft that sounds like something that would need to be fixed in GraphQL.js - this package doesn't deal with execution at all. However, if you have some ideas for where documentation could be improved, please file an issue here: https://github.com/apollostack/tools-docs


Prevent saving too big bem

Dear Developers
I recently tried to create a BEM using the mne-python code, but I ran into the issue that it is too big to save, and there is no automatic splitting for it.
I guess I could use a source space with fewer sources in it this particular time,
Below is the output from mne.write_bem_solution()
Best
Lau

3 BEM surfaces found
Reading a surface...
[done]
Reading a surface...
[done]
Reading a surface...
[done]
3 BEM surfaces read
Approximation method : Linear collocation
Three-layer model surfaces loaded.
Computing the linear collocation solution...

Inverting the coefficient matrix...
IP approach required...
Matrix coefficients (homog)...
inner_skull (10242) -> inner_skull (10242) ...
Inverting the coefficient matrix (homog)...
Modify the original solution to incorporate IP approach...
Combining...
Scaling...
Solution ready.
BEM geometry computations complete.
Traceback (most recent call last):
File "", line 4, in
mne.write_bem_solution(save_path + bem_solution_name, bem_solution)
File "/home/lau/src/mne/mne/bem.py", line 1416, in write_bem_solution
bem['solution'])
File "/home/lau/src/mne/mne/io/write.py", line 135, in write_float_matrix
check_fiff_length(fid)
File "/home/lau/src/mne/mne/io/write.py", line 270, in check_fiff_length
raise IOError('FIFF file exceeded 2GB limit, please split file or '
IOError: FIFF file exceeded 2GB limit, please split file or save to a different format
_

don't use 10242 vertices per layer. Using half will be enough and will fix
the problem.

You really don't need a BEM with so many points, you can do this for the
source space instead if you need it. The conductivity model can be
estimated based on fewer points then the dipole grid used for inverse
solutions.
On Mon, Apr 11, 2016 at 9:11 AM Alexandre Gramfort<EMAIL_ADDRESS>wrote:

don't use 10242 vertices per layer. Using half will be enough and will fix
the problem.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/3112#issuecomment-208218911

Ok, thanks for the input - I'll make the source space smaller.
Lau

Reopened and re-titled. This is the second time this has come up, so we should probably add a warning in the bem code that 5120 should be enough, and also in the writing function if too many data points exist to save.

+100


which node publish the topic "brake_cmd"

which node publish the topic "brake_cmd"

runtime manager.

runtime manager.


Shield of Arrav Marked as Incomplete

Issue: Shield of Arrav does not seem to be handled by the plugin - or on the backend - as it does not reflect on the wiki itself.
Heroes' quest

Dragon Slayer II

In-Game

Yeah, I think I have an idea of what's going on here. It's definitely server side (I get an "IN_PROGRESS" for SOA from the server), but I'll keep this up to date with progress since there isn't a better place to put issues. FWIW, this is known to happen with miniquests as well, basically anything with special conditions on the varbs. Pretty sure they are all the same issue. Thanks for the report!

Shield of Arrav fixed server-side about 10 minutes ago!

Still a bug - Sept 2nd 2022


Grafana 8.3.1

Upgrade internal Grafana to 8.3.1
Testing plan: Deployed.

lgtm; we can merge once CI passes.


Changed "server" to "instance"

This is avoid confusion of some users that can guess Windows need be rebooted.

@rrg92 That is an excellent clarification. :) Thank you very much.


Parsing of OWS parameters placed in to a function

flake8 fixes

Pull Request Test Coverage Report for Build 45

43 of 45 (95.56%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.03%) to 75.309%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

GeoUsage/log.py
43
45
95.56%

Totals

Change from base Build 42:
1.03%

Covered Lines:
244

Relevant Lines:
324

💛  - Coveralls


define "OAuth" in quick start code

I went to run the quick start code in the Node REPL and got an undefined error. It makes for a nicer quick start experience when you can just copy/paste the code and it works

thanks


[BUG] Change put method to use mongodb replace

Describe the bug
put method will replace the whole record in db.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
The whole record will be replaced by the given data.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

done.


vagrant up fails with a VMware vagrant provider

The most recently published vagrant box for VMware does not work with current levels of vagrant and the vagrant fusion provider.  Attempts to bring up bosh-lite fail with the following:
$ vagrant up
Bringing machine 'default' up with 'vmware_fusion' provider...
==> default: Box 'cloudfoundry/bosh-lite' could not be found. Attempting to find and install...
    default: Box Provider: vmware_desktop, vmware_fusion, vmware_workstation
    default: Box Version: 388
==> default: Loading metadata for box 'cloudfoundry/bosh-lite'
    default: URL: https://atlas.hashicorp.com/cloudfoundry/bosh-lite
==> default: Adding box 'cloudfoundry/bosh-lite' (v388) for provider: vmware_desktop
    default: Downloading: https://atlas.hashicorp.com/cloudfoundry/boxes/bosh-lite/versions/388/providers/vmware_desktop.box
==> default: Successfully added box 'cloudfoundry/bosh-lite' (v388) for 'vmware_desktop'!
/Users/sykesm/.vagrant.d/gems/gems/vagrant-vmware-fusion-3.1.2/lib/vagrant-vmware-fusion/action_farm.rb:430:in `call': undefined method `metadata' for nil:NilClass (NoMethodError)
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/builtin/handle_box.rb:56:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/builder.rb:116:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/runner.rb:66:in `block in run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/util/busy.rb:19:in `busy'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/runner.rb:66:in `run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/builtin/call.rb:53:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /Users/sykesm/.vagrant.d/gems/gems/vagrant-vmware-fusion-3.1.2/lib/vagrant-vmware-fusion/action_farm.rb:1236:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /Users/sykesm/.vagrant.d/gems/gems/vagrant-vmware-fusion-3.1.2/lib/vagrant-vmware-fusion/action_farm.rb:99:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /Users/sykesm/.vagrant.d/gems/gems/vagrant-vmware-fusion-3.1.2/lib/vagrant-vmware-fusion/action_farm.rb:273:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/builtin/config_validate.rb:25:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /Users/sykesm/.vagrant.d/gems/gems/vagrant-vmware-fusion-3.1.2/lib/vagrant-vmware-fusion/action_farm.rb:160:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/builder.rb:116:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/runner.rb:66:in `block in run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/util/busy.rb:19:in `busy'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/action/runner.rb:66:in `run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/machine.rb:212:in `action_raw'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/machine.rb:189:in `block in action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/environment.rb:516:in `lock'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/machine.rb:176:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/machine.rb:176:in `action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.1/lib/vagrant/batch_action.rb:82:in `block (2 levels) in run'

Versions:
$ vmware-vmx -v

VMware Fusion Information:
VMware Fusion 7.1.0 build-2314774 Release

$ vagrant plugin list | grep vmware
vagrant-vmware-fusion (3.1.2)

$ vagrant --version
Vagrant 1.7.1

I can confirm this bug. Box 387 did not work either.

Using version 6.0.5 of vagrant
Tried both 2.5.2 and 3.2.0 (latest) of the provider plugin

There is some peculiarities with how box_version is set on Vagrantfile that cause this.
I just pushed a fix at c82c939833fd05ad8e7ad93595ece6d2a054f806
@sykesm, @squeedee Please pull and try again

I used the patch before it was pushed, Dmitri shared it with me. It worked thankyou. I'll also retry using the repo tommorow.

Closing. Reopen if problem still exists.


Error of the docker endpoint not working

The error results in the following description :
This error may indicate that the docker daemon is not running.: Post "http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.24/build?buildargs=%7B%7D&cachefrom=%5B%5D&cgroupparent=&cpuperiod=0&cpuquota=0&cpusetcpus=&cpusetmems=&cpushares=0&dockerfile=Dockerfile&labels=%7B%7D&memory=0&memswap=0&networkmode=default&rm=1&shmsize=0&t=trueagi%2Fhyperon&target=&ulimits=null&version=1": open //./pipe/docker_engine: The system cannot find the file specified.

Hi @Nahom32 , could you please write steps to reproduce the issue?

@vsbogd sorry I think it was a problem with my docker not being updated!

@vsbogd I have resolved it


Set dynamic pathing

Remove relative paths ('temp/blah.text')
Use __dirname to get script path
Set tmp path using found script path
Create all files and folders as needed

Hi thanks, you've done a lot of work.
I am relatively new to git & extremely new to github. I have gone through your suggestions and they look appropriate on first look.  I will take a day or two to work out how to implement these because I have gone and mucked up my git main by merging something into from a testing branch by accident. I can find no way to undo it. In the past when I have done this I think I just rebase eveything or use a local backup of the entire project. However, with your interest in the package it will muck things up for you.
I will get back to you, thanks so much for your help.
On a slightly different note. I find the preview of the asciidoc files in pulsar relatively useless. This seems to be because all sorts of things are constrained by what I take to be security limitations in the editor package.  Scrolling does not work properly and for zoom I use the chrome style extension. Also most of the links do not work in the preview including moving around within the html file..
To be honest I find it far more acceptable to simply open the html5 file created by asciidoc in falkon by pressing Ctrl+Alt+F. Then in the browser window virtually everything works out of the box just as it would in a real including zoom. scroll. Automatic update when the source file is saved in pulsar works because falkon has creates its own local server.
I have wondered about removing the ascidoc preview in pulsar and just using the falkon browser as described above and concentrating on making that  synchronise wi the pulsar editor.

It is importance that the base directory be relative and ./ is what I found worked best. __dirname leads to an absolute directory name and will not work in the ...view file.

So this one is perplexing to me. I do not 100% understand this response (or rather possibly what you see that differs than on my system). So here is what I found when using relative paths for temp:
The relative path (./temp for example) ends up working out to the path of your adoc file's directory. So, if I preview an adoc file in ${HOME}/Documents it wants  there to be a temp folder in that directory (${HOME}/Documents/temp) which now leaves a bunch of temporary render files in that temp folder. Now, if I preview another adoc file in ANOTHER location (say ${HOME}/git/myproject/README.adoc) it now needs a temp folder there as well (again leaving all temp render files in that new temp folder).
So, the issue I see with this method is that we create new temp folders all over the filesystem versus in one location (like the plugin tree .pulsar/packages/ianasciidocpreview-active-editor-info). To make matters worse, many times developers are writing adoc files and need a preview they are in a git repository and these temp files could (and will) then get checked into git accidentally.

Just wanted to add:
You are doing great and I applaud you for learning all of this. Whatever direction you take will be awesome because you are learning new stuff every day. I agree on the complexities of all this and I wish I had the time to devote more to this. I simply just don't have that time.
I am happy to contribute and test here and there, but cannot fork and devote time to this (that could change here and there and allow more time though).  Do not take my suggestions as criticisms as they are not.   I am just trying to add what I can here :-)

It has dawned on me that we have a different way of working and we are ending talking at cross purposes while we agree, I think that is the case anyway.

The relative path (./temp for example) ends up working out to the path of your adoc file's directory. So, if I preview an adoc file in ${HOME}/Documents it wants there to be a temp folder in that directory (${HOME}/Documents/temp) which now leaves a bunch of temporary render files in that temp folder. Now, if I preview another adoc file in ANOTHER location (say ${HOME}/git/myproject/README.adoc) it now needs a temp folder there as well (again leaving all temp render files in that new temp folder)

The above is exactly the way that I want things. I use asciidoc only for very large projects with say 600 web pages and many links to subdirectory levels with the final child pages with 100s of links to the likes of GoogleNew[papers]. Each subdirectory level has its own index.html with links into the next subdirectory which has its own index.html page and so on.  What happens is that the temp directory is placed at the top project level directory regardless of which subdirectory files pulsar is editing.  That is what I experience anyway. There are no temp directories in any of the subdirectories no matter which adoc file I edit in pulsar. I aways run /home/ian/Downloads/pulsarlatest . --no-sandbox from a terminal in the top project directory. If I accidentally enter pulsar from say the lib/ directory then I will end up with a lib/temp dir there. So as long as I always enter pulsar from the top ProjectDirectory I'll get the structure listed below. I have added a few comments. (The extra extentions .html.erb in addition to adoc are because I use Middleman CMS for the static websites and it uses Ruby hence ".erb".)
ProjectDirectory
├── source
│   ├── Press
│   │   ├── GlasgowHerald
│   │   │   ├── GlasgowHerald1971.html.erb.adoc
│   │   │   ├── GlasgowHerald1972.html.erb.adoc
│   │   │   ├── index.html.erb.adoc  # index to GlasgowHerald
│   │   ├── Guardian
│   │   │   ├── index.html.erb.adoc
│   ├── DiverseArtists
│   │   ├── Exhibitions
│   │   │   ├── ExhibitionA.html.erb.adoc
│   │   │   ├── ExhibitionB.html.erb.adoc
│   │   ├── Shows
│   │   ├── index.html.erb.adoc
│   ├── index.html.erb.adoc     # index to Press
├── index.html.erb.adoc         # main index to all levels
├── temp                        # the only temp directory for all directies.
│   ├── htmloutputFileRawFromAsciidoc.html
│   ├── iansasciidocpreviewhtmloutput.html
│   ├── iansasciidocpreviewscrolltohere.txt
│   ├── iansasciidocpreviewtransformscale.txt
│   ├── XXXasciidocInputFromEditor_words
│   └── XXX.txt

Now, as you point out,  if we have lots independent adoc pages in their own directories, each will have its own temp directory, and yes there will be lots of these all over the place.
I think that we can easily accommodate both ways of working by a couple of possibilites which come to mind:

Have the option to delete the temp file when we finish editing the adoc file, or
Choose where the  htmloutputDir  will be either  "./", or some other directory like "/home/myhome/AllMyAdocPreviewTempFiles" .  This would need a bit more thought.

THis is all for now. But thanks so much for your interest and taking time to think about it all and comment.  No one else has done that and what I need is an outside referee - thee!

I've created a branch Test_Changing_htmloutputDir  which puts the temp directory in /home//ZZZZianasciidocpreview-active-editor-info/temp/ . This is based on a minimal change to my original main push.  All the same files as before should appear in temp/. Many thanks to @dadreggors  for the stimulation as to why this might be a useful change.
Please try this and let me know how you get on.
The htmloutputDir can be changed  to "./" to have the temp/ directory appear in the project directory. See the comments in the lib/...view.js file for how to do this.
(There are several new .html files starting htmloutputFileRawFromAsciidoc which show different modes of asciidoctor.js  .adoc conversions. In the future these will be used to generate the linked style css files which will be incorporated into the outputhtml file.  Eventually, but not yet, this output will be controlled from within the .adoc file  as css attributes.)
Note: this is the first time I have pushed a branch, so it might not have gone according to plan. Please let me know.


Implementation of blossom algorithm for maximal cardinality matching

Code by D. Eppstein, UC Irvine, 6 Sep 2003.
Implementation of maximum cardinality matching code taken from https://code.activestate.com/recipes/221251-maximum-cardinality-matching-in-general-graphs/
I recently used this code on a Google Foobar challenge and it would be a nice addition to this repository.

Nice work. :)
I am thinking of checking in a fast UnionFind (#830) and even though I have written a good number of tests for it, I would like to test its integration with the blossom algorithm implementation as well.
If you find some time to add your tests, I can refactor the API myself and test out the integration.
Thanks!

Thanks @aalekhpatel07, I've added unit tests for blossom algorithm implementation. You can test your union find integration now. All the best and let me know if there's any way I can help!

Nice. (927e73) contains the new blossom that uses the UnionFind API from algorithms.unionfind.
The tests are passing, so yay! :smile:

Looks great!


Colors not consistent rendered according to theme specification

Insider version

Version: 1.53.0-insider
Commit: 32b28f6f8f9dfe36efad102cc460e14a84ba756a
Date: 2021-01-19T06:10:13.886Z
Electron: 11.2.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0

Regular

Version: 1.52.1

Commit: ea3859d4ba2f3e577a159bc91e3074c5d85c0523

Date: 2020-12-16T16:32:10.090Z

Electron: 9.3.5

Chrome: 83.0.4103.122

Node.js: 12.14.1

V8: <IP_ADDRESS>-electron.0

OS: Linux x64 5.4.0-62-generic

nVidia driver: 460.32.03

Ubuntu color space configuration: Standard RGB

The colors rendered by VSC are not consistent with the specification in any given theme.
E.g. tried out the "City Lights theme" in VSC and Atom (same author) and you can see the difference below
Steps to Reproduce:

VSCode

Atom

This holds for any theme I've tried.
Moreover, tried separately the fixes previously reported here, i.e.
	"disable-color-correct-rendering": false,
	"force-color-profile":"srgb",
	"disable-hardware-acceleration": true,

and no luck.

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

This is dup of https://github.com/microsoft/vscode/issues/24569.
VSCode interprets themes colors as sRGB, as used by browser, while TextMate (and probably Atom) interprets them as Apple RGB.
While we came from TextMate themes, VSCode color themes have become more than TextMate themes. We will stick with sRGB.
As a consequence, when you convert a TextMate theme to a VSCode theme and you want to be 100% color compatible, a color conversion is necessary.

This is dup of https://github.com/microsoft/vscode/issues/24569.
VSCode interprets themes colors as sRGB, as used by browser, while TextMate (and probably Atom) interprets them as Apple RGB.
While we came from TextMate themes, VSCode color themes have become more than TextMate themes. We will stick with sRGB.
As a consequence, when you convert a TextMate theme to a VSCode theme and you want to be 100% color compatible, a color conversion is necessary.


uselisteners does not produce listeners listener of choices

Empire Version
2.3
OS Information (Linux flavor, Python version)
linux Kali GNU/Linux Rolling \n \l
python 2.7
Expected behavior and description of the error, including any actions taken immediately prior to the error. The more detail the better.
when typing uselisteners I expect a list of listeners as demonstrated: http://www.harmj0y.net/blog/empire/the-empire-strikes-back/
instead the following is returned:
(Empire: listeners) > listeners
[!] No listeners currently active
(Empire: listeners) > uselistener
[!] Error: invalid listener module
Screenshot of error, embedded text output, or Pastebin link to the error

Any additional information
wiped and resinstalled twice to ensure there wasn't an issue during install

uselistener [TAB] [TAB] to see available listeners
example : uselistener http

@buglessdr, @ThePirateWhoSmellsOfSunflowers is correct. Use tab to list them.


Add option to show high alch values in loot tracker

Add option to show GE or HA values in the loot tracker.
New config:

When both GE and HA is checked:

When only GE is checked:

When only HA is checked:

This should be an enum with ge, alch, or both

Config now looks like this:

You can just click the resolved button. Github shows the commit history.

LootTrackerRecord.java 100644 → 100755 i dont think this is correct?


set-content needs to perform more like > for simple redirection scenarios

Steps to reproduce
get-content  | set-content foo.txt
get-content  > bar.txt
Expected behavior
both succeed with similar performance
Actual behavior
first one fails due to memory usage
Reported from:
https://twitter.com/troyhunt/status/769108300973563904
I think the meta issue is discovery for end users.  This may be a common obvious operation for many.

At least conceptually, Set-Content should always be faster, because it needn't involve the default output formatter.
A quick, informal test on my macOS 10.12.6 machine shows that Set-Content is indeed about twice as fast as > (though note that as of beta.6 > still produces UTF16-LE, while Set-Content produces UTF-8).
With output from an external utility - which is invariably text - Set-Content is arguably the better tool, though > / Out-File work fine functionally too (encoding differences in Windows PowerShell aside, Set-Content / Out-File produce the same output with string input).
That Set-Content would involve array allocation is somewhat baffling (haven't looked at the source yet).
The error aside, it's fair to say that PowerShell has a performance problem here.
Let's compare commands that write a 1-million-element array to a file (sample timings from my machine):
PowerShell:
> $a = 1..1000000; Measure-Command { $a > t.txt } | % TotalSeconds
83.074664  # as stated, with Set-Content this would roughly be halved on my machine

Bash:
$ a=( {1..1000000} ); time printf '%s\n' "${a[@]}" > t.txt
real	0m6.342s
...

That's a factor of 13.


Unlimited timeout in python requests causes library to wait for requests forever.

In contrast with version 1 the actual version 2 is not providing timeout to to calls.
python Requests is used and in their dosumentation is written this:
Timeouts Most requests to external servers should have a timeout attached, in case the server is not responding in a timely manner. By default, requests do not time out unless a timeout value is set explicitly. Without a timeout, your code may hang for minutes or more.
In some cases - like network od DNS problems the library calls hangs forever.
I find this serious issue.

I would appreciate it if the timeout could be configured for each query. For example, I can't stop the client application for 300 seconds when trying to create a payment link.

Thank you. It should be enough for me.


fix: fix compatibility issues with fs

Resolving compatibility issues with file systems across different Android systems.

Thank you!


More flexible data visualization

currently only target/gender distribution plots are available..
but there might be age, gender, emotion, duration, snr etc. information available
change value_counts in [EXPL]
so that
[DATA]
target = emotion
[EXPL]
value_counts = [['gender'], ['age', 'duration']]

would result in two plots: one showing the distribution of gender per emotion, and the other the scatterplot of age and duration colored by emotion.

done with version 0.52.0


fix reporting when commited to master

trigger to telegram

https://github.com/EpicMorg/atlassian-json/commit/64153b05301636cfcdcfce9aefdec96ceedf9224


Circleci project setup

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

What is the intention of this PR? I do not think we want to add another CI system.

cowsay, sorry you're spamming us.


Additional Resource Section to Legacy Newsletter

Added a Additional Resource section to Legacy Newsletter module, this pull request should cover all pages within that module.
Description of the change: Added a Additional Resource section to Legacy Newsletter module
Reason for the change: Relates to Issue #3089
@ksigler7

Hello @Aonepathan,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Pre-bundling should be more resilient to errors

Clear and concise description of the problem
When Vite fails during the pre-bundling of a particular dependency for the browser, it seems to get into a state where it refuses to pre-bundle any further dependencies. Even if the reference to the offending dependency is removed, in dev mode Vite remains in this state until the server is restarted.
This is an issue because some dependencies simply can't be pre-bundled for the browser (for example, if they make use of Node APIs). Mentioning these in code that's meant to be isomorphic for the browser and the server - even if it's inside if (false) { import ('something'); } - causes Vite to attempt to pre-bundle them, resulting in an error, and also resulting in other dependencies not being pre-bundled, resulting in (for example) CJS being served to the browser when it's expecting ESM.
Suggested solution
I do not know Vite's internals well enough to suggest what should be changed.
Alternative
Other options that seem less good to me (because they don't solve the underlying brittleness issue) would be to make Vite smarter about import()s that it sees could never be reached, or to make Vite only actually bundle dependencies for the browser when they are requested. Both of these might be good ideas by themselves, but even if someone does accidentally import something on the browser that cannot be bundled for the browser, this shouldn't prevent other dependencies from being pre-bundled until Vite is restarted.
Additional context
See https://github.com/sveltejs/kit/issues/1570 which contains a reproduction using SvelteKit.
For now, my workaround is to hide the import()s from Vite that I don't want it to try to pre-bundle by wrapping them in another function call, which then calls import().

We experience this issue aswell, when updating from 2.3.0 -> 2.5.7 (through svelte-kit) most dependencies that were previously only exposed to the server is now exposed to the browser. And everything fails.

Could be a vite-plugin-svelte issue, what dependency in particular that is causing that? disableDependencyReinclusion might help too.

I'm running into a similar issue related to an attempt at client-bundling an unnecessary package.
In my case, I have two packages in a mono-repo:

A library that exports two functions; validate and request
a SvelteKit app (which maybe complicates things?) that consumes this library

validate is totally safe to run on the server and client, and it's imported and used in both.
request wraps node-fetch, and is only every imported on code that is server-side only. Despite this, I'm getting errors from Vite that node-fetch can't be bundled for the client (which isn't even something I want it doing anyway). Adding node-fetch to optimizeDeps.exclude doesn't help, nor did using import('node-fetch') rather than a static import; I still am getting the error.
I would assume that, between SvelteKit and Vite, the bundler should be smart enough to know that the node-fetch import path is only relevant for the server-side code and not bother trying to prepare it for client-side execution.
SvelteKit itself could definitely be the problem here, but they don't really have any settings related to building; as far as I can tell, it delegates entirely to Vite, which makes me things there's something wrong or missing on the Vite side of things as well.

It seems like there's two issues with prebundling here:

Named import of nodejs/builtin modules in dependency fails. This is because Vite's optimizers stubs builtin modules with export default only. I'm trying a few ways locally to fix this.
Dynamic imports should be lazily prebundled. Currently it's eager when analyzing the file for imports. I'm also testing this locally and have a POC that seems to work.

I'll be sending PRs soon.

I found that lazily prebundle for dynamic imports to not always be a good default as some apps may re-prebundle more often which would be poor DX in some cases. I think we could keep it as-is for now given no1 is solved. Closing for now.


fix changelog for v2.4.6

Motivation and Context
The changelog for the PR for this version needs to be moved into unreleased side. When I created the PR, in the middle of it, there were probably updated changes to the master branch, which was not captured in my feature branch.
Your Changes
fixed changelog to have my PR in the unreleased section
Type of change (select all that apply):

[x] Other (please specify): fixing changelog

Checklist

[ ] I have performed a self-review of my own code.
[ ] I have verified that the pre-commit.ci checks have passed.
[ ] I have verified that the CI tests have passed.
[ ] I have reviewed the test coverage changes reported on Coveralls.
[ ] I have added tests for my changes.
[x] I have updated the Changelog.md file.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
39137

Relevant Lines:
42024

💛  - Coveralls


Weak form of a PDE

Hi again :)
I'd like to bring up a topic that's been keeping me busy lately in the field of Physics-Informed Neural Networks (PINNS). During research I've come across a number of papers that discuss how accurate and effective PINNS are, particularly when dealing with the strong form of a Partial Differential Equation (PDE). Interestingly, these papers reveal that in most cases PINNS don't perform well and might even give us incorrect results. However, they also highlight that PINNS work really well when it comes to weak forms of PDEs.
Just as an example take a look at page 6 of Paper1
Another example can be seen here Paper2 where they had to add an integration formulation of the pde to the strong form to keep the global consistency.
In light of these observations, I would like to propose the incorporation of an integration method. Such an addition would enable the formulation of the loss function in weak form, facilitating integration across the entire domain (effectively summing over collection points). By doing this, you could really enhance the performance of PINNS, bringing them up to date with the latest techniques, particularly in accurately solving PDEs using Neural Networks.

Hey @LoveFrootLoops thank you for the suggestion. With the maintainers we are thinking to include variational losses (weak formulation), but we are still not sure how is the best way to do it.
In the meanwhile, consider that we made a LossInterface class so the user can create a loss by inheriting from it. By using torch integration schemas (look at this nice library torchquad), and by defining some basis test functions for your problem I think you should already be able to train by using a weak formulation.

Hey @LoveFrootLoops did you manage to work it out?😊

Hi @dario-coscia,
I haven’t progressed with the project since I found it challenging to keep up with the constant updates to the packages. Each time I attempted to update to a newer version, I encountered issues that prevented anything from working properly. Consequently, I’ve put a hold on using it until it stabilizes. I would appreciate an update from you regarding the version you’d recommend and whether it’s currently stable.
Regarding the implementation, I have been considering a potential solution. Given that the training process is batch-oriented, one possible approach could be to modify the on_epoch_end function to calculate the integral. This would align the calculation with the completion of an epoch rather than on a per-batch basis, which could be a more feasible integration point for the functionality I need.

Hi @LoveFrootLoops we plan to release the v0.1 version this November as stable version, I will update you when the release is ready. We made new documentation with more details, updated all tutorials and examples, and introduced the Neural Operator learning framework (MIONet, DeepOnet, Fourier Neural Operator, and more to come..). Also, we plan to make a mailing list for University users, Students, and developers where we will put all the (minor) updates in the package.
I will work out after the release to provide a tutorial on weak PINN. After looking in more details on VPINN and WPINN, the best is to define a new solver inheriting from PINN, where in the __init__ constructor you choose the basis functions you want. Then you will just need to overwrite the loss_phys function in PINN and everything should work.  The loss_phys is a method for computing the physical loss, in our case the weak loss (integration can be done by quadrature rules using torchquad, or by a simple sum).

@dario-coscia, that sounds impressive. Well done!
I do have a query about the loss_phys function. Is it assessed solely at a batch of collocation points? It so, how would you evaluate an integral over the complete domain?

@dario-coscia, that sounds impressive. Well done!
I do have a query about the loss_phys function. Is it assessed solely at a batch of collocation points? It so, how would you evaluate an integral over the complete domain?

It depends the batch_size you use. If you use batch_size=None in the SamplePointLoader you will evaluate the integral over all points used during training. Otherwise, points are batched, and loss_phys is assessed solely at a batch of collocation points (as you correctly said).

@dario-coscia, I believe there might be an issue because you're looking to evaluate the local losses at batches of collocation points, whereas the integral losses need to be evaluated over the entire domain.
Just as a recommendation, I would distinguish between local and global losses. Global losses can be evaluated at each epoch end (on_epoch_end) while local losses should be trained for batches i.e. training steps in the epoch (training_step).

Hi @LoveFrootLoops we merged the new version as well as making a complete new documentation.
I do not understand what you are saying. If batch size is None all points sampled for training are used to evaluate the loss. In that case 1batch iteration = 1epoch iteration.
Anyway, would you like to start a draft PR in which we can collaborate to make up a simple tutorial using VPINN or WPINN? This way I can help you with the problem directly on the code. Let me know

Hi @LoveFrootLoops 👋🏻 and happy new year! Just checking if you managed the implementation of the integral loss function using torchquad and the on_epoch_end  method of the callback. If you need extra help let us know, also in case you want something to be implemented lets post it here since we plan to add integral losses soon.

Hi @LoveFrootLoops 👋🏻 and happy new year! Just checking if you managed the implementation of the integral loss function using torchquad and the on_epoch_end  method of the callback. If you need extra help let us know, also in case you want something to be implemented lets post it here since we plan to add integral losses soon.

Hi @dario-coscia,
Happy New Year! Just wanted to let you know that I got the integral working with torchquad. I'm thinking of setting up a benchmark problem, but things are a bit hectic right now. Should have some time around mid-February to sort that out.
About the PyTorch Lightning thing – yeah, it's all about batches and training steps. We'll need to figure out how to handle all the collocation points in each batch with something like a domain_points variable. Also, there's a bit of a juggle to make sure the loss sizes match up, since we'll have residuals for the batch size and others for all the domain points.

Wow! That's super you managed to do it 🚀🚀
Yeah, it would be great to do a tutorial for the integral loss using torchquad on a simple benchmark problem (I added the label to the PR). Maybe a simple Helmotz equation with PBC using trigonometric coordinates? (this way we just minimize the residual and we do not care about boundaries). Let me know what you think!

Hi! I'm planning to implement VPINNs in #263 by introducing a variational loss class. Could be useful?

Hi! I think @LoveFrootLoops already implemented a sort of Variational PINN using a variational Loss, maybe we can all work together to introduce Variational Losses in PINA?