feat: CLIN-3616 allow to pass dbsnp option in genotypegvcf

Allow to optionally add dbsnp identifiers in the ID column of the output vcf files.
Tests

Run the pipeline locally without dbsnp parameters
The pipeline runs successfully and the character . appears in the ID column of the output vcf file after GenotypeGVCFs process.

Run the pipeline locally with dbsnp parameters
The pipeline runs successfully and we can see dbsnp ids in the ID column of the output vcf file after GenotypeGVCFs process.

Try specifying only dbsnpFile without dbsnpFileIndex
The job fail with a message mentioning that the dbsnpFileIndex parameter is missing

Replicate tests in juno

Why this is a draft PR

tests in juno
go through pr checklist

PR checklist

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[x] Make sure your code lints (nf-core lint).
[x] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[x] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[x] Reference Data Documentation in docs/reference_data.md is updated.
[x] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Add dbsnp file and index to the nexflow.config allowing for versioning of these database


feature: 安装上报 bk_agent_id 持久化 (closed #781)

Codecov Report

Merging #838 (971d997) into V2.2.X_IPV6 (f0c7e64) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           V2.2.X_IPV6     #838   +/-   ##
============================================
  Coverage        71.84%   71.85%
============================================
  Files              303      303
  Lines            21822    21834   +12
============================================
+ Hits             15679    15689   +10
- Misses            6143     6145    +2

Impacted Files
Coverage Δ

...ackend/components/collections/agent_new/install.py
89.51% <100.00%> (+0.45%)
:arrow_up:

...nents/collections/agent_new/choose_access_point.py
95.12% <0.00%> (-0.49%)
:arrow_down:

apps/node_man/handlers/job.py
86.13% <0.00%> (-0.27%)
:arrow_down:

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 f0c7e64...971d997. Read the comment docs.


add /r/lib/ in .gitignore

What is this PR for?
after compiling/building Zeppelin project with default/R mvn $BUILD_FLAG $PROFILE -B, this directory /r/lib/ gets created.
What type of PR is it?
Improvement
Todos

[x] - add /r/lib/ in .gitignore

What is the Jira issue?
N/A
How should this be tested?
Compile/build Zeppelin project with R/default options mvn $BUILD_FLAG $PROFILE -B, then do a git status. This directory /r/lib/ should not show up.
Screenshots (if appropriate)
N/A
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

LGTM

Merging if no more discussion.

+1


Added destructor comments for basic_endpoint and client.

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

Project coverage is 97.15%. Comparing base (dc54b3f) to head (146b2be).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #330   +/-   ##
=======================================
  Coverage   97.15%   97.15%
=======================================
  Files         152      152
  Lines        9160     9161    +1
=======================================
+ Hits         8899     8900    +1
  Misses        261      261

Files
Coverage Δ

include/async_mqtt/client.hpp
100.00% <100.00%> (ø)

include/async_mqtt/endpoint.hpp
100.00% <ø> (ø)

... and 2 files with indirect coverage changes


Disable spinner in CI environments (closes #733)

\сс @inikulin @AlexanderMoskovkin @VasilyStrelyaev

BTW, can you condirm it works well on Travis?

Yep: https://travis-ci.org/DevExpress/testcafe-browser-provider-saucelabs/jobs/153573126, https://travis-ci.org/DevExpress/testcafe-browser-provider-saucelabs/jobs/153573127

Why it writes "undefined" in output before error message in provided logs? Is this somehow related?

lgtm

Nope, "undefined" belongs to provider debugging (In more words, it tries to print out the secret environment var SAUCE_USERNAME, but fails, because Travis CI doesn't pass secrets to pull requests).

lgtm then

lgtm


BuildInfo API

Chisel now spits out a nice string at the top of generated firrtl with some build information. I think it would be nice to have a standard way of adding more build information about e.g. library versions, seeds.
@chick and @ucbjrl and I talked some about how the sbt-buildinfo plugin works and the possibility of using reflection for chisel to grab all the BuildInfo objects out there.
I also found this potential problem:  sbt/sbt-buildinfo#97

Apparently fixed.
Future requests for updating this API should include a use case as a driving example.


Does this not work anymore?

I installed the mod and after changing level i quit to play later. But i see no option to continue my save.

It does work. Start a run, complete 1 stage, exit the game, and then send log file, so I would have some info to be able to help you. To get log you can follow these instructions from ror2 modding discord
If you are using r2modman or Thunderstore Mod Manager:
You can copy your log by going to the Settings screen, selecting the Debugging tab and clicking Copy LogOutput.log to clipboard
If you have manual install:
You can find such log file in your Risk of Rain 2/BepInEx/ folder, the file is called LogOutput.log.

Sorry. It is working now when i tried it.  Thank you for this great mod.


Webhook API

This PR adds a Webhook API to syncplay.
For example you can have a rocket-chat server, with channels that put every posted link in a channel in syncplay.
The API also works with persistent rooms and if somebody is in the channel it will post as someone randomly, since there is no user/watcher assigned to the API.
Note that currently this API does not allow reading the current playlist but only appending.
Example configuration for Outgoing WebHook integration in rocketchat:
URLs: http:// SYNCPLAY_SERVER : WEBHOOK_PORT
Script:
class Script {
  prepare_outgoing_request({ request }) {
    url = request.data.text;
    c = request.data.channel_name;
    request.data = {"text": text, "channel_name": c};
    return request;
    }
  }

  /**
   * @params {object} request, response
   */
  process_outgoing_response({ request, response }) {
    return;
  }
}

I am happy to hear your feedback and improve the PR if necessary.

Hi, this an interesting concept so thanks for sharing it.  It looks like it'd need some work to get it to be merge-ready and then there is the question of whether we want to add this level of additional complexity to the software.
As noted in the principles set out at https://github.com/Syncplay/syncplay/issues/315 we have to balance increasing functionality with keeping things as simple as possible to minimise bugs and make the software easier to maintain.
I'm not sure how many people would want every URL posted in their chat server to be added to a Syncplay playlist and webhooks isn't really an area I have that much familiarity with to be able to properly maintain and test the code. I'd be interested to hear from others what their thoughts are and whether or not it provides a feature which they would be likely to use.

Hi, this an interesting concept so thanks for sharing it. It looks like it'd need some work to get it to be merge-ready and then there is the question of whether we want to add this level of additional complexity to the software.
As noted in the principles set out at #315 we have to balance increasing functionality with keeping things as simple as possible to minimise bugs and make the software easier to maintain.
I'm not sure how many people would want every URL posted in their chat server to be added to a Syncplay playlist and webhooks isn't really an area I have that much familiarity with to be able to properly maintain and test the code. I'd be interested to hear from others what their thoughts are and whether or not it provides a feature which they would be likely to use.

A webhook API seems to bring a lot of value here, but it might be better to move the webhook server out into its own package.

I'm closing this as we have no plans to support a Webhook API at present.


PSCE-249 - Add entrypoint for trestle YAML rule transformation

Description
Adds rules transform entrypoint

Updates CONTRIBUTING.md with updated components and code information
Adds trestle rules transformation entrypoint script

Blocked by #53
Type of change

[X] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected) - Updates the location of the actions in the repo

How has this been tested?

[X] Unit test
[ ] Integration tests

Checklist

[X] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@afflom I can break this down into multiple PRs if needed. The addition of the new entrypoint needed some restructuring and there is some cleanup of the test code in this PR.

@afflom I moved the generic test code changes/cleanup to a new PR - https://github.com/RedHatProductSecurity/trestle-bot/pull/56

Moved to draft while resolving conflicts.


treewide: migrate packages to pkgs/by-name, take 1

[RFC 140 2b] pkgs/by-name: Automated migration, take 1
We are migrating packages that meet below requirements:

using callPackage
called path is a directory
overriding set is empty ({ })
not containing path expressions other than relative path (to make nixpkgs-vet happy)
not referenced by nix files outside of the directory, other than pkgs/top-level/all-packages.nix
not referencing nix files outside of the directory
not referencing default.nix (since it's changed to package.nix)

The tool is here: https://github.com/Aleksanaa/by-name-migrate. Try reproduce yourself.

Todo:

[ ] Check whether passthru.updateScript has a path instead of checking files having "update" in the name
[ ] Try evaluating files moved to pkgs/by-name (probably by nix repl stdin/stdout pipe again)

Result of nixpkgs-review pr 354531 run on aarch64-linux 1
I consider the criteria and checks used by the script to be adequately conservative. There will be conflicts for open PRs, but that seems unavoidable; we could skip packages that have PRs active in a recent time‐frame but Git is decent at handling renames, so I think that’s probably not necessary, especially since so many PRs enact these moves themselves lately. Let’s get this done 🚀
If there are unforeseen painful consequences, I will try to revert ASAP.

Any plans to automatically update pending PRs? Or people have to fix the arisen conflicts manually?

I think that we could automatically rebase PRs but it would have consequences (e.g., breaking commit signatures). It seems like a lot of the conflicting PRs already involve a manual move to by-name, which would be trickier to handle (e.g. do we just drop the resulting empty commits? if they include formatting too as some of them do, the resulting commit message would become inaccurate and there’s no good way to automate fixing that)… maybe an approach avoiding touching packages with in‐flight PRs would have been better. OTOH, it would also probably have stretched out the process longer as we keep batching up packages that had fallen inactive over time, and increased the amount of active packages that need someone to manually migrate them.
Not sure if this was the right call or not – it seemed like the migration plans had stalled out for long enough, and the obligatory “move to pkgs/by-name” commits were so ubiquitous, that the time had come to bite the bullet and get it over with, but I’m open to feedback. It’d probably be even more disruptive to revert at this point, though, so it would probably just constitute lessons for next time…

I don't consider merge conflict as necessary, it only affects people not knowing how to use git. Once you pick up git as your workflow, you have to learn to deal with merge conflicts.
This may have an impact on some older PRs, but if it is an unnecessary functional improvement, "the contributor is no longer there" is basically the same as "there is a pending issue but no response", and is often not what we want to see. For other more critical/valuable PRs, rebasing PR in case contributor is not responding has long been a common practice among committers (and gh pr checkout is also quite handy for this).
I do hope that GitHub can have rebase/fixup tools as web app, so that we will not break signatures and can it be more friendly to people who do not know how to use Git.


Docs Update

I've decided to cleanup the README (it's getting absurdly long) and start a documentation process that is easier to follow. This is going to be a long process but here are the initial set of changes.
You can check out the updated docs here: https://github.com/blessedcoolant/stable-diffusion/tree/docs-update
I'll keep pushing changes regularly over the next day or two and then squash them all into one update.
Done

[x] Moving all documentation to a new folder called docs.
[x] Intro clean up
[x] Create a full list of contributors to ensure everyone is credited properly
[x] Initial Install Docs
[x] Initial Feature Docs
[x] Move all assets to Docs folder.
[x] Format cleanup on all the existing docs

Todo

[ ] Update, fix language and formalize all existing documentation.

Help

[ ] Localization -- If you are willing to help translate the docs into different languages, you are more than welcome to lend a hand. Thank you.

@lstein If you have any thoughts, suggestions or ideas in regards to this, please let me know.

This is good to go for now. I will formalize all language across all the docs over time. @lstein

@blessedcoolant; Should we put Stable-Diffusion-local-Windows.ipynb (it's primary purpose is to help with installation, not serve as a runtime notebook) and the text from the Easy-peasy-Windows-install Wiki page into the docs folder as well?

@blessedcoolant; Should we put Stable-Diffusion-local-Windows.ipynb (it's primary purpose is to help with installation, not serve as a runtime notebook) and the text from the Easy-peasy-Windows-install Wiki page into the docs folder as well?

Not sure about the book but the other page can do into the docs. Can this be integrated into the Windows Installation doc somehow? Or does this need to be a separate thing?

Then too... it's seeming more Wiki-like, but I'm not sure if we want to have the docs "tied up" in the Wiki or not - the Wiki is cool, but the 'docs' directory will travel with the repo (so if you're triple-firewalled like lstein, you can still have the docs with you).
I'm not sure what the right answer is here.

Then too... it's seeming more Wiki-like, but I'm not sure if we want to have the docs "tied up" in the Wiki or not - the Wiki is cool, but the 'docs' directory will travel with the repo (so if you're triple-firewalled like lstein, you can still have the docs with you).
I'm not sure what the right answer is here.

I feel for now we should keep all the docs with the repo. They'd be accessible offline too.

Can this be integrated into the Windows Installation doc somehow? Or does this need to be a separate thing?

I need to see what the Windows install page looks like in this PR so I can decide... the Easy-peasy guide is reeallly step-by-step detailed, so it's pretty long (and there's a separate page just for installing Git for Windows 🙄); we might end up with a Windows Install index page pointing out to other pages.

I need to see what the Windows install page looks like in this PR so I can decide... the Easy-peasy guide is reeallly step-by-step detailed, so it's pretty long (and there's a separate page just for installing Git for Windows 🙄); we might end up with a Windows Install index page pointing out to other pages.

Won't be a problem. The more detailed the better. The idea of having extra detailed docs is that we can forward people raising issues that have been addressed to the docs.
Have a look at the repo and let me know if it can be integrated somehow. Either you can do it or I can do it. Both work.

Right, so the current Windows Install doc is structured pretty well.
If you want, you can just pull the Markdown out of the Wiki into two individual markdown docs, fix up the main link in https://github.com/blessedcoolant/stable-diffusion/blob/docs-update/docs/installation/INSTALL_WINDOWS.md, fix up the Git for Windows install link in the Easy-peasy page, and all will be well.

Right, so the current Windows Install doc is structured pretty well.
If you want, you can just pull the Markdown out of the Wiki into two individual markdown docs, fix up the main link in https://github.com/blessedcoolant/stable-diffusion/blob/docs-update/docs/installation/INSTALL_WINDOWS.md, fix up the Git for Windows install link in the Easy-peasy page, and all will be well.

I suggest we let it be as it is for this particular push.
In a few days, I'll do a formalization pass on the docs so the language is consistent and professional across all the docs. While doing that, I'll try to reframe the Install_Windows doc by taking the best of both yours and the existing doc and make it good to go. There's no sense in having two docs for the same purpose in my opinion.

As usual, you folks are running ahead of me! I was planning to undertake a docs cleanup tomorrow morning. I'll pull this in this evening (have an all-day obligations today)

As usual, you folks are running ahead of me! I was planning to undertake a docs cleanup tomorrow morning. I'll pull this in this evening (have an all-day obligations today)

All good. :) Split the workload. Take your time. No rush.


limit the directory

sorry，I am not good at English，so i just  my upload my situation by photos

as you can see,i want to blast a domain like "www.xxx.com/cn" ,It's important to note that i don't  want to blast www.xxx.com ,i just want to blust www.xxx.com/cn  , but the machine works and automatictally add this "www.xxx.com" to blast,i konw i can enter and cancle this，but it can‘t work with this way when i use feroxbuster in  my scripts, i'll appreciate if someone can help！thankyou！

Good morning and thanks for the report!
The scan cancel menu does exactly what you want, simply cancel the directory you don't want to scan.

Good morning and thanks for the report!
The scan cancel menu does exactly what you want, simply cancel the directory you don't want to scan.

yeah，thank you,you're right,i can Cancel a Recursive Scan Interactively,but i mean how can i  avoid that happened when i use the scripts write by myself like a python script maybe and the ferobuster just a part of this script,and i can't  Cancel the Recursive Scan Interactively in this way,because i want my scritpt run itself automatictally！Am I explaining this clearly? thanks again for you help！

Yup, I understand what you're asking for. You basically want a domain blacklist
--dont-scan

or something like that. Seems reasonable, ill see what kind of effort this would require soon.

i think that we can safely roll this request and #130 into the same request, as blacklisting a domain should take effect whether it's via a redirect or not.


Fix export of missing component

I'm just assuming that the export should be removed, because it isn't exported from ObjectHome.js anymore.

I think this should be export class ObjectHomeRaw extends Component {  in ObjectHome.js.
The default export is ObjectHome with clickOutside wrapper and ObjectHomeRaw without wrapper.

@brunottonurb done


Failure to terminate the transaction when the broker cannot be connected

Failure to terminate the transaction when the broker cannot be connected
Hello.
There is a serious problem with the library.
The scenario is:
Broker : Mosquitto is installed on the docker.

I shut down my broker. (Docker Container shutdown)
I wrote and ran a console application like the one below.
I opened the broker. (Docker Container open)

I subscribed to the broker with MQTTBox and did not receive messages. Also, when I want to close the application, although the application is closed, it appears in the task manager and says Access Denied.
Testing was done on Windows 10. And MQTTnet 3.0.13
The problem is that we cannot get TcpAck and Conack information from mosquitto because we have closed the docker container directly.
static void Main(string[] args)
        {
            IMqttClient mqttClient = new MqttFactory().CreateMqttClient();

            MqttClientOptions mqttOptions = new MqttClientOptions
            {
                ChannelOptions = new MqttClientTcpOptions
                {
                    Server = "localhost",
                    Port = 1883 // Eg. 9000
                },
                CleanSession = true,
                ClientId = "xXx"
            };

            mqttClient.UseConnectedHandler(e =>
            {
                Console.WriteLine(mqttClient.Options.ClientId + " - bağlandı.");

            });

            mqttClient.UseDisconnectedHandler(e =>
            {
                Console.WriteLine(mqttClient.Options.ClientId + " - brokera bağlantı kesildi yeniden deneniyor...");

                Thread.Sleep(1000);
                mqttClient.ReconnectAsync();
            });

            mqttClient.ConnectAsync(mqttOptions);

            Task.Run(() =>
            {
                while(true)
                {
                    if(mqttClient.IsConnected)
                    {
                        mqttClient.PublishAsync("asd", Encoding.UTF8.GetBytes(Guid.NewGuid().ToString()));
                    }

                    Thread.Sleep(1000);
                }
            });

            Console.Read();
        }

Please attach the trace from the client (activation is described in wiki). Without having this I can only guess what the problem might be. In general the code looks OK to me and works in other projects without issues.

Please attach the trace from the client (activation is described in wiki). Without having this I can only guess what the problem might be. In general the code looks OK to me and works in other projects without issues.

I found the error in more detail. It works fine when we provide a remote ip address. (Like <IP_ADDRESS>) but when we give localhost or <IP_ADDRESS> it doesn't work if there is no broker.

I found the error in more detail. It works fine when we provide a remote ip address. (Like <IP_ADDRESS>) but when we give localhost or <IP_ADDRESS> it doesn't work if there is no broker.

Please attach the trace from the client (activation is described in wiki). Without having this I can only guess what the problem might be. In general the code looks OK to me and works in other projects without issues.

I found the error in more detail. It works fine when we provide a remote ip address. (Like <IP_ADDRESS>) but when we give localhost or <IP_ADDRESS> it doesn't work if there is no broker.

Please attach the trace from the client (activation is described in wiki). Without having this I can only guess what the problem might be. In general the code looks OK to me and works in other projects without issues.

I found the error in more detail. It works fine when we provide a remote ip address. (Like <IP_ADDRESS>) but when we give localhost or <IP_ADDRESS> it doesn't work if there is no broker.

That sounds really strange to me. Because the string is passed directly to the .net socket.
Which target platform do you use?

That sounds really strange to me. Because the string is passed directly to the .net socket.
Which target platform do you use?

That sounds really strange to me. Because the string is passed directly to the .net socket.
Which target platform do you use?

.NET Core 3.1 on Windows 10

That sounds really strange to me. Because the string is passed directly to the .net socket.
Which target platform do you use?

.NET Core 3.1 on Windows 10

OK then we have to ty some things: Please set DualMode to false in the client settings.

OK then we have to ty some things: Please set DualMode to false in the client settings.

And please test with latest 3.0.14-rc5.

And please test with latest 3.0.14-rc5.

And please test with latest 3.0.14-rc5.

How can I set the dual mode? Also 3.0.14-rc5. where can i download the version?

And please test with latest 3.0.14-rc5.

How can I set the dual mode? Also 3.0.14-rc5. where can i download the version?

And please attach the trace.
DualMode is a property at the client options.
rc5 is available at nuget.

And please attach the trace.
DualMode is a property at the client options.
rc5 is available at nuget.

I installed RC5 and after setting dual mode to false the error was fixed.
Then I went back to 3.0.13. It worked again. It was dual mode.
Thank you...

[2021-01-02T11:41:33.4658581Z] [1] [MqttClient] [Verbose]: Trying to connect with server 'localhost:1883' (Timeout=00:00:10).
[2021-01-02T11:41:35.7012247Z] [8] [MqttClient] [Error]: Error while connecting with server.
MQTTnet.Exceptions.MqttCommunicationException: Error while connecting with host 'localhost:1883'.
---> System.Net.Internals.SocketExceptionFactory+ExtendedSocketException (10061): Hedef makine etkin olarak reddettiğinden bağlantı kurulamadı. [::1]:1883
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw(Exception source)
at System.Net.Sockets.Socket.EndConnect(IAsyncResult asyncResult)
at System.Net.Sockets.Socket.DoMultipleAddressConnectCallback(Object result, MultipleAddressConnectAsyncResult context)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw(Exception source)
at System.Net.Sockets.Socket.DoMultipleAddressConnectCallback(Object result, MultipleAddressConnectAsyncResult context)
at System.Net.Sockets.Socket.MultipleAddressConnectCallback(IAsyncResult result)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw(Exception source)
at System.Net.Sockets.Socket.EndConnect(IAsyncResult asyncResult)
at System.Net.Sockets.Socket.<>c.b__277_0(IAsyncResult iar)
--- End of stack trace from previous location where exception was thrown ---
at MQTTnet.Implementations.CrossPlatformSocket.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
--- End of inner exception stack trace ---
at MQTTnet.Implementations.CrossPlatformSocket.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
at MQTTnet.Implementations.MqttTcpChannel.ConnectAsync(CancellationToken cancellationToken)
at MQTTnet.Internal.MqttTaskTimeout.WaitAsync(Func`2 action, TimeSpan timeout, CancellationToken cancellationToken)
at MQTTnet.Adapter.MqttChannelAdapter.ConnectAsync(TimeSpan timeout, CancellationToken cancellationToken)
at MQTTnet.Client.MqttClient.ConnectAsync(IMqttClientOptions options, CancellationToken cancellationToken)
[2021-01-02T11:41:35.7551017Z] [8] [MqttClient] [Verbose]: Disconnecting [Timeout=00:00:10]
[2021-01-02T11:41:35.7560265Z] [8] [MqttClient] [Verbose]: Disconnected from adapter.
[2021-01-02T11:41:35.7572828Z] [8] [MqttClient] [Info]: Disconnected.
xXx - brokera bağlantı kesildi yeniden deneniyor...

I installed RC5 and after setting dual mode to false the error was fixed.
Then I went back to 3.0.13. It worked again. It was dual mode.
Thank you...

[2021-01-02T11:41:33.4658581Z] [1] [MqttClient] [Verbose]: Trying to connect with server 'localhost:1883' (Timeout=00:00:10).
[2021-01-02T11:41:35.7012247Z] [8] [MqttClient] [Error]: Error while connecting with server.
MQTTnet.Exceptions.MqttCommunicationException: Error while connecting with host 'localhost:1883'.
---> System.Net.Internals.SocketExceptionFactory+ExtendedSocketException (10061): Hedef makine etkin olarak reddettiğinden bağlantı kurulamadı. [::1]:1883
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw(Exception source)
at System.Net.Sockets.Socket.EndConnect(IAsyncResult asyncResult)
at System.Net.Sockets.Socket.DoMultipleAddressConnectCallback(Object result, MultipleAddressConnectAsyncResult context)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw(Exception source)
at System.Net.Sockets.Socket.DoMultipleAddressConnectCallback(Object result, MultipleAddressConnectAsyncResult context)
at System.Net.Sockets.Socket.MultipleAddressConnectCallback(IAsyncResult result)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw(Exception source)
at System.Net.Sockets.Socket.EndConnect(IAsyncResult asyncResult)
at System.Net.Sockets.Socket.<>c.b__277_0(IAsyncResult iar)
--- End of stack trace from previous location where exception was thrown ---
at MQTTnet.Implementations.CrossPlatformSocket.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
--- End of inner exception stack trace ---
at MQTTnet.Implementations.CrossPlatformSocket.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
at MQTTnet.Implementations.MqttTcpChannel.ConnectAsync(CancellationToken cancellationToken)
at MQTTnet.Internal.MqttTaskTimeout.WaitAsync(Func`2 action, TimeSpan timeout, CancellationToken cancellationToken)
at MQTTnet.Adapter.MqttChannelAdapter.ConnectAsync(TimeSpan timeout, CancellationToken cancellationToken)
at MQTTnet.Client.MqttClient.ConnectAsync(IMqttClientOptions options, CancellationToken cancellationToken)
[2021-01-02T11:41:35.7551017Z] [8] [MqttClient] [Verbose]: Disconnecting [Timeout=00:00:10]
[2021-01-02T11:41:35.7560265Z] [8] [MqttClient] [Verbose]: Disconnected from adapter.
[2021-01-02T11:41:35.7572828Z] [8] [MqttClient] [Info]: Disconnected.
xXx - brokera bağlantı kesildi yeniden deneniyor...


Can around-next take multiple args?

In the tutorial.kbd it says that layer-next is equivalent to:
(around-next (layer-toggle layer-name) some-button)

But everywhere else that I see around-next in use, there is only one argument passed. For example, the quick-reference provides the following example:
(defalias ns  (around-next sft))  ;; Shift the next press

Should the tutorial instead say (around (around-next (layer-toggle layer-name)) some-button)?

You're right, this seems to be a mistake in the tutorial.
On Tue, Nov 16 2021 21:18, Duane Johnson wrote:

Should the tutorial instead say (around (around-next (layer-toggle layer-name)) some-button)?

I believe it should just be
(around-next (layer-toggle layer-name))

because around-next already works around the next key, so specifying
more here is not necessary.


first open on model date

It seems to me that the first open refer today, and not the model in the input form.
that's how i changed your init() to acheive this
          init = function() {
            var days, endDate, firstMonday;

            var initDate=scope.date ? moment(scope.date) : moment();

            firstMonday = initDate.clone();
            firstMonday = firstMonday.date(1).startOf('isoweek');
            if (firstMonday.date() === 1) {
              firstMonday.subtract(1, 'weeks');
            }
            days = initDate.date(1).daysInMonth();
            endDate = initDate.clone();
            endDate = endDate.add(1, 'months').date(0);
            scope.month = $filter('date')(new Date(endDate), 'MMM');
            if (endDate.day() !== 7) {
              endDate = endDate.add(7 - endDate.day(), 'days');
            }
            scope.currentDate = firstMonday;
            scope.year=initDate.year();
            return scope.weeks = getWeeks(endDate.diff(firstMonday, 'days'), firstMonday, initDate.month());
          };

create a pull request


Allow name of Pi to be specified for GUI install

Since I have multiple installations I needed a way to prevent the gui nameing all of them 'allsky'.
This change allows the hostname to be specified when installing the GUI. If no name is specified then the default 'allsky' is used.
Tested on a clean pi after installing allsky. Also re run the gui installer several times to confirm the hostname changes correctly.

Thanks Alex, nice improvement!


Send Adam Luckenbach a final copy of the assessment

A reminder to send Adam Luckenbach a final copy of the 2023 assessment.

I sent a draft so closing this issue.


Built in mod downloader

What feature do you want to get added on the base engine?
So I really want the built in mod downloader from psych engine online to normal psych engine people say it’s in the source code but I don’t know what to do with the source code.
To test your sight, and reliability, please select the option of what should NOT be requested.
Proper credit that was forgotten

Proper credit that was forgotten

Failed reliability test


File Browser Style should be more similar to VS Code

Material Design Icons:

chevron-right
chevron-down
circle

97ae887150c270d33bc5f0b4f3f0be1ae9fe9f1c


[Feature] Cache

Description
NgDoc doesn't have a cache, which means that every time the application starts, NgDoc renders and builds the application from scratch. The long build time is particularly noticeable when building an application with many API entities.
Proposed solution
A cache would solve the problem of not having to render what was rendered in the previous build, significantly increasing the speed of subsequent application launches.
Alternatives considered

:tada: This issue has been resolved in version 16.2.0 :tada:
The release is available on:

v16.2.0
GitHub release

Your semantic-release bot :package::rocket:


prompt: fix some keybindings to work with multilines

The behavior of the Home/End keys has been changed: now, the cursor moves to the beginning/end of the entire command (instead of the line) when these keys are pressed.

The behavior of the alt-b/alt-f keys has been changed: now, the cursor moves one word backward in multiline commands, and the word delimeter characters are \n and  .

The associated emacs keybindings has been changed: now ctrl-b/ctrl-f allow to move over the entire multiline command. ctrl-a/ctrl-e same as Home/End.

Added integration tests for emacs keyset.

Closes #6

Also, module was renamed to github.com/tarantool/go-prompt.


Focus/select file in file explorer sidebar for active editor

When learning / navigating a codebase, knowing where a given file exists in the filesystem (and seeing its siblings) is a very useful tool. It would be great if the file explorer sidebar selected/showed/focused the currently active editor's file (by expanding all folders in the hierarchy until the file becomes visible).

Seconded. I wanted to use this editor but this was my very first stumbling block!

I'd also like this feature! Initially I thought the keybinding "Reveal Active File in File Explorer" would do this, but this opens a Finder window (on Mac) with to show the file. In Sublime, there's a keybinding to do just this.


jsii-docgen doesn't find child jsii packages in a monorepo with npm workspaces

Hi there,
in an npm monorepo with the following structure:
packages/
├─ child-jsii-package/
│  ├─ package.json
├─ parent-jsii-package/
│  ├─ package.json   # includes child-jsii-package
package.json

jsii-docgen works fine on child-jsii-package, but fails on the parent-jsii-package with:
Error: Assembly "child-jsii-package" not found
    at TypeSystem.findAssembly (/Users/home/office/maintainer/contractcase/node_modules/jsii-reflect/lib/type-system.js:147:19)
    at TypeSystem.findFqn (/Users/home/office/maintainer/contractcase/node_modules/jsii-reflect/lib/type-system.js:156:26)
    at get type [as type] (/Users/home/office/maintainer/contractcase/node_modules/jsii-reflect/lib/type-ref.js:52:28)
    at TypeScriptTranspile.typeReference (/Users/home/office/maintainer/contractcase/node_modules/jsii-docgen/lib/docgen/transpile/transpile.js:317:17)
    at TypeScriptTranspile.parameter (/Users/home/office/maintainer/contractcase/node_modules/jsii-docgen/lib/docgen/transpile/typescript.js:134:30)
    at /Users/home/office/maintainer/contractcase/node_modules/jsii-docgen/lib/docgen/transpile/typescript.js:156:73
    at Array.map (<anonymous>)
    at TypeScriptTranspile.callable (/Users/home/office/maintainer/contractcase/node_modules/jsii-docgen/lib/docgen/transpile/typescript.js:156:35)
    at new InstanceMethod (/Users/home/office/maintainer/contractcase/node_modules/jsii-docgen/lib/docgen/view/instance-method.js:10:37)
    at /Users/home/office/maintainer/contractcase/node_modules/jsii-docgen/lib/docgen/view/instance-methods.js:9:25

Other jsii commands work (with the exception of this issue, which affects non-jsii packages marked with bundleDependencies, so my guess is it is a different cause - but I'm mentioning it in case it isn't).
Let me know if you need any further information

This is because jsii-docgen only loads .jsii assemblies it finds under $PWD, and since some here are hoisted up, they're not loaded upfront nor resolved... so the jsii-reflect type system it builds up is incomplete...
Seems to me that it should actually resolve dependencies "as node would", so that it would work... I'm looking at this.

Thank you so much!!


Winget update error

Brief description of your issue
An error occurred while trying to update packages. Logs report absence of package Microsoft.Winget.Source_8wekyb3d8bbwe, which shouldn't have happened as this is a new Canary install. I suspect this issue stems from the said package with no working method of obtaining it.

Steps to reproduce
Run winget update with admin privileges (either CMD or PS).
Expected behavior
winget updates its sources and returns upgradable packages, if any.
Actual behavior
winget reports an error in updating its repo.
Environment
PS C:\Users\ntauthority> winget update
Failed in attempting to update the source: winget
Failed when searching source; results will not be included: winget
No installed package found matching input criteria.
PS C:\Users\ntauthority> winget --info
Windows Package Manager (Preview) v1.7.3172-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.25997.1010
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.22.3172.0

Additional comment: Getting the source APPX package did not return anything, but reregistering it returns an error, saying the path to the package does not exist. The Get-AppxPackage command seems to have encountered an issue. I've also tried downloading the package from AdGuard Store, but every channel returned an empty list. Is there something with the package itself?

Try doing winget source reset --force from an admin terminal.
If that doesn't help, try
Add-AppxPackage -Path "https://cdn.winget.microsoft.com/cache/source.msix"

Try doing winget source reset --force from an admin terminal.
If that doesn't help, try
Add-AppxPackage -Path "https://cdn.winget.microsoft.com/cache/source.msix"

Unsuccessful. It returns Add-AppxPackage : Deployment failed with HRESULT: 0x80073CF0, Package could not be opened. error 0x80072EFD: Opening the package from location source.msix failed.

@sprono I've seen this behavior with Add-AppxPackage in Windows PowerShell with passing the -Path parameter. You can try
$ProgressPreference = "SilentlyContinue"
iwr "https://cdn.winget.microsoft.com/cache/source.msix" -OutFile source.msix
Add-AppxPackage .\source.msix

Or you can simply download the source.msix from the URL and double click on the downloaded file to install

@mdanish-kh well this pic says all

That's unusual and this would be the cause of the Failed in attempting to update the source: winget issue. Have you checked whether a proxy or firewall is blocking your internet requests?

That's unusual and this would be the cause of the Failed in attempting to update the source: winget issue. Have you checked whether a proxy or firewall is blocking your internet requests?

No, no proxy or VPN in use. (I do have Cloudflare WARP installed, but when I attempted to execute the commands the WARP service was not running.) I checked both inbound & outbound rules in Windows Firewall, but I could not find any entry, neither inbound nor outbound, related to winget or Windows Package Manager (if that's how Microsoft calls it.)

Maybe MS has worked on the CDN and has fixed the issue. Everything seems to be back online now. Case closed.

I'm also getting weird CDN issues.  Some of my machines can get to https://cdn.winget.microsoft.com/cache/source.msix . But my main PC gets a "connection refused"

The only thing that worked for me (and I suspect will work for lots of people): use a VPN to the US region.
These broken CDN issues rarely get resolved quickly because "works for me" for most Microsoft employees.

Add-AppxPackage -Path "https://cdn.winget.microsoft.com/cache/source.msix"

Add-AppxPackage: Deployment failed with HRESULT: 0x80073D0D, The deployment operation failed because the volume is offline. For a package update, the volume refers to the installed volume of all package versions.

Windows cannot install package Microsoft.Winget.Source_2024.325.1003.593_neutral__8wekyb3d8bbwe to volume S: with media ID {30353253-584E-4741-7D05-853047358899} because the volume is offline.

NOTE: For additional information, look for [ActivityId] d42af1bf-7e4d-0000-1d2a-55d44d7eda01 in the Event Log or use the command line Get-AppPackageLog -ActivityID d42af1bf-7e4d-0000-1d2a-55d44d7eda01

I do have an S: drive, but Windows, etc. are all installed on C: drive. It's strange that it insists on installing to my S: drive.


Remove GIN index for PostgreSQL search

While implementing PostgreSQL full text search we decided to use GIN index by default to speed up full text search of products name and description. It might be a good idea to leave this decision to a user, since choose of index scope and type (GIN, GIST) depends strongly on ones needs.
Problem could be solved by more advanced manual index setting in PostgreSQL, if we choose to go down this path we should discuss it here.
Fixes #1420
Pull Request Checklist
(Please keep this section. It will make maintainer's life easier.)

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[ ] JavaScript code quality checks pass: eslint.

Codecov Report

Merging #1445 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1445      +/-   ##
==========================================
- Coverage    83.4%   83.39%   -0.01%
==========================================
  Files         128      128
  Lines        5791     5789       -2
  Branches      664      664
==========================================
- Hits         4830     4828       -2
  Misses        803      803
  Partials      158      158

Impacted Files
Coverage Δ

saleor/product/models.py
92.16% <ø> (-0.06%)
:arrow_down:

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 a8d167c...f893322. Read the comment docs.


feat(network): Update VPC service with API version 2020-11-17

:tada: This PR is included in version 0.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


shouldUpdateScrollViewInsets method

Added:
func hidingNavigationBarManagerShouldUpdateScrollViewInsets(_ manager: HidingNavigationBarManager, insets: UIEdgeInsets) -> Bool
This is method for better customization. I need this to configure hiding manager change autolayout config instead of scrollview insets in my application.

Sorry for the extremely slow review. I have not taken a look at this project in ages.
Thanks for the addition!


Deploy new iov-faucet to xerusnet

Depends on https://github.com/iov-one/iov-faucet/issues/12

Should be done by now. I will take over to ensure it is deployed on xerusnet and yaknet

Done by @husio

🎉 thanks @husio


Added "Quick Start with the ABP Framework"

I also added an "Application Frameworks" section under the C# category.

Can you rename the project's title to for example "TODO list with APB Framework"? And this should be in the C# category.


isogram: update generator to put input variable in failure message

I received feedback on my PR saying that the failure message in the unit tests should leverage the same string variable. I changed the isogram generator to reflect that change. Not sure if this requires an update to the test suite version or not.

Thank you again @bmkiefer!


update the new behaviors in the config

there has been no new behaviors in Iron and Humble from last year


fix: update Readme.md

:tada: This PR is included in version 5.0.19 :tada:
The release is available on:

v5.0.19
GitHub release

Your semantic-release bot :package::rocket:


No exit button in VR mode on mobile

When entering the VR mode on mobile, putting it back in portrait doesn't change anything, and there are no "exit VR" button, which makes it a bit difficult to exit VR on phones with no physical back button (in my case Motorola X 2nd gen)

Sorry, wrong repo - could you file at https://github.com/aframevr/aframe/issues/new instead? Thanks! (Sorry, @AFrame!)


Remove "native bundle" support

The split start level capabilities has always been dubious, and there are better ways to handle it (including breaking out into different packages). It over-complicates the plugin and the sequence of things starting in CQ.

Note that this would be a major version change and would require a "migration guide" to help people understand how to change their builds.


Use the Hijacked syntax analyzer for syntactic tests

Pretty simple,
Allows syntactic tests to contain the inline modifier.
See #37 for context

Closing this as it's probably not too relevant anymore. If anyone wants it revived, open one in meta.


Provider bedrock is not supported when trying to use bedrock examples listed in repo.

Here is my config, got it off the examples.
graph_config = {
"llm": {
"client": "bedrock",
"model": "bedrock/anthropic.claude-3-sonnet-20240229-v1:0",
"temperature": 0.0
}
}
Clarity on what "client" is helps.
Here is the error:
line 139, in _create_llm
raise ValueError(f"Provider {llm_params['model_provider']} is not supported. If possible, try to use a model instance instead.")
ValueError: Provider bedrock is not supported. If possible, try to use a model instance instead.

Looks like the bug is caused because of a missing comma between anthropic and bedrock.

Working on it, thanks for reporting this bug. The fix will most likely be rolled out in the next beta in a few hours.

Hi please update to the new beta


Tag Management UI

Describe the feature you'd like
I would like a single user interface where I can manage tags and tag values with the ability to see a mapping of where tags are (listing of shelves, books, chapters, and pages).
Describe the benefits this feature would bring to BookStack users
Tags are very easy to add and the live suggestions as you type are great, however, as a project ages, it is quite likely that the tags will get out of control unless there is an easy way to manage them and clean them up periodically.  It would be great to be able to go to one place in the UI and be able to both see what tags are associated with what entities (shelves, books, etc.)
Additional context
The essential functionality requires no more than a new UI that simply reverses the normal mapping, which would be this:
[Shelf | Notebook | Chapter | Page] --> [tag1:tag-value1,tag2:tag-value2]
to this:
tag1:tag-value1 --> [Notebook1, Page21, Page23, Chapter3]
A nice to have feature down the road would be to include some data cleaning utilities such as what is included in tools like OpenRefine (similar strings, semantic similarity, misspellings, etc.)...But under no circumstances, should the latter feature be construed as necessary for an initial capability.

After recently having found how useful tags are for searching, i would love this feature!

Related to #868 and #1354.

I would love a way to manage and edit tags. Right now, I have no way to modify a tag created in error due to a typo. I can remove it and create a new tag that's spelled correctly, but the old, incorrect tag remains.

@saijojoe The old tag should not remain, and you should be able to simply edit the name or the value of existing tags.
Maybe play with the demo instance to see if the behaviour is the same there, and if so, might be worth raising as an issue if you're able to provide clear steps in not being able to edit or remove tags.

I'm going to go ahead and close this off since a tag overview dashboard, with a full breakdown of usage, was added as part of BookStack v21.11:
https://www.bookstackapp.com/blog/bookstack-release-v21-11/#new-tags-overview


Fixed Watch/Event flags.

The Watch/Event flags were mixed up, I have corrected them.

Coverage remained the same at 96.709% when pulling 740f090137ef600f85d367e8422db4a32c59b831 on filonik:master into 48f65257110aa8a37bbb38ec76a035c164e1eced on skypjack:master.

Coverage remained the same at 96.709% when pulling a2f94836fc349502c9318b021507fb94100b833a on filonik:master into 48f65257110aa8a37bbb38ec76a035c164e1eced on skypjack:master.

Coverage remained the same at 96.709% when pulling 3a2f4fa89b919b1541cf955772f4394cd7c5ca0c on filonik:master into 48f65257110aa8a37bbb38ec76a035c164e1eced on skypjack:master.


In Use label is unreadable in dark mode

In the Rule Template section of analytics, the in use label is unreadable in dark mode.

Hi @anfisher1967 Gentle Reminder: We are waiting for your response on this issue. If you still need to keep this issue active, please respond on it in the next 2 days. If we don't receive response, we will close this issue as per our standard procedures, thanks.

Hi @anfisher1967 since we have not received a response in the last 5 days, we are closing your issue #7749, as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation.


SPIKE: Saturn status page

Users wish to know whether Saturn or any of it's components are down. What is the Saturn equivalent of status.firecloud.org?

Firecloud status page uses Pingdom's public status page. Pingdom monitors a website's functionality and the public status page lets users see how the website is currently/has been behaving. Looks like a feature of the Pingdom membership and seems fairly straightforward to set up from their guides.

Ticket generated as a result of this spike is #791 . Closing this Spike since a path forward has been made.


Simplify and optimize Julia implementation

Two changes:

Replace haskey, d[k] = x and d[k] with get! to cut down on code size and the number of required hashes. get! is a builtin function designed specifically for this common use case of getting an element from a dictionary or instantiating it if it's missing.
Use an appropriately sized integer type for post index and counts. This will cause bugs if there are more than 2^16 posts. If that possibility for error is "cheating", we can add a near-zero-cost dispatch to dynamically determine the appropriate number type. I'd estimate that that dispatch will add about 6 lines of code, increase compilation time a bit, and not affect runtime at all.

Hey, the rules state that it must be able to handle 100k posts so that tweak is not allowed.


Fix the loading of the currency format types

In order to fix #201, it was necessary to properly interpret the type attribute of the currencyFormat elements in CLDR data.  It apparently changed between version 23 and 26.

Current coverage is 83.06%

Merging #203 into master will decrease coverage by -0.01% as of 15d9f56

@@            master    #203   diff @@
======================================
  Files           21      21
  Stmts         3780    3778     -2
  Branches         0       0
  Methods          0       0
======================================
- Hit           3140    3138     -2
  Partial          0       0
  Missed         640     640

Review entire Coverage Diff as of 15d9f56

Powered by Codecov. Updated on successful CI builds.

The decrease in overall coverage is due to a change in a doctest.  Maybe codecov is not running them?

@etanol tests are run by travis using pytest, if doctests aren't run that's a bug but IIRC they are run.

@sils1297 Then, how can I avoid this coverage decrease?

@etanol you should be able to reproduce the coverage report with make test-cov after pip install pytest-cov. Either doctests aren't executed, should be easy to test locally, or your tests are insufficient. In the first case, please file a bug and refer to it but IIRC pytest should do that.

There seems to be a problem with the process environment on Windows for Python 2.7.  The function babel.core.default_locale returns None when there is no system locale information available from the environment variables (LANGUAGE, LANG, LC_*).
I'm going to change the doctest so this doesn't fail.  However, maybe some review needs to be performed on the AppVeyor configuration, since all Python 3 setups seem to work fine.

@sils1297 this looks good to me.

@etanol
Can you add an entry to the CHANGES file, under the upcoming 3.x version just to keep track of changes you've made?
We don't have any formal guideline regarding this, but I think asking PR authors to do this it's the easiest way to keep our change log updated. Thank's

@erickwilder Done.  I took the chance to include information related to my other PR #117.  I hope is not an issue.

@etanol it seems we are still not capable of maintaining this repository without larger delays for basically everything and you're more of a Babel dev than I am tbh. You do seem to have some kind of interest in Babel, would you be interested in help us reviewing and triaging bugs a bit?

@sils1297 Sure, I'll be glad to help.  Although my availability is variable, I always try to dedicate a couple of hours a week to free coding.

@etanol cool, nobody is full time availabe for this one anyway. Whenever your passion takes over feel free to review some "pending review" PRs. I'll try to get you permissions to relabel stuff, we still need @mitsuhiko to do that kind of stuff and it's usually hard to grab him.

Hi @etanol
Other pull requests were approved and it seems to have caused some conflicts here. Can you rebase this so I can merge it to our master branch?
Sorry for the delay. My daughter was born a couple days ago and I was out of the radar for a while.
Thanks

@etanol marking this one in-progress as it seems to need a rebase and coverage still complains.

Current coverage is 83.40%

Merging #203 into master will increase coverage by +0.04% as of 2151786

@@            master   #203   diff @@
=====================================
  Files           21     21
  Stmts         3781   3785     +4
  Branches         0      0
  Methods          0      0
=====================================
+ Hit           3152   3157     +5
  Partial          0      0
+ Missed         629    628     -1

Review entire Coverage Diff as of 2151786

Powered by Codecov. Updated on successful CI builds.

@etanol are there any notable changes from the state where @erickwilder approved this? Does it need another full review?

@sils1297 I need to fix the CHANGES file, give me a day or two please.

You may want to merge my PR that sets the version numbers in CHANGES right
in the meantime.
On Sep 26, 2015 8:28 PM, "Isaac Jurado"<EMAIL_ADDRESS>wrote:

@sils1297 https://github.com/sils1297 I need to fix the CHANGES file,
give me a day or two please.
—
Reply to this email directly or view it on GitHub
https://github.com/python-babel/babel/pull/203#issuecomment-143478262.


Support shadow dom locator

We should add support for Shadow dom locators.
In an attempt to figure out how to get selenium to find elements within a shadow dom I came up with an (incomplete) solution where I made another version of Target.
ShadowTarget.the("welcome message").rooted_at('unique_tag').located_by('h2[name=welcome]')

It's unclear to me yet if it makes more sense to simply extend Target or to create a new class.  For the sake of simplicity I created a new class; which underlined the necessity for protocol for Target-like classes.
My solution is a little bit of a hack since selenium doesn't currently have native shadowdom support either.  Also the below example was written prior screenpy 4.x.
from typing import List, Tuple, Union

from selenium.common.exceptions import WebDriverException
from selenium.webdriver.common.by import By
from selenium.webdriver.remote.webdriver import WebElement

from screenpy.abilities.browse_the_web import BrowseTheWeb
from screenpy import Actor, Target
from screenpy.exceptions import TargetingError

class ShadowTarget:
    """Describe an element with a human-readable string and a locator.

    Examples::

        ShadowTarget.the("header search bar").located_by("div.searchbar")

        ShadowTarget.the("welcome message").rooted_at('unique_tag').located_by(
            'h2[name=welcome]')
    """

    target_name: str
    locator: Union[None, Tuple[str, str]]

    @staticmethod
    def the(desc: str) -> "ShadowTarget":
        """Name this Target.

        Beginning with a lower-case letter makes the logs look the nicest.
        """
        return ShadowTarget(desc)

    #TODO: try using cssify to convert xpath

    def located_by(self, locator: Union[Tuple[str, str], str]) -> "ShadowTarget":
        """Set the locator for this Target.

        Possible values for locator:
            * A tuple of a |By| classifier and a string
                (e.g. ``(By.ID, "welcome")``)
            * A CSS selector string
                (e.g. ``"div.confetti"``)
        """
        if isinstance(locator, tuple):
            self.locator = locator
        else:
            self.locator = (By.CSS_SELECTOR, locator)

        return self

    located = located_by

    def get_locator(self) -> Tuple[str, str]:
        """Return the stored locator.

        Raises:
            |TargetingError|: if no locator was set.
        """
        if self.locator is None:
            raise TargetingError(
                f"Locator was not supplied to the {self} target. Make sure to use "
                "either .located() or .located_by() to supply a locator."
                )
        return self.locator

    @staticmethod
    def make_css(by, value):
        if by == By.ID:
            value = '[id="%s"]' % value
        elif by == By.TAG_NAME:
            pass
        elif by == By.CLASS_NAME:
            value = ".%s" % value
        elif by == By.NAME:
            value = '[name="%s"]' % value
        elif by == By.XPATH:
            raise ValueError('ShadowTarget locator cannot support xpath')
        return value

    def elem_in_shadow(self, target: Union[Target, 'ShadowTarget'],
                       *shadow_tags: Union[Tuple[str, str], str]):
        s = ["return document"]
        for shadow_tag in shadow_tags:
            s.append(f".querySelector('{self.make_css(*shadow_tag)}').shadowRoot")
        s.append(f".querySelector('{(self.make_css(*target))}')")
        return ''.join(s)

    def all_elem_in_shadow(self, target: Union[Target, 'ShadowTarget'],
                           *shadow_tags: Union[Tuple[str, str], str]):
        s = ["return document"]
        for shadow_tag in shadow_tags:
            s.append(f".querySelector('{self.make_css(*shadow_tag)}').shadowRoot")
        s.append(f".querySelectorAll('{(self.make_css(*target))}')")
        return ''.join(s)

    def rooted_at(self, shadow_locator: Union[Tuple[str, str], str]):
        if isinstance(shadow_locator, tuple):
            self.shadow_locators.append(shadow_locator)
        else:
            self.shadow_locators.append((By.TAG_NAME, shadow_locator))
        return self

    def found_by(self, the_actor: Actor) -> WebElement:
        """Retrieve the |WebElement| as viewed by the Actor."""
        browser = the_actor.ability_to(BrowseTheWeb).browser
        try:
            # script = self.elem_in_shadow(self, *self.shadow_locators)
            return browser.execute_script(self.elem_in_shadow(self, *self.shadow_locators))
        except WebDriverException as e:
            raise TargetingError(f"{e} raised while trying to find {self}.") from e

    def all_found_by(self, the_actor: Actor) -> List[WebElement]:
        """Retrieve a list of |WebElement| objects as viewed by the Actor."""
        browser = the_actor.ability_to(BrowseTheWeb).browser
        try:
            return browser.execute_script(self.all_elem_in_shadow(self, *self.shadow_locators))
        except WebDriverException as e:
            raise TargetingError(f"{e} raised while trying to find {self}.") from e

    def __repr__(self) -> str:
        return self.target_name

    __str__ = __repr__

    def __getitem__(self, index: int) -> str:
        return self.get_locator()[index]

    def __init__(self, desc: str) -> None:
        self.target_name = desc
        self.locator = None
        self.shadow_locators: List = []

I'm just now looking over your implementation—man, fascinating. So locating elements in Shadow DOMs can't be done through XPATH, is that right?
If i'm remembering our conversation earlier, there was some way to switch to a shadow root, right? Maybe there is an easier way to support Shadow DOMs by switching recursively until we get a Target at the bottom?

After trying to implement this myself, i think i've caught up to your implementation. I found a great blog post by Titus Fortner, written late last year, discussing how to deal with shadow DOM.
I'm hoping that their implementation will come soon, and i think we might want to wait until that is available to add this feature.

I agree on waiting until the selenium folks come up with a more universal solution.  My hope is that all of the driver groups will agree on a means of looking inside shadow DOMs.

Closing for now.  We can revisit when the driver groups address how to access shadow doms.


FreeRDP crash on startup in Windows when started in service mode

Description of the bug
When FreeRDP is started from a service (SYSTEM user) it crashes with access violation on startup.
The crash happens in function wfreerdp_client_global_init (client\Windows\wf_client.c) on
strcat(home, getenv("HOMEDRIVE"));
as getenv returns NULL.
As SYSTEM user there is no HOME, HOMEDRIVE, HOMEPATH environment variables by default.
To Reproduce
Steps to reproduce the behavior:

Start FreeRDP from a service to just login a user

Expected behavior
Proper check of environment variables that does not exist
** Application details

Latest version of FreeRDP (3Oct2018)
OS version connecting to is Windows 8.1

Desktop (please complete the following information):

OS: Windows 8.1

Sounds like a bad idea to start a user process as service. (=root on unix ;))
Anyhow, the missing null check is definitely a bug.


Regarding the labels (symptom and diagnosis dates)

Dear sir/madam,
thanks so much for your great work and I am doing some research in this domain as well at West Virginia University. I couldn't find the symptom and diagnosis dates for the subjects so that I can train models on them. Could you kindly help?
Best regards,
Omid

Please check our supplemental data of the Phase-1 paper.  https://www.nature.com/articles/s41551-020-00640-6.

Thanks you so much with kind regards,
Omid
On Wed, Sep 8, 2021 at 7:12 AM Gireesh Bogu @.***>
wrote:

Please check our supplemental data of the Phase-1 paper.
https://www.nature.com/articles/s41551-020-00640-6.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gireeshkbogu/LAAD/issues/1#issuecomment-915143639,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLW2MBQTK72E57GL4WDFZTUA5AKVANCNFSM5DTYP2FA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


WARNINGS: ?: (ckeditor.W001) django-ckeditor bundles CKEditor 4.22.1

OS Type: Debian
OS Version: 12
Database Type: PostgreSQL
Database version: 15
Modoboa: 2.3.2
installer used: Yes
Webserver: Nginx
Hello my mailbox postmaster
floods you with monotonous messages like
WARNINGS:
?: (ckeditor.W001) django-ckeditor bundles CKEditor 4.22.1 which isn't supported anmyore and which does have unfixed security issues, see for example https://ckeditor.com/cke4/release/CKEditor-4.24.0-LTS . You should consider strongly switching to a different editor (maybe CKEditor 5 respectively django-ckeditor-5 after checking whether the CKEditor 5 license terms work for you) or switch to the non-free CKEditor 4 LTS package. See https://ckeditor.com/ckeditor-4-support/ for more on this. (Note! This notice has been added by the django-ckeditor developers and we are not affiliated with CKSource and were not involved in the licensing change, so please refrain from complaining to us. Thanks.)
As far as I understand from https://github.com/modoboa/modoboa/issues/3316
required to be added SILENCED_SYSTEM_CHECKS in settings.py.
what are  settings.py we talking about ?
find / -name settings.py
/srv/modoboa/env/lib/python3.11/site-packages/modoboa_radicale/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/drf_spectacular/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/modoboa_contacts/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/tutorial/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/modoboa_amavis/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/rest_framework_nested/runtests/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/oauth2_provider/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/rest_framework/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/django_rq/tests/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/django_rq/settings.py
/srv/modoboa/env/lib/python3.11/site-packages/ckeditor_demo/settings.py
/srv/modoboa/instance/instance/settings.py

I had the same issue
For silencing the message it's in /srv/modoboa/instance/instance/settings.py
add something like this at the end of the file after checking SILENCED_SYSTEM_CHECKS does not already exist in the file
SILENCED_SYSTEM_CHECKS = [
"ckeditor.W001",  # CKEditor 4.22.1 warning
]


[release-v1.15] Update golang.org/x/crypto to 0.31.0

Addresses https://github.com/advisories/GHSA-v778-237x-gjrc

cc @matejvasek

/cc @matejvasek
/cc @jrangelramos

\lgtm

\lgtm

/lgtm

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (release-v1.15@6242c15). Learn more about missing BASE report.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@               Coverage Diff                @@
##             release-v1.15    #1016   +/-   ##
================================================
  Coverage                 ?   59.68%
================================================
  Files                    ?      129
  Lines                    ?    15408
  Branches                 ?        0
================================================
  Hits                     ?     9197
  Misses                   ?     5345
  Partials                 ?      866

Flag
Coverage Δ

e2e-test
36.86% <ø> (?)

e2e-test-runtime-go
26.49% <ø> (?)

e2e-test-runtime-node
25.89% <ø> (?)

e2e-test-runtime-python
25.89% <ø> (?)

e2e-test-runtime-quarkus
26.04% <ø> (?)

e2e-test-runtime-rust
24.97% <ø> (?)

e2e-test-runtime-springboot
25.07% <ø> (?)

e2e-test-runtime-typescript
26.00% <ø> (?)

unit-tests
50.61% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Use 'context' in generated code

Replace golang.org/x/net/context with context in generated code:

[x] google.golang.org/grpc (https://github.com/grpc/grpc-go/pull/1160)
[ ] github.com/golang/protobuf (TODO https://github.com/golang/protobuf/issues/336)
[ ] github.com/gogo/protobuf (TODO https://github.com/gogo/protobuf/issues/282)
[ ] github.com/grpc-ecosystem/grpc-gateway (TODO https://github.com/grpc-ecosystem/grpc-gateway/pull/343)

Not a blocker for Go 1.9 migration, or 3.3 release.
c.f. https://github.com/coreos/etcd/issues/6174

With Go type alias and 1.10, this is resolved.


Fix/quantel chose latest clip

When a ClipGUID resolves to 2 or more ClipIDs on the same disk pool, choose the one with the latest (most recent) created date for further processing, including metadata and media processing.
Implementation matches https://github.com/nrkno/tv-automation-media-management/pull/32

Codecov Report

Merging #165 into release24 will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           release24     #165   +/-   ##
==========================================
  Coverage      99.79%   99.79%
==========================================
  Files             17       17
  Lines            490      490
  Branches           1       43   +42
==========================================
  Hits             489      489
  Partials           1        1

Impacted Files
Coverage Δ

src/types/src/superfly-timeline.ts
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Powershell only....


Crash when tapping on "Nearby item" card in Contributions

Summary:
Tapping on the Nearby item card in Contributions crashes the app.
Steps to reproduce:
Make sure your GPS is on and you have location permissions enabled for the app. When the card at the top of Contributions screen appears, tap it.
System logs:
STACK_TRACE=java.lang.NullPointerException: Attempt to invoke virtual method 'android.content.res.Resources androidx.fragment.app.FragmentActivity.getResources()' on a null object reference
at fr.free.nrw.commons.nearby.fragments.NearbyParentFragment.centerMapToPlace(NearbyParentFragment.java:655)
at fr.free.nrw.commons.contributions.MainActivity.centerMapToPlace(MainActivity.java:296)
at fr.free.nrw.commons.nearby.NearbyNotificationCardView.lambda$setActionListeners$0$NearbyNotificationCardView(NearbyNotificationCardView.java:86)
at fr.free.nrw.commons.nearby.-$$Lambda$NearbyNotificationCardView$VOPNkXLFp2_oU8vL3IdOKygUkxo.onClick(Unknown Source:4)
at android.view.View.performClick(View.java:7862)
at android.view.View.performClickInternal(View.java:7831)
at android.view.View.access$3600(View.java:879)
at android.view.View$PerformClick.run(View.java:29359)
at android.os.Handler.handleCallback(Handler.java:883)
at android.os.Handler.dispatchMessage(Handler.java:100)
at android.os.Looper.loop(Looper.java:237)
at android.app.ActivityThread.main(ActivityThread.java:8167)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:496)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1100)
Device and Android version:
Samsung Galaxy S20FE, Android 10
Commons app version:
ProdRelease, e454cfd98fe7537a9b95960031aa46edaf7a50ab
Would you like to work on the issue?
No

I would like to work on this issue. I think I can fix that.

Great, please feel free to start on this once you have a PR up for #4115 @Saral-code

This issue is already create - #4086 and and solved my pull request  #4105

Sorry about that, this is indeed a duplicate of #4086 , closing for redundancy.


build: include instructions for runc build

Including some build information about runc that has been helpful in my
development environment. This should allow one to build and install runc
next to containerd so that both can be worked on simultaneously.
Signed-off-by: Stephen Day<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2460   +/-   ##
=======================================
  Coverage   45.04%   45.04%
=======================================
  Files          92       92
  Lines        9424     9424
=======================================
  Hits         4245     4245
  Misses       4496     4496
  Partials      683      683

Flag
Coverage Δ

#linux
49.29% <ø> (ø)
:arrow_up:

#windows
41.33% <ø> (ø)
:arrow_up:

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 cb4bf20...dd104de. Read the comment docs.

LGTM

LGTM


matech

i  created  a  design  for  the cryptocurrency landpage  and  just  added  the  image but  could not get  the  path  to  insert  the  image  in the html  code .

Hi @mato002
Thanks for working on this. However, the design does not align with the size requirements in the issue description. Also can you update the colors to align with Chimoney's brand colors? Here's our brand guidelines for reference.
When you're done, kindly send the design via the comment section. Thank you

Okay I shall do that
On Sun, Mar 17, 2024, 8:20 PM Phyllis Atieno @.***>
wrote:

@.**** requested changes on this pull request.
Hi @mato002 https://github.com/mato002
Thanks for working on this. However, the design does not align with the
size requirements in the issue description. Also can you update the colors
to align with Chimoney's brand colors? Here's
https://drive.google.com/file/d/1vv_T3UKf4kzuEafQU7E6CThCmS7yKY22/view?usp=sharing
our brand guidelines for reference.
When you're done, kindly send the design via the comment section. Thank you
—
Reply to this email directly, view it on GitHub
https://github.com/Chimoney/Community-Website/pull/106#pullrequestreview-1941535520,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3HUK3Y4GXFBQMEOECO6AWTYYXGG3AVCNFSM6AAAAABEZLVLN6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNBRGUZTKNJSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


You are not prompted to save the changes when you change a path and close PathPlanner

After editing a .path file, PathPlanner should have a save/don't save/cancel dialog when closing it.

Added in 3e325db


android S3 upload issue.

Hi,
We are using android S3 upload with cognito in our project. While uploading a file, once after it has reached 100% upload, bytes transferred exceeds total bytes and total percentage exceeds 100%, Upload never completed, once we stopped and started again, it uploaded properly.
This issue occurs even in android aws s3 sample latest project not only in our project. we have given the output and attachment from android aws s3 sample application (latest sample used) for yor refernce.
We have used the following ANDROID SDK'S versions in our project as below,
minSdkVersion="14"
targetSdkVersion="23"
Tested android OS
Lollypop, Kitkat & Jellybean
We have tested the following AWS Mobile SDK versions for Android in our project but we are getting the same issue.
SDK Tested
aws-android-sdk-core-2.2.13
aws-android-sdk-s3-2.2.13
aws-android-sdk-core-2.2.14
aws-android-sdk-s3-2.2.14

Output
    Uploading file bytesCurrent exceeded from bytesTotal.
	 total: 17914286, current: 16300462
	 total: 17914286, current: 16447918
	 total: 17914286, current: 16562606
	 total: 17914286, current: 16808366
	 total: 17914286, current: 16931246
	 total: 17914286, current: 17283502
	 total: 17914286, current: 17430958
	 total: 17914286, current: 17545646
	 total: 17914286, current: 17914286 - Reached the max size
	 total: 17914286, current: 18028974
	 total: 17914286, current: 18258350
	 total: 17914286, current: 18479534
	 total: 17914286, current: 18725294
	 total: 17914286, current: 18749870
	 total: 17914286, current: 18889134
	 total: 17914286, current: 19118510
	 total: 17914286, current: 19356078
	 total: 17914286, current: 19470766
	 total: 17914286, current: 19732910
	 total: 17914286, current: 20085166
	 total: 17914286, current: 20191662
	 total: 17914286, current: 20543918
	 total: 17914286, current: 20715950
	 total: 17914286, current: 21068206
	 total: 17914286, current: 21420462
	 total: 17914286, current: 21780910
	 total: 17914286, current: 22034862

I have attached the screenshot of those files below,

@prakashr87 Thanks for the detailed information. A few questions

does it happen often?
in which S3 region?
will the upload complete eventually?

@fosterzhang Thanks for the reply, kindly refer below answers.

does it happen often?
Failure upto 30% of the uploads (i.e. 3/10 files failure, but it happens randomly). Same file will upload in 2nd or 3rd try.

in which S3 region?
US Standard

will the upload complete eventually?
It tooks more time (ex: 1 hour) for 10 MB file once, then still progressing, we did pause and resume then it started with 61% and went complete on 100%

@prakashr87 what's your network status? I suspect some network issue causes the transfer to retry and the progress listener doesn't update correspondingly. When you pause and resume, the progress is recalculated.

I have the same question in some time.

@jinguoliang This is a notorious bug to fix. I'll put it in our backlog.

Well, I hope make some contribution，what can I do？

@jinguoliang This is fixed in v2.2.19 release.

Where can i find this v2.2.19?
I am also facing same issue but only in marshmallow  HTC M9.

Finally i found the latest version v2.2.19.
But I tried "S3TransferUtilitySample" It is asking "COGNITO_POOL_ID" for authenticating unfortunately my region not supported for COGNITO. What i need to do?Is there any alternative?

@syam82 Cognito isn't available in all regions. However, you don't have to use the same region as your service. For example, you can use Cognito in us-east-1 and S3 in ap-southeast-1. Please open another issue if you have other questions.


[Bug]: Actions v2 throws database error on event

Preflight Checklist

[x] I could not find a solution in the documentation, the existing issues or discussions
[x] I have joined the ZITADEL chat

Environment
Self-hosted
Version
v2.63.1
Database
PostgreSQL
Database Version
postgres:16-alpine
Describe the problem caused by this bug
I wanted to test out Zitadel actions v2, especially sending a webhook to an external service on user creation. However when creating a user, I get the following error in the zitadel logs and no request is sent to my webhook. Don't know if is related, but
http://localhost:8080/resources/v3alpha/actions/executions/_search
[]

returns
{
    "details": {
        "appliedLimit": "100",
        "timestamp": "0001-01-01T00:00:00Z"
    }
}

Even after creating an execution
2024-10-04 19:30:44 zitadel  | time="2024-10-04T17:30:44Z" level=info msg=activity caller="/home/runner/work/zitadel/zitadel/internal/activity/activity.go:105" domain="http://localhost:8080" grpcStatus=0 httpStatus=200 instance=287838681627099139 isSystemUser=false method=/zitadel.user.v2.UserService/AddHumanUser org=287838837806202883 path=/v2/users/human requestMethod=POST trigger=resourceAPI user=287838878088298499
2024-10-04 19:30:44 db-1     | 2024-10-04 17:30:44.440 UTC [13130] ERROR:  could not obtain lock on row in relation "current_states"
2024-10-04 19:30:44 db-1     | 2024-10-04 17:30:44.440 UTC [13130] STATEMENT:  SELECT
2024-10-04 19:30:44 db-1     |      aggregate_id
2024-10-04 19:30:44 db-1     |      , aggregate_type
2024-10-04 19:30:44 db-1     |      , "sequence"
2024-10-04 19:30:44 db-1     |      , event_date
2024-10-04 19:30:44 db-1     |      , "position"
2024-10-04 19:30:44 db-1     |      , filter_offset
2024-10-04 19:30:44 db-1     |  FROM
2024-10-04 19:30:44 db-1     |      projections.current_states
2024-10-04 19:30:44 db-1     |  WHERE
2024-10-04 19:30:44 db-1     |      instance_id = $1
2024-10-04 19:30:44 db-1     |      AND projection_name = $2
2024-10-04 19:30:44 db-1     |  FOR UPDATE NOWAIT;
2024-10-04 19:30:44 db-1     | 2024-10-04 17:30:44.441 UTC [13133] ERROR:  could not obtain lock on row in relation "current_states"
2024-10-04 19:30:44 db-1     | 2024-10-04 17:30:44.441 UTC [13133] STATEMENT:  SELECT
2024-10-04 19:30:44 db-1     |      aggregate_id
2024-10-04 19:30:44 db-1     |      , aggregate_type
2024-10-04 19:30:44 db-1     |      , "sequence"
2024-10-04 19:30:44 db-1     |      , event_date
2024-10-04 19:30:44 db-1     |      , "position"
2024-10-04 19:30:44 db-1     |      , filter_offset
2024-10-04 19:30:44 db-1     |  FROM
2024-10-04 19:30:44 db-1     |      projections.current_states
2024-10-04 19:30:44 db-1     |  WHERE
2024-10-04 19:30:44 db-1     |      instance_id = $1
2024-10-04 19:30:44 db-1     |      AND projection_name = $2
2024-10-04 19:30:44 db-1     |  FOR UPDATE NOWAIT;
2024-10-04 19:30:44 zitadel  | time="2024-10-04T17:30:44Z" level=warning msg="missing translation" args="map[ChangeDate:2024-10-04 17:30:44.428196 +0000 UTC Code:C0SL3Y CreationDate:2024-10-04 17:30:44.428196 +0000 UTC DisplayName:Example_given Example_family FirstName:Example_given<EMAIL_ADDRESS>LastName:Example_family LastPhone<EMAIL_ADDRESS>NickName<EMAIL_ADDRESS><EMAIL_ADDRESS>VerifiedEmail: VerifiedPhone:]" caller="/home/runner/work/zitadel/zitadel/internal/i18n/translator.go:161" error="message \"VerifyEmail.Footer\" not found in language \"en\"" id=VerifyEmail.Footer
2024-10-04 19:30:44 zitadel  | time="2024-10-04T17:30:44Z" level=error msg="could not create email channel" caller="/home/runner/work/zitadel/zitadel/internal/notification/types/user_email.go:28" error="ID=QUERY-fwofw Message=Errors.SMTPConfig.NotFound Parent=(sql: no rows in result set)"
2024-10-04 19:30:44 zitadel  | time="2024-10-04T17:30:44Z" level=error msg="statement execution failed" caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/handler.go:640" error="ID=PHONE-w8nfow Message=Errors.Notification.Channels.NotPresent" projection=projections.notifications
2024-10-04 19:30:44 zitadel  | time="2024-10-04T17:30:44Z" level=info msg="process events failed" caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/handler.go:414" error="statement failed: ID=PHONE-w8nfow Message=Errors.Notification.Channels.NotPresent" projection=projections.notifications
2024-10-04 19:30:47 db-1     | 2024-10-04 17:30:47.070 UTC [13166] FATAL:  role "root" does not exist

To reproduce

Create target via api

POST: http://localhost:8080/resources/v3alpha/actions/targets
{
"name": "created_user_webhook_new",
  "restWebhook": {
    "interruptOnError": true
  },
  "timeout": "10s",
  "endpoint": "http://<IP_ADDRESS>:8000/api/v1/webhooks/user-created"
}

Create execution

PUT: http://localhost:8080/resources/v3alpha/actions/executions
{
    "condition": {
        "request": {
            "method": "/zitadel.user.v2.UserService/AddHumanUser"
        }
    },
    "targets": [
        {
            "target": "287841604754997251"
        }
    ]
}

Create user using the v2 UserService

POST: http://localhost:8080/v2/users/human
{
    "profile": {
        "givenName": "Example_given",
        "familyName": "Example_family"
    },
    "email": {
        "email"<EMAIL_ADDRESS>    }
}

Screenshots
No response
Expected behavior
Zitadel should send the event to my webhook
Operating System
Windows 10, however Zitadel is being run in Docker
Configuration, if needed:
version: '3.8'

services:
  zitadel:
    container_name: zitadel
    restart: 'always'
    networks:
      - 'zitadel'
    image: 'ghcr.io/zitadel/zitadel:v2.63.1'
    command: 'start-from-init --masterkey "MasterkeyNeedsToHave32Characters" --tlsMode disabled'
    environment:
      - 'ZITADEL_DATABASE_POSTGRES_HOST=db'
      - 'ZITADEL_DATABASE_POSTGRES_PORT=5432'
      - 'ZITADEL_DATABASE_POSTGRES_DATABASE=zitadel'
      - 'ZITADEL_DATABASE_POSTGRES_USER_USERNAME=zitadel'
      - 'ZITADEL_DATABASE_POSTGRES_USER_PASSWORD=zitadel'
      - 'ZITADEL_DATABASE_POSTGRES_USER_SSL_MODE=disable'
      - 'ZITADEL_DATABASE_POSTGRES_ADMIN_USERNAME=postgres'
      - 'ZITADEL_DATABASE_POSTGRES_ADMIN_PASSWORD=postgres'
      - 'ZITADEL_DATABASE_POSTGRES_ADMIN_SSL_MODE=disable'
      - 'ZITADEL_EXTERNAL_SECURE=false'
      - 'ZITADEL_EXTERNAL_DOMAIN=<IP_ADDRESS>'
    depends_on:
      db:
        condition: 'service_healthy'
    ports:
      - '8080:8080'

  db:
    restart: 'always'
    image: postgres:16-alpine
    environment:
      - POSTGRES_USER=postgres
      - POSTGRES_PASSWORD=postgres
      - POSTGRES_DB=zitadel
    networks:
      - 'zitadel'
    volumes:
      - postgres:/var/lib/postgresql/data
    healthcheck:
      test: ["CMD-SHELL", "pg_isready", "-d", "zitadel", "-U", "postgres"]
      interval: '10s'
      timeout: '30s'
      retries: 5
      start_period: '20s'

networks:
  zitadel:
    external: true
volumes:
  postgres:

Relevant Configuration
Nothing special. Token for target and execution creation was generated from a service user who is a IAM_OWNER
Additional Context
No response

Solved. Was not an actual issue of the Zitadel application itself
Was following this guide (https://zitadel.com/docs/apis/actions/v3/testing-locally)
which had an outdated documentation (which is understandable, as it is still in alpha state 😄)
had to update the body for the execution creation from
{
    "condition": {
        "request": {
            "method": "/zitadel.user.v2.UserService/AddHumanUser"
        }
    },
    "targets": [
        {
            "target": "287952341628354562"
        }
    ]
}

to
{
    "condition": {
        "request": {
            "method": "/zitadel.user.v2.UserService/AddHumanUser"
        }
    },
    "execution": {
        "targets": [
            {
                "target": "287952341628354562"
            }
        ]
    }
}

Additionally had another problem, but I will add it here, if someone is having the same problem.
Add port mapping to make Zitadel Actions access microservice running on WSL
My dev environment is on Windows, while running the microservice via WSL. So by default Zitadel, running in a Docker container, is not able to reach it via the http://host.docker.internal:8000/my/path url. To solve this, you have follow these steps

In WSL cmd get ip address of WSL

ip addr show eth0

In Powershell with admin rights, setup port forwarding to WSL address

netsh interface portproxy add v4tov4 listenport=8000 listenaddress=<IP_ADDRESS> connectport=8000 connectaddress=<YOUR_WSL_ADDR>

Validate if Zitadel docker container is able to access you microservice running on WSL

docker run --rm -it --network="zitadel" alpine /bin/sh
apk add curl
curl http://host.docker.internal:8000/my/path


Multiversion Sphinx

We would like to have a toggle button to switch between master and other versions like mne has https://mne.tools
As suggested in https://github.com/braindecode/braindecode/pull/248 , we could use sphinx-multiversion:

have a look at
https://holzhaus.github.io/sphinx-multiversion/master/index.html
it's used here for example:
https://github.com/mne-tools/mne-nirs/blob/master/doc/conf.py#L49

Right now our Github Action deploys the docs from the master branch to a subdirectory master:
https://github.com/braindecode/braindecode/blob/364bd331f522f5ce18ef3b50781dfbf105882358/.github/workflows/tests-and-docs.yml#L81
This was done to prevent that the stable/last-release docs are being overriden, which caused quite some confusion in users before.
We now would want a proper setup where older docs still exist and can be seen through the toggle button, the default on braindecode.org should be the last release docs. If you build locally, the default might better be the docs from current code.

Related with #272.


Insert (inline) span, rather than (full line) div, when dragging reporters

This PR makes it so that when reporters are dragged over the code area, they will insert a  (within the targeted line), rather than a  (next to the targeted line). It also makes some minor code and style changes, particularly displaying a preview of the text that will be inserted with the element. Things to think about going forward, that aren't addressed by this PR:

Currently, the temp-span will be placed at the start of the line (column zero). In the near future, I want to write code to guess good positions to insert the block, like between the parentheses in a line that reads if () {.

Replacing literals (e.g. 0 in this.goTo(0, 0)) would be handy too!
If I'm going to tackle this, I'd rather do it in a separate PR, or at least get the code here reviewed and tested first.

Blocks which contain multiple lines of code, like "if", look weird in the preview. Would it make sense for the preview to stretch however many lines the content takes?

Or we could just display the preview without any line breaks, like in the block palette.

Code still isn't inserted at all when it's dropped.
There may be some general code cleanup that can be done with the stuff I've written here too, shrugs! :P

Definitely encourage reviewing the code and leaving review comments (the "+" button on each line of code) if you're wondering about the behavior of anything in this PR! That way I can clarify and potentially add code comments describing the purpose of this PR's behavior. Also be sure to test this yourself locally! I messed around with some existing code; though I tried to not mess up any existing behavior, getting a second pair of eyes on it would help confirm it's good, since this is a moderately large PR.

Thank you for the contribution :D! Will merge right now

ok, how to open a closed pr? lol


Unterminated single-quoted strings do not emit errors or properly slice

Observe:
~> put `working string`
▶ `working string`

That works. However, if you don't end the string:
~> put `cut-off string
▶ `cut-off strin`

Compare with double-quoted strings, which error on the same input:
~> put "cut-off string
<tty 5>:1:5: parsing error: invalid syntax
put "cut-off string
    ^

That's not where it ends. It gets worse:
~> put `
panic: runtime error: slice bounds out of range [recovered]
    panic: runtime error: slice bounds out of range

elvish/errutil.Catch(0xc82005fbb8)
    elvish/errutil/exception.go:23 +0xe0
elvish/parse.unquote(0x5, 0x4, 0xc82000b5c4, 0x1, 0x2, 0x0, 0x0, 0x0, 0x0)
    elvish/parse/parse.go:335 +0x153
elvish/parse.(*Parser).primary(0xc82005fbe0, 0xc8201da5e0)
    elvish/parse/parse.go:385 +0x247

...

Seems the parser has some holes... Will look into it.

This is fixed by the new parser. Also, the syntax for single-quoted strings now use ' instead of ` .


Atom version

Any plan for atom version?

No, we're afraid but there aren't any plans to provide an atom version, this project was dedicated to vscode and probably will ever be, as far as I know.


Domains that resolve to localhost do not show up in recommended policy

https://github.com/Automattic/vip-go-mu-plugins/pull/2815#issuecomment-1014226438

@varunsh-coder I am handling this issue


shipping tab content is not displayed in product details page

Should fix test 'should contain tabs with correct text' skipped in #1128

Problem has occurred after merge https://github.com/SAP/cloud-commerce-spartacus-storefront/pull/1391

passed on dev17 feb27 with latest spartacus


Readme: make examples easier to read

The \ still makes it possible to paste it into a terminal.

@rstacruz thanks for the contribution!


Item counter in chat messages is off by one

In Fetchr 5.1.4 Beta 1, the item counter that is shown next to the time in chat messages when someone has gotten an item is off by one. The very first "player got item" message will show an outdated value from the previous game, then the second message will be shown as item 1, even though it is item 2.
Bug in action: https://www.youtube.com/watch?v=yw1E9pU-0ss&t=4770s

The very first item in a completely new instance will also be shown as # https://www.youtube.com/live/yw1E9pU-0ss

Thanks for reporting this @bl4ckscor3, life got in the way of me getting it ready 🙄
K


management: Don't prepend "HTTP_" to certian headers in parse_headers.

Django does not prepend "HTTP_" to "Content-Type" or "Content-Length" headers. So parse_headers should not do that either This was just a small bug I noticed while working on something else.

Nice, merged as 1a33d73554ef77230e3fa25148af3d6b9c051ca7, after adding a comment linking to the right part of the Django docs for understanding that this code is correct.
Thanks @Hypro999!


Do we need to calibrate per field?

Look at how 971 does per-comp mapping of AprilTags.  Link to talk is below:
https://www.chiefdelphi.com/t/psa-field-tolerances-for-vision/457135/22

Check out Holy Cows process as well.

from other issue:
Not sure if we want to go back to xml parsing for this or store it in a normal header file, but we want to be able to easily edit the relative positions of the April tags, so that it's easier to calibrate per field.


[feat] Add check_error_condition function to engine

After a script executes with the Shinigami script engine, there are various checks that need to be made to ensure the script was successful.
BTCD implemented these inside a CheckErrorCondition function. We will need a similar function in our engine to ensure the Bitcoin transaction is valid.
Resources

btcd CheckErrorCondition
Shinigami engine

I would love to take this issue @b-j-roberts

Hi @b-j-roberts can I work on this?

@b-j-roberts, i would like to work on this.

I'll assign you @Jemiiah , please lmk asap if you have any questions!