python310Packages.pylitterbot: 2023.1.2 -> 2023.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.pylitterbot is: Modulefor controlling a Litter-Robot
meta.homepage for python310Packages.pylitterbot is: https://github.com/natekspencer/pylitterbot
meta.changelog for python310Packages.pylitterbot is: https://github.com/natekspencer/pylitterbot/releases/tag/2023.4.0
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2023.4.0 with grep in /nix/store/bisx4j0a7jfsfxv32j547938p2sz0dda-python3.10-pylitterbot-2023.4.0
found 2023.4.0 in filename of file in /nix/store/bisx4j0a7jfsfxv32j547938p2sz0dda-python3.10-pylitterbot-2023.4.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
home-assistant-component-tests.litterrobot
python310Packages.pylitterbot
python311Packages.pylitterbot

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bisx4j0a7jfsfxv32j547938p2sz0dda-python3.10-pylitterbot-2023.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.pylitterbot https://github.com/r-ryantm/nixpkgs/archive/4e2abf09265d3a3f26ae1f88751a033d0386a09a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bisx4j0a7jfsfxv32j547938p2sz0dda-python3.10-pylitterbot-2023.4.0
ls -la /nix/store/bisx4j0a7jfsfxv32j547938p2sz0dda-python3.10-pylitterbot-2023.4.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

home-assistant-component-tests.litterrobot

2 packages built:

python310Packages.pylitterbot
python311Packages.pylitterbot

Maintainer pings
cc @fabaff for testing.

Is the postPatch still necessary @fabaff?

Is the postPatch still necessary @fabaff?

I will remove it.


Nishnha/gsu multi dir python

creates a different group for each directory
I plan on merging the different groups together by making the first directory a DependencyChange object and merging it with the next directory's DependencyGroupChangeBatch objects and so forth

Closing out since https://github.com/dependabot/dependabot-core/pull/8331 seems like the better approach.


possible use case?

Hi folks, is it possible to use these cards as a note-card plugin, I would like to have something that would allow me to display relatively short markdown notes that I could shuffle and reorder, merge etc.

I don't think this is possible with this plugin, but you can check out the obsidian kanban plugin. It might come close to what you want.

As @jedth has said, it is not possible at the moment with CardBoard.  In addition to looking at kanban, the next item that the core obsidian dev team are working on is Canvas:
You can use the Canvas view to organize notes visually.  With Canvas view you can create mind maps, flow charts, org charts, moodboards, family trees and more.
This sounds like it is exactly what you are asking for!


Readthedocs

Move the README to docs and use it in docs landing page

Coveralls failed
🚀 Posting coverage data to https://coveralls.io/api/v1/jobs
HTTP error:
---
Error: Gateway Time-out (504)

Their status page says their edge service provider is doing some maintenance in Salt Lake City and Houston that may add latency, so I think this is just an upstream issue. I'll try again in a bit.


Support to gokart in github actions marketplace

hey hey do you have any plans to create a gokart action for us to use in github actions?
I'm testing the gokart, I'm finding it really cool and I'd like to add it to github actions for an action very similar to gosec because of this the my question
thanks 😊

Thanks for your interest and test driving @renanpalmeira!  While support for GitHub actions is not currently on our list of fast-follow features, the creation and use of SARIF absolutely is!
Once we have SARIF output properly supported, it should be an easy lift to create a Github action around it.  Please keep an eye on the repo over the next couple weeks for new releases.

I've made this! Please try it out: https://github.com/kitabisa/gokart-action

@isp1r0 is there any update on the SARIF support? Do you plan to support taxonomies in your SARIF report?


Include Page URL or template include() stack trace information with each hint

We use a fairly robust "content block" setup for all of our sites, which results in several levels of included templates. While the current hints include the template:line that are creating the recommendation, it sometimes makes it pretty difficult to track down when that particular template can be included several different ways. It would greatly help to have the page URL that triggered the hint, or even a stack trace of the template include()s to help track down the actual occurrence.

Thanks for the suggestion. Added in 2.2.0.


Pretrained model got poor result (RMSE=1343.609)

Hi @fangchangma Thanks for sharing the code. I evaluated the pretrained model provided in readme. The result is not as good as reported in the paper (rmse 1343 vs 814). It was a clean clone and I followed the data folder structure. I attached the command and the screenshot of the results. Please let me know if there is an error or if I missed something here. Thank you.
python main.py --evaluate<EMAIL_ADDRESS>=> output: ../results/mode=sparse+photo.w1=0.1.w2=0.1.input=gd.resnet34.criterion=l2.lr=1e-05.bs=16.wd=0.pretrained=False.jitter=0.1.time=2019-05-08@10-21
Val Epoch: 8 [990/1000]	lr=0 t_Data=0.001(0.001) t_GPU=0.014(0.023)
	RMSE=1086.59(1347.03) MAE=308.10(359.76) iRMSE=4.29(4.27) iMAE=1.50(1.64)
	silog=4.67(5.24) squared_rel=0.00(0.01) Delta1=0.994(0.992) REL=0.018(0.020)
	Lg10=0.007(0.008) Photometric=0.000(0.000)
=> output: ../results/mode=sparse+photo.w1=0.1.w2=0.1.input=gd.resnet34.criterion=l2.lr=1e-05.bs=16.wd=0.pretrained=False.jitter=0.1.time=2019-05-08@10-21
Val Epoch: 8 [1000/1000]	lr=0 t_Data=0.001(0.001) t_GPU=0.014(0.023)
	RMSE=1005.17(1343.61) MAE=262.70(358.79) iRMSE=4.57(4.28) iMAE=1.57(1.64)
	silog=4.98(5.24) squared_rel=0.00(0.01) Delta1=0.993(0.992) REL=0.018(0.020)
	Lg10=0.007(0.008) Photometric=0.000(0.000)
*
Summary of  val round
RMSE=1343.609
MAE=358.790
Photo=0.000
iRMSE=4.277
iMAE=1.642
squared_rel=0.006554501281207195
silog=5.2404233943858145
Delta1=0.992
REL=0.020
Lg10=0.008
t_GPU=0.023
(best rmse is 1343.609)
*

I have a trained model myself on half of the kitti data and got mae=0.36 and rmse=1.08. Exact same parameters as the repo but using my own training script.

I have a trained model myself on half of the kitti data and got mae=0.36 and rmse=1.08. Exact same parameters as the repo but using my own training script with only depth supervision (no photometric stuff)

Thanks for the reply. I just found it could be because I mixed the self-supervised training model with the supervised training model. I'll do some more experiments. Closing the issue for now.


ValidationRule only on custom button action

Hi @volkanceylan,
I managed to add a new validationRule in an EntityDialog, but I was wondering if there is a way to "activate" this validation rule only when I press my custom button and not (for example) when I press the standard update / apply buttons.
Thanks

Store which button clicked and check it there.

Do you mean in the OnClick event of the buttons?
So every ToolButton set itself (id?) in a global Dialog variable?

Only save buttons trigger validation. You should override their Click events, storing reference to old handlers, when clicked write it to a variable, then in validation method check which is clicked. But i'd recommend skipping validation system completely and checking form input value separately for your button, showing a Q.Alert if there is an error.

Copy that!
I'll try...once again thanks for your support!


Fix JSONDecodeError on get_channel from EU countries

This fixes #36
More info here.

hey,
it looks like there is an smarter way - i checked this for the videos
def get_initial_data(session: requests.Session, url: str) -> str:     url = url+"&ucbcb=1"     response = session.get(url)     html = response.text     return html

Hey @Nutzer123!
I certainly prefer your fix better.  I've implemented it into this PR, but I've changed how the parameter is passed to the request.
Thanks!

thanks for this @twissell- and @Nutzer123


Decrypt mode should be in decrypt doc section

Also add a toc entry

This actually seems to be a bug in the ruby OptionParser.

I care less about this... I don't feel like testing it in newer versions of ruby to see if it is fixed.

This was fixed in b89ffe07e0d49661c27c29bc00c5bfc8de78fc91 by using opts.on instead of opts.on_tail.


[BUG]  Indicate Aux In(x18) Mute status Feedback appears to be broken in 3.3.1

This feedback doesn't seem to be working properly since we updated to 3.3.1, using an XR18 on FW version 1.22. The mute status for channels seems to still work ok as far as I can tell

Confirmed. I'll look into this later this evening.


nixos/test-driver: don't add QEMU_OPTS twice

Description of changes
As the TODO says, this is already included by the script.
If adding a device, including this again here would result in either two devices being added, or, if they were explicitly named, an error due to reuse of the name.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thank you!


Automated cherry pick of #4658: fix: 避免同步安全组失败后,没有任何日志记录

Cherry pick of #4658 on release/2.11.
#4658: fix: 避免同步安全组失败后,没有任何日志记录

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@ad7a545). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.11   #4663   +/-   ##
==============================================
  Coverage                ?   7.09%
==============================================
  Files                   ?     480
  Lines                   ?   78151
  Branches                ?       0
==============================================
  Hits                    ?    5541
  Misses                  ?   72144
  Partials                ?     466

Flag
Coverage Δ

#aFlag
7.09% <ø> (?)

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 ad7a545...54fd08c. Read the comment docs.

/lgtm

/approve


About how to get and use ILoggerFactory

[Enter feedback here]
I had an issue to use ILoggerFactory. While I was trying to resolve the issue, I found one thing below.
There are 3 methods in Startup.cs. 1. Startup(), 2. ConfigureServices() and 3. Configure(). Calling sequences are 1, 2 and 3. Then 1 and 3 can have “ILoggerFactory loggerFactory” as a parameter. And we use “services.AddApplicationInsightsTelemetry();” in 2. After finish to call 2, ILoggerFactory is created and configured and can be used. So, we have to store the ILoggerFactory instance from 3 not 1. I couldn’t find this information from anywhere in MS documents. If this is added to your site (https://docs.microsoft.com/en-us/azure/azure-monitor/app/ilogger), it will be very helpful for many developers.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b09a09da-72ae-d5c0-fe25-69274054d302
Version Independent ID: 6911ee60-7c91-f024-8788-94939567d477
Content: Explore .NET trace logs with ILogger - Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/ilogger.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @lgayhardt
Microsoft Alias: lagayhar

@ss185475 Thanks for your comment.
Assigning to content author @lgayhardt for review.

@ss185475 The ILogger doc that you linked to is mainly for to how use App Insights to capture ILogger logs. The issue you had is more related to logging in general. This doc on the Startup class in the ASP.NET Core documentation talks about how to setup Startup(), ConfigureServices() and  Configure() in more detail.
Thank you
#please-close


[GoTrue] Implement a way to reauthenticate the current user

moved to https://github.com/zoedsoupe/gotrue-ex


⚠ Bofrost.es has degraded performance

In 101d3fa, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1627 ms

Resolved: Bofrost.es performance has improved in 3acb960 after 6 minutes.


iOS / Android experiment: Moire Camera "Viewer lens" app

We need to prototype a Moire lens with iphone and android.  Let's start with which ever one is easier to get going.
My objective, create a moire pattern with a camera on a phone using something like opencv
https://en.wikipedia.org/wiki/Moiré_pattern
I'll outline the code base to start with an how we will run this experiment.  The app will just  :

launch
load a camera in the app
modify the camera view with an overlay of black lines at alternating distances (even) and with a particular width
we will need to select a test pattern/image to verify against

For technical background on a prototype I already made, try this on the PC/mac: https://jsfiddle.net/truedat101/befu6n2t/2/

The lens can be a simple vertical line set as seen in the example jsfiddle above.

Test pattern can be any one of these:

Marking this as done.


Does not load after reload or stop

Name
Does not load after reload or stop
Description:
After i have reloaded or restaret the server either this or the 1vs1 plugin loads its map and resets to 0 maps
How to reproduce?

Run PMMP with this plugin
Restart the server with a arena allready configured

Server version:

3.8.0
Other informations:

#53


set x-request-id header in XMLHttpRequestController only for node js

@kettanaito @msutkowski @marcosvega91 @timdeschryver could you please review this


Misc fixes typo on Mad Libs

Hello there,
Little fix where initially, the sentence that needed to be rewritten started with a majuscule but the assertion asks for a minuscule.
Screens
before:

after:

I think we need to change the hint rather than the code cell - the code cell should actually have nil for each variable, so that's also an issue!
Great catch! I'll make a quick edit to this and merge it in.


Added extra logging.

Coverage increased (+0.71%) to 87.24% when pulling 1c6d37af066e20f6dbe117d4bd7d9ce015bebed3 on feature/verbose-logging into 211322645b89e18ea685467ddb0c85feeb7001ba on master.

LGTM and released.


Schema in multiple files

Is there a way to have the schema spread out in multiple files? In large code bases having one large file might become difficult to maintain.

run the script before your build process (maybe using go generate or Makefile)
schema.sh
for f in ./graphql/schema/*.gql; do cat $f; echo; done > ./graphql/schema_generated.gql

I'm hoping to add support for graphql-import syntax into the new parser (until something lands in a spec).

Closing in favour of https://github.com/vektah/gqlgen/issues/6


Fix folly::sorted_vector_set and minor codegen comment change

Summary
This change corrects how we handle folly::sorted_vector_set handling and adds another space into the codegen comments (so you can easily cut'n'paste the symbol without picking up the trailing slash).
I have logged task #62 as this change doesn't contain a test for this container and we need one.
Test plan
'make test-devel` passes all tests. The integration tests are failing for me currently but it's not owing to anything here (I think!).

Yeah, I've tested these changes against an application and have just transplanted the changes across.


Add Git log, globalConfig, and tree diff API

Fix for #62609,
cc @joaomoreno

The test failures doesn't look to have anything to do with my changes.


Move conformance job to shared libs

Why is this PR needed?
Does it fix an issue? addresses a business case?
add a description and link to the issue if one exists.
Fixes https://github.com/SUSE/avant-garde/issues/1955
Reminder: Add the "fixes bsc#XXXX" to the title of the commit so that it will
appear in the changelog.
What does this PR do?
please include a brief "management" technical overview (details are in the code)
Anything else a reviewer needs to know?
Special test cases, manual steps, links to resources or anything else that could be helpful to the reviewer.
Info for QA
This is info for QA so that they can validate this. This is mandatory if this PR fixes a bug.
If this is a new feature, a good description in "What does this PR do" may be enough.
Related info
Info that can be relevant for QA:

link to other PRs that should be merged together
link to packages that should be released together
upstream issues

Status BEFORE applying the patch
How can we reproduce the issue? How can we see this issue? Please provide the steps and the prove
this issue is not fixed.
Status AFTER applying the patch
How can we validate this issue is fixed? Please provide the steps and the prove this issue is fixed.
Docs
If docs need to be updated, please add a link to a PR to https://github.com/SUSE/doc-caasp.
At the time of creating the issue, this PR can be work in progress (set its title to [WIP]),
but the documentation needs to be finalized before the PR can be merged.
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

job run with this passed correctly: https://ci.suse.de/view/CaaSP/view/v4.5 Dashboard/job/caasp-jobs/job/conformance/job/v4.5/job/vmware/200/

I will reaprove after conflcit resolution


Modify security when disable_tls =True, 0x10

https://github.com/EDF-Lab/eVDriveFlow/blob/8ac55fc2594517a756a6a60db909cb42f4b641f4/shared/payloads.py#L61

TLS can now be disabled by modifying SECURITY_PROTOCOL in /shared/global_values.py


Alternative TCP DNS server to allow pipelined query handling

As previous issues have noted (https://github.com/miekg/dns/issues/646, https://github.com/miekg/dns/issues/1314), for any single TCP connection, DNS queries are handled serially. This is generally not a big problem, as

DNS over TCP is not all that common, and
typically the overhead of establishing the TCP connection after having received a truncated UDP packet means the latency is already high, the serial processing of queries doesn't significantly worsen it.

However, in the specific case of a DNS proxy, the serial processing becomes a significant contributing factor: a proxy has to forward the queries as part of the handler, synchronously. (The current handler interface doesn't seem to allow doing so asynchronously - the ResponseWriter is alive while the handler function is running, but not longer.) Since clients often send two (or more) queries approximately simultaneously, the second query incurs a delay of a full RTT to the upstream server, just waiting for the first query to be handled. Similarly so, any further simultaneous queries incur more RTTs waiting for exchanges with the upstream, which isn't great.
Now, as previous issues have established, changing the existing TCP server to be pipelined is not an option (and probably a breaking change), since the handler would have to deal with the fact that it is being called concurrently and the response writer would to be made thread-safe without losing performance.
Long story short, I'm opening this issue to inquire whether the library would accept a contribution of an alternative TCP server which does do pipelining or whether that would be considered out of scope.

It would be nice if this could be done in folded into the current tcp server? Is that doable?

It would be nice if this could be done in folded into the current tcp server? Is that doable?

@miekg I do not think it's doable, to expand on what @tmthrgd said previously:

The current handler interface is ServeDNS(w ResponseWriter, r *Msg). We would have to ensure that concurrent writes to the ResponseWriter are safe (could compromise performance if we need to lock). Also, ResponseWriter.Write([]byte) becomes an unsafe method, since it's possible to accidentially interleave bytes of multiple messages if the writer doesn't write full messages. Finally, ResponseWriter.Hijack() becomes inherently racy.
It would be a breaking change: There currently was no way the handler function would be running concurrently for the same connection, but if we implement pipelinining this can occur. That seems like a breaking change of API semantics, and hence would have to be opt-in or in a v2 of the module.

If we end up making it a configurable thing on the existing TCP server, I think it will make the implementation more complex than having two separate implementations, but that's mostly a matter of taste.

thanks for digging that up.
Does the Golang http impl do pipelining btw? As I've followed that model here a bit?

thanks for digging that up.

Sure thing.

Does the Golang http impl do pipelining btw? As I've followed that model here a bit?

The short answer is no. I admit that it's not entirely clear what the correct handler interface would look like. I've thought about it briefly, and I think we can make the handler
type handler interface {
    ServeDNS(*dns.Msg) *dns.Msg
}

The ReponseWriter in http is, afaik, primarily necessary since HTTP is a text streaming protocol. DNS doesn't have this requirement, hence we can "just" return a complete message - side-stepping the problem of writing interleaved message parts.

ok, frankly I'm worried about a sprawling API that needs to exist just for tcp pipelinging, assuming you get this for free with QUIC. Also I have a server now (say coredns) and I want to do pipelining : what needs to be changed?
OTOH: pipelning is a nice enhancement, but I'm not sure how much of a perf it will get you (assuming that's what is why you want this?)

ok, frankly I'm worried about a sprawling API that needs to exist just for tcp pipelinging, assuming you get this for free with QUIC.

Yeah, that's fair and definitely an understandable maintainer's perspective. It may be possible to instead of adding a second TCP server to add another field to the Server struct, say Pipelined bool which enables the pipelining at the cost of complicating the serve loop.

Also I have a server now (say coredns) and I want to do pipelining : what needs to be changed?

If we do the above, Pipelined: true to enable the pipelining and then a lot of difficult thinking about whether your handler is safe to be called in parallel.

OTOH: pipelning is a nice enhancement, but I'm not sure how much of a perf it will get you (assuming that's what is why you want this?)

I work on cilium which uses the Server to implement a DNS proxy. For proxying, pipelining is a big win. Imagine two queries (A/AAAA) arriving over a TCP conn. Without pipelining, the second query waits until the first query is proxied to an upstream, which increases the latency. I think recursive resolvers have the same use case.

yeah, pipelining is a win there, is there a way where you can do a small concept PR to see how this looks?
Going further, DoQ is the correct solution here, although there is no std Go support for quic yet


Add Gradient to NavigationBar

Instead of adding a layer, I set the background image for UINavigationBar

for the frame height it's not very clean. I set 64. Don't know if there is a constant somewhere
If you put only the bound height you do not draw gradient to the top of the screen

Tested on playground

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          2 Warnings

:warning:
Consider adding supporting documentation to this change. Documentation can be found in the docs directory.

:warning:
Consider adding / updating the demo ap.

  Generated by :no_entry_sign: Danger

@phimage can you please merge it when you think it is ready? 🚢🇮🇹 Well done. A great step to extend the protocols to non-UIView components. 🎉🎉🎉 I learned how to do it as well 👍

Thanks for the review


Use from_random_bytes from Zexe in try and increment

This removes a large chunk of our try and increment code which was error prone.
This is WIP as we're waiting for this to be merged into Zexe.

Fantastic! Exactly what we needed to unblock us from making the library generic over any PairingEngine

Think we need to adjust the gadget code as well

Yeah, because we changed the position we take the mask bit from. On it.


New package: Canon.IJPrinterAssistantTool version 1.90.3

Pull request has been created with komac v2.6.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

c-wss.com domain is controlled by Canon. Source: https://www.whois.com/whois/c-wss.com

Would this be a good PackageUrl?
https://www.canon-europe.com/support/consumer_products/software/ij-printer-assistant-tool.html?softwareid=tcm:13-1871303&language=EN&productid=tcm:13-2267884&os=Windows 11

Manual Validation ended with:

Should this be a nested installer?

Close with reason: Stale pull request;


Minor typo fixes

Learn Build status updates of commit 89e4217:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/defender-for-cloud/concept-attack-path.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#sign-off


chore(angular): upgrade clr-angular to Angular 12

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] clarity.design website / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

We need a new branch for this!

any news on this PR?

We need a new branch for this!

https://github.com/vmware/clarity/tree/angular-v12


nixos/graphite: delete.

Motivation for this change
The graphite module is unmaintained (although upstream is still
developing graphite). It pulls in a closure of insecure python
packages, and is the only one that depends on them.
One of its deps, Django 1.8, has been marked insecure for >1 year,
and nobody stepped up to fix it.
Fixes #52679.
Signed-off-by: David Anderson<EMAIL_ADDRESS>Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

cc @lsix because of recent django work

Thanks Jon!
Note, this does not remove Django from Nix/NixOS. We're still shipping 2 more recent versions of Django that are still maintained (by both upstream and nixpkgs). This is only removing the EOL Django 1.8.

I'd like to see the removed attributes being replaced with throws with a decent error message (probably in pkgs/top-level/alias.nix), and a mkRemovedOptionModule in nixos/modules/rename.nix for the module.

Done, I think. This is my first time doing a removal, PTAL.

I'm just not super "in-tune" with django ecosystem, which is why I would like @lsix to take a look.

Hi,
Apparently graphite-web-1.1.6 now supports django 2.x (released last October). The upstream project was dependent on unsupported django version for quite a long time.
I should probably try bring everything up to date. If I do not succeed shortly, then removal is the way to go. I’ll try to give it some time later today.
Anyway, it cannot stay as it is, so unless it is successfully updated, it should be removed.

Yes, in theory we could upgrade Graphite by multiple versions and fix it that way. I tried briefly but the derivations are quite complex and I wasn't confident I could fix it.
Combined with the fact that it's been alerting for >1y about security vulnerabilities and nobody decided to fix it then... I decided that probably means nobody on NixOS is using graphite, so removal sounded better.
If you get a new version going, please cc me on the PR and I can help review.

Why not remove just the services.graphite.web part of the module and keep the rest? Just curious, not using it myself.

Basically, I didn't feel confident that I could remove the CVE-ridden parts of the module and still leave something usable/useful behind. And given that nobody fixed this module even though it's been depending on insecure stuff for >1y tells me nobody seems to be using it... So I defaulted to cleaning it up and leaving a clean slate for someone to re-add support in the future.

I have not still much work to do, but I am getting closer. All hope is not lost (yet).
Here is a WIP: https://github.com/lsix/nixpkgs/commit/ddca6b93752fc4a3dca121ea13120bc3d82a6b88 (https://github.com/lsix/nixpkgs/tree/fix-graphite).
hopefully I’ll be able to finish this in the next few days.
Anyway, we clearly do not have users, so we could drop the module until it is properly reborn.

If possible, I think continuity is better than delete+add. I can wait a couple of days for you to upgrade Graphite, and then I can rework this PR to just delete the EOL Nix packages and not Graphite. Sounds good?

Thank you! I'll close this and review your PR.


Fixed the extents function

Minor JSDoc updates

This simplifies the extents function. I'm pretty sure the original did not work, this line here invokes teh D3 mn function:

Much neater, I like it.
I originally had the extents function in the fc namespace to mirror the d3 function but it is, in reality, a utility? (I know this isn't actually a question, but thoughts on this would be appreciated)


Proposal: React Navigation Vs React Native Flux Router / React Router

Seems like the new https://github.com/react-community/react-navigation project could replace the need for the multiple handling of navigation in https://github.com/amoghbanta/reactNativeEverywhere/tree/master/src web integration is documented here - https://reactnavigation.org/docs/guides/web
Still new but curious if this would help reduce duplication. As well redux integration seems to flow a bit better than with react-native-flux-router
Thoughts?

Hi @christopherhein, I came across react-navigation couple of days back, haven't tried it yet but it is in the near plan to test it for web and native using  an all common code base. I completely agree that it  "could replace the need for the multiple handling of navigation" and reduce the duplication. Seems theoretically perfect to me  but i could comment on it better once I give it a try.

For now it doesn't work so good.
They web implementation is very limited for now. I'm trying to do something about it but so far no success.
That's the differences:
https://github.com/react-community/react-navigation/blob/master/src/react-navigation.js
https://github.com/react-community/react-navigation/blob/master/src/react-navigation.web.js
Without StackNavigator/TabNavigator it seems to be very limited.
I would love to have it.

Ah sweet, thanks for the info!

After some tweaks I'm able to run StackNavigator :)
It should work the same on both Android, iOS and web.
The example from web:

But TabNavigator is really what I desire here.
I will try to push some example how to implement it when I make some code cleanup 👍

Oh should I reopen @kkszysiu ?

@kkszysiu , how did you get the css to work? Changing the react-navigation.web.js file allowed me to use StackNavigator but the css is all broken

@kkszysiu did you do this in a fork somewhere? wanted to check out what was required to get it working

I guess this should be reopened as react-navigation is the way to go

@amoghbanta Any update on this?

hey @fotiDim, so I've working on a complete rewrite of react-native-everywhere. The update on this is that react-router guys released v4 and that is exactly what fits our requirement. My aim is to keep it more of plug and play where in you can just replace react-router with react-navigation if you prefer that. It'll be out in a couple of days.

@amoghbanta any insights to share on where react navigation fell short?

@amoghbanta - great work and thanks!!! Any news on the rewrite?

hey @ajostergaard, this branch is where the rewrite is going on.
Now, since react-native supports templates, so this project will be more of a template(supporting multiple platforms) with some additional scripts. No more git cloning. We are also working on a component library which would support multiple platforms too.

hey @fotiDim, its just that react-router's API was an exact match/requirement for us. I personally haven't played much with react-navigation so I can't comment. As i said earlier, it would be more of a plug and play thing, so you can use the one you feel is right for your project.

Thanks @amoghbanta. How mature is the template? Ready for a relative beginner to use for an app?

Template is ready to be used, you guys can give it a try and as of now it works on react-router v4.
Closing this one.


Direct Routing SBA - Edit request to add a note on ETA for the new token lifetime (known issues section)

Hello,
As reference to the following known issue about the SBA related token 24 hours lifetime that limit the Teams users survivability from 1h to 24 hours (too restrictive for many customers who want to buy the solution but can't accept it as few hours of survivability is not enough to get the time to fix the issue, for many locations).
My request, if possible: Microsoft confirmed last year the development starting on Q1-2023 to extend the token lifetime from 24hours to at least 72hours (up to a week). Can you please sync with the product team to add the estimated time of delivery to the related note in the "Known issues" section:
Known issues

As SBA relies on authentication tokens that are valid for 24 hours and are renewed daily, presently SBA can support outages for up to 24 hours from the last authentication. This means that if an outage occurs 20 hours after the last authentication token renewal, SBA will be operational only for the remaining 4 hours.

Thank you,
Best regards,
Anthony.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 183fc921-e9b1-cc74-2bcb-26d32b994919
Version Independent ID: 1765bda3-dd62-76d7-1e2d-9f5496ac9094
Content: Direct Routing SBA - Microsoft Teams
Content Source: Teams/direct-routing-survivable-branch-appliance.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

Hello @ajeob
Can you please point me into this announcement?

Microsoft confirmed last year the development starting on Q1-2023

So I can start digging into this. Thank you

Adding @FilippSe

Hello @get-itips,

Hello @ajeob
Can you please point me into this announcement?

Microsoft confirmed last year the development starting on Q1-2023

So I can start digging into this. Thank you

Here is the answer we had from Filipp Seljanko:

Thank you for you support,
regards.

Hello @get-itips, @ajeob, @FilippSe:  GitHub issues should be about content issues--it is not the forum to discuss development dates.  @ajeob: Please follow your normal technical support channels.  Thank you.

Hello,
Sorry, I thought it would be possible to add the planning information on the documentation as there is an impact for customers.
The Direct Routing SBA can help them on sites where the WAN/Internet connectivity is not reliable (for example in south America), but they can't accept having only few hours of survivability. So the Microsoft DR-SBA product can't be sold.
We'll see if we can try again with normal technical support channel.
Thank you,
best regards.

Closing.


SNOW-862622 Add support for timestamp variations

Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
SNOW-862622 Add support for timestamp variations

Fill out the following pre-review checklist:

[x] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.

This PR adds support for Snowflake timestamp variations, e.g., TIMESTAMP_NTZ, TIMESTAMP_TZ, TIMESTAMP_LTZ. Today Snowpark Python only supports Timestamp alias for Timestamp types and the underlying type is controlled by TIMESTAMP_TYPE_MAPPING. This PR allows user to specify which variation of TIMESTAMP type to use by adding a TimestampType attribute tz.

Looks good. You might want to test in stored proc (job) and see if it works.

running https://ci-dev-142.int.snowflakecomputing.com/job/PythonStoredProcBuildPrecommitTest/464/


psh: remove login applet ambiguity

Changes remove ambiguity between well known login command in UN*X like systems and current implementation of login applet in psh.

login applet renamed to auth and moved to pshapp directory
pshlogin moved to separate directory to allow execution, testing and excluding from building
trying to start second interactive psh (pshapp) under one psh process returns a message (either called from psh or pshlogin command)

JIRA: BES-164

seems good, please fix styling errors so we could merge this.

My first change did not pass the scenario:

enter pshapp commandline
enter command pshlogin
escape the login prompt without entering good credentials (e.g ctrl+d)
It was impossible to escape pshlogin prompt without passing good credentials.

I think exposing pshapp internals to other applets isn't the way to go. Commonly we have:

init - some init scripts (init.rc on older systems, systemd on newer ones) - which monitor system services and restarts them when necessary - especially getty for every physical/virtual terminal available
getty - prepares serial console + executes login when needed  - when exited will be restarted by init
login - authorizes the user and spawns the shell if succeeded

(note: the login than can be easily reused, it's used for example in telnet server)
I think our approach should be to have pshlogin - getty + init equivalent - calling auth repeatedly and psh if succeeded, not exitable from inside (in the future: ability to run pshlogin on other UARTs). That way we get rid of complex isrunpsh logic (exiting from "top" psh will bring us back to pshlogin - so we get the auth prompt. (of course having root privileges mean we could still kill pshlogin directly, but that's not an issue for now).

There are still couple of valid remarks from reviewdog to be fixed

There are still couple of valid remarks from reviewdog to be fixed

Please note that my issues from previous comment are still not addressed ;)

niewim19 what is the status of addressing mentioned issues?

Addressing  issues of previous reviewers was postponed due to involvement in micropython in last week. I will rewrite psh to form proposed by nalajcie asap.

Latest changes in this PR are very different from the first approach so now it is almost as new PR for review (however - shorter)
Summary of new approach
Logging into psh is now handled in psh, not through pshlogin applet. pshlogin app is a facade for pshapp (just to handle executing from different name)
login applet ambiguity changes:

moved login to pshapp directory not to collide with well known UN*X like login applet
change name login to auth

pshlogin applet changes:

pshlogin is pure facade applet for pshapp
pshapp and pshlogin exitability is handled through check of foreground terminal process id
first psh is not exitable no matter if it was launched using login prompt or not.

Changed return codes of auth and left nonexitability only for psh call under pshlogin name.
In next psh related PR i will address the issues of the rest of psh return codes, together with removing isrunpsh hack and assuring good psh exitability if needed.

As changes in both commits were somehow back-and-forth-ing some features i squashed two commits.


TypeError: Cannot read properties of undefined (reading 'array')

const account = {
authority: anchorWalletObj.publicKey,
collectionMint: collectionPDA,
metadataAccount: metadataAccount,
masterEdition: masterEditionAccount,
tokenAccount: associatedTokenAccount,
tokenMetadataProgram: MPL_TOKEN_METADATA_PROGRAM_ID,
systemProgram: anchor.web3.SystemProgram.programId,
rent: anchor.web3.SYSVAR_RENT_PUBKEY,
};
const tx = await program.methods
  .createCollectionNft(name, symbol, jsonUri)
  .accounts(account)
  .signers([])
  .rpc();

You seem to be using v0.29 which has the following code:
https://github.com/coral-xyz/anchor/blob/fc9fd6d24b9be84abb2f40e47ed3faf7b11864ae/ts/packages/anchor/src/program/accounts-resolver.ts#L552
This has already been fixed in v0.30 (https://github.com/coral-xyz/anchor/pull/2824):
https://github.com/coral-xyz/anchor/blob/852fcc77beb6302474a11e0f8e6f1e688021be36/ts/packages/anchor/src/program/accounts-resolver.ts#L451

i,m using v0.30.1

That's weird because there is no unchecked .array access in v0.30 as mentioned above. Let's reopen it then, but note that there isn't enough information to reproduce this problem.
Also FYI: you should be able to remove at least half of those accounts (from client), see account-resolution.

That's weird because there is no unchecked .array access in v0.30 as mentioned above. Let's reopen it then, but note that there isn't enough information to reproduce this problem.
Also FYI: you should be able to remove at least half of those accounts (from client), see account-resolution.

still can,t resolve it


Update fix/analytics data export cleanup

Description
[Provide a brief description of the changes made in this PR]
This PR creates a more readable response when the wrong data filter is posted for the data download.

Summary by CodeRabbit

New Features

Enhanced error handling for invalid filter requests in data export and summary functionalities.
Improved consistency in API responses, including structured error messages.

Bug Fixes

Refined exception handling to omit sensitive details in error messages for security.

Documentation

Updated method signatures to reflect changes in error handling and response structure.

thanks @NicholasTurner23 , I am now waiting for the deploy preview to confirm that all is working well. @Codebmk

@Baalmart I think we have talked about the google auth issue in the preview about 3 or so times. That is what is causing the failure.

@Baalmart I think we have talked about the google auth issue in the preview about 3 or so times. That is what is causing the failure.

alright @NicholasTurner23 I had forgotten


Toggle to hide/show hidden folders? (ElFisho functionality)

Just as an option to unlock the features that are included here. https://chrome.google.com/webstore/detail/custom-colors-for-desmos/clcdbeolcdigfefgbhlgghepjfkpdlgi

Do you want the feature "Set up and use custom colors" (without using rgb/hsv function)?
Otherwise, duplicate of https://github.com/jared-hughes/DesModder/issues/10 (which I am currently working on) for the hidden folders, clickableObjects, and advancedStyling.

Closed by https://github.com/jared-hughes/DesModder/commit/62e5a39a55b7381e6a7e33ae3bbdac97d9acd3a5


Missing active_low attribute

Hi
I am currently working with a GPIO chip controller on a HP server (I don't know the model no.) - one of the attributes of pins on this chip is active_low, which determines whether the logic of activating the pin is inverted, i.e. if active_low = 0 then basically for pin values 1 = high and 0 = low, but if active_low = 1 then this is inverted to become 1 = low and 0 = high.
I cannot see this being indicated in the Pin class constructor or the Controller class you have written.  I would like to add this via a PR.  Is that OK?
SM

Hey SM,
First, I'm glad to see that you are using this package!
Indeed active_low isn't actually implemented. Although I haven't noticed this on GPIO controllers I used when implementing the module, active_low is a Sysfs GPIO standard, and I would be very glad to accept a PR implementing it! Go for it! :+1:
Thanks!
Derek

Thanks for the info.  Actually I wrote my own API for the sysfs GPIO before I came across this, and noticed there were some differences.  It seems the way you read and write to the sysfs GPIO filesis via Python file objects, but I found that this was not actually not possible with my setup (I was running Ubuntu 15.10 and the GPIO driver was of type ich) because for example /sys/class/gpio/export and /sys/class/gpio/unexport could not be read or written to, even as root.  So I had to use system calls via subprocess.  So for example, here's some of my code for exporting a pin, writing a pin value (assuming it is a 'out' pin), and listing exported pins as integers.
from subprocess import CalledProcessError, check_output, STDOUT
....
# export pin
try:
    subprocess.call('echo %d > /sys/class/gpio/export' % pin_number, stderr=STDOUT, shell=True)
except CalledProcessError
    raise

# set pin value
try:
    call('echo %d > /sys/class/gpio/gpio%d/value' % (value, pin_number), stderr=STDOUT, shell=True)
except CalledProcessError:
    raise

# list exported pins as numbers
['gpio436,gpio462']
import re
try:
    map(
        lambda s: int(s.strip('gpio')),
        filter(
            lambda t: re.match(r'gpio[0-9]{3}$', t),
            check_output('ls /sys/class/gpio', stderr=STDOUT, shell=True).strip('\n').split('\n')
        )
    )
except CalledProcessError:
    raise

Damn, this is really sad! The scenario I use is popular embedded devices, like Raspberry Pi, in which case the module works perfectly. I found strange that Python can't write to the files, as echo 1 > /sys/class/gpio/gpioN/value actually opens a subprocess (in this case user's shell) as root, and does pretty the same as what Python does: Opening a file and then writing to it.

So that seems an issue then with the hardware perhaps?  What make is the GPIO chip you use with the Pi?  The ICH chip is Intel I believe.
http://cateee.net/lkddb/web-lkddb/GPIO_ICH.html

I don't think it is a Python issue in my case - I could not for example do cat /sys/class/gpio/export, it would complain of an I/O error or some input error (I can't remember, I'm not at my desk).  These files that sysfs exposes are not real files, if they were you could cat them.

Solved by #5!


Adding localization coverage for Career Coach

Included list of supported language to docs:
Career Coach is localized in the following languages:  - Chinese (Simplified, Mainland China)  - Chinese (Traditional, Taiwan)  - English (US)  - English (UK)  - French (Canada)  - French (France)  - German (Deutschland)  - Japanese (Japan)  - Portuguese (Brazil)  - Spanish (Spain)  - Spanish (Mexico)

I'm approving and merging.


Implement task callbacks

When a task is completed, app servers need to be able to notify the task executer of the new task status. The results will then be processed by the task executer and the relevant task plug-in, before the task is marked as complete.
Task callbacks will be executed using a dedicated event in the message broker. This event will include the information the task executer needs in order to complete the task. When the workflow manager picks up that event, the following needs to happen:

Task information retrieved from MongoDB
The correct task plug-in instantiated
The plug-in is given the callback event payload
The plug-in is responsible for tidying up, and returns:
4.1. A list of artefacts that have been generated by this task execution
4.2. A dictionary with the metadata generated by the task execution
4.3. The status of the task
Task status, artefact and metadata information is saved with the task in MongoDB
Task complete event published to RabbitMQ

Event Schema
Queue: md.task.callback
{
  "Timestamp": timestamp,
  "ExecutionId": string,
  "CorrelationId": string,
  "EventDetails": dictionary - specific to the app server, provided to the plug-in.
}

The format of the callback event or API call can differ between app servers. We need to define an overall wrapping object with the minimum set of information that we need, for example the task ID-but otherwise the content of this object will eventually be passed to the plug-in itself, so the main interface needs to be between the plug-in and its app server.
Task statuses
Tasks can finish in one of two statuses:

Success
Error

If the task finished with an error, an optional error message can also be provided by the plug-in and should be persisted to MongoDB.
Note: Inference operations can have more complex statuses, for example the task may finish successfully but fail to reach a decision due to lack of statistical significance. This is still represented as a success for the purposes of the workflow manager. To allow for these more complex nuanced statuses the metadata dictionary may be used by plug-ins to indicate the result in further detail.
Database schema
the task object is the same task object created by the workflow executer. The attributes that should be updated by the task executer upon completion of the task are:

status
output_artifacts
metadata

Note that the artefact path should be the relative path from the output directory, and not the absolute path in MinIO.
{
   ...
    "status": "created/in_progress/success/error",
    "output_artifacts": {
        "ARTIFACT_NAME": "/relative/path/to/artifact/in/MinIO/output/directory"
    },
    "metadata": {
      "key": "value"
    }
}

Acceptance Criteria

GIVEN an app server finished running the task, WHEN a callback event is published to RabbitMQ THEN it is picked up by the task executer for processing
GIVEN the callback event has been picked up by the task executer, WHEN the task executer completes the task THEN a task complete event is published to RabbitMQ
GIVEN the callback event is valid WHEN an error occurs while processing the callback event THEN the event is not acknowledged
GIVEN the callback event is invalid WHEN an error occurs while processing the callback event THEN the event is rejected
GIVEN the task plug-in has provided output artefacts WHEN the event is updated in the database THEN the artefact details should be persisted
GIVEN the task plug-in has provided output meta data WHEN the event is updated in the database THEN the metadata should be persisted
GIVEN the task resulted in an error AND the task plug-in has provided an error message WHEN the event is updated in the database THEN the status and error message should be persisted

__

@mocsharp please take a look at the above. This is what I had in mind, and I'm not sure how it will work if the task executer doesn't have access to the database – let me know what you think.

There are 3 events used to communicate between the Workflow Manager <-> Task Manager & Task Manager <-> Task Plug-ins (including external services). TaskDispatchEvent.cs, TaskUpdateEvent.cs and TaskCallbackEvent.cs.

TaskDispatchEvent is sent from WM to TM to start a task execution
TaskUpdateEvent is sent from TM to WM to indicate a change in the task, including acceptance of a task as defined in TaskStatus.cs.
TaskCallbackEvent  is sent from either the task plug-in or the task execution service (external) to indicate completion of the task.

As discussed earlier, we will share the same database but in different collections (mongo) for the POC release and separate them in later releases.
In future releases, once TM & WM are separated, each instance of TM can have its own instance of DB and the TaskCallbackEvent can be extended from md.task.callback to md.task.callback.tm_cloud1 where tm_cloud1 is the name of a collection of (one or more) TMs running an environment.

@mocsharp as discussed, please update the description of this ticket to match the current design

Task Update & Task Callback Events
When a task is completed by the app servers (external orchestration services), it needs to notify the Task Manager of its status via the md.tasks.callback event. Task Manager then queries & retrieves details statuses, metadata and output artifacts.
The Task Manager communicates any changes to an accepted task through the md.tasks.update event. For example, when a md.tasks.dispatch is accepted, a md.tasks.update is emitted with status=Accepted.
Task Callbacks - md.tasks.callback
The event is emitted by external orchestration services to notify Task Manager that a task has been completed.
Queue name: md.tasks.callback
Type: TaskCallbackEvent
Event schema:
{
  "correlation_id": "d1ffa435-1582-4aaa-8cab-912d86fb7e38",
  "execution_id": "91a70c2a-4a97-4103-a9a5-c77aed9f4d2a",
  "identity": "md-list-input-artifacts-template-xfl6v",
  "metadata": {},
  "task_id": "caa10114-db70-4655-8b5d-273e16833c03",
  "workflow_id": "65666477-a775-4914-825b-5ecc33d0d6b4"
}

This event includes the identity of the task on the remote orchestration service for the Task Manager and the plug-in to query the status. When the Workflow Manager picks up that event, the following needs to happen:

Validate incoming message
Call the plug-in's GetStatus() method
Emit a md.tasks.update event
Release allocated resource.

Task Update - md.tasks.update
The md.tasks.update is emitted by the Task Manger to notify any changes in the status of the accepted tasks.
Queue name: md.tasks.update
Type: TaskUpdateEvent
Event schema:
{
  "correlation_id": "d1ffa435-1582-4aaa-8cab-912d86fb7e38",
  "execution_id": "91a70c2a-4a97-4103-a9a5-c77aed9f4d2a",
  "message": "Argo status = Running. Messages = .",
  "metadata": {
    "JobIdentity": "md-list-input-artifacts-template-xfl6v"
  },
  "reason": "None",
  "status": "Succeeded",
  "task_id": "caa10114-db70-4655-8b5d-273e16833c03",
  "workflow_id": "65666477-a775-4914-825b-5ecc33d0d6b4"
}

The event includes the status of a task, failure reasons, and an optional failure message.
Metadata generated by the task and any artifacts registered by the task are returned in the event.
The format of the callback event or API call can differ between app servers. We need to define an overall wrapping object with the minimum set of information that we need, for example, the task ID-but otherwise the content of this object will eventually be passed to the plug-in itself, so the main interface needs to be between the plug-in and its app server.
Task statuses

Unknown
Created
Dispatched
Accepted
Succeeded*
Failed*
Canceled*

Tasks can finish in status denoted with *.
Note: Inference operations can have more complex statuses, for example, the task may finish successfully but fail to reach a decision due to a lack of statistical significance. This is still represented as a success for the purposes of the workflow manager. To allow for these more complex nuanced statuses the metadata dictionary may be used by plug-ins to indicate the result in further detail.
Acceptance Criteria

GIVEN an app server finished running the task, WHEN a callback event is published to RabbitMQ THEN it is picked up by the task executer for processing
GIVEN the callback event has been picked up by the task executer, WHEN the task executer completes the task THEN a task update event is published to RabbitMQ
GIVEN the callback event is invalid WHEN an error occurs while processing the callback event THEN the event is rejected with requeue=false
GIVEN a valid callback event WHEN status is retrieved from the plug-in THEN the Task Manager shall include any metadata and output artifacts in the task update event.
GIVEN a valid callback event WHEN status is retrieved from the plug-in THEN the Task Manager shall include any error messages in the task update event.

ROADMAP ESTIMATE: 8 Days
ACTUAL:


Website / Donor Portal About Us: partners section, please add Eclektik Designs logo and text

[ ] Website / Donor Portal About Us: partners section, please add Eclektik Designs logo and text

Creation of the project overview video

Also add Dr. Laura Leets to the Volunteer section please

May I work on this @nshamit

@anjula-sack says please go ahead. @RandilaP please complete this today (Monday 17th Oct)

@nshamit Where can I find the text and the details about Dr.Laura Leets.

@RandilaP you don't need any, Randila, just list her as a volunteer in the volunteer section of the About Us page.

@nshamit okay. And what about the text of Eclektic Designs

@nshamit Don't they have a link?

I searched for it last evening, Randila, when I was creating the issue. They didn't have anything I could use. Shall we just quickly finish this off ?

okay

@RandilaP @anjula-sack
Text for Eclektic designs:
Eclektik Designs is a US based start-up that provides a full suite of design solutions that include online video marketing materials, trade-show graphics, logo designs and professional brochures.


CB-19609 Create a new flow step in the External DB start flow. Create the skeleton of the CERT rotation step in External DB flow.

See detailed description in the commit message.

something looks off to me as the first commit is already merged: https://github.com/hortonworks/cloudbreak/pull/13786

something looks off to me as the first commit is already merged: #13786

fixed


chore: Added Anaconda

Related Issue
Closes #1086

Description

Data science platform that includes tools for Python, R, and distribution management
Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@Karmpatel960, it's a well-known tool. For that reason, we can't have that tool.


Any methods to make attribute declarable in C++ as well?

Right now the only (popular) feature of midl that requires non-trivial annotation is attribute. Commonly used attribute include:

Windows repo's contract
Xaml content
bindable

Is it possible to declare these in C++ as well without needing to write [[idlgen::attribute("bindable",)]]? Note that C++ attribute is not extensible.

Idea
struct based
namespace idlgen
{
template <char const* Property = nullptr>
struct attribute
{
};
}

namespace wux
{
struct bindable_attribute : idlgen::attribute<>
{
};
template<char const* Property>
struct content_property_attribute : idlgen::attribute<Property>
{
};
}

const char property_name[] = "Property";
struct ViewModel : ViewModelT<ViewModel>, bindable_attribute, content_property_attribute<property_name>
{
}

SO says it's possible to use const char* as template parameter. Since we need to support C++17 C++20's std::string-in-template-parameter is not something idlgen can enjoy yet.

It works, but property_name must be defined outside of the class which can leak impl detail/introduce pollution (C++17 doesn't support module yet). It works in C++20 though...


Private/shared kifus

Well, what the title says. This would be a neat feature for teachers/private study groups, don't you think?

I agree. I've thought about at least the private thing before, and deliberately set that aside to focus on what I thought of as the minimum viable product for GoKibitz. Having private kifus that don't show up in public lists shouldn't be too difficult: I already have a public flag in the DB. Now that you've mentioned it, I'll bump that up in priority.

Well, don't work on it if there are more interesting things ;) I just thought this may bump its use-ability by go teachers (a little like the Go Teaching Ladder: post the game and get a notification/ask questions a teacher can answer)

There are a million things to work on. But I launched this rather than keep
it private because I needed some encouragement and incentive (I was getting
overwhelmed with all the features it Ought To Have before it was as good as
I wanted it to be), so if someone asks for something and I think it's a
good idea, that helps me know what to prioritize.
On Mon, Jan 12, 2015 at 12:12 PM, Ruben Berenguel<EMAIL_ADDRESS>wrote:

Well, don't work on it if there are more interesting things ;) I just
thought this may bump its use-ability by go teachers (a little like the Go
Teaching Ladder: post the game and get a notification/ask questions a
teacher can answer)
—
Reply to this email directly or view it on GitHub
https://github.com/neagle/gokibitz/issues/12#issuecomment-69605918.


ios post build script require flutterfire to be available globally

Description
When running flutterfire configure (0.3.0-dev.19), post build scripts are added to ios. Those post build scripts require flutterfire to be installed on the machine building the ios application.
This is a problem because the flutterfire installed locally might not be the same version with which configure was run. Currently, for example, I'm using an hard coded version of the flutterfire cli in this github action so the build process can run https://github.com/cedvdb/action-flutter-build-ios/blob/main/action.yml#L54
I'd propose to use flutterfire_cli installed as a dev_dependency instead and fallback to a global one, so everyone, the CI and all developers can use the same version deterministically.
Related:

https://github.com/invertase/flutterfire_cli/issues/242
https://github.com/invertase/flutterfire_cli/issues/14

Reasoning
It should be deterministic
Additional context and comments
No response

I noticed this issue today as well and came up with a manual workaround for building my iOS app via Codemagic. First, here's the error I got on codemagic:
    /Users/builder/Library/Developer/Xcode/DerivedData/Runner-edaimyiflreloheqntgnhkmwcclv/Build/Intermediates.noindex/ArchiveIntermediates/Runner/IntermediateBuildFilesPath/Runner.build/Release-iphoneos/Runner.build/Script-EAEA39BC9122492C49AD1220.sh: line 5: flutterfire: command not found
    Command PhaseScriptExecution failed with a nonzero exit code

This (flutterfire: command not found) makes sense because unlike my local dev environment, Codemagic never ran dart pub global activate flutterfire_cli 0.3.0-dev.20 --overwrite anywhere.
My workround was to manually add dart pub global activate flutterfire_cli 0.3.0-dev.20 --overwrite to the FlutterFire: flutterfire upload-crashlytics-symbols script that flutterfire configure originally created.

I noticed this issue today as well and came up with a manual workaround for building my iOS app via Codemagic. First, here's the error I got on codemagic:
    /Users/builder/Library/Developer/Xcode/DerivedData/Runner-edaimyiflreloheqntgnhkmwcclv/Build/Intermediates.noindex/ArchiveIntermediates/Runner/IntermediateBuildFilesPath/Runner.build/Release-iphoneos/Runner.build/Script-EAEA39BC9122492C49AD1220.sh: line 5: flutterfire: command not found
    Command PhaseScriptExecution failed with a nonzero exit code

This (flutterfire: command not found) makes sense because unlike my local dev environment, Codemagic never ran dart pub global activate flutterfire_cli 0.3.0-dev.20 --overwrite anywhere.
My workround was to manually add dart pub global activate flutterfire_cli 0.3.0-dev.20 --overwrite to the FlutterFire: flutterfire upload-crashlytics-symbols script that flutterfire configure originally created.

Do you need to install firebase tools too? Or upload-crashlytics-symbols doesn't require them?

Do you need to install firebase tools too? Or upload-crashlytics-symbols doesn't require them?

No, flutterfire_cli is enough. I just added pre-build script (in Codemagic):
#!/bin/bash

dart pub global activate flutterfire_cli

Do you need to install firebase tools too? Or upload-crashlytics-symbols doesn't require them?

No, flutterfire_cli is enough. I just added pre-build script (in Codemagic):
#!/bin/bash

dart pub global activate flutterfire_cli

Works!!!

not stale


Automatically block bot accounts through a setting in my preferences

Describe the solution you'd like
I'd like to be able to prevent bot accounts from following me, with a setting in my preferences, so I don't have to block each of them manually.
Recent examples:
https://bsky.app/profile/funnyordie.bsky.social
https://bsky.app/profile/90spiictures.bsky.social
https://bsky.app/profile/poeticalphotos.bsky.social
https://bsky.app/profile/9gag.bsky.social
and any other account that follows an absurdly high number of accounts.
Which is not the same as an account that has a huge number of followers - that can happen and still be genuine.
Describe alternatives you've considered
Blocking them manually one by one.
Additional context
If it technically isn't easy to distinguish these accounts, I'd also be happy with a number setting, in analogy to "reply must have n likes before I want to see it".
For myself, I'd probably start at accounts which are following more than 5000 people, and lower the number later if necessary.

(related https://github.com/bluesky-social/social-app/issues/1160)

This only was a problem in the first days after opening my account. You can mark it as closed, if you want.

This can be closed in my opinion, has been solved by moderation lists / block lists.


feat: add backup settings to app storage

This is draft from copilot workspace as a starter for #2192
Add a setting option to toggle backup of settings to app storage.
Pending: testing disable of sync shouldn't remove the stored data, so you could retoggle it back on and it should restore based on whatever was last there.
This didn't work yet, but the initial sync seemed to be toggled without error.
Also this is local storage and pretty sure that's a nice backup for local but wouldn't sync remotely, so this is just a start.

For more details, open the Copilot Workspace session.

✨✨ Here's an AI-assisted sketch of how you might approach this issue saved by @sheldonhull using Copilot Workspace v0.25

Closing in favor of #2208


Using properties in

Description
When you have an element that uses a property as a CSS style value, and you use this element two times with a different value, then the value you set on the first element also gets used in the second element in Firefox. In Chrome it works, possibly because they have a real shadow DOM and doesn't use the webcomponents loader.
Live Demo
Demo: https://evanescent-tiglon.glitch.me/
Source: https://glitch.com/edit/#!/evanescent-tiglon?path=index.html
Steps to Reproduce

Create an element that has a property that is used as a CSS style value
Make 2 instances of this element with different values for this property
See the difference of the result in Firefox and Chrome

Expected Results
You would expect the second element to get the style you provide to it, but it gets the same style you give to the first element. It happens only in Firefox, in Chrome it works like expected.
Actual Results
Each element it's rendered according to the properties you set on it.
Browsers Affected

[x] Firefox
[ ] Chrome

Versions

lit-element: v0.6.1
webcomponents: v2.1.3

This is a limitation of the ShadyCSS library that's used to help scope styles when using Shady DOM to simulate real Shadow DOM. The styles for a given element type are specially scoped only one time per element definition. That's why the "first render" wins.
It's also not a particularly good for performance to render dynamic values in styles like this since altering stylesheets is pretty expensive in the browser. Instead we'd recommend using css custom properties for this. Here's an example: https://glitch.com/edit/#!/nice-coke.

Aha, clear, thank you!


Typo in privacy policy

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
github-general-privacy-statement.md
What part(s) of the article would you like to see updated?
See
https://github.com/github/docs/blob/6415c00c11eb55bf199ab319e0ce4568969aea4d/content/site-policy/privacy-policies/github-general-privacy-statement.md?plain=1#L61
I'm pretty sure "this" should be a link to "What are your cookie choices and controls?"
Additional information
These documents are great, thank you for making them available under a permissive license.

@samuelcolvin Thank you for raising this issue! I'll get this triaged for review ✨ Our team will provide feedback regarding the best next steps for this issue - thanks for your patience! 💛

@samuelcolvin 👋 - thanks for spotting that there is a link missing 🦅 👀
I agree with your suggestion for the section to link to.
Would you be happy to make the fix yourself in a pull request, or shall I open this to the OS community and let someone help?

You go for it.

I've added the help wanted label to this issue which means that it's up for grabs ⚡

I would like to fix it!, can you assign it to me


feat: rpc server supports QUIC connections

Description

Blocked by hyper not supporting QUIC/HTTP3 yet.

Maybe something nice to track: https://github.com/hyperium/h3
It's purpose is to be used internally by hyper once it's stable

Yup and it's blocking this too https://github.com/seanmonstar/reqwest/pull/1599.

Yup and it's blocking this too seanmonstar/reqwest#1599.

Another one to track https://github.com/hyperium/hyper/issues/1818.

just for visibility we also have this https://github.com/aws/s2n-quic


fix(ops): Terminate scope of args and temporaries in op2 to avoid borrow errors

Instead of:
let arg0 = ...;
let arg1 = ...;
let result = Self::call(...);

We now do:
let result = {
  let arg0 = ...;
  let arg1 = ...;
  Self::call(...)
}

This ensures that the scope of all args and temporaries ends after the call to Self::call.

Instead of:
let arg0 = ...;
let arg1 = ...;
let result = Self::call(...);

We now do:
let result = {
  let arg0 = ...;
  let arg1 = ...;
  Self::call(...)
}

This ensures that the scope of all args and temporaries ends after the call to Self::call.


Added shebang to build script

This avoids build problems when you're using another shell like zsh.

Hi @TorbenKoehn, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Thanks for the change! Should we use /bin/bash? The script used to have problems with /bin/sh. I think previously we had trouble with the license header checker for this change. Seems that problem is gone. @dilijev

Absolutely. Changed it.

Should we use /bin/bash? The script used to have problems with /bin/sh.

bin/sh forces POSIX. So, the differences. However makes sure the experience due to strict bacward compatibility. On the other hand, for the range of platforms we target, 'bash' makes more sense for us.

LGTM

Thanks @TorbenKoehn, and @obastemur for confirmation. I'll merge this.

@jianchun yes, last time this came up I @obastemur and I fixed the copyright check to support shebangs as the first line of the file before the required copyright header.

@dilijev Thanks for confirm!

For reference, that logic is here: https://github.com/Microsoft/ChakraCore/blob/master/jenkins/check_copyright.py#L46


add --quiet=true for swag init, make the debug logger quiet.

Describe the PR
Add a --quiet=true flag for swag init.
make the debug logger quiet.
Relation issue
https://github.com/swaggo/swag/issues/1193

@jixiuf Thanks for your contribution.


replace legacy setup.py with pyproject.toml

other changes which where required:

move configure_invokeai.py into ldm.invoke

update files which imported configure_invokeai to use new location:

ldm/invoke/CLI.py
scripts/load_models.py
scripts/preload_models.py

update test-invoke-pip.yml:

remove pr type "converted_to_draft"
remove reference to dev/diffusers
remove no more needed requirements from matrix
add pytorch to matrix
install via pip3 install --use-pep517 .
use the created executables
this should also fix configure_invoke not executed in windows

To install use pip install --use-pep517 -e . where -e is optional

not sure if you wanted testing but installs fine on ubuntu 22.04, I used pip install --use-pep517 -e .

not sure if you wanted testing but installs fine on ubuntu 22.04, I used pip install --use-pep517 -e .

Yes, your testing is greatly appreciated!!

pyproject.toml changes look good
can I ask to please not modify configure_invokeai.py and related changes from your first point in the description, because I'm already making similar changes (and more) to all of that in #2292? Is it possible to keep the changes here limited to pyproject.toml and the CI workflows?

pyproject.toml changes look good
can I ask to please not modify configure_invokeai.py and related changes from your first point in the description, because I'm already making similar changes (and more) to all of that in #2292? Is it possible to keep the changes here limited to pyproject.toml and the CI workflows?

Since I do not want to include the scripts folder in the package I need to move configure_invokeai.py, but that's all I do with it, I do not change anything in it.
After looking at your PR, I could mimic the file structure and put it to ldm/invoke/config/configure_invokeai.py if that would help you?


AddBodyParameter sometimes doesn't work on first try

I have a spreadsheet which sends a POST request that contains body parameters. I am using Fiddler to analyze the traffic sent by VBA-Web. Occasionally, my requests are being sent without the body parameters. When this happens, the request is just sent with an empty body and Content-Length is 0. It happens almost every time I first open the spreadsheet. It always works if I try again a second time. It is very frustrating because it is intermittent. It is as if the Request.AddBodyParameter lines in my code are being ignored. I know those lines are running though, because if I set a breakpoint, on them, it does stop. Why could this be happening?
I am using 64-bit Excel 2013 on Windows 8.1.

Interesting, I have been getting this occasionally with the test suite, but I figured (incorrectly) that it was a problem with httpbin.org. I'll dig in and see what I can find.

Thanks. It's reassuring to hear that you have also encountered this bug.

Hi Tim, I was just wondering if you've made any progress on this. It is a frustrating issue that I deal with every day, and even a temporary solution (commenting out a line of code, etc) would be very helpful. Thanks.

@bdr99 I haven't been able to reproduce it consistently to really debug what is happening. If you're getting it everyday, I could use some help debugging:

Set WebHelpers.EnableLogging = True before the request is executed and then check if the body is being sent in the logs in the Immediate Window ctrl + g
At the end of WebRequest.AddBodyParameter (~689), check the current Body value with Debug.Print Me.Body
In WebClient.Execute (~275), check Request.Body before Send with Debug.Print Request.Body

Sure, I'd be happy to help. I'll try these later when I have some time. Thanks.

While I was trying to do this debugging, I ran into another problem. I started with a blank copy of VBA-Web - Blank.xlsm and added the following code to a new sub:
Dim Client As New WebClient, request As New WebRequest, response As WebResponse

Client.BaseUrl = "http://httpbin.org"
Client.EnableAutoProxy = True

request.Resource = "post"
request.Method = HttpPost
request.AddBodyParameter "param1", "test1"
request.AddBodyParameter "param2", "test2"
request.AddBodyParameter "param3", "test3"

Set response = Client.Execute(request)

This code works fine, sending the following request body:
{"param1":"test1","param2":"test2","param3":"test3"}

But if I add Debug.Print Me.Body to the end of WebRequest.AddBodyParameter, it only sends the first parameter:
{"param1":"test1"}

This made no sense at first. But then I looked at line 451 in WebRequest (Public Property Get Body() As Variant). This converts the body to a string and saves it in web_pConvertedBody. If this string already exists, however, it will use the already stored value from web_pConvertedBody. Therefore, subsequent calls to the Body of the WebRequest will always return the same value, even if additional body parameters are added. It seems like it should always generate a new value every time the Get Body() is called. I'm not even sure why the value needs to be stored in web_pConvertedBody.
I'm not sure if this problem is related to or is the cause of my original problem, but fixing it would certainly make it easier to debug that problem.

That is definitely a bug and I'll fix it shortly. I too am curious if that is the root cause, it certainly could lead to some unexpected behavior.

Thanks. Once this bug is fixed, I will try to help you debug the original issue I was dealing with, if it still exists.

Ok, it's been fixed in v4.0.14

I'm actually running into a new problem now. Somehow my WebRequest bodies are becoming malformed. I added Debug.Print "body=""" & Body & """" to the end of the Public Property Get Body() As Variant line in WebRequest at line 463. This is what I'm getting:

At first I didn't even realize that this was a separate problem because it causes my code to fail in the same way that the original problem did. I can't find any pattern as to when this one happens and when it doesn't happen. Any help would be appreciated.

Wow, that's very strange. What Format/RequestFormat/custom formatter are you using? (the default is WebFormat.Json)

I am using the FormUrlEncoded format with three body parameters.

Are you able to send me the parameters/Workbook or a portion of them so that I can try to reproduce locally? (my email is in the code headers) My best guess is some implicit conversion issues between Variant and String, but I would need to see more to fully diagnosis the bug.

Ok, I just sent you an email. Please let me know if I can help in any other way.


Load climate and terrain data for provinces.

Adds methods for reading province climate and terrain, making this information available for other features. I intend to use this later to enable randomising RGOs in terrain/climate buckets, for users who want less-historical setups. I will create a utility to generate the terrainData file from the Victoria bitmap in a future PR.

Sorry, I messed that this was ready for review.
Just one nit: I'd prefer the new parser to be used where possible. Though the fact that you didn't need to add the header shows this file hadn't been converted yet, and this is a use case where setting up a helper class might be a pain, so no big deal.


Should we call parent::decode() or $this->decode() in GuidStringCodec?

Scrutinizer turned up this issue for line 54 of GuidStringCodec.
Inside decodeBytes(), you call parent::decode(), and Scrutinizer wonders if maybe you meant to call $this->decode() instead.
At a glance, I wasn't sure whether this was intentional, so I decided to open an issue about it.

@aztech-dev

I need to check, but I think it's intentional...

I did my homework, I wasn't sure anymore as it's been a while since I wrote the GUID implementation, but basically it boils down to preserving the correct byte order on the first three fields.
I guess you'll have to ignore the issue in Scrutinizer :) I'll open up a P/R over the weekend to add a comment specifying it's intentional and improve the implementation (to handle properly encoding/decoding GUID's based on the endianness of the host).

Great. Thanks!


On duckdb engine, some DataFrame functions fail after joins if next functions use the key/id column without referencing df source

Hi,
I encountered following errors related to some functions, like .select() and .withColumn(s) if you use the key name, after a join, without referencing the df that it came from. Here is some code to test:
from sqlframe.duckdb import DuckDBDataFrame as DataFrame
from sqlframe.duckdb import DuckDBSession

spark = DuckDBSession()

initial: DataFrame = spark.createDataFrame(
    [
        ("some_id"),
        ("another_id"),
    ],
    ["example_key"],
)

for_join: DataFrame = spark.createDataFrame(
    [
        ("some_id", "some_id"),
        ("another_id", "slightly_different_id"),
    ],
    ["example_key", "new_key"],
)

joined = initial.join(for_join, on="example_key", how="left").select(
    F.col("example_key").alias("just_renamed_key")
)

or

joined = initial.join(for_join, on="example_key", how="left").withColumn(
    "just_renamed_key", F.col("example_key")
)

This fails with:
python sqlframe_playground.py
Traceback (most recent call last):
  File "/workspaces/redacted/sqlframe_playground.py", line 24, in <module>
    joined = initial.join(for_join, on="example_key", how="left").select(
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/operations.py", line 48, in wrapper
    df: t.Union[_BaseDataFrame, _BaseGroupedData] = func(self, *args, **kwargs)
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/dataframe.py", line 679, in select
    cte = ctes_with_column[resolved_column_position[ambiguous_col]]
IndexError: list index out of range

Also, there is another error, when trying a function on the key column, but only after calling .show()
joined = initial.join(for_join, on="example_key", how="left").select(
    F.coalesce("example_key", "new_key").alias("example_key")
)
joined.show()

This fails with:
python sqlframe_playground.py
Traceback (most recent call last):
  File "/workspaces/redacted/sqlframe_playground.py", line 39, in <module>
    joined.show()
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/dataframe.py", line 1512, in show
    result = self.session._fetch_rows(sql)
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/session.py", line 455, in _fetch_rows
    self._execute(sql, quote_identifiers=quote_identifiers)
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/session.py", line 427, in _execute
    self._cur.execute(self._to_sql(sql, quote_identifiers=quote_identifiers))
duckdb.duckdb.BinderException: Binder Error: Ambiguous reference to column name "example_key" (use: "t24194196.example_key" or "t26006366.example_key")

Looks like both are due to ambiguous reference, which would happen in sql as you need to point to the id, supported by the fact that replacing example_key with initial.example_key and new_key with for_join.new_key works in both cases.

joined = initial.join(for_join, on=["example_key"], how="left").select(
    F.coalesce(for_join.new_key, initial.example_key).alias("example_key")
)

joined.show()

works as expected
Although implicitly mentioning dataframes would solve the issue, there are a lot of cases where just having the same key on both sides simplifies readability, especially when dataframes names can get quite long.

Thanks for reporting this! Fixed in 1.6.1

@eakmanrq , indeed now it works for alias, but using it in an expression will still throw an error
from sqlframe.duckdb import DuckDBDataFrame as DataFrame
from sqlframe.duckdb import DuckDBSession
from sqlframe.duckdb import functions as F

spark = DuckDBSession()

initial: DataFrame = spark.createDataFrame(
    [
        ("some_id"),
        ("another_id"),
    ],
    ["example_key"],
)

for_join: DataFrame = spark.createDataFrame(
    [
        ("some_id", "some_id"),
        ("another_id", "slightly_different_id"),
    ],
    ["example_key", "new_key"],
)

joined = initial.join(for_join, on="example_key", how="left").select(
    F.coalesce("example_key", "new_key").alias("final_key")  # here on the coalesce
)
joined.show()
Traceback (most recent call last):
  File "/workspaces/redacted/frame.py", line 27, in <module>
    joined.show()
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/dataframe.py", line 1527, in show
    result = self.session._fetch_rows(sql)
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/session.py", line 455, in _fetch_rows
    self._execute(sql, quote_identifiers=quote_identifiers)
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/session.py", line 427, in _execute
    self._cur.execute(self._to_sql(sql, quote_identifiers=quote_identifiers))
duckdb.duckdb.BinderException: Binder Error: Ambiguous reference to column name "example_key" (use: "t24194196.example_key" or "t26006366.example_key")

or any function for that matter:
F.size("example_key").alias("final_key")

Btw, changing from
joined.show() to joined.sql() hits sqlglot OptimizerError, instead of DuckDB one:
Traceback (most recent call last):
  File "/workspaces/redacted/frame.py", line 27, in <module>
    joined.sql()
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/dataframe.py", line 544, in sql
    exp.Select, self.session._optimize(select_expression, dialect=dialect)
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlframe/base/session.py", line 422, in _optimize
    return optimize(expression, dialect=dialect, schema=self.catalog._schema)
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlglot/optimizer/optimizer.py", line 92, in optimize
    optimized = rule(optimized, **rule_kwargs)
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlglot/optimizer/qualify.py", line 90, in qualify
    validate_qualify_columns_func(expression)
  File "/workspaces/redacted/.venv/lib/python3.10/site-packages/sqlglot/optimizer/qualify_columns.py", line 102, in validate_qualify_columns
    raise OptimizeError(f"Column '{column}' could not be resolved{for_table}")
sqlglot.errors.OptimizeError: Column '"example_key"' could not be resolved

Doing some debugging on the size example, generated sql would name correctly on selection, but not on functions
F.size
"WITH t26006366 AS
(SELECT CAST(example_key AS TEXT) AS example_key FROM (VALUES ('some_id'), ('another_id')) AS a1(example_key)),
t24194196 AS (SELECT CAST(example_key AS TEXT) AS example_key, CAST(new_key AS TEXT) AS new_key FROM (VALUES ('some_id', 'some_id'), ('another_id', 'slightly_different_id')) AS a2(example_key, new_key)),
t37939134 AS (SELECT ARRAY_LENGTH(example_key) AS final_key FROM t26006366 LEFT JOIN t24194196 ON t26006366.example_key = t24194196.example_key)
SELECT final_key FROM t37939134 LIMIT 20"
vs
F.col
"WITH t26006366 AS (SELECT CAST(example_key AS TEXT) AS example_key FROM (VALUES ('some_id'), ('another_id')) AS a1(example_key)),
t24194196 AS (SELECT CAST(example_key AS TEXT) AS example_key, CAST(new_key AS TEXT) AS new_key FROM (VALUES ('some_id', 'some_id'), ('another_id', 'slightly_different_id')) AS a2(example_key, new_key)),
t36021190 AS (SELECT t26006366.example_key AS final_key FROM t26006366 LEFT JOIN t24194196 ON t26006366.example_key = t24194196.example_key) SELECT final_key FROM t36021190 LIMIT 20"

Yeah good catch. I was a bit too strict in my search for ambiguous columns and therefore the fix was taking whatever expression I find and search within that for any ambiguous columns and resolve that. Thanks for reporting!

Fix in 1.6.3


[mongo]: Hashes are not computed

I created a brand new little nodejs project and installed prisma + this package.
After creating a document in my MongoDB Atlas (normal mongodb+srv connection), Hashes are not computed...

is this a bug?
Because as I know: Mongo is not saving nullable fields into the db. But I don't know why prisma is not computing the hashes...

Notice: I've added the @default("") fields to try to fix this, but it also didn't work...

Do you have a reproduction repository?

Do you have a reproduction repository?

If you give me your gitlab.com username or email I can add you to the project. Just write me via<EMAIL_ADDRESS>

I think I found the issue: the annotation for hash fields should be /// @encryption:hash(fieldName), not /// @encrypted:hash(fieldName).

I think I found the issue: the annotation for hash fields should be /// @encryption:hash(fieldName), not /// @encrypted:hash(fieldName).

you're totally right 🙈.
Thank you for your finding ❤.


Wont let me try to install

running it as admin, outside of downloads, and it wont let me install it

Yes, i even tried installing the other 2 versions too

Click the checkmark next to "Install to Stable"

aaaaaaahhhhhhhh
was hard to see the check box


Improve PatientSearchInput.test.tsx

Fixes #234

Not sure why these are failing and passing, we can figure it out.

Not sure why these are failing and passing, we can figure it out.


Unable to copy integer values t.value.trim is not a function

Preflight

[x] I have searched the repo for this duplicates

Description
Expect Result
Copy value to the clippboard
Steps to reproduce

use an int instead of a string for the field
click the copy button

Logs, error output, etc
Uncaught TypeError: t.value.trim is not a function

Version information
nove-text-copy-field 1.5
Laravel Nova  v3.8.4

If you use a computed field and cast the int to a string before it works just fine.

in your model:
protected $casts = ['your int variable' => 'string'];

Thanks.


Text Random Line broken after update

More precisely:
When I drag an existing image into ComfyUi and press generate, all the "Text Random Line" nodes are circled red and don't work.
I am including an example.
The way I work is, I have generated a huge bunch of images at low res and manually selected the good ones. I was planning to reload them, switch on the high.res part of the workflow and  requeue them.
Now it looks like I can just throw them away.
Or is there any way to reinstall the old version of that node?
In the future, please, please don't change existing nodes in a non-compatible way.

I haven't updated them in a long time, so not sure what the issue is. Most the time if there is changes you need to remake the nodes.


gccgo: internal compiler error: in struct_field_expression

The following program (reduced from a gosmith-generated one)
package p

var F func() [0]struct{
	A int
}

var i int
var V = (F()[i]).A

crashes gccgo (GCC) 8.0.1 20180205 (experimental) with the following error:
go1: internal compiler error: in struct_field_expression, at go/go-gcc.cc:1539
0x6e164b Gcc_backend::struct_field_expression(Bexpression*, unsigned long, Location)
	../../gcc-trunk/gcc/go/go-gcc.cc:1539
0x6b8a81 Field_reference_expression::do_get_backend(Translate_context*)
	../../gcc-trunk/gcc/go/gofrontend/expressions.cc:11778
0x6c839b Type_conversion_expression::do_get_backend(Translate_context*)
	../../gcc-trunk/gcc/go/gofrontend/expressions.cc:3542
0x6ef008 Variable::get_init(Gogo*, Named_object*)
	../../gcc-trunk/gcc/go/gofrontend/gogo.cc:6670
0x703979 Gogo::write_globals()
	../../gcc-trunk/gcc/go/gofrontend/gogo.cc:1377

This gcc trunk build includes the fix for #22305, which no longer causes a crash; so I believe this is not a dup of #22305.

(Still reproducible in gccgo 11.1)

I just committed a patch to fix this on GCC tip.  This was a bug in the gofrontend's GCC interface.  Thanks for reporting the problem.


Feature request: Container Checkpoint Support

I would like to suggest adding support for container checkpoint functionality to Falco Talon.
This feature allows saving the state of running containers, which could enhance the capability to perform forensics and investigation on the compromized containers/pods.
This feature has been merged into Kubernetes v1.25.

https://kubernetes.io/blog/2022/12/05/forensic-container-checkpointing-alpha/
https://kubernetes.io/docs/reference/node/kubelet-checkpoint-api/

Supports:

Containerd ( See https://criu.org/Containerd)
CRI-O ( with flag --enable-criu-support=true )

I did some searches, you can correct me if I'm wrong, but here's the results:

the feature has a lot of requirements:

the container runtime must have the feature and be started with it enabled
the feature in k8s is behind the feature gate, it requires the kubelet to be started with a specific option
criu must be installed on all nodes

moreover, the checkpoints are stored under /var/lib/kubelet/checkpoints, it means, we have to follow this procedure:

trigger the checkpoint with talon
wait til the completion
start a pod on the exact same node with /var/lib/kubelet/checkpoints mounted from the host
use that pod to get the checkpoint and push it to an available output (s3 or minio) for now

Even if it's technically doable, I would prefer to wait this feature to be GA and avoid to spend to much time on it today, and see it totally removed or modified in the future.
wdyt?
cc @IgorEulalio @xinity

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten


Beautiful line break

How to make a beautiful line break? Now the words are spelled, I'm trying to do it according to the words. Example
Before:

After:

Try setDefaultStyle("word-beak: unset;")

Tried to

Hi @AkioSarkiz
Try it
suneditor.create('..', {
    defaultStyle: 'word-wrap: normal;'
})

@JiHong88
Tried to

@AkioSarkiz

Did you copy and paste words with spaces?
If so, the space is copied by " "
ex) wording\ wording\ wording\ ...
The above text are recognized as one word.
It is separated by a space and does not wrap.
Test it by typing it yourself.

No, I didn't copy the text. Here's a video of me typing and the words aren't transferred to the new line.
https://youtu.be/nIDS8PnA-_o
I also turned off app.css to check, but that didn't help either.

Hi @AkioSarkiz
Sorry for the late reply.
Modify css like below:
word-break: normal;
word-wrap: break-word;

Thanks!


Maintenance

Who is maintaining this repo?

Homebrew/Homebrew's maintainers. It has fallen behind the list of priorities a little, due to the sheer amount of work it takes to look after older versions of software, but they'll get around to it soon I'm sure.

If they have a lot to do maybe they can accept new maintainers? It should be possible to take in some of the pull requests that have been there for some time https://github.com/Homebrew/homebrew-versions/pulls?q=is%3Aopen+is%3Apr

There are 6 core maintainers, and two who tend to deal with Versions PRs, Jack and Mike. I'm sure they've considered whether it's worthwhile snagging someone to handle the Homebrew/Versions repo, but it's kind of a complicated repo to handle, there's a lot of old, outdated code that needs modernising, and that takes considerable time and effort. I appreciate PRs have been standing for a while.

I didn't really understand your last comment "I appreciate PRs have been standing for a while."
I understand that there are complicated parts of this repo but it should be possible to pull in new "old" versions like https://github.com/Homebrew/homebrew-versions/pull/642 introduces.

I didn't really understand your last comment "I appreciate PRs have been standing for a while."

As in "I know PRs have been standing for a while, and obviously that's not ideal for people, but please bear with the maintainers who have their hands full and are aware of the situation here". You can @ Mike or Jack into the conversation if you wish for their direct perspective, I don't speak for them, nor can I read their minds, I just wanted to highlight the amount of work it takes to maintain this particular repo and how that can potentially have delaying effects. It isn't a dead repo.

I managed to fix my problem with the brew edit command. Mike and Jack are probably doing a really good job. I just didn't see anyone working on the pull requests and could not find anything useful for that in the readme. If they really have a lot to do then maybe they should consider taking in more people to help. Anyway, thanks for the chat @DomT4 =)

Yeah, you can make local changes. You can also do things like brew pull https://github.com/Homebrew/homebrew-versions/pull/657 which patches the formula for you as if the PR had been merged.
Apologies if my tone was abrasive. Blame my being British and our national collective grumpiness :wink:.

I didn't find it abrasive. Just informative =)