[WIP] recognize .netstandard framework

Apparently, .netstandard is going to replace all dnxcore monikers as described here: https://github.com/dotnet/corefx/blob/master/Documentation/architecture/net-platform-standard.md#deprecated-monikers

I don't know if this is related, but I tried a paket update on this and it didn't resolve:

source https://dotnet.myget.org/F/dotnet-core/api/v3/
source https://api.nuget.org/v3/index.json
source https://www.nuget.org/api/v2/

nuget NETStandard.Library 1.5.0-rc3-24013

giving
C:\misc\repro-54>.paket\paket.exe update
Paket version <IP_ADDRESS>
Resolving packages for group Main:
 - NETStandard.Library is pinned to 1.5.0-rc3-24013
Paket failed with:
        Could not find versions for package NETStandard.Library on any of ["http
s://dotnet.myget.org/F/dotnet-core/api/v3";
 "http://api.nuget.org/v3/index.json"; "https://www.nuget.org/api/v2"].

I don't know if it is driver error, e.g. incorrect feeds, or that the myget feeds are not coughing up the lists of available versions, or the same as the bug being fixed here.

source https://www.myget.org/F/dotnet-core/api/v3/index.json

brings:

source https://www.myget.org/F/dotnet-core/api/v3/index.json
source https://api.nuget.org/v3/index.json
source https://www.nuget.org/api/v2/

nuget NETStandard.Library 1.5.0-rc3-24013-00

seems to work.

@forki Thanks! silly me :)

I think we will make that more robust. Nobody really knows how these new
endpoints are really meant to be used. (or at least I don't know).
On Apr 13, 2016 10:13 PM, "Don Syme"<EMAIL_ADDRESS>wrote:

@forki https://github.com/forki Thanks! silly me :)
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/pull/1605#issuecomment-209626872

I removed the [WIP] prefix - I think we can merge it and fix whatever comes next

I added couple of other things to this, but Paket 2.61 now works with .NETSTANDARD. At least with Kestrel in https://github.com/CumpsD/kestrel-classic/pull/1

Cool thanks!

would love to hear if it works for you.

Will see in next days but hopefully it does :+1:

checked my use case - works fine. The remaining concern is indeed the unofficial prelease source feeds, which are required to pull down latest packages. Eventually the packages are to be published to the official feed, but till then it would be good to somehow guide users to work with prerelease feeds - not sure how to do this though

what concern do you mean? Adding endpoint to deps file should be enough, right?

Yes its enough but just as you stated above it's not obvious at once which feeds should be added


Should the list of Wallets Supported include only Apps, or LN wallets from websites as well?

It looks like some sites where the user has a LN wallet are now also adding Lightning Address support.  For example. every account nym on Stacker.News gets a corresponding Lightning Address (e.g., @Stacker.News ).
Would these get added to the list of Wallets Supported?
Or should websites providing Lighting Address support for their users be a separate list from the existing list which consists of LN Wallet apps (e.g., BlueWallet) and LN wallet services (e.g., @LNTxBot on Telegram)?
Now that I think about it, this might not just apply to websites with Lightning Addresses for their users.  Let's say some mobile gaming app uses LN and each player gets a Lightning Address, ... kind of would be the same issue.

Looks like these are reference to #services on fiatjaf's lnurl repo:
Services
https://github.com/fiatjaf/lnurl-rfc#services

Yeah this is definitely a place where we can improve the separation here. Services !== Wallets, I agree with you.
I could see possibly see refactoring things to relate this change, but the distinction becomes blurred very quickly. Services with Wallets have almost (if not all) of the functionality of Wallets. Some Services even have mobile apps which allow for the same capabilities.
Maybe we need a broader term: Products and Services supporting Lightning Addresses and change the UI of it a bit? Not sure how to proceed tbh

PRs are welcome. Going to close this for now.


remove rebuilds and buildconfigs

Signed-off-by: Robert Cerven<EMAIL_ADDRESS>Maintainers will complete the following section

[ ] Commit messages are descriptive enough
[ ] Code coverage from testing does not decrease and new code is covered
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Changes to metadata also update the documentation for the metadata
[ ] Pull request has a link to an osbs-docs PR for user documentation updates
[ ] New feature can be disabled from a configuration file

tests are failing because coverall is down

Can we have description in commit message why pls?

Can we have description in commit message why pls?

added

rn updated


[quill ] update to v.1.6.3

Describe the pull request

What does your PR fix?

Required for #18581

Which triplets are supported/not supported?


fix(msrv): Put MSRV-aware resolver behind a config

What does this PR try to resolve?
This is a part of #13540 which is a party of #9930.
The config is resolver.something-like-precedence with values:

something-like-maximum (default)
something-like-rust-version

This is punting on the actual config schema so we can implement
package.resolver and edition = "2024" support as we want the
MSRV-aware resolver available without cargo_features.
How should we test and review this PR?
One of the included test cases shows a bug with cargo install.  Resolving that will be tracked in #9930
Additional information

Thanks!
@bors r+

:pushpin: Commit 18a5940046d737959a3ab2483d991a60b243058c has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 18a5940046d737959a3ab2483d991a60b243058c with merge<PHONE_NUMBER>f0ff36067af698eaa934beb9cd98e3...


Add support for onError resource and implement error types for Artemis connector

Purpose
Add error types and add support for onError resource for Artemis connector
Tasks

[x] Add Artemis error type
[x] Add dispatching errors to onError resource
[x] Add validations to the compiler plugin
[ ] Add compilation tests
[ ] Change and test BBEs
[ ] Enable and add integration tests
[x] Refactor Artemis native function implementations

Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[x] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #16388 into master will increase coverage by 1.63%.
The diff coverage is 9%.

@@             Coverage Diff              @@
##             master   #16388      +/-   ##
============================================
+ Coverage     11.25%   12.89%   +1.63%
  Complexity     2969     2969
============================================
  Files          1500     1495       -5
  Lines         82798    72275   -10523
  Branches       9696     8707     -989
============================================
- Hits           9318     9317       -1
+ Misses        72555    62030   -10525
- Partials        925      928       +3

Impacted Files
Coverage Δ
Complexity Δ

...ng/messaging/artemis/externimpl/consumer/Stop.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ng/messaging/artemis/externimpl/producer/Send.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...saging/artemis/externimpl/message/GetProperty.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...g/messaging/artemis/externimpl/producer/Close.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...messaging/artemis/externimpl/consumer/Receive.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ssaging/artemis/externimpl/message/GetPayload.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...saging/artemis/externimpl/message/PutProperty.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...saging/artemis/externimpl/message/Acknowledge.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...messaging/artemis/externimpl/session/IsClosed.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...mis/externimpl/consumer/CreateServiceConsumer.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

... and 22 more

Continue to review full report at Codecov.

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


Import/Export

Allow migrating between instances with import and export.
This could also allow members to export their information

This is now supported using backup/restore.
It relies on the django dumpdata and loaddata commands - it has not been very well tested and will almost certainly fail if you try migrating between different versions of the software - but as a first cut it'll solve the current problems.


URL based addressing

What: Serve websites from Swarm, using RNS/ENS domain names
Sub-stories:

Amend manifest to allow for serving websites in browser
Be able to connect Swarm node to Ethereum node and use ENS/RNS

This is redundant, given that we have #153 and #154


[deny list v2] Charge for gas on receiving coins

Description

Charge for gas based on the number of config objects loaded (estimated worst case)

Test plan

TODO the current gas charging is too low to be picked up by existing tests

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:

looking and adding @tzakian to the convo


Updated Bumpers and Broadswords MAXIMUM to latest version

Updated the game.droid to the one from the latest update of Bumpers and Broadswords MAXIMUM.

Updated the game.droid to the one from the latest update of Bumpers and Broadswords MAXIMUM.

Was this also tested?

Yes, but I ran into some issues. Copied from my Discord post:

So I decided to test this (why I didn't do that BEFORE submitting the pull request) and it crashed. I just used the files from the cloned repository I have on my hard drive, so it should be exactly the same as the one on Github with the new game.droid.

I included the Log file in my Discord post. What's strange is that if I just did a fresh install and then replaced the old game.droid with the one from the new version it worked fine.

As I said before, the game.droid from the update works.


Generic IdP Configuration

For those that run an IdP, such as IdentityServer, is there a possibility of creating an ability to use a JWT token and/or OpenID Connect type token to do a sign-on to the system rather than using Google or a username/password with this system?  It seems like the system is very close as-is, but am unsure.
If guidance could be given, I would like to at least research out the possibility of creating this module.

As an extension of #145, OnlyOFFICE could implement a generic, branded IdP that can link to multiple social accounts (google/facebook/twitter/etc) and give the onus to the customer of OnlyOFFICE that, should their specific requirements be different than what OnlyOFFICE could support, the customer could develop their own solution to the problem.
This, of course, works great for self-hosted scenarios.  Further analysis would need to be performed to determine what changes would be required to support the hosted model with OnlyOFFICE.

If we were to implement this, using something like IdentityServer, how best is it to communicate how to configure CommunityServer to utilize the SSO solution?

It seems like 10.0.1 supports Bearer auth, which may be a way to offload the captcha work for implementation by a 3rd party.  I realize that it may still allow for a DDos attack on the OnlyOFFICE system, but there may be a way to rate-limit with a few lines of code, etc.

Closed for a reason - there has been no activity for a long time.


XP customization not working

In GitLab by @Asacolips on Apr 25, 2022, 14:35
Notes
From Rant in the Foundry Discord:

Having a weird issue in the Dungeon World system with level-up XP. I'm currently playing Homebrew World but wanted to change the amount of XP needed for an advancement. In HBW, the default is 5 per level. This works just fine if XP Formula (in System Settings) is set to 5. But any time I try to change it to something that's not just a constant, it disregards whatever I've put there and treats it as the default Dungeon World value (what amounts to @attributes.level.value + 7. Basically I want it to be @attributes.level.value + 4 but that doesn't work. I've tried other variations ((@attributes.level.value + 4) or 4 + @attributes.value.level) but it disregards those. Referencing that value (@attributes.level.value) with rolls works fine, it's just the XP formula part.
One thing I did notice is that if I have that formula in there, whenever I click the + or - XP button the char sheet, a console warning pops up about Roll#evaluate becoming asynchronous. That doesn't happen if the field is just 5 (HBW default). Not sure if that has anything to do with it though.

Acceptance Criteria

[x] Resolve issue with custom XP formulas
[x] Resolve Foundry v9 warnings

In GitLab by @Asacolips on Apr 28, 2022, 20:16
marked the checklist item Resolve issue with custom XP formulas as completed

In GitLab by @Asacolips on May 29, 2022, 22:18
mentioned in commit 51d5c052a77ecf2c00df98833da3629b5501b760

In GitLab by @Asacolips on Apr 28, 2022, 20:24
This has been merged and will be included in the next release of the system.

In GitLab by @Asacolips on Apr 28, 2022, 20:18
marked the checklist item Resolve Foundry v9 warnings as completed


refactor: fallible conversion

Refactors to allow fallible conversion, required simply to eliminate the unimplemented!() statements in the runtime when converting to a versioned result/error.

Codecov Report
Attention: Patch coverage is 52.17391% with 11 lines in your changes missing coverage. Please review.

Project coverage is 42.65%. Comparing base (206757b) to head (dc0f90a).

Files with missing lines
Patch %
Lines

runtime/devnet/src/config/api/versioning.rs
0.00%
6 Missing :warning:

pallets/api/src/extension.rs
0.00%
4 Missing :warning:

extension/src/functions.rs
87.50%
0 Missing and 1 partial :warning:

@@             Coverage Diff              @@
##           daan/api     #275      +/-   ##
============================================
+ Coverage     42.59%   42.65%   +0.05%
============================================
  Files            47       47
  Lines          4139     4145       +6
  Branches       4139     4145       +6
============================================
+ Hits           1763     1768       +5
  Misses         2319     2319
- Partials         57       58       +1

Files with missing lines
Coverage Δ

extension/src/mock.rs
95.23% <100.00%> (+0.11%)
:arrow_up:

extension/src/functions.rs
92.33% <87.50%> (-0.29%)
:arrow_down:

pallets/api/src/extension.rs
48.33% <0.00%> (ø)

runtime/devnet/src/config/api/versioning.rs
60.00% <0.00%> (ø)


Roll Flutter from 2303f42250b1 to<PHONE_NUMBER>de (11 revisions)

https://github.com/flutter/flutter/compare/2303f42250b1...9996126740de
2023-02-09<EMAIL_ADDRESS>Add proper disabled values for input chips (flutter/flutter#120192)
2023-02-09<EMAIL_ADDRESS>Update gallery.dart (flutter/flutter#120366)
2023-02-09<EMAIL_ADDRESS>Manual roll Flutter Engine from 2f2e2e27cb28 to eb346ba63f69 (7 revisions) (flutter/flutter#120364)
2023-02-09<EMAIL_ADDRESS>Roll Plugins from f59c08db3f27 to 73986f4cc857 (5 revisions) (flutter/flutter#120362)
2023-02-09<EMAIL_ADDRESS>Manual roll Flutter Engine from 89c8a1393d4b to 2f2e2e27cb28 (3 revisions) (flutter/flutter#120333)
2023-02-09<EMAIL_ADDRESS>Fix widget inspector null check (flutter/flutter#120143)
2023-02-09<EMAIL_ADDRESS>Revert "[web] Move JS content to its own .js files (#117691)" (flutter/flutter#120275)
2023-02-08<EMAIL_ADDRESS>[Tool] [Windows] Output build duration (flutter/flutter#120311)
2023-02-08<EMAIL_ADDRESS>Update .cirrus.yml (flutter/flutter#120315)
2023-02-08<EMAIL_ADDRESS>Manual roll Flutter Engine from cc4ca6a06ab3 to 89c8a1393d4b (6 revisions) (flutter/flutter#120319)
2023-02-08<EMAIL_ADDRESS>Reland "Stop recursively including assets from asset directories" (flutter/flutter#120312)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC rmistry@google.com,stuartmorgan<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


[bug] Wrong CUDA_VERSION for hipsparseSpVecDescr_t and hipsparseDnVecDescr_t

The corresponding to hipsparseSpVecDescr_t and hipsparseDnVecDescr_t  cuSPARSE APIs cusparseSpVecDescr_t and cusparseDnVecDescr_t appeared in CUDA 10.2, not in CUDA 10.1.
https://github.com/ROCmSoftwarePlatform/hipSPARSE/blob/develop/library/include/hipsparse.h#LL8777C1-L8783C7:
/* Generic API opaque structures holding information */
#if(!defined(CUDART_VERSION) || CUDART_VERSION >= 10010)
typedef void* hipsparseSpVecDescr_t;
typedef void* hipsparseSpMatDescr_t;
typedef void* hipsparseDnVecDescr_t;
typedef void* hipsparseDnMatDescr_t;
#endif

Hi @emankov. I am just skimming through the cusparse documentation and it appear that both cusparseSpVecDescr_t and cusparseDnVecDescr_t exist in the 10.1 version. Ill take a look at the actual cusparse shared library and see if the functions cusparseCreateDnVec are actually present or not and report back.

@emankov Here is what I have determined so far:
cuSPARSE actually has three versions for 10.1, namely:
10.1
10.1 update 1
10.1 update 2
as can be seen here https://developer.nvidia.com/cuda-toolkit-archive
Now for 10.1, cusparseSpVecDescr_t and cusparseDnVecDescr_t are not defined and the shared library does not contain routines using these descriptors like cusparseCreateSpVec, cusparseCreateDnVec etc. In 10.1 update 1 and 10.1 update 2 the shared library does have them.
Now for all three versions of 10.1, the CUDART_VERSION define is 10010. This define is what we use to determine cusparse backend compatibility. So what happens is if you have cusparse 10.1 installed then you get compile errors. If you have cusparse 10.1 update 1 or cusparse 10.1 update 2 then you should be fine.
I am in conversions about how to resolve this issue and will report back once I have a solution.

Starting from the CUDA_VERSION >= 10020, the separate versioning of the CUDA libs including CUSPARSE_VERSION appeared. I use it together with CUDA_VERSION in some cases related to CUDA Update X differences, for instance:
#if CUDA_VERSION >= 12010 && CUSPARSE_VERSION >= 12100
  // CHECK: rocsparse_spmv_alg SPMV_SELL_ALG1 = rocsparse_spmv_alg_ell;
  cusparseSpMVAlg_t SPMV_SELL_ALG1 = CUSPARSE_SPMV_SELL_ALG1;
#endif

But cases with CUDA_VERSION < 10020 need a different solution.

Options other than the #if <version> are:

version.txt file, which was shipped till CUDA 10.2 (inclusively)
nvcc --version with the subsequent grabbing of the X.Y.X version

After some thought I think I am going to try actually read from the cusparse shared library itself and check if the symbol exists or not.
The issue with reading from the version.txt file (or more recently the version.json) or checking nvcc is that neither one of these is strictly required for using hipsparse so I don't know if we can guarantee their existence. I don't want to have the situation where we sometimes are able to find the version.txt or nvcc on some users machines and sometimes not on others. This could create an inconsistent user experience.
Ill report back if my efforts to read from the shared library are successful.

Fixed in up coming 5.7 release.


Create libavoidrepitition.c

this is where the code for the functions lives

Kindly refer to #5  commentry.

It seems very reasonable to separate. But I've tried it as MinGW and MSVS - trust me it doesn't matter much.


Add AtorchTemperatureController

First try #206
Not sure if this device is generic or not

Thanks for the contribution @Poil !


Remove sync calls

Turn off whitespace in the diff to understand the change.

I'm going to just release this as is and we can sort out issues in subsequent PRs.


cmd/gofmt: add blank line between functions

What version of Go are you using (go version)?

$ go version
go version go1.13.3 linux/amd64

Does this issue reproduce with the latest release?
Yes, it does.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/wdc/snap/code/common/.cache/go-build"
GOENV="/home/wdc/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/wdc/go"
GOPRIVATE=""
GOPROXY="https://goproxy.cn"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/wdc/文档/algorithms4/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build918630619=/tmp/go-build -gno-record-gcc-switches"

What did you do?

gofmt this code
package main
import "fmt"
func add() {
}
func main() {
	fmt.Println("gofmt")
}

What did you expect to see?
package main

import "fmt"

func add() {
}

func main() {
	fmt.Println("gofmt")
}

What did you see instead?
package main

import "fmt"

func add() {
}
func main() {
	fmt.Println("gofmt")
}

Thanks for the request.
This is a dup of  #22337, and the final decision was to not do this (see https://github.com/golang/go/issues/22337#issuecomment-337943177).
Closing here as a dup of #22337.

Thank you!


[lang] Add JavaScript

Current Behavior

This project doesn't have JavaScript language, not even in the pull request I find something relates fs!
Desired Behavior

JavaScript language must be consider inside lang folder.
Implementation

The way of implementation listed on README.md, section consisting contribution guidelines.

Contributor Guides and Resources

🛠 Reference for building
📚 Instructions for contributing to documentation
:exclamation: Code of Conduct
🙋🏾🙋🏼 Questions: Discord discussion server

Thanks for this, happy to be a part of the same!
Super excited!! I'll be starting by today :)


Add config option so Jade AMD can specify dependencies

So I was having some trouble with the way grunt-contrib-jade wrapped up the jade templates as an AMD module. The plugin currently wraps using
define(['jade'], function(jade) { if(jade && jade['runtime'] !== undefined) { jade = jade.runtime; }

but it would be nice to be able to specify any requirements of the templates like
define(['jade', 'moment', 'another'], function(jade, moment, another) { if(jade && jade['runtime'] !== undefined) { jade = jade.runtime; }

That way I can references things like moment inside my jade templates without using the global namespace -- something that moment has deprecated and will be removing in future releases.
My suggested change is that the plugin adds another config option called amdDeps (totally open to a name change suggestion but I can't think of anything better without creating a breaking change to the config)
compile: {
  options: {
    amd: true,
    amdDeps: [
      'moment',
      'another'
    ],
  }
}

I added two tests to show that my changes work -- one might have done but I kinda wanted to prove the point.
I haven't updated the documentation in this PR but if it seems like a change that makes sense to add into the plugin then I'm more than happy to add documentation changes in too.

I can't review this since I'm not an actual require.js user...
@gruntjs/owners Could anyone help reviewing?

@shinnn Any update on this?

@Specialkbyte Still cannot review this.
@tkellen, sorry to bother you. I'd like you to review this PR.

This is great, I was fixing the same, I'm trying to import my i18n module... Jade is using a global i18n by now.
I hope you merge it soon, will be very useful.

@felquis Still needs some fixes https://github.com/gruntjs/grunt-contrib-jade/pull/133#discussion_r25124875 https://github.com/gruntjs/grunt-contrib-jade/pull/133#discussion_r25124877, at least.

Can I push these little fixes?

@XhmikosR Can you review this PR? As I said in https://github.com/gruntjs/grunt-contrib-jade/pull/133#issuecomment-72351124, I'm not familiar with RequireJS.

I can't review it either, @shinnn

Can I push these little fixes?

@felquis Sure, if you can push them to Specialkbyte:amd-deps.Contact @Specialkbyte to access his fork.

@felquis please fork and just send a new pull request to update this. then we will merge it.


[Patterns] Getter is not invoked in case of Object pattern and wildcard

The following test fails now on dart2js, dart2wasm-hostasserts-linux-x64-d8, dartdevk, and dartk configurations. The code is
String log = "";

void logger(String s) {
  log += s;
}

main() {
  var Square(area: _) = Square(2, logger);
  Expect.equals("Square.area", log);
}

Getter Square.area adds "Square.area" to log when called. Test fails with
Expect.equals(expected: <Square.area>, actual: <>) fails.
It means that getter area is not called at all. According to the spec it should be called but its value should be ignored.

2.  Otherwise, for each field `f` in `p`, in source order:

    1.  Call the getter with the same name as `f` on `v`, and let the result
        be `r`. The getter may be an in-scope extension member.

cc @eernstg

Yes, my expectation is also that the relevant rule is 'the getter is called, but its value is ignored'.

Perhaps there is a source of confusion here: I seem to remember that there was some debate about not invoking operator [] in cases like a map pattern containing a someKey: _, because it's sufficient to call containsKey. However, we don't have anything like containsKey with an object (we can tear off a non-operator method in order to see whether it exists, but for a getter we can't check that it's there without invoking it).
Found https://github.com/dart-lang/language/issues/2671: Contains discussion about this topic, has not been settled.
Found https://github.com/dart-lang/language/issues/2634: Same topic comes up, not settled.
This means that we haven't settled whether the wildcard pattern will actually retrieve the value (and then immediately discard it) when the enclosing pattern is a listPattern or mapPattern.
For the objectPattern I think it has been settled: The getter must be invoked.

Yes, I think the getter should be invoked. If you don't want to call the getter... don't write it in the object pattern. :)


Add docs

Reviewer checklist

[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage remained the same at 98.0% when pulling 4794b9cbd1743ef1072d98a4617c355c5bea67e9 on docs into d2421c3f8c9f045704577d1585d565e74334c003 on main.


Dockerfile.rhel7 cleanup for hybrid core/plugin install behavior on a…

…pi.ci
/assign @bparees
@smarterclayton fyi

@bparees made a good point during devex arch ... specifically, the FROM registry.access.redhat.com/openshift3/ose-cli in Dockerfile.rhel7 should be changed to the new UBI equivalent
The current setting does not matter for api.ci per se (it is substituted there) but for building locally it appears to be prereq
For either this PR or a subequent one

@bparees and I off line agreed changing the FROM and sorting through disabling repos is a subsequent PR

ok tests look good

/lgtm


CLA for pull requests

Contributor License Agreement (CLA) for pull requests are mandatory going forward.
If contributors don't agree to the CLA then the PR must be rejected.

How i can agree to the CLA? Is a comment from me in the PR sufficent or will be there a agreement on the mainpage of the github project as form, that we must sign?

We are implementing a CLA process for tracking contributors. I will provide more information when that is complete.

Ok, thanks for your quick response :)

Update here, we have a process defined. Waiting on some administrative things to get done before we implement it.

CLA workflow is implemented:
https://github.com/winsiderss/systeminformer/pull/1308
https://github.com/winsiderss/systeminformer/pull/1309


Implement secure passphrase handling on the sign command

The passphrase should be stored in either a password manager or users head.
It should only be prompted for at last possible moment and then purged from memory when used.
Also: remove TODO, near def sign

According to my research the only way t o do it sensibly is implement an end to end flow of credentials (so: acquisition, crypto operation, purge) on a platform that allows fine grained memory control (I'm thinking rust or zig) and embedding that into dtcli

Out of scope


FerretDB document defaults, PostgreSQL default URL, disable telemetry

Description of changes

Move the default settings into options, so that they are documented in the options list
set a default value for the PostgreSQL URL, to provide a nice default if the end user wants to use the PostgreSQL backend
Disable telemetry by default

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

cc @JulienMalka @camillemndn

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

Friendly ping: @JulienMalka @camillemndn

FerretDB developer there. Why disable telemetry by default?

@AlekSi hey! It's kind of an unwritten rule inside of nixpkgs to disable telemetry by default. Some of our users expect it.

Running the sqlite test, I get:
The option nodes.machine.services.ferretdb.settings' is used but not defined.`


HSEARCH-4727 OutboxEvent "payload" column created as blob(255) with DB2

HSEARCH-4727 OutboxEvent "payload" column created as blob(255) with DB2
HSEARCH-4732 Run tests against DB2 and DB2 JDBC driver version <IP_ADDRESS>

I just updated the version of DB2 and its JDBC driver in the hope that this will workaround the (apparently client-side) deadlock we've experienced in the last build...


Dropdowns Not Showing In Some Cases

E.g. https://translate.google.com/ ...the dropdown to choose language doesn't show. While this is clearly not a select box, something else is clearly wrong here like js mouse events?

@cretz I can't reproduce this here. Clicking the arrow to the right of "Detect language" brings up the box of all the languages, and I can interact with it.

@terinjokes - Yeah, it's working now for me too, but I still get this intermittently. As for many of these, it seems to not have an easy replication path. I'll leave this open for now hoping someone can give me steps to replicate (or maybe, after long enough, we can say this is no longer an issue).


Allow to open only the selected notebooks / tags / search results in the graph

Good day!
This is a great plugin! Thanks for giving your time to the community!
It would be great if we could open only the selected notebook/notebooks and/or tag/tags and/or search results in the graph view. That would allow to scope/slice things down in big databases and to have different unrelated projects in one database. Not exactly Zettelkasten way, but a flexible one)

Hey, this is a great idea and something I'm realizing I really want too!
I'm picturing a little toolbar at the top of the graph area that let's you set up some filtering of notebooks/tags.  If you have any UI ideas, definitely feel free to share a picture because I think it requires a little thought so that it isn't too complicated.

Another (maybe additional) way to do it is adding an item to the right-click menu on the notebooks/tags in the sidebar and the search bar in the notes list. Something like "Show in node graph". And to indicate somewhere in the graph area what selection is currently showing.

@ivangretsky I'm not sure if modifying the right-click menu in the sidebar is possible for a plugin to do.
For now, I added a setting where you can add comma-separated list of Notebook names to filter out.  I started playing around with something nicer (like a Tree with checkboxes for notebooks to include or exclude) but i remembered I don't know how to HTML/JS/CSS and I resorted to this as a temporary bandaid.
v0.9.0 should show up in Joplin any minute and will have the Settings UI for this.

I'm not sure if modifying the right-click menu in the sidebar is possible for a plugin to do.

Should we ask Laurent about this?

Hi,
would it be possible to add a function to recursively exclude notebooks? When I add a notebook to the comma-separated list, only the contents of that specific notebook are hidden from the graph, all the sub-notebooks still show up.
As I have a hierarchical notebook structure, it would be tedious to write down hundreds of notebooks into the comma-separated list...
Thanks in advance!

I'm going to add in the recursive filtering option this week (hopefully)

I just cut a new version v0.9.7 that will have the option to recursively filter out all sub-notebooks enabled by default.  It will be available as soon as Joplin picks up the new version from NPM

Cool!
I've updated to the new version. Bit cant figure out, how to use this new feature. Could you give me a hint?

There should be a checkbox in the Graph UI section of Preferences (Open Joplin’s Preferences and the graph UI/other plugin settings should be towards the bottom )

I think I found it. Thanks @treymo !

It says "filter out" notebooks. Am I right that this means exclude those from the graph? Asking this as I thought there will be an "include only" option to only show graph of one or more selected ntebooks. Is it on the roadmap?
For the "include only" option manual selection of the notebooks in the UI seems preferable (actually it seems preferable in any case))) Do you think it will be possible to implement some kind of UI for this other than in the plugin settings? The best would be to just right click select all the notebooks one needs, or at least have the input next to the graph.

That's correct, right now it's exclusion instead of inclusion.  Yes, I left this issue open because I do plan on adding an "include only" version of this setting.
I agree but it's a much larger development task - especially for me who doesn't work with UI/JS development professionally. If you want to provide some UI mock-ups or something, maybe someone else or I can try to tackle the UI improvements down the road.

haha, It's clear enough.  Maybe I'll try creating a little toolbar up top that has this filtering as one of the options.
I also notice that I really should support other languages.... I'll see what i18n support looks like in Joplin plugins

I am trying to filter out unnecessary notebooks in the plugin settings. Why doesn't it work. I see all notes in graph. What am I doing not atk?

Why doesn't it work. I see all notes in graph. What am I doing not atk?

It is hard for me to know from just the screenshot.   You need to make sure the names in that list have a comma between them and matches the notebook name exactly.  For example:  Are you sure its "01.TODO" and not "01. TODO" (there is a space in the second one vs no space in the first one and your screenshot)?


feat(core): implement transaction lifecycle hooks

Current lifecycle hooks cover entity and unit of work events, but
transaction-related ones are missing. Add beforeTransactionStart,
afterTransactionStart, beforeTransactionCommit, afterTransactionCommit,
beforeTransactionRollback, and afterTransactionRollback
Closes #1175

Although #1175 mentions implementing "after" entity hooks post commit, they are not included in this PR as they are not needed. Refer to the UserSubscriber example class in test file GH1175.test.ts to see how to implement an ad-hoc afterCommitCreate hook using a combination of afterTransactionStart, afterCreate, and afterTransactionCommit.
Currently, all this PR is missing right now is documentation. Please let me know if you need me to do that as well, but I was thinking maybe @B4nan could lend a hand with that as he might have a better idea regarding form, style, and content.

Btw, these hooks work correctly with nested transactions. Testing is very exhaustive and covers many cases including those.

Looking good, thanks! Will have few code style hints, might change it myself as I want to fiddle around with this locally anyway to understand it better.
Also few lines in MongoConnection are not covered (probably just branches caused by the optional chaining - if so, let's just force ignore those via istanbul ignore comments).
That TransactionEventBroadcaster thing looks a bit unneeded, but will need to dig deeper to better understand the code, might be missig something now.
One thing that looks stinky is that you are using isRootTransaction only in the SQL driver, not in mongo one.

@B4nan glad to help.

Also few lines in MongoConnection are not covered (probably just branches caused by the optional chaining - if so, let's just force ignore those via istanbul ignore comments).

I will check this today.

That TransactionEventBroadcaster thing looks a bit unneeded, but will need to dig deeper to better understand the code, might be missig something now.

The idea here is that what the broadcaster is is agnostic to the connection and we could provide any extra context we want from the call site in the future. For example, we could choose to provide the unit of work when doing implicit transactions. I actually had planned to define the EventBroadcaster interface in the Connection package. I will do that today.

One thing that looks stinky is that you are using isRootTransaction only in the SQL driver, not in mongo one.

Currently mongo doesn't allow for child transactions (save points), but they might in the future and in that case checking to see if it is a root transaction or not sounds like a generic concern. Let me know your thoughts. I can easily move that to AbstractSqlConnection.

Pushed some of the changes mentioned, added tests for mongo, and fixed coverage errors.

Thank you, @B4nan.

Hi @rhyek, I was struggling with afterTransactionCommit hook because I didn't understand why uow param was undefined.
Watching the GH1175.test.ts file I realize that it only is defined in implicit transactions.
Is there a reason why uowis undefined in explicit transactions? I would love that it was defined as in afterFlush event. The thing is that I don't know if it is a constraint of mikro-orm or if it could be done.
I've created the question in SO too: https://stackoverflow.com/questions/76851480/in-mikro-orm-is-it-possible-to-track-unit-of-work-changes-in-the-aftertransacti
Thanks in advance.


JISON for AlaSQL2

We definitely need to start AlaSQL2 with smart architecture and without error and bugs inside!:)

Inputs and ideas on how to structure the parser for AlaSQL2

Perhaps trying to decouple the parser from the compiler would be a good idea. I tried to make an inline string compiler using Babel but couldn't because SELECT statements can not be de/serialized.

What if you use the output from alasql.parse? Does that help?

Sort of.
More interesting is if I use alasql.compile
compileSelectStatement.prototype.constructor.query.selectfns

It seems to have a stringified version of the function, but I need to make sure I include the right scope.

Closed as its put on The Long List - only to be reopend when its time has come.


Externalize Active MQ storage directory in application-context.xml

In mdw-common/src/META-INF/mdw/spring/application-context.xml, change the hardwired amq:broker dataDirectory to use property syntax like the values in the MDWDataSource bean.  The default dataDirectory when no property is specified should be the existing relative location (../activemq-data).

New properties:
activemq:
location: activemq-data   # Has to be relative to startup directory - default=../activemq-data
port: 61619   # default=61618
maxConnections: 10   # Max connections in JMS pool - default=8
Unfortunately, there's a problem with Spring Expression Language and the activemq schema that means the expression cannot be the first thing in the value, meaning the following doesn't work:
dataDirectory="#{T(com.centurylink.mdw.config.PropertyManager).getProperty('mdw.activemq.location', '../activemq-data')}">
But it does work if you add a leading space to it (but then the specified path doesn't work because it starts with a space), so instead, we're specifying ./ before the expression like this:
dataDirectory="./#{T(com.centurylink.mdw.config.PropertyManager).getProperty('mdw.activemq.location', '../activemq-data')}">
This means that the specified value via property has to be relative to the startup directory of the application.   This is still better than having to override the entire application-context.xml file


Component Token Naming

In the component token layer (currently exported as one sass file with two maps), components are not consistently named. Notifications and Buttons are two components, but Button tokens are nested in a list mixed with other interactive components while Notification tokens live in their own list.
/** Content of the maps is shortened for demo purposes */
$post-interactives: (
  'post-comp-interactive-color-primary-bg-enabled': var(--post-sem-color-interactive-primary-enabled-bg),
  'post-comp-interactive-button-primary-color-fg-enabled': var(--post-sem-color-interactive-button-primary-enabled-fg),
);

$post-notifications: (
  'post-comp-notification-color-bg-badge': var(--post-sem-color-notification-badge-bg)
);

Potential issues
This approach makes it hard for developers looking at the exported sass files to determine what components the tokens should be used for (are Tabs "interactive"?). While it's possible to look up tokens used in Figma Dev Mode, it's not a guarantee to catch all tokens, especially in deeply nested components. There's no overview of what tokens are used for this component when the outer layer is selected. With the Figma approach, it's easy to miss tokens.
The danger of component tokens that need renaming is also higher when we use shared component names. A renamed component token will most likely be a breaking change for development because they will be documented in Storybook (adding more component tokens would be features, removing component tokens would be a breaking change also).
Proposed solution
Component tokens are the most important interfacing point between Figma and Code. Therefore naming and consistency are most important in this area.
Consistent naming
The naming schema should be consistent across the whole token system.
// Component layer: [namespace] [layer] [group] [component] [variant] [property group] [property] [state]
// Semantic layer:  [namespace] [layer] [property group] [group] [component] [variant] [state] [property]
'post-comp-interactive-button-primary-color-fg-enabled': var(--post-sem-color-interactive-button-primary-enabled-fg),

Naming schema
For component tokens, especially the development will profit from short, memorable token names. Unnecessary categorization should be omitted. The layer naming (-comp-) is good to have an overview in the token system, but is irrelevant to end users of tokens because the only thing they should interact with are the component tokens. As for the grouping of component tokens, it does not provide any benefit if every component has it's own collection of tokens.
/* Comp interactive could be left out */
--post-comp-interactive-button-primary-color-fg-enabled

--post-button-primary-color-fg-enabled

Sass output
Ideally, each component will generate a list of tokens available for this component. This makes it possible to iterate over existing tokens not only in the Styles, but also in the documentation. If lists with tokens for multiple components are used, it's not possible to assign the tokens used to a specific component.
$post-textfield: (
  'post-comp-interactive-textfield-primary-color-bg-enabled': var(--post-sem-color-interactive-primary-enabled-bg),
);

$post-button: (
  'post-comp-interactive-button-primary-color-fg-enabled': var(--post-sem-color-interactive-button-primary-enabled-fg),
);

$post-notifications: (
  'post-comp-notification-color-bg-badge': var(--post-sem-color-notification-badge-bg)
);

Proposal accepted, need dev support for renaming.

Added new state on the production board and moved affected tickets to this column. Moving to discussions for documentation purposes.


melt model support for isolation.

Description
Adding support for solution isolation for the melt model.
Type of Change

[X] New Feature
[X] Refactor

Codecov Report

Merging #133 (2ebacdf) into main (084a1b8) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #133   +/-   ##
=======================================
  Coverage   32.34%   32.34%
=======================================
  Files          27       27
  Lines        3568     3568
=======================================
  Hits         1154     1154
  Misses       2323     2323
  Partials       91       91


How do I change body class within each page?

Does anyone know how I should go about changing body class for each html page?

Have a look here: https://github.com/luruke/barba.js/issues/51

If you have jquery, could simply insert on your loaded page:


refactor: align module request of CommonJsExportRequireDependency

Summary

Require Documentation?

[ ] No
[ ] Yes, the corresponding rspack-website PR is __

[!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

Current dependencies on/for this PR:

main

PR #5542

PR #5543

PR #5502

PR #5612   👈

PR #5588

This stack of pull requests is managed by Graphite.


add graphics 2D

see eslint warnings here: https://github.com/TurboWarp/extensions/pull/281/files
try to resolve those; they're warnings but with the new setup (implemented after you opened this), warnings will cause pull request checks to fail

you should rename some of these to be more clear to the average user:

it's not obvious that "line section" means length
it's not obvious that "square" actually refers to "area"
it's not obvious that "round" refers to a circle with a given radius or diameter


Artwork Issue

What version of Music Assistant has the issue?
2023.2.1
The problem
When viewing Albums with Firefox, some Album artwork doesn't appear.  Using Chrome, it works fine.  This happens on different PC's.
How to reproduce
Switching between browsers produces the same result.
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2023.2.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Could be a caching issue but a new front end is coming with V2 so wait until then and see what happens.

Thanks!  I will wait for V2 and see if it fixes the issue.

Closing. I can reopen if this is still an issue with MA V2


AlignAndDistribute with Group

I'd like to draw a draggable Rectangle with some Text in it. I want to able to align these Rectangles with guides.
I'm trying to get AlignAndDistribute working with Groups and it seems like that isn't possible. A Group is not a Shape, so you can't add it to AlignAndDistribute. My attempts to fake a Shape/BoundingBox didn't work well.
If you pick one of the child shapes of the group (e.g. the Rectangle...) as an alignment bounding box, dragging a group doesn't trigger the attribute handlers of its children.
Any ideas?

I'll look into it, AlignAndDistribute was contributed from Red Hat, and is quite complex... let me see what I can do.

The main issue is Group at the time did not support BoundingBox. It can be made to work, if we can create a common interface that Group and Shape share. That supports BoundingBox. Like BoundingBoxable ;)

It's not that difficult to construct a BoundingBox for a Group, it's the sum of all the bounding boxes of everything in the Group. The issue for AlignAndDistribute is that you generally bail out if rotation, scale, or shear is set on one of the shapes..., so the question is what to do if a component of a Group is rotated, scaled, or sheared. Then we have Group's of Group's, and it's Turtles all the way down  :-)

this is now fixed


Add tests for CLI

Followup to #598. We don't seem to have automated tests for shiny run and its args.

Should also add more (examples etc.) to the docs.


Fix grafana dashboard to work with scrape interval greater than 15s

This adds the changes done in #10936 to the Grafana Standalone Dashboard (original PR only changed it for the Kubernetes version of the Dashboard).

Hi @swiffer,
Thanks your contribution. Could you please rebase your PR on v3.1 branch?

@mmatur done ;)


error in creating a new service plan

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp create
Errors:
update_site_config() takes 2 positional arguments but 3 were given
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 608, in execute
    raise ex
cli/core/commands/__init__.py, ln 666, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 657, in _run_job
    cmd_copy.exception_handler(ex)
...
cli/core/commands/__init__.py, ln 306, in __call__
    return self.handler(*args, **kwargs)
azure/cli/core/__init__.py, ln 493, in default_command_handler
    return op(**command_args)
cli/command_modules/appservice/custom.py, ln 135, in create_webapp
    match['setter'](cmd, match, site_config)
TypeError: update_site_config() takes 2 positional arguments but 3 were given

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp create --resource-group {} --plan {} --name {} --runtime {} --deployment-local-git

Expected Behavior
Environment Summary
Linux-4.15.0-1067-azure-x86_64-with-debian-stretch-sid
Python 3.6.5
Shell: bash

azure-cli 2.0.81

Additional Context

@hassanalbudairiqts , thanks for reporting, we are looking at this ?

I also have this problem - during the follow learning lab:
https://docs.microsoft.com/en-us/learn/modules/load-balance-web-traffic-with-application-gateway/3-exercise-create-web-sites
Section "Create App Service and deploy the license renewal site"
Step 3.

Same as #12159 please use the edge builds until the fix is released in 2.0.82


Process junit xml even when full junit is not enabled

Fixes #902
#798 introduced a new argument full-junit-result. It is false by default. First problem, it is not added to keys list so flank doctor report new argument as unknown key.
Second problem is about junit xml report for smart flank run. full-junit-result and legacy-junit-result are used to decide to send junit report. As I understand from #798, there are now there junit result, legacy and full. Consideration; when both flag are true, flank can throw exception at argument checking phase.
Test Plan

How do we know the code works?

While using flank without legacy and full junit test report, smart test sharding and junit xml upload need to work.
Checklist

[x] Documented
[x] Unit tested
[ ] release_notes.md updated (Waiting for review before updating notes)

Thanks for the pull request! The team will review this soon.

I will update the release notes and merge if everything looks good. Next step is preparing fladle for Android uswr for full xml report.


NEXT-12345 - Example Issue

Description
This is just an example
Possible Solution
See example PR here: https://github.com/shopwareBoostDay/platform/pull/95
Additional context
This is intended as a guideline for a good PR/Issue

@shopwareBoostDay/shopware-design Test - Bekommt ihr eine Benachrichtigung?


KMP algorithm for string pattern matching is added.

This is a standard string algorithm, used for Pattern Searching.

Can you mark this with the Hacktoberfest label, It would help.


tests: Run the flaky tests, too

What type of PR is this?
/kind bug

What this PR does / why we need it:
Turns out that my previous patch to run the flaky tests was incomplete.
We need some more changes:
- the testify.m parameter must be, for some reason, specified last
or else the go test call fails with "no Go files in XYZ".
- after the namespaced tests (last in the regular non-flaky tests),
we need to reset the namespace or else we test everything in the
test-ns namespace
- we need to tear down the operator after the namespaced tests to
be able to roll it out again in the flaky tests
The tests are still flaky, but at least they run now.
Which issue(s) this PR fixes:
None

Does this PR have test?
implicit

Special notes for your reviewer:
N/A
Does this PR introduce a user-facing change?

NONE

Codecov Report

Merging #1153 (0f255a1) into main (7d5b719) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1153   +/-   ##
=======================================
  Coverage   50.70%   50.70%
=======================================
  Files          42       42
  Lines        4753     4753
=======================================
  Hits         2410     2410
  Misses       2266     2266
  Partials       77       77


overload feature is not working properly

Hi,
I'm unable to get the ---@overload feature to work.
This is my code:
--- Create and return a new AnimationGroup as a child of this Region.
---@param name string
---@param inheritsFrom string
---@return boolean test
---@overload fun():boolean
---@overload fun(name:string):boolean
function Region:CreateAnimationGroup(name, inheritsFrom) end

and this is the result:

You can see there are no parameters shown and I'm also not able to set a name for the return value.
The expected behavior for me would look something like this:
function Region:CreateAnimationGroup(name: string, inheritsFrom: string)
 -> test: boolean
function Region:CreateAnimationGroup(name: string)
 -> test: boolean
function Region:CreateAnimationGroup()
 -> test: boolean

Best regards

Has supported.


Default credentials for PostgreSQL

Problem description
I cannot log into PostgreSQL through adminer. The error stipulates that no such role exists - Unable to connect to PostgreSQL server: FATAL: role "root" does not exist
Expected behavior
Login with username: root with a blank as my password
I have tried:

pgAdminIII, it cannot connect to WPN-XM PostgreSQL server for me to create roles to solve the issue.
Editing the pgpsss.conf in ../AppData/postgresql according to these specifications to solve the issue.
Using psql in WPN-XM installation

C:\WPN-XM\bin\pgsql\bin>psql -U postgres
psql: FATAL:  role "postgres" does not exist

Possible solution
Proper user documentation. This is not enough.

Hello Joseph!
Please run pgsql -l first, which lists the created databases including owners.
There should be a database postgres.
The username should be the user you ran the installer with.
Alternatively, you can also find out the default username by running pgsql --help.
The name is displayed at the end of the line containg -U, --username=NAME with "default: username".
We do a very minimal setup for PostgreSQL from the installer:

it fetches the zip and extracts it into /bin/pgsql
then it runs initdb.exe --encoding=UTF-8 --pgdata=..\data to create an initial database
Then, from the Server Control Panel the following command is used:
for starting: pg_ctl -D ..\data -l logfile.txt start

Are you ok with the changes? Please give me a feedback.
Do you have any other ideas for things i need to change?

...

Thank you for the support. I found a workaround where I used my computer account's name as the username and a blank password. root username will not work. in reference to @jakoch commit:  yourdb option is not necessary in my opinion


Show event locations

I think this extension is great -- it is just missing 1 killer feature! Locations for events are super important, especially inside an office where there are dozens of undifferentiated conferences rooms. It would be SUPER helpful if you could see event location right inside the extension. Right now you have to click and that brings you to google.com/calendar, which TOTALLY defeats the purpose of the extension itself.
I would love love it you make that update. With that feature you could easily get IT departments to pay for it as well.
I'm Nathan
<EMAIL_ADDRESS>@nathanzaru here
@yes on twitter
System Information:

Version: 2.6
Language: en-US
Timezone Offset: -07:00
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.82 Safari/537.36

What steps will reproduce the problem?
1.
2.
3.
What is the expected result?
What happens instead?
Screenshot URL:
Logs:

Hello,
Maybe we can just show location next to the icon like this :

That if it is an adresse or the name of a room, you can quickly where you need to go.
Hope this can help.
Dexterneo

This ticket can be closed now.
@manastungare

I think this feature was never implemented because I cannot see locations on my screen.  It would be extremely helpful to have this functionality, as I am always cutting it close and don't know where I'm going!

In version 2.8 the extension display the location if defined on the event, make sure you have the last version installed.

OK after check, if the event has a location and a hangout_url the location is not displayed.
I will fix this part of the code


Landing Page Text Content

💡 Description
Page content that features headers and body text, as well as allows for embedded images and various link types as well as simple, non-interactive data tables.

See Airtable for additional design notes.
Implementation Phase: 2
⚔ Parent Epic / Related Tickets
No response

[x] Load description content from OpenSearch
[ ] Determine how content will be rendered in the future when curated (i.g. will the markup be separate from the content, where will the markup be maintained (in WordPress?), etc.)


feat: SEO improvements

based on #47

[x] fixes og:image not showing (needs testing) (tested in https://github.com/Eggrror404/tnfshcec-web-pages-test)
[x] uses svelte-meta-tags for SEO tags, so we probably got more of them (which is good for SEO i think)
[x] adds dynamically changing title
[ ] adds dynamically changing og:image

[x] fixes github action failure on deploying to github pages

不過 #47 已經合的還在這
是不是該重開一個PR

@HeyLoon
圖片生成現在不想做（
所以可以 review 來合併了👍


Asserts should be able to reference query variables

In the Assert tab, I can already make use of environment variables, but there is no way to reference the values from the "Variables" tab. For example, if I have a variable called "id", I'd like to be able to assert that its value appears in the body of the response. Maybe my query tries to retrieve a particular user record. I'd like to assert that the "id" value on the response matches what I asked for. This example is highly simplified, of course.
I'm not sure what that syntax would look like, though. Would it use the same double curly braces? Would the variable name need a prefix of some sort like {{vars.id}} and how would you prevent collisions with environment variables other than by saying "don't start your variables with "vars""?

Hi @MelGrubb  this works for me

Hi @MelGrubb  this works for me

Ah, I'm doing GraphQL queries and the UI is slightly different for those. There's a separate "variables" tab there. Those are the ones I need to reference in the assertions.


Adding 2022-04 REST resources

This PR adds the auto-generated resources for API version 2022-04.

It's counting most of them as renames from 2021-04!


Proxy Settings

I want to send temperature values with amqp_ws to the Azure Iot-Hub. In my private Network all works fine, but my client has a protected network and all traffic must be send over the proxy server.
I'm using a Raspberry 3B with Windows 10 Iot Core (10.0.17763.107)
How can I config the proxy settings?

In my code?
In the Dashboard?
With PowerShell?
I have no Idea.

Thanks for help.

@mgrontzki you might want to file this bug under this repo: https://github.com/Azure/azure-iot-sdk-csharp/issues/

I changed the repo. Thanks


Issue about update cell's attribute without construct a new one.

My code here
import ipysheet
mysheet = ipysheet.sheet(rows=2, columns=2)
mysheet
c = cell(0,0,1, background_color = 'red')

I want to change background_color, I expect to do something like this
c.background_color = 'yellow'

Or later
mysheet[0, 0].background_color = 'yellow'

instead of this
c = cell(c.row_start, c.column_start, c.value, background_color = 'yellow')

Excuse me if I make a read-doc-for-me issue. But I didn't find method I expect. If there isn't, could anyone explain? Thanks

No need to apologies, I agree this is super confusing
The helper cell function for creating the Cell object takes a background_color argument but the Cell class has no background_color attribute...
Instead, there is the style attribute that you can change. So you can do:
import ipysheet
mysheet = ipysheet.sheet(rows=2, columns=2)
display(mysheet)

c = ipysheet.cell(0, 0,1, background_color = 'red')

# And later on
c.style = {'backgroundColor': 'yellow'}

I would suggest not using the helper functions and directly instantiate the Sheet and the Cell objects, adding the cell to the sheet manually with mysheet.cells = [c]. That is more complicated but also more explicit.

Thanks for your response. I think I tried use style to access backgroundColor before, but in a wrong way.
I did c.style['backgroundColor'] = 'yellow' instead of c.style = {'backgroundColor': 'yellow'}. After read some ipywidget code, looks later approach will trigger add_traits which tigger send_state which make front-end update. Could you confirm that?

Yes, that's exactly it. Ipywidgets uses traitlets internally, and traitlets
does not observe internal changes on list/dict elements. It only observes
attribute assignements on HasTraits instances.
On Mon, Sep 30, 2019, 8:31 PM Ke Zhang<EMAIL_ADDRESS>wrote:

Thanks for your response. I think I tried use style to access
backgroundColor before, but in a wrong way.
I did c.style['backgroundColor'] = 'yellow' instead of c.style =
{'backgroundColor': 'yellow'}. After read some ipywidget code, looks
later approach will trigger add_traits
https://github.com/jupyter-widgets/ipywidgets/blob/f8cad9dceeef1f911da66c995a3877ffe964e6c0/ipywidgets/widgets/widget.py#L594
which tigger send_state
https://github.com/jupyter-widgets/ipywidgets/blob/f8cad9dceeef1f911da66c995a3877ffe964e6c0/ipywidgets/widgets/widget.py#L594
which make front-end update. Could you confirm that?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/QuantStack/ipysheet/issues/146?email_source=notifications&email_token=AFBXEE624N23UU7NGHLMI73QMJAZLA5CNFSM4I3H3CMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD76UJEQ#issuecomment-536691858,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFBXEE7NAMQZGDEFZX74VCLQMJAZLANCNFSM4I3H3CMA
.


CommandQuery context type hint

Add genric type on CommandQuery to define the type of CommandQuery.context.
Put generic type on Command and CommandSet too.
Note: Because commands are loaded at runtime there is no way to make type checking. So it will be possible to load a Command<A> with a CommandSet<B>. This is only for IDE auto-completion.

implemented in 4.0.0


Deploy CF using BOSH instructions contain outdated link

https://www.pivotaltracker.com/story/show/157165712
is
https://github.com/cloudfoundry/cf-deployment/blob/master/gcp-deployment-guide.md#2-deploying-cf
should be
https://github.com/cloudfoundry/cf-deployment/blob/master/deployment-guide.md#deploy-cf

It looks like this was addressed in this commit: https://github.com/pivotal-cf/onboarding/commit/73aeba4b76043f7eaf98f6d7f0b11a5ae933ae35


Error: "read ECONNRESET" in random commands

Reported by mephisto5558<PHONE_NUMBER>25055775) with bot<PHONE_NUMBER>02710047
Error: read ECONNRESET
at TLSWrap.onStreamRead (node:internal/stream_base_commons:218:20)

seems like a network error on the host server


Reference formatting problem: encoding_opts::space_as_plus

https://www.boost.org/doc/libs/master/doc/antora/url/reference/boost/urls/encoding_opts/space_as_plus.html

I believe this is fixed by https://github.com/cppalliance/mrdocs/commit/a89e82fd1bcffb6c1cd4a48cc9422023fd727dd8


Bump JLine to 3.13.2

Command parameter completion is still missing...
@remkop, if you can add some command parameters to example I can try to implement completers.

PS. top command is working ok... what you see is asciinema problem ;)

Codecov Report

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

@@            Coverage Diff            @@
##             master     #888   +/-   ##
=========================================
  Coverage     92.84%   92.84%
  Complexity      388      388
=========================================
  Files             2        2
  Lines          5812     5812
  Branches       1512     1512
=========================================
  Hits           5396     5396
  Misses          205      205
  Partials        211      211

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 389e46f...23f195e. Read the comment docs.

@mattirn I like the asciinema a lot! This looks like a very nice PR, thank you!
Can I ask you to also update the README for the picocli-shell-jline3 module?
It would especially be good to explain what the new PicocliCommands class does and how developers can use it to enhance their application.
Perhaps the example in the README should also use the new PicocliCommands class.
Again, thank you! This looks really nice!

@mattirn I saw you pushed updates to the documentation, thank you!
I am happy to merge it as is, but I really liked the asciinema you created and I wondered if we could include those in the readme - if you are okay with that, that is...
What do you think?

I have already included the asciinema linked in this pull request in README. You are free to include also others if you like JLine demos :).

Merged into master.
Also updated the release notes.
Thank you for the contribution. Very nice!


Working on Wear Decompose

Working on Wear Decompose
TODO

Complete all Screens and navigation
Reimplement deeplinks
Remove all existing ViewModels
Reimplement a Wear Nav Scaffold (Scaffold, TimeText, PositionIndicator)

FYI @arkivanov


Use parse to format modules.

parse inlines at least any INSTANCE statement:
---------------------- MODULE SyncTerminationDetectionAP ----------------------
CONSTANT N

VARIABLES tpos, active, terminationDetected

INSTANCE SyncTerminationDetection
=============================================================================

../../apalache/bin/apalache-mc parse --output=SyncTerminationDetectionAPparse.tla SyncTerminationDetectionAP.tla
11:08:16.394 [main] INFO at.forsyte.apalache.tla.tooling.opt.ParseCmd -- Loading configuration
Output directory:specifications/ewd840/_apalache-out/SyncTerminationDetectionAP.tla/2023-12-13T11-08-16_12289306914144930095
# APALACHE version: 0.44.2 | build: 3360796                       I@11:08:16.604
Parse SyncTerminationDetectionAP.tla                              I@11:08:16.613
PASS #0: SanyParser                                               I@11:08:16.727
Parsing file specifications/ewd840/SyncTerminationDetectionAP.tla
Parsing file specifications/ewd840/SyncTerminationDetection.tla
Parsing file /private/var/folders/7d/4x6z2cc91jl588ysynlc1tfc0000gn/T/Naturals.tla
Substitution of LEADS_TO needs care. The current implementation may fail to work. W@11:08:16.911
Parsed successfully
Root module: SyncTerminationDetectionAP with 16 declarations. I@11:08:16.971
It took me 0 days  0 hours  0 min  0 sec                          I@11:08:16.971
Total time: 0.365 sec                                             I@11:08:16.971
EXITCODE: OK

-------------------- MODULE SyncTerminationDetectionAPparse --------------------

EXTENDS Integers, Sequences, FiniteSets, TLC, Apalache

CONSTANT
  N

VARIABLE
  tpos

VARIABLE
  active

VARIABLE
  terminationDetected

Wakeup(i, j) ==
  active[i]
    /\ active' = [ active EXCEPT ![j] = TRUE ]
    /\ UNCHANGED terminationDetected

Node == 0 .. N - 1

vars == <<active, terminationDetected>>

terminated == \A n \in Node: ~(active[n])

(*
  has termination been detected?
*)
TypeOK == active \in [(Node) -> BOOLEAN] /\ terminationDetected \in BOOLEAN

Live == terminated ~> terminationDetected

Stable == [](terminationDetected => []terminated)

...

This makes it impossible to use parse to format modules.

Do I understand correctly that you would like an auto-formatter to, e.g., enforce a normalized style? If so, it is true that parse is not designed with this purpose in mind.

Since we are using Sany as the parser here, I'm not sure it would make sense to try to support auto-formatting TLA+ via apalache instead of supporting that in SANY itself. I'll close this for now, but if I am missing something about the motives here, feel free to expand and we can open it back up.


Fix #876 - Improve config migration by chaining together

This implements the proposal in #876.
The migrations are now chained together so each migration is only responsible for migrating from the previous version to the next version. This means one can now simply define a migration which takes in the output of the previous migration and migrates it forward without having to modify any previous migration.
The PR also updates the 3.2.0 migration to work with the changes made in 2017.07.01 which broke the migration (due to the interfaces changing in the schema).
Each migration now should take in a string (an exception has been made for the legacy migration) and output a string. This provides a uniform interface that lets them easily be chained together (as they input and output the same things).
I have tested the migrations both mid way through the chain and from legacy. The legacy migration I tested is located https://people.igalia.com/dpino/lwaftr/lwaftr.conf with this binidng table https://people.igalia.com/dpino/lwaftr/binding_table.txt I believe it to be 20M softwires entries.

There were a few changes to previous migrations I had to make when looking at them. This is a side-effect of us keeping the same schema (i.e. the same name for the schema) and having no way for the migrations to use the schema it was actually written against. These changes are far less than otherwise would have been required under the old migration system though.
This PR also introduces unit testing which should flag up these problems when migrations are written. One further solution that would be trivial to do would be to include a schema for each version (if it requires one) which is what it was written against. This should reduce (hopefully to almost zero) any modifications to previous schemas. I have not done that right now, I don't feel like it's required and would be an easy change to make in the future if we do desired.

Looks good to me.  Please push when/if the CI goes green.

I actually took another look at util.string_output_file and it occurred to me the way it's implemented maybe the out table it builds up internally sticks around even when file has gone away. I added a method to clear it and it actually produces memory usage similar to using an external file now:
$ sudo ./snabb lwaftr migrate-configuration -f legacy lwaftr.conf
Migrating version:	legacy	(memory: 9.0425720214844)
Migrating version:	3.0.1	(memory:<PHONE_NUMBER>2526)
Migrating version:	<IP_ADDRESS>	(memory:<PHONE_NUMBER>7281)

Turns out it was easier than I had initially thought. PTAL @wingo, hopefully this is better :)


TodoReview opens an empty file, no header and nothing

Hey, I came back to this package after a while because I used to use it all the time.
However, when I try to use it now, it opens a new, uneditable and empty tab.
The status bar shows "23 files scanned", but there is no header printed in the file itself.
I already tried ignoring all other packages and even used it in a fresh install of SublimeText.
Is there anyone else who had this issue and might be able to point me in the right direction?
I'm running ST4 on a Windows 10 machine.

I've the same issue, in my case I've to run the command twice to get an answer.

Oh wow, running the command twice actually does show the results!
Thank you for that hint!
The issue persists, but I can work with that.

@ktrutmann I've a better hint: you can use https://github.com/jfcherng-sublime/ST-TodoReview, this have all fix and work perfectly (for now).

@ktrutmann I've a better hint: you can use https://github.com/jfcherng-sublime/ST-TodoReview, this have all fix and work perfectly (for now).

Oh wow, thank you so much! Works like a charm!
One detail, though: You probably copied the readme from another project and it sais to install CommandAndMenu. Instead I had to uninstall TodoReview and just reinstall it from your source.
Thanks again!

That isn't my repository, I just found it

Doesn't work on ST4, Linux as well. And invoking multiple times doesn't help.  When invoked for "open files" it shows a new tab with timestamp (why zero files when plenty of files are open?)
// Tuesday 04/16/24 at 10:23pm - 0 files in 0.07 secs
When invoked for project it doesn't open a tab or show anything.


Please design as public interface than pass RenderEditable everywhere

Thanks for flutter, we can define any extended packages base on flutter.
I'm the author of following packages:
https://pub.dev/packages/extended_text
https://pub.dev/packages/extended_text_field
I have recode something refer to RenderEditable to support WidgetSpan can be selected.
But i found it's more and more difficult to merge official code from flutter SDK.
RenderEditable is passed everywhere.
https://github.com/flutter/flutter/blob/c9d5485361ddfe5d3d04b73149537dff5b74d51f/packages/flutter/lib/src/widgets/text_editing_action.dart#L27
abstract class TextEditingActionTarget {
  /// The renderer that handles [TextEditingAction]s.
  ///
  /// See also:
  ///
  /// * [EditableTextState.renderEditable], which overrides this.
  RenderEditable get renderEditable;
}

https://github.com/flutter/flutter/blob/c9d5485361ddfe5d3d04b73149537dff5b74d51f/packages/flutter/lib/src/widgets/text_selection.dart#L376
/// An object that manages a pair of text selection handles.
///
/// The selection handles are displayed in the [Overlay] that most closely
/// encloses the given [BuildContext].
class TextSelectionOverlay {
  /// Creates an object that manages overlay entries for selection handles.
  ///
  /// The [context] must not be null and must have an [Overlay] as an ancestor.
  TextSelectionOverlay({
    required this.renderObject,
  }) :
  // TODO(mpcomplete): what if the renderObject is removed or replaced, or
  // moves? Not sure what cases I need to handle, or how to handle them.
  /// The editable line in which the selected text is being displayed.
  final RenderEditable renderObject;

it' nice to public RenderEditable as interface which method or property are used in other classes, thanks.

@LongCatIsLooong  @justinmc @YeungKC

@zmtzawqlp
Please check this comment for your reference and see if it resonates with your request.
Also, from that comment:
I'm considering exposing a widget that has a TextSpan builder (like what we have in TextEditingController) and an onChanged field (somewhat similar to _Editable but I think it would be nice to make it a public class), to avoid direct communication between the controller and RenderEditable, and also fill the missing link of TextSpan <=> TextEditingValue conversion.

@darshankawar  It's not the same request. just want to pass public interface/Mixin instead of pass  RenderEditable directly

Thanks for clarifying.

cc @GaryQian Maybe this can be gathered with #84317 ?

Thanks for the issue @zmtzawqlp, I've been thinking the same thing.  I think we have a lot of momentum to refactor some of these classes.  I'll keep this issue in mind as we try to move in that direction.


Prevent beyla self-instrumentation

Depending on how customers choose to deploy beyla, we can easily end up in a situation where Beyla is self-instrumenting itself. One example would be setting "all" for an executable name or a kubernetes pod/service/namespace. This is particularly common actually, because our default helm configuration instruments all namespaces, so Beyla will self-instrument even when it's installed in its own namespace.
Self-instrumentation comes with serious consequences, mainly because of self-perpetuating instrumentation of calls to /v1/metrics and /v1/traces. This creates unnecessary load on all components in the system and drowns the collection pipeline.
I've added an option to purposefully enable Beyla self-instrumentation, however I think we shouldn't even document it. It should be purely internal option we may choose to enable in some special circumstances.

Codecov Report
Attention: Patch coverage is 33.33333% with 4 lines in your changes missing coverage. Please review.

Project coverage is 50.99%. Comparing base (f9a8d4b) to head (f245fec).

Files
Patch %
Lines

pkg/internal/discover/attacher.go
33.33%
4 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (f9a8d4b) and HEAD (f245fec). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (f9a8d4b)
HEAD (f245fec)

oats-test
1
0

k8s-integration-test
1
0

integration-test
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1048       +/-   ##
===========================================
- Coverage   81.50%   50.99%   -30.52%
===========================================
  Files         139      139
  Lines       11113    11119        +6
===========================================
- Hits         9058     5670     -3388
- Misses       1543     5058     +3515
+ Partials      512      391      -121

Flag
Coverage Δ

integration-test
?

k8s-integration-test
?

oats-test
?

unittests
50.99% <33.33%> (-0.01%)
:arrow_down:

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

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


Slow ActionView::PathResolver#find_template_paths

I'm not sure if there is a clear fix that will work for all cases, but I wanted to bring this to your attention.
We've have really slow view loading in certain contexts. The glob that it searches for views ends up compiling to:
app/views/rodauth/login{.en,}{.html,.text,.js,.css,.ics,.csv,.vcf,.vtt,.png,.jpeg,.gif,.bmp,.tiff,.svg,.mpeg,.mp3,.ogg,.m4a,.webm,.mp4,.otf,.ttf,.woff,.woff2,.xml,.rss,.atom,.yaml,.multipart_form,.url_encoded_form,.json,.pdf,.zip,.gzip,}{}{.raw,.erb,.html,.builder,.ruby,.coffee,.slim,.jbuilder,.haml,}

I'm supposing we don't have this problem elsewhere because Rails might eliminate options here from information from routing layer, but that's just speculation.
I think somehow incorporating respond_to { html { } } around the renders may cut this down significantly.
For some members of our team (particularly macOS users where Docker's filesystem access is pretty indirect), these renders can be > 300ms.
We are on Rails 5, so perhaps Rails 6 wouldn't have this problem due to https://github.com/rails/rails/pull/33860
We've worked around this by inserting the following into our rodauth controller:
  def _render_template(options)
    variant = options.delete(:variant)
    assigns = options.delete(:assigns)
    context = view_context

    context.assign assigns if assigns
    lookup_context.rendered_format = nil if options[:formats]
    lookup_context.variants = variant if variant

    lookup_context.formats = [:html]
    lookup_context.handlers = [:slim, :erb]

    view_renderer.render(context, options)
  end

Thanks for the report. Yes, I would definitely like to add a performance patch for this for Rails 5.x. I'll try to get more familiar with the related Action View code, and see if it's possible to add a safe patch. Using respond_to might be possible.

Does this problem occur when rendering any template in Rails 5 (in other controllers)? Or just with manually created controller instances used in rodauth-rails?

Good question. It was my colleague @chendo who did the benchmarking, so hopefully he can weigh in on that, but it was my understanding from his internal PR that it was just Rodauth. He only added that _render_template call in our RodauthController

It was specifically just Rodauth. Regular Rails controllers in the same project use globs that look like this: "/app/app/views/controller/action{.en,}{.html,}{}{.raw,.erb,.html,.builder,.ruby,.coffee,.slim,.jbuilder,.haml,}"
That particular controller action has no explicit render call, so something in Rails would be limiting the extensions to just .html

After some looking, I found out that Action Controller normally limits the template formats for lookup to the ones accepted by the current request (code). This is why classic rendering inside a controller is fast even without using respond_to.
In rodauth-rails case, template rendering is happening outside of action processing, so this code isn't being executed. I've opened https://github.com/janko/rodauth-rails/pull/52 which should fix this by adding this logic. @chendo could you test and verify whether the proposed fix works for you?


[Bug]: Text is not on the same height as Icon

What happened?
The Text is always higher than the icon

Card Version
1.1.2
HomeAssistant Version
2022.12.8
Relevant javascript log output
No response

Yep that happens in some sizes. Thanks for reporting

I noticed that you also miss the rounded corners, and the text size is greater than what I have specified. What browser are you using?

Updates fixed it. Thank you very much.
(I’m using iOS Home Assistant App)


Support changing permissions on secret directories

I have a whole bunch of secrets stored in the /run/secrets/scripts directory. I want to put the directory /run/secrets/scripts into a non-root user's $PATH, but since that user doesn't have permission to read the directory, it doesn't work.
This can be worked around by manually symlinking to the secrets, but it would be great to have this as a normal feature.

Not sure what the configuration would look for this since permissions are nested...

Two ideas pop into my head, one in yaml, one in nix:
slapd:
  __dirMode: 0755
  __dirOwner: slapd
  password: abc

{
  sops.secrets."slapd/password" = {};
  sops.dirPermissions.slapd = {
    mode = "0755";
    owner = "slapd";
  };
}

Mhm, this seems like quite a bit of extra complexity for each feature that not many people use.


Excavator action is not continuing to update other manifests after failing.

The Excavator workflow is not continuing on error and failing to update all the Scoop buckets.

I dont think
$Global:ErrorActionPreference = 'Continue'

Is getting scoped into
https://github.com/ScoopInstaller/GithubActions/blob/23a63d8c4d9f2f1f486c75d876590d7f25ff6068/src/Action/Scheduled.psm1#L31
But I don't know PS...

Damn, thanks for bringing this to our notice, @sedlund!


Metrics upload is failing due to multiple tasks found in the datastore

Used the following hash: 92bb306bcd1b62f56ea39f359bd8b1423bb74beb
An example: https://ci.chromium.org/ui/p/flutter/builders/prod/Linux run_debug_test_linux/125/overview
In this case several tests failed due to this issue. The run cannot complete because there are two entries for the task.
Here is an example for a task on the same run:

Somehow the run for the post submit build shown here was duplicated:

The count is shown as 708 for all of the tasks:

Okay so we definitely somehow got two pull_request events with the action being "closed". The pubsub message ids were different but for the same type of request. I was able to verify this in the logs. Both went to the same instance. These were 2 to 3 seconds apart. I am not sure what the latency is here if any.
We processed<PHONE_NUMBER>972805 at 11:18:35 on 03/02

Then we processed<PHONE_NUMBER>701923 at 11:18:38 on 03/02

This somehow lead to us putting two entries for that pull request in the database. When we look up values we do not create a transaction (Not sure if that matters at this point). I did not see the error message that prints if we do find the commit in the database.
Then we processed the following check_suite with id<PHONE_NUMBER>5 request at 11:18:48 on 03/02

And the other check_suite with id<PHONE_NUMBER>0 for the same pr at 11:18:50 on 03/02

Then later we received two different check_suites created for that single pull request. Those messages went to two different instances. This resulted in double the checks being generated for the post submit run.

Okay looking at the github payload I found that github delivered two of the same message to us!
Request 1:

Request 2:

Does it mean it's a github issue and we can close this one?

Please reopen if you see this issue again.


Lecture11 Balancing image combination ratios for enhanced performance

I understood that Cutmix's principle is to combine the two pictures and do the labeling in this way [0.5, 0.5]. However, if you combine the two pictures like [0.9,0.1] or [0.95,0.05], the performance is expected to be lower. How did you solve this problem? Or is there an optimal range?

I believe this is related to using the beta function in probability sampling.

As illustrated, by setting hyperparameters, we can set the probability density function to be lower at the extremes, such as near 0 or 1. This helps in adjusting so that extreme values like 0.95 or 0.1 are less likely to be sampled. Additionally, by tuning the hyperparameters of the beta function, it is possible to empirically determine the optimal hyperparameters for sampling probabilities.
Could you please confirm if my understanding is correct? Thank you!

@junehyeok129 @imsongpasimin Thank you for the comment and the correct answer.
By adjusting the hyperparameters of the Beta distribution, we can control the mixing ratio of two images (@imsongpasimin).
It's important to note that even if the mixing ratio between class 0 (represented as [1, 0]) and class 1 (represented as [0, 1]) is skewed towards one class, such as [0.9, 0.1] or [0.05, 0.95], it does not pose a significant issue because the labels are smoothed to the same ratio. For instance, in the case of a [0.05, 0.95] mixing ratio, we use the smoothed label [0.05, 0.95].


Unified implementation for TARGET_STM and TARGET_NXP

I merged ARMmbed/sdio-driver#1 and ARMmbed/sdio-driver#2,  which were respectively based on JojoS62/sdio-driver/master from @JojoS62 and NXPmicro/sdio-driver/Add_SDIO_LPC55S69 from @mmahadevan108.
I had to make some changes for the coexistence of STM and NXP implementations, and I put everything in a COMPONENT_SDIO directory.
I am creating this pull request to start the discussion. I have not yet been able to test this. I am preparing another patch (pull request) for ARMmbed/mbed-os to support the SDIO component which should be make testing easier (without having to resort to other storage type).

I just found ARMmbed/mbed-os#10235 (which was closed and masked by issue 10909 in the comment in #1).

Replaced by #4.


mandelbulber: 2.24 -> 2.26

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/buddhi1980/mandelbulber2/releases.
meta.description for mandelbulber is: "A 3D fractal rendering engine"
meta.homepage for mandelbulber is: "https://mandelbulber.com"
meta.changelog for mandelbulber is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/4c1nhvidyv7q3h44np68bn3y69bwhxd5-mandelbulber-2.26/bin/mandelbulber2 passed the binary check.

/nix/store/4c1nhvidyv7q3h44np68bn3y69bwhxd5-mandelbulber-2.26/bin/.mandelbulber2-wrapped passed the binary check.

2 of 2 passed binary check by having a zero exit code.

2 of 2 passed binary check by having the new version present in output.

found 2.26 with grep in /nix/store/4c1nhvidyv7q3h44np68bn3y69bwhxd5-mandelbulber-2.26

found 2.26 in filename of file in /nix/store/4c1nhvidyv7q3h44np68bn3y69bwhxd5-mandelbulber-2.26

directory tree listing: https://gist.github.com/211c288c6413fb1e9d8eeefdabb8fbed

du listing: https://gist.github.com/9e37ea9d51b0a64529bb9ab6e21b1c5a

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

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
mandelbulber

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4c1nhvidyv7q3h44np68bn3y69bwhxd5-mandelbulber-2.26 \
  --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 mandelbulber https://github.com/r-ryantm/nixpkgs/archive/889e6328f59e7d124e0f0e4882d101bc19607635.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4c1nhvidyv7q3h44np68bn3y69bwhxd5-mandelbulber-2.26
ls -la /nix/store/4c1nhvidyv7q3h44np68bn3y69bwhxd5-mandelbulber-2.26/bin

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

1 package built:

mandelbulber

Maintainer pings
cc @kovirobi for testing.

@ofborg eval

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

1 package built:

mandelbulber


Validate the baking status of an address

We need a function to validate if the delegate address entered by the user is registered as a baker or not.

With the recent betanet protocol upgrade, the wallet is hanging due to this issue. Arthur himself wrote on Slack about this. We must fix this with the 2nd release


Update to Angular 15

The latest version of Angular (as of Jan 2023) is v15, and our code is at v13. We should copy over code from the latest version of their tutorial (as I did last year, described in #6) to make sure we line up with what they're doing.

This was closed by a PR


Update requirements and tests.

Summary
Update to lxml version 4.6.2.
Description
Update tests to reflect the new version of lxml can emit the namespaces in random orders. Added CI tests via travis-ci.com.

Fixes #14 and includes #18

Looks great to me!


Your database (default) is offline or not responding

Hey there,
First of all, I love this product.
Now for my issue.  I have my entire site (DB and web) on a single server.  This weekend I moved to a new similarly configured server but now I've got thousands of those alerts in the Notifications list.  I'm not sure where it's looking, but as far as the databases in config/database.php go, I haven't changed anything there.  I changed the entries in my app's .env file but the rest of the app works great, it's only EyeWitness that's having problems.
Am I missing something?

Hi @nexxai - glad you like it :)
For this - have you tried starting all the queues again, as well as flushing the config cache and flushing Opcache?


Remove dead code in saving lib

I think this was just straggler code from a previous refactor.

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

Project coverage is 54.11%. Comparing base (104fe8e) to head (06391e0).
Report is 3 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #19415       +/-   ##
===========================================
- Coverage   75.97%   54.11%   -21.86%
===========================================
  Files         366      366
  Lines       40740    40756       +16
  Branches     7944     7945        +1
===========================================
- Hits        30952    22056     -8896
- Misses       8075    17094     +9019
+ Partials     1713     1606      -107

Flag
Coverage Δ

keras
54.11% <ø> (-21.71%)
:arrow_down:

keras-jax
?

keras-numpy
54.11% <ø> (+0.08%)
:arrow_up:

keras-tensorflow
?

keras-torch
?

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

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


[enhancement] Solving issues with current hemodynamic models

I see at least two issues:

the naming is inconsistent e.g.
the values for time derivatives depend on the oversampling factor, which is rather weird.

@bthirion shall we close this issue?


Blockchain state meta API returns 404 for compressed files

Blockchain states meta API returns wrong url_path and urls fields if the file is compressed:
> GET /api/v1/blockchain-states-meta/-1/

{
    "last_block_number": null,
    "url_path": "/blockchain/blockchain-states/0/0/0/0/0/0/0/0/0000000000000000000!-blockchain-state.msgpack",
    "urls": [
        "http://<IP_ADDRESS>:8555/blockchain/blockchain-states/0/0/0/0/0/0/0/0/0000000000000000000!-blockchain-state.msgpack"
    ]
}

Expected result:
{
    "last_block_number": null,
    "url_path": "/blockchain/blockchain-states/0/0/0/0/0/0/0/0/0000000000000000000!-blockchain-state.msgpack.xz",
    "urls": [
        "http://<IP_ADDRESS>:8555/blockchain/blockchain-states/0/0/0/0/0/0/0/0/0000000000000000000!-blockchain-state.msgpack.xz"
    ]
}

@dmugtasimov Please review https://github.com/thenewboston-developers/thenewboston-node/pull/373

@andbortnik please, address comments

@dmugtasimov Please see the changes

@andbortnik please, see my replies. Also take a look at block chuck meta implementation (I merged it today). It does have issue with compressor uncertainty (when filename sometime treated as original name and sometimes as actual filename). May you can fix them in the same style. I think we need original_filename and actual_filename tuple attributes to be explicitly clear what is meant


docs: pick methods belong to Deck, not DeckGL

For #2658 (first section)
Background
Page linked from documentation doesn't actually have any information on the picking methods mentioned.
Change List

Just at this line, change "DeckGL" to "Deck",
and point at the "Deck" documentation page.

Perhaps an anchored link would be better?
Apologies again if I'm confused.

Perhaps an anchored link would be better?

Not quite sure what you are proposing, but good to understand that links are processed during markdown to HTML conversion when we generate the website. Our ambition is for links to work both when reading docs on github and in the website.