Apply WASI support patch against ICU source tree

467631d2a7359c12bcf1c2254a3e797471d70b46 cherry-picks ICU-22838 Add WebAssembly/WASI cross-compilation support
The original patch is still under review in the upstream ICU project, but it is needed to unblock the swift-foundation build on WebAssembly.
And also adjust CMakeLists.txt and Package.swift to disable timezone and dynamic loading features.

Hi @kateinoigakukun thanks for the patch! We recently upgraded our ICU to 74 and I'll reapply your patch on top of 74

Reapplied my patch on the top of the latest main branch :)

@swift-ci test

Confirmed this change does not break other platform builds https://github.com/apple/swift-foundation/pull/823

SwiftCorelibsFoundation and SwiftFoundation integration tests both passed! Merging now...

Thanks!


where is this function in sklearn?

I didn't find the "loss_curve_" function in official documents of sklearn . When I try to use this function in VS2017, it say "this model don't have loss_curve_ ".
My sklearn version is 2.3.0.
could you please help me,thanks?

Hi, the loss_curve_ function comes from sklearn.neural_network.MLPRegressor -- looks like it isn't in the documentation, but it is in the code: https://github.com/scikit-learn/scikit-learn/blob/master/sklearn/neural_network/_multilayer_perceptron.py
loss_curve_ : list of shape (n_iters,)
        Loss value evaluated at the end of each training step.


adding boilerplate coverity scan to submit to public analysis

Generic boilerplate coverity scan workflow:
Downloads coverity build tool
Builds from project code
Uploads a build to scan queue

 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.


ENH: Use IEX Trading data instead of pandas-datareader

Got fed up with the breaking/removal of Google & Yahoo Finance APIs, and so since we've been using pandas-datareader to get data from those APIs I removed that dependency for when we want to get data for a specific benchmark (in this case SPY).
I changed our data source to IEX which will give us up to 5 years worth of data. It has really great documentation and seems like its a lot more reliable.
Should fix:

https://github.com/quantopian/zipline/issues/2029
https://github.com/quantopian/zipline/issues/2024
https://github.com/quantopian/zipline/issues/1965
https://github.com/quantopian/zipline/issues/2002
https://github.com/quantopian/zipline/issues/2036
https://github.com/quantopian/zipline/issues/2030

And should be better than PR https://github.com/quantopian/zipline/pull/1972

:+1: on moving to the IEX api.  Do we need to add an attribution somewhere?  From https://iextrading.com/api-terms/:

If you sell, lease, furnish, permit or provide access to, or otherwise redistribute Investors Exchange (“IEX”) API data, which includes API data licensed from third-parties, to any other person:
a. you must provide a link to https://iextrading.com/api-exhibit-a or otherwise obtain consent from such person to the terms of use substantially in the form of the above Exhibit A;
b. you must cite IEX as the source in accordance with the instructions available on https://iextrading.com/developer;

MIght be worth opening an issue/asking a question on https://github.com/iexg/IEX-API.

@ssanderson Thanks for catching that. I read that attribution comment at some point and just kinda forgot. Will open an issue there.

Coverage increased (+0.006%) to 87.378% when pulling 181a0f7479ad3faa02f143b1f6edcdb428d4a8a2 on use-iex-for-benchmark-data into 49a3a4a2785a4a75dc21f04389094681eb408579 on master.

Opened up an issue: https://github.com/iexg/IEX-API/issues/124

Coverage increased (+0.006%) to 87.378% when pulling 8b87424ff57358536f4e1ca2dbe7558148ac74a2 on use-iex-for-benchmark-data into 42b32007dd6df8b2ff8d75bd1b90bf1a902da064 on master.

Coverage increased (+0.006%) to 87.378% when pulling 8b87424ff57358536f4e1ca2dbe7558148ac74a2 on use-iex-for-benchmark-data into 42b32007dd6df8b2ff8d75bd1b90bf1a902da064 on master.

Coverage increased (+0.006%) to 87.378% when pulling c2a735bd72d6f87c0eb62c61580c6213e5074fb3 on use-iex-for-benchmark-data into 42b32007dd6df8b2ff8d75bd1b90bf1a902da064 on master.

This looks like a good change for zipline master, but FYI it still does not fix #2024.  I'm getting the same KeyError using this branch and trying to run the do-nothing test algo I described in that issue through latest available close (12/1):
KeyError: 'the label [2017-11-30 00:00:00+00:00] is not in the [index]'
The loader seems to pick the wrong benchmark ending date of 11/29:
root@1400ce2d197f:/projects# zipline run -b quantopian-quandl -f test.py --start=2017-1-1 --end=2017-12-31
[2017-12-02 19:10:15.709130] INFO: Loader: Cache at /root/.zipline/data/SPY_benchmark.csv does not have data from 1990-01-02 00:00:00+00:00 to 2017-11-29 00:00:00+00:00.
[2017-12-02 19:10:15.709493] INFO: Loader: Downloading benchmark data for 'SPY' from 1989-12-29 00:00:00+00:00 to 2017-11-29 00:00:00+00:00
[2017-12-02 19:10:16.352607] WARNING: Loader: Still don't have expected benchmark data for 'SPY' from 1989-12-29 00:00:00+00:00 to 2017-11-29 00:00:00+00:00 after redownload!
[2017-12-02 19:10:16.353117] INFO: Loader: Cache at /root/.zipline/data/treasury_curves.csv does not have data from 1990-01-02 00:00:00+00:00 to 2017-11-29 00:00:00+00:00.
[2017-12-02 19:10:16.353323] INFO: Loader: Downloading treasury data for 'SPY' from 1990-01-02 00:00:00+00:00 to 2017-11-29 00:00:00+00:00

When data through 12/1 is available:
root@1400ce2d197f:/projects# curl -s https://api.iextrading.com/1.0/stock/SPY/chart/5y | python -c "import sys, json; print(json.load(sys.stdin)[-1])"
{'changePercent': -0.208, 'change': -0.55, 'date': '2017-12-01', 'low': 260.76, 'changeOverTime': 0.8696359137504418, 'label': 'Dec 1', 'vwap': 263.4229, 'unadjustedVolume': 164390902, 'close': 264.46, 'high': 265.31, 'volume': 164390902, 'open': 264.76}

The issue is this line:
https://github.com/quantopian/zipline/blob/5421ec033712b383dfd924525ddb590b58546b5d/zipline/data/loader.py#L156
It appears to be in place because the treasury data from the Fed H15 report is a day lagged from the Quandl and IEX stock market data.
In my opinion, treasury curve data should be filled through the latest available market data close rather than hold up the trading algorithm a full day behind the market.
Alternatively, it appears users can implement their own load_market_data callable and pass via the load arg in when instantiating the TradingEnvironment object.

Hello, what would be the steps to call it after having pulled the code changes?

Currently version 1.1.1 is broken pending this fix getting released. Any chance we can get a conda install for 1.1.0 to keep things going in the meantime?

Just getting started on zipline.  Anxious for this branch to get implemented so the benchmarking is fixed with the IEX historical data.


Mail template option

Contribution Type: new feature
Primary Area: plugins
Link to Bug: #907
Description of change:

Add an "template" option for the email plugin to select the template to use (between "short" and "long").

Description of solution:

Add a common layout for emails, with dynamic parts (color and introducing phrases).
Rename the old "failed" and "success" templates to "short" and "long".
For backwards compatibility, when the "template" option is missing, the selection is based on the build status: "short" for failed builds, "long" for successful builds.

@mavimo Changes done.


Docker Compose script not set up to be plug-and-play (yet) - need to change image or build directives

I'm finding the docker-compose script is not plug-and-play on v2.0.0. Is that expected?

Pulling repository docker.io/library/hygieia-api
ERROR: Error: image library/hygieia-api not found

I realized this is just a small issue probably, so I tried...
diff --git a/docker-compose.yml b/docker-compose.yml
index 78e2137..b739942 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -8,7 +8,7 @@ mongodb:
    - ./mongo:/data/db:rw
   volume_driver: local
 hygieia-api:
-  image: hygieia-api:latest
+  image: capitalone/hygieia-api:latest
   container_name: hygieia-api
   ports:
   - "8080:8080"
@@ -17,7 +17,7 @@ hygieia-api:
   links:
   - mongodb:mongo
 hygieia-ui:
-  image: hygieia-ui:latest
+  image: capitalone/hygieia-ui:latest
   container_name: hygieia-ui
   ports:
   - "8088:80"
@@ -26,7 +26,7 @@ hygieia-ui:

and so on. But applying this change to some of the collectors did not work. First, the versionone collector. I don't use that so I just commented-out. But then jira collector isn't working, and I'd like to use that image. It doesn't seem like hygieia-jira-feature-collector is on https://hub.docker.com/.
I can't build them all with docker-compose build, and that seems like reasonable behavior ...
$ docker-compose build
mongodb uses an image, skipping
hygieia-api uses an image, skipping
hygieia-ui uses an image, skipping
hygieia-jenkins-build-collector uses an image, skipping
hygieia-subversion-scm-collector uses an image, skipping
hygieia-udeploy-collector uses an image, skipping
hygieia-chat-ops-collector uses an image, skipping
hygieia-github-scm-collector uses an image, skipping
hygieia-stash-scm-collector uses an image, skipping
hygieia-jira-feature-collector uses an image, skipping
hygieia-jenkins-cucumber-test-collector uses an image, skipping
hygieia-sonar-codequality-collector uses an image, skipping

... since indeed the docker-compose.yml is set up to refer to images.
I'm just thinking, for plug-and-play demos, it would be nice to grab Hygieia and "just" run docker-compose :)
To that end, maybe if they all say image:, they should all be on DockerHub ... else, maybe they should say build: and refer to the Dockerfiles within this repository. Once consistent, it would be easy to add reference to this in the README or documentation, as a way for someone to get up and running quickly.
Thoughts? Happy to contribute to a solution once I know the consensus.

Containers: 11
 Running: 3
 Paused: 0
 Stopped: 8
Images: 17
Server Version: 1.11.2
Storage Driver: aufs
 Root Dir: /mnt/sda1/var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 138
 Dirperm1 Supported: true
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.4.12-boot2docker
Operating System: Boot2Docker 1.11.2 (TCL 7.1); HEAD : a6645c3 - Wed Jun  1 22:59:51 UTC 2016
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.955 GiB
Name: default
ID: UAOQ:DT74:42PW:EQ2A:RAAK:DKNH:6AUY:AFQQ:CXTQ:KTE5:TB3Y:UEWA
Debug mode (server): true
 File Descriptors: 27
 Goroutines: 66
 System Time: 2016-08-11T00:30:48.850282621Z
 EventsListeners: 1
 Init SHA1:
 Init Path:
 Docker Root Dir: /mnt/sda1/var/lib/docker
Labels:
 provider=virtualbox

I also try this and it seems to be invalid on each 2.x.x release :(

@hangtwenty @tatemz - we have recently released the first in a series of updates to our docker setup. I believe this update solves the issue you were facing here. Please pull the latest version of Hygieia, give it a shot and let me know how it goes. Feel free to continue the discussion here or open a new issue if you run into any problems.


Introduce XRun command

For iOS projects, this should use the XBuild command to build the project, then use simctl (xcrun simctl) to install the app.
# build
xcrun simctl install booted <path to built app>
xcrun simctl launch booted <app identifier> # this might be problematic

simctl usage:
Usage: simctl [--noxpc] [--set <set path>] <subcommand> ... | help [subcommand]
Command line utility to control the Simulator

For subcommands that require a <device> argument, you may specify a device UDID
or the special "booted" string which will cause simctl to pick a booted device.
If multiple devices are booted when the "booted" device is selected, simctl
will choose one of them.

Subcommands:
        create              Create a new device.
        delete              Delete a device or all unavailable devices.
        pair                Create a new watch / phone pair.
        unpair              Unpair a watch and phone phone.
        erase               Erase a device's contents and settings.
        boot                Boot a device.
        shutdown            Shutdown a device.
        rename              Rename a device.
        getenv              Print an environment variable from a running device.
        openurl             Open a URL in a device.
        addphoto            Add photos to the photo library of a device.
        install             Install an app on a device.
        uninstall           Uninstall an app from a device.
        get_app_container   Print the path of the intsalled app's container
        launch              Launch an application by identifier on a device.
        spawn               Spawn a process on a device.
        list                List available devices, device types, runtimes, or device pairs.
        icloud_sync         Trigger iCloud sync on a device.
        help                Prints the usage for a given subcommand.

So the process (in theory) is this:

Boot an iOS simulator (xcrun simctl boot <UUID>)
Install app in simulator (xcrun simctl install booted <PATH>)
launch the app in the simulator (xcrun simctl launch <BUNDLE_IDENTIFIER>)

if we want to give users a choice of simulator (maybe g:xcodebuild_ios_simulator or XSetSimulator), we can get a list of options with xcrun simctl list devices.
This is all great, except that doing a simple install hangs for me and never finishes.

the nice thing is that I think this will be simpler for OS X: open -a build/MyCoolApp.app
Determining the target name is still going to be a pain in the ass, even if we don't need to figure out the bundle identifier.

simctl seems horribly broken. I can't get any of this to work.

ios-sim seems to still work, I should be able to look into it and their node wrapper around simctl to figure out what to do/what I'm doing wrong.

Looks like most of the bugs I ran into were from Xcode 7? I got a working prototype of this working as a shell script.
Other things to think of:

Can we tell the difference between a Mac .app and an iOS .app?
How do we help people not try launching iPad apps in an iPhone sim? Do we care?
How are watch or TV apps handled?

I think we can punt on these for MVP

I got a working prototype of this working as a shell script.

Using ios-sim or simctl? Any chance you could share a gist? I'm trying to automate installing and launching a build app into a Simulator and haven't been able to achieve it without ios-sim :disappointed:
Thanks :smile:

I was able to get something working with bare simctl, which is important because I'd like to avoid adding external dependencies to this project.
I believe this is the working prototype although it's been a while since I've tried it so it might be broken.

@gfontenot Thanks for pointing me to it.
Your script works great 👍


E2E Test for tailing multi_line logs with a custom pattern

What does this PR do?
Adds an e2e test for the logs agent to test tailing multi_line logs with a custom pattern

Motivation
This adds more e2e test coverage for multi line logs. (Auto multi line detection was covered in a previous PR)

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
This is the test

/merge


Thefootest patch 4

Can one of the admins verify this patch?

test this please

test this please

watch

test this please

watch

test this please

watch

add to whitelist

add to whitelist

add to whitelist

add to whitelist


Release history is behind

[Enter feedback here]
Shouldn't this page be reflecting the gateway releases for Nov/Dec 2021? https://powerbi.microsoft.com/en-us/blog/on-premises-data-gateway-december-2021-update-is-now-available/

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

ID: 27fa3259-c8cf-bd35-7e6a-6f8a1f2a36cf
Version Independent ID: 187906ab-5531-bce0-084c-d90e3c009c82
Content: Previous monthly updates to the on-premises data gateways
Content Source: data-integration/gateway/service-gateway-monthly-updates.md
Product: on-premises-data-gateway
Technology: ****
GitHub Login: @arthiriyer
Microsoft Alias: arthii

Hi @JamesTStanley
Thanks for your feedback. This was an oversight on our part. The release history is being watched more closely, and is currently up to date.


[5.8] Smooth redis throttling

Use case
Throttling jobs in redis queue. For example, sending a lot of emails and smoothing out the job execution in order to prevent many emails at once, which would increase a risk of being flagged as spam.
Code example
Redis::throttle('key')->allow(120)->every(60)

We say here, that we want to allow 120 jobs to execute every 1 minute. However, that would run them all as fast as possible till it executes all 120 and then just waits and does nothing for the remaining seconds.
Redis::throttle('key')->smoothly()->allow(120)->every(60)

Now in this case, would be equivalent to:
Redis::throttle('key')->allow(2)->every(1)

thus smoothing out job execution.
Case when allowed locks is less then decay:
Redis::throttle('key')->smoothly()->allow(6)->every(60)

is equivalent to
Redis::throttle('key')->allow(1)->every(10)

Now I couldn't find a test for DurationLimiterBuilder class. Should we cover this class?
Please review and give comments on how suggested code could be improved (naming, structuring)

I guess I don't understand. If it's the same as allow(2)->every(1) just write that to begin with? 😄


Add database session config field in configuration

1. Why is this change necessary?
In case you need to change e.g. wait_timeout for shopware but your hosting environment does not offer the choice of editing the my.cnf. Just add a 'db' => [ 'session' => [ 'wait_timeout' => 1 << 16 ] ] to wait for a decent amount of seconds before your database or hosting partner is about to die.
2. What does this change do, exactly?
Simply sets these values on the database as well everytime a connection is created.
3. Describe each step to reproduce the issue or behaviour.

Have inperformant SQL query
MySQL gone away
Increase timeout in my.cnf
my.cnf is readonly

4. Which documentation changes (if any) need to be made because of this PR?
Maybe a hint in the dev cheatsheet or hosting docs?
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-25020
Please use this issue to track the state of your pull request.

Thanks for your Contribution! :blue_heart:

You're always welcome :) 💙


GEMM kernel stalls due to dimension mismatches

For some dimensions of the optimized kernels the SSR region will stall, since the buffer will not be consumed entirely due to SIMD/loop unrolling. Need to add constraints on dimension in datagen.py

Fixed by https://github.com/pulp-platform/snitch_cluster/pull/137/.


Add documents to manage multiple versions on Windows with Scoop

I believe this PR is incorrect.
Switching deno versions using Scoop is not possible at the moment.
The linked article starts with the following command scoop bucket add versions. This adds an additional scoop bucket which contains major/minor versions of well-known programs.
The main bucket contains only ruby (https://github.com/ScoopInstaller/Main/blob/master/bucket/ruby.json). The versions bucket contains ruby19 (https://github.com/ScoopInstaller/Versions/blob/master/bucket/ruby19.json) and ruby24 (https://github.com/ScoopInstaller/Versions/blob/master/bucket/ruby24.json), for example.
As far as I know, that is the only concept of "versioning" that is currently possible with Scoop. There is no scoop install<EMAIL_ADDRESS>as described in the README. Thus, I believe this PR should be reverted.
On another note, we can create our own bucket (for example, https://github.com/denoland/deno-versions.git) with a bucket directory and put deno-v0-26-0.json, deno-v0-25-0.json, etc. in there, and voilà, we got versioned Deno with Scoop. Users would then have to add the bucket via scoop bucket add deno-versions https://github.com/denoland/deno-versions.git and could then run, for example, scoop install deno-v0-26-0. (Not sure whether Scoop works with .'s in the tool name, perhaps even deno-v0.26.0 could work.)

@MarkTiedemann

@zhmushan Awesome! Thanks for correcting my incorrect correction attempt.
(Well, this didn't work when I started using Scoop a couple of years ago... Perhaps I should make a PR to update their documentation about this. :)


Added CollectionAssertions

❓ Type of change

[x] ✨ New feature (adds functionality)

📚 Description
Added common-unit module & Added CollectionAssertions
Closes #188

🧪 How Has This Been Tested?
Unit test

📝 Checklist

[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

Summary by CodeRabbit

New Features

Introduced new modules and classes: common-unit, UUIDParser, GameMode parser, CollectionAssertions, Rotation, Transform, and ValueRequirement.
Enhanced command processing with new attributes and methods, improving functionality and usability.

Bug Fixes

Resolved issues in the spigot component related to player visibility and command sender type inversion.

Documentation

Updated changelog to reflect new features, fixes, and improvements.

Tests

Added CollectionAssertionsTest to validate new collection assertion methods.

When I think about it this is useless we already have this utility There is no need for it
we might want to add this tho
public static RuntimeException buildException(String reason, Object expected, Object actual, String message, Throwable cause) {
    reason = (reason == null ? "" : reason + ", ") + formatValues(expected, actual);

    message = message == null || message.isBlank()
        ? reason
        : message + " ==> " + reason;

    if (cause == null)
        return new IllegalArgumentException(message);

    return new IllegalArgumentException(message, cause);
}

private static String formatValues(Object expected, Object actual) {
    return String.format("expected: <%s> but was: <%s>", toString(expected), toString(actual));
}

private static String toString(Object obj) {
    return obj instanceof Class<?> clazz
        ? clazz.getName()
        : obj == null ? "null" : obj.toString();
}


Fix crazy physics

First half of #517

Download:
Environment: QT=5.12, ARCH=32, PYTHON_PATH=C:\Python37
Environment: QT=5.12, ARCH=64, PYTHON_PATH=C:\Python37-x64


[Feature] Allow presets for soxr

Allow creating presets for soxr settings.
Variable name: SOXR_PLUGIN_PRESET
One possible value: "goldilocks"

Available in 2022-11-12.1


debug: false not working?

I'm trying to set up logging following the instructions in the current release's documentation. I'm using debug: false to try and cut down on the noise. I'm still seeing a fair bit of noise though. Example:
[info] GET https://feeds.npr.org/381444908/podcast.xml -> 200 (406.203 ms)                                                            [46/4731]
[debug]
>>> REQUEST >>>
(no query)
accept-encoding: gzip, deflate
accept: application/rss+xml, application/rdf+xml;q=0.8, application/atom+xml;q=0.6, application/xml;q=0.4, text/xml;q=0.4

(no body)

<<< RESPONSE <<<
server: Apache/2.4.25 (Debian)
x-powered-by: PHP/7.2.14-1+0~20190113100742.14+stretch~1.gbpd83c69
content-encoding: gzip
x-served-by: prod-web-2
x-content-type-options: nosniff
x-xss-protection: 1; mode=block
referrer-policy: no-referrer-when-downgrade
strict-transport-security: max-age=604800; includeSubDomains
x-frame-options: SAMEORIGIN
content-length: 21462
content-type: text/xml;charset=UTF-8
cache-control: max-age=0
expires: Wed, 20 May 2020 19:43:35 GMT
date: Wed, 20 May 2020 19:43:35 GMT
vary: Accept-Encoding

���r�H����
����n�ġp�hM�҈��HjF��ᘨ�!��^�V��/�7�k���J�YU����DNϨ7f)(�Bf֓U�O~���¹Ru�W�ޑ{��RT2/�?���9I���߾��Ɓ�e�]9�8�������x�X����C���u�i�n�f���>յ��y�����:��X��9�
UR��%е#8�])��Ue��t6�

Note that the stuff at the end appears to be the gzip'd response body.

Try mix deps.compile tesla --force.
This is a known issue, I have Logger middleware internals redesign planned for the near future

Worked like a charm, but I had to rm -rf deps first. Many thanks! Feel free to close the issue unless you wanted to keep it around for tracking purposes.


Adding inventory fails on production

Woohoo! This was as easy as just running migrations. I guess migrations don't run automatically... we should look into making that fully automated.
Here was a useful set of commands I ran in a Heroku console that helped figure the problem out, since the log wasn't really showing everything I needed (taken from http://stackoverflow.com/questions/22078627/how-do-i-render-a-partial-from-the-rails-console):
> view_paths = Rails::Application::Configuration.new(Rails.root).paths["app/views"]
> av_helper = ActionView::Base.new view_paths
> @item = Item.new(category_id: "4")
> @categories = Category.all
> av_helper.extend Rails.application.helpers
> av_helper.render template: "items/edit", locals: { :@categories => @categories, :@item => @item }


native: fix chat actions sheet on Android

This was another instance of the Android modal context bug (see TLON-2618 for details). The pattern of using context to grab these handlers feels good, so I wanted to avoid removing it in hopes we can return to normal use once the bug gets squashed (sounded like Facebook RN core team has it on their radar).
To that end, the workaround introduced is to update the sheets to take a new prop that matches the missing context value. The context file itself exports a hook for returning the needed value in addition to its existing provider based API. My thinking was this way we can avoid modifying the business logic as much as possible and instead just remove the new prop threading once the normal usage is restored.
Open to better ideas on solutions if y'all have any!
Fixes TLON-2618

Lgtm, but wanna grab me a sec before you merge in?

Going to ship a patch in the demo branch and close this in favor of other approaches.


how to untoggle date

For example 28th is toggled, after toggling on the 30th the 28th should be untoggled. How to accomplish this usecase?

Hi @feldmarv :)
Sorry about late 😢
This issue need snippets...
If you okay, Please tell me more specific detail
Thank you for your time 👍

Thanks for the reply.
I do load certain events in Array then display it in the calendar.
var slotDates = [];
var url = "../../../events/getSlotDates/" + getCookie("eventId");
$.ajax({
method: "get",
url: url,
dataType: 'json',
cache: false,
async: false,
success: function (data) {
$.each(data.slotDates, function (i, d) {
slotDates.push(d);
});
    }
});

$('#terminslot-kalender').pignoseCalendar({
lang: 'de',
toggle: true,
enabledDates: slotDates,
date:moment(slotDates[0]),
select: loadSlotsbyDate
});

As you can see when an event in the calendar is selected it loads specific information about this event by the function loadSlotsbyDate(), The toggled event is marked as active. If the user toggles an other event this information is loaded by loadSlotsbyDate(). Now the problem is that both dates are active even if only information about one is displayed.

I didn't think in async connection calling styles @feldmarv
In your code,
success callback will be executed after Calendar initialized.
And select, set method in PIGNOSE Calendar seems need be fixed.
I will check an updates about this issue.

set, select method duplication problem.
async connection date update method.

was able to fix it by just remove this line  toggle: true

Sorry @feldmarv
I be late for checking the issue.
I think i could answer the issue at tonight 😄

Okay this sounds good to me 👍
Toggle option is making calendar date button to toggle button.
If you disable toggle option,
Calendar will be ordinary calendar.

This issue will be closed tomorrow.
If you have other issues, Please create a new issue 🍺


Gateway timeout for small values

Hello, I have found another small problem in the gateway code.
If you set the gateway timeout values to a value smaller than your sleep interval in the main loop, you will get a timeout error without sending the SDO message at all. The reason for this is that the variable timeDifference_us (value is 50000 or higher because there was nothing to do in the last iteration) is passed to the function 'CO_SDOclientUpload', but no time has passed since the call of CO_SDOclientUploadInitiate. This only happens when the command queue is empty since we fixed the unnessaray wait time in my last issue.
My first thought was to reset the variable timeDifference_us when a read/write command is detected, but I'm not sure if this has any effect on other commands.

I did, as you say in last sentence.
timeDifference_us argument is used later in the function, but in other states. So it is no problem.
I didn't change timeDifference_us for LSS functions, because there it is not a problem.

Thanks again, works great now.


Encrypted userinfo jwt not working

I'm integrating with an identity provider who uses OpenId connect with userinfo in an encrypted jwt.
Unfortunately, OpenIdConnectHandler is unable to handle this specific case. Crashes at: https://github.com/aspnet/Security/blob/beaa2b443d46ef8adaf5c2a89eb475e1893037c2/src/Microsoft.AspNetCore.Authentication.OpenIdConnect/OpenIdConnectHandler.cs#L815-L819 because userInfoEndpointJwt.Payload is null in this case
See also https://github.com/aspnet/Security/pull/517#discussion_r42279158 where @PinpointTownes suggests it will possibly crash but he can't test it versus a server with this behavior.
Since it's allowed in the openid connect spec, the handler should probably be able to handle it.
I managed to make it work by replacing the code above with this:
else if (contentType.MediaType.Equals("application/jwt", StringComparison.OrdinalIgnoreCase))
        {
            var validationParameters = Options.TokenValidationParameters.Clone();
            validationParameters.RequireSignedTokens = false;
            validationParameters.ValidateLifetime = false;
            if (_configuration != null)
            {
                var issuer = new[] { _configuration.Issuer };
                validationParameters.ValidIssuers = validationParameters.ValidIssuers?.Concat(issuer) ?? issuer;
                validationParameters.IssuerSigningKeys = validationParameters.IssuerSigningKeys?.Concat(_configuration.SigningKeys)
                                                         ?? _configuration.SigningKeys;
            }

            var princip = Options.SecurityTokenValidator.ValidateToken(userInfoResponse, validationParameters, out SecurityToken validatedToken);
            var userInfoJwt = validatedToken as JwtSecurityToken;
            user = JObject.FromObject(userInfoJwt.Payload);
            userInfoResponse = new JwtSecurityTokenHandler().WriteToken(userInfoJwt.InnerToken);
}

But this is just an ugly workaround for this specific provider, I'm sure someone can come up with a more generic implementation which fixes both cases. (The reason I'm replacing the userInfoResponse is because later in the method the userInfoResponse is being validated and this method ALSO assumes it's unencrypted)

See also #517 (comment) where @PinpointTownes suggests it will possibly crash but he can't test it versus a server with this behavior.

Glad to know I was right :rofl:
More seriously: another workaround may consist in disabling the built-in userinfo retrieval support and doing it yourself using the TokenValidated event.

Should I really be writing workarounds when integrating with a party who follows the specification? :wink:
My goal is to improve the current implementation so it supports more variants of openid connect out of the box, and hopefully this issue can help with that.

Should I really be writing workarounds when integrating with a party who follows the specification?

Workarounds are not long-term fixes, they're just meant to unblock you until a proper fix is released :smile:
Fixing the OIDC handler is certainly the right thing to do, but you'll have to wait for 2.2 to ship to benefit from your fix :smile:

@brentschmaltz - do  you have a tracking item for supporting this in IdentityModel?

Got it; we'll evaluate what's needed in ASP.NET Core.

What identity servers provide encrypted tokens that we can test against?

I'm afraid the one I encountered is private and I'm unable to share it for
testing.
Maybe someone can setup a IdentityServer4 repo PoC which encrypts userinfo
tokens?
On Wed, Jul 18, 2018, 23:03 Chris Ross<EMAIL_ADDRESS>wrote:

What identity servers provide encrypted tokens that we can test against?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Security/issues/1810#issuecomment-406073607,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEt_B4ClGOlbArz8zgDW2NZuf5TBXpcks5uH6KNgaJpZM4VLC4x
.

@Tratcher http://testingsts.azurewebsites.net/ has a discovery endpoint and will return jwe.

Thanks for the info, everyone. We will consider this in the 3.0 release.

@Eilon when is 3.0 planned for release?

@brentschmaltz looks like next year (definitely not this year).


Representing metadata and conditionally encoding text for a specific task

Thanks for the work on Instructor and making it accessible via open source. I enjoyed reading your research work!
I prototyped the implementation of a book and it works fine. I'd like to extend it to several books and so have a few questions related to it.

What's the best way to tag metadata related to the chunks that are embedded?

Text from an arbitrary page will be on a specific page, specific chapter and section (also source language of the text). This sort of metadata doesn't fall into any of the proposed attributes (text type, text objective, domain).
One possibility is that I can concatenate the metadata along with each chunk as
Chapter: NAME
Page number: NUMBER
Section: NAME
chunk text

Is it fair to assume that each chunk is embedded for all 70 tasks? If so, can I conditionally encode it for a specific task type (ex: instruction for retrieval) thereby bringing down the compute time?

Hi, Thanks a lot for your interest in the INSTRUCTOR!

The page number does not seem to provide meaningful information. In this case, you may try to transfer the chapter name and section name into domain descriptions.
Yes, you only need to encode a text chunk for a specific task with customized instructions.

Thanks for your response. Makes sense.


Devcenter mgmt 2023 04 DRAFT

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.DevCenter


chore: fix documentation info section version in rest api, fixes RHOAIENG-2889

Description

Change version field in info in rest api
fixes RHOAIENG-2889
How Has This Been Tested?

Merge criteria:

[X] The commits and have meaningful messages; the author will squash them after approval or will ask to merge with squash.
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has manually tested the changes and verified that the changes work

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

Comparison is base (44ce399) 76.42% compared to head (d32ceb0) 81.57%.

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

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #296      +/-   ##
==========================================
+ Coverage   76.42%   81.57%   +5.14%
==========================================
  Files          17       12       -5
  Lines        1909      597    -1312
  Branches       73       73
==========================================
- Hits         1459      487     -972
+ Misses        276       87     -189
+ Partials      174       23     -151

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


Babel plugin is dropping keys from object types.

This is a:

[x] Bug Report
[ ] Feature Request
[ ] Question
[ ] Other

Which concerns:

[x] flow-runtime
[x] babel-plugin-flow-runtime
[ ] flow-runtime-validators
[ ] flow-runtime-mobx
[ ] flow-config-parser
[ ] The documentation website

What is the current behaviour?
I have a type which looks like this:
type ODataAST = {|
  '$path'?: $path,
  '$select'?: $select,
  '$expand'?: $expand,
  '$filter'?: $filter,
  '$orderby'?: $orderby,
  '$callback'?: string,
  '$format'?: string,
  '$search'?: string,
  '$count'?: boolean,
  '$skip'?: number,
  '$top'?: number
|}

When I add flow-runtime validations using the babel plugin I end up with a bunch of RuntimeTypeErrors that look like this:
RuntimeTypeError: Return Type must be one of: {|
  $path?: $path | $select | $expand | $filter | $orderby | string | string | string | boolean | number | number;
|}

Expected: {|
  $path?: $path | $select | $expand | $filter | $orderby | string | string | string | boolean | number | number;
|}

Actual: {
  $select: Array<string>;
}

Notice that the the error message claims that the type only has a single key $path and all of the other keys that should be there have instead had their types joined together in a union and put onto $path.

What is the expected behaviour?
I would expect babel-plugin-flow-runtime to correctly interpret the following type:
type ODataAST = {|
  '$path'?: $path,
  '$select'?: $select,
  '$expand'?: $expand,
  '$filter'?: $filter,
  '$orderby'?: $orderby,
  '$callback'?: string,
  '$format'?: string,
  '$search'?: string,
  '$count'?: boolean,
  '$skip'?: number,
  '$top'?: number
|}

Which package versions are you using?
{
  "babel-core": "^6.25.0",
  "babel-plugin-flow-runtime": "^0.11.1",
  "babel-plugin-syntax-flow": "^6.18.0",
  "babel-preset-env": "^1.6.0",
  "flow-bin": "^0.50.0"
}

It turns out that if I remove the quotes around the keys of the object type then everything works. So it still seems like a bug, but it has a proper workaround at least.
type ODataAST = {|
  $path?: $path,
  $select?: $select,
  $expand?: $expand,
  $filter?: $filter,
  $orderby?: $orderby,
  $callback?: string,
  $format?: string,
  $search?: string,
  $count?: boolean,
  $skip?: number,
  $top?: number
|}

That type works and produces the correct flow-runtime validations.

This ended up being a bug with Babylon. Fixed in next release.


CNCF TAG Runtime and/or Working Group AI Discussion/Presentation

Hello Ollama maintainers,
We at CNCF TAG Runtime which is the Technical Advisory Group for workload execution on cloud native infrastructure in the Cloud Native Computing Foundation are interested to explore the space of intersection between AI and cloud native technologies.
As the Tech Lead of CNCF TAG Runtime, I am reaching out to you to present/discuss Ollama at one of our meetings.
We have slots available on:

the TAG Runtime biweekly meeting
and the CNCF Working Group AI meeting

It would be great to learn about the overview of ollama and get to know what lies on your roadmap. We would also be interested in knowing how AI can benefit Cloud Native and vice versa.
If you are interested, please feel free to add it to the agenda or reach out to us in the #tag-runtime channel on the CNCF slack.
Thanks!
cc: @raravena80 @nikhita
cc: @jmorganca if you or anyone else on the team are interested! :)

Going to go ahead and close this. :-D

There was no presentation yet.  Are you interested in presenting?  Ty!

@pdevine we would love to have ollama present at TAG Runtime meeting. Please let us know if you need any help or have any interest to present


evdev to read the joystick

https://gist.github.com/peterhellberg/5a1f66927972587ea3aef24375a1a89d via @peterhellberg

We should use a pure Go implementation of evdev, I'm going to see if I can adapt some of the existing implementations (and only support arrow keys + enter)

I can probably work on this issue later this week. Do you want me to send PRs or can I commit directly on master?

I’m hacking on this issue today :) Will send a PR later today or tomorrow.

We should probably bump the Go version to 1.7.1 on Travis-CI


Error on transferring Python 3.6 to custom image

Hi
running through a test deployment after setting up AWS, and am finding this:
INFO : Copy files to local lambda/
INFO : Copy file: python/lambda_get.py to lambda/
INFO : Copy file: python/lambda_post.py to lambda/
INFO : Copy file: example.R to lambda/
INFO : Remove EC2 directory ~/rtest-aws
INFO : Copying local lambda/ to EC2 ~/rtest-aws ...
.gitignore                                                                                100%   84     2.8KB/s   00:00
example.R                                                                                100% 1281    42.3KB/s   00:00
lambda_get.py                                                                                100%  672    22.2KB/s   00:00
lambda_post.py                                                                                100%  673    22.3KB/s   00:00
README.md                                                                                100%  267     8.8KB/s   00:00
INFO : Configure EC2, create Lambda package/function & API method ...
INFO : Configure AWS on EC2
INFO : Check AWS configuration on EC2 ...
INFO : AWS Account ID: ********3729
INFO : Lambda Function Name: rtest-aws-master-alpha-resource-v1-get
INFO : Creating deployment package
INFO : PWD: /home/ec2-user/rtest-aws
INFO : Transferring Python 3.6 packages to the deployment package
bash: line 392: /home/ec2-user/env/bin/activate: No such file or directory
ERROR: Cannot create Lambda package/function & API method. Terminating end exiting ...

one thought is that perhaps my custom ami is not right, or perhaps a bug somewhere?

The error seems to be related to the installation of Python 3.6 and virtualenv on the custom AMI (assuming you are using one). Could you please try recreating the custom AMI?
Could you please

Hi
I tried that, but it seems to do the same thing:
I ran 22_setup_custom_ami.sh
IT produced a new AMI id, with the result from teh script being (Ive included the entire output for completeness)
bash scripts/22_setup_custom_ami.sh
INFO : Checking project directories
INFO : Loading default settings and secrets
INFO : Starting an AWS t2.micro Instance from AMI ID ami-30041c53 ...
INFO : AWS EC2 Instance ID is: i-02cda8fcc8e307b5f
INFO : Waiting for the AWS EC2 Instance to initialize ...
INFO : Check # 1. Trying again in 5 seconds ...
INFO : Check # 2. Trying again in 5 seconds ...
INFO : Check # 3. Trying again in 5 seconds ...
INFO : Instance i-02cda8fcc8e307b5f is running, its address is ec2-13-211-178-232.ap-southeast-2.compute.amazonaws.com
INFO : Waiting for the SSH Server to start ...
ssh: connect to host ec2-13-211-178-232.ap-southeast-2.compute.amazonaws.com port 22: Connection refused
INFO : Check # 4. Trying again in 5 seconds ...
ssh: connect to host ec2-13-211-178-232.ap-southeast-2.compute.amazonaws.com port 22: Connection refused
INFO : Check # 5. Trying again in 5 seconds ...
ssh: connect to host ec2-13-211-178-232.ap-southeast-2.compute.amazonaws.com port 22: Connection refused
INFO : Check # 6. Trying again in 5 seconds ...
ssh: connect to host ec2-13-211-178-232.ap-southeast-2.compute.amazonaws.com port 22: Connection refused
INFO : Check # 7. Trying again in 5 seconds ...
Warning: Permanently added 'ec2-13-211-178-232.ap-southeast-2.compute.amazonaws.com,<IP_ADDRESS>' (ECDSA) to the list of known hosts.
ec2-user
INFO : Can connect to ec2-13-211-178-232.ap-southeast-2.compute.amazonaws.com
INFO : Using Custom AMI ami-30041c53, skip updating.
INFO : Attempting to stop EC2 Instance ID i-02cda8fcc8e307b5f ...
---------------------------
|      StopInstances      |
+-------------------------+
||   StoppingInstances   ||
|+-----------------------+|
||      InstanceId       ||
|+-----------------------+|
||  i-02cda8fcc8e307b5f  ||
|+-----------------------+|
|||    CurrentState     |||
||+-------+-------------+||
||| Code  |    Name     |||
||+-------+-------------+||
|||  64   |  stopping   |||
||+-------+-------------+||
|||    PreviousState    |||
||+--------+------------+||
|||  Code  |   Name     |||
||+--------+------------+||
|||  16    |  running   |||
||+--------+------------+||
INFO : Instance i-02cda8fcc8e307b5f is being stopped ...
INFO : Waiting for the AWS EC2 Instance to stop ...
INFO : Check # 1. Trying again in 5 seconds ...
INFO : Check # 2. Trying again in 5 seconds ...
INFO : Check # 3. Trying again in 5 seconds ...
INFO : Check # 4. Trying again in 5 seconds ...
INFO : Check # 5. Trying again in 5 seconds ...
INFO : Check # 6. Trying again in 5 seconds ...
INFO : Check # 7. Trying again in 5 seconds ...
INFO : Check # 8. Trying again in 5 seconds ...
INFO : Check # 9. Trying again in 5 seconds ...
INFO : Create custom AMI from Instance ID
INFO : Waiting for AMI ID ami-52cf6830 to be available ...
INFO : Check # 1. Trying again in 5 seconds ...
INFO : Check # 2. Trying again in 5 seconds ...
INFO : Check # 3. Trying again in 5 seconds ...
INFO : Check # 4. Trying again in 5 seconds ...
INFO : Check # 5. Trying again in 5 seconds ...
INFO : Check # 6. Trying again in 5 seconds ...
INFO : Check # 7. Trying again in 5 seconds ...
INFO : Check # 8. Trying again in 5 seconds ...
INFO : Check # 9. Trying again in 5 seconds ...
INFO : Check # 10. Trying again in 5 seconds ...
INFO : Check # 11. Trying again in 5 seconds ...
INFO : Check # 12. Trying again in 5 seconds ...
INFO : Check # 13. Trying again in 5 seconds ...
INFO : Check # 14. Trying again in 5 seconds ...
INFO : Check # 15. Trying again in 5 seconds ...
INFO : Check # 16. Trying again in 5 seconds ...
INFO : Check # 17. Trying again in 5 seconds ...
INFO : Attempting to terminate EC2 Instance ID i-02cda8fcc8e307b5f ...
----------------------------
|    TerminateInstances    |
+--------------------------+
||  TerminatingInstances  ||
|+------------------------+|
||       InstanceId       ||
|+------------------------+|
||  i-02cda8fcc8e307b5f   ||
|+------------------------+|
|||     CurrentState     |||
||+-------+--------------+||
||| Code  |    Name      |||
||+-------+--------------+||
|||  48   |  terminated  |||
||+-------+--------------+||
|||     PreviousState    |||
||+--------+-------------+||
|||  Code  |    Name     |||
||+--------+-------------+||
|||  80    |  stopped    |||
||+--------+-------------+||
INFO : Instance i-02cda8fcc8e307b5f is being terminated ...
INFO : Appending to setup_auto.sh:

# Added on: 2018-07-19 23:10:20 UTC
EC2_CUSTOM_AMI_ID="ami-52cf6830"

I then also deleted the old  EC2_CUSTOM_AMI_ID variable from setup_auto.sh
re-ran 01_main.sh and got the same error
INFO : Using Custom AMI ami-52cf6830, skip updating.
INFO : Copy files to local lambda/
INFO : Copy file: python/lambda_get.py to lambda/
INFO : Copy file: python/lambda_post.py to lambda/
INFO : Copy file: example.R to lambda/
INFO : Remove EC2 directory ~/rtest-aws
INFO : Copying local lambda/ to EC2 ~/rtest-aws ...
.gitignore                                                                                100%   84     2.8KB/s   00:00
example.R                                                                                100% 1281    45.2KB/s   00:00
lambda_get.py                                                                                100%  672    23.4KB/s   00:00
lambda_post.py                                                                                100%  673    24.0KB/s   00:00
README.md                                                                                100%  267     9.3KB/s   00:00
INFO : Configure EC2, create Lambda package/function & API method ...
INFO : Configure AWS on EC2
INFO : Check AWS configuration on EC2 ...
INFO : AWS Account ID: ********3729
INFO : Lambda Function Name: rtest-aws-master-alpha-resource-v1-get
INFO : Creating deployment package
INFO : PWD: /home/ec2-user/rtest-aws
INFO : Transferring Python 3.6 packages to the deployment package
bash: line 390: /home/ec2-user/env/bin/activate: No such file or directory
ERROR: Cannot create Lambda package/function & API method. Terminating end exiting ...```

👍  I am glad that it worked out, I was just about to mention this.


[1.12.2] Autocrafting bug with enchanted books

Issue description:
I tried to setup the Auto-Anvil from Openblocks to combine two enchanted books of the same kind (2x Protection I = Protection II) with a processing pattern. The problem is, if I click on start during the crafting process, nothing happens (the button stays gray and nothing happens)

For the output I used a book (protection II) combined in an anvil, which got the following two nbt tags:
RepairCost:1
StoredEnchantments:[{lvl:2s,id:0s}]
If I use a cheated book without the RepairCost nbt tag the crafting process works fine!
I could reproduce this with only Refined Storage (version 1.5.32) and JEI (version <IP_ADDRESS>) on forge (<IP_ADDRESS>25) installed, by using a chest (as a placeholder) next to the crafter instead of an Auto-Anvil. (for the screenshots I added Actually Additions, but tested it also without it...)
Here you could see the nbt tags of the working recipe (with the cheated book):

And that's the one with the book created in an anvil (with the RepairCost nbt tag):

There is nothing in the log file, but I've uploaded it anyway.
What happens:
Tried to combine two enchanted books, autocrafting doesn't work.
What you expected to happen:
Autocrafting doesn't hang there forever... :smile:
Steps to reproduce:

Build some basic RS system to autocraft
Combine two Protection I books in an Anvil to get a Protection II book (also tested other enchantments, same thing there...)
Set up a processing recipe 2x Prot. I = Prot. II
Try to craft the book

Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: <IP_ADDRESS>25
Refined Storage: 1.5.32

Does this issue occur on a server? maybe (only tested in singleplayer)
If a (crash)log is relevant for this issue, link it here:
https://pastebin.com/FZ7kKLnu
(No crashlog, not really much information... :/)

Don't autocraft with enchanted books, it's not meant for that. RS specifically ignores the enchanted book tag so enchanted books work with other recipes.


Generate typescript variable declarations

Problem 1: ffe-core problematic .ts-files
I recently merged a change to ffe-core which implemented generation of typescript declarations for constants defined in .less-files. This meant that .ts-files were published (including .js and .d.ts files), which in turn lead to confusion for the typescript compiler and lots of error messages.
solution
The solution was to only publish .js and d.ts files, so this PR deletes the .ts files after they are used for type declarations.
Problem 2: generate typescript declaration files for all Icons in  ffe-icons-react
@henrikhermansen suggested that we also add typescript types all the Icon components in ffe-icons-react.
solution
My suggested solution is to replace babel with tsc in order to use that to generate declaration files from the .tsx files generated from the build-script. This could have unintented consequenses as the final transpiled code will be a bit different, but the behaviour should be the same. Another solution I thought about would be to just generate the .d.ts files directly with a template and keep the old babel setup, but I thought that could lead to problems down the line should the component change.

Coverage remained the same at 84.949% when pulling 7480efc0eabcc207b36130e7bec95dc8a5eb8a16 on tutturen:fix-generate-typescript-variable-declarations into 15e6a92badbdc71ad53d9e83697ed5a80a1062a4 on SpareBank1:develop.

Looks good to me! A renaming of build-jsx-components.js to build-tsx-components.js might be in order, though!


Move to std::future::Future

I'm sure you're aware that a new release of rust is imminent with async/await support. Hyper, tokio, and other libraries used by this crate are migrating to std::future::Future to fully support the new syntax. Are there any plans to migrate this crate to std::future::Future? I expect the rest of the ecosystem to migrate to the new trait relatively quickly once async/await support lands on stable in a couple days. Would you be open to a PR that moves to the latest hyper, tokio, and Future?

Yes, I would be very happy about a PR! I have become a bit tired of the constant change so I was happy with the state as it was, but now that the ecosystem becomes more stable this would be very nice to have.

Okay. I've been looking into this. I was hoping it would be a mostly mechanical change, but it is definitely not. A lot of the futures utilities the current implementation uses are no longer available because async/await is a more natural way to solve it, but that means restructuring the code away from the callbacks that exist today. Overall, it should definitely be an improvement to the code quality, but just beware that it's going to be a massive change without any ability to break it into a series of smaller commits.
Additionally, some of the mutexes in the current code are being held while futures are executing. This is asking for trouble and could easily lead to system wide deadlocks. Refactoring the locking to correct that is most likely going to require changing some of the trait definitions. Since this is going to be a major breaking change anyway I'm hoping that's ok.

Okay. I have a working change here:
https://github.com/ggriffiniii/yup-oauth2/commit/e1dca50f9b13d566c2f4e32fefa4d887ec471db7
it pins to the alpha versions of tokio and hyper which is why I didn't just send it as a pull request. Figuring you probably want to wait for those pieces to release.
There are still a lot of remaining things that could be cleaned up. I think there's an unnecessary amount of cloning and things that was probably to workaround lifetime issues with the callbacks. Some of that may involve changing function signatures to accept slightly different types. I'm thinking now would be a good time to try and do all of these things since moving to std futures is already a breaking change.
Here are some of the breaking changes I would propose after looking over the code while making this change.

I would change InstalledFlowMethod::HTTPRedirect to not take a port number instead just do what HTTPRedirectEphemeral does. I can't think of any reason a user would want to specify a port number.
Force all delegates to implement Send + Sync and no longer require them to implement Clone.
Change delegate methods to take &self rather than &mut self.
In general minimize allocations by changing methods to accept slices rather than String and Vec when all they want is to read them.

Do you have a problem with any of the above ideas? I can throw together some of the changes to see how they turn out. If they don't look like an improvement I have no problem abandoning them.

w.r.t. the points

I agree, the current way is for backwards compatibility but it shouldn't be a big issue to break it in a major release
yes
if possible, yes
if possible, yes -- I unfortunately added Clone or non-ref arguments when refactoring for tokio. I didn't have a whole lot of asynchronous rust experience at the time, and despite spending quite some time I didn't manage to figure out something better (and often the compiler errors did sound reasonable)

so, no, I don't have problems with your proposed changes!


PdoStorage missing in compiled sismo.php

While switching from sqlite to mysql due to a database locking issue i noticed, that the provided Sismo/Storage/PdoStorage class was not included in sismo.php
This pull request adds the class to the compilation process.

Good catch, thanks @Tschebel.


How do I exactly download protected videos from Facebook?

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE
Hi,
I have created a ~/.netrc and am passing --netrc switch while trying to download a protected video. However, I am always getting the below.
[facebook] Downloading login page
[facebook] Logging in
[facebook]<PHONE_NUMBER>5802509: Downloading webpage
[facebook]<PHONE_NUMBER>5802509: Downloading webpage
ERROR: This video is only available for registered users. Use --username and --password or --netrc to provide account credentials.

Then I used --username and --password on cli, which throws the same error. I even passed --twofactor, since I have 2FA enabled on my Facebook account.
What else do you suggest I try?
Thanks.
UPDATE: My .netrc file looks like this:
machine facebook login myemail password mypassword

Incomplete how? I thought I followed the template.

@kartagis Did you figure that out? Having same issue.

Actually, I'm not sure if youtube-dl uses 2FA for facebook at all...

Nope, I tried everything I could think of. I tried username, I tried e-mail
address, I tried escaping the exclamation mark in my password; nothing
seems to work.
Tolga
On Thu, 27 Aug 2020 at 23:53, Bartosz<EMAIL_ADDRESS>wrote:

Actually, I'm not sure if youtube-dl uses 2FA for facebook at all...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ytdl-org/youtube-dl/issues/25025#issuecomment-682184987,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMIOXQD6PKOOOBWSTQDVPDSC3BUDANCNFSM4MRW4OXA
.

Any progress on this?

How is this incomplete anyway?

Can you please reopen this issue? It persists. Or at least tell me how this bug report is incomplete.


start server after coordinator server is started in dev mode

Since server is trying to connect to coordinator server during initalization, start coordinator server first.

@maxkozlovsky instead of changing the order let’s make the server keep retrying to reach the coordinator


Fixes usage of -vf string arguments in filter_rotate

I'm not sure why this is failing when using string arguments. But the truth is that I'm not being able to use filter_rotate unless I use '' around the -vf arguments.

Hey @owahab, any news on this. We've been using this in production for a couple of months now and it's working perfectly.


The picture for Performance Counter needs to be updated.

The above picture for Performance Counter needs to be updated as we don't have "Terminal Services Session Lag" today.

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

ID: 9561716a-21fa-fd7c-1807-300a0bc73869
Version Independent ID: 7aed7f1a-3924-1bd2-6cd1-02a94382a392
Content: Use performance counters to diagnose application responsiveness problems on Remote Desktop session hosts
Content Source: WindowsServerDocs/remote/remote-desktop-services/rds-rdsh-performance-counters.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @lizap
Microsoft Alias: wscontent

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Add CI to fftvis repo

This PR adds testing to fftvis

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Create student.txt

https://www.southessex.ac.uk/

@dnlib First stage of verification passed, official domain is southessex.ac.uk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@dnlib @Bran04don Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!


Django debug toolbar errors on 'python manage.py migrate' during installation

What I'm trying to achieve
I was using the installation instructions for Linux here: https://saleor.readthedocs.io/en/latest/gettingstarted/installation-linux.html
I got stuck at step 7 python manage.py migrate with the errors
ImportError: No module named 'debug_toolbar'
and
ImportError: No module named 'ddt_request_history'
Steps to reproduce the problem

Follow the steps here: https://saleor.readthedocs.io/en/latest/gettingstarted/installation-linux.html
Bug occurs when running python manage.py migrate

What I expected to happen
No errors on install.
Proposed Solution
Add
django-debug-toolbar==1.10.1
django-debug-toolbar-request-history==0.0.9

to the requirements.txt
System information
Operating system: Ubuntu 16.04
If you'd like, I can submit a pull request for this issue.

The problem is, we've moved both of the packages into the requirements_dev.txt, while they should stay inside of requirements.txt

Do you want them to be in requirements_dev.txt and requirements.txt or just requirements.txt.
Cool, I'll create a PR in a little while!

I think just the requirements.txt is fine

Maybe just update the docs about pip install requirements_dev.txt if saleor needs running for development?
requirements.txt needs to be installed for production server. Including dev packages may cause security and performance issues.

@ProgrammingJoe Or you could just export DEBUG=False to enable production mode, which does not needs intalling requirements_dev.txt. 😃

Yeah, good point actually. I'll adjust my PR to make that change to the docs and I'll revert the requirements change.

@jxltom @ProgrammingJoe Please move them to regular requirements. They are imported by the codebase even if it depends on a specific environment variable. Dev dependencies should be stuff that never makes it to a server like the ipython console or some linting tools.


Problem with i18n when page is not translated to all target languages

Expected behavior and actual behavior
I have a Middleman site which is translated to 4 languages: en, es, de, fr. i18n config is as follows:
activate :i18n, :langs => [:en, :es, :de, :fr], :mount_at_root => false

For most pages there is a single source file which lives under source/localizable; i18n-specific strings are translated via the <%= t('key') %> helper. This works fine.
For some pages, though, there are different language specific versions.
If a language-specific version is available for all languages, then this works fine as well. Let's assume I have:

source/localizable/test.html.erb
source/localizable/test.es.html.erb
source/localizable/test.fr.html.erb
source/localizable/test.de.html.erb

If I build the site then all works as expected.
However if the language-specific version is missing for some languages, then the "default" version seems to be used for all languages. Let's assume I have:

source/localizable/test.html.erb
source/localizable/test.es.html.erb

When I build the site, I see that the default version is used for French and German (good) but also for Spanish (bad)
Not sure if this is related but in the build log I see:
      create  build/en/test.html
      create  build/es/test.html
      create  build/fr/test.html
      create  build/de/test.html
   identical  build/es/test.html

The Spanish page is being processed twice -- but both times with the wrong content (the content for the default language).
Steps to reproduce the problem (from a clean middleman installation)
See above.
Additional information

Ruby version: ruby 2.6.8p205 (2021-07-07 revision 67951) [arm64-darwin20]
Middleman version: I was using 4.3 but this is also reproducible with 4.4.2
OS version: macOS BigSur 11.6 with Apple M1

@guillerodriguez did you ever figure anything out? We had pages where I18n isn't falling back to using the default translations.

No, unfortunately never managed to figure out what’s wrong..
El lunes, 23 de mayo de 2022, Jarrett Lusso @.***>
escribió:

@guillerodriguez https://github.com/guillerodriguez did you ever figure
anything out? We had pages where I18n isn't falling back to using the
default translations when they are missing.
—
Reply to this email directly, view it on GitHub
https://github.com/middleman/middleman/issues/2529#issuecomment-1134972853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAS67KGZ4D3K5Q2CELMHOWLVLPBJNANCNFSM5NAKXHKQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Guillermo Rodriguez Garcia
@.***

Do not close.

@guillerodriguez is this still valid after latest merged pull requests?

hi @guillerodriguez did you have a chance to check it out 🙏🏼? Thank you so much in advance!

Hello @guillerodriguez,
Could you please 🙏🏼 try against main?
gem "middleman", github: "middleman"


New Waveshare (b) devices use Different CH422G outputs

The new Waveshare revision (B) devices for example esp 4.3(b) use the 8 directional i/o's and also the General purpose Output Pins of the CH422G.
As per CH422G Datasheet :-
The pins OC3-OC0 of CH422 are push-pull or open-drain output pins, and are push-pull output pins by
default.
Waveshare schematic :-

Datasheet Layout:-

On the new revision boards OC0 and OC1 are used as digital Outputs to drive BC817-40's  via PC817X3's
The current library has support for the 8 Directional I/Os but not the OCx Outputs.
Marty

Hi @bondango, sorry for the late. I tried modifying the driver according to the CH422G manual so that it can control the OCx (pin number: 8-11) output level. However, since I don't have the hardware, could you please help me test it? Thank you very much!

Thank

Yes I will try and report hopefully this weekend

I did try a simple 1 second pause loop
expander - digitalWrite(8, HIGH);
expander - digitalWrite(9, HIGH);
delay(500);
expander - digitalWrite(8, LOW);
expander - digitalWrite(9, LOW);
delay(500);
simple light test
power gnd >>>output GND
+12v >>> bulb
blub >>> DI0
But nothing on the DO0 and D01 outputs
inputs are working fine

I have installed this and am using it against a Waveshare s3-4.3LCD board.  This particular board is using the CH422G to also manage which USB port is active (the board sports two). I am receiving USB disconnects when monitoring the application as it transitions the backlight from off to on. I suspect this has to do with the code assuming the initial state of the pins and not reading it into the "cache". I'm trying to instrument the code to validate this.
TBH, I don't understand where the register values have come from. The PDFs of the datasheet I've found don't mention them. In fact, they don't really outline an I2C interface but rather some unusual two wire interactions. Can someone help me source the I2C interface information? I can't find the example program cited in the datasheet either.

Thank you for the testing! Could you please provide your complete code and the serial output logs? It would also be best if you could add expander->printStatus(); in the loop.

the changes made by H-sw123 work fine my end


spo move: unhandled exception when trying to move file from non-existent location, Closes #4537

the function fileExists was called without the await operator which resulted in the bug. With the await operator, the output results in Error: The file /Shared Documents/non-existent-file.zip does not exist.
Closes #4537

Awesome work @nicodecleyre. We will review it ASAP

Awesome find @nicodecleyre 👏

Merged manually.
Bug squashed, thank you!


fix Fix regression case test_nereids_row_policy run in clo…

…ud mode
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Specify which Layout::tab layout to be active

I just want a way to specify which layout in Tabbed layout to be active, Is there any workaround for this?

There is currently no way to specify the active tab element. That would be a nice improvement

There is currently no way to specify the active tab element. That would be a nice improvement


refactor(shell): Write at once rather than in fragments

What does this PR try to resolve?
For cargo add and cargo search, rather than expose termcolors API, we wrote a styled fragment at a time.
This is no longer needed as of #12751 because we switched from termcolor to anstream which decorates stdout and stderr with any of the logic we need to adapt to the configured terminal. .
How should we test and review this PR?
Ran the commands locally to verify styled output.  Tests verify unstyled output.
Additional information

@hi-rustin this is an example of how to style without write_stderr / write_stdout

Looks good. Thanks
@bors r+

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

:hourglass: Testing commit de691b9a02dba95c3ebb4d262480309f83c3b553 with merge f327dbd253665d5f09bb8564dcca98372637bb51...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing f327dbd253665d5f09bb8564dcca98372637bb51 to master...

@hi-rustin this is an example of how to style without write_stderr / write_stdout

Thank you! Got it!


fixed where query without any parameters

changed in PR.

fixed where query without any parameters

i want to query without any parameters.
in my case..
i used firebase database query with optional parameters.
Future<dynamic> listPost({String? writerId}) {
  return _myPostCollection
      .where('writerId', isEqualTo: writerId)
      .get();
}

i will get all post function in "test code" without any parameters.
/// test code
final posts = await postRepositiry.listPost();
expect(posts.isNotEmpty, equals(true));

above code, i will expect get list of post.
but, occurs Unsupported  Error.
final posts = await postRepositiry.listPost(writerId: 'userId-1');

it's working when with parameters.

https://github.com/firebase/flutterfire/pull/11896
updated this way, closed th PR.


error i got and dont know

SyntaxError: Invalid regular expression: /^/_next/data/development/{root).json$/: Unmatched ')'
at new RegExp ()
at buildDataRoute (node_modules\next\src\server\lib\router-utils\build-data-route.ts:32:6)
at page (node_modules\next\src\server\lib\router-utils\setup-dev-bundler.ts:834:39)
Additional context

Nothing in the error you posted that looks like it has anything to do with Highlight.js at all.


Ensure Data Model Matches

Right now the data model doesn't match across labs.  To work around it, we're deleting the data and then reloading it.  Make the data model match everywhere and remove the delete statements as they'll no longer be needed.

Lab 3 - Fix Date to string so filters in Lab 6 work

Data model seems fine for Form 13 now.  Should double check.

I think we're good here.


weekly live

Please use this template to ensure easier processing of your pull request
Category

[x ] Content fix
[ ] New article

Docs Build status updates of commit f8d487c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/accessibility-by-design.md
:white_check_mark:Succeeded
View

Community/glossary.md
:white_check_mark:Succeeded
View

Community/microsoft365-maturity-model--staff-and-training.md
:white_check_mark:Succeeded
View

Community/power-automate-vs-logic-apps.md
:white_check_mark:Succeeded
View

Community/query-string-url-tricks-sharepoint-m365.md
:white_check_mark:Succeeded
View

README.md
:white_check_mark:Succeeded

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


Randomly very low image resolutions in apps

Version
<IP_ADDRESS>9
Apps in dashboard pull in very low resolution images. According to dev it currently scrapes Google images which is the cause.

A suggestion is using TVDB and/or TMDB instead. Could also cache them if you want to go overboard.

For what it's worth, if you want to prioritize other stuff it would be nice to be able to toggle off background image in the app. As a quick fix while it's this low res. This very low res is making the dashboard look very low quality compared to all the other nice things :)

you can just change the size of the app to now show the carousel/images

you can just change the size of the app to now show the carousel/images

Understood but then I miss out on the history.


NXP imx rt1064 evk: Application does not boot when flash/flexSPI driver is enabled

Describe the bug
I am currently trying to build and flash the MCUBoot bootloader to a nxp mimxrt1064 evaluation board. Building and flashing using west works fine without any warnings or error messages but unfortunately the application does not boot. There is nothing on the console and also GDB debugging using a Segger J-Link does not work.
After searching for the reason for a while I noticed that the same behaviour occurs when I try to include the flash driver in other sample applications as well. So I am able to provide a minimal example to reproduce this issue:
To Reproduce
Steps to reproduce the behavior:

Add CONFIG_FLASH=y to samples/hello_world/prj.conf
Build the hello world application (no warnings or errors should show up)
Flash the application to the mimxrt1064_evk
See that the application does not boot and nothing is shown on the console

Expected behavior
Show the zephyr boot banner and boot as usual when including the flash driver.
Impact
I am not able to integrate MCUBoot with Zephyr on my RT1064 evaluation board.
Environment :

OS: Windows
Toolchain: gnu arm embedded
Commit: Current master (2.5.0-rc1)

Zephyr currently doesn't support mcuboot for the i.MX RT1064 internal QSPI flash.
I've submitted this PR that should help you get started.

Hey @hendriklehmkuhl, can you verify again, now that this PR has been merged?
As mentioned, check that you build the bootloader with CONFIG_BOOT_MAX_IMG_SECTORS=512.

Fixed by #33701 and #34845. Please reopen if you still the issue after pulling in these changes.


santa extension error message

after executing
sudo osqueryi --extension /var/osquery/extensions/trailofbits_osquery_extensions.ext --allow_unsafe=true
getting the following error:
osquery> SELECT * FROM santa_denied;
Error: no such table: santa_denied
osquery> SELECT * FROM santa_allowed;
Error: no such table: santa_allowed

but seems like the following worked:
SELECT * FROM santa_rules;
Please provide some guidance on how to troubleshoot. Appreciate it.

@mike-myers-tob I grabbed it from your most recent release.

@mike-myers-tob I grabbed it from your most recent release. https://github.com/trailofbits/osquery-extensions/releases/download/1.1/trailofbits_osquery_extensions-macos-v1.1.zip

I think that's the reason; we need to put up a newer release soon. In the meantime, if you build from source, it will include the new Santa tables.

also getting compile error with CMakeLists.txt
CMakeLists.txt:15: *** missing separator.  Stop.
Whitespace/Tab issue?

Hello @tonynotears
It seems like osquery is being compiled with SKIP_TESTS set to 1 (environment variable); if that's not the case, then the dependencies are missing. Make sure the system has been initialized correctly:
make sysprep
make deps

@alessandrogario  this is the message compling in the folder:
$ make sysprep
make: *** No rule to make target `sysprep'.  Stop.
Tried both in the /santa folder and in the /osquery-extention folder

Hey @tonynotears
The build instructions are located in the README file: https://github.com/trailofbits/osquery-extensions#building
It is similar to how osquery is normally built, except you need to create a symbolic link to our source folder.

now getting this error
TypeError: cannot use a string pattern on a bytes-like object
-- Unable to detect osquery platform: ./tools/get_platform.py failed

Maybe a osquery binary issue? will post that in the #general channel in osquery

This has been fixed with the newly release 1.2 version (https://github.com/trailofbits/osquery-extensions/commit/724123ac19ecd801de379cdfac3b939ca4011001)


add tests and remove all deps!

this really makes me want to get rid of the node dep…!

also we should rename this repo to learn-deno-ssr


flutter doctor issue

I was using flutter 2.2.2 and I upgraded to 2.2.3. I'm unable to run flutter doctor --android licenses. I already tried to solve it using a previous issue about it on github but nothing is working.
This is the output of flutter doctor -v:

This is the output of flutter doctor --android-licenses:

Someone suggested that I download the Android SDK Command-Line Tools (latest) on my Android Studio.
This is the output of flutter doctor --android-licenses after doing this:

I am using Android studio 4.2.1. Is it occurring because of that?

Hi @VishakhMehta,
Thanks for filing the issue, Please remember to share the proper logs in future issues rather than the screenshots. Did you download the Android SDK command-line tools?
if not can you try downloading by following this link , And in case if you have already downloaded it, try to uncheck the

Android SDK command-line tools
Apply (Let it delete)
recheck and apply (redownload)

Yes, I downloaded the Android SDK Command-Line Tools which caused the error as shown in image 3.

Please paste the output of flutter doctor -v.

I've followed the instructions in issue #56778 and it worked.


Duplicate detection enabled topic endpoint Is functioning

Hi,
It is mentioned in this article "Service Bus queues and topics used as IoT Hub endpoints must not have Sessions or Duplicate Detection enabled. If either of those options are enabled, the endpoint appears as Unreachable in the Azure portal.".
However I was able to configure a topic with duplicate detection enabled, set it as a custom endpoint in iothub and still receive messages from iothub. My duplicate detection history time window is 2 mins.
Can someone confirm if this is not a recommended practice for any other reason as what is mentioned in the document is not the case.
Thanks

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

ID: 245b8dbe-fbf6-484f-43a2-64aa315ebe2b
Version Independent ID: d4b99500-0b73-20d3-ebc4-a2d42ade9520
Content: Understand Azure IoT Hub message routing
Content Source: articles/iot-hub/iot-hub-devguide-messages-d2c.md
Service: iot-hub
GitHub Login: @ash2017
Microsoft Alias: asrastog

Thanks for the question! We will review and get back to you shortly.

Please request that they open a support ticket for this. Thanks.

@bulathacl As suggested, Please file a support request @ https://aka.ms/azsupt? If you do not have access to a support plan, please reach out @ AZCommunity at microsoft.com with a link to this Issue as well as your subscription ID and we can help get the support ticket opened for this issue.

Please file a support request as suggested.
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.

@nehsin please investigate.

@AshokPeddakotla-MSFT I looked into the support ticket but there isn't any new information from what I find here. Is the issue already investigated by a SE? If yes, do we have an ICM I can see?

@nehsin No, I can open one right now and will post the ICM ID right after I open it.

@nehsin Here's the ICM number :194464975.


WIP feat(test): improvement of performance --maxWorkers=4

WIP for testing performance of UTs with max 4 workes. Unit tests take approximately 1 min. Times: 01:01, 01:00, 00:59.

Note, CircleCi gives us a max parallelism of 4.  This allows us to split tests up across 4 machines at once.  We can drastically reduce the amount of time for testing if we use this feature.
https://circleci.com/docs/2.0/parallelism-faster-jobs

agreed to start with tweaking jest settings first (i.e. --max-degree-of-parallelism), will make changes to CI infrastructure's config by means of separate PR.


[Bug]: Protobuf emitter incorrect map value type

Describe the bug
The protobuf emitter drops the package prefix from the map value type for maps that use an imported model as the value.
message MyLocal {
  map<string, Dummy> m = 1;
}

service MyServices {
  rpc Echo(MyLocal) returns (myshared.Dummy);
}

Reproduction
https://typespec.io/playground?c=aW1wb3J0ICJAdHlwZXNwZWMvcHJvdG9idWYiOw0KDQp1c2luZyBUeXBlU3BlYy5Qxx3FHEBwYWNrYWdlKHsNCiAgbmFtZTogIm15c2hhcmVkIiwNCn0pDQrEF3NwYWNlIE15U8UaIMUvbW9kZWwgRHVtbXnGESAgQGZpZWxkKDEpIHg6IHN0cmluZzvEGn0NCn3ccGVydmljZdVxxhvMck15TG9jYWzSdG06IE1hcDzGeCzpAK4u5QCjPucAiMQHQOgBBS7HesQVaW50ZXJmzXZzyGRlY2hvKC4uLsd7KTrPXesA5Q%3D%3D&e=%40typespec%2Fprotobuf&options={}
Checklist

[X] Follow our Code of Conduct
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

Thank you for reporting this issue. The same logic that applies to return types and property types for resolving cross-package references was missed in the calculation of map value types. The following PR will resolve this issue: https://github.com/microsoft/typespec/pull/3561


Unable to use filereader.readAsText

Description
Hi! I've been toying around with exokit for awhile and like it so far. Here's is a thing I found during messing with it.
In the exokit runtime, while being able to create a filereader, it won't allow me to run the following code:
let fs = new FileReader()
fs.addEventListener('load', e=> {

    processing logic here...

    console.log(e)}
)

fs.readAsText()

The error is:

TypeError: fs.readAsText is not a function

Configuration
exokit: 0.0.495 (installed via CLI)
os: ubuntu 18.04 64bit
content: custom aframe-website

Thanks, looks like the JS just doesn't implement it here: https://github.com/webmixedreality/exokit/blob/5f15e7a39ee23139005940523928a38baf18b6a5/src/core.js#L579

@modulesio  Thx for your reply. This makes sense now.
I was curious to know whether exokit is built on-top of node or something else, since node didn't support FileReader api natively either.
Are there plans to be compatible to the browser spec? Since I assume exokit is more like a intermediate layer for contents written directly for webvr(which is targeting browsers mostly)
If no, It would be nice to know it in the docs.
Glad to know it is indeed implemented in a different way. Guess this is for performance reasons?
Once again, thx for making this wonderful project.


Create libvirt xml files from templates with python

Great work, but please cleanly separate the CLI code from the library code like I suggested.

thanks for all the input :) i will continue after lunch

Great work, but please cleanly separate the CLI code from the library code like I suggested.

@aplanas @aspiers
I also had the intention to write this more modular, hence not one CLI for each "function", but I was really not sure how.
e.g. how I should pass all the computed variables that we need from mkcloud, how should I call it from within mkcloud and so on. I was even thinking about one central script with subcommands for each function but I don't know what makes most sense in the long term (and longer... if  we go beyond libvirt with mkcloud)
I would definitely need some more help on that from the python side, due to lack of expertise.
So my suggestion would be doing this in a separate PR (maybe with some dedicated discussion of the python experts about the architecture), as it affects also the python scripts that have already been merged. This here is just another small step in the refactoring of mkcloud. What do you suggest?

@MaximilianMeister I see.
So far the logic behind this code is not big. Initially I would put all the functions in a single Python package, and create several CLI (as you did) that parse the args and call the appropriate (and slim) library.
This approach can be used if mkcloud is finally written in Bash or Ruby.  But if is done in Python we will need to refactor the library into several new packages with coupled functionality, because it will grow up a lot : )
Do we have a final decision about mkcloud?

Do we have a final decision about mkcloud?

@aplanas No, we don't. And this is IMHO not related. The overall goal is to split mkcloud into separate tools (not just libraries). It should become separate tools that can be used individually, and mkcloud is using all of them.
Once mkcloud is small enough we can do a rewrite of it. I see python as the most likely option here. Once mkcloud itself is python it can use the librarlies of the other tools directly. Until then it will call the scripts (ideally with subcommands, and not one script per function).

@jdsn @MaximilianMeister I can help to refactor this to create subcommands.  I will do it after the merge of this PR.
Can we provide some test file for this PR, so we can start the refactoring in a good way?

I will try to setup some basic unittests as well for now. And maybe extract some of the logic in each CLI into the libvirt_helpers "module", which will be the base for the unittests.

Great work, but please cleanly separate the CLI code from the library code like I suggested.

@aplanas @aspiers
I also had the intention to write this more modular, hence not one CLI for each "function", but I was really not sure how.

No problem, but since then I explained how you can do it :-)

e.g. how I should pass all the computed variables that we need from mkcloud, how should I call it from within mkcloud and so on.

The bash -> python interface you have done is already good.  You only need to change the scripts so they're also usable as modules.

I was even thinking about one central script with subcommands for each function but I don't know what makes most sense in the long term (and longer... if  we go beyond libvirt with mkcloud)

Big -1 for a single script.  Code should always be in small, clean chunks, where each chunk does one thing really well.
I think you're too focused on the CLI aspect.  Sure we can support a CLI as long as we need one, but the focus should be on writing reusable, testable, maintainable code, and the right way to do that is to ensure clean separation of concerns into separate modules, and to keep the CLI code separate from the business logic.

I would definitely need some more help on that from the python side, due to lack of expertise.
So my suggestion would be doing this in a separate PR (maybe with some dedicated discussion of the python experts about the architecture), as it affects also the python scripts that have already been merged. This here is just another small step in the refactoring of mkcloud. What do you suggest?

I suggest:

make each script reusable as a module by separating the CLI code from the business logic
later, move each script into a new module namespace which groups them all together, and move the CLI code into a single script which provides subcommands which delegate to each submodule in that namespace.

@aplanas We have some unittests now :-)
Does it make sense to unittest the vm + network creation + cleanup as well? is it possible to start VM's in the Travis VM?

I suggest:

make each script reusable as a module by separating the CLI code from the business logic

@aspiers done.
Please have another look. Thank you!

:+1: successfully tested with initial version of mkcloud self-gating (currently manually triggered)

codewise :+1:

Retriggering Travis Build: closing and reopening

:+1: but waiting for results of final test run

I would suggest making the code more modular as Adam did. I would like to note that keeping the test cases in the same folder is a bad idea. May be after splitting mkcloud into multiple repositories, the test cases can lie in their special tests/ folder (but suggest to do it sooner than later).
Some hard coding in the XML files, and some other minor issues which may be carried out in following patches. Looks good to me. +1


Signals Dashboard UI

v2 design for dashboard
Overview tab

Signal status tab/ zero-state: Select a signal to load data

Signal status tab / signal selected w/ charts loaded

Link to figma designs
https://www.figma.com/file/IiYu74VmEi4GVS4bnYaTd6/Delphi-Site-Redesign?node-id=2700%3A0
Link to proto
https://www.figma.com/proto/IiYu74VmEi4GVS4bnYaTd6/Delphi-Site-Redesign?node-id=2700%3A0&scaling=min-zoom

Link to signals dash deck: https://docs.google.com/presentation/d/1k61NFrHFzd91t1L8CZapwqzcG1iX4ir3lp-UqS6kVMo/edit#slide=id.p

closing since released


JAVA-1824: Make policies overridable in profiles

For LBP, retry and speculative execution policy, and timestamp generators.
Profiles can have separate configuration. The driver traverses all profiles at startup, and initializes all distinct policies.

Timestamp generation is a bit of a stretch, but I decided to include it because it was cheap once the mechanism was in place.
Throttling and query logging fall into the same category, they can definitely be added as well.
But let's not forget that you can always write a custom policy that tests the request's profile name too, so we're completely closing the door if we don't do it. Maybe we should stick to just LBP, retries and specex.
As for the metrics, I find the idea compelling on paper, but indeed it adds a dimension in the metric space and will probably complicate things. I'll take a look.

My opinion on the additional options ( throttling, query logging, and metrics ) - I would like to keep this to the minimal set at first ( I also question whether we need timestamp generation here ). We can always add these later if we see use cases requiring them.

I added a couple integration tests and also plan on adding some more validation on the speculative execution tests and a LBP specific test soon.

Rebased and force-pushed.
The Travis build fails because we depend on a native-protocol snapshot since JAVA-1592, but I've run the full integration suite locally.
The last commit passes profile information to the metric updaters when relevant/practical. The default updaters ignore it at this point, but that leaves a door open if we decide to do per-profile metrics later.


[macOS, sandbox] Implement optional native file selection dialog support for sandboxed apps.

Alternative to #47056, implements https://github.com/godotengine/godot-proposals/issues/1123 on macOS.
Adds ability to access arbitrary folders for macOS sandboxed apps (Entitlements → App Sandbox → Enabled is selected during export).

Adds files.user-selected entitlement support to the export dialog.
Adds OS.is_sandboxed() to detect running in the macOS sandbox.
Adds DisplayServer.file_dialog_show() to display native file selection dialog. Which also saves security-scoped bookmarks for selected files/folders, to automatically gain access in the next session.
Adds FileDialog.use_native_dialog property to seamlessly integrate native dialog to existing apps, disabled by default, used automatically when sandboxed app is accessing file system.
Implements OS_OSX::get_granted_permissions() to get list of accessible folders/files.
Adds OS.revoke_granted_permissions() to clear list of accessible folders/files.
Can be used in non-sandboxed apps.

@bruvzg This needs a rebase and addressing Remi's documentation notes, please 🙏

Thanks!


[Doc] Update main page + add some modules

This PR reworks the main page of the documentation and adds/update some modules
Main page update:

Add the abstract of the README
Add overview page with links to project resources
Add Tutorials placeholder page
Add ChangeLog and License pages

Modules added to documentation: testutils, resources, gui.testutils.
Some changes to installation steps and gui.icons page.
Fix requirements.txt

It looks good to me.

Added 'getting started' for plot + ImageView widget doc


Edit Comments

As a User I want to edit the comments i make so that I can correct errors i may have made or change my mind about an art piece.
Acceptance Criteria :

[ ]
[ ]
[ ]

completed


Update the pcluster site config to use pcluster 3.9.3

As detailed in https://github.com/JCSDA-internal/jedi-tools/issues/372 we need to update the parallel cluster from 3.7.1 to 3.9.3. I've built a new site-config and updated the sections in the readme and the site-config assets for this new cluster.
Because this pull request only edited the site configs it was affected by the same test exclusion block noted in https://github.com/JCSDA/spack-stack/pull/1208 - I've moved the changes discussed there into this pull request ensuring that all required tests run.

There is a readthedocs build error - https://readthedocs.org/projects/spack-stack/builds/25158270/

I don't think it's related to this PR:
Running Sphinx v8.0.2
loading translations [en]... done
making output directory... done
Converting `source_suffix = '.rst'` to `source_suffix = {'.rst': 'restructuredtext'}`.
ERROR: Invalid value `None` in intersphinx_mapping['https://docs.python.org/']. Expected a two-element tuple or list.

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/spack-stack/envs/1210/lib/python3.11/site-packages/sphinx/cmd/build.py", line 332, in build_main
    app = Sphinx(args.sourcedir, args.confdir, args.outputdir,
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/docs/checkouts/readthedocs.org/user_builds/spack-stack/envs/1210/lib/python3.11/site-packages/sphinx/application.py", line 277, in __init__
    self.events.emit('config-inited', self.config)
  File "/home/docs/checkouts/readthedocs.org/user_builds/spack-stack/envs/1210/lib/python3.11/site-packages/sphinx/events.py", line 100, in emit
    results.append(listener.handler(self.app, *args))
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/docs/checkouts/readthedocs.org/user_builds/spack-stack/envs/1210/lib/python3.11/site-packages/sphinx/ext/intersphinx/_load.py", line 127, in validate_intersphinx_mapping
    raise ConfigError(msg)
sphinx.errors.ConfigError: Invalid `intersphinx_mapping` configuration (1 error).

Configuration error:
Invalid `intersphinx_mapping` configuration (1 error).

But it seems that https://github.com/JCSDA/spack-stack/blob/develop/doc/source/conf.py#L203 should be changed to:
intersphinx_mapping = {'python': ('https://docs.python.org', None)}


[mui-system][style] bug fix for style value check color in nullable object

Description
In PR #39071, we added checks to getStyleValue to throw a warning in development mode to double check if the prop value resolved from themeMapping was intentionally object. This was to ensure that setting object values to css such as color does not fail silently. But, the warning was also thrown on setting object values when necessary such as typography to body1 which quickly led to spam a lot of warnings.
Changes Made:

Added a function to check if a given string is a colour in any of the standard color styles (hex, rgb, rgba, var, scss).
Updated the getStyleValue function to check if the resolved value is non-null object and only throw warning if the object immediately contains color values.
Updated all tests from the previous PR to better fit the new getStyleValue.

Testing:
I have updated all relevant tests from the original PR, style.test.js, palette.test.js, styleFunctionsSx.test.js and Typography.test.js file to ensure that the changes work as expected. The tests include checking that the warning is only thrown on objects which immediately contain colours.

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

Closes #39399

Should we reopen https://github.com/mui/material-ui/issues/38478?

@DiegoAndai I do think we need to reopen but as of now I haven't been able to find a better way to solve it.


PHPLIB-1056: Add missing types in reference doc

Fix PHPLIB-1056
I checked all the methods to make sure that all the arguments have a defined. I used the syntax of the latest PHP versions but that's not a problem for the doc.

I can't merge because the psalm and phpcs jobs did not run.


Button lighten on hover when default color is changed

If the default color is changed on a button, then the color no longer "lightens" on mouse hover.
For example with this button,
Button

the color ligthens on hover. But after changing the color to red
Button

the button stays the same color on hover.
It isn't obvious to me how to get that effect back for different button colors. Is it possible to have the buttons always "lighten" regardless of the background color?

+1
Perhaps the best way to do this is to add the hover effects inside the color classes themselves (Such as .red:hover ).

@brandonSc did you resolve the problem?
Closed due inactivity, feel free to reopen it if it is still necessary.

This is an actual problem

I worked a little on it few months ago.
I got no more time now, maybe it could be helpful


problem in linux (ubuntu)

in linux
open an empty devtools tab in vscode after choose target

+1 I am having the same issues on Kubuntu 20.04

Same issue here, also on Kubuntu 20.04. Tried running with a clean instance of Chrome and no extensions installed, also tried using the --disable-extensions flag. Neither produced any results, just an empty devtools tab. It does seem to be recognizing the url, as a false url does produce an error message, there just isn't anything rendering in the view.


add rules for blog

@Satnam14 is this all you had to do?

I'm not sure yet. I'm still figuring some stuff on the blog side. But since we're gonna need to do this regardless I wanted to give this a shot on the staging website-cms in the meantime

got it 👍