SetRecursiveAcl refinement.

Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[x] C++ Guidelines
[x] Doxygen docs
[x] Unit tests
[x] No unwanted commits/changes
[x] Descriptive title/description

[x] PR is single purpose
[x] Related issue listed

[x] Comments in source
[x] No typos
[x] Update changelog
[x] Not work-in-progress
[x] External references or docs updated
[x] Self review of PR done
[x] Any breaking changes?

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

In .Net, PathClient has {Set,Update,Remove}AccessControlRecursive and SetAccessControlList. DirectoryClient inherits these functions and override SetAccessControlList
what's the consideration behind this unalignment?

In .Net, PathClient has {Set,Update,Remove}AccessControlRecursive and SetAccessControlList. DirectoryClient inherits these functions and override SetAccessControlList
what's the consideration behind this unalignment?

In .Net, PathClient has {Set,Update,Remove}AccessControlRecursive and SetAccessControlList. DirectoryClient inherits these functions and override SetAccessControlList
what's the consideration behind this unalignment?

What does it mean if a file resource is initialized as a path client and AccessControlRecursive related API is called? It does not make sense.
We can always move it into the path client in the future as a non-breaking change.

In .Net, PathClient has {Set,Update,Remove}AccessControlRecursive and SetAccessControlList. DirectoryClient inherits these functions and override SetAccessControlList
what's the consideration behind this unalignment?

What does it mean if a file resource is initialized as a path client and AccessControlRecursive related API is called? It does not make sense.
We can always move it into the path client in the future as a non-breaking change.

What does it mean if a file resource is initialized as a path client and AccessControlRecursive related API is called? It does not make sense.

What does it mean if a file resource is initialized as a path client and Delete with recursive = true is called?

What does it mean if a file resource is initialized as a path client and AccessControlRecursive related API is called? It does not make sense.

What does it mean if a file resource is initialized as a path client and Delete with recursive = true is called?

What does it mean if a file resource is initialized as a path client and AccessControlRecursive related API is called? It does not make sense.

What does it mean if a file resource is initialized as a path client and Delete with recursive = true is called?

That's different, a meaningless combination of the parameters and a meaningless API itself are different levels of exposure to the customer.
If you insist, I can delete the delete methods in the path client.

What does it mean if a file resource is initialized as a path client and AccessControlRecursive related API is called? It does not make sense.

What does it mean if a file resource is initialized as a path client and Delete with recursive = true is called?

That's different, a meaningless combination of the parameters and a meaningless API itself are different levels of exposure to the customer.
If you insist, I can delete the delete methods in the path client.

/azp run cpp - storage

/azp run cpp - storage

Force push to resolve the merge issue caused by github.

Force push to resolve the merge issue caused by github.


Multiple KeyResolver beans allowed?

I'm trying to build multiple KeyResolvers. Each of them is for different route. But this results in a runtime error (see below). Is it a correct approach to mark one of them @Primary?
***************************
APPLICATION FAILED TO START
***************************

Description:

Parameter 1 of method requestRateLimiterGatewayFilterFactory in org.springframework.cloud.gateway.config.GatewayAutoConfiguration required a single bean, but 3 were found:
	- ipKeyResolver: defined by method 'ipKeyResolver' in class path resource [com/RateLimitingConfig.class]
	- clientKeyResolver: defined by method 'clientKeyResolver' in class path resource [com/RateLimitingConfig.class]
	- customerKeyResolver: defined by method 'customerKeyResolver' in class path resource [com/RateLimitingConfig.class]

Action:

Consider marking one of the beans as @Primary, updating the consumer to accept multiple beans, or using @Qualifier to identify the bean that should be consumed

There needs to be a default one. Is there one?

There needs to be a default one. Is there one?

Hi @spencergibb , how to make one of them default? Put a "@Primary" on top of one of them?

Yes

Yes

That's what I'm doing right now. Thanks for your confirmation. Closing this issue now.


Add Gluon (fka UnUniFi)

UnUniFi has been rebranded to Gluon.
UnUniFi will remain as is, and will be updated this in the future.

Looking at the version history, we can see this is an update of ununify, so I think we should not create anew gluon directory, but just update the branding for ununify.

@JeremyParish69  Currently, there is a mainnet for UnUniFi and a testnet for Gluon.
We would therefore like to register both; UnUniFi will be removed later.

@JeremyParish69 Currently, there is a mainnet for UnUniFi and a testnet for Gluon. We would therefore like to register both; UnUniFi will be removed later.

Is gluon a testnet for ununify?

Is there still interest in proceeding with this @Senna46 ?


ISSUE-160

Resolve #160

coverage: 100.0%. remained the same when pulling 69913e76b64926fe715cdcd1fc6f30fcdaf5e3fc on ISSUE-160 into b3a34891b4f2d13b3e28832049344ece3f3b2bcf on master.


Consider merging automatically PRs with only a version bump

Most rosdistro PRs are releases of a new version of an already released repo. It would save us a significant amount of time if the PRs that have only a 1 line diff that consists in a version bump were merged automatically

I'm not sure how much time it actually costs us. I typically spend <10 seconds on those PRs and we only get less than 10 releases a day. And if we were to automate it we'd have to monitor it and have ways to disable it when we're in a sync period etc. With the extra overhead of managing another webhook and service and generally higher complexity would negate the time savings.

I don't mind one way or the other. I used to merge a lot of these, but lately I've been getting beat to them or I hesitate because I couldn't remember if we're doing a sync or not.
I tend to agree that they don't take up much time, but since I'm not the one merging most of them lately, I'd also be fine with automating the process.

Closing due to age and inactivity.


Fix gh-pages

Now workflow broken on master; dylanowen/mdbook-graphviz#20 is the likely cause.
Same error on local machine on stable with cargo install mdbook-graphviz --force.

The messed up the pulldown-cmark update, had similar fallout.


Components: Introduce a component for heading/copy, etc.

The problem
Most of the components with heading contents in Web Components codebase has a corresponding component for heading contents, for the styling purpose. OTOH <dds-link-list> doesn't.
Also, <dds-image-with-caption> supports heading attribute but it doesn't have a corresponding <slot>.
The solution
Consider introducing a similar one for <dds-link-list>. Also scan the codebase to see if we need any more components for heading purpose, for other areas.
Additional information
Similar thing is seen with <dds-card>, etc.

@asudoh Can this issue be closed?  I looks like all of the linked PRs are merged.

@asudoh Can this issue be closed?  I looks like all of the linked PRs are merged.

@RobertaJHahn Thank you for checking - Would like to keep this open if possible. Further code inspection is needed to ensure we have finished everything.

@RobertaJHahn Thank you for checking - Would like to keep this open if possible. Further code inspection is needed to ensure we have finished everything.

@RobertaJHahn This work is complete now. Thanks again for checking!

@RobertaJHahn This work is complete now. Thanks again for checking!


The issue you are having is environment related. You'll need to figure out a way to synchronize the time between the server instances.

          > @loren-coding The issue you are having is environment related. You'll need to figure out a way to synchronize the time between the server instances.

thanks for the reply.  OAuth2Authorization.isBeforeUse()   the  Instant.now.isBefore(Instant otherInstant)  uses nanosecond comparisons, and the difference is small:
public boolean isBeforeUse() {
	Instant notBefore = null;
	if (!CollectionUtils.isEmpty(getClaims())) {
		notBefore = (Instant) getClaims().get("nbf");
	}
	return notBefore != null && Instant.now().isBefore(notBefore);
}

Maybe should add clockSkew like  org.springframework.security.oauth2.jwt.JwtTimestampValidator
Originally posted by @loren-coding in https://github.com/spring-projects/spring-authorization-server/issues/1346#issuecomment-1793361564

Closing this and will respond to original comment


reviewer-guide: Update notes on use of informers

/cc @rmohr
What this PR does / why we need it:
Based on the following ML thread:
https://groups.google.com/g/kubevirt-dev/c/q_hR1tFH4Rk
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm
/approve


Mobile Suit Gundam Extreme Versus Force [PCSB00983]

App summary

App name: MOBILE SUIT GUNDAM EXTREME VS-FORCE
App serial: PCSB00983
App version: 1.00

Vita3K summary

Version: v01.8
Build number: 3222
Commit hash: https://github.com/vita3k/vita3k/commit/[commit]
CPU Dynamic
GPU Vulkan

Test environment summary

Tested by: Sol
OS: Windows 10/
CPU: Intel?
GPU: NVIDIA 960m
RAM: 16GB

Issues
Goes in game but screen is warped with UI and in game graphics bleeding into each other
Screenshots

Log
Recommended labels

A Graphics Bug
B In game
C?

Goes in game but when I play game I'm met with this graphics issue. youtube shows that it worked on previous versions.   https://youtu.be/YPG7Ky0HOWI

report is incomplete, you have forget log
and write issue number, for open issue juste using option inside emu, right click on game, andcreate state report

Where do I get the Log file ?
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows
From: @.>
Sent: 22 June 2023 20:11
To: @.>
Cc: @.>; @.>
Subject: Re: [Vita3K/compatibility] Mobile Suit Gundam Extreme Versus Force [PCSB00983] (Issue #1689)
report is incomplete, you have forget log
and write issue number, for open issue juste using option inside emu, right click on game, andcreate state report
—
Reply to this email directly, view it on GitHubhttps://github.com/Vita3K/compatibility/issues/1689#issuecomment-1603184487, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJ67C7CZINIHZFZPT3PQOA3XMSKGRANCNFSM6AAAAAAZQK6TCE.
You are receiving this because you authored the thread.Message ID: @.***>

Ok managed to locate the log file realized it generates as a text file when the game is open in App location
vita3k.log


Bad Gateway

{"level":50,"time":1606398944498,"pid":1,"hostname":"5105fb8c4ead","name":"IndexerAgent","component":"Network","indexer":"0x14c7dB0Bf796060DA7212C0F851Ce62A47805502","operator":"0x14c7dB0Bf796060DA7212C0F851Ce62A47805502","err":{"type":"IndexerError","message":"Failed to query indexer allocations","stack":"IndexerError: Failed to query indexer allocations\n    at Object.indexerError (/opt/indexer/packages/indexer-common/dist/errors.js:91:12)\n    at Network.<anonymous> (/opt/indexer/packages/indexer-agent/dist/network.js:311:46)\n    at Generator.next (<anonymous>)\n    at fulfilled (/opt/indexer/packages/indexer-agent/dist/network.js:24:58)\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (internal/process/task_queues.js:97:5)","code":"IE010","explanation":"https://github.com/graphprotocol/indexer/blob/master/docs/errors.md#ie010","cause":{"type":"b","message":"[Network] Bad Gateway","name":"CombinedError","graphQLErrors":[],"networkError":{"type":"Error","message":"Bad Gateway","stack":"Error: Bad Gateway\n    at /opt/indexer/node_modules/@urql/core/dist/e624b925.js:150:37\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (internal/process/task_queues.js:97:5)"},"response":{"size":0,"timeout":0}}},"msg":"Failed to query indexer allocations"} {"level":40,"time":1606398944498,"pid":1,"hostname":"5105fb8c4ead","name":"IndexerAgent","err":{"type":"IndexerError","message":"Failed to synchronize with network","stack":"IndexerError: Failed to synchronize with network\n    at Object.indexerError (/opt/indexer/packages/indexer-common/dist/errors.js:91:12)\n    at Agent.<anonymous> (/opt/indexer/packages/indexer-agent/dist/agent.js:120:47)\n    at Generator.throw (<anonymous>)\n    at rejected (/opt/indexer/packages/indexer-agent/dist/agent.js:25:65)\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (internal/process/task_queues.js:97:5)","code":"IE004","explanation":"https://github.com/graphprotocol/indexer/blob/master/docs/errors.md#ie004","cause":{"type":"IndexerError","message":"Failed to query indexer allocations","stack":"IndexerError: Failed to query indexer allocations\n    at Object.indexerError (/opt/indexer/packages/indexer-common/dist/errors.js:91:12)\n    at Network.<anonymous> (/opt/indexer/packages/indexer-agent/dist/network.js:311:46)\n    at Generator.next (<anonymous>)\n    at fulfilled (/opt/indexer/packages/indexer-agent/dist/network.js:24:58)\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (internal/process/task_queues.js:97:5)","code":"IE010","explanation":"https://github.com/graphprotocol/indexer/blob/master/docs/errors.md#ie010","cause":{"type":"b","message":"[Network] Bad Gateway","name":"CombinedError","graphQLErrors":[],"networkError":{"type":"Error","message":"Bad Gateway","stack":"Error: Bad Gateway\n    at /opt/indexer/node_modules/@urql/core/dist/e624b925.js:150:37\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (internal/process/task_queues.js:97:5)"},"response":{"size":0,"timeout":0}}}},"msg":"Failed to synchronize with network"}

This error message represents a transient error on the gateway side and is not something to be worried about generally.


Add more test cases

following up from #145

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

Project coverage is 59.37%. Comparing base (1f6b63c) to head (3048156).

:exclamation:  There is a different number of reports uploaded between BASE (1f6b63c) and HEAD (3048156). Click for more details.
HEAD has 137 uploads less than BASE

Flag
BASE (1f6b63c)
HEAD (3048156)

139
2

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #147       +/-   ##
===========================================
- Coverage   76.21%   59.37%   -16.84%
===========================================
  Files         130      130
  Lines       33911    33886       -25
===========================================
- Hits        25844    20121     -5723
- Misses       8067    13765     +5698

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


feat: [PIPE-16896]: Add an use case for  matrix

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66224921ce8c96bb6a723067--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66224a4b64af1ebf178dcd52--harness-developer.netlify.app


GitLab Mattermost Guide

Review and revision of information at https://docs.gitlab.com/omnibus/gitlab-mattermost/
Consider:

Order of information presented
Accessing information from one topic to the next in a sequenced flow
Improved information retrieval from search and browsing

The GitLab Omnibus product documentation is being reviewed as part of January 2022 Growth Spike initiatives. Closing.


Set-CsOnlineVoiceApplicationInstance: command not found

I am connected to Teams and can execute get-team, but
PS C:\WINDOWS\system32> Set-CsOnlineVoiceApplicationInstance
Set-CsOnlineVoiceApplicationInstance : The term 'Set-CsOnlineVoiceApplicationInstance' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the
spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1

Set-CsOnlineVoiceApplicationInstance

  + CategoryInfo          : ObjectNotFound: (Set-CsOnlineVoiceApplicationInstance:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

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

ID: 81fa6268-92b2-546f-9200-f323bb48e90e
Version Independent ID: 6ce89bc0-ae69-b6f1-df50-532e46b2e67b
Content: Manage resource accounts in Teams - Microsoft Teams
Content Source: Teams/manage-resource-accounts.md
Service: msteams
GitHub Login: @dstrome
Microsoft Alias: dstrome

You can mark this as closed now. The cause was that voice functions are still in the Skype module not in the Teams module.
To configure a call queue you need to:

Create a Resource Account of Call Queue type
Create a service number
Purchase a Phone System - Virtual User license in Purchase service ->bottom of the page -> addons
Assign the license to the Resource Account
Set the location of the service account in Active Users -> Licenses and apps  to the same country as the service number
Assign Phone System licenses to the users of the agents
Create a call queue and add the agents to it
Associate the call queue and associate it with the resource account
Assign the number to the resource account
If you don't want to assign calling plans to the agents then enable enterprise voice for them
Connect to the Skype for Business Online shell:
$sfbSession = New-CsOnlineSession
Import-PSSession $sfbSession
Set-CsUser -identity<EMAIL_ADDRESS>-EnterpriseVoiceEnabled $true
remove-PSSession $sfbSession

Br,
Kris

@krisztian-andre thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Inconsistent library names for static and shared libraries on Linux & OS X

You can ready about some of the problems I was having over here for the full story:
https://github.com/raysan5/raylib/issues/332
But the tl;dr is that if -DBUILD_SHARED_LIBRARIES=ON I get myself a libglfw.so file.  But if I compile and install the static version, I'm getting libglfw3.a.  That inconsistant 3 is causing me some linking problems across two machines where I have GLFW installed from source, but one was with static libraries and the other using dynamic.
This is a bit of the pain in the but for me.
I was also using 3.2.1 from the .zip on the GLFW website.
EDIT:
After checking the other OS's, I think that 3 needs to be added there for Linux's shared libraries.

This is sorta documented in http://www.glfw.org/docs/latest/moving_guide.html

The GLFW 3 header is named glfw3.h and moved to the GLFW directory, to avoid collisions with the headers of other major versions. Similarly, the GLFW 3 library is named glfw3, except when it's installed as a shared library on Unix-like systems, where it uses the soname libglfw.so.3.

So let's see if that is the case:
$ cmake . -DBUILD_SHARED_LIBS && make
...
$ ls -la src | grep libglfw -a
lrwxrwxrwx  1 tmerr tmerr     12 Jul 22 13:10 libglfw.so -> libglfw.so.3
lrwxrwxrwx  1 tmerr tmerr     14 Jul 22 13:10 libglfw.so.3 -> libglfw.so.3.2
-rwxr-xr-x  1 tmerr tmerr 163600 Jul 22 13:10 libglfw.so.3.2

When you link -lglfw to gcc it looks up the filename libglfw.so, resolves the symlink to libglfw.so.3.2, then pulls this information out of it:
$ objdump -p src/libglfw.so.3.2 | grep SONAME
  SONAME               libglfw.so.3

Which it puts in as NEEDED in the compiled program:
$ objdump -p examples/boing | grep libglfw
  NEEDED               libglfw.so.3

So it does do what it says, though I am not sure about the motivation for that decision of not including the version number. There are some libraries like libpython that let you link against specific versions since it names its symlinks  libpython2.7.so and libpython3.so so both packages can be installed simultaneously. Hrm

I took a look at the OS X build output to see what it does and it also has the same behaviour.  When linking with static it would need -lglfw3 and for .dylibs you'll need -lglfw.  My personal opinion is that should both have the 3 at the end.  Not having this can cause a lot of confusing (e.g. take a look at what I was trying to do with raylib and Nim).

Just my 2 cents: the library names are working as intended and as documented, and it is quite convenient to be able to switch between dynamic and static linking simply by changing the name. However, this did trip me up when I was getting started with glfw, and I'm sure it has caused countless others problems too over the years, so I wonder if the benefits actually outweigh the costs?

Isn't that the expected behavior on Unix systems, almost as a rule? I see that in most libraries, at least on Linux. Python is one of the exceptions (and it only causes confusion the ability to have both Python 2 and 3 on the system).

It does seem like the expected way but the alternative isn't unusual either. Naming it libglfw<major>.so would have been good in that:

It would have prevented this ticket https://bugs.launchpad.net/ubuntu/+source/glfw3/+bug/1635684 .
Arch wouldn't need to run glfw2's makefile through sed https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=glfw2 .
Cross platform Makefiles could rely on linking as gcc -lglfw<major> ...

Talking about it now seems kind of moot though.. it would be hard to change the library names after the library is already so widely adopted, wouldn't it?

Those are good points, but I agree with the last sentense:

Talking about it now seems kind of moot though.. glfw2 is basically dead and it would be hard to change the library names since it's already so widely adopted.

The confusing naming is my fault. Didn't realize until after 3.0 was released. I'm thinking the next major version should use the libglfw4.so naming.
It's possible to paper over this somewhat with the CMake package for CMake users or pkgconfig for non-CMake users, but it's definitely not ideal.
I'm tempted to implement this naming in future 3.x releases as well and then install extra symlinks for backward compatibility. That would simplify documentation and other user-facing things but potentially create even more hassle for package maintainers.
What do you think?

Looking specifically at raylib, it looks like it could definitely benefit from some pkg-config love on both macOS and free Unices.
The glfw3.pc file is generated with the right library name depending on the value of BUILD_SHARED_LIBS. It would save you from keeping the ever changing dependency lists up to date as well.

@define-private-public Sorry, hadn't read the whole raylib thread when I wrote about pkg-config just now.

Hello every one here.
I'm encountering some difficulties in setting up an openGL development environment. I'm on Ubuntu Mate 18.04, I'm using openGL superbible 7 source code as a testrun for my compilation. The system runs a skylake intel cpu-gpu and since not very long the intel team releases their driver directly to the kernel guy (so that have me skip the "install proprietary driver) so i have been able to compile glfw3.2.1 but it also comes as a package in this version. I also red this guide:https://stackoverflow.com/questions/17768008/how-to-build-install-glfw-3-and-use-it-in-a-linux-project
I also red other pages discussing linker loader shared libraries static and dynamic, symlinking to the right name but i'm stuck with this error:
/usr/bin/x86_64-linux-gnu-ld : ne peut trouver -lglfw3 (aka cannot find)
I'im not really a developper, I just would want to get into it and i'm commited to follow the principles of my tutorial book ans i want to stay on linux.
Shall I wait for a update on these package or did I miss something.
$whereis libglfw3
Gives me:
libglfw3: /lib/x86_64-linux-gnu/libglfw3 /lib/x86_64-linux-gnu/libglfw3.so
tried a simlink
sudo ln -s /lib/x86_64-linux-gnu/libglfw3 /usr/lib/x86_64-linux-gnu/libglfw3.so
I just wanted to report this
Thanks for reading.


Move Route extension to router

Resolved: https://github.com/gouef/diago/commit/2d4139c729d0928464c64ac53b4e930bdeea1f29


SDDM Crashes when displayManager.autoLogin is set

Describe the bug
On a cold boot, sddm-helper exits uncleanly, leaving a black screen.
This issue seems to have appeared on 20.03 (stable) up to the (at the time of this report) latest nixos-unstable-small channel.
Running sudo systemctl restart display-manager.service from a different TTY will immediately load Plasma with the expected automatic login.
To Reproduce
Steps to reproduce the behavior:

Include nixpkgs/nixos/modules/profiles/graphical.nix as an import at the top of the system configuration
Enable services.xserver.displayManager.autoLogin and set services.xserver.displayManager.autoLogin.user

Expected behavior
Automatic login to Plasma after the Plymouth screen.
Screenshots
No screenshots, but here's an excerpt of the output of systemctl -b -u display-manager.service:
...
Sep 11 19:08:24 nixos sddm-helper[842]: [PAM] Starting...
Sep 11 19:08:24 nixos sddm-helper[842]: [PAM] Authenticating...
Sep 11 19:08:24 nixos sddm-helper[842]: [PAM] Preparing to converse...
Sep 11 19:08:24 nixos sddm-helper[842]: [PAM] Conversation with 1 messages
Sep 11 19:08:24 nixos sddm[812]: Authentication error: "\"System is booting up. Unprivileged users are not permitted to log in yet. Please come back later. For technical details, see pam_nologin(8).\""
Sep 11 19:08:24 nixos sddm-helper[842]: [PAM] authenticate: Authentication failure
Sep 11 19:08:24 nixos sddm-helper[842]: [PAM] returning.
Sep 11 19:08:24 nixos sddm[812]: Authentication error: "Authentication failure"
Sep 11 19:08:24 nixos sddm-helper[842]: [PAM] Ended.
Sep 11 19:08:24 nixos sddm[812]: Auth: sddm-helper exited with 1
Sep 11 19:09:24 nixos xserver-wrapper[826]: (EE) event12 - Telink Wireless Receiver: client bug: event processing lagging behind by 17ms, your system is too slow
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) event2  - Power Button: device removed
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) event13 - Video Bus: device removed
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) event0  - Power Button: device removed
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) event1  - Sleep Button: device removed
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) event9  - Telink Wireless Receiver Mouse: device removed
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) event11 - Telink Wireless Receiver System Control: device removed
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) event12 - Telink Wireless Receiver: device removed
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) event7  - HP WMI hotkeys: device removed
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) event10 - Telink Wireless Receiver Consumer Control: device removed
Sep 11 19:09:24 nixos xserver-wrapper[826]: (II) AIGLX: Suspending AIGLX clients for VT switch
Sep 11 19:11:05 nixos sddm[812]: Signal received: SIGTERM
Sep 11 19:11:05 nixos sddm[812]: Display server stopping...
Sep 11 19:11:05 nixos systemd[1]: Stopping X11 Server...
...

Additional context
Using the same configuration this issue appears on only 2 of my 3 machines. The affected ones are using Intel integrated graphics with SSDs, while the other one has an older AMD GPU and HDD. I'm unsure if this could be an issue relating to drivers (with autologin off, SDDM and X work OOTB) or the significant difference in boot times.
Notify maintainers
@abbradar @ttuegel
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.8.8, NixOS, 21.03pre243391.b38c90caa22 (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.7`
 - channels(johannes): `""`
 - channels(root): `"nixos-21.03pre243391.b38c90caa22"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

I can reproduce this on a machine with AMD graphics; the issue seems to be a race condition with individual user login not being permitted before SDDM starts. Normally you don't notice because it's done by the time you finish typing in your password, but with autologin enabled it doesn't have to wait for the human.
I have had luck with this configuration:
systemd.services.display-manager.wants = [ "systemd-user-sessions.service" "multi-user.target" "network-online.target" ];
systemd.services.display-manager.after = [ "systemd-user-sessions.service" "multi-user.target" "network-online.target" ];

Which may be overkill, but does appear to work.

Which may be overkill, but does appear to work.

That doesn't sound like overkill to me. As a user, I definitely want the network and user sessions available before I log in! :sweat_smile:

Well yes :P But I mean, it's possible that it only needs one of wants or after, not both, or that, say, multi-user.target implies systemd-user-session.services so there's no need to explicitly list the latter.

it's possible that it only needs one of wants or after

I doubt this part: wants and after are independent, and we need the display manager to wait until the other services are started, so we probably do need both. I'm not sure about the second part, I need to investigate.

Shouldn't be marked as stale, there are still people affected by this.
Would it be wise to start a PR for the tweaks suggested by @ToxicFrog? They seem to work on hardware and configurations affected by this.

Would it be wise to start a PR for the tweaks suggested by @ToxicFrog? They seem to work on hardware and configurations affected by this.

I agree, this is not stale. I think the suggestion above is the appropriate solution.

Thanks a lot, the annoying workaround is no longer needed and time to desktop is fast again!


Finalize activation parameters

This includes an activation block height, as well as finalizing the runestone signature.
Activation height:

The activation height is initially un-announced.

When it is announced, it will be at least 2016 blocks (144 * 14, the length of a difficulty adjustment) in the future.
It will fall on a difficulty adjustment, since that seems more auspicious.
This gives everyone a fortnight to prepare.

We should call the activation the etchening.

Is there any update about activation parameters @casey

Nope! There is a lot of work still to be done. I need to add wallet commands in particular.

how to inscribe on runes?

The activation block height is the halving, on block 840,000.


Question: How to know how much memory is a BtreeMap using?

Is it possible to determine the size in bytes used by a BTreeMap with unbounded data types? I have a use case where I need to remove items from the map if the total size exceeds a certain threshold.

I don't think that's possible right now -- at least not in a clean way. I suppose it would make sense to add a size() function to the StableBTreeMap which would give you this information.
@ielashi Do you have an idea of how much work that would be?

Correct, this information isn't exposed right now. It's actually quite easy to expose, since we already track how many pages we're using in memory. We can add that to the list of improvements for us to take on next.


Scan Report Web App not opening // Cannot read properties of undefined (reading 'length')

Hi everybody,
we are able to produce a scan-result.yml and also create a so called web-report-app.html. Unforts the web-report cannot be opened as it produces the following Javascript Error:

Processing the data gets stuck :

Any advise / help would be appreciated.
Thanks
P.

Please find the mentioned web-report.html below:
scan-report-web-app.html.zip

@tsteenbe a quick fix for you?

@meeraa247 I manually export the scan result data from the WebApp report your provided and loaded into reporter-webapp/public/index.html, ran yarn start and it shows me the follow error in the browser console.
TypeError: license is undefined
    licenseToHslColor index.js:48
    WebAppLicense WebAppLicense.js:39
    WebAppOrtResult WebAppOrtResult.js:127
    processOrtResultData processOrtResultData.js:28

The call to licenseToHslColor fails in WebAppLicense.js#L39 as the licenses in the EvaluatedModel contains { _id: 51, id: "" } which is an invalid entry.
@meeraa247  Can you provided the Scanner output file so @sschuberth  or @mnonnenmacher can see why an empty string license is part of the ORT results?

HI @tsteenbe , first of all. Thanks for picking this up so quickly. Appreciate it ;-D
I attached the "scan-result.yml" file for further investigation (@sschuberth)
scan-result.yml.zip

The problem is that the Pod::BVLinearGradient:2.5.6 package literally declares an empty license. I'll create a PR with a fix.


feat(observability)(WIP): allow custom latency distribution buckets

Description
Work In Progress.

This is intended to address #7641 and add support for users to provide an alternative list of distributions thresholds.
No tests.  Not entirely sure this is the best approach.  Not validated at all yet.
Issue reference

Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@jfreeland are you still working on this?

I think there's still a lot of work to do here but I think this is 'closer'.  I didn't get a chance to dig around the Makefile and reverse engineer the quickest path to building my own containers so that I could test this out in my development environment.
I seem to recall a couple months ago that I regenerated the generated files myself but I didn't do that tonight.  I don't recall seeing that I needed to in CONTRIBUTING.md, but maybe I wasn't looking in the right place.
It might be cool if there was some simple app that could be used for local testing with https://tilt.dev like https://github.com/hashicorp-demoapp/hashicups-setups.
I also spotted a little typo https://github.com/dapr/dapr/blob/master/pkg/diagnostics/grpc_monitoring.go#L57 healthProbeRoundripLatency should be Roundtrip along the way that bothered me enough to make a note.  Probably worthy of a separate PR just to ensure consistency and searchability.

I seem to recall a couple months ago that I regenerated the generated files myself but I didn't do that tonight. I don't recall seeing that I needed to in CONTRIBUTING.md, but maybe I wasn't looking in the right place.

I found @ItalyPaleAle 's reference to make code-generate and ran it again but there was no diff.  I updated configuration.yaml by hand in this PR.  That was probably not correct.

this is not ready to rerun tests yet.  please hold.

Update...
Some of the unit test errors were because I was missing the new-ish statusKey.  Maybe that's also related to integration test failures but I didn't quickly determine that.
I haven't yet tracked down what's causing the runtime/service_invocation/res_recv_latency_ms: cannot register view "runtime/service_invocation/res_recv_latency_ms"; a different view with the same name is already registered errors.  I removed this from handler_test.go in https://github.com/dapr/dapr/pull/7646/commits/d5c590a1cf51b04ce35dbfc59dbf01d1da26ec88 but that was actually probably the wrong thing to do.  I believe there's something about these tests running in parallel that was working fine before but no longer is the case after the distribution bucket changes, which is not what I expected.  Now that I've addressed the statusKey change I've got more cannot register view errors in e.g. https://github.com/jfreeland/dapr/blob/8519d226924000dab0793bdecbc8191cea10a0a5/pkg/diagnostics/resiliency_monitoring_test.go#L188.


support table comment

ref:
https://dev.mysql.com/doc/refman/8.0/en/create-table.html

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 8 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 68.714%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
716

Relevant Lines:
1042

💛  - Coveralls


Stabilize the rustc-link-arg option

This change removes the unstable option (tracked by #9426) and unconditionally accepts additional linker arguments (as implemented in #7811 and #8441). Documentation is moved from unstable to what appeared to be the correct location.
I am not aware of any significant concerns with the option and it appears consistent with some other existing stable linker options.
Please let me know if this is not the appropriate process or if there is anything that I am missing from the PR.

Thanks for the PR!  Sorry, we haven't been ignoring this.  There were some issues with the implementation that we were trying to resolve (such as #9563).  We will try to review this soon, though I think the team was generally favorable to moving forward.

The CI error can be resolved by rebasing on the latest master.

Thanks for the update @ehuss
Let me know if there are any questions or other issues that I could help out with here as well.

@rfcbot fcp merge
@rust-lang/cargo, this is a stabilization for the extra-link-arg feature. This allows build scripts to pass additional linker arguments to rustc. This has been historically allowed for cdylibs, and this stabilizes for some other crate types:

cargo:rustc-link-arg-bins=FLAG:  Pass the flag for all [[bin]] targets.
cargo:rustc-link-arg-bin=BIN=FLAG: Pass the flag for a specific [[bin]] target.
cargo:rustc-link-arg=FLAG: Pass the flag for all cargo targets of this package.

Just to confirm, before stabilizing this: all of these only apply to the current crate, not to crates depending on the current crate, right?

Correct.

Thanks @ehuss for the review.
I have updated the change, let me know if there are any other further tweaks I should make before we land this.

Thanks!  🎉
@bors r+

:pushpin: Commit e73703944eabcfbdad64906b3f21260dea26a96f has been approved by ehuss

:hourglass: Testing commit e73703944eabcfbdad64906b3f21260dea26a96f with merge 97007c99a5657a9b15333228007a8bdd4920eacf...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing 97007c99a5657a9b15333228007a8bdd4920eacf to master...


Allow colon in SSH username

Fixes #693

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Question: Multiple instances of same app in same cluster

To me it looks like the layout in this repo assumes that each instance of the app is in the same namespace but in separate clusters. We have some cases where we have multiple instances of the same app in the same cluster. Usually it is the development cluster that has some extra instances that is more stable than the CI/CD build. In some cases we have the staging version in the same cluster as dev.
My question is how this is best accomplished using flux2 multi-tenancy. For example could I somehow create multiple namespaces for the same tenant repository (one ns per app instance)? Or is there some other way to handle this that is better suited for flux2 (not using separate ns)?
I know there are HNS which seems suited for this but it also seems a bit experimental and adds another extension to manage.

Hi, I have the same question. Any update on this? Thanks!

In my case I ended up using a single namespace for the tenant and two kustomiztion.yaml files with different nameprefix setting for the two deployments of the same app. Both kustomiztion.yaml for the deployed instances reference a base kustomiztion.yaml where all the manfiests for the app lives. The nameprefix setting will give  unique names to the objects created for each deployment.

In my case I ended up using a single namespace for the tenant and two kustomiztion.yaml files with different nameprefix setting for the two deployments of the same app. Both kustomiztion.yaml for the deployed instances reference a base kustomiztion.yaml where all the manfiests for the app lives. The nameprefix setting will give unique names to the objects created for each deployment.

Thank you @jonaskello, this makes sense.
@stefanprodan - just wondering if there is any other flux-recommended way to install the same app into multiple namespaces?

FYI. I have been looking for to create multi-tenant setup with hierarchical namespaces noticed similar limitation on it that multiple copies of application on same cluster would be problematic especially in-case where application is split to multiple namespaces. I have been written about it to https://github.com/kubernetes-sigs/hierarchical-namespaces/issues/79 which might be useful on here too.

In my case I ended up using a single namespace for the tenant and two kustomiztion.yaml files with different nameprefix setting for the two deployments of the same app. Both kustomiztion.yaml for the deployed instances reference a base kustomiztion.yaml where all the manfiests for the app lives. The nameprefix setting will give unique names to the objects created for each deployment.

Hi @jonaskello  Did you do this in flux2 or flux v1? I'm unable to achieve this in flux2. Can you please share your kustomization file?

@Bujail I'm using flux2. The prefixing is not a flux specific feature, you just use nameprefix in regular kustomization.yaml like this:
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
metadata:
  name: myapp
namespace: myns
nameprefix: my-prefix-
resources:

I fixed my issue using following kustomize.toolkit file:
apiVersion: kustomize.toolkit.fluxcd.io/v1beta1 kind: Kustomization metadata:   name: external-dns-one   namespace: flux-system spec:   interval: 5m0s   path: "./external-dns-v0.10.2/"   prune: false   sourceRef:     kind: GitRepository     name: external-dns-one   validation: none   targetNamespace: kube-system   patches:     - patch: |         - op: add           path: /spec/template/spec/containers/0/args/0           value: --source=mygateway-gateway       target:         group: apps         version: v1         kind: Deployment         annotationSelector: "patchsel=external-dns"   postBuild:     substitute:       name: "external-dns-one"       replicas: "1"       zone: "private"       owner_id: "nginx-one-"       ingress_class: "nginx-one"       cpu_limit: "100m"       mem_limit: "200Mi"       cpu_req: "20m"       mem_req: "100Mi"       node_group: "agent"


remove reference to username in getting-started

There were the following issues with your Pull Request

Commit: 78823da4ffbda826441d322155438c9a7e4a8d0c
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com

@joshuawilson snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1241-fabric8-ui.int.rdu2c.fabric8.io


error install python-shogun on server

I am going to use shogun in an experiment written by tensorflow running on a  ubuntu server, I followed the install instructions but still get error when try to install python-shogun.

Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package python-shogun

Any suggestion? Thanks.

Without the exact steps you did and system specs, we cannot help you


Opps app not registered with office 365 (altho registered)

When i run, i keep getting this message

altho the app is already registered

as shown in code

If you're using VS 2015, when you run the Add connected services wizard, you end up with a key in your App.xaml file that looks like this:
x:String x:Key="ida:ClientId"
This line, in MainPage.xaml.cs: checks for different casing:
if (!App.Current.Resources.ContainsKey("ida:ClientID"))
So you'll get past this problem by changing that line to this:
if (!App.Current.Resources.ContainsKey("ida:ClientId"))
Also, there is a line in AuthenticationHelper.cs that looks for this key:
private static readonly string ClientID = App.Current.Resources["ida:ClientID"].ToString();
So you'll need to change that line to this:
private static readonly string ClientID = App.Current.Resources["ida:ClientId"].ToString();
I'll be submitting a pull request shortly that fixes this issue for VS 2015 users.

Another way to approach this (and probably the best approach, for now) is simply to change the key name in the App.xaml file.
In other words, change this:
<x:String x:Key="ida:ClientId"></x:String>
to this:
<x:String x:Key="ida:ClientID"></x:String>
That will fix the problem, and the sample will still work for developers who are using VS 2013.

Hi @jamescro, your solution did it.

x:String x:Key="ida:ClientID"></x:String

Yet another VS2015 issue.
thx

Perfect jamescro.
Small change case ClientID in MainPage, AuthenticationHelper, App


Spinner support

@thii, any ideas if it's possible to use the spinners?

@luisgizirian Can you please explain more clearly?

Of course @thii. Sorry about that.
Font awesome, has the ability to display spinner icons (those used for "Loading..." kinda screens & overlays).  I think these is some CSS trickery on top of icons. Are these kind of animations also ported inside this swift library?
Hope I made myself clear this time! Thanks!

@luisgizirian You'll probably have to use UIAnimation to rotate the container  (UILabel...) of the icon you want to spin. I'll have to look into this to see if we can have spinner support inside this library.

I, too, would appreciate this feature.


A suggestion on LCD peripheral (IDFGH-7191)

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when I use LCD peripherals.
SPI master settings cannot be configured in function esp_lcd_new_panel_io_spi(), but I want to change TX to LSB first, to ensure the color of my LCD is correct.
Describe the solution you'd like
in /esp-idf/components/esp_lcd/src/sep_lcd_panel_io_spi.c,
spi_device_interface_config_t devcfg = {
        .flags = SPI_DEVICE_HALFDUPLEX,

The flag can be set as a parameter to ensure the flexibility of the SPI master device driving the screen.
Describe alternatives you've considered
Give some extra functions of API to change these settings.
Chinese
LCD外设生成的SPI主设备难以更改flag参数，但有屏幕修改LSB/MSB等一些需求。

Thanks for reporting.

@LucivorLin Yes, we didn't expose all the underlying spi device flags to LCD IO layer by design (because not all spi flags are suitable for LCD). But left the space to extend them in the future by adding more flags into esp_lcd_panel_io_spi_config_t.flags.
Which flag in your case want to have? SPI_DEVICE_TXBIT_LSBFIRST or SPI_DEVICE_RXBIT_LSBFIRST or SPI_DEVICE_BIT_LSBFIRST?

@suda-morris  SPI_DEVICE_TXBIT_LSBFIRST is I want.

@LucivorLin
spi-lcd-support-transmit-lsb-first.patch.zip
Is this patch sufficient for you? Sorry I don't have an LCD at hand that acquires an LSB first transmission.
Ways to apply the patch:

decompress the zip file, and get the .patch file (e.g. my_fix.patch)
cd $IDF_PATH , check out the master branch
git apply my_fix.patch or git am my_fix.patch

After apply the patch, what should I do?
Since the flags of esp_lcd_new_panel_io_spi didn't change, I would like to ensure my operation is correct.
xxx@LAPTOP-XXXXXXXX MINGW64 /e/Softwares/ESP/Container/esp-idf (master)
$ git apply temp/spi-lcd-support-transmit-lsb-first.patch
error: can't open patch 'temp/spi-lcd-support-transmit-lsb-first.patch': Permission denied

xxx@LAPTOP-XXXXXXXX MINGW64 /e/Softwares/ESP/Container/esp-idf (master)
$ git am temp/spi-lcd-support-transmit-lsb-first.patch

xxx@LAPTOP-XXXXXXXX MINGW64 /e/Softwares/ESP/Container/esp-idf (master)

Should there be any reaction of the latter command line? @suda-morris

@LucivorLin  The patch file extracted from the attached zip file is 0001-lcd-spi-lcd-support-transmit-lsb-first.patch. Maybe you didn't decompress it successfully?

Now I successfully applied, and I added .flags.lsb_first = 1 in esp_lcd_panel_io_spi_config_t io_config.
However, since I am new to SEP-IDF master branch, I cannot build the project as before, the error shows as follow:
Traceback (most recent call last):
  File "E:/Softwares/ESP/Container/esp-idf/tools/ldgen/ldgen.py", line 16, in <module>
    from ldgen.entity import EntityDB
  File "E:\Softwares\ESP\Container\esp-idf\tools\ldgen\ldgen\entity.py", line 12, in <module>
    from pyparsing import (Group, Literal, OneOrMore, ParseException, SkipTo, Suppress, White, Word, ZeroOrMore, alphas,
ImportError: cannot import name 'rest_of_line' from 'pyparsing' (e:\Softwares\ESP\Tools\python_env\idf4.4_py3.8_env\lib\site-packages\pyparsing.py)
[2/8] Performing build step for 'bootloader'
[1/1] cmd.exe /C "cd /D E:\ESP32_Projects\S2_projects\lvgl\build\bootloader\esp-idf\esptool_py && e:\Softwares\ESP\Tools\python_env\idf4.4_py3.8_env\Scripts\python.exe E:/Softwares/ESP/Container/esp-idf/components/partition_table/check_sizes.py --offset 0x8000 bootloader 0x1000 E:/ESP32_Projects/S2_projects/lvgl/build/bootloader/bootloader.bin"
Bootloader binary size 0x5100 bytes. 0x1f00 bytes (28%) free.
ninja: build stopped: subcommand failed.
ninja failed with exit code 1

@suda-morris

the pyparsing dependency of ldgen has been upgraded on master branch. I guess you need to rerun the "install.sh" script to install the python environment needed for master branch. Currently you're using the idf 4.4 environment for compiling the master branch (e:\Softwares\ESP\Tools\python_env\idf4.4_py3.8_env)

Well, I tried using the patch in v4.4.
However, the LCD cannot work, I think it may be because of the priority of the operator in C language.
Is that better to change the code from
spi_device_interface_config_t devcfg = {
        .flags = SPI_DEVICE_HALFDUPLEX | io_config->flags.lsb_first ? SPI_DEVICE_TXBIT_LSBFIRST : 0,

to
spi_device_interface_config_t devcfg = {
        .flags = SPI_DEVICE_HALFDUPLEX | (io_config->flags.lsb_first ? SPI_DEVICE_TXBIT_LSBFIRST : 0),

?
@suda-morris

@LucivorLin  Ah yes, my bad. Nice found. So with your fix, does that work now?

It works properly now.


Add github action based build

Add Github Action based build to catch build issues in PRs
Fix tests broken by #43

:+1:

:+1:


Not disconnected from server, when removed from webinterface

When I remove all servers in the webinterface and click save, my znc remains connected to that server!

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/101805-not-disconnected-from-server-when-removed-from-webinterface?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

Fixed as I removed two networks yesterday and ZNC disconnected me which made my client reconnect and then it said "no such network".


Saving and loading snapshots .

Hi ,
Is it possible to take a snapshot and reload it in another .ty file with few new sheets  connecting to sheets  of that snapshot  ?
Simply to ignore unnecessary running of partially working part every time !
PS : I have tried this already . does not give any syntax  error ! so If that affects logically then It would help.

Sure.  A .ty file is just Python code (the special extension is just so that Windows will launch Topographica's Python on it rather than any other Python that might be installed), so you can do anything Python allows.  E.g. you can write a new .ty file that starts out with load_snapshot("file.typ"), and then add whatever commands you like to add additional sheets, etc.

Thank you .


Support for $ref in securityDefinitions

In a similar vein to #677: I'd also like to see support for references in securityDefinitions - again I have several APIs scattered across a number of files, and it would be helpful to share the definitions between them.

Parent: #560

This is about a structure in 2.0 that is no longer relevant.  Security Schemes have been in the Components Object since 3.0, and see

#1972
#3776

for more recent similar requests.  Closing.


Black and white list doesn't appear with large extra fields

When is listed a "large" list of extra fields the black and white checkbox list list doesn't appear.
I am using Ubuntu 16.04.
Normal condition:

Issue:

Work around: the extra filed checkbox list should have the scrollbar enabled.

This was a tricky one. wxCheckListBox sets it's size on frame init based on it's contents and doesn't care about any setSize, setMaxSize() calls later. Overrides any proportion directives of parent sizers etc.


Modify Calculation of SE in Simulation

Pooled variance estimate is not exactly the same as unpooled in the t-test. Since t-stat = effect size / SE, we can calculate SE as effect size / t-stat.

💯 thanks for catching this!


Snippets with children headings

From the README.org:

Only headings without children are considered as snippets.

Have you thought about an option to get the ability to navigate through large snippets by collapsing and expanding sub-headings? This would make it much more easier to manage large and complex snippets from my perspective.
Thanks for your thoughts. Maybe you've already faced similar issues and found a way to deal with them.
Backround for my question
I do have some snippets that contain templates for larger projects, consisting of up to fifty headings with pre-defined dependencies among them using org-edna.
While yankpad gives me the advantage of being able to define snippets within org mode as major mode, the limitation of "no snippets with children" is some kind of a downer for managing large snippets. I usually locate a specific sub-heading by collapsing the headings and looking for the heading text of choice.
Related
For all readers that wonder how I'm able to have sub-headings within snippets: you may escape asterisks of sub-headings with a backslash within the snippet:
** This is a snippet

Snippet text.

\** This is a sub-heading of level 2 within the snippet

Sub-heading text.

If somebody is interested in some thoughts on task dependencies (although not particular yankpad-related): Project management, from org-depend to org-edna, Dependencies, How to Define Projects.

Hi! It may be doable, but I think it would requirie some changes on how snippets work internally. However I think your problem could be solved (but perhaps a bit less elegantly) by using src block snippets. I made a small tweak to make them respect the current indentation of the org-mode buffer when yanked. Here's an example:
** test:Test source block            :src:

#+BEGIN_SRC org
  ,* My test
  ,** Level 2

     I want to write something here $1
  ,** Level 2 again

     Lorem ipsum.
  ,* Back to the root
  $0
#+END_SRC

If you use this in your Yankpad file then you could go to the source block and press C-c ' (by default) and edit the snippet in a separate buffer, where you could expand headings etc.

Hi @Kungsgeten ,
I just installed yankpad-20210619.1756 which expand your example as expected.
Your workaround seems to be a clever one which actually might have some advantages as well such as limiting the search space to the headings of the current snippet instead of the while yankpad file. :+1:

Oh, one thing though: when my snippet already contains a SRC block, this trick does not work, right?
Reason: the first #+END_SRC ends the snippet, not the corresponding SRC block within the snippet.

Yes I think that org-mode does the character escaping automatically if you edit using C-c '. Happy to hear that you're still using Yankpad and that the suggested method works for you!

yankpad was and is one of my most important packages!


Environment variables for URLs

closes #8

The only issue what I see with this is that what happens if in the future we want to add more and more flags/envs. At the moment EnvConfig.set_environment accept optinal parameters, but what happens if there will be 300 flags. I'm only worried about because this system should be configurable based on flags/envs, so I assume there will be a ton.

That's a valid point. I can replace the two parameters with a list of key-values or something like that. But If we are going to have a lot of variables maybe we should read in a config file to set them or create the flags in a more generic way (or both).

The requested changes pushed later. @jmezo please create an issue about.


Questions about github-proxy

@pGrovesy , thanks a lot for sharing this code, the idea sounds too good to be true, and I'm more than happy to help with code.
Before getting into technology, I'd like to ask you some questions about the desired solution.
I see that the roadmap mentions to Become a "true" proxy that can be set via "git config http.proxy...."; I guess this solution will have to work around TLS end-to-end encryption in some way, which would probably force the use of some unconventional libraries/computation.
I wonder if - instead - we could introduce a simple naming convention to generate the Git endpoint that is read by the server, so that developers get easily used to it.
For example, Git remotes can be defined as git://<github-proxy-host>/gh/<github org or user>/<github repo name>.git, so that the server can extract <github org or user> and <github repo name> and be able to dispatch incoming commands (if/when validation occurs and is passed).
Another confirmation is that this proxy will only manage Git activity.... and therefore:

It could actually be called git-proxy
GitHub Issues, Pull Requests, Discussions and other way of collaborations that are not done via Git commit/push, are out of scope for this project

Is that correct?
TY!

So right now it works by setting up a remote which assumes the target is https://{host}/{project}/{repo} and is setup locally as 'remote' e.g. 'git remote origin http://{proxy}/{project}/{repo}.git
We would have to decide if we just stick with it being a 'remote' or we decide to support being a 'proxy' - "git config --global --add HTTP.proxy HTTP://{git-proxy}"
SSH is another topic, right now supporting Http and Https

So right now it works by setting up a remote which assumes the target is https://{host}/{project}/{repo} and is setup locally as 'remote' e.g. 'git remote origin http://{proxy}/{project}/{repo}.git

Got it now (also looked at the code, it's clearer now). So am I right assuming that there will be a way to configure the proxy so that {host} is github.com , allowing us to - for example - do git clone https://localhost:3001/finos/open-developer-platform.git ?

We would have to decide if we just stick with it being a 'remote' or we decide to support being a 'proxy' - "git config --global --add HTTP.proxy HTTP://{git-proxy}"

I'd vote for both; I kinda like the first approach too, as it's less hidden; you can always do git remote -v and see the Proxy in the URL, which makes it clear that you're not communicating directly with the github.com (upstream) endpoint.
I agree that some firms may find the proxy setup easier to integrate, so I'm in favour of this approach too; could it be a configuration switch?

Got it now (also looked at the code, it's clearer now). So am I right assuming that there will be a way to configure the proxy so that {host} is github.com , allowing us to - for example - do git clone https://localhost:3001/finos/open-developer-platform.git ?

Correct and yes I think both styles are good tool have- I'll put some more of a roadmap together in the next day -- feature suggestions welcome, audit-log-synchs, synch vs asnych pushes etc


Unix sockets

Add a parameter to the web host resources that listens on a unix socket instead of a tcp port.

Codecov Report

Merging #93 into master will decrease coverage by 1.17%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
- Coverage   67.06%   65.89%   -1.18%
==========================================
  Files          14       14
  Lines         331      346      +15
==========================================
+ Hits          222      228       +6
- Misses        109      118       +9

Impacted Files
Coverage Δ

lib/poise_application_ruby/resources/puma.rb
66.66% <33.33%> (-6.07%)
:arrow_down:

lib/poise_application_ruby/resources/unicorn.rb
66.66% <33.33%> (-6.07%)
:arrow_down:

lib/poise_application_ruby/resources/thin.rb
63.33% <33.33%> (-4.67%)
:arrow_down:

Continue to review full report at Codecov.

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


diffoscope: 259 -> 260

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for diffoscope is: Perform in-depth comparison of files, archives, and directories
meta.homepage for diffoscope is: https://diffoscope.org/
meta.changelog for diffoscope is: https://diffoscope.org/news/diffoscope-260-released/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 260 with grep in /nix/store/1wsflyi3a3fsh6x36wqb9wz2jhnms1dm-diffoscope-260
found 260 in filename of file in /nix/store/1wsflyi3a3fsh6x36wqb9wz2jhnms1dm-diffoscope-260

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
diffoscope
diffoscopeMinimal

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1wsflyi3a3fsh6x36wqb9wz2jhnms1dm-diffoscope-260 \
  --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 diffoscope https://github.com/r-ryantm/nixpkgs/archive/52afd5d25f30c6ab0a4c25597a391619656eba8a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/52afd5d25f30c6ab0a4c25597a391619656eba8a#diffoscope

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1wsflyi3a3fsh6x36wqb9wz2jhnms1dm-diffoscope-260
ls -la /nix/store/1wsflyi3a3fsh6x36wqb9wz2jhnms1dm-diffoscope-260/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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

6 packages built:

diffoscope
diffoscope.dist
diffoscope.man
diffoscopeMinimal
diffoscopeMinimal.dist
diffoscopeMinimal.man

Maintainer pings
cc @dezgeg @danielfullmer @RaitoBezarius for testing.

Already broken on master for certain dependencies, so  not a regression.


added vue setup instructions to README

I spent an unreasonable amount of time figuring out how to make this work with Vue
Initial annotorious setup was simple thanks to this guide from the documentation But figuring out how to add these new selectors was rough with the documentation provided.
I added a minimal working example to the README for how to install and set up annotorious and the selector pack in a vue project. If you don't think it's appropriate for the main readme, if it can be added to the documentation somewhere I think it may save someone a headache in the future.
Thank you for making this super useful tool

Thanks so much for this! I agree: the Readme probably isn't the right place. I think I'll move it over to the Vue guide, as an additional section. And then just add a link to this Readme, pointing to the Guide. (Actually: your example should actually be valid for most other plugins, too. So this is super helpful more generally!)

P.S.: I'll no need to update this PR, BTW. I'll take care of updating the guide + readme.

Done! https://recogito.github.io/guides/annotorious-in-vue/#using-plugins
Thx again!


Add augmentations for demucs

Current demucs augmentations found in config.yaml in demucs repo:
augment:
  shift_same: false
  repitch:
    proba: 0.2
    max_tempo: 12
  remix:
    proba: 1
    group_size: 4
  scale:
    proba: 1
    min: 0.25
    max: 1.25
  flip: true

Implemented in demucs/augment.py while used in demucs/solver.py and in demucs/train.py

According to the paper, pitch/tempo shift is a new technique used in augmentation. https://arxiv.org/pdf/1911.13254.pdf section 5.3
I'll think about whether we should use both shift and other augments at one time (if it's shown in the paper, that it helps) or experiment with both separately looking at our label-noise dataset.
@dabrze @SebChw any opinions?


Default constructor all colored as type

Code

VS

Closing, moved to https://github.com/ionide/ionide-fsgrammar/issues/38


HDDS-4668. Intermittent failure in TestOMRatisSnapshots

What changes were proposed in this pull request?
TestOMRatisSnapshots#testInstallSnapshot fails intermittently.  PR includes small patch to integration test to fix intermittent problem.  Problem occurs when transactions are added to leader OM that are then applied to a follower OM that is placed into the active state from inactive.  Test puts the follower OM into active state from inactive and expects the follower to initially have less transactions than the leader index, however the updates are done with a nonblocking background thread.  Checking the OM index in this manner is prone to error due to the nonblocking install, at times the number of transactions, the index of the follower OM is >= the leader.
To fix the intermittent issue, a simple wait and retry construct is used to check the follower index periodically until it updates, installs the snapshot.  A 3s timeout is used to propagate a fatal error.  The wait and retry block checks the state of the snapshot install and ensures the follower index is <=   leader lastAppliedTermIndex -1.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4668
How was this patch tested?
Tested though hadoop-ozone/dev-support/checks/integration.sh script in CI environment with environment variables $ITERATIONS=60, $MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.class=standard -Dmaven.wagon.http.retryHandler.count=3
hadoop-ozone/dev-support/checks/integration.sh  -Dtest=TestOMRatisSnapshots#testInstallSnapshot
Run cat target/integration/summary.txt
Iteration 1 exit code: 0
Iteration 2 exit code: 0
Iteration 3 exit code: 0
Iteration 4 exit code: 0
Iteration 5 exit code: 0
Iteration 6 exit code: 0
Iteration 7 exit code: 0
Iteration 8 exit code: 0
Iteration 9 exit code: 0
...
Iteration 60 exit code: 0

Thanks @neils-dev for fixing this test.


Message ack regression

Fix for #511.
Includes some commits cherry-picked from #504:
f5901f3a1d80fdaea8bf4cb16fd1ff8541fd6a86: standard toxiproxy tests for RpcConsumer
8841dd1464b56906e0d28c967ac58644daaeb381 and 891a91c94fb20e4328cf6a31765f3f8982d1e823: fix for hanging toxiproxy process

Hi @mattbennett
We'd love to see this land as the issue is a blocker for us getting nameko into production.
What are your expectations for getting this into a new release?
thanks

@noisyboiler I'm happy to release as soon as it's been through review. Some critical eyes would be helpful, thanks. I would also like @davidszotten's approval before landing it.

Race may be wrong word, but connection might close between checking and acking
Sent from my phone

On 16 Feb 2018, at 11:23, Matt Yule-Bennett<EMAIL_ADDRESS>wrote:
@mattbennett commented on this pull request.
In test/conftest.py:

@@ -161,6 +161,12 @@ def reset(self):
yield controller
controller.reset()

delete proxy

allow some grace period to ensure we don't remove the proxy before

other fixtures have torn down

resource = 'http://{}/proxies/{}'.format(toxiproxy_server, proxy_name)
eventlet.spawn_after(10, requests.delete, resource)
No, a process won't wait for greenthreads to finish before exiting.

This cleanup really isn't required because we stop the whole toxiproxy server at the end of the test run. I added it while debugging a hanging toxiproxy (which turned out to be http://tanin.nanakorn.com/blogs/373) thinking that the problem might be the growing number of proxies that we add and never remove.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

much appreciated chaps. 👍


RelayerMessageCallback invokes all session request handlers for each event (and causes json deserialization exceptions)

Context (Web3Wallet)
I've two different complex RPC request payloads (for eth_sendTransaction and personal_sign methods):
[RpcMethod("personal_sign"), RpcRequestOptions(Clock.ONE_MINUTE, 99994)]
public class EthPersonalSignRequest : List<string>

[RpcMethod("eth_sendTransaction"), RpcRequestOptions(Clock.ONE_MINUTE, 99997)]
public class EthSendTransactionRequest : List<TransactionInput>

// handlers
client.Engine.SignClient.SessionRequestEvents<EthSendTransactionRequest, EthSendTransactionResponse>().OnRequest += OnEthSendTransactionRequested;
client.Engine.SignClient.SessionRequestEvents<EthPersonalSignRequest, EthPersonalSignResponse>().OnRequest += OnEthPersonalSignRequested;

The Issue

The wallet client throws a JSON deserialization exception when it receives a personal_sign (or eth_sendTransaction) request.

Reason

The RelayerMessageCallback function of the TypedMessageHandler invokes all session request handlers for each session request event (regardless of method type).
When the client receives a personal_sign request (an array of strings), it tries to deserialize it to the payload of the eth_sendTransaction method (an array of objects) and throws an exception.

Actual Root Cause

When registering session request handlers, the HandleMessageType function of the TypedMessageHandler obtains the method name by calling RpcMethodAttribute.MethodForType<T>
RpcMethodAttribute.MethodForType<T>, finds the method name from the RpcMethodAttribute of the given type (T). But, for session request events, the type (T) will always be SessionRequest<TR> (TR: the actual RPC request type) and it always returns wc_sessionRequest:

[RpcMethod("wc_sessionRequest")]
public class SessionRequest<T> : IWcMethod

It registers all session request event handlers (to the messageEventHandlerMap) with the key of wc_sessionRequest.
When an event is received, the RelayerMessageCallback function retrieves all handlers with the key of wc_sessionRequest and invokes all of them. In the RequestCallback local function (of the HandleMessageType function of the TypedMessageHandler), it tries to decode the message to the target request type, and it throws a JSON deserialization exception when the payload schema does not match:

var payload = await this.Core.Crypto.Decode<JsonRpcRequest<T>>(topic, message, options);

The WrappedRefOnOnRequest function of the SessionRequestEventHandler checks the method of the event, and it does not execute the RequestCallback function of the TypedEventHandler (not TypedMessageHandler) if it does not match, but it is not enough to prevent this issue.

Proposed Solution

RpcMethodAttribute.MethodForType<T> should check if the type T is a generic type:

If so, it should check whether any of the generic arguments of type T has the RpcMethodAttribute attribute and return the method name of the generic argument.
In this way, the event handler of the personal_sign request will be registered as personal_sign instead of wc_sessionRequest (in messageEventHandlerMap)

The Method property of the JsonRpcPayload class should check whether the param.request.method value exists in the _extraStuff dictionary. If so, it should return the param.request.method instead of the method.
I created a PR to fix this issue: https://github.com/WalletConnect/WalletConnectSharp/pull/187

How to Reproduce the Issue?
You can add this integration test to the SignTests.cs test file of the WalletConnectSharp.Sign.Test project.
 // represents array of strings requests, similar to personal_sign
[RpcMethod("complex_test_method"), RpcRequestOptions(Clock.ONE_MINUTE, 99990)]
public class ComplexTestRequest: List<string>
{
    public ComplexTestRequest()
    {
    }

    public ComplexTestRequest(params string[] args) : base(args)
    {
    }

    public int A
    {
        get
        {
            if (Count != 2 || !int.TryParse(this[0], out var a))
            {
                return 0;
            }

            return a;
        }
    }

    public int B
    {
        get
        {
            if (Count != 2 || !int.TryParse(this[1], out var b))
            {
                return 0;
            }

            return b;
        }
    }
}

// represents array of objects requests, similar to eth_sendTransaction
[RpcMethod("complex_test_method_2"),
 RpcRequestOptions(Clock.ONE_MINUTE, 99991),
 RpcResponseOptions(Clock.ONE_MINUTE, 99992)]
public class ComplexTestRequest2 : List<TestRequest2>
{
    public ComplexTestRequest2()
    {
    }

    public ComplexTestRequest2(params TestRequest2[] args): base(args)
    {
    }

    public string X => this.FirstOrDefault()?.x ?? string.Empty;

    public int Y => this.FirstOrDefault()?.y ?? -1;
}

[Fact, Trait("Category", "integration")]
public async Task TestTwoUniqueComplexSessionRequestResponse()
{
    await _cryptoFixture.WaitForClientsReady();

    var testAddress = "0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045";
    var testMethod = "complex_test_method";
    var testMethod2 = "complex_test_method_2";

    var dappConnectOptions = new ConnectOptions()
    {
        RequiredNamespaces = new RequiredNamespaces()
        {
            {
                "eip155",
                new ProposedNamespace()
                {
                    Methods = new[] {testMethod, testMethod2},
                    Chains = new[] {"eip155:1", "eip155:10"},
                    Events = new[] {"chainChanged", "accountsChanged"}
                }
            }
        }
    };

    var dappClient = ClientA;
    var connectData = await dappClient.Connect(dappConnectOptions);

    var walletClient = ClientB;
    var proposal = await walletClient.Pair(connectData.Uri);

    var approveData = await walletClient.Approve(proposal, testAddress);

    var sessionData = await connectData.Approval;
    await approveData.Acknowledged();

    var rnd = new Random();
    var a = rnd.Next(100);
    var b = rnd.Next(100);
    var x = rnd.NextStrings(AllowedChars, (Math.Min(a, b), Math.Max(a, b)), 1).First();
    var y = x.Length;

    var testData = new ComplexTestRequest(a.ToString(), b.ToString());
    var testData2 = new ComplexTestRequest2(new TestRequest2() {x = x, y = y});

    var pending = new TaskCompletionSource<int>();

    // Step 1. Setup event listener for request

    // The wallet client will listen for the request with the "test_method" rpc method
    walletClient.Engine.SessionRequestEvents<ComplexTestRequest, TestResponse>()
            .OnRequest += ( requestData) =>
        {
            var request = requestData.Request;
            var data = request.Params;

            requestData.Response = new TestResponse()
            {
                result = data.A * data.B
            };

            return Task.CompletedTask;
        };

    // The wallet client will listen for the request with the "test_method" rpc method
    walletClient.Engine.SessionRequestEvents<ComplexTestRequest2, bool>()
        .OnRequest += ( requestData) =>
    {
        var request = requestData.Request;
        var data = request.Params;

        requestData.Response = data.X.Length == data.Y;

        return Task.CompletedTask;
    };

    // The dapp client will listen for the response
    // Normally, we wouldn't do this and just rely on the return value
    // from the dappClient.Engine.Request function call (the response Result or throws an Exception)
    // We do it here for the sake of testing
    dappClient.Engine.SessionRequestEvents<ComplexTestRequest, TestResponse>()
        .FilterResponses((r) => r.Topic == sessionData.Topic)
        .OnResponse += (responseData) =>
    {
        var response = responseData.Response;

        var data = response.Result;

        pending.TrySetResult(data.result);

        return Task.CompletedTask;
    };

    // 2. Send the request from the dapp client
    var responseReturned = await dappClient.Engine.Request<ComplexTestRequest, TestResponse>(sessionData.Topic, testData);
    var responseReturned2 = await dappClient.Engine.Request<ComplexTestRequest2, bool>(sessionData.Topic, testData2);

    // 3. Wait for the response from the event listener
    var eventResult = await pending.Task.WithTimeout(TimeSpan.FromSeconds(5));

    Assert.Equal(eventResult, a * b);
    Assert.Equal(eventResult, testData.A * testData.B);
    Assert.Equal(eventResult, responseReturned.result);

    Assert.True(responseReturned2);
}

Hey @aliarbak, could you please check if #193 solves it for your?

yes, thanks 🙏 @skibitsky


Fix regression due to handling of normal range

Fix regression due to https://github.com/flackr/scroll-timeline/pull/223
We should return normal as the default  range start and end, but the parsing still expects a TimelineRangeOffset which can cause an issue.
Fixing the issue by only returning normal when no range was set by the user, keeping the TimelineRangeOffset from getNormalStart/EndRange otherwise.
There's already a todo to handle the full syntax later.

@bramus Do you have time to review this?

Merged it in.
Besides the current TODO’s for this code, passing in explicit normal values also doesn’t work properly. I have filed https://github.com/flackr/scroll-timeline/issues/236 for this.


It's not possible to pass in the endpoint parameter

It's not possible to pass in the optional endpoint string:
const storyblokApi = new StoryblokClient(apiOptions);   return { storyblokApi };
Only apiOptions are supported.

Hello @philipp-bender,
I think this change lately introduced should help for that purpose:
https://github.com/storyblok/storyblok-js-client/pull/598
cc @alexjoverm


Remove references to undeclared argument and add ability to specify version file name

There are a number of places in the git helper that reference checkout_root which isn't declared anywhere (some methods have it as an argument but others do not). The proposed fix for this is to make path a required argument instead of defaulting to nil so that the checkout path doesn't need to be inferred. In fact, the path can't be inferred anyway because checkout_root isn't declared in the methods being adjusted.
To make this work, it was necessary to remove the default value for the extra_cmd argument which immediately precedes the path argument, however since leaving a default nil value for path would cause an exception, I think it's safe to assume that users are also supplying a value for extra_cmd.
This also gives users the ability to specify the name of the version file if they so desire.

This lgtm 👍  Doesnt change behavior for our use case.


add codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@1456847). Click here to learn what that means.
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #20   +/-   ##
=======================================
  Coverage        ?   73.88%
=======================================
  Files           ?        3
  Lines           ?      157
  Branches        ?        0
=======================================
  Hits            ?      116
  Misses          ?       41
  Partials        ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Support Alluxio allowed Fuse truncates

What changes are proposed in this pull request?
Fixes https://github.com/Alluxio/alluxio/issues/14768
Why are the changes needed?
Support the truncate cases that Alluxio Fuse can support,
error out and give clear information when cannot support.
Does this PR introduce any user facing changes?

@maobaolong @yyongycy PTAL, thanks!

@LuQQiu Please correct the style issue
[ERROR] /usr/src/alluxio/integration/fuse/src/main/java/alluxio/fuse/AlluxioJniFuseFileSystem.java:25:8: Unused import - alluxio.exception.FileIncompleteException. [UnusedImports]

Error: 1.859 [ERROR] Errors:
Error: 1.860 [ERROR] alluxio.client.cli.job.DistributedCommandsCancelStatsTest.testDistributedCpCancelStats
Error: 1.862 [ERROR]   Run 1: DistributedCommandsCancelStatsTest.testDistributedCpCancelStats:128 » Timeout ...
Error: 1.863 [ERROR]   Run 2: DistributedCommandsCancelStatsTest.testDistributedCpCancelStats:128 » Timeout ...
21:07:51.863 [INFO]

@maobaolong @yyongycy PTAL, thanks!

@maobaolong @yyongycy PTAL, thanks

I like this change.

@LuQQiu hi lu, in issues 14678, since the business logic is black box, I guess it belongs to case 2, which now seems to solve the problem, that is, it will not throw exceptions again. But for this scenario, is it possible that it is case 3 or 4?

@codings-dan for issue https://github.com/Alluxio/alluxio/issues/14768, it's possible that it belongs to case 1, 2, 4.
We don't know in the previous implementation but will have much more informative error message, so we can know what case it's in the future

Thanks, the change looks good.

Error: 9.289 [ERROR] Errors:
Error: 9.290 [ERROR] alluxio.client.cli.job.DistributedCommandsCancelStatsTest.testAsyncPersistCancelStats
Error: 9.292 [ERROR]   Run 1: DistributedCommandsCancelStatsTest.testAsyncPersistCancelStats:172 » Timeout T...
Error: 9.293 [ERROR]   Run 2: DistributedCommandsCancelStatsTest.testAsyncPersistCancelStats:172 » Timeout T...

Added to the sheet, not related to this PR

alluxio-bot, merge this please

alluxio-bot, merge this please


qgo run

The qgo run command should build the executable and run it. The following is assumed:

Run go mod tidy before the build
The go build command will have all LDFlags defined in the package.json dynamically defined in the command.
The appropriate environment variables are auto-configured, such as GOOS and GOWASM.

When the command is run, the color-coded command should be displayed so the user knows what actually happened.

Completed


The attribute [name] either does not exist or was not retrieved for model [App\Models\User]

Pulse Version
v1.0.0-beta7
Laravel Version
v10.37.3
PHP Version
8.2
Livewire Version
v3.3.0
Database Driver & Version
10.4.28-MariaDB
Description
We have an e-commerce application and in the users table we use 'first_name' and 'last_name' in favor of 'name' column. This doesn't cause any problems and when we visit the pulse endpoint it shows 'Unknown' for the user name in the Application Usage card. However, when we add Model::shouldBeStrict() in the AppServiceProvider and then visit the pulse endpoint, this time it throws an Exception showing that the [name] attribute doesn't exist in the User model. On one hand this makes sense but on the other it make this package unusable for us. So we either have to rename the 'first_name' column back to 'name' (which is a huge work), or don't use strict models at all.
So this is the method from Pulse.php that causes the error:
    public function resolveUsers(Collection $ids): Collection
    {
        if ($this->usersResolver) {
            return collect(($this->usersResolver)($ids));
        }

        if (class_exists($class = \App\Models\User::class) || class_exists($class = \App\User::class)) {
            return $class::whereKey($ids)->get()->map(fn ($user) => [
                'id' => $user->getKey(),
                'name' => $user->name, // Throws exception if Model::shouldBeStrict()
                'email' => $user->email,
            ]);
        }

        return $ids->map(fn (string|int $id) => [
            'id' => $id,
            'name' => "User ID: {$id}",
        ]);
    }

It would be great if we were able to specify the user's name key in the config/pulse.php file, so in our situation something like:
'username' => 'first_name'

and then use this value in the resolveUsers method
public function resolveUsers(Collection $ids): Collection
{
...
if (class_exists($class = \App\Models\User::class) || class_exists($class = \App\User::class)) {
    $name = config('pulse.username');
    return $class::whereKey($ids)->get()->map(fn ($user) => [
        'id' => $user->getKey(),
        'name' => $user->{$name},
        'email' => $user->email,
    ]);
}
...
}

Steps To Reproduce

Install a new laravel application
Add Model::shouldBeStrict() in boot method of AppServicesProvider
In users table, rename the 'name' column to anything (like first_name)
Install pulse + run migrations
Visit pulse endpoint

Solving our own issue, I just found in the docs that we can provide our own users resolver:
use Laravel\Pulse\Facades\Pulse;

Pulse::users(function ($ids) {
    return User::findMany($ids)->map(fn ($user) => [
        'id' => $user->id,
        'name' => $user->name,
        'extra' => $user->email,
        'avatar' => $user->avatar_url,
    ]);
});


Add dhcp status metrics collector

This collector calls NSX DHCP api group to get the list of DHCP server,
then get the status of each of them. Statistics is out of scope,
because it contains IP pool stats that we might have to treat differently.
Signed-off-by: Giri Kuncoro<EMAIL_ADDRESS>

Address #7

On the second thought, this metric should specify that it's a status of DHCP server


How to recreate an instance avoiding the setup fee

Usually, with Terraform we can do something like this to recreate an instance (on other cloud providers):
terraform destroy
...
terraform apply

But, as warned in the README.md, since Contabo has a setup fee for the VPS, doing the above is not viable because it would incur an elevated cost:
# CAUTION:  with example main.tf you are about to order and pay a Cloud VPS instance

What is the suggested alternative in the case of Contabo to force a reinstallation of an existing VPS?
Currently, my approach is to make a small change on the cloud-init which triggers a reinstallation, but it doesn't feel right.

The only solution is to perform a state change where something changes in the instance so that it will basically perform a reinstall.


Feature request: assert function

I was trying to add a new function, assert, in be_baselib.c but soon I came across an impairment and that is: the lack of getting the current line and file from bvm structure.
From what I understand reading the source code this is not possible. Please advise.

The bvm structure is designed to be invisible to external apis, and the berry api interface operates on a virtual stack.

Please wait a moment, I will write some tutorials for this.

I made some simple instructions, please see this page: API Example. If you need any help, please tell me.

Thanks for the example but I already figured how to do that :)
What I was trying to say is this: Right now there's no way of knowing the location where a "bvalue" has been created at (file and line number). I need this information in order to create an "assert" function (on the C side), eg:
def assert(cond)
if (!cond)
  push_error(linenumber, filename)
end
end
The solution(?) I came with is to have this modified in "be_object.h":
```C
typedef struct {
#if BE_RUNTIME_DEBUG_INFO > 1
    uint16_t linenumber;
    uint16_t endpc;
#else
    int linenumber;
    int endpc;
#endif
} blineinfo;

typedef struct bvalue {
    union bvaldata v;
    int type;
    blineinfo lineinfo;
} bvalue;

And basically everytime you create a variable or modify the value of it change the the information accordingly. I would also add in structure blineinfo a const char *fname.

I don't provide such an API, but you can do it.
Please refer to "be_debug.c", this file has examples of source code information at runtime.

I will, thank you!

Calling the C function be_debug_error will terminate the program and output a runtime error message, I think it should be what you want.

Not quite, it doesn't report correctly the line number if it's called in the global space aka main function but works correctly if I call it into another function.
def a()
 def init()
 assert(1 > 2) #works fine!
end
end
assert(1 > 2) # doesn't work fine, the line number is wrong

You need trace call stack.

Do you have any progress?
I think it would be easier to assert the implementation as a C function. Or, we can implement an error function straight throw, this time will naturally display the call information.

You can commit the code to your repository and I can provide some advice.

I implemented the assert function as a C function in be_baselib.c and using the blineinfo structure added in bvalue but sometimes I don't get the right line number on which the assert function is being called on the berry side. I have to look more into it.

You can run this code:
a = 0
print(a + 'str')

Then you will get the following runtime error message:
stack traceback:
        test.be:2: in function `main`
        [C]: in ?

Is this expected?

I get the same output as you. I'll see if I can reuse some error handling logic from operator '+' and apply it to the assert function.

If an error occurs while calling the function, the call stack will be printed, like:
def func(x)
    return x + 'str'
end
print(func(0))

error message:
test.be:2: error: unsupported operand type(s) for +: 'int' and 'string'
stack traceback:
        test.be:2: in function `func`
        test.be:4: in function `main`
        [C]: in ?

If functions would take as a parameter in be_baselib.c the parser structure then error checking would be much easier to handle :)

If you call the function be_debug_error, you will get the debug message.

It will be easier, but it is not conducive to the realization of the external parser.

static int l_assert(bvm *vm)
{
    int pass = be_tobool(vm, 1);

    if (!pass) {
        be_debug_error(vm, BE_EXEC_ERROR, "Assert failed!");
    }
    be_return_nil(vm);
}

class Base1
    var b
    def init(a)
        self.b = a * 3
    end
end
class A1 : Base1
    var c
    def init(c)
        self.c = c
        super(self).init(c)
    end
    def func(a, b)
        return 1 + a + b
    end
    def -*(other)
    end
end

assert(1 > 2) # line 20 in editor

obj = A1(2)
print(obj.func(5, 6))
print("exiting from class.be")

Output:
native error: Assert failed!
: in native function
test/class.be:24: in function main
[C]: in ?

Nice!

The line number in editor does not match the one in stack trace

Emmm...

It seems that if the error is thrown in the native function, there will be problems with the stack information.

Yes!

I think the function sourceinfo may have a problem.

Isn't it related to https://github.com/gztss/berry/blob/36be542d7721283d0d7221d9af2eee4793810bd7/src/be_debug.c#L107 ?

check this https://github.com/gztss/berry/blob/36be542d7721283d0d7221d9af2eee4793810bd7/src/be_vm.c#L470

Please wait a moment, I will confirm.

Please review the new submission and this issue should be resolved.
Since the IP (instruction pointer) is not pushed onto the stack when the native function is called,
There is a problem with the calculation of the PC. This is not a VM bug, but a performance consideration. Now I have written the patch_native function to correct the IP value when I need to print debugging information.

Now it works perfectly! Well done! I was looking at IP and PC this whole time, glad you figure it out!

Do you have a more in depth explication why IP is not pushed onto the stack when calling the native function (native function == main function,right?)?

Native functions are functions implemented in C, such as print and assert.
Because there is no bytecode when calling a native function, IP is not used and there is no need to push it onto the stack. If the Berry function is called after this, the IP will be pushed onto the stack and restored after this function returns, so there is no problem.

I understand! Thanks!

If you can, please do further testing.

I will!


feat(zod): add native zod schema support

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#130  👈
#129
#128
#127
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @pevisscher and the rest of your teammates on  Graphite


Bug? js file does not load

Serilog no longer loads issues for me and i'm seeing the following error in the browser:
GET https://[REDACTED]/serilog-ui/index.6bbf471b.js net::ERR_ABORTED 404

Any clues what is going on?

@sommmen thanks for reporting that. I will look into it.

Please upgrade to version 2.2.3.

Please upgrade to version 2.2.3.

Works like a charm again, thanks!


QR per Location

Hi,
Is it possible to include qrcode on location via configuration?
Because in the documentation i only see devices/cables/racks.
As this might be handy when you create location per room or area and you place the QRcode next to the roomnumber for example.
Someone with QRcode scanner then sees what is located in this room/area when scanning and goes directly to netbox url.

#53


🆙 PyQt5 version up.

:up: Let's split PyQt5 version up Pull Request.
Originally posted by @tkoyama010 in https://github.com/pyvista/pyvistaqt/pull/51#issuecomment-696480605

I think we have multiple versions in CI now so I'll close


K ver fix

Summary
Fixed the kind runner bootup.
Implementation Notes :hammer_and_pick:
added configs/kind/kubeadm.conf to boot kind cluster without cgroup errors.
fixed scripts/cluster/create_one_node_cluster.sh to use the above config while executing kubeadm init

also, please squash the commits