Xcode Version 12.0 beta 5 (12A8189h) Ad-Hoc build with 'iOS' and 'App Clip' target fails

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
With Xcode Version 12.0 beta 5 (12A8189h) an Ad-Hoc build with iOS and App Clip target fails after 'Archive Succeeded' on  'xcodebuild -exportArchive'. Directly in Xcode there is now the option to choose between 'iOS' and 'App Clip' where also the provisioning profile has to be selected.
Steps to reproduce: Just add a 'App Clip' target to an existing App and try to export 'Ad-Hoc' again.
Command executed
build_ios_app(export_method: "ad-hoc")
Complete output when running fastlane, including the stack trace and command used
[15:34:06]: ▸ ** ARCHIVE SUCCEEDED **
[15:34:06]:
[15:34:06]: ⬆  Check out the few lines of raw xcodebuild output above for potential hints on how to solve this error
[15:34:06]: 📋  For the complete and more detailed error log, check the full log at:
[15:34:06]: 📋  /Users/YXZ/Library/Logs/gym/YXZ iOS-YXZ iOS.log
[15:34:06]:
[15:34:06]: Looks like fastlane ran into a build/archive error with your project
[15:34:06]: It's hard to tell what's causing the error, so we wrote some guides on how
[15:34:06]: to troubleshoot build and signing issues: https://docs.fastlane.tools/codesigning/getting-started/
[15:34:06]: Before submitting an issue on GitHub, please follow the guide above and make
[15:34:06]: sure your project is set up correctly.
[15:34:06]: fastlane uses xcodebuild commands to generate your binary, you can see the
[15:34:06]: the full commands printed out in yellow in the above log.
[15:34:06]: Make sure to inspect the output above, as usually you'll find more error information there
Environment
| xcode_path | /Applications/Xcode-beta.app |
| gym_version | 2.156.1 |
| export_method | ad-hoc |
| sdk | iPhoneOS14.0.sdk

🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.15.6

Ruby
2.7.1

Bundler?
false

Git
git version 2.24.3 (Apple Git-128)

Installation Source
/usr/local/Cellar/fastlane/2.156.1/libexec/bin/fastlane

Host
Mac OS X 10.15.6 (19G2021)

Ruby Lib Dir
/usr/local/Cellar/ruby/2.7.1_2/lib

OpenSSL Version
OpenSSL 1.1.1f  31 Mar 2020

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

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

Xcode Version
12.0

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./fastlane/Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
update_fastlane

default_platform(:ios)

platform :ios do

    before_all do |lane, options|
        #cocoapods
    end

    before_each do |lane, options|

    end

    desc "Push a new beta build to Firebase"
    lane :updateFirebase do |options|

        build_ios_app(workspace: "XYZ.xcworkspace", configuration: "Release", scheme: "XYZ iOS", clean: true, silent: false, include_bitcode: true, export_method: "ad-hoc", include_symbols: true, output_name: "XYZ.ipa", codesigning_identity: "Apple Distribution: XYZ GmbH (XYZ)", skip_codesigning: true, skip_profile_detection: true, export_options: {
            method: "ad-hoc",
            provisioningProfiles: {
                "XYZ" => "match AdHoc XYZ",
                "XYZ.clip" => "match AdHoc XYZ.clip",
                "XYZ.widgets" => "match AdHoc XYZ.widgets"
            }
        })
    end
end

`./fastlane/Appfile`
```ruby
For more information about the Appfile, see:
https://docs.fastlane.tools/advanced/#appfile

### fastlane gems

| Gem      | Version | Update-Status |
| -------- | ------- | ------------- |
| fastlane | 2.156.1 | ✅ Up-To-Date  |

### Loaded fastlane plugins:

| Plugin                                    | Version | Update-Status |
| ----------------------------------------- | ------- | ------------- |
| fastlane-plugin-firebase_app_distribution | 0.1.4   | ✅ Up-To-Date  |
| fastlane-plugin-versioning                | 0.4.3   | ✅ Up-To-Date  |

Loaded gems

| Gem                                       | Version      |
| ----------------------------------------- | ------------ |
| did_you_mean                              | 1.4.0        |
| slack-notifier                            | 2.3.2        |
| atomos                                    | 0.1.3        |
| CFPropertyList                            | 3.0.2        |
| claide                                    | 1.0.3        |
| colored2                                  | 3.1.2        |
| nanaimo                                   | 0.3.0        |
| xcodeproj                                 | 1.18.0       |
| rouge                                     | 2.0.7        |
| xcpretty                                  | 0.3.0        |
| terminal-notifier                         | 2.0.0        |
| unicode-display_width                     | 1.7.0        |
| terminal-table                            | 1.8.0        |
| plist                                     | 3.5.0        |
| public_suffix                             | 4.0.5        |
| addressable                               | 2.7.0        |
| multipart-post                            | 2.0.0        |
| word_wrap                                 | 1.0.0        |
| tty-screen                                | 0.8.1        |
| tty-cursor                                | 0.7.1        |
| tty-spinner                               | 0.9.3        |
| babosa                                    | 1.0.3        |
| colored                                   | 1.2          |
| highline                                  | 1.7.10       |
| commander-fastlane                        | 4.4.6        |
| excon                                     | 0.76.0       |
| faraday                                   | 1.0.1        |
| unf_ext                                   | <IP_ADDRESS>      |
| unf                                       | 0.1.4        |
| domain_name                               | 0.5.20190701 |
| http-cookie                               | 1.0.3        |
| faraday-cookie_jar                        | 0.0.6        |
| faraday_middleware                        | 1.0.0        |
| fastimage                                 | 2.2.0        |
| gh_inspector                              | 1.1.3        |
| mini_magick                               | 4.10.1       |
| rubyzip                                   | 2.3.0        |
| security                                  | 0.1.3        |
| xcpretty-travis-formatter                 | 1.0.0        |
| dotenv                                    | 2.7.6        |
| bundler                                   | 2.1.2        |
| naturally                                 | 2.2.0        |
| simctl                                    | 1.6.8        |
| jwt                                       | 2.2.1        |
| uber                                      | 0.1.0        |
| declarative                               | 0.0.20       |
| declarative-option                        | 0.1.0        |
| representable                             | 3.0.4        |
| retriable                                 | 3.1.2        |
| mini_mime                                 | 1.0.2        |
| multi_json                                | 1.15.0       |
| signet                                    | 0.14.0       |
| memoist                                   | 0.16.2       |
| os                                        | 1.1.1        |
| googleauth                                | 0.13.1       |
| httpclient                                | 2.8.3        |
| google-api-client                         | 0.38.0       |
| google-cloud-env                          | 1.3.3        |
| google-cloud-errors                       | 1.0.1        |
| google-cloud-core                         | 1.5.0        |
| rake                                      | 13.0.1       |
| digest-crc                                | 0.6.1        |
| google-cloud-storage                      | 1.27.0       |
| emoji_regex                               | 3.0.0        |
| jmespath                                  | 1.4.0        |
| aws-eventstream                           | 1.1.0        |
| aws-sdk-core                              | 3.104.3      |
| aws-sdk-kms                               | 1.36.0       |
| aws-sdk-s3                                | 1.78.0       |
| uri                                       | 0.10.0       |
| json                                      | 2.3.1        |
| aws-partitions                            | 1.356.0      |
| aws-sigv4                                 | 1.2.2        |
| forwardable                               | 1.3.1        |
| logger                                    | 1.4.2        |
| cgi                                       | 0.1.0        |
| timeout                                   | 0.1.0        |
| stringio                                  | 0.1.0        |
| ipaddr                                    | 1.2.2        |
| openssl                                   | 2.1.2        |
| ostruct                                   | 0.2.0        |
| strscan                                   | 1.0.3        |
| date                                      | 3.0.0        |
| delegate                                  | 0.1.0        |
| fileutils                                 | 1.4.1        |
| io-console                                | 0.5.6        |
| zlib                                      | 1.1.0        |
| singleton                                 | 0.1.0        |
| rexml                                     | 3.2.3        |
| open3                                     | 0.1.0        |
| yaml                                      | 0.1.0        |
| psych                                     | 3.1.0        |
| mutex_m                                   | 0.1.0        |
| webrick                                   | 1.6.0        |
| fastlane-plugin-firebase_app_distribution | 0.1.4        |
| fastlane-plugin-versioning                | 0.4.3        |

*generated on:* **2020-08-25**

There is the Apple example app 'Fruta' which has a target 'App Clip' % 'Widgets'.
'Fruta'

With 'distributionBundleIdentifier' inside a 'ExportOptions.plist' file the build works with:
build_ios_app(workspace: "YXZ.xcworkspace", configuration: "Release", scheme: "YXZ iOS", clean: true, silent: false, include_bitcode: true, export_options: "YXZ iOS/ExportOptions.plist")


Config object injection fails in abstract class

In the following code snippet, config object named config  is null:
@Component
class AbstractT {
    @ApolloConfig
    private Config config;
}

@Service
class T extends AbstractT {
    @ApolloConfig
    private Config config1;
}

Thanks! This issue has been fixed in v0.10.0, verified with unit tests in #1039.

This case is closed for now. If there is any other problem, you could provide more information to this issue, thanks!


Fails to provision iscsi volumes on truenas scale

ElectricEel-24.10-RC.2
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG TrueNAS GET Request through fetch: 504
Tue, 22 Oct 2024 22:44:13 +0000 backend ERROR Not found: Volume with name pvc-7ec72f22-d747-4813-9027-72f25459836c not found.
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG Falcon Response (to HPE CSI): 404 Not Found
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG Last backend requests Response: 504 Gateway Time-out
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG API Key detected. Will use token authentication.
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG TrueNAS GET request URI: core/ping
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG TrueNAS response: "pong"
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG API fetch caught 1 item
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG HPE CSI Request <==============================>
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG          uri: http://truenas-csp-svc:8080/containers/v1/volumes?name=pvc-b59d2d9e-29f4-47b3-9757-0299fca39228
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG         body: None
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG        query: name=pvc-b59d2d9e-29f4-47b3-9757-0299fca39228
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG       method: GET
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG content_type: application/json
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG      headers: {"HOST": "truenas-csp-svc:8080", "USER-AGENT": "Go-http-client/1.1", "CONNECTION": "close", "ACCEPT": "
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG API Key detected. Will use token authentication.
Tue, 22 Oct 2024 22:44:13 +0000 backend DEBUG TrueNAS GET request URI: pool/dataset
Tue, 22 Oct 2024 22:44:16 +0000 backend DEBUG TrueNAS response:
<title>504 Gateway Time-out</title>

504 Gateway Time-out

nginx

Tue, 22 Oct 2024 22:49:16 +0000 backend ERROR Backend Request (GET) Exception: Traceback (most recent call last):
  File "/app/backend.py", line 404, in get
    self.req_backend.raise_for_status()
  File "/app/lib/python3.12/site-packages/requests/models.py", line 1024, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 504 Server Error: Gateway Time-out for url: https://hl15-1.home.thecrimsontint.com/api/v2.0/pool/dataset

Proof that truenas-csp pod can hit the API:
/app # hostname; wget -q -O- https://hl15-1.home.thecrimsontint.com/api/v2.0 | head -n5
truenas-csp-dbc4f9d86-xg9xn
{
 "openapi": "3.0.0",
 "info": {
  "title": "TrueNAS RESTful API",
  "version": "v2.0"
/app #

If I had to guess, your problem is here:
https://github.com/hpe-storage/truenas-csp/issues/66#issuecomment-2408110774
If I manually run that API call, it takes about 25s to return.

If I manually run that API call, it takes about 25s to return.

Thank you for reporting this. How many datasets (including snapshots) do you have?

About 20

I did some adhoc curl'ing. In the current shape and form, one second is added to curl -s<EMAIL_ADDRESS>for every 100 datasets. On my VM running on 10 year old hardware, 600 datasets took 5 seconds to GET.
Using the query-filters and dialing down query-options to just retrieve the bare minimum, the same GET takes 700ms and this is definitely the optimization I'm looking to do, but I think there's something else that is broken in your case.

Same truenas version?

Dragonfish-24.04.2

Ah.  I am ElectricEel-24.10-RC.2.
Also:  the entire way you do business will break in 25.04:

Seems this will be its replacement: https://github.com/truenas/api_client

That client performs the same api call in 17s.

Regardless... your hypothesis is overall correct.
I have two TrueNAS scale appliances.  One this RESTful API call works fine as is.  The other, it does not.  Happen to know where on a TrueNAS scale box the restful api logs?  Would be interesting to fully RCA this.

Runtime of RESTful call at localhost:
real    1m17.001s
user    0m0.004s
sys     0m0.033s

Indeed.  That, however, is an issue for iXSystems.  I'll file a ticket over there.
So it sounds like the path here is probably filters in a future release then websocket come 25.04 or maybe websocket first?

So it sounds like the path here is probably filters in a future release then websocket come 25.04 or maybe websocket first?

The next release by the end of the year will have query option filters. This will most likely be the last RESTful release before switching to websocket but that has not been scoped yet.


What happens in vkb::Camera::update()

Could someone please explain what's supposed to happen in vkb::Camera::update, especially on keys.up and keys.down?

What part specifically? It's mostly based on my samples but I guess it could be simplified.

Can't identify the issue any more. Closing.


some problems

1. Not working with python 3.2, had to use with 2.7
2. Can't use SSL for DICOM connection

Do you planning to develop pynetdicom?

Original issue reported on code.google.com by nixargh on 25 May 2012 at 10:59

Tracked by
https://github.com/patmun/pynetdicom/issues/38
https://github.com/patmun/pynetdicom/issues/41


healthchecks : Add Network, API and Ports startup checks.

Description
This PR creates the healthchecks package and adds the execution of the following checks at the Ops Agent startup :

API (Enabled, Permission) Check
This check consists of verifying that the Monitoring and Logging API’s are enabled and have the correct permissions .

Network (Failures) check
This check consists of verifying that the VM has a correctly configured network connection (firewalls, proxies, etc) enough to run the Ops Agent correctly.

Ports (Available) check
This check consists of verifying that the local ports used to communicate Fluent-Bit (20202) and Opentelemetry-Collector (20201) self metrics are available (listen and serve) to be used.

The results of the checks will be reported within the service (linux or windows) it runs (main or subagent). The checks will describe the failure with an ERROR_CODE (HealthChecksError struct, health_checks_error.go) and the following information is going to be user visible :

Failure description.
Error Code
Solution to the failure.
Link to a resource.

Most of the development effort was done in this branch https://github.com/GoogleCloudPlatform/ops-agent/pull/993 .
Related issue
b/262563192, b/266080802, b/261467774
How has this been tested?

Added unit tests to verify the functionality of health_checks package.
Added two new integration tests for the functionality of Network, API and Ports startup checks :

Implemented TestNetworkHealthCheck()
Implemented TestPortsAndAPIHealthChecks()
The execution of this tests will be restricted to a couple of platforms ( b/266080802 ), but for now it is open to running all of them to have full visibility.

Checklist:

Unit tests

[ ] Unit tests do not apply.
[x] Unit tests have been added/modified and passed for this PR.

Integration tests

[ ] Integration tests do not apply.
[x] Integration tests have been added/modified and passed for this PR.

Documentation

[ ] This PR introduces no user visible changes.
[ ] This PR introduces user visible changes and the corresponding documentation change has been made.
[x] This PR introduces user visible changes and the corresponding documentation will release at same time.

Minor version bump

[ ] This PR introduces no new features.
[x] This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
[ ] This PR bumps the version.

Current status of PR:

There is a problem with Ports Check since it fails (flake/unintended) when doing systemctl restart. This happens since fluent-bit takes an undetermined amount of time while stopping, so the fluent-bit port 20202 is still in use when the check runs again. The systemd doesn't have any ordering for the units (like using After), but even then if then there could still be problems with this check.

I added some small time.Sleep to address the flakiness, but it still fails sometimes (e.g. fluent-bit takes a while to wind down and stop).
Modifying systemd to address this issues seems too big of a big change.
An future iteration of the check using netstat could fix the issue (checking process name).
It only happens when restarting.
Thoughts ?

All of the current comments have been addressed.
Decided to disable testing on most of the distributions ( Will keep one (or two) Linux and one windows) as discussed with @martijnvans (see b/266080802).
Additionally skip tests on distributions that can't install netcat. This in conjunction of previous bullet.

CC @jefferbrecht , @rafaelwestphal , @jinghan-ma , @mtabasko

All healthchecks tests passed (running in debian-11 and windows-2019 only). All other tests passed, aside from one unrelated flake in Windows.


Casting to string makes the whole script fail in release build

Godot version:
6f4384f20
Steps to reproduce:

Add some Sprite node with a script
Put this code

func _ready() -> void:
	modulate = Color.red
	var a = 1.5
	var b = str(a)
	modulate = Color.blue

The sprite doesn't change color at all in release build

If you do var b = a it works fine.
Not sure about other conversions.

Most built-in functions can not be called in release build as there is a function call validation that requires information only available from DEBUG_ENABLED.
https://github.com/godotengine/godot/blob/07112b92a85de9b2134f3208149b295f871d3642/modules/gdscript/gdscript_analyzer.cpp#L1854 validate_call_arg compares argument count and type, even without DEBUG_ENABLED. It gets information about the function to call from few lines above
https://github.com/godotengine/godot/blob/07112b92a85de9b2134f3208149b295f871d3642/modules/gdscript/gdscript_analyzer.cpp#L1817 however get_info returns no information without DEBUG_ENABLED so comparing argument counts fails.
removing validate_call_arg from release builds solves the issue, but I do not know what kind of validations are still expected to run in release builds.

broken builtins have been reported here (https://github.com/godotengine/godot/issues/41706) but issue remained unresolved

Can't reproduce in https://downloads.tuxfamily.org/godotengine/testing/4.0/4.0-dev.20210727/
Here is a project with exports for Windows and Linux. (I couldn't export for macOS or some reason)
I tested Windows and it works just fine. I'm looking for someone with Linux to test
https://drive.google.com/drive/folders/1fZLyHgh6b7o5hQ6VUa622w4p_SsCgRTM?usp=sharing

Confirmed, it seems to work fine in<PHONE_NUMBER>50005f84d7b51a527bc779d76b905e.


Use anchors instead of line numbers for 1.1-1.3

Uses anchors instead of line numbers for chapters 1.1 through 1.3. Cherry picked from my larger PR #75. First part in resolving #46

hey @desttinghim, thanks a lot for this change! I think this is too big of a change to make after the book has already been written and lots of risk to mess up the code line numbers, will require a similar PR for all subsequent chapters, so I think I will close this for now. I might bring this back one day if I have more time to work on the book, thanks again! :)


Parse semicolons inside BEGIN/END block without ending statement

Right now, semicolons indicate the end of statement. If there is a BEGIN/END block with a semicolon, the outer statement is prematurely ended inside of the block.

I don't think this is a very promising way to solve this—it doesn't produce a meaningful syntactic structure for the content of the begin/end block, doesn't handle indentation, and doesn't cover similar constructs like if and loop. What kind of construct are you using the begin/end notation in?


⚠ Bofrost.fr has degraded performance

In 0bd80af, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 818 ms

Resolved: Bofrost.fr performance has improved in 573c128.


Cdk listbox control accessor

Implemented ControlValueAccessor in listbox, which entailed adding a value property and implementing several required methods. This is done to allow for listbox to be used with form control.

Addressed feedback and added some unit tests.

Feedback address. Ready for review and check on the changes.


Broken links in the Readme file.

The provided links in the Contribution Guidelines part of the README are not working.
I can fix them!

Hey @noviicee, please attach a screenshot marking the link that is broken. Thankyou.

@geekquad
Sure!
In the Contribution Guidelines of the README file,
When you click here-

You are redirected to here-

And when you click here-

You are redirected to here-

Hey @noviicee, go ahead!


Convert the actionResponse into a ViewModel if it's an associated array

Rather than return an ArrayObject, return a ViewModel. I think that this will have the side effect of allowing someone to return an array() from a controller action and the view-layer will do the right thing. This was noticed by ocramius on #zftalk.2.

Closing, as I've accomplished this now in a different fashion.

@weierophinney just for reference, is it 170cecda873ceca645fcb4b9b49856715b0ff88b and e97a53598b0556583d7b2ced3e277b4b6b790130 ?

Oramius: Yes. e97a535 is the outcome of this specific pull request and 170cecd accomplishes converting a returned array to a view model if required.


fix bug in pairs plot of recdevs

Turns out that the problem is with r4ss which accidentally was no longer returning the recruitpars element in the list created by SS_output. Adding fix now, but will require update of r4ss to get this plot to work.

Should be sorted out in recent commit.
Update to r4ss is associated with packageVersion('r4ss') 1.25.1.

Weird.. I didn't get that bug - it works for me. I'll update anyway. Thanks

This is the plot that wasn't working for me before. Perhaps it's because I was was using a too-new version of r4ss that broke it and now you're just leapfrogging over that bad version.

Ha ha, yes that might be what happened. I just re-installed r4ss to 1.25.1 and it's all good.

This was the same crash I had when I was having trouble building (without errors) the document.  I'll try again but I suspect it will be good once I devtools r4ss and source new version.

Aaron if it works for you, can you close this issue? Thanks - I'm just trying to whiddle down the open issues.

I've updated to 1.25.1 but still have the issue. I think I've re-run everything from scratch, but can do so again (on Monday), and can then close if Aaron has it working.

Ok, we can figure it out on Monday Andy. I am probably heading offline soon and not coming on tomorrow. We have a full week ahead of working on this and I think we are all doing a good job of keeping on top of it. No worries!

I'll try deleting the new .RData file in the base MCMC directory, since that was created before I updated r4ss.

Hmm... still don't get the pairs plot. Will figure out on Monday.

I would try this, it removes all model rdata files:
delete.rdata.files("c:/github/hake-assessment/models")
then source(:all.r") again and try to rebuild.
This is the "turn-it-off" and "turn-it-on-again" approach.

Okay, pairs plot had built. I had a latex error (that I thought was still due to the pairs plot), but turns out it was for some other plots (which I think might be just because we haven't redone the calculations yet).
Aaron can close this when he's good.

Hopefully everybody is sorted out, but the key is that for some versions of r4ss, the base.model object was missing the $recruitpars element. This was used to get the names of all recdev parameters for subsetting the things to show in the pairs plot.
As long as base.model$recruitpars isn't NULL, then it should be OK. Deleting \models\41_BasePreSRG_v3\41_BasePreSRG_v3.RData and then re-running all.r should be enough.
This is what the head of that object is supposed to look like
head(base.model$recruitpars)
                     Value Parm_StDev          type   Yr
Early_InitAge_20 -0.373814    1.20514 Early_InitAge 1946
Early_InitAge_19 -0.116222    1.32603 Early_InitAge 1947
Early_InitAge_18 -0.140671    1.31201 Early_InitAge 1948
Early_InitAge_17 -0.169516    1.29605 Early_InitAge 1949
Early_InitAge_16 -0.203272    1.27811 Early_InitAge 1950
Early_InitAge_15 -0.242445    1.25819 Early_InitAge 1951

Worked for me so closing...


NuoDbParameter - Decimal Value 0 - 100% CPU Load

Hi,
if i try to insert a record into a table, with a decimal field and fill the value within c# with a decimal value of 0, it seems the driver is hanging (and produces 100% cpu load).
Simple Table DDL
create table T1 (ID integer, VALUE numeric(15,2));
Attached a VS2015 Example with Driver Version <IP_ADDRESS>
synesc.NuoDbDecimal.zip

Thanks for the bug report; it is fixed now


Use the legacy Crout solver : nrn / scopmath / crout.c

Experimentation with different versions of the Crout solver

@pramodk
FYI, I decided to replace the previous crout solver that I have introduced with the legacy one found in NEURON: https://github.com/neuronsimulator/nrn/blob/master/src/scopmath/crout.c
I took this decision in order to have a way more bullet-proof solver and be sure that our tests do not fail because of some implementation error.

@olupton @pramodk this PR passes all the tests, i.e.:

https://bbpgitlab.epfl.ch/hpc/coreneuron/-/pipelines/81127 (correctness)
https://bbpgitlab.epfl.ch/hpc/cellular/coreneuron-perf-tests/-/pipelines/81133 (good performance)
Therefore, it is safe to be merged and replace the previous and merged Crout implementation (with the bug in the error handling).

I have opened this issue #971 to enhance the error checking. Currently, the error handling is through an assert. Not ideal but for now it is a sufficient solution.


Allow null for bufferData and namedBufferData

Closes #23

I think this is a bad way as it introduces more footguns than it helps. I would either make two functions where one takes a length, the other takes a slice or make a function that takes a union(enum) { length: usize, data: []const T }

Made a new PR #36 that uses separate functions for uninitialized buffers


Allow spaces in pyproject.toml config file

Closes #148

Codecov Report

Merging #149 (c4f0e45) into main (8ca3612) will decrease coverage by 0.07%.
The diff coverage is 66.66%.

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

@@            Coverage Diff             @@
##             main     #149      +/-   ##
==========================================
- Coverage   97.04%   96.97%   -0.08%
==========================================
  Files          24       24
  Lines        1288     1290       +2
==========================================
+ Hits         1250     1251       +1
- Misses         38       39       +1

Impacted Files
Coverage Δ

robotidy/utils.py
95.55% <50.00%> (-0.69%)
:arrow_down:

robotidy/cli.py
93.83% <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 8ca3612...e65a091. Read the comment docs.


TypeError: Cannot read property 'email' of null

Hello, I just installed this plugin with no bond fans set up idk if this is the issue, but if it is please lmk. My unit comes by Thursday.
#Config:
{
	"bridge": {
		"name": "Homebridge",
		"username": "CC:22:3D:E3:CE:30",
		"port": 51826,
		"pin": "031-45-154"
	},
	"description": "Homebridge",
	"accessories": [{
			"accessory": "Chamberlain",
			"name": "Double garage",
			"username"<EMAIL_ADDRESS>			"password": "Canada113",
			"deviceId": "xxx"
		},
		{
			"accessory": "Chamberlain",
			"name": "Single Garage",
			"username"<EMAIL_ADDRESS>			"password": "Canada113",
			"deviceId": "xxx"
		},
		{
			"accessory": "DummySwitch",
			"name": "Lights",
			"stateful": true
		},
		{
			"accessory": "SonyTV",
			"name": "TV",
			"mac": "xx",
			"ip": "10.0.0.xx",
			"tvsource": "tv:dvbs",
			"maxchannels": 500,
			"soundoutput": "speaker",
			"onscript": "/home/pi/scripts/dolbyon.sh",
			"offscript": "/home/pi/scripts/dolbyoff.sh",
			"savefilenameprefix": "/home/pi/sonycookie"

		},
		{
			"accessory": "DummySwitch",
			"name": "Living Room Lights",
			"stateful": true
		},
		{
			"accessory": "DummySwitch",
			"name": "Master BedRoom Lights",
			"stateful": true
		},
		{
			"accessory": "DummySwitch",
			"name": "BackYard Lights",
			"stateful": true
		}
	],
	"platforms": [{
		"platform": "Doorbell",
		"doorbells": [{
				"name": "Doorbell Front"
			},
			{
				"platform": "TplinkSmarthome",
				"name": "TplinkSmarthome",

				"devices": [{
						"host": "10.0.0.xx"
					},
					{
						"platform": "Bond",
						"email": "xx",
						"password": "xx"
					}

				]
			}
		]
	}]

}

And with this I get the error:
Load homebridge-bond.Bond
/usr/local/lib/node_modules/homebridge-bond/dist/index.js:10
let email = config['email'];
^
TypeError: Cannot read property 'email' of null
at new BondPlatform (/usr/local/lib/node_modules/homebridge-bond/dist/index.js:10:27)
at Server._loadDynamicPlatforms (/usr/local/lib/node_modules/homebridge/lib/server.js:347:30)
at Server.run (/usr/local/lib/node_modules/homebridge/lib/server.js:88:8)
at module.exports (/usr/local/lib/node_modules/homebridge/lib/cli.js:45:10)
at Object. (/usr/local/lib/node_modules/homebridge/bin/homebridge:17:22)
at Module._compile (module.js:569:30)
at Object.Module._extensions..js (module.js:580:10)
at Module.load (module.js:503:32)
at tryModuleLoad (module.js:466:12)
at Function.Module._load (module.js:458:3)
at Function.Module.runMain (module.js:605:10)
at startup (bootstrap_node.js:158:16)
at bootstrap_node.js:575:3

You need to configure,your credentials for bond.
In your config, add the platforms,section and provide your Creds
"bridge": {
"name": "Homebridge",
"username": "CC:22:3D:E3:CE:30",
"port": 51826,
"pin": "031-45-154"
},
"platforms":[{
"platform": "Bond",
"email": "",
"password": ""
}],
"accessories": [

I have my credentials in there, just removed them so people don't login to my account

I have no fans synced with my bond, yet I just wanted to make sure the plugin works while my unit arrives Thursday. Is this why it may not be working?

OMG I got it I figured out why it wouldn't work. It had to be at the top of the platforms since it was trying to be a switch. Thanks for the reply!

I had a similar error but at a different location. Checked my config and even tried the version in config.example.json without success.
`Load homebridge-bond.Bond
/usr/lib/node_modules/homebridge-bond/dist/index.js:10
let email = config['email'];
^
TypeError: Cannot read property 'email' of null
at new BondPlatform (/usr/lib/node_modules/homebridge-bond/dist/index.js:10:27)
at Server._loadDynamicPlatforms (/usr/lib/node_modules/homebridge/lib/server.js:347:30)
at Server.run (/usr/lib/node_modules/homebridge/lib/server.js:88:8)
at module.exports (/usr/lib/node_modules/homebridge/lib/cli.js:45:10)
at Object. (/usr/lib/node_modules/homebridge/bin/homebridge:17:22)
at Module._compile (module.js:653:30)
at Object.Module._extensions..js (module.js:664:10)
at Module.load (module.js:566:32)
at tryModuleLoad (module.js:506:12)
at Function.Module._load (module.js:498:3)
at Function.Module.runMain (module.js:694:10)
at startup (bootstrap_node.js:204:16)
at bootstrap_node.js:625:3
`
I know it's bad, but I did hardcode my email and pass in /usr/lib/node_modules/homebridge-bond/dist/index.js and it worked, so is there an error locating my config.json? or am I missing something else?


Sending Commands to CBD6S

I have a Steelcase Ology which uses a Linak CBD6SP00020A-109. It has a Steelcase bluetooth adapter and I am able to connect successfully.
However using --move-to ### does nothing.
I would be happy to help diagnose and add support for this desk. Do you have any suggestions on how to proceed? The issue seems similar to #32. Looks like an error with sending commands.
Connected E5A13E83-F9E0-FC7F-6886-8ECCB0C8B30F
Capabilities: {'memSize': 4, 'autoUp': True, 'autoDown': True, 'bleAllow': False, 'hasDisplay': True, 'hasLight': True}
User ID: 01 e5 a1 3e 83 f9 e0 fc 7f 68 86 8e cc b0 c8 b3 0f
Base height from desk:  686mm
Height:  686mm
Moving to height: 700
Final height:  686mm (Target:  700mm)
Disconnected

Hello! Sorry for the delayed response.
Can you describe the setup more? Its a Linak CBD6SP00020A-109 connected to some non-Linak bluetooth adapter? (And for clarity, the adapter is plugged into your desk, not your computer?). How do you normally control the desk?
A bit suspicious of the 'bleAllow': False being reported in the capabilities. If that means that connecting via BLE is not allowed then its unlikely we can make it work with this script. On the other hand those capabilities are fetched via BLE so I'm not really sure if thats meaningful 😄

No worries. The bluetooth adapter is a Linak controller. Yes the adapter is plugged into the desk and not my computer. I normally control it with this adapter. It's sold by Steelcase as an Active Touch Controller
I can see the device info in Bluetility
The physical tag on the device says this:
Linak
Type: DPG1C72-000017
Item: DPG1C001-02
SW: 007088 ver 1.08
Model: DBG1C


Plugin do not work on 2021.3

Hello, plugin do not work on 2021.3
Plugin error: Plugin 'Regex Rename Files' (version '1.1.5') is not compatible with the current version of the IDE, because it requires build 212.* or older but the current build is WS-213.5744.224

Thanks for letting me know! I will try update it this weekend.


Fullscreen doesn't work in mobile browser

hi @CookPete - not sure if this is a known issue or something I need to configure. Fullscreen functionality works great on desktop but doesn't seem to work in mobile browsers. I tried Chrome and Safari and neither seem to work. Any chance you know why?

Which mobile browsers and which devices have you tried this on? Fullscreen functionality on mobile OS's is a lot different than on the desktop, so I'm not surprised that there are issues.

@nikolas Chrome and Safari. iOS

Fullscreen functionality is a separate beast from the ReactPlayer library. The demo page uses screenfull.js as an example of how to use the JS fullscreen API with ReactPlayer (which is not supported by iOS or Android) and the fullscreen buttons inside the player are powered by the various different APIs for each video type, which are out of our control.
Alternatively, this might be solved by just adding the allowfullscreen attribute in the right places.

@CookPete yep, feel free to close this issue. I learned that iOS Safari does not support the native Fullscreen API http://caniuse.com/#search=fullscreen (which is what screefull.js uses underneath to implement it's API).


Fix GitHub Contributors Action

Expected Behavior
Run Contributors action weekly to generate an SVG
Actual Behavior

I think this is the blocker
https://github.com/orgs/community/discussions/23064

@rishabkumar7  Not sure, if this is the correct fix, but in my main branch I was able to get the action successfull by enabling the following in the settings of the repo:

@rishabkumar7 take a look when you can please

@rishabkumar7
So I was looking at the error, and it says that it needs a PR to make changes.
As per the action github, I was reading a issue, and the owner mentioned that it creates a PR using the default branch of the repo because of the github API.
When I repro it did worked for me, and my branch is not Protected, is staging protected?

@rishabkumar7

@azgcloudev yeah, staging is protected hence it creates a PR and can't just merge it.

@rishabkumar7  what are we doing about this?

@madebygps I am trying to see if there is a workaround. Will disable the action for now.


Update chainlist

[x] Add support for Blast Mainnet https://github.com/Koniverse/SubWallet-ChainList/issues/288
[x] Add in-app staking support for DENTX https://github.com/Koniverse/SubWallet-ChainList/issues/289

https://github.com/Koniverse/SubWallet-Extension/issues/3455

Version test : http://<IP_ADDRESS>:8900/s/FLe4goGZe6wGXyF
Validation passed in case :

Install/Update version
https://github.com/Koniverse/SubWallet-ChainList/issues/288

https://github.com/Koniverse/SubWallet-ChainList/issues/289


About training code

Halo guys，
Thanks for your great work.
Will your training code be released on time(7.8) and will it include the training code of RAM?

Hi, thanks for your attention!
We have open-source Tag2Text forward function, which can refer to README.md.  While I would personally love to open source all the training codes right away, they are still in the company's process.  We are actively working towards making them available as soon as possible.
Best wishes!

Hi, thanks for your attention! We have open-source Tag2Text forward function, which can refer to README.md. While I would personally love to open source all the training codes right away, they are still in the company's process. We are actively working towards making them available as soon as possible. Best wishes!

Hi, thanks for your great job! I want to ask if I replace 'blip.py' with 'ram.py', it also seems to be able to run(but I haven't tried), is that right? I see the code structure is similar.Thanks for reply

Besides, another key improvement is: BLIP only read two key-value pairs: {'image': path_of_image, 'caption': text_of_image}, and Tag2Text need to read three key-value pairs: {'image': path_of_image, 'caption': text_of_image, 'tag': the tags of the image, parsed from the caption}.

Thanks. So when will the ram training code be released? maybe an approximate time？

非常赞的工作，非常期待训练代码～～


pandas.DataFrame variables with MultiIndex won't be showed correctly in data viewer

In data viewer, index of a pandas.DataFrame variable with MultiIndex will not be correctly showed.

Jupyter server running:                       Local
Extension version:                               2019.10.44104 (22 October 2019)
VS Code version:                                1.39.2
Setting python.jediEnabled:               false
Python and/or Anaconda version:     3.7.4(x64)
OS:                                                     Windows
Virtual environment:                         N/A

a sample of code
import pandas as pd
import random
list1 = [[random.uniform(0, 1) < 0.5 for i in range(20)], [random.uniform(
    0, 1) < 0.5 for i in range(20)], [random.uniform(0, 1) < 0.5 for i in range(20)]]
data1 = pd.DataFrame(list1)
data1 = data1.T
data1.columns = ['a', 'b', 'c']
data1['d'] = 1
data2 = data1.groupby(['a', 'b', 'c']).count()
data2

Thanks for letting us know. We'll look into it.
Also thanks for your code sample, it made it quick to reproduce the issue.

Duplicate of #6663


Reasonable way to avoid locale resetting on preload?

I have a few buttons for selecting the user language which change the $locale store to the locale string. I had an issue where $locale would get reset to whatever initLocale was in the +layout.js when hovering over links (preload) and when changing the route.
I fixed(?) it by using the following code:
const initLocale = get(locale) ? get(locale) : "en"; // get from cookie, user session, ...

await loadTranslations(initLocale, pathname);

which seems to work as it uses the $locale store value if it exists, otherwise "en" as a default. (Should probably use cookies/localstorage for this later)
My main question is, is there a better way to do this? I'm very new to i18n.

Hi @henrikvilhelmberglund!
Your solution is quite tricky as the locale store is shared between users on server. Recommended is to use cookies to keep user’s preferred language.. see example here.


Upgrade most javascript dependencies to latest versions

I have manually tested that this works correctly for me in both Jupyter lab 3 and notebook.
With this change all javascript dependencies are up to date except the following which requires an apation of the
tooltip calls since tippy/popper has changed their api.

❯ npm outdated
Package           Current  Wanted  Latest  Location                       Depended by
cytoscape-popper    1.0.7   1.0.7   2.0.0  node_modules/cytoscape-popper  ipycytoscape
tippy.js            5.2.1   5.2.1   6.3.7  node_modules/tippy.js          ipycytoscape

Hi @jenshnielsen, thanks for the PR!
So, actually, one thing worries me about the PR, setting everything for the latest version could cause some version conflicts in user's machines that depend on older software.
I asked my colleagues and the preferred way to go about versioning is by adding a lower boundary of suport and a high one.
For the low, I'd say we can use whatever it was the lowest before and for the highest, we can use the new values you added on this PR. What do you think?
example:
node>=13.2.0,<=17.0.0

Hey @jenshnielsen thanks for opening this, but I updated it with npm update seems like the recommend way of doing by npm and seems to work fine.


reduce configuration of authentication with named servers

This PR removes use_named_servers config from binderhub and makes Launcher to get it from an env variable which is set by jupyterhub.hub.allowNamedServers.

Spoke too soon... ran into another problem:
traitlets.traitlets.TraitError: The 'allow_named_servers' trait of a Launcher instance must be a boolean, but a value of 'true' <class 'str'> was specified.

`jupyterhub` config:
```yaml
jupyterhub:
  custom:
    binderauth_enabled: true
  hub:
    baseUrl: /jupyter/
    redirectToServer: false
    services:
      binder:
        oauth_redirect_uri: "http://example.org/oauth_callback"
        oauth_client_id: "binder-oauth-client-test"
    # Allow up to 2 servers per authenticated user
    allowNamedServers: true
    namedServerLimitPerUser: 2

@manics sorry, I forgot about that. Now I tested this locally and it works.
And you are right we have to find a way to add tests for these cases.

@betatim what do you think? can we merge this?

I've not deployed a BinderHub with auth/named servers in a long time and won't get around to trying (and thinking about) this before I leave for my holiday. Maybe @minrk has a moment to do so.
If the layout/format of the yaml files changes (where you define something) we should figure out a way to emit a helpful error message to users who upgrade and wonder why everything breaks. Also, it should not quietly (but in a broken state) keep working if someone uses their old helm chart. (I've not looked closely enough to know if you changed how the helm chart values are organised so this is a "shot into the dark", ignore it if it is irrelevant :))
Otherwise, if you think this is ready to go: merge it. When we find bugs we can always fix them later.

If the layout/format of the yaml files changes (where you define something) we should figure out a way to emit a helpful error message to users who upgrade and wonder why everything breaks.

I think this is one example where having defined releases helps, otherwise there's ambiguity about whether something that's changed in one PR and reverted/modified in another PR a week later is "breaking".

I think this is one example where having defined releases helps

Agreed. In this case I was thinking "are we changing something that has been that way for a long time"? What is a long time? Months?
If we have releases we need to come up with a policy for deprecating features (marked as deprecated in release N, emits a warning if you turn on warnings, release N+1 always emits a warning, and then in release N+2 using it is "broken") and giving people advice on how to transition. We could build that infrastructure pretending that each merge to master is a release and then maybe do time based releases? I don't think just doing the semver thing of bumping the version numbering appropriately is enough because it doesn't solve the problem that an update breaks people's setup and not everyone can update their setup "over night".
(Maybe this conversation should be in its own issue though)


When will the next new stable release be published?

Hi there,
I'm hoping to utilize the change that reduce dependencies when one imports quic-go.
Do you know when that change will be published into a stable release?
thank you in advance

Soon(-ish).

Targeting a new release by middle / end of the week. If you want to get any changes in, now would be a good time to send the PR.


how to plot the output from Quadrature file.

Dear Dorian,
Thank you for creating such a nice toolbox.  I created the gmsh and orientation file for input using the MTEX2Gmsh toolbox.
I ran some simulations. The output is the Quadrature file. I have now the x, yz, r_x,r_y and r_z as output. I wish to plot it.
(1) Is there any possibility of plotting it directly or I have to convert it to Bunge angles first.
(2) I suppose its unstructured mesh. So I should use paraview or some other software.
Regards
Rajesh

Hello Rajesh,
I have written a small function for reading Quadrature outputs files and converting those data in MTEX's EBSD format (see here), so that you can easily plot the results. Indeed, since we use an unstructured mesh, the EBSD map will appear somehow sparse. Consider using the fill function from MTEX if needed.
Also, have a look on the cavehat (mentioned in the help section) if your quadrature output files are too large files.
Regards.

Thanks Dorian,
You have taken care of NaN also. .
Another question, Possibly you are using Prisms.
When you use 2D data, how large a strain you can give? For example, if you want to do plane strain compression, upto what strain you can get reliable results. I am getting strange stress strain curves at large strains.
Regards
Rajesh

I have no idea how large the strain can be. You will for sure have issues in compression if your elements are too distorted. The only workaround I know is remeshing; but AFAIK, this feature is not (yet) available in Prism-plasticity. Also note that full-hex mesh (required by PRISMS-Plasticity) inevitably results in some poor-quality elements (see for instance the stats in Gmsh); if those elements are heavily distorted, this can lead to unrealistic results.
Maybe have a look on FEPX, which does support remeshing. Note that MTEX2Gmsh can be used for FEPX as well, but the latter only works with tetrahedral elements, which is obviously not the best choice for 2D problems.
Regards.


setuptools-markdown is deprecated

This is a ticket to migrate away from setuptools-markdown since it is deprecated:
https://github.com/msabramo/setuptools-markdown?tab=readme-ov-file#this-project-is-deprecated

Latest commits include a revamp of package to use hatch instead of setuptools, which also eliminates this dependency.

That's great! Thanks for adding that. Much appreciated.


Refactored a Code a bit

@iamusmanazeem the code is untested can you deploy and test if things are still working?

@iamusmanazeem if this is working on AWS then i will fix the test cases


Support for ._domainkey. CNAMEs

Most SaS uses a dotted CNAME postfixed with ._domainkey., currently those entries do not show up on VinylDNS. Creating a zone just for this entries feels odd. Can this entries be allowed as CNAMEs?
I understand and agree with the idea of not allowing dotted names for most entries, and that this would be yet another exception to this rule.
I can work on a PR implementing this if needed.

There has been some discussion on this, specifically around whitelisting certain dotted labels via configuration. Given that there are specific patterns for various verifications, I think it's a safe approach to define some sort of configuration. This way, VinylDNS allows administrators to define what works for their system instead of imposing hard rules which may not apply in all circumstances.
Example config:
vinyldns {
  dotted-label-whitelist = [
 	{
		type: "CNAME"
		pattern: "^[^.]+\._domainkey$"
	}
  ]
}

Seems like the best way to handle this. I'm not sure if my scala skill is sharp enough to tackle this issue, with some mentoring and reviews I might be able to. Would that work?


...infer X extends TypeParameter[] in conditional type inappropriate binding

Bug Report

🔎 Search Terms
conditional type, extends, infer, spread, tuple
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about extends

⏯ Playground Link
Playground link with relevant code
💻 Code

type Foo<K, T extends readonly unknown[]> = T extends readonly [any, ...infer X extends readonly unknown[]] ? X : never;
type Bar<K, T extends readonly unknown[]> = T extends readonly [any, ...infer X extends readonly K[]] ? X : never;
type Baz<K, T extends readonly unknown[]> = T extends readonly [any, ...infer X extends readonly (K | "a" | "b")[]] ? X : never;

type x = Foo<"a" | "b", ["a", "b", "b"]>
//   ^? - type x = ["b", "b"]
type y = Bar<"a" | "b", ["a", "b", "b"]>
//   ^? - type y = readonly ("a" | "b")[]
type z = Baz<"a" | "b", ["a", "b", "b"]>
//   ^? - type z = ["b", "b"]

🙁 Actual behavior
The conditional types do not produce the same output:
type x = Foo<"a" | "b", ["a", "b", "b"]> // ["b", "b"]
type y = Bar<"a" | "b", ["a", "b", "b"]> // readonly ("a" | "b")[]
type z = Baz<"a" | "b", ["a", "b", "b"]> // ["b", "b"]

🙂 Expected behavior
The conditional types should all produce the same output. The fact that K is a type parameter should not affect the output.
type x = Foo<"a" | "b", ["a", "b", "b"]> // ["b", "b"]
type y = Bar<"a" | "b", ["a", "b", "b"]> // ["b", "b"]
type z = Baz<"a" | "b", ["a", "b", "b"]> // ["b", "b"]

Alternatively if that is infeasible for technical reasons, the conditional type should fail. Inferring the constraint makes traversing the tuple impossible.

Here's another reproduction, showing how it works if the tuple is reversed.
type Heads<T extends any[]> = T extends [... infer Rest, any] ? Rest : never;
type Tails<T extends any[]> = T extends [any, ... infer Rest] ? Rest : never;

// Expected: [1, 2]
// Actual:   never
type heads = Heads<[1, 2, 3]>;

// Expected: [2, 3]
// Actual:   [2, 3]
type tails = Tails<[1, 2, 3]>;


Support pulsar jwt auth type

Codecov Report

Merging #115 (903ec2a) into main (fcca5d9) will increase coverage by 0.42%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main     #115      +/-   ##
============================================
+ Coverage     23.05%   23.48%   +0.42%
  Complexity      147      147
============================================
  Files            80       80
  Lines          1327     1303      -24
  Branches         75       73       -2
============================================
  Hits            306      306
+ Misses         1005      981      -24
  Partials         16       16

Flag
Coverage Δ

unittests
23.48% <0.00%> (+0.42%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../com/github/shoothzj/dev/constant/PulsarConst.java
0.00% <0.00%> (ø)

...zj/dev/simulator/pulsar/PulsarClientSimulator.java
0.00% <0.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 fcca5d9...903ec2a. Read the comment docs.


YouTube app history not being recorded.

I use pi-hole, which by default ships with this filterlist. Recently, I've noticed that in the youtube app for freebsd (ios i meant) doesn't properly record history. It may record some videos, but not all. Which filter may be causing this?

Hi @iambeingtracked this issue has come up before, notably

#1379
#1862

... and some others.
YouTube uses this domain for multiple purposes, including ads, and we decided to continue blocking this.

IMHO this it's a bad decision to block important features. A lot of people have this issue, not knowing that pihole is the problem.


Distinguish components from external add-ons

This fixes #2536 I'm expecting to make small fixes depending on reviews.

@shannonbradshaw, looks good but I think we need to update content/en/_redirects to prevent dead links.
@RFMVasconcelos any other thoughts, or is this good to merge (after those redirects), so we can close #2536

Thank you for this work @shannonbradshaw!
A couple of missing things:

Redirects in _redirects file so that external links are not broken
Do a search for internal links and update them accordingly not to break them upon these change

Do you think you could take on this extra work?

Yes. I'm checking through to do redirects.

@RFMVasconcelos @thesuperzapper, thanks for reviewing! I believe I've addressed your comments, including redirects and internal links. I did test and I believe the redirects and changes I made are working; however, given the tedious nature of these changes it's possible I missed something. Please let me know if you notice any further changes that need to be made.

@shannonbradshaw can you resolve the merge conflict?
@RFMVasconcelos I think this is a pretty good first pass, once the merge conflict is resolved, I think you can /approve

I broke this PR while attempting to rebase. Replacing this with a clean PR: #2630


Error when running the program outside the installed program folder

Hi, I have a question regarding the job submission and running the vina- gpu+,
After installation I already run the example as show in the instruction.it was successful. However, I run the job in different directory, I got an error. It said something like “ ./OpenGL/…. Cannot be found” (sorry I cannot remember it exactly, and will update it asap). So, I tried running the job like the example by creating the subfolder within the program directory and submitted the job like this
./Vina-GPU —config ./input_file/config.txt
It turned out successfully.
Anyone got the same issue? How to solve it? Or I have to run my jobs inside this folder only?
FYI. I’m using a Linux version on Ubuntu.
.

Hi, you need to include the Kernel files in the directory, either make symlinks or copy them before running the program

Thank you for the solution. I will try


reset train dataloader after OOM

What does this PR do?

Fixes #9625
binsearch scaling in batch size scaling wasn't reseting the dataloader batchsize after an OOM error.
Calling the reset function for the train dataloader fixes that.
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Is this coming in to the next release?

@mogwai This PR still needs to address the comments and write a test etc.
You can work on a new PR to finish it if you are interested in contributing!


Remove synom

Removed synom due to merge into syn in  https://github.com/dtolnay/syn/commit/c5ab8c6b9fb7bf3f303abac2086485f272a73ebf
syn::tokens is now syn::token :unamused:

@alexcrichton I would additionally lock down the version of syn so that this stuff doesn't happen again, but I'd leave that up to you.
Fixes #1

Thanks!


Use UITableViewController as parent type for menu

This is to fix #37 for everybody who need to subclass UITableViewController. I don't think this can be merged as is – it's rather here as a workaround until the real solution is implemented. Simply use my fork until then.
For Carthage users the inclusion would look like this as an example:
github "Dschee/InteractiveSideMenu" "master"

@Dschee Thanks for making your solution to this  #37.  We really appreciate it.
Unfortunately, these changes don't provide appropriate flexibility to client code and we don't feel able to merge it in the current state.


allow click 8.*

Updates the project requirements to allow click 8 to be installed. (^7.1.2 locked it to >=7.1.2,<8)

Hi, any update on this ?

Hey guys, so sorry but I totally missed this


AWS: Use RSA to Generate Fingerprint

I took the daring leap into OS X 10.11 and found that openssl no longer supports the pkey command.  Update the aws script to use rsa instead which seems to be functionally equivalent. I verified rsa exists on older OS X and ubuntu as well and all seem to produce the same output.
@justinsb PTAL

Did apple also change the behavior of curl wrt certs again (like they did in the last two OS updates)? :)

curl --version:
curl 7.37.1 (x86_64-apple-darwin15.0) libcurl/7.37.1 SecureTransport zlib/1.2.5
Protocols: dict file ftp ftps gopher http https imap imaps ldap ldaps pop3 pop3s rtsp smtp smtps telnet tftp
Features: AsynchDNS GSS-Negotiate IPv6 Largefile NTLM NTLM_WB SSL libz

Which is the same as 10.10, so I'm guessing hoping that means they left it alone. :)

I can confirm the issue using OS X 10.10.3.
openssl:Error: 'pkey' is an invalid command.

Standard commands
asn1parse      ca             ciphers        crl            crl2pkcs7
dgst           dh             dhparam        dsa            dsaparam
ec             ecparam        enc            engine         errstr
gendh          gendsa         genrsa         nseq           ocsp
passwd         pkcs12         pkcs7          pkcs8          prime
rand           req            rsa            rsautl         s_client
s_server       s_time         sess_id        smime          speed
spkac          verify         version        x509

Message Digest commands (see the `dgst' command for more details)
md2            md4            md5            mdc2           rmd160
sha            sha1

Cipher commands (see the `enc' command for more details)
aes-128-cbc    aes-128-ecb    aes-192-cbc    aes-192-ecb    aes-256-cbc
aes-256-ecb    base64         bf             bf-cbc         bf-cfb
bf-ecb         bf-ofb         cast           cast-cbc       cast5-cbc
cast5-cfb      cast5-ecb      cast5-ofb      des            des-cbc
des-cfb        des-ecb        des-ede        des-ede-cbc    des-ede-cfb
des-ede-ofb    des-ede3       des-ede3-cbc   des-ede3-cfb   des-ede3-ofb
des-ofb        des3           desx           rc2            rc2-40-cbc
rc2-64-cbc     rc2-cbc        rc2-cfb        rc2-ecb        rc2-ofb
rc4            rc4-40         rc5            rc5-cbc        rc5-cfb
rc5-ecb        rc5-ofb        seed           seed-cbc       seed-cfb
seed-ecb       seed-ofb

PEM_write_RSA_PUBKEY failed

Please let me know if I can provide additional debug information.

@iterion Can you check whether this actually prints an error if you run it directly?  It doesn't look like (either version) is actually working correctly on OSX (10.9.5) for me.
I'm tempted to just go with ssh-keygen -lf  ${pubkey_path}  | cut -f2 -d'   It doesn't match the AWS fingerprint, but there's only so much we can do here.

No errors for me. If I run it directly I get a fingerprint as expected. It also matches what amazon generates if I upload it through the aws web console.
Your proposed change also generates a fingerprint but doesn't match AWS as you said. Does AWS care if the fingerprint matches?

AWS has a really "unusual" way of computing their fingerprint (e.g. http://serverfault.com/questions/603982/why-does-my-openssh-key-fingerprint-not-match-the-aws-ec2-console-keypair-finger).  I tried matching it, because I thought it would be weird to have two fingerprints, hence the crazy openssl lines rather than just ssh-keygen -lf.  But AWS doesn't care if the fingerprint is wrong - it's just a name, and we just want some form of unique identifier.
I'll see if I can get the EC2-style fingerprint to work on my OS (10.9.5) - your rsa command is at least recognized, so it is definitely an improvement :-)  I'd rather use your version, but at the same time it's more important to just have something that works on most machines.  Better to explain to people why the fingerprints don't match than to have to deal with kube-up failing.

Here's the problem on Mavericks, from https://www.netmeister.org/blog/ssh2pkcs8.html
 However, it turns out that on OS X Mavericks, the OS provided version of ssh-keygen(1) lost the ability to handle the PKCS8 format:

$ ssh -V
OpenSSH_6.2p2, OSSLShim 0.9.8r 8 Dec 2011
$ ssh-keygen -e -m PKCS8 -f ~/.ssh/id_rsa.pub
-----BEGIN PUBLIC KEY-----
-----END PUBLIC KEY-----
PEM_write_RSA_PUBKEY failed
$

Writing a go helper to do the hashing would be attractive, but I don't think we have anywhere convenient to put it (especially this close to v1).  I think your version works on every supported OS except for Mavericks.
I'm going to LGTM this patch; it fixes the behavior on OSX except for Mavericks, which is better.
I don't know yet what to do about Mavericks.

@justinsb I have the same problem on OS X Yosemite (10.10.3). Actually I have installed a newer version of openssl from Homebrew:
brew update
brew install openssl
brew link openssl --force

openssl version
OpenSSL 1.0.2a 19 Mar 2015

This version of openssl support pkey command. But then I found Issue with ssh-keygen.

Finaly, I figure this out.
I have updated openssh and it works like a charm:
$ ssh -V
OpenSSH_6.8p1, OpenSSL 1.0.2c 12 Jun 2015
$ ssh-keygen -f ~/.ssh/id_rsa.pub -e -m PKCS8
-----BEGIN PUBLIC KEY-----
... removed content ...
-----END PUBLIC KEY-----

To update openssh using Homebrew:
brew install homebrew/dupes/openssh

@bronislav Interesting, unfortunately I'm not sure if we can rely on everyone having openssh installed through homebrew.
@justinsb Creating the fingerprint in go or some other lang would probably be the most portable solution. Could we use python? I know we use python elsewhere in kube-up, so it's not unreasonable to expect it to be there.

@iterion Sure. Pushing everyone to use homebrew is quite strange. It is works for me but may not work for someone else.


🐛 Minor issue: Zero/Dummy mysql date ie. '0000-00-00' in date column, running sqlmode="" (ie. not strict mode), is presented as -0001-11-30

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?
The date on the view is in the format:
eg.  ($client->getClient_birthdate())->format(....users...date...format....choice...eg. 'Y-m-d');
ie. DateTimeImmutable date with the format command.
Version
ORM 2.0.0
PHP 8.1.9

The date was presented as -0001-11-30 as a string

I can avoid this issue completely if I simply present the current date on the client form. In this way the user will have to edit the current date to their birthdate. This ensures that an official date is entered in the column.  Or I can do something like this in the view:
eg.
(($client->getClient_birthdate())->format($datehelper->style())) === '-0001-11-30' ? 'Please fill in your birthdate'
:
(($client->getClient_birthdate())->format($datehelper->style()))

This is the mySql documentation:
5.1.11 Server SQL Modes
https://dev.mysql.com/doc/refman/8.0/en/sql-mode.html
With strict mode disabled, invalid dates such as '2004-04-31' are converted to '0000-00-00' and a warning is generated. NO_ZERO_DATE The NO_ZERO_DATE mode affects whether the server permits '0000-00-00' as a valid date. If this mode is not enabled, ... The MySQL server can operate in different SQL modes, and can apply these modes differently for different clients, depending on the value of the sql_mode system ...
B.3.4.2 Problems Using DATE Columns
https://dev.mysql.com/doc/refman/8.0/en/using-date.html
The special “zero” date '0000-00-00' can be stored and retrieved as '0000-00-00'. When a '0000-00-00' date is used through Connector/ODBC, it is automatically converted to NULL because ODBC cannot handle that kind of date. MySQL permits you to ...You should use this format in UPDATE expressions and in the WHERE clause of SELECT ...

are you sure the problem is not about timezone?


Which version to install?

Hi,
Just so you know, there are three different versions of this that show up in Vue ui:

It's very confusing, especially since "vue-cli-plugin-antv" has a higher version number and appears to have updated/newer docs.
I'm not sure how a repo gets added to the search but if this repo is the correct version  can you remove the others from showing up in the search? Thanks!


:package: feat(schemas): Init schemas package

Description
Initialized schemas package. Schemas package is the source of truth for types in EVMts. It exports types, effect validators, Error types, and vanilla js validators for all types
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #506   👈

This comment was auto-generated by Graphite.


not playing station

Just hangs never plays
http://rogers-hls.leanstream.co:8000/rogers/edm1029.stream/48k/playlist.m3u8
CHDI "Sonic 102.9" Edmonton, AB

Try another streaming link.


WASM median cut

It works and it's incredibly fast. Would be nice to add a dropdown menu in the wasm color palette extraction demo, to select the algorithm from the three that are currently available, but I don't know how to do that in pure js, without altering the html page that all demos share.

ok, we will add in the future


Make DNSSD queries cancellable

Resolves https://github.com/apple/swift-async-dns-resolver/issues/32

@swift-server-bot add to allowlist

Hi, sorry for the wait, I updated the implementation to use DispatchReadSource to process the query results.
I converted the DNSSD struct to a class to use the deinit and deinitialize the pointers when we are done. The current implementation deinitialized the pointers at the end of the query function, which caused a crash when calling DNSServiceProcessResult from the event handler.
Let me know if converting it to a class is OK or if we should change it.
Other than that I only removed code that was no longer necessary.

Don't bother reviewing for now as the code is not correct, we only have one reply handler pointer so concurrent requests cause a crash. Trying to fix it.

I added a Query class to handle the pointer initialization and deallocation, now a Query instance is created for every query made and the deallocation happens in the deinit of each instance.
I have tested with many concurrent requests and it seems to be working correctly. This implementation uses DispatchSource, as stated above.
Let me know if there is any changes I should make!

Thanks @victorherrerod.
@weissi @Lukasa @ktoso Can you please review the latest changes when you have time? Thanks in advance.

Thanks everyone for the feedback, I followed @weissi advice and implemented a cancellation handler for the dispatch source that manages the ressources. I also deleted the Query class, which was kind of a hack to handle the pointers deallocation.
I have tested concurrent queries and everything seems to work correctly. Unit tests all passed too.
Let me know if there are other changes that have to be made.

Hi, could someone review the changes? We'd like to use this in a project to update our SRV resolver to use async/await, and being able to timeout the requests by cancelling them is necessary.
I've tried to answer and fix most of the issues, let me know if there is anything else that's needed.

Anything new for this PR? It would be nice to be able to use it in a project, and with the requests being uncancellable and the timeout being so long, it's kind of difficult to use.
I understand that it's still a pretty new lib that may not be prod-ready, so no pressure 🙌


Update @atproto/api and implementation

atproto as default export is deprecated in API. This MR updates the dependency in package.json and utilizes the newer preferred implementation in bot.ts.

Ooh, thanks. I hadn't kept up with changes in the library. Will take a look at this later.


[CP] [dart2wasm] Fix partial instantiation constants

Commit(s) to merge
f7fc4d037466a099000962f3f0eb20e01048e37c
Target
stable
Prepared changelist for beta/stable
https://dart-review.googlesource.com/c/sdk/+/380120
Issue Description
Using partial instantiation constants of generic methods with named parameters can in certain situations lead to compiler crashes.
What is the fix
Fix the code that handles partial instantiation constants in dart2wasm.
Why cherry-pick
Users have hit this problem.
Risk
If all tests pass: low.
Issue link(s)
https://github.com/dart-lang/sdk/issues/56372
Extra Info
No response

/cc @kevmoo for visibility (we want this to be also included in the dot release)

@athomas Could we get this approved so it gets included in the upcoming hotfix release?

@mkustermann based on previous discussions, I am going to mark this cherry-pick approved.

@mkustermann based on previous discussions, I am going to mark this cherry-pick approved.

Thanks a lot, @itsjustkevin. No more cherry-picks for dart2wasm in the SDK (still one dependent in flutter/flutter - https://github.com/flutter/flutter/pull/153310)


Set font size in the settings

Is your feature request related to a problem? Please describe.
Yes. Because the box already is a small portion of the screen, small font sizes make it very difficult to read for many people.
Describe the solution you'd like
I want to be able to set the font size in the settings.
Additional context
None.

Yes please

+1   this feature would be very helpful for me as well.
Thanks!


On a touch device, it is not possible to ungroup or group by drag'n'drop.

could you implement the posibillity to have checkboxes to group or ungroup?
Like it is in the songs app?

My standpoint is that I won't try to support touch devices in the desktop UI, but instead have a touch-centric interface that is more similar to the mobile version of the sonos app and try to make that responsive.
I started with that a long time ago (you can check it out on the /m url), but it is still lacking a lot of the features from the regular interface. With that said, having checkboxes (or some other way of grouping multiple players in one go) might still have a need, it would just not be because of touch support.
What kind of device are you trying to use for this?

I have not tried the touch support for the web controller yet (as I'm currently waiting for my screen). But I'm going to be trying to use the Official 7" Raspberry Pi screen with my Raspberry Pi 2 model B as both the host and interface of this web controller and I'd love to see some love for touch support.

Touch is supported in the /m layout. Issues should be filed against missing functionality in that layout instead.


Instantiating a Snippet from a partial xml code

Currently in order to create a Snippet from partial XML (i.e list items without the ul or ol wrappers) the TempladoSnippet can be used. However it requires that the user knows to wrap the partial in code in the Templado namespace:
<page:snippet xmlns:page="https://templado.io/snippets/1.0" xmlns="http://www.w3.org/1999/xhtml" id="whatever">
    ... partial code ...
</page:snippet>

A DOMNode must be created, the partial code must be wrapped, and the wrapped code then loaded into the DOMNode.
It would be a nice convenience if this process were moved under the hood. Perhaps a factory method could be added to the TempladoSnippet:
const TEMPLADO_NAMESPACE_OPENER =  '<page:snippet xmlns:page="https://templado.io/snippets/1.0" xmlns="http://www.w3.org/1999/xhtml" id="contentHeader">';
const TEMPLADO_NAMESPACE_CLOSER = '</page:snippet>';

public static function fromPartialCode(string $targetId, string $partialCodeString)
{
    $partialBlock = new \DOMDocument();
    $wrappedPartial = self::TEMPLADO_NAMESPACE_OPENER . $partialCodeString .  self::TEMPLADO_NAMESPACE_CLOSER;
    $partialBlock->loadXML($wrappedPartial);
    return new self($targetId, $partialBlock);
}

Or perhaps this code could be added to the Snippet loader.

While I like the idea, I'm not sure where this functionality belongs.
Adding it to the TempladoSnippet sounds like a good fit at first glance. But not having a general fromString feels incomplete. That though would lead to having a fromString method on TextSnippet and SimpleSnippet as well. That in turn would lead to duplicate code as at least TempladoSnippet and SimpleSnippet need to ensure the given string translates into valid (=parsable) XML.
Adding it to the SnippetLoader feels wrong as this is not actually "loading" anything, more like mapping.
So, does that mean we should add a SnippetMapper that can translate strings into their respective best matching type of snippet? Or more like explicit map calls?
Still pondering what is best here...

What about the idea of creating a simple wrapper class (i.e. Templado\Engine\SnippetPartial) that is instantiated with a string and internally wraps it with the namespace. Then TempladoSnippet takes that as a dependency.

The concept of "snippets" got removed from Templado 5. Everything is a document now.
That doesn't exactly solve the problem of this issue though. Still pondering whether we should have a wrapper here..


Support English

Basically the application supports i18n to a certain extent. The whole Language/Label infrastructure is prepared to support multiple languages.
The report generation could be tricky (either set labels dynamically in the code, or create separate/duplicate JRXML files for each language).

NOPE


[Errno 2] No such file or directory: '/tmp/salt_dashboard.tmp'

hi,
we installed saltshaker with following commands:
echo deb http://ppa.launchpad.net/saltstack/salt/ubuntu `lsb_release -sc` main | sudo tee /etc/apt/sources.list.d/saltstack.list

wget -q -O- "http://keyserver.ubuntu.com:11371/pks/lookup?op=get&search=0x4759FA960E27C0A6" | sudo apt-key add –

apt-get update

apt-get install salt-master salt-minion salt-syndic salt-api python-pip python-cherrypy3 mysql-server python-mysqldb

useradd -g admin admin && passwd admin
vim /etc/salt/master.d/saltapi.conf
     rest_cherrypy:
        port: 8000
        host: <IP_ADDRESS>
        disable_ssl: true
     external_auth:
        pam:
          admin:
            - .*
            - '@runner'
            - '@wheel'

/etc/init.d/salt-master restart

/etc/init.d/salt-api restart

netstat -tnlp | grep 8000

cd /opt
pip install Django==1.8.4
git clone https://github.com/yueyongyue/saltshaker.git
cd /opt/saltshaker
vi saltshaker/settings.py
mysql -u root –p

In mysql-shell:
show databases;
create database saltshaker;
show databases;
create user saltshaker@localhost;
set password for saltshaker@localhost=password('saltshaker');
grant all on saltshaker.* to saltshaker@localhost;

mysql -u saltshaker –p
show databases;
use saltshaker;
quit

pip install django-crontab
python manage.py migrate
python manage.py runserver <IP_ADDRESS>:9000&
python manage.py createsuperuser

by using the web-gui we can logg in but than following mistake returns:
IOError at /

[Errno 2] No such file or directory: '/tmp/salt_dashboard.tmp'

Request Method: 	GET
Request URL: 	http://<IP_ADDRESS>
Django Version: 	1.8.4
Exception Type: 	IOError
Exception Value:

[Errno 2] No such file or directory: '/tmp/salt_dashboard.tmp'

Exception Location: 	/opt/saltshaker/dashboard/views.py in index, line 12
Python Executable: 	/usr/bin/python
Python Version: 	2.7.6
Python Path:

['/opt/saltshaker',
 '/usr/lib/python2.7',
 '/usr/lib/python2.7/plat-x86_64-linux-gnu',
 '/usr/lib/python2.7/lib-tk',
 '/usr/lib/python2.7/lib-old',
 '/usr/lib/python2.7/lib-dynload',
 '/usr/local/lib/python2.7/dist-packages',
 '/usr/lib/python2.7/dist-packages']

Server time: 	Wed, 3 Feb 2016 12:45:29 +0100`
```

when we touch the missing file manually, following error message returns:
```
`IndexError at /

list index out of range

Request Method: 	GET
Request URL: 	http://<IP_ADDRESS>/
Django Version: 	1.8.4
Exception Type: 	IndexError
Exception Value:

list index out of range

Exception Location: 	/opt/saltshaker/dashboard/views.py in index, line 14
Python Executable: 	/usr/bin/python
Python Version: 	2.7.6
Python Path:

['/opt/saltshaker',
 '/usr/lib/python2.7',
 '/usr/lib/python2.7/plat-x86_64-linux-gnu',
 '/usr/lib/python2.7/lib-tk',
 '/usr/lib/python2.7/lib-old',
 '/usr/lib/python2.7/lib-dynload',
 '/usr/local/lib/python2.7/dist-packages',
 '/usr/lib/python2.7/dist-packages']

Server time: 	Wed, 3 Feb 2016 12:29:48 +0100`
```

What´s wrong ?

Best regards

hi
django-crontab
CRONJOBS = (
('*/1 * * * *', 'shaker.cron.dashboard_scheduled_job'),
)
django-crontab will create /tmp/salt_dashboard.tmp file every minute.

We add the cronjob with your python comman but the file is not created.
If we execute the command in the crontab we get an error 401:
`# /usr/bin/python /opt/saltshaker/manage.py crontab run a69abdead75461c29e5d444fd9d79a91
Traceback (most recent call last):
  File "/opt/saltshaker/manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python2.7/dist-packages/django/core/management/__init__.py", line 338, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python2.7/dist-packages/django/core/management/__init__.py", line 330, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python2.7/dist-packages/django/core/management/base.py", line 393, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python2.7/dist-packages/django/core/management/base.py", line 444, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python2.7/dist-packages/django_crontab/management/commands/crontab.py", line 29, in handle
    Crontab().run_job(options['jobhash'])
  File "/usr/local/lib/python2.7/dist-packages/django_crontab/crontab.py", line 142, in run_job
    module = import_module(module_path)
  File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "/opt/saltshaker/shaker/cron.py", line 44, in <module>
    dashboard = dashboard_scheduled_job()
  File "/opt/saltshaker/shaker/cron.py", line 7, in dashboard_scheduled_job
    status = sapi.runner_status('status')
  File "/opt/saltshaker/shaker/shaker_core.py", line 151, in runner_status
    self.token_id()
  File "/opt/saltshaker/shaker/shaker_core.py", line 22, in token_id
    content = self.postRequest(obj, prefix='/login')
  File "/opt/saltshaker/shaker/shaker_core.py", line 31, in postRequest
    opener = urllib2.urlopen(req)
  File "/usr/lib/python2.7/urllib2.py", line 127, in urlopen
    return _opener.open(url, data, timeout)
  File "/usr/lib/python2.7/urllib2.py", line 410, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 523, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 448, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 531, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
urllib2.HTTPError: HTTP Error 401: Unauthorized
`

hi !
HTTP Error 401: Unauthorized
You can check salt-api authentication.
/etc/salt/master.d/saltapi.conf ,There are strict format requirements.
see  https://github.com/yueyongyue/saltshaker/blob/master/saltapi.conf

This is how it looks like:
rest_cherrypy:
  port: 8000
  host: <IP_ADDRESS>
  disable_ssl: true
external_auth:
  pam:
    admin:
        - .*
        - '@runner'
        - '@wheel'

it doesn't work.

user name has to be saltapi:
saltapi@salt-master:~$ cat /etc/salt/master.d/saltapi.conf
rest_cherrypy:
  port: 8000
  host: <IP_ADDRESS>
  disable_ssl: true
external_auth:
  pam:
    saltapi:
        - .*
        - '@runner'
        - '@wheel'


Allows default values for select and multiselect elements

Close #1047

Notes on implementation: default values for select elements match on value if it exists, or if the options array is a simple array of strings, it matches on the string. If a single select has multiple matching options, than the last one that matches will be selected.
On multiselect elements, you can define value as an array of strings, or a single string. Either way, this value gets normalized to an array and matching is done by checking whether the option's value is in the array using Array.indexOf.
One thing to note about defining multiselect defaults is that, even if the multiselect element is alterable (ie you can define new options) defining a default value that isn't included in the options will not cause that value to appear or be selected.
This is the most straightforward way to do it, and seems to cover the most cases.

Fantastic work, I agree with your implementation notes. I'll do a code review.

I just set the value field on my site's multiselect and this works perfectly, thank you! This has been my biggest pet peeve about metadata.


Private Menu component code optimization

Changes

After PR TODO from #1942.
TL;DR;

Remove redundant keyboard navigation code from Menu in favor of the code from useListNavigation.
Menu handles lots of boiler plate code to set up a Menu. This reduces the amount of code in the components that use Menu.
Focus no longer moves from the trigger to the first focusable item upon clicking the trigger with a mouse. This only happens when the trigger is activated using the keyboard's Enter or Space keys.
Menu no longer focuses the selected item upon opening.

Since #1942 introduces Floating UI's useListNavigation in DropdownMenu that handles keyboard navigation, the keyboard navigation code in Menu becomes redundant. This PR extends the use of Floating UI's useListNavigation to other uses of Menu too (ComboBox, SplitButton, Select) and completely removes the Menu's keyboard navigation code.
Additionally, Menu now handles a lot of the boilerplate thus significantly reducing the code in the components that use Menu. The boiletplate code that Menu now handles (not exhaustive):

the portaling: offers the optional portal prop for more customization
conditional rendering based on the popover's open state
spreading the popover props (getFloatingProps, getReferenceProps, getItemProps)
setting the refs: offers the optionalpositionReference prop to set the position reference (instead of doing ref={popover.refs.setPositionReference})
keyboard navigation: offers the interactions.listNavigation prop for more customization
registering a FloatingNode in the FloatingTree if an ancestral FloatingTree is found
focus management:

focuses items on hover.
if not in a FloatingTree, focus moves to the trigger when the menu is closed. If in a FloatingTree, focus does not move back to the trigger since MenuItems handle the focus.

setting aria-expanded accordingly depending on the menu open state.
When the Menu is within a FloatingTree, if a submenu has focus, the hover interaction is automatically disabled. This helps to keep the last hovered/focused submenu open even upon hovering out.

Some non-user facing changes:

Floating UI's useListNavigation boilerplate setup (e.g. listRef,activeIndex, etc.) is handled by Menu.
Menu's keyboard navigation code is removed in favor of FloatingUI's useListNavigation that handles keyboard navigation, among other things.
usePopover also returns interactionsState.listNavigation. This contains activeIndex and onNavigate that can be used to get or set the activeIndex from outside usePopover.
Since setFocus was set to false in all uses of Menu, the setFocus prop is no longer necessary and this was removed. In the future, if we want to set the focus on a particular item, we can use activeIndex in useListNavigation.
Menu adds new props (trigger, positionReference, portal) to help handle the boilerplate aspects of the code itself.

Since components that use Menu don't call usePopover, they can use Menu's new popoverProps to customize the default props passed to usePopover.

FloatingUI's useClick requires two clicks in some instances to close the submenu on a MenuItem. https://github.com/iTwin/iTwinUI/pull/1942#discussion_r1576373640 addressed this by manually implementing the onClick functionality instead of relying on FloatingUI. But manually handling it got complicated in this PR since the trigger can be a button or a div and pressing Enter on a button by default triggers onClick, which is not something div does. Thus, I reverted to using FloatingUI's useClick click that handles different types of triggers and wrote some code in usePopover to overcome the "two clicks" issue.

Some user facing behavior changes:

Earlier, clicking on the trigger would move the focus to the first focusable item. Now, the focus stays on the trigger. Only activating the trigger with the Enter or Space keys will move the focus to the first focusable item. This is the default behavior of FloatingUI (focusItemOnOpen). I believe this behavior is better since it seems incorrect imo for the focus to jump away from the trigger without any mouse movement or opening of a modal.
Earlier, MenuItem's tabIndex used to depend on the disabled and role. Now, the default depends on isSelected. But if the MenuItem is in a Menu, this default value is replaced with a roving tabIndex (https://github.com/iTwin/iTwinUI/pull/2021#discussion_r1640163588).

After #2034 and #2036, Menu was no longer used in Select and ComboBox. As a result, some Menu features are no longer being used and thus were removed:

Menu no longer focuses the selected item upon opening

Unrelated:

I noticed that the types of DropdownMenu were slightly contradictory. DropdownMenuProps included component props for a ul and DropdownMenu included component props for a div. This PR fixes it to only include component props for a div. This fixes some of the type errors when working on this PR.

Self TODOs

[ ] Undo all changes, make a commit, redo all the changes, then make another commit. This is because I temporarily committed with --no-verify.

Testing

Moved the failing Menu unit tests to be e2e tests instead.
Removed outdated tests (mostly unit tests and one image test).

Replaced failing Menu unit tests with e2e tests.

Approved failing test images

Docs

Added JSDocs appropriately
Added a changeset for the public facing change

After PR TODOs:

[ ] (near future) See why FloatingUI's focusItemsOnHover is not working. After that, we can remove manually focusing the item in onMouseEnter (https://github.com/iTwin/iTwinUI/pull/2021#discussion_r1617671460).
[ ] Consider deprecating MenuItem (https://github.com/iTwin/iTwinUI/pull/2021#discussion_r1619449538)

Over the last few commits since May 20 (https://github.com/iTwin/iTwinUI/pull/2021/files/ed8d1fdc8cc4934d7092708787a3c58f859ec400..7986e482c24fa68ad05fd8556aec52fda487c53e), I have been fixing the last remaining bugs/regressions, adding/fixing the tests, and also addressing the remaining GitHub comments. I updated the PR description to reflect the current state of the PR.
There was a small regression that happened a few hours ago after dae20ab (#2021)/https://github.com/iTwin/iTwinUI/pull/2021#discussion_r1609930747. I am currently trying to fix it. But apart from that, the PR is ready for review.

Optional: @Ben-Pusey-Bentley Since some parts of this PR have changed since your review, you could optionally take a quick look at the PR in case you have any comments.

Looks good to me. Tested the new behaviors with the DropdownMenu, SplitButton, and Table column manager, and everything is worked as expected based on the PR description.


Task 08 starter code missing clippy directive

Task 08 is missing the clippy directive to #[allow(clippy::vec_init_then_push)] above main, leading to linter errors with a correct solution.

I believe this was closed by #22 .


Generate Elixir log filters from deriving register_event

If the Elixir log filters are receiving log info from Coda, they should match on the log output generated by deriving register_event. The Elixir filter code could be generated by the deriver.

We can use the output from the existing structured events dump to create log filters.


Allow registering tool lints

Existing lints work as before. Example use:
dylint_linting::declare_late_lint! {
    /// …
    pub dylint::QUESTION_MARK_IN_EXPRESSION,
    Warn,
    "`?` operators embedded within an expression"
}

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

Thanks very much for your PR.
I am okay with adding this functionality, and I like the idea of having one set of macros that handle both tool and non-tool lints, but I don't know how I feel about this:
https://github.com/trailofbits/dylint/blob/6853c35630431c90984565c165b43de3f93cf820/utils/linting/src/lib.rs#L175
Specifically, introducing special syntax ([@tool]) into a macro a user is expected to use.
What I am about to propose is not ideal, but I think I would prefer have two (nearly identical) alternatives within the macro definition, similar to what you did for _declare_and_register_lint.
Also, we will definitely want a test for this new functionality, e.g., something like:

create a new Dylint template in a temporary directory
use dylint_internal::find_and_replace to add a tool name
verify that the modified code builds and the UI tests pass

The boundary_toolchains test could probably be used as a starting point.

@bb010g Do you think you might continue to work on this?

@bb010g I'm going to close this for bookkeeping purposes. But I would be happy to revisit this, especially if you have a need for it.


android splash screen image is low resolution

On Pixel 6 Pro running Android 13

never seen this before, interesting


Pip integration

It would be convenient to provide CMake for Pytest the same way most people will install pytest: with pip.
pip install pytest pytest-cmake

CMake does not provide an extensive search procedure for modules, so this will only work if the CMake file is provided as a configuration instead (https://cmake.org/cmake/help/latest/command/find_package.html)
By doing so, we can use the data_files to install the generated PytestConfig.cmake and PytestVersionConfig.cmake along with the other Cmake files in a location that CMake will search via the system environment path when find_package(Pytest) is invoked:
<PREFIX>/share/Pytest/cmake

The PytestConfig.cmake file will simply include the FindConfig.cmake module file:
include(${CMAKE_CURRENT_LIST_DIR}/FindPytest.cmake)

The PytestVersionConfig.cmake file will be generated by CMake using the write_basic_package_version_file function. The version will be given via the pytest module fetched via the build system table. CMake will also be fetched that way to ensure that the version config can be generated.
Question:

Is it safe to include a CMake module file via a config file? As far as I can tell, the find_package_handle_standard_args function will not mess up with the config. But it might be preferable to escape it when generating the config.

I opened a discussion thread of the CMake repo:
https://gitlab.kitware.com/cmake/cmake/-/issues/24171
And the conclusion seems to be that this method is unconventional but makes sense in this scenario.


standardize use of AES-RSA hybrid encryption

Whats the context for this change -- why back out the byte-size decision in this code?

@diranged See discussion and links at the bottom of this PR: https://github.com/Nextdoor/zuul_alpha/pull/7
Let me know your thoughts.


[4.x] - Refactor to Tracing providers

Resolves #6181

fyi brave 6 will be out soon (removing deprecation), and sorry about the spam. I noticed refactoring notices here to a new api. when that's together, if still using brave.. let's get a canonical frontend/.backend example here. This will help make sure people can test it perf and dependency wise, or just to have fun. If not interested, excuse the spam https://github.com/openzipkin/brave-example


[release-1.21] Fix eds cluster loadassignment with invalid endpoint

Please provide a description of this PR:
fixes: #50718
Manual pick: #50690

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.21
b410f74d4317839c5c6f83974b680611589030c0
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.21
b410f74d4317839c5c6f83974b680611589030c0
link
true
/test unit-tests-arm64

unit-tests_istio_release-1.21
b410f74d4317839c5c6f83974b680611589030c0
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.