make default sonar system user configurable. Default to sonar

Thank you for making this awesome role. I felt like it would be better not to run Sonar as root by default, hence this PR. I've tested this inside your geerlingguy/centos7 box.
Let me know if this is acceptable  or if you have any feedback.
Regards,
Barry

Well, that's odd.
Seems the build failed but I can't really see why.
It fails the idempotence check with 22 changes, which are not just mine.
Any idea how to move this forward?

👍

I'll try to take a look soon—there's a fix in my Jenkins role's .travis.yml that lets you see what idempotent task causes a change in the output from Travis... I haven't had time to apply that change to this role's travis file yet.


Azure MobileServices - background sync with foreground updates = Busy errors

In simple terms, with a background thread performing PushAsync and PullAsync's, any foreground database inserts (or updates) can result in 'Busy' errors if the timing happens to clash.
We are writing a Xamarin Forms UWP app and using the Azure Mobile Services to pull data into a SQLite db (IMobileServiceSyncTable), work offline and push changes back and in general this works really well.
However, more extensive testing is throwing up some SQLite 'Busy' errors when a background sync is in progress and we try to update or insert new records in response to actions on the UI ie the user edits a record.
All actions are performed on the IMobileServiceSyncTable, all the same context etc so should all be using the same connection underneath. I know what the busy message means but was hoping that by doing everything thru the same context that the locking would somehow be better.
At first it seems like we just neeed to implement our own locking between the updating and sync processess, but this isn't really practical and in the real world where a sync process could take several minutes due to many factors then it's not really practical to leave the user hanging whilst it waits.
We are using Microsoft.Azure.Mobile.Client version 2.0.1. Previously we have written similar apps using the Microsoft Sync Framework with SQLite on the client for offline access, background syncs and foreground updates and didn't get this problem.

We are no longer working on Mobile.Client v2.0.1.  Can you retry on the latest version.  If it is still happening, then please reopen.
It would also be helpful to see a minimal repro project.  We've had problems in the past reproducing threading issues (which is obnoxious), so if you have one you can share, all the better.
Closing this for now, as later versions fixed many threading issues.


Fix issue with localized groups

See https://github.com/minishift/minishift/issues/1541
Doing:
([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdentity]::GetCurrent()).IsInRole('Hyper-V Administrators')

will return False on a localized version of Windows. Eg. on a pt_br vesion of Windows the test needs to be:
([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdentity]::GetCurrent()).IsInRole('Administradores do Hyper-V')

According to MSDN, the best way to solve this is to use a security identifier needs to be used:
Relative identifiers (RIDs) are components of a Windows user group's security identifier (SID) and are supported to help prevent cross-platform localization issues. Many user accounts, local groups, and global groups have a default RID value that is constant across all versions of Windows.
For example, the RID for the BUILTIN\Administrators role is 0x220. Using 0x220 as the input parameter for the IsInRole method results in true being returned if the current principal is an administrator.
Ref: https://msdn.microsoft.com/en-us/library/86wd8zba(v=vs.110).aspx
According to:
https://support.microsoft.com/en-us/help/243330/well-known-security-identifiers-in-windows-operating-systems the following S-1-5-32-578 will work:
([Security.Principal.WindowsPrincipal][Security.Principal.WindowsIdentity]::GetCurrent()).IsInRole('
S-1-5-32-578')

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "localized-groups" git@github.com:gbraad/docker-machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

LGTM, thank you!


Add launch script for NXP MCUXpresso Config tools generator

Created scripts to launch NXP MCUXpresso Config tools generator. Scripts reports not supported platform except Windows amd64.
The MCUXpresso Config tools have to be v16.

@LukasTycNXP unless there is other changes you want to make, are you happy to move this PR from draft to ready for review?

Please make sure the scripts will be added to the release archives by updating the GH Actions release job configuration:
https://github.com/Open-CMSIS-Pack/generator-bridge/blob/main/.goreleaser.yml
I would suggest the add the following entry under the archives: files: node:
archives:
  - files:
      - src: scripts/MCUXpresso_Config_Tools/{{.Os}}-{{.Arch}}/launch-MCUXpressoConfigTools*
        strip_parent: true

Docs:
https://goreleaser.com/customization/archive/

Thank you


Distinguish the 4 different causes for I/O errors.

This PR displays specific error messages instead of the generic ERROR: Could not open file .....

@catull cool! Thanks for cleaning this up!


Typo

- You use this object to to run various commands in your session.
+ You can use this object to run various commands in your session.

Thanks agains!


Session with K8S helm install

Hello,
I just went through and installed clearml-server with the helm charts on our on-prem k8s cluster. It appears like this uses the persistent agent option instead of the glue option (link). Is it possible to use clearml-session with this setup? If so, how do I need to configure the session? Will I need to add a service to route the ssh connection through or is it being routed through an existing service?
Thanks for the help!

Hi @SpontaneousDuck,
The clearml-session basically depends on an agent running the interactive session. However, assuming the interactive session will be spun up in a k8s pod, you'll need (as you mentioned), a pre-configured ingress to route the SSH connection. The k8s glue contains a mode called the "ports mode" which basically automatically labels pods spun up by the k8s glue (which acts as a daemon agent), allowing you to pre-configure a fixed ingress for such-labeled pods.
This will not be possible using the default "persistent agent" (as you've called it) which is an optional part of the Helm chart.


Fix the bug of tensorrt: can not infer the model which batch size is not 1

Closes #284
Signed-off-by: zhangkaili<EMAIL_ADDRESS>

bors r+


Added another level in Guides section (issue #105)

Fixes issue #105

/assign @jlewi

This is great thank you so much!
/lgtm
/approve


Evaluating project file from .Net Core application throws StackOverflowException

Steps to reproduce
I'm trying to evaluate project file from .Net Core application using Microsoft.Build packages. Target project uses Microsoft.NET.Sdk. The approach is similar to Microsoft.Build.Locator package, where Microsoft.Build dependencies are marked with ExcludeAssets="runtime". Instead loading them from application directory, the application uses AppDomain.CurrentDomain.AssemblyResolve to load assemblies from SDK directory.
You can find sample over here: https://github.com/kovalikp/MSBuildCoreTest
Expected  behavior
Being able to load project get list of targets from project file.
Actual behavior
Application crashes on Exception Unhandled: System.StackOverflowException. Exception does not have stack trace or any other useful data.
Environment data
OS info:
Windows 10
Version	10.0.17134 Build 17134
Microsoft Visual Studio Enterprise 2017
Version 15.7.2
VisualStudio.15.Release/15.7.2+27703.2018
Project target framework:
netcoreapp2.0
NuGet package
Microsoft.Build 15.7.179
dotnet
sdk 2.1.201

I ran into the same issue. I was able to work around it by not excluding the assets from the MSBuild nuget packages, delivering them with my application, and disabling my custom MSBuildLocator. I would prefer to simply load the version supplied in the dotnet SDK since the workaround makes my deliverable much bigger.

I found a preferable work around. Instead of including the package assets, if I load the MSBuild assemblies from the dotnet sdk folder upfront (Assembly.LoadFrom), everything works as expected.


datadog-agent: 7.38.1 -> 7.45.1

Description of changes

https://github.com/DataDog/datadog-agent/blob/7.45.1/CHANGELOG.rst
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] Fits CONTRIBUTING.md.

I've just added a separate config for the datadog process agent in the nixos module as well. It was hard-coded, and it should likely be configurable as well (for situations like this).

Hi, is this planned on being merged? The datadog agent is currently broken in the latest nixpkgs.


Able to query records with acl set to public:false

When a record is set with ACL [{"public": false}], fetching records by a user other than the owner exhibits correct behavior:
$ http localhost:3000 action=record:fetch access_token=5e6f8c43-3f6b-44ce-96c8-59a97525a4ba ids:='["comment/6a4f53e2-8102-4de1-b845-3d0d9249acdc"]'
{
    "result": [
        {
            "_id": "comment/6a4f53e2-8102-4de1-b845-3d0d9249acdc",
            "_type": "error",
            "code": 102,
            "message": "no permission to read",
            "name": "PermissionDenied"
        }
    ]
}

But querying for the record type will return the record:
$ http localhost:3000 action=record:query access_token=5e6f8c43-3f6b-44ce-96c8-59a97525a4ba record_type=comment
{
    "result": [
        {
            "_access": [
                {
                    "level": ""
                }
            ],
            "_created_at": "2016-06-28T05:58:58.174264Z",
            "_created_by": "f25fca56-073e-4b74-8a2f-ca040a085804",
            "_id": "comment/6a4f53e2-8102-4de1-b845-3d0d9249acdc",
            "_ownerID": "f25fca56-073e-4b74-8a2f-ca040a085804",
            "_type": "record",
            "_updated_at": "2016-06-28T05:58:58.174264Z",
            "_updated_by": "f25fca56-073e-4b74-8a2f-ca040a085804",
            "body": "Lovely!",
            "post": {
                "$id": "post/5ec13be4-b701-4502-b26c-a19ee1122180",
                "$type": "ref"
            }
        }
    ]
}

Skygear Server Date/Version: 0.12.1
Application Platform:
[ ] Is this a regression?
[ ] Attached logs, screenshots

Expected Results
Should not return the record because of ACL setting in place.
Actual Results
Returned the record, ignoring ACL setting.

And I don’t know why the _access settings become [{"level":""}] instead of the intended [{"public": false}].

I just tried with this ACL settings and record:query returns the record anyway:
                {
                    "level": "read",
                    "relation": "$direct",
                    "user_id": "713a3a64-743d-4e79-9b43-cb3ea888b504"
                }

Turns out [{"public": false}] is undefined and should not appear in the database.

This is due to the request lacking database_id field, and this cause the query handler to ignore ACL.

superseded by #58


ReversedVPN connections being reset by istio ingress reconfigurations

How to categorize this issue?

/area networking
/kind bug
/priority 3
What happened:
Each reconfiguration of the istio-ingress gateway causes all established vpn connections of clusters on the same seed to be reset/killed.
Below is an example coming from a tcpdump for reset packets in the istio-ingress gateway:
# tcpdump -ni eth0 'tcp[13] & 4 != 0 and port 8132'
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on eth0, link-type EN10MB (Ethernet), capture size 262144 bytes
09:04:00.956700 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.25052: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.956922 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.25052: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.957136 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.32728: Flags [R], seq 677275936, win 0, length 0
09:04:00.957246 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.50335: Flags [R.], seq<PHONE_NUMBER>, ack<PHONE_NUMBER>, win 8197, options [nop,nop,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>], length 0
09:04:00.957273 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.50335: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.957323 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.50335: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.957387 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.32728: Flags [R], seq 677275936, win 0, length 0
09:04:00.957387 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.50335: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.957402 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.50335: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.958070 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.23669: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.958294 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.23669: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.964219 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.964565 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.965063 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.965213 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.965242 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.965784 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.965834 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.965876 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.965903 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.965943 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:00.965963 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:04:01.179392 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.21511: Flags [R], seq<PHONE_NUMBER>, win 0, length 0
09:07:09.990264 IP <IP_ADDRESS>.8132 > <IP_ADDRESS>.52484: Flags [R], seq 250348339, win 0, length 0

If debug level logging was enabled on the istio-ingress gateway it showed that a configuration change was performed and therefore connections were drained. Approximately 40-50 seconds later the draining was complete (with another log message) and all connections were terminated.
In the vpn-seed-server and vpn-shoot logs, you see corresponding connections reset messages:
Fri Jul 16 12:29:10 2021 vpn-shoot-client/<IP_ADDRESS>:40706 Connection reset, restarting [0]

Fri Jul 16 12:29:10 2021 Connection reset, restarting [0]

What you expected to happen:
Reconfigurations of the istio-ingress gateway should not affect existing vpn connections and certainly not disrupt/kill them. Cluster creation/deletion should not affect the ReversedVPN tunnel connection of other clusters on the same seed.
How to reproduce it (as minimally and precisely as possible):

Have a cluster running on a seed with ReversedVPN being enabled (per cluster or per gardenlet feature gate)
Create a second cluster on the same seed with ReversedVPN being enabled
Watch the VPN connection of the first cluster being disrupted.

This works the same with cluster deletion or a larger amount of clusters.
Anything else we need to know?:
The behaviour is documented by envoy proxy here: https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/operations/draining
It might affect the SNI infrastructure in the same way.
Environment:

Gardener version: v1.28-dev
Kubernetes version (use kubectl version): v1.20.8

/cc @DockToFuture @mvladev @marwinski

@mvladev Feel free to share how this issue affects/does not affect the SNI infrastructure.

Due to #4362 this issue is less relevant as the overall connection will continue to exist for established connections and new connections have a bigger buffer until an error is reported back.

@mvladev Could you please update this ticket?
Also, @marwinski opened https://github.com/gardener/vpn2/issues/4 which seems to describe the same problems as https://github.com/gardener/gardener/issues/4381 and https://github.com/gardener/gardener/issues/4382 opened by @ScheererJ . Which ones do we keep?


moment().format() always output in local time, utcOffset() odd behaviour

Description of the Issue and Steps to Reproduce:
// NOT what I expect
moment("2017-07-03T00:00:00+08:00").format()
// => "2017-07-02T10:00:00-06:00"

// What I expect, but a bit clumsy
moment("2017-07-03T00:00:00+08:00").utcOffset("2017-07-03T00:00:00+08:00").format()
// => "2017-07-03T00:00:00+08:00"

Also moment somehow use local time as utcOffset which cause the following behaviour:
// Wrong, gives  -360
moment("2017-07-03T00:00:00+08:00").utcOffset()

Can we make moment to use its internal _tzm by giving an option into format() ?
So that
moment("2017-07-03T00:00:00+08:00").format('', {useTZM: true})
 // => "2017-07-03T00:00:00+08:00"

Environment:
Chrome 58 on OSX
moment.version "2.14.2"
Other information that may be helpful:

The time zone setting of the machine the code is running on 2017-07-07T01:57:48-06:00

You need .parseZone.
See this blog post:
https://maggiepint.com/2016/05/14/moment-js-shows-the-wrong-date/


New predictions approach

[x] Create template R script for each species
[x] Update predict_sdm function
[ ] Create bash script
[x] Create output/suitabilities folder

For each time period

Predict suitability based on each SDM (save only current time period: suitabilities/genus_species-[sdm]-current.rds)
Create raster with weighted average of suitability values (save: suitabilities/genus_species-ensemble-[time].rds)
Calculate threshold value
Create raster with predicted distribution (save: distributions/genus_species-distribution-[time].rds)

@jcoliver I think things are finished and working. I also closed an separate issue that was created just to update predict_sdm.

As with the other two bash-builders, want to test this out before closing the issue. @ezylstra


Request for combined dashboards for clusters and projects

What kind of request is this (question/bug/enhancement/feature request): Enhancement
Steps to reproduce (least amount of steps as possible): n/a
Result: (Desired result)
It would be nice if there was a single dashboard where top-level metrics from each cluster were displayed.
It would also be nice if there was a single dashboard where top-level metrics from each project were displayed.

This would be a great feature to have. Additionally it would be awesome, If multiple clusters could aggregate to a single grafana dashboard.

It seems that another issue has been opened for this that is getting more traction. We may want to close this one in favor of the other.
https://github.com/rancher/rancher/issues/23057

Close in favor of https://github.com/rancher/rancher/issues/23057


xcodebuild faild: 'collection_notifications.hpp' file not found

Version 2.0.4 file not found when using cocoapods trunk push command.
Goals
To be able to compile successfully.
Expected Results
No errors occur.
Actual Results
Build failed with error:
#include "collection_notifications.hpp"
         ^
1 error generated.

Steps to Reproduce
pod trunk push --verbose --allow-warnings
Code Sample
/var/folders/m1/vyls4x9167b98rtbtwmyvjnw0000gp/T/CocoaPods/Lint/Pods/Realm/Realm/ObjectStore/src/impl/collection_change_builder.cpp:19:10: fatal error: 'impl/collection_change_builder.hpp' file not found
#include "impl/collection_change_builder.hpp"
         ^

CompileC /Users/devedbox/Library/Developer/Xcode/DerivedData/App-gyjmrpvlfiduejbdxaszesswculw/Build/Intermediates/Pods.build/Release-iphonesimulator/Realm.build/Objects-normal/i386/collection_notifications.o Realm/Realm/ObjectStore/src/collection_notifications.cpp normal i386 c++ com.apple.compilers.llvm.clang.1_0.compiler

/var/folders/m1/vyls4x9167b98rtbtwmyvjnw0000gp/T/CocoaPods/Lint/Pods/Realm/Realm/ObjectStore/src/collection_notifications.cpp:19:10: fatal error: 'collection_notifications.hpp' file not found
#include "collection_notifications.hpp"

 - ERROR | [iOS] xcodebuild: Returned an unsuccessful exit code.
  - NOTE  | [iOS] xcodebuild:  Realm/Realm/ObjectStore/src/impl/collection_change_builder.cpp:19:10: fatal error: 'impl/collection_change_builder.hpp' file not found
    - NOTE  | [iOS] xcodebuild:  Realm/Realm/ObjectStore/src/collection_notifications.cpp:19:10: fatal error: 'collection_notifications.hpp' file not found

Version of Realm and Tooling
Realm version: 2.0.4

Xcode version: 8.1

iOS/OSX version: 10.12.1

Dependency manager + version: 1.0.0

ProductName:	Mac OS X
ProductVersion:	10.12.1
BuildVersion:	16B2555

/Applications/Xcode.app/Contents/Developer
Xcode 8.1
Build version 8B62

/usr/local/bin/pod
1.1.1
(not in use here)

/bin/bash
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin16)

(not in use here)

/usr/bin/git
git version 2.9.3 (Apple Git-75)

Hi @devedbox, are you trying to push Realm.podspec, RealmSwift.podspec or a different podspec altogether?
If you're using Realm in a different podspec and you're specifying a location other than CocoaPods trunk, make sure you set it to use git submodules, as the Realm pods depend on those.
If you're trying to push Realm.podspec or RealmSwift.podspec to say your own private CocoaPods trunk server, make sure you've updated your local submodules prior to doing so:
git submodule update --init --recursive

And that you're using the latest version of CocoaPods (1.1.1), and that you have Xcode 8.x installed and xcode-selected. I just pushed the Realm and RealmSwift pods version 2.0.4 yesterday to CocoaPods trunk with no issue.

@jpsim I tried as you said and still failed. this is my podspec file:
Pod::Spec.new do |s|

  s.name         = 'JYObjectModule'
  s.version      = '0.0.27'
  s.summary      = 'A module of models definition.'
  s.description  = <<-DESC
                    A module of models definition on iOS platform for jiangyou tec. only.
                   DESC

  s.homepage     = 'https://github.com/jiangtour/JYObjectModule'
  s.license      = { :type => 'MIT', :file => 'LICENSE' }

  s.author             = { 'aiXing' =><EMAIL_ADDRESS>}
  s.platform     = :ios, '7.0'

  s.source       = { :git => 'https://github.com/jiangtour/JYObjectModule.git', :tag => '0.0.27' , :submodules => true }
  s.source_files  = 'JYObjectModule/Models/*.{h,m}', 'JYObjectModule/Models/Normal/*.{h,m}', 'JYObjectModule/Models/Storage/*.{h,m}'

  s.frameworks = 'Foundation'

  s.requires_arc = true
  s.dependency 'Realm'
  s.dependency 'MJExtension'
  s.dependency 'SDWebImage'
  s.dependency 'CocoaSecurity'
  s.dependency 'UICKeyChainStore'

end

It works fine in version 2.0.3.

I add realm-object-store as a submodule and build successfully. You can close the issue now. Thanks anyway. @jpsim

Glad to hear it!

Hi,
I am using RealmSwift and getting the same issue. Both cocoapod and realm are latest version.

@amit-k-gupta please file a new issue with steps to reproduce, and please include the entire output from pod install -verbose

Same issue using RealmSwift v2.1.1, no problem with v2.1.0.

@mathroule please file a new issue with steps to reproduce, and please include the entire output from pod install -verbose, and taking care to fill out all the sections in the new issue template.
I'll be closing this issue to prevent future messages being posted without including the necessary information prompted by the issue template. Thanks for understanding!


Return accounts data from nctl in JSON format

So we can display data like here
Address
Balance in CSPR (1CSPR is<PHONE_NUMBER> motes - API should always return string with motes)

Add new endpoints GET accounts/ and GET accounts/:number  (the second one needs to return private keys associated with this accounts)
It MUST return PublicKey and balance
The detail endpoints should return private key as well as associated keys with this account

Example:
https://cspr.live/account/0202ed20f3a93b5386bc41b6945722b2bd4250c48f5fa0632adf546e2f3ff6f4ddee

for now it doesn't make sense to do this as we're not sure if we will use nctl in new release.


Feature/opensearch integration

First pass at the integrating with opensearch official client libraries

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

First pass at the integrating with opensearch official client libraries

It is looking good. I pushed some commits. However, we still need to do some work.

[ ] Tests for the converters
[ ] Test for the health endpoint
[ ] Evaluate if we can get rid of the jacksondatabind dependency
[ ] Graal Test
[ ] Test for the OpenSearchAsyncClient
[ ] Groovy Code samples
[ ] Kotlin samples
[ ] Show in the docs embedded samples


Fresh Install, error artisan route:list

Laravel Version: v9.3.0
PHP Version: PHP 8.1.2
Database Driver & Version: None

Description:
After a fresh install, execute route:cache and route:list and exception fired.,
Steps To Reproduce:
I made a fresh install with
composer create laravel/laravel test
cd test
php artisan route:cache
php artisan route:list
And error...
` ReflectionException
Class "O:47:"Laravel\SerializableClosure\SerializableClosure":1:{s:12:"serializable";O:46:"Laravel\SerializableClosure\Serializers\Signed":2:{s:12:"serializable";s:295:"O:46:"Laravel\SerializableClosure\Serializers\Native":5:{s:3:"use";a:0:{}s:8:"function";s:77:"function (\Illuminate\Http\Request $request) {
return $request->user();
}";s:5:"scope";s:37:"Illuminate\Routing\RouteFileRegistrar";s:4:"this";N;s:4:"self";s:32:"00000000000002d30000000000000000";}";s:4:"hash";s:44:"Wl3vmm7PTYqPG0N8Pm85TCwJiITBVFJMMMs+7LPOb5A=";}}" does not exist
at vendor/laravel/framework/src/Illuminate/Foundation/Console/RouteListCommand.php:223
219▕         if ($route->action['uses'] instanceof Closure) {
220▕             $path = (new ReflectionFunction($route->action['uses']))
221▕                                 ->getFileName();
222▕         } elseif (is_string($route->action['uses'])) {
➜ 223▕             $path = (new ReflectionClass(explode('@', $route->action['uses'])[0]))
224▕                                 ->getFileName();
225▕         } else {
226▕             return false;
227▕         }`

Caused by https://github.com/laravel/framework/pull/41254. We're investigating, thanks.

Weird, I tried this and couldn't reproduce it with a fresh install. Found out I was on version 9.3.1. Downgraded Laravel from 9.3.1 to 9.3.0.
Looks like it got fixed in 9.3.1. @dgironella could you verify that?

In version 9.3.1 is solved. My error was produced in version 9.3.0 caused by #41254


Possibility for page numbers in PDF?

Hi everyone! Currently I use PhantomJS to generate PDF documents from html output. Because PhantomJS isn't really up-to-date, I'd like to switch to Headless Chrome.
The thing that keeps me from making the switch is that it's not possible to print the page number on the bottom of every page. I found out that it's possible to set a header and/or a footer on every page with CSS.
header {
  display: block;
  position: fixed;
  top: 0px;
  left: 0px;
  right: 0px;
}

footer {
  display: block;
  position: fixed;
  bottom: 0px;
  left: 0px;
  right: 0px;
}

The only downside is that this only works for static data as it's not possible to somehow detect and print the page number in the footer or header. PhantomJS allows to set a header and footer of a document and print the page number.

displayHeaderFooter will add a page number to the bottom of the page. Unfortunately, chrome's print to pdf doesn't allow much customization of this and few browsers have support for the full set of features in CSS @page.
The following worked for me. To see the header, I had to add a top/bottom margin so the page's content wouldn't cover them up in the final pdf.
await page.pdf({
  path: 'page.pdf',
  displayHeaderFooter: true,
  margin: {top: 40, bottom: 40}
});

@JoelEinbinder @aslushnikov might have a better way.

I think CSS counters can help you get the page number in your CSS:
https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Lists_and_Counters/Using_CSS_counters

I tried that together with the CSS from above and a :after pseudo selector to set the content of the footer. It prints the total number of pages on every page because in fact it's only one element.

Has anyone found any workaround for that?

The only thing that worked is displayHeaderFooter. The downside is that it also prints the page title and page date at the top. Page title can be removed by removing the title tag in the html file. Date is OK for me. The only thing that's a no go for me is the filepath/pageurl in the bottom left corner.

Sadly no support to add page numbers with puppeteer and no way to do this in CSS. However there is a workaround for that. Just edit your pdf file after puppeteer is doing magical things.
For my project I've used HummusJS (https://github.com/galkahana/HummusJS) to add page numbers after puppeteer. Performance is quiet good with puppeteer+hummus and even better than wkhtmltopdf or phantomjs.
HummusJS example to replace strings in an existing pdf file:
https://github.com/galkahana/HummusJS/issues/71
Here you can find different ways to do this:
https://stackoverflow.com/questions/1603301/how-to-add-page-numbers-to-postscript-pdf

Is there something I can do or help with to get this implemented? Does this mean that we have to add/change things in Chromium directly? Because the Page.PrintToPdf only accepts a boolean for displayHeaderFooter. I would be happy to help you guys out with this but then I need a pointer at where I should/can start.
// @aslushnikov

I'm not familiar with the PDF generation code in Chromium, but our entry point is here:
https://cs.chromium.org/chromium/src/headless/lib/browser/headless_devtools_manager_delegate.cc?type=cs&l=395

I've been fixing the height on a "page" itself. For example, If you set pdf size to A4, you can set a fixed height of 1122px to a container. Then just some small JS to dynamically input page numbers.
await page.goto('https://bl.ocks.org/seripap/raw/81241195e182b62adc3c87c27258f85f/', {waitUntil: 'networkidle'});
await page.pdf({
  path: 'hacks.pdf',
  format: 'A4'
});

more clean option to customize header and footer will be awesome.

Specifically looking for this spec to be implemented: https://www.w3.org/TR/css3-page/#page-based-counters

Imhop, this isn't really a puppeter issue, discussion should be moved to: https://bugs.chromium.org/p/chromium/issues/detail?id=368053 or https://bugs.chromium.org/p/chromium/issues/detail?id=740496

Thank you very much @aslushnikov 🍻 !

:thumbsup: :thumbsup: :thumbsup:

@aslushnikov omg yes!! Thank you so much!! 👍👍👍

All credits go to @pavelfeldman and @aj-dev! 🎉

Thank you as well and some feedback... I have two points.

The header seems to be several times smaller than the content in the output pdf. Simple
...</h1 in header and content looks in the output like this. At least on my windows.

The headerTemplate doesn't seems to evaluate javascript. Is this expected? I would like to use it to for example hide the page number for the first pdf page.

I was fiddling with the package installed through npm i GoogleChrome/puppeteer. I apologize if I did miss something or was too inpatient.

@pofider I've noticed the same with small header and footer fonts but had no time to dig deeper. There's some scaling issue, I guess. It works ok if I declare inline style, eg. style="font-size: 12px;"

What is the expected format of header/footerTemplate?
I set the following as options in page.pdf:
        displayHeaderFooter: true,
        footerTemplate: '
Test Footer
'

but nothing is shown in the generated pdf file. I have tried also with simple text and other html tags, but nothing produces any results.

@acgrama me too,it seems that chrome version not right. maybe @pofider can give some advise, thanks.

@aslushnikov  can headerTemplate make a function like phantomjs header ？This may more flexible
`
header: {
    contents: phantom.callback(function(pageNum, numPages) {

        if (pageNum % 2 != 0) {

            return '
' + pageNum + ' / ' + numPages + '
';

        }

        return "";

    }),

},

`
thanks very much

have you tried not using displayHeaderFooter: true?, AFAIK that option is to show default header and footer that chrome has on printed pages but when you use headerTemplate or footerTemplate that option does not makes sense and you are likely getting into a conflict. try don't using it or put it in displayHeaderFooter: false

@bjrmatos yes，i had set displayHeaderFooter: true.

Here is what Chrome itself uses. You can use date, title, url, pageNumber, totalPages classes to specify nodes values are injected into.
headerTemplate = `<div class='date'></div><div class='title'></div>`
footerTemplate = `

      /

`

@pavelfeldman  thanks, this method is perfect, but i just want add header in odd page num , so this method can not use.

@yale8848 did you tried nth-child?

@bjrmatos @pavelfeldman I left out the displayHeaderFooter option and tried the suggested footerTemplate, but I can still see no footer.

Are you sure you installed the version from Github? The new feature is not yet released through npm.

@SamVerschueren Ok, then that's the problem! We installed through npm, as part of our docker image build chain.
Thanks a lot, looking forward to testing out the npm-based version!

@k15a thanks，i will try it

Hey @pavelfeldman ,
I tried the following code it is not showing the template as expected.
I have used the following code:
index.js
var puppeteer = require('puppeteer');

(async () => {
    console.time('pdf-gen');
    const browser = await puppeteer.launch({
        args: ['--no-sandbox', '--disable-setuid-sandbox']
    });
    console.log("Puppeteer Browser Initiation Successful!");
    const page = await browser.newPage();
    var response = await page.goto('https://github.com/GoogleChrome/puppeteer/releases', {
        waitUntil: 'networkidle0'
    });
    if (response.ok) {
        var report = await page.pdf({
            path: 'hn.pdf',
            headerTemplate: `

            OF
`
        });
    }
    await browser.close();
    console.timeEnd('pdf-gen');
})();

package.json
{
  "name": "puppeteer-sample",
  "version": "1.0.0",
  "main": "index.js",
  "author": "suri",
  "license": "MIT",
  "private": true,
  "dependencies": {
    "puppeteer": "https://github.com/GoogleChrome/puppeteer"
  }
}

It is not printing the the headers as Expected.
Any help would be appreciated.
Thank you.

@ayal I had the same issue, and managed to style the header/footer without using inline styles, without the !important marks, and without setting the margins (I guess Chrome is using the default margins for format A4). All I do is embed a

  Page
   /

It seems that some stylings just won't work though, like background-color.

@yale8848 do you have any progress with nth-child?

Did you try using

Re: adding image, are you using data url? Pointing to a resource would not wait for it to load, you have better chances with data:

@pavelfeldman

test1

test2

test3

I try add

Confirm, can't get JS works in template too. Also, looks like chromium prerenders frame separate for every pdf page, so neither css hack won't help you remove page number only for the first page.

@paul-pro Get it , thanks you very much, it seem that change header custom is difficult.

@paul-pro get it , thanks

i find a method to execute javascript in headerTemplate and footerTemplate:
https://github.com/GoogleChrome/puppeteer/issues/2167#issuecomment-425448926

The most recent documentation link that @odykyi referenced is here: https://github.com/puppeteer/puppeteer/blob/main/docs/api.md#pagepdfoptions

If you need customizing the page number and end up here like me, for now until the puppeteer let us customize the header/footer I create this. It uses puppeteer and let fully customize header, footer, and pagination
https://github.com/PejmanNik/puppeteer-report#readme

What is the expected format of header/footerTemplate?
I set the following as options in page.pdf:
        displayHeaderFooter: true,
        footerTemplate: '
Test Footer
'

but nothing is shown in the generated pdf file. I have tried also with simple text and other html tags, but nothing produces any results.

https://github.com/puppeteer/puppeteer/issues/5345#issuecomment-747325977

The only thing that worked is displayHeaderFooter. The downside is that it also prints the page title and page date at the top. Page title can be removed by removing the title tag in the html file. Date is OK for me. The only thing that's a no go for me is the filepath/pageurl in the bottom left corner.

I think that is the default value for header/footer, just add '' to the one you do not need and it will not display the default value.


Control node does not provide global position if placed under container

Godot version
4.1 RC1
System information
Windows 10
Issue description
Control node does not provide global position if placed under container
Steps to reproduce

Create a Horizontal/Vertical box container
Create Texture Rect node inside it and duplicate 3-4 times
Check the last node global position ~ print($HBoxContainer/TextureRect3.get_global_rect()) or print($HBoxContainer/TextureRect3.global_position)
Result  = (0,0)

Minimal reproduction project
N/A

Thanks @Sauermann, closing as duplicate then.


Update xref links

Docs Build status updates of commit 8c0e23d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Error 404 - Document not found

[Wed Jul 16 22:43:25 2014] [error] [client xx.xx.xx.xx] File does not exist: C:/Program Files (x86)/Apache Software Foundation/Apache2.2/htdocs/a
<?php
require 'flight/Flight.php';

Flight::route('/', function(){
echo 'first / route matched';
});
Flight::route('/a', function(){
echo 'second a route matched';
});
Flight::start();
?>

The issue I'm having is when calling the first / route, it works, get the correct response. When calling the '/a' i get:
Not Found
The requested URL /a was not found on this server.
I feel like this is a noob thing, but I can't see where I'm going so wrong

sorry for reviving this old thread, but this is one of top 5 search results on google and in case someone having same problem on apache 2.4 despite .htaccess and AllowOverride All in vhost config:
My solution was removing **Options +Indexes +... ** from vhost config.
Hopefully it helps somebidy.
cheers

This doesn't work, but this:

works! Why is that and can I do something about it because it is going to make problems in further development?


Alight declarative interface

Add declarative interface:
alight.namespace('ns')
.directive('name', {link})
.directives({
    dirName: {link}
})
.controller(name, fn)
.controllers({
    ctrlName(scope) {/*...*/}
});
// filters are the same

Namespace create alight new namespace or use existing. It is pretty and simple.
Extended example:
(function() {
    'use strict';

    alight.namespace('ns')
    .controllers({userCtrl, settingsCtrl});

    function userCtrl() {}
    function settingsCtrl() {}
})();

Hi!
I don't very like this Angular 1 style, but I can add it to "ext" package.
Also filters doesn't have namespace, controllers were removed, now direct-directives instead of them.
Also filters and direct-directives can be located in scope, so you don't need always create namespaces, e.g.:
alight('body', {
  data: '',
  filter: (a, b) => a+b,
  directive: () => {}
})

What is direct-directives? How to define it?

Declarative interface is about to avoid ugly constructions like alight.directives.ns = alight.directives.ns || {}; in modular applications.

And about namespaces itself. It very important to separate code into modules. Namespaces is the best trick to avoid conflicts. I have two different projects where I've realised toDate filters but they parsed different date formats which cause an error. This make me to rewrite one of the components.


Make installation of vivado tools (updatemem) a non-optional part of getting started to work on FPGAs

As FPGA tests rely on a growing set of bitstreams it's become challenging to allow users to run FPGA tests without installing vivado tools to provide updatemem. We can run updatemem without purchasing or acquiring a license and can use it to perform necessary bitstream splices to construct a number of bitstreams necessary for FPGA tests.
We should also consider setting it up in all CI environments so that we can discontinue tools we've constructed to work around it.

[ ] Announce that it's a new requirement and work with contributors to get it set up
[ ] Update docs to reflect it's required
[ ] Update in CI
[ ] Cleanup vivado and manual tags

Currently the installation is marked as optional and we've discussed making it non-optional in slack but I'd like to track the issue more formally here.

As of #16998, vivado / updatemem is no longer optional.


How to set default value for url if enviroment variable is undefined

Hi there. I have my spring app with following configuration property in application.properties:
loki4j.url: ${LOKI4J_URL}
and in logback-spring.xml I have following:
`
    <http>
        <url>${loki4j.url:-localhost:3100/loki/api/v1/push}</url>

.....
`
Inspite of information about possibillity of setting default value via :-, when I dont explicitly set LOKI4J_URL variable I get
java.lang.IllegalStateException: java.lang.IllegalStateException: Logback configuration error detected:  ERROR in ch.qos.logback.core.joran.spi.Interpreter@38:16 - RuntimeException in Action for tag [appender] java.lang.IllegalArgumentException: URI with undefined scheme
Could you please tell me is it possible to get what I want or no? Thank you in advance.

Hi @setofaces,
First off, you should be able to use env vars directly in your Logback config.
Second, the error that you're getting - "URI with undefined scheme" - is exactly what is wrong, you just need to add scheme to your URI:
<url>${LOKI_URL:-http://localhost:3100/loki/api/v1/push}</url>

Closing this issue for now, but feel free to ask any further questions here or in dedicated discussions section.


Minimum working example of how to use geospatial time series

Please test whether you can instantiate, retrieve, and visualise geospatial time series data using the provided instructions

I personally prefer the API key to be written by the script, specified in the properties file. Also missing instruction to spin up Blazegraph with geospatial enabled, I know how to initialise a new namespace programatically, let me know if you wish to do that.

better to provide the command to run the python scripts explicitly, e.g. \Scripts\python.exe instantiate_data
It's not super clear that the docker compose script will spin up a container named "docker", I had to work a bit to find the "Open in Browser" option

better to provide the command to run the python scripts explicitly, e.g. \Scripts\python.exe instantiate_data. It's not super clear that the docker compose script will spin up a container named "docker", I had to work a bit to find the "Open in Browser" option

Included suggested changes in commit 3e7eaec17c2537fa70788eade520d15e0c77f3c1


Feedback for "Syntax Options in the hdbgrants File"

https://help.sap.com/docs/HANA_CLOUD_DATABASE/c2b99f19e9264c4d9ae9221b22f6f589/f49c1f5c72ee453788bf79f113d83bf9.html
Hi,
In the documentation for schema_roles the definition of the schema itself is missing, e.g. "schema": "MySchemaS1"
best regards
Martin
{
  "EPM_XXX-table-grantor": {
   "object_owner": {
      "schema_roles": [
        {
          "schema": "MySchemaS1",
          "roles" : ["Schema_RoleR1", "Schema_RoleR2"],
          "roles_with_admin_option" : ["Schema_RoleR3", "Schema_RoleR4"]
        }
      ]
    }
  }
}

Thanks for catching this! The Cloud documentation will be updated where appropriate to reflect the on-premise version, which does include the information about the "schema" property.


License

Sass Guidelines still does not have a license. The code from the Sass Guidelines should be licensed under MIT.
However the content has to use another license given it’s not a software. ChooseALicense says:

Mixed Projects
If your project contains a mix of software and other material, you can include multiple licenses, as long as you are explicit about which license applies to what parts of the project. See the license notice for this site as an example.

Their README for instance states:

The content of this project itself is licensed under the Creative Commons Attribution 3.0 license, and the underlying source code used to format and display that content is licensed under the MIT license.

Any thought? Maybe CCA 4.0?

The CCA 3.0 license seems to be specified for the United States? At least this is what I get out of the link.
I would suggest the Creative Commons Attribution 4.0 International license then.

Done with https://github.com/HugoGiraudel/sass-guidelines/commit/465037c305b302583436fe3013bbf6cf8348abf2.


Robux icon next to "Was XXX" on limiteds is broken / BC icon is premium UGC items is glitched

Released CSS bug report
The Robux icon that shows right next to the "Was (original price here)" on limiteds is broken,
To Reproduce
Steps to reproduce the behavior:

Go to https://www.roblox.com/catalog
Find a limited
See error

To Reproduce (2)
Steps to reproduce the behavior:

Go to https://www.roblox.com/catalog
Click on premium and select anything (or just find a premium item)

See error

Desktop (please complete the following information):

OS: Windows
Browser: Brave
Script Version: 1.2.3
1920 x 1080

Should be fixed  now.

Thanks for the fix. Although, the problem with the "Was (og limited price)" still happens. And also, I kinda feel like I have to ask, could the premium icon in the catalog be changed the the OBC icon like the icon on profiles?

haha sorry I just instinctively scrolled to the bottom of the page. Ill fix the limited prices stuff

Alr thanks lol, thanks for doing all the fixes and stuff I've been suggesting, fun to watch this theme slowly get better and better


Backup/Restore Suite - Recursefix

Type of Change

[ ] Bug fix (non-breaking change, fixes #7387)
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[x] Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Unit test is included
[x] Documentation
[ ] Build system

Purpose
Should make the Restore-Dbadatabase documentation match up with reality
The appv failures don't appear to have anything to with this
Approach

Commands to test

Screenshots

Learning

@lauraberanek could you read the new documentation please, and let me know if it reads as how the command actually works

will wait for explicit approval or/ stuart you can merge when it's ready


A suggestion for  collaboration

Hi,
We would like to collaborate and contribute to your project.
We have ideas to enrich the form capabilities.
Are you interested?
Shaul Almog
Sapiens Technologies

Hi @AlmogShaul,
Sure, any contribution is welcome!!
I think the best to discuss new ideas is to use GitHub issues if you are ok with that.

Great!


@apply in

I was playing around and noticed I couldnt @apply in <style> or <style lang="postcss"> and I am sure its some order of operations in how things are built, but just wondering if it was somehow configurable to be able to use that or not?

If you're using out of the box Phoenix asset pipeline with Tailwind, the entrypoint is assets/css/app.css. so any style tags embedded in your heex templates or render calls will not be processed.
I'm implementing a CSS blueprint and @apply used in CSS works just fine without any changes to the asset pipleine.

I'm not sure I follow the last part of the comment about blueprint CSS. Is there some change you made to make @apply in style tags in Heex and render calls to work?

I'm not sure I follow the last part of the comment about CSS blueprint. Is there some change you made to make @apply in style tags in Heex and render calls to work?

No.  I can't image that could ever work trying to apply tailwind styles if your styles are not imported via assets/css/app.css unless you were using a svelte preprocessor in your asset pipeline.
This might be sometime look into:
https://github.com/sveltejs/svelte-preprocess

This is fairly common thing to deal with when you use just svelte (instead of sveltekit) so usually I do mess with the preprocessor and/or webpack/vite but in this case it seems Phoenix uses esbuild to do it (I guess?). I am not familiar with how Phoenix does all this asset stuff since I always do projects w/out the asset pipeline and (last few years) use vite to handle all that stuff. I can have a look, I am not sure the magic needed to make svelte work in liveview so likely won't know the magic to make css compile in elixir land 😅


feat(ktable): adds props.paginationPageNumber

Summary
Adds support for props.paginationPageNumber which allows one to set KTable’s current page from an external source. This behaves exactly as interactions with the built-in pagination do.
Signed-off-by: Philipp Rudloff<EMAIL_ADDRESS>Notes

I think this ability should be built into KTable. Currently, the only alternative would be updating the table’s key prop whenever a page changes externally. This would trigger KTable’s onMounted hook and let it read the now-changed page number from props.initialFetcherParams. While this works, I consider it a hack. It shouldn’t require a re-mount of KTable to change the page number in situations that KTable can’t control (e.g. browser history navigation).

PR Checklist

[x] Conventional Commits all commits follow the conventional commit standards outlined in the main README.
[x] Functional: all changes do not break existing APIs and if so, bump major version.
[x] Tests pass: check the output of yarn test
[x] Naming: the files and the method and prop variables use the same naming conventions as other Kongponents
[x] Framework style: abides by the essential rules in Vue's style guide
[x] Cleanliness: does not have formatting issues, unused code (e.g., console.logs, debugger), or leftover comments
[x] Docs: includes a technically accurate README, uses JSDOC where appropriate

@kleinfreund I have the same concerns here as we chatted about yesterday as I don't understand the use-case for this prop.
Also, what happens if two things change the page simultaneously, e.g. the table itself and the host app prop are out of sync?

I have the same concerns here as we chatted about yesterday as I don't understand the use-case for this prop.

I think the use case is pretty well explained. My assumption is that it’s expected by users that paginated documents (e.g. once containing tables) can be navigated using regular browser semantics including history navigation. For that to work, pagination state needs to be persisted on the client side (ideally in the URL). I think it’s reasonable that that duty does not fall to KTable as it can’t know how a host application will do that.
With that assumption, I think KTable should support a way to control the current page. The next best alternative (which I think is an undesirable compromise) would be disabling KTable’s pagination (there’s a prop for that) and using KPagination in the host application. Then this can be wholly controlled on the host application side, but considering how easily KTable can be put in the position to handle this scenario, I think it’s far preferable to do that than hand-write the pagination interactions.

Also, what happens if two things change the page simultaneously, e.g. the table itself and the host app prop are out of sync?

I think it can reasonably be assumed that, if someone decides to control the current page of a KTable instance from the outside, they would be the responsibly party for keeping the URLs up-to-date with what KTable provides (e.g. as a side effect of calling props.fetcher which will always hold that current page).
As far as I can tell, what I propose to add here can’t make the internal state of KTable inconsistent. If two interactions happen in quick succession, it’s either the new prop triggering last and superseding a KTable-internal interaction or the other way around. In both cases, two calls to props.fetcher would be triggered (unless they happen during the same event loop cycle I guess? Even then it’s consistently going to be the last one) and I think that’s very much expected.

it’s expected by users that paginated documents (e.g. once containing tables) can be navigated using regular browser semantics including history navigation

I personally don’t agree with this assumption at all. The table is a component on a page; not the page itself.
Coupling URL navigation to KTable IMO is outside the scope of the component and not the intended usage; however, if you’re wanting to control the KTable page based on the URL, the suggested incrementing of a key prop on the KTable to remount the component with an updated initial page seems acceptable IMO and is supported with the current implementation without introducing additional watchers and new props that are essentially doing the same thing, albeit without a remount.

This brings up a good point that the current implementation isn’t ideal for a lot of implementations and should likely allow the host app to perform the data fetching…


Include license file

Linux distros require licensing information to be distributed alongside the installed (binary) package – hence the need for a separate file.

Ping! It's been 2.5 months; please close if you do not want to do this, because that allows me to move forward to including the license in my own packaging, which is suboptimal, but better than not packaging this.

@marcusmueller Sorry for the late reply on this one... and thanks a lot for the ping and the patch. I'll make a new release soon which will include the license.


licenses for JetBrains tools to students

@ojarnaoui I am sorry, I have to close the request without merging, because you made a mistake in a request and requested to merge files other than located in lib/domains/ directory. Also, ofppt-edu.txt.ma is not a proper website URL. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for the understanding!

@ojarnaoui Please provide us with:

the school official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.


Mention remote debugging in news file

For #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #7770 (aa8ac2c) into main (8d51eeb) will decrease coverage by 0%.
The diff coverage is n/a.

:exclamation: Current head aa8ac2c differs from pull request most recent head a68a47d. Consider uploading reports for the commit a68a47d to get more accurate results

@@          Coverage Diff          @@
##            main   #7770   +/-   ##
=====================================
- Coverage     68%     68%   -1%
=====================================
  Files        363     363
  Lines      22593   22593
  Branches    3437    3437
=====================================
- Hits       15544   15516   -28
- Misses      5709    5735   +26
- Partials    1340    1342    +2

Impacted Files
Coverage Δ

.../client/datascience/debugLocationTrackerFactory.ts
42% <0%> (-8%)
:arrow_down:

src/client/datascience/jupyter/jupyterDebugger.ts
19% <0%> (-7%)
:arrow_down:

...ent/datascience/jupyter/pythonVariableRequester.ts
47% <0%> (-6%)
:arrow_down:

...datascience/editor-integration/codelensprovider.ts
69% <0%> (-5%)
:arrow_down:

...datascience/editor-integration/cellhashprovider.ts
70% <0%> (-2%)
:arrow_down:

src/client/datascience/jupyter/kernels/kernel.ts
73% <0%> (-2%)
:arrow_down:

src/client/datascience/jupyter/kernelVariables.ts
53% <0%> (+1%)
:arrow_up:

src/client/datascience/raw-kernel/rawSocket.ts
84% <0%> (+1%)
:arrow_up:


Basic.richcmp(): Make Rel import global, not inline.

This seems to have a major impact on comparison performance, e.g.:
Before:
>>> import timeit; timeit.timeit('a == b', 'from symengine import sympy_compat as sp; a = sp.Symbol("a"); b = sp.Symbol("b")', number=10000)
0.3318144249933539

After:
>>> import timeit; timeit.timeit('a == b', 'from symengine import sympy_compat as sp; a = sp.Symbol("a"); b = sp.Symbol("b")', number=10000)
0.0019199100061086938

Failing test is unrelated and fixed in #120


Update causes all caravans to stop moving

My personal fix is to add a check on Scout.cs, line 358 and 368:
if (this.WarSettlementComp != null)
<= This check prevents null reference exceptions and allows my game to continue..

For Trader.cs, line 352:
I added a check for rwsc.parent:
if(rwsc != null && rwsc.parent != null && rwsc.parent.Faction != this.Faction && !tradedWithSettlement)


fix(version): update fabric8-stack-analysis-ui to 0.1.10

There were the following issues with your Pull Request

Commit: 6c1ca8f138f6f92c17e285b0e123e5f1b06a2a4a
Your subject line is longer than 50 characters

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

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


test: add unit test case for func ParseEnv and ValidateEnv

Ⅰ. Describe what this PR did
test: add unit test case for func ParseEnv and ValidateEnv
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
added unit test
Ⅳ. Describe how to verify it
go test
Ⅴ. Special notes for reviews

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

Codecov Report

Merging #2572 into master will decrease coverage by 0.13%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #2572      +/-   ##
==========================================
- Coverage   69.16%   69.03%   -0.14%
==========================================
  Files         278      278
  Lines       18686    18690       +4
==========================================
- Hits        12925    12903      -22
- Misses       4290     4300      +10
- Partials     1471     1487      +16

Flag	Coverage Δ
#criv1alpha1test	31.35% <0%> (-0.05%)	⬇
#criv1alpha2test	35.72% <0%> (+0.07%)	⬆
#integrationtest	40.71% <0%> (-0.02%)	⬇
#nodee2etest	32.56% <0%> (-0.27%)	⬇
#unittest	26.68% <66.66%> (+0.01%)	⬆
Impacted Files	Coverage Δ
apis/opts/env.go 85.71% <66.66%> (+5.71%)	⬆
cri/ocicni/cni_manager.go 58.82% <0%> (-11.77%)	⬇
cri/v1alpha2/cri_wrapper.go 63.2% <0%> (-2.4%)	⬇
ctrd/image.go 76.95% <0%> (-2.18%)	⬇
cri/v1alpha2/cri.go 68.49% <0%> (-1.25%)	⬇
ctrd/container.go 58.1% <0%> (-0.8%)	⬇
cri/v1alpha1/cri.go 60.59% <0%> (-0.34%)	⬇
cri/v1alpha2/cri_utils.go 89.64% <0%> (-0.29%)	⬇
daemon/mgr/container.go 58.79% <0%> (-0.22%)	⬇
daemon/mgr/snapshot.go 94.2% <0%> (+4.34%)	⬆
... and 1 more

Why the coverage is down?


Add iBBTC to retroactive sushi logic

Add iBBTC to the list of Setts to gather retroactive sushi from. Ensure subgraph has required info for this.

This code has duplicated logic and hardcoded aspects and should be refactored. Though as we intend to only use it once more we can leave as it.

Done

Dapp reports this was done a while ago.


Alpha compositing differentiable rendering

Hi!
First of all thank you for the great work and codebase! :smiley:
I wanted to ask you about the design choice of adding the transparency value of the primitives to the differentiable rendering process. Specifically, in the paper you mention that it behaves better during optimization in comparison to the standard differentiable rendering pipeline. What do you mean by better behavior?  How worse are the results when using directly the standard Pytorch3D renderer, are there any examples that you could share showing the difference? Finally, do you have any intuition why this is happening?
Best,
Konstantinos

Hi @ktertikas thanks for reaching out!
Compared to standard Pytorch3d mesh renderer, we modify two things:
1. Softmax blending > Alpha-compositing blending
We propose a new formulation in our previous work UNICORN on SVR because we observed that pytorch3d simply does not work when learning geometry from RGB rendering comparisons only (they use silhouettes to make it work). You can have a look at appendix A for why this is happening, but basically the standard softmax RGB blending formulation prevents gradient from flowing to the occupancy maps, which is necessary to update geometry. We raised an issue with working examples in pytorch3d repo showing that our alpha-compositing blending function make the learning work with RGB comparisons.
2. Adding mesh transparency values
We incorporate transparency values inside the blending function to be able to model a variable number of meshes in a differentiable manner, by simply setting some mesh transparency values to 0. This is our proposed solution to optimize over the number of meshes in a differentiable manner, other solutions typically imply reinforcement learning or greedy algorithms which are difficult to handle.
Let me know if this makes sense!

Hey @monniert , thanks for the great explanation!
I was not aware of the gradient issues on the current Pytorch3D Renderer, thanks for letting me know!
Re the transparency values, from my experience experimenting with parsimony losses similar to your proposed loss, the number of primitives that eventually contribute to the scene (alpha > 0.5) is quite noisy, and can even change significantly on different runs of the same scene (different seed per run). Did you experience something similar in your work?
Best,
Konstantinos

That's a good point. From what I observed, there is indeed some noise in the number of primitives contributing to the scene but nothing unrealistic. For example for the BlendeddMVS scene of the egg, it can sometimes be reconstructed with 2 or 3 primitives, with one sphere at the bottom and another small ellipsoid to reconstruct the top.
However, as stated in the paper, we observed that the training rendering loss was a good proxy to automatically select the good run amond severals ones.
Closing the issue, feel free to reopen if needed!

Great, thanks for the help and the insights!


Added documentation for final logic functions

This PR is to add the examples for the final four logic functions. As requested from https://github.com/evilsoft/crocks/issues/304

[x] not
[x] or
[x] unless
[x] when

Coverage remained the same at 100.0% when pulling 3f1093e0fa36c98ff56207d849361f8b3b17cc06 on docs/logic-functions into 0a9369da7ad5e92c1b30b1eab6d7b7ad71bbbd0f on master.

I think the examples for when and unless may be a bit lacking, if you agree any suggestions? i'd like to add Pred to them but couldn't think of a good example

Thanks for the review @evilsoft, I just reformatted my computer today, so give me 30min to finish setting this up and i'll get these up ASAP

moar 👀

@bennypowers just to get this out the 🚪 I am going to dismiss your review. I think I addressed your issue and I am not being dismissive of your review. I value your input!


Investigating assemblyScript 0.20 compatibility issues.

Since Assemblyscript 0.20 released, some old package maybe meet usage problem.
We need to investigate the dependencies we use and make sure they work fine.

The visitor-as/as-pect should be adapted to 0.20, if not we cannot upgrade our deps.


Grouped apps in taskbar

Description of the new feature / enhancement

Drag 1 app on the other, take 1 second for it to "Snap" into a group on hovering the second app.
Click a group -> a contextual taskbar appears containing the grouped applications.
Drag an application out of the taskbar, either to the desktop or taskbar.
Right click context menu per app in a group with the default action items

In essence, what discord provides for grouping servers and ios has for applications.
Scenario when this would be used?
Any kind of scenario where you want to group applications.
In this example i would have

"Terminals"

Powershell
Cmd
Cygwin
GitBash

"VS IDE's"

Visual Studio 2019
Visual Studio 2022
Visual Studio 2022 Preview

"Communication"

Discord
Whatsapp
Slack

"Entertainment"

Spotify
Netflix

Supporting information
No response

/dup https://github.com/microsoft/PowerToys/issues/264


Try

Submitted via Docs Editor.

Hi @adunndevster - is it ok if we close this?

ping @adunndevster

We're going to close this @adunndevster, let us know if you need to open it again.


ERROR: operator does not exist: date >= integer (SQLSTATE 42883)

GORM Playground Link
https://github.com/go-gorm/playground/pull/463
Description
On PostgreSQL 14.2, was comparing date in a WHERE clause, but get:
ERROR: operator does not exist: date >= integer (SQLSTATE 42883)

The comparison should work since date(now()) - 0 returns a date type (via psql), but somehow in gorm, it evaluates as an integer type.

	err := DB.
		Model(&User{}).
		Select("max(id)").
		Where("date(created_at) >= date(now()) - ?", clause.Expr{SQL: strconv.Itoa(0)}).
		Group("date(created_at)").
		Find(&max).Error

gorm sql log is not really executed sql, it will actually be converted to $1 $2 and create a prepared statement and deduce the type based on the incoming parameters, that's why you fail.
https://www.postgresql.org/docs/9.3/sql-prepare.html

Thanks @a631807682, those are very useful pointers.


Remove output tensors 1 - 4 from FusedBatchNorm

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): 1.10.0
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
Downloaded frozen inference model MobileNet_v1_1.0_224 from https://github.com/tensorflow/models/blob/master/research/slim/nets/mobilenet_v1.md
FusedBatchNorm has five outputs. 1-4 seem to be related to training. How do I remove them?
I tried the following code:
		if op.type == 'FusedBatchNorm':
			new_node = node_def_pb2.NodeDef()
			new_node.name = op.node_def.name
			new_node.op = op.node_def.op
			for i in op.node_def.input:
				new_node.input.extend([i])
			for a in op.node_def.attr:
				new_node.attr[a].CopyFrom(op.node_def.attr[a])
			output_graph.node.extend([new_node])

		else:
			new_node = node_def_pb2.NodeDef()
			new_node.CopyFrom(op.node_def)
			output_graph.node.extend([new_node])

It still has the output tensors 1 - 4.
Can this feature be added to Graph Transform tool?
Will this change the current api? How?
It won't. It's a new transformation in Graph Transformation tool.
Who will benefit with this feature?
Mobilenet users
Any Other info.

Have I written custom code? Yes
if op.type == 'FusedBatchNorm':
new_node = node_def_pb2.NodeDef()
new_node.name = op.node_def.name
new_node.op = op.node_def.op
for i in op.node_def.input:
new_node.input.extend([i])
for a in op.node_def.attr:
new_node.attr[a].CopyFrom(op.node_def.attr[a])
output_graph.node.extend([new_node])
	else:
		new_node = node_def_pb2.NodeDef()
		new_node.CopyFrom(op.node_def)
		output_graph.node.extend([new_node])

OS Platform and Distribution: Ubuntu 16.04
TensorFlow installed from: pip
Bazel version: N/A
CUDA/cuDNN version: N/A
GPU model and memory: No GPU support
Exact command to reproduce: N/A
Mobile device: N/A

@santoshchilkunda This is a valid point though Batch Normalization can be used for inference as well.
You can delete the 4 extra outputs in FusedBatchNorm leaving only output y which is a 4D tensor for output data. Try to use onnx tool to remove those extra outputs.

Nagging Assignee @wt-huang: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@wt-huang You can take a look at the solution provided here. Look at the comments of the answer for the solution.  Also, if the issue still persists, please post this issue in tensorflow/models as anything related to the models in that repo are discussed there. Thanks!!


Replace deprecated constants to fix warnings in HA 2024.1

Had these warnings in my logs, which are linked to changes in 2024.1 (https://developers.home-assistant.io/blog/2023/12/19/constant-deprecation/ and https://developers.home-assistant.io/blog/2023/12/28/support-feature-magic-numbers-deprecation/):
2023-12-31 14:43:35.731 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_AUTO was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.AUTO instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.737 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_COOL was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.COOL instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.741 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_DRY was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.DRY instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.745 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_HEAT was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.HEAT instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.749 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_HEAT_COOL was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.HEAT_COOL instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.753 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_OFF was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.OFF instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.757 WARNING (MainThread) [homeassistant.components.climate.const] SUPPORT_PRESET_MODE was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use ClimateEntityFeature.PRESET_MODE instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.762 WARNING (MainThread) [homeassistant.components.climate.const] SUPPORT_TARGET_TEMPERATURE was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use ClimateEntityFeature.TARGET_TEMPERATURE instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.766 WARNING (MainThread) [homeassistant.components.climate.const] SUPPORT_TARGET_TEMPERATURE_RANGE was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use ClimateEntityFeature.TARGET_TEMPERATURE_RANGE instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.771 WARNING (MainThread) [homeassistant.const] TEMP_CELSIUS was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.778 WARNING (MainThread) [homeassistant.components.fan] SUPPORT_PRESET_MODE was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use FanEntityFeature.PRESET_MODE instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.782 WARNING (MainThread) [homeassistant.components.fan] SUPPORT_SET_SPEED was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use FanEntityFeature.SET_SPEED instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.786 WARNING (MainThread) [homeassistant.components.fan] SUPPORT_PRESET_MODE was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use FanEntityFeature.PRESET_MODE instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.791 WARNING (MainThread) [homeassistant.components.fan] SUPPORT_SET_SPEED was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use FanEntityFeature.SET_SPEED instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.805 WARNING (MainThread) [homeassistant.const] ELECTRIC_CURRENT_AMPERE was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfElectricCurrent.AMPERE instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.809 WARNING (MainThread) [homeassistant.const] ELECTRIC_POTENTIAL_VOLT was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfElectricPotential.VOLT instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.814 WARNING (MainThread) [homeassistant.const] ENERGY_KILO_WATT_HOUR was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfEnergy.KILO_WATT_HOUR instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.819 WARNING (MainThread) [homeassistant.const] POWER_WATT was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfPower.WATT instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:35.824 WARNING (MainThread) [homeassistant.const] TEMP_CELSIUS was used from sonoff, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please create a bug report at https://github.com/AlexxIT/SonoffLAN/issues
2023-12-31 14:43:55.538 WARNING (MainThread) [homeassistant.components.light] Entity sonoff.sonoff_1000f7eeca (<class 'custom_components.sonoff.light.XLightL1'>) is using deprecated supported features values which will be removed in HA Core 2025.1. Instead it should use <LightEntityFeature.EFFECT|TRANSITION: 36> and color modes, please create a bug report at /api/sonoff/a97f87d0-f357-4739-b9c8-641792345b61 and reference https://developers.home-assistant.io/blog/2023/12/28/support-feature-magic-numbers-deprecation

With the changes, all warnings are gone. Tested locally, runs fine. I don't have a Sonoff climate device - someone should test this.
Needs HA >2022.5 because of https://developers.home-assistant.io/blog/2022/04/02/support-constants-deprecation/ and https://developers.home-assistant.io/blog/2022/05/03/constants-deprecations/.
Should fix #1278

For some reason, my L1 led light isn't working properly anymore with these changes. If I hit the light in HA, it will actually turn on but HA won't show the correct state (looks like it's still off). HA also can't turn it off anymore.

Hmm, looks like that might be unrelated to my changes, as I reverted back to the latest version and that didn't change the issue.

But UnitOfEnergy, UnitOfPower, UnitOfTemperature - 2022.11
UnitOfElectricCurrent, UnitOfElectricPotential - 2023.1

Yes, you are correct, I missed that. Don't know how you want to go about this? Do you want to keep supporting HA <2023.1, or is it ok to raise the minimum version to that one?
After all, the constant deprecation is valid for a year (final removal in 2025.1), but it will keep filling people's logs with the warning so I would suspect a lot of issues reported.

I've updated the PR to raise minimum version of HA to 2023.1. That's a year ago already....

@hmmbob Despite using the code you modified, the lights and switches still cannot synchronize status

@hmmbob Despite using the code you modified, the lights and switches still cannot synchronize status

That is correct, those are unrelated

Thanks! I agree that 1 year backward is OK. Support for older versions is very time consuming.

As for the other issues with 2024.1, bdraco has pinpointed where to look.

Commenting L40 in entity.py did fix the issue on my side.
As it is a backward compatibility thing for 2021 (and this PR is raising minimum version to 2023), it can probably be gone

Backward compatibility fixes break backward compatibility :)

@hmmbob I don't have permissions for writing to beta channel. So it's useless to ping me there :)

Yeah, they closed the channel after the release - no one can write to it currently. I pinged you somewhat before that. But hey at least you saw the info (which I copy-pasted here earlier 😉 )

@hmmbob I don't have permissions for writing to beta channel. So it's useless to ping me there :)

beta gets closed when current released is shipped, it will open with next beta.

https://github.com/AlexxIT/SonoffLAN/releases/tag/v3.5.4

Thanks for the update and fix!


Remove k6 tests and add plugin-e2e

Remove k6 tests and replace with the plugin-e2e package tests.

Closed in https://github.com/grafana/clickhouse-datasource/pull/940.


@import url('') deleted when minifying

Hi !
CleanCss made a hot update 4.0.0 that change the configuration...
For external @import to work you need to pass the configuration inline: ['none'] or inline: false or inline: ['local']
So... We must have this configuration in your package too, to pass it to cleanCss after !
Is it possible to do this soon ?
Thanks !

@emoc11 so this package uses<EMAIL_ADDRESS>which is the most current release.
You should be able to pass the option here. Does passing the option not work as expected?

It seems not i'm afraid...
Since 4.0 we can configure inline: false in cleanCss to be sure that it do nothing to @import but my @import url("") are still removed when i build with gulp-clean-css.. !

@emoc11 I was unable to track this down and am closing due to staleness. I have made various subsequent changes that may resolve your issue if you try to update. If the issue persists, please re-open a new issue with as much detail as possible and I'll help you look into it. Thanks!

I have exactly the same problem as described by the author.


Remove unnecessary configuration and global state usage

See issues #4 and #5.
Only the config file is used for storing the file list now. Additionally the ImportTo and ExportFrom configuration was removed. This allowed me to remove the configuration dialog and remove the dependency on system wide shared state.
Note that the ImportTo and ExportFrom feature was removed since it didn't support relative path names. By removing it, we obtain a simple, working implementation. We can add these features back when the time comes to implementing it.

@mattpalermo I've just gone to review this and got in a bit of a mess :confused: , which branches relate to this PR please?
Just some other things to note, when trying to make the project I found that all the modules were already in the binary so I had to remove them all and start again, the frmConfigure.frm couldn't be imported for some reason either.

The noconfig branch contains the changes relating to simplifying / removing the configuration and removing the dependence on global state.
Make sure that you checkout the branch properly with a clean working directory. In this branch, there is no binary. The instructions to build the binary can be found in the README.
The next pull request is for the self-use branch. It builds on changes made in the noconfig branch. The changes made in this branch simplifies the build process by including a template binary and a configuration file for using a copy of this add-in to automatically import the files into the template.
I intended for you to look at this pull request first and then the other pull request if this one is accepted.

Here is a link to my noconfig branch on github https://github.com/mattpalermo/VBA-IDE-Code-Export/tree/noconfig

Just so we're clear:

noconfig is the one you want me to look at first
self-use is to be looked at after noconfig has been merged

Thanks :smile:

That's right :)


GH #1234 - Ignore empty cookie keys

Without this check for the 'key' variable, there is an empty key in the resulting dictionary, e.g:
{
   'first': u'IamTheFirst',
   'a': u'2',
   'second': u'andMeTwo',
   '': ''
}

Manually triggered re-build. There is a style error, please fix it by validating with flake8.

Flake8 issue resolved


Leaving Navbuddy via either actions.insert_name or actions.select removes cursor

Version
NVIM v0.9.0-dev-1117+ga4f443994-dirty
Build type: Release
LuaJIT 2.1.0-beta3
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/opt/homebrew/Cellar/neovim/HEAD-a4f4439/share/nvim"
Configuration (Lazy.nvim)
return {
    "SmiteshP/nvim-navbuddy",
    lazy = false,
    dependencies = {
        "neovim/nvim-lspconfig",
        "SmiteshP/nvim-navic",
        "MunifTanjim/nui.nvim"
    },
    keys = {
          {"<leader>nv", "<cmd>Navbuddy<cr>", desc = "Nav"},
    },
    config = function ()
        local actions = require("nvim-navbuddy.actions")
        local navbuddy = require("nvim-navbuddy")
        navbuddy.setup({
            window = {
                border = "double"
            },
            mappings = {
                ["k"] = actions.next_sibling,
                ["i"] = actions.previous_sibling,
                ["j"] = actions.parent,
                ["l"] = actions.children,
                ["h"] = actions.insert_name,
                ["<enter>"] = actions.select,

            },
            lsp = {auto_attach = true}
        })
    end
}

Additional notes
Full dot files can be found here
I should note that I have fairly  unconventional mappings for movement and entering insert mode:
vim.keymap.set("n", "j", "h")
vim.keymap.set("n", "k", "j")
vim.keymap.set("n", "i", "k")
vim.keymap.set("n", "h", "i")

vim.keymap.set("v", "j", "h")
vim.keymap.set("v", "k", "j")
vim.keymap.set("v", "i", "k")

Here as an example screenshot of what normal mode looks like after exiting navbuddy by pressing enter:

It strangely almost looks like visual line mode, but is not, and the cursor no longer can be found.

Seems related to #14

My bad, did not see this, closing this.