[WIP] Improve index resource usability by adding individual setting fields

Resolves #124
As mentioned in #124, current settings field's usability is low due to the limitation of type(string object with no nest) and causing issues like #108 and #119 .
This PR introduces individual fields to improve usability by allowing json for some fields like analyzer and robustness by having strict key and value type for each setting key.
NOTE

This PR is backward compatible, so the settings field is deprecated but still usable until we make major version update.
The schema is pretty much same as phillbaker/terraform-provider-elasticsearch's index resource.  So it also makes it easier to migrate from phillbaker/terraform-provider-elasticsearch.

TODO

[ ] Support import
[ ] Add tests

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@tobio
Could I ask you to review if the concept of this PR looks good?

@k-yomo yep, I agree this is the right approach to take.
What do you think about keeping the general settings field and merging those with the individual settings?
It looks like your code's already doing that, and it would mean the provider was flexible in supporting new index settings before those individual fields were added.

What do you think about keeping the general settings field and merging those with the individual settings?
It looks like your code's already doing that, and it would mean the provider was flexible in supporting new index settings before those individual fields were added.

@tobio
Totally agree! It would support smooth migration from settings.
And yes, this PR's code is doing merge unless they are conflicting.
Thank you for the review! I'll proceed to complete the PR🙏

@tobio
This PR is also ready for review!
While developing, I noticed it will be problematic to continue to have both individual settings and settings fields. Since it's gonna be confusing when importing, refreshing, and updating.   In fact, currently, settings are not really refreshed because of it.
Thus I believe settings should be removed in the future, and I put a deprecated flag and comments. But let me know if you or the other maintainers have a different opinion.
We can also deprecate settings later if it's controversial.

jenkins test this


out of date readme ?    Web5.did.create("ion"); does not exist

The main repo read me suggests   one can register an ION did right from the cals
import { Web5 } from "@web5/api";
const myDid = await Web5.did.create("ion");
Uncaught TypeError TypeError: Cannot read properties of undefined (reading 'create')
Web5.did  does not exist on the class.     Also not as a function of the instantiated object returned from  web5.connect()
% npm info "@web5/api"  version
0.8.2
https://github.com/TBD54566975/web5-js#web5didcreatemethod-options

Maybe its possible that one must configure the DWN accept plain/text  without any protocol because in the same environment if I configure the Shared ToDo list Protocol from the tutorial the two nodes are able to send to each other.

@fkkfrank, I have updated the pull request with the following changes:

Edited the README.md file to provide clearer instructions for users.
Added step-by-step guidelines for making edits, updating content, and committing changes to the repository.
Included troubleshooting tips to help users address potential issues.
Presented alternative methods for creating Decentralized Identifiers (DIDs) to offer users more flexibility and options.

Please review the changes, and if you have any feedback, suggestions, or specific modifications in mind, please share your thoughts. Your input is valuable in ensuring that these updates align seamlessly with the project's objectives.
look forward to hearing from you.


fix(job-ui-plugin): Remove refresh button that appears when you click start job.

When job is "running" there appears a button.

Oh, no I haven't touched any refresh buttons. I actually think that we saw the "get results button", but that is no longer visible. I think you can close this issue.


Race condition with CLI spinner

When running some commands, such as the non-working bacalhau docker run ubuntu hello, the spinner will show:
Finding node(s) for the job  ................ err  ❌  02m01.4s

Error while executing the job.
Error submitting job: Could not start spinner.: spinner already running or shutting down

This appears to be caused by the spinner being restarted when it is still shutting down from the previous use.
We might simplify the WaitForJobAndPrintResultsToUser function (https://github.com/bacalhau-project/bacalhau/blob/46ac14168d6887c701234a25ec7126bc38573787/cmd/bacalhau/utils.go#L586)  to make it a little simpler and use the version of the spinner in spinner.go.

Do you know if there is any temporary work around for this bug or which types of strings trigger this condition? Ie I've tried removing and re-installing the bacalhau cli, but it doesn't seem to help.
Testing with this command:
bacalhau docker run -v QmTYadG6K4LocouLu6wFzeMXNB5z8ikdpwuGqpdZNxp5qR:/project/inputs/LC08_L1TP_001028_20220615_20220627_02_T1_mosaic.tif -o outputs:/project/outputs/ ${USERNAME}/segmentation_testbed:latest

Using  --id-only should help, but you'll have to rely on describe to check in on it.
bacalhau docker run --id-only -v QmTYadG6K4LocouLu6wFzeMXNB5z8ikdpwuGqpdZNxp5qR:/project/inputs/LC08_L1TP_001028_20220615_20220627_02_T1_mosaic.tif -o outputs:/project/outputs/ ${USERNAME}/segmentation_testbed:latest

Was fixed in #2236


Update the README for Swift 3

Update the usage instructions and code examples for the Swift 3 API.

Codecov Report

Merging #125 into develop will not impact coverage.

@@           Coverage Diff            @@
##           develop     #125   +/-   ##
========================================
  Coverage    94.75%   94.75%
========================================
  Files            6        6
  Lines          381      381
========================================
  Hits           361      361
  Misses          20       20

Continue to review full report at Codecov.

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


Empty list doesn't contain element at index 0.

Hi, I encountered an "Empty list doesn't contain element at index 0" error when trying to post a picture on Instagram using the postPicture method. The error seems to be related to an issue with the uploadPicture or configure stage of the process. Below are the details of my code and the error stack trace.
It occurs right here:
   protected static String getCrsf() throws IOException {         Request request = (new Request.Builder()).url("https://www.instagram.com/api/v1/web/login_page/").headers(Headers.of(Constants.BASE_HEADERS)).addHeader("user-agent", "Mozilla/5.0").get().build();         Response response = client.newCall(request).execute();

First, correct the code blocks. Secondly, share your piece of code that is causing the problem along with the picture you are posting

One, if you're going to help, please keep it concise.
Two, I understand you're from india, but that doesn’t prevent you from writing clean, high-quality code. Take some time to improve your skills.

classic ind*an developer. don't use this useless library.

One, if you're going to help, please keep it concise. Two, I understand you're junior, but that doesn’t prevent you from writing clean, high-quality code. Take some time to improve your skills.

(In the same manner you replied)
One, I just asked you for the code and the image you are trying to post to debug the issue. Second, I think you didn't read the CODE_OF_CONDUCT.md or don't know how to behave. You wrote India and then edited it to junior after thinking for total of 3 minutes.
The code is clear and understandable, Take some time to improve your understanding

Yes you are right :)

classic ind*an developer. don't use this useless library.

Now it's clearly understandable that the your problem is not the code, but a specific country

One, if you're going to help, please keep it concise. Two, I understand you're junior, but that doesn’t prevent you from writing clean, high-quality code. Take some time to improve your skills.

(In the same manner you replied)
One, I just asked you for the code and the image you are trying to post to debug the issue. Second, I think you didn't read the CODE_OF_CONDUCT.md or don't know how to behave. You wrote India and then edited it to junior after thinking for total of 3 minutes.
The code is clear and understandable, Take some time to improve your understanding

I know it's hard to find healthy food there, but I recommend eating meat so your brain can develop. You said, 'In the same manner you replied'. My attitude developed that way because your initial approach was disrespectful. Then again, who am I even explaining this to? You won't understand anyway :)


command not found: theme ,even after calling echo or export

Hi,
I am working with Theme Kit last few weeks. It worked perfectly. However, today I tried to start my environment(in same folder where I was working on Friday). All commands with theme are not working because of this message zsh: command not found: theme .
I tried to reinstall Theme kit (installation without any problems), even manually put export path to .zshrc and .bashrc...both options had no result. I am desperate and no longer know what to try.
Here is for example shown one of many attemps.

Thanks for your advices.

@Tr1po how did you solve this? I’m having a similar problem

@shanetheboland

did you run the commands outputted from the installation script?
did you verify that theme exists at ~/.themekit/theme?
are you using a bash shell or something else?

Figured it out.

My .bash_profile was a mess.
I added PATH="/Users/shane/.themekit:$PATH" to fix this.
I had to uninstall the shopify_theme gem. It also uses the theme command.


Creating dynamic menu items

I am trying to create dynamic menu items but not sure how to code the following. I have a midiManager that will print an array of midi devices. I want to create a midi item that has the name of each device
I am getting the following error
Argument type '()' does not conform to expected type 'MenuElement'
if as use as I get a different error
 StandardMenu(title: "Midi Devices")
            {
              midiManager.endpoints.outputs.forEach { device in
                TextMenuItem(title: device.name) { _ in
                  print(device.uniqueID)
                }
              }
            }

You're not far from having it.  Based on what it looks like you are trying to do in your code example, this should do the trick:
StandardMenu(title: "Midi Devices")
{
    ForEach(midiManager.endpoints.outputs)
    { device in
        TextMenuItem(title: device.name)
        {
            print(device.uniqueID)
        }
    }
}


Add Multi OS Build Pipeline

Adds a build pipeline that build on Windows, Alpine, and OSX

I recommend to merge this pr as a squash and merge, since I had to commit a lot of times to have the pipeline trigger with new versions


Implement a content security policy

Final cleanup on #4244.

Rebased, merge commit was 3b2872fe0229d833c354e0e2ba0886d4e5b52310.

@EdOverflow If you're good with 1f71e8c88553b2158a7b07c257f874fdfe9e59c8 then I will merge to master and deploy. Our general habit is to make PRs from branches in our main repo rather than forks, so that its easy for multiple people to add commits. Would you be willing to make an onboarding ticket for yourself so we can get you plugged in as a collaborator so you can make branches in the main repo here?

OK, I went ahead and submitted an onboarding ticket: https://github.com/gratipay/inside.gratipay.com/issues/952 :)


Update mlflow.py

Fixes #6745: adds additional info about the run, as in the native API.

Hello @stllfe! Thanks for updating this PR.

In the file pytorch_lightning/loggers/mlflow.py:

Line 39:1: E302 expected 2 blank lines, found 1
Line 42:1: E305 expected 2 blank lines after class or function definition, found 1
Line 49:1: W293 blank line contains whitespace

Any suggestions on code formatting and typing checks?

@stllfe mind check the failing test?

@Borda done! Removed that _MLFLOW_AVAILABLE so that resolve_tags would be defined either way. However, it seems there still are some failed tests, idk why though... Can you check it? I'm not that familiar with pytest :(

However, it seems there still are some failed tests, idk why though... Can you check it? I'm not that familiar with pytest :(

The failed tests are unrelated.
Can you also update the CHANGELOG?
Do you see a way to test this?


BrotliInterceptor V4.1.0

cannot resolve symbol BrotliInterceptor .
Please help how to get it working

Did you add the okhttp-brotli dependency (https://search.maven.org/artifact/com.squareup.okhttp3/okhttp-brotli/4.1.0/jar)?

Yes but still not able to implement in java :
OkHttpClient client = new OkHttpClient.Builder()
               .addInterceptor(new BrotliInterceptor())
               .build();

It gives error that BrotliInterceptor() has private access
Please help how I can implement in java

I’ll add an example. There is a singleton instance of it so use the static field.

Waiting for your reply

@krishnameena Can you try following these instructions https://github.com/square/okhttp/pull/5391/files

Thanks it is working now


Added my profile links

Proposed Changes
added my profile links to the public file

Do I need to make a new PR after the changes made by Schmelto? Or this PR is good to be merged?

good to be merged


ocamlPackages.lustre-v6: update hash

Description of changes
https://github.com/NixOS/nixpkgs/pull/206095#issuecomment-1405046100
Things done

Built on platform(s)

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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg build ocamlPackages.lustre-v6

Thanks.


Draft: kernel: Add a patch to fix an NPD when using the UART

When /dev/ttySTM1 is closed while there are still some characters pending to be sent, the kernel dereferences a null pointer which locks up the kernel.
Fix that by picking a patch from today's linux-next.
While the commit log claims this indeed fixes the problem noticed by @SmithChart, this isn't tested yet.

This seems to solve part of the problem. The good thing is that the kernel errors seen when closing a /dev/ttySTM* while there is still pending data is gone and now there is only a
$ dmesg -w
…
[   68.147733] stm32-usart 4000f000.serial: Transmission is not complete
…

in its place.
But it gets worse when one tries to use the device again:
$ dmesg -w
…
[   78.434633] panel-mipi-dbi-spi spi2.0: SPI transfer timed out
[   78.439387] spi_master spi2: failed to transfer one message from queue
[   78.450284] spi_master spi2: noqueue transfer failed
[   78.454254] panel-mipi-dbi-spi spi2.0: error -110 when sending command 0x2a
[   78.461289] lmp92064 spi1.0: SPI transfer timed out
…

followed by a watchdog induced reboot.
To me this looks like e.g. lower-priority interrupts no longer being handled.

followed by a watchdog induced reboot.
To me this looks like e.g. lower-priority interrupts no longer being handled.

What @hnez reports still sounds like the behavior I've observed.

Closed by mistake, I think by accidentally pushing a wrong rev to my branch. Anyhow, now it should be right and fixed.

I've tested this PR now and it looks like it fixes the issues we had. Nice to see this endeavour come to an end!
Before we merge it however #136 should be merged first and this PR be rebased so that the version number consistency check does not throw errors.

#136 has been merged. Please rebase.

(Pushed once more because I got the committer info wrong in the first force-push)

I don't know what the version consistency check is about, but I rebased anyhow.

It makes sure that we do not generate any images containing e.g. 24.04 as a version number after the 24.04 version tag is already set and instead have 24.04+dev as a ver


Unity 2022.2 Fails to serialize objects to json properly

Bug Report
2023/05/25 12:16:36.835 9596 9634 Error Unity JsonSerializationException: Unable to find a constructor to use for type ElevenLabs.Voices.Sample. A class should either have a default constructor, one constructor with arguments or a constructor marked with the JsonConstructor attribute. Path 'samples[0].sample_id', line 1, position 78.
2023/05/25 12:16:36.835 9596 9634 Error Unity   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateNewObject (Newtonsoft.Json.JsonReader reader, Newtonsoft.Json.Serialization.JsonObjectContract objectContract, Newtonsoft.Json.Serialization.JsonProperty containerMember, Newtonsoft.Json.Serialization.JsonProperty containerProperty, System.String id, System.Boolean& createdFromNonDefaultCreator) [0x00000] in <00000000000000000000000000000000>:0
2023/05/25 12:16:36.835 9596 9634 Error Unity   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject (Newtonsoft.Json.JsonReader reader, System.Type objectType, Newtonsoft.Json.Serialization.JsonContract contract, Newtonsoft.Json.Serialization.JsonProperty member, Newtonsoft.Json.Serialization.JsonContainerContract containerContract, New

Overview
Execution Error when  call  api.VoicesEndpoint.GetVoiceAsync(IdElevenLabs) which Unity 2022.2.20 and URP
To Reproduce
Steps to reproduce the behavior:

Create a new project in Unity 2022.2.20
Configure for Android with I2CPP
Add this pakage
Put the api key
Put a script with this in the start method:

        ElevenLabsClient api = new ElevenLabsClient();
        Voice voice = (await api.VoicesEndpoint.GetVoiceAsync(IdVoiceElevenLabs));

Buid & Run and it woks OK with no error
Add URP Pakage and configure Graphics in Player settings to use it
Build & Run and produce an error when execure GetVoiceAsync

Expected behavior
No error and recive a correct voice return
In Unity  2021.3.24f1 works good with no error

@fernanbg what should IdVoiceElevenLabs be set as?

So the error only crops up when URP is installed?

@fernanbg seems to be a bug with Unity 🤔
I tested with 2021.3 LTS and it seems okay.
When I test with 2022.2 the json serialization seems to always return an empty json string.

@fernanbg try updating the newtonsoft package 3.2.1

Hmm updating to 3.2.1 didn't help either.
I also tried changing from .net standard to .net framework still same results

I have found that perhaps it is a problem with IL2CCP in Android and the UnityLinker's bytecode stripping.  But I don´t know what happens only when I put URP in the project, and with the 2022.2 Unity version.
https://github.com/jilleJr/Newtonsoft.Json-for-Unity/wiki/Fix-AOT-using-link.xml

It's failing for me even without URP

That's a great link for the Newton soft stuff but I've already mitigated the problem using the following:
https://github.com/RageAgainstThePixel/com.rest.elevenlabs/blob/08ab1489438e55004a26445614b97a4dfd465b01/ElevenLabs/Packages/com.rest.elevenlabs/Runtime/AssemblyInfo.cs#L7

Also we're using unity's Newton soft package directly

@fernanbg I just tried with 2022.2 LTS and it worked when I changed Managed Stripping Level down to Minimal.
Crazy cuz I have both the link.xml and the preserve and link assembly attributes in there and it is still getting stripped from the build. Crazy.


[WIP][KYUUBI #999] Build the basic framework for rest frontend service

Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #1003 (142e2e9) into master (01bebe3) will increase coverage by 0.01%.
The diff coverage is 90.10%.

:exclamation: Current head 142e2e9 differs from pull request most recent head 36fa671. Consider uploading reports for the commit 36fa671 to get more accurate results

@@             Coverage Diff              @@
##             master    #1003      +/-   ##
============================================
+ Coverage     80.35%   80.37%   +0.01%
  Complexity       14       14
============================================
  Files           154      158       +4
  Lines          5626     5717      +91
  Branches        647      652       +5
============================================
+ Hits           4521     4595      +74
- Misses          719      733      +14
- Partials        386      389       +3

Impacted Files
Coverage Δ

...pache/kyuubi/server/api/v1/ApiRequestContext.scala
50.00% <50.00%> (ø)

...in/scala/org/apache/kyuubi/server/api/v1/api.scala
80.00% <80.00%> (ø)

...org/apache/kyuubi/server/RestFrontendService.scala
89.65% <89.65%> (ø)

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
95.23% <90.00%> (-0.12%)
:arrow_down:

.../apache/kyuubi/server/api/v1/ApiRootResource.scala
100.00% <100.00%> (ø)

.../org/apache/kyuubi/operation/KyuubiOperation.scala
38.18% <0.00%> (-16.37%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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

thanks merged to master


ERROR: MAX_TRIANGLES_BATCH overflow

I am making a game which prints a whole bunch of hexagons - each made using DrawPoly.
After putting down a bunch of tiles,  the program closes, and I see the error message "MAX_TRIANGLES_BATCH overflow".
Which seems to come from this line:
https://github.com/raysan5/raylib/blob/53280a56e3f4ab576fafeb75a68031fcdc4089fa/src/rlgl.c#L677
It looks like this is defined here:
https://github.com/raysan5/raylib/blob/53280a56e3f4ab576fafeb75a68031fcdc4089fa/src/rlgl.h#L98
It's a 36 x 19 map, and I'm using 2 hexagons for each tile (1 for the tile, 1 for a shadow of the tile).  At 684 hexagons, it crashes(4104 triangles)..
How should I get around this issue?
I can probably find a way to render this with fewer triangles, but is this a hard upper limit?
Thanks!

Hi @SethArchambault! Thanks for pointing this issue! Also related to https://github.com/gen2brain/raylib-go/issues/12 and https://github.com/raysan5/raylib/issues/267.
I already solved this issue for MAX_QUADS_BATCH but not for MAX_TRIANGLES_BATCH, my idea was replacing all elements drawing just by quads instead of using triangles, that way, I have to manage internally only one vertex buffer (quads) instead of three (lines, triangles, quads).
I'll correct this issue in the following days, in the meantime you can just change value in rlgl.h and recompile the library with Notepad++ script raylib_source_compile or using provided Makefile.
#define MAX_TRIANGLES_BATCH     8192

Solution implemented in commit https://github.com/raysan5/raylib/commit/4dcc02ff1ef4d6ab6c10ece84c47892f12290908
It's not the best solution but it will work for now.


fix: bug where mulit_gpu was being set and warning being printed even with num_processes=1

What does this PR do?
Fixes a bug in the accelerate launch command which enables multi_gpu even with num_processes being set to 1.
Fixes # (issue)
https://github.com/huggingface/accelerate/issues/2920
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


TODO

[ ] add pagination
[ ] add back clustering
[ ] update mapbox.js
[ ] clean up jquery

Personally, I didn't like clustering, so having a toggle for that in the map generating method would be :+1:

@deltamualpha, yeah actually I'm feeling the same way. Just took it off the list :sparkles:


Auto-Complete PDO class.

Please add Auto-Complete support for PDO class.

same with you.. need PDO class
+1

+1

+1

+3 :+1:


Add storage object create form in dev console

It would be pretty handy to be able to create objects directly in the dev console without having to import.

@oscargoldman Nice suggestion 👍
In the meantime we do have a way to import storage objects in JSON or CSV format into the game server. These objects will become owned by the system user.

yup, have been using it to import our existing schema. Took me a while to get the scripts to escape the value node properly but it's working now.


Support custom deployments for github_app.py and add more options for automatic review actions

Some background:
In our team we can't use the PR-Agent GitHub App (for internal reasons), so what we did is fork the repo and update the github_app.py code to allow us to deploy it as a listener server in our environment, that listens to GitHub webhook events (specifically: pull_request and issue_comment) that are sent to us from GitHub through a messaging queue. The reason we used the existing github_app.py code instead of creating a new file for our server, is because this way we can always pull upstream changes to github_app.py and merge them with our code.
The main changes in the github_app.py to support this deployment:

Allow deploying the app in "user" mode instead of "app" and setting the user name for our bot user (not always github-actions[bot])
Control certain verification code that is not relevant in our deployment
Handle duplicate requests that may be generated by the messaging queue that passes the github webhook messages to our server (due to long request handling times)

Additionally, we wanted to give the app the ability to do more stuff automatically:

Add handling for pull request actions such as "opened"/"reopened"/"ready_for_review" (moved from Draft to Open) to automatically trigger the PR Agent
Control the commands that the PR Agents will execute once triggered (for example: "/describe" followed by "/auto_review" instead of just "/auto_review")

In this PR I tried to generalize our "custom" github_app.py to the best of my abilities in hopes that it will be useful for other users who might have similar deployment architecture. And at the same time, maybe you will find the new automatic review triggers useful.
This PR should not affect the ability to run the github_app.py as a GitHub App as it is done today, the only apparent changes are:

Automatically triggering the PR Agent when a pull request is opened, reopened, ready for review (moved from draft to open), and when the bot user is specifically requested as a reviewer in an open PR.
When the PR Agent is automatically triggered, it will run "/describe" followed by "/auto_review" instead of just "/auto_review"
Both of these changes are completely configurable, and it's completely possibly to keep the current behavior of the app by simply changing their defaults.

@zmeir so we're ready to merge?

@zmeir so we're ready to merge?

Yes :)


Simple Test case crashes server

Server code
var WebSocketServer = require('ws').Server
, wss = new WebSocketServer({port: 1337});
wss.on('connection', function(ws) {
ws.on('message', function(message) {
console.log('received: %s', message);
});
});
Code from Mozilla scratchpad
var ws = new MozWebSocket("ws://localhost:1337");
ws.onopen = function()
{
for(var i = 0; i < 10000;i++)
{
    ws.send("<mary>had a little lamb</mary>");
}

};

Thanks, I'll look into it in a few.
On Feb 6, 2012 9:44 PM, "cmgit" <
<EMAIL_ADDRESS>wrote:

Server code
var WebSocketServer = require('ws').Server
, wss = new WebSocketServer({port: 1337});
wss.on('connection', function(ws) {
ws.on('message', function(message) {
console.log('received: %s', message);
});
});
Code from Mozilla scratchpad
var ws = new MozWebSocket("ws://localhost:1337");
ws.onopen = function()
{
for(var i = 0; i < 10000;i++)
{
ws.send("had a little lamb");
}
};

Reply to this email directly or view it on GitHub:
https://github.com/einaros/ws/issues/20

Alright, I've dealt with this issue and published 0.4.4 to npm. See if that helps on your end as well.

Looks like it's working! Thanks!


Mochi 1 not creating anything at all

Your question
All available backends failed to load the model 'H:\SwarmUI\Models\Stable-Diffusion/OfficialStableDiffusion\mochi1PreviewVideo_fp8Scaled.safetensors'. when i try to do a text to video.
Logs
https://paste.denizenscript.com/View/128250
Other
No response

Use this copy of the model: https://huggingface.co/Comfy-Org/mochi_preview_repackaged/blob/main/all_in_one/mochi_preview_fp8_scaled.safetensors
The ones you have are diffusion_models versions, which naturally go in the diffusion_models folder and need some special handling

Something went wrong while generating images.
Looks like its doing something then just as its finished it throws that error.
https://paste.denizenscript.com/View/128252

That type of sudden crash without an error message, especially right while it's starting up the VAE, implies out of memory crash.
See the notes in the docs @ https://github.com/mcmonkeyprojects/SwarmUI/blob/master/docs/Model Support.md#genmo-mochi-1-text2video re VAE Tile params


[WIP] Update Readme with genesis overriding examples

What was wrong?
#123 did not include an update to the Readme.
How was it fixed?
Provided walkthough and code samples explaining how to build and use custom genesis state and parameters.
Fixes #132
Cute Animal Picture

Sorry to keep this on WIP for so long - I've been quite busy.  Ill be back to give it another go soon.

Following up again here - I've been using this version for some time now, and use the documented API to continue development.
I'm still feeling somewhat unable to tend this PR due to time constraints; However, I'd still like to remove the WIP and follow-up with any requests for changes on the current version (d83afef for posterity).
Thanks ahead of time for looking! :cactus:


[Snyk] Upgrade org.mybatis.generator:mybatis-generator-core from 1.4.0 to 1.4.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade org.mybatis.generator:mybatis-generator-core from 1.4.0 to 1.4.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 2 months ago, on 2022-03-08.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

@murphysec-service test


hypervisor: support user space mounts

This depends on #43
This commit introduces the ability to mount custom files and folders
from the host operating system onto a partition.
The hypervisor configuration is adjusted as follows at partition layer:
mounts:
  - [ "/dev/urandom", "/dev/urandom"]
  - [ "/dev/urandom", "/dev/random"]
  - [ "/a/file/on/the/host", "/guest" ]

Future features could additionally include the ability to mount things
read-only, etc.
Fixes https://github.com/aeronautical-informatics/apex-linux/issues/42
cc @wucke13

Rebased

@emilengler Thank you!


Share with Pypi

Then you can install it with pip install portSpider and download all dependencies.

Thank you, I considered it, but it would take a lot of effort and it would'nt really worth it since updates and fixes are more complicated that way.


Down orientation does not work on iOS (Android not tested)

The down orientation does not work properly on iOS. Has it been tested? I have not tested Android.

See https://github.com/mastermoo/react-native-action-button/pull/113.


Add additional separator keys for tag creation

What this does
Allows more keys to create tags than just enter key. It is an optional prop called separatorKeys which is an array of numbers. I'm excited to use it on commas, spaces, and tabs.

PR probably not needed if either of these other PRs are accepted: https://github.com/pathofdev/react-tag-input/pull/15 ... looks like a lot of us are into tabs, spaces, and commas!

This PR also adds a small amount of code to apply text on blur to tags (useful for mobile)

If anyone stumbles onto this PR in the future, https://github.com/yairEO/tagify seems to be a well maintained alternative. I haven't had the opportunity to test it out extensively yet, but it seems like it has all the features of this library and some (including the one this PR adds).

@alexanderl19 Thx it looks awesome


[wip] Add conditional for hiding bills with restricted views

This PR adds a conditional to the is_visible property: it explicitly hides bills marked as having a restricted view in Legistar.
It works in cooperation with https://github.com/opencivicdata/scrapers-us-municipal/pull/258, https://github.com/opencivicdata/python-legistar-scraper/pull/81, and https://github.com/datamade/django-councilmatic/pull/232
N.b., I'll get the tests passing once I can add the above-mentioned changes in Django Councilmatic to the testing requirements.

Overview
With the newly provided API token, DataMade scrapers now have access to private Metro bills. But….why do we need to scrape private bills?
We need to know which bills to show and which bills to hide on Councilmatic: the MatterRestrictedViewViaWeb field tells us which bills are public (show!) and which are private (hide!). The process:

scrape bill and include the value of MatterRestrictedViewViaWeb in the extras (https://github.com/opencivicdata/scrapers-us-municipal/pull/258 and https://github.com/opencivicdata/python-legistar-scraper/pull/81)
import bills and indicate if they are restricted (https://github.com/datamade/django-councilmatic/pull/232)
hide or show bills based on this value. (this PR)

But wait. Why not just skip private bills? Sometimes, Metro will toggle bills between public and private, meaning that the Councilmatic system may have bills that were previously public. Scraper needs to access those now-private bills, so that Councilmatic can make sense of their new status.
Not knowing which bills had changed between public and private has caused issues in the past:
https://github.com/datamade/la-metro-councilmatic/issues/345#issuecomment-455683240
https://github.com/datamade/la-metro-councilmatic/issues/328


Angular 15 Support

Angular 15 released recently, If possible it would be nice to see an angular 15 compatible release.

Hi @Destreyf . The current build should be compatible with angular 15. Are you facing any errors?

I cannot install due to the package.json dependencies tying it to angular
14.
On Sat, Nov 19, 2022, 1:25 AM Sibiraj @.***> wrote:

Hi @Destreyf https://github.com/Destreyf . The current build should be
compatible with angular 15. Are you facing any errors?
—
Reply to this email directly, view it on GitHub
https://github.com/sibiraj-s/ngx-tiptap/issues/30#issuecomment-1320833466,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHMFCGBQ7JWOR3F5KUKSSDWJCFIXANCNFSM6AAAAAASE4O6KY
.
You are receiving this because you were mentioned.Message ID:
@.***>

At least that's the error I got earlier today, I'll be back at my computer in 8 to 10 hours and i will double check

Yes. It should support any version greater than 14. See https://github.com/sibiraj-s/ngx-tiptap/blob/master/projects/ngx-tiptap/package.json#L18.
If there is something else please share the error. Will try to make a release.

I have done some additional testing, it appears it should work, we had some other dependencies conflicting that are causing us to delay our migration but as @sibiraj-s stated, it should be compatible, I believe we had a slightly older version at the time we originally tried to upgrade.
Part of the confusion on this is that we use NX internally and it didn't complain about any dependency issues in it's migrations, but once we tried to reinstall at a later time a bunch of packages were broken, this was one of them but it no longer complains about ngx-tiptap so I'm going to close this.

Thanks for the update @Destreyf


rest calls implementation for the websocket api

this only allow the user to command the websocket with api calls (start, keep alive and close websocket stream)

wrong files committed, working on a fix


GPL-368 As a dev team we would like the validation error message propagated to Sequencescape when a tube rack fails validation

Can you update the sequence flow doc: https://www.lucidchart.com/invitations/accept/fb09193a-4b0b-41c4-8a4d-ee250f1ef539
ta

Done!


Kustomize

Goals:

qat plugin to be deployable with kubectl apply -k qat_plugin/VERSION (VERSION being devel or a release tag)
gpu plugin to be deployable with kubectl apply -k gpu_plugin/..., possibly configurable command line parameters if needed
fpga plugin to be deployable with kubectl apply -k fpga_plugin/..., different kustomizations for different run modes (preprogrammed vs orchestrated)
and support for other plugins (Movidius) as well

PRs contributing to this issue:

QAT plugin - PR #286
GPU plugin - PR #288


Fixed axe violation for keyboard accessibility of scrollable content

Summary
Fixes #2716
Additional Details
Changes in terra-scroll also fixes the issue for terra-dialog and terra-content-container

@saket2403 There are some test examples in terra-image and terra-responsive-element that are also having this violation when using axe-core 3.4.0. These are the test examples I see that needs updated.

After discussing the implications of the changes on this PR for addressing this rule introduced in axe-core-3.3.0, we decided to hold off on these changes and disable this new rule for now in terra-toolikit. We will revisit this rule once we have a better plan on how to address it. I will close this PR after I disable the rule to ensure the violations no longer appear.

Closing this and we will visit how to properly handle accessibility with scrollable content when we tackle #2787.


RaygunTraceListener sometimes causes infinite blocking on call to Trace.TraceError

I've just run into an issue today where I noticed my application locking up on a call to Trace.TraceError. At first I assumed it was an issue with our own custom color console listener but after some investigation the culprit seems to be RaygunTraceListener. This is a multi-threaded console project application that is using Mindscape.Raygun4Net v5.3.1 and Raygun.Diagnostics v1.2.0. In this application Trace.TraceError is used successfully and at other times it does not and blocks the output so I'm guessing this may have to do with internal handling perhaps not being thread safe. It's also important to note that in particular places we can get Trace.TraceError to consistently hang each time.

Here is a trace of the calls that I see when this happens.

Interesting. In my code I had a block like this where I was testing this issue.
this blocks indefinitely for me
Trace.TraceError("Error");

Based on the call stack that I pasted above I tried using the RaygunClient directly instead to send a message and when I added that line before the TraceError call not only did the call to Client.Send succeed but the call to Trace.TraceError also no longer hangs and I've verified that both messages make it to Raygun.
this works without hanging
Raygun.Diagnostics.Settings.Client.Send(ex);
Trace.TraceError("Error");

Assuming that this may have something to do with the type of message that is sent to the Raygun client I attempted to formulate a similar message directly and send it like so.
var listener = Trace.Listeners.OfType<Raygun.Diagnostics.RaygunTraceListener>().Single();
var message = listener.MessageFromTraceEvent(null, "source", TraceEventType.Error, 1, "Error");
listener.WriteMessage(message);

This however worked with no problem and once again surprisingly any subsequent calls to TraceError also succeed without hangling. After a few itterations of this example to minimalize the effect I have the following which also succeeds.
var listener = Trace.Listeners.OfType<Raygun.Diagnostics.RaygunTraceListener>().Single();
var message = listener.MessageFromTraceEvent(null, "source", TraceEventType.Error, 1, "Error");
Trace.TraceError("Error");

The two notable things from above is that I didn't have to actually send a message but only to call the MessageFromTraceEvent method to change the undesirable hanging behavior of Trace.TraceError. Also note that I'm passing null for the TraceEventCache parameter mainly just because I don't have access to it here and this may impact the behavior.

The most recent test I've done just makes the behavior seem even more arbitrary which tells me that this may be a deeper issue.
this also works without hanging
var client = Raygun.Diagnostics.Settings.Client;
Trace.TraceError("Error");

Unfortunately I'm going to have to remove RaygunTraceListener from my projects until I can get this straightened out.

Hey @jpierson , sorry about the issue. Where are you setting your raygun key?
I've not been able to reproduce the blocking you are seeing, so the first thing that comes to mind is a configuration error. I also spent a few mins looking at mindscape's code for the raygun .NET provider. They are using WebClient.UploadString(), so it's possible that could block...though they have good error handling and fail back. What I did discover that was interesting, is a SendInBackground method on the RaygunClient...that offloads the Send() to a backround thread using QueueUserWorkItem. So by definition, that should not block the main thread. I'll look into supporting that as an option through the trace listener...it would be a nice option to have.

Another thought on the issue. Can you try adding a secondary listener? If the raygun trace listener sees another listener and an exception occurs within the code, it will push that exception to the other trace listener in your app.

I did have a secondary listener stacked after RaygunTraceListener and that is how I found the issue originally. I was intentionally throwing an exception, catching it, and then tracing it but I wasn't seeing it get picked up in the trace listener. In my tests it doesn't appear that an exception happens it just blocks on the call to Monitor.Enter as shown above. Putting my custom trace listener before RaygunTraceListener at least allowed me to see those traces but then I still have the indefinitely hanging issue.

I should add that it appears that this may be if anything pointing to an issue in the Raygun Client implementation or within the WebProxy implementation but since I don't yet have a way of reproducing a similar issue with either of those directly I'm unsure how to follow up there.

@jpierson what version of .NET are you targeting?

.NET Framework 4.5.2

Hmmm, I'm no longer able to reproduce this issue :( The only significant thing I've done is uninstalled and reinstalled the Nuget packages for both the Raygun Client and the Raygun.Diagnostics. My concern however is that it may happen again in production.

Not surprised that a reinstall fixed it, that has happened to me many times with other libraries.
For comparison, we are running about 20 apps in production with v1.2 and are not seeing the issue. But let's keep the issue open for a while just in case. I'll create a separate issue for adding background send support to the trace listener.


[Issue #29] (Team 2) Dynamic Programming Algorithm for Query Rewriter

Added English dictionary file, with frequencies.
Modified FuzzyParser to implement the dynamic programming algorithm, implemented by @JavierJia which is referenced in the comment thread, under #29
Modified the QueryRewriterTest to reflect the new expected results from the QueryRewriter

@chenlica : Can you give this a review?

Actually QueryRewriter is the main class... It in turn calls the FuzzyTokenizer class to detect missing white spaces. The reason we have separated the two classes is because later, QueryRewriter would also perform "synonym" matching as well.

@chenlica : Added an example in the comments as suggested... Kindly take a look

In the QueryRewriter class, is it better to make fuzzyTokenizer.getFuzzyTokens() a static function if it only needs a query string?

High-level comments:

Make FuzzyTokenizer.getFuzzyTokens() a static function.
Possibly rename FuzzyTokenizer to QuerySegmenter.  If you don't agree, that's fine.
Wait for @JavierJia to review the DP algorithm.

Refactored FuzzyTokenizer to QuerySegmenter
Refactored phrase to queryString
Refactored getFuzzyTokens to getTokens
Converted getTokens to static function and made corresponding changes to QueryRewriter

@chenlica : Kindly take a look.

To unblock you, no need to wait for the review from @JavierJia .  Go ahead to do the merge after making those changes.  We will let @JavierJia doublecheck the logic even after the merge.

Looks good.  Please go ahead to do the merge!

Missed refactoring a variable name...

Looks good.  Please go ahead to do the merge.

Overloaded QueryRewriter constructor to accept flag for returning all possible tokenizations
Included corresponding test cases in QueryRewriterTest
Cleaned some noisy entries in word-frequency map

@chenlica : Can you take a look at the changes?

I left some minor comments on your commit.  After taking care of them, please do the merge.  Thanks!

@chenlica : Refactored changes as suggested. I am going ahead with the merge.
Thank you, Professor.


Merge code updates from the robotframeworklexer upstream

The upstream has pushed some changes not yet in the Pygments version of the lexer:

Add But to GherkinTokenizer
Stricter header and setting syntax: 1) Space-sensitivity; 2) Unknown headers are marked as errors

and probably more.
If nobody volunteers, I may find some time and submit a pull request myself, but it would probably better if someone familiar with the code has done so.
cc @pekkaklarck, @birkenfeld

Thanks @andrewshadura for bringing this up! There have been also other changes to Robot Framework syntax that aren't yet supported by our old external lexer project that basically is the current Pygments plugin. The main reason we haven't updated the external lexer is that Robot Framework 3.2 contained a new parser that, unlike the old one, has a separate lexer component that is generally used by tools needing to parse Robot Framework data. Updating the external lexer has been on my todo list but because it's basically only used by Pygments, and because there's still effort to get Pygments to use it, it hasn't been too high on the list. Now that you @andrescarrasco are interested on helping with the latter task, this whole thing gets higher priority.
The main question here is how should the external lexer be updated now and later if/when Robot Framework syntax is enhanced. Having an external lexer that needs to implement same rules as the built-in one is not too nice from maintainability point of view, but Pygments vendoring the whole robot package is probably even worse solution. Some ideas what we could do include:

Just bite the bullet and update the external lexer. Syntax changes are relatively rare and not immediately supporting all latest features probably isn't a huge deal.
Recreate external lexer based on the new built-in lexer. A problem here is that the current external lexer is just a single file that's very easy to vendor by Pygments while the built-in lexer is a package containing multiple files. A bigger problem is that the external still needed to be updated in the future when the built-in one is updated.
Just directly copy the built-in lexer code into Pygments and deprecate the external lexer altogether. This requires making it sure the built-in lexer has a stable API that allows copying the code. Since the Pygments lexer probably only needs our current get_tokens (and possibly get_resource_tokens and get_init_tokens) which is part of our public and stable parsing API this shouldn't be a big problem.
Use Robot Framework as an external dependency for Pygments and use the aforementioned public and stable parsing API. Could pin the version to make sure possible API changes don't accidentally break something. If Pygments already has external dependencies this probably would be the easiest solution. Requiring the whole Robot Framework installation would mean also code not used by the lexer would be installed, but I doubt that's a big problem in itself.

Any comments by others how should we move forward? I'd be happy to help regardless what approach we choose.


Faster Player

By default in iOS 10 and later, the AVPlayer stalls until it calculates that it could play until the end of the AVPlayerItem at the current download rate. However, for large podcast audio files this calculation is iffy and leads to big delays when a new track is put into the Player, even though we can safely start playing right away. This PR makes this small change to make the Player play immediately by default.

Looks good merge it


Head rotation

When you are in a mothership your head doesn't rotate in 3rd person but it does for other players and in 1st person

Could you test it on a vanilla GC space station as well? It really shouldn't behave any different...

Sorry for the long delay.
I can't reproduce this. Do you have any other mods which might affect player rendering?

Modpack:https://www.dropbox.com/s/zgu237awbshorqo/Superheroes.zip?dl=0


Cyrillic characters in filename not suported

Processing a file  with cyrillic chracters in the filename with:

$ scour Система_виробничих_відносин.svg output.svg

lead to an errormessage: IOError: [Errno 22] invalid mode ('rb') or filename: '???????_??????????_????????.svg'
Traceback (most recent call last):
  File "C:\Python27\Scripts\scour-script.py", line 11, in <module>
    load_entry_point('scour==0.36', 'console_scripts', 'scour')()
  File "c:\python27\lib\site-packages\scour\scour.py", line 3919, in run
    (input, output) = getInOut(options)
  File "c:\python27\lib\site-packages\scour\scour.py", line 3833, in getInOut
    infile = maybe_gziped_file(options.infilename, "rb")
  File "c:\python27\lib\site-packages\scour\scour.py", line 3828, in maybe_gziped_file
    return open(filename, mode)
IOError: [Errno 22] invalid mode ('rb') or filename: '???????_??????????_????????.svg'

Hi,
I cannot reproduce this in git on master (nor at the v0.36 tag):
$ time PYTHONPATH=. python3 -m scour.scour -i Система_виробничих_відносин.svg -o output.svg
Scour processed file "Система_виробничих_відносин.svg" in 371 ms: 29940/58953 bytes new/orig -> 50.8%

@JoKalliauer: What version of scour were you using when you got this error?

FTR, my OS locale defaults to an UTF-8 encoding and it also works flawlessly with my python 2.7 installtion.

Thanks I know now the mistake, but changeing the default encoding in the registry of Windows 10 can damage the OS (not bootable any more).

I read https://stackoverflow.com/a/17177904/6747994
then I run chcp and got Aktive Codepage: 850., which is a encoding used in DOS in Western Europe, more infos see: https://en.wikipedia.org/wiki/Code_page_850
Reading https://superuser.com/questions/269818/change-default-code-page-of-windows-console-to-utf-8 said that I have to change the registrykey HKEY_LOCAL_MACHINE\Software\Microsoft\Command Processor\Autorun to @chcp 65001>nul and would solve the problem, but in the sentence above the user wrote: Changing [registrykey] value to 65001 appear to make the system unable to boot in my case.

Therefore I don't want to change the default encoding in the registry of Windows 10 to UTF-8 just because of scour.
svgo and svgcleaner don't have any problems with UTF-8 characters in the filename on my Windows-10-OS, but scour does.

The problem seems to be the executable python creates.
I'm on Windows 10 myself and if I run
python -m scour.scour zСистема.svg zСистема2.svg

it works flawlessly.

@nthykier

time PYTHONPATH=. python3 -m scour.scour -i Система_виробничих_відносин.svg -o output.svg

i get

/usr/bin/python3: Error while finding module specification for 'scour.scour' (ModuleNotFoundError: No module named 'scour')
real    0m0,149s
user    0m0,062s
sys     0m0,046s

How can I solve this issue?
@Ede123
When I run

python -m scour.scour zСистема.svg zСистема2.svg

i get /usr/bin/python: No module named scour, how can I solve this issue?
According to https://stackoverflow.com/a/2326045/6747994 and https://stackoverflow.com/a/7587545/6747994 I might have to add the path of scour to sys.path or $PYTHONPATH? (Maybe you can help me?)
In cygwin.exe as well as in cmd.exe I get:

which scour
/cygdrive/c/Python27/Scripts/scour

I haven't used cygwin in a long time, sorry.
Just noticed I can not make it work in Python 2 either, so maybe it's a general issue in Python 2 (Python 3 works when loading the module directly for me).
You can also try to use short filenames (they usually work when everything else fails).

@Ede123

I installed Python 3.6
run pip3 install scour
using which scour leads to /cygdrive/c/Program Files (x86)/Python36-32/Scripts/scour
running the command  scour -i zСистема.svg -o zСистема2.svg leads to:

Traceback (most recent call last):
File "C:\Program Files (x86)\Python36-32\Scripts\scour-script.py", line 11, in
load_entry_point('scour==0.36', 'console_scripts', 'scour')()
File "c:\program files (x86)\python36-32\lib\site-packages\scour\scour.py", line 3919, in run
(input, output) = getInOut(options)
File "c:\program files (x86)\python36-32\lib\site-packages\scour\scour.py", line 3833, in getInOut
infile = maybe_gziped_file(options.infilename, "rb")
File "c:\program files (x86)\python36-32\lib\site-packages\scour\scour.py", line 3828, in maybe_gziped_file
return open(filename, mode)
OSError: [Errno 22] Invalid argument: 'z???????.svg'

Therefore I don't think it is a Python-2-bug.

Please read properly: https://github.com/scour-project/scour/issues/158#issuecomment-366466536

@Ede123
I think I don't understand your comment.
scour.exe does not work properly, but the source code does?
If I would use the code python -m scour.scour zСистема.svg zСистема2.svg then there shouldn't be any encoding problem?
But if i run python -m scour.scour zСистема.svg zСистема2.svg on my computer I get the error: /usr/bin/python: No module named scour. I'm not very familiar to programming and I don't have a glue what python is, therfore I don't know how to solve this issue.
I googed and found:

https://stackoverflow.com/a/339220/6747994
https://stackoverflow.com/a/2326045/6747994
https://stackoverflow.com/a/7587545/6747994

but I do not understand them, and I don't know what(and how) to append to sys.path.

scour.exe does not work properly, but the source code does?

The executable is only a wrapper automatically created by pip - it seems the wrapper causes additional problems when dealing with non-ASCII characters (and is broken for both, Python 2 and Python 3.

If I would use the code python -m scour.scour zСистема.svg zСистема2.svg then there shouldn't be any encoding problem?

It rules out problems of the wrapper and leaves you with problems of python itself - in Python 3 it seems to work, in Python 2 it seems to be broken.

But if i run python -m scour.scour zСистема.svg zСистема2.svg on my computer I get the error: /usr/bin/python: No module named scour.

As suggested before try not to run it in a Cygwin shell. If you have CPython installed from python.org it should work (in cmd.exe). The Cygwin environment will likely create even more problems.

@Ede123
Thanks:
python -m scour.scour zСистема.svg zСистема2.svg works with Python3 (not in Python2) in cmd.exe
Since Python2.7 is still default, I'm not sure if this issue can/should be closed.

Well, as I wrote it's basically a Python issue... See https://bugs.python.org/issue2128 for some older information on this.
I'm not sure if it even could be worked around from our side and I also feel there are more important things...


Propagate Caching information to FlyteAdmin

When Propeller successfully looks up a cached task execution, it needs to propagate that information in the NodeEvent. FlyteAdmin should store the information that the node execution has a cached task execution. This information could include the original execution which populated the cache and expose that in the NodeExecutions response objects.
The UI/CLI then has the capability to show an indication if a task execution is cached and from what lineage. Here is the corresponding UI issue: https://github.com/lyft/flyte/issues/21

related to https://github.com/lyft/flyte/issues/71

In Review:
https://github.com/lyft/flyteidl/pull/70
https://github.com/lyft/flyteplugins/pull/105
https://github.com/lyft/flytepropeller/pull/167
https://github.com/lyft/flyteadmin/pull/110

now merged


gambit: Fails to build since deterministically built gcc PR merge

Describe the bug
gambit fails to build since commit 48c952c039b224ea4bde5afa6c9a9b88fa46e550 ( PR #112928 - stdenv: make a deterministically built gcc ).  Found by git bisect.
This is on Darwin.
To Reproduce
nix-build . -A gambit

Expected behavior
It builds.
Screenshots
this derivation will be built:
  /nix/store/dp954vrzrq27fgnh0dsi9b874vv1msyn-gambit-4.9.3.drv
building '/nix/store/dp954vrzrq27fgnh0dsi9b874vv1msyn-gambit-4.9.3.drv'...
unpacking sources
unpacking source archive /nix/store/argbv6rzp25h5qyspav4knmj8zgf0zdm-gambit-v4_9_3.tgz
source root is gambit-v4_9_3
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file gambit-v4_9_3/doc/gambit.txt
patching sources
configuring
configure: WARNING: unrecognized options: --enable-targets
checking build system type... x86_64-apple-darwin19.4.0
checking host system type... x86_64-apple-darwin19.4.0
checking target system type... x86_64-apple-darwin19.4.0
checking for gcc... /nix/store/4akw6n9a2f1cxwcwihzf1diblbyzhkkc-clang-wrapper-7.1.0/bin/gcc
checking whether the C compiler works... no
configure: error: in `/private/tmp/nix-build-gambit-4.9.3.drv-0/gambit-v4_9_3':
configure: error: C compiler cannot create executables
See `config.log' for more details
error: builder for '/nix/store/dp954vrzrq27fgnh0dsi9b874vv1msyn-gambit-4.9.3.drv' failed with exit code 77;
       last 10 log lines:
       > configuring
       > configure: WARNING: unrecognized options: --enable-targets
       > checking build system type... x86_64-apple-darwin19.4.0
       > checking host system type... x86_64-apple-darwin19.4.0
       > checking target system type... x86_64-apple-darwin19.4.0
       > checking for gcc... /nix/store/4akw6n9a2f1cxwcwihzf1diblbyzhkkc-clang-wrapper-7.1.0/bin/gcc
       > checking whether the C compiler works... no
       > configure: error: in `/private/tmp/nix-build-gambit-4.9.3.drv-0/gambit-v4_9_3':
       > configure: error: C compiler cannot create executables
       > See `config.log' for more details
       For full logs, run 'nix log /nix/store/dp954vrzrq27fgnh0dsi9b874vv1msyn-gambit-4.9.3.drv'.

Notify maintainers
Gambit: @thoughtpolice @raskin @fare
gcc stdenv: @baloo
Metadata

system: "x86_64-darwin"
host os: Darwin 19.4.0, macOS 10.15.4
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.4pre20210326_dd77f71
channels(root): "nixpkgs-20.09pre240394.ca3fa9c32a4"
nixpkgs: /nix/store/6dwr7zpzbhhfwna0wr7i8073gz1vnarc-source

Maintainer information:
attribute:
- gambit
- gcc

It's working in my repo/branch https://github.com/MuKnIO/nixpkgs/tree/alpha but #114449 is blocked because I can't get #116275 is blocked by RFC0091 https://github.com/NixOS/rfcs/pull/91 having no support.

I don't have access to a darwin setup easily :/ Not sure i'll be able to help too much here :(

The attribute gccStdenv.cc no longer contains a gcc-based compiler on Darwin, but the clang wrapper.
The breaking change is deleting this one line here: https://github.com/NixOS/nixpkgs/commit/48c952c039b224ea4bde5afa6c9a9b88fa46e550#diff-ab5748dc9567516fefba8344056b51ec1866adeace380f46e58a7af3d619ea22L10285
Reverting that one line makes gambit build again.
@baloo  I don't understand what deleting that cc = gcc; line is about.  It seems wrong, but I'm not sure reverting is now right.  Thoughts.

Build fails differently with cc = gcc; restored:
DYLD_LIBRARY_PATH=..: ../gsc-boot -:~~bin=../bin,~~lib=../lib,~~include=../include -f -c  -prelude "(define-cond-expand-feature|enable-type-checking|)(define-cond-expand-feature|disable-auto-forcing|)(define-cond-expand-feature|enable-sharp-dot|)(define-cond-expand-feature|enable-bignum|)(define-cond-expand-feature|enable-ratnum|)(define-cond-expand-feature|enable-cpxnum|)(define-cond-expand-feature|disable-smp|)(##include\"../lib/header.scm\")" _io.scm
DYLD_LIBRARY_PATH=..: ../gsc-boot -:~~bin=../bin,~~lib=../lib,~~include=../include -f -c  -prelude "(define-cond-expand-feature|enable-type-checking|)(define-cond-expand-feature|disable-auto-forcing|)(define-cond-expand-feature|enable-sharp-dot|)(define-cond-expand-feature|enable-bignum|)(define-cond-expand-feature|enable-ratnum|)(define-cond-expand-feature|enable-cpxnum|)(define-cond-expand-feature|disable-smp|)(##include\"../lib/header.scm\")" _num.scm
DYLD_LIBRARY_PATH=..: ../gsc-boot -:~~bin=../bin,~~lib=../lib,~~include=../include -f -c  -prelude "(define-cond-expand-feature|enable-type-checking|)(define-cond-expand-feature|disable-auto-forcing|)(define-cond-expand-feature|enable-sharp-dot|)(define-cond-expand-feature|enable-bignum|)(define-cond-expand-feature|enable-ratnum|)(define-cond-expand-feature|enable-cpxnum|)(define-cond-expand-feature|disable-smp|)(##include\"../lib/header.scm\")" _std.scm
DYLD_LIBRARY_PATH=..: ../gsc-boot -:~~bin=../bin,~~lib=../lib,~~include=../include -f -c  -prelude "(define-cond-expand-feature|enable-type-checking|)(define-cond-expand-feature|disable-auto-forcing|)(define-cond-expand-feature|enable-sharp-dot|)(define-cond-expand-feature|enable-bignum|)(define-cond-expand-feature|enable-ratnum|)(define-cond-expand-feature|enable-cpxnum|)(define-cond-expand-feature|disable-smp|)(##include\"../lib/header.scm\")" _repl.scm
make[2]: *** [makefile:168: _std.c] Illegal instruction: 4
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [makefile:168: _repl.c] Illegal instruction: 4
make[2]: *** [makefile:168: _num.c] Illegal instruction: 4
make[2]: *** [makefile:168: _io.c] Illegal instruction: 4
make[2]: Leaving directory '/private/tmp/nix-build-gambit-4.9.3.drv-0/gambit-v4_9_3/lib'
make[1]: *** [makefile:466: all-recursive] Error 1
make[1]: Leaving directory '/private/tmp/nix-build-gambit-4.9.3.drv-0/gambit-v4_9_3'
make: *** [makefile:114: from-scratch] Error 2
error: builder for '/nix/store/38nfp84g07psjircpnwnrilmg911x2k9-gambit-4.9.3.drv' failed with exit code 2;
       last 10 log lines:
       > DYLD_LIBRARY_PATH=..: ../gsc-boot -:~~bin=../bin,~~lib=../lib,~~include=../include -f -c  -prelude "(define-cond-expand-feature|enable-type-checking|)(define-cond-expand-feature|disable-auto-forcing|)(define-cond-expand-feature|enable-sharp-dot|)(define-cond-expand-feature|enable-bignum|)(define-cond-expand-feature|enable-ratnum|)(define-cond-expand-feature|enable-cpxnum|)(define-cond-expand-feature|disable-smp|)(##include\"../lib/header.scm\")" _repl.scm
       > make[2]: *** [makefile:168: _std.c] Illegal instruction: 4
       > make[2]: *** Waiting for unfinished jobs....
       > make[2]: *** [makefile:168: _repl.c] Illegal instruction: 4
       > make[2]: *** [makefile:168: _num.c] Illegal instruction: 4
       > make[2]: *** [makefile:168: _io.c] Illegal instruction: 4
       > make[2]: Leaving directory '/private/tmp/nix-build-gambit-4.9.3.drv-0/gambit-v4_9_3/lib'
       > make[1]: *** [makefile:466: all-recursive] Error 1
       > make[1]: Leaving directory '/private/tmp/nix-build-gambit-4.9.3.drv-0/gambit-v4_9_3'
       > make: *** [makefile:114: from-scratch] Error 2
       For full logs, run 'nix log /nix/store/38nfp84g07psjircpnwnrilmg911x2k9-gambit-4.9.3.drv'.

looks like it was fixed in b863a7c8c261b650d2f87479ee30ef8b0fb88135 (available in current master in #123421)

Nice!  It gets as far as the illegal instruction on master now.
@baloo Any thoughts on how that PR could make gambit fail with "Illegal instruction: 4"?  Is the deterministic gcc somehow pinned to an architecture or does it make weird executables?

gcc used to not be profiled on darwin:
-        # PGO seems to speed up compilation by gcc by ~10%, see #445 discussion
-        profiledCompiler = with stdenv; (!isDarwin && (isi686 || isx86_64));

so nothing should have changed in this regard (except for my dumb mistake)


Add fall back for availableProcessor()

Due to https://github.com/Alluxio/alluxio/issues/14243#issuecomment-941441855, Netty is not able to to get the correct number of processors in certain container environments. Thus set a higher default value. The default value is debatable.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.8 please


The result icon is not changed for .py file in "Test" panel

Visual Studio Code - Insiders: 1.37.0-insider
python-insider: 2019.7.27813-dev
Repro step:

Input below code:

import unittest
class PassingTests(unittest.TestCase):
    def test_passing(self):
        self.assertEqual(42, 42)
    def test_passing_still(self):
        self.assertEqual("silly walk", "silly walk")
class FailingTests(unittest.TestCase):
    def test_failure(self):
        self.assertEqual(42, -13)
    def test_failure_still(self):
        self.assertEqual("I'm right!", "no, I am!")

Run tests with unittest framework
Run all tests

Actual Result:
The result icon is not changed for .py file in "Test" panel

Expect Result:
It should display failed icon.
Note:
This issue is repro on pytest and nosetest

@xiaofeng0123 just wanted to let you know that we did get this report and we did triage it as an actual bug. :)

Closing as stale. If you still experience this issue, let us know and we can reopen!


Added Apache 2.0 License File

You have a license stated in your README but no license file in the repository.

Thanks for the PR. Would you mind squashing your commits?


how to train a bot

This quick start guide will hlep you https://chatterbot.readthedocs.io/en/stable/quickstart.html

Hello, I would like to know if there is way to indicate to the bot if one of his answers is wrong.
For example: if I say to him 'Hello, my name is Daniel'
and he answers me : 'The weather is great'
How do I tell him that it is a bad response?

It all depends on confidence value and logic adapter you have chosen
You can put a benchmark for your corpus if confidence value below x (assume 0.70) then it would be a bad answer.


Less hacky fix for Razor explicit expression completions

Fixes https://github.com/dotnet/aspnetcore/issues/21154
Depends on this External PR
We need ( and ) to be commit characters to make extent calculations happen correctly on VS Client side. Having them as trigger characters work but it is not the correct fix.
This shouldn't affect HTML completions because the HTML language server provides CommitCharacters on individual CompletionItems which overrrides what is set in AllCommitCharacters.
Roslyn is yet to do that https://github.com/dotnet/roslyn/issues/43851

This is now unblocked.


[azure-sdk-for-java/sdk/eventhubs/azure-messaging-eventhubs] Filter fails if related to the last sequenceNumber of the partition

Describe the bug
In eventhub, startPosition filter fails if related to the last sequenceNumber of my partition. As far as I understand, it might be server-side issue.
To Reproduce
I work with some eventhub topic on which the traffic is really low. It contains 4 partitions but only partition 0 contains data, here are the SequenceNumber/EnqueuedTime contained
1271 - 2022-06-07T06:48:31.138Z
1272 - 2022-06-07T06:48:31.138Z
1273 - 2022-06-07T06:48:31.138Z
1274 - 2022-06-07T06:48:31.138Z
1275 - 2022-06-07T06:48:31.138Z
1276 - 2022-06-14T14:20:47.254Z

I use the code snippet to execute several scenarii.
First I focus on filtering sequenceNumber=1275
Here are the results:
EventPosition.fromSequenceNumber(sequenceNumber=1275, inclusiveFlag=false)
1276 - 2022-06-14T14:20:47.254Z

EventPosition.fromSequenceNumber(sequenceNumber=1275, inclusiveFlag=true)
1275 - 2022-06-07T06:48:31.138Z
1276 - 2022-06-14T14:20:47.254Z

This was conform to what I expected so far, let's try with sequenceNumber=1276
EventPosition.fromSequenceNumber(sequenceNumber=1276, inclusiveFlag=false)
1272 - 2022-06-07T06:48:31.138Z
1273 - 2022-06-07T06:48:31.138Z
1274 - 2022-06-07T06:48:31.138Z
1275 - 2022-06-07T06:48:31.138Z
1276 - 2022-06-14T14:20:47.254Z

EventPosition.fromSequenceNumber(sequenceNumber=1276, inclusiveFlag=true)
1271 - 2022-06-07T06:48:31.138Z
1272 - 2022-06-07T06:48:31.138Z
1273 - 2022-06-07T06:48:31.138Z
1274 - 2022-06-07T06:48:31.138Z
1275 - 2022-06-07T06:48:31.138Z
1276 - 2022-06-14T14:20:47.254Z

Code Snippet
import com.microsoft.azure.eventhubs.*;

import java.util.Iterator;
import java.util.concurrent.Executors;
import java.util.concurrent.ScheduledExecutorService;

public class EventHubFilterFails {
    public static void main(String[] args) throws Exception {
        String connectionString = "Some secret";
        String consumerGroup = "$default";
        String partitionId = "0";
        long seqNo = 1276L;
        boolean inclusiveFlag = true;
        EventPosition eventPosition = EventPosition.fromSequenceNumber(seqNo, inclusiveFlag);
        ScheduledExecutorService executor = Executors.newSingleThreadScheduledExecutor();

        EventHubClient client = EventHubClient.createFromConnectionStringSync(
                connectionString,
                executor);

        ReceiverOptions receiverOptions = new ReceiverOptions();
        receiverOptions.setReceiverRuntimeMetricEnabled(true);
        receiverOptions.setPrefetchCount(500);
        receiverOptions.setIdentifier("test-filters");
        PartitionReceiver receiver =
                client.createEpochReceiver(consumerGroup, partitionId, eventPosition, 0L, receiverOptions).join();

        while (true) {
            Iterator<EventData> it = receiver.receiveSync(1).iterator();
            while (it.hasNext()) {
                EventData msg = it.next();
                EventData.SystemProperties sp = msg.getSystemProperties();
                System.out.println(sp.getSequenceNumber() + " - " + sp.getEnqueuedTime());
            }
        }

    }
}

Expected behavior
For executions with sequenceNumber=1276, I would have expected:
EventPosition.fromSequenceNumber(sequenceNumber=1276, inclusiveFlag=false)
<EMPTY>

EventPosition.fromSequenceNumber(sequenceNumber=1276, inclusiveFlag=true)
1276 - 2022-06-14T14:20:47.254Z

Setup:

OS: Windows 11 Business (10.0.22000 Build 22000)
Library/Libraries: azure-eventhubs:3.3.0
Java version: Zulu version of java (https://www.azul.com/downloads/?version=java-8-lts&os=windows&architecture=x86-64-bit&package=jdk) : zulu<IP_ADDRESS>-ca-jdk8.0.312-win_x64

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Bug Description Added
[X] Repro Steps Added
[X] Setup information Added

Thanks for filing this github issue, @fanchuo. @liukun-msft could you please take a look?
/cc @Azure/azsdk-sb-java

Sure, I will check this.

Hi @JamesBirdsall, Could you please help triaging this track1 EH question?


docker won't connect to tvheadend

Hi there,
I've configured the docker container on my synology and can get to http://:5004/ without any probs. I get the "Antennas are operational!" screen.
config.yml:
tvheadend_url: http://user:pass@<IP_ADDRESS>:9981
tvheadend_weight: 300 # subscription priority
tuner_count: 2 # numbers of tuners in tvheadend

The docker log shows this whenever i try to add a tuner to plex:
4:46:22,[development] Kemal is ready to lead at http://<IP_ADDRESS>:5004,stdout
4:46:40,2017-09-18 04:46:40 +0000 200 GET / 362.3µs,stdout
4:46:40,2017-09-18 04:46:40 +0000 404 GET /favicon.ico 159.1µs,stdout
4:46:43,2017-09-18 04:46:43 +0000 200 GET / 44.0µs,stdout
4:47:21,2017-09-18 04:47:21 +0000 200 GET /discover.json 109.5µs,stdout
4:47:21,2017-09-18 04:47:21 +0000 200 GET /lineup_status.json 37.9µs,stdout
4:47:21,2017-09-18 04:47:21 +0000 200 GET /discover.json 43.4µs,stdout
4:47:21,2017-09-18 04:47:21 +0000 200 GET /lineup_status.json 61.9µs,stdout
4:47:25,Exception: No address found for x.x.x.x:9981 over TCP (Socket::Error),stdout
4:47:25,0x4d717e: ??? at ??,stdout
4:47:25,0x4d6878: socket at /opt/crystal/src/io.cr 558:7,stdout
4:47:25,0x4d0beb: exec_internal_single at /opt/crystal/src/http/client.cr 500:19,stdout
4:47:25,0x4ce68f: exec_internal at /opt/crystal/src/http/client.cr 487:5,stdout
4:47:25,0x4cb497: exec at /opt/crystal/src/http/client.cr 482:5,stdout
4:47:25,0x4c6cbc: ??? at /opt/crystal/src/http/client.cr 329:3,stdout
4:47:25,0x536be6: call at /home/jfa/antennas/lib/kemal/src/kemal/route_handler.cr 255:3,stdout
4:47:25,0x529a67: call_next at /opt/crystal/src/http/server/handler.cr 24:7,stdout
4:47:25,0x5283c1: call at /home/jfa/antennas/lib/kemal/src/kemal/static_file_handler.cr 56:9,stdout
4:47:25,0x533602: call at /opt/crystal/src/http/server/handler.cr 24:7,stdout
4:47:25,0x530f13: call at /opt/crystal/src/http/server/handler.cr 24:7,stdout
4:47:25,0x535fe6: call_next at /opt/crystal/src/http/server/handler.cr 24:7,stdout
4:47:25,0x51d50a: process at /home/jfa/antennas/lib/kemal/src/kemal/init_handler.cr 11:7,stdout
4:47:25,0x4177d8: ??? at /opt/crystal/src/fiber.cr 255:3,stdout
4:47:25,0x0: ??? at ??,stdout

is there something simple im missing here?

Woah, /home/jfa/antennas/, that doesn't seem right.
But the error seems to be related to Exception: No address found for x.x.x.x:9981 over TCP (Socket::Error),stdout, it looks as if it's not reading your config.yml
By default, the address is x.x.x.x, so it looks like the Docker container hasn't successfully mounted the volume with your updated config.yml.

If you pull down the latest Docker, you'll be able to configure the URL using the TVHEADEND_URL environment variable, and that'll be easier and more reliable.


Data Save

Hello ! CalendarFX is a great Framework ! i just have a problem with storing Data in local i tried iCal4j but no luck ! can anyone expailn the process for me ! Thanks !!

There is no standard process for persisting the calendar data. You need to take the calendar model and implement a persistence solution that works for you. If you do find one that might be interesting for others, then please feel free to provide a pull request.


Add support for builder.AddExpression

Second part of #5 : generate key-value settings from an Expression<Func<LoggerConfiguration, LoggerConfiguration>>
Most of the work is in class ConfigurationExpressionSettingsSerializer.
Currently supported features :

MinimumLevel.Is() / MinimumLevel.{Level}() / MinimumLevel.Override()
Enrich.WithProperty()
Enrich.{ExtensionMethod}()
WriteTo.{ExtensionMethod}()
AuditTo.{ExtensionMethod}()
Filter.{ExtensionMethod}()

Currently supported "parameters" :

constant expressions
new Class() where Class inherits/implements an abstract class / interface
public static property / field to acces an abstract class / interface

Tests :

expression serilization tests
sanity checks (AddExpression(exp) behaves the same as applying the expression)
combination tests (combines different settings sources)

Known advanced missing features as TODOs in ConfigurationExpressionSettingsSerializerTests.cs , but they should probably be included in future separate PRs.
Note: this PR sits on top of #9 , so I will rebase it properly once #9 has been merged .

Ready for review ! :nerd_face:
Thanks !

Super nice! 💯


Docs: Improve Known Issues pages

Jira Service Desk does not seem to support searching by IDs... Can we make a feature request? Or am I overlooking something? There does not seem to be any other persistent ID to reference a knowledge base article. So not sure how useful it is to have the ID in the link label if you can't really do anything with it.

@sleto-it What is the # column actually for? Will it be used in support to reference it in the docs? Are the updated link labels better perhaps?

https://github.com/arangodb/arangodb/pull/7640 needs to be pulled in here


Allow to parse/validate programmatically defined schema right away

Currently in order to parse/validate one needs to use parse and validate function either directly or use x closure. At first this felt like a good solution with clear separation of concerns, I mean not mixing up schema definition and parse/validate processes together however user expectation for this is different.
What I'm proposing is ability to parse/validate right away after schema definition like it's done in zod or superstruct:
Example:
import { string } from 'schematox'

const parsed = string().parse('x')
const validated = string().validate('x')

Another thing which might be annoying is necessity to import each programmatic schema unit separately. Yes one can make a statement like this import * as x from 'schematox' however I'm not sure if this is actually good approach.
What we can do is to extend x closure with ability of schema definition like this:
import { x } from 'schematox'

const parsed = x.string().parse('x')
const validated = x.string().validate('x')

If we will mange to make the first and the second requirement work without touching functions responsible for programatic schema definition it might be a win. However in that case we get inconsistent behaviour that need to be describe in docs which I want to avoid. By inconsistent behaviour I mean:
import { x, string } from 'schematox'

x.string().parse('x')

// expect error here because `parse` after schema definition is implemented on the x closure level
string().parse('x')

So having this work only in x.string might be not very good idea and better to deal with that at programmatic schema functions .

Closed by #18


Slash command args optional params and params with more than one string?

Support Question
Hello Symphony Developers! Thanks for the work on: https://github.com/finos/symphony-bdk-python/pull/284/files
It was helpful for us to grab @mention users. However we ran into a few issues below.
Borrowing off:
 @activities.slash("/echo {@mention_argument}")
        async def on_echo_mention(context: CommandContext):
            mentioned_user = context.arguments.get_mention("mention_argument")
            message = f"Mentioned user: {mentioned_user.user_display_name}, id: {mentioned_user.user_id}"

We tried:
@activities.slash("/echo {@mention_argument} {a_second_arg")
        async def on_echo_mention(context: CommandContext):
            mentioned_user = context.arguments.get_mention("mention_argument")
            message = f"Mentioned user: {mentioned_user.user_display_name}, id: {mentioned_user.user_id}"

But it appears that if {a_second_arg} is not passed the message won't expand. That's fine and we can get around it by declaring two decorators I guess one with the arg and one without but is there a way to make {a_second_arg} here optional if not passed the message will still trigger?
The second issue we faced was passing an arg/string with spaces in it. That appears to not work at all. In the example above if I were to:
@bot /echo @someone How are you doing?

This would completely fail, I'm guessing it's thinking that any string with a space is considered another arg it can't process but any way to allow args with spaces? If there's a cleaner workaround to this I'd love to hear it as well!
We need this because we typically take the full string "How are you doing?" and do something with that value and it would be annoying to tell users to have to quote the string.
Thank you for all the awesome work on this, it's made our Symphony experiences much more fun and pleasurable to use.

Hello @dky, thank you for raising this issue.
Regarding the first point, the parameters cannot be optional. A workaround would be to use different decorators with the same function. Here is an example:
@activities.slash("/echo {string_arg}")
@activities.slash("/echo")
async def on_echo_mention(context: CommandContext):
       argument = context.arguments.get_string("string_arg") if context.arguments else None
       message = f"Argument received: {argument}" if argument else "No Argument received"
       logging.info(message)

Regarding the second one, the white space is used by the input tokenizer to tokenise the message parts. Therefore "How are you doing" is split to 4 tokens. I don't see an efficient workaround but I encourage you to open a feature request issue and we can work on it.
For further documentation: https://github.com/finos/symphony-bdk-python/blob/main/docsrc/markdown/activity-api.md#slash-command-pattern-format


Add document for node problem detector.

For https://github.com/kubernetes/kubernetes/issues/23028.
This PR added a "Monitoring Node Health" section in "Administering Clusters".
@dchen1107
/cc @kubernetes/sig-node

@Random-Liu I think this feature is new to 1.3.  Please close this PR, and rebase your work on top of release-1.3, and open a new PR.
To rebase your work on 1.3 do something like this starting in the branch that you used for this PR:
COMMITS=$(git log master.. | grep commit | cut -f 2 -d " " | tac)
BR=$(git rev-parse --abbrev-ref HEAD)
git fetch upstream
git checkout upstream/release-1.3
git checkout -b $BR-1.3
for c in $COMMITS
do
  git cherry-pick $c; done
done
git push

@erictune ACK. Thanks a lot! :)

Close this one and will send a new one after rebase.

ACK.


GCE Deploy

Set up automated nightly builds on GCP.

@zaneselvans I made some changes to the output layer and the census db to accommodate the GCS cache. I've also restructured pudl_etl.sh to notify slack, run the etl and copy the outputs to GCS.

I think this PR may have the most commits-per-line-of-code ever.

Would it make sense to do a squash merge to condense all of these iterative changes into a single commit?

Great idea to make this a squash merge. A couple of thoughts and questions before I wrap this up:

@zaneselvans Should the container create the CEMS partitioned directory of parquet files and the single parquet file for the intake catalogs? If it needs to create both, do I need to run the CEMS pipeline twice?
The github workflow is now a single job because github does not allow you to pass env vars between jobs. The two previous jobs need the same values for CHECKOUT_BRANCH, GCE_INSTANCE, and ACTION_SHA.

On the partitioned vs. monolithic CEMS issue, the pudl-catalog currently expects both to exist, but I don't think that's the long-term plan. We want a single version which is fast and space efficient, which I think will be the partitioned version, but we need to play around with how the metadata is stored to make that work. Right now I'm using the monolithic version in the examples because querying it is faster, and the caching mechanism downloads every file from the partitioned version whenever anything is queried because it has to look at the metadata inside each file to know where the right data is.
So either I should disable the partitioned version of the data in the catalog for the moment, or we should generate both outputs.
The epacems_to_parquet script should be able to generate the partitioned output with the --partition output after the normal ETL has run.

Ok, I'll just run epacems_to_parquet --partition after the full ETL is run we have the partitioned and monolithic versions.

Changes:

Create a Docker image that installs PUDL and runs the ETL.
Create a Github Action that builds the PUDL image, pushes it to Docker Hub, runs the ETL on a GCP VM and copies the outputs to intake GCS buckets on a schedule and tags.
Add a --loglevel arg to the package entrypoint commands.
Add GoogleCloudStorageCache support to ferc1_to_sqlite and censusdp1tract_to_sqlite commands and pytest.
Allow users to create monolithic and partitioned EPA CEMS outputs without having to clobber or move any existing CEMS outputs.
Add requester pays support to GoogleCloudStorageCache.

Ok! The YAML formatting has been sorted out and the unit and CI tests have been re-enabled. I think this is good to merge in.

Looks good to me. Weird that you apparently can't transfer ownership of a PR to someone else. Since I created the initial PR I can't "approve".

Hmm weird. I'll squash and merge it in.


HTTP code is 500 which indicates error: server error - {"message":"Get https://registry-1.docker.io/v2/datarhei/restreamer/manifests/latest: Get https://auth.docker.io/token?scope=repository%3Adatarhei%2Frestreamer%3Apull&service=registry.docker.io: read tcp <IP_ADDRESS>:41556-><IP_ADDRESS>:443: read: connection reset by peer"}

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

Please make sure that you're on a stable connection and use a solid DNS such as Google DNS.


How do I prevent hot-update.json/js files from crowding my project directory

I've never seen this before - It'll be related to webpack itself rather than this module - as the actual hot-update is handled there.
If you post your webpack config I can still take a look.

It happened because I had --watch on webpack. Removing that made it stop.


Can you explain how do you generate the cache key?

Great idea and project, thanks
can you explain how are you generating the cache key based on the query and variables of the operation?
https://github.com/oslabs-beta/rediQLess/blob/npm_rediqless/src/RediCache.js#L39

The keys are the field name followed by an id number, and each original query is given an ID number to reference the field array associated with the query/type. There are some changes in the npm package that aren't reflected here. The SpaceX API was used for testing during development.


fix request button for items with long call number

see http://dev-discovery.nypl.org/item/b14342378

fixed in 4c27ba35481980b92974cbb949a75b4f61e0c77a