Resource not accessible by integration

Error: Unhandled error: GraphqlResponseError: Request failed due to following response errors:

Resource not accessible by integration

getting the above error while integrating the action. what's the graphql your requesting?

Hi @sszemeresky,
Thank you for reporting the issue. I'm also encountering this issue. Maybe, specifying permissions in your workflow would resolve this problem. Would you try this configuration?
https://github.com/yykamei/block-merge-based-on-time?tab=readme-ov-file#usage

getting two errors now
Error: Unhandled error: GraphqlResponseError: Request failed due to following response errors:

Resource not accessible by integration
Resource not accessible by integration

OK, thank you for reporting. I want to see the detailed log by enabling debug mode. Would you mind giving me the debug log?
By the way, I forgot to answer your first question.

getting the above error while integrating the action. what's the graphql your requesting?

You can see what queries this action sends to GitHub.
https://github.com/yykamei/block-merge-based-on-time/blob/b1ff83334834ea3a76d311be080af0183d9775a5/src/github.ts

unfortunately i'm unable to turn debug mode on

Thank you for your patience. I understand that enabling debug mode is not an option for you. Could you please share any additional logs or information you have that might help me diagnose the issue? It would also be helpful to know more about your environment and the specific steps you took when encountering the error. Plus, I want to know whether this change works or not.

I want to know whether this change works or not.

unfortunately permissions didn't help

Thank you for the update. OK, could you provide any additional details about your environment and the specific steps that lead to the error? Any extra information would help in diagnosing the issue.

i just made a pull request adding this workflow
`name: Block Merge Based on Time
on:
pull_request:
types:
- opened
- reopened
- synchronize
- labeled
- unlabeled
schedule:
- cron: '*/30 * * * *'
jobs:
block:
runs-on: ubuntu-latest
permissions:
pull-requests: read
statuses: write
steps:
- uses: yykamei/block-merge-based-on-time@main
id: block
with:
timezone: UTC+2
after: '16:00'
before: '07:00'
base-branches: '(default)'
prohibited-days-dates: 'Friday, Saturday, Sunday, H:Poland, BH:Poland'
- run: echo pr-blocked=${{ steps.block.outputs.pr-blocked }}
if: github.event_name == 'pull_request'`

@sszemeresky
Thank you for giving me your workflow file. Thank to that, I found out contents: read is also required to successfully run this workflow. Please try to add this permission in your file!

adding contents: read permission worked! thx! could add this to your readme ;)

Thank you! I added this via https://github.com/yykamei/block-merge-based-on-time/commit/7bff24c92bd4f38a513384ff7cfc242a3578442b, so I close this issue. I appreciate you opened this issue 😄


[Feature Request]: Batch transcription with multiple output type!

Describe the feature
Batch transcription is great!
Podcasts need html and one of the machine-readable formats like srt.
Could there be a way to do batch transcription and target multiple output formats?
Or perhaps even easier, generate all output formats and the user can easily delete the file types that are not needed?
There is a same issues here：https://github.com/thewh1teagle/vibe/discussions/273

Batch transcript is great. Just a couple of suggestion.
config the general batch options via config file (language, model, output format, destination directory...)
export the config from the UI to be used as base for the batch config

Added in https://github.com/thewh1teagle/vibe/releases/tag/v2.6.6


Starting level ideas

We need a start to the episode. I haven't really done a base map, except at the end of my massive contract revoked submission. I would like to take advantage of GFX and placed models to create something semi-unique.
As all the easy enough to kill with the shotgun enemies are base level enemies we should start there and then work on expanding the textures to support new realms or a theme for the rest of the maps.
I like the following ideas:

A research lab with hints to what is to come (can be added later)
I am a fan of creating cryo tubes with enemy models in them... I want to take that to the next level using sprites
Hints of a larger base out windows or being doors.
Develop a visual language using colored lights to direct the player. (maybe some of the fullbright light wads could help here)

A new version of progs_dump (1.1.3) should be ready by February if not sooner. Depends on a few things that are in progress. Not sure if you'd want to use it but there are some nice additions like killable bosses, more built-in texture options for breakables.
The 1.44MB limitation is pretty interesting. progs_dump has not been tested on legacy game engines. I'm wondering if there is going to be a recommended source port for this?

Updating to the latest Progs Dump. I will branch then merge once completed.

Updating to the latest Progs Dump. I will branch then merge once completed.

Moving everything into the "projects" board as cards

Moving everything into the "projects" board as cards


Possible memory leak with cgltf_data not cleaned up

Hello.
I was using the Visual Studio 2019 Diagnostic Tools' Memory Usage tool on Windows 10.
After a bit of digging and prodding I ended up with this test, where I call a function to import a gltf and return. This is from the main function.
There are two breakpoints, 1 before the importing function is called and 2 after the function is called. Snapshots of the heap allocated are taken and it shows the following. I am interpreting this as the cgltf_data that was allocated is not being freed correctly.
Main function

The which imports the gltf into a cgltf_data and frees the data.

The snapshot of the heap at the breakpoint at the return statement. Ideally there should be no values here.

This is the call stack snapshot.

Tried with imported 3 gltf files and the "count" parameter in the snapshot goes to 3.
Please let me know if you need more information, and if there is another way to verify this.
Best regards.
Nihal Kenkre.

#112 solves this.
Cheers!


[Dev Site] bump sdk version on developer site [Nov]

we got a hero ping that our sdk docs are out of date with one core. let's get those up to date.
Acceptance Criteria

[ ] follow these instructions
[ ] update SDK version release-3366
[ ] ensure no errors are present in the components

I'm getting a fair amount of errors in the console include this  error on  http://localhost:8000/components/nrql-queries/


Add CI that runs tests on every commit

It should run tests for every supported GHC version.

IMHO it would be enough just to run tests on HEAD


Vue randomly adding v-move onto transition group elements

Vue version
3.3.4
Link to minimal reproduction
https://github.com/VividLemon/bootstrap-vue-next/tree/carousel-hack
Steps to reproduce

Clone my repo
switch to branch carousel-hack
pnpm i
pnpm dev --filter bootstrap-vue-next
open up the console
view one of the picture elements
start clicking the controls

When you click back and forth on the controls, when v-move is applied to an element it "stutters", I'm not sure why this is. I'm never applying v-move to any element, so I'm not sure where it's coming from.
I apologize it's not a minimal reproduction. But the actual internals are not very important.
What is expected?
v-move should not be applied. I don't know why it's ever added, I don't see it ever mentioned on https://vuejs.org/guide/built-ins/transition.html#transition
What is actually happening?
v-move is applied at the end of transitions. It seemingly is the cause of some stuttering in transitions
System Info
System:
    OS: Linux 6.2 Pop!_OS 22.04 LTS
    CPU: (16) x64 AMD Ryzen 7 7800X3D 8-Core Processor
    Memory: 21.58 GB / 30.49 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 20.3.1 - /run/user/1000/fnm_multishells/9176_1690985460458/bin/node
    npm: 9.6.7 - /run/user/1000/fnm_multishells/9176_1690985460458/bin/npm
    pnpm: 8.6.5 - ~/.local/share/pnpm/pnpm

Any additional comments?
No response

It's not mentoned in the Transition docs because it's part of TranstitionGroup (which you are using), particularly to provide an class to attach move styles to. like other transition classes, the default name starts with v- , unless you give your transition a specfic name.
https://vuejs.org/guide/built-ins/transition-group.html#move-transitions
And as you can see in the docs examples, this doesn't cause stuttering. So we would need to assume this is not a bug with our implementation unless you can provide a specific minimal reproduction demonstrating how it is.


Could not start stub server with requestBody as xml format. Getting the following error: getProperties(...) must not be null with 81 version of specmatic.jar and getting schema.properties must not be null error with 56 version.

Could not start stub server with requestBody as xml format. Getting the following error: getProperties(...) must not be null with 81 version of specmatic.jar and getting schema.properties must not be null error with 56 version.

hello @Jaganudumula, thanks for reporting the issue. Can you please share a sample of the OpenAPI specification you are using? That will help us understand / reproduce the issue.

Sample api.txt
It did not accept .yaml file to upload, so I changed the extension to .txt and uploaded.

@harikrishnan83 any solution for this issue?

Hi @Jaganudumula, thanks for reporting this, we'll have a look.
However, the issue is occurring because the schema of the XML request payload does not have any properties defined. You should be able to get past this by defining them, as described in the OpenAPI documentation here.
Hope this helps.

Hello @joelrosario, thank you for the response.
I tried by adding the properties field in request body but still getting the same error.

@Jaganudumula , apologies for the delay in responding.
Would it be possible for you to provide the updated yaml file in which you have added properties.

Hi @Jaganudumula
I was able to get this loading as a stub, after adding properties. Please try the specification below.
openapi: "3.0.3"
info:
  description: This documentation contains REST API contract details
  version: "1.0.0"
  title: REST API

paths:
  /xyz/api/rest/v1/book:
    post:
      tags:
        - V1
      summary: Post an entry
      description: |
        - Post an entry
      operationId: postEntry
      requestBody:
        content:
          application/xml:
            schema:
              $ref: '#/components/schemas/PostEntryXmlRequest'
            examples:
              file:
                summary: A sample message
                value: >-
                  <?xml version="1.0" encoding="UTF-8"?>
                  <Document xmlns="urn:iso:std:iso:20022:tech:xsd:camt.054.001.02">
                  –<BkToCstmrDbtCdtNtfctn>
                  - <GrpHdr>
                  <MsgId>XML990920151125223537343</MsgId>
                  <CreDtTm>2015-11-25T21:35:37Z</CreDtTm>
                  - <MsgRcpt>
                  - <Id>
                  - <OrgId>
                  - <Othr>
                  <Id>7786849266</Id>
                  - <SchmeNm>
                  <Cd>CUST</Cd>
                  </SchmeNm>
                  </Othr>
                  </OrgId>
                  </Id>
                  </MsgRcpt>
                  <AddtlInf>/DEBT/</AddtlInf>
                  </GrpHdr>
                  - <Ntfctn>
                  <Id>2015-11-25-22.35.37-SEK2121</Id>
                  <CreDtTm>2015-11-25T21:35:37Z</CreDtTm
      responses:
        '202':
          description: >-
            Accepted
          headers:
            X-Request-ID:
              schema:
                type: string
              description: ID
        '400':
          description: Invalid input for posting entry.

    parameters:
      - $ref: '#/components/parameters/X-Token'
      - $ref: '#/components/parameters/X-Request-ID'

components:
  parameters:
    X-Token:
      name: X-Token
      in: header
      required: true
      schema:
        type: string
    X-Request-ID:
      name: X-Request-ID
      in: header
      required: true
      schema:
        type: string

  schemas:
    PostEntryXmlRequest:
      title: PostEntryXmlRequest
      type: object
      xml:
        name: Document
      properties:
        id:
          type: integer
        name:
          type: string

  responses:
    Unauthorized:
      description: Unauthorized action

    Forbidden:
      description: The user is not allowed to access this service.

We have recently added a fix so that the error message reports the name of the schema intended to define an XML node, if it has no properties. The fix will be out soon.
Btw Specmatic is now at version 1.2.17, with many improvements and feature since the last time you tried it.
Closing this issue for now. But please reopen it if you have any followup questions.


It should be easier to add new settings

We should make it very easy to create new settings, and the preferences screen should dynamically show form controls for these settings. This way we are more inclined to add new settings, knowing that there is little overhead

I agree with the basic premise and do not yet see how will that be done. Since I already created the Preferences chapter, please add what needs to be in that chapter.


Ignore to vectorize constants only generic op in LoadStoreVectorization

This fixes all internal tests for enabling the pass as default.

Interesting case. This seems to be constant foldable?

I added some patterns for this kind of pattern before, e.g., sqrt op. I was thinking to add another pattern for rsqrt, but I didn't find a library call for it. We could add it at some point with simple implementation 1.0 / sqrt(x). However I feel that there could be missing patterns, so still send out this PR.


ISPN-150002 TestClient resource shutdown can block indefinitely resul…

https://issues.redhat.com/browse/ISPN-15002

It looks like I added an extra 0 to the commit message. Probably easiest to remove via "squash and merge" instead of waiting for another CI run :slightly_smiling_face:


how to use the native hooking?

Sorry for not a bug, but I can't quite figure out the native hooking:
{"classname":"libc.so","methodname":"recvfrom"}
{"classname":"libc.so","methodname":"open"}

seems to load fine but neither get hit.  I can see them hit in frida-trace however. Happy to write up some docs on it so figured file as a bug for record rather than emailing.

Hey, sorry for the late reply; it should work with your json input; honestly I am using frida-trace other than house when it comes to native hooking as there are some edge case that house cannot handle at this point; I will def work on it when I get the chance


⚠ Bofrost.be has degraded performance

In 0e89609, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1095 ms

Resolved: Bofrost.be performance has improved in 2b0ff02.


building perl fails to create the Makefile

...
cflags.SH: cc       = gcc
cflags.SH: ccflags  = -march=x86-64 -mtune=generic -O2 -pipe -fwrapv -fno-strict-aliasing -fstack-protector-strong -march=x86-64 -mtune=generic -O2
 -pipe -fwrapv
cflags.SH: stdflags =
cflags.SH: optimize = -march=x86-64 -mtune=generic -O2 -pipe
cflags.SH: warn     =  -Wall -Werror=declaration-after-statement -Wextra -Wc++-compat -Wwrite-strings
Extracting cflags (with variable substitutions)
Extracting config.h (with variable substitutions)
Extracting makedepend (with variable substitutions)
Extracting Makefile (with variable substitutions)
./op.c: line 3: amigaos4: command not found
./op.c: line 4: syntax error near unexpected token `('
./op.c: line 4: ` *    Copyright (C) 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000,'
./perl.c: line 3: amigaos4: command not found
./perl.c: line 4: syntax error near unexpected token `('
./perl.c: line 4: ` *    Copyright (C) 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000, 2001'
Extracting myconfig (with variable substitutions)
Extracting pod/Makefile (with variable substitutions)
Extracting Policy.sh (with variable substitutions)
Extracting runtests (with variable substitutions)
Configure done.

If you compile perl5 on a different machine or from a different object
directory, copy the Policy.sh file from this object directory to the
new one before you run Configure -- this will help you with most of
the policy defaults.

make: *** No targets specified and no makefile found.  Stop.
==> ERROR: A failure occurred in build().
    Aborting...
                                                                                ```

That's because NTFS is case-insensitive. Perl has two makefiles - "$firstmakefile" (which is usually detected as "makefile") and Makefile. Makefile is created first. It is then executed with make to create the main makefile, i.e. "$firstmakefile". Before creating "$firstmakefile", Makefile will run rm -f $firstmakefile.
Since NTFS is case-insensitive, "makefile" and "Makefile" are the same file, thus Makefile deletes itself.
One possible workaround for this is to rename Makefile to something else (such as MyMakefile) , and add "-f MyMakefile" to all make invocations that are expected to execute that file, since otherwise make won't know what to run.


Add a --context-path for build.sh

Add a --context-path for build.sh, allowing to test Dockerfiles in different directories, and still get relative paths to work.

Add new option --context-path to build.sh
Keeps the default as the dirname of the Dockerfile, so
no changes expected in the current behaviour

cc @Mousius @areusch @mehrdadh for reviews

@leandron LGTM. Thanks for this change!

just curious what's the use case for this change?

just curious what's the use case for this change?

We have some tooling that will use Dockerfiles out of tree (in a temp directory), but it needs to keep the context pointing to the tree, in order to resolve relative paths.

It seems CI is happy now, can I get a review and a merge if you're happy? @areusch


reference/docker: synchronize fork with upstream distribution/distribution

reference/docker: align files with original code
This package was forked from the https://github.com/distribution/distribution
repository in commit 901bcb2231466229d27aee8d38a6e2fcdc95015e (https://github.com/containerd/containerd/pull/3728), but that commit
did a plain copy of the code (minus tests), and rewrote the code to be in a
single file. The same commit also removed some deprecated code for handling
"shortid" references (ParseAnyReferenceWithSet() function), in order to avoid
the "digestset" dependency from the distribution repo.
At the time, containerd used the distribution/distribution package from this
commit: https://github.com/distribution/distribution/commit/0d3efadf0154c2b8a4e7b6621fff9809655cc580
Since the code was forked, both containerd and distribution have received
improvements and fixes, so unfortunately, the code started to diverge.
I'm planning to reconcile the packages (potentially by using a shared module),
and this is the first commit to assist with that.
This patch restructures the reference/docker package to split the code into the
same files as are used in the upstream distribution/distribution repository.
This makes it easier to compare the implementations in both repositories (to
allow synchronizing changes).
No changes are applied yet, other than splitting the code (follow-up commits
will take care of syncing changes across).
reference/docker: add unit-tests from upstream distribution/distribution
This copies the unit-tests from the distribution repository from the time when
the package was forked. The tests are copied from this commit:
https://github.com/distribution/distribution/commit/0d3efadf0154c2b8a4e7b6621fff9809655cc580
Tests for ParseAnyReferenceWithSet() have been removed, as that function has not
been included in the containerd fork.
Forward port (partial) patches:
Bringing this fork back in sync with https://github.com/distribution/distribution

https://github.com/distribution/distribution/commit/a19e1847a3a815894bdc9ffdc064392c96fa38c6 (https://github.com/distribution/distribution/pull/2979)
https://github.com/distribution/distribution/commit/c26fa61ec9234e3b52181c6d6dc1204f6713329f (https://github.com/distribution/distribution/pull/3265)
https://github.com/distribution/distribution/commit/1d33874951b749df7e070b1c702ea418bbc57ed1 (https://github.com/distribution/distribution/pull/3225)
https://github.com/distribution/distribution/commit/32c77af6ac94ba4e34fc9aaa784c6ca69c114cf0 (https://github.com/distribution/distribution/pull/3201)
https://github.com/distribution/distribution/commit/1c89ce5fc1cec85e0726f598bccc12b873d04de3 (https://github.com/distribution/distribution/pull/3566)
https://github.com/distribution/distribution/commit/89622d99a13e25d9a17286eee562b9284e351fe8 (https://github.com/distribution/distribution/pull/3566)
https://github.com/distribution/distribution/commit/53a6f7d7aa2d0e423d85e3bf263521778c4a414d (https://github.com/distribution/distribution/pull/3489)

relates to https://github.com/kubernetes/kubernetes/issues/104491

Approving with the expectations this package will find a new home outside our main repo 😄

Yup! Working on that


README review from a newcomer to css-blocks

After reading the entire README, I still don't understand when I should be using block or the scope pseudo selector.

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L271

In this example, it was first mentioned that ":scope selector is automagically applied to the template's root-level element." It will be helpful to have the file name, output HTML file and CSS file to see exactly how :scope selector is applied.   The code snippet example in its current state doesn't provide any clarification.

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L128

I think that it will be helpful to have an example in the beginning that can demonstrate what block, scope, class, state, sub-state selectors are. Maybe a tweet?  The code snippets from line 128 - 195 are all abstract at the moment, e.g.
:scope {
  block-name: custom-block-name;
  /* 👆 optional! */
  /* ... more styles ... */
}

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L81

"To learn how to install css-blocks for in your application"  -- for should be removed

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L153

"Blocks may can contain other classes " -- may should be removed

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L293

"You may not use a Block class outside of it's " -- should be its
"hasn't yet been implemented yet, " - should be "has not yet been implemented"

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L324

"The resolution logic for @blocks to node_modules " -- should be @blocks

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L180

"Sub-states of a State are mutually exclusive and an element may only be in one sub-state of that state at any given time."
Not everyone may be familiar with this terminology, it may be helpful to give a quick explanation before the code snippet. e.g.
"For example, a button can only be one color (state) at any time. It can be red, blue, or yellow (sub-state)"

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L478

Is this suppose to beClass State?

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L479

Is this suppose to beClass Sub-State?

[ ] https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L616

"If you have dynamic classes or states in your template, it will ask you to provide explicit resolutions between Blocks that even only have a chance of being used together on the same element." - what are you trying to say here? Suggest to change the wording, and provide an example of this problem and how it will be resolved.

[ ]  https://github.com/linkedin/css-blocks/blame/45969004e55545d397e9c8762507c1f52f1ba6da/README.md#L709

"In rare occasions, a Block may choose to declare declare that a certain State is global. " -- delete a "declare"

Thanks for the review!


Run Error

   I have run the TSP demo, but the nodes of TSP more than about 10 could not get the result and return a “ValueError: no embedding found” error in Python. After many times tests for this demo using D-Wave, the results of TSP with 5 to 10 nodes were apparently wrong. I did not test in the Forest.
   So if you have any great solutions about this problem, please tell me how you solved it.
   Many thanks in advance.

Hi! It's not possible to solve problems of size 10 or more using D-Wave 2000Q, since this problem is too big to fit onto the QPU. Perhaps there exists an embedding, that you can find using minorminer, but I've never suceeded.


Consider adding grid components

It might be worth adding some higher-level grid layout abstractions to make page layout work like Bootstrap grid.
E.g.
const Row = props => (
  <Flex
    mx={-3}
    {...props}
  />
)

const Column = props => (
  <Box
    px={3}
    {...props}
  />
)

Closing this out since there hasn't been much activity on this front


Use File.OpenHandle() instead CreateFileW pinvoke. Part 2

PR Summary
Contributes to #18553.

Use File.OpenHandle() in RemoteSessionNamedPipe
Use File.OpenHandle() in RunspaceConnectionInfo

I cannot test the change.
PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Close since in any case we have to add CreateFileW pinvoke. Will do in follow PRs.


Update BCR tutorial

Update repGermline description. Add information about .align_j_gene

PR https://github.com/immunomind/immunarch/pull/260 is on top of this and contains fixes for this PR, so better to review only PR 260 and close this one


Fixed PasswordDigest Generation

The WSSE password digest is best calculated from a byte array of the concat of NONCE, CREATED TIMESTAMP and PASSWORD

please provide tests and fix the build

Please fix the build

Please fix the build

I am not sure what you want me to fix, none of the issues appear to be in any of the files I have touched

have you rebased your branch onto master?

have you rebased your branch onto master?

Yes, just did it again now and no changes. The Travis build is reporting an error in code I haven't touched on Node 6
1 failing

SOAP Server
should properly handle compression:
Uncaught AssertionError: false == true

expected - actual
-false
+true

have you rebased your branch onto master?

Yes, just did it again now and no changes. The Travis build is reporting an error in code I haven't touched on Node 6
1 failing

SOAP Server
should properly handle compression:
Uncaught AssertionError: false == true

expected - actual
-false
+true

I refactored the failing test for the compression check that wasnt mine :-)

thanks @martinholden-skillsoft !


Incorrect quoting in Content-Disposition headers?

I've having a hard time grokking the various RFCs, but I just came across a service that refused to parse Content-Disposition lines like this:
CONTENT-DISPOSITION: form-data; name=file; filename="upload.gpx"; filename*=utf-8''upload.gpx

But accepted this:
CONTENT-DISPOSITION: form-data; name="file"; filename="upload.gpx"; filename*=utf-8''upload.gpx

The only difference being the quoting of the name parameter. Is aiohttp doing the right thing here? Is there a way to force quoting to happen?

According the spec:
     content-disposition = "Content-Disposition" ":"
                           disposition-type *( ";" disposition-parm )
     disposition-type = "attachment" | disp-extension-token
     disposition-parm = filename-parm | disp-extension-parm
     filename-parm = "filename" "=" quoted-string
     disp-extension-token = token
     disp-extension-parm = token "=" ( token | quoted-string )

Where token is:
    token          = 1*<any CHAR except CTLs or separators>
    separators     = "(" | ")" | "<" | ">" | "@"
                   | "," | ";" | ":" | "\" | <">
                   | "/" | "[" | "]" | "?" | "="
                   | "{" | "}" | SP | HT

So name=file satisfies requirement to be disp-extension-parm since both name and file are valid tokens.

We can enforce quoting, however. It shouldn't cause any harm. But suddenly, you cannot enforce it without a patch.
With what service do you have such problem btw?

This is with Strava's upload API. I posted to their developer support forum as well:
https://groups.google.com/forum/#!topic/strava-api/onA_ANTsPY4

BTW SimpleCookie starting from Python 3.5 enforces quoting for cookie names

Fixed by #641


chore(NumberTheory/Fermat): rename fermat to Nat.fermatNumber

(And also namespace and rename the lemmas about them.)
As discussed in the reviewers chatroom.

I think Nat.fermat is enough, unless there are something else named "Fermat" for natural numbers. This makes the name Nat.coprime_fermatNumber_fermatNumber too long IMHO.

Agree, Nat.fermat seems fine unless there's some clashing with the FLT project or something like that.

There's no clashing with FLT naming but I look at Nat.fermat and think "whatever does that mean?" whereas I look at Nat.fermatNumber and think "I know exactly what that will be" which is I think what we're aiming for.

Re "it's too long" -- it will also be never used, so this is fine. There's no big theory of Fermat numbers, I think I would rather argue "this shouldn't even be there" than "this should have a shorter name". Note that 2^2^n+1 is shorter than Nat.fermat. My instinct is just to merge this but if people really want to bikeshed about the name then feel free to do it here.

I guess one difference is that Mersenne is not famous for anything else! So if I see mersenne I can guess what it is.

Thanks :tada:
bors merge


Example of configuration setup for a single bucket?

Hi there, we at Safespring were really intrigued when we saw this project released, it seems to solve a lot of the same use cases we also work with in our day-to-day operations. We have actually started to develop a very similar tool internally but you seem to have progressed further than we have, so far.
New to the project, what would be the easiest way to get a working replication setup for a single bucket s3->s3, in a standalone configuration?
Docker compose as the basis for the setup? Is there anything to go by that you have?
Looking forward to possibly collaborate!

Thanks, this should be good to get us started, one of us will surely be in touch if we get stuck 👍🏼  The part that we couldn't figure out quickly ourselves was the cli tool example.

one more thing that may be useful: if you are familiar with Go, you can run Chorus e2e tests against your s3 storages with debug, breakpoints, and stuff from your IDE.
Then you can create a new test, or edit the existing one. The test is just a Go script that puts some data to src s3 with s3 go client and communicates with chorus directly over gRPC (instead of cli) to run replication and verify the results.

This is a little bit low-level and more relevant for developers but if you are fluent with Go, it might be easier to build all from source to run on your machine and use gRPC instead of CLI .


iOS simulator needs additional x86_64 target

"New" iOS simulators are 64bit, while current ios-simulator only builds i386 target. So we probably need both.

Le me see if I can make a PR for that.

Ok, this worked for me:

https://github.com/bkaradzic/bgfx/pull/1293
https://github.com/bkaradzic/bx/pull/173
https://github.com/bkaradzic/bimg/pull/7

PS. Making changes to build configurations between 3 repos are annoying, is there a better way to do it than 3 forks-PR's?

Merged

PS. Making changes to build configurations between 3 repos are annoying, is there a better
way to do it than 3 forks-PR's?

Nope. But technically important change is toolchain.lua, the rest is just helpers (I for example don't even use those, but they are there to get libs up and running quickly).


Authorization prompt doesn't appear

Briefly summarize your issue:
When attempting to start SampleApp and find authorization prompt, SampleApp does not properly open.
What is the expected behavior?
When running:
./SampleApp/src/SampleApp ./Integration/AlexaClientSDKConfig.json
in the sdk-build folder, a popup like this should appear to authorize product:
##################################
NOT YET AUTHORIZED
##################################
################################################################################################
To authorize, browse to: 'https://amazon.com/us/code' and enter the code: {XXXX}
################################################################################################
What behavior are you observing?
Instead, this command returns the lines:
2018-06-27 10:32:35.512 [  1] I sdkVersion: 1.7.1
USAGE: ./SampleApp/src/SampleApp <path_to_AlexaClientSDKConfig.json> <path_to_inputs_folder> [log_level]
File SampleApp says that it is executable, but only opens for a second when prompted.
Tell us about your environment:

What version of the AVS Device SDK are you using?
  <1.7.1>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[x] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[x] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Nevermind! Fixed it with this prompt.
./SampleApp/src/SampleApp ./Integration/AlexaClientSDKConfig.json /home/pi/sdk-folder/third-party/alexa-rpi/models


Alert dialog is transparent when they are drawn over platform view on iOS

I don't know if it is a google maps specific problem but I believe that this is global
After upgrade to flutter > v1.9.1 if you put a google maps (or web view) on Cupertino page, if you want to display a Cupertino alert dialog, the alert is transparent (transparency more than 50%)
The problem exist to the current stable version too (now 1.12.13+hotfix.7)

Hi @gmeles
Can you edit this code in the way to reproduce the transparency?

Code sample
import 'package:flutter/cupertino.dart';
import 'package:webview_flutter/webview_flutter.dart';

void main() {
  runApp(FlutterView());
}

class FlutterView extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  Future<void> _handleClickMe() async {
    showCupertinoDialog(
      context: context,
      builder: (BuildContext context) => CupertinoAlertDialog(
        title: Text('Title'),
        content: Text('content'),
        actions: <Widget>[
          CupertinoDialogAction(
            child: Text('Don\'t Allow'),
            onPressed: () {
              Navigator.of(context).pop();
            },
          ),
          CupertinoDialogAction(
            child: Text('Allow'),
            onPressed: () {
              Navigator.of(context).pop();
            },
          ),
        ],
      ),
    );
  }

  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      child: Column(
        children: <Widget>[
          Container(
            width: MediaQuery.of(context).size.width,
            height: MediaQuery.of(context).size.height * 0.8,
            child: WebView(
              initialUrl: 'https://flutter.dev/',
            ),
          ),
          CupertinoButton(
            onPressed: _handleClickMe,
            child: new Icon(CupertinoIcons.add),
          ),
        ],
      ),
    );
  }
}

I see such result

Or do you mean that the alert is the same dark as the background overlay?

Your code produce this

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.12.13+hotfix.7, on Mac OS X 10.15.1 19B88, locale el-GR)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.41.1)
[✓] Connected device (2 available)
• No issues found!

I just check android platform
It works perfect
The problem is on iOS

How frustrating.
I ended up reverting my project to v1.9.1+hotfix.6
I prefer loosing dark mode than having such ugly alerts
We cannot Design beautiful apps with current stable version at least for iOS
Please try to fix
Thanks

This may be an issue in the engine if the same code works differently between iOS and Android for the same version of flutter.  cc @flar

What has been changed between 1.9 and 1.10?
This works perfect on 1.9 till hotfix 6.  It is broken ever since

Do you mean v1.9.1+hotfix.6 or v1.9.6?

v1.9.1+hotfix.6 according to the previous comments.

Correct

This issue showed up in da151c95ba2b56a1da72c252a547adcdde86d013 cc @LongCatIsLooong
This means it showed up in v1.10.2.

This code tweaks the look of the dialog, still figuring out what the right thing to do is.
diff --git a/packages/flutter/lib/src/cupertino/dialog.dart b/packages/flutter/lib/src/cupertino/dialog.dart
index 305bba5d5..8500e71a4 100644
--- a/packages/flutter/lib/src/cupertino/dialog.dart
+++ b/packages/flutter/lib/src/cupertino/dialog.dart
@@ -60,7 +60,7 @@ const double _kDividerThickness = 1.0;
 // dialog's background color
 // Extracted from https://developer.apple.com/design/resources/.
 const Color _kDialogColor = CupertinoDynamicColor.withBrightness(
-  color: Color(0xCCF2F2F2),
+  color: Color(0xFFF2F2F2),
   darkColor: Color(0xBF1E1E1E),
 );

This is what the standard UIKit alert looks like, seems like the opacity should be 0xFF for that color.

I changed the background to just a bunch of Text to get rid of the PlatformView and it still looked pretty white so I went into CupertinoPopupSurface and changed its BackdropFilter blur radius from the default of 20 to 1.0 and I could see the surface behind it through the translucent background of the dialog. It would appear that if the code was properly blurring the content of the WebView then the dialog would look as intended. The question is why we aren't successfully blurring the Platform content.

When we draw any content (the alert view or the backdrop filter in this case) on top of the platform view, we draw it on the overlay instead of the main canvas. And the overlay is by default transparent.
I haven't looked into this in too much detail, just want to throw that out, might be related :)

@cyanglaz Thanks, that explains a lot. The alert and its backdrop filter can't see the platform pixels in that case. Do the platform view pixels exist at all at that point in rendering the layer tree, or are they going to be produced asynchronously at another time?

@flar On iOS: the platform view is not produced by us, we rather insert iOS UIView into the UIView tree. Skia doesn't know any pixels in the platform view.
On Android: the platform view is a texture layer so skia actually knows all the pixels in the platform view.

@cyanglaz so then the actual compositing would be done by iOS itself well outside our jurisdiction?
The issue here is that the decision to use a BDF+translucent color (which works well on background content we have access to) is made in the Cupertino widgets in the framework code which are far removed from the implementation detail that learns about the lack of background pixels...

@cyanglaz is there an alternate way to handle the platform views on iOS that behaves more like android? We already detect that readback is required on a surface during PreRoll so we could potentially expand that to dynamically determine whether the current scheme would work fine or if we might want to implement an alternate solution...?

I understand that it should render correctly, but I don't understand why it worked previously?
If what we are saying about the blur compositing incorrectly on platform views, why did it look decent before this commit?  The alpha wasn't 255 for those dialog colors.

I checked the commit, so previously a white colored container was drawn under the alert view. So I guess that caused the blur not being transparent?

It sounds like getting the blur working on platform views on iOS would be a huge effort.  You'd have to do a screen capture of the underlying UIView for the blur calculation.  Maybe we should just put the alpha to 255 on iOS until that point.  The difference is super super subtle on iOS 13.

I thought we had identified above that the color used to be opaque and now is translucent. Is that not true?
I can't seem to build and run an app on 1.9.1hotfix6 now (after upgrading to Xcode 11?) so I am having trouble verifying the dissection seen above.

Nothing about the commit identified as the culprit above should have affected our ability to blur on top of the UIView, though, so I'm not sure that is what is going wrong here...?

I clobbered my project and recreated it and was able to run it on 1.6.1hotfix6. It works fine and then I went in and changed the CupertinoAlertDialog to use a very small blur (changed from the default of 20 to 1.0) and the dialog was still opaque, so it looks like the dialog used to be opaque for whatever reason.

I am suspecting this is related to the code I added to avoid the extra compositing step on iOS related to https://github.com/flutter/engine/pull/14104

I've narrowed this down to the removal of the decoration on the dialog container. Before https://github.com/flutter/flutter/commit/da151c95ba2b56a1da72c252a547adcdde86d013 all of the Cupertino dialogs and popups that used a blur backdrop would include a decoration that would render white with BlendMode.OVERLAY over the blurred backdrop to lighten it. One of the side effects of that operation is that it would make sure that the background was opaque. If all of the pixels were transparent then it would just turn them white, but if they contained (blurred) colors, it would move them towards white. The code in this PR removed those decorations with the OVERLAY operation and so now the transparent pixels would be unmodified and the platform view would show through unblurred behind the dialog.
The patch below undoes that part of the commit and restores the old look for the test case as a proof of concept. We'll need to get @LongCatIsLooong involved for a proper fix, though, as this de-patch doesn't take dark mode into account for one thing.
diff --git a/packages/flutter/lib/src/cupertino/dialog.dart b/packages/flutter/lib/src/cupertino/dialog.dart
index 009fea3402..ce012b5ac0 100644
--- a/packages/flutter/lib/src/cupertino/dialog.dart
+++ b/packages/flutter/lib/src/cupertino/dialog.dart
@@ -49,6 +49,16 @@ const TextStyle _kCupertinoDialogActionStyle = TextStyle(
 const double _kCupertinoDialogWidth = 270.0;
 const double _kAccessibilityCupertinoDialogWidth = 310.0;

+// _kCupertinoDialogBlurOverlayDecoration is applied to the blurred backdrop to
+// lighten the blurred image. Brightening is done to counteract the dark modal
+// barrier that appears behind the dialog. The overlay blend mode does the
+// brightening. The white color doesn't paint any white, it's just the basis
+// for the overlay blend mode.
+const BoxDecoration _kCupertinoDialogBlurOverlayDecoration = BoxDecoration(
+  color: CupertinoColors.white,
+  backgroundBlendMode: BlendMode.overlay,
+);
+
 const double _kBlurAmount = 20.0;
 const double _kEdgePadding = 20.0;
 const double _kMinButtonHeight = 45.0;
@@ -332,8 +342,11 @@ class CupertinoPopupSurface extends StatelessWidget {
       child: BackdropFilter(
         filter: ImageFilter.blur(sigmaX: _kBlurAmount, sigmaY: _kBlurAmount),
         child: Container(
-          color: isSurfacePainted ? CupertinoDynamicColor.resolve(_kDialogColor, context) : null,
-          child: child,
+          decoration: _kCupertinoDialogBlurOverlayDecoration,
+          child: Container(
+            color: isSurfacePainted ? CupertinoDynamicColor.resolve(_kDialogColor, context) : null,
+            child: child,
+          ),
         ),
       ),
     );

And to close the loop on the problem diagnosis, applying the following one-line patch to 1.9.1+hotfix.6 causes that release to demonstrate the bug, so it appears that the loss of the container decoration with the OVERLAY operation is the cause.
diff --git a/packages/flutter/lib/src/cupertino/dialog.dart b/packages/flutter/lib/src/cupertino/dialog.dart
index c35f9391a4..25c581e3b6 100644
--- a/packages/flutter/lib/src/cupertino/dialog.dart
+++ b/packages/flutter/lib/src/cupertino/dialog.dart
@@ -338,7 +338,6 @@ class CupertinoPopupSurface extends StatelessWidget {
       child: BackdropFilter(
         filter: ImageFilter.blur(sigmaX: _kBlurAmount, sigmaY: _kBlurAmount),
         child: Container(
-          decoration: _kCupertinoDialogBlurOverlayDecoration,
           child: Container(
             color: isSurfacePainted ? _kDialogColor : null,
             child: child,

Any update on this issue?

Over webview

Over flutterlogo

code sample
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';
import 'package:webview_flutter/webview_flutter.dart';

void main() {
  runApp(FlutterView());
}

class FlutterView extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  Future<void> _handleClickMe() async {
    showCupertinoDialog(
      context: context,
      builder: (BuildContext context) => CupertinoAlertDialog(
        title: Text('Title'),
        content: Text('content'),
        actions: <Widget>[
          CupertinoDialogAction(
            child: Text('Don\'t Allow'),
            onPressed: () {
              Navigator.of(context).pop();
            },
          ),
          CupertinoDialogAction(
            child: Text('Allow'),
            onPressed: () {
              Navigator.of(context).pop();
            },
          ),
        ],
      ),
    );
  }

  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      child: Column(
        children: <Widget>[
          Container(
            width: MediaQuery.of(context).size.width,
            height: MediaQuery.of(context).size.height * 0.8,
            child: WebView(
              initialUrl: 'https://flutter.dev/',
            ),
          ),
          // FlutterLogo(
          //   size: 300,
          // ),
          // FlutterLogo(
          //   size: 300,
          // ),

          CupertinoButton(
            onPressed: _handleClickMe,
            child: new Icon(CupertinoIcons.add),
          ),
        ],
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel master, 1.22.0-10.0.pre.317, on Linux, locale en_US.UTF-8)
    • Flutter version 1.22.0-10.0.pre.317 at /home/taha/Code/flutter_master
    • Framework revision 10febea877 (6 hours ago), 2020-09-23 07:22:10 +0200
    • Engine revision 57ee3d4b32
    • Dart version 2.10.0 (build 2.10.0-156.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /home/taha/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /home/taha/Code/sdk
    • Java binary at: /home/taha/Code/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 4.0)
    • Android Studio at /home/taha/Code/android-studio
    • Flutter plugin version 49.0.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.49.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.14.1

[✓] Connected device (3 available)
    • Linux (desktop)  • linux      • linux-x64      • Linux
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome
      85.0.4183.102

• No issues found!

I am experiencing the same problem when displaying a Cupertino alert dialog over an Apple map (the apple_maps_flutter package).

Same here when displaying a Dialog over a native admob ads. Flutter 2.5.0

This issue still persists with Flutter 2.5.0. Is there any workaround or when it is expected this to be fixed?

Can't you just make your own CupertinoAlertDialog that has its own background color like I mentioned in https://github.com/flutter/flutter/issues/50183#issuecomment-586002873?

I don't have an environment set up to test this, but it looks like the CupertinoPopupSurface produces:
ClipRRect(
  child: BackdropFilter(
    child: Container(
      color: isSurfacePainted ? Cupertino_kDialogColor : null,
      dialog child,
    )
  )
)

Perhaps an alternate way of doing this but ensuring a background for the BDF to blur would be:
ClipRRect(
  Container(
    // This decoration has no effect if the background is opaque, but it steps in
    // to make sure the background is more completely obscured if the popup
    // is on a transparent layer and the BackdropFilter cannot blur its background.
    // dstAtop basically sets a minimum opacity "floor" for all of the pixels
    decoration: BoxDecoration(
      color: Cupertino_kDialogColor.withOpacity(1.0),
      backgroundBlendMode: BlendMode.dstAtop,
    ),
    child: BackdropFilter(...see above...),
  )
)

It would be more expensive but probably not noticeable given the BackdropFilter operation. The color used doesn't necessarily need to be fully opaque, but should be a lot more opaque than the default dialog color which assumes that it is rendering over a blurred background.

I also proposed a mechanism for accomplishing the same effect using just the BackdropFilter widget to avoid complicating the widget hierarchy, but that proposal would require some new functionality in the engine.
See https://github.com/flutter/flutter/issues/90213

Try to use https://pub.dev/packages/alert package. It worked for me.

Same here.

showCupertinoDialog + CupertinoAlertDialog is transparent above the Yandex Maps view.
Flutter 2.8.1

Yea, same here.

This behaviour also occurs on Web over HtmlElementViews on CanvasKit renderer, although may not be related the same exact problem. This issue becomes especially irritating when blurring a video from video_player plugin, for example.

Example code
// ignore_for_file: avoid_web_libraries_in_flutter

import 'dart:html' as html;
import 'dart:js' as js;
import 'dart:ui' as ui;

import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Stack(
          children: [
            const WebImage('https://picsum.photos/800'),
            BackdropFilter(
              filter: ui.ImageFilter.blur(sigmaX: 15, sigmaY: 15),
              child: Center(
                child: Container(
                  padding: const EdgeInsets.all(100),
                  decoration: BoxDecoration(
                    borderRadius: BorderRadius.circular(20),
                    color: Colors.white.withOpacity(0.5),
                  ),
                  child: js.context['flutterCanvasKit'] != null
                      ? const Text('Wow, I\'m NOT blurred :(')
                      : const Text('Wow, I\'m blurred!'),
                ),
              ),
            ),
          ],
        ),
      ),
    );
  }
}

class WebImage extends StatefulWidget {
  const WebImage(
    this.src, {
    Key? key,
  }) : super(key: key);

  final String src;

  @override
  State<WebImage> createState() => _WebImageState();
}

class _WebImageState extends State<WebImage> {
  html.ImageElement? _element;

  @override
  void initState() {
    // ignore: undefined_prefixed_name
    ui.platformViewRegistry.registerViewFactory(
      '__webImageViewType__${widget.src}__',
      (int viewId) {
        _element = html.ImageElement(src: widget.src)
          ..style.width = '100%'
          ..style.height = '100%'
          ..style.objectFit = 'cover';
        return _element!;
      },
    );

    super.initState();
  }

  @override
  void dispose() {
    _element?.removeAttribute('src');
    _element?.remove();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return HtmlElementView(viewType: '__webImageViewType__${widget.src}__');
  }
}

(left is --web-renderer canvaskit, right is --web-renderer html)

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.1, on macOS 12.1 21C52 darwin-arm, locale ru-RU)
    • Flutter version 3.0.1 at /opt/homebrew/Caskroom/flutter/2.5.3/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (3 недели назад), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/mayday/Library/Android/sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

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

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

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

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

[✓] Connected device (3 available)
    • M2101K7BNY (mobile) • D64HPJUSY9EIZLVO • android-arm64  • Android 11 (API 30)
    • macOS (desktop)     • macos            • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)        • chrome           • web-javascript • Google Chrome 102.0.5005.61

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

• No issues found!

+1

The issue is related to https://github.com/flutter/flutter/issues/43902, but instead of filtering the whole background, a region (the same size of the alert view) needs to be blurred. Here is a PR that we are currently working on to make the whole platform view being able to be blurred. https://github.com/flutter/engine/pull/34596
I don't think the current solution in the above PR would work for a selected region of the PlatformView. We might need to find a different approach for that.

Still an issue, would be nice to get a transient method of allowing a user to specify a background colour for the CupertinoActionSheet to at least remove the opacity until fixed.

@euphio I have just tried latest flutter (from master branch) and yes, unfortunately in iOS whole PlatformView is blurred instead of only the selection portion (like on Android)
Android:

iOS

The issue is related to #43902, but instead of filtering the whole background, a region (the same size of the alert view) needs to be blurred. Here is a PR that we are currently working on to make the whole platform view being able to be blurred. flutter/engine#34596
The current solution in above PR adds a gaussian blur filter to the layer of the PlatformView, which by definition blurs the whole layer (the whole PlatformView). I don't think the current solution in the above PR would work for a selected region of the PlatformView as is.

Which is worse:

not applying the filter at all
applying the filter to the entire page

I have a prototype ready for making a rect inside the PlatformView to be blurred. https://github.com/flutter/engine/pull/36015
I'm currently working on something else. If anyone is interested in picking it up, please feel free to do so. Otherwise, I will continue to work on it when I circle back.

The issue is related to #43902, but instead of filtering the whole background, a region (the same size of the alert view) needs to be blurred. Here is a PR that we are currently working on to make the whole platform view being able to be blurred. flutter/engine#34596
The current solution in above PR adds a gaussian blur filter to the layer of the PlatformView, which by definition blurs the whole layer (the whole PlatformView). I don't think the current solution in the above PR would work for a selected region of the PlatformView as is.

Which is worse:

not applying the filter at all
applying the filter to the entire page

IMO blurring the entire view is going to provide unexpected behaviour from an end developer perspective. I'd say not blurring is the safer of the two options until a solution such as @cyanglaz is implemented.
A simple example is pushing a map view up into the safe area on IOS. If done within a navigation view, the blur effect added by the navigation bar will cause the entire map view (ie screen) to blur out, which would seem unexpected.
Example:

https://github.com/flutter/engine/pull/36015 is a working prototype that fixes the issue, I still need to add tests for it.
I would appreciate if someone can test it locally and give feedbacks.

any upodates?

The PR is reverted due to some breakage and I'm working on re-landing the PR.

@cyanglaz
I just have to say thank you for the effort 🍻. I have tried the latest master branch of flutter which has the latest flutter engine

Flutter 3.5.0-10.0.pre.158 • channel master • https://github.com/flutter/flutter.git
Framework • revision e41f588cc5 (74 minutes ago) • 2022-10-29 06:24:15 -0400
Engine • revision 88e9c6574e
Tools • Dart 2.19.0 (build 2.19.0-351.0.dev) • DevTools 2.18.0

and I have found a small problem. In out app we have a Sheet component overlaying a native Map view which is translated with user gesture. When blur effect is applied the translation becomes really glitchy like it's dropping gesture events but when the ClipRect is removed from the Sheet everything works fine (please see the attached video)
https://user-images.githubusercontent.com/16222284/198829506-e2a52ef4-99d2-4c7a-9e22-ccf54d839999.mov

@mario-jerkovic Could you open a new issue with details, including how this issue can be reproduced?

@mario-jerkovic Could you open a new issue with details, including how this issue can be reproduced?

@cyanglaz I've made a new issue and also created and example app which reproduces the issue. Hope it helps 🤞


Failure to read FST files on *BSD systems

The *BSD stdio library seems to buffer data from the FST file leading to the seek position of the underlying file description being ahead of the stdio maintained seek position (that's what buffering is for after all).
fstapi.cc uses gzdopen(dup(fileno())) to open and so zlib is given the underlying seek position instead of the seek position that the stdio using application is expecting with the result that gzread() starts reading from EOF instead of byte 17.
This PR moves the underlying seek position so that gzread() will read from the location that the application expects. This doesn't appear to affect stdio (which is still sharing that underlying seek position) because in one of the cases, the original FILE is promptly closed and in the other case the FST library seeks the original file anyway.
The grom.ys unittest now succeeds on NetBSD when it didn't before.

Please note that this should be sent upstream as well. FST library is actually part of gtkwave
Note same file is on 3 different places:
https://github.com/gtkwave/gtkwave/tree/master/gtkwave3/src/helpers/fst
https://github.com/gtkwave/gtkwave/tree/master/gtkwave3-gtk3/src/helpers/fst
https://github.com/gtkwave/gtkwave/tree/master/gtkwave4/src/helpers/fst

Please note that this should be sent upstream as well. FST library is actually part of gtkwave
Note same file is on 3 different places: https://github.com/gtkwave/gtkwave/tree/master/gtkwave3/src/helpers/fst https://github.com/gtkwave/gtkwave/tree/master/gtkwave3-gtk3/src/helpers/fst https://github.com/gtkwave/gtkwave/tree/master/gtkwave4/src/helpers/fst

Thanks. I'll create an issue and PR over in https://github.com/gtkwave/gtkwave as well.


"Dara", "Pietroasa Mică" and "Șarânga" are listed in the drop-down menu

Description:
"Dara", "Pietroasa Mică" and "Șarânga" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Dara" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Dara, BUZĂU (Pietroasele)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Pietroasa Mică" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Pietroasa Mică, BUZĂU (Pietroasele)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Șarânga" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Șarânga, BUZĂU (Pietroasele)".


NDigits2 Trigger does not handle digit removal properly

The Ndigits2 trigger removes digits from the digits collection in order to avoid duplication when then applying the second level NDigits trigger scan.
However, digit times in the digits collection are stored in a map of digit number vs time.
When digits are scanned over in WCSimWCTriggerBase::AlgNDigits and WCSimWCTrigger::FillDigitsCollection, digit times are retrieved with a key that is the iteration number of a  simple for loop. This assumes keys follow a monotonic increase from 0 to ndigits. However, when WCSimWCDigi::RemoveDigitizedGate is called it simply erases the digit from the map, breaking this assumption.
At present the bug goes unnoticed due to the unsafe c-style map addressing in WCSimWCDigi::GetTime (and many others). Replacing this with a safe map::at style addressing and placing try{ } catch (... ) statements around the digit time retrieval demonstrates the problem. (apply the attached patches to include/WCSimWCDigi.hh and src/WCSimWCTrigger.cc
trigpatch.txt
digipatch.txt

Good spot! Thanks for this.
Could you put a pull request in for a fix for this? I guess it means changing the loops over  in WCSimWCTriggerBase::AlgNDigits() and WCSimWCTriggerBase::FillDigitsCollection():
for ( G4int ip = 0 ; ip < (*WCDCPMT)[i]->GetTotalPe() ; ip++)
to loops over the e.g. times map
Swapping from [] to map::at() in the times & other maps/vectors in WCSimWCDigi would also be a good plan

Sure, i'm working on a fix and should have something to submit soon.

Pull requested created: https://github.com/WCSim/WCSim/pull/247
For some reason this link gives me a 404 error, but i can't re-submit the PR because github tells me one is already open. Hopefully it will become visible soon.

Closed via #247


Update mobile-platform-home-page.md

Adjusting: Intune support and conditional access. Further investigation has determined that manual steps to add app to Intune are not supported.

@jasongre


Cant parse only season, multiple season and subs excess with only season

Hello,
Is it possible to add season parsing, if episode is not existing?
The.X.Files.S01.DKsubs.1080p.BluRay.HEVC.x265 - Returns:
{'quality': 'BluRay', 'codec': 'x265', 'resolution': '1080p', 'excess': 'HEVC', 'title': 'The X Files S01 DKsubs'}
I'm in the need of this result
{'quality': 'BluRay', 'season': '1', 'codec': 'x265', 'resolution': '1080p', 'excess': ['HEVC', 'DKsubs'], 'title': 'The X Files'}
Is it also possible to add multiple season parsing?
The.X.Files.S01-S03.DKsubs.1080p.BluRay.HEVC.x265 - Returns:
{'quality': 'BluRay', 'codec': 'x265', 'resolution': '1080p', 'excess': 'HEVC', 'title': 'The X Files S01-S03 DKsubs'}
I'm the need of this result
{'quality': 'BluRay', 'season':['1','2','3'] 'codec': 'x265', 'resolution': '1080p', 'excess': ['HEVC', 'DKsubs'], 'title': 'The X Files'}
Also - if episode is not existing "DKsubs" becomas a part of the title and not in excess.
Thanks

Oh,it seems same problem as I put forward at issue15.
The regax pattern for season in this package is LIKE ('season', '(s?([0-9]{1,2}))[ex]').It always ask e (or x, as the begining of episode)
In my mind, It is easy to parse only season ( with change season pattern to (s\d{1,2})).
But parse S01-S03 to 'season':['1','2','3'] may difficulty.

I actually work on this and will add pull request when I can for multiple seasons and solve the examples mentioned in this ticket


fails on repos that master it's called main

We could get the branch from settings.ini anyway.

This should fix the issue:
def release(self):
        "Tag and create a release in GitHub for the current version"
        ver = self.cfg.version
        notes = self.latest_notes()
        self.gh.create_release(ver, self.cfg.branch, body=notes)
        return ver

please @jph00 fix this =)

@tcapelle would you be interested in contributing a PR for this?

Oh, I forget about this. I can propose a fix.


FancyZones and X11 in windows 11

Description of the new feature / enhancement
wsl2 seems to include an X server in windows 11. However, unlike VcXsrv, fancy zones will not snap X11 windows running on the built in windows x11 server.
Scenario when this would be used?
Whenever an WSL2 launches a graphical application using X11.
Supporting information
No response

The WSL team is working on supporting.  Soon as the needed support lands for Windows proper, PowerToys will be sure to make sure it also work appropriately if it doesn’t “just work”

Fyi, powertoys and wsl are on the same parent team so we do actively chat.

Good to hear. Looking forward to the fix.


Updated Styles

Updated styles to add more space for better tapping.
Grouped completed items separate from items to be done for better focus.

Done!


[Bug]:  Old Profile Picture Not Deleted on New Upload and ERR_HTTP_HEADERS_SENT Error

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
When a new profile picture is uploaded, the application encounters two issues:
Old Profile Picture Not Deleted: The previous profile picture file is not removed from the server storage after a new one is uploaded.
ERR_HTTP_HEADERS_SENT Error: The server throws an ERR_HTTP_HEADERS_SENT error, indicating that headers are attempted to be sent after the response is already sent. This might happen due to multiple res.send() or res.redirect() calls in the code.
Expected behavior
Old Profile Picture should be deleted when a new profile picture is uploaded.
No ERR_HTTP_HEADERS_SENT Error should occur; only one response should be sent to the client.
Add ScreenShots
No response
On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Dear @Soujanya2004 ,
Backend implementation is needed with some extra packages with proper middleware to handle it, So please add Level 3.


CTPK export

I have no idea what I'm doing and I would appreciate some help.
Technically this is working because I changed a texture for our Metroid: Samus Returns (MSR) randomizer with this code and it also recreates the original file when loading and saving it.
Also I don't know if MSR has any file with multiple textures within one bctex file, so it is hard to write the code to support that.
Eventually, it would be nice to directly load an export a bctex file which is only a wrapper around the ctpk file but don't know what's the proper way here because OnSave always directly writes to a file. Otherwise I would suggested to let the OnSave of ctpk write to a byte array such that the OnSave of bctex can wrap around it.
Let me write some annotations via GH now.

Looks like the code is now part of a submodule / a different repository.


Cosmos full-text query diagnostics doesn't work with cosmos javascript client

Package Name: @azure/cosmos
Package Version: 3.17.3
Operating system:
[X] nodejs

version: 16.19.1

[ ] browser

name/version:

[X] typescript

version: 4.6.4

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
When Cosmos has feature "Diagnostics full-text query" enabled and diagnostigs settings are turned on for DataPlaneRequests and QueryRuntimeStatistics events it is expected that all queries executed against this database will be logged, however it's not happening when queries are executed using javascript client. But when the same queries are executed using other clients (.NET and Java) for the same database, they are fully logged. Just to be clear - only CDBQueryRuntimeStatistics table is not getting populated, CDBDataPlaneRequests works just fine.
To Reproduce
Steps to reproduce the behavior:

Enable "Diagnostics full-text query" feature in Cosmos
Enable diagnostics settings for DataPlaneRequests and QueryRuntimeStatistics events and route all logs to Analytics Workspace
Write js application, which queries data in Cosmos
Examine table CDBQueryRuntimeStatistics in Log Analytics Workspace

Expected behavior
All queries texts are logged in table CDBQueryRuntimeStatistics
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
This was working perfectly until 06.04.2023. Something happened on this date from the Cosmos side and it just stopped to work. There were no changes in our code at that time.

@jshaptic Can you provide more details on this issue? looks like this is related to the Diagnostic settings on the server. If you are asking about Diagnostic string support on the JS SDK it will be available with the next release. (ETA June end 2023)

@sajeetharan That's the thing there were no changes to the Diagnostics settings on the server, and no changes on the client side, just at some point CDBQueryRuntimeStatistics table stopped getting populated.
And also, other Cosmos clients (Java and .NET) are working correctly with the same database. Query texts are logged in the CDBQueryRuntimeStatistics table. For some reason only JS SDK is getting affected.
What details exactly are you looking for? I would more than happy to provide it.

Can you raise a support ticket to investigate this?


[SPARK-21654][SQL] Complement SQL predicates expression description

What changes were proposed in this pull request?
SQL predicates don't have complete expression description. This patch goes to complement the description by adding arguments, examples.
How was this patch tested?
Existing tests.

cc @HyukjinKwon @gatorsmile Please help to take a look when you have time. Thanks.

Test build #80332 has started for PR 18869 at commit abfa867.

Test build #80332 has finished for PR 18869 at commit abfa867.

This patch fails Scala style 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/80332/
Test FAILed.

Test build #80334 has started for PR 18869 at commit 9895843.

Test build #80334 has finished for PR 18869 at commit 9895843.

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/80334/
Test PASSed.

Could you check all of binary predicates you are trying to modify? Thanks!

@gatorsmile I've checked the binary predicates in this change. Related test cases are added.

Test build #80435 has started for PR 18869 at commit 1369fd5.

Test build #80439 has started for PR 18869 at commit bca2b0b.

Test build #80440 has started for PR 18869 at commit b64c9e6.

@viirya Could you hold this PR at first? There is a related PR https://github.com/apache/spark/pull/18818 that is changing the underlying semantics. We also need to update the description after this. Also, welcome to review that PR too.

Test build #80435 has finished for PR 18869 at commit 1369fd5.

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

@gatorsmile Sure. Let's wait after #18818. I'll look at it too.

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/80435/
Test PASSed.

Test build #80440 has finished for PR 18869 at commit b64c9e6.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test build #80439 has finished for PR 18869 at commit bca2b0b.

This patch fails due to an unknown error code, -9.
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/80440/
Test FAILed.

Merged build finished. Test FAILed.

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/80439/
Test FAILed.

@gatorsmile @HyukjinKwon #18818 is merged now. Can this PR go ahead?
As what #18818 did is to allow structs, arrays to be input expression for predicates, currently looks like we don't have explicit description to say those types are not supported.

retest this please.

Test build #81306 has started for PR 18869 at commit b64c9e6.

map is not supported, right?

@gatorsmile Right. Isn't too verbose if we describe map is not supported in each description?

Test build #81306 has finished for PR 18869 at commit b64c9e6.

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/81306/
Test PASSed.

Previously Equal and EqualNullSafe don't state that the type must can be used in equality comparison. Now added the note.

Test build #81311 has started for PR 18869 at commit 099c671.

Test build #81311 has finished for PR 18869 at commit 099c671.

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/81311/
Test PASSed.

@gatorsmile, I understand the previously discussed point, the documentation should be correct, tested well and in details. However, if I observed this correctly, this argument descriptions have not been well filled up for the last year (since I proposed Arguments description) - if I checked correctly, we have three 3 instances fixed, replace, rlike and like, out of 266 instances.
I mean, practically, it sounds a bit demanding to me. (In this way, for the perfectness, I think we should also require to add test cases with all the possible types and check if there is any duplicated test too). I wonder if we really should add the examples about all the cases we assume  users might be confused of, and describe it that precisely for the current state, given the information did not exist before.
Of course, it is important and it should be correct but I have been thinking of incrementally improving this, rather than requesting the perfectly correct documentation for PRs, practically to roll it better.
What do you think about minimising the test cases and rather trying to describe it a bit more in an abstract, short and general manner for now? We could ask additional test cases for additional information in the followup PRs.
For this PR, it looks good enough to me (to be honest, the examples and test cases look too much) but I believe I need your opinion and I think we should have a compromised position.

If @viirya were a new contributor, I am fine to merge it.
In contrast, @viirya is experienced and knew the code base pretty well. Thus, I hope we can make our docs and test coverage much better.
Below is the doc I am comparing. You can easily see the gap we have. The test cases we have are so few. We will see more bugs when adding new test cases.  https://www.ibm.com/support/knowledgecenter/en/SSEPGG_11.1.0/com.ibm.db2.luw.sql.ref.doc/doc/r0000746.html

Test build #81347 has started for PR 18869 at commit a8bb457.

Test build #81348 has started for PR 18869 at commit 54c71e1.

Test build #81349 has started for PR 18869 at commit 444c64d.

Test build #81347 has finished for PR 18869 at commit a8bb457.

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/81347/
Test PASSed.

Test build #81348 has finished for PR 18869 at commit 54c71e1.

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/81348/
Test PASSed.

Test build #81349 has finished for PR 18869 at commit 444c64d.

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/81349/
Test PASSed.

Could you also add negative test cases for map?

@gatorsmile As I said in https://github.com/apache/spark/pull/18869#discussion_r136706025, we already have tests in ExpressionTypeCheckingSuite to check the predicates don't support map type. Do we need to add more negative test cases for map?

Test build #81356 has started for PR 18869 at commit ec9199a.

Test build #81356 has finished for PR 18869 at commit ec9199a.

This patch fails SparkR unit 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/81356/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Test build #81359 has started for PR 18869 at commit ec9199a.

Test build #81359 has finished for PR 18869 at commit ec9199a.

This patch fails PySpark unit 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/81359/
Test FAILed.

retest this please.

Test build #81362 has started for PR 18869 at commit ec9199a.

Test build #81362 has finished for PR 18869 at commit ec9199a.

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/81362/
Test PASSed.

@HyukjinKwon @gatorsmile Any more comments on this? The added tests should be enough.

LGTM

Thanks! Merging to master.

Thanks @HyukjinKwon @gatorsmile


[Archive github repository] bpdm-certificate-management

Details
We(@maximilianong and @SujitMBRDI ) want a support to archive one of our github repository.
Repository details -
Url - https://github.com/eclipse-tractusx/bpdm-certificate-management
Also, where we can find steps if we want to unarchive same repo in future?

@SujitMBRDI you can create a PR in our repository configuration https://github.com/eclipse-tractusx/.eclipsefdn (take a look at the closed PRs, there is at least one for archving a repo).
Unarchiving is the same road just the PR setting the flag the other way.
@maximilianong Evelyn and Mathias are still active,i'm active until October. Sebastian is gone though

Maybe it makes sense to document this process in an TRG?
I couldn't find anything there.

@SujitMBRDI you can create a PR in our repository configuration https://github.com/eclipse-tractusx/.eclipsefdn (take a look at the closed PRs, there is at least one for archving a repo).
Unarchiving is the same road just the PR setting the flag the other way.
@maximilianong Evelyn and Mathias are still active,i'm active until October. Sebastian is gone though

@Siegfriedk, thank you for your inputs here.
I have raised pull request to archive eclipse-tractusx/bpdm-certificate-management repo.

Maybe it makes sense to document this process in an TRG? I couldn't find anything there.

TRGs are standards not workflows. We had it documented on system team pages. It should be part of our Developer Hub https://eclipse-tractusx.github.io/docs/developer


Validator._guardExtended: deleted restriction for getMessage attribute.

This caused severe problems when using multilanguage as it overrided the translation for the current locale.
Now this implementation is compliant with the documentation wich says "can contain a getMessage method", but it's not required.

Thanks for the PR, but do you mind fixing the failing test? Again thanks!

Ready. I forgot about the tests :(
Now Mr. Codecov is not happy. What should I do? Is it ok?

Well I don't know what Codcov deal is with this PR, I guess I can just merge it 😄
Thanks for the PR! looking forward to the next one!


Remove country borders file

I'm putting this in Satsuma for now, but we should move it to Poblano if we decide to do it, even if part of Satsuma.
Before the Sudan training we noted that there was a disputed border between Sudan and S. Sudan and since we didn't have time or the....something.... to tackle the disputed border issue, we just removed Sudan from the file that creates the country borders entirely.
I'm wondering if we should find a better basemap, one with more prominent country borders.  This could potentially eliminate the need for our extra layer.  One alternative is to a more official set of borders than the really bad ones that we have right now.  The potential downside is that the couple that I've seen are much bigger files, and the current crappy border file is already one of the more time consuming ones that we load.  There may be ways to compress these files that we haven't explored though.
This ticket is just a way of starting this conversation.

@sanjuroj Why put it in Satsuma for now if it belongs in Poblano?

Because I think it might disappear in Poblano.  I sort of wanted to see how priority it was before dropping it into the pile.

This work should be done as part of Sites (per Marie)

This may already have been done.  @camgnostic  worked on the maps as part of levelup.

This will be done once Level Up launches.


[MRG] fix glossary path

glossary is not reachable through search

nor through index

are you sure of your fix?
I think the online doc is simply not updated.

you are right.


Update to version 0.40.0

Update our fork from 0.28.0 to 0.40.0
The biggest change is the introduction of RequestChainNetworkTransport which replaces their HTTPNetworkTransport
https://github.com/apollographql/apollo-ios/releases/tag/0.34.0
I've also reverted some changes that we've made which have either been merged upstream or replaced by something else.
The only one remaining is the delegate webSocketTransport(_ webSocketTransport: WebSocketTransport, didReceiveMessage message: (payload: JSONObject?, error: Error?))

@ketenshi can we close this now?

@ketenshi can we close this now?

Yep, thx for the reminder

Needed to restore because the TSB Apollo feature branch is still on connect-0.40.0


Create a FastAPI app that will check lag on a given topic-consumerGroup-brokerlist combination

Create a FastAPI app that will check lag on a given topic-consumerGroup-brokerlist combination. A corresponding Dockerfile would be appreciated.

assign this to me please


SQLITE returns wrong id on bulk insert

From the docs:

insert — .insert(data, [returning])
Creates an insert query, taking either a hash of properties to be inserted into the row, or an array of inserts, to be executed as a single insert command. Resolves the promise / fulfills the callback with an array containing the first insert id of the inserted model, or an array containing all inserted ids for postgresql.

But If I try this with sqlite, the last id will be returned.  (with knex 0.11.10)

I'm using knex 0.12.6 and I can confirm this issue with sqlite. Maybe it's just a typo in the docs?

If I recall correctly the only way to get id of inserted row is by making another query to request id of the last row added https://www.sqlite.org/c3ref/last_insert_rowid.html.
So this sounds like invalid documentation to me as well. Need to check from code or try insert data with sqlite3 with debug prints (`DEBUG=knex:*) to verify that from the implementation.

I have a test that inserts 6 rows to a table "unidades" inside a transaction. I ran it with debug prints and it only shows a single query for that table:
  knex:query insert into "unidades" ("expiracion", "producto", "venta") select ? as "expiracion", ? as "producto", ? as "venta" union all select ? as "expiracion", ? as "producto", ? as "venta" union all select ? as "expiracion", ? as "producto", ? as "venta" union all select ? as "expiracion", ? as "producto", ? as "venta" union all select ? as "expiracion", ? as "producto", ? as "venta" union all select ? as "expiracion", ? as "producto", ? as "venta" +1ms
  knex:bindings [ 2016-11-08T00:32:16.967Z, 1, 1, 2016-11-08T00:32:16.967Z, 2, 1, 2016-11-08T00:32:16.967Z, 2, 1, 2016-11-08T00:32:16.967Z, 3, 1, 2016-11-08T00:32:16.967Z, 3, 1, 2016-11-08T00:32:16.967Z, 3, 1 ] +0ms

@GAumala I couldn't find from sqlite documentation what that kind of insert should return...


Trains Maintain Their Lanes (hey that rhymes)

The code may already be in here (just needs to be reorganized) but as quad tracks are soon to become a thing, it would be nice to have vehicles (trains in this case) favor the lane they are on rather than switching lanes for a slight distance difference in curves.  The exception to this, of course, would be if the placed stops mandated a lane change which would mainly be an issue for quad tracks with central bypass lanes.  Otherwise there would be no reason for a lane change as all junctions naturally connect all compatible lanes.  I don't remember if the pathfinding is a weighted system but basically this would all be fixed if more 'weight' was added to changing lanes.  That way trains would not do it unless there was no other way to reach their endpoint or the speed difference between the lanes outweighed the lane change (which I would imagine would be rare).

Ok giving this 2 seconds of thought, it is very similar to #89 Medians. It will operate the same as a road that has medians separating each lane, except at 3+ network junctions.

The issue of vehicles changing lanes for a slightly shorter distance at curves also affects highways in a most annoying manner: #112

Path-finding uses average segment length for cost calculation, not lane length. Your issue must be caused by something else.

@andreharv Could you please post a screenshot or a savegame that shows this behavior? I would like to investigate this issue.

@andreharv This only occurs for one-way tracks, correct?

Yes.  This is only an issue on one way tracks and quad tracks (MOM).  If you send me your email, I can give you a copy of the MOM build (it's near complete).  It will be a lot easier to see when putting down quad tracks and stations.  All the lanes are the same speed except for the bypass station tracks (outer tracks are stations and inner tracks are bypass).


Update Math and Time library APIs to handle RTE

Description:
Currently, Time library APIs and some Math APIs does not handle RunTimeExceptions correctly. The API signature should change to reflect errors.

Fixed with https://github.com/ballerina-platform/module-ballerina-time/pull/49


Select icon to display in mobile app (MEGH-1438)

It would be a great enhancement if it is possible to specify what icon to use in the mobile app. Basically a list of default images we can select to display when we create the device in rainmaker firmware. For example, I want to switch a pump, so I use a switch device, but would like the icon to be of a pump or similar.

You can achieve this by creating a new custom device and set the switch param as the main param. Also, you need to create a new icon and build your own phone app, just add your own icon here (for android). Anyway, I think more icons and devices are coming in the near future.

You can achieve this by creating a new custom device and set the switch param as the main param. Also, you need to create a new icon and build your own phone app, just add your own icon here (for android). Anyway, I think more icons and devices are coming in the near future.

Will this not replace the switch icon for any device using it?

No. I will give you an example for humid sensor which I have implemented in IO Connect
In EspDeviceAdapter.java:
 else if (AppConstants.ESP_DEVICE_HUMID_SENSOR.equals(device.getDeviceType())) {
deviceVh.ivDevice.setImageResource(R.drawable.rmk_ic_device_humid_sensor);
}
and in AppConstants.java
   public static final String ESP_DEVICE_HUMID_SENSOR = "esp.device.humidity-sensor";


Azure Blob Storage & Elementary Data Cli

Share observability report to Azure blob storage by using edr send-report command

Hi @dharitsura248 !
The design for this goes as follows -

Add new set of parameters that are required to upload the report to Azure blob to the Elementary CLI in monitor/cli.py. Go to the send_report endpoint and see how the S3 / GCS params are configured.
Populate the new params to Config.py (see how S3 / GCS params are populated there)
Add new AzureBlobClient (take a look at S3Client or GCSClient). We will need to implement 2 main things here -

create_client - static method to create the client
send_report - uploads the report to Azure blob. if self.config.update_bucket_website is enabled it means that we will need to make this bucket a static website. This will enable users to access the report as a static website. The implementation on how to make a bucket a static website is a bit different between cloud providers but feel free to reach out and I can help with that as well.

In DataMonitoringReport initialize the new AzureBlobClient and use it in both send_report and upload_report functions in case the Azure blob params were given

Please let me know if you have any questions! would love to help with this


exp/services/recoverysigner: use unique signing keys per registered account

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

What
This PR adds the feature of using unique signing keys per registered account. The secret keys of the signers are encrypted and stored in the database.
Why
The application uses a single signing key for all accounts that are registered. The issue with this is that anyone who knows that single signer can determine what accounts are registered with a recoversigner server. This public association may be undesirable for a user using a recoverysigner.
Known limitations
N/A
Close #2343

@accordeiro So that it is easy to pickup where we left off, I've made a list of things to do before bringing this out of draft:

[ ] Add tests for the areas of code that aren't tested that use KMS https://github.com/stellar/go/issues/2750
[ ] Add docs for how to use unique signing keys https://github.com/stellar/go/issues/2795
[ ] Add use of public key so service can startup without decrypting the keyset https://github.com/stellar/go/issues/2733
[ ] Support key rotation of unique signing keys https://github.com/stellar/go/issues/2631

Do you still want to pursue this or shall we close it?

We still plan to pursue this. Thanks @2opremio.

sorry for asking the same question again but do you still want to keep this open?

sorry for asking the same question again but do you still want to keep this open?

@2opremio I'm going to close this now because I don't know the answer to that and this PR is getting more out-of-date as time moves on. We still have the open issue here https://github.com/stellar/go/issues/2343 which I'll keep open until @shanzzam can chime in there.

@2opremio I'm going to close this now because I don't know the answer to that and this PR is getting more out-of-date as time moves on. We still have the open issue here https://github.com/stellar/go/issues/2343 which I'll keep open until @shanzzam can chime in there.


Better Animations (Help Wanted)

If anyone is good with any type of 2d animation be it SVG or pixel art, please consider chipping in. If you contribute you will be rewarded 0 USD
The current animations are in assets/public for reference

If you are not an artist but like Fable and you are capable of a one-time donation visit https://github.com/sponsors/ker0olos all the money donated will go into a prize pool that will be split between the artists who end up creating the animations. Current All-Time Donations: 0 USD

Requested Assets

Better 1-5 star rating reveal
Unique loading spinner themed for gacha pulls
Unique loading spinner themed for synthesis
Unique loading spinner themed for stealing

Fable Brand Colors
primary: #feb500
secondary: #fed33c

I will chip in myself with 5 USD every month until this is finished


Pin sassc-rails to current ref

As per conversation in original PR: https://github.com/alphagov/whitehall/pull/2176/files#r30508635

:+1:

:+1: Thanks


Add measure/scalar/swap/display/qft support to import-from-quirk

This is ready for review now @viathor

Hm. Thought github lost the comments the first time.


Action to expand selection.

Trying to expand a selection in linear view to a whole section/segment is tedious. It would be helpful if there were actions which could help:
Expand Selection\To end/start of bv
Expand Selection\To end/start of section
Expand Selection\To end/start of segment
Expand Selection\To end/start of function
Expand Selection\N Bytes

Related to https://github.com/Vector35/binaryninja-api/issues/1827

Added via https://github.com/Vector35/binaryninja-api/commit/015314d34d073a8ebbd958edf8150a81381f12c2, should be available in dev 4.1.5260