Generalized injection library

I have started putting together some of the code for the PE project, and naturally have seen opportunities for generalization, specifically with the injection library. This might be a good time to consider how we may implement this in its own repo. Is the injection code large enough to warrant its own repo? If not, what else should go in? related to ML4GW/PE#6

I feel that it's large enough on its own at this point, but there are also things that could be added. At the moment, it feels pretty rigid in how it gets used. Generalizing to broader use cases, and maybe making it a bit more user-friendly, could be good improvements to add.

I totally agree. 'm a bit confused about the logic behind some of the recent changes we've made to the library. For example, I  liked how the old generate_gw expected a fully initialized WaveformGenerator. It gets a bit confusing now with the **waveform_generator_args, as its not exactly obvious what arguments these can be. Additionally, many of the defaults are hidden behind the get_waveform_generator function.
I think the idea behind these changes was to make these arguments in generate_waveforms more general. I think simply passing them to the generate_waveforms main function, and then explicity initializing the waveform_arguments dict in the WaveformGenerator makes the most sense. That way, generate_gw is as most general as possible, since the details of the waveform generator are left to the user.
@deepchatterjeeligo let me know your thoughts.

Overall I agree that a redesign is needed. There are several function which are rather long and have parts that are hard to test. But also I have been looking at this code rather recently. But to answer

It gets a bit confusing now with the **waveform_generator_args, as its not exactly obvious what arguments these can be.

It is all the keyword arguments that go into the bilby waveform generator. In our code, we are wrapping around a bilby function and hence do not need to add them once again to the function signature. If in a new release of bilby, the keyword argument changes to something else, we should don't need a change in our end. We can assume that the user will take note that the bilby version has changed. So I think the docstring should contain a cross ref to the corresponding bilby docs to know what the arguments can be.
However, it is true that the code tries to unwrap a nested dictionary waveform_arguments. While I made that change, I don't feel very strongly about it since it break the logic I mentioned above. But I thought that was a preference in #70

I think the idea behind these changes was to make these arguments in generate_waveforms more general.

I don't see a generate_waveform in injection.py, but if you're suggesting putting all of those keyword arguments explicitly in the function signature, I'll suggest against it.

So generate_waveforms refers to the project it's actually not too bad to add arguments thanks to typeo.
I do see what you mean though. The motivation for this discussion from my end was trying to utilize these same tools for the PE project, where non-CBC waveforms might be used. In that case, some of the default waveform_arguments (i.e. waveform_approximant and reference_frequency aren't relevant). I like the idea of using the ** syntax for the waveform_arguments , as these are not bilby specific, but dependent on the waveform you pass it. In either case what we have now obviously works fine, I was just thinking about the best way to reduce copy paste between projects.

I will take a closer look at the code for the our project starting this week. I have customized bilby previously for my own projects, and have a overview of the code base, so if something rings a bell, I'll bring it up in our meetings/slack. If there are changes that are in general helpful to bilby, we should consider contributing patches to bilby itself. The bilby-help channel on ligo mattermost is active, we can check with them about a certain design choice before diving in to make a change. In fact, the bilby waveform interface is really a wrapper around lalsimulation python API, and there are ongoing discussions about making that more user friendly. But that maybe a digression at this point, so I am OK punting that down the road and using wrappers around bilby, but also having an eye out for the lalsimulation development.

Last quick point here: been thinking more about it, and I'm not sure generate_gw or the software model that went into it are ripe for generalization. The idea the more I've thought was that there are some project-wide sensible defaults we'd like to use, but rather than make the function signature "all or nothing" (either provide a fully custom waveform generator or accept all of our defaults), you can pick and choose the defaults you want to override more simply.
This obviously makes more sense in a specific context like this, where we know the boundaries of the problem we're trying to solve for right now, and are fine to set them explicitly to simplify development. Not sure if this is entirely what's at issue here but just wanted to clarify the concept behind that API choice, which still might not be ideal.

rather than make the function signature "all or nothing" (either provide a fully custom waveform generator or accept all of our defaults), you can pick and choose the defaults you want to override more simply.

I agree. In the long run, it will be helpful to have the configuration separately. But I don't know if the scope of this issue was just config.
For example, in https://github.com/ML4GW/PE/blob/main/libs/injection/mlpe/injection/injection.py, there is another generate_gw, and project_raw_gw, which is the same in injection lib in BBHNet. So maybe the generalization here is to have refactor the injection lib out in its own thing? @EthanMarx

BTW, by same I meant similarly named. Maybe they are not doing exactly the same thing.

Reworking the purpose of this issue:

Use lalsimulation directly for generating time domain waveforms
Remove reliance on bilby waveform_generator arguments (will naturally be solved by 1.)
Rewrite with machine learning in mind i.e. batch , use torch where possible etc.


SNS Topic resource remains healthy after spec.name is changed

Describe the bug
We have observed that changing the spec.name of a resource such as an SNS topic has no effect on the resource. The topic name in AWS is not changed - this is expected as the topic name is immutable. But the k8s topic resource remains in a healthy state. For example, if the description of a “faulty” topic resource is edited, then the change is also seen on AWS. Is this behaviour expected? I believe a better behaviour here is for the resource to fail with a message saying that the spec.name is an immutable field so ACK cannot implement the change of name of the topic on AWS.
Steps to reproduce
Run kubectl apply to create an example topic:
apiVersion: sns.services.k8s.aws/v1alpha1
kind: Topic
metadata:
  name: my-ack-topic
spec:
  displayName: "Test topic"
  name: my-ack-topic

Run kubectl edit and change spec.name to something different such as my-ack-topic-edited. Resource remains as my-ack-topic on AWS as the name of a topic is immutable. However, the ACK Kubernetes resource remains healthy. If for example, spec.displayName is then changed to something different such as "Test topic edited", the description on AWS will be updated.
Expected outcome
The expected behaviour will be for the Kubernetes resource to error with a message saying that "the topic name cannot be updated because it is an immutable field".
Environment

Kubernetes version - 1.30
Using EKS (yes/no), if so version?  - yes, eks.8
AWS service targeted (S3, RDS, etc.) - SNS Subscription

@a-hilaly, I was able to reproduce this with the latest sns controller 1.0.14 version.

@daniduhnev1 this is fixed in the latest version of sns-controller. Now the controller treats spec.name as an immutable field, and will set a Terminal condition if a user modifies the field


Move the sign-in form from the options page to the popup page

When the user clicks Sign In, opening a new tab can be disruptive.
It would be better to have the sign-in process in the popup instead.

This is a duplicate of #61.


Visualisation of failed speed tests

Describe the bug
My speedtest is scheduled every hour at :55. Today I had problems with my internet provider, my connection was first slower and then completely collapsed. Of course I wanted to have a look at that in the dashboard.
I noticed that failed speed tests (those during the interruption of the connection) are not displayed at all. If you look at several tests in the table, you might not even notice the leap in time (see screenshot). The tests in between are simply missing.
To Reproduce
Steps to reproduce the behavior:
Not sure if it is possible to repoduce it when you stop your network connection and have no idea how to simulate a provider malfunction.
Expected behavior
I would expect the failed tests to be displayed with a warning sign in the table and no speed/server. This way you could also evaluate how many tests failed.
Screenshots
If applicable, add screenshots to help explain your problem.

I like this, absolutely a good suggestion and something that should be introduced.

I thought we already had something like that (with debug on for now):

I thought we already had something like that (with debug on for now):

Not sure how to activate this?! In my version I can't find this feature.
But anyway I would suggest to combine it in into one results list.
Thank you for the great project!

Failed jobs are different from failed speedtests and should only be used for debugging purposes. Jobs are the Speedtest Tracker trying to do something.
If the speedtest-cli fails I don't currently capture and report that.

so jobs=internal tasks. got it. :)
definitely, we need to capture failed speedtests, I didn't know that was missing.

Finally getting to this, you can follow the progress in #318. It's really more work than I had initially thought.


Spacesuit Chestplate Crash

When trying to figure out how to fill the chest plate, I placed it in the anvil and it crashed the game this this log. I've also noticed that shift+click to move the chest plate creates client sided duplicates of the plate. It'll appear in one inventory and not the other and vise versa.

Trying to enchant a set of space suits, also running into this issue.
Here's my log in case it helps: https://paste.ee/p/2neM7
This issue was tagged as "other mods". How come? Is it Cardinal Components' fault?

Indeed, it is CCA's fault.

It is in fact (mostly) not CCA's fault 😎

Of course it is CCA's fault, knew it. /s

Solved in development version. Will be released today if nothing goes catastrophically wrong.


Input: bind native attrs to input and textarea element

Please make sure these boxes are checked before submitting your PR, thank you!
Resolves #10314

[ ] Make sure you follow Element's contributing guide (中文 | English | Español).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

input $attr后 type="password"无效了


Discussion: behavior of deliminated and terminated

First of all, thanks for the great work for creating nom. It makes writting a parser to become a much fun process.
Prerequisites
Here are a few things you should provide to help me understand the issue:

Rust version : rustc 1.63.0 (4b91a6ea7 2022-08-08)
nom version : 7.1.1
nom compilation features used: default without adding feature flags

Problem statement
This issue is not reporting a bug, this just want to raise some limitations of existing deliminated/terminated and to see if there is a better way to resolve this, potentially either:

change the current behavior of the deliminated/terminted
or keep existing function the same for backward competibility, but create new methods for alternative use cases
or something else, potentially no change because there is a better way to do things which I did not aware

Please allow me to elaborate using the following examples.
I have a input string which has both leading and trailing spaces, I want to the parser to remove all the leading & trailing spaces.
e.g.
input: "    ABC CDE    "
expected parser output: "ABC CDE"

However, when I was using the recipe as suggested by document, it gives me
nom parsed out: "ABC CDE   "
expected: "ABC CDE"

Basically, the trailing spaces are still there. Note, if the space is between character, it is a valid token and I want to take it.
My nom parser is basicly
fn parse_header_value(input: &[u8]) -> IResult<&[u8], &[u8]> {
        delimited(
            multispace0,
            take_while(|c| is_alphanumeric(c) || b"_ :;.,\\/\"'?!(){}[]@<>=-+*#$&`|~^%".contains(&c)),
            multispace0,
        )(input)
    }

    #[test]
    fn test_parse_header_value() {
        assert_eq!(
            parse_header_value(b"   ABC CDE   "),
            Ok((&b""[..], &b"ABC CDE"[..]))
        ) // assert failed as the output is &b"ABC CDE   "
    }

This seems because when deliminate is doing the parsing, it parser the input by sequence. However, in this case, the second parse already contains the third parser, therefore the second parser consumes all the input, and leave the third parser a empty input.
The current implementation for deliminate or terminate will parse the input by order. So if the parser executed before contains the later parser, it will not be able to remove the later part. Looks in reality, we will have some use cases for such situation.
My initial thought is, could we instead of parsing them by order, for deliminated, we execute the first & third parser first, and then the second parser. Or we create a seperate function to reversely chunk the bytes from tail, and then return the rest of input. Or maybe something like regex pattern matching. I just notice there exist an nom-regex. But just want to learn from you if you know  some better ways to solve this.

FYI, I used map to mitigate this "limitation", but I am wondering if there is a better way to resolve this.
fn parse_header_value(input: &[u8]) -> IResult<&[u8], &[u8]> {
    map(
        preceded(
            multispace0,
            take_while(|c| {
                is_alphanumeric(c) || b"_ :;.,\\/\"'?!(){}[]@<>=-+*#$&`|~^%".contains(&c)
            }),
        ),
        |v: &[u8]| {
            let mut mid = v.len();
            for (i, v) in v.iter().enumerate().rev() {
                if *v != b' ' {
                    mid = i + 1;
                    break;
                }
            }
            v.split_at(mid).0
        },
    )(input)
}

    #[test]
    fn test_parse_header_value() {
        assert_eq!(
            parse_header_value(b"   4 0   "),
            Ok((&b""[..], &b"4 0"[..]))
        )
    }

Thank you very much. Happy to contribute if it is needed.

Hi @Geal. Tag you to get some input if it is possible. Feel free to at@ anyone who would be relavent.

It's not a limitation, that the intended behavior, you have space between "ABC" and "CDE", there is no reason delimited take CDE, what you have here is: spaces, list separated by space, spaces.
        delimited(
            multispace0,
            separated_list1(multispace0, alpha1)
            multispace0,
        )(input)


rust-analyzer: 2021-01-04 -> 2021-01-18

Motivation for this change
Bump version.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109962 run on x86_64-linux 1

3 packages built:

rust-analyzer
rust-analyzer-unwrapped
vscode-extensions.matklad.rust-analyzer

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109962 run on x86_64-linux 1

3 packages built:

rust-analyzer
rust-analyzer-unwrapped
vscode-extensions.matklad.rust-analyzer


feat: refactor login buttons to use dropdown

With more providers getting added, this design allows better organization and presentation.
https://github.com/Atinux/nuxt-auth-utils/assets/38922203/12324c55-2f8f-45ed-9507-3a9a61703a73

I was actually thinking of doing this or directly use card in the main page for each provider
You read my mind :D

I'm glad we're thinking the same :D
Card will look good too actually, maybe we can add both? or dropdown is enough?

Let's keep the dropdown for now 😀
I will have to deploy the playground also to have a nice demo.
But need to create all the OAuth apps 😆

Yeah that'd be nice, also I think some docs would be helpful, I can work on that if you think it's a good idea

More docs than what we have on the readme? What are you thinking of?

Oh sorry my last comment is not that clear now that I read it :d I think we can add the providers developer links(similar to what you wrote in nuxt-todo-edge setup for github)
and with a website maybe we can add examples page so user can try the features out


docs: consistent header sizing, small grammar changes

Some small README changes: adjusted header sizes to be consistent with the rest of the document, and a few grammar changes.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #366   +/-   ##
=======================================
  Coverage   94.76%   94.76%
=======================================
  Files          14       14
  Lines         955      955
  Branches      198      198
=======================================
  Hits          905      905
  Misses         50       50

Continue to review full report at Codecov.

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

Thanks for the updates!


SIGSEGV in ru.naumen.metainfo.shared.ClassFqn.equals(Ljava/lang/Object;)

Please provide a brief summary of the bug
The bug manifests itself some time after the application is launched in the Tomcat servlet container.
hs_err_pid97770.log.zip
Please provide steps to reproduce where possible
Ready to reproduce if necessary and send the necessary data.
Expected Results
No crash
Actual Results
A fatal error has been detected by the Java Runtime Environment:
SIGSEGV (0xb) at pc=0x00007f52866894b3, pid=97770, tid=98800
JRE version: OpenJDK Runtime Environment Temurin-11.0.17+8 (11.0.17+8) (build 11.0.17+8)
Java VM: OpenJDK 64-Bit Server VM Temurin-11.0.17+8 (11.0.17+8, mixed mode, tiered, compressed oops, parallel gc, linux-amd64)
Problematic frame:
J 14262 c1 ru.naumen.metainfo.shared.ClassFqn.equals(Ljava/lang/Object;)Z (39 bytes) @ 0x00007f52866894b3 [0x00007f5286689460+0x0000000000000053]
What Java Version are you using?
OpenJDK Runtime Environment Temurin-11.0.17+8 (build 11.0.17+8)
What is your operating system and platform?
Ubuntu 20.04.5 LTS, x64
How did you install Java?
Binary archive, tar.gz.
Did it work before?
This worked on OpenJDK Runtime Environment Temurin-11.0.12+7 (build 11.0.12+7) before updating the OS version from 18.04 to 20.04.

Did you test with other Java versions?
Yes.

OpenJDK Runtime Environment Temurin-11.0.12+7.

Relevant log output
No response

Dup of #654


suggestions for the reading list for reproducibility and open science

Hi,
for the reading list I would suggest to have a look at the "Ten simple rules" collection from PLOS, some of them are probably very useful. I for example like this one:
Ten simple rules for making research software more robust.
Cheers,
Andrej

That sounds great @awakenting! Thank you!
Would you like to create a pull request to add these two to the reading list?

Yes. Done.

Closing this issue because pull request #51 is merged! THANK YOU! 💯


Can't add emoji and see markers of emoji to a message.

Can't add emoji to a message until someone has already added emoji to the message, we can just increment count. Also, we can't see who added emoji to a message.
App Details:
Android 5.0.46

For the first part, we need to implement an emoji picker.
The second part, I suspect is not as important. We also need to figure out what the UI will look like.

Dup of https://github.com/zulip/zulip-mobile/issues/303 ?  (and I saw another issue about the UI for viewing who added emoji)


The indentation is incorrectly changed after pasting

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

VS Code Version: 1.66.0
OS Version: Windows 7

Steps to Reproduce:
The indentation of the existing closing code is changed after pasting, depending on the level of indentation of the copied code. Here is the animated screenshot:

the first copied block (top) has indentation with one tab more than the resulting indentation -> after pasting, the existing closing bracket has one tab less indentation;
the second copied block (bottom) has indentation with three tabs less then the resulting indentation -> after pasting, the existing closing bracket has three tabs more indentation.

I have "editor.autoIndent": "full" because I like that option of VSCode very much with its automatic indentation based on the surrounding code.
Maybe these issues are related: #86301 and #85781.

It looks like that the indentation rules for JS is not covering this case. Currently the indentation rules for
auto indent feature in VS Code is regex based, which doesn't really understand the scope (function, statement) etc. Formatting works more consistently in this scenario, so I would recommend to enable formatting on type or save. Closing this as we don't expect to get around it in the near future.

This is not problem only with JS files. Here is HTML:

Here is CSS:

And JSON:

I don't understand how your regex code could add the proper number of tabs/space characters just fine to all pasted lines, except the last one.
Could you please point me in the right direction, where should I look in your code? I am not that good with TypeScript, but I am quite fine with regular expressions and maybe I could fix that for you.


Revert "Simple Item Filter Editor"

Reverts EmmittJ/PoESkillTree#331

There was some problems that I didn't see when merging the item filter code


fix top border width for user list updates

Why
While reviewing https://github.com/bluesky-social/social-app/pull/4315, I noticed a couple of existing nits to clean up:

Still using some borderTopWidth: 1 instead of hairline in the add/remove users to list screen
Title was hitting the side of the screen and could take up multiple lines
Border above the Done button was also using 1 instead of hairline.

Before

After

Bad screenshot in the main comment, doesn't show the border between items. Here's what it actually looks like.


add number of occurences calc.

fix #1774
Added number of occurence calculator

@sairish2001 KIndly help with merge conflicts


Create project for USA presidents

Can I work on this issue.

I'm on it

solved by @kendoriddy with PR https://github.com/unclebay143/my-country-presidents/pull/25


:Gitsigns show upstream/master fails with message that buffer already is opened

Description
Can not open previously opened buffer to show upstream master version of file.
Neovim version
NVIM v0.10.0-dev-472+g0370e4def
Operating system and version
Linux pc 6.3.5-arch1-1 #1 SMP PREEMPT_DYNAMIC Tue, 30 May 2023 13:44:01 +0000 x86_64 GNU/Linux
Expected behavior
If the buffer already exists, gitsigns should find it and open it without error message.
Actual behavior
:Gitsigns show upstream/master
Error executing Lua callback: ...cal/share/nvim/lazy/gitsigns.nvim/lua/gitsigns/async.lua:105: The coroutine failed with this message: .../share/nvim/lazy/gitsigns.nvim/lua/gitsigns/diffthis.lua:83: Vim:E95: Ein Puffer mit diesem Namen exis
tiert bereits
stack traceback:
        [C]: in function 'nvim_buf_set_name'
        .../share/nvim/lazy/gitsigns.nvim/lua/gitsigns/diffthis.lua:83: in function 'run'
        .../share/nvim/lazy/gitsigns.nvim/lua/gitsigns/diffthis.lua:157: in function 'show'
        ...l/share/nvim/lazy/gitsigns.nvim/lua/gitsigns/actions.lua:1113: in function 'f'
        ...local/share/nvim/lazy/gitsigns.nvim/lua/gitsigns/cli.lua:101: in function <...local/share/nvim/lazy/gitsigns.nvim/lua/gitsigns/cli.lua:73>
stack traceback:
        [C]: in function 'error'
        ...cal/share/nvim/lazy/gitsigns.nvim/lua/gitsigns/async.lua:105: in function 'step'
        ...cal/share/nvim/lazy/gitsigns.nvim/lua/gitsigns/async.lua:129: in function 'run'
        ...on/.local/share/nvim/lazy/gitsigns.nvim/lua/gitsigns.lua:101: in function <...on/.local/share/nvim/lazy/gitsigns.nvim/lua/gitsigns.lua:100>

Translation: Vim:E95 A buffer with this name already exists.
Minimal config
vim.o.packpath = '/tmp/nvim/site'

local plugins = {
  gitsigns = 'https://github.com/lewis6991/gitsigns.nvim',
  -- ADD OTHER PLUGINS _NECESSARY_ TO REPRODUCE THE ISSUE
}

for name, url in pairs(plugins) do
  local install_path = '/tmp/nvim/site/pack/test/start/'..name
  if vim.fn.isdirectory(install_path) == 0 then
    vim.fn.system { 'git', 'clone', '--depth=1', url, install_path }
  end
end

require('gitsigns').setup{
  debug_mode = true, -- You must add this to enable debug messages
  -- ADD GITSIGNS CONFIG THAT IS _NECESSARY_ FOR REPRODUCING THE ISSUE
}

-- ADD INIT.LUA SETTINGS THAT IS _NECESSARY_ FOR REPRODUCING THE ISSUE

Steps to reproduce

Setup worktree with fish:

  function makeWorkTree -d "create worktree into cwd from https url with ending .git"
    if test (count $argv) -eq 1
      set worktreesfolder (basename -s .git "$argv[1]")
      echo "$worktreesfolder"
      mkdir "$worktreesfolder"
      cd "$worktreesfolder"
      git clone --bare "$argv[1]" .bare
      # make sure git knows where the gitdir is
      echo "gitdir: ./.bare" > .git
      git worktree add master
    else
      echo "invalid argument number"
    end
  end
  makeWorkTree https://github.com/ziglang/zig

Have a gitconfig, which fetches all branches (for convenience):

[core]
	repositoryformatversion = 0
	filemode = true
	bare = true
[remote "upstream"]
	url = https://github.com/ziglang/zig.git
	fetch = +refs/heads/*:refs/remotes/upstream/*
	fetch = +refs/pull/*/head:refs/remotes/upstream/pr/*
	gh-resolved = base

git remote rename origin downstream
git fetch
nvim --clean -u minimal.lua
:Gitsign show upstream/master
Escape or do another cancellation quickly
Try :Gitsign show upstream/master again and get aforementioned error message.

Gitsigns debug messages
There should be no debug messages necessary.

Can you check if #815 fixes this?

Was fixed, thanks alot. Now I can only get it showing an expected error, if I open the same branch in two windows.

Which error is that?

My bad. I should have run :Lazy sync. It works now flawlessly.


Added device tests for online device labs

Fixes #1965
Testing
Locally with Android
Prerequisites:

A locally installed Android emulator and one virtual device you want to test with
Python 2.x
npm

How to test:

Install appium: npm i -g appium

After starting the emulator, start appium: appium  --device-name="some name"  --platform-name="Android" --browser-name="Chrome"
Important note: For appium to find the emulator, the ANDROID_HOME environment variable has to be correctly set.

In the appium directory, activate the python virtual environment: . bin/activate

Install dependencies: pip install -r requirements.txt

Run the tests: py.test tests/

Via AWS Device Farm
Prerequisites:

An AWS Account
Python 2.x
zip

How to test:

Run the script deploy.sh, optionally supplying a name for the resulting zip file
Follow the instructions here beginning with "Upload Your Web Application Appium Python Tests"

Running into errors when trying to do the android-studio setup on our machines. It throws some errors during install and can't create the virtual device. All of the error messages sound like permission problems with our wonderful windows setup. Can somebody with a Mac or who's already switched to Linux check this one?


Photo properly sized on cards for /photos and /event page

Closes #101

Coverage remained the same at 72.258% when pulling 7b615acbffea583e2ed1a7ec39d6280a6b961717 on sam-shih:master into e0d29b0d45488941686a94d36d9a91b748c7780c on TIRaveler:master.


error_reporting has influence on behavior of collection->where()

Laravel Version: 5.5.11
PHP Version: 7.1.11 & 7.1.3

Description:
When I change error_reporting to E_NONE the behavior of Collection->where changes and then a unit test fails.
Steps To Reproduce:
I have forked the repository and used travis to check. This build fails because of a unit test that is failing https://travis-ci.org/leonrenkema/framework/builds/303214891.
The only file I have changed is tests/Support/SupportCollectionTest.php for the sake of testing and executed error_reporting(0). Changing php.ini also works.
Cause
The test works because of a PHP notice that gets generated when a stdClass is compared with an integer. With error reporting on Object of class stdClass could not be converted to int will be generated and catched and returns false.
But when error_reporting is off there is no error generated and this function will return true.
Illuminate/Support/Collection.php:484
try {
    switch ($operator) {
        default:
        case '=':
        case '==':  return $retrieved == $value;
        case '!=':
        case '<>':  return $retrieved != $value;
        case '<':   return $retrieved < $value;
        case '>':   return $retrieved > $value;
        case '<=':  return $retrieved <= $value;
        case '>=':  return $retrieved >= $value;
        case '===': return $retrieved === $value;
        case '!==': return $retrieved !== $value;
    }
} catch (Exception $e) {
    return false;
}

Thanks for this catch, here's a PR with a fix: https://github.com/laravel/framework/pull/22172


Option to match source account alert state

Summary
Currently any alert conditions that are migrated are set to disabled. We should add an option to match the state of the alert condition (enabled/disabled) from the source account.
Desired Behavior
The default behavior should still be to disable all alert conditions in the target account. We should be able to optionally override that default behavior to match the enabled / disabled state of each condition in the source account to the target account.
Possible Solution
Add an optional argument to the migrateconditions.py script and check to see if it is set before overriding the enabled attribute when copying alert conditions to the target account.
Additional context
This provides an additional option for customers who do not want to review and enable alert conditions individually in the target account post migration.

This capability was merged with https://github.com/newrelic-experimental/nr-account-migration/pull/16.


Animation appears at (0,0) of frame

So this is how I set up the activity indicator:
let loading = NVActivityIndicatorView(frame: CGRectZero, type: .BallPulse, color: .whiteColor())
loading.translatesAutoresizingMaskIntoConstraints = false
signInButton.addSubview(loading)
loading.snp_makeConstraints{(make) -> Void in
    make.center.width.height.equalTo(signInButton)
}
loading.backgroundColor = UIColor.blueColor() // this is to see where the frame is.
loading.startAnimation()

But in the simulator it looks like this:

It seems size of animation layer is calculated before the actual size of view is determined by autolayout.
One quick fix I suggest is to set frame to something else if the size is already known.

That works, thanks!


redesign

Anybody like my design? here is the .sketch file https://dl.dropboxusercontent.com/u/244623/github pulse.sketch
Hope someone can pull a request to make the design to css and html. Sorry, i am not familiar with css.
fonts:

Montserrat
Open Sans font

Wow, very nice! Thanks!
If nobody's done it so far I'll push it myself later today!

@wjp2013 I draw something yesterday too.

I released it on the experimental branch, if you guys wan't to check it out first.

cool :+1:


Isabelle review: phase-to-phase migrations

Acceptance

[x] Able to enable phase migrations in competition editor
[ ] Phase does not migrate when it's already migrating
[ ] Phase does not migrate when it is scoring submissions
[x] Phase auto-migrates after date passes, submissions are run

Not possible to migrate by hand.

Could not get it to work by changing the date.

Which phase did you set to migrate? auto_migrate_to should be set on the phase to migrate to, not the phase to migrate from. Hope that makes sense.

TODO: Add video demonstrating this working

VIDEO is not needed. I understand what needs to be done, this is not the problem.
I created a collection of bundles to test V2.
V1 bundles can be found here
V2 bundles can be found here

Cannot complete upload a bundles from v1: stuck at the unpack level. An example is

![Uploading Screen Shot 2019-11-27 at 8.02.53 PM.png…]()

Cannot complete

The competition yaml of all v2 competitions needs to have version: 2 in it, otherwise it attempts to unpack it as a v1.5 bundle (hence HTML pages being required). So your Iris competition and Sample competition will not upload correctly. The wheat seed competition should upload correctly, though.
Thank you for catching the error with dumps, looks like they are not including versions in their yaml files. I can fix this. In the mean time, you can open the dump competition yaml and add version: 2 to it, and it should upload.
Can you be specific about which v1 bundle is getting stuck in unpack forever? I'd be happy to look in to it, but I need to know which bundle isn't working, or if all of them aren't working.

I do not know how to test:
Phase does not migrate when it's already migrating
Phase does not migrate when it is scoring submissions
In the future, for testing I suggest to provide:

the URL of the server to be tested
competition bundles that work
also website with the corresponding bundle already uploaded on which I am an admin
sample submissions that work
step-by-step instructions of what to do including screen shots if there is something tricky

The competition yaml of all v2 competitions needs to have version: 2 in it, otherwise it attempts to unpack it as a v1.5 bundle (hence HTML pages being required). So your Iris competition and Sample competition will not upload correctly. The wheat seed competition should upload correctly, though.
Thank you for catching the error with dumps, looks like they are not including versions in their yaml files. I can fix this. In the mean time, you can open the dump competition yaml and add version: 2 to it, and it should upload.

I did that but then got another error:

Please test a bunch of dumps and re-upload them to make sure they work.

Can you be specific about which v1 bundle is getting stuck in unpack forever? I'd be happy to look in to it, but I need to know which bundle isn't working, or if all of them aren't working.

Malaria detection challenge.zip
from V1 bundles


AbstractPluginManager raises deprecation warning (possible bug?)

As many of you know, migration to Zend\ServiceManager 3.0 isn't straightforward as it includes many BC breaks. This includes the removal of ServiceLocatorAwareInterface.
This is the place to ask if this is an actual bug or just early notice, but in version 2.7.6 of zend-servicemanager, the Zend\ServiceManager\AbstractPluginManager tends to implement the interface above, causing an E_WARNING to be thrown during it's initialization.
Is it possible to create a fix for it or I should consider migration to version 3.0 of the module, when possible?

@MaDKaTZe The zend-mvc 2.7 series fixes this issue (see zendframework/zend-mvc#88) by injecting the parent locator during instantiation, and a patch applied for 2.7.4 (see zendframework/zend-mvc#114) ensured that if a service locator was already present in a plugin manager, it would not attempt the injection — and therefor no longer raise a deprecatoin message.
The only reason you should see such messages at this point are for custom plugin manager implementations; no plugin managers implemented by ZF should cause the issue to be raised any longer.


Adjust our semver test to also allow for tags matching 'v0.0.0' form.

Also, rebuild API documentation for master branch and semver tags.

Coverage remained the same at 100.0% when pulling 1ba3e82df8a455f4f81db6779c50c630c2a8fdab on semver-test into 0874dca593f79305a963dc0f40fc99e8fc495b78 on master.

Coverage remained the same at 100.0% when pulling d380a68bda8a645422eb4f967de9504476c47cf3 on semver-test into 0874dca593f79305a963dc0f40fc99e8fc495b78 on master.

Coverage remained the same at 100.0% when pulling 727c0824b8a99fc3b9bd93021aa368cb3d073961 on semver-test into 0874dca593f79305a963dc0f40fc99e8fc495b78 on master.

I switched out documentation strategy slightly here..


Open Shortcuts app URLs with a confirmation dialog

Task/Issue URL: https://app.asana.com/0/414235014887631/1203051209824446/f
Tech Design URL:
CC:
Description:
We define platform schemes that prompt user for confirmation as opening those URLs redirects user to another app. iOS provides a shortcuts scheme for launching the Shortcuts app which should be handled in the same way.
Steps to test this PR:

Create on a simulator or device a shortcut with a predefined name (e.g. "Test shortcut")
On a static webpage or jsfiddle.net create a following link (provide the right name as the name parameter):
LINK
Tap the link to run the shortcut 4.

Device Testing:

[ ] iPhone
[ ] iPad

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16

Small note: it probably would be good to add this test case to https://github.com/duckduckgo/privacy-test-pages


[Bug] Positioning of suggestions incorrect when monaco editor is placed inside a Fluent panel.

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[X] Not reproducible in the monaco editor playground

Monaco Editor Playground Code
No response
Reproduction Steps
Here is the code pen for a min repro - https://codepen.io/cheriequek/pen/XWBjKOX
I was unable to use the monaco editor playground as it did not seem to allow me to load the fluent libraries.
Load the codepen and wait for a little bit (need to wait for monaco to load)
Then click Load editor
When the editor loads, attempt to type something that would bring up the intellisense suggestions (eg. console)
Actual (Problematic) Behavior
The intellisense suggestions show up in the wrong location and most of it is not visible.

Expected Behavior
I expect to see the intellisense suggestions (and other widgets) to respect the size of the panel and be fully visible within the panel width.
Additional Context
I've seen this other issue which is similar - https://github.com/microsoft/monaco-editor/issues/2503. and have attempted other user's workarounds and none of them seem to work. I've opened a different issue mainly because the positioning issue seems to be different from the screenshots provided.

Sorry, but we cannot investigate issues that neither occur in vscode nor in the monaco playground


datebase not support key contain '.'

Because mongoDB don't support key value contains '.' .
It's really troublesome....

possible solution:

https://github.com/mongodb-js/extended-json/issues/14
https://stackoverflow.com/questions/12397118/mongodb-dot-in-key-name

Since like it can be added & followed, but the database will throw error. Don't know why.


[SPARK-13803] restore the changes in SPARK-3411

What changes were proposed in this pull request?
This patch contains the functionality to balance the load of the cluster-mode drivers among workers
This patch restores the changes in https://github.com/apache/spark/pull/1106 which was erased due to the merging of https://github.com/apache/spark/pull/731
How was this patch tested?
test with existing test cases

Test build #53097 has started for PR 11702 at commit c82e90f.

Test build #53099 has started for PR 11702 at commit 3e1882d.

Test build #53097 has finished for PR 11702 at commit c82e90f.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53097/
Test FAILed.

jenkins, retest this please

Test build #53100 has started for PR 11702 at commit 3e1882d.

Test build #53099 has finished for PR 11702 at commit 3e1882d.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53099/
Test FAILed.

Test build #53100 has finished for PR 11702 at commit 3e1882d.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53100/
Test FAILed.

Merged build finished. Test FAILed.

Shouldn't the changes in #2436 be restored as well?

Test build #53127 has started for PR 11702 at commit 0bb54f6.

This is now working as expected for me. Thanks!

Test build #53127 has finished for PR 11702 at commit 0bb54f6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53127/
Test PASSed.

@bwongchaowart thanks, @srowen it's done,

I'm going to go ahead and merge. It passes, and clearly this change was already intended to be in the source.

Merged to master back to 1.4

thanks @srowen


useMenuTrigger doesn't set "disabled" attribute on button when isDisabled is true

🐛 Bug Report
When using useMenuTrigger() and when isDisabled is passed to the component, trigger props doesn't include disabled property for button element. This results in button trigger being still active (able to interact with) when list revealing is expanding.

🤔 Expected Behavior

Props for button received from useMenuTrigger() hook include disabled attribute, which can be passed directly to <button/>.
😯 Current Behavior

Props for button received from useMenuTrigger() hook doesn't include disabled attribute. It results in <button/> being active despite that trigger is blocked. It feel odd, when using isDisabled property on select component.
💁 Possible Solution

Just add disabled property in object returned from useMenuTrigger() hook:
https://github.com/adobe/react-spectrum/blob/40d086a558b5a2b77e0816c0820e449edfa36b5f/packages/%40react-aria/menu/src/useMenuTrigger.ts#L122-L126
Of course disabled should be true only when isDisabled is true.
🔦 Context

I'm using useSelect() hook and passing isDisabled to my select component doesn't lock <button/>.  If anyone finds it confusing enough to warrant additional description, we accept contributions. Thank you for understanding.


Plugin attempts to validate API keys when the Sezzle payment method is disabled

Issue
There appears to be a bug in the Plugin/Config/Controller/Adminhtml/System/Config/SavePlugin.php file related to validating the API keys.
In the aroundExecute method, there's a section that appears to try to validate the Sezzle API keys when they've changed:
            // only validate keys if they are changed
            if ($this->hasKeysChanged($oldConfig, $newConfig)) {
                if (!$merchantUUID = $this->authenticationService->validateAPIKeys(
                    $newConfig[SezzleConfig::KEY_PUBLIC_KEY],
                    $newConfig[SezzleConfig::KEY_PRIVATE_KEY],
                    $newConfig[SezzleConfig::KEY_PAYMENT_MODE]
                )) {
                    throw new ValidationException(__('Auth token not found.'));
                }
            }

The core issue is there's no way to leave empty or invalid keys in the config when you try to save the configurations on the payment methods, effectively locking store owners out of modifying their payment methods without first setting up valid Sezzle API keys. This can cause headaches with the store owners if they need to change other payment settings before they've gotten their API keys, or if they're running multiple stores and only want the payment method enabled on one of the stores.
Steps to Replicate

Clone the repo and follow the setup instructions. (You may need to update the database image in the docker-compose.yml file too. mysql:5.6.23 seems to use an older docker scheme that didn't work for me. Updating it to mysql:5.6.51 fixed the setup issue for me.)
Navigate to the payment method settings - Store > Config > Sales > Payment Methods
Try to save the page without changing anything. You should receive an error message saying "Unable to validate the Sezzle API Keys." and any changes you've made won't save.

Suggested Change(s)
I've got two ideas for fixing this issue that seem to work for me:

Add an additional check to only validate the keys when they've changed AND when the payment method is enabled in the config. This seems like the most logical since there shouldn't really be any reason to check if API keys are working if the store owner doesn't even have the payment method enabled. Just changing the if statement above to $newConfig['sezzle_enabled'] && $this->hasKeysChanged($oldConfig, $newConfig) would do the trick.
Modify the logic of the hasKeysChanged method to use the inequality operator != rather than the non-identity operator !==. The reason the extension tests the API keys when they're empty even when they've not really changed is because the key data in the post request is an empty string, while the data in the database is null and the non-identity operator will see these as different values thus triggering the API key validation logic. This fixes the issue, but doesn't make as much sense to do since it only really covers the case where the store owner doesn't have them set in the first place, and presumably they'd still have the payment method disabled anyway.

We are facing issues in releasing the version in the Magento marketplace. We will update here once the new version is released. Thanks.

Changes released in 7.0.17


redundant code lines in metric

Looks like this line and the one before it do the same thing:
https://github.com/huggingface/evaluate/blob/7fa754cc38c523b69054c67b8c3b8e3f4962fbf6/metrics/mean_iou/mean_iou.py#L145

Hey,
Can I solve this issue?


Diff view: output metadata change UX is confusing

Version: 1.63.0-insider (user setup)
Commit: ed3018f4310e906daa0a9119d997e75b8a38b879
Date: 2021-11-23T05:33:39.595Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.22000
Steps to Reproduce:

open this notebook in VS Code https://github.com/claudiaregio/data-science/blob/main/Titanic.ipynb
"Run All" cells
switch to the diff view for the notebooks file
scroll down to this cell

It's confusing that the two tables are highlighted in red and green colors, even though their content is identical. Also, the "Switch output rendering" button is hard to discover as it's on the far-right side.

Related: https://github.com/microsoft/vscode/issues/135820

I knew I saw this before. Closing #135820 in lieu of this one. I think Claudia may have run into this too.

Actually closing this one instead. #135820 is the original and contains much more detail.


ci: smoke test improvement

Describe pull-request
Some improvements:

Removed unnecessary code
Exported core version to make sure it uses the same in wrappers

How to test
Run action in GitHub

Maybe add option to set have the possibility to run on different branches and different node versions by adding the following:

Code is taken from beta-release.yml so you can copy it from there. Pay attention how are later on Node version and branch referenced.


Add suggestions when user is searching I'd and Improve the navbar

Hello @SmrutiRanjan-L
Can I do this?

Yeah, you can. Provide a preview SS before making PR.

Okay. Will do so.

suggestions when the user is searching I'd and Improved the navbar.

Great. It is as expected. Go for making a PR.

Thank you.


Nothing happens

So when I set it up, I run the program. The loading screen launches, but nothing else pops up. I ran the debug program. Number 2 seems fine, but number 1 starts off like this:
Select number: 1
INFO:captions.json not found.
INFO:corruption.json not found.
INFO:discord.dat not found.
INFO:info.json not found.
INFO:media.json not found.
INFO:prompt.json not found.
INFO:web.json not found.
Exception in thread Thread-4 (listen):
Traceback (most recent call last):
File "C:\Users\Shez\AppData\Local\Programs\Python\Python312\Lib\threading.py", line 1075, in _bootstrap_inner
self.run()
File "C:\Users\Shez\AppData\Local\Programs\Python\Python312\Lib\threading.py", line 1012, in run
self._target(*self._args, **self._kwargs)
File "C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\src\panic.py", line 33, in listen
with Listener(address=ADDRESS, authkey=AUTHKEY) as listener:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\Shez\AppData\Local\Programs\Python\Python312\Lib\multiprocessing\connection.py", line 464, in init
self._listener = SocketListener(address, family, backlog)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\Shez\AppData\Local\Programs\Python\Python312\Lib\multiprocessing\connection.py", line 608, in init
self._socket.bind(address)
then pictures do show up in the command prompt:
OSError: [WinError 10048] Only one usage of each socket address (protocol/network address/port) is normally permitted
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35742_484e0cd48956ae59431b7c71945fd31f92cfddf7.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35741_72ffaac72eb193ddf2196d99d8fa8c32a66ef239.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35726_4502878716727eb4fda868708a987890751990b3.jpg
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35725_dd1a9aac3f4eb564b0f82394266a6c5d7af3d488.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35711_839252b40c48073a21382b6ea0214cb38d3886d7.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35710_cdd5f1c94e42289458a7ac54791d85d949e25eae.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35709_8e3d581ea4375c826e1eb6dc327339d6e458ca95.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35703_29674184362610407b96655ca251830a963c29c5.jpg
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35698_4ac5211952fab38210ace044ff7a30fec7d5c5fa.jpg
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35682_ea0a214895f8fbee95240b588a732a15a507d940.jpg
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35681_e690af5a3c35bb93fe6ea6c64aac93e78f21d30a.jpg
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35674_3a5c9969fa6303f08023b2bcc7af2ec0afac8b9a.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35672_98ea416b8226827c4ea67ef39878954e7ae573fc.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35669_80c300dfabee60121594427987f95225990a8e18.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35668_6fc30fc128fce02a2051198fdba71462666bd71d.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35664_e6ed2f5f17813010cdabaec0119ae5d5773769c2.png
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35663_74b7da6162da3b97511d0bbc0b42c25342c86e9d.gif
C:\Users\Shez\Desktop\EdgewarePlusPlus-main\edgeware\data...ensored_35655_fe06cf549c914e6c57f24edb0d6738f904f42f20.jpg
I can confirm that when I go to my "data" and then "download," images are coming in. However, even after all that, nothing pops up on my screen (I have VLC installed), except for the first starter image that then fades away.
what could be my issue here? thanks for reading!

btw i do have a resource pack installed and  imported

Edgeware uses sockets for sending and receiving panic signals in certain cases, it listens on localhost:6000 and it looks like you may already have something else running on that address. Try changing 6000 in line 10 of edgeware/src/panic.py to something else:
ADDRESS = ("localhost", 6000)

Did that error seem to have gone away, but the same issue is still there. The loading flair appears and then just fades away. When I run through the debug way, I get this while the loading flair pops up and then fades away. I re-downloaded your latest version today and nothing changed. It's the same thing. I read the code and noticed that you removed the lanczos function. Yeah, the description you added was pretty accurate. Even in the other version made by the creator, it did not work when I tried it a couple of months ago. It seems like I'm having the same issue. Also tried disconnecting it from github from the config menu did not do much. Another thing I would like to bring up is whether it is normal for import functions and from functions to be highlighted in yellow, indicating that it "Could not be resolved" in Pylance. Is that supposed to be normal?

Since it isn't finding any of the JSON files, have you loaded any pack and are you sure it's in the correct location? Packs should be placed/loaded under edgeware/resource.

I'm going to assume your problem has been fixed. If you still have problems, feel free to ask again!


Request: add support for unusual characters in feature names

These will be particularly useful for utility features, enabling clearer names that keep us organized.
E.g. I would like to add "Grants ability U: untap — Aura" (we're trying out categorizing utility features by card type, hence the final bit)

Use - instead of —


Events

Is there any way to catch the response from TS server in browser as variable or something? I can't find it in Instantsearch.js
I mean - any time the search return hits I need somehow to get those and lets say send some analytics or something else

@victorbosh Have a look at the analytics widgets in InstantSearch.js.
Here's an example where I used it to send analytics to GA: https://github.com/typesense/showcase-songs-search/blob/39b3a402da0d0df4cda458daeb3909e030e3ba58/src/app.js#L187-L196

I tried it but it is deprecated. I also tried upgrade to insights middleware as mentioned here https://www.algolia.com/doc/guides/building-search-ui/upgrade-guides/js/#analytics-widget
But I can not even start to list the problems for example events are fired randomly times (just like 'render' event too) which makes no sense at all. I spent last week just dealing with Instantsearch.  It is more tricky to implement that stated. Well, thank you anyway

I've noticed a bit of flakiness with the analytics widget every now and then as well. I'd recommend opening an issue in the InstantSearch.js repo to see if this is a known issue and/or if they have any workarounds.


UI ontology should be added

With forms and other UI elements using the UI ontology, it would be nice to add it as a top-level context, at least, so we can use this context in our code.
http://www.w3.org/ns/ui#

Thanks. I added the prefix, but no other shortcuts yet, as I believe ui is still pretty much in flux.


fetch can not send point

i want to send request with<EMAIL_ADDRESS>when use fetch ,it always is<EMAIL_ADDRESS>const searchResponse = await fetch(
        `http://gitlab.xxxxxx.com:4001/api/v3/users/liutanrong@gmail%2ecom`,
        { headers: getHeaders() }
      );

%2e in URLs is the same as .. I don't know what you're asking, or what your problem is, but this is no place to ask. This is an issue tracker for a fetch polyfill used in old browsers that don't support window.fetch natively.


MINOR: Fix display names for parameterized KRaft and ZK tests

This patch adds display names for KRaft and ZK tests. Without this, it becomes hard to understand in Jenkins test reports which test failed. With this addition, it becomes more clear which method in the test suite fails.

Thanks for the PR, @skaundinya15 . This seems like a good readability improvement. Is it possible to use a constant here rather than repeating the literal string? If we had a constant, we could put it in TestInfoUtils, I suppose.

@cmccabe Thanks for the feedback, just pushed a new patch where there's a string in TestInfoUtils and used across all the tests.

retest this please


Add template check upfront

It's very annoying to see a lot of plugin-related messages if you don't want to use in current project

Neat!


Switch to using ThemeResource for icon in InfoBar

Switching InfoBar icon foreground to use ThemeResource instead of StaticResource so that it updates with theme changes. Fixes #3581

/azp run


Added hover effect in "start" Button #197

Change cursor to pointer on hover
Slightly increase button size and lighten background color
Add smooth transition for hover effects"

PR Description 📜

Change cursor to pointer on hover
Slightly increase button size and lighten background color
Add smooth transition for hover effects"

Mark the task you have completed ✅

[ ✅] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[✅ ] I have performed a self-review of my own code or work.
[ ✅] I have commented my code, particularly in hard-to-understand areas.
[ ✅] My changes generates no new warnings.
[ ✅] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[✅ ] I have added screenshot for review.

## Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

I Have a pull request could you please review it

which issue does it address, attach it in the PR
conflicts present

this issue i have mentioned the issue number in the name.


‘static func stories<C: Collection>(by identifiers: C)’ always return error.

[x] I've searched past issues and I couldn't find reference to this.

Hi Sbertix,
I  try to get reelMedia list by:
static func stories<C: Collection>(by identifiers: C) -> Endpoint.ResponseDisposable where C.Element == String

But no matter I if the parameter I pass in is archived media's id or user's pk:
Endpoint.Feed.stories(by:["archiveDay:18038182546262472","archiveDay:17918118640295965"])
                .unlocking(with: secret)
                .task(
                    by: .instagram,
                    onComplete: { result in
                        switch result {
                        case .success(let response):
                            print(response)
                        case .failure(let error):
                            print(error)
                        }
                }).resume()

or

Endpoint.Feed.stories(by:[my.pk])
                .unlocking(with: secret)
                .task(
                    by: .instagram,
                    onComplete: { result in
                        switch result {
                        case .success(let response):
                            print(response)
                        case .failure(let error):
                            print(error)
                        }
                }).resume()

It always return an error like this:
["status": Response("fail"), "message": Response("Invalid reel id list")]

In SwiftyInsta
Sometimes I get the correct result, sometimes I get empty data.

The changes in #60 made it work for me.
I'm merging it right now and bumping to 3.1.1.
Let me know if it fixes for you 💪

If you found this useful, please consider starring :star: this repo, or even sponsoring it :dollar:.Even just $5 can help me maintain this.Thank you for using Swiftagram.

I've found out that the error can still occur on 4.0.*.
I've pushed a fix that will be released with 4.1.0.
Sorry.


Use relative path in file_util and gl_pickle tests

Use relative path in the following test:
test_file_util.py
test_gl_pickle.py

Fixed!


A very good SAM fine-tuning job

This is an exciting work, I read your article, I would like to ask if your team has tested it on remote sensing imagery, which is a relatively complex task.

Hi, thanks for watching our work. We didn't try on the remote sensing images yet.

Hi, thanks for watching our work. We didn't try on the remote sensing images yet.

Thank you for your reply and very much look forward to your work being open sourced in the community. I will try to put your research to the test in our study if I get the chance!

We will release it within June. Please stay tuned!

respect

A very good improvement over the 'Foundation Model". Could this be extended for point clouds also in the future?

I am adding support for HQ-SAM in the segment-geospatial package, which is designed for segmenting remote sensing image. See https://github.com/opengeos/segment-geospatial/pull/161


Cannot change Scaffold background when using PageBuilder with transition in go_route inside ShellRoute

Is there an existing issue for this?

[X] I have searched the existing package issues
[X] I have read the guide to filing a bug

What package does this bug report belong to?
go_router
What target platforms are you seeing this bug on?

Yes
Dependency versions
pubspec.lock
# Generated by pub
# See https://dart.dev/tools/pub/glossary#lockfile
packages:
  animations:
    dependency: "direct main"
    description:
      name: animations
      sha256: fe8a6bdca435f718bb1dc8a11661b2c22504c6da40ef934cee8327ed77934164
      url: "https://pub.dev"
    source: hosted
    version: "2.0.7"
  async:
    dependency: transitive
    description:
      name: async
      sha256: "947bfcf187f74dbc5e146c9eb9c0f10c9f8b30743e341481c1e2ed3ecc18c20c"
      url: "https://pub.dev"
    source: hosted
    version: "2.11.0"
  boolean_selector:
    dependency: transitive
    description:
      name: boolean_selector
      sha256: "6cfb5af12253eaf2b368f07bacc5a80d1301a071c73360d746b7f2e32d762c66"
      url: "https://pub.dev"
    source: hosted
    version: "2.1.1"
  characters:
    dependency: transitive
    description:
      name: characters
      sha256: "04a925763edad70e8443c99234dc3328f442e811f1d8fd1a72f1c8ad0f69a605"
      url: "https://pub.dev"
    source: hosted
    version: "1.3.0"
  clock:
    dependency: transitive
    description:
      name: clock
      sha256: cb6d7f03e1de671e34607e909a7213e31d7752be4fb66a86d29fe1eb14bfb5cf
      url: "https://pub.dev"
    source: hosted
    version: "1.1.1"
  collection:
    dependency: transitive
    description:
      name: collection
      sha256: "4a07be6cb69c84d677a6c3096fcf960cc3285a8330b4603e0d463d15d9bd934c"
      url: "https://pub.dev"
    source: hosted
    version: "1.17.1"
  cupertino_icons:
    dependency: "direct main"
    description:
      name: cupertino_icons
      sha256: e35129dc44c9118cee2a5603506d823bab99c68393879edb440e0090d07586be
      url: "https://pub.dev"
    source: hosted
    version: "1.0.5"
  fake_async:
    dependency: transitive
    description:
      name: fake_async
      sha256: "511392330127add0b769b75a987850d136345d9227c6b94c96a04cf4a391bf78"
      url: "https://pub.dev"
    source: hosted
    version: "1.3.1"
  flutter:
    dependency: "direct main"
    description: flutter
    source: sdk
    version: "0.0.0"
  flutter_lints:
    dependency: "direct dev"
    description:
      name: flutter_lints
      sha256: "2118df84ef0c3ca93f96123a616ae8540879991b8b57af2f81b76a7ada49b2a4"
      url: "https://pub.dev"
    source: hosted
    version: "2.0.2"
  flutter_test:
    dependency: "direct dev"
    description: flutter
    source: sdk
    version: "0.0.0"
  flutter_web_plugins:
    dependency: transitive
    description: flutter
    source: sdk
    version: "0.0.0"
  go_router:
    dependency: "direct main"
    description:
      name: go_router
      sha256: b3cadd2cd59a4103fd5f6bc572ca75111264698784e927aa471921c3477d5475
      url: "https://pub.dev"
    source: hosted
    version: "10.0.0"
  js:
    dependency: transitive
    description:
      name: js
      sha256: f2c445dce49627136094980615a031419f7f3eb393237e4ecd97ac15dea343f3
      url: "https://pub.dev"
    source: hosted
    version: "0.6.7"
  lints:
    dependency: transitive
    description:
      name: lints
      sha256: "0a217c6c989d21039f1498c3ed9f3ed71b354e69873f13a8dfc3c9fe76f1b452"
      url: "https://pub.dev"
    source: hosted
    version: "2.1.1"
  logging:
    dependency: transitive
    description:
      name: logging
      sha256: "623a88c9594aa774443aa3eb2d41807a48486b5613e67599fb4c41c0ad47c340"
      url: "https://pub.dev"
    source: hosted
    version: "1.2.0"
  matcher:
    dependency: transitive
    description:
      name: matcher
      sha256: "6501fbd55da300384b768785b83e5ce66991266cec21af89ab9ae7f5ce1c4cbb"
      url: "https://pub.dev"
    source: hosted
    version: "0.12.15"
  material_color_utilities:
    dependency: transitive
    description:
      name: material_color_utilities
      sha256: d92141dc6fe1dad30722f9aa826c7fbc896d021d792f80678280601aff8cf724
      url: "https://pub.dev"
    source: hosted
    version: "0.2.0"
  meta:
    dependency: transitive
    description:
      name: meta
      sha256: "3c74dbf8763d36539f114c799d8a2d87343b5067e9d796ca22b5eb8437090ee3"
      url: "https://pub.dev"
    source: hosted
    version: "1.9.1"
  path:
    dependency: transitive
    description:
      name: path
      sha256: "8829d8a55c13fc0e37127c29fedf290c102f4e40ae94ada574091fe0ff96c917"
      url: "https://pub.dev"
    source: hosted
    version: "1.8.3"
  sky_engine:
    dependency: transitive
    description: flutter
    source: sdk
    version: "0.0.99"
  source_span:
    dependency: transitive
    description:
      name: source_span
      sha256: dd904f795d4b4f3b870833847c461801f6750a9fa8e61ea5ac53f9422b31f250
      url: "https://pub.dev"
    source: hosted
    version: "1.9.1"
  stack_trace:
    dependency: transitive
    description:
      name: stack_trace
      sha256: c3c7d8edb15bee7f0f74debd4b9c5f3c2ea86766fe4178eb2a18eb30a0bdaed5
      url: "https://pub.dev"
    source: hosted
    version: "1.11.0"
  stream_channel:
    dependency: transitive
    description:
      name: stream_channel
      sha256: "83615bee9045c1d322bbbd1ba209b7a749c2cbcdcb3fdd1df8eb488b3279c1c8"
      url: "https://pub.dev"
    source: hosted
    version: "2.1.1"
  string_scanner:
    dependency: transitive
    description:
      name: string_scanner
      sha256: "556692adab6cfa87322a115640c11f13cb77b3f076ddcc5d6ae3c20242bedcde"
      url: "https://pub.dev"
    source: hosted
    version: "1.2.0"
  term_glyph:
    dependency: transitive
    description:
      name: term_glyph
      sha256: a29248a84fbb7c79282b40b8c72a1209db169a2e0542bce341da992fe1bc7e84
      url: "https://pub.dev"
    source: hosted
    version: "1.2.1"
  test_api:
    dependency: transitive
    description:
      name: test_api
      sha256: eb6ac1540b26de412b3403a163d919ba86f6a973fe6cc50ae3541b80092fdcfb
      url: "https://pub.dev"
    source: hosted
    version: "0.5.1"
  vector_math:
    dependency: transitive
    description:
      name: vector_math
      sha256: "80b3257d1492ce4d091729e3a67a60407d227c27241d6927be0130c98e741803"
      url: "https://pub.dev"
    source: hosted
    version: "2.1.4"
sdks:
  dart: ">=3.0.6 <4.0.0"
  flutter: ">=3.3.0"

Steps to reproduce

Create an app with Material.router.
Use a GoRoute with pageBuilder in ShellRoute with CustomTransitionPage

Expected results
I am expecting to use the Scaffold background color that I set.
Actual results
The Scaffold background color cannot be applied. It uses ColorScheme.background
Both ThemeData.scaffoldBackgroundColor and Scaffold.backgroundColor won't work

It works when the route is not in ShellRoute

It works when using GoRoute(path: '/',builder: (context, state) => const HomeScreen())

It does not work when using CustomTransitionPage inside ShellRoute

Code sample
Dartpad example
https://dartpad.dev/?id=63605a22db2bd742d3f4eb6169fc1c88
Code sample
import 'package:animations/animations.dart';
import 'package:flutter/material.dart';
import 'package:go_router/go_router.dart';

void main() {
  runApp(const MyApp());
}

final router = GoRouter(
  initialLocation: '/',
  routes: [
    ShellRoute(
      builder: (BuildContext context, GoRouterState state, Widget child) {
        return Scaffold(
          backgroundColor: Colors.lightGreen,
          body: child,
          bottomNavigationBar: BottomNavigationBar(
            showSelectedLabels: false,
            showUnselectedLabels: false,
            items: const <BottomNavigationBarItem>[
              BottomNavigationBarItem(
                icon: Icon(Icons.home_outlined),
                label: 'Home',
              ),
              BottomNavigationBarItem(
                icon: Icon(Icons.person_outline),
                label: 'Camera',
              ),
            ],
            onTap: (int index) {
              if (index == 0) {
                context.go('/');
              } else if (index == 1) {
                context.go('/anim');
              } else if (index == 2) {}
            },
          ),
        );
      },
      routes: [
        GoRoute(
          path: '/anim',
          pageBuilder: (context, state) => CustomTransitionPage(
            key: state.pageKey,
            child: const Center(
              child: Text('With transition'),
            ),
            transitionsBuilder:
                (context, animation, secondaryAnimation, child) =>
                    SharedAxisTransition(
              animation: animation,
              secondaryAnimation: secondaryAnimation,
              transitionType: SharedAxisTransitionType.scaled,
              child: child,
            ),
          ),
        ),
        GoRoute(
          path: '/',
          builder: (context, state) => const Center(
            child: Text('home'),
          ),
        ),
      ],
    ),
  ],
);

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp.router(
      routerConfig: router,
      title: 'Flutter Test',
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        scaffoldBackgroundColor: Colors.cyan,
        colorScheme: ColorScheme.fromSeed(
          seedColor: Colors.red,
          brightness: Brightness.dark,
        ).copyWith(background: Colors.blue),
        useMaterial3: true,
        brightness: Brightness.dark,
      ),
    );
  }
}

Screenshots or Videos

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.6, on macOS 13.4 22F66 darwin-arm64, locale
    tr)
    • Flutter version 3.10.6 on channel stable at
      /Users/user/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (3 hafta önce), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/user/Library/Android/sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: /opt/homebrew/opt/openjdk/bin/java
    • Java version OpenJDK Runtime Environment Homebrew (build 20.0.1)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✗] Chrome - develop for the web (Cannot find Chrome executable at
    /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[!] Android Studio (not installed)
    • Android Studio not found; download from
      https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup
      for detailed instructions).

[✓] VS Code (version 1.81.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.70.0

[✓] VS Code (version 1.81.0-insider)
    • VS Code at /Applications/Visual Studio Code - Insiders.app/Contents
    • Flutter extension version 3.68.0

[✓] Connected device (1 available)
    • macOS (desktop) • macos • darwin-arm64 • macOS 13.4 22F66 darwin-arm64

[✓] Network resources
    • All expected network resources are available.

Thanks for the detailed report. Seeing the same behavior as reported.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.10.6, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.10.6 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (5 days ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.13.0-16.0.pre.19, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.13.0-16.0.pre.19 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c51aa3d2de (79 minutes ago), 2023-08-02 11:42:06 +0800
    • Engine revision 9dae7b708b
    • Dart version 3.2.0 (build 3.2.0-22.0.dev)
    • DevTools version 2.25.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

I can not reproduce this on Flutter 3.22.1 anymore. I used the sample code as provided and Scaffold.backgroundColor is applied correctly.


docs: Changing all occurrence of older name

Amend your commit message in accordance to the Conventional Commits specification and add appropriate description for the PR.


It need some time to establish connect

Hello,
My Server is Spring,  my client is android, i find that i need five seconds or even more (quickly sometimes)  between beforeHandshake() and afterHandshake(),I don't know why and i want to make it faster,do you have some suggestions?

I use Android -> HAProxy (handling TLS) -> Jetty and the connection is quick


BVTCK-132 Add tests for section 4 - Value extractor definition

https://hibernate.atlassian.net/browse/BVTCK-132
https://hibernate.atlassian.net/browse/BVTCK-133

This PR introduces the new testing utility we talked about the other day and uses it in some tests.
It also introduces most tests required to test the assertions of the paragraph Value extractor definition.
@gunnarmorling we have an issue with JavaFX and the incontainer mode: we need to make the JavaFX classes visible and apparently we would need something like that https://developer.jboss.org/thread/238426 . But then we would need to include a WildFly specific configuration file in the TCK (and other container will probably have issues to test that too). WDYT about it?

@gunnarmorling force pushed an update to fix the typos and explained the meaning of "testing consistently". As for the explanation, tell me if you have ideas on how to improve the wording if you think it's necessary.

Force pushed another update to improve the wording of the error message: https://github.com/beanvalidation/beanvalidation-tck/pull/111/commits/6aa122d8c156372c497ffaf048fbdf01c47fb8f7#diff-24ce6001f0f28ab8c04cf5ad206a97a3R258


Upload TKEX

please let me know when the TKEX Icon/Logo is uploaded. Thanks and appreciate it.

Please lowercase file explanation

Please lowercase file explanation

Completed

Please lowercase file explanation

Please let me know if I need to do anything else. The “File Name/ Token contract” and .png is lowercase. Please notify me of any other changes that need to be completed to upload icon to the TRUST wallet. Thanks 🙏🏽🌊

Added lowercased image file name here: https://github.com/TrustWallet/tokens/commit/eaf62ed069c7116b39684e678b8ba4f25d925485
Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwallet.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp


Implement Application Completion Workflow with Certificate Generation and Email Notification

Purpose

The purpose of this PR is to fix #122
Goals

Enable admins to change the application status to 'COMPLETE'.
Notify mentees via email upon status change.
Generate a completion certificate for each mentee and attach it to the notification email.

Approach

To achieve the above goals, the following steps were taken:

Integrated the pdf-lib library (version 1.17.1) to modify and generate completion certificates in PDF format.
Updated utils.ts to support sending emails with PDF attachments.
Created a separate function dedicated to handling the PDF generation and email notification logic
Created a new folder to store certificates

Screenshots

Checklist

[x] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[x] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Related PRs

Test environment

OS: Linux Mint 21.3
Browser Firefox
Version 128.0
Nodejs 20.15.0

Learning

During the development of this feature, I explored several resources to understand how to effectively use the pdf-lib library for PDF manipulation and how to attach files to emails programmatically. This included reviewing documentation, blog posts, and tutorials related to pdf-lib and email handling in Node.js applications.

Can you resolve the conflicts? @mayura-andrew
I'm working on this


Possible to use 2 digit count for ec2 instances?

We use the name tag for puppet. Is it possible to ditch the - and make the count 2 digits?

Unfortunately, it is not currently possible unless you fork this module and make this change there.


meowify: translate English into Meow

lower/upper/mixed cases, capitalize; word-to-meow length
examples:
>>> meowify('Hello world!')
'Meow meow'
>>> meowify('Areallylongword areallylongword aREALLYlongword')
'Meoweoww meoweowew mEOWWOwoow'

This is wonderful.


args: Merge allowedHelp into allowed

This issue was originally filed by @seaneagan

Currently it looks something like:
  var allowed = {
    'x': 'x help',
    'y': 'y help',
    'z': 'z help',
  }
  parser.addOption('foo', allowed: allowed.keys.toList(), allowedHelp: allowed);
Nicer would be:
  parser.addOption('foo', allowed: {
    'x': 'x help',
    'y': 'y help',
    'z': 'z help',
  });
So allowed: would allow Iterable<String> or Map<String, String>

This issue has been moved to dart-lang/args#25.


Parameterised scenario names appear as blank in extent reports

I have a cucumber test in which the scenario outline title is parameterised. After execution the parameterised value is getting populated in IDE test execution status but in extent reports it is blank.
Feature File :
Scenario Outline: FreeCRM login scenario '<username>'
	Given User is on login page
	When Login page title is displayed
	Then user enters "<username>" and "<password>"
	Then user clicks on login button
	And user is navigated to HomePage
	And browser is closed

	Examples:
		| username | password |
		|<EMAIL_ADDRESS>| test|

IDE result :
In the below snapshot, The scenario title is displayed as FreeCRM login scenario<EMAIL_ADDRESS>as expected.

Extent Report Result : But in extent report it is FreeCRM login scenario ''. I am using 1.2.0 version. Please let me know how to resolve this issue. Am I missing some configuration?

Yep this seems to be an issue. Will have a look.

@grasshopper7 Sure. Thanks for quick response. Will look forward to hear from you regarding the resolution for this issue.

@grasshopper7 Any update on this as we are using this plugin and have multiple parameterised scenarios so we would need this fix as soon as possible.

Time permitting, will fix this today or tomorrow. Hope u realize that i volunteer for these open source contributions in my spare time, which in these 'covid days' is rare.
If it is urgent for your project u are welcome to fix this and send in a pull request.

Fixed in version 2.1.0. Should be available in maven when synced.


Fix Class CustomDeserializer, #3306

全局自定义解析函数补充为可通过比较获取
自定义解析器的时候，应该不能使用 ASM 字节码解析
在判断字段的时候将通过 1 的比较获取为是否 Custom

Codecov Report

Merging #3324 into master will decrease coverage by 0.49%.
The diff coverage is 96.93%.

@@             Coverage Diff              @@
##             master    #3324      +/-   ##
============================================
- Coverage     80.37%   79.88%   -0.50%
+ Complexity     7408     7381      -27
============================================
  Files           157      157
  Lines         27797    27824      +27
  Branches       6685     6693       +8
============================================
- Hits          22343    22228     -115
- Misses         3436     3596     +160
+ Partials       2018     2000      -18

Impacted Files
Coverage Δ
Complexity Δ

...json/parser/deserializer/JavaBeanDeserializer.java
76.99% <0.00%> (-1.64%)
306.00 <0.00> (-8.00)

...java/com/alibaba/fastjson/parser/ParserConfig.java
80.71% <97.70%> (+0.18%)
231.00 <6.00> (+6.00)

.../parser/deserializer/DefaultFieldDeserializer.java
90.00% <100.00%> (+0.29%)
26.00 <0.00> (+1.00)

...ava/com/alibaba/fastjson/util/IdentityHashMap.java
100.00% <100.00%> (ø)
21.00 <6.00> (+6.00)

...ava/com/alibaba/fastjson/parser/JSONLexerBase.java
66.71% <0.00%> (-3.29%)
785.00% <0.00%> (-28.00%)

...com/alibaba/fastjson/parser/JSONReaderScanner.java
73.58% <0.00%> (-1.26%)
45.00% <0.00%> (-1.00%)

...on/parser/deserializer/ASMDeserializerFactory.java
94.95% <0.00%> (-1.21%)
140.00% <0.00%> (-3.00%)

Continue to review full report at Codecov.

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


Add hpx::iostreams::ostream overload to format_to

Fixes #3013.
Proposed Changes

Adds an overload to hpx::util::format_to taking a hpx::iostreams::ostream to make sure that the correct operators (<<) are called.

Any background context you want to provide?
When a hpx::iostreams::ostream is used as a std::stream the correct virtual functions are not called, thus bypassing locks that are supposed to be taken in the hpx::iostreams::ostream implementation (base class functions are not virtual?). A proper fix would make sure the correct functions are called, but at the moment I have no fix for that.

For the records, the actual problem is that hpx::cout has it's own operator<<(T) that does the locking (see https://github.com/STEllAR-GROUP/hpx/blob/master/hpx/components/iostreams/ostream.hpp#L330-L335). This operator will not be invoked if the stream is used through its std::ostream base class. This should be fixed by a separate PR.


GPG signing key for Terraform Registry

Per #57, we need to create a GPG signing key in order to publish the provider to the Terraform Registry.
Once the key has been created, it should be stored in corporate LastPass or similar secure storage facility.

@mrunalk  reports this has been completed.


Add validations for model predictions

This validation prevents the downstream failure when trying to serialize the RAIInsights object or computing explanations if the predicted set of values aren't passed as valid classes to RAIInsights.
Description

Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #2008 (71228b1) into main (52ec38e) will increase coverage by 2.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2008      +/-   ##
==========================================
+ Coverage   91.58%   93.63%   +2.05%
==========================================
  Files         105       31      -74
  Lines        5143      581    -4562
==========================================
- Hits         4710      544    -4166
+ Misses        433       37     -396

Flag
Coverage Δ

unittests
93.63% <ø> (+2.05%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 74 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


How to disable registration and login?

How to disable registration and login?

It's not supported yet

Please add public download link would be very useful to share a file

Please add public download link would be very useful to share a file
yes.

#249

By default, authentification is disabled.
You can enable it in the config file


Set updated_at when executing counter_culture_fix_counts

class Foo < AR::Base
  has_many :bars
end

class Bar < AR::Base
  belongs_to :foo
  counter_culture :foo, touch: true
end

and
Bar.counter_culture_fix_counts

I think in this case also the updated_at field of the foos needs to be updated.
Does this make sense to you?

Yeah, that makes sense. Feel free to submit a PR. 😄

@magnusvk I'm willing to submit a PR. What I've done so far:

forked the project;
cloned locally the fork;
ran rspec spec

I get the following failure:
Failures:

  1) CounterCulture should fix a STI counter cache correctly
     Failure/Error: expect(company.twitter_reviews_count).to eq(1)

       expected: 1
            got: 0

       (compared using ==)
     # ./spec/counter_culture_spec.rb:1018:in `block (2 levels) in <top (required)>'

Finished in 2.31 seconds (files took 1.55 seconds to load)
35 examples, 1 failure

Failed examples:

rspec ./spec/counter_culture_spec.rb:1003 # CounterCulture should fix a STI counter cache correctly

After some investigation I found out that the failure is cause by the after_create callback in Review. When I comment out this callback, the test passes. Also, if I put the callback after:

# TwitterReview
counter_culture [:user, :manages_company]

that is, I move it from Review to TwitterReview (for the sake of executing it after counter_culter has finished updating the counters) the test also passes. I am currently investigating this, but it appears that after after_create :update_some_text is ran, @_counter_culture_active is not properly set to false in _wrap_in_counter_culture_active(&block), so that when it gets to decide whether to update the counter for twitter_reviews_count in Company, @_counter_culture_active is still true and thus does not execute the block.
Before I continue my investigation - can you tell me if the test fails on your side as well?
I'm running ruby 2.4.1 and rails 5.1.5.
https://github.com/magnusvk/counter_culture/blob/master/lib/counter_culture/extensions.rb#L78

O.K., this does not happen on 5.1.4., it only happens on 5.1.5. I think the problem is somehow related to https://github.com/rails/rails/pull/27780

Yeah, this used to work fine with Rails 5.1. I haven’t had a chance to look into this yet but will try to soon. Thanks for digging. For purposes of your PR feel free to just test against 5.1.4 for now where the test is green.

Fixed by #212


bugfix

The isFirst variable is always true. Therefore, I think these codes are unnecessary(but I do not know the detailed reasons why this process was necessary...)

isFirst is a flag to skip the first process called.

Why should it skip? I am troubled because the thumbnail image is not displayed with the following code...

    func configure(_ cell: HomeCollectionViewCell, atIndexpath indexPath: IndexPath) {
        guard let post = dataSource?[indexPath.item] else { return }
        cell.disposer = post.listen { post, error in
            if let ref = post?.thumbnail?.ref {
                cell.imageView.sd_setImage(with: ref)
            }
        }
    }

    func collectionView(_ collectionView: UICollectionView, didEndDisplaying cell: HomeCollectionViewCell, forItemAt indexPath: IndexPath) {
        cell.disposer?.dispose()
    }

Is the usage of the listen function incorrect?

post has already been loaded.
guard let post = dataSource?[indexPath.item] else { return }

Using listen will load the same value asynchronously.
Loading the same value unnecessarily will cause the screen to flicker.
You should use observeObject.

Thank you. The problem was solved thanks!


Change Polarion field which stores tmt id

There is a requirement to use different field to store tmt id in Polarion.
This patch changes which field is used for any new imported/exported test case to newly created 'tmtuuid' field.
Any existing cases will still be found with existing ids in current field so no need to export again just for this change, just during the next export the id will be exported to the new field.
Pull Request Checklist

[x] implement the feature
[x] write the documentation
[x] include a release note

@KwisatzHaderach I'm afraid there's at least one failed polarion test in /plans/remote/polarion, can you take a look at it?

Fixed the tests (hopefully) here https://github.com/teemtee/tests/pull/8 which of these chickens/eggs we merge first?

Rebased on the latest main and dropped the temporary hack to actually execute tests.

Unrelated failures, merging.


⚠ gorzdrav.org has degraded performance

In a9a32c2, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 2404 ms

Resolved: gorzdrav.org performance has improved in eb01bab.


Missing area in COCO's .json output?

https://github.com/laclouis5/globox/blob/005789310c25d6bd2bee2edf409b34680fdab38b/src/globox/annotationset.py#L754
Is there a field missing for "area"?
Annotations might have:
“annotations”: {
“id”: int,
“image_id: int”,
“category_id”: int
“segmentation”: RLE or [polygon],
“area”: float,
“bbox”: [x,y,width,height],
“iscrowd”: 0 or 1
}

Thank you for the explanation. I was using another tool to analyze the .json file saved with globox. It attempts to access the area field for a bounding box and throws an error as a result.

Thanks for the clarification. What is the other tool you used? I could add the area field if this simplifies this workflow.

I used datatsetinsights from Unity's Perception. This function threw the error:
https://github.com/Unity-Technologies/datasetinsights/blob/0705bead5e592bdfe59e10a749ed7bf5264a2b68/datasetinsights/stats/object_detection_stats.py#L26

I updated the package so that area and segmentation fields are populated for COCO export. Could you try updating Globox to its last version (v2.1.2) then try exporting the annotations to COCO again and finally import in Perception SDK?


http://host.docker.internal Does not work

Environment: cypress docker image -> Amazon AMI2 image
Using host.docker.internal does not work to access the localhost. I ended up having to use <IP_ADDRESS>.

What operating system and version of Docker are you using?
host.docker.internal is not supported in all environments.

The official cypress docker is running in linux

As I said, host.docker.internal only works on 'Docker for Windows' and 'Docker for Mac' when the version is 18.03 or beyond.
If you are using an older version than this, or running Linux on your host machine then it will not work.
Additionally, this is a feature of Docker not a feature of Cypress, so if there is still a problem you should be making a bug report to Docker instead of here.


Quotas per image type (readonly ISO / System / Datablock)

Author Name: kvaps kvaps (kvaps kvaps)
Original Redmine Issue: 5142, https://dev.opennebula.org/issues/5142
Original Date: 2017-05-05

I want to allow my users upload custom ISO's but I don't want to allow use it as persistent storage for their vm's.

Original Redmine Comment
Author Name: kvaps kvaps (kvaps kvaps)
Original Date: 2017-05-10T08:04:22Z

I think it may be done by separating ISO datastore from Images datastore
Or by make option for choose which images types may be stored in this datastore.

Original Redmine Comment
Author Name: kvaps kvaps (kvaps kvaps)
Original Date: 2017-05-10T09:01:16Z

For now I solved this by adding this hook:
https://gist.github.com/kvaps/fda530aaf2b1d8b7d71e1c4c0ec10280
And adding option into my datastore:
ISO_ONLY=yes