Added missing ../ to flashloader cmake.

That's... one level outside the 32blit repo? Are you running inside the firmware folder? I meant for that to be run at the root.

I think the docs are missing the step for building the Flash loader tool (for Windows/Linux/Mac)
There might be a clearer way to say that the build should be run in the root- but then I suspect it's also possible for CMake to find these directories no matter where you run the build from? I'll have a tinker!

I put a "Run the build for your platform in the top level" to avoid duplicating the instructions for every platform.

I think the docs on the whole differ between "build in root" and "build in subfolder" so we probably need to canonicalise on the latter so people don't get into bad habits.
Stuff like the "32Blit" programmer almost feels like it belongs completely outside of this source tree- since it's not valid to build it with anything but a platform specific toolchain, but you'd only be using it when building with the device toolchain. I wonder if it's feasible to stick it into another repository and just ship binaries here?
That said- with all the attention on this repository it's the best place for it to be maintained!

Maybe split out all the tools into their own repo when things are more stable. You can get CMake to find the tool anywhere if it's in your PATH.
Something else that should probably be covered in the docs is building an example vs building your own project, which should probably happen outside the repo entirely. I'm using something like the CMakeLists in vs/game and then doing cmake -D32BLIT_PATH=path/to/this repo for my stuff.
I'm going a bit off topic now though. Might go do adjust some docs...

That's... one level outside the 32blit repo? Are you running inside the firmware folder? I meant for that to be run at the root.

I was just thinking if people are following the other build examples they would probably go to the files with the cpp file and make a build folder from there.

As aside note feel free to chuck any documentation my way to test. Should give you a more realistic idea of how its likely to be interpreted as you guys clearly are much more knowledgeable.

I've started reshuffling some of the 32Blit documentation to hopefully isolate the different stages and make them clearer: https://github.com/pimoroni/32blit-beta/commit/f5ef27c75a3ca5d8dc45f7720fe926d0699d394c
Your input on these changes would be greatly appreciated!


Android Basics: Viewmodel and State in Compose

Second block of code in codelab step 7 references property that does not exist in state
7. Verify guess word and update score
import kotlinx.coroutines.flow.update

   if (_uiState.value.userGuess.equals(currentWord, true)) {
   } else {
       // User's guess is wrong, show an error
       _uiState.update { currentState ->
           currentState.copy(isGuessedWordWrong = true)
       }
   }

Here is the code in the solution:
    fun checkUserGuess() {
        if (userGuess.equals(currentWord, ignoreCase = true)) {
            // User's guess is correct, increase the score
            // and call updateGameState() to prepare the game for next round
            val updatedScore = _uiState.value.score.plus(SCORE_INCREASE)
            updateGameState(updatedScore)
        } else {
            // User's guess is wrong, show an error
            _uiState.update { currentState ->
                currentState.copy(isGuessedWordWrong = true)
            }
        }
        // Reset user guess
        updateUserGuess("")
    }

This is still an issue. Very confusing.

I noticed the same. It got me confused for a while.

Aaah, here I thought I wasn't doing something right

Hello @CalebEverett,
Thank you for reaching out and reporting this issue. The code snippet in the section 7 was fixed:

Our apologies for the confusion it caused. Happy coding!


Progress Component - Type 'string' is not assignable to type 'number'.

TypeScript type checking yields an error when using the progress component:

Since it is defined in Progress.vue for progress prop to be Number you need to pass it as a Number.
In your example you're passing progress as static prop, which is String.
Try changing progress="45" to :progress="45", which is dynamic prop and it will be passed as a number.
More info can be found on: https://vuejs.org/guide/components/props.html#prop-passing-details

As @josipgagro said, you need to provide a number, not a string.


Build failing on caddyserver.com

Describe the issue
When I try to download Caddy with this plugin built in it hangs for a long time and ultimately fails: https://caddyserver.com/download?package=github.com%2Fgreenpau%2Fcaddy-git
Running a build with another plugin downloads a binary rather quickly without issue.
Is there a chance that this plugin isn't compatible with the latest version of Caddy? Do we have to build from source with a particular dependency that they can't provide through the website?
Configuration
N/A
Version Information
N/A
Expected behavior
I expect the binary with the plugin included to download.
Additional context
N/A

@jimafisk , have you tried specifying the version in your download URL?

@jimafisk , just tried downloading from caddyserver.com and it worked fine.

That's odd, I've tried on a couple of different computers and get the same thing each time:

Screenshot of failed build

I just tried setting version 1.0.7 but that didn't seem to work either unfortunately.

i got the same problem i think ...
i am guessing that the version of xcaddy has to match the version of the this repo in some way ???
go install github.com/caddyserver/xcaddy/cmd/xcaddy@latest

xcaddy build --with<EMAIL_ADDRESS>2023/02/15 18:44:14 [INFO] Temporary folder: /Users/apple/workspace/go/src/github.com/gedw99/charty-gui/install/exp/caddy/git/buildenv_2023-02-15-1844.2888214292
2023/02/15 18:44:14 [INFO] Writing main module: /Users/apple/workspace/go/src/github.com/gedw99/charty-gui/install/exp/caddy/git/buildenv_2023-02-15-1844.2888214292/main.go
package main

import (
        caddycmd "github.com/caddyserver/caddy/v2/cmd"

        // plug in Caddy modules here
        _ "github.com/caddyserver/caddy/v2/modules/standard"
        _ "github.com/greenpau/caddy-git"
)

func main() {
        caddycmd.Main()
}
2023/02/15 18:44:14 [INFO] Initializing Go module
2023/02/15 18:44:14 [INFO] exec (timeout=10s): /usr/local/opt/go/libexec/bin/go mod init caddy
go: creating new go.mod: module caddy
go: to add module requirements and sums:
        go mod tidy
2023/02/15 18:44:14 [INFO] Pinning versions
2023/02/15 18:44:14 [INFO] exec (timeout=0s): /usr/local/opt/go/libexec/bin/go get -d -v github.com/caddyserver/caddy/v2
go: added github.com/beorn7/perks v1.0.1
go: added github.com/caddyserver/caddy/v2 v2.6.4
go: added github.com/caddyserver/certmagic v0.17.2
go: added github.com/cespare/xxhash/v2 v2.1.2
go: added github.com/go-task/slim-sprig v0.0.0-20210107165309-348f09dbbbc0
go: added github.com/golang/mock v1.6.0
go: added github.com/golang/protobuf v1.5.2
go: added github.com/google/pprof v0.0.0-20210407192527-94a9f03dee38
go: added github.com/google/uuid v1.3.0
go: added github.com/klauspost/cpuid/v2 v2.2.3
go: added github.com/libdns/libdns v0.2.1
go: added github.com/matttproud/golang_protobuf_extensions v1.0.1
go: added github.com/mholt/acmez v1.1.0
go: added github.com/miekg/dns v1.1.50
go: added github.com/onsi/ginkgo/v2 v2.2.0
go: added github.com/prometheus/client_golang v1.14.0
go: added github.com/prometheus/client_model v0.3.0
go: added github.com/prometheus/common v0.37.0
go: added github.com/prometheus/procfs v0.8.0
go: added github.com/quic-go/qpack v0.4.0
go: added github.com/quic-go/qtls-go1-18 v0.2.0
go: added github.com/quic-go/qtls-go1-19 v0.2.0
go: added github.com/quic-go/qtls-go1-20 v0.1.0
go: added github.com/quic-go/quic-go v0.32.0
go: added go.uber.org/atomic v1.9.0
go: added go.uber.org/multierr v1.6.0
go: added go.uber.org/zap v1.24.0
go: added golang.org/x/crypto v0.5.0
go: added golang.org/x/exp v0.0.0-20221205204356-47842c84f3db
go: added golang.org/x/mod v0.6.0
go: added golang.org/x/net v0.7.0
go: added golang.org/x/sys v0.5.0
go: added golang.org/x/term v0.5.0
go: added golang.org/x/text v0.7.0
go: added golang.org/x/tools v0.2.0
go: added google.golang.org/protobuf v1.28.1
2023/02/15 18:44:16 [INFO] exec (timeout=0s): /usr/local/opt/go/libexec/bin/go get -d -v<EMAIL_ADDRESS>github.com/caddyserver/caddy/v2
panic: internal error: can't find reason for requirement on<EMAIL_ADDRESS>
goroutine 1 [running]:
cmd/go/internal/modget.(*resolver).updateBuildList.func1({{0xc0009542b8?, 0xc000884588?}, {0xc0000d0c60?, 0xc000b1e5c8?}})
        /usr/local/Cellar/go/1.19.6/libexec/src/cmd/go/internal/modget/get.go:1760 +0x114
cmd/go/internal/modget.(*resolver).updateBuildList(0xc000556900, {0x1732870, 0xc0000a8000}, {0xc001278d00, 0x1, 0x1})
        /usr/local/Cellar/go/1.19.6/libexec/src/cmd/go/internal/modget/get.go:1765 +0x597
cmd/go/internal/modget.(*resolver).applyUpgrades(0xc000556900, {0x1732870, 0xc0000a8000}, {0xc000482240?, 0x1?, 0x1071c85?})
        /usr/local/Cellar/go/1.19.6/libexec/src/cmd/go/internal/modget/get.go:1312 +0x105
cmd/go/internal/modget.runGet({0x1732870, 0xc0000a8000}, 0xc0000c62b8?, {0xc0000bc160, 0x2, 0x2})
        /usr/local/Cellar/go/1.19.6/libexec/src/cmd/go/internal/modget/get.go:351 +0x45e
main.invoke(0x1a37120, {0xc0000bc130, 0x5, 0x5})
        /usr/local/Cellar/go/1.19.6/libexec/src/cmd/go/main.go:225 +0x34e
main.main()
        /usr/local/Cellar/go/1.19.6/libexec/src/cmd/go/main.go:179 +0x7d1
2023/02/15 18:44:17 [FATAL] exit status 2

i tried the 2 latest tagged versions of xcaddy at  https://github.com/caddyserver/xcaddy/releases/tag/
XCADDY_VER=v0.3.2
XCADDY_VER=v0.3.1
both versions got the same error.

i also tried the ''' make all ''' with the latest xcaddy just like the make file does.
also fails...

@jimafisk , @gedw99 , there is some issue with Go. What is the go version you are compiling with?

I get the same error as @gedw99 using Docker:
FROM caddy:2.6.4-builder AS builder

RUN xcaddy build \
    --with<EMAIL_ADDRESS>
FROM caddy:2.6.4

COPY --from=builder /usr/bin/caddy /usr/bin/caddy

Throws the same error above.

@gedw99
see：
https://github.com/zuohuadong/caddy-git
run:
./xcaddy build     --with<EMAIL_ADDRESS>

Upgraded to the latest: https://github.com/greenpau/caddy-git/releases/tag/v1.0.8


Add integration testing

Adds runtimes_common integration tests to the jetty runtime. This is analogous to the work done for the openjdk-runtime in https://github.com/GoogleCloudPlatform/openjdk-runtime/pull/75.

Readme updated :)

Looks good, but what about instructions on how to run the Cloud Container Runtimes Integration Tests, like here.
Would probably also need to clarify that these are different from the integration tests in /tests.


iOS Chrome not working

I get an error on Chrome for iOS
Erorr: Pusher Beams does not support this browser version.
Chrome 87.0.4280.77
iOS 14.2
iPhone Xr

At this moment iOS browsers are not supported by Pusher Beams, as iOS browsers do not support the Push API. This is something that may be addressed in the future but is not on our current roadmap.
You can use our push-notification-swift library for any native iOS apps that require push notifications.


Client and Operation properties should be virtual

@KrzysztofCwalina , @heaths , @annelo-msft should we update our support for mocking guideline? https://azure.github.io/azure-sdk/dotnet_introduction.html#dotnet-mocking

This section doesn't say anything about properties, it mentions only methods.

https://github.com/Azure/azure-sdk/pull/2738


[client] Fix inconsistent parameter of TopicPolicies.getSubscriptionDispatchRate

Motivation
getSubscriptionDispatchRate and getSubscriptionDispatchRateAsync in org.apache.pulsar.client.admin.TopicPolicies use "namespace" in parameter,
but the implementation in org.apache.pulsar.client.admin.internal.TopicPoliciesImpl uses "topic".
This should be a typo. "Topic" should be used.
Modifications
Use "topic" instead of "namespace".
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] no-need-doc
fix typo and doc error.

/pulsarbot run-failure-checks


Highlight and warning background colors off

Describe the solution you'd like
I might have missed this but trying to find a way to turn off the background colors on warnings and errors, or this is not a feature yet?

When you are referring to warning background colors are you referring to the virtual text from vim.diagnostic. If so then yes there is a way to not set the background. Nightfox's modules allow for more specific options for different areas/plugins. For the diagnostic module there is an option defined to toggle the background color.
require("nightfox").setup({
  options = {
    modules = {
      diagnostic = {
        background = false,
      },
    },
  },
})

See usage for more details.

Thank you, that was what I was looking for.


Add ability to disable individual options in RadioButtonGroup

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
It would be nice to have the option to disable individual Radio Buttons with the RadioButtonGroup. Right now, it is only possible to disable the entire group at once.
Examples 🌈
One example:
<RadioButtonGroup
  label="Label"
  name="My Radio Button Group"
  options={[
    {
      id: 1,
      label: 'ABC',
      disabled: watch('other_field') === "2"
    },
    {
      id: 3,
      label: 'XYZ'
    }
  ]}
/>

never had a use for it, but happy to review a PR for this feature

It would be very nice for the MultiSelectElement too


Problem importing check-disk-space. Minor fix suggestion

I've been trying to work on node installations that support esm, and found that check-disk-space doesn't work out of the box on them. I've tested this on node 16.9.1. Using require works fine however.
Whether or not if my package.json include "type": "module", and then try to import check-disk-space via
import checkDiskSpace from "check-disk-space", I get the following error in the console:
(Use `node --trace-warnings ...\` to show where the warning was created)
/Users/mark/ch_test/node_modules/check-disk-space/dist/check-disk-space.es.js:1
import { execFile } from 'child_process';
^^^^^^

SyntaxError: Cannot use import statement outside a module`

However, if I make some minor changes to the module in such that I do the following:

Add "type": "module" to check-disk-space's package.son
Rename check-disk-space.cjs.js to check-disk-space.cjs
Rename check-disk-space.es.js to check-disk-space.mjs
change "main": "./dist/check-disk-space.cjs.js" to "main": "./dist/check-disk-space.cjs"
change "import": "./dist/check-disk-space.es.js" to "import": "./dist/check-disk-space.mjs"
change "require": "./dist/check-disk-space.cjs.js" to "require": "./dist/check-disk-space.cjs"

then  I can use the import syntax without any problem in esm environments, and require in commonjs environments. I am not sure if the above fix is the most optimal, but it allowed me to work in both environments without any problems.
Testing code used was simply:
import checkDiskSpace from "check-disk-space";
//const checkDiskSpace = require("check-disk-space").default; // for testing require

(async () => {
  console.log(await checkDiskSpace("/"));
})();

I've also tested this with typescript with "target": "ESNext", "module": "ESNext", and "moduleResolution": "node" in my tsconfig.json.

Same issue here. Reverting to 2.1.0 until fixed.

Thank you for the report and the documentation of the issue :)
This is fixed in the new 3.1.0 release


First draft for collections

TODO:

[x] Coverage
[ ] where method

Coverage decreased (-12.8%) to 87.179% when pulling 521c213ce3c12958671d7c14451a798ae8e25c08 on collections into d85cea4bbbac878e54282995531fef011d10281a on master.

Coverage remained the same at 100.0% when pulling 705502cd8ca73a8735feee6d4f28bdc522533bb7 on collections into d85cea4bbbac878e54282995531fef011d10281a on master.

Coverage remained the same at 100.0% when pulling 77cb9468246d245abca2cc41e20c5c83c8ad2a9d on collections into d85cea4bbbac878e54282995531fef011d10281a on master.


Document Text Tricks in docs for each info_*() function

There is a collection of text tricks that can be used when creating info text. A paragraph briefly explaining how to use these should be included in each of the info_*() functions that deal expressly with adding info text.

Fixed with https://github.com/rich-iannone/pointblank/commit/7a3019fd78f71db81f01d0f102ef36fb8beff1e8.


west: runners: jlink: Disable DAP after flashing

Disables the Debug-Access-Port of the microcontroller after flashing.
If not disabled, the DAP consumes ~1.6mA until the debugger disables it
or a hard power cycle is applied.
The added command reset the value of the CTRL/STAT register of the DAP.
This clears the CSYSPWRUPREQ and CDBGPRWUPREQ bits, leaving the debug
hardware free to power off the appropriate hardware. In no way does it
hinder the ability to later connect to the device for debugging.
The second write is required as the first write is not applied.
Alternate approaches like read before write and sleep before write
were tested and did not work. Write before write works 100% of the
time.
This resolves the jlink portion of #26139
Signed-off-by: Jordan Yates<EMAIL_ADDRESS>

@simtind  Please re-review. I have updated the docs and the method in light of your comments.
As noted in the issue, I have also determined this is only a problem when other copies of JLink tools are running (they don't have to be connected to anything).
If this change is undesirable in those contexts, let me know.
In that case I will update the runner docs instead.

@anangl can you review?

@JordanYates with this change, are you able to west attach after flashing? since you disable the debug port.

@JordanYates with this change, are you able to west attach after flashing? since you disable the debug port.

I'll answer my own question here: yes, attaching works fine even when flashing with this PR.

@JordanYates with this change, are you able to west attach after flashing? since you disable the debug port.

I'll answer my own question here: yes, attaching works fine even when flashing with this PR.

Most tools know to power the debug port when performing a new action, so it shouldn't be a problem. Can be an issue with long running sessions like rtt or a live debugging session, but you're reprogramming, so you're breaking that kind of session anyways.

Changes look good.

@JordanYates with this change, are you able to west attach after flashing? since you disable the debug port.

I'll answer my own question here: yes, attaching works fine even when flashing with this PR.

Most tools know to power the debug port when performing a new action, so it shouldn't be a problem. Can be an issue with long running sessions like rtt or a live debugging session, but you're reprogramming, so you're breaking that kind of session anyways.

Thanks for checking


CombineLatest for IObservable<IObservable>

Feature request

Which next library version - major

What are the platform(s), environment(s) and related component version(s)? - Software can be upgraded as needed

How commonly is this feature needed - Several projects

Concat & Merge both have overrides that can operate on IObservable<IObservable>.
This is great, but a use case I keep running into is wanting the last values of each group within a GroupedObservable.
This could be implemented as an extension method similar to that of below:
public IObservable<IList<TSource>> CombineLatest<TSource>(IObservable<IObservable<TSource>> sources)
{
  return sources
    .ToList()
    .Select(agg => agg.CombineLatest())
    .Switch();
}

However, I feel that this feature would be a logical extension to the core api.
I do however understand that this could lead to memory management problems when dealing with an outer observable that is chatty.
sourceObserable
  .GroupBy(s => s.Id)
  .CombineLatest()
  .Subscribe(latestItems => {
    // Process latest item from each Id grouping
  });

Is this something that could be added to QueryLanguageEx
I will happily implement a pull request if this feature is deemed appropriate.

It is trivial to aggregate observable into a list and then apply combineLatest or zip, which require a known and fixed number of sources to work. I don't think such overloads should be in Rx.NET proper but you can have it in your project.

You could say the same for Merge.
You could aggregate the outer observable into a list of observables and then call the IList<IObservable> variant to get the same behavior. But in this case, you provide the override for IObservable<IObservable> which is a lot more convenient.

Wrong. Merging an observable sequence of observables is a completely different operation with different semantics. Each inner observable can arrive over time and there could be any number of such inner observables being merged into a single stream over the lifetime of the outer observable, without them needing any item-based correlation.
CombineLatest operates by taking a full row of latest values available and transform them into one output value per row. For this, the number of sources have to be known, otherwise there is no way to know when a row can be considered full. It is very rare the number of sources to combine is not known when the subscription happens.

I do recognize that Merge has the benefit of outputting a simpler type signature.
However, the idea here is to add the same concept  to combineLatest.
The source is the same and has the same constraints IObservable<IObservable<T>>. That outer observable could produce a new inner observable at any time and there is no way of telling when that might happen.
This is outputting outputting a list of length n + 1 wherever a new inner observable is emitted and an updated list wherever any of the inner observables emit. It's not waiting for the completion of the outer observable to complete before outputting.
As a usage example. Say you have a stream of messages coming from multiple sources on a single stream, like a message bus topic.
Each of these messages could come from a different source and contain an Id property.
Now say that you want a stream of the latest value from each of the services.
This means that you need to aggregate state in some way.
Note: the version of combine latest I'm referencing in that Select has the signature IEnumerable<IObservable<T>> -> IObservable<T>. I don't think it's much of a strech to want to extend that to monitor an outer observable.

Still wrong. Merge doesn't need multiple of the inner sources to have items produced in order to emit; they can go completely independently.
Also your GroupBy + CombineLatest example can't work. You collect up all groups, which at that point will no longer signal any more items because the GroupBy operator has completed all groups and the sequence of groups itself (otherwise, ToList would never complete and emit the list of groups to be selected over).
As for any kind of live, rolling array-combination, it is prone to all sorts of problems. For example, any lack of data from a fresh source can prevent forming a new row; or if the number of sources keeps growing indefinitely, leading to massive performance degradation due to constant need to copy a large array of latest values all the time.
In addition, in your example with the one group per Id case, why do you want to correlate different Id groups and possibly correlate an arbitrary number of such groups? Why would you want to combine every fresh item of a group with the latest item of all the other groups?
See also #149.

GroupBy will emit a new observable whenever a new group appears. It does not need to consume the entire source stream, that's the cool thing about GroupBy.
I do agree with the notion that CombineLatest requires an item from each source, but this is a problem with CombineLatest in any use of it's setup. One option in this case would be to call .Select(innerOb => innerOb.StartWith(<someValue>)).
And I again do agree that any rolling array combination is prone to problems, especially when growing infinitely, so this is more of a niche operator in that regard.
But do take into regard that the example above is to illustrate a concept. An append only linked list would solve the memory inefficiencies for instance. Another flaw in that design is that the Scan / ToList combined with Select(CombineLatest) will force a re-subscription to all inner observable's whenever the list changes as it goes through Scan.
Ideally the code would be based off of an operator similar to Merge where the previous subscriptions are always left open.

It doesn't work. If you don't believe me, create a small project, create the operator and then write some tests to verify it behaves as you think it would.

I have code like this in a running project.
What is the thing that you don't think works here.
GroupBy emits over time
Scan creates an IObservable<IList<T>> that will emit an updated list every time the outer observable.
So you get IObservable<IObservable<IList<T>>
Then there is a mapping to CombineLatest so you get IObservable<IObservable<IList<T>>
Then you switch so that the latest CombineLatest is the only source consumed.
But again, the implementation is not important here as it can be optimised if it's built from the ground up.

ToList doesn't work and that Scan will create larger and larger pool of sources which have to be hot, otherwise every CombineLatest application will restart the sequences. If they are hot, you have a race condition between abandoning the previous subscriptions to start the new subscriptions, which could lead to lost signals on one or many of those sources. The general setup works only when you have groups from GroupBy and the whole switcharound can happen synchronously to it.
You presented use cases that are quite error prone in general and represent a very peculiar behavior, thus not suitable for Rx.NET proper. You can, of course create your own extension method in your own library and publish it with as you see fit.

I'm okay with this implementation being error prone, it's a good distinction that messages within the inner observables could be lost during resubscription.
I regret putting forward the example code as I had only put it there as a discussion point to the concept of a high level operator for CombineLatest.
I do believe that an operator that could handle this behavior in a safe manner would be valuable.
As stated in the request you linked, each high level operator should be evaluated individually.
If the comminity does not agree with the utility of such an operator, then so be it.


More v2 work

FIx v2 rate limits
break swagger doc in to multiple files
v2 work

Pull Request Test Coverage Report for Build 863

0 of 0   changed or added relevant lines in 0 files are covered.
165 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.04%) to 80.423%

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/mining.js
5
77.09%

dist/routes/v2/transaction.js
6
83.98%

dist/routes/v2/blockchain.js
38
81.38%

dist/routes/v2/rawtransactions.js
54
82.09%

dist/routes/v2/address.js
62
75.63%

Totals

Change from base Build 834:
-0.04%

Covered Lines:
2493

Relevant Lines:
2916

💛  - Coveralls

:tada: This PR is included in version 1.14.0 :tada:
Your semantic-release bot :package::rocket:


[com.google.fonts/check/fontvalidator] ERRORs if FontValidator is not installed

Observed behaviour
If FontValidator is not installed, com.google.fonts/check/fontvalidator fails with two ERROR messages.

Expected behaviour
The first ERROR should be converted into a FAIL, because there's nothing wrong with openbakery itself.
The second ERROR needs to be fixed; it's caused by trying to run FontValidator before checking if it's available.
Resources and steps needed to reproduce
Install openbakery and run,
openbakery check-fontval data/test/source-sans-pro/OTF/SourceSansPro-Regular.otf

The first ERROR should be converted into a FAIL, because there's nothing wrong with openbakery itself.

A FAIL is a problem in a font.
An ERROR is a bug in the program or a bad setup, such as a missing third-party tool in the system.

When FontValidator is not installed, it is treated as a bad system setup, thus, it is classified as an ERROR. It is not a FAIL because it is not a font problem.

I think neither status is quite right for conveying that a tool or package needed for running a check is missing. However,

there's a precedent for yielding FAIL when a package is missing, and
the check-run ends with a message that instructs users to report ERROR statuses.

So even though FAIL is commonly regarded as a font problem, it seems appropriate (and less harsh) to use when a check cannot be run. I suppose SKIP could be used as well if we interpret it as "the conditions for running the check have not been met", but that would result in a silent failure.

A FAIL is a problem in a font.
An ERROR is a bug in the program or a bad setup, such as a missing third-party tool in the system.

That division is sensible for a programmer, I think.
Instead, my goal is to aim toward a more user-centric organization of the statuses. As such, I see them fitting into two main categories:

The check didn't run in its entirety; it exited abruptly or unexpectedly. There is nothing a user can do on their end to fix the failure. The only action is for the user to report the problem. Yield ERROR in such scenario.
The check ran in its entirety. The user is able to address any check failures on their own, without requiring the tool's codebase to be updated. In such scenario, yield all appropriate statuses except ERROR.

I think this way there's a clear distinction between when a tool-run completes in an expected way and when it doesn't. The benefit for users is that overtime they'll know that an ERROR result is something that's beyond their control, unlike all the other check results.

The check ran in its entirety. The user is able to address any check failures on their own, without requiring the tool's codebase to be updated. In such scenario, yield all appropriate statuses except ERROR.

The part where the user is able to address a bad setup of the system, such as installing a missing software in the system,  on their own is requiring a very specific type of a user. E.g. the machine must be administered by the user, the user must be allowed to install software on their own and the user must be skilled to do so. I've seen organizations where that is not a given, and I.T. would have to approve all the executed software. Or think of a remote deployment on a server. To aid the the role that is setting up the environment, it would be easier to yield ERROR, it sticks out and signals brightly that the tool is not completely ready. If it can't be fixed in the font, which is the part that the "user" is controlling usually it should not be a FAIL.

there's a precedent for yielding FAIL when a package is missing, and

Maybe that precedence should be fixed instead. Also, maybe we need a way beyond status messages to handle these cases. Configuration could be used to switch statuses or to exclude checks.

the check-run ends with a message that instructs users to report ERROR statuses.

That could be reworded easily.

I think this way there's a clear distinction between when a tool-run completes in an expected way and when it doesn't.

If have a function (a check) and I call it and it can't execute because a dependency is missing, how does that account to  completing in an expected way?

The benefit for users is that overtime they'll know that an ERROR result is something that's beyond their control, unlike all the other check results.

As I stated above, you require a very specific type of user/scenario that has control over that type of ERROR, for less powerful/enabled/skilled users, having a FAIL that they can't fix on their own in the font can be frustrating.
Consider another example, the conversation between a client and their foundry: the client requires by contract from the foundry to fix all FAILS (make *bakery *profile pass) but the incomplete system setup of the client can't be fixed by the foundry. I imagine a lot of emails going back and forth until the product manager on the client side has fixed their setup, requiring attention/work from everyone involved.

@graphicore Thank you for the input. Appreciated.

If have a function (a check) and I call it and it can't execute because a dependency is missing, how does that account to completing in an expected way?

By "expected way" I meant that a check yields a known code (e.g. fontval-not-available, fontval-returned-error) instead of a traceback dump.

@miguelsousa, this merged PR changes the previously established semantic distinction between the FAIL and ERROR status codes.  I agree that there are some aspects of the code-base that may have caused confusion regarding those meanings (such as that user message on the terminal reporter and that vendor-specific override) and I am willing to amend those to remove any remaining ambiguity.
It is my intention to minimize the differences between the Font Bakery repository and your personal fork of it, in the hope that we'll soon get back to collaboration on a shared repository. For that reason, I'd like to ask you to reconsider this change or, alternatively, to open an issue in which you could propose and discuss this semantic change with the wider user/developer community.

open an issue in which you could propose and discuss this semantic change with the wider user/developer community

Will do. Allow me some time to put it together.


Also get the project open in "pop-up" when using self defined menu.

I'm trying out this wonderful theme.
I love that in the example site that when you click on a project that it opens as a pop-up or overlay.
But actually when I'm using my self defined menu with a separate page for "Projects" I don't seem to get the popup.
So i.e. when on site localhost:1313/projects/ the projects are listed but when I click a project it doesn't open as a overlay pop-up. Instead it opens as a separate page as well http://localhost:1313/projects/photography/
Can someone help me out to adjust some code so that on http://localhost:1313/projects/ the projects can also appear as popups?
Cheers!

Hey @batch22,
Glad you like the theme. Does your site by any change has more projects then three?
Could you please try and comment out params.home.numberOfProjectsToShow in config.toml?

Hello @hanzei,
Thanks for you reply.
I did try your suggestion to comment out the line but there is no difference.
The site still has the basic content with 3 projects. In future I would love to populate to more projects.
Since it's a separate page popup would be awesome so vistors could just click to close the popup and remain in projects overview page so they won't have to browse 'back'.

Did you alter exampleSite in any way? If so, mind sharing this code?

No I didn't alter it. I just tested it with a new site with clean introduction theme.
created custom menu with separate /projects/ page and there it also happens not to pop-up.

Could you provide some easy to follow steps on how to reproduce this exactly?

Yes sure...
-You can quickly see reproduce it when you make new site in HUGO.
-Add clean introduction theme to themes folder.

and alter the example site, config.toml. then you can easily see it.
Here's the adjustment in the toml.
 #If you don't want to use the default menu, you can define one by yourself [[menu.main]]    name    = "Home"    url     = "/"    weight  = 0 [[menu.main]]    name    = "projects"    url     = "/projects/"    weight  = 1

After editing the toml you can cd to exampleSite dir:
hugo serve  --themesDir ../..
Then you can see so no popups are being used when using self defined menu

I'm still not able to repo this issue. Would you be open to share your site's code?

I appreciate your help but  I already skipped this project on Hugo.
When you test it by altering as mentioned above you could easily replicate the issue.
Cheers!


Create LICENSE

I copied the one from https://github.com/tarides/meio/blob/main/LICENSE.
Are you ok with that choice?

Great, thank you !


cannot place limit on update query mySQL

am trying to place a limit on an update query in pypika.
MySQLQuery.update(conversations).set(
            conversations.text, text).limit(1)

When I print the query I get
UPDATE `TEST_conversations` SET `text`=245

Is there any way I can set a limit on an update query in pypika??

This would need to be added. Are you able to raise a PR?

@maciek-d This should now be fixed in the above merged PR.


Messages cannot be parsed if Specification contains condition with Message'Size

A message is not parsed by PyRFLX if its specification contains a condition where a comparison with Message'Size is performed.
Consider the following example:
package Test is

   type T is mod 2**8;

   type Test is
      message
         A : T
            then B
               with Size => A * 8;
         B : Opaque
            then null
               if Message'Size = (A + 1) * 8;
      end message;
end Test;

It think the problem is that the expression (Message'Size = (A + 1) * 8) is not completely simplified into a BooleanTrue/BooleanFalse by __simplified(l.condition) inside the _next_field method.
That leads to a comparison between an expression.Equal and an expression.BooleanTrue inside the __eq__ method of expression.Expr, which returns NotImplemented.
The core problem is that Message'Size is not substituted with a concrete value (not in __type_literals nor in _simplified_mapping).
As no outgoing condition can be satisfied, the _next_field method returns an empty String, which leads to a KeyError when attempting to get the next field from _fields in the parse method.
The raised KeyError also contains no proper error message, only an empty String.

The core problem is that Message'Size is not substituted with a concrete value (not in __type_literals nor in _simplified_mapping).

Message'Size needs to be substituted with the sum of the size of all fields on this path. If there are multiple paths we have to create a disjunction of multiple subsitutions. In this concrete example the result needs to be Message'Size = A'Size + B'Size  which then should be resolved to A'Size + B'SIze = (A + 1) * 8.


remove/replace default --no-suggest composer option on 6.8 branch

Deployer version: 6.8.0
Deployment OS: Linux Ubuntu 20.04 / RHEL 8.6

https://github.com/deployphp/deployer/blob/6.x/recipe/common.php#L90
This is causing the following composer error:
[MY-ENV] > cd /path/to/my/deployment/releases/34 && /usr/local/bin/composer install --verbose --prefer-dist --no-progress --no-interaction --no-dev --optimize-autoloader --no-suggest
[app-prod.ce.berkeley.edu] < [error] You are using the deprecated option "--no-suggest". It has no effect and will break in Composer 3.

I realize you are working on a major 7 release, but I think this warrants a quick and easy patch to the 6.x branch with a new release.
Thanks for the awesome project!

Sorry but v6 is no longer maintained. You can override this config in your own recipe.


Stream without session continue to try to reconnect

If the session is lost, the stream try to reconnec in an infinte loop-
This will never be success since the X-session is wrong and it will waste our resources.

After 10 attempts, it should force exit.


Fixes for OpenJpeg/Jpeg2000

There are couple of things here:
First of all, make OIIO look for j2k libraries in the multiarch folder on linux
Second of all, it seems alpha wasn't handled totally correct as we believe. This in fact was already submitted and some initial discussion started in #943 but while preparing other patches i needed to update master branch, which closed the original pull request. Not sure i can point existing pull-request to a different branch, so made a new pull-request from a dedicated branch.
Would be cool to nail down what's the proper way to deal with alpha in j2k :)

Seems reasonable, LGTM.
This patch assumes that the only alpha you'll ever find in a JPEG-2000 file is "unassociated alpha", i.e. the colors are NOT already "premultiplied" by the alpha value. Can you confirm that this is your understanding of the J2K spec? Do you have any documentation that this is the case?

Oh, just noticed that now you are premultiplying the colors upon reading, but you have not altered jpeg2000output.cpp to do the inverse when writing -- that is, divide colors by alpha (like we do for PNG and other unassociated-alpha-only formats).
Without that change, our writer would not be producing output that our reader would correctly interpret, and vice versa.

If you don't want to bother, let me know, and I can pretty easily do the output part myself and commit both changes at the same time.

Yeah, don't worry about this. I can easily fix the writer.
Also, in looking this over, I have a number of additional fixes for JPEG-2000 beyond your patch. I will review those separately.

Ok, cool. Thanks for the reviews :)

I'm going to merge what you've done -- which is definitely a step forward -- and close the ticket. Be aware that the "jpeg2000" test will fail, because the writer and reader no longer match. But this will be temporary, I will submit a fix (and more!) hopefully later today.
Sergey, do you need the JPEG2000 improvements backported to RB-1.5, or is it sufficient to have it in master?

Awesome. And it should sufficient to have it in master.

More J2K improvements in #1097


refactor: enhance LLM configuration and initialization

BREAKING CHANGE: Complete overhaul of LLM configuration and initialization system
Core Changes:

Add dynamic model selection for all LLM backends
Implement availability checking for all LLM types

Features:

Add color-coded menu for LLM backend selection
Show setup instructions for unconfigured backends
Add real-time status checks for all LLM types
Implement proper error handling and user feedback

Configuration:

Clear Gemini API key
Reset Ollama model name
Add proper validation for model paths and API keys

Security:

Add checks for default/placeholder values
Improve error messages for missing credentials

Dependencies:

Add google-generativeai>=0.3.0 to requirements.txt

I thought I was modifying https://github.com/TheBlewish/Automated-AI-Web-Researcher-Ollama 🤣 only after I finished I did realize that I had modified an older project.
I wanted to add Gemini after seeing a Reddit comment saying that Gemini has a large context window and I realized he was right. I hope you can copy or get inspired 😢


When project delete confirmation popup appears focus doesn't switch to it - pressing Enter enters name change

Discord username
No response
What type of issue is this?
Permanent – Occurring repeatably
Is this issue blocking you from using Enso?

[ ] Yes, I can't use Enso because of this issue.

Is this a regression?

[ ] Yes, previous version of Enso did not have this issue.

What issue are you facing?
https://github.com/enso-org/enso/assets/108271897/672e3113-df75-4afb-b1c5-9ab19208e832
1st done with mouse - works
2nd done with keyboard - focus doesn't switch
Expected behaviour
After popup appears pressing enter should confirm the delete action
How we can reproduce it?
No response
Screenshots or screencasts
No response
Logs
No response
Enso Version
2023.8.17 nightly
Browser or standalone distribution
Standalone distribution (local project)
Browser Version or standalone distribution
standalone
Operating System
Windows
Operating System Version
Win11pro 22H2 22621.2134
Hardware you are using
12th Gen Intel(R) Core(TM) i9-12900HK / RTX3060 Laptop / Nvidia Drivers 531.68

note that the delete modal currently intentionally doesn't submit when pressing enter (this has always been the case)
but, the enter keypress definitely shouldn't be handled by anything else either. i think the best solution here is to handle the keypress on the delete modal and just have it do nothing - what do you think?
(alternatively we can just make enter confirm the dialog, as expected)

on second thought - maybe all keyboard events should be disabled when a modal is open? e.g. ctrl+r for renaming a project/folder?

on second thought - maybe all keyboard events should be disabled when a modal is open? e.g. ctrl+r for renaming a project/folder?

I believe this is correct behavior, except for maybe the shortcuts that are used in the modal. At least that's what I would expect as a user.

for now i'll just prevent propagation of all keydown events, except for Escape (which is the keybind used to close modals)

Fix implemented in #7602


Parsing xml data types - integer, booleans etc.

Hello,
I have xml fragment that looks like this:
<id type="integer">2</id>

And it parses to
:id=>{:type=>"integer", :value=>"2"}

I think parser should automatically detect data types like this, and parse automatically to:
:id=>2

Same with booleans and other data types (I think it already does this with arrays) fallbacking to value, type pair only if data type is not known. Let me know if you are interested in this kind of improvement.
Cheers

@michaldarda,
It's a reasonable ask.  However, I don't think it should be the default behavior since it's not backward compatible.
If you want to add option flag to turn this behavior on, I am open to merge your patch.
Thanks!

After many years I think we can close this issue.


[Refactor]  UI/restructuring/spinner

Signed-off-by: Yash Kamboj<EMAIL_ADDRESS>DescriptionMissing component in the new refactoring ui branch
Conflicted PR #5397
Previous PR #4828
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

@YashKamboj how are we taking this one forward?

@warunicorn19 good to go?

@warunicorn19 Done!!


optimizeOptimize to_string's performance.

Problem summary
When we call DataType::to_string(IColumn, row_num) for column_const, the code now would convert const column to full column. This is expensive and necessary. This pr removed it.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[x] Is this PR support rollback (If NO, please explain WHY)

run buildall

run buildall

run p0

run buildall


Improved error handling style: assert -> require

🚀 Description
Replaced assert convenience function by  require following Solidity documentation's advice.

Indeed, such exceptions can be reached even if the contract is correct, and are more a valid condition to be met.
This is not really an issue but a style improvement.

The assert function should only be used to test for internal errors, and to check invariants. The require function should be used to ensure valid conditions, such as inputs, or contract state variables are met, or to validate return values from calls to external contracts. If used properly, analysis tools can evaluate your contract to identify the conditions and function calls which will reach a failing assert. Properly functioning code should never reach a failing assert statement; if this happens there is a bug in your contract which you should fix.

http://solidity.readthedocs.io/en/develop/control-structures.html#error-handling-assert-require-revert-and-exceptions

[x] 📘 I've reviewed the OpenZeppelin Contributor Guidelines
[x] ✅ I've added tests where applicable to test my new functionality.
[x] 📖 I've made sure that my contracts are well-documented.
[x] 🎨 I've run the JS/Solidity linters and fixed any issues (npm run lint:all:fix).

Those asserts are invariants, and follow the documentation correctly :)
https://github.com/OpenZeppelin/zeppelin-solidity/issues/565

Correct me if I'm wrong.
Because SafeMath uses asserts then we should take care to verify input, i.e, using 'require'. This may lead in a double verification of inputs.

@shrugs I disagree that those asserts are invariants.
For example, take the sub code:
function sub(uint256 a, uint256 b) internal pure returns (uint256) {
    assert(b <= a);
    return a - b;
}

b <= a is not an invariant, since you can have pretty much any values reaching this point.
This code is pretty much filtering inputs, which is exactly Solidity's recommendation for require.

@leonardoalt
I guess the correct thing I should have said is is they "should be invariants". But obviously they aren't because nobody is doing requires before calling safemath anyway.
I've come around to your argument in #1120, so let's continue it there?


Parsing ITU-R RR (2020)

As I understand this is the identifier's document here: https://search.itu.int/history/HistoryDigitalCollectionDocLibrary/1.44.48.en.104.pdf
So it's a compilation of documents related to Radio Regulations. (also here https://www.itu.int/en/publications/Pages/publications.aspx?lang=en&media=electronic&parent=R-REG-RR-2020)
Identifier in URL: "R-REG-RR-2020".
So this is not a standard identifier, should we still parse it?

Since users can utilize these references we need to parse them.

@ronaldtse @andrew2net do we have other identifiers like this?

For “Regulatory Publications” there are other documents:
https://www.itu.int/pub/R-REG

Radio Regulations
RR Navigation Tool
RR5 Table of Frequency Allocations
Rules of Procedure
High Frequency Broadcasting Schedule

I can add to parser all this publication, but if they will add something else I'll need to update parser.
If we use something like “ITU-R REG-RR”, “ITU-R REG-RTX”, “ITU-R REG-RR5”, etc. I don't need to add all these publications to parser, and it can accept any symbols after “ITU-R REG-”.
@ronaldtse what do you think? Why do we decide to use “ITU-R RR” without “REG”?

Why do we decide to use “ITU-R RR” without “REG”?

It's not our decision so we cannot speculate... we just have to accept it and produce that as given.
We need to also be aware that ITU-D, ITU-R and ITU-T use different identifier patterns (even for the same document type).


Upstream DNS-over-TLS support?

As in allow leng to perform DoT queries to upstream resolvers, much like it does for DoH.
I think this is a nice to have only, as we are already capable of secure private DNS via DoH. DoT can be blocked and it is easier to spot (since it uses port 853). If we implement it, should be disabled by default I reckon

Documenting here that this is actually already the case when leng itself receives DoT requests


Rasa NLU is not recognising entities ?

Rasa NLU version (e.g. 0.7.3):
Used backend / pipeline (`["nlp_spacy", "tokenizer_spacy", "intent_featurizer_spacy", "ner_crf", "ner_synonyms",  "intent_classifier_sklearn", "ner_duckling"]):
Operating system (windows, osx, ...):
Issue:
text: The world's tallest restaurant in the Burj Khalifa offers what.
intent : restaurant_search and
entity should be  Burj Khalifa
I trained with single example with entity as Burj Khalifa many other examples tells the intent as restaurant_search.
I'm getting the correct intent but unable to identify the Burj Khalifa as the entity.do I need add more examples with the name Burj Khalifa as entity ?
I'm making use of ner_crf in config with other pipelines mentioned above.
Please suggest me a way to identify the entities like  Burj Khalifa using rasa NLU
thanks

I don't know any statistics about your training data, but I guess you need a couple more examples to properly train the NER model. These don't necessarily need to contain this specific entity as a value, but a couple more examples with it won't hurt.

thanks. I'm closing the ticket ! If any problem I will reopen it thanks

if there's no examples of "x-mas" as an entity, then yes you need to add some.

Then what ever entities I trained rasa is able to identify only those entities. Is it Right ?

Rasa can generalize beyond provided examples, though the examples have to be representative of a wide spectrum of possibilities. but in this case x-mas is too different from Christmas for it to generalize.
Let us know if you are still having trouble.

The same problem is what i am facing, i am not able to look for the entities that are not in the training data.

Hi @jainsaurav22 , I have the same problem. I am not able to look for the entities that are not in the training data. Were you able to solve it?


End-time shifts for klog today --diff --now as time goes by

Maybe I'm missing something, but I find it surprising that End-time shifts as time goes by, even after the open-ended range is closed.
Example output

15:09

             Total    Should     Diff   End-Time
Today        6h35m    6h25m!     +10m      14:59 # <-- this
Other       31h10m    32h5m!     -55m
          ======== ========= ========
All         37h45m   38h30m!     -45m      15:54 # <-- and this

15:11

             Total    Should     Diff   End-Time
Today        6h35m    6h25m!     +10m      15:01 # <-- ticked two minutes
Other       31h10m    32h5m!     -55m
          ======== ========= ========
All         37h45m   38h30m!     -45m      15:56 # <-- same here

15:12

             Total    Should     Diff   End-Time
Today        6h35m    6h25m!     +10m      15:02 # <-- ticked yet another minute
Other       31h10m    32h5m!     -55m
          ======== ========= ========
All         37h45m   38h30m!     -45m      15:57 # <-- dito

This is expected, but I agree it might not be intuitive, so the UI could potentially be improved.
The reasoning is this:

When using --now, klog behaves as if the ? placeholder was substituted with the wall clock time “right now”. If you use --follow additionally, then it keeps doing that as time goes on. That way, you continuously “consume” time, and move closer to your should-total goal.
If there is no open time range, then you don’t “consume” time, so the designated end-time moves forward with every minute that elapses – which makes sense, because you are not doing anything after all. E.g., if you have a time debt of 3h, then it keeps pushing forward the end-time, until you start an activity.

UI-wise, the output could make it obvious whether there is actually an ongoing activity (open time range) that keep “consuming” time. I’ll think about it, but if you have ideas how the semantics could be improved to be more intuitive, please let me know.

Ideas: Maybe there shouldn’t be any end-time at all, if there is no ongoing activity? Or it’s displayed in parenthesis, or with subdued color?

Still I find it weird to have an end-time forecast when the should-total have already been reached (or even surpassed).
I like your last suggestion!


feat(ModelZoo): rework prebuilt assets management

Description
ModelZoo artifacts changed from a single archive to one archive per model/backend combination. It allows users to only download needed archives, but prevents keeping the current design.
Change from all models being handled by the "neural_networks_provider" package to models being downloaded by packages that need them.
Leverage the newly added versioning of the prebuilt models.
Related links
Part of https://github.com/autowarefoundation/autoware.universe/issues/1879
Tests performed
Tested locally by building the TVM packages with --cmake-args -DDOWNLOAD_ARTIFACTS=ON and running the unit tests of those packages.
Notes for reviewers
https://github.com/autowarefoundation/autoware.universe/pull/1181 needs to be merged first.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Currently, it seems that neural_network_provider is only used by tvm packages while tensor_rt packages depend on trtexec_vendor. If we are not going to extend neural_network_provider its usage to cover tensor_rt, I think it makes sense to merge neural_network_provider into tvm_utility.

I think it makes sense to merge neural_network_provider into tvm_utility.

@angry-crab Done


[FR] Additional zoom options

The program's default zoom settings are generally fine for what I need but when I'm using my laptop, it would be nice to have some additional views.
Current behavior

Zoom to width
Zoom to height
The above two but with full screen enabled

** Requested behavior**

Optional scroll wheel zooming, ideally CTRL+scroll wheel to zoom in or out
Other custom or pre-set zoom settings, mostly for when images have a large side-border

I'll add these to my 'think about' list, but no promises :)

I want to keep the interface/options as minimal/simple as possible

While I think my idea is important but so is the next person and pretty soon you've got a really cluttered interface.  So that makes sense.  Feel free to close this issue if you decide against it.

I just uploaded another beta (2.1.0-beta2) that includes a new view, scale to height, allowing setting the zoom scale to one of the preset values or a custom one. Also, the view's zoom can now be increased or decreased from the menus or using 'ctrl++', 'ctrl+-', or 'ctrl+scroll wheel'.


Geometric Spread Backoff

Parodus basically uses a goemetric spread backoff algorithm like this:
inputs:
    max_time        time.Duration
    unit_duration   time.Duration
    skip_steps      int

skip_steps jumps ahead in the progression to limit the retries that are "close to zero" delays.
unit_duration is used as the multiplier for determining the range/new attempt time
max_time is used to crop the time window that is randomized over.  Don't truncate the value or you'll mess up the statistical magic of the randomness up.
The steps are:
0, 1, 3, 7, 15, 31, 63 ... (2n+1)
Try at time 0
Try at a random time between 1 and 3 units inclusively
Try at a random time between 3 and 7 units inclusively
Try at a random time between 7 and 15 units inclusively
Try at a random time between 15 and 31 units inclusively
....

How is this different from an exponential backoff?  I'm not seeing the benefit.


when render video , it will error

Describe the bug
A clear and concise description of what the bug is.
Loading latest checkpoint from load_dir
__ Done loading checkpoint from outputs/rewjj/nerfacto/2023-07-02_092345/nerfstudio_models/step-000029999.ckpt
Creating trajectory video
__ Rendering __ ________________________________________   0% ? --:--
Traceback (most recent call last):
File "/opt/conda/lib/python3.7/site-packages/scripts/render.py", line 147, in _render_trajectory_video
writer.add_image(render_image)
File "/opt/conda/lib/python3.7/site-packages/mediapy/init.py", line 1426, in add_image
if self._proc.stdin.write(data) != len(data):
BrokenPipeError: [Errno 32] Broken pipe
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/opt/conda/bin/ns-render", line 8, in
sys.exit(entrypoint())
File "/opt/conda/lib/python3.7/site-packages/scripts/render.py", line 354, in entrypoint
tyro.cli(RenderTrajectory).main()
File "/opt/conda/lib/python3.7/site-packages/scripts/render.py", line 347, in main
camera_type=camera_type,
File "/opt/conda/lib/python3.7/site-packages/scripts/render.py", line 147, in _render_trajectory_video
writer.add_image(render_image)
File "/opt/conda/lib/python3.7/contextlib.py", line 524, in exit
raise exc_details[1]
File "/opt/conda/lib/python3.7/contextlib.py", line 509, in exit
if cb(*exc_details):
File "/opt/conda/lib/python3.7/contextlib.py", line 377, in _exit_wrapper
return cm_exit(cm, exc_type, exc, tb)
File "/opt/conda/lib/python3.7/site-packages/mediapy/init.py", line 1390, in exit
self.close()
File "/opt/conda/lib/python3.7/site-packages/mediapy/init.py", line 1439, in close
raise RuntimeError(f"Error writing '{self.path}': {s}")
RuntimeError: Error writing '/workspace/nerfstudio3/renders/rewjj/2023-07-02_092345.mp4': Unrecognized option 'qp'.
Error splitting the argument list: Option not found
To Reproduce
Steps to reproduce the behavior:

Go to 'CUDA_VISIBLE_DEVICES=4 ns-render --load-config /workspace/nerfstudio/nerfstudio/scripts/outputs/r
ewjj/nerfacto/2023-07-02_092345/config.yml --traj filename --camera-path-filename /workspace/nerfstudio3/datasets/rewjj/camera_paths/2023-07-02_09
2345.json --output-path /workspace/nerfstudio3/renders/rewjj/2023-07-02_092345.mp4'

conda install ffmpeg
solve it


Settings for WD MyCloud

I have looked around and don't see anything on this. I am wondering if anyone has suggestions for configuring a WD MyCloud NAS to work with this extension?  I can connect to a Linux box using smbd just fine but I am always getting a failure (-100 error) connecting to my WD NAS.  I don't see any settings that jump out at me in the Windows share options on the NAS.  There must be something here I'm missing.  Any help would be appreciated.

No sooner had I posted this than I found the solution.  I'll leave it here in case anyone else has the same issue.  The WD MyCloud NAS defaults to SMBv2.   I had to downgrade to SMBv1 and then it started working.  Strange because the setting in the WD interface says "Max SMB Protocol" which leads me to believe it will support this or anything below.  Not sure if the extension is not negotiating down to SMB1 or the NAS is enforcing SMBv2.  In any case this solved the issue.


String interpolation example.

First off, great pod. Is there a string interpolation example that one might point me to? It doesn't seem to work when you just tack on localize() at the end of an interpolated string.

Figured it out. You have to do this:
 String(format: " • Range: %@ - %@".localized(), formattedLowPrice!, formattedHighPrice!)
If you interpolate variables, it will not work. I am not sure if this was intended but correct me if I am wrong.

@electic Thanks for mentioning, I suggest you add it the readme file.


play.spotify.com to player.spotify.com

Spotify now redirects 'play' subdomain to 'player'

For me if I go to play.spotify.com it's stays in that subdomain. If I go to player.spotify.com it redirects me to play.spotify.com.
So just to make sure, when you go to play.spotify.com it redirects you to player.spotify.com? If so, does the page still look the same as before?
I suppose it's something country dependent? If you don't mind can you tell me your OS and country? (So maybe I can find a VPN and try it).

I am on win7 in the usa

On May 31, 2015, at 4:56, "Paco Hobi"<EMAIL_ADDRESS>wrote:
For me if I go to play.spotify.com it's stays in that subdomain. If I go to player.spotify.com it redirects me to play.spotify.com.
So just to make sure, when you go to play.spotify.com it redirects you to player.spotify.com? If so, does the page still look the same as before?
I suppose it's something country dependent? If you don't mind can you tell me your OS and country? (So maybe I can find a VPN and try it).
—
Reply to this email directly or view it on GitHub.

Sorry for the delay (have been busy with college). I'm not able to replicate your problem, but I tried something that I hope that fixes the issue.
Download this version of the add-on, and if it fixes it I will publish it as a official update:
https://dl.dropboxusercontent.com/u/1404483/currentsong.xpi
Again, sorry for the long wait.

I have included the fix with release v0.2.7.


支持mybatis @SelectKey注解

mybatis插件回放支持@SelectKey注解，讲录制的id回填给insert参数对象

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Peng HU1 胡鹏 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: avoids eslint and prettier conflict in Guardian UI

Adds prettier to the eslint config so they do not conflict.

gateway/ui could use a similar fix: https://github.com/fedimint/fedimint/blob/master/gateway/ui/.eslintrc.json


⚠ Bofrost.fr has degraded performance

In 5f1eb53, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1529 ms

Resolved: Bofrost.fr performance has improved in 612151b after 7 minutes.


python310Packages.MDP: fix tests

Description of changes
Fix build (https://hydra.nixos.org/build/209282768)

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 217923 run on x86_64-linux 1

2 packages built:

python310Packages.MDP
python311Packages.MDP


Add matrix product example

This PR adds a new example that synthesizes a program which multiplies a 2x2 matrix with a 2d vector and returns 2 times the element-wise sum of the product vector.
Tested and verified to find the correct synthesis out of 3 possible outputs.

Checked to ensure FnDecl renaming didn't affect this example


PushNotificationIOS.cancelLocalNotifications(userInfo) not working

Here is the code to reproduce the bug:
          PushNotificationIOS.presentLocalNotification({
            alertBody: 'simple alert',
            soundName: 'default',
            alertAction: 'view',
            userInfo: {id: 1},
          });

...

          PushNotificationIOS.cancelLocalNotifications({id:1});

The notification is not removed after calling cancelLocalNotifications
react native 0.29.0
ios 8.2 & ios 9.3.2

I couldn't reproduce this.


Units of measurement for 3D structure

Hi.
I am trying to use these datasets for contact prediction. A contact is defined as a pair of residues within 8A of each other. However, when I compute the pairwise distances of residues for proteins in this dataset, they are typically in the thousands and none are 8A away from each other. What are the units used?
Thanks

Hi, sorry about the late response. I missed this somehow. The units used are picometers.


Show partial hierarchy for selected term, allow switching for a broader/narrower/related term

Example from http://www.ukoln.ac.uk/projects/enhanced-tagging/demonstrators/:

This tries to solve the same problem as #11 Full-size term selector and we don't have to do both.

One idea would be to replace the term options menu with a richer pop-up, hosting the options as well as a browser where selecting a broader/narrower/related term will add/replace it in the search.


[feature] Add perishable tuple (expiration date time)

Hello There,
We have a use case where we want to create a tuple but for a given amount of time.
Would it be something that openfga could support (maybe it deserves an RFC)?
Regards,
Yann

Thanks for raising the issue @le-yams!
We are working on this as part of our improved ABAC support https://github.com/openfga/roadmap/issues/32
That would allow you to add context to tuples and add conditions that would evaluate the context in order for a relationship to be valid.
Timestamps will be one of the possible values to context, and you can add a condition that would return true if the period since that timestamp is greater than some value.

Great news! And sorry I missed that.
Thanks a lot  :)

You can find a very early demo of this work here: https://youtu.be/sFqk42fJy_E?t=665
Please let us know if you have any feedback!


Using a ciggie to interact with a thruster should light it

From @DogZeroX

Sergal

Isnt this in already? I tried using an unlit cigarette on a thruster, turned on, and it was lit.
should this be closed?


GistPad not fully functionnal when used via authenticated proxy

When using GistPad on an enterprise network that requires an authenticated proxy connection, GistPad will properly login (with token) and load the list of my Gist but will not display them.
E.g. list:

But attempting to load example.ps1 fails with this error in the console:

Curiously most of the time I only get the loading indicator without any error, like if the request was sent WITHOUT going thru the proxy.
VS Code works properly, will update and download extensions via proxy etc.

Any updates on this? I'm facing a similar error when trying to use GistPad from within my companies network.
The editor could not be opened due to an unexpected error: Request failed with status code 500


How to prevent crashing of wallet.dat ?

When I run my premine-qt application on Ubuntu 14.04, it always give me Error message. I don't know what is the problem every time I am getting this and I'm very much frustrated with that. Actually I developing new coin. How I can resolve this.
and error is
"Error initializing block database.
Do you want to rebuild the block database now?"

It sounds like you are talking about PTS--which is obsolete has been upgraded to DPOS: https://github.com/PTS-DPOS/PTS
There is no mining in BitShares.


FileNotFoundException: Mono.WebAssembly.Interop

I followed the instructions regarding using MatBlazor for server-side deployment and tried using the Tooltip component, however when I test the site I get the following error:
FileNotFoundException: Could not load file or assembly 'Mono.WebAssembly.Interop, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'. The system cannot find the file specified.
And the stack trace shows the error is occurring on the line @(await Html.RenderComponentAsync()).  In order to get rid of the error, I had to comment out both the tooltip component markup and the /app.UseEmbeddedBlazorContent(typeof(MatBlazor.BaseMatComponent).Assembly); line in the Startup.cs file.

@dmarciano, this issue is very old. Please comment / re-open if you are still having issues.


Improved connection paths

First of all: Thanks for this great library.
I have very crowded networks, where it can get hard to see which node is connected to which other node. I think this is basically due to the way connections are drawn.
To demonstrate my point, I connected two nodes and draged one of them around:

With this pull request, the connection path now starts (almost) perpendicular to the nodes side and then starts to smooth out leading to:

It even works, if inputs and outputs are on the same side:

This may be too much of a change to directly push it to the library, so if you would like an exchangable approach, where the previous and this approach can be exchanged, just let me know.

With the second commit I removed redudant points. The original creation of the PathFigure and the one from the previous commit, had doubled startPoint and doubled mid point. It should look a little smoother and more symmetric now.

Hi, thanks for working on this! I've tried it out, and this does indeed seem to give a more clear view of connections that take unconventional bends.
However, when the connection is fairly straight and short, the new connections take zig-zag bends.
Current system:

Proposed system:

In this situation, I do find the current look to be more clear. I haven't looked at the code in detail yet, but you can probably fix this by scaling the 'gradient' distance based on the distance between the endpoints, as well as relative orientation of the endpoints.

In this situation, I do find the current look to be more clear. I haven't looked at the code in detail yet, but you can probably fix this by scaling the 'gradient' distance based on the distance between the endpoints, as well as relative orientation of the endpoints.

There will be a distance where this zig-zag cannot be avoided to provide the visual feedback when the width is basically zero:

However I can reduce it so that it is not so annoying:

Looks good, thanks!


OPENJSON (Transact-SQL) claims Azure still defaults to Compat 120

Fromm OPENJSON (Transact-SQL):

Compatibility level 120 may be the default even in a new Azure SQL Database.

According to this article, from way back in August 2017, this is no longer true:

As of today [28 August, 2017], the default compatibility level for new databases created in Azure SQL Database is 130. Very soon, we’ll be changing the Azure SQL Database default compatibility level for newly created databases to 140.

I can't see any other documents that make this exact claim (that doesn't mean there aren't any), however, T-SQL differences between SQL Server & Azure SQL Managed Instance certainly agrees that 140 is the default and the SQL Azure Database(s) I have agrees with that.

@LarnuUK -- Thom, thank you for your well-researched feedback.
@jovanpop-msft -- Jovan, please look into this issue.

Thanks for bringing this issue to our attention, @LarnuUK! The current compatibility level is 150. I'll remove that text from the doc.

#reassign:VanMSFT


add Compat back to REQUIRE

it is still used by the package so it still needs to be required unless all uses of it are removed

Coverage remained the same at 68.008% when pulling 01f6cc0b97d4b9fc0dd470af1e68776c53fe25ca on tk/Compatreq into 92f47e4647ebd6eb5feec06f2ea46b430a1e7c66 on julia-0.4.

[Current coverage][cc-pull] is 68.01%

Merging [#20][cc-pull] into [julia-0.4][cc-base-branch] will not change coverage

@@           julia-0.4        #20   diff @@
===========================================
  Files              6          6
  Lines            472        472
  Methods            0          0
  Messages           0          0
  Branches           0          0
===========================================
  Hits             321        321
  Misses           151        151
  Partials           0          0

Powered by Codecov. Last updated by [92f47e4...e0e7f40][cc-compare]
[cc-base-branch]: https://codecov.io/gh/JuliaLang/Combinatorics.jl/branch/julia-0.4?src=pr
[cc-compare]: https://codecov.io/gh/JuliaLang/Combinatorics.jl/compare/92f47e4647ebd6eb5feec06f2ea46b430a1e7c66...e0e7f40f87d30a3446774d473bf747553000f25a
[cc-pull]: https://codecov.io/gh/JuliaLang/Combinatorics.jl/pull/20?src=pr

Bump. It would be awesome to get this merged since HypothesisTests will need to use this package.


Error adding predicates after actions

The system seems to produce an error Action client execution error applying at_end effects if you apply a predicate that is already true. This can easily be seen with the terminal tool and the simple example (simple_example.pddl). From the tool, type set predicate (robot_at leia entrance) twice. The second time produces the error Predicate not valid. If this happens during the actual execution of the plan, the execution will abort. Is this intentional? Is there any way to allow true predicates to be set anyway?

Hi @dr-steveb
That behavior is not correct and I thought I had fixed it some time ago...
I can fix it, but not immediately. Meanwhile, if you want to fix it, a PR is welcome ;)
Best

Hi,
Which branch should I be looking at for this? Note that I am using Foxy. Happy to fix if I can.
Best,
Stephen
From: Francisco Martín Rico<EMAIL_ADDRESS>Sent: Tuesday, December 8, 2020 11:33 AM
To: IntelligentRoboticsLabs/ros2_planning_system<EMAIL_ADDRESS>Cc: Balakirsky, Stephen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [IntelligentRoboticsLabs/ros2_planning_system] Error adding predicates after actions (#38)
Hi @dr-stevebhttps://github.com/dr-steveb
That behavior is not correct and I thought I had fixed it some time ago...
I can fix it, but not immediately. Meanwhile, if you want to fix it, a PR is welcome ;)
Best
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/IntelligentRoboticsLabs/ros2_planning_system/issues/38#issuecomment-740744422, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB4BN2UBINGWDEJD6YTCJWDSTZIJ7ANCNFSM4UN4KSMA.

Hi @dr-steveb
The development should be always done in the master branch. Foxy or other distribution-specific branches are used only to deliver packages.
Thanks!!

Created a bug fix branch off of master for this and fixed issue. However, I can't push the new branch to the repo. How do I get permissions?

I don't know if you used to follow the usual open-source contribution procedure. If not, these are the steps:

Fork PlanSys2 repo (this repo) in your account
Clone your repo (or add a remote) in your computer.
Create a branch with your contribution.
Push this branch to your forked repo.
Do a Pull Request (PR) to PlanSys2 repo.
Then we could start talking, reviewing, ...
When it is finished, I will merge your PR in the PlanSys2 repository.

@dr-steveb I am closing this, as fixed in #51

@dr-steveb I am closing this, as fixed in #51


Transforms: Renderer transform origin should be centered w.r.t. current variable

I believe that when we apply transforms to individual renderers, the origin should be the center of that variable's extents.
To reproduce:

Spin up Vapor, load KatrinaMaster.nc
Create a 2D renderer and enable it
In the Geometry tab, scale the renderer by 2.  The rendering is no longer centered within the domain

Stas, I looked over the code and I think setting a more reasonable default origin for the transforms is pretty straight forward for both the renderer transform (on the Render/Geometry subtab), and the data set transform (Navigation/Viewpoint) tab.
For the renderer transform you just need to set the origin on the _transform member inside the RenderParams constructor. Note: the RenderParams constructor is overloaded and only the one with the signature:
RenderParams::RenderParams(
DataMgr *dataMgr, ParamsBase::StateSave *ssave, const string &classname,
int maxdim
);
should be changed. You can simply use the _Box class to find the center of the domain
For the dataset transform it's a little more tricky. Your best bet may be to modify the ViewpointEventRouter::updateTransforms() method so that the first time it sees a new transform (dataset name) it sets the origin of the transform returned by vParams->GetTransform(names[j]) The tricky part is picking a good value. The DataStatus class has a method:
DataStatus::GetActiveExtents() that returns the union of the extents of all of the "active" variables. However, the current version does this for ALL data sets. You would need to implement an overloaded version that allows the specification of a particular data set.
This should work! Let me know if you have questions.


Add docker configuration

Description
This enables you to optionally run your database in Docker.
Fixes # (issue)
How to test?

Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have merged the latest version of the source branch (typically develop) into this branch before pushing my changes
[x] This PR is ready to be merged and not breaking any other functionality

Great 👍😃
Whats not to like about docker


[SPARK-5553][SQL] Replace Array[Byte] with Java NIO ByteBuffer as binary type representation

JIRA: https://issues.apache.org/jira/browse/SPARK-5553
This pr attempts to replace Array[Byte] with Java NIO ByteBuffer as SQL binary type representation.

Test build #30852 has started for   PR 5666 at commit d77990f.

Test build #30852 has finished for   PR 5666 at commit d77990f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #30858 has started for   PR 5666 at commit 68c9b00.

Test build #30858 has finished for   PR 5666 at commit 68c9b00.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Hey, we probably do want to do this at some point, but I'm not sure the answer is ByteBuffer.  Big changes like this should be proposed in JIRA and discussed before coding begins.  Since we are close to the merge deadline for Spark 1.4 it is unlikely this patch will be merged anytime soon.

Thanks for submitting this. It's our fault to have that JIRA ticket there and suggest nio.ByteBuffer.
ByteBuffer is not the right abstraction for this. It's API is pretty clunky (flip, etc), and cannot be serialized. We will do something as part of https://issues.apache.org/jira/browse/SPARK-7075

ok. I'm closing this.


Improve lodash imports to reduce size of serverless bundles

Summary

Serverless is becoming popular and bundle size is important. This PR avoids importing the whole lodash code, even in the server.
Test plan

Tests pass for ChunkExtractor.js, the only affected file.

I like that, thanks @luisherranz!

That was fast. Thanks @neoziro! :)

@bitttttten it works if you build your bundle using webpack or ncc but I agree that maybe adding them as packages may cover more cases.

Ohh I see. I am not using webpack to bundle my lambdas! To be honest I didn't ever think of doing that. I guess that answers my question, thanks!
If you bundle your lambda with webpack, then wouldn't you loose all the niceties of the console editor? Or does it some how keep all that?

I'm not familiar with AWS because I use other service. If code don't need to be bundled, then maybe it's not a good idea to make it necessary if someone is using loadable component.
Could you please describe what is the normal workflow in AWS?

use webpack alias to resolve dependencies

use webpack externals to keep node_modules in node_modules

Can you run require on a lambda? And require node modules on that side?

Of course, you can. Majority of lambdas(they were not created just for SSR task) are written in pure nodejs, and haven't ever experienced any bundler.


Add test_i32x4_swizzles

This raises the test coverage in SIMD from 10.04% to 10.27%, just a bit.
@pcwalton, I see there is a comment about completing I32x4 swizzles, do you want me to do that? In this commit I'm only testing the 4 that exist.

@luisbg Sure, feel free to complete them if you'd like.


Set estimated destination time to 12 hrs for all transfers with Ethereum destination

Users are expecting 5 hours. Current latency is around 12 hours
We should set Estimated Time to <12 hours for all transfers (any asset) to Ethereum destination

https://github.com/connext/frontend/pull/55

@alexwhte could you confirm if this is for all transfers to Ethereum including fast/slow or just slow?

updated for slow only

sorry missed that detail

:+1: Updated!

See deploy here: https://frontend-connext-bridge-git-sm-sof-6128-9b02d3-connext-979dc950.vercel.app/EZETH-from-arbitrum-to-ethereum?amount=1


Error rejecting Domestic Payment Consent

The UI displays an error saying "Bad Request" when rejecting a Domestic Payment Consent.
Nothing is logged in the RCS to indicate what the issue is.
In the IG logs we see:
{
"Code": "BAD_REQUEST",
"Id": "8427891f-daea-405c-9776-3469d6a4f014",
"Message": "Bad Request",
"Errors": [
    {
        "ErrorCode": "OBRI.RCS.consent.decision.Invalid",
        "Message": "Invalid remote consent decision token. Invalid format"
    }
}

TO DO Test other consent types

When the consent is rejected by the PSU user, currently the debtorAccount value is "" if the user have not select any account when rejects the consent.
Example decision submitted
{
    "consentJwt": "eyJ0eXAiOiJKV1QiL....",
    "decision": "Rejected",
    "debtorAccount": ""
}

in the RCS backend the jackson mapper deserialize the json payload submitted from RCS-UI including empty properties, and fails because the property debtorAccount defined in the object ConsentDecisionDeserialized is defined as  FRFinancialAccount type, jackson try to deserialize it as String raising an exception.
To prevent that issue and never submit the debtorAccount as "" (empty string) the UI consents submit event has been modified for all payments to set the debtorAccount always as null when the consent is rejected by the user.
The debtorAccount can be ignore when the consent is rejected, the consent no needs to be patched with the accountId selected by the user when the consent is rejected.
{
    "consentJwt": "eyJ0eXAiOiJKV1QiL....",
    "decision": "Rejected",
    "debtorAccount": null
}

So that the property debtorAccount will be ignored by jackson when deserialize the payload.


apply NIU cookiecutter

Description
What is this PR

[ ] Bug fix
[ ] Addition of a new feature
[x] Other

Why is this PR needed?
For consistency, we'd like to adhere to the NIU cookiecutter template
What does this PR do?

no changes to Python code functionality
pure refactor of the package structure and CI to adhere to the new tempalte

References
Closes #11
How has this PR been tested?
tests ported to reflect name and structure changes
Is this a breaking change?
Not for the user, but very breaking for developers
Does this PR require an update to the documentation?

High-level docs still TODO
existing docstrings unchanged.

Checklist:

[ ] The code has been tested locally
[ ] Tests have been added to cover all new functionality (unit & integration)
[ ] The documentation has been updated to reflect any changes
[ ] The code has been formatted with pre-commit

I will update this PR with changes to actual code from #54 before marking as ready-to-review.


[e2e]: generic workflow_dispatch main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6464266510
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Tue Oct 10 03:20:08 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6477514220
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Wed Oct 11 03:20:25 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6490787998
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Thu Oct 12 03:20:13 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6503566595
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Fri Oct 13 03:19:47 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6515239032
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Sat Oct 14 03:19:54 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6521651080
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Sun Oct 15 03:19:08 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6528537156
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Mon Oct 16 03:19:51 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6542053242
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Tue Oct 17 03:46:06 UTC 2023
Tests are passing now. Closing this issue.


Gallery: State Flutter's status and link to our website

We should state the status Flutter is in (tech preview) in the Gallery app, as well as link to our website (triggering an intent to open a browser?).
This way, if someone builds and deploys the gallery, users that stumble across it will know what they are looking at.
One idea is to add this info to the drawer of the Gallery.
"Flutter is an early-stage, open-source project to help developers build high-performance, high-fidelity, mobile apps for iOS and Android from a single codebase. This gallery is a preview of Flutter's many widgets, behaviors, animations, layouts, and more. Learn more about Flutter at https://flutter.io."

cc @mpcomplete @HansMuller

cc @tvolkert who is waiting for this :)

Add an "about" dialog to the pull-out drawer?

I'm working on the about dialog.

This could also address https://github.com/flutter/flutter/issues/4584

I agree. Will plug this hole today :-)

@HansMuller: Ping. An example of connecting to the URI launcher https://gist.github.com/chinmaygarde/d778498418e30825d687147b79d070eb.


failed to rewind wordlist: seek /dev/fd/11: illegal seek

gobuster dns -d https://example.com -w <(curl -fskSL bit.ly/dsstorewordlist):
===============================================================
Gobuster v3.5
by OJ Reeves (@TheColonial) & Christian Mehlmauer (@firefart)
===============================================================
[+] Domain:     https://example.com
[+] Threads:    10
[+] Timeout:    1s
[+] Wordlist:   /dev/fd/11
===============================================================
2023/04/29 12:17:26 Starting gobuster in DNS enumeration mode
===============================================================
2023/04/29 12:17:26 [-] Unable to validate base domain: https://example.com (lookup https://example.com: no such host)

Error: error on running gobuster: failed to rewind wordlist: seek /dev/fd/11: illegal seek

pipe curl into gobuster and use -w - to use stdin as a wordlist


RichEmbed doesn't allow > 25 fields, but embed object does?

Please describe the problem you are having in as much detail as possible:
RichEmbed is not allowing me to create anything above 25 fields, but when I create an object with 25+ fields, it works.
Include a reproducible code sample here, if possible:
//// I use "m" as message.
// RichEmbed
let embed = new Discord.RichEmbed();
embed.setTitle = "Your Current Permissions";
embed.setDescription = "These are the permissions you currently have."
let perms = m.member.permissions.serialize();
for (let perm in perms) {
  let theP = perms[perm];
  if (theP) embed.addField(perm, "✅", true);
  else embed.addField(perm, "❎", true);
}
m.channel.sendEmbed(embed);
// Output: RangeError: RichEmbeds may not exceed 25 fields.

// Object
let embed = {"title": "Your Current Permissions", "description": "These are the permissions you currently have.", "fields": []};
let perms = m.member.permissions.serialize();
for (let perm in perms) {
  let theP = perms[perm];
  if (theP) embed.fields.push({"name": perm, "value": "✅", "inline": true});
  else embed.fields.push({"name": perm, "value": "❎", "inline": true});
}
m.channel.sendEmbed(embed);
// Output: [object Promise] (It sends successfully)

Further details:

discord.js version: v11.0.0
node.js version: v7.8.0
Operating system: Ubuntu v16.04
Priority this issue should have – please be realistic and elaborate if possible: Low

[x] I have also tested the issue on latest master, commit hash: 930519c494f316e1cc709aa3afc140ec0a36491d

Did you count the number of fields?

@iCrawl 29

Found out it was cutting it short.


[FEATURE] Add configuration from plugin download timeout

Description
Add configuration from plugin download timeout
Why is this needed?
Poor connection quality or some issues in a server side could generate timeouts.
Additional Information
Configuration for timeout in jenkins_plugin: https://docs.ansible.com/ansible/latest/collections/community/general/jenkins_plugin_module.html#parameter-timeout

@ommarmol, could you update the changelog, please? :)

@ommarmol, could you update the changelog, please? :)

Sure! ;)


Add demo button

Add the demo button to the loginpage

@ntorsen This is where the demo button was added to Mad-expo-core for https://github.com/equinor/mad-project-hearing-test-client/issues/84. For now only Hearing test uses this build of Mad-expo-core

Demo button is added, and I'm able to click on it and get into demo version 👍


Adding Deepinfo

Hi, it's a good idea to add Deepinfo, which provides extensive cybersecurity data.

Thanks!


"Run Python File in Terminal" missing

Issue Type: Bug
"Run Python File in Terminal" will usually disappear from the right click menu for a bit and will magically appear after some time.
https://imgur.com/a/h4SEX3S
VS Code version: Code 1.30.1 (dea8705087adb1b5e5ae1d9123278e178656186a, 2018-12-18T18:12:07.165Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-3570K CPU @ 3.40GHz (4 x 3503)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
23.94GB (18.95GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

vscode-great-icons
emm
2.1.46

code-runner
for
0.9.5

theme-monokai-pro-vscode
mon
1.1.9

python
ms-
2018.12.1

code-settings-sync
Sha
3.2.4

(3 theme extensions excluded)

Is this after reload? Is this with new files? Is this with the first python file you open?

I get this whenever I first start VScode, and it usually ends up appearing after like 20-30 minutes by itself. Sometimes I have to open up settings and then go back to my file for it to come back. But it is pretty consistent whenever I first open it up it's not there.

/duplicate of #65041