Only load scripts the first time the adapter connects

This should fix issue #919 where hubot reloads scripts
each time the adapter emits 'connected'
Because some adapters handle retries for lost connections, they
may emit such an event many times. However hubot should not load
scripts more than once.
No tests added, bin/hubot is hardly testable and the modification
does not break any existing test

+1

the automatic checks say I didn't agree to the Github CLA but I did :(

This is a duplicate of https://github.com/github/hubot/pull/1069

yes.

but mine passes travis somhow :/

@Floby we dropped node 0.8 for travis between my PR and yours.


Allow page-level types to be deprecated and superseded

This is a change from any content being able to be deprecated, to just top-level page types (i.e. those that extend SeguePage). We did not do anything with deprecated set on any other content, so this change is safe to make and reflects the actual behaviour.
This also moves supersededBy to the SeguePage and subclasses, so that more than just question pages can be superseded.
It also tidies up other classes that had one or both of these properties but where we were not using them. A similar change to the frontend types exists to demonstrate that we only used these for SeguePage level objects.
I have tested that this has not broken any existing content, and also verified that ContentSummaryDTOs still get the two properties mapped across as expected.

This is probably where the properties should have been added in the first place. I am still a little worried about odd cases like wildcards or past question attempts where deprecated might have gotten into the database and now be unable to be un-mapped. I believe wildcards are the only problematic case where we used an unconfigured ObjectMapper which included null values, however!

This goes with https://github.com/isaacphysics/isaac-react-app/pull/963, although neither depends on the other! They both confirm that the move is safe to make, however.


log_pos() isn't working

log_pos() only returns command prompt.  Entry is not made in Olog. grabit and logit work okay.

does wh_pos() work?  Do either work if you pass them a list of devices?

I suspect you need to add https://nsls-ii.github.io/beamline-configuration.html#integration-with-ophyd
# This is for ophyd.commands.get_logbook, which simply looks for
# a variable called 'logbook' in the global IPython namespace.
logbook = simple_olog_client

to your startup files.

Tried by hands, it seems to work fine now.
Thanks.
I am going to add it to our olog_config file

It looks like this was resolved. Closing.


esp carFw missing. Car is 2021 Hyundai Sonata SEL

Description
Added another fingerprint for 2021 Hyundai Sonata SEL
Verification Made sure my car would be recognized by the comma two.
Route
Route: comma two 654df30d2b0bf782|2021-01-29--17-21-04

Description
Added another fingerprint for 2021 Hyundai Sonata SEL
Verification Made sure my car would be recognized by the comma two.
Route
Route: comma two 654df30d2b0bf782|2021-01-29--17-21-04

The carFW is empty in my case. Did you see the same issue? If so, have you figure it out?


[QUESTION] graphq subscription error/support

How can I make subscriptions of my graphene schema work with fastapi?
Right now I following the tutorial of the documentation and subscriptions don't work with that. Maybe it is a question for Starlette lib.
Thanks?

Yeah, that comes directly from Starlette, and any GraphQL stuff would be handled directly by Graphene.
You might also want to check out https://ariadnegraphql.org/


Yash

Sign/Login Page (Purple/Lavender Theme)

Improvements
Hello, @yashgoyal2802. Loved  the layout. Exactly what was needed, but just a couple of things to update upon.

[ ] Remove the scroll for the login and signup page. Let it occupy the whole page!
[ ] Use a good font for the page. I'd suggest this or maybe Trebuchet MS
[ ] Add a required attribute to the <input type="checkbox" > for the Sign-Up page so the user cannot create an account without agreeing to the terms and conditions!
[ ] Add a vector image on the left (  I'll be sending one later tonight )

Re-open the PR, when the changes are made!


Supplement the Portuguese (Portugal) translation

Supplementing translations regarding the resource language preference switcher (as asked in issue #143).

Thanks @joseborges!

Your welcome @mattstauffer


Can you please link to similar projects?

For those of us new to the React space, and shopping for component libraries, especially in a functional style, it would be useful to get some links to what you consider "close" in style, or "on the right path" but not quite there, which is why you made rebass. I'd also like to hear about the trade-offs you made designing this library. Why did you alias well-known style attributes, for example? Are you optimizing for JSX read ability? Compactness? Simplicity at runtime? And so on.
Optional, but appreciated.

There is a Related section in the docs https://github.com/jxnblk/rebass#related
I don't know of any other component libraries out there like Rebass (stateless, customizable, design-system-based, etc.), and that's part of the reason I started this. This post gets at a little more of the philosophy behind Rebass http://jxnblk.com/writing/posts/patterns-for-style-composition-in-react/
As for trade-offs, personally I don't think that I've made any in this library. There might be some performance issues which I hope will generally improve over time, but I think it's good enough for a production application.
Feel free to continue the conversation here or reopen this issue if you have any other specific questions


Help!

I can't get the script to stay monitoring, i got like 30+ working proxies every time i start the script this happens,

Download and run in same folder: https://github.com/supthunder/premeStock/blob/master/run.py
Also no proxies are loaded, you have to paste proxies list from proxies.txt into proxies variable

https://github.com/supthunder/premeStock/wiki/How-to-run

Unrelated:
Is it possible to add Slack Integration into your script @supthunder

Is it possible to make the masterstock.py save masterstock.txt in this format {'170459':'Studded Belt'} (just like proxyloader.py)  instead of, to make it easier to copy and paste.
"category_name": "Accessories",
"id": 170459,
"image_url": "//d17ol771963kd3.cloudfront.net/128530/ca/r2u51G9hEqY.jpg",
"image_url_hi": "//d17ol771963kd3.cloudfront.net/128530/rc/r2u51G9hEqY.jpg",
"name": "Studded Belt",
"new_item": true,
"position": 30,
"price": 14800,
"sale_price": 0
}

@Bleez103 added slack

Much appreciated @supthunder

quick question how can i make it tweet or use slack just for testing @supthunder

@Jeisonx3 tweet: go to apps.twitter.com create an API key, and paste that in tokens.py, with slack just edit the slackMsg() -> slack_token variable in monitor.py
Links:

Twitter API Key
Slack API Key

To test uncomment this line

@supthunder Getting this while testing.

Slacked addition worked perfectly for this weeks release @supthunder . If you have a paypal I'd be happy to donate.
Will it be possible to do a restock say for Adidas with similar Slack compatibility?

@Bleez103 will add more next week, busy this week with school


Write Gauge value to display not deleting prior value

Hello,
Excellent library. I am enjoying it.
I have a gauge on the display with a value 90 as it's default value. I send an Arduino command to set it to 180 and the dial moves to 180. However, it is still showing the old 90 value dial hand. So basically I have two dial hands; one at 90 and the other at 180. Is there some clear command I should put in prior to changing the value?
I tried both these ways:
NextionComponent gauge0(nextion, 0, 2);

void loop(){
   //gauge0.value(180);
    gauge0.attribute("val", 180);
}

thanks for the help.

Did you update your comport instance?
NextionComPort nextion;
...
void loop(){
gauge0.attribute("val", 180);
nextion.update();
}

I tried the update but didn't work either. When debuging inside Nextion Editor the issue is the same. I found out that if the gauge is set for crop image and located ontop of a picture as a full background this happens. If I switch the gauge to "solid color" the gauge works correctly. So it is a refresh thing.
I found a refresh command "ref 0" where 0 is the id of the control to refresh. Page0 is ID 0. So by changing my guage values followed by sending a page refresh fixed the issue in the Nextion Editor.
How do I send a page refresh via this library?
Thanks again

I think you must the command "page x". See https://wiki.iteadstudio.com/Nextion_Instruction_Set

Thanks, but how do I send a generic command via the library?

Have a look to the documentation. e.g. nextion.command("cir 50,50,20,WHITE");


[BACKPORT][1.0.1] Allow to preserve OS images when run 'epicli apply' for existing cluster

Backport issues #2288
Related pull requests: #2329

Tested together with #2333.
Tested:
✅ apply v0.9 -> upgrade to v1.0 -> apply v1.0
✅  apply v1.0-> apply v1.0
I checked different values for default_os_image parameter:

ubuntu-18.04-x86_64

redhat-7-x86_64

centos-7-x86_64

centos-7-arm64 (only terraform part as arm support will be introduced in v1.1.0)

default

also with changed default specification:
---
kind: infrastructure/virtual-machine
title: "Virtual Machine Infra"
provider: aws
name: default
specification:
  os_full_name: RHEL-7.9_HVM-20210208-x86_64-0-Hourly2-GP2

both for Azure and AWS.


dotnet commands (test/pack) do not work if project depends on authenticated feed

There's dotnet commands which implicitly build the projects and implicitly restore packages if needed.
In the case for dotnet pack, the task should probably pass the --no-restore to avoid restoring packages.
There should also be documentation saying that if the project depends on packages coming from authenticated builds, a dotnet restore should be put in place before other dotnet tasks.

Ran into the same issue, it's too bad that the preview task for dotnet when in pack mode doesn't accept arguments (like test does for example)

Closing this as WF. In the upcoming quarter, we'll be moving away from the full-featured Artifacts-related tasks (which have many issues like this because they don't provide the full range of commands supported by the underlying tool) and recommending calling dotnet directly. See the design here for more details.


Replace goamz with official go AWS SDK

Does anyone have any reason not to replace the goamz AWS client binding with the official AWS client binding, now that it has reached feature-complete:
https://aws.amazon.com/blogs/aws/preview-the-latest-updates-in-the-master-branch-aws-sdk-for-go/
The goamz binding we're using:

is incomplete - for example, there is no way to page through EC2 instances, so fails if you have more than 50 (I think that's the default page size)
has been forked all over the web but never re-integrated - there is no real "goamz" library.
is likely dead now that there is an official AWS library

Obviously the new library may have bugs, but I personally would rather deal with those than continue to invest in goamz.

Sounds good to me.

I'm working on this. Some features from goamz are still being merged into aws-sdk-go.

@jsdir : it's great that you are working on this.  Metadata retrieval is a big one to leave out.  My guess is that it's because there isn't a schema for it (maybe!)
How are you working on the port & how is it going?  Are you building a second implementation of the EC2 interface?  (That is what I had in mind when putting so many methods into EC2/goamzEC2).  Is your code visible on a branch somewhere?

@justinsb The initial port is at my 6162-aws-sdk-go branch. Although some parts of the interface were still bound to the goamz implementation, aws-sdk-go's Instance struct is very similar to that of goamz.

Waiting for the following:

[ ] GetMetadata https://github.com/awslabs/aws-sdk-go/pull/192
[ ] IsRegionValid https://github.com/awslabs/aws-sdk-go/issues/108

@jsdir: I wonder if we should just pull in your code for GetMetadata and implement something similar for IsRegionValid, until something is put into the official aws-sdk-go.
I'm going to be (re)implementing ELB soonish, which brings in a whole new set of API calls, so I'd much rather be using aws-sdk-go than creating more porting work...
Thoughts?

@justinsb: I agree, the development at aws-sdk-go is not as punctual as I had hoped. I'll add GetMetadata and IsRegionValid to the AWS cloudprovider for now.

@justinsb: You can continue porting the cloudprovider to aws-sdk-go. There have been many additions since my last changes and you would probably be more familiar with their use.

@justinsb I've got some free time today, I can try to pick up the implementation of this if you want?
As it stands @jsdir's PRs are still not merged into aws-sdk-go so I'll just reimplement in kubernetes.

@iterion That would be awesome - thank you!

This should be closed as of #8297, no?

Yeah, this should be closed now.


#5120: Unit Tests failing due to testcontainers

Description
Upgraded the version of testcontainers
Related issue and scope

#5120 Unit Tests failing due to testcontainers

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Scheduler
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[x] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Thanks for catching this @vrann !

Codecov Report

Merging #5121 (1f5ad87) into master (f1829e1) will decrease coverage by 13.69%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #5121       +/-   ##
===========================================
- Coverage   81.86%   68.17%   -13.70%
===========================================
  Files         225      218        -7
  Lines       11423    11158      -265
  Branches      501      497        -4
===========================================
- Hits         9352     7607     -1745
- Misses       2071     3551     +1480

Impacted Files
Coverage Δ

...che/openwhisk/core/invoker/LogStoreCollector.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...hisk/core/scheduler/message/ContainerMessage.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...core/database/cosmosdb/RxObservableImplicits.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedConsumer.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0.00% <0.00%> (-95.85%)
:arrow_down:

.../openwhisk/core/scheduler/FPCSchedulerServer.scala
0.00% <0.00%> (-95.84%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0.00% <0.00%> (-93.90%)
:arrow_down:

...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala
0.00% <0.00%> (-92.31%)
:arrow_down:

...sk/core/scheduler/grpc/ActivationServiceImpl.scala
0.00% <0.00%> (-88.00%)
:arrow_down:

... and 62 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 f1829e1...1f5ad87. Read the comment docs.


Copying a project board doesn't copy/retain notes & issues from original board to the newly created board

Prerequisites

[x] ~Can you reproduce the problem with Debug -> Reload Without Extensions?~ Tried in a browser that didnt' have extensions
[x] Did you perform a cursory search to see if your bug or enhancement is already reported?
[x] Did you read the Troubleshooting guide?

For more information on how to write a good bug report read here
For more information on how to contribute read here
Description
Copying a project board doesn't copy/retain notes & issues from original board to the newly created board
Steps to Reproduce

Select a project board, ensuring there are notes and cards on it that can be copied to the new board
Select Menu > ... > Copy
Choose an owner then create copy of board

Expected behavior: [What you expected to happen]
New board is created with all of the same columns, automation, notes and issues on it. (Previous functionality.)
Actual behavior: [What actually happened]
New board is created with columns & automation, but notes and issues are not present.
Note: Our team uses this functionality to define all essential issues (in the form of notes) that need to be on a project when it is created. With these notes now missing, this workflow is now impacted. 😞
Versions
Win 10.1803 (17134.1365)
Chrome Version 80.0.3987.162 (Official Build) (64-bit)
Firefox 75.0b9 (64-bit)

Wrong repo! 😊


Add disable_xcpretty option to scan

🔑Guidelines read
New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Debugging CI can be a challenge when using xcpretty. While gym has support for disabling it, scan does not.
It'd be great to have the same functionality in scan by adding a disable_xcpretty option.
As a reference, this is the original Issue to add this option to gym: https://github.com/fastlane/fastlane/issues/5035
Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
10.13.4

Ruby
2.2.5

Bundler?
true

Git
git version 2.14.1

Installation Source
~/.rvm/gems/ruby-2.2.5/bin/fastlane

Host
Mac OS X 10.13.4 (17E199)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.2.5/lib

OpenSSL Version
OpenSSL 1.0.2e 3 Dec 2015

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.3

System Locale

Variable
Value

LANG
en_GB.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./.fastlane/Fastfile`
desc "Runs tests on the primary platforms and configurations"
lane :test do
    verify(
        scheme: "MCTDataStructures iOS"
    )
    verify(
        scheme: "MCTDataStructures OSX",
        destination: "platform=macOS"
    )
    verify_swiftpm
    pod_lib_lint
end

desc "Runs unit tests"
lane :verify do |options|
    scheme = options[:scheme]
    destination = options[:destination]
    sdk = options[:sdk]
    output_directory = ".fastlane/test_output/" + scheme + "/"
    opts = {
        :project => 'MCTDataStructures.xcodeproj',
        :scheme => scheme,
        :destination => destination,
        :sdk => sdk,
        :code_coverage => true,
        :xcargs => "ONLY_ACTIVE_ARCH=YES",
        :output_directory => output_directory,
        :clean => true
    }
    scan(opts)
    upload_cov
end

desc "Runs unit tests using Swift Package Manager"
lane :verify_swiftpm do
    sh("swift test")
end

desc "Upload code coverage reports (if running on CI)"
lane :upload_cov do
    if is_ci?
        sh "bash ./codecov.sh"
    else
        say "No need to upload code coverage. Not running from CI."

This is still an issue.

Of note: if you specify output_style: "raw" in your options, it has the effect of disabling xcpretty for  fastlane scan.
https://github.com/fastlane/fastlane/blob/master/scan/lib/scan/test_command_generator.rb#L82-L84
I came across this because I too wanted to disable xcpretty for scan and was working on a PR to add it, but then saw the above code.
The fastlane scan docs could probably be a bit clearer on this point, especially since other fastlane actions (like gym) do have an explicit disable_xcpretty option.


Fix Chris' crash

handleBackEvent() {
    const { navigation } = this.navigator.props;

you want to check here @edmofro , this is where it crashes

Right you are @andreievg, was moving too fast. Should be fixed now. Again, untested.


[v1.2.0]heapster crash when get version

[root@ip-172-18-6-200 heapster]# git checkout v1.2.0
HEAD is now at eea8c96... Merge pull request #1291 from piosz/rel-1.2
[root@ip-172-18-6-200 heapster]# git branch
* (detached from v1.2.0)
  master
[root@ip-172-18-6-200 heapster]# make
...
[root@ip-172-18-6-200 heapster]# ./heapster version
F1107 04:26:47.195087   15124 heapster.go:77] Wrong number of sources specified
goroutine 1 [running]:
k8s.io/heapster/vendor/github.com/golang/glog.stacks(0x3549e00, 0x0, 0x0, 0x0)
	/data/src/k8s.io/heapster/vendor/github.com/golang/glog/glog.go:766 +0xb8
k8s.io/heapster/vendor/github.com/golang/glog.(*loggingT).output(0x3529b00, 0xc800000003, 0xc8200b6300, 0x34f5662, 0xb, 0x4d, 0x0)
	/data/src/k8s.io/heapster/vendor/github.com/golang/glog/glog.go:717 +0x259
k8s.io/heapster/vendor/github.com/golang/glog.(*loggingT).printDepth(0x3529b00, 0x3, 0x1, 0xc8204b7d90, 0x1, 0x1)
	/data/src/k8s.io/heapster/vendor/github.com/golang/glog/glog.go:646 +0x1b8
k8s.io/heapster/vendor/github.com/golang/glog.(*loggingT).print(0x3529b00, 0x3, 0xc8204b7d90, 0x1, 0x1)
	/data/src/k8s.io/heapster/vendor/github.com/golang/glog/glog.go:637 +0x50
k8s.io/heapster/vendor/github.com/golang/glog.Fatal(0xc8204b7d90, 0x1, 0x1)
	/data/src/k8s.io/heapster/vendor/github.com/golang/glog/glog.go:1125 +0x49
main.main()
	/data/src/k8s.io/heapster/metrics/heapster.go:77 +0x48f

Master branch works correctly.
[root@ip-172-18-6-200 heapster]# ./heapster version
I1107 04:34:24.818192   18164 heapster.go:65] ./heapster version
I1107 04:34:24.818241   18164 heapster.go:66] Heapster version 1.2.0
F1107 04:34:24.818248   18164 heapster.go:73] Failed to get kubernetes address: No kubernetes source found.

@DirectXMan12 @mwringe

There is no 'heapster version' command. Heapster always prints out the version as part of its startup, and as you can see in both casese it ends in an error. All that has changed between 1.2.0 and master is the error message.

Closing as WAI (see @mwringe comment).


Fix thread leak in FileDownloadDelegate

Fixes https://github.com/swift-server/async-http-client/issues/364
Modification

remember if we own the thread pool or not.
shutdown the thread pool if we own it after we are done with it

Result
we no longer leak one thread per download

API breakage failure is a false positive
16:14:03 1 breaking change detected in AsyncHTTPClient:
16:14:03   💔 API breakage: constructor FileDownloadDelegate.init(path:pool:reportHead:reportProgress:) has removed default argument from parameter 1

The change in question can be simplified to:
struct Foo {
    // old init
    init(bar: Int = 0) {}
    // new inits
    init() {}
    init(bar: Int) {}
}

replacing a default argument with two inits, one with the argument and one without, is an ABI breaking change but not an API breaking change.

I have changed the implementation because we would otherwise now only ever use one thread for all file IO. This is not good because file IO can block the thread and there is now easy way of making it non-blocking. We therefore initialize the thread pool with ProcessInfo.processInfo.processorCount threads. As this is potentially much more expensive operation then just creating one thread we also delay initialization until the first actual write.

@Lukasa test added in https://github.com/swift-server/async-http-client/pull/614/commits/9ee5c82cfbf732d7cf2aab533c1f35392b8d8680


application startup takes more than 50 secs

Thanks for providing the slush-wean to build native apps quickly.
After build the app on windows using gulp build-win.When open  the app ,it takes more than 50 secs.Can you please provide a solution to reduce the startup time.

closing as no update


A few minor cleanups

This PR contains a few minor cleanups in preparation for requestifying default argument typechecking.

@swift-ci please smoke test

Why does the entire interface have to drop const?  Is it possible to provide a mutable accessor for friend access?


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter twinkly for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.twinkly
   -
:thumbsup: No errors found
WARNINGS:

[ ] :eyes: [W028] Minimum node.js version 18 recommended. Please adapt "{'engines' : { 'node' >= '16' } }" at package.json.
[ ] :eyes: [W156] js-controller 4.0.24 listed as dependency but 5.0.19 is recommended. Please consider updating dependency at io-package.json.
[ ] :eyes: [W529] Plugin "manual-review" missing at .releaseconfig.json. Please add.
[ ] :eyes: [W953] .npmignore found - consider using package.json object "files" instead.

SUGGESTIONS:

[ ] :pushpin: [S522] Please consider migrating to admin 5 UI (jsonConfig).

Please review issues reported and consider fixing them as soon as appropiate.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Fri, 13 Sep 2024 13:57:10 GMT based on commit 5c207e071e46f5fcf5f554231315deaffd38690c
ioBroker.repochecker 3.0.5

This issue has been updated by ioBroker Check and Service Bot
The following issues have been fixed
[E955] iob_npm.done not found in .npmignore
:thumbsup:Thanks for fixing the issues.


Fix explicit property when making autoprop

Closes #687

@giggio This branch has conflict.

@carloscds fixed


Is it possible to merge cells in table?

like colspan in html. I see something maybe helpful:
https://shd101wyy.github.io/markdown-preview-enhanced/#/markdown-basics?id=table

seems that a plugin for markdown-it is the way out?
https://www.npmjs.com/package/markdown-it-multimd-table

well, I write a simple helper to do some regex/replace hack to get it work.
| header1 | header2 | header3 | header4 |
| --- | --- | --- | --- |
| a | b | c |
| {colspan=4} | < | < | < |
| | {colspan=3} merge 3 cells | < | < |
| d | e | {colspan=2} f | < |

Handlebars.registerHelper('custom_transform', function (options) {
  // get the original output html
  var origin = options.fn(this);

  // do some hacks
  var transformed = origin.replaceAll("<	", "");
  transformed = transformed.replaceAll(/\{colspan=([0-9]+)}/g, "");

  return new Handlebars.SafeString(transformed);
});


CookieHandler.php not allowed on a free hosting

Never saw such a thing. But you can rename the file to something else, and change the new file name wherever necessary,

I see i use token system.
Let me remove it.

What release are you guys using? I just finished implementing MOFHY-Lite into my hosting site, and my CookieHandler.php doesn't look like the one currently in the repository.

Latest Dev release


Use lower-case gRPC header keys

As upper-case letters are illegal per http2 spec.

#2204

@yang-g Thanks for the fix!  Travis is going to barf whenever it gets around to running, because the unit tests assert the upper-case field names:
======================================================================
FAIL: test___call__ (gcloud.test__helpers.TestMetadataPlugin)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/tseaver/projects/agendaless/Google/src/gcloud-python/gcloud/test__helpers.py", line 845, in test___call__
    self.assertEqual(callback_args, [(cb_headers, None)])
AssertionError: Lists differ: [([('authorization', 'Bearer F... != [([('Authorization', 'Bearer F...

First differing element 0:
([('authorization', 'Bearer FOOBARBAZ'), ('user-agent', 'USER_AGENT')], None)
([('Authorization', 'Bearer FOOBARBAZ'), ('User-agent', 'USER_AGENT')], None)

- [([('authorization', 'Bearer FOOBARBAZ'), ('user-agent', 'USER_AGENT')], None)]
?      ^                                      ^

+ [([('Authorization', 'Bearer FOOBARBAZ'), ('User-agent', 'USER_AGENT')], None)]
?      ^                                      ^

Can you please update the PR accordingly?
@dhermes, @daspecster For future reference, here is the HTTP/2 spec reference which mandates lower-case for header field names (I was completely unaware of that requirement).

I cancelled the not-yet-started Travis build after verifying that it would fail.

@tseaver I pushed eebb83a about the same time of your comment. I did a search and did not find anything else. Could you restart the travis-ci or point me to more places to fix? Thanks.

@yang-g I restarted the build.  Sorry for my haste in cancelling -- Travis has been so glacially slow lately that we've been minimizing "known broken" jobs in the queue.

@tseaver I think Travis have sorted out their issues

@yang-g Thank you again!


⚠ Bofrost.de has degraded performance

In 34b858f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1206 ms

Resolved: Bofrost.de performance has improved in cd653cf after 46 minutes.


Raffle Wheel...Spin to Win!

TEST PLAN

If you are not in the list of names who can spin the wheel, you will either need to get one of those people to spin it or be added to the list. The wheel will only work in a domain where the spinner is an ADMIN. There should be more than one person present with an account to be sure you can see the names change.
-Go to zaru, up on the roof /91.6864,18.2118,45.1662 to the wheel and trigger or mouse click on the outer edge (away from the text entity on top of it) it to spin it.
-It should flash through all usernames of avatars in the domain who have accounts as the wheel spins. As the wheel, slows down, the names change more slowly and eventually stop when the wheel stops.
-The winner's name is displayed.

Approved as long as I win every time xD


feature/changes to publish the library on npm

ScrollingFix: Changing scroll detection algorithm to use relative scroll distances
ScrollingFix: Preventing negative offset values
ScrollingFix: Changing start/end to be the rendered start and end
StickyFix: Last test fixes
fix: fixes behaviour where upwards scrolling of first rendered data did cut off half of the header, if header and rows are not equal in height
chore: changes everything to publish to npm

Looks good 👍
Not sure if you are allowed to remove the author with MIT License 🤔

Looks good 👍
Not sure if you are allowed to remove the author with MIT License 🤔

@philippspindler but I didn't remove the license file and the origin author from it?

@philippspindler but I didn't remove the license file and the origin author from it?

looks good, how you did it now

@philippspindler but I didn't remove the license file and the origin author from it?

looks good, how you did it now

ah u meant the info in the package.json ? that's the only thing I was not sure about.. because actually, if we publish it, aren't we the author?
the license file with the copyright of the creator did always stay in this PR


My OLED is not working

I followed the instructions here: https://github.com/juj/fbcp-ili9341/blob/master/README.md
And I was trying to connect the Adafruit OLED 128x128 1.5' board to my Raspberry Pi 4B+, but it is just not working. The display seemed like it never turned on. I don't know what is happening. I had it working before on the exact same setting but I don't know why it is not working this time. I checked the wiring and everything.
I followed this:
sudo apt-get install cmake
cd ~
git clone https://github.com/juj/fbcp-ili9341.git
cd fbcp-ili9341
mkdir build
cd build
cmake -DILI9341=ON -DGPIO_TFT_DATA_CONTROL=27 -DGPIO_TFT_RESET_PIN=22 -DSPI_BUS_CLOCK_DIVISOR=30 -DARMV8A=ON -DDISPLAY_CROPPED_INSTEAD_OF_SCALING=OFF -DDISPLAY_BREAK_ASPECT_RATIO_WHEN_SCALING=OFF -DSTATISTICS=2 -DUSE_DMA_TRANSFERS=OFF -DDISPLAY_INVERT_COLORS=OFF -DDISPLAY_ROTATE_180_DEGREES=ON ..
make -j
and I made sure to edit /etc/rc.local to make sure it launches at startup.
But it is not working at all.
A few months ago, I used the exact same setting and it worked. This time I also tried using different -DSPI_BUS_CLOCK_DIVISOR ranging from 20 to 60. I tried all the even numbers between them. It is just not working. I'm not sure what is going on

Do you have a link to the display? The -DILI9341=ON option will definitely not work without modifications on a 128x128 display, because it assumes a 320x240 ILI9341 controller based display.

Do you have a link to the display? The -DILI9341=ON option will definitely not work without modifications on a 128x128 display, because it assumes a 320x240 ILI9341 controller based display.

If it is this one https://www.adafruit.com/product/1431, try using -DSSD1351=ON instead of -DILI9341=ON option (delete CMakeCache.txt when changing that option)

If it is this one https://www.adafruit.com/product/1431, try using -DSSD1351=ON instead of -DILI9341=ON option (delete CMakeCache.txt when changing that option)


Email not sent in Docker Login to Auth Server

It appears that when performing a docker login against the registry v2, no email address is sent to the token auth server.  I believe this to be a bug.
This is a deviation from v1 and I believe it necessary to include the email address for account identification, password resets, etc.
Leaving the email address out is acceptable if the docker login process should no longer be allowed to create accounts, but I think would be bad user experience and according to the docker client documentation login should still register accounts.
/cc @stevvooe @jlhawn

Creating accounts from the "login" command is a terrible idea in the first place IMO.

@dmp42 perhaps, however it does provide for a nice user experience all from the command line.
If this is to change that is fine, but I believe there should be a docker register for example to handle this then. If we remove this functionality from the client entirely then it puts the burden on having a web based UI on the auth server to register accounts which I believe would be less than ideal for user experience.

@dmp42 either way for feature parity and with the impending 1.6 release I think that this needs to be fixed to submit the email address somehow to the auth server.

Seems like this needs to be done on the docker client side. Around here (https://github.com/docker/docker/blob/08b2f5b9415f566f5f80c3ff26ff35ed21035541/registry/token.go#L55) I would send email if the scope is empty.

@dmp42 perhaps, however it does provide for a nice user experience all from the command line.

I disagree, and I think this is very odd currently.
For example, git does not provide a way to create "accounts" on remote git repositories.
Furthermore, such a mechanism imposes undue constraints on the registry registration mechanism and is a security risk (not to mention questionable UX currently).
The functionality is not removed from the client, it is still present in the v1 code, so, I don't think we are loosing anything here (it's not part of the v2 protocol, though).
Would be happy to discuss this more, and if you still disagree, by all means bring this up for discussion as a proposal for the v2 protocol.

I disagree, and I think this is very odd currently.

I agree that the implementation could be a little better, perhaps a separate register command, or prompting to verify password, etc.

For example, git does not provide a way to create "accounts" on remote git repositories.

I don't see this as a valid comparison. It all depends on implementation. Git was never design to be centralized with authentication, it has evolved over time. Solutions like Gitosis allows for management of repositories via git. GitHub has had to develop software specifically to manage permissions for git repos.
AFAIK Docker was designed from near day 1 to have a registry with authentication, it makes sense to have account creation in the command line client when the public registry is the primary place and by default the only place to get images. (especially for new users)

Furthermore, such a mechanism imposes undue constraints on the registry registration mechanism and is a security risk (not to mention questionable UX currently).

How is it imposing undue constraints? How is it a security risk?
There is nothing to say that the index (in v1) or the auth server (in v2) HAS to accept the user's information and create the account, that is entirely up to the implementation of the auth service. If you take a look at my index implementation for v1, there is an option to disable account creation via docker login, I plan to do the same for v2.

The functionality is not removed from the client, it is still present in the v1 code, so, I don't think we are loosing anything here (it's not part of the v2 protocol, though).

You are correct that as long as v1 registry is being used, no functionality is lost. However as I understand it Docker internally is working on moving to the new token auth server and using registry version 2 with the public registry, this would be a loss in functionality, as the auth service would never be able to know the email address of new users.

Would be happy to discuss this more, and if you still disagree, by all means bring this up for discussion as a proposal for the v2 protocol.

Just point me to where I can bring this up. I think it is the only logical step unless the docker login process is going to be changed for v2 and no longer allow users to register via the client.

Thanks for your answer @ekristen

it makes sense to have account creation in the command line client when the public registry is the primary place and by default the only place to get images. (especially for new users)

You don't need an account to consume public images.
You do need one to publish images, and to consume private images, true.

I agree that the implementation could be a little better, perhaps a separate register command, or prompting to verify password, etc.
[...]
How is it imposing undue constraints? How is it a security risk?

The implementation must be better IMO.
UX and undue constraints:

docker currently reports account activation is required on any registry (see for example: https://github.com/docker/docker/issues/7376) (causing quite some confusion)
implied account creation on a "login" command is (very) deceptive

Security risks:

registries without validation mechanisms: fat-fingering your account name will transparently create a new account on that registry, and then happily let you publish your (possibly private) content into public.
having an endpoint that would blindly create account without any sort of restriction opens the door for massive spam / dos

Other comparable systems: IIRC neither npm, nor pip (others?) let you create an account on the command line.

Just point me to where I can bring this up. I think it is the only logical step unless the docker login process is going to be changed for v2 and no longer allow users to register via the client.

To summarize:

I'm not inherently opposed to such a feature, and this is definitely an open-ended conversation
the current v1 implementation of this feature is ill-formed, and either require manual activation through an email + web browser (defeating the purpose), or is a dangerous feature
adding this to the v2 protocol is certainly open to discussion, in the form of a proposal (either here, or on docker/docker)

that proposal should detail the UX (probably a separate command, like you suggested with "register")
normative information on what an auth service should do if (not) implementing this must be documented extensively (handled error codes in different conditions, etc)

v2 is an open protocol, and we can certainly extend it by adding (any) new functionality - as long as they are properly thought about and documented (to avoid the kind of snafoos that v1 suffered from)

Also, I would definitely encourage you to reach out to @stevvooe or me on irc to kick this of.
Thanks again for caring, and for your ideas here - this discussion is certainly welcome!
Cheers.

@dmp42 point taken with respect to pip, npm, etc ...
I'm all for moving it out of the client and into some other mechanism (ie. website)
My main question is this ...
With version 2 should account creation be allowed via the docker client?
If no, then the client needs to have 2 workflows, one for v1 and one for v2 and the documentation needs to reflect this, both on the website and in the client itself.
If backwards compatibility is a concern then v2 would need to still send an email address.
Ultimately whether the index/auth system actually creates an account or not has nothing to do with the docker daemon or client.  There is nothing stopping me from using the basic auth creds, sent by the client to the auth service to receive a token, to create an account for the user.

@ekristen I'm not fully caught up on the back and forth at this point, but I think would be right to say that the v1 login architecture was really oriented towards getting up and running. While this is a valuable feature, overloading docker login with registration was an architectural mistake, probably made out of expediency. As a user, I would find it to be unexpected to register an account during a login process.
That said, the question arises, what features should the docker daemon provide? Should it have full user management in a docker register command? How does it work? What protocols does that support? User management all seems way outside of the scope of the docker project.
For your use case, where you are building an "index" that has login and registration, we do have to ask how one would do user management via the docker client, if at all. If this is as simple as sending a user email address with the token retrieval, we may want to explore that. Another option may be to have a cli that interacts with your system for user management. The latter approach would provide a lot more flexibility and security, at the cost of some convenience.

I don't think this needs to be open anymore? email was completely deprecated.
Thanks for the hard work guys and girls. Cheers!


Ad does not update if path changes

The ad should update when the path changes for cases when the page containing the ad does not unmount but just re-renders when the route params change (example: category drill down).

+1


Problem font rendering

I propose an option to disable the column layout and put the comment directly after the binding without spacing.

If you don't mind I'ma wait with removing this feature until some more people will agree with your proposal. Since some may say afterwards that it made the readability worse.

I meant making it optional via conf option, doesn't matter to me what the default is. I wonder why you are not getting this problem. Which OS do you use? Maybe we can estimate how likely it is users getting this problem. I can try Windows 11 since I have it installed.

Is there maybe a good monospaced font I can try to install?

i think this issue was resolved with @stax76 pull request #5


C# Support

I'm using C# for my project, despite the addon being written in gdscript, and have been successful up until it comes time to implement the dialogue_generated signal, as it recognizes the DialogueResponse as Variant type.  This renders the output, as far as I can tell, non-functional.
Is this intended behavior (meaning that the addon is incompatible with C#) or is it a bug? Are there any known workarounds or is a partial C# rewrite necessary to fix the issue?
It is my understanding that the purpose of this addon is to primarily function as a personal tool, and secondarily as a community tool, thus I don't see C# support being high on its priority list. Is there any framework for a potential C# support update being PR'ed to this repo?

thanks for cutting this issue. I'll look further into this! since I have not personally tested the usage along with C#.

hey came across this from the video; you should be able to use .Get("property name") on variants and they should retain props or .Call("method name", args) for methods, i think as long as the variant is an Object type that is (which it is in this case). Resource also retains this, you may have to force the signal to connect but it should be fine. You can validate the items exist as well using things like .HasMethod, .GetPropertyList, etc., though naturally it uses Reflection so it can be slowish.

example:
i didn't load up the whole extension myself but i've done a little facsimile of it in the interest of being quick:

and if that's sound (which it should be enough), i tested grabbing the info out of it thusly:
public static void Test () {
  // instantiating a test node
  GDScript readerScript = GD.Load<GDScript>("res://Src/Externals/EzDialogueReader.gd");
  Node exampleNode = (Node)readerScript.New();

  // setting up a quick lambda to ensure the custom resource retains its props
  // this'd be whatever func you connected the signal to
  Callable myCallable = Callable.From((Resource dialogueResponse) => {
    GD.PrintT(
      dialogueResponse.Get("text"),
      dialogueResponse.Get("choices")
    );
  });

  // doin' it (just setting up the signal to call our lambda)
  exampleNode.Connect("dialogue_generated", myCallable);
  exampleNode.Call("test_call");
}

and the output:

from there you can use something like dialogueResponse.Get("text").As<string>() (must be variant compat type) to convert to something c#'ll understand easily enough. Naturally conversion from dynamic to static can be a little sketchy but this should do as long as the underlying stuff stays the same.
There's some more info in these here:
https://docs.godotengine.org/en/stable/tutorials/scripting/c_sharp/c_sharp_variant.html#variant-compatible-types
https://docs.godotengine.org/en/stable/tutorials/scripting/cross_language_scripting.html
https://docs.godotengine.org/en/stable/classes/class_object.html#class-object-method-get


how can i check if user has signed in for first using FB

what i mean is ..is it possible to check for the signed in user ID wheather that user already have an ID generated  or not ..and if so, where is user ID located

Hi @job111,
On logging in Facebook, you can get the user information (username and id) from Facebook as described here. Every facebook user has a unique Facebook id.
After that, it is up to your app's implementation how and where you store that users information. If you have decided to use Firestore, please refer to the Firebase plugin documentation as well as Firebase documentation itself.


feat: add umee price feeder

Add new flag providers
Add oracle structure to control umee price-feeders
Add get symbol API to coingecko struct
Issue #181
PR at Umee

How to test

Take a look at https://github.com/umee-network/peggo/pull/239/files#diff-8055950eb8b0895f6a87971bd9ea7e936ed738a9cb7c566db7396f39ccc5e2df
Still in progress though :sweat_smile:

Codecov Report

Merging #231 (9907000) into main (4caf509) will decrease coverage by 0.36%.
The diff coverage is 61.90%.

@@            Coverage Diff             @@
##             main     #231      +/-   ##
==========================================
- Coverage   60.68%   60.31%   -0.37%
==========================================
  Files          17       17
  Lines        1989     2031      +42
==========================================
+ Hits         1207     1225      +18
- Misses        685      703      +18
- Partials       97      103       +6

Impacted Files
Coverage Δ

orchestrator/orchestrator.go
87.50% <ø> (ø)

orchestrator/relayer/relayer.go
80.95% <ø> (ø)

orchestrator/coingecko/coingecko.go
66.20% <60.65%> (-9.53%)
:arrow_down:

orchestrator/relayer/options.go
100.00% <100.00%> (ø)

Awesome! Give me some time to review it 👌👌

Starting to review this today! 💪

I'll check the E2E tests, something's up 🤔

LGTM


build AOT error

Import TagInputModule into my project leads build error, but in run it works well.
Error: Child compilation failed:
Module build failed: Error: "extract-text-webpack-plugin" loader is used without the corresponding plugin, refer to https://github.com/webpack/extract-text-webpack-plugin for the usage example

Hi @phamtienmanh, please fill out the template


Fixing issue #6

Fix for https://github.com/flokosiol/kirby-focus/issues/6

Thanks a lot for your extensive testing and your work. Unfortunately I was already working on this issue in the develop branch. Because I changed the calculation of the ratio I couldn't merge your pr. But you pointed me to the right direction. Thanks again for that!

Awesome! I'm so glad it works now. I wanted to fix this issue as fast as possible because the plugin is a perfect fit for the project I'm currently working. I'll probably use this plugin on every single website now. Client is happy, I am happy 👍

😄


handle sql error if there is no data

ref: https://issues.redhat.com/browse/ACM-6983

ref to this blog about the workaround to handle the multiple values for variables - https://www.dbi-services.com/blog/grafana-sql-and-in-list-for-multi-value-variable/
if there is no data, show No data until tomorrow morning for Policy Group Compliancy Overview and Cluster Group Compliancy Overview

if there is no data in other dashboards, show No data in response. for example:

/lgtm


Move instance setup scripts to usr/bin.

usr/local/bin does not appear on the path in certain distros.

LGTM


cargo: update ring to avoid segfault

When building a package that depends on oauth1, I got this error:
instapaper-rs $ cargo build                                                                                [121/403]
   Compiling failure v0.1.3
   Compiling ring v0.12.1
   Compiling reqwest v0.9.4
error: failed to run custom build command for `ring v0.12.1`

Caused by:
  process didn't exit successfully: `/Users/work/src/github.com/sirupsen/instapaper-rs/target/debug/build/ring-0e15c2552dc743d0/build-script-build` (signal: 11, SIGSEGV: invalid memory reference)
  --- stdout
  ALACRITTY_LOG: /var/folders/ch/_pklv2hx6zl4sr9tpyxtxb7m0000gn/T/Alacritty-2161.log
  BIGTABLE_EMULATOR_HOST: localhost:8086
  BUTTONDOWN_TOKEN: 9fcbc0d5-578c-4d84-8dc0-dd4dee3832ff
  CARGO: /Users/work/.rustup/toolchains/stable-x86_64-apple-darwin/bin/cargo
  CARGO_CFG_TARGET_ARCH: x86_64
  CARGO_CFG_TARGET_ENDIAN: little
  CARGO_CFG_TARGET_ENV:
  CARGO_CFG_TARGET_FAMILY: unix
  CARGO_CFG_TARGET_FEATURE: fxsr,sse,sse2,sse3,ssse3
  CARGO_CFG_TARGET_OS: macos
  CARGO_CFG_TARGET_POINTER_WIDTH: 64
  CARGO_CFG_TARGET_VENDOR: apple
  CARGO_CFG_UNIX:
  CARGO_FEATURE_DEFAULT: 1
  CARGO_FEATURE_DEV_URANDOM_FALLBACK: 1
  CARGO_FEATURE_USE_HEAP: 1

  --- stderr
  thread '<unnamed>' panicked at 'attempted to leave type `nodrop::NoDrop<(epoch::Epoch, garbage::Bag)>` uninitialized, which is invalid', /rustc/2fd73fabe469357a12c2c974c140f67e7cdd76d0/library/core/src/mem/mod.
rs:671:9
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: build failed

Resolved by upgrading the ring dependency :)

Can I bother you to release 1.0.0? If it's trouble, you can also add me as an owner<EMAIL_ADDRESS>and I'll do it


Package not compatible with remotion 4.

This package requires Remotion 3. The following error appears when installing it with remotion 4.
$ npm install remotion-time --save-dev

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/remotion
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/remotion-time
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

When forcing the installation, this package seems to be working on remotion 4.

@LoganTann Until the maintainer updates the peer dependency versions, you can use npm install remotion-time --legacy-peer-deps. It already works with remotion v4.

Thanks for the reply @mohsentaleb, this is what I did 👍

Fixed in 0.0.3.


Dark mode

Is your feature request related to a problem? Please describe.
Colors are set in each components of the app and are difficult to maintain
Describe the solution you'd like
Define dark mode and put it at default

In light mode, checkboxes are not visible

Thanks for your feedback, I just push a fix


404 at /language/golang/storage/index.md

I found a broken link : https://docs.docker.com/language/golang/storage/index.md
I arrived on this page through: https://docs.docker.com/language/golang/develop/

Fixed by #13295


Unsafe implementation of the X509TrustManager interface

The AcceptAllCertificates class defines empty checkServerTrusted() and checkClientTrusted() methods, thereby disabling SSL validation and accepting any SSL certificate as valid if the class is used when connecting to a server over SSL/TLS.
Regardless of whether the affected classes are actually used at runtime, Google Play is blocking any app that defines such an insecure X509TrustManager, as detailed on Google's support page:
"Beginning May 17, 2016, Google Play will block the publishing of any new apps or updates containing an unsafe implementation of the X509TrustManager interface."
reference

Duplicated issue:
https://github.com/pedroSG94/RootEncoder/issues/1562
Closing issue as dupicated


[BACKLOG-29110] - Fix product icons

@pentaho/kashyyyk Please review.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  assemblies/static/src/main/resources/docs/English/welcome/favicon.ico
  assemblies/static/src/main/resources/spoon.ico
  ui/src/main/resources/ui/images/kettle_logo_small.svg
  ui/src/main/resources/ui/images/spoon.ico
  ui/src/main/resources/ui/images/spoon_lowres.ico

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:


Does not allow Mouse Scroll(Mouse Wheel) ?

Mouse scroll is not allowed or not

Hi @Valgeria is this issue resolved?


feat: Handle duplicate schema names

Description
This addresses the issue reported by @scsmithr, where running the command CREATE SCHEMA IF NOT EXISTS test; twice would result in a 'Duplicate name' error. The expected behavior is that the IF NOT EXISTS clause should prevent an error from being thrown if the schema already exists.
What This Enables

Handling the 'IF NOT EXISTS' clause in schema creation to prevent the 'Duplicate name' error.
Adding logging for schema creation and mutation for better debugging experience.

Impacts
This pull request addresses issue #1320
Test Case
First, run the GlareDB server, and then test the fix with this Python script.
import psycopg2
import json

hostname = "localhost"
port = 6543
username = "glaredb"
dbname = "glaredb"

# The SQL query to create the schema
sql_query = "CREATE SCHEMA IF NOT EXISTS test"

try:
    # Creating a connection to the GlareDB server
    conn = psycopg2.connect(
        host=hostname,
        port=port,
        user=username,
        dbname=dbname
    )

    # Creatin the cursor object
    cur = conn.cursor()

    # Executing the SQL query here
    cur.execute(sql_query)
    print("Executed SQL query successfully: ", sql_query)

    # Executing the query again
    cur.execute(sql_query)
    print("Executed SQL query successfully: ", sql_query)

    cur.close()
    conn.close()

except psycopg2.Error as e:
    print("Unable to connect to the database")
    print(e.pgerror)
    print(e.diag.message_primary)

@scsmithr the check that's failing is repo secret related...
please have a look, and let me know.

Hi @0xf333
Thanks for your contribution to GlareDB partying_face
I added a few comments in the PR, mainly addressing the following points:
* Removing log statements since they would create a lot of noise.

* Removing the `if_not_exists` check from the client side.

* Setting `if_not_exists` to `false` in tests (instead of `true`) everywhere. Since the tests were written when `if_not_exists` wasn't supported, changing current tests might cause issues (as mentioned in one of the comments).


Question about '3D joint position y' in Projective Attention

Hello!
I'm wondering what y refers to in the Projective Attention module.
Is it the ground truth of the 3D keypoints? If so, how do we handle it when the model is inferencing?
I'm looking forward to your reply and thanks in advance ~

hi it means the 3d joint location prediction of the current decoder layer

hi it means the 3d joint location prediction of the current decoder layer

Thanks for your quick reply !
I have checked the source code. It seems that reference_points is y ?
Does this mean that y is actually the reference_points output by the previous decoder layer?
https://github.com/sail-sg/mvp/blob/8b2ccc576a450841a5b344597cec26e5ac77eaf7/lib/models/mvp_decoder.py#L314-L345
I am not sure if my understanding is correct. If it is not, could you please explain it in detail? Thank you.


error when using using convert_estimator with MultiOutputRegressor

Describe the bug
ValueError: Cannot find 'pure_sklearn' counterpart for MultiOutputRegressor
To Reproduce
Convert a pickled MultiOutputRegressor model to pure_sklearn using:
model = pickle.load(filename, 'rb')
model = convert_estimator(model)
Expected behavior
Expected conversion to pure_sklearn version
Additional context
Add any other context about the problem here.

Yes unfortunately this meta-estimator isn't supported. By design, pure predict only supports a subset of sklearn estimators and meta-estimators. Please feel free to contribute.


feat: reuse auth token within the same org

Fix #113

:tada: This PR is included in version 1.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Support pre-signed requests for CloudFront distribution or S3 URL.

Example environment:
AWS_CLOUDFRONT_PUBLIC_DISTRIBUTION_URL="http://assets.mydomain.com"
AWS_CLOUDFRONT_PROTECTED_DISTRIBUTION_URL="http://assets.mydomain.com/protected"
AWS_CLOUDFRONT_PRIVATE_KEY_ID="<key_id>"
AWS_CLOUDFRONT_PRIVATE_KEY_PATH="<key path>"

Looks good, would merge.

Mateusz says yes. Merging


Users only in default group not appearing in / being deleted from editor

A user brought to my attention that not all players that had joined the game since LP was installed were appearing in the editor - these users being those that were only in the default group. As we investigated, a new editor session was run, and the new one included one less user than the previous, again someone only in default group.
lp user <user> info for the players not appearing in the editor shows them as having logged in before and being only in default group, with mojang-assigned UUIDs (the server is online-mode: true)
This is their lp info output:

And this is surprising bit:
First Editor

Second Editor (Moments Later)

Turbo thought maybe it has something to do with the anti-duplicates editor change recently made (commit 918ea75 fix for issue 235 in LPWeb repo) and I honestly have no clue, just thought I'd let ya know!

This seems like normal behaviour.
The editor user list will always include online players. It is then "topped up" (to a limit of 1000) with other offline players with permissions already set.
Therefore, offline players, with no permissions/parents defined beyond the default group will not be shown.
If you think this ^ could be useful, then I'm happy to consider adding it - but it's not a bug. :)

Oh, interesting! Figured it might be like the default group members not all listing if offline etc. Perhaps add it so that if there are less than 1000 players in the editor, start to list default players? Up to you, but seems it could help small servers and their owners configure individual players.

Would it be possible for the editor to show offline players using the usercache.json (or an option to show these players in the editor)?

Would it be possible for the editor to show offline players using the usercache.json (or an option to show these players in the editor)?

- [ ] @~~~~

This seems like normal behaviour.
The editor user list will always include online players. It is then "topped up" (to a limit of 1000) with other offline players with permissions already set.
Therefore, offline players, with no permissions/parents defined beyond the default group will not be shown.
If you think this ^ could be useful, then I'm happy to consider adding it - but it's not a bug. :)

I have the same issue and can not add a permission to a user when they are only in the default group and they are offline from the web editor since they are not shown.
I can only add permission(s) via the command line and until they login again they still will not show up in the users list on the web editor.


Merge from master to stable

Script generated clean merge here: https://github.com/infinyon/fluvio-website/pull/790
closing this one


Ensure formatting of output is same for adapt=false samplers

Submission Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
These two overloads weren't passing num_chains to the run_sampler method, which meant they would never prefix their output with Chain [#] like their adaptive counterparts.
Intended Effect
How to Verify
Side Effects
Documentation
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

@SteveBronder mind giving this a look? I think it got lost in the travel for ISBA


add graphs for incidence/R-Value

Feature description
Add graphs showing incidence and R-value on a timeline.
Enable user to e.g. click on the daily values and get a graph showing the last n days of the values development. It should be possible to select n from a small set, e.g. "all time", "past 90 days", "past 30 days".
Also, show the last 3 or 4 day numerical values directly on the numerical indicator.
Problem and motivation
CWA currently shows the current day value for incidence and R-Value only. However, both values are dynamic and instable for at least 4 days. This is often misunderstood. Furthermore, press often covers invalid graphs plotting the daily reported R-value instead of the updated values from https://github.com/robert-koch-institut/SARS-CoV-2-Nowcasting_und_-R-Schaetzung/blob/main/Nowcast_R_aktuell.csv. This leads to wrong perception of the development and correctness of the value. The same holds true for the incidence.
This is often being criticized, for real-world arguments have a look at https://twitter.com/RainerGerhards/status/1411757405319438336
Data sources are readily available

github link above for R
incidence: https://www.rki.de/DE/Content/InfAZ/N/Neuartiges_Coronavirus/Daten/Fallzahlen_Inzidenz_aktualisiert.html;jsessionid=EE5AC87955DC1EC0267C5D2B11D2861B.internet091?nn=13490888

Very little storage space is required:

date,
7-day r (float)
incidence (float)
If space is ultra-sparse, floats could be encoded as integers where real value r is stored as (int) (r*10 or 100) and r restored before display by appropriate float division.

Is this something you're interested in working on
No (I am not a mobile developer)

Internal Tracking-ID: EXPOSUREAPP-8289

@rgerhards
Thanks for your suggestion. I have informed the dev team about your proposals. We will report back here on Github if we get any news on the status of the issue.

Corona-Warn-App Open Source Team

It would be great if the incidence increase since the last day was displayed.
Tell me if i should open a new Feature Request for that.

@lm41

It would be great if the incidence increase since the last day was displayed.
Tell me if i should open a new Feature Request for that.

I had a similar request logged under https://github.com/corona-warn-app/cwa-wishlist/issues/395. I closed it though because trends are already available.

Auf der Status-Seite der Corona-Warn-App herunterscrollen bis zum Bereich "Häufige Fragen" und darauf tippen.
Im Menu rechts oben, "Analysis" auswählen.
Auf der Seite  "Corona-Warn-App (CWA): Kennzahlen" bis zum Link "RKI: Covid-19-Trends (DE)" nach unten scrollen.
Auf diesen Link tippen.
Um beispielsweise den Trend für NRW anzuschauen, bis zur Rubrik "Bundesländer" herunterscrollen.

It would be great if the incidence increase since the last day was displayed.

There is a problem with "since last day". The volume of cases changes by Weekday (usually Mondays least, Wednesdays most). While the 7-day incidence takes care of much of the noise, one can see a periodic change in it, too. This is why I asked for a graph.
If a single value is to be given, it would be best to compare not the day before but the same day in the previous week. Removes most of the noise (at least outside of holiday periods).

@rgerhards
Your suggestion is implemented in CWA 2.28. See https://www.coronawarn.app/en/blog/2022-10-26-cwa-2-28/ which announces the "pandemic radar".
You can access it from the app or directly through https://www.rki.de/DE/Content/InfAZ/N/Neuartiges_Coronavirus/Situationsberichte/COVID-19-Trends/COVID-19-Trends.html?__blob=publicationFile#/home


[✨ Feature Request]:

Is your feature request related to a problem? Please describe.
Currently, I am implementing Subgraph network rules for GNNs, and I need to operate on edges and create new graphs based on the connectivity of these edges, but I am not aware of any way to do it in this framework at this moment. I think it may be beneficial to add some feature which would allow creation of new nodes based on some given rules.
Describe the solution you'd like
I would need some way of creating new nodes and graphs based on the rules, for example some code similar to the following would be expected:
Relation.edge_node(V.E) <= Relation.edge(V.X, V.Y) # creates a node for each edge of the original graph
Relation.edge(V.X, V.Y, V.E) <= Relation.edge_node(V.E), Relation.edge(V.X, V.Y) # defines a relation that ties the edge node and its original nodes together

Relation.edge(V.E1, V.E2) <= (Relation.edge(V.X, V.Y, V.E1), Relation.edge(V.X, V.Z, V.E2))# creates edge between the nodes created above

Describe alternatives you've considered
No response
Additional context
No response

is the implementation based on any published paper?

Hi, thank you for the feature request.
I'm working on implementing this feature, but it requires changes to the underlying grounder, and those can easily break something. So I'm not sure when or if this feature will be possible.
I think that in your scenario, you do not necessarily need this feature. You could do something like:
R.edge(V.A, V.B, V.C, V.D) <= (R.edge(V.A, V.B), R.edge(V.C, V.D), R.special._in(V.A, V.C, V.D))
R.edge(V.A, V.B, V.C, V.D) <= (R.edge(V.A, V.B), R.edge(V.C, V.D), R.special._in(V.B, V.C, V.D))

Here we treat edge/2 as "nodes" and edge/4 as "edges" between them. The in predicate is there to make it more compact. Otherwise, it would require four rules instead two.

Or just
R.edge_sym(V.A, V.B) <= R.edge(V.B, V.A)
R.edge_sym(V.A, V.B) <= R.edge(V.A, V.B)

R.edge2(V.A, V.B, V.C) <= (R.edge_sym(V.A, V.B), R.edge_sym(V.B, V.C))

if you really need to consider symmetric edges.
To generalize to the higher orders, I'd say just remember the first and last node from the edge composition (path), and take it as the new hypernode/edge - this way you can compose the "higher-order" graphs:
R.edge2(V.A, V.C) <= (R.edge(V.A, V.B), R.edge(V.B, V.C))
R.edge4(V.A, V.C) <= (R.edge2(V.A, V.B), R.edge2(V.B, V.C))
...

and so ons.

Thank you for your replies!
The implementation is based on the original paper on Subgraph Networks by Qi Xuan et al.: https://arxiv.org/abs/1903.09022
I would also like to ask what exactly does the R.special._in predicate do? There seems to be no explanation in the docs, so far: https://pyneuralogic.readthedocs.io/en/latest/advanced/modifiers.html#special-modifier

The R.special._in(V.X, V.Y0, V.Y1, ... V.Yn) is satisfiable if X == Y1, X == Y2, or X == Yn.
In my example, it's checking that V.A is either equal to V.C (source node of the second edge) or V.D (target of the second edge).


[Feature request] Add pipeline script over file browser

It would be nice if we had a button to browse local scripts from the device, which can then be uploaded. This would be much "easier" than trying to access the "pipeline/" docker mount to push a file or upload a script to GitHub.
Just a suggestion :)

so basically an upload feature? 😅 Coming soon 🫡

Added on dev!

Oh wow, that was fast! You're amazing. Thank you!


DS-4275: Fix case of resourcepolicies endpoint

The endpoint is specified in lowercase in the REST contract.

Hi @AlexanderS  thanks for your PR, your commit has been cherry-picked with proper attribution as part of a different PR and it is now in the master https://github.com/DSpace/DSpace/pull/2604/commits/57df5a3f981e4cfe61bfc4ffea0aba235e6afa23 so I'm closing this PR


Missing value 1 0xe00

I was able to debug the blue screen 0x119 with the first parameter 0xe00 whose meaning I do not see on this page. Can you add a description of this first parameter?

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

ID: 7917857b-97bc-efec-67ab-5e3af061524c
Version Independent ID: 8c00d20e-9b21-7bb0-a041-220aca9d2487
Content: Bug Check 0x119 VIDEO_SCHEDULER_INTERNAL_ERROR - Windows drivers
Content Source: windows-driver-docs-pr/debugger/bug-check-0x119---video-scheduler-internal-error.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @DOMARS
Microsoft Alias: domars

@MrPepka - I added a description for the 0xE00 parameter. Thank for this issue and your interest in the debugger stop code documentation for driver developers.


no Svg SvgData in widget

Referring to the toml file in the example, I think it only needs to add a reference to "usvg", but why can't I find the compile time prompt?
main file.
use druid::{widget::{FillStrat, Flex, Svg, SvgData, WidgetExt}, AppLauncher, LocalizedString, Widget, WindowDesc, Color};
use druid::widget::{Button, Container};

fn main() {
    let main_window = WindowDesc::new(ui_builder)
        .window_size((960.0, 580.0))
        .set_position((1024.0, 960.0));
    let data = 0_u32;
    AppLauncher::with_window(main_window)
        .use_simple_logger()
        .launch(data)
        .expect("launch failed");
}

fn ui_builder() -> impl Widget<u32> {
    let browse_svg = match include_str!("./assets/browse.svg").parse::<SvgData>() {
        Ok(svg) => svg,
        Err(err) => {
            // error!("{}", err);
            // error!("Using an empty SVG instead.");
            SvgData::default()
        }
    };

    // The label text will be computed dynamically based on the current locale and count
    let text =
        LocalizedString::new("hello-counter").with_arg("count", |data: &u32, _env| (*data).into());
    let svg = Svg::new(browse_svg).padding(5.0).center();
    let button = Button::new("increment")
        .on_click(|_ctx, data, _env| *data += 1)
        .padding(5.0);

    let container_menu = Container::new(svg).background(Color::WHITE);

    Flex::row()
        .with_child(container_menu)
        //.with_child(label)
        .with_child(button)
}

cargo.toml
[package]
name = "IndexJi"
version = "0.1.0"
authors = ["me"]
edition = "2018"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
druid = "0.7.0"
druid-shell = "0.7.0"
druid-derive = "0.4.0"
rusqlite = "0.25.3"

usvg =  "0.14.1"
harfbuzz-sys =  "0.4.0"

error is
error[E0432]: unresolved imports `druid::widget::Svg`, `druid::widget::SvgData`
 --> src\main.rs:1:39
  |
1 | use druid::{widget::{FillStrat, Flex, Svg, SvgData, WidgetExt}, AppLauncher, LocalizedString, Widget, WindowDesc, Color};
  |                                       ^^^  ^^^^^^^ no `SvgData` in `widget`
  |                                       |
  |                                       no `Svg` in `widget`

you need to enable svg feature for druid.

you need to enable svg feature for druid.

I don't know how to do it. Can you teach me?
In the beginning,I wanted to create an image, but I didn't know how to load the image. "Druid / Druid / examples / image.rs" uses from_data, but when I compile, I prompt that this method does not exist. In the code prompt of clion, there is only from_ raw.
I'm new to durid, but the Druid document doesn't have the knowledge I need, and the sample code can't be used normally (maybe I won't use it). It bothers me.

you need to enable svg feature for druid.

Thank you, I passed the code to prompt the correct setting successfully.

However, how to load file resources when creating an image, "Druid / Druid / examples / image.rs" uses from_data, but when I compile, I prompt that this method does not exist. In the code prompt of clion, there is only from_raw.


RKE2 Recommended Tolerations Fail to Scan

The recommendation for RKE2 deployments (found here: https://docs.rke2.io/install/ha/#2a-optional-consider-server-node-taints) is to taint Server nodes with CriticalAddonsOnly=true:NoExecute. As a result, the CIS Operator appears to not properly scan server nodes. When modifying this manually in a cluster I have deployed, the operator appears to work properly.
PR here: https://github.com/rancher/cis-operator/pull/101

Issue Duplicate/Similar to: https://github.com/rancher/rancher/issues/37129
PR merged: https://github.com/rancher/cis-operator/pull/101
Hence Closing.


Update CVE-2024-5232.json

Removed Spaces from CPE on CVE-2024-5232

@patrickmgarrity Thank you! This was previously fixed upstream and just recently republished before I saw this PR.


Add a shortname to federatedtypeconfig

What this PR does / why we need it:
Adds a short name ftc to federatedtypeconfig which is easier to type in command line.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Thanks for doing this! And special thanks to @sohankunkerkar for enabling this capability in kubebuilder!
Would it make sense to enable this for KubeFedCluster and KubeFedConfig as well?
/lgtm

Would it make sense to enable this for KubeFedCluster and KubeFedConfig as well?

yes, it would be easy to use them in command line. My suggestion is kfcluster for KubeFedCluster and kfconfig for KubeFedConfig. Are there any other suggestions?

sgtm


Reuse cat and system ID when expanding probes

As system and cat work basically the same as printf we need to use
the same id reuse trick.
trigger:
tracepoint:syscalls:sys_enter_open,tracepoint:syscalls:sys_enter_openat
{
  if (str(args->filename) == "") {
    cat("");
  }
}

Results in:
terminate called after throwing an instance of 'std::out_of_range'
  what():  vector::_M_range_check: __n (which is 1) >= this->size() (which is 1)
Aborted

Closes #957

Thanks!! :)

updated @danobi


Comments field (for edits)

Would it be possible to add another field to the end of the form for comments, which are displayed in the pull request but not actually added to the list record?
This could function as a simple way of enabling edits; someone who has previously submitted an entry can just submit a new entry and let us know that we should delete their previous record.
Thoughts?

Not sure this is possible. I don't see any way in the docs to have a comment go into the PR text instead of the JSON data file.

I see; fair enough! In that case we'll just stick with the "raise issue or send email" option for edits.


Some loops are erroneously turned into infinite loops

Hi,
When this kind of source code (loop with a small body) is compiled in 32-bit with gcc, RetDec fails to properly decompile it. In fact, this source code :
int myfunc(unsigned char* S) {
   for(int i = 0; i < 256; i++)
      S[i] = i;
   printf("Supposed not to be removed.");
   return 0;
}

becomes :
int32_t myfunc(int32_t * a1) {
   __x86_get_pc_thunk_ax(g2, 0);
   char v1 = 0;
   while (true) {
      *(char *)((int32_t)v1 + (int32_t)a1) = v1;
      v1++;
   }
}

RetDec properly decompiles it when it's compiled in 64-bit though :
int64_t myfunc(int64_t * a1) {
   int32_t v1 = 0;
   for (int64_t i = 0; i < 256; i++) {
      *(char *)(i + (int64_t)a1) = (char)v1;
      v1++;
   }
   printf("Supposed not to be removed.");
   return 0;
}

Three passes seem to cause this issue : Instruction Combine (-instcombine), Global Value Numbering (-gvn) and Early Cse (-early-cse). In that case they tend to transform a "br <cond> <label> <label>" into a "br false <label> <label>", as my IR dumps show, resulting in a "while true". The code that follow is then considered unreachable and removed. When I disable these passes in RetDec, the decompilation outputs  the expected C code that is just like the 64-bit example above.
I also tried with llvm-8 (on your new branch), doesn't work either.
This is probably due to an upstream problem.
However, this problem doesn't occur when the binary is compiled with clang.
Note : This is not gcc's fault since the generated assembly doesn't represent an infinite loop, and the binary is working as expected.
Thank you for your time !

Thank you for the report. @PeterMatula, can you please take a look at this?


[MRG] improve ignore deprecation warnings

What does this implement/fix?
This PR adds mne.utils.filter_out_warnings to remove warnings from a warning record, when using pytest.warns.
The main problem is that by construction, @pytest.mark.filterwarnings and pytest.warns cannot be used together since pytest.warns overwrites all the filters to not miss any potential warning. (see https://github.com/pytest-dev/pytest/issues/4011)
Additional information
This PR would allow to simplify #4936 in the following manner:
 @requires_h5py
 @testing.requires_testing_data
-@pytest.mark.filterwarnings('ignore:Function read_events_eeglab is deprecated')
 @pytest.mark.parametrize('fnames', [raw_mat_fnames, raw_h5_fnames])
 def test_io_set_raw(fnames, tmpdir):
     """Test importing EEGLAB .set files."""
@@ -84,8 +83,6 @@ def test_io_set_raw(fnames, tmpdir):
                                event_id=event_id)
         raw4 = read_raw_eeglab(input_fname=raw_fname, montage=montage)

-        [w.pop(DeprecationWarning) for _ in range(5)]  # read_events_eeglab
-
         assert raw0.filenames[0].endswith('.fdt')  # .set with additional .fdt
         assert raw2.filenames[0].endswith('.set')  # standalone .set

@@ -101,11 +98,9 @@ def test_io_set_raw(fnames, tmpdir):
         raw0.filter(1, None, l_trans_bandwidth='auto', filter_length='auto',
                     phase='zero')  # test that preloading works

-        while 'FutureWarning' in [xx._category_name for xx in w]:
-            w.pop(FutureWarning)
-        while 'ImportWarning' in [xx._category_name for xx in w]:
-            w.pop(ImportWarning)
-
+    filter_out_warnings(w, category=DeprecationWarning)
+    filter_out_warnings(w, category=FutureWarning)
+    filter_out_warnings(w, category=ImportWarning)
     assert len(w) == 4  # check `preload=False` raises RuntimeWarning

ping @agramfort
green, merge if happy.

#4936 is merged, can you improve the usage in that code, too? Doing that one example makes it even clearer how it can make our tests in general cleaner going forward

@larsoner yes! I can apply the patch here. Instead of a different PR.

CIs are not happy...

thx @massich


Consider GitHub Templates for PR and Issues

If you see https://github.com/sunpy/sunpy/issues/new/choose
sunpy has a range of yaml templates that allow people to select a range of issues types.

Good idea, we should add it!

This is a high level topic that would need to be done on the project side and not this repo. So I will be closing this.


using box to forward declare?

Our core model of the application is growing quite large, and impacting our compilation time, as every reducer/selector/etc is #include "Model.h"
There is the note about using box to work around incomplete types, so this seemed like a good place to start (change all the includes in the Model.h to be forward declared, as they are already in box's already.
https://sinusoid.es/immer/design.html?highlight=box#recursive-types
But this fails to instantiate as the type is incomplete. Did I misunderstand the use case here? Is there an easy way to support foward-declaration of the model types with immer containers?

Hi @kevin-- ! How did you solve this in the end?

Hi @kevin-- ! How did you solve this in the end?

Hi @arximboldi I did not actually solve it, I assumed it was not possible -- am I mistaken?


implemented event bubbling

See #2342.

Would it be possible to see an example of this working? I've tried this approach but the event.target value is always the parent DisplayObject and not the children, although looking in event.children[1], I can see the one that I clicked.
A working example would be great, thanks!

Hello @gurmukhp! Here is an example for you: http://jsbin.com/noboliqamo/1/edit?html,js,console,output
Click events show up in the console.
Let me know if this helps!

Ok, I think I see your issue.
My example works great unless the outer Pixi container is set as interactive. If it is, it prevents its children from hearing the click.
I'll see if I can find time to take a closer look soon.

Thank you! That's really useful! Do you know if event bubbling from within a Pixi Canvas brings any performance benefit?
In the HTML world the more events bound, the slower it can be, do you know if this also applies to Pixi?

@gurmukhp It will of course be slower, it will have to emit events many more times than before.
.

Thanks, so to confirm, we should try to limit the number of event listeners possible.
Any idea when this will make it to the public release?

Thanks, so to confirm, we should try to limit the number of event listeners possible.

@gurmukhp No, if an event happens it will emit from the target through all the parents. It doesn't matter how many listeners you add. Aside from the overhead of actually calling your callbacks, there isn't any significant difference between 10 listeners and 100. It still walks the same number of nodes, and emits the same number of events.

Yep: Any performance penalty would come from deeply-nested targets bubbling through lots of parents, not the volume of interactive targets. (If you want to use event bubbling in a deep scene graph, you can use stopPropagation to mitigate this.)
I use event bubbling not for performance reasons, but because it can help me write cleaner code. In some cases, I think it's best to separate event logic from individual sprites.

Any idea when this will make it to the public release?

This doesn't seem to be a high priority issue. (I might be the only one who really cares!)
Given the potential performance implications, and the fact most Pixi devs don't seem to use bubbling, I'm wondering if bubbling should ultimately be implemented as an optional toggle (defaulting to "off").

@akbr Where there is clear trade-off for performance v. usability, having an opt-in toggle for usability is probably a better approach.

About performance, I don't think this would be an overhead for any PIXI application. Some benchmarks would be necessary to confirm exactly how much time it would consume having to bubble in a long display list.

I'm closing this in light of https://github.com/pixijs/pixi.js/pull/2658, which adds global events to the InteractionManager. As @andrewstart notes, global events allow for "some of the use cases of event bubbling without the performance hit of deep display lists." (:beers:)
If there is a desire for "proper" event bubbling -- which should probably be opt-in, as discussed above -- someone else can pick this up.


add column expr

Codecov Report

Merging #221 (db9a338) into master (f1ff762) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##            master      #221    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           77        81     +4
  Lines         7304      7653   +349
==========================================
+ Hits          7304      7653   +349

Impacted Files
Coverage Δ

fugue/execution/execution_engine.py
100.00% <ø> (ø)

fugue/column/__init__.py
100.00% <100.00%> (ø)

fugue/column/expressions.py
100.00% <100.00%> (ø)

fugue/column/functions.py
100.00% <100.00%> (ø)

fugue/column/sql.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


exome-seq_tour_and_workflow

tour
workflow
@bebatut

@wm75 could you have a look at the tool and workflow, and probably update with the new Gemini version? Thanks 😄

Thanks @polkhe for the effort.
@wm75 rewrote the exome-seq tutorial this summer so this tour is probably out of date


Invoke shell to execute easy-rsa scripts

Pull Request (PR) description
Invoke shell to execute easy-rsa scripts
They appear to be non-executable.
This Pull Request (PR) fixes the following issues
Fixes #313

The root cause was the permission of the files as installed by Puppet, not the way they were being executed. No change is needed here.


How does pilot retrieve and apply configuration in Kubernetes?

I am using Istio with Kubernetes currently. As far as I know, the pilot in control plane is responsible for the configuration of data plane (Envoy proxy), and we configure Istio with YAML file in Kubernetes.
For example, if we want to configure a new virtual service, we will use kubectl apply -f newvs.yaml. I wonder how does Istio pilot retrieve the configuration and apply the new configuration in each data plane? Does it get from the Kubernetes master or somewhere else?
I was searching around and read several blogs but I didn't find an answer.

Generally Galley is used to read from kubernetes api server and pushes the config to Pilot. I am not very familiar with Galley but afaik all of the code for that can be found here https://github.com/istio/istio/tree/master/galley/pkg/source/kube.
I believe everything is based on shared informers

Galley subscribes to resources and when said resources are pushed to your k8s cluster, Galley will receive them, do some validation and push them to Pilot. Pilot then convert resources to Envoy config and distribute it to appropriate sidecars.


Union Syntax Requires Python 3.10

We replaced typing.Union with the pipe syntax (|) in this PR, but this is a feature that requires python 3.10. We should either bump the required version in pyproject.toml or revert the change.

I think itt should work as long as we have annotations exported:
from __future__ import annotations

I also run all of my projects on 3.9 to make sure it actually works :)

But...am I wrong?

Ahh, I think you are correct. I was going off a pyright error but I had forgot the annotations import.


Repository design: IR

aferritto created the following on Feb 09:
Analogue of #2 for IR

After requirements established (#26), create a design to support those requirements.

Original issue: https://github.ibm.com/ai-foundation/PrimeQA/issues/27

duplicate #22


Use corfu--initial-state instead of corfu--initial-state

Fixes https://github.com/radian-software/prescient.el/issues/158
I don't have a good understanding of corfu and prescient, so I haven't confirmed that they work properly. This patch may be insufficient. In that case, please feel free to close this PR.

This seems to work for me. Thank you for making this pull request.
@raxod, how do you want to handle the changed variable? Do you want the Prescient code to still work with the older variable until a new stable version of Corfu is published (which this PR currently does)?

That seems prudent, if the change was recent, then people might be using either the new version or the old version. Unless it's difficult, it makes sense to me to support both.
Of course, it goes without saying that it'd be nice if there were a public interface exposed that could be used instead.

Currently, the package dependencies are as follows:
;; Package-Requires: ((emacs "27.1") (prescient "6.1.0") (corfu "0.28"))

Corfu has been released 10 times since 0.28, but the variable change was included in 1.1.

Currently, the package dependencies are as follows:
;; Package-Requires: ((emacs "27.1") (prescient "6.1.0") (corfu "0.28"))

Corfu has been released 10 times since 0.28, but the variable change was included in 1.1.

I think that Emacs treats the 0.28 as a minimum version number, so it shouldn't cause any problems for the version before the change. Still, I have bumped it to 1.1. I will increase the required version number again once the next stable version of Corfu is released.
I have merged the PR. Thank you for the creating the issue and providing the fix.

@okamsn Thanks for improving and merging!


Broken link for 2.0.0-preview1-download

https://github.com/dotnet/core/blob/master/release-notes/download-archives/2.0.0-preview1-download.md
Corrected link: https://download.microsoft.com/download/0/4/8/048C286D-59CB-4B7C-95A1-D0F7FD4D37D2/dotnet-win-x<IP_ADDRESS>-preview1-002111-00.exe

Thanks @colbylwilliams, fixed.


Test around workload updates specifically labels/annotations

We are making some changes to accommodate Rio, that may have impact on updates.
There are some workload API updates and updates on labels/annotations that need to be checked for regressions.

@adingilloRancher Can you test use cases specific to  workload API updates and updates on labels/annotations to make sure no regressions are seen?

rancher/rancher:master at July 25th
I tested creating/deleting/updating workloads and tried updating labels and annotations for workloads and changing scale as well, all seems to be working fine.

@galal-hussein can you try with an upgraded scenario as well?

@deniseschannon upgrades are broken because of the bug https://github.com/rancher/rancher/issues/14733

@deniseschannon i was able to test upgraded scenario and confirm that label/annotations works as expected with workloads.


Cc/legacy shortcode fix

Pre-Flight checklist

[ ] Screenshots and design review for any changes that affect layout or styling
[ ] Migrations

[ ] Migration is backwards-compatible with current production code

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

[ ] Settings

[ ] New settings are documented and present in app.json
[ ] New settings have reasonable development defaults, if applicable
[ ] Opened issue for DevOps regarding necessary configuration changes to deployed environments

What are the relevant tickets?
(Required)
What's this PR do?
(Required)
How should this be manually tested?
(Required)
Where should the reviewer start?
(Optional)
Any background context you want to provide?
(Optional)
Screenshots (if appropriate)
(Optional)
What GIF best describes this PR or how it makes you feel?
(Optional)

Codecov Report

Merging #1335 (975c7d3) into master (669c4a7) will increase coverage by 5.39%.
The diff coverage is 97.87%.

@@            Coverage Diff             @@
##           master    #1335      +/-   ##
==========================================
+ Coverage   89.03%   94.42%   +5.39%
==========================================
  Files         230      109     -121
  Lines        9546     3354    -6192
  Branches     1778      802     -976
==========================================
- Hits         8499     3167    -5332
+ Misses        929      180     -749
+ Partials      118        7     -111

Impacted Files
Coverage Δ

static/js/lib/ckeditor/plugins/LegacyShortcodes.ts
98.71% <91.66%> (-1.29%)
:arrow_down:

static/js/lib/ckeditor/plugins/util.ts
98.20% <98.38%> (-0.05%)
:arrow_down:

static/js/lib/ckeditor/plugins/ResourceEmbed.ts
92.00% <100.00%> (ø)

...lib/ckeditor/plugins/ResourceLinkMarkdownSyntax.ts
100.00% <100.00%> (ø)

...mands/markdown_cleaning/rules/shortcode_logging.py

videos/urls.py

main/utils.py

videos/signals.py

videos/conftest.py

videos/tasks.py

... and 115 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 669c4a7...975c7d3. Read the comment docs.


[GHSA-78xj-cgh5-2h22] NPM IP package vulnerable to Server-Side Request Forgery (SSRF) attacks

Updates

Affected products

Comments
The vulnerability has been fixed in the 1.x and 2.x branches.
https://github.com/indutny/node-ip/pull/138#issuecomment-1951675086

Duplicate of #3553


Appended query to ttl file (or create a new one each time). Notifies ttl status upon log in. Then clears query.

Write ttl file to inbox of different user pod
explore if either ttl shall be existing and we just append or we create every time a new one
On Login to said pod, system needs to check if ttl fail existing / has values
if it has system shall query the data
on Successful query entry in ttl file or ttl file shall be deleted


⚠ Bofrost.lu has degraded performance

In 47bb757, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1098 ms

Resolved: Bofrost.lu performance has improved in 5da10cd after 6 minutes.


Define get Member by ID API Specification

Task to track the definition of get member by ID API

API Design completed