[BUG] ReactiveCommand throws exception in UWP app when activated through sharing

Describe the bug
In a UWP app, if the app is activated through receive data api the ReactiveCommand invocation raises exception
Steps To Reproduce
I am providing a bare minimum app that reproduces this problem.
ReactiveUITestApp1 code

Start the Debugging the app in visual studio
Using firefox or edge browser, click on the share menu
Select ReactiveUITestApp1 from the list of apps
From the new application window, click the Complete button
After the command's Action is executed an exception is thrown

Expected behavior
There should not be an exception or there should be a way to safely invoke the command
Environment

OS:       Windows 10
Version   1903

Additional context
The project target was set to 1903 with min target of 1809. The ReactiveUI package 9.15.1 was used.
If I remove the command creation in MainViewModel in the main UI of the app, I do not get the exception.
Others have faced similar exception whilst sharing/receiving in UWP, here are some links to provide some pointers why it might be happening
https://stackoverflow.com/a/18643977/1139856
https://github.com/luberda-molinet/FFImageLoading/issues/166
Exception Stacktrace
| Name | Value | Type
-- | -- | -- | --
▶ | Exception | {System.Exception: A COM call to an ASTA was blocked because the call chain originated in or passed through another ASTA. This call pattern is deadlock-prone and disallowed by apartment call control.  A COM call (IID: {C50898F6-C536-5F47-8583-8B2C2438A13B}, method index: 3) to an ASTA (thread 12132) was blocked because the call chain originated in or passed through another ASTA (thread 22768). This call pattern is deadlock-prone and disallowed by apartment call control.    at System.Runtime.InteropServices.WindowsRuntime.ICommandAdapterHelpers.<>c__DisplayClass1_0.b__0(Object sender, EventArgs e)    at ReactiveUI.ReactiveCommandBase2.OnCanExecuteChanged() in D:\a\1\s\src\ReactiveUI\ReactiveCommand\ReactiveCommandBase.cs:line 195    at ReactiveUI.ReactiveCommand2.<.ctor>b__9_6(Boolean ) in D:\a\1\s\src\ReactiveUI\ReactiveCommand\ReactiveCommand.cs:line 671    at System.Reactive.AnonymousSafeObserver1.OnNext(T value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\AnonymousSafeObserver.cs:line 45    at System.Reactive.Linq.ObservableImpl.RefCount1..OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\RefCount.cs:line 65    at System.Reactive.Subjects.FastImmediateObserver1.EnsureActive(Int32 count) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Subjects\ReplaySubject.cs:line 850    at System.Reactive.Subjects.FastImmediateObserver1.EnsureActive() in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Subjects\ReplaySubject.cs:line 752    at System.Reactive.Subjects.ReplaySubject1.ReplayBase.OnNext(T value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Subjects\ReplaySubject.cs:line 271    at System.Reactive.Subjects.ReplaySubject1.OnNext(T value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Subjects\ReplaySubject.cs:line 167    at System.Reactive.Linq.ObservableImpl.AsObservable1._.OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\AsObservable.cs:line 31    at System.Reactive.Linq.ObservableImpl.DistinctUntilChanged2..OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\DistinctUntilChanged.cs:line 77    at System.Reactive.Linq.ObservableImpl.CombineLatest3._.SecondObserver.OnNext(TSecond value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\CombineLatest.cs:line 180    at System.Reactive.Linq.ObservableImpl.RefCount1..OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\RefCount.cs:line 65    at System.Reactive.Subjects.FastImmediateObserver1.EnsureActive(Int32 count) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Subjects\ReplaySubject.cs:line 850    at System.Reactive.Subjects.FastImmediateObserver1.EnsureActive() in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Subjects\ReplaySubject.cs:line 752    at System.Reactive.Subjects.ReplaySubject1.ReplayBase.OnNext(T value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Subjects\ReplaySubject.cs:line 271    at System.Reactive.Subjects.ReplaySubject1.OnNext(T value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Subjects\ReplaySubject.cs:line 167    at System.Reactive.Linq.ObservableImpl.AsObservable1._.OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\AsObservable.cs:line 31    at System.Reactive.Linq.ObservableImpl.DistinctUntilChanged2..OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\DistinctUntilChanged.cs:line 77    at System.Reactive.Linq.ObservableImpl.Concat1._.OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\Concat.cs:line 33    at System.Reactive.Linq.ObservableImpl.Select2.Selector..OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\Select.cs:line 48    at System.Reactive.Linq.ObservableImpl.Scan2._.OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Linq\Observable\Scan.cs:line 49    at System.Reactive.SafeObserver1.OnNext(TSource value) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Internal\SafeObserver.cs:line 40    at System.Reactive.ScheduledObserver1.Run(Object state, Action1 recurse) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Internal\ScheduledObserver.cs:line 254    at System.Reactive.Concurrency.Scheduler.<>c__DisplayClass49_01.<InvokeRec1>b__0(TState state1) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Concurrency\Scheduler.Recursive.cs:line 55    at System.Reactive.Concurrency.Scheduler.InvokeRec1[TState](IScheduler scheduler, Pair2 pair) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Concurrency\Scheduler.Recursive.cs:line 55    at System.Reactive.Concurrency.CoreDispatcherScheduler.<>c__DisplayClass10_01.<Schedule>b__0() in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Platforms\UWP\Concurrency\CoreDispatcherScheduler.cs:line 99 --- End of stack trace from previous location where exception was thrown ---    at System.Reactive.Concurrency.CoreDispatcherScheduler.<>c__DisplayClass10_21.b__1(Object o, Object e) in D:\a\1\s\Rx.NET\Source\src\System.Reactive\Platforms\UWP\Concurrency\CoreDispatcherScheduler.cs:line 119} | System.Exception

Hmm interesting one. We are a bit backlogged on bugs at the moment so hopefully can get a fix in soon. Happy to take a PR if you're able to earlier too

@glennawatson @harvinders
Here is some more reading on the subject: http://www.dansuleski.com/raising-events-across-windows/
We are seeing this bug as well (and posted about it on the reactiveui slack channel a couple days ago). It is caused by the fact that on UWP each new window gets its own ui thread and the way WaitForDispatcherScheduler (which is the default scheduler for RxApp.MainThreadScheduler) works it is inherently only going to work on a single window app. As @anaisbetts put it in Slack, UWP UI thread situation is a "tire fire" and to quote "ReactiveUI is not designed to handle multiple window apps" - which essentially means no ProjectionManager, no ShareTarget, no ApplicationViewSwitcher, nothing multi-window. While I agree with the first part (and begrudgingly the second part), I am not so sure that RxUI shouldn't be considering these issues.
Anyways. Back to the issue.
This specific exception is caused by work continuing from one ui thread to another and back. Each one of these windows runs its own dispatcher thread and is an “Application Single Threaded Apartment” (ASTA). Essentially the framework is blocking what it thinks is a potential deadlock. This particular exception is thrown when code running under one STA (1) calls out to another STA (2) and that STA (2) might have to re-enter STA (1) in order to do its work. The framework throws that exception at this point to try and stop you getting into a mess with deadlocks.
I am currently testing a scheduler that can handle multiple windows/dispatchers and will hopefully manage this as well, but we are still testing it and I'm not sure if I'll be able to block this exception yet (which would make my new scheduler useless... 👎).

It looks like the exception is thrown because ICommand marshals back to the a COM object for event handling... .NET Reference Source

Bounty Added: BountySource

Any funding requests should go through the open collective. Given we vetted this source of revenue we know it's safe for the rxui organisation to recommend to potential community members to target.

I'll look at your issue free of charge tomorrow for you.

Any funding requests should go through the open collective. Given we vetted this source of revenue we know it's safe for the rxui organisation to recommend to potential community members to target.

Done! 🙌

I was able to get past this problem personally by changing
            CompleteCommand = ReactiveCommand.Create(() => { }, outputScheduler: CoreDispatcherScheduler.Current);

I might make a new issue about the RxApp having a singular access point and support multiple windowing technologies, and close this one since you can use the dispatcher scheduler to correctly. I don't think it's a workaround per say since it's a threading requirement for this problem.


1436 show random fun facts on preloader

Thank you for your contribution to the KodaDot NFT gallery.
👇  _ Do a quick check before the merge.
PR type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[x] I've tested PR on mobile and everything works
[ ] I found edge cases

What's new?

[x] PR closes #1436>
[x] Implemented a textfield that shows on of 32 random fun facts while loading

Had issue bounty label ?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[ ] My fix has changed something on UI, a screenshot for others, is best to understand changes.

Funky 🧙

@kkukelka payout sent https://kusama.subscan.io/extrinsic/0x7a63ea9060fb100f438e9ce9d8d6c38f355816e4a23002e5b59316400a19be7c
Let's make PR to main branch as well as I guess it will take some time to be on beta for while 😄

#1460 seems I've found bug 🪲🐛


Feature Request OracleDatabase - Run scripts on startup.

It would be great to be able to run scripts after the database has started. For example I want to do something mount a volume with:
For example I want to launch the container with a volume that contains scripts, and execute those scripts in the context of the container.
-v ./init:/docker-entrypoint-initdb.d
This mount would contain things like:
12c.init.sql
which would result in echo "exit" | $ORACLE_HOME/bin/sqlplus -s "/ as sysdba" @"/docker-entrypoint-initdb.d/12c.init.sql" being executed in the context of the container.

Can you do that by customizing your own image?

The MySQL container does something similar: https://github.com/mysql/mysql-docker/blob/mysql-server/5.7/docker-entrypoint.sh#L98-L105
I think it's actually a pretty good idea and means the base container is more flexible and doesn't require customisation.

I added a pull request with what I was thinking #389. There are tons of use cases for this functionality. For example I could populate the schema after the database is launched.

@brunoborges Following back up. I sent you an updated OCA.

Thank you @jcustenborder. There are some minor changes that I did request for this PR. Perhaps you can work on them in the meantime?

This has been implemented via #389 and #471. Thanks a lot for the contribution!


Need an example for multiple tests adding up to 100

Hi,
I work with Java and Gradle. I followed the example repository .
I tried it myself, but the Action runs all 3 tests as a single test, 3 times.
So one broken test causes 0 points total.
Can you give an example that works? maybe improve on the example repository.
My assignment:
https://classroom.github.com/classrooms/83712254/assignments/olympics

An update:
I found in Gradle CLI docs that you can specify the test classes:
gradle test --tests MyTestClass
But If there is a compilation problem in ANY of the tests, the score is zero.
Only if all tests compile, each of them get their respective points.
Is there a way to easily fix this, perhaps in Gradle or Actions config?

I'm no Gradle expert, but I had a similar issue in the past and used the sourceSets attribute to exclude all classes that were not required for a given test. For example:
task test_constructor (type: Test) {
	filter.includeTestsMatching "ConstructorTest"
}

// other test tasks here
// ....

sourceSets {
    if (project.gradle.startParameter.taskNames.contains("test_constructor")) {
        test.java.exclude '**/LotsOfGuessesTest*'
        test.java.exclude '**/TestMain*'
        test.java.exclude '**/TestMainLoop*'
        sourceSets.main.java.excludes = ['Main*']
    }
    else if (project.gradle.startParameter.taskNames.contains("test_loser")) {
        test.java.exclude '**/TestMain*'
        test.java.exclude '**/TestMainLoop*'
        sourceSets.main.java.excludes = ['Main*']
    }
    else if (project.gradle.startParameter.taskNames.contains("test_lots_of_guesses")) {
        test.java.exclude '**/TestMain*'
        test.java.exclude '**/TestMainLoop*'
        sourceSets.main.java.excludes = ['Main*']
    }
    else if (project.gradle.startParameter.taskNames.contains("test_main")) {
        test.java.exclude '**/TestMainLoop*'
    }
}

Then, in my autograding.json file I call the tests one by one. To call the "constructor" test from the Gradle snippet above for instance, the test is:
{
  "name": "Test constructors",
  "setup": "",
  "run": "timeout 60 gradle test_constructor",
  "input": "",
  "output": "",
  "comparison": "included",
  "timeout": 1,
  "points": 20
}

I hope this helps.

I'll try that, thanks Steve!
On Thu, Nov 3, 2022 at 9:00 PM Steve Bitner @.***>
wrote:

I'm no Gradle expert, but I had a similar issue in the past and used the
sourceSets attribute to exclude all classes that were not required for a
given test. For example:
task test_constructor (type: Test) {
filter.includeTestsMatching "ConstructorTest"
}
// other test tasks here// ....
sourceSets {
if (project.gradle.startParameter.taskNames.contains("test_constructor")) {
test.java.exclude '/LotsOfGuessesTest*'
test.java.exclude '/TestMain*'
test.java.exclude '/TestMainLoop*'
sourceSets.main.java.excludes = ['Main*']
}
else if (project.gradle.startParameter.taskNames.contains("test_loser")) {
test.java.exclude '/TestMain*'
test.java.exclude '/TestMainLoop*'
sourceSets.main.java.excludes = ['Main*']
}
else if (project.gradle.startParameter.taskNames.contains("test_lots_of_guesses")) {
test.java.exclude '/TestMain*'
test.java.exclude '/TestMainLoop*'
sourceSets.main.java.excludes = ['Main*']
}
else if (project.gradle.startParameter.taskNames.contains("test_main")) {
test.java.exclude '/TestMainLoop*'
}
}
Then, in my autograding.json file I call the tests one by one. To call the
"constructor" test from the Gradle snippet above for instance, the test is:
{
"name": "Test constructors",
"setup": "",
"run": "timeout 60 gradle test_constructor",
"input": "",
"output": "",
"comparison": "included",
"timeout": 1,
"points": 20
}
I hope this helps.
—
Reply to this email directly, view it on GitHub
https://github.com/education/autograding/issues/56#issuecomment-1302544003,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH66V74TLPPDZDL4TUDT6OTWGQDVHANCNFSM542U26ZQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Refs and more

This is to add some basic support for $refs in Json Schema objects.

@geoffsmith82,
Thank you for your great contribution.


Schemagen/external refs

Description
Adds support for mapping types to external schemas and just referencing them instead of generating them.
Links

Resolves #768
Checks

[x] I have read and understand the Code of Conduct and Contribution Guidelines.

Crap.  Forgot to update the version and release notes.


AttributeError: 'brotli.Decompressor' object has no attribute 'unused_data'

Hello, I'm using warcio to read a WARC archive containing brotli encoded HTTP responses like so.
with open(sys.argv[1],'rb') as stream:
    for record in ArchiveIterator(stream):
        if record.rec_type != 'response':
            continue
        print(record.content_stream().read())

This gives me the following error, because there is not attribute unused_data in brotli.
Traceback (most recent call last):
  File "load_warc.py", line 22, in <module>
    print(record.content_stream().read())
  File "/home/sebastian/.asdf/installs/python/3.7.2/lib/python3.7/site-packages/warcio/recordloader.py", line 34, in content_stream
    return ChunkedDataReader(self.raw_stream, decomp_type=encoding)
  File "/home/sebastian/.asdf/installs/python/3.7.2/lib/python3.7/site-packages/warcio/bufferedreaders.py", line 284, in __init__
    super(ChunkedDataReader, self).__init__(stream, **kwargs)
  File "/home/sebastian/.asdf/installs/python/3.7.2/lib/python3.7/site-packages/warcio/bufferedreaders.py", line 72, in __init__
    self._init_decomp(decomp_type)
  File "/home/sebastian/.asdf/installs/python/3.7.2/lib/python3.7/site-packages/warcio/bufferedreaders.py", line 89, in _init_decomp
    self.decompressor = self.DECOMPRESSORS[decomp_type.lower()]()
  File "/home/sebastian/.asdf/installs/python/3.7.2/lib/python3.7/site-packages/warcio/bufferedreaders.py", line 31, in brotli_decompressor
    decomp.unused_data = None
AttributeError: 'brotli.Decompressor' object has no attribute 'unused_data'

What am I missing?

I'm using Google's official brotli package, is this correct?
I did
git clone https://github.com/google/brotli.git
cd brotli
git log -p | grep unused_data
grep -r unused_data

This doesn't give any results, so there is no unused_data in brotli, and there also has never been any. :thinking:

warcio uses the brotli-py library, installable with pip install brotlipy
I think it was available before the python bindings on the official google brotli..
Sorry for any confusion! Perhaps worth looking if it makes sense to switch..

This is a doc problem, this project has no requirements.txt, setup.py doesn't mention brotli because it's optional, the only place the package name is mentioned is in .travis.yml and appveyor.yml

@wumpus you're right, probably a good solution would be toadd brotlipy as an extras_require and also mention in the README

I think this is still an issue? I had to install brotlipy manually to be able to use record.content_stream().read() on br records. I think adding a little something in README would be a good first step :)

This is still an issue, currently failing on this, resolved by pip install brotlipy.

fixed in PR https://github.com/webrecorder/warcio/pull/179, not sure when this will appear on pypi


Remove unused submodule support

The CI passes and the commits are short/few. I think this is safe to merge.


Moving owning-a-home/closure-disclosure pages into cfgov-refresh

Changes

Modified CSS, HTML, and images to move owning-a-home/closing-disclosure/ into cfgov-refresh.

Testing

Run gulp build.
Visit https://www.consumerfinance.gov/owning-a-home/closing-disclosure/ and
http://localhost:8000/owning-a-home/closing-disclosure/.
Click the expandables on and ensure they expand / contract as expected.
Compare localhost to production.
Mouse over the highlighted areas on the image and ensure the related expendables receive focus.
Click the 'Get definitions` tab and ensure the expandable terms switch, as expected.
Click the pagination buttons and ensure the pages fadeIn/Out, as expected.
Resize the browsers and ensure the form explainer images resize properly.
Scroll the pages and ensure the form explainer images doesn't overlap the ratings well.

Screenshots
Notes

The focus-target.js code runs on all pages, and adds an event listener to the all anchor links. This causes the page to jump unexpectedly in some situations via Chrome.  We should modify the modules to require the code instead of including it in common.js.

Todos

Write unit tests.

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewer tool :arrow_right:

@jimmynotjim, give it another look when you get a chance.

I get a jquery error, but it appears to be coming from GTM and is unrelated.

Oddly the expandables on localhost appear more sluggish.

When resizing image grows at 599px and pagination right arrow gets clipped slightly on right. Not sure if this matters. On production the image continues to decrease in size at small screen sizes.

Scrolling is choppy at mobile size and styles look too narrow compared to prod:
local:

prod:

Scrolling is choppy at mobile size and styles look too narrow compared to prod:

This was due to the throttling, it should be fixed. I also shortened and adjusted the animations to use translate3d instead of translateX.
@anselmbradford, please take a look when you get a chance.

Do we care about things like this?

Resize http://localhost:8000/owning-a-home/closing-disclosure/ down

Hmm layout seems better before the recent edits


Documentation Error  for Azure Synapse SQL DW Update statement

Hello All
In the Update statement page (https://docs.microsoft.com/en-us/sql/t-sql/queries/update-transact-sql?view=sql-server-ver15) at section AH. ANSI joins for update statement,  the sample has two errors that are not supported by Azure Synapse SQL DW.
These are
INSERT INTO dbo.Table1 VALUES(1, 10.0), (1, 20.0);
Returns
Msg 103010, Level 16, State 1, Line 1
Parse error at line: 1, column: 39: Incorrect syntax near ','.
Correct statements are
INSERT INTO dbo.Table1 VALUES(1, 10.0);
INSERT INTO dbo.Table1 VALUES(1, 20.0);
And the Update
UPDATE dbo.Table2
SET dbo.Table2.ColB = dbo.Table2.ColB + dbo.Table1.ColB
FROM dbo.Table2
INNER JOIN dbo.Table1
ON (dbo.Table2.ColA = dbo.Table1.ColA);
GO
Returns
Msg 104058, Level 16, State 1, Line 1
FROM clause in UPDATE and DELETE statements cannot contain subquery sources or joins.
Correct statement is
UPDATE dbo.Table2
SET dbo.Table2.ColB = dbo.Table2.ColB + dbo.Table1.ColB
FROM dbo.Table1
WHERE dbo.Table2.ColA = dbo.Table1.ColA;
GO
Also the syntax of UPDATE Statement for Azure Synapse is wrong

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

ID: bc5aa35c-e2c9-1cce-0026-6aae93b30504
Version Independent ID: 85e9daaa-2af1-95c6-8795-75976c36ccba
Content: UPDATE (Transact-SQL) - SQL Server
Content Source: docs/t-sql/queries/update-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @VanMSFT
Microsoft Alias: vanto

@antoniosch -- Antonios, thank you for your feedback.
@VanMSFT -- Van, please look into this issue.

@julieMSFT - Hey Julie, can you please take a look at this? Thanks!

@antoniosch Hi Antonios, thank you for contacting Microsoft and bringing this to our attention. I have correct the syntax in the document. The join syntax you highlighted does work in the latest version of Azure Synapse (formerly SQL DW). Please run @@version to ensure you have a version greater than Microsoft Azure SQL Data Warehouse - 10.0.11115.0 May 16 2020 02:43:26 Copyright (c) Microsoft Corporation. To get the latest version, you can pause and resume, scale, or the latest version will be applied to your SQL pool during your next maintenance window.
This new update syntax is not supported in Azure Synapse (workspaces preview).
I will go ahead and close this issue. The @notification mechanism continues to work even after an item is closed, in case further communication is needed. We hope to hear from you again.

#please-close


Require issue references for changes to Provisional PEPs?

Currently the new change log in PEP 440 links directly to the relevant discussion threads on distutils-sig.
That can work - we'd just follow up on those threads with a reference back to the issues here, so the linkage would be in place.
Another alternative would be to change the PEP references to point directly at the issue tracker, and then add the thread references here.
And the final (and my currently preferred option) would be to keep the direct references to the discussion thread archives, but also add references to the PEPs.
(Whatever we decide to do here is what I'll propose as a normative references for the PEP 1 change proposal).

I don't have a major opinion either way, except that I trust the distutils-sig links to continue working for a lot longer then I trust Github links to work for... but I don't think it's a major big deal either way.

Yeah, link stability is one of the major benefits of referring to the mailing list archives rather than a third party issue tracker.
Since I'm not inclined to set up a link redirector to provide stable links to PEP issues, lets leave this as is for now.


Lost of connection requires steam VR restart.

This is the best way I can put it, this is some excellent software. But one of the biggest problems it has is when I lose connection for whatever reason (I'm using windows 11 and Beta Software. It just happens) and I need to reconnect. When I do it reboots Steam VR despite already being open. This also means any games I have running like VRChat just close. See the issue? It would be nice to connect without needing to reboot Steam VR, so I can quickly resume a conversation.

Also having this issue, if my headset's left idle for too long it'll likely disconnect from the computer and reconnecting means that everything Steam VR gets restarted.

Would love to see this fixed, while oculus killer does wonders for my performance, force-quitting my game every time the screen turns off is tough to cope with

Commenting because I lose AirLink connection every time my headset goes to sleep, causing SteamVR to restart every time I go back into it. I hope it would be an easy fix, just to check if SteamVR is already running before deciding to run it. Absolutely love this software, has saved me so many lag issues, but it is annoying to basically crash out of the game and have to load everything up again.


[all] (index.yaml) Adding links to main landing page

This PR includes:
-- my attempt at adding a paragraph to the main landing page (mostly unsuccessful because the page schema is very restrictive -- doesn't allow ":" characters, couldn't figure out how to do a line break, etc). Any suggestions here??
-- since the paragraph didn't work, I experimented with adding MS Learn links to the About Office Add-ins card. If we include MS Learn links here, my intuition is that they should open in a new tab, and also that there be some visual indicator that these thinks go to a different section of the site (a small icon?). Not sure how much freedom we have. Open to suggestions here, too!

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

File
Status
Preview URL
Details

docs/index.yml
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 31445e0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/index.yml
:white_check_mark:Succeeded
View

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

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

File
Status
Preview URL
Details

docs/index.yml
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit cd2f292:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/404.md
:warning:Warning
View
Details

docs/index.yml
:white_check_mark:Succeeded
View

docs/404.md

Line 0, Column 0: [Warning-custom-404-page] Custom 404 page will be deprecated in future. Please remove the 404.md file to resolve this warning.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit b1c7095:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/404.md
:warning:Warning
View
Details

docs/index.yml
:white_check_mark:Succeeded
View

docs/404.md

Line 0, Column 0: [Warning-custom-404-page] Custom 404 page will be deprecated in future. Please remove the 404.md file to resolve this warning.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Thanks @AlexJerabek!


[RESOLVED] High ICMP ping response time

Problem started at 15:58:14 on 2023.06.01
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Operational data: Value: 496ms
Original problem ID: 23054
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23132&eventid=23054

Problem has been resolved in 5m 0s at 16:03:14 on 2023.06.01
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Original problem ID: 23054


Update navigation

Updated the nav with new pages as well as adjusted to account for homepage hero header.

LGTM!


Update kube-prompt to v1.0.5

What kind of request is this (question/bug/enhancement/feature request):
enhancement
Steps to reproduce (least amount of steps as possible):
Update kube-prompt to v1.0.5

@superseb please describe how to test this for QA

Open embedded kubectl
Run kube-prompt, version should show v1.0.5
See if autocomplete for namespaces works (kube-prompt, get deploy -n<space> -> should list namespaces)

rancher/rancher:master 12/27
kube-prompt is at version v1.0.5
Autocomplete is functional for kube-prompt commands and namespaces


Port VS Code 1.56.0 Hotfixes

1.56.1 - https://github.com/microsoft/vscode/releases/tag/1.56.1
https://github.com/microsoft/vscode/commit/7259955ce318a2bb2a86ee121c34dcaab0a86e45
https://github.com/microsoft/vscode/commit/5f3466a21e80b2c016a1c0cbe2af683c9072d0a7
https://github.com/microsoft/vscode/commits/1.56.1
(note this one is weird, they say they fixed them but the only thing I see in that branch is a distro bump. My best guess is that their distro has all these files in it, but ours doesn't so I'm going to make the changes here for us as well)
1.56.2 - https://github.com/microsoft/vscode/releases/tag/1.56.2
https://github.com/microsoft/vscode/commit/fb3e51a1f0912c186d91d021b8885dee4ab9b638
https://github.com/microsoft/vscode/commit/e95e22e03aacbfdde8339de5a4982d3de4d53279
https://github.com/microsoft/vscode/commit/17670a16e845984c7bc65f37c852c5b73c83b6dd
https://github.com/microsoft/vscode/commit/c15aa347f5959a68d6e26f8c44a1a9cbdfc0cd03
https://github.com/microsoft/vscode/commit/df4e68ddfbfd35b890309fd49cbad8a280815c22
https://github.com/microsoft/vscode/commits/1.56.2

I'm working on the 1.56.1 fixes, but I don't actually see the commits for those in the 1.56.1 branch so I'm trying to figure out what's going on there.

I'll follow up with looking at removing some of the fixes we did for the icon scaling since that was something they fixed in https://github.com/microsoft/vscode/commit/c15aa347f5959a68d6e26f8c44a1a9cbdfc0cd03

Pull Request Test Coverage Report for Build 972163768

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.004%) to 42.948%

Files with Coverage Reduction
New Missed Lines
%

extensions/import/src/wizard/pages/fileConfigPage.ts
1
73.46%

Totals

Change from base Build 970553828:
0.004%

Covered Lines:
26340

Relevant Lines:
55699

💛  - Coveralls


Add Int24 type

I really appreciate the contribution, but I've decided that Sera will remain a speed-first library - serializing integer types supported by the buffer library will be faster


Add docs on troubleshooting with DNS

We just spent some time investigating why some domain names weren't showing up, and after the investigation, it turned out it was down to the DNS servers in the datacentre we run the production servers in.
The DNS servers we lookup couldn't see specific domains, and the fix was to change the DNS servers we refer to.
We should document this.

This was a DNS issue in the end.
I checked these links to get an idea why and how the DNS lookups for a specific Malaysian domain was failing, and ended up adding a new domain name server from our current host, Leaseweb's list of DNS servers:
# is a domain name service 123.245.455.65
dig non-resolving.domain.my @123.245.455.65

There's a convention called netplan that i guess we might prefer to use or support in future if this is a problem again:

Netplan is a utility for easily configuring networking on a linux system. You create a YAML description of the required network interfaces and what each should be configured to do. From this description Netplan will generate all the necessary configuration for your chosen renderer tool.

https://netplan.io/
Links used:
https://kb.leaseweb.com/support/cheat-sheet
https://askubuntu.com/questions/346838/how-do-i-configure-my-dns-settings-in-ubuntu-server


No Transition

i want the slider to 'change slide' on click, thats not a problem. I just want to remove the animation. So i dont want a fade or a slide. I just want it to change. Is this possible?

Simply set speed parameter to 0.
myselector.slick({speed:0})


Task - Observation Matrix Dashboard - filter params do not work

@proceps please give me some example data/URls to test with.

3i database, species of the genus Kybos, for example:
https://sfg.taxonworks.org/tasks/observation_matrices/dashboard?fieldsets[]=observations&limit=5000&taxon_name_id=373470&ranks[]=genus&ranks[]=species&validity=false

Looking at this further we have to move the functionality to Unified Filters and associated reports.  Tabular is too complicate to maintain.

Functionality in this interface is extremely useful, but yes, some unification with filters will be beneficial.

We do these things already in unified, we just need to add some more f(n) to the matrix options, and generate a report with counts.


Realtime

For realtime map styling, does it make sense to add a Kino.MapLibre.periodically? Similar to Kino.VegaLite.periodically but for MapLibre.update_layer and/or feature-state expression. This would allow updating the “state” of a feature to allow for live data and map rendering.
Or, maybe there is a better way to handle this using setFeatureState/getFeatureState/removeFeatureState in the Elixir bindings to MapLibre Style Spec
some relevant examples:
real time maps
live-electoral-maps


Missing service account error during setup_on_cloudshell.sh

This is a small issue with the initial environment setup with ./scripts/setup_on_cloudshell.sh: by default, GCP does not have the Cloud Build API enabled, so the Cloud Build service account may not exist yet.
Thus, line 21, which attempts to grant a role on that service, will fail with an error message like, "ERROR: (gcloud.projects.add-iam-policy-binding) INVALID_ARGUMENT: Service account<EMAIL_ADDRESS>does not exist."
Hopefully the solution should be as simple as adding the following command before line 21:
gcloud services enable cloudbuild.googleapis.com
(I would create the pull request myself, but external contributions are not being accepted right now. 😉)

Another alternative is to leverage the scripts/setup_on_jupyterlab.sh script already written.  Then the only requirement might be for a small amendment to the README.md addressed at a 'new' audience that may need to ensure their account/project has the correct APIs enabled.

Thanks @ewall and @bigfleet .
That enablement command is already included in the Makefile, and the instruction is also included in the README.md.
Hope it helps!
https://github.com/GoogleCloudPlatform/asl-ml-immersion/blob/master/Makefile
https://github.com/GoogleCloudPlatform/asl-ml-immersion/blob/master/README.md


Copy tasks do not consider filter/eachFile/expansion properties in up-to-date checks

@lptr commented on Tue Jul 12 2016
Code review: REVIEW-6089
See JIRA issues:

GRADLE-1646
GRADLE-1276
GRADLE-1298

GitHub issues:

https://github.com/gradle/gradle/issues/861
https://github.com/gradle/gradle/issues/1015
#4729

@lptr commented on Tue Jul 12 2016
Added tests in gradle/gradle@e42e1038fb759155c9f1eca0c1c7d168ce339a0a

@lptr commented on Tue Jul 12 2016
Discussion on Slack with @melix about deprecating the imperative methods on CopySpec like filter(Closure): https://gradle.slack.com/archives/build-tool/p1468322257003567

@lptr commented on Thu Jan 12 2017
Since we don't want to cache archive or copy tasks, we are moving issues related to them to the main Gradle board. This issue is already reported on the main board (see description for links).

@lptr Consider adding my test in #861 regarding nested (for lack of a better word) expands, i.e. the filesMatching '**/*.properties', { expand expandMap } part.

@huxi do you think we should have them besides these? https://github.com/gradle/gradle/commit/e42e1038fb759155c9f1eca0c1c7d168ce339a0a

@lptr Yes.
I was able to fix the first of the two tests in #861 but that change wouldn't have affected the second test scenario where the expand was a parameter to filesMatching.

The straight-forward idea of adding the line getInputs().properties(properties); to AbstractCopyTask.expand(Map<String, ?> properties) fixes the first but not the second test.

Are there any plans to fix this issue, or a workaround I can use in the interim? How can I get my build to update the application.properties file if one of the input variables have changes?

As a workaround you can register the input variables via task.inputs.property().

Thank you @lptr.
So I have:
processResources {
	filesMatching('application.properties') {
		expand(project.properties)
	}
}

If I change it to
processResources {
	inputs.properties(project.properties)
	filesMatching('application.properties') {
		expand(project.properties)
	}
}

I get this error:
Could not add entry ':processResources' to cache taskHistory.bin (C:\Development\...\.gradle\3.4\taskHistory\taskHistory.bin).
> Unable to store task input properties. Property 'classLoaderScope' with value 'org.gradle.api.internal.initialization.DefaultClassLoaderScope@2b00d37' cannot be serialized.

Any idea what is wrong? I get the same error if I use inputs.property with only the single property I want to add to the inputs (which is just a string).
I am using Gradle 3.4

project.properties contains a lot of things, some of which is not serializable, and thus we can't track changes to them. You could take the properties that you know will be used during expansion (say, propA and prop2), and add them specifically:
processResources {
	def expandProps = ["propA": propA, "prop2": prop2]
	inputs.properties(expandProps)
	filesMatching('application.properties') {
		expand(expandProps)
	}
}

Using inputs.properties(props) to copy when filters change saved me! This took awhile to find. Thank you!

@wolfs Now that we have polymorphic inputs, can this be tackled more easily?

I think we could handle the CopySpec methods that don't take a Closure/Action (like eachFile {} does) now, yes. We could also handle the ones with the actions, but they would produce a lot of false cache misses in the typical case when the action is declared in the build script. Still, for a lot of use cases we could be doing things better.

we don't want to cache archive or copy tasks

Why not? My Sync tasks are generally just merging output from one or more tasks into a new folder

I guess the idea is that "build cache" would require Gradle to download the entry from build cache and unpack it.
It would take the same or even extra time as just performing copy/archive actions locally.

While implementing a custom task type with similar inputs than a copy/archive task, I wanted to get inspired by the implementation of the Gradle types and it was unexpected that renaming is not properly implemented for up-to-date checks.
I think that at least a warning is missing in the documentation, with a not that the users currently needs to workaround by declaring inputs themselves (for example on the working with files page)
Correct me if I was just too blind to find a note about this topic in the documentation.

This is actually an annoying issue to work around. Typically if something is not marked as an input one can just add it to the inputs of a task explicitly, but because CopySpec is also used outside of concrete tasks (for example, the distribution extension has a copyspec), this is not possible here.


Remove allocation in server_auth filter

This PR adds a new API call for server_auth filter to allocate the server_security_context on the call's arena instead of doing a separate allocation.

****************************************************************

libgrpc.so

     VM SIZE                                    FILE SIZE
 ++++++++++++++ GROWING                      ++++++++++++++
  +0.0%    +112 [None]                          +376  +0.0%
  +0.1%     +16 src/core/lib/surface/call.cc     +16  +0.1%
      +2.7%     +11 [Unmapped]                       +11  +2.7%
      [NEW]      +5 grpc_call_get_arena               +5  [NEW]

  +0.0%    +128 TOTAL                           +392  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                    FILE SIZE
 ++++++++++++++ GROWING                      ++++++++++++++
  +0.0%    +112 [None]                          +384  +0.0%
  +0.1%     +16 src/core/lib/surface/call.cc     +16  +0.1%
      +2.7%     +11 [Unmapped]                       +11  +2.7%
      [NEW]      +5 grpc_call_get_arena               +5  [NEW]

  +0.0%    +128 TOTAL                           +400  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

I would move the new function to a separate PR

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                                           FILE SIZE
 ++++++++++++++ GROWING                                             ++++++++++++++
  +0.1%    +392 [None]                                              +1.05Ki  +0.0%
  +2.0%     +64 src/core/lib/security/context/security_context.cc       +64  +2.0%
      [NEW]     +42 destroy_server_ctx_internals                            +42  [NEW]
       +18%     +30 [Unmapped]                                              +30   +18%
      [NEW]     +10 grpc_server_security_context_create_from_arena          +10  [NEW]
      [NEW]      +5 grpc_server_security_context_destroy_from_arena          +5  [NEW]

  +0.0%    +456 TOTAL                                               +1.12Ki  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc.so

     VM SIZE                                                           FILE SIZE
 ++++++++++++++ GROWING                                             ++++++++++++++
  +0.1%    +392 [None]                                              +1.05Ki  +0.0%
  +2.0%     +64 src/core/lib/security/context/security_context.cc       +64  +2.0%
      [NEW]     +42 destroy_server_ctx_internals                            +42  [NEW]
       +18%     +30 [Unmapped]                                              +30   +18%
      [NEW]     +10 grpc_server_security_context_create_from_arena          +10  [NEW]
      [NEW]      +5 grpc_server_security_context_destroy_from_arena          +5  [NEW]

  +0.0%    +456 TOTAL                                               +1.12Ki  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                                           FILE SIZE
 ++++++++++++++ GROWING                                             ++++++++++++++
  +0.1%    +392 [None]                                              +1.05Ki  +0.0%
  +2.0%     +64 src/core/lib/security/context/security_context.cc       +64  +2.0%
      [NEW]     +42 destroy_server_ctx_internals                            +42  [NEW]
       +18%     +30 [Unmapped]                                              +30   +18%
      [NEW]     +10 grpc_server_security_context_create_from_arena          +10  [NEW]
      [NEW]      +5 grpc_server_security_context_destroy_from_arena          +5  [NEW]

  +0.0%    +456 TOTAL                                               +1.12Ki  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Are there cases where a security context is created other than for a specific call?  If not, then I'm not sure I see the value in having an alternative ctor/dtor -- the original approach of just modifying the existing ctor/dtor to use the arena seems fine to me.

****************************************************************

libgrpc.so

     VM SIZE                                                           FILE SIZE
 ++++++++++++++ GROWING                                             ++++++++++++++
  +0.1%    +392 [None]                                              +1.07Ki  +0.0%
  +2.0%     +64 src/core/lib/security/context/security_context.cc       +64  +2.0%
      [NEW]     +42 destroy_server_ctx_internals                            +42  [NEW]
       +18%     +30 [Unmapped]                                              +30   +18%
      [NEW]     +10 grpc_server_security_context_create_from_arena          +10  [NEW]
      [NEW]      +5 grpc_server_security_context_destroy_from_arena          +5  [NEW]

  +0.0%    +456 TOTAL                                               +1.13Ki  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

@markdroth It seems there are no other places where a security context is created besides for a specific call. However, there is an internal test that would need to access the call's arena in order to use the modified existing ctor. That's when I would need the get_call_arena function

[microbenchmarks] No significant performance differences

I think adding the grpc_call_get_arena() method is probably less messy than having multiple ctor/dtor for security context.
Alternatively, consider changing the internal test to create its own arena instead of trying to use the one from the call.

[microbenchmarks] No significant performance differences

Adding grpc_call_get_arena to our surface API worries me a little... Hyrums law would say that if we do it, consumers of core will start allocating their stuff on the arena (which may not be a bad thing, but if it is a feature we should discuss)

I'm not proposing that we add it to the external API.  It should be declared in src/core/lib/surface/call.h, not in include/grpc/grpc.h.  We already have a bunch of internal stuff in there, so I don't see why this is a problem.

[microbenchmarks] No significant performance differences

Good point... sorry for the runaround Hope...

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                    FILE SIZE
 ++++++++++++++ GROWING                      ++++++++++++++
  +0.0%    +112 [None]                          +376  +0.0%
  +0.1%     +16 src/core/lib/surface/call.cc     +16  +0.1%
      +2.7%     +11 [Unmapped]                       +11  +2.7%
      [NEW]      +5 grpc_call_get_arena               +5  [NEW]

  +0.0%    +128 TOTAL                           +392  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                    FILE SIZE
 ++++++++++++++ GROWING                      ++++++++++++++
  +0.0%    +112 [None]                          +376  +0.0%
  +0.1%     +16 src/core/lib/surface/call.cc     +16  +0.1%
      +2.7%     +11 [Unmapped]                       +11  +2.7%
      [NEW]      +5 grpc_call_get_arena               +5  [NEW]

  +0.0%    +128 TOTAL                           +392  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                                         FILE SIZE
 ++++++++++++++ GROWING                                           ++++++++++++++
  +0.0%     +96 [None]                                               +368  +0.0%
  +0.1%     +16 src/core/lib/surface/call.cc                          +16  +0.1%
      +2.7%     +11 [Unmapped]                                            +11  +2.7%
      [NEW]      +5 grpc_call_get_arena                                    +5  [NEW]

 -------------- SHRINKING                                         --------------
  -0.5%     -16 src/core/lib/security/context/security_context.cc     -16  -0.5%
      [DEL]     -10 grpc_server_security_context_create                   -10  [DEL]
      -5.1%      -9 [Unmapped]                                             -9  -5.1%
      -2.9%      -8 grpc_client_security_context_destroy                   -8  -2.9%

  +0.0%     +96 TOTAL                                                +368  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                                         FILE SIZE
 ++++++++++++++ GROWING                                           ++++++++++++++
  +0.0%     +96 [None]                                               +368  +0.0%
  +0.1%     +16 src/core/lib/surface/call.cc                          +16  +0.1%
      +2.7%     +11 [Unmapped]                                            +11  +2.7%
      [NEW]      +5 grpc_call_get_arena                                    +5  [NEW]

 -------------- SHRINKING                                         --------------
  -0.5%     -16 src/core/lib/security/context/security_context.cc     -16  -0.5%
      [DEL]     -10 grpc_server_security_context_create                   -10  [DEL]
      -5.1%      -9 [Unmapped]                                             -9  -5.1%
      -2.9%      -8 grpc_client_security_context_destroy                   -8  -2.9%

  +0.0%     +96 TOTAL                                                +368  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                    FILE SIZE
 ++++++++++++++ GROWING                      ++++++++++++++
  +0.0%    +112 [None]                          +376  +0.0%
  +0.1%     +16 src/core/lib/surface/call.cc     +16  +0.1%
      +2.7%     +11 [Unmapped]                       +11  +2.7%
      [NEW]      +5 grpc_call_get_arena               +5  [NEW]

  +0.0%    +128 TOTAL                           +392  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Existing test flake: #15827

[microbenchmarks] Performance differences noted:
Benchmark                                                                                cpu_time    real_time
----------------------------------------------------------------------------------------  ----------  -----------
BM_PumpStreamClientToServer<InProcessCHTTP2>/262144                                       -4%         -4%
BM_StreamingPingPong<MinInProcess, NoOpMutator, NoOpMutator>/262144/2                     +4%         +4%
BM_StreamingPingPongMsgs<InProcess, NoOpMutator, NoOpMutator>/32768                       +4%         +4%
BM_StreamingPingPongWithCoalescingApi<InProcess, NoOpMutator, NoOpMutator>/262144/1/0     +8%         +8%
BM_StreamingPingPongWithCoalescingApi<InProcess, NoOpMutator, NoOpMutator>/262144/2/1     +7%         +7%
BM_StreamingPingPongWithCoalescingApi<MinInProcess, NoOpMutator, NoOpMutator>/262144/1/0  +4%         +4%

****************************************************************

libgrpc.so

     VM SIZE                                    FILE SIZE
 ++++++++++++++ GROWING                      ++++++++++++++
  +0.0%    +112 [None]                          +368  +0.0%
  +0.1%     +16 src/core/lib/surface/call.cc     +16  +0.1%
      +2.7%     +11 [Unmapped]                       +11  +2.7%
      [NEW]      +5 grpc_call_get_arena               +5  [NEW]

  +0.0%    +128 TOTAL                           +384  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[microbenchmarks] No significant performance differences

LGTM on the allocation from the arena.


Expose multiple audio streams.

I'd like to make all available audio streams accessible through the movie object returned.
I've kept the the previous audio_stream attributes in tact for the purposes of backwards compatibility but, if you choose to go with this change and would like me to, I can add in a deprecation warning when accessing those attributes.

[x] Expose all audio streams
[x] Add tests specific for multiple audio tracks

@CSilivestru thanks for this enhancement; for the specs and for keeping the backward compatibility! I'll look at this in a bit more depth and merge it or get back to you with questions.


Additional Information Needed

Where is the link to the page that explains how "remote servers" are determined?

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

ID: 761d0db6-1f42-1ec5-ea01-c1543d4f7e6c
Version Independent ID: ac0eafd4-9efa-272c-b873-fc4d9e129973
Content: Network security Restrict NTLM Outgoing traffic (Windows 10) - Windows security
Content Source: windows/security/threat-protection/security-policy-settings/network-security-restrict-ntlm-outgoing-ntlm-traffic-to-remote-servers.md
Product: m365-security
Technology: windows-sec
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Thanks @Turranx for the feedback, and thanks @michaelAngeloEgypt for adding the link


ecl, ecl_16_1_2: fix linking of natively built libraries and executables

Motivation for this change
On current master, both ecl versions are unable to link a native fasl using c:build-fasl from a bunch of objects that are built using compile-file with :system-p t. This change addresses this.
I tested compiling alexandria and babel manually in this fashion using the new ecl derivations to verify my change.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 133176 at ccef83bd run on x86_64-linux 1

2 packages skipped due to time constraints:

sage
sageWithDoc

4 packages built successfully:

ecl
ecl_16_1_2
eql
maxima-ecl

11 suggestions:

warning: name-and-version
Did you mean to pass pname instead of name to mkDerivation?
Near pkgs/development/compilers/ecl/default.nix:31:14:
   |
31 |   inherit (s) name version;
   |              ^

Near pkgs/development/compilers/ecl/default.nix:31:14:
   |
31 |   inherit (s) name version;
   |              ^

warning: build-tools-in-build-inputs
autoconf is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/compilers/ecl/16.1.2.nix:33:10:
   |
33 |   inherit buildInputs propagatedBuildInputs;
   |          ^

warning: build-tools-in-build-inputs
makeWrapper is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/compilers/ecl/16.1.2.nix:33:10:
   |
33 |   inherit buildInputs propagatedBuildInputs;
   |          ^

warning: build-tools-in-build-inputs
makeWrapper is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/compilers/ecl/default.nix:32:10:
   |
32 |   inherit buildInputs propagatedBuildInputs;
   |          ^

warning: build-tools-in-build-inputs
automake is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/compilers/ecl/16.1.2.nix:33:10:
   |
33 |   inherit buildInputs propagatedBuildInputs;
   |          ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/development/compilers/ecl/16.1.2.nix:66:5:
   |
66 |     ./ecl-1.16.2-libffi-3.3-abi.patch
   |     ^

warning: build-tools-in-build-inputs
libtool is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/compilers/ecl/default.nix:32:10:
   |
32 |   inherit buildInputs propagatedBuildInputs;
   |          ^

warning: build-tools-in-build-inputs
libtool is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/compilers/ecl/16.1.2.nix:33:10:
   |
33 |   inherit buildInputs propagatedBuildInputs;
   |          ^

warning: name-and-version
Did you mean to pass pname instead of name to mkDerivation?
Near pkgs/development/compilers/ecl/16.1.2.nix:32:14:
   |
32 |   inherit (s) name version;
   |              ^

Near pkgs/development/compilers/ecl/16.1.2.nix:32:14:
   |
32 |   inherit (s) name version;
   |              ^

warning: build-tools-in-build-inputs
automake is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/compilers/ecl/default.nix:32:10:
   |
32 |   inherit buildInputs propagatedBuildInputs;
   |          ^

warning: build-tools-in-build-inputs
autoconf is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/compilers/ecl/default.nix:32:10:
   |
32 |   inherit buildInputs propagatedBuildInputs;
   |          ^

Result of nixpkgs-review pr 133176 run on x86_64-linux 1

6 packages built:

ecl
ecl_16_1_2
eql
maxima-ecl
sage
sageWithDoc


Autocomplete on script imports not working

Ive just updated my atom-typescript version (I was a couple weeks behind maybe). And it seems that now when importing a file it doesnt autocomplete the script when hitting enter.
So when it gets to this part and hit enter, it doesn't fill in the rest of the import statement anymore.

All autocompletion UI is handled by the autocomplete-plus core package. Check its settings?

Oh! Hmm okay i'll dig around and see what I can find. Thanks, I'll post an answer here when i find one.
Ps. Its the same for directories when autocompleting.

Also, a quick note: make sure the enter keybinding isn't stolen by something else. You can use the keybinding resolver to debug those ('key binding resolver: toggle' in command palette)


Update articles pages with links to demo code

As more demo code becomes available in delphidabbler/article-demos add links to those demos to article pages - including the pages that still link to PDFs.
To Do:

[ ] article 2 (same demo as article 3)
[ ] article 3 (same demo as article 2)
[x] article 4
[ ] article 5
[x] article 6
[ ] article 7
[ ] article 8
[ ] article 9
[x] article 10
[x] article 11
[ ] article 12
[ ] article 13
[ ] article 14
[ ] article 16
[ ] article 17
[ ] article 18
[ ] article 19
[ ] article 20
[ ] article 21
[ ] article 22
[ ] article 23
[ ] article 24
[ ] article 25

Revise links added before Dec 21 to access required folders rather than repo root directory:

[ ] article 4
[ ] article 6
[ ] article 10
[ ] article 11

Add demo code for article 27


Improve correlation for optimized search in rest source and improve general correlation by downloading manifests

Unit tests added. Manual validation that microsoft edge can now be successfully correlated.
Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


Link status

More work on link status and polling.  Driver is receiving/processing Async events and is even receiving Rx data packets.  Link status tracking remains illusive.
This is a good snapshot to stop at for the night.

So upon inspection of the latest changes I'm feeling like we need to go back to the drawing board. I think concurrent code is great for solving the revent polling problem but that should ONLY be for polling the interrupt pin, not for the actual sdpcmPoll function! What are your thoughts @scottfeldman ?
My gut instinct is telling me to follow the CYW43 way of doing things at a high level (to a point, ofc), which is doing it single threadedly. In this case doing so yields simpler, easy to understand code. I feel like there is more to lose by writing concurrent code. This would actually also make it more performant imho.
As for the parts that should be concurrent: The irq pin polling, which ideally would happen through an interrupt, which we have not yet designed. This implies reverting most concurrent heavy latest changes except for pollStart and pollEnd functions. Concurrency primitives like channels and the lot should be exclusively limited to this part! We should take inspiration from cy43 for the rest!

As for the parts that should be concurrent: The irq pin polling, which ideally would happen through an interrupt, which we have not yet designed. This implies reverting most concurrent heavy latest changes except for pollStart and pollEnd functions. Concurrency primitives like channels and the lot should be exclusively limited to this part! We should take inspiration from cy43 for the rest!

Ok, let me undo the last part and follow cyw43 style.  It's going to make for a messy PR; I'm not opposed to scrapping this PR and opening a new PR with the rework.


Added cross-domain (CORS) capabilities to XHR backend

Added cross-domain (CORS) capabilities to the XHR i18next backend.  For CORS support, requests need to follow specific guidelines which are detailed on the HTTP access control MDN page.  The X-Requested-With custom header falls outside of this spec, but is set by the XHR backend.  To work around this, the new crossDomain option can be set to true, which specifically requests that this header is not sent.  This is similar to the approach that jQuery takes and still allows the old operation if crossDomain is a falsey value.

published v0.2.0


change over_window_compute_count to over_window_accessed_entry_count for better observability

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

#18943  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @stdrc and the rest of your teammates on  Graphite


Sometimes the device is not registered after CustomerIO.identify

HI!
Today I was replacing the native SDK with the flutter package, and in the end, I had problems with the iOS platform.
For Android: I get the device token from the console and send a test notification via CIO, and everything works as expected. When I run the app, I see the registered device in the CustomerIo dashboard, and the same token is there as in the console.
For iOS: I noticed that sometimes the iOS devices don't appear in the CIO dashboard under the People section. I was able to find the account via email, but when I looked under the Device tab, there is no recent activity for that particular device, only events that are triggered. Also, I noticed that I don't see that message when the device is registered from CIO in the iOS device console.
But what's interesting is when I used the FCM token from Firebase to send a test push notification via CIO, everything was working fine. Because from code in the console, I can always get an FCM token and it is identical to a CIO device token.
My questions:

Why don't I see that registered device token message in iOS as I do for Android?
Why is there no registered device in Dashboard?
Is there any other way of getting device tokens for QA testing?

Console message when a device is registered (Android case):
registering device token dsatQJETtIdsa3d42s2IfbsXpc-:APA91bEVGfeZvOitdasdaJz4ULT8erhgNFE0VMtOpSemSNDGah5Ahd4nlN72_Y5DT9r9FUISLMJNR4kuyAwgD4WDL6E7kdF4WaDV0G_Hh5z9hdScH09CeIokz0, attributes: {device_os=31, device_model=SM-G990B2, device_manufacturer=samsung, app_version=1.0.0, cio_sdk_version=1.0.0-alpha.8, device_locale=en-US, push_enabled=true}

Hi @NemanjaLugi
I'm sorry you had to deal with this.

I noticed that I don't see that message when the device is registered from CIO in the iOS device console.

Did you try configuring the package with logLevel : "debug" ? Setting this property would let the package show logs in their respective IDE's (Xcode for iOS).
Also, I wanted to learn more about the problem you're having. Could you please share the exact steps you took when you didn't see the registered device token? A quick video sent to<EMAIL_ADDRESS>would also help us better understand the problem.

Hi @ami-aman
Yes, I checked the logs using log-level debugging, but I didn't see anything strange coming up. Everything was implemented as mentioned in the docs only Firebase was added through the flutter package.
We still have a branch with a native implementation of CIO (Flutter project with native iOS and Android SDK), and we have managed to reproduce only on one device.
How much time is needed for a device to send all data to the CIO dashboard? I guess it depends but approximately?
Is there any additional message if something with Firebase is not added correctly? Because when I run the app on Android, I see a message showing if Firebase is available, but there is no such message for iOS. Will Pushh notification work if there is no Notification Service Extension?
Logs - This is all logs from the CIO on the load of the app :)
[CIO] (siteid:) ℹ Customer.io SDK 2.1.2 initialized and ready to use for site id: <ID>
[CIO] (siteid:) deleting expired tasks from the queue
[CIO] (siteid:) Setting up in-app module...
[CIO] (siteid:) ℹ In-app module setup with SDK
[CIO] (siteid:) deleting tasks older then 2023-03-17T14:29:36+0100, current time is: 2023-03-20T14:29:36+0100
[CIO] (siteid:) deleting 0 tasks.
[CIO] (siteid:) ℹ identify profile <PROFILE - ID>
[CIO] (siteid:) identify profile attributes none
[CIO] (siteid:) ℹ adding queue task identifyProfile
[CIO] (siteid:) added queue task data {"identifier":"<PROFILE - ID>"}
[CIO] (siteid:) processing queue status QueueStatus(queueId: "<ID>", numTasksInQueue: 1).
[CIO] (siteid:) ℹ queue timer: scheduled to run queue in 30.0 seconds
[CIO] (siteid:) storing identifier on device storage "<PROFILE - ID>
[CIO] (siteid:) Timer (nckapmqmlh) making a timer for 30.0 seconds.
[CIO] (siteid:) Timer (nckapmqmlh) timer is done! It's been reset.
[CIO] (siteid:) ℹ queue timer: now running queue
[CIO] (siteid:) ℹ queue run request sent
[CIO] (siteid:) queue querying next task. criteria: QueueQueryCriteria(excludeGroups: Set([]))
[CIO] (siteid:) queue tasks left to run: 1
[CIO] (siteid:) queue next task to run: 4020D, identifyProfile, {"identifier":""<PROFILE - ID>"}, QueueTaskRunResults (totalRuns: 0)
[CIO] (siteid:) queue task 4020D ran successfully
[CIO] (siteid:) queue deleting task 4020D
[CIO] (siteid:) queue out of tasks to run.
[CIO] (siteid:) resetting queue tasks query criteria
[CIO] (siteid:) ℹ queue timer: queue done running

Thanks for the response, @NemanjaLugi ! Appreciate it.

... and we have managed to reproduce only on one device.

Did you do anything differently for the device that didn't get registered?
You are right, the time taken to send data to the workspace depends. Here is more information on how the queue gets processed.
As per the logs you shared, I do not see a device token getting generated for the device. If you are using logLevel as debug then you will notice a log like [CIO] (siteid:) ℹ registering device token <device_token>. Can you check if the device token was generated for the device that did not get registered for you ?
Also, Notification Service Extension is for rich push notifications - notifications with an image, link etc. You will receive simple push notifications even if notification service extension is not there in your app.

Thanks for your response @ami-aman :)
We ran the same code on all devices, and we noticed some differences:

For Android devices, we got a device token every time we run the app, and the token is successfully registered and we can see all the device information on the CIO dashboard.
For iOS devices, we got a device token only when we run an app for the first time and only then we can see that  registering device token <device_token> massage and after that iOS device is registered. But in the second run and all the others we could no longer see that message, only after the app is fully reinstalled(deleted and installed) again.

What could be causing the different behaviors between iOS and Android? Or is it the flow of how it should work?

Gotcha! Thanks for explaining your observations.
This is happening because of lifecycle differences between flutter and native modules, that means that the native methods are called before flutter lifecycle starts. Flutter package provides a flexibility to register a device token manually that would help you overcome this scenario.
Let me know if this helped!

@ami-aman  Thank you very much!
Manually registering the device token will probably work for us, so I'll close the issue now :)


Improved the home window

Issue: https://github.com/roonyh/fossasia-gci-companion/issues/1 I have made whole page with materialize css. The files of materialize css are attached with this pull request. I have attached the fossasia-dark logo at top and then I have added simple logo near input fields. After that there is one submit button edited and at bottom I have added footer. At footer I have added about FOSSASIA and then at right hand corner there are various links for FOSSASIA in different social networking sites. Now it looks like this as shown in image :
Test Link : http://shashank-sharma.github.io/fosasia-site-test.github.io/

Thanks! Please have the issue assigned to you before you start working on it. This is important to avoid duplication.
Just to make sure you are not confused, GCI FOSSASIA Companion is supposed to be a desktop app. Not a website. So we don't need minified js files. They would not make much different when the source code is compiled to binary. Read about electron which is the tool we are using for this, to understand more.
App should work offline too. So don't link to external resources. They should be inside the app and shipped with it.
We don't need to base64 encode images in the footer. Again because this is a desktop app. I don't think we need a footer at all :)
I don't think we need multiple font formats because again this is a desktop app. So if fonts work in the used chromium version we are good.
Text inputs of the form should be in a single column.
I think you should try to build the app locally and get more familiar with electron. [READ ME] (https://github.com/roonyh/fossasia-gci-companion/blob/master/README.md) and comment here if you need help.

@roonyh I have some doubts regarding this. At first if it is an app then how can I run it in my system. Second if I want to edit the UI of this app then I must edit index.html right?  and last since there are less content so do I have to edit those content only for better view like editing input field and buttons only or do I have to add some more things also?  Like description or anything like that.

Brief instructions on how to run are given in the readme file. https://github.com/roonyh/fossasia-gci-companion/blob/master/README.md#development
You need to install nodejs  latest stable. https://nodejs.org/en/download/stable/
You just have to do npm install and npm start from the root folder of the repo. It should work for any system but let me know if you have troubles.
Yes you have to edit the index.html. Only editing current content is enough. Just add fossasia logo. And a suitable background with colors.


Can a pod discover its namespace?

In #9473 I asked if there was an easy way to discover resource limits without querying the API.  But maybe I should just be querying the API...  It seems easy, except that I don't know how to find my namespace.
I think I can do this:
POD_NAME=`cat /etc/hostname`
NAMESPACE=default # ???
curl http://${KUBERNETES_RO_SERVICE_HOST}:${KUBERNETES_RO_SERVICE_PORT}/api/v1beta3/namespaces/${NAMESPACE}/pods/${POD_NAME}

And then I get everything (labels, resources etc), which is great.
But:

How can I discover my namespace?
Will this continue to be supported?  I can imagine that we don't want to give access to the entire cluster state to pods, but hopefully at least same-namespace or self-identity will be allowed.
Is it safe to assume that hostname == pod-name (and if not, is there a way to discover my pod-name)?

I filed #9433 yesterday related to pod and namespace and more discussion can be found at: https://github.com/GoogleCloudPlatform/kubernetes/issues/9356#issuecomment-110155706

@dchen1107: thanks for that link, although I don't know that it applies here.  To be clear - I'm inside a container, and I would like to get the pod info that applies to me from the API.  I don't know how to get my namespace, nor am I certain that my hostname was always my name.
I did figure out that I could look at my IP, which is my podIP in the API.  Sadly I don't think I'm allowed to use that as a fieldSelector ("message": "field label not supported: podIP"), but then I can at least use that to unambiguously figure out which pod I am.  It's not great, but it does work!

@justinsb - you can get the namespace information from the downward API.
see the example here:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/downward_api.md
Closing the issue, re-open if this did not address your concern.

Thanks @derekwaynecarr - that is a good option that I wasn't aware of!
I was hoping for something that didn't require the manifest writer to do anything, but maybe that's over-optimizing.  (I'm thinking about packaged k8s apps that can just be deployed and will self-configure, but it'll probably involve a service & and an rc anyway so this would actually work great).
Between this and the option of querying the k8s API I have some reasonable options!


CRON Expression to run webjob on 1st day of Aug every year at 01:00 AM

I have given CRON as '0 0 1 1 8 *' to run on 1st day of Aug every year. But in job_scheduler.log I found as below
"Scheduling WebJob with 0 0 1 1 8 * next schedule expected in 40.00:00:00"
what is the next execution time?
Is my CRON Correct ?
More details: I have CRON 1 to run after 31 days, it worked as expected.
But i have given CRON to run after 43 days, But it didnt work as expected.
CRON 1  "0 0 1 19 11 *" - Expected next run after 31 days, 13 hours, 24 minutes  - Yes its correct
[10/18/2019 11:35:12 > ec1845: SYS INFO] Scheduling WebJob with 0 0 1 19 11 * next schedule expected in 31.13:24:47.7563432
CRON 2  "0 1 1 12 *" - Expected next run after 43 days, 13 hours, 17 minutes  - But webjob is giving wrong
[10/18/2019 11:42:58 > ec1845: SYS INFO] Scheduling WebJob with 0 0 1 1 12 * next schedule expected in 40.00:00:00

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


not taking turn for player character

Whenever I try to activate the module, it doesn't do anything. Then, I realized (saw info in console) it doesn't do anything for player owned characters. This is a very big problem for the very campaign I downloaded this module for. In this campaign, the players can move the monsters as well, so everyone is the owner of every monster. Might be an edge case, but allow for a setting to allow the ai to overtake player controlled non player character tokens. I looked into the code and saw it would be pretty easy to implement as well. Thanks for your consideration.

I'm not sure there's an easy way to determine if a token is an NPC other than by checking if they are owned by a player.
However, this check predates the confirmation screen, and I'm not sure it's needed any more. If I add a warning to that dialog, then the GM can reject a PC plan if they want to. This would allow GMs to have more balanced "mind control" effects, for example.
I'm still thinking about this, but I'm leaning toward adding this warning into the patch that adds functionality to the confirmation dialog.

I have added it


Update SETUP.md

SETUP.md could use a little expansions now that we have good support for both GCE and AWS we'll want to describe both install paths well.

Fixed in #368


Add startAutoplay method reference

Short description of what this resolves:
Fixes the TypeScript compiler error when using Slides.getSlides().startAutoplay()
As specified in the Swiper docs
mySwiper.startAutoplay();
start auto play. It may be useful for custom "Play" and "Pause" buttons
Changes proposed in this pull request:

Adding startAutoplay() method reference

Ionic Version: 2.0-beta.11

I'm going to close this since slides have since been refactored, thank you though!


It's hard for users to find their logs on console.cloud.google.com.

After collecting friction logs from many people, I've observed a common issue that users suffer is that it's difficult to find their logs in console.cloud.google.com.  If the app is running locally, then they appear under Global.  If the app is running on App Engine flex, then the logs appear under App Engine.
Admittedly, this client library is far removed from the user experience of console.cloud.google.com.  But I do see an opportunity to help users avoid this pain:  Write a log statement (which appears on stdout when the user runs in development mode) that tells them exactly how to see their logs.  Something like:
Logs are being written to Stackdriver and can be seen at https://console.cloud.google.com/logs/viewer?resource=global

Like https://github.com/googleapis/google-cloud-dotnet/issues/2630, it will take some cleverness to avoid infinite recursion.

Shouldn't that information be part of normal user documentation instead? We can describe the different scenarios in which logs end up under one or another "environment label".

Yes, it should and it is.  But it's just really hard for users to find the logs.  3 of the 5 of us who recorded friction logs had trouble finding our logs in console.cloud.google.com.

Drive-by thought: suppose our samples had something that wrote the URL to the console. If we expose an easy way of getting it printed to the console, but without writing to the console by default, then users following the samples will see it, but then can easily remove it once they know where to look.
Does that work, or do you think people don't follow the samples closely enough?

Hi @SurferJeffAtGoogle, I'm working on this now and we were thinking of adding some more data to the URL (apart from the monitored resource) so that users can truly land on the logs being logged. This data would include the project ID, the log level, the name of the log, etc. With this info included on the URL users would only need to change on the UI the time spans they want to see logs for. Are you OK with this?

@SurferJeffAtGoogle I just created #2860. Take a look and leave comments, specially if you think the snippets are enough.

@jskeet the flagging of this as a P2, even though it was a feature request, was causing it to go out of SLO.  I went ahead and removed that label - just wanted to make sure it's ok with you :)

@JustinBeckwith: Fine by me. I thought everything had to have a priority :)

I think this is now fixed as far as we want to; it's documented at
http://googleapis.github.io/google-cloud-dotnet/docs/Google.Cloud.Diagnostics.AspNetCore/#finding-out-the-url-where-logs-are-written
It does help that we've made diagnostics easier to get started with anyway.

Closing as "this is as good as we're going to get".


Script is trying to activate the skill on an Order Changed servant too quickly and missing the click

Using Bluestacks. Basically it tries to activate the next skill before the servant is quite finished appearing, and before the button is clickable. I am using the latest version of Ankulua.
Can you add a small delay after Order Change is used to prevent this? Alternately let us add it manually in the autoskill.

use ankulua 8.5.0 can temporarily fix the issus

Certainly it is possible, if you are talking about PlugSuit switching:
local function SelectSubMember(location)
	return function()
		click(location)
		wait(0.3)
		click(game.BATTLE_ORDER_CHANGE_OK_CLICK)
		WaitForAnimationToFinish(15) // change this value 15 to something higher, experiment

		ChangeArray(DEFAULT_FUNCTION_ARRAY)
	end
end

A workaround that works 100% of the time for me is to put a dummy skill usage command right after the plugsuit switch that doesn't matter for your setup. It will whiff that skill and then execute the rest correctly afterwards.


simulcast codecs support

@davidliu added you as a reviewer too to confirm there's nothing here that'll break react-native. We won't need to worry about AV1 for RN for awhile.. just wanted to make sure this PR doesn't break it.


Trigger observers that inbox is up to date after start

Background
When start is called, onInboxChanged was not triggered if there was nothing to update form the server. After start the OnInboxChanged should be called at least once so that observers know that the inbox is up to date.
Implementation
Trigger OnInboxChanged since no updates are downloaded from server.

Any info when this could be merged?

Hi @ollitapa,
We need to check if this is the right callback since we use another approach in forceContentUpdate. We do have your request on our list.

Fixed in #498 when inbox loads, the InboxChanged callback is executed no matter if new messages will be downloaded or not.


[Feature] Add a cargo binary for running diesel migrations locally

Objective
Add a new binary in the router crate that would run migrations locally via cli i.e cargo run migrations.
Context
Currently migrations are managed via the diesel_cli which rely on explicitly providing the database username/password & host,
This is a bit cumbersome when you have to provide the variables over & over again.
diesel migration --database-url postgres://$DB_USER:$DB_PASS@localhost:5432/$DB_NAME run

It would be somewhat easier if we could leverage diesel_migrations to read the credentials from config file / env & apply migrations.
(Open to alternative approaches as well for this)
Outcomes

A new rust binary that can be used for running local db migrations from file
Updates to the local setup documentation outlining the usage of this cli tool

I am interested in taking on this issue!

Hey @davidkurilla , thanks for your interest.
I'm assigning you this issue, you can ask any doubts here or reach out to me or @SanchithHegde  on our slack/discord.