Allow HTML in the copyright message

It would be nice to allow HTML in the copyright so that you can link to a website, like so:
copyright = "Jared Dillard"

I don't know if this creates a security risk by using safe, but if it does I think you'd have to try and shoot yourself in the foot. I'm also not sure if docs of some kind should be added to point out that use can use HTML (carefully).

The bigger problem with doing this is that... well... this is not what other Sphinx themes do and people use name <email> style fairly often.


Expensify shows that I paid someone but the payment failed

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @saracouto
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1702072130270899
Action Performed:

Receive a IOU request from Account B to Account A
Pay the request using  "Pay with expensify"
Open the details

Expected Result:
There should be no error as it was a successful payment
Actual Result:
Unexpected error please try again later message appears
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub

Getting clarification on platforms impacted via Slack

@Beamanator any thoughts on this one? I see it referenced in https://github.com/Expensify/Expensify/issues/335366 - looks like the wallet is 'Pending' which is throwing the error, even though the payment was successful?

Yeah that seems to be the problem, though I haven't looked into why this is happening, @robertjchen not sure if you've looked into this since this seems to be between you & sara?

@Beamanator lemme know if I should assign someone else or if you can take a look at this one

Sorry I'm hoping to have more free time to look into this next week when I'm back in cairo after the holidays - if this currently only affects sara, i think we can lower the priority maybe? Unless it's a huge paid and delaying her a ton from important things

I think that it's fair to lower the priority on this

Not overdue, low priority.

@Beamanator what do you think here? Since there have been no other reports, should we close this?

@bfitzexpensify I was just thinking that, yeah I like that plan

Cool, let's close this out and re-investigate if we get more reports.


When using aiDeepRequired inside anyOf , a Segment ... does not exist error is thrown

When using aiDeepRequired inside anyOf , a Segment ... does not exist error is thrown
Does not depend on the type of object to which you specify the path

please resubmit with full code sample, answering the questions in template


[Feat]: Add Loading Animation to Page

What feature?
As a user visiting the website, I often encounter situations where I need to wait for content to load, especially when navigating between different pages or performing resource-intensive actions. Currently, the absence of a loading animation creates a somewhat jarring experience, as I'm unsure whether the page is still loading or if there is an issue.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Please show us the design and animation for the loading.

You can choose from here:https://cssloaders.github.io/ or any custome one you want,i can integrate any loading animation

@PiyushKalyanpy ,please assign me this issue

@PiyushKalyanpy ,which animation do you want or i should add from my choice?

add any of your choice.


doc(skeleton-typescript-aspnetcore): update readme to include getting type definition files

Since the packages no longer include the type definition files, the initial project setup needs to include a step for getting the type definition files.

I included a change to the documentation in PR#553 that covers this PR. I documented the need for the typing tool to be installed, I had already included a post-install step in package.json to run the typings tool.

New skeletons are in master with updated implementations and readmes. Thanks for all the help. Even though we weren't able to merge this exact PR, it guided us as we were making fixes.


fix(jest): Allow being used inside Jest beforeAll

Fix #72

Ah, and I no longer have access to publish this package (it used to be under Codecademy) -- will work on that behind the scenes with @jakemhiller.

Update: this should be published as<EMAIL_ADDRESS>Thanks @jakemhiller!


initial confirmed stable release of stiebel-isg

initial and confirmed stable relase of the stiebel-isg adapter for stiebel-eltron/tecalor internet service gateway. (heatpumps)
forum-thread:
https://forum.iobroker.net/viewtopic.php?f=24&t=14309

will repair the conflicts


i18n(zh-cn): Update content.mdx

Description (required)
Update content.mdx
Related issues & labels (optional)

#10046
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

zh-cn/community-resources/content.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


No Response for Xiaomi Air Purifier 4 Compact

Describe the bug
Xiaomi Air Purifier 4 Compact (zhimi.airp.cpa4) is showing No Response all the time. I just bought the air purifier but the Filter Condition is already "Change Soon" and Filter Life is always 0.
Expected behavior
It should have on/off button and fan level. The air quality status should be showing Excellent/Good/etc instead of "Unknown".
Your config.json
                {
                    "name": "Xiaomi Smart Air Purifier 4 Compact",
                    "ip": "<IP_ADDRESS>",
                    "token": "d30023e2376894bd6de5b8394a90ea89",
                    "deviceId": "533733331",
                    "model": "zhimi.airp.cpa4",
                    "pollingInterval": 10,
                    "deepDebugLog": true,
                    "buzzerControl": false,
                    "ledControl": false,
                    "childLockControl": false,
                    "modeControl": false
                }

Debug log
[12/09/2022, 22:32:48] [miot] [Xiaomi Smart Air Purifier 4 Compact] Starting property polling.
[12/09/2022, 22:33:40] [miot] [Xiaomi Smart Air Purifier 4 Compact] Poll failed 4 times in a row! Stopping polling and trying to reconnect! Reason: Error: Call to device timed out
[12/09/2022, 22:34:41] [miot] [Xiaomi Smart Air Purifier 4 Compact] Device found: zhimi.airp.cpa4
[12/09/2022, 22:34:41] [miot] [Xiaomi Smart Air Purifier 4 Compact] Setting up miot device!
[12/09/2022, 22:34:41] [miot] [Xiaomi Smart Air Purifier 4 Compact] Device setup finished! Miot device ready!
[12/09/2022, 22:34:41] [miot] [Xiaomi Smart Air Purifier 4 Compact] Connected to device: zhimi.airp.cpa4
[12/09/2022, 22:34:41] [miot] [Xiaomi Smart Air Purifier 4 Compact] Doing initial property fetch.
[12/09/2022, 22:34:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Starting property polling.
[12/09/2022, 22:35:35] [miot] [Xiaomi Smart Air Purifier 4 Compact] Poll failed 4 times in a row! Stopping polling and trying to reconnect! Reason: Error: Call to device timed out

Screenshots

Additional context
Even after I disable the buttons below, they are still being shown in the main buttons UI area.
"buzzerControl": false,
"ledControl": false,
"childLockControl": false,
"modeControl": false

Call to device timed out more or less means that the plugin cannot connect to your device. This might have a couple of reason, the most likely are that either the ip address or the token are incorrect.

I added the device through MiCloud. All other devices (Mi Standing Fans, Xiaomi Air Purifier 3H, Mi Robot Vacuum-Mop 2) have been working perfectly, except Xiaomi Air Purifier 4 Compact.

The token or ip might be wrong in your MiCloud account. Also some users experienced issues when the smart devices where in a separate IoT network and could not be reached.

The token or ip might be wrong in your MiCloud account. Also some users experienced issues when the smart devices where in a separate IoT network and could not be reached.

I got the IP from router and the token through Android device. Call to device timed out error does not appear now, but the air purifier is still showing "No Response".
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Model known: zhimi.airp.cpa4!
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing device!
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing device services
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Device services: [
  "air-purifier",
  "environment",
  "filter",
  "alarm",
  "physical-controls-locked",
  "screen",
  "custom-service",
  "aqi"
]
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing device properties
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Device properties: [
  "air-purifier:on",
  "air-purifier:fault",
  "air-purifier:mode",
  "environment:pm2.5-density",
  "filter:filter-life-level",
  "filter:filter-used-time",
  "filter:filter-left-time",
  "alarm:alarm",
  "physical-controls-locked:physical-controls-locked",
  "screen:brightness",
  "custom-service:moto-speed-rpm",
  "custom-service:country-code",
  "custom-service:favorite-level",
  "custom-service:filter-used-time-dbg",
  "aqi:aqi-updata-heartbeat"
]
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing device actions
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Device actions: [
  "air-purifier:toggle",
  "filter:reset-filter-life",
  "custom-service:toggle-mode"
]
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Successfully created a AirPurifier device! It is a Xiaomi Air Purifier 4 Compact.
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing accessory!
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Accessory successfully initialized!
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Registering 1 accessories!
[13/09/2022, 01:40:53] [miot] [Xiaomi Smart Air Purifier 4 Compact] Everything looks good! Initiating property polling!

I do not see any error in the log that you posted, that looks perfectly fine. You would need to post a debug log, ideally as described here: https://github.com/merdok/homebridge-miot#debug

I do not see any error in the log that you posted, that looks perfectly fine. You would need to post a debug log, ideally as described here: https://github.com/merdok/homebridge-miot#debug

Debug log
[13/09/2022, 08:19:32] [miot] [Xiaomi Smart Air Purifier 4 Compact] No cached device info found!
[13/09/2022, 08:19:32] [miot] [Xiaomi Smart Air Purifier 4 Compact] Model known: zhimi.airp.cpa4!
[13/09/2022, 08:19:32] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing device!
[13/09/2022, 08:19:32] [miot] [Xiaomi Smart Air Purifier 4 Compact] Creating device instance by model: zhimi.airp.cpa4!
[13/09/2022, 08:19:33] Error: listen EADDRINUSE: address already in use :::51809
    at Server.setupListenHandle [as _listen2] (node:net:1372:16)
    at listenInCluster (node:net:1420:12)
    at Server.listen (node:net:1508:7)
    at EventedHTTPServer.listen (/var/lib/homebridge/node_modules/homebridge/node_modules/hap-nodejs/src/lib/util/eventedhttp.ts:155:20)
    at HAPServer.listen (/var/lib/homebridge/node_modules/homebridge/node_modules/hap-nodejs/src/lib/HAPServer.ts:332:21)
    at Bridge.<anonymous> (/var/lib/homebridge/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Accessory.ts:1279:18)
    at step (/var/lib/homebridge/node_modules/homebridge/node_modules/tslib/tslib.js:144:27)
    at Object.next (/var/lib/homebridge/node_modules/homebridge/node_modules/tslib/tslib.js:125:57)
    at fulfilled (/var/lib/homebridge/node_modules/homebridge/node_modules/tslib/tslib.js:115:62)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Not a airconditioner device! Device class could not be found: ../modules/airconditioner/devices/zhimi.airp.cpa4.js
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Not a airer device! Device class could not be found: ../modules/airer/devices/zhimi.airp.cpa4.js
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Not a airfryer device! Device class could not be found: ../modules/airfryer/devices/zhimi.airp.cpa4.js
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Not a airmonitor device! Device class could not be found: ../modules/airmonitor/devices/zhimi.airp.cpa4.js
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] It is a airpurifier device! Found device class at ../modules/airpurifier/devices/zhimi.airp.cpa4.js!
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing device services
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Device services: [
  "air-purifier",
  "environment",
  "filter",
  "alarm",
  "physical-controls-locked",
  "screen",
  "custom-service",
  "aqi"
]
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing device properties
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Device properties: [
  "air-purifier:on",
  "air-purifier:fault",
  "air-purifier:mode",
  "environment:pm2.5-density",
  "filter:filter-life-level",
  "filter:filter-used-time",
  "filter:filter-left-time",
  "alarm:alarm",
  "physical-controls-locked:physical-controls-locked",
  "screen:brightness",
  "custom-service:moto-speed-rpm",
  "custom-service:country-code",
  "custom-service:favorite-level",
  "custom-service:filter-used-time-dbg",
  "aqi:aqi-updata-heartbeat"
]
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing device actions
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Device actions: [
  "air-purifier:toggle",
  "filter:reset-filter-life",
  "custom-service:toggle-mode"
]
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing device events
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Device events: [
  "custom-service:fault-motor-stuck",
  "custom-service:childlock-trigger",
  "custom-service:filter-exhausted"
]
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initial properties to monitor: [
  "air-purifier:on",
  "air-purifier:mode",
  "air-purifier:fault",
  "screen:brightness",
  "environment:pm2.5-density",
  "filter:filter-life-level",
  "filter:filter-used-time",
  "filter:filter-left-time",
  "alarm:alarm",
  "physical-controls-locked:physical-controls-locked"
]
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Successfully created a AirPurifier device! It is a Xiaomi Air Purifier 4 Compact.
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Found cached accessory for this device! Unregistering it first!
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Initializing accessory!
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Creating AirPurifier accessory for device Xiaomi Smart Air Purifier 4 Compact!
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Found accessory class at ../modules/airpurifier/AirPurifierAccessory.js!
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Accessory successfully initialized!
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Final properties to monitor: [
  "air-purifier:on",
  "air-purifier:mode",
  "air-purifier:fault",
  "screen:brightness",
  "environment:pm2.5-density",
  "filter:filter-life-level",
  "filter:filter-used-time",
  "filter:filter-left-time",
  "alarm:alarm",
  "physical-controls-locked:physical-controls-locked"
]
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Registering 1 accessories!
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] Everything looks good! Initiating property polling!
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] (Protocol) Start handshake <IP_ADDRESS>
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] (Protocol) Server listening <IP_ADDRESS>:33190
[13/09/2022, 08:19:33] Got SIGTERM, shutting down Homebridge...
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] (Protocol) <IP_ADDRESS> -> Handshake reply
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] (Protocol) Call <IP_ADDRESS>: miIO.info - {"timeout":5000,"retries":3} - {}
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] (Protocol) Start handshake <IP_ADDRESS>
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] (Protocol) <IP_ADDRESS> <- (2) {"method":"miIO.info","params":{"timeout":5000,"retries":3},"id":1}
[13/09/2022, 08:19:33] [miot] [Xiaomi Smart Air Purifier 4 Compact] (Protocol) <IP_ADDRESS> <- !1p��,$4L�;�I��RE��x

When I run miot test <ip> -t <token>
(Protocol) Call <IP_ADDRESS>: miIO.info - {"retries":3,"timeout":5000} - {}
(Protocol) <IP_ADDRESS> <- Reached maximum number of retries, giving up miIO.info - {"retries":3,"timeout":5000}
Call to device timed out
 ERROR  Could not connect to the device at <IP_ADDRESS>! Make sure the ip and token are correct!

It seems that the either one of the IP and token are incorrect, but I got them from local device, not MiCloud.

Yeah, you need to make sure that miot test will succeed. Maybe it has something to do with your network setup?

Should be fixed in version 1.3.1


Feature / Custom helpers

Implement feature recorded on the issue #8

Pull Request Test Coverage Report for Build 8

23 of 23 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.03%) to 94.167%

Totals

Change from base Build 6:
2.03%

Covered Lines:
90

Relevant Lines:
90

💛  - Coveralls


Is this page still up to date ?

Is this page still up to date ?

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

ID: afda862d-5084-45c0-43d6-fa1cf6723e2a
Version Independent ID: 12cb954d-fd9d-0877-6437-31bdd292abb2
Content: Differences between global Azure, Azure Stack Hub, Azure Stack HCI - Azure Stack Hub
Content Source: azure-stack/operator/compare-azure-azure-stack.md
Service: azure-stack
GitHub Login: @Justinha
Microsoft Alias: justinha

@Ilkoboy This describes the high level differences between the 3 products, and it is still up to date.
Even though the date lists as last May, there have been 10 updates to this page in the past month.
Did you have a specific question about what is discussed in this doc?

@Ilkoboy We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Fix macOS linking with ARM CPU

Change is related to the default installation path for homebrew on ARM macOS. See homebrew documentation

confirmed to be working


Add collecting metrics for inverted index too

labels for inverted indexes are {db="",collection="",shard="",index=""}
Scope & Purpose
(Please describe the changes in this PR for reviewers, motivation, rationale - mandatory)

[ ] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.10: (Please link PR)
[ ] Backport for 3.9: (Please link PR)
[ ] Backport for 3.8: (Please link PR)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket:
[ ] Design document:

some tests?


Laravel 11.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 11.x.
Before merging, you need to:

Checkout the l11-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: Shift detected a dependency for the laravel/framework. Unless you have a special requirement for depending on the entire Laravel framework, it's recommended to add dependencies for the specific illuminate packages instead.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-notification-subscriptions.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "liran-co/laravel-notification-subscriptions": "dev-l11-compatibility",
    }
}

Finally, run: composer update

Can you please merge this?

Thank you !!


nixos/stage1-init: Add multi-device btrfs wait (closes #32588)

Motivation for this change
As noted in #32588, there is a race condition due to the btrfs device scan in the stage-1 init script is sometimes running before all the devices are available because udevadm settle does not guarantee this (whether by error or design I do not know) as the stage-1 init writer expected.
There is a btrfs device ready command that

wait[s] until all devices of a multiple-device filesystem are scanned and registered within the kernel module ... to provide a way for automatic filesystem mounting tools to wait before the mount.

Presumably using this is the correct thing to do.
Things done
I build this on my machine and rebooted. It came up okay. I don't know for sure though if this is correct way to use this, nor, as it is a race condition, if it actually solved it or the timing just worked out okay.
I am also not 100% clear on how btrfs device scan interplays with btrfs device ready. From the man page, I get the feeling that we should maybe drop the btrfs device scan call (which this patch does not do) in favour of just using btrfs device ready.

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

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

[X] Fits CONTRIBUTING.md.

@edolstra @florianjacob @shlevy @bluescreen303 @nckx @aanderse CCing the various people who have made modifications to the btrfs module for any comments/thoughts they might have.

Still important to me.

Do you think we can add a NixOS test with a BTRFS RAID1?

Should we still do this?

NixOS 23.11 supports using a systemd based initrd which also seems to support populating /etc/udev/rules.d via services.udev.packages . btrfs-progs contains udev rules that do the same thing as this PR but integrates nicely with udev. I think we should be using boot.initrd.supportedFilesystems to populated services.udev.packages with btrfs-progs when btrfs is enabled in the initrd.
That said for non-systemd initrd's we should still do something; either add the udev rules verbatim or merge this PR. I think having a VM test would be very useful; in my testing I'm still having trouble getting multi-device btrfs systems to mount consistently in stage-1. That said I have not experienced this issue much outside of Qemu VM tests (but I've also switched to using a systemd initrd so perhaps that avoids this race condition).
Similar issue in dracut based initrd; https://github.com/dracutdevs/dracut/issues/947


Client Certificate ABAC

We can configure Helidon 4 to require client certificate authentication with the following
server:
  port: 8443
  host: <IP_ADDRESS>
  tls:
    client-auth: REQUIRED
    endpoint-identification-algorithm: NONE # This is a bit counter-intuitive
    private-key:
      keystore:
        ...
    trust:
      keystore:
        ...
        trust-store: true

Ignoring the need to set endpoint-identification-algorithm: NONE which is not very intuitive, this works well.
However, coming from a Servlet background I would then expect to be able to do the following

register Certificate DN to Role mappings
use those Roles in the ABAC authoriser

There is no way to achieve this with Helidon.
Environment Details

4.0.0
Helidon SE
JDK version: Zulu21.30+15-CA (build 21.0.1+12-LTS)
OS: MacOS 14.0

Problem Description
I would like to be able to implement security authorisations using ABAC against clients authenticated using client certificate authentication.
Given Helidon has ABAC the Certificate properties could be exposed to ABDC for dynamic checking against attributes of the Certificate.
In order to get this working in my test lab I have had to do the following

Register a io.helidon.webserver.http.Filter to add the certificate to the request

    @Override
    public void filter(
            final FilterChain chain,
            final RoutingRequest req,
            final RoutingResponse res) {
        final var context = req.context();
        context.get(SecurityContext.class).ifPresent(sec -> {
            final var envBuilder = sec.env().derive();
            req.remotePeer().tlsCertificates().ifPresent(certs -> envBuilder.addAttribute(X509_CERTIFICATE, certs));
            sec.env(envBuilder);
        });
        chain.proceed();
    }

This would seem to be a core feature missing from io.helidon.webserver.security.SecurityContextFilter

Implement an io.helidon.security.spi.AuthenticationProvider

This takes the code from io.helidon.security.providers.httpauth.HttpBasicAuthProvider to configure SecureUserStore instances but then looks up the user by the certificate DN
    @Override
    public AuthenticationResponse authenticate(final ProviderRequest providerRequest) {
        final var foundUser = X509SecurtyFilter.certificates(providerRequest.env())
                .stream()
                .flatMap(Arrays::stream)
                .flatMap(cert -> findUser(cert).stream().map(user -> new SimpleImmutableEntry<>(user, cert)))
                .findFirst();

        return foundUser.map(e -> {
            if (subjectType == SubjectType.USER) {
                return AuthenticationResponse.success(buildSubject(e.getKey(), e.getValue()));
            }
            return AuthenticationResponse.successService(buildSubject(e.getKey(), e.getValue()));
        }).orElseGet(this::invalidUser);
    }

    private Optional<SecureUserStore.User> findUser(final Certificate certificate) {
        if(!(certificate instanceof X509Certificate)) {
            return Optional.empty();
        }
        final var dn = ((X509Certificate) certificate).getSubjectX500Principal();
        return userStores.stream()
                .flatMap(userStore -> userStore.user(dn.getName()).stream())
                .findFirst();
    }

    private Subject buildSubject(final SecureUserStore.User user, final Certificate certificate) {
        Subject.Builder builder = Subject.builder()
                .principal(Principal.builder()
                        .name(user.login())
                        .build())
                .addPublicCredential(
                        X509Credentials.class,
                        new X509Credentials(user.login(), certificate));
        user.roles()
                .forEach(role -> builder.addGrant(Role.create(role)));
        return builder.build();
    }

    private static final class X509Credentials {
        private final String username;
        private final Certificate certificate;

        private X509Credentials(String username, Certificate certificate) {
            this.username = username;
            this.certificate = certificate;
        }
    }

Hello
the endpoint-identification-algorithm: NONE  should not be required at all if you have correctly defined certificates. It is what is called "endpoint verification" in other places, and it is related to the common name of the server certificate - if that is set correctly to the host name used, you do not need to specify this option.
Regarding the feature request, we will investigate it. Right now you can only use the common name from certificate to assert user identity through our header based security provider (we use X_HELIDON_CN as the generated header for common name from client certificate)

@tomas-langer I understand that - but what endpoint-identification-algorithm does when enabled for client certificate authentication on the server side is that it tries to identify the client by its SAN. As a client certificate will identify a user and not a machine, and moreover the user's machine could have any IP address and likely not a domain name, this verification will always fail.
As an example, my WAN ip address might now be <IP_ADDRESS> - I have no reverse DNS for that IP as it's given to me by my ISP by DHCP. I make a request with my certificate for DN=CN=boris.morris provided for me by some public CA which issues client certificates. The server, if endpoint verification is enabled, will try and verify that there is an IP SAN for <IP_ADDRESS> in my certificate - which there obviously can never be.
Endpoint identification is used when the client talks to a server - which will have some fixed address(es) that can be named in a SAN.
For the server side of a mutual TLS connection, endpoint verification is normally disabled for this reason.


Added very basic vs2015 support

I've noticed a few issues with this that need to be fixed up before merging, this module has a dependency on another module that happened to be loaded when I was generating the projects (oops). I'll also split this out into multiple files, vsandroid_vstudio.lua, vsandroid_sln2005.lua and vsandroid_androidproj.lua. This should make it consistent with premake-core, as well as making it easier to extend and understand.

@TurkeyMan I've split this up into multiple files and fixed a couple of issues with it. Let me know if there's any issues.

I finally tried it out, works as advertised :)
I'm thinking the old 3rd party solution stuff can safely be stripped out now MS have committed to supporting it themselves.

The MS implementation only works in VS2015+, whereas the vs-android implementation doesn't work with VS2015+, just VS2010-2013. I think it still has a purpose, but I guess it depends on how much use it gets.

True, but I'm thinking it's safe to say that anyone doing Android dev with VS will have updated no?
Safe to say that it'll never be tested by either of us...


Update the update yaml script

Move update_yaml.py script into main cli()
Add unit tests

Making this a draft, because for some reason click can't find the update_yaml command

TODO:

per convo with @caufieldjh, fix bug when updating multiple nested keys - solution for now is to support specifying nested keys like this --keys graph_data:graph:node_path --values my/new/node/path

Adding - @caufieldjh I've made a unit test here for this - unskip this once you've implemented the nested key thing above

TODO:

per convo with @caufieldjh, fix bug when updating multiple nested keys - solution for now is to support specifying nested keys like this --keys graph_data:graph:node_path --values my/new/node/path

Done - keys now need to be specified like key1:key2:key3, but a list of different keys can still be provided, e.g. key1:key2:key3,key4:key5,key6


Added 'hello-world' quickstart for SDKs

Description
Added "Hello World" quickstart for SDK
Issue reference
We strive to have all PRs being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ x] The quickstart code compiles correctly
[ x] You've tested new builds of the quickstart if you changed quickstart code
[ x] You've updated the quickstart's README if necessary

@artursouza @yaron2 plz take a look

We have decided to keep SDK specific "quickstarts" in the corresponding SDK repo. Please, move this to the go-sdk repo.
@orizohar @AaronCrawfis thoughts?

When discussed with @wcs1only and @halspang it seemed there is a need to simplify the quickstarts - to use SDKs instead of Dapr API. I also share this idea. I think we should have a single and simple quickstarts repository.
In this PR I provide list of SDKs where we have more advanced examples.

We have decided to keep SDK specific "quickstarts" in the corresponding SDK repo. Please, move this to the go-sdk repo.
@orizohar @AaronCrawfis thoughts?

When discussed with @wcs1only and @halspang it seemed there is a need to simplify the quickstarts - to use SDKs instead of Dapr API. I also share this idea. I think we should have a simple gateway quickstarts repository, to expedite user's adoption.
In this PR I also provide a full list of language-specific SDKs where we have more advanced examples.

Changing the quickstarts from using APIs to SDKs must be agreed within the maintainers since that is a change in direction we have decided until now.
@AaronCrawfis @orizohar @yaron2  Thoughts?

We have decided to keep SDK specific "quickstarts" in the corresponding SDK repo. Please, move this to the go-sdk repo.
@orizohar @AaronCrawfis thoughts?

When discussed with @wcs1only and @halspang it seemed there is a need to simplify the quickstarts - to use SDKs instead of Dapr API. I also share this idea. I think we should have a simple gateway quickstarts repository, to expedite user's adoption.
In this PR I also provide a full list of language-specific SDKs where we have more advanced examples.

Changing the quickstarts from using APIs to SDKs must be agreed within the maintainers since that is a change in direction we have decided until now.
@AaronCrawfis @orizohar @yaron2 @msfussell Thoughts?

We should definitely not replace the HTTP examples with SDK examples.
The HTTP APIs are the means by which Dapr guarantees language agnosticism and they are consistent across languages.
SDK quickstarts can be added to complement these.
Something like
hello-world
hello-world-go-sdk
etc.

My main argument for having basic SDK quickstarts in a single repository is that having them in multiple repositories is simply confusing.
This dapr/quickstarts repo is a logical place for all types of quickstarts. We still can have more advanced examples in the language-specific SDK repos.
I agree with @yaron2.  I would also suggest making one step further and renaming hello-world into hello-world-api.

My main argument for having basic SDK quickstarts in a single repository is that having them in multiple repositories is simply confusing.
This dapr/quickstarts repo is a logical place for all types of quickstarts. We still can have more advanced examples in the language-specific SDK repos.
I agree with @yaron2. I would also suggest making one step further and renaming hello-world into hello-world-api.

I would not rename the hello world sample since it is already linked from multiple places.
Regarding having the quickstarts reproduced per SDK in this repo, it would require to have all of them done in a single change (IMO). Having them partially implemented will be more confusing since it would give the impression that some SDKs don't exist or are partially implemented. I think this effort is not small enough to just be closed with this PR and would need to have an issue created and prioritized within the other work items we have.

Regarding having the quickstarts reproduced per SDK in this repo, it would require to have all of them done in a single change (IMO). Having them partially implemented will be more confusing since it would give the impression that some SDKs don't exist or are partially implemented. I think this effort is not small enough to just be closed with this PR and would need to have an issue created and prioritized within the other work items we have.

I personally don't see any issue with gradually adding hello-world-sdk quickstarts. IMO, use of SDK will significantly simplify and expedite Dapr adoption. We can start with one SDK quickstart, and add more later.
Regarding "having them partially implemented will be more confusing since it would give the impression that some SDKs don't exist or are partially implemented", how do the developers know which language is supported today? They should somehow find the right SDK repository. If we have a centralized quickstart repo (which I'm advocating for), we can provide list and links to all SDK repos. I've already done that in this PR, but we can relocate that info where it will be more visible.

I agree that SDKs simplify Dapr adoption, the issue is that SDK maintainers/approvers need to be better at creating quickstart samples in  their repo  and most have not done this enough today. The approach here is that each SDK has its own quickstarts. The discoverability issue is easily solved in docs, where you can then navigate either from here https://docs.dapr.io/ or https://docs.dapr.io/getting-started/ for the respective SDK. This cannot be done until there are better samples in the SDK.  Given there are 7 SDK (5 crafted SDKs) we should certainly not push these all into one repo. IMO this particular example is best moved into a the /examples folder in the GO SDK repo although we could also consider a quickstart repo in the Go SDK, which has a tutorial like walkthrough.  This is also agreeing the quickstart content.

Personally, I totally agree.  As much as everybody wants to be inclusive across a host of languages, or to hand craft an HTTP message each time a remote request is needed, I'm NOT interested!
Give me a client side SDK that simplifies the transfer, I've simply got to much to do to learn about a 100 different choices, from a 100 different languages....
Rant over...  :)
Get Outlook for Androidhttps://aka.ms/ghei36

From: Mark Fussell @.>
Sent: Sunday, June 13, 2021 9:38:17 PM
To: dapr/quickstarts @.>
Cc: Subscribed @.***>
Subject: Re: [dapr/quickstarts] Added 'hello-world' quickstart for SDKs (#442)
I agree that SDKs simplify Dapr adoption, the issue is that SDK maintainers/approvers need to be better at creating quickstart samples in their repo and most have not done this enough today. The approach here is that each SDK has its own quickstarts. The discoverability issue is easily solved in docs, where you can then navigate either from here https://docs.dapr.io/ or https://docs.dapr.io/getting-started/ for the respective SDK. This cannot be done until there are better samples in the SDK. Given there are 7 SDK (5 crafted SDKs) we should certainly not push these all into one repo. IMO this particular example is best moved into a the /examples folder in the GO SDK repo although we could also consider a quickstart repo in the Go SDK, which has a tutorial like walkthrough. This is also agreeing the quickstart content.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/dapr/quickstarts/pull/442#issuecomment-860356480, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEVJVNS3676O3OY2XNCHULTSWBTTANCNFSM46PRVXXA.

@msfussel, I think we all agree that SDK is the quickest way to implement Dapr application. Not having SDK example in dapr/quickstarts defies the purpose of this repo. Also, as the name suggests, it is not a quickstarts for API only, it is a quickstarts for entire Dapr.
I'm not suggesting to move all quickstarts from the language-specific SDK repos into this repository, but having some basic SDK examples, and referring to the SDK repos for more advanced cases.

@msfussel, I think we all agree that SDK is the quickest way to implement Dapr application. Not having SDK example in dapr/quickstarts defies the purpose of this repo. Also, as the name suggests, it is not a quickstarts for API only, it is a quickstarts for entire Dapr.
I'm not suggesting to move all quickstarts from the language-specific SDK repos into this repository, but having some basic SDK examples, and referring to the SDK repos for more advanced cases.
I also think that having SDK examples in different languages in this repo will serve as another proof of Dapr adoption and support within open source community.

We try to keep the quickstart repo general so it is relevant to developers using any language and at the same time small so it can be maintained and verified on each release. If we add SDKs we will need to add several quickstarts and at that point it will be grow too large in my opinion. I think this is a great addition that belongs in the go-sdk Dapr repo and I strongly disagree that not having SDK examples here "defies the purpose if this repo".
I think the maintainers can reopen this decision of having SDK specific examples in the SDK specific repos but it requires more discussion.

Let's agree to disagree :)  Closing this PR.

Thanks @dmitsh for the contribution, I think having a go quickstart that mirrors the HTTP quickstart is a great idea.
I think that in a perfect implementation we have the quickstarts implementations side by side in the docs.dapr.io website, with a tabbed UI for code, showing that HTTP and SDKs are co-equal and users can choose between them.
For now though, I think we can PR this into the go SDK, and once the docs/quickstarts maintainers have time to standardize the quickstarts experience we can make it unified.

Happy to agree – but I do like learning from history.  WCF was an awesome tech, but it was trying to be too much for too many people…
It feels that DAPR is also doing the same thing in terms of trying to be too much too many people.
My own personnel opinion is that .Net would be the implementation of choice, but that seems to be only one of many that is being focused on.
One of the reasons why I’m still waiting to introduce it to my clients – as IMO – DAPR has a bit of too many things to get good at any one thing.
But as you say – we can agree to disagree
From: Dmitry Shmulevich @.>
Sent: June 14, 2021 9:51 AM
To: dapr/quickstarts @.>
Cc: Richard Reukema @.>; Comment @.>
Subject: Re: [dapr/quickstarts] Added 'hello-world' quickstart for SDKs (#442)
Let's agree to disagree :) Closing this PR.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/dapr/quickstarts/pull/442#issuecomment-860834569, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEVJVL724FI5FIZ432644DTSYXN7ANCNFSM46PRVXXA.


Fix UG errors

Codecov Report

Merging #207 (a30beac) into master (2b35a75) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #207   +/-   ##
=========================================
  Coverage     80.33%   80.33%
  Complexity      981      981
=========================================
  Files           146      146
  Lines          2828     2828
  Branches        332      332
=========================================
  Hits           2272     2272
  Misses          478      478
  Partials         78       78

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 2b35a75...a30beac. Read the comment docs.


Browser doesn't re-size if window's horizontal size changes

As the title says: if one resizes the Panda3D window horizontally, the CEF browser simply stretches along instead of re-sizing to the new resolution.
I've currently fixed this by changing...

if self._cef_texture.get_x_size() != width AND self._cef_texture.get_y_size() != height:

to

if self._cef_texture.get_x_size() != width OR self._cef_texture.get_y_size() != height:

... in CEFPanda._set_browser_size, on my local version of cefpanda.
I considered posting this as a pull request, but since I don't know for sure how you intended that method to operate, I'll leave it to you.

You are correct that this should be an "or" and not an "and"


CR and DR are removed from basic block

@pgoodman  In the commit #276, you have removed DR and CR from basic block.
Now, how we can lifting instructions accessing these registers? Because this check always fails.

This check should always pass: https://github.com/trailofbits/remill/commit/71b24d6cf031ba98bf0225aee278cb2f0dee7240#diff-cc2c1f23701c1aa7eb1c9bf42cb78cd1R147
If it's not, then try adding [[gnu::used]] to the renamed variables and see if that helps. If it does, please submit a PR.

Hi @pgoodman
Unfortunately adding [[gnu::used]] doesn't work for me :(
I believe that the extern global variable will be stripped off from the module if it is not referenced anywhere. A very simple example is that:
__attribute__((__used__)) extern int a;

int func() {
	return 1;
}

then
clang -c -emit-llvm extern.c -o extern.bc
extern.c:1:16: warning: 'used' attribute ignored [-Wignored-attributes]
__attribute__((__used__)) extern int a;

and we can see that there is no exernal global for a
; ModuleID = 'extern.bc'
source_filename = "extern.c"
target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
target triple = "x86_64-unknown-linux-gnu"

; Function Attrs: noinline nounwind optnone uwtable
define i32 @func() #0 {
  ret i32 1
}

attributes #0 = { noinline nounwind optnone uwtable "correctly-rounded-divide-sqrt-fp-math"="false" "disable-tail-calls"="false" "less-precise-fpmad"="false" "no-frame-pointer-elim"="true" "no-frame-pointer-elim-non-leaf" "no-infs-fp-math"="false" "no-jump-tables"="false" "no-nans-fp-math"="false" "no-signed-zeros-fp-math"="false" "no-trapping-math"="false" "stack-protector-buffer-size"="8" "target-cpu"="x86-64" "target-features"="+fxsr,+mmx,+sse,+sse2,+x87" "unsafe-fp-math"="false" "use-soft-float"="false" }

!llvm.module.flags = !{!0}
!llvm.ident = !{!1}

!0 = !{i32 1, !"wchar_size", i32 4}
!1 = !{!"clang version 5.0.2 (tags/RELEASE_502/final)"}

What happens if you do:
(void) CR0;
(void) CR1;
...

inside of __remill_basic_block?

See #286

Another approach #287

Merged your PR.


Low coverage site marked on variant page

Similar to when sites are filtered can we mark sites that have low coverage, either by using AN or median coverage.

Fixed in 9f6daf0


Upgrade to Bazel 4.x

We should upgrade to the latest Bazel 4.x release. This involves futzing with RBE configs as well

Likely also requires https://github.com/bazelbuild/bazel-toolchains/issues/970 to be fixed


Neater hashing interface

Please read docs/source/yosys_internals/hashing.rst for what the interface is now.
We want to be able to plug-and-play hash functions to improve hashlib structure collision rates and hashing speed. Currently, in some ways, hashes are incorrectly handled: for deep structure hashing, each substructure constructs a hash from scratch, and these hashes are then combined with addition, XOR, sometimes xorshifted to make this less iffy, but overall, this is risky, as it may degrade various hash functions to varying degrees. It seems to me that the correct combination of hashes is to have a hash state that is mutated with each datum hashed in sequence. That's what this PR does.

[x] check performance impact since this PR isn't NFC as it changes how structures are hashed
[x] clean up things left over from experiments with inheriting from Hashable
[x] fix pyosys
[x] finish the plugin compatibility part of the doc

Cool, I got hit in the face with a downright gcc bug

Passes like extract_fa and opt_dff take 5-10% longer with this change. This is definitely a problem

Leaving a note so I don't forget it: We should provide a way for plugins to be compatible with both pre and post this change. I am thinking a define to advertise the new API.

Thanks @jix and @povik for your reviews of the large amount of code change. I don't think I've left anything unaddressed. Since we're nearing the end of the release cycle, we'll have to target the next one

This change hid hash_ptr_ops from the Yosys namespace causing an avoidable API breakage. See https://github.com/povik/yosys-slang/commit/a2e91d4ffa969a1c1908dbc7fc84bfe64ca00edf. We can walk that back before the release.

See povik/yosys-slang@a2e91d4.

This patch isn’t enough to be compatible post the API change. I am not sure how to declare the dicts in a way which works on both sides of the API change without resorting to preprocessor logic.


Add system CTO for contracts in new cicero-common package

Coverage remained the same at 89.499% when pulling 7e766d2f41601c36bd8fd18d9fa1f193617e1652 on jeromesimeon:contracts into d56d2fcab295cd3c4fc29f10eaf95a0b49e5e393 on accordproject:master.


[bug] UNKNOWN units in device METER CC

Version
HA OS 5.12 on RPi4 4G

Describe the bug
UNKNOWN entries in Plug device METER section.
Initially no one, but now 9.
To Reproduce
When device paired - no UNKNOWN entries.
After several Core & Host restart - appears.
Expected behavior
Only entries actually exposed by device.
Additional context

node_2.zip

Ok, closing for now. If it reappears and you happen to catch it in a log, feel free to reopen.

Sure. Not appeared anymore.


[WIP] Add documentation for widgets, common patterns and examples

Todo:

[x] Fix autodoc, since the current RTD it is broken.
[x] Add autodoc pages for all of the base widgets and link to them from a table
[ ] Autogenerate 'supported platforms' table using inspect module, since this will change a lot

Work in progress, request merging early for comment

FYI, I'd like to start putting in docstrings, I couldn't see which style you preferred so I'll raise a seperate issue.

@freakboy3742 applied changes in https://github.com/pybee/toga/pull/98/commits/23646be6be88d55d0fc17f6be9ca23fb48453f17

Looks good - do you want to keep this PR open while you work on the last point on your TODO, or are you happy for this to be merged?

@freakboy3742 let's do it in a separate PR. The idea is:
iterate through each of the platforms (we need an enum type for this?)
import the directory
check that each of the widgets exist in each package
Collect that and represent it as an RST table of supported platforms and widgets.
Happy to merge this when you're ready


Ec2

@bors-servo r+

:pushpin: Commit f4ab5fa has been approved by Ms2ger

:hourglass: Testing commit f4ab5fa with merge 16f5c71...

:sunny: Test successful - travis


Implement inactivity reminders

Current coverage is 98.32% (diff: 100%)

Merging #492 into master will increase coverage by 0.03%

@@             master       #492   diff @@
==========================================
  Files            38         37     -1
  Lines           935        956    +21
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            919        940    +21
  Misses           16         16
  Partials          0          0

Powered by Codecov. Last update 9684e52...2be2a90

I don't see anything else that needs addressing.  I intend to merge this into master soon.  Any last comments?


Removed entries in RMapCache Through Eviction Process Still able to see through Redis-cli

I am Using the RMapCache<K,V> and using the Method  putIfAbsent(K key, V value, long ttl, TimeUnit ttlunit) .
I want to use the Eviction functionality i.e provided so I am giving ttl as 45 Sec.
After 45 Sec when i check the Entry in the MapCache From the ReddisonClient at runtime i didnt found the entry
But when i check the same in the Redis-cli using HGETALL i can still able to found this Entry that was Deleted through Eviction Process by Redisson
SomeTimes even after 5 min i can able to see the Entry in the Redis-cli even Though it can't be able to find from the Redissonclient after 45 Secs

Hi, removing of stale RMapCache entries is not executed in realtime. That's why you able to see stale value through redis-cli

Hi ,
Could you Please Eloberate the statement
removing of stale RMapCache entries is not executed in realtime

Hi,
When checking for the above issue I found a similar issue was raised in the below link
https://stackoverflow.com/questions/46824834/expired-redisson-keys-still-visible-in-redis-cli/46838604#46838604
where @Redisson_RuiGu Mentioning that
Redis does not support individual element eviction out of the box for hashes. So Redisson has built its own solution and named it MapCache.
the hash in the redis-cli eventually will disappear when the Redisson expiration process kicks in.
so Is this Issue because of delay in Redisson expiration process kicks in
Here Redisson expiration process kicks in process means Removing of stale RMapCache entries could you please confirm on this

I confirm that Redisson cleans stale entries of RMapCache and there are no Redis process to handle that

Thank You For Confirming. I am closing this Issue


Google Chrome not found in Arch Linux

I installed google-chrome from Arch Linux AUR, which uses the binary name 'google-chrome-stable', but web-component-tester doesn't look for it.
My workaround was creating a symbolic link:
sudo ln -s /usr/bin/google-chrome-stable /usr/bin/google-chrome

To avoid modifying your system installation, you can instead tell launchpad where to find chrome via the dedicated environment variable:
export LAUNCHPAD_CHROME=/usr/bin/google-chrome-stable

This can be done for other browsers as well. More information on https://github.com/ekryski/launchpad.

:+1:


test case for Rails 6.1 issue when queries contain an IN clause

Issue #121
In Rails 6.1, queries which contain an IN clause seem to get simple-valued bind parameters, which breaks the LogSubscriber render_bind_vxxx method.
Description of changes
I have added a test case to reproduce this issue, but not identified a fix yet
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thank you for the test, it was a great help to isolate the issue.

Thank you for the test, it was a great help to isolate the issue.

Thank you for the test, it was a great help to isolate the issue.

No problem, thank you for all your work on this gem! :1st_place_medal:

Thank you for the test, it was a great help to isolate the issue.

No problem, thank you for all your work on this gem! :1st_place_medal:


Unable to cross-compile for android

I'm building an android application that requires support for the socket.io protocol to connect to a server, however, this requires cross-compilation and this crate requires openssl-sys
I have already tried many alternatives, like using Cross, or even bare docker images with qemu, but none have worked so far
Host: x86_64-unknown-linux-gnu (Ubuntu)
Target: aarch64-linux-android
#366 might be related, and maybe could fix this, but it seems to have no progress so far
additionally, could this be reasonably easy to implement?

Mhm... openssl is an inherent problem of such crates and it is not super easy to get rid of it IMO. Feel free to fork this repo and try to get rid of the openssl dependency though. If you have questions, feel free to ask them as part of this issue.


Longest Palindromic Substring in C++

This issue is about [Algo/DS Name/Question Name](link to resource for the Algo/DS/Question)

[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate

[x] Issue Title is in form [Algo/DS Name] in [Language] (ex:- BubbleSort in Java)

[x] I will send my PR only after I'm assigned to this issue by any maintainer

[x] Assign this issue to me (I want to work on this)

[x] I ensure that I am not already assigned to 2 or more issues

Please find PR -> https://github.com/SobhanBera/algorithms/pull/48


show uid_map in podman info

show uid_map in podman info
close #4067
Signed-off-by: Qi Wang<EMAIL_ADDRESS>

https://github.com/containers/libpod/issues/4067#issuecomment-533550391
I also added the log for newuidmap/newgidmap, when will the changes happen to newuidmap/newgidmap?
@mheon @giuseppe @rhatdan  PTAL

How does the output look when you run podman info?  Are the line separations ok?

How does the output look when you run podman info? Are the line separations ok?

It directly prints out the IDMap struct
$ podman info
host:
  ...
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
  MemFree:<PHONE_NUMBER>
  ...

The output looks good, but I'm concerned that if there are several hundred or more mappings on a system, then podman info will have an enormous amount of output.  Could we create a '--quiet'  option for podman info that would not show all of this, or at that time perhaps just show a count for the number of uid-map entries found?  Maybe instead have a --verbose option that would only show this info at that time?  @mheon WDYT?

Code LGTM other than my voluminous output concern.

@TomSweeneyRedHat I think you are creating a solution to a problem that does not exist.  I believe you used to be able to specify only 5 ranges, maybe you can specify more now.  But bottom line, unless someone is trying to cause problems, we should never see more then 5 lines
/lgtm
/approve


Only watch specific configmaps to reduce traffic

I see it's WIP, but I've ran some testing of it locally by adding extra logging to OnChange.
First, OnChange is not firing up on Config Map changes we shouldn't be monitoring which is great.
However, after creating and modifying both k3s-etcd-snapshots and rke2-etcd-snapshots Config Maps it looks like only changes to one trigger OnChange. And in my limited testing one time it was only k3s-etcd-snapshots that was triggering OnChange and in the other testing it was only rke2-etcd-snapshots. I'm not sure why it's happening, but it is happening.

Since we're filtering on subscribe, it's probably a good idea to remove check in OnChange (if configMap.Namespace != "kube-system" || !snapshotNames[configMap.Name]).

@snasovich Sorry, I accidentally merged this PR without the follow-ups to you comments.  Those fixes are in https://github.com/rancher/rancher/pull/33577 now.

@snasovich Sorry, I accidentally merged this PR without the follow-ups to you comments. Those fixes are in #33577 now.

@ibuildthecloud , thank you. Just for my understanding, was the fix to behavior I observed in not seeing OnChange triggered by one or the other config map to start controller factories sequentially vs. via goroutine?
Also, any specific reason check if configMap.Namespace != "kube-system" || !snapshotNames[configMap.Name] was left in OnChange code (my last comment)?


feat: dropdown new version

📑 Description
Dropdowns re-written:

introduced the concept of header and footer slots
changed to use rounded instead of rounded-lg - this is a major inconsistency from Flowbite.com
DropdownItems implement as active element <a/> or <button/> to allow catching events
custom dropdown items achieved by using <li/> directly

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed
[x] My pull request is based on the latest commit (not the npm version).

ℹ Additional Information
API changes introduced.

Dropdown navbar in a mobile size
Before opening:

After opening:

In the before opening, the logo and the hamburger menu is too close. When you open the menu, the logo position changes.

This in not a drop-down bug. It's the implementation of navbar.

From #23 of validator
Error: Attribute href not allowed on element div at this point.
From line 414, column 5; to line 414, column 297


More robust test for 'dot' command

The current RoadRunner.draw code has the following test for the dot command from graphviz:
if not 'PATH' in os.environ or any([ os.access( os.path.join( p, 'dot' ), os.X_OK ) for p in os.environ['PATH'].split( os.pathsep )])

I get a key error for 'PATH' when I run this from the Tellurium notebook viewer, presumably because Python gets executed as a kernel and doesn't inherit the environment variables of the calling process (Jupyter may be different). These are the environment variables I have access to in the kernel:
>>> print(os.environ)
environ({'IPYTHONDIR': '/home/poltergeist/.config/Tellurium/telocal/ipython', 'TERM': 'xterm-color', 'CLICOLOR': '1', 'PAGER': 'cat', 'GIT_PAGER': 'cat', 'MPLBACKEND': 'module://ipykernel.pylab.backend_inline'})

However, I can use the shutil.which command in Python 3:
>>> import shutil
>>> shutil.which('dot')
'/bin/dot'

A distutils-based solution also fails due to the lack of the PATH env var:
import distutils
distutils.spawn.find_executable('dot')
<traceback>
KeyError: 'PATH'

Therefore, I'm going to use the shutil.which solution where available (Python 3.3+), and it looks like I have to remove the check in Python 2. Not ideal, but it looks like there isn't a robust way to check for the presence of a command in Python 2.

Ended up putting it back because not having a PATH also prevents pygraphviz from working.


Allow for custom load balancer options to be passed to baragon

Baragon exposes an nginxExtraConfigs option which seems to be ignored when a singularity deploy is created.
From https://github.com/HubSpot/Baragon/blob/master/docs/baragon_api_docs.md

{
"loadBalancerRequestId":"requestId", # Unique request id
"loadBalancerService":{
"serviceId":"testService",
<EMAIL_ADDRESS>"serviceBasePath":"/basepath",
"loadBalancerGroups":["vagrant"],
"options":{
"nginxExtraConfigs": [
"rewrite ^/test(.*) /test/v1/$1 last;"
]
}
},
"addUpstreams":[
{"upstream":"example.com:80","requestId":"requestId","rack":"us_east_1a"}
],
"removeUpstreams":[]
}

It would be great if we could somehow specify a hash in a Deploy that will get passed to Baragon.

@edhurtig , objects added to the loadBalancerOptions field of the deploy will show up in the options hash on the Baragon request

Thanks @ssalinas! I will check that out. Closing for now

Worked like a charm

:smile:


Not able to create a new project

Hi! Thanks for this tool but when I run npx new-tailwind-app my-tw-app --basic it gives me an error:

How should I fix this? I have installed it globally.

@Kvaibhav01 have you tried changing the name of your folder when you are running the CLI? It says a folder named my-tw-app already exists in the directory where you are trying to create this project.

I have created a project with name my-tw-app. You can take a look at the screenshot.

Now if I create another project with the same name my-tw-app, it will give you an error that the folder already exists.

Yes, I know but here's how it happens:

I navigate to Web Projects folder: cd Web\ Projects
I run npx new-tailwind-app hello  --basic
It says:

✖ Couldn't create basic Tailwind app.

✖  hello  directory already exists. Try changing the name.

I see there is a new folder created under /Web Projects/hello. But nothing else.

Then I do something else like:

Navigate to this new folder: cd hello.
Run npx new-tailwind-app  --basic.
It asks me for the project name in kebab case so I enter: "hy-lo".
But still, I get this:

And yes, if we go to the "hello" folder, an empty folder called "hey-lo" is created but nothing apart from this:

There must be some bug in the code. I will take a look at it. Thank you pointing this out.


ENH: Supporting REPEATED schema for list types

Basically, at the moment if there are lists inside a column, we have no way to put that data in GBQ. This enables REPEAT mode in a simple way.

Codecov Report

Merging #60 into master will decrease coverage by 45.12%.
The diff coverage is 75%.

@@             Coverage Diff             @@
##           master      #60       +/-   ##
===========================================
- Coverage   73.44%   28.32%   -45.13%
===========================================
  Files           4        4
  Lines        1540     1543        +3
===========================================
- Hits         1131      437      -694
- Misses        409     1106      +697

Impacted Files
Coverage Δ

pandas_gbq/gbq.py
19.62% <75%> (-55.71%)
:arrow_down:

pandas_gbq/tests/test_gbq.py
27.94% <0%> (-54.94%)
:arrow_down:

Continue to review full report at Codecov.

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

tests!

I don't get what if failing, the test is there but doesn't look like it's being executed.... 🤔

FWIW it's not 'pandantic' to put lists in a dataframe, and IIRC if they're on an index you'll face issues with a lot of pandas functionality
We approach this with either:
a) separate dataframes uploaded to BQ, and then run a BQ query to merge them into a nested format in a single table
b) stack / unstack, with a BQ query to merge columns

@MaximilianR I guessed it is not, but is something we use, thankfully not too often, but still need.

Should we put a warning or something to warn about a list schema being generated?
Can you tell me why the test isn't being executed?

What makes you think it's not being executed?

Thanks @txomon! I think #25 may help us provide support for BigQuery repeated fields. I'd like to keep this PR open just in case #25 doesn't provide this functionality.
Regarding the integration tests being skipped, some integration tests will be skipped if a BigQuery project id is not set. I think the tests should fail if a project id is not set. I've created #72 to track this improvement.
Follow these steps to run the BigQuery integration tests on Travis:
https://pandas-gbq.readthedocs.io/en/latest/contributing.html#running-google-bigquery-integration-tests

Yeah, #25 may help. This is also something to make sure we handle when we switch from streaming writes to bulk upload https://github.com/pydata/pandas-gbq/issues/7.

Closing as stale, but open to a solution for writing repeated fields


[Autocomplete] Add test cases for createFilterOptions

[x] I have followed (at least) the PR section of the contributing guide.

@netochaves Thanks


Cursorline in mini.hues is not highlighted properly in few lines.

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.hues
Description
cursorline in mini.hues is not highlighted properly in few lines in mini.hues.
In some places it looks normal like below

While in others it does not show properly

Neovim version
nightly latest
Steps to reproduce
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
	spec = {
		-- add any other plugins here
		"nvim-treesitter/nvim-treesitter",
		"echasnovski/mini.hues",
	},
})

require("nvim-treesitter.configs").setup({
	ensure_installed = { "svelte", "bash" },
	highlight = {
		enable = true,
	},
})

vim.cmd("set cursorline")
vim.cmd.colorscheme("default")
vim.cmd.colorscheme("randomhue")

With the above config open the file below with name *.svelte

{#if name && description}
  <Accordion {name} {url}>

      {#each badges as badge}
        {badge}
      {/each}

        {description}

  </Accordion>
{/if}

Expected behavior
Cursorline should look uniform across all lines.
Actual behavior
Cursorline is not uniform across all lines.

This is not an issue with 'mini.hues' (or any color scheme, for that matter). This is the effect of tree-sitter highlighting: as it is done with extmarks, if highlight group defines background color, it is used on top of the cursorline. You can check which highlight group overrides cursorline by executing :Inspect in the target range.

This is not an issue with 'mini.hues' (or any color scheme, for that matter). This is the effect of tree-sitter highlighting: as it is done with extmarks, if highlight group defines background color, it is used on top of the cursorline. You can check which highlight group overrides cursorline by executing :Inspect in the target range.

But with tokyonight colorscheme, the same code works fine.

Running :Inspect at the position of the cursor in screenshot shows the following:
Treesitter
  - @none.svelte links to Normal svelte

As it links to Normal highlight group, it defines background which overrides cursorline (as this is how extmarks with background work).
This comes from 'nvim-treesitter/nvim-treesitter' and which has the following its 'CONTRIBUTING.md': @none    ; completely disable the highlight. The 'mini.hues' module interprets "completely disable" as "disable both foreground and background" and does so by linking to Normal.
This particular use of @none comes from 'svelte' queries and seems intentional.
'folke/tokyonight.nvim' has different behavior because it ignores the @none capture completely, which doesn't align with documentation in 'nvim-treesitter/nvim-treesitter'.

If you don't like this behavior, you can follow the Tokyonight route and add vim.api.nvim_set_hl(0, '@none', {}) after enabling 'mini.hues' based color scheme. A more elaborate approach is to define @none to only have foreground from Normal, but that is left as an exercise in how to use vim.api.nvim_get_hl().


Avoid negative hmtx values when creating font from variable CFF2 font

Fix #2821.

Fix looks arbitrary though. This is speced nowhere.

well, hmtx advance can't be negative, they are encoded as unsigned 16-bit integer, no?

well, hmtx advance can't be negative, they are encoded as unsigned 16-bit integer, no?

Correct. Though applications might interpret them otherwise. I'd say this is a case of a bad font. The fact that the advance is encoded as unsigned in hmtx is orthogonal to whether the combined advance has to be nonnegative.

We have other places in the format also that the variations can take a value outside the limits of its original value.

FWIW we also do a similar thing for truetype-flavored VFs in the varLib.instancer, see
https://github.com/fonttools/fonttools/blob/a2eef358be401cfa1c783f24f48a000424457152/Lib/fontTools/ttLib/tables/_g_l_y_f.py#L435-L438

Okay I see; I forgot we are talking about instancing. Nevermind.


Utilise scope in pytest.fixture

Is Your Feature Request Related to a Problem? Please Describe
In general it makes sense to only generate pytest.fixture once, unless it's required to not do that https://docs.pytest.org/en/6.2.x/fixture.html#scope-sharing-fixtures-across-classes-modules-packages-or-session
Describe the Solution You'd Like
No response
Describe Alternatives You've Considered
No response
Additional Context
No response

Closed in #266


Update domains.rst

without indentation, properties becomes None.
Proposed changes:

Documentation of required indentation in domain file.

Status (please check what you already did):

[ ] made PR ready for code review
[ ] added some tests for the functionality
[X] updated the documentation
[ ] updated the changelog
[ ] reformat files using black (please check Readme for instructions)

 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 for the PR, we'll give it a review as soon as possible.

@erohmensing Sorry for the really late reply, but I did, actually. Guess it's doing its job properly :)


Fix Icon Materials that Don't Respect Depth

Overview
A handful of materials are using the GUI/Text Shader which does not respect the depth buffer. This results in icons drawing though other objects. This icons should use the MRTK/Standard shader and be marked as TransparentCutout or Transparent.
Changes

Fixes: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/5521

Verification

Open the HandInteractionExamples scene.
Look at the ToggleFeaturesPanel from the backside.

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run


layout free inline_asm_elementwise

Hi
I want to perform fused right shift (uint8) on inputs of dot with inline_asm_elementwise.
 b = tl.inline_asm_elementwise("shr.b32 $0, $0, 4;",
                         "=r,r", [b, ], dtype=tl.int8, is_pure=True, pack=4)

acc += tl.dot(a, b)

It would be cool if data is shifted on fragments. But there is a dialect type check here
https://github.com/openai/triton/blob/424e67e7275f0cb2cd231e7a4d17ff8570530b77/lib/Dialect/TritonGPU/Transforms/OptimizeDotOperands.cpp#L111-L114
The dot layout can not be propagated to inline_asm_elementwise, Which makes the kernel slower.
Would it be better if we could add a flag to manually enable the layout propagation?
 b = tl.inline_asm_elementwise("shr.b32 $0, $0, 4;",
                         "=r,r", [b, ], dtype=tl.int8, is_pure=True, pack=4, layout agnostic=True)

Or is there any other solution?

adding a flag specific to this optimization seems too ad hoc and not something we would want to expose at the language level.
This technically could be optimized as long as the input and output types have the same size otherwise it would change the order of values within one thread.

It looks like the check should be op->hasTrait<Elementwise>(), that would automatically include elementwise_inline_asm.

@peterbell10 I guess that might not be safe. inline asm with pack>1 could be layout sensitive.

pack is not really what matters (pack only allows manual vectorization). What matters is the element types of the inputs/output tensors. I think @peterbell10's recommendation is correct but we need to make sure we don't generate invalid cases due to different element types.


cross compile to scala 2.12 & 2.11

fix https://github.com/MfgLabs/akka-stream-extensions/issues/25

Can somebody review it? Thx!

Yes, thank you. I will look at the other PR and make a release next week.

thx a lot!


[BUG]

Computer Information:

OS: [e.g. Windows 10 Version 1809 (Start + 'winver' to find out)]
Program Version [e.g. 1.0.2]
Pirated version :(

Describe the bug
The tool does not work and occasional error message shows
Steps to Reproduce
Steps to reproduce the behavior:

Opened SRT
Opened Pirated ver of RE2
Then this error shows up and tool won't work.

Expected behavior
Maybe this only happens to the pirated version of the game.
Screenshots
ERROR POP UPhttp://prntscr.com/mp2wi3
Additional context
Add any other context about the problem here.

I'll look into this, only because it may affect non-pirated versions.

It seems to be specific to pirated versions only. It never occurred on a original copy of the game in almost a week. I tried downloading a pirated version of the game and it showed up after a couple boots of the game.

Non-pirated copies are unsupported.


Error importing Python packages

import scipy as sp

ImportError                               Traceback (most recent call last)
 in ()
----> 1 import scipy as sp
ImportError: No module named 'scipy'

Keep on getting this error message for any package I try to import such as numpy, matplotlib, etc.

Does import scipy as sp work in the IPython console?
You can start the IPython console by running ipython in your terminal.

No, it's not working there either

Igeiger thanks
I already solved the issue, your comment was of much help.

@joseclcodes Great to hear you got it working! 🎉


aws: Fix caching of credentials when using DefaultCredentialsProvider

The code would previously recreate the credential providers at each
Credentials() call, which means the cache of each credential provider
would be cleared.

Thanks for the PR @abustany I am actually in the process of refactoring and cleaning up the Credentials logic/structure which will include caching for the providers.  This refactor should be out soon and will address issues just like the one your PR addresses.


Technical Blog: "How to deploy Doom using Zarf"

Outcome: We want to provide a well-written technical blog that gives step-by-step instructions on how to deploy Doom using Zarf.
Background: We believe that by providing a technical blog, our target user persona (i.e. junior engineers) can see the problem, solve the problem, and understand that Zarf is a tool that makes delivering software in an air gapped environment easy.
Acceptance Criteria / Definition of Done:
-Target persona understands the problem Zarf is trying to solve
-Target persona is able to deploy Doom using Zarf
-Target persona understands/ gains value from the blog
This is blocked or made more difficult by a few things:

Need to make sure the technical content is 100% accurate.

Target finish date: January 17th

IMO this page already has (or should have) this content. Is it worth having a separate technical blog here? Can we make this page in the repo more "blogg-y" if the need isn't being met? I believe this will double the maintenance burden on keeping things up to date since we'll now need to update the Zarf docs linked above as well as the technical blog as Zarf changes.
/cc @jeff-mccoy

I thought the GitHub page worked great.  If the target audience is junior engineers, I don't know why you'd need a separate tech blog.  I'm not a software engineer, but I made my way through it.  One thing I did notice, however, was that I was unable to package and deploy the game without manually copying the game files to my working directory.  When I did the "cd" step, my zarf commands wouldn't work.  As I said, I'm not a software engineer, so this could easily be user error but thought it worth a mention.

I'm going to do a couple videos with the future (new mega multi distro branch) tonight, would like to chat with some humans tomorrow about that. I think it will make some things even easier.


[now-dev] initial build error on bash example

To reproduce:

now init bash
now dev bash --debug

Error:
> Error! Command failed<EMAIL_ADDRESS>index.sh

Full logs:
> [debug] [2019-04-22T02:10:20.589Z] Found config in file /Users/xilo/git/zeit/now-examples/now.json
> [debug] [2019-04-22T02:10:20.595Z] Using Now CLI 15.0.7-canary.2
> [debug] [2019-04-22T02:10:20.596Z] user supplied known subcommand
(node:97828) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
> [debug] [2019-04-22T02:10:20.657Z] The nsfw module is already cached, not re-downloading
> [debug] [2019-04-22T02:10:20.672Z] Reading `now.json` file
> [debug] [2019-04-22T02:10:22.686Z] Adding build match for "index.sh"
> Running initial builds
> [debug] [2019-04-22T02:10:22.687Z] Building index.sh with "@now/bash" v0.1.4 (workPath = /var/folders/r0/_2kzb3ld0bx996dpzqrxdc_w0000gp/T/co.zeit.now/dev/workPaths/dm6vqq15)
downloading files...
import: using '/usr/bin/shasum'
Caching imports in "index.sh"…
import: importing<EMAIL_ADDRESS>import: normalized URL<EMAIL_ADDRESS>import: end of headers<EMAIL_ADDRESS>import: resolved location<EMAIL_ADDRESS>-><EMAIL_ADDRESS>import: calculated hash<EMAIL_ADDRESS>-> 'a9aafe4fcee4a7168dbdf80592864593fdab5b8f'
import: creating symlink<EMAIL_ADDRESS>-> ../../../data/a9aafe4fcee4a7168dbdf80592864593fdab5b8f
import: successfully imported<EMAIL_ADDRESS>-> '/var/folders/r0/_2kzb3ld0bx996dpzqrxdc_w0000gp/T/co.zeit.now/dev/workPaths/dm6vqq15/.import-cache/data/a9aafe4fcee4a7168dbdf80592864593fdab5b8f'
<EMAIL_ADDRESS>line 1: HTTP/2: No such file or directory
> Error! Command failed<EMAIL_ADDRESS>index.sh
> [debug] [2019-04-22T02:10:30.083Z] Error: Command failed<EMAIL_ADDRESS>index.sh
    at makeError (/Users/xilo/Library/Caches/co.zeit.now/dev/builders/node_modules/execa/index.js:174:9)
    at Promise.all.then.arr (/Users/xilo/Library/Caches/co.zeit.now/dev/builders/node_modules/execa/index.js:278:16)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Env:
Now     15.0.7-canary.2 (dev)
Node.js 10.15.3
MacOSX  10.14.4

I checked and it in 15.0.7!


Aqcu 1135 addition -- adding additional sorting

Coverage increased (+0.003%) to 98.489% when pulling 45c110139ad41ec589f17da863fed371c436f75c on mhines-usgs:AQCU-1135-addition into fc5e2112c1efb8364d42d665ef5ea467f32e473d on USGS-R:master.

Coverage increased (+0.003%) to 98.489% when pulling 45c110139ad41ec589f17da863fed371c436f75c on mhines-usgs:AQCU-1135-addition into fc5e2112c1efb8364d42d665ef5ea467f32e473d on USGS-R:master.

Coverage increased (+0.003%) to 98.489% when pulling 45c110139ad41ec589f17da863fed371c436f75c on mhines-usgs:AQCU-1135-addition into fc5e2112c1efb8364d42d665ef5ea467f32e473d on USGS-R:master.

Coverage increased (+0.003%) to 98.489% when pulling 14b0056865806afbe4786a8bea91eb457b3d4997 on mhines-usgs:AQCU-1135-addition into fc5e2112c1efb8364d42d665ef5ea467f32e473d on USGS-R:master.

So fumbling a bit with this but this sometimes works but sometimes doesn't, depending on the data..
#' Parse TSS Thresholds
#'
#' @description Given the full report JSON object reads the
#' thresholds and handles read errors.
#' @param reportObject the full report JSON object
#' @param timezone the timezone of the report
parseTSSThresholds <- function(reportData, timezone){
  thresholds <- tryCatch({
    readThresholds(reportData)
  }, error=function(e){
    warning(paste("Returning NULL for TSS thresholds. Error:", e))
    return(list())
  })

  if(!isEmptyOrBlank(thresholds)){
    thresholds <- mutate(thresholds, sortTime = sapply(thresholds$periods, "[[", "startTime"))
    thresholds <- thresholds[order(thresholds[['sortTime']]),]
    thresholds[['periods']] <- lapply(thresholds[['periods']], function(p){
      p[['startTime']] <- formatOpenDateLabel(flexibleTimeParse(p[['startTime']], timezone))
      p[['endTime']] <- formatOpenDateLabel(flexibleTimeParse(p[['endTime']], timezone))
      return(p)
    })

    thresholds <- attachFullDataToSubFrame(thresholds, 'periods')
  }

  return(thresholds)
}

Hmm perhaps if you did the thresholds sort time mutation and ordering after the ordering within the periods? It might be that the first period isn't necessarily the earliest so when you pull it out to sort by it you might not end up sorting by the earliest? So if you add back the sorting of the periods within the lapply and then do the outside threshold sorting and mutating after that it could help?
The other thing is that there are situations when the threshold could have multiple periods and I'm not totally sure how that would end up with your sapply? I'm not too familiar with how sapply works so I could be wrong but that might end up giving you a list of all of the start times for the sortTime and then the sort after that might be sorting the overall thresholds based on a list for within each row and I'm not sure how that works

Yeah, still trying to wrap my head around the variations across the test datasets. Trying something else now!

maybe I'm crazy, but I just used this data for thresholds, and these seem to be sorting without me doing anything already..
 "thresholds": [
        {
            "severity": "0.0",
            "name": "Operational limit (maximum) Equipment 2010",
            "description": "Values above a specified threshold limit",
            "periods": [
                {
                    "startTime": "2015-10-07T00:00:00-05:00",
                    "endTime": "9999-12-31T23:59:59.9999999Z",
                    "appliedTime": "2017-02-02T21:27:53.3106Z",
                    "referenceValue": 19.2,
                    "secondaryReferenceValue": 17.28,
                    "suppressData": true
                }
            ],
            "referenceCode": "Operational limit - high-Public",
            "type": "ThresholdAbove"
        },
        {
            "severity": "0.0",
            "name": "Operational limit (minimum) DCP offset 2016",
            "description": "Values below a specified threshold limit",
            "periods": [
                {
                    "startTime": "2015-10-02T00:00:00-05:00",
                    "endTime": "9999-12-31T23:59:59.9999999Z",
                    "appliedTime": "2017-02-02T21:27:53.1881899Z",
                    "referenceValue": -0.07,
                    "secondaryReferenceValue": 0.13,
                    "suppressData": true
                },
                {
                    "startTime": "2015-10-05T00:00:00-05:00",
                    "endTime": "9999-12-31T23:59:59.9999999Z",
                    "appliedTime": "2017-02-02T21:27:53.1881899Z",
                    "referenceValue": -0.07,
                    "secondaryReferenceValue": 0.13,
                    "suppressData": true
                },
                {
                    "startTime": "2015-10-06T00:00:00-05:00",
                    "endTime": "9999-12-31T23:59:59.9999999Z",
                    "appliedTime": "2017-02-02T21:27:53.1881899Z",
                    "referenceValue": -0.07,
                    "secondaryReferenceValue": 0.13,
                    "suppressData": true
                }
            ],
            "referenceCode": "Operational limit - low-Public",
            "type": "ThresholdBelow"
        },
		{
            "severity": "0.0",
            "name": "Operational limit (minimum) DCP offset 2016",
            "description": "Values below a specified threshold limit",
            "periods": [
                {
                    "startTime": "2015-10-04T00:00:00-05:00",
                    "endTime": "9999-12-31T23:59:59.9999999Z",
                    "appliedTime": "2017-02-02T21:27:53.1881899Z",
                    "referenceValue": -0.07,
                    "secondaryReferenceValue": 0.13,
                    "suppressData": true
                }
            ],
            "referenceCode": "Operational limit - low-Public",
            "type": "ThresholdBelow"
        },
		{
            "severity": "0.0",
            "name": "Operational limit (minimum) DCP offset 2016",
            "description": "Values below a specified threshold limit",
            "periods": [
                {
                    "startTime": "2015-10-03T00:00:00-05:00",
                    "endTime": "9999-12-31T23:59:59.9999999Z",
                    "appliedTime": "2017-02-02T21:27:53.1881899Z",
                    "referenceValue": -0.07,
                    "secondaryReferenceValue": 0.13,
                    "suppressData": true
                }
            ],
            "referenceCode": "Operational limit - low-Public",
            "type": "ThresholdBelow"
        },
		{
            "severity": "0.0",
            "name": "Operational limit (minimum) DCP offset 2016",
            "description": "Values below a specified threshold limit",
            "periods": [
                {
                    "startTime": "2015-10-01T00:00:00-05:00",
                    "endTime": "9999-12-31T23:59:59.9999999Z",
                    "appliedTime": "2017-02-02T21:27:53.1881899Z",
                    "referenceValue": -0.07,
                    "secondaryReferenceValue": 0.13,
                    "suppressData": true
                }
            ],
            "referenceCode": "Operational limit - low-Public",
            "type": "ThresholdBelow"
        }
    ],

Coverage increased (+0.005%) to 98.491% when pulling d4c906f9a545d3b5d7590a882d1860cb3ad6b1f2 on mhines-usgs:AQCU-1135-addition into fc5e2112c1efb8364d42d665ef5ea467f32e473d on USGS-R:master.

Thanks.. yeah. I might test the other sections when I'm bored to see if they also don't need any explicit sorts. Probably some other function handled it here.

It would be worthwhile to throw in tests for the parseTSS... functions for ordering.

Yeah I think the check to see if the dates are open ended transforms the dates into characters. I'll look at adding some tests. Thanks! Would you think arrange is the better solution here instead of order?

Yeah that function transforms them to characters because if they are open-ended it needs to replace them with the word "Open" and I was getting errors when the data frame was mixed with dates and strings

@mhines-usgs I don't think it's necessary. dplyr can be faster depending on how large the data frames are because it's running C (C++ ?) under the hood.  In this case, it's mostly a style thing.

@zmoore-usgs yeah, you can't have a column with mixed values - it's a vector so it needs one class. I guess ordering by characters should work, but just feels strange. Could the conversion to character happen after the ordering?

It could yeah, it just needs to happen before the call to formatOpenDateLabel

Yep, I can move the calls above that part. No problem!


[image_picker] PlatformException(invalid_source, Invalid image source., null, null) on iPhone 11 (ios 15.5)

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
in the ios 15.5 simulator, the selection of photos from the gallery does not work.
image_picker: ^0.8.7+2
Expected results
this works in the latest version of ios
Actual results
in the ios 15.5 simulator, the selection of photos from the gallery does not work.
image_picker: ^0.8.7+2
Code sample
Code sample
 final image = await picker.pickImage(source: ImageSource.gallery);

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
[VERBOSE-2:dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(invalid_source, Invalid image source., null, null)
#0      ImagePickerApi.pickImage
messages.g.dart:136
<asynchronous suspension>
#1      ImagePickerIOS.getImageFromSource
image_picker_ios.dart:78
<asynchronous suspension>
#2      __ProdileEditScreenState._getImage
profile_edit_page.dart:63
<asynchronous suspension>

Flutter Doctor output
Doctor output
flutter doctor -v
[✓] Flutter (Channel stable, 3.7.12, on macOS 13.3.1 22E261 darwin-arm64, locale ru-KZ)
    • Flutter version 3.7.12 on channel stable at /Users/rayyvon/FlutterDev/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (9 days ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/rayyvon/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.77.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (4 available)
    • iPhone 14 Pro Max (mobile) • 593DC49F-C551-4F4B-BF13-D816D2854487 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • iPhone 11 Pro (mobile)     • 81053BFE-EF27-4EFF-AAA2-BCC2A19DC177 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-5 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 13.3.1 22E261 darwin-arm64
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 112.0.5615.137

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!```

</details>

Hi @Aboneyx
Can you please provide the complete output logs (flutter run -v) so that we may verify the issue properly? Also, what kind of image type have you picked?
Taking a look at the error message, it may be caused by source type or supported type.

I found out that this bug exists in versions below ios 16.1

I can reproduce this issue even with iOS 15.2 or iOS 16.4. And it only happens with a specific image (image has flowers above) whilst other images can display well.
After checking image format, it's a HEIC image: IMG_0006.HEIC (IMG_0006.HEIC.zip). This is a known issue that is being noted on the document (please see iOS note). When I test this image on physical device, the issue doesn't happen.
Please retry on a physical device for HEIC format. I will close this issue as this is a foreseeable issue. If the crash still occurs on physical device, please write a comment and I will reopen this for further investigation. Thanks!


StringLookup fails as first layer in a Sequential model in TF 2.8.0

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04 (Google Colab)
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary (preinstalled on Google Colab)
TensorFlow version (use command below): v2.8.0-0-g3f878cff5b6 2.8.0
Python version: 3.7.12
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A

Describe the current behavior
Using a tf.keras.layers.StringLookup layer as the first layer in a Sequential model raises an exception when calling the model: UnimplementedError: Exception encountered when calling layer "sequential" (type Sequential). Cast string to int64 is not supported [Op:Cast] (see full stacktrace below).
Standalone code to reproduce the issue
import tensorflow as tf

cat = ["Paris", "Singapore", "Auckland"]
str_lookup_layer = tf.keras.layers.StringLookup()
str_lookup_layer.adapt(cat)
lookup_and_embed = tf.keras.Sequential([
    str_lookup_layer,
    tf.keras.layers.Embedding(input_dim=str_lookup_layer.vocabulary_size(),
                              output_dim=2)
])
lookup_and_embed(tf.constant([["Paris"], ["Singapore"], ["Auckland"]]))  # ERROR!

This code is available in this gist.
Describe the expected behavior
This should work like it did in TensorFlow 2.7.1, I believe it's a regression. It should output something like this:
<tf.Tensor: shape=(3, 2), dtype=float32, numpy=
array([[-0.02887753, -0.01268407],
       [ 0.04601531, -0.02668235],
       [ 0.03409723, -0.03205377]], dtype=float32)>

Other info / logs
Instead, it raises this exception. Full stacktrace:

---------------------------------------------------------------------------
UnimplementedError                        Traceback (most recent call last)
[<ipython-input-2-ee4b4b94a15e>](https://localhost:8080/#) in <module>()
      7                               output_dim=2)
      8 ])
----> 9 lookup_and_embed(tf.constant([["Paris"], ["Singapore"], ["Auckland"]]))

1 frames
[/usr/local/lib/python3.7/dist-packages/keras/utils/traceback_utils.py](https://localhost:8080/#) in error_handler(*args, **kwargs)
     65     except Exception as e:  # pylint: disable=broad-except
     66       filtered_tb = _process_traceback_frames(e.__traceback__)
---> 67       raise e.with_traceback(filtered_tb) from None
     68     finally:
     69       del filtered_tb

[/usr/local/lib/python3.7/dist-packages/tensorflow/python/framework/ops.py](https://localhost:8080/#) in raise_from_not_ok_status(e, name)
   7184 def raise_from_not_ok_status(e, name):
   7185   e.message += (" name: " + name if name is not None else "")
-> 7186   raise core._status_to_exception(e) from None  # pylint: disable=protected-access
   7187
   7188

UnimplementedError: Exception encountered when calling layer "sequential" (type Sequential).

Cast string to int64 is not supported [Op:Cast]

Call arguments received:
  • inputs=tf.Tensor(shape=(3, 1), dtype=string)
  • training=None
  • mask=None

Hi @ageron !
Please post this issue on keras-team/keras repo.
To know more see;
https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999 . Thanks!

Oh right, will do, thanks.


Added rebus

Associated Issue: N/A
Summary of Changes

Added a rebus by following along with the README. Plan to start working on #93 in this my forks 'persistent-storage' branch.

Pull Request Test Coverage Report for Build 217

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.71%

Totals

Change from base Build 216:
0.0%

Covered Lines:
113

Relevant Lines:
125

💛  - Coveralls


Is this library ready to usein production?

Because its private in npm.

it’s kinda ready, just is not in npm yet.
However consider the fact that all post quantum libs are not really ready until the fips draft is finished.

Thank you.
also thanks for the other very useful libraries that you've written.
It helped a lot, now i dont have to write native code or wasm.