Cannot use usage commands

Hello, sorry if this is quite the 'noob' issue but I can't seem to type into the executable or figure out how to actually use the options provided on the main page.
I am running the "maniac-win64-static-debug.exe" version because the others return the "The code execution cannot proceed because libwinpthread-1.dl was not found. Reinstalling the program may fix this problem." error.
If you could instruct me on how to use these options that would be useful for lots of testing I'm doing.
Thank you! & Good work.

The executable is designed to be run from the command line and takes arguments like any other command line program.
To run it with a humanization level of 100 and a replay humanization delta of 5 you would do ./maniac-win64-static-debug.exe -l 100 -r 5 (screenshot)


docs: added reference to create commit message

Closes https://github.com/kawalcovid19/wargabantuwarga.com/issues/644
Description
it will help new contributor especially new developer to understand creating good (and accepted) commit message
Current Tasks

[x] update docs

@all-contributors please add @ryanadhi for doc.


fix: initialize editor with empty paragraph if root is empty

FX-4190
Description
When { root: { children: [] } } is provided as a default value to the RichTextEditor, the internal editor state is initialized without an empty paragraph. According to Lexical docs, this is not a valid state, there should always be at least one paragraph node by default.
This causes the editor to crash when trying to append items to the root node (for example, creating an empty list in editor with an empty default value).
How to test

Temploy
Story for RichText can be used for testing
Change the HTML story example code to have the default value of root: { children: [] }
Without doing anything else, click on any list button in the editor

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@TomasSlama it looks like we have previously not been checking for empty value in the correct way at all.
root.isEmpty() is always true because value is not yet converted and appended to the root node in Lexical. I've updated the example to check for value instead.


kwm doesn't seem to tile borderless iterm windows

To illustrate the issue I'm having I posted a screenshot of overlapping windows that should be tiled.
I'm fairly new to kwm so this may be an issue with how I am using it.

This is #40 — check the kwmc config add-role AXDialog iTerm2 line in your kwmc, maybe change the iTerm2 to iTerm?

In regard this topic, I also have a doubt: If I place that code in my
kwmrc it tiles also my 'visor' like hotkey iTerm, so... there's a way to
have untitled iTerm windows be tiled BUT the Hotkey window?
Eden S.-G. wrote:

This is #40 https://github.com/koekeishiya/kwm/issues/40 — check the
|kwmc config add-role AXDialog iTerm2| line in your kwmc, maybe change
the |iTerm2| to |iTerm|?
—
Reply to this email directly or view it on GitHub
https://github.com/koekeishiya/kwm/issues/187#issuecomment-189021101.

--

Andrea Spada :: Liuteria d'Autore
www.andreaspada.com

@EdenSG could you further elaborate? My kwmc ~/.kwm/kwmc is non-existant. I tried creating and re-adding that line but it didn't change anything.

@Nolski you'll need to make one then. Copy the example one from here.

@EdenSG Is that path correct? Even when I created one and ran kwmc config reload I didn't notice any change in functionality. I've been looking around for some docs to troubleshoot this but I haven't found any yet.

The config file for kwm is: $HOME/.kwm/kwmrc (short for 'kwm runcom')
kwmc is a tool to configure Kwm during runtime, and is also the prefix-string used in the configuration file to let Kwm know what kind of command is on that line.

@Nolski You shouldn't notice a change — correct me if I'm wrong, but by default kwm uses the default kwmc file.

@koekeishiya Thanks! Adding kwmc config add-role AXDialog iTerm2 to my $HOME/.kwm/kwmrc solved the issue and opened my eyes to a bunch of the configuration abilities available in kwm. :100:

I will hopefully be writing more detailed information on the wiki during the weekend

I had to use kwmc config add-role AXUnknown iTerm -- the Accessibility Inspector showed it as AXUnknown instead of AXDialog (which I tried first). I believe I am using iTerm 1 (it doesn't say iTerm 2)


need help with background borders - not working for me

Have added background borders to the demo file form.html using the following:
$('.image-editor').cropit({
  	imageBackground: true,
  	imageBackgroundBorderWidth: 15
});

However, borders do not display.  What am I missing?

$('#image-cropper').cropit({
change your ID to image-cropper instead of using an image-editor class.

Thank you so much.  My bad thinking the name could be changed.  Not smart on my part.


Add Test Runs using Http Fault Injector

Description
Adds the ability for tests to run with HTTP fault injecting functionality to simulate scenarios where the network is unstable.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - appconfiguration - tests

API change check
API changes are not detected in this pull request.

/azp run java - appconfiguration - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

@mikeharder, I've disabled running HTTP fault injector on macOS while a solution for it preventing requests to the HTTPS port is being found.


⚠ dev-deces.matchid.io search API has degraded performance

In 8341304, dev-deces.matchid.io search API (https://dev-deces.matchid.io/deces/api/v1/search?q=pompidou) experienced degraded performance:

HTTP code: 200
Response time: 8517 ms

Resolved: dev-deces.matchid.io search API performance has improved in 3d9ea5b after 29 minutes.


USBIP attached device gets disconnected after a few seconds

I'm trying to pass a USB device from macOS to Linux. It's disconnecting after about 5 seconds. On the macOS host/server, I run env RUST_LOG=info cargo run --example host
The client is running Debian 11 and usbip:

Package: usbip
Version: 2.0+5.10.136-1
Priority: optional
Section: admin
Source: linux (5.10.136-1)
Maintainer: Debian Kernel Team<EMAIL_ADDRESS>
On the client, I run sudo usbip list -r <IP_ADDRESS> (this is the local IP address of the mac), and I get the full list of USB devices exposed by usbip server implementation on the mac.
At this point, I get some info logs on the server that seem to be pointing to some issue, although the operation on the client did appear to complete successfully:
[2022-09-06T22:45:56Z INFO  usbip] Got connection from Ok(<IP_ADDRESS>:37120)
[2022-09-06T22:45:56Z INFO  usbip] Handler ended with Err(Custom { kind: UnexpectedEof, error: "early eof" })
Then I attach a device. In my case:
sudo usbip attach -r <IP_ADDRESS> -b 3-16-2
At this point, I get some success logs from the server, and the client shows the attached device if I run sudo usbip port:
Server:
[2022-09-06T22:49:34Z INFO  usbip] Got connection from Ok(<IP_ADDRESS>:58240)
[2022-09-06T22:49:34Z INFO  usbip] Found device "/sys/bus/3/16/2"
After about 5 seconds, the server throws the same error message from earlier:
[2022-09-06T22:49:44Z INFO  usbip] Handler ended with Err(Custom { kind: UnexpectedEof, error: "early eof" })
At this point, the device is no longer attached to the client.

Following up, I found a different behavior with different devices:
 3-16-2: Sony Corp. : DualSense wireless controller (PS5) (054c:0ce6)
       : /sys/bus/3/16/2
       : (Defined at Interface level) (00/00/00)
       :  0 - Audio / Control Device / unknown protocol (01/01/00)
       :  1 - Audio / Streaming / unknown protocol (01/02/00)
       :  2 - Audio / Streaming / unknown protocol (01/02/00)
       :  3 - Human Interface Device / No Subclass / None (03/00/00)

This is the "problem device." I don't know if you might have any intuition about macOS doing something goofy with game controllers that's interfering with the connection or if there might be workarounds.
I did a passthrough on this device and it is able to maintain the attached USB device without the server throwing any errors (it'll stay connected even if I also attach the game controller, when the game controller gets disconnected)
  3-9-4: Blue Microphones : Yeti Stereo Microphone (b58e:9e84)
       : /sys/bus/3/9/4
       : (Defined at Interface level) (00/00/00)
       :  0 - Audio / Control Device / unknown protocol (01/01/00)
       :  1 - Audio / Streaming / unknown protocol (01/02/00)
       :  2 - Audio / Streaming / unknown protocol (01/02/00)

I think the error UnexpectedEof is due to TCP connection closing.

is there something I can do to resolve it, or get additional logs?

@Aussiedler  You need to setup Rust development environment.

I suspect this issue is caused by reading too much data (> than the ep max packet size) of the usb ip device.
This commit: https://github.com/jiegec/usbip/pull/8/commits/ee51a4003b95ef54400aa6990ce2b05823f03967 fixed the instability with my device.

Please try again, and if the issue persists, you can reopen the issue.


add check_no_self_qualified_accesses

closes https://github.com/ericphanson/ExplicitImports.jl/issues/55

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.84%. Comparing base (911b915) to head (e7e08f5).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #64      +/-   ##
==========================================
- Coverage   96.84%   94.84%   -2.00%
==========================================
  Files           9        9
  Lines        1015     1048      +33
==========================================
+ Hits          983      994      +11
- Misses         32       54      +22

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Make HuskChat command executable from console

This pull request makes /huskchat, as well as its sub-commands (like the reload command), executable from console as requested in #27.
To prevent MineDown text from being shown in console, links are extracted by regex before the whole string is passed to MineDown to filter out formatting codes, leaving just the text behind. The regex solution isn't great, but it works.

I messed up my branches, so please submit a new PR if you're up for it :) sorry!

Yeah, I didn't like my solution much either. While it certainly is possible to send MineDown components directly to the Velocity console (without needing to parse and strip them), and converting to legacy formatting is also easily done, I can't find any good solution for showing links in console.

Hmm, good point. To be honest, it's probably fine that way even if it's imperfect. Perhaps something to detect and extract links in the message manager? But even that's a bit annoying and specific.
I plan on eventually moving some of the boilerplate I use across my plugins to a library, I may look into handling this all in that to save this stuff.

I could modify the regex from this PR to extract the links before passing the message to MineDown. It isn't great, but I can't think of any better solutions.


Validate and Modify Record on Server

Is there any way to change the data on server side before its written to cache or db?
I need some kind of data validation, so i can make sure that the schema is valid.
Are there any plans for adding such a feature? Some layer like permissions service, where u can hook in and modify the data.
OnMessageRecieve() => do something and callback
OnMessageSend() => do something and callback
Like the Middleware in express.js

You can put that inside of your permission layer and say the message is denied if it doesn’t match your schema, plus a custom message.
Please look at
https://deepstream.io/tutorials/custom-plugins/permission/index.html

Thank you. I tryed this but i do not get it to work as expected.
i cant access the data on record create or record read.
canPerformAction(username, message, callback, authData, socketWrapper, passItOn) {
  if (message.topic === 5) {
            // Create
            if (message.action === 15) {
                // **message.data always undefined**
            }

            if (message.action === 18) {
               // this one works kind of. Does not update record in cache. Client does not recieve the data
                let data = {};
                if(message.data) {
                    data = JSON.parse(message.data);
                }

                if(!data.owner) {
                    data.owner = username;
                }
                data.updated_at = Date.now();
                message.data = JSON.stringify(data);
            }
        }
}

i think i miss something :/

You want to validate on messages coming into the server, so that should be a RECORD_ACTION.UPDATE or RECORD_ACTION.PATCH.
CREATE is what is used when you want to create a new empty record, and READ is when you request the data from the server.

My server code:
canPerformAction(username, message, callback, authData, socketWrapper, passItOn) {
        // TOPIC RECORD
        if (message.topic === 5) {
            // ACTION CREATE RECORD
            if (message.action === 15) {
                let data = {};
                if(message.data) {
                    data = JSON.parse(message.data);
                }

                data.created_at = Date.now();
                data.owner = username;
                message.data = JSON.stringify(data);
            }

            // ACTION UPDATE RECORD
            if (message.action === 18) {
                let data = {};
                if(message.data) {
                    data = JSON.parse(message.data);
                }
                if(!data.owner) {
                    data.owner = username;
                }
                data.updated_at = Date.now();
                message.data = JSON.stringify(data);
            }

        }

        if (message.topic === 6) {
            if (message.action === 2) {
                let data = JSON.parse(message.data);
                data.authData = authData;
                data.username = username;
                message.data = JSON.stringify(data);
            }
        }
        // pass request to Valve permission service
        this.permission.canPerformAction(username, message, callback, authData, socketWrapper, passItOn)
        return
    }

expected outcome in database
{ "_id" : { "$oid" : "5d63861fedac8cd86098ed02" }, "ds_key" : "123456", "__ds" : { "_v" : 0 }, created_at: 123456789, owner: "Some User"}
actually stored in database
{ "_id" : { "$oid" : "5d63861fedac8cd86098ed02" }, "ds_key" : "123456", "__ds" : { "_v" : 0 } }
Client code:
// ... Login etc
let record = client.record.getRecord('test/123456') // create new record

record.whenReady(rec => {
        console.log(rec.get()) // expected: {created_at: 123456789, owner: "Some User"} what i get: {}

       rec.set({some:"data"});
       console.log(rec.get()) // expected: {created_at: 123456789, owner: "Some User", some:"data"} what i get: {some:"data"}
}

Do i have to subscribe to the record?
what i missing here

create does not actually set any data and just sets it as empty, regardless of how you transform the data.
// ... Login etc
await client.record.setData('test/123456', {})
let record = client.record.getRecord('test/123456') // create new record

record.whenReady(rec => {
        console.log(rec.get()) // expected: {created_at: 123456789, owner: "Some User"} what i get: {}
       rec.set({some:"data"});
       console.log(rec.get()) // expected: {created_at: 123456789, owner: "Some User", some:"data"} what i get: {some:"data"}
}

That should work since your doing a CREATEUPDATE (which sets the data) and then subscribes.
It's a valid usecase though, I'll have a think about it.
canPerformAction(username, message, callback, authData, socketWrapper, passItOn) {
        // TOPIC RECORD
        if (message.topic === 5) {
            // ACTION UPDATE RECORD
            if (message.action === 18) {
                socketWrapper.parseData(message)
                if(!message.parsedData.owner) {
                    message.parsedData.owner = username;
                }
                message.parsedData.updated_at = Date.now();
            }
        }

        if (message.topic === 6) {
            if (message.action === 2) {
                let data = JSON.parse(message.data);
                data.authData = authData;
                data.username = username;
                message.data = JSON.stringify(data);
            }
        }
        // pass request to Valve permission service
        this.permission.canPerformAction(username, message, callback, authData, socketWrapper, passItOn)
        return
    }

Also please use the RECORD_ACTION enum that is provided via @deepstream/protobuf, I don't know the enums by heart 😅
Thanks!

Sorry. Will import RECORD_ACTION. Just playing around with it.
rec.get()

hast the expected data if i restart the server between requests.
Is there any way to trigger an update message from within the canPerformAction function?

nope, its a one way pipeline to just verify data is permissioned. Not sure why you would want to do that but generally doing any other side effects than saying yes or no is usually a bad idea. If the user is supposed to know it is the owner and when it was last updated then the client should set those and you should permission against them accordingly. That being said all other clients will get the update if you augment it in the permission layer, so you can let each client have to connections so that it receives updates for its writes.

Hmm. I dont like that the client have to track created an updated timestamps. But ok. Than i have to do it this way. I mus t restructure some data, so that records with sensetiv data can not be requested by the client. Example User records have encrypted Password and some Tokens. This data must be moved out of user record because it is not possible to modify records before they are send.
I still recommend some kind of middleware that can do such things. I will take a closer look at the source, maybe i will find a solution
For now i close this issue. Thanks for the support


Android marker icon

Hi, when i try add icon to marker i get following error:
TypeError: Cannot read property 'android' of null File: "/data/data/app.test.test/files/app/tns_modules/nativescript-google-maps-sdk/map-view.js, line: 311, column: 115
I have images placed right in AppResource file (in xml element Image is normal showed).

Thanks - I pulled in your fix


Version

Can you add version selector? that would be great for servers that are 1.7;1.8;1.9! :+1:
EDIT: Also can you add a download link too?

Also can you add a download link too?

I just thought I could use spigot for providing downloads, because there is already a stress tester there.

Can you add version selector? that would be great for servers that are 1.7;1.8;1.9

I think that would work. It would be just ~300kB additional file size for each version

ok, thank you :+1: !


bridge_mod.js turned into a Userscript

for whoever wants to use that ¯_(ツ)_/¯

Didn't notice one was already made, my mistake!


ISPN-14207 Make MemcachedDecoder non blocking

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

looks like I have some failures in integration tests...

@diegolovison can we have a perf run with memcached client?

http://perfrepo.mw.lab.eng.bos.redhat.com/reports/tableComparisonReport/19584

@diegolovison are you sure the job did run the right thing?

the snapshot commit is from 8 days ago.
the logs show the test run for 240 sec but the report shows almost a ~400 sec duration.
the radargun version used is different

http://perfrepo.mw.lab.eng.bos.redhat.com/reports/tableComparisonReport/19586

Merged, thanks


Integrated terminal can not be opened, flash back

VSCode Version: Code 1.3.0 (e724f269ded347b49fcf1657fc576399354e6703, 2016-07-07T16:56:12.476Z)
OS Version: Windows_NT ia32 10.0.10586

Steps to Reproduce:

Please provide detailed steps of how to reproduce the issue.


Rename Let widget to Value

As pointed out last time, we have a Let widget to define a value (let x = 12) but in the case of F#, let can mean several things: value, constant, and function.
So I renamed the Let widget to Value so it's clearer the specific use case this widget cover.
AnonymousModule() {
    Value("x", "12")
}

Agreed Value is clear. Can we update the code snippets in the Readme? .Thanks


fix(file-router): ignore "config" export for HMR

Allows correct HMR update in case export const config exists in the route file.

The java test failure would be alleviated after #2130 is merged, stay tuned...

I don't pretend to have a good enough understanding of how HMR works without some additional explanations but I might still consider this change to be good enough if it's guarded by the integration test that we discussed.

Basically, I was struggling with the fact that I really don't know how HMR is working and how that regex is helping there :)

This is basically one of those cases where it would be helpful with some comments in the code to give some additional contest in the form of an examples of what the targeted code looks before the transformation and what it is supposed to look like after.


legacy 'django common' templates and assets

This is the templates and static assets that had been in the private repo 'django-cfgov-common. This makes sense, at this point, because with the 'jobmanager' code, we have code in this repo that depends on it, and it's better for this repo not to depend on private repos.
Additions

A whole bunch of files

Testing

'pip uninstall cfpb_common'
checkout out pages like jobs/supervision to see that everything still works.

Review

@kurtw @kave @richaagarwal

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

Things work for me 👍

👍


gitlab-runner: 15.1.0 -> 15.2.0

Description of changes
https://gitlab.com/gitlab-org/gitlab-runner/blob/v15.2.0/CHANGELOG.md

Things done

Built on platform(s)

[x] 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/)
22.11 Release Notes (or backporting 22.05 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

/cc @bachp @zimbatm @globin @dlouzan

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

1 package built:

gitlab-runner


Clicking 'Resolve Conversation' deletes the currently typed comment

Command attempted:
In a PR, when a user has commented in a review:
Typing a comment and clicking 'Resolve Conversation' should send the comment, then resolve the conversation.
What happened:
The comment is deleted and the conversation resolved.

Hi, I believe this is a question about GitHub.com, the website. This issue tracker is for a command-line utility called hub, not the website itself. For reporting bugs and requesting features for GitHub the website, please go to https://github.com/contact or visit https://github.community. Thank you!

Hi, I believe this is a question about GitHub.com, the website. This issue tracker is for a command-line utility called hub, not the website itself. For reporting bugs and requesting features for GitHub the website, please go to https://github.com/contact or visit https://github.community. Thank you!


Docs: fix typo in ws recipe

There is no i defined in this function; it seems like a leftover from some of the other recipes on this page.
Copy-pasting the function as it currently is results in a compiler error, but works as expected when (i) is removed.

Thanks! It was fixed in #1234


Add Coupon Functionality for Discount on Cart Total

Description
Added functionality to apply a coupon code for a 30% discount on the cart total. Users can enter the coupon code 3p9xLnDE, and upon validation, the total amount will reflect the discount.
Related Issues
None

Closes #427

Type of PR

[X] Feature

Screenshots / videos (if applicable)

Checklist

[X] I have gone through the contributing guide
[X] I have updated my branch and synced it with project main branch before making this PR
[X] I have performed a self-review of my code
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional context:

@Shweta-1902 resolve the conflicts and also the UI of input doesn't match with theme retro


ARROW-14071: [R] Try to arrow_eval user-defined functions

Once this PR is merged, users should be able to use their own functions in an {arrow} dplyr-like pipeline (provided they can be translated with the help of existing bindings).
library(dplyr)
library(arrow)

nchar2 <- function(x) {
  1 + nchar(x)
}

# simple expression
tibble::tibble(my_string = "1234") %>%
  mutate(
    var1 = nchar(my_string),
    var2 = nchar2(my_string)) %>%
  collect()
#> # A tibble: 1 × 3
#>   my_string  var1  var2
#>   <chr>     <int> <dbl>
#> 1 1234          4     5

# a slightly more complicated expression
tibble::tibble(my_string = "1234") %>%
  mutate(
    var1 = nchar(my_string),
    var2 = 1 + nchar2(my_string)) %>%
  collect()
#> # A tibble: 1 × 3
#>   my_string  var1  var2
#>   <chr>     <int> <dbl>
#> 1 1234          4     6

nchar3 <- function(x) {
  2 + nchar(x)
}

# multiple unknown calls in the same expression (to test the iteration)
tibble::tibble(my_string = "1234") %>%
  mutate(
    var1 = nchar(my_string),
    var2 = nchar2(my_string) + nchar3(my_string)) %>%
  collect()
#> # A tibble: 1 × 3
#>   my_string  var1  var2
#>   <chr>     <int> <dbl>
#> 1 1234          4    11

# user function defined using namespacing
nchar4 <- function(x) {
  2 + base::nchar(x)
}

tibble::tibble(my_string = "1234") %>%
  mutate(
    var1 = nchar(my_string),
    var2 = 1 + nchar4(my_string)) %>%
  collect()
#> # A tibble: 1 × 3
#>   my_string  var1  var2
#>   <chr>     <int> <dbl>
#> 1 1234          4     7

Thoughts around the design were captured in this design doc.

I'm not wholly against requiring registration if it is significantly simpler or is the only way. But I'm not sure that requiring registration will necessarily help make errors easier to understand (we'll need to have some good, clear error messages when anything goes wrong here regardless of the path we take).
And I might even argue that having only one kind of registration (for the user-defined scalar function) sets that apart in a positive way (crafting one of those will take care + do things that are surprising compared to all of our other bindings, after all). IMO: people shouldn't (need to) think too much about how the bindings work, just use the code you're used to, and if it works, great, if it gives you a (clear) error that some function isn't native to arrow yet, you can respond to that as is. If we have two different types of registration we'll need to find a way to explain how they are different (and even with great docs explaining that difference, there will be a contingent of people who will wonder what's the difference between the hypothetical register_binding(...) and register_scalar_function(...).
But, again, if autoregistering is a sticking point (though looking through the code that's here it doesn't look like it is...), then sure we can go the registration route first. (or if all the autoregistration stuff is overly burdensome from a maintenance perspective too, of course).

Thanks for this! One comment, so far. It would be nice to have an idea of which parts of this are working, and which parts still need work before they are ready (I see a handful of todos in the code, which is great, but also in a comment of "I've done these bits: ..., still working on these: ...")

All the examples listed in the PR description work. So far this works for:

mutate with:

[ ] simple expression (nchar2(my_string)) - contains a simple call to a user-defined function
[ ] a slightly more complicated expression (1 + nchar2(my_string))
[ ] multiple unknown calls in the same expression (to test the iteration) - nchar2(my_string) + nchar3(my_string)
[ ] user function defined using namespacing 1 + nchar4(my_string)

@paleolimbot I like your suggestion in https://github.com/apache/arrow/pull/13789#issuecomment-1214992856. Even if we don't require the user to self-register, that function can really simplify my current logic (as we can use it instead of having to walk the AST).

Update:

user-defined functions work with mutate() and filter()
and work with nested function definitions

Give both of these work and the approach is rather complex, by walking the AST, I will now go more in the direction suggested by @paleolimbot (in #13789 (comment)) and refactor / simplify the implementation.

Could you say more about what you're thinking about doing here, there were a few suggestions in that comment | thread. Are you going to now require registration? Or something else?

Not requiring registration. I was thinking about switching from translation + registration to registration only.
Currently my approach is to take:
nchar2 <- function(x) 1 + nchar(x)

walk its abstract syntax tree, translate it to
nchar2 <- function(x) call_binding("+", 1, call_binding("char", x))

and then register it + update .cache$functions.
However, the translation might not be necessary and only complicates the implementation, as we only need to register the "nchar2" binding without actually translating its body for this to work. The reason it doesn't work in main is because the data mask has no knowledge of "nchar2". So updating the context of the evaluation (to be aware of the translation, but without actually translating the body) should work.
Translating the body also highlighted the overlap between nse_funcs and .cache$functions. We could potentially reduce that complexity too - ARROW-17495, but I'm happy to leave that for another PR.

Ah cool, that sounds good, thanks!

The CI failure (Windows & R 3.6) seems to be related to https://github.com/apache/arrow/pull/13661.

I'm closing this PR for now as it was handed over to me to take a look at, but I don't think I'm going to end up with time to work on this in the near future, and we're just having a bit of a tidy of PRs ahead of the upcoming release.


Wrapper shell scripts for starting and stopping services

Start and stop scripts (for *nix environments) to automate the instructions provided at https://github.com/bitpay/bitcore-wallet-service/blob/master/installation.md
If this is accepted, I can update the documentation as well.

Yes! thanks for the contribution!
Could you update the docs and also add 'start.sh' to package.json ->  npm start?

Done. Also added a "stop" target to package.json
All doc changes in installation.md. Cleaned up some inaccurate npm references there as well. The installation document is completely accurate and tested.

LGTM
This is very useful, thank you!


KeyboardKit appearance issue on Snapchat

I thought that I have a problem with my custom keyboard extension, I installed KeyboardKit and added its default keyboard. I run into same issue. While Snapchat appearance settings based on System appearance which is dark in my case but the keyboard still appeared in light appearance.

Any hint?

Hi @mrhaxic 👋
I can have a look, but it all depends on the information that KeyboardKit receives.
I previously struggled with a problem that appeared when editing a text field that applied a dark keyboard appearance while the system was in light mode. In these cases, iOS told the keyboard extension that the system was actually in dark mode, which made it impossible to determine which colors to apply.
I (and many others, such as Grammarly) solved this by applying a transparent color instead, which looks correct in light and dark mode and appearance.
The question here is what information that is given to the keyboard extension. Can you inspect the extension and see what the color scheme and keyboard appearance is?

Hi @danielsaidi ‒ I'll check and get back to you.

Closing this since I haven't heard back.


Regression: setting recordingName is ignored

Description
Affects v5 and still works on v4.3.0.
basically, no matter if you specify the recording name in v5 or not, it always generates the recording in a fully namespace directory. not according to the set recordingName.
Shareable Source
describe('.thing', function () {
    setupPolly({
        adapters        : ['node-http'],
        persister       : 'fs',
        mode            : 'replay',
        recordIfMissing : true,
        persisterOptions: {
            keepUnusedRequests: true,
            fs                : {
                recordingsDir: POLLY_RECORDINGS
            }
        },
        matchRequestsBy: {
            body   : true,
            headers: false,
            order  : false
        }
    });

  it('should successfully send', async function() {
    this.polly.server
      .post(APP_URL)
      .recordingName('200-Raw0');
  });
});

{
  "@pollyjs/core": "^5.0.0",
  "@pollyjs/adapter-node-http": "^5.0.0",
  "@pollyjs/persister-fs": "^5.0.0"
}

Environment
Node.js v14.12.0
linux 5.4.0-47-generic
npm 6.14.8

I tried to reproduce this but using recordingName worked as expected:
const { Polly } = require('@pollyjs/core');
const NodeHttpAdapter = require('@pollyjs/adapter-node-http');
const FSPersister = require('@pollyjs/persister-fs');
const fetch = require('node-fetch');

Polly.register(NodeHttpAdapter);
Polly.register(FSPersister);

(async function () {
  const polly = new Polly('polly-recording-name-test', {
    adapters: ['node-http'],
    persister: 'fs',
    mode: 'replay',
    recordIfMissing: true,
    persisterOptions: {
      keepUnusedRequests: true,
    },
    matchRequestsBy: {
      body: true,
      headers: false,
      order: false,
    },
  });

  polly.server
    .any('https://jsonplaceholder.typicode.com/*')
    .recordingName('jsonplaceholder');

  await fetch('https://jsonplaceholder.typicode.com/todos/1');

  await polly.stop();
})();

Generated recordings/jsonplaceholder_1752295936/recording.har.
This can be an issue with your polly.server.post matcher or its an actual issue with setup-polly-jest. Does something like this.polly.server.post(`${APP_URL}/*`) work?

It was exactly that! I was matching against the wrong URL, I only figured this out after a full day and a meeting with a vendor.
This leads me to another request, for configuration purposes there should be a way to switch polly into "debug" mode in order to see where exactly the match is failing.
For example:
cant find a request matching the URL ${request url}
request for recording  ${recording name} partially matched a request but failed to match headers <print diff>
... etc


scrollIntoView() or setScrollPosition() method?

I need to be able to update the scroll position. Is there some sort of scrollIntoView() method? So I could pass in a node and it would scroll until it was visible. Another, simpler, option would be a setScrollPosition method where I could find the element position myself and just update the scroller to where I need it to go. Is there something available to accomplish what I need currently? As of now I'm planning on just hacking the top position css... but that's not very pretty.
Thanks so much for ngTinyScrollbar – it's great!

Yes, it exposes an update method. Specify the position you want to scroll to in your outer directive.

Can you help me how to use this update function?
For example I have:  and how to call update(x) from the controller in which is this directive?

@Mossar you would do it inside another directive's link function. So in your example your outer div would need to be another directive and when defining that controller you would add your update code in the outer directive's link function. See the documentation here under 'Creating Directives that Communicate'


Error in Grafana: "Invalid interval string, expecting a number followed by one of "Mwdhmsy"

I've installed all prerequisites and get the error "Invalid interval string, expecting a number followed by one of "Mwdhmsy"" in Grafana.
Controller version used: 5.13.29
Any hints where the error is coming from?

Maybe you can give me a few more hints where it's coming from. What version of Grafana? What dashboard?

David,
Grafana version: v6.6.1 (21bf8b71bc)
Dashboards: I tried UAP, Sites, and DPI. All of the three showed the same error (see screenshot).

Great, glad you found the problem!


Lato font resulting in incorrect bold on pages?

Over in Jupyter Book we got a report that for some Mac users, bold wasn't showing up properly on some browsers:
https://github.com/executablebooks/sphinx-book-theme/issues/122
I think that this is because of the Lato font family that is loaded by default, because disabling it then makes the fonts look correct.
I'm wondering if this is something anybody has experienced with the pydata theme? I think Jupyter Book is getting its fonts straight from this theme, so if it's a bug there, it's probably a bug here. @hoetmaaiers maybe you have encountered something like this?

HI @choldgraf , I did the same check on macOS in Safari. I do see Lato ren the <strong> text rendered in bold. I have no Lato installed on my system.
What browser appears to be the issue? Are  we sure it is about Safari?
This screenshot does looks correct?

This might also be impacted by https://github.com/pandas-dev/pydata-sphinx-theme/pull/169 ?

Now that #169 is merged, it might be good to test whether this changed / fixed anything. But @choldgraf I think you can't reproduce it yourself? (so testing might be a bit difficult, without waiting on the next release?)

digging into this a bit more, a few more pieces of information:

The error is not Safari, I was mistaken. The error is on Chrome on OSX
I can reproduce the error from within browserstack
The issue seems to be that in Chrome on OSX, the Lato font is used. On Safari it is not used and defaults to the system sans-serif font (e.g. helvetica)
So...it seems like the issue is that on most browsers, Lato isn't used. When Lato is used, this looks like a bug with Lato

That's a bit weird, because I'd assume that our explicitly-specified font we'd want to work the most, right?
I can confirm that on my own browser (W10, Chrome) from this page:
https://pydata-sphinx-theme.readthedocs.io/en/latest/demo/demo.html
Inspecting the bolded section shows that Lato is not being used and it's defaulting to the system sans-serif (Arial)

Thanks for digging in ..
On Firefox / Ubuntu, my web developer inspection tools indicate that Lato Bold is used for the bold word in the first sentence. But I am not sure how to actually check for sure

In Chrome the error is indeed clearly printed in the console. Since #169 the Lato font is imported as a vendor font in which I am also able to reproduce the behaviour.
The issue  is that the bold version lato-700.woff is not present, only the 400 version of Lato..

osed by #224


Update readme

Covers the basic structure, entry point, config options.
The structure the old readme referred to has been deleted.

Could you use the README structure from the styleguides?
https://github.com/alphagov/styleguides/blob/master/use-of-READMEs.md

One typo, ditto @tijmenb's comment re: structure from the stylguides.
Otherwise, content looks fine.

@tijmenb @mobaig thanks, I think your comments have been addressed

👍 excellent!

@benhyland my typo wasn't ; ).

@mobaig yep, confusion was due to my git command line fingerfailure. Thanks for chasing :)


Correct "Option 2: Running Solana Testnet via a proxy hosted locally"

after the appearance of the image cybercoredev/proxy:v0.2.0, you need to fix the documentation on
https://docs.neonlabs.org/docs/devportal/testnet/connect_to_solana_via_proxy#option-2-running-solana-testnet-via-a-proxy-hosted-locally
now launching proxy from docker is done by commands:
for devnet:
docker run --rm -ti --network=host -e CONFIG=devnet cybercoredev/proxy:v0.2.0
for testnet:
docker run --rm -ti --network=host -e CONFIG=testnet cybercoredev/proxy:v0.2.0
https://github.com/neonlabsorg/proxy-model.py/pull/181

Description of the option 2 has been corrected. The fix is waiting for an approval.


Ast2600 A1 not booted after enable secure boot

setups i am followed to complied secure boot

generate rsa public private key using opnssl .
-> openssl genrsa -out rsa_oem_dss_key.pem 1024
-> openssl rsa -in rsa_oem_dss_key.pem -pubout -out rsa_pub_oem_dss_key.pem
add public key name into this  (2600-a1_mode2-rsa1024-sha224.json).
configure and compile.
programed  otp memory using u-boot utility.
Burn secure boot image and boot.

problem:- during booting only two word( S2 ) print on uart console.

Please try AST2600 A3 chip.
We are not going to support A1 anymore.
Thanks.


Interpolate differences in vector layer alpha property in canvas/exported AVDs

Exported AVDs is done. Still need to render the changes in the canvas.


[Question] Additional Struct Fields and scan values

I add TotalLikes field to post type with Additional Struct Fields template
{{ define "model/fields/additional" }}
	{{- if eq $.Name "Post" }}
        // TotalLikes defined by template.
        TotalLikes int `json:"totalLikes,omitempty" gqlgen:"totalLikes"`
    {{- end }}
{{ end }}

And I update the Post type in the GraphQL schema
extend type Post {
	totalLikes: Int!
}

TotalLikes field is not stored in the database but I added it with custom SQL Modifiers in query like that
client.Post.Query().
		Modify(func(s *sql.Selector) {
			t := sql.Table(likes.Table)

			s.LeftJoin(t).
				On(
					s.C(post.FieldID),
					t.C(likes.FieldPostID),
				).
				AppendSelect(
					sql.As(sql.Count(t.C(likes.FieldUserID)), "total_likes"),
				).
				GroupBy(s.C(post.FieldID))
		}).All(ctx)

But when I use this query in GraphQL, the following error is displayed:
unexpected column \"total_likes\" for type Post

How can I add the TotalLikes custom field in Post type scan values?

Hey @sahmadreza 👋
First, if you need to get the totalCount of entities, you can get it without any additional code using our Relay Connection integration. There's an example here: https://github.com/ent/contrib/tree/master/entgql/internal/todo.
If you still need to extend the GraphQL schema with additional fields, you can create another schema file (non-generated) and use the extend keyword to extend objects. Then, gqlgen will generate resolvers for these extra fields. Check our this example: https://github.com/ent/contrib/blob/master/entgql/internal/todo/todo.graphql#L13
Closing, but please feel free to reopen or join our Discord community in case you need more help with this 😎


[feat] react-x/no-unknown-property for MDX and other non-TS envs

Describe the problem
Similar to https://github.com/Rel1cx/eslint-react/issues/834
I ran into an issue today that react-x/no-unknown-property is not implemented in eslint-plugin-react-x, as noted in https://github.com/Rel1cx/eslint-react/issues/85#:~:text=members %23234-,The following can be enforced by TypeScript%2C no need to implement them,-react/jsx-no:

The following can be enforced by TypeScript, no need to implement them

react/no-unknown-property

This causes problems for environments where TypeScript is not available
Eg. there is not yet full TypeScript support for MDX files, and it appears ESLint doesn't support multiple parsers for a set of files (eg. for trying to use typescript-eslint parser along with the MDX parser).
Describe the solution you'd like
It would be great to get react-x/no-unknown-property implemented for other languages like MDX? (and also just for completeness, for other situations where TypeScript cannot be used)
Alternatives considered
Keep using react/no-unknown-property via eslint-plugin-react
Additional context
Ideally, this rule would be implemented also with abilities to extend for other environments like React Three Fiber (a missing feature of react/no-unknown-property):

https://github.com/jsx-eslint/eslint-plugin-react/issues/3423

I will add this rule, but as you mentioned, the issue of applicability across different environments, rather than adding options to rules, I think a better approach is to provide react-dom/no-unknown-property, similar to react/no-unknown-property, which is essentially a react-dom rule. Other environments (not necessarily provided by us) would offer corresponding react-native/no-unknown-property, react-three-fiber/no-unknown-property, etc.
This way, not only can the no-unknown-property rule in each environment be updated and released according to its own versioning cadence (to sync the propertires changes), but it also reduces the complexity of JS tool configurations, especially with ESLint, where we've heard too many complaints about overly complex configurations. This is one of the main reasons many users have switched from ESLint to biome, oxlint recently.
We believe that through better rule design and plugin positioning, we can avoid configuration complexity without sacrificing functionality.


Idea: load Channel ID's with data package

What about importing the Discord Data package's information such as Channel ID's? This can be used for automatic deleting most chats

#39 seems to be it, but the author didn't get back to it, so I would have to try to merge it myself later


Improved UI when using a language other than English

Improved UI when using a language other than English:

added MinWidth and set Width to Auto for all buttons;
set HorizontalTitleAlignment to Centered for all buttons for consistent look and feel across languages.

When I added the Romanian language I noticed that the buttons were either too small to accommodate the text or the text was overlapping with the icons. Few screenshots (before and after):
Before

After

Before

After

Before

After

Before

After

P.S. This is the last PR for a while, unless I find a bug or something.

This looks great, did you check that it doesn't break other languages?

It shouldn't break any other languages since the controls will increase the width to accommodate the whole text. But yes, I did check some other languages as well. In fact I switched thru all of them in the Settings dialog, but also checked some other dialogs.


Profile options for passing reference data to workers in dataflow

From @droazen on May 5, 2015 18:32
Copied from original issue: broadinstitute/hellbender#490

From @droazen on May 5, 2015 18:34
Options include:
-sharding the reference data + GroupByKey
-embedding reference bases as an attribute in reads
-downloading reference to each worker on demand from a bucket
-containers?

From @pgrosu on May 14, 2015 17:33
What is the current I/O throughput, the amount of data being pushed and the desired response-time?  Usually keeping read-only data locally is usually faster, but it depends on how frequently it is used.  Otherwise there would be dedicated sharded indexed servers, which would perform part of the repetitive data-processing to increase the response-time back to any workers and returned results.


Copyright claim for 'multipart' developers

Hi
The LICENSE-MIT file says that copyright is owned by The multipart Crate Developers here: https://github.com/abonander/const-cstr/blob/master/LICENSE-MIT#L1
But Cargo.toml says that the author is Austin Bonander<EMAIL_ADDRESS>I presume that this is a copy-paste error? Would be great to get clarification on this in order for me to be able to package the crate for debian.
best regards

Fixed, it was a copypaste error.
For the record, I am the principal developer of both crates but they have received minor contributions.


NullPointerException in client.HttpResponseDecoder

I got reported a NPE issue from a Armeria Slack thread.
Caused by:java.lang.NullPointerException: Cannot invoke "com.linecorp.armeria.client.HttpResponseDecoder$HttpResponseWrapper.close(java.lang.Throwable)" because "res" is null(Cannot invoke "com.linecorp.armeria.client.HttpResponseDecoder$HttpResponseWrapper.close(java.lang.Throwable)" because "res" is null)
  at com.linecorp.armeria.client.HttpResponseDecoder.failUnfinishedResponses(HttpResponseDecoder.java:131)
  at com.linecorp.armeria.client.HttpSessionHandler.channelInactive(HttpSessionHandler.java:415)

HttpResponseDecoder MUST be executed in an event loop. The NPE might be caused by recursive calls of failUnfinishedResponses().
Even though we use Iterator, a null reference could be returned.
private final IntObjectMap<String> intObjectHashMap = new IntObjectHashMap<>();
private final Map<Integer, String> hashMap = new HashMap<>();

@Test
void intObjectHashMap() {
    intObjectHashMap.put(1, "A");
    intObjectHashMap.put(2, "B");
    intObjectHashMap.put(3, "C");
    intObjectHashMap.put(4, "D");
    intObjectHashMap.put(5, "E");
    // java.lang.AssertionError: null is not a legitimate internal value. Concurrent Modification?
    removeAll(intObjectHashMap);
}

@Test
void jdkHashMap() {
    hashMap.put(1, "A");
    hashMap.put(2, "B");
    hashMap.put(3, "C");
    hashMap.put(4, "D");
    hashMap.put(5, "E");
    // java.util.ConcurrentModificationException
    removeAll(hashMap);
}

void removeAll(Map<Integer, String> strings) {
    for (final Iterator<String> iterator = strings.values().iterator();
         iterator.hasNext();) {
        final String res = iterator.next();
        System.out.println("removed: " + res);
        // To avoid calling removeResponse by res.close(cause), remove before closing.
        iterator.remove();
        removeAll(strings);
    }

/cc @kojilin

Just curious there is a possibility that calling failUnfinishedResponses recursively?

Just curious there is a possibility that calling failUnfinishedResponses recursively(didn't see the stacktrace and it should be in the same thread)?

My words were wrong. failUnfinishedResponses will not be recursively called. However, the problem is still reproduced if Map.remove() is invoked while iterating a Map.
private final IntObjectMap<String> intObjectHashMap = new IntObjectHashMap<>();

@Test
void intObjectHashMap() {
    intObjectHashMap.put(1, "A");
    intObjectHashMap.put(2, "B");
    // java.lang.AssertionError: null is not a legitimate internal value. Concurrent Modification?
    removeAll(intObjectHashMap);
}

void removeAll(Map<Integer, String> strings) {
    for (final Iterator<String> iterator = strings.values().iterator();
         iterator.hasNext(); ) {
        final String item = iterator.next();
        iterator.remove();
        removeOne(strings);
    }
}

void removeOne(Map<Integer, String> strings) {
    strings.remove(2);
}


Add checks/badge for 1.59.0

Fixes #10

I think this will hit the double panic in the condvar test I mentioned earlier. It would be nice to fix that.
Also, not every project needs to support an MSRV, so it's up to @kprotty if he cares to.

Addressed the condvar double panic in #12

Could you rebase on master?

Could you rebase on master?

Done


Release date is not imported for itch.io games

We already import it for games that have that information, most sadly don't have it on itch. You will need to use an alternative metadata source.


[MastodonBridge] Update feed URL format

Changes feed URL from https://instance/users/username.atom to<EMAIL_ADDRESS>Closes #1716

gj @VerifiedJoseph !


Support go1.21

While pre-release testing go1.21, we found some regression test issue

error build log
  ==> Testing ibazel
  iBazel [4:01PM]: Querying for files to watch...
  iBazel [4:01PM]: Building //:bazel-test
  Loading:
  Loading:
  Loading: 0 packages loaded
  Analyzing: target //:bazel-test (0 packages loaded, 0 targets configured)
  ERROR: /tmp/ibazel-test-20230625-66545-1f7x4j4/_bazel/925cec60fe9edcc34fd0fd7780b797bf/external/go_sdk/version.bzl:5:21: unclosed string literal
  ERROR: /tmp/ibazel-test-20230625-66545-1f7x4j4/_bazel/925cec60fe9edcc34fd0fd7780b797bf/external/go_sdk/version.bzl:6:1: syntax error at 'time': expected newline
  ERROR: /tmp/ibazel-test-20230625-66545-1f7x4j4/_bazel/925cec60fe9edcc34fd0fd7780b797bf/external/go_sdk/version.bzl:6:11: invalid octal literal: 06 (use '0o6')
  ERROR: /tmp/ibazel-test-20230625-66545-1f7x4j4/_bazel/925cec60fe9edcc34fd0fd7780b797bf/external/go_sdk/version.bzl:6:20: invalid octal literal: 00 (use '0o0')
  ERROR: /tmp/ibazel-test-20230625-66545-1f7x4j4/_bazel/925cec60fe9edcc34fd0fd7780b797bf/external/go_sdk/version.bzl:6:23: invalid octal literal: 01 (use '0o1')
  ERROR: /tmp/ibazel-test-20230625-66545-1f7x4j4/_bazel/925cec60fe9edcc34fd0fd7780b797bf/external/go_sdk/version.bzl:7:1: unclosed string literal
  ERROR: /tmp/ibazel-test-20230625-66545-1f7x4j4/BUILD:3:10: While resolving toolchains for target //:bazel-test: invalid registered toolchain '@go_sdk_toolchains//:go_aix_ppc64': error loading package '@go_sdk_toolchains//': compilation of module 'version.bzl' failed
  ERROR: Analysis of target '//:bazel-test' failed; build aborted:
  INFO: Elapsed time: 0.541s
  INFO: 0 processes.
  FAILED: Build did NOT complete successfully (29 packages loaded, 6 targets configured)
  iBazel [4:01PM]: Build error: exit status 1

  ################################################################################
  # Did you know iBazel can invoke programs like Gazelle, buildozer, and         #
  # other BUILD file generators for you automatically based on bazel output?     #
  # Documentation at: https://github.com/bazelbuild/bazel-watcher#output-runner  #
  ################################################################################

  Killing child processes...

full build log, https://github.com/Homebrew/homebrew-core/actions/runs/5370484128/jobs/9742789964?pr=134468#step:4:514
relates to Homebrew/homebrew-core#134468

What repo are you testing? In the BUILD file in the root we do not have a target //:bazel-test

oh, that might be some test we need to update in the homebrew side, let me check really quick

indeed an issue with updating the rules_go to use the commit in this PR
iBazel [1:15PM]: You are using an experimental filesystem watcher. If you would like to disable that, please set the environment variable
	IBAZEL_USE_LEGACY_WATCHER=1
iBazel [1:15PM]: Error getting Bazel info exit status 1
iBazel [1:15PM]: Querying for files to watch...
iBazel [1:15PM]: Error getting Bazel info exit status 1
iBazel [1:15PM]: Error finding bazel info: exit status 1

iBazel [1:15PM]: Error querying for source files: exit status 1
iBazel [1:15PM]: Error getting Bazel info exit status 1
iBazel [1:15PM]: Error finding bazel info: exit status 1

iBazel [1:15PM]: Error querying for source files: exit status 1
iBazel [1:15PM]: Building //:bazel-test
ERROR: Error computing the main repository mapping: Every .bzl file must have a corresponding package, but '@io_bazel_rules_go//go:deps.bzl' does not have one. Please create a BUILD file in the same or any parent directory. Note that this BUILD file does not need to do anything except exist.
Loading:
iBazel [1:15PM]: Build error: exit status 1

################################################################################
# Did you know iBazel can invoke programs like Gazelle, buildozer, and         #
# other BUILD file generators for you automatically based on bazel output?     #
# Documentation at: https://github.com/bazelbuild/bazel-watcher#output-runner  #
################################################################################

I'm not sure I understand what the linked PR has to do with this. Can you elaborate a little bit more?

@chenrui333  I believe the underlying problem is actually that homebrew needs to update the version of rules_go in (https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/ibazel.rb#L37-L84) to support Go's new version output. There's a new rules_go v0.41.0 that does so
The bazel-test target is one that the Homebrew project creates to test when ibazel or one of its dependencies  is bumped to a new version or changed. You can see [the test definition in homebrew's repo]
However, the log outputs linked in this ticket aren't super explanatory and don't match what's elsewhere. I suspect that's because the links that GitHub generates for the failed checks in on this page don't include the same --verbose info that's on that page.
If you look at that page, you see these error outputs mentioned in a section titled "brew test --retry --verbose ibazel failed on Linux!":
ERROR: /tmp/ibazel-test-20230808-72283-emd4kd/_bazel/2223a70abfa57191e29592d0d9893a55/external/go_sdk/version.bzl:5:21: unclosed string literal
ERROR: /tmp/ibazel-test-20230808-72283-emd4kd/_bazel/2223a70abfa57191e29592d0d9893a55/external/go_sdk/version.bzl:6:1: syntax error at 'time': expected newline
ERROR: /tmp/ibazel-test-20230808-72283-emd4kd/_bazel/2223a70abfa57191e29592d0d9893a55/external/go_sdk/version.bzl:6:11: invalid octal literal: 08 (use '0o8')
ERROR: /tmp/ibazel-test-20230808-72283-emd4kd/_bazel/2223a70abfa57191e29592d0d9893a55/external/go_sdk/version.bzl:6:14: invalid octal literal: 04 (use '0o4')
ERROR: /tmp/ibazel-test-20230808-72283-emd4kd/_bazel/2223a70abfa57191e29592d0d9893a55/external/go_sdk/version.bzl:6:23: invalid octal literal: 06 (use '0o6')
ERROR: /tmp/ibazel-test-20230808-72283-emd4kd/_bazel/2223a70abfa57191e29592d0d9893a55/external/go_sdk/version.bzl:7:1: unclosed string literal
ERROR: /tmp/ibazel-test-20230808-72283-emd4kd/BUILD:3:10: While resolving toolchains for target //:bazel-test: invalid registered toolchain '@go_sdk_toolchains//:go_aix_ppc64': error loading package '@go_sdk_toolchains//': compilation of module 'version.bzl' failed
ERROR: Analysis of target '//:bazel-test' failed; build aborted:

When you inspect the code, it's the bits parsing the output of the go command's version code.
So, homebrew needs to bump the rules_go version in the ibazel formula tests

cool, I am gonna try that later, thanks!


VM template to attach to external nw/detach external nw *

#Corresponds to API:

/cloudapi/machines/attachExternalNetwork (should have an extra arg to reference networkid)
/cloudapi/machines/detachExternalNetwork

Docs and tests must be included
part of #64
For supported virtual machine images (ubuntu / redhat), the blueprint also needs to try to statically configure the provisioned external ipaddress into the vm and bring up the interface.

Blocked on https://github.com/0-complexity/openvcloud/issues/861 and https://docs.greenitglobe.com/openvcloud/openvcloud/issues/50

After discussion with @grimpy , this is by design.

software version
master 4ab35d89


Document why OPENAI_API_KEY in getting started

I am looking into adopting promptfoo, and in the getting started step 3, randomly it asks for an OpenAI API key.
https://promptfoo.dev/docs/getting-started
Why is the OpenAI API key needed?  Can we document this (or add a link to the explanation) into the Getting Started instructions?

Is there some way we can have the "getting started" instructions not require OpenAI being set up?

Yes, if you aren't using GPT then you don't need to set your OpenAI API key!  promptfoo init creates a config that uses GPT by default, hence its inclusion in Getting Started.
Also worth noting that if you use any model-graded assert, you will have to override the LLM grader (and potentially the rubric prompt) to select a model other than GPT.

Thanks, I can contribute a docs PR for this shortly

I was in the docs anyway so I made an edit - feel free to add further clarification if you see a need :)

Just to confirm, is the default Getting Started supposed to use OpenAI's chat endpoint or completion endpoint?

It uses OpenAI's chat endpoint by default.

Thanks, I opened https://github.com/promptfoo/promptfoo-docs/pull/6 to upstream my learnings, thank you!


Fix for crash: unbind (texture) asset references when material is destroyed

Fixes #2982
I confirm I have signed the Contributor License Agreement.

Approving - but for extra points on this, a unit test never hurts. But that shouldn't block merging this, IMHO.

I'll merge now - and write a unit test today (after finishing the other issue I'm looking at) :)


Over resource limits on Streamlit Cloud

Hey there :wave: Just wanted to let you know that your app on Streamlit Cloud deployed from this repo has gone over its resource limits. Access to the app is temporarily limited. Visit the app to see more details and possible solutions.

https://huggingface.co/spaces/giswqs/Streamlit


cukinia: consider name length for _cukinia_process

In /proc, process name maximal length is 15. We need thus to cut the
argument of _cukinia_process if it exceeds 15 characters.

Also, because it is linked to an issue, you can add Closes #18. to the end of your commit message.
Otherwise, this looks good to me.

@gazoo74, according to @pkeroulas  ${pname:0:15}  is a bashism,
So we should keep the cut

merged, thanks


Errors preventing databases from being created

When trying to create databases, I came across a couple or errors that were preventing the databases from being created properly.  A stray space character was in the Energy ADE config parameters, and there were some issues with some of the string formattings in the DML Functions sql file for utility network ADE.
I was able to create the tables after fixing them here.
Reference: #dec6275

Hi,
thanks for providing the bugfix for the stored procedures. Fixed.
Regarding the install script, the database connection parameters are intentionally left as text string description. The user needs to set them according to his/her own database.

Oh, I accidentally left those in.  I changed it to remove them but they got back in somehow when making the merge request.  But there is a problem with the PGBIN parameter line, there is an extra space before the equals sign that is causing it to fail.


Add an item template for a custom xaml control.

Custom xaml controls require a few things to ensure they work correctly across languages. Template would prevent common mistakes.
Requires #568

/cc @jevansaks

I think the templates would come from @danzil and @fabiant3-zz

For WinUI we already have one for .Net and we will add one for C++/WinRT, too, in the WinUI VSIX (not the C++/WinRT VSIX). WinUI feature requests are tracked at https://github.com/microsoft/microsoft-ui-xaml/issues

@danzil Is there some overlap with the two VSIX? Perhaps we should trim down the C++/WinRT VSIX if so.

Yes, there is overlap, but the C++/WinRT VSIX targets Windows.UI.Xaml (OS Xaml) while the WinUI one targets Microsoft.UI.Xaml (WinUI 3.0) and it's not yet released (it's an Alpha).

OK thanks. Either way we don't have the resources to expand the C++/WinRT templates so I'd like to focus any new investment on the WinUI side.


operator automotive-infra (0.0.4)

Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Are you familiar with our operator pipeline?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want to use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Your submission should not

[ ] Add more than one operator bundle per PR
[ ] Modify any operator
[ ] Rename an operator
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description of the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/pipeline restart operator-release-pipeline

/pipeline restart operator-release-pipeline

@mavaras Looks like the problem initiated in https://github.com/redhat-openshift-ecosystem/community-operators-prod/pull/5263, then we need to get 4.12 here and the 0.0.5 version should pass.
However, looks like my comment doesn't trigger anything there since it "passed".

/pipeline restart operator-release-pipeline


CSS not compiling

I've been trying to figure this out for awhile now. I have my stylesheet under app/styles/main.css.
My brunch-config looks like this:
      stylesheets: {         joinTo: {                 "app.css": [/^vendor/, /^bower_components/, /^app/]             },             order: {                 before: [/^app/]             }
When I run Brunch my terminal says that main.css is compiled but is not used in browser and styles revert to default.
Here is a link to the repo: https://github.com/ruskibenya/DolDem/tree/working
Thank you for all help.

Hey @ruskibenya, how is your day going?
Brunch does not perform any magic on html, so you have to manually add
<link rel=stylesheet href=app.css>

Does that help?

You didn't include your stylesheet here. Please, check it out.

Wow. Can't believe I missed something so simple. I've tried to add the stylesheet but it still seems to not compile in browser.
 Please see http://brunch.io/docs/config for more info.
I will close this one, but feel free to contact us if you have another issue.

@shvaikalesh thank you. Would my stylesheets come before or after the material design?

@ruskibenya vendor files always come first

@shvaikalesh I think the problem I'm running into is that material design lite isn't really a vendor file. I'm just linking to a cdnjs of it and including it as a stylesheet in my index.
I've tried messing around with the jointo order but can't get brunch to correctly compile my own css file.
Am I correct that this is what's causing brunch to ignore my css file? Is the only solution to install CDL into my project as an actual vendor file? I was hoping to avoid adding the extra size.

@shvaikalesh Spasibo for all the extra help!


Health endpoint returns 2 bytes of content causing AWS ALB to crash

Bug Report

Version
Version 0.56.0

Platform
AWS EC2 containers

Description
AWS load balancer crashes when a 204 is returned with content. Since the /api/v1/health endpoint returns 2 bytes, it is not functioning as expected. I expect to see 0 bytes returned.

Whoops, thanks for the report! Expect a fix imminently.

CC @svix-dylan

@alexrejto Thanks for bringing this to our attention! We've just patched this on main, and plan on releasing 0.57.0 later today or tomorrow, which will include this patch.

Thanks so much for the rapid fix @svix-dylan!
Will this patch also be released to docker hub?

@alexrejto Of course! And yes, 0.57.0 will also ship to Docker Hub.

@alexrejto just following up to confirm v0.57.0 with this patch has shipped to Docker Hub! Apologies, this went out a little later than we’d hoped, we found some small issues in our pre-release testing we wanted to nail down. You should be good to pull the latest image to your EC2 now!

Thanks again for the speedy turn around. The fix solves the issue (which was mostly a bug in AWS)!

With pleasure! And yeah, I would have expected AWS to either relay verbatim or ignore the extra bytes.


Reduce ProviderFactory.Get method signature

The ManagedClusterInfo is part of SubmarinerConfig.Status so no need to pass it in separately.

/approve

/lgtm


use CORE_IMAGE_BIN_PATH on e2e for core image, skip compile

What
use CORE_IMAGE_BIN_PATH with CORE_IMAGE on e2e
Why
for core image, skip compile, to build e2e faster

Nice!! Thanks!
It still takes 22 mins thought!

Nice!! Thanks!
It still takes 22 mins thought!

yes cli make install_rust from source and the rpc make build-soroban-rpc targets take up the majority of that remaining build time, system-test can optionally do cargo install soroban-cli <ver> , but I've noticed that takes almost as much time as build from repo/source since the the crate install builds rust locally anyways.


upgrading to azure sdk 2.6.0

This PR upgrades the Azure SDK dependency to v2.6.0 (upgrade from v2.4.0) and targets Azure OS family 4 (upgrade from 3).
Tested in local emulator and did a test deploy to DEV, all looks good to me.
Please review @maartenba @yishaigalatzer @bhuvak

Have tried this in DEV, looks ok. Did the functional tests on build.nuget.org run?

@maartenba all green! (check the history :))

:shipit:


feat: split long articles to query ChatGPT

@a00012025 Thank you for the PR. I'll take a look and review soon.

@a00012025 Is this ready to merge? I tried it and it is working fine
Tried on a Paul Graham essay - Do things that don't scale


Fix: The database name is displayed when the TenantDatabaseAlreadyExistsException exception is thrown.

When TenantDatabaseAlreadyExistsException  was thrown, the reason did not display the concerned database name. This PR fixes that behavior.

Thanks


Update to work with upcoming<EMAIL_ADDRESS>

With these changes I think we should move this library to 1.0 - ping @timothyej

Yes, sounds good to me :+1:
I will start working on the view models.


Officially recommended way to install?

What is the officially recommended installation procedure via helm for any given LTS (K8S v1.13.2 on AWS, if that matters) ?

checkout the repo, then helm install... from master
OR checkout the repo, then git checkout release-1.0 (how do I know which LTS I would be on?), then run helm install...
OR checkout the repo, then git checkout tags/1.0.6 -b branch2InstallFrom, then run helm install...
OR download the release source for any LTS, then helm install...
OR some other way I missed

Can't get any deterministic, idempotent behaviour. What am I getting wrong?
Also: Out of curiosity.. why is there no istio helm chart in https://github.com/kubernetes/charts stable/incubator channels?

whats wrong with official docs on this: https://istio.io/docs/setup/kubernetes/quick-start/ ?

nothing wrong, figured that out in the meantime, thx! closing this.


support treq

Add support for issuing requests on Twisted via Treq.

coveralls should fail on combined coverage (this should pass, but I only see the 1 check)
we need to land https://github.com/twisted/treq/pull/106/ first to land this, since it's using an unreleased test utility


rp2_common/hardware_pio/pio: Add 'pio_add_program_at_offset' implemen…

…tation
fixes #58

fixed in another pull request - please use correct code formatting!

fixed in another pull request - please use correct code formatting!


Error while updating translation package

I ran into a problem while updating the Official & Machine TL package today. The update start off fast, up to 3/9 package, then it slow to almost a halt, after a while it show error like

Also I see that some event from 2017 until now has not been translated, should I install another one or because those translation has not been updated? I would love to contribute to it.

I fixed cloudflare's caching, try again and see if it's any better.
Event from 2017 onwards as well as LB2-LB5 is not translated yet, maybe soon.

That's strange? Since the NA one is already on 2018 content of the JP server so I think at least those from 2017 to the start of 2018 should have official translation. Or it is not available yet to the public?

Oh, you are right, 2018 onwards. What 2017 events are not translated properly?

Oh sorry I was mistaken one of the Ryo style event of 2019 for the Paul Bunyan event. Just recheck some and all of the checked one are translated


Find a way to configure where the map starts

https://github.com/codeforraleigh/wmsb/blob/master/app/assets/javascripts/views/map_view.js.coffee#L52

That is a start. Now we need to trace back and figure out where to change that stuff.


Wrong placeholder scroll behavior in dialog on firefox mobile

Bug, feature request, or proposal:
Bug report
What is the expected behavior?
Placeholder of an input in a dialog should scroll same as the input itself.
What is the current behavior?
The placeholder scrolls further then the rest of the dialog content.
What are the steps to reproduce?
https://stackblitz.com/edit/angular-sexkpn
open the stackblitz link above on an android phone with the firefox browser, open the dialog, focus one of the inputs, then start scrolling.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Latest (61.0) firefox mobile browser on android (7.0)
Is there anything else we should know?
It seems to be only happening during the scrolling (placeholder gets prepositioned correctly after scroll) and only if a input had a focus (scrolling right after opening the dialog the scrolling seems to be working fine).

I wasn't able to reproduce this. Given that we haven't had any more similar reports since this issue was posted, I'm assuming that it has been resolved.


Build failure with libvsg 1.0.9 on openSUSE_Tumbleweed/x86_64 distribution

A binary package for vsgPoints version 0.1.0 was recently added to the openSUSE Tumbleweed distribution. After updating libvsg to version 1.0.9, there are unfortunately build errors (see https://build.opensuse.org/package/live_build_log/games/vsgPoints/openSUSE_Tumbleweed/x86_64):
[   16s] cd /home/abuild/rpmbuild/BUILD/vsgPoints-0.1.0/build/src/vsgPoints && /usr/bin/c++ -DVSG_SHARED_LIBRARY -DvsgPoints_EXPORTS -I/home/abuild/rpmbuild/BUILD/vsgPoints-0.1.0/include -I/home/abuild/rpmbuild/BUILD/vsgPoints-0.1.0/include/vsgPoints -O2 -Wall -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=3 -fstack-protector-strong -funwind-tables -fasynchronous-unwind-tables -fstack-clash-protection -Werror=return-type -flto=auto -g -O2 -g -DNDEBUG -std=gnu++17 -fPIC -Wall -Wparentheses -Wno-long-long -Wno-import -Wreturn-type -Wmissing-braces -Wunknown-pragmas -Wmaybe-uninitialized -Wshadow -Wunused -Wno-misleading-indentation -Wextra -MD -MT src/vsgPoints/CMakeFiles/vsgPoints.dir/create.cpp.o -MF CMakeFiles/vsgPoints.dir/create.cpp.o.d -o CMakeFiles/vsgPoints.dir/create.cpp.o -c /home/abuild/rpmbuild/BUILD/vsgPoints-0.1.0/src/vsgPoints/create.cpp
[   16s] /home/abuild/rpmbuild/BUILD/vsgPoints-0.1.0/src/vsgPoints/create.cpp: In function 'vsg::ref_ptr<vsg::StateGroup> vsgPoints::createStateGroup(const Settings&)':
[   16s] /home/abuild/rpmbuild/BUILD/vsgPoints-0.1.0/src/vsgPoints/create.cpp:200:13: error: 'class vsg::GraphicsPipelineConfigurator' has no member named 'colorBlendState'
[   16s]   200 |     config->colorBlendState->attachments = vsg::ColorBlendState::ColorBlendAttachments{
[   16s]       |             ^~~~~~~~~~~~~~~
[   16s] /home/abuild/rpmbuild/BUILD/vsgPoints-0.1.0/src/vsgPoints/create.cpp:203:13: error: 'class vsg::GraphicsPipelineConfigurator' has no member named 'inputAssemblyState'
[   16s]   203 |     config->inputAssemblyState->topology = VK_PRIMITIVE_TOPOLOGY_POINT_LIST;

This will be an issue with vsgPoints-0.1.0 being older than VSG-1.0.9, so I guess you need a new 0.2.0 tag.
A quick check and I see recent updates to vsgPoints master to keep things compiling against the latest changes to the VSG.

Yes, can you add such a tag? https://github.com/vsg-dev/vsgPoints/commit/b740a3ef26f1a07be47356b6759e698745616ff7 seems to be the latest suitable commit for this, as with newer commits vsgPoints cannot be built with libvsg 1.0.9:
git clone git@github.com:vsg-dev/vsgPoints.git
cmake -B ../vsgPoints-build/
-- The CXX compiler identification is GNU 7.5.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at CMakeLists.txt:22 (find_package):
  Could not find a configuration file for package "vsg" that is compatible
  with requested version "1.0.10".

  The following configuration files were considered but not accepted:

    /usr/lib64/cmake/vsg/vsgConfig.cmake, version: 1.0.9

-- Configuring incomplete, errors occurred!

I have created a v0.2.0 tag to build against VSG v1.0.9:
https://github.com/vsg-dev/vsgPoints/releases/tag/v0.2.0

Thank you for fixing this issue, the package has been updated accordingly


Alert  manager fails to send email through smtp server configured with port 465

The issue have been raised here too
https://groups.google.com/d/msg/prometheus-users/962iFpDPFZU/4mimkxSuCQAJ .
https://groups.google.com/d/msg/prometheus-users/w2hUOAQFD7I/1L8FNTvaCgAJ
With smtp port 465, it does not work (whether or not you are in proxy). Not all the smtp servers provide us with the priviledge of port 587. So, kind of stuck getting it worked with port 465. It responds with either a reset flag set or a EOF
note:- copy the links and paste in browser

does this  a confirmed issue? I'm using another email ISP(not gmail) which using 465 as secure port, and have simillar issue as @IamPrvn .

@IamPrvn Any progress on this?

@hiscal2015 No, PR is yet to be reviewed/merged .


more workspaces or tags

i like to have more workspaces. maybe 10 like i3gaps i get used to it. which file to modify? i have a feeling this one is the most stupidest question from all i have asked.

Pull the last commits
2)Edit https://github.com/PapyElGringo/material-awesome/blob/master/conf/tags.lua

added more tags with icons.


ERRAI-1075: Add Errai APT Generators mechanism

Description
Having the goal to replace all GWT generators by Annotation Processors, this PR creates a mechanism to generate code using the Java APT for errai-bus and errai-jaxrs initially.
This was achieved by using and improving the existing APT MetaClass implementation so that errai-codegen remains the library for generating code in Errai.
Two annotation processor projects were created. One of them to export relevant files (errai-[module]-export-apt) for an specific module and another to actually generate the code to be used by Web Apps (errai-generators-apt). An adicional module called errai-common-apt was created to share code between the two types of annotation processors.
errai-bus and errai-jaxrs both come with their own export files annotation processors as a transitive dependency, so that every module that depends on them is able to generate its own export files. Export Files are files which contain references to relevant types for an Errai module. [1]

NOTE: The errai-[module]-export-apt modules require maven-compiler-plugin 3.2 or higher.

Usage
Errai APT generators can be configured for a Web App in four steps:

Add an @ErraiApp annotation to any class on the project.
Create a profile named "apt-generators" that can be activated by the "apt-generators" property, includes the errai-generators-apt jar as a provided dependency and sets a property named "gwt.module.prefix" to "AptGenerators_". [2]
Create a module named AptGenerators_[moduleName] that inherits the actual module and overrides a property named "errai.useAptGenerators" with the value "true". (this property is defined in ErraiCommonApt.gwt.xml. errai-bus and errai-jaxrs inherit it and the other modules will too as they have its generators migrated as well. [3]
Add (or modify) the property <module> on the gwt-maven-plugin to be in the format ${gwt.module.prefix}ModuleName.

The property apt-generators will then switch to APT generators instead of GWT generators. (e.g. mvn clean install -Dapt-generators)

NOTE: Every module of your Web App should have been built generating its own Export Files.

Next steps

Migrate the rest of Errai modules to use APT generators.
Parse GWT module files to restrict visible classes to APT generators like the GeneratorContext does in GWT generators.
Skip @GenerateAsync specific generators when APT generators are enabled.

@mbarkley Thanks for your review! I replied your inline comments. The APTMetaClassTest was removed by accident. I'll make sure to bring it back :)


Blosc extension factor out common buffer access code

Factor out common code to obtain buffer pointer with PY2 compatibility for array.array.

Done in https://github.com/alimanfoo/numcodecs


Add partial support for Nordea

Hi, and thanks a lot for making this library!
We used this to integrate with Nordea's Web Services API, and modified the library to support that use case.
If you are interested in upstreaming these changes, here they are :)
I didn't get the certificate requesting part to quite work right with little effort and we don't really need it, with certs generated using other means, so that functionality unfortunately isn't part of this PR. I have included a mention in the readme that the Nordea client lacks the *certificate -methods.
Other WS API calls work though, and I added a new method uploadFile to the client.
I'm happy to answer questions and to try make changes to the implementation if any are needed. As far as I can see, this shouldn't break anything with the Osuuspankki client, but to be honest I don't have a client account there so I haven't really tested either. If possible, I'd recommend testing that it still works before publishing these changes.
Again, thanks for your work

The build seems to fail due to reduced test coverage. I did play around with the pankkiyhteys.test.ts specs and could duplicate some of those Osuuspankki client tests for the Nordea client, but I'm not quite convinced that that would add much real test coverage "safety". The methods tested in those specs mostly fall through to the Client-class, which both of these clients inherit, so that would mostly test the same thing twice. The tests do a lot of mocking and don't (obviously) send request to the actual banks, so they can't really catch errors coming from bank API differences either.

Hi,
Awesome that you found this useful and were able to get Nordea API working! I'll test that my use case with Osuuspankki continues to work with these changes. After that I'll happily merge this.
Don't worry about coverage limits. I think I'll end up removing it.

Thanks, sounds good. No rush on our end, we are currently using our version directly from the github fork and that works well enough for now.
I also did some other chore work, upgraded all dependencies and cleaned up some ESLint warnings here. I didn't include those in this PR because they are kinda irrelevant to the Nordea feature, but I can make another PR if you'd like (perhaps if/when this gets merged).


Try pipeline caching

/azp run net - core - ci

/azp run net - core - ci

Seems to save 1-4mins

@benbp, @weshaggard what do you think?

I didn't know about this feature! I wonder where else this might be useful.


WIP: Cached KV-based store

Summary
A cluster aware, plugin-KV based list store for things like App's and Subscription's.

added an initial implementation, based on pluginapi.cluster.Mutex + local tests
moved the AppStore to use it

Ticket Link
TODO - several tickets
TODO

[ ] Add Clean() to the CachedStore API to use in debug clean
[ ] Document the design
[x] Test command that can be executed on bigger clusters
[ ] Implement data migration
[ ] Restore MessageHasBeenPosted, separate PR?
[x] Alternate, single-writer implementation based OnClusterLeaderChanged + benchmark (ideally in a cluster)
[x] Manually test with cluster
[x] Move the subscription store

@levb Before I jump into the code: Is there a rough overview of the design of the cached store?

@hanzei not yet, still WIP
in brief - for a given CachedStore (e.g. apps, subscriptions) a single index key in the KV holds a list of (key, data-SHA) and the individual items are stored as keyed, in the prefixed namespace. The in-memory index/cache is a sync.Map. KV writes are from whatever host received the request, protected for exclusivity by cluster.Mutex. Plugin cluster messages are sent to all hosts to notify them of new cache entries (PUT, DELETE).
The upcoming single-writer version will send the same cluster messages, but the writes will be serialized on the master, with no need for clusterMutex.Lock()

The upcoming single-writer version will send the same cluster messages, but the writes will be serialized on the master, with no need for clusterMutex.Lock()

What are the advantages of that approach vs the current usterMutex.Lock() one?

What are the advantages of that approach vs the current usterMutex.Lock() one?

Hypothetically:

Fewer DB queries
Better reliability (TL;DR - cluster leader is "server-native", trustworthy, and cheap. cluster.Mutex - homegrown. I specifically do not understand how cluster.Mutex is reset if the locking plugin terminates). The argument about distributing the load - I do not 100% agree with. Specifically, 1/5 consolidating DB writes to a single host may actually achieve better performance; definitely not sub-par unless other overhead (CPU, memory, IO) requires distributed writers.

1/5 I'd much prefer to submit both implementations, with tests to demonstrate the differences.

This PR will be closed, re-submitted as a series of PRs: #457, #458, #459


Fix the missing parallel feature flag for ark-serialize

Description
This appears to be the blocker for a number of PRs since the latest test no longer allows a feature that is not declared in the Cargo.toml.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (master)
[x] Linked to GitHub issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[x] Re-reviewed Files changed in the GitHub PR explorer

N/A:

[ ] Wrote unit tests
[ ] Updated relevant documentation in the code
[ ] Added a relevant changelog entry to the Pending section in CHANGELOG.md

Actually this is not as easy as I think: the problem is that we also implement Valid for things like Rc, but such structure is not "sendable".

Should we comment out the current parallel implementation then? The problem seems to be as follows: parallel check always requires the elements to be sendable to other threads, and therefore this places a limitation

This is a problem introduced by https://github.com/arkworks-rs/algebra/pull/811. I would rather that we revert the part there with removes Valid: Send instead of giving up on parallel check

I made an edit---now it has a different implementation.
Since all the batch_check requests would be reduced to the base primitives, and many trivial data structures may not benefit from parallel check:

now, the parallel check is not going to be applied to Valid in general
the base primitive needs to opt-in to accept parallel check
now, only EC points and pairing element are opted in

limitation of the current implementation: it copies.


Lic

Hi. what is the license for table.py ie the formatter code? MIT? folder w/o license.

Hello, it's MIT


Polling minutes

If I want to modify the 15 min polling in a docker container, which is the launch environment variable name I have to set?

try
FiatChamp_RefreshInterval=15
15min. is the default. hope it will work. not sure if i ever tried other values than the default.


Add mutation tests

Adds mutation testing to CI/CD pipelines. Should run once a week with the goal of maintaining 60% pass rate (mutations are successfully killed)
For the time being, uses a fork of go-mutesting in order to take advantage of several improvements necessary for CI/CD pipeline use that have been sitting in PRs for six~seven months. If/when the parent repo merges those changes, will probably switch back to that and delete the fork.

Had to remove fault/parser from the mutant list. That is autogenerated parser files from ANTLR.


Create monotonic-splines.py

This PR adds an example to the gallery related to the livestream from last Friday. I can make changes to it if you like but figured a PR to be the easiest way to get the ball rolling.

I have read the CLA Document and I hereby sign the CLA

recheck


implement aspect-ratio in avatar component

this PR implements the aspect-ratio mixin within chec-avatar, making it possible to render a responsive 1:1 aspect-ratio avatar, I added a story to demonstrate the use-case.

Here's a screenshot from the story. This helps out making the avatar a bit responsive, currently need this in the beta dashboard for the user content on the nav, when it hover/expands.  (see https://github.com/chec/dashboard/pull/123#issuecomment-650310396 for a screenshot of the user-content overflowing)

Yes, I needed it for the aspect ratio styles to work.
Can modify, what do you think @kvisca?

Needs a rebase for the linting issue - it's fixed in develop.

Needs a rebase for the linting issue - it's fixed in develop.

Rebased