Bug: Dark mode styling errors

Describe the bug
In dark mode, the following styling issues occur:

/user/profile: "Receive notifications and reminders on email" in dark font
/crew/sponsorships/{id} : "When deleted the company will no longer appear as a sponsor. However, the company representative is allowed to apply for another sponsorship" in dark font
/moderator/presentations/{id}: Presentation information and Participants
/moderator/rooms/{id}: Room information

To Reproduce
Steps to reproduce the behavior:

Go to each page in the list
Click on 'dark mode toggle button'
See error

Expected behavior
Font color should be consistent
Screenshots
Example:

Desktop (please complete the following information):
N/A
Smartphone (please complete the following information):
N/A
Additional context
Add any other context about the problem here.

Fixed in #438


Design System: Add in keyboard functionality

Feature Description
Design system needs access to the useKeyboardOnly util used by editor and dashboard. Ideally, this helps the design system become the source of truth for that.
Alternatives Considered

Additional Context
Realized this ticket was missing while working on #4950 dropDown. Creating a new ticket in an effort to avoid scope creep.

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

This will get tested with components, background work. There's no real reason to spend time in UAT on this alone, my thoughts - but I'll let you make that call, @o-fernandez  :)

This will get tested with components, background work. There's no real reason to spend time in UAT on this alone, my thoughts - but I'll let you make that call, @o-fernandez  :)

Not quite sure to understand this ticket, but I don't see any Keyboard section in the DS.

Understandable, @ivayloklissarov  - I thought as much with this comment here: https://github.com/google/web-stories-wp/issues/5810#issuecomment-755698270
Essentially what this is for is a shared style for all keyboard focuses. which are these:

So this'll get tested every time you test anything with focus, there's nothing to really uat. just doing due diligence with the ticket pipeline by passing it to UAT. We can probably just close this.


fix(types): async Component types

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[ ] No
[x] Not sure if this is considered a breaking change

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

Other information:
Look at this codeexample:
import { AsyncComponent, Component } from 'vue'
const a: AsyncComponent = () => ({
  component: new Promise<Component>((res, rej) => {
    res({})
  }),
})

const b: AsyncComponent = () => ({
  component: () =>
    new Promise<Component>((res, rej) => {
      res({})
    }),
})

The first example is like in the docs https://vuejs.org/v2/guide/components-dynamic-async.html#Handling-Loading-State and the second example should not work (I also looked at the implementation and I think this would not work).
The types are currently like such that the first example throws a type Error and the second throws no error. This PR fixes that.

Can you add a test?

What exactly should be tested? -  As I understand the repro there are no type tests. I just found https://github.com/vuejs/vue/blob/dev/test/unit/features/component/component-async.spec.js#L322
I can add there a test that the second example doesn't work (see https://codepen.io/mathe42/pen/poNNNYx)
there are tests that the first example works.

In this case a type test case would suffice: https://github.com/vuejs/vue/tree/dev/types/test

Thanks didn't saw that. Just pushed some tests (also for the simpe usages of AsyncComponent).
Not shure about my const b test. This tests that the syntax that currently is allowed (by the types) failes with this PR.

Yes sorry, there was more in the commits than I wanted. Working on it.

@posva Removed the styling changes.
Hope this is fine now.

I repleced the Vue.component tests with a single one to test if it accepts AsyncComponents...


Bluetooth: Mesh: Check TTL max value on transport TX

Adds check for TTL max in the transport send functions, and moves
setting of default TTL to transport.
Fixes #29855.
Signed-off-by: Trond Einar Snekvik<EMAIL_ADDRESS>

Does this really need to be backported? Unless I'm misunderstanding something, what we're really covering for here are application bugs (no valid mesh application should try to use more than 127) rather than a serious stack bug (IMO), so this doesn't feel like something critical enough to justify backporting.

Your assessment is correct, and I'm still calibrating my backport-meter a bit, so I won't fight for it. It's definitely not very critical.


fixed wrong env variable name

I think there is a typo on the env variable name POSTGRES_DB in postgres/scripts/load.py

Thanks! There are a couple of other places where this occurs too. I'll fix those after merging.


Fixes #943: Handling of method parameters without @ApiParam has been added.

Fixes #943: Handling of method parameters without @ApiParam annotations has been added.

Ignoring of whitespaces will make reviewing easier https://github.com/tomtit/swagger-core/commit/1861659a917c7a64309f5ff7e06f997ceeecd64f?w=1


Added newsletter relation to subscribe events

refs https://github.com/TryGhost/Team/issues/1478
Changes

Added the newsletter relation to subscribe events

Changes in members-api

Compare: https://github.com/TryGhost/Members/compare/%40tryghost/members-api%406.0.0-alpha.0...%40tryghost/members-api%406.0.0
Makes sure the newsletter relation is returned in the activity feed for susbcribe events (aka newsletter events).

Tests

Added first test for activity feed to check if the newsletter relation is correctly fetched

Tests fail in CI only 😬


Just fixed format with go fmt ./...

+1 for this. Many people, including myself, work with editors that automatically run go fmt on saved files. Need to do git add in patch mode then if you don't want to include irrelevant formatted stuff.

Yeah just run into that while preparing another PR, so I thought let's fix that entirely ;)


Menu items not available after enabling

Hi, thank you for this addon, I was wondering if anybody has seen an issue where the addon is installed and enabled but we get no additional options for pod logs etc, only the default openlens options?
I had to install it via the tar download, no direct access to github but it installed fine and enabled fine.
thanks
Paul

Not to worry, I believe its probably down to the restricted internet access again. Fro developer view I can see the error below, looks like it needs to get out to download some stuff and I dont think it can.  I have configured OL to use our proxy but the settings dont cater for requiring authentication so I suspect its failing
BrowserConsole.js:47 [EXTENSIONS-LOADER]: can't load renderer for "@alebcay/openlens-node-pod-menu": Error: Cannot find module<EMAIL_ADDRESS>Require stack:
- electron/js2c/renderer_init
    at Module._resolveFilename (node:internal/modules/cjs/loader:940:15)
    at i._resolveFilename (node:electron/js2c/renderer_init:33:1095)
    at Module._load (node:internal/modules/cjs/loader:785:27)
    at c._load (node:electron/js2c/asar_bundle:5:13343)
    at i._load (node:electron/js2c/renderer_init:33:356)
    at Module.require (node:internal/modules/cjs/loader:1012:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at ExtensionLoader.requireExtension (https://1f3515d….lens.app:59766/build/lens.js?5a1bf22…:542270:20)
    at https://1f3515d….lens.app:59766/build/lens.js?5a1bf22…:542222:53
    at Array.map (<anonymous>)

Happened here, the same exception. But after a hard refresh (cmd+R) started working again 🤷.

The same here, OpenLens 6.4.15, behind proxy, had to install using TGZ file. The same error as above, hard refresh did not help.

I got the same problem. In my case, the seems to be that OpenLens is looking for the extension code in a different place, than where the extension was installed, from OpenLens GUI.
Installed: ~/.var/app/dev.k8slens.OpenLens/.k8slens/extensions/alebcay--openlens-node-pod-menu
Expected: ~/.var/app/dev.k8slens.OpenLens/config/OpenLens/node_modules/@alebcay

Thank you very much @brpaz, that helped me solve this same issue.

Thanks everyone for the info. Can someone please open an issue upstream for this? From what has been presented so far, this looks like an issue with the extension loading and installation process within Lens itself (this extension does not, for instance, dictate the npm command listed above as a workaround, or control the path into which it is installed).
Upstream may also ask for other information in the issue description, hence why I would appreciate if someone who has run into the issue to report there; I have not been able to reproduce this locally and am missing parts of what they're asking for.

We had the same problem.
This worked for us under MacOS:

Install the extension with the UI (note: it will install "forever")
Wait for 1 or 2 minutes and then close OpenLens
Execute on Terminal to create a symlink:

mkdir ~/Library/Application\ Support/OpenLens/node_modules/\@alebcay

ln -s ~/.k8slens/extensions/alebcay--openlens-node-pod-menu ~/Library/Application\ Support/OpenLens/node_modules/\@alebcay/openlens-node-pod-menu

Start OpenLens again
Enable the extension if necessary

same problem here,  installed on windows :
debug :
BrowserConsole.js:47 [EXTENSIONS-LOADER]: can't load renderer for "@alebcay/openlens-node-pod-menu": Error: Cannot find module<EMAIL_ADDRESS>Require stack:
- electron/js2c/renderer_init
    at Module._resolveFilename (node:internal/modules/cjs/loader:940:15)
    at i._resolveFilename (node:electron/js2c/renderer_init:33:1095)
    at Module._load (node:internal/modules/cjs/loader:785:27)
    at c._load (node:electron/js2c/asar_bundle:5:13343)
    at i._load (node:electron/js2c/renderer_init:33:356)
    at Module.require (node:internal/modules/cjs/loader:1012:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at ExtensionLoader.requireExtension (https://0db381312a321e2d62459b2eb7f4dbee.lens.app:37052/build/OpenLens.js?d8e99657584b2f5b7773:488590:20)
    at https://0db381312a321e2d62459b2eb7f4dbee.lens.app:37052/build/OpenLens.js?d8e99657584b2f5b7773:488542:53
    at Array.map (<anonymous>) {extension: {…}}

so i tried to find a renderer.js file in the .tar  but there is not this file
there is the file renderer.tsx  i renamed .js and put in a dist directory i created : C:\Users\xxxxxxxx\AppData\Roaming\OpenLens\node_modules@alebcay\openlens-node-pod-menu\dist
but it didn't work
why is there no "renderer.js" file while the logs request it ?

It sounds like you are trying to download the source files from this repository, which is uncompiled TypeScript. The build process converts this into plain JavaScript which OpenLens consumes as a plugin.
You can see that the version pushed to npm contains the renderer.js:
$ curl<EMAIL_ADDRESS>| tar tzf -
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4456  100  4456    0     0   9958      0 --:--:-- --:--:-- --:--:-- 10104
package/LICENSE
package/dist/renderer.js
package/package.json
package/README.md

Installing by providing the package name to OpenLens should from npmjs and thus contain the expected files.

thanks, it works perfectly now with the tgz you send

Success: Installation works perfectly (for me on a Mac) if I install using "@albecay/openlens-node-pod-menu"
If instead I give the URL to the .tgz or select the downloaded .tgz in the file system, OpenLens claims that the installation was successful, but the Status flickers rapidly(!) back and forth between Enabled/Disabled until I restart OpenLens. After restarting, the Status is then steady (Enabled), but the menus still do not show up, and none of the solutions shown above make any difference. Attempting to uninstall did not remove the extension, and I had to edit various json files to truly uninstall the extension before I could install using the '@'-reference.
I hope this helps someone :)

I had same issue: windows without github acces and OpenLens 6.5.2-366 without logs button.
I was download https://github.com/alebcay/openlens-node-pod-menu/archive/refs/tags/v0.1.2.tar.gz, unpack it and install from Extention page. Nothing was change :)
Than I was download<EMAIL_ADDRESS>and copy "dist" folder from it in c:\Users%USER%\AppData\Roaming\OpenLens\node_modules\alebcay\openlens-node-pod-menu
Restart OpenLens - and works!

uninstall and then reinstalling fixed this for me

On Windows:  For me, it looks like it was creating either a symbolic link or a shortcut.  When I recreated the folder and copied the contents over, it started working.  Here's what my folder looked like:

On Windows doesn't seem to work here. The extensions were "installed" and green, but no button. I went into c:\Users%USER%\AppData\Roaming\OpenLens\node_modules\ and followed @rakker91 advice of renaming the shortcut, creating a folder and copying files into it, then deleting the shortcut, and now I open lens and the extension menu is empty.
This could definitely be improved.


Meteor.LoggingIn return true when not loggedIn, and Meteor.user() is not saved to AsyncStorage

Meteor.LoggingIn return true when not loggedIn, but when loggedIn it return false.
And the most blocker thing is Meteor.user() is not saved to asyncStorage. So when I am refreshing the app, the Meteor.user() is not there anymore

Yes see meteor docs for loggingIn but it is what is expected ;) LoggingIn returns true if logging is in progress.
When you app is launched, React Native Meteor has to re-sign in. So it checks if a token exists (AsyncStorage), and sign in with it.
Am I being clear ?

Yea thanks for confirm. btw how about the Meteor.user() that undefined although already lloggedin when rn is refreshed

Yeah Meteor.user() get defined only when re-signed in. Is it bad in your opinion ?

Yeah, meteor default behaviour are using it as the saved object to be used
when a user already loggedin. So can you explain how is it now to detect
that user already logged in and get their data everytime its needed? Btw
your Meteor.userId() also not saved to storage, and that should be saved yo
storage too
On Mar 10, 2016 9:23 PM, "Mokto"<EMAIL_ADDRESS>wrote:

Yeah Meteor.user() get defined only when re-signed in. Is it bad in your
opinion ?
—
Reply to this email directly or view it on GitHub
https://github.com/inProgress-team/react-native-meteor/issues/19#issuecomment-194870108
.

the diagram should be like this.
Login with password > success > save Meteor.user() obj and Meteor.userId string to asyncStorage by default. > app closed/refreshed on emulator > Meteor.user() and Metror.userId should still exist so i can redirect router to main page rather than login page for example

Sorry I am misunderstand your concept, it's already there if I call Meteor.user() or Meteor.userId() inside getMeteorData method, and the behaviour is already correct. But why its called so many time? and also with undefined data included. Is it possible to called only once when ready?

Actually the Meteor.user is never saved in localstorage, only userId is ;)

I will add this to asyncStorage soon, but it is not a priority in my opinion.

this is very useful when needing something like this, after logged In
Meteor.userId() use case on every part of method:
startMeteorSubscriptions(){
  Meteor.subscribe('todos', {createdBy: Meteor.userId()}); //but currently Meteor.userId() can't be accessed like this
}

Meteor.user() use case
render(){
   const user = Meteor.user();

   return (
     <NavigationBar
     title="My Profile"
     renderLeftButton={<Text>user.name</Text>}/>
   );
}

there are still so many case need this to be accessed Synchronously in any method provided by component. For now I can doing this by saving manually to AsyncStorage, and get data via props/state, and that's not so clear solution.

I agree with you Meteor.userId() use case, but in you Meteor.user() use case, why don't you just :
getMeteorData() {
   return { user: Meteor.user() };
}
render(){
   const { user } = this.data;

   return (
     <NavigationBar
     title="My Profile"
     renderLeftButton={<Text>user.name</Text>}/>
   );
}

??
Meteor is not saving Meteor.user(), so I'm do not really like adding Meteor.user() to AsyncStorage...

yeah the Meteor.user is not something that important, I just think that it was easier like that. But the Meteor.userId() in subscriptions etc, i think should be done :)

Looking into this this week-end ;)

Sorry but I couldn't add Meteor.userId() to AsyncStorage, because Async is asynchronous.
But now you can do that :
startMeteorSubscriptions(){
  Meteor.subscribe('todos', {createdBy: Meteor.userId()});
}

Reactive variables are now accepted in startMeteorSubscriptions ;)
In Beta13

I'll give this a try. Hope this not make any memory leak, because reactive variable always called too many time and cannot be stopped

working like charm! I can get Meteor.userId() from everywhere and it also able to help hack something like onReady in subscriptions etc.
Thanks @Mokto


Rename AuthorizationServerEndpoints to AADEndpoints

Rename AuthorizationServerEndpoints to AADEndpoints.
Add tenantId in AADEndpoints.
Sort lines in spring.factories.

/azp run java - spring - tests

/azp run java - spring - tests


Unhandled exception in ConsolePal.ReadKey

Steps to reproduce
I don't have a consistent repro, sorry. See version details below. Typed c space d backspace backspace, pwsh hung for about 10 seconds then produced error below and exited
PS C:\onebranch> c
An error has occurred that was not properly handled. Additional information is shown below. The PowerShell process will exit.

Unhandled Exception: System.InvalidOperationException: Cannot read keys when either application does not have a console or when console input has been redirected. Try Console.Read.
   at System.ConsolePal.ReadKey(Boolean intercept)
   at System.Console.ReadKey(Boolean intercept)
   at Microsoft.PowerShell.PSConsoleReadLine.ReadOneOrMoreKeys()
   at Microsoft.PowerShell.PSConsoleReadLine.ReadKeyThreadProc()
   at System.Threading.Thread.ThreadMain_ThreadStart()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

Expected behavior
Don't crash
Environment data
> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.1.0
PSEdition                      Core
GitCommitId                    6.1.0
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

We need repo.

Can you open this in the PSReadLine repo?  PSConsoleReadLine is coming from PSReadLine

@daxian-dbw Could you please make a conclusion about the issue while you is on the wave?


fix sidebar paths

https://github.com/phuochungus/HomeLand_Apartment_Management/issues/65
https://github.com/phuochungus/HomeLand_Apartment_Management/issues/63
https://github.com/phuochungus/HomeLand_Apartment_Management/issues/62
https://github.com/phuochungus/HomeLand_Apartment_Management/issues/64

https://github.com/phuochungus/HomeLand_Apartment_Management/issues/57
https://github.com/phuochungus/HomeLand_Apartment_Management/issues/58
https://github.com/phuochungus/HomeLand_Apartment_Management/issues/56

https://github.com/phuochungus/HomeLand_Apartment_Management/issues/53

https://github.com/phuochungus/HomeLand_Apartment_Management/issues/55


DBZ-6013: Data will miss when use archive log only mode with Oracle RAC

DBZ-6013
Data will miss when use archive log only mode with Oracle RAC
The cause of the problem is that there is an overlap between the SCN range contained in the redo logs of the two threads。
We should check the SCN range of online redo log and the current  startScn  and endScn of the connector.
1). If the startScn is contained both  in a archived log and online redo log, we should pause the task and wait untill the online redo log is archived.
2). We need limit endScn to no more than the smallest scn in all online redo logs when calculate endScn.

Hi @LangYuanzh, could you please run the following command locally and commit the format changes:
./mvnw clean install -DskipITs -DskipTests -Passembly,oracle-all,oracle-tests


OS X WriteCharacteristic requires GODEBUG=cgocheck=0

When running code that uses the WriteCharacteristic() function, you must use the GODEBUG=cgocheck=0 flag or else you receive the error panic: runtime error: cgo argument has Go pointer to Go pointer
This is the same issue as https://github.com/paypal/gatt/issues/76

Note that ReadCharacteristic() can run without this flag without error.

Seems this is no longer an issue in Golang 1.8.x+ so I am going to close now.


Can't save change

Describe the Bug with repro steps

Open my App
See there's a new UI (glad to see it's finally happening but wasn't this announced a very long time ago?)
Delete an action and click Save
See Save confirmation toast at top-right corner
See Save button become active immediately as if nothing were saved
Try again and get same result
Switch to old UI
Successfully save

Thanks!
What type of Logic App Is this happening in?
Consumption (Portal)
Are you using Preview Designer or GA Designer
Preview
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Edge
Additional context
No response

This is in the known issues list. We'll have a fix out for it soon.


Why Callkit redirect user to the mobile app after accepting the call?

Description
I have a react-native application that uses a React-Native Plugin (react-native-twilio-programmable-voice) to handle Twilio communication.
Whenever a call comes in , User clicks on accept call button and call is get accepted but immediately at the same time user is routed back to the App while the call screen goes in the background but I want my app to stay on the Call Screen but no to move user to the mobile app.

If User taps on the Topbar ( Red line as shown in the Attached image ) It  takes the user back to the app which shouldn't be happening
Steps to Reproduce
Launch the application on the iPhone with iOS 14.x and log in. Make sure Twilio is registered
Call the phone.
Accept the call.
Expected Behavior
That user should stay on the call screen of the call kit.
Actual Behavior
When user answers the call he/she routed to the mobile app after accepting the app.
Reproduces How Often
Always
Versions
Voice iOS SDK
5.5.1
Xcode
12.3.0
iOS Version
14.2.0
iOS Device
iPhone 7 Plus

There doesn't seem to be any CallKit API that allows application to config such behavior.

There doesn't seem to be any CallKit API that allows application to config such behavior.


Active EQA

Thank you so much for open sourcing the project. I was very interested in the Active EQA task in your paper, but I don't seem to see the Active EQA baseline in this repository. Is it convenient to provide Active EQA related codes?

I am also very interested in the Active part of the benchmark.

Same question

Same question

Same question

same


Burve Protocol W3F Grant Application

Project Abstract

Burve Protocol is a cutting-edge AMM protocol designed to support a wide range of cryptocurrency use cases, from simple memecoin launches to sophisticated AMM solutions. Our protocol addresses the limitations of existing AMMs, such as initial liquidity challenges and potential manipulation, by integrating bonding curves inspired by S. De la Rouvier’s work. This allows for a more inclusive and efficient financial ecosystem, offering a seamless pathway for a token’s journey from creation and minting to exchange.

Grant level

[ ] Level 1:  Up to $10,000, 2 approvals
[x] Level 2:  Up to $30,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied and aptly renamed (project_name.md).
[x] I have read the application guidelines.
[x] Payment details have been provided (Polkadot AssetHub (DOT, USDC & USDT) address in the application and bank details via email, if applicable).
[x] I understand that 30% of each milestone will be paid in vested DOT, to the Polkadot address listed in the application.
[x] I am aware that, in order to receive a grant, I (and the entity I represent) have to successfully complete a KYC/KYB check.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
[ ] I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @kari19says:matrix.org (change the homeserver if you use a different one)

Thanks for the application @WenSwen199a

It appears that the markdown table didn't work for milestone 2. Can you please fix this?
We don't support the funding of audits. Therefore M1.6 can be removed.
It appears that Burve Protocol is already live and well-established. What new components would you be building for Polkadot integration?
Would the smart contracts need to be migrated or built from scratch?
Would all code be open-source?

Hey @WenSwen199a ,
there are some default deliverables that need to be included in your milestones. E.g. testing guide and license. Please check our website and FAQ.

pinging @WenSwen199a

Thank you @keeganquigley @PieWol @semuelle !

Milestone 2 table fixed
Milestone 1.6 audit deliverable deleted
[-] will add default deliverables to milestones tomorrow

will add default deliverables to milestones tomorrow

pinging @WenSwen199a

Hey @WenSwen199a ,
are there any updates you are able to share with us?


[Bug report]: Line-endings of the trickest top-10000.txt wordlist

Describe the bug:
When cloning either this repository, or the original Seclists repository on Windows, the file "top-10000.txt" always shows up as modified after being cloned or doing a "git pull".

To reproduce:

git clone https://github.com/ItsIgnacioPortal/SecLists-Express
git status

Expected behaviour:

Files shouldn't show up as "modified" unless they are manually modified by a contributor on their local copy of the repository.

Additional context:

Many solutions have been attempted to solve this issue, but it seems to persist.

Solution 1 - Adding --renormalize to the .github/workflows/remote-wordlists-updater.yml workflow. See: cc38b6f6697d78ed889d7b89747a8888f8220e51
Solution 2 - Manually renormalizing the wordlist. See: https://github.com/danielmiessler/SecLists/pull/1003
Solution 3 - Adding a redundant git add --renormalize to the .github/workflows/remote-wordlists-updater.yml workflow to deal with CRCRLF edge-case line endings. See: 839a56fd91ae7e4bdec014c7f24dccf395a93b4c
Solution 4 - Adding the line Discovery/Web-Content/trickest-robots-disallowed-wordlists/top-10000.txt text eol=lf to the .gitattributes file so the same type of line endings would be used on all systems for the top-10000.txt wordlist. See: 3e78005167e38624e97c781a9de068499fbd6443 and https://docs.github.com/en/get-started/getting-started-with-git/configuring-git-to-handle-line-endings

Despite all of these attempts at fixing the issue, the problem persists.
@righettod @molangning do you know what could be causing this issue?

Mo Langning's proposal of running git rm -rf --cached . && git reset —hard HEAD does seem to work as a bandiad solution, but it gets undone when either of these things happen:

The repo is cloned again
User runs git pull
User creates a new branch before running the fix, runs the fix, and then switches from that branch back to the main branch.

To be honest, I have no idea on such issue 🤔

This issue is likely to be related with inconsistencies in the UTF-8 encoding of the file, as discussed in https://github.com/facelessuser/RawLineEdit/issues/29


WCF service crashes when RabbitMq service is stoped

I implemented a wcf service on .net 4.6 with masstransit <IP_ADDRESS>. The service is publish only, no consumers.
When service is running if I stop RabbitMq service, my WCF service crashes.  My service has to be restarted when RabbitMq service comes back up, to return it to running state. Other services written with .Net core do not have these issues. So it's just full framework issue.
How can we resolve this issue.
System.IO.FileLoadException: Could not load file or assembly 'Microsoft.Diagnostics.Tracing.EventSource, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
File name: 'Microsoft.Diagnostics.Tracing.EventSource, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'

An unrelated issue,  wcf service creates a temporary queue every time I start the service. I tired configuring queue with an endpoint with no consumers, but the temporary queue is created regardless. Is there a way to resolve this?

Are you calling Start (or StartAsync) on the bus instance when your application starts? If not, that could be a related issue. I'm not sure about the assembly load issue for the diagnostic tracing, since you're using the full framework - that is only used with .NET core.
The temporary queue is always created for a bus instance.

Yes I call start, thats why it creates the queue. I can also route messages, but if I stop rabbitmq service the wcf service crashes with that exception. Further search showed that its related to rabbitmq client library reference issue. Not sure how to resolve or what masstransit versions to use.

I don't know how you can resolve it, to be honest. The .NET framework version of MassTransit doesn't use that assembly.

Do you have a MVC or WCF sample on the full framework I can try this on.?


Upgrade - Update code to the new content renderer (dupe)

See https://github.com/prezly/content-renderer-react-js/pull/19

Is this a duplicate of https://github.com/prezly/theme-nextjs-bea/pull/119?

@e1himself should this PR be closed in favour of yours?
If yes, could you increase the slate renderer version number

@aspirisen Yeah, I was just thinking about which one it's better to keep. Let me upgrade the package version in #121.

Good, by the way, why i have failed nextjs builds, maybe i should have access to it? @kudlajz

@aspirisen I'll grant you access. But JFYI I am sure you can reproduce the error locally by running npm run build.

It failed because of
Type error: Module '"@prezly/content-renderer-react-js"' has no exported member 'Html'.

I'll grant you access.

Nope, I can't. Please ask Camil to let you in.


Docker containers with the sample SQL schema?

We could have a Docker container per DB type (built on top of an existing base image) with a way to start with a fresh copy of the schema (pulled from this repo automatically). This would be useful to both authors and editors when testing the SQL scripts in an article, to have a reproducible environment.

Hi @theangrydev
I've added a JIRA task to look into this.

Hi @lor6
I think we can close this issue.


Copy to final destinaion feature

It would be nice to be able to have plots created on final directory (ssd) but then moved to HDD.
For example, I currently use madmax and using a secondary temp drive didn't help much but when I had temp and destination on my nvme drives, I shaved off 7 minutes. I am however using the "plot whilecopy" which is nice too.
thanks for the hard work.

I'll have to test to see if there actually an improvement with ssd as destination since I'm using the "plot while copy" feature.

I think I know what you mean, you kinda want a buffer drive where you place plots quickly while plotting and offload them to your HDD from there. I do want to implement a "Mover" option but have not done so yet. The "Plot While Copy" sort of does that in way, but having that and a Mover would be nice.

I guess "mover" is a much more clear and simple way to describe what i wrote above. haha. but yea that is exactly what i meant. I'm glad that's in the works. Great work btw, donated already :D


Want to make issue tamplete for this repo

Assign me under Aperture'24

Very well but do not make it hard.

Best of Luck. Happy Coding.


[noco] Retrieve video language according to user options

Get the user audio language preference in order to retrieve video with chosen language instead of 'fr'.

What is the benefit of this improvement?

Current version of Noco extractor only get a show with French as audio language. Some shows are available in multiple language, for example French and Japanese for some animes.
With this modification, the Noco extractor will get the show audio language according to your account preferences and fallback to French as previously if your preferred audio language isn't available for the show.

Can you add a test video with multiple languages?

Thanks.


Add @mixin Query\Builder to EloquentBuilder.stub

[ ] Added or updated tests (in progress, failing tests at the moment)
[x] Documented user facing changes (not needed)
[x] Updated CHANGELOG.md

Changes
Resolves #1346 as @xwillq is proposing in the issues comments.
As larastan is reading docblocks from stubs instead from source code, we need to add the mixins from Query\Builder in the Eloquent\Builder stub.
Reference: Laravel 9 docblock source code of Eloquent\Builder
Breaking changes
No breaking changes, only functionality is added

Hi,
The underlying issue of #1346 is more complicated. And this fix is not correct. We already have an extension that handles the normal query builder calls on Eloquent builder.
Thank you though.


Weird Linking

I guess this is related: https://github.com/foambubble/foam/issues/585

Still no info after a few months so unfortunately doesn't help much :/


Hyper support

Does YADR support the Hyper terminal? I've found that the vim color sets of YADR look no good in Hyper.

Setting theme to 'hyper-solarized-dark' should fix the issue.


Deprectation Notices

Hey, I noticed some deprecation notices when running composer update in my symfony2 application.
Deprecation Notice: The callback Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::buildBootstrap declared at /var/www/foo/vendor/sensio/distribution-bundle/Sensio/Bundle/DistributionBundle/Composer/ScriptHandler.php accepts a Composer\Script\CommandEvent but post-update-cmd events use a
Composer\Script\Event instance.
Please adjust your type hint accordingly, see https://getcomposer.org/doc/articles/scripts.md#event-classes in phar:///usr/local/bin/composer/src/Composer/EventDispatcher/EventDispatcher.php:289
Deprecation Notice: The callback Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache declared at /var/www/foo/vendor/sensio/distribution-bundle/Sensio/Bundle/DistributionBundle/Composer/ScriptHandler.php accepts a Composer\Script\CommandEvent but post-update-cmd events use a Composer\Script\Event instance. Please adjust your type hint accordingly, see https://getcomposer.org/doc/articles/scripts.md#event-classes in phar:///usr/local/bin/composer/src/Composer/EventDispatcher/EventDispatcher.php:289
Deprecation Notice: The callback Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::installAssets declared at /var/www/foo/vendor/sensio/distribution-bundle/Sensio/Bundle/DistributionBundle/Composer/ScriptHandler.php accepts a Composer\Script\CommandEvent but post-update-cmd events use a Composer\Script\Event instance. Please adjust your type hint accordingly, see https://getcomposer.org/doc/articles/scripts.md#event-classes in phar:///usr/local/bin/composer/src/Composer/EventDispatcher/EventDispatcher.php:289
Deprecation Notice: The callback Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::installRequirementsFile declared at /var/www/foo/vendor/sensio/distribution-bundle/Sensio/Bundle/DistributionBundle/Composer/ScriptHandler.php accepts a Composer\Script\CommandEvent but post-update-cmd events use a Composer\Script\Event instance. Please adjust your type hint accordingly, see https://getcomposer.org/doc/articles/scripts.md#event-classes
in phar:///usr/local/bin/composer/src/Composer/EventDispatcher/EventDispatcher.php:289
Deprecation Notice: The callback Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::prepareDeploymentTarget declared at /var/www/foo/vendor/sensio/distribution-bundle/Sensio/Bundle/DistributionBundle/Composer/ScriptHandler.php accepts a Composer\Script\CommandEvent but post-update-cmd events use a Composer\Script\Event instance. Please adjust your type hint accordingly, see https://getcomposer.org/doc/articles/scripts.md#event-classes
in phar:///usr/local/bin/composer/src/Composer/EventDispatcher/EventDispatcher.php:289^
Is there something broken?
I'm using symfony 2.8 and sensio/distribution-bundle 2.3
Best
Christian

sensio/distribution-bundle 2.3 is totally outdated. The current stable release is 5.0.6

Ok, I updated to ~5.0 and notices are gone now. Thank you.

Well with a SF2 installation I guess everything is outdated now.
Shouldn't we still be able to do composer installs or updates ?

@Glideh the fact that composer displays a deprecation notice does not prevent composer install from running.

Ho sorry that's right, I was mixing with another issue.
I going to remove my comments.


Add custom SSL certificates configuration

Allow users to configure a custom certificate for every repository

Config http.sslVerify not working on first clone


What is the meaning of "requires True" in lemmata?

F* code seems to often include requires True in a lemma - what is this for? This clause appears several times even in the tutorial, without an explanation.

On explanation could be that while F* supports some abbreviations for lemmas (e.g. Lemma (ensures (P)) or Lemma (P) instead of Lemma (requires (True)) (ensures (P)), it does not work when you add extra clauses (typically a decrease clause or a SMTPat clause)
To use one of those you may need to add a trivial precondition.


[Croissant] JSON-LD requests are full=true by default.

waiting for https://github.com/huggingface/moon-landing/pull/8794 (internal) before merging and deploying


ThrottlesLogin trait missing

Laravel Version: 7.4.0
PHP Version:  7.3.13
Database Driver & Version: -

Description:
I am in the process of upgrading to laravel 7, and it seems that currently Illuminate\Foundation\Auth\ThrottlesLogins; is missing
Missing in 7.x: https://github.com/laravel/framework/blob/7.x/src/Illuminate/Foundation/Auth/ThrottlesLogins.php
But available in 6.x: https://github.com/laravel/framework/blob/6.x/src/Illuminate/Foundation/Auth/ThrottlesLogins.php).
Looking at the upgrade guide, there is no mention of it: https://laravel.com/docs/7.x/upgrade
Is it moved? Or deleted?
Steps To Reproduce:

See in: https://github.com/laravel/ui/blob/2.x/auth-backend/ThrottlesLogins.php

I had the same problem and the above answer was not clear what was going on, but it's as simple as doing:
composer require laravel/ui
For some reason the file has the same namespace rather than being under a laravel/ui namespace so after that there's nothing you need to do if you were already using it.
Also copying the error message I had for easier googling since it took me a while to find this:
"Trait 'Illuminate\Foundation\Auth\ThrottlesLogins' not found"


A Series Of Suggestions

-1.19.4 Support
-Plugin Support (As in mineflayer scripts so that you can run any mineflayer script you want)
-Other basic functions (bodyguarding a player, add 'friends' to killaura (people to ignore), following a player, mining a specific block, area mining out an area, following a certain entity (ANYPLAYER, ANYMOB, or ANYVEHICLE)
Thats about it, since this runs on mineflayer , it should be pretty easy for the "other basic functions", and there is obviously tons of code for everything I listed already out there. As for the other 2 should be relatively simple!

Why did you ignore my issue 😆 @RattlesHyper

1.19.4 support will be added as soon as possible. Other features are noted.

Thank you, forgot to mention this feature as well: travelling to certain coordinates


Firewall configuration wrapper script using iptables-restore to apply rules

This pull request adds a firewall-update.sh script for wrapping the existing firewall.conf.
The wrapper script collects the rules for application on completion of the wrapper. The wrapper checks for changes before using iptables-restore to apply, so will result in the FILTER table being flushed less frequently. Due to the reduced flush any packed/byte counters would not be frequently reset, which is a change in behaviour (but seems preferable)
When the wrapper is used we no longer flush tables other than the FILTER table so it would be up to the user to ensure these were correct for use with an AppScale deployment.

Daily build no. 6694

Is there a reason the tempfile is needed instead of changing the format of the firewall.conf file itself?

The firewall.conf file is a bash script that gets run to generate rules. I'm not sure if we could eliminate the need to template/generate the firewall configuration (using ipsets for hosts in the cluster for example)
Longer term, we would probably want to move away from using iptables directly (perhaps firewalld)

One thing that needs to change with this pull request is using only:
iptables -A INPUT -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT

and removing the older:
iptables -A INPUT -m state --state ESTABLISHED,RELATED -j ACCEPT

either should generally work, but the check for a failing iptables command does not make sense with the wrapper (which will accept all rules and apply them later)

Eventually firewall.conf needs a proper home in the src tree other than at the root level.

I think I'm with Chris here (and with Steve on the long term): why don't we switch firewall.conf to be in the format of ipdates-save so we can use iptables-restore directly? The only script that modifies it is the datastore viewer script. This way we remove a shell script (we use iptables-restore instead) and we just have a config file instead.

Closing this pull request. We can look at simpler solutions with static firewall configuration.


fix: bad colors in dialog when using dark theme

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[x] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[ ] Documentation content changes

What is the current behavior?
Closes #1334
What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

White background in dark theme looks really bad, in my opinion. I think right way is to change wrapper background color in dark theme, not font color

But I dont know is it important to use exactly #f6f7f8 color or not, its better to ask designer

@vladimirpotekhin fixed


share button on photo browser with RTL direction

share button on photo browser work fine when direction is LTR But with RTL direction for arabic direction it doesn't work at all. Any solution please !

try changing line number : 37 in SKToolbar.swift
from:
if SKMesurement.screenWidth - point.x < 50 { // FIXME: not good idea
to
if SKMesurement.screenWidth - point.x < 50 ||  point.x < 50 { // FIXME: not good idea

You found solution for this or not yet muhammadalkhalaf ?


chore(gulp): maintain a single set of release artifacts under one directory

Changes the behavior of gulp release-prepare such that a new version-specific directory is not created under release/ unless a new --with-dir flag is given on the command line but rather release artifacts are overwritten in the release/ dir for each new release.
Problem
The current release process creates a new version-specific directory for each release in the git repo, which means each release adds more "perpetual" static content and grows the size of the repo unnecessarily each time we create a release. Right now we don't use these release artifacts, so it seems wasteful to let them pile up in the repo, especially since each tag will contain the release artifacts for every previous release.
Possible Solutions

Do not add release artifacts to the git repo at all
As reflected by commit 91234fe (when the --with-dir CLI flag is not given). This was just the simplest thing I could think of, but after talking with @aredridel, I realized it may not be the best solution.

Maintain a single release/ directory in the git repo and have the release process (gulp tasks) overwrite its contents on each new release
As reflected by commit a0761d9. To me, this option (along with the assumption of tagging releases) is probably best, since it would still make version-specific release artifacts accessible from GitHub via tag references like https://raw.githubusercontent.com/npm/pivotal-ui/v5.1.2/release/pivotal-ui.min.css but would not accumulate release artifacts for every prior release in the master branch.

Discussion
This PR currently represents what I think is close to the best solution, but I also want to facilitate discussion about the problem and, hopefully, agreement about an appropriate solution. Please add comments with your feedback.

I totally love this. As @nexdrew and I talked, I really think solution 2 is a great place to be. I don't love that we commit it with the main repo, as it's awful reflexive -- I prefer a derivative branch -- but as pure practicality, this works delightfully. And it's so much better thant the status quo.


Unable to login after install

Core_users table is empty
Traceback (most recent call last):   File "manage.py", line 10, in <module>     execute_from_command_line(sys.argv)   File "/srv/modoboa/env/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 353, in execute_from_command_line     utility.execute()   File "/srv/modoboa/env/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 345, in execute     self.fetch_command(subcommand).run_from_argv(self.argv)   File "/srv/modoboa/env/local/lib/python2.7/site-packages/django/core/management/base.py", line 348, in run_from_argv     self.execute(*args, **cmd_options)   File "/srv/modoboa/env/local/lib/python2.7/site-packages/django/core/management/base.py", line 399, in execute     output = self.handle(*args, **options)   File "/srv/modoboa/env/local/lib/python2.7/site-packages/modoboa/core/management/commands/load_initial_data.py", line 31, in handle     admin.set_password("password")   File "/srv/modoboa/env/local/lib/python2.7/site-packages/modoboa/core/models.py", line 164, in set_password     "PasswordUpdated", self, raw_value, self.pk is None   File "/srv/modoboa/env/local/lib/python2.7/site-packages/modoboa/lib/events.py", line 90, in raiseEvent     callback(*args, **kwargs)   File "/srv/modoboa/env/local/lib/python2.7/site-packages/modoboa_pdfcredentials/modo_extension.py", line 39, in password_updated     credentials(account, raw_password)   File "/srv/modoboa/env/local/lib/python2.7/site-packages/modoboa_pdfcredentials/documents.py", line 54, in credentials     story.append(resized_image(imgpath + "/modoboa.png", 6*cm))   File "/srv/modoboa/env/local/lib/python2.7/site-packages/modoboa_pdfcredentials/documents.py", line 32, in resized_image     img = utils.ImageReader(path)   File "/srv/modoboa/env/local/lib/python2.7/site-packages/reportlab/lib/utils.py", line 807, in __init__     annotateException('\nfileName=%r identity=%s'%(fileName,self.identity()))   File "/srv/modoboa/env/local/lib/python2.7/site-packages/reportlab/lib/utils.py", line 1387, in annotateException     rl_reraise(t,v,b)   File "/srv/modoboa/env/local/lib/python2.7/site-packages/reportlab/lib/utils.py", line 771, in __init__     self.fp = open_for_read(fileName,'b')   File "/srv/modoboa/env/local/lib/python2.7/site-packages/reportlab/lib/utils.py", line 669, in open_for_read     raise IOError('Cannot open resource "%s"' % name) IOError: Cannot open resource "/srv/modoboa/instance/sitestatic/css/modoboa.png" fileName='/srv/modoboa/instance/sitestatic/css/modoboa.png' identity=[ImageReader@0x7f8380b03bd0 filename='/srv/modoboa/instance/sitestatic/css/modoboa.png']

@padred123 Have you tried recently ?

Yes right before I posted this I did a git clone https://github.com/modoboa/modoboa-installer and reran the installation. My Ubuntu 14.04 isn't fresh so maybe something is throwing it off but all of the dependencies appeared to install correctly.

Yes, you have to upgrade your dependencies as described in https://github.com/modoboa/modoboa-installer/issues/39, if that helps.

@almereyda Worked like charm! Thx!


[Backend] User, UserMaterial, ThreeDP Resolver

What is your new feature? Please describe.
A clear and concise description of what the feature is.
Describe How will you do it
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Closed due to duplicated issues


Audit fixes

In advance of the virtual sprint at the end of September, updates for auditing:

Make it so that curators can access audit history (check perms)
Change page layout for audit history to be helpful for logged in curators

@taniadoa did I forget anything on this that I said I'd do?

That was it! Thank you!

I created a new page layout for curators and adjusted the Experience layout for the audit object.
I updated curator profile permission to have read access to the audit object and added a sharing setting to give the Curator Public Group read access to all audit records.

I added a new picklist value for audit status and updated the audit flow to use the new status. The flow uses a formula to set the audit status based on the information in the screen flow, so that's where the update was made. This formula is used to create the new audit record and on the "Last Audit Status" field on the Resource. I updated the existing resources and audit records to use the new status instead of the "FAIL" status. Deactivated the FAIL picklist value.


Hover events likely shouldn't raise if an active drag is in progress

Checked that this is standard behaviour on macOS and windows
https://user-images.githubusercontent.com/191335/165684195-5fd49b7f-ac21-4819-ab29-4544395342a0.mp4
https://user-images.githubusercontent.com/191335/165684538-630f5599-3768-431a-aef1-6c255544b97c.mp4
Doesn't feel great

It depends on the context though. For example it will raise hover events if you're dragging files around in Finder (or on the desktop), whether it is onto other folders or onto the sidebar tree.

That is true, but I think it is the unlikely case for o!f since these hover events come from whether the hovered element accepts receiving the dragged element and handling it. Maybe we can add a rule against this once o!f supports "drop targets"

Drag/drop targets are definitely an edge case. but we don't really have anything like that implemented yet. maybe something to consider when addressing this issue though.


WinUi 3 - Combobox inconsistency in Windows 10 Enterprise LTSC Deployment Machine

Describe the bug
We are using multiple Combobox in our application.  It is working as expected in all most all the deployment machines.
We are seeing some inconsistency with Windows 10 Enterprise(build 17763). Some of the Combobox are not working as expected. sometimes the Combobox are not populated or It is not triggering the events in case of selection changed.
This issues we are only seeing in the Windows 10 Enterprise(build 17763)
Steps to reproduce the bug
It is inconsistent.
Expected behavior
No response
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.1.5
Windows version
Windows 10 (1809): Build 17763
Additional context
No response

@ravi-chandra-t, can you help provide a simple repro app for this issue?


Menu items should hide based on transition settings when using dropdown-content-expand

Menu dropdowns using the class the dropdown-content-expand class for mobile should use the currently set transition type and duration.

Fixed.


Feature request: support for unquoted destructuring assignment

In ADVANCED mode, the following code produces a questionable output:
const obj = JSON.parse('{ "foo": "hello", "bar": "world" }')
console.log(obj)            // Output: {foo: "hello", bar: "world"}
const { foo, bar } = obj
console.log(foo, bar)       // Output: undefined undefined

The workaround is to replace line 3 with:
const { ['foo']: foo, ['bar']: bar } = obj

However, this feels illogical. I think a destructuring assignment always corresponds to a search-by-name intent, so the names should be left untouched by the closure compiler.

FYI, the square brackets aren't needed.
const { 'foo': foo, 'bar': bar } = obj;

repro
https://closure-compiler.appspot.com/home#code%3D%252F%252F%2520%253D%253DClosureCompiler%253D%253D%250A%252F%252F%2520%2540compilation_level%2520ADVANCED_OPTIMIZATIONS%250A%252F%252F%2520%2540output_file_name%2520default.js%250A%252F%252F%2520%2540formatting%2520pretty_print%250A%252F%252F%2520%253D%253D%252FClosureCompiler%253D%253D%250A%250A%252F%252F%2520ADD%2520YOUR%2520CODE%2520HERE%250Aconst%2520obj%2520%253D%2520JSON.parse('%257B%2520%2522foo%2522%253A%2520%2522hello%2522%252C%2520%2522bar%2522%253A%2520%2522world%2522%2520%257D')%250Aconsole.log(obj)%2520%2520%2520%2520%2520%2520%2520%2520%2520%2520%2520%2520%252F%252F%2520Output%253A%2520%257Bfoo%253A%2520%2522hello%2522%252C%2520bar%253A%2520%2522world%2522%257D%250Aconst%2520%257B%2520foo%252C%2520bar%2520%257D%2520%253D%2520obj%250Aconsole.log(foo%252C%2520bar)%2520%2520%2520%2520%2520%2520%2520%252F%252F%2520Output%253A%2520undefined%2520undefined%250A

FYI, the square brackets aren't needed.

@brad4d, you are right, thanks. But notice that prettier (and VS Code) removes the quotes when there is no bracket, without any mean to disable this behavior. See this thread.

Better repro with more explanation in comments.
https://closure-compiler.appspot.com/home#code%3D%252F%252F%2520%253D%253DClosureCompiler%253D%253D%250A%252F%252F%2520%2540compilation_level%2520ADVANCED_OPTIMIZATIONS%250A%252F%252F%2520%2540output_file_name%2520default.js%250A%252F%252F%2520%2540formatting%2520pretty_print%250A%252F%252F%2520%253D%253D%252FClosureCompiler%253D%253D%250A%250A%252F%252F%2520ADD%2520YOUR%2520CODE%2520HERE%250Aconst%2520obj%2520%253D%2520JSON.parse('%257B%2520%2522foo%2522%253A%2520%2522hello%2522%252C%2520%2522bar%2522%253A%2520%2522world%2522%2520%257D')%250Aconsole.log(obj)%2520%2520%2520%2520%2520%2520%2520%2520%2520%2520%2520%2520%252F%252F%2520Output%253A%2520%257Bfoo%253A%2520%2522hello%2522%252C%2520bar%253A%2520%2522world%2522%257D%250A%250A%252F%252F%2520This%2520is%2520equivalent%2520to%250A%252F%252F%2520let%2520foo%2520%253D%2520obj.foo%252C%2520bar%2520%253D%2520obj.bar%253B%250A%252F%252F%2520compiler%2520warns%2520that%2520as%2520far%2520it%2520knows%250A%252F%252F%2520obj.foo%2520and%2520obj.bar%2520are%2520undefined%250Alet%2520%257B%2520foo%252C%2520bar%2520%257D%2520%253D%2520obj%250A%250A%252F%252F%2520because%2520they%2520are%2520unquoted%252C%2520properties%2520foo%2520and%2520bar%2520get%2520renamed%252C%250A%252F%252F%2520but%2520then%2520they%2520no%2520longer%2520match%2520the%2520literal%2520%2522foo%2522%2520and%2520%2522bar%2522%250A%252F%252F%2520properties%2520in%2520the%2520JSON%2520object.%250Aconsole.log(foo%252C%2520bar)%2520%2520%2520%2520%2520%2520%2520%252F%252F%2520Output%253A%2520undefined%2520undefined%250A%250A%252F%252F%2520this%2520way%2520works%2520correctly%250A(%257B%2520'foo'%253A%2520foo%252C%2520'bar'%253A%2520bar%2520%257D%2520%253D%2520obj)%253B%250Aconsole.log(foo%252C%2520bar)%253B%250A
This is working as intended.
The compiler cannot just decide that property names used in destructuring should be treated as quoted.
This would break destructuring cases where the code was designed with the intention that the properties be renamed.
Also note that the compiler does complain that you're accessing properties that appear to it to be undefined.

@steph643 Thanks for the link to the prettier issue. I was unaware of that problem.

This would break destructuring cases where the code was designed with the intention that the properties be renamed.

My initial post was denying the possibility that such intention ever exists:

I think a destructuring assignment always corresponds to a search-by-name intent

So it seems I was wrong. Would you be kind enough to explain me where?

One of the major goals of using ADVANCED mode is to be able to rename properties below. This code should be completely safe in ADVANCED mode and should rename the properties.
const obj = {foo: 'foo', bar: 'bar'};
const {foo, bar} = obj;
console.log(foo, bar, obj.foo, obj.bar);

Or to say another way, the only reason your code above had problems is you used JSON.parse so the original properties were not visible to the compiler.

Thanks, @ChadKillingsworth, I understand.
For the record, this code shows the issue without relying on JSON.parse:
const obj = { 'foo': 'hello', 'bar': 'world' }
console.log(obj)              // Output: {foo: "hello", bar: "world"}
const { foo, bar } = obj
console.log(foo, bar)       // Output: undefined undefined

Same for this one:
function doIt({ foo, bar }) {
  console.log(foo, bar)
}
doIt({ foo: 'hello', bar: 'world' })     // Output: hello world
doIt({ 'foo': 'hello', 'bar': 'world' }) // Output: undefined undefined


Feature/wjamieso/mapl refactor

Added in Aerosols_GridComp. Adjusted directory structure.

Note the legacy GOCART in this repo will never be used in GEOS.  GEOS will switch to this repo when GOCART_2G is complete.


Will it upgrade to the latest opencore?

I don't think I'll do it because I changed my PC in the meantime. In any case, the official guide is available at: https://dortania.github.io/OpenCore-Post-Install/universal/update.html#updating-opencore . I also suggest taking a look at the tool: https://github.com/ic005k/OCAuxiliaryTools because it simplifies manual work. I have already tested this tool personally on the project and with the last update I did not encounter any problems. I hope I was helpful. Good work


{IoT} Fix DPS policy key regeneration when setting empty strings as keys

Related command

az iot dps policy update
Description

Currently the service rejects empty key values of "" as invalid input, this sends the correct None / null values to the service for these values and will successfully regenerate keys for that policy.
Testing Guide

Force service to regenerate primary key:
az iot dps policy update -g {group} --dps-name {dps} --pn {policy} --primary-key ""
Force service to regenerate secondary key:
az iot dps policy update -g {group} --dps-name {dps} --pn {policy} --secondary-key ""
History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

IoT

Hi @c-ryan-k
since we are in a releasing process and the code is frozen during this process, this PR cannot be merged right now. We will merge this PR after the process is completed.

/azp run


JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Hello ; please i run this code :
import json
import requests
url = "http://add_server"
data = json.dumps({'sl':5.84,'sw':3.0,'pl':3.75,'pw':1.1})
r = requests.post(url,data)
print(r.json)
and i get this error :
<bound method Response.json of <Response [500]>>
and when i run this code :
import json
from pprint import pprint
import requests
url = "http://add_server"
data = json.dumps({'sl':5.84,'sw':3.0,'pl':3.75,'pw':1.1})
r = requests.post(url,data)
pprint(r.json)
i get this error:
JSONDecodeError Traceback (most recent call last)
in ()
4 data = json.dumps({'sl':5.84,'sw':3.0,'pl':3.75,'pw':1.1})
5 r = requests.post(url,data)
----> 6 print(r.json())
~/anaconda3/lib/python3.6/site-packages/requests/models.py in json(self, **kwargs)
890 # used.
891 pass
--> 892 return complexjson.loads(self.text, **kwargs)
893
894 @property
~/anaconda3/lib/python3.6/json/init.py in loads(s, encoding, cls, object_hook, parse_float, parse_int, parse_constant, object_pairs_hook, **kw)
352 parse_int is None and parse_float is None and
353 parse_constant is None and object_pairs_hook is None and not kw):
--> 354 return _default_decoder.decode(s)
355 if cls is None:
356 cls = JSONDecoder
~/anaconda3/lib/python3.6/json/decoder.py in decode(self, s, _w)
337
338 """
--> 339 obj, end = self.raw_decode(s, idx=_w(s, 0).end())
340 end = _w(s, end).end()
341 if end != len(s):
~/anaconda3/lib/python3.6/json/decoder.py in raw_decode(self, s, idx)
355 obj, end = self.scan_once(s, idx)
356 except StopIteration as err:
--> 357 raise JSONDecodeError("Expecting value", s, err.value) from None
358 return obj, end
JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Any help please. Thanks

I think an informative exception should be raised in this case.

What's more informative than JSONDecodeError meaning we literally can not decode any JSON from this.
A response with a status code of 500 may return valid JSON. It may not. It's up to the user to know whether that's the case and call response.json() accordingly. response.json() being called with reckless abandon is about as likely to work as the user doing import json; json.loads(response.text) with the same amount of though.


Is there a plugin for frontmatter metadata?

Rather than the existing metadata extension, I'd rather use YAML frontmatter in my markdown files.
Is there a plugin for this?

I've come to conclude that it is better to process front-matter with a separate wrapping preprocessor, rather than an extension to the parser itself. Therefore I have created docdata, which can handle MultiMarkdown style meta data as well as YAML data. In the future I expect this will replace the existing extension (as the separate "wrapper" that it is now, rather than as an extension).
As there is no actionable item here, I am closing this.

@waylan Yep, fair enough. Thanks.
For posterity, here's my solution:
import frontmatter  # python-frontmatter package
import markdown

markdown_parser = markdown.Markdown(extensions=markdown_extensions)
file_parts = frontmatter.load(filepath)
html_content = markdown_parser.convert(file_parts.content)
metadata = file_parts.metadata

@nottrobin Always great when someone shares his solution to save others time. Thanks!


Convert from double to signed data type for reference hamming distance.

@aressem : please review
@arnej27959 , @havardpe : FYI

__builtin_popcount expects an unsigned int argument, thus we would likely need to pass (uint8_t) (x ^ y) to __builtin_popcount to avoid an off by 24 error if x and y were of type int8_t.


Mfa system updates, tests and docs

Moving some functionality out of MFASystem and into simson (please look and consider other possible usecases if this makes sense or not).
closes #2; this pr makes flows and stocks be initialised from process objects.
closes #14.
some testcases added (requires pytest to be installed, tests can be run using python -m pytest).
adds README and sphinx documentation (not yet hosted by readthedocs).

To run the sphinx documentation locally, sphinx, sphinx-rtd-theme, sphinx-pydantic, autodoc-pydantic need to be installed. Then inside the docs directory, run .\make.bat html
You can install the vscode extension Live Server, and then on the sodym\docs\build\html\index.html click on "go live" in the bottom right corner of the vscode editor. This opens the docs in your browser.
Update (30th August)

dynamic stock model refactoring #21
MFASystem as a pydantic model, closes #24

Great stuff! Great to see the docs and tests!

Waiting to merge to resolve this simson pr

I'm honestly a bit overwhelmed by this PR. I think I've looked through most of it (not all of the documentation, yet).
I'm putting a list of comments here:

MFASystem looks great!
Open question: The {name: item} type dictionaries in MFASystem (flows, stocks, ...) could be made a dict-like object, which is initialized with a list, and provides the __getitem__ by name. These objects could also have those items accessible as an attribute if the name allows it, like columns in pandas. Just for future development.
The model_cfg is still a remnant of the horrible implementation of Sankey plotting. We could get rid of it, I think. Sorry I didn't catch that earlier.
The changes to DynamicStockModel also look great. They don't include what I had suggested on constant inflow yet, right? No worries, it's a functional extension, and we can do it later.

I have a few things I'd like to change (slightly), but it doesn't have to be as part of this PR, and some of these are too small for an issue. So I'd suggest we merge this (simson PR review pending) and I'm going from there when I have the time.

I'll discuss these changes with you when/before I make them. At the latest during PR review, hopefully earlier.

* Open question: The `{name: item}` type dictionaries in MFASystem (flows, stocks, ...) could be made a dict-like object, which is initialized with a list, and provides the `__getitem__` by name. These objects could also have those items accessible as an attribute if the name allows it, like columns in pandas. Just for future development.

Yes, we could pass a list that gets transformed to a dict during initialisation. Not sure if this loses some of the clarity of type specifications from pydantic though. I think it does make sense to store flows, stocks as dicts, since it makes look up much faster.

* The model_cfg is still a remnant of the horrible implementation of Sankey plotting. We could get rid of it, I think. Sorry I didn't catch that earlier.

It's also currently holding the ldf_type which is telling us which survival method to use. I think it's also somehow useful to have somewhere to store "extra" options in a very flexible way, although it does feel a bit like a hack :D

* The changes to `DynamicStockModel` also look great. They don't include what I had suggested on constant inflow yet, right? No worries, it's a functional extension, and we can do it later.

No, I tried to do this, but got different results. Something to look at together as a next step.


Error parsing enum values with numbers in them.

Given the following ron file:
Configuration(
    paths: Paths(
        data: "data"
    ),
    video: Video(
        // Video mode options: window, fullscreen, borderless
        mode: window,
        // Refresh rate used in fullscreen mode
        refresh_rate: 60,
        // Vertical sync
        vsync: true,
        // Sets the fullscreen or window resolution. Ignored in borderless mode.
        width: 1920,
        height: 1080,
        // Vertical FOV (float)
        fovy: 70,
        // Anisotropic texture filtering options: off, x1, x2, x3, x4, x8, x16
        anisotropic_filtering: x16,
    ),
    force_level: Some("level0")
)

I get the following error for the anisotropic_filtering enum value (x16):
RONTokenType."(, RONTokenType.) or RONTokenType. expected, got '16'.

The file can properly parsed by the ron deserializer.

Hi, Nils! Thanks for an issue, indeed it looks to be broken right now. I'll do a fix and release it with a next version of a plugin, I'll inform you here when it would be available in a market

I fixed it and you can download it from here now:
https://github.com/madwareru/intellij-ron-remix/releases/tag/v0.2.1
It's also right now on a moderation in a marketplace as well

@hasenbanck a new version should be available today, stay tuned! I'll inform you as soon as it will be ready to download from a marketplace, and if it suits you, I'll close an issue

Okay, the update just poped up in Clion, so that I could test the changes. The bug is fixed on my side. Thank you very much.


Branch for preview 1

It’s probably time to branch for preview 1. We need to:

[ ] Create preview 1 branch release/1.0.0-preview1 (leave the branding as 0.1.0-alpha1 until tomorrow when we’re going to do a publishing dry run and update branding)
[ ] Update master branding to 1.0.0-preview.2
[ ] Update master subscriptions to get .NET from .NET 5 Preview 5 channel (which is about to snap)

Would like to merge https://github.com/microsoft/reverse-proxy/issues/128 first since it's quite breaking.


Module upgradation

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


Update [min/max]PoolSize with the same default value than Play/Hikaricp

As discussed in #41

Thank you, guys. I will do a release just right now.

Release was done here edulify/edulify.github.io@be6554deb6cbbb113c27bd13cd3dcef78c492dd7. Again, thanks for your contributions.


All permissions no access

I am the project site administrator.  I am a portfolio manager. I have all access permissions.  Why do I continue to get the error "Access to the resource is forbidden?

Reference: https://docs.microsoft.com/en-us/power-bi/desktop-project-online-connect-to-data

Thanks for your comment, @cschmid16  This looks like a support request or product question rather than documentation feedback. You might find a solution in the Power BI Community. If not, try our support page.


contents: add linked list implementation in C++

Add std::list as the linked list implementation in C++.

std::forward_list should also be added as std:list is  doubly-linked list

Thank you 🙇🏻


[Job] Could I ask you to help me to integrate it into my project for a payment?

It crashes. I've managed to run this one - https://github.com/vel-jack/GodotLocalNotification but it is only for android.
please, reach me out<EMAIL_ADDRESS>Thank you!

@gen4sp Hello! Now this plugin uses NativeLib for installation. See README for details.


[Feature]: Implement Stanley Controller

Description
Implement node that calculates steering angle based on Stanley controller
Definition of Done
Publishes correct steering angle based on Stanley Controller

Blocked by #113


Episode 0: communicators and groups

Describe what communicators are and why it might be useful to have more than one.
Exercise: computing pi with Monte Carlo.

To follow up:

figures
better style for the exercises
exercises with increasing levels of difficulty
more quiz questions

MPI_Group_excl is missing from the glossary

Some comments in comms-groups-divide-evenly.c refer to ranks1 and ranks2 but those have been commented out for the exercise. We could probably express the task more clearly. Eg. the rst describes the task here, but we could have a comment in the code that repeats that.

comms-groups-divide-evenly.c suggests using MPI_Group_incl but that's not in the lesson text nor the glossary

I don't think comms-groups-divide-evenly benefits from introducing Allreduce to return a cryptic recvbuf. It should be enough to report the rank of the process in MPI_COMM_WORLD, its size, the rank within new_comm, and its size.
It would also be good for the example to tell the student whether it is working correctly, since we know what the answer is if they do it right!


Add menu item Signs > Recently added

...with all entries created in the last 7 days.
...leading to the standard list of search results, but
a) without the search fields on top
b) and with the added two fields at the left "Entry date" and "User name"
See #72, #73

Done, but not visible yet because of Ponyland upgrade

( now also visible on signbank.science.ru.nl )

• Change the contents of this page to all signs for which Proposed New Sign = True.
• Widen the columns so that the list better fills the page

Done?

Final touch: please remove the limit of showing only 20 items; just display all for which 'proposed new sign = true'

http://www.google.nl/imgres?imgurl=http://brettmcfall.com/wp-content/uploads/2015/01/brettmcfall-lovetheboss.png&imgrefurl=http://brettmcfall.com/why-getting-a-better-paying-job-in-2015-is-the-biggest-mistake-you-could-ever-make/&h=700&w=600&tbnid=x2FS1F7m6xshSM:&docid=8nQH4LwcAmRK7M&ei=AGHxVZCTD4iwaorAkLgN&tbm=isch&client=safari&ved=0CDgQMygEMARqFQoTCJCT-LKm7McCFQiYGgodCiAE1w


No module named 'gym_chess.envs'

Hi everyone,
after intallation in my anaconda environment, I want to import full library into Jupyter notebook. However, the submodule gym_chess.envs  cannot be identified, see full trace below. I am using Python 3.8.8.
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_3868/639769641.py in <module>
      4 import Queens
      5 import gym
----> 6 import gym_chess

C:\ProgramData\Anaconda3\lib\site-packages\gym_chess\__init__.py in <module>
      1 from gym_chess.gym_chess import ChessEngine  # rust module
----> 2 from gym_chess.envs import ChessEnvV0, ChessEnvV1, ChessEnvV2  # envs
      3 from gym.envs.registration import register  # to register envs
      4
      5

ModuleNotFoundError: No module named 'gym_chess.envs'

It seems that the installer does not include sub-folders of gym_chess, therefore subpackages cannot be found.

I've run into the same issue. Which folder specifically should I copy&paste the missing package folders into? The /anaconda3/Library one? Or is there a folder with the main gym_chess folder that I should keep the missing folders with? I was trying to find the latter option in the anaconda terminal, but I couldn't find any promising options. Thanks for any help!

I've run into the same issue. Which folder specifically should I copy&paste the missing package folders into? The /anaconda3/Library one? Or is there a folder with the main gym_chess folder that I should keep the missing folders with? I was trying to find the latter option in the anaconda terminal, but I couldn't find any promising options. Thanks for any help!


Removed jQuery from auto provided variables in webpack config

window.jQuery is already defined in autoProvidejQuery function, there's no need to override:
https://github.com/symfony/webpack-encore/blob/v0.9.1/lib/WebpackConfig.js#L271

You are right ... but when we did the transition to Webpack Encore, this didn't work. In fact, the autoProvidejQuery() option is the only Encore feature that never works for me. I'm sure it's not Encore's fault but legacy jQuery code fault ... but still, we must be careful. Could you please double check that everything works in all browsers after removing this? Thanks!

@javiereguiluz at some point, Encore reverted this change, before adding it again (after figuring out why something was not working). I think this explicit stuff might have been added in the demo during the reverting phase.

In fact, the autoProvidejQuery() option is the only Encore feature that never works for me. I'm sure it's not Encore's fault but legacy jQuery code fault ... but still, we must be careful.

there is 2 kind of legacy code handling:

legacy libraries not supporting CommonJS require to get jquery, which are part of your webpack-processed assets. This is the case that autoProvidejQuery is covering
legacy application code in your project, which is not processed by webpack (for instance inline JS in your template) which expects to have a jQuery global variable (with plugins registered in it). autoProvidejQuery is not covering this case (autoProvideVariables is not either).

@javiereguiluz Sorry for the delay, it seems working fine with different browsers.

Thanks @mathop.


Update README.md

Putting default setting in "Host *" directive at the end of file seems to be more effective for OSX system.
However, very good work :clap: :clap:

Weird, but everything is possible.

This is in line with man ssh_config (which reads the same on Mac OSX):

For each parameter, the first obtained value will be used

and

Since the first obtained value for each parameter is used, more host-spe‐
cific declarations should be given near the beginning of the file, and
general defaults at the end.

I propose a first clarification part of Pull Request #12 : https://github.com/eric-brechemier/whosthere/commit/498b86550da64cd1c0b94cf54f92266f00d3405c


add-tailwind-grid-generator

Added Tailwind Grid Generator to the tools list and also added a reference to it in the layout page.

Please submit your tool here: https://github.com/aniftyco/awesome-tailwindcss


Remove lex and parsing from formatter benchmark

Summary
This PR changes the formatter benchmark to only benchmark the formatting time by excluding the lexing and parsing time.
Excluding the lexing and parsing time (which make up more than 50% of the whole formatting time) allows us to more cleary identify performance changes in the formatter itself.
It further helps to gather profiles that don't include the whole parsing noise.

Current dependencies on/for this PR:

main

PR #6547   👈

PR #6548

This comment was auto-generated by Graphite.


Fix link typos

Links pointing to wrong EIP

Thanks for this @biafra23 !
The file got moved so I was unable to merge this PR but I've applied the changes here: https://github.com/safe-global/safe-docs/commit/db0858c1ce8ea45610bd962f0a6cb8161391fad0


[#6018] Move Dotnet generators from Samples to "generators" - dotnet-templates - Echo Bot

Addresses #6018
#minor
Description
This PR adds the dotnet-templates' Echo bot from the BotBuider-Samples repository.
It also includes the root documentation for the dotnet-templates folder.
Detailed Changes

Added HOW_TO_VERSION, LICENSE, and README docs.
Added generators/dotnet-templates/Microsoft.BotFramework.CSharp.EchoBot.
Fixed some StyeCop issues in the code.

Testing
These images show the dotnet-template working.

Pull Request Test Coverage Report for Build 281522

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.02%) to 78.656%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Connector.Streaming/TaskExtensions.cs
1
82.61%

/libraries/Microsoft.Bot.Connector.Streaming/Application/TimerAwaitable.cs
2
85.71%

/libraries/Microsoft.Bot.Connector.Streaming/Application/WebSocketClient.cs
4
71.82%

Totals

Change from base Build 281321:
-0.02%

Covered Lines:
24849

Relevant Lines:
31592

💛  - Coveralls

Hi @BruceHaley, this is the first PR of the series to move the dotnet generators from BotBuilder-Samples. Please let us know if you have any questions.
Thanks!

Are we missing launchSettings.json?

Hi @mrivera-ms, you're right, it was missing. I haven't noticed that it was excluded by the .gitignore file.
Thanks!

/azp run BotBuilder-DotNet-CI-PR-(MacLinux)-yaml

/azp run BotBuilder-DotNet-CI-PR-(MacLinux)-yaml


Async / Await with IObserver

So, I have a very simple eventing system which is surprising difficult it seems to make work right with Rx.NET.  I have producers and consumers with an at least once message guarantee.  The consumers utilize both IObserverable to pull messages from an event source log and IObserver to handle those messages.  The handling / observing of the messages requires interfacing with external system which are wrapped in API's that return Tasks.  Following best practices we should async / await the tasks all the way up the stack.  Here comes the issue, IObserver does not support returning a Task.  This means some sort of blocking operation must be done against the returned Task in the implementation of the IObserver which violates best practice and introduces a lot of risk for deadlocks and other threading issues depending on hosting platform.  So, should I use Rx.NET or roll my own here.  Is there a solution to this that I’m just not seeing?

Perfect, thanks. This answers my questions.

@ghuntley Please close.

I just want to clarify - there is no build in way in Rx to handle async method in Subscribe?

What would be really nice to have is equivalent of this (from RxJS):
await observable.first()**.toPromise()**;

@davidnemeti thank you so much for these detailed clarifications.
When I talked to my team about these issues, one question came up: Namely, if it made any difference if we would use asyncOnNext().ToObservable() instead of 'Observable.FromAsync(() => asyncOnNext(x)'
For example
instead of

public static class MyObservableExtensions
{
    public static IDisposable SubscribeAsync<T>(this IObservable<T> source, Func<Task> onNextAsync) =>
        source
            .Select(number => Observable.FromAsync(onNextAsync))
            .Concat()
            .Subscribe();

    public static IDisposable SubscribeAsyncConcurrent<T>(this IObservable<T> source, Func<Task> onNextAsync) =>
        source
            .Select(number => Observable.FromAsync(onNextAsync))
            .Merge()
            .Subscribe();

    public static IDisposable SubscribeAsyncConcurrent<T>(this IObservable<T> source, Func<Task> onNextAsync, int maxConcurrent) =>
        source
            .Select(number => Observable.FromAsync(onNextAsync))
            .Merge(maxConcurrent)
            .Subscribe();
}

could we write
public static class MyObservableExtensions
{
    public static IDisposable SubscribeAsync<T>(this IObservable<T> source, Func<Task> onNextAsync) =>
        source
            .Select(number => onNextAsync().ToObservable()) // note ToObservable instead of FromAsync!
            .Concat()
            .Subscribe();

    public static IDisposable SubscribeAsyncConcurrent<T>(this IObservable<T> source, Func<Task> onNextAsync) =>
        source
            .Select(number => onNextAsync().ToObservable()) // note ToObservable instead of FromAsync!
            .Merge()
            .Subscribe();

    public static IDisposable SubscribeAsyncConcurrent<T>(this IObservable<T> source, Func<Task> onNextAsync, int maxConcurrent) =>
        source
            .Select(number => onNextAsync().ToObservable()) // note ToObservable instead of FromAsync!
            .Merge(maxConcurrent)
            .Subscribe();
}

and if no, what is the difference between 'Observable.FromAsync' and '.ToObservable'?
This question is just too hard for me 😫

@gentledepp, Observable.FromAsync and .ToObservable are different.
The difference is that Observable.FromAsync is deferred, while .ToObservable is not; i.e. the latter will execute the given async function immediately once a new item is being yielded from the observable.
This difference is due to the fact that the parameter of Observable.FromAsync(...) is a Func<Task>, while the parameter of .ToObservable is a Task; i.e. the former is a not-yet-started asynchronous operation which can be started when it is desired, while the latter is an already-started asynchronous operation.
It means that when you create an observable by using Observable.FromAsync(...) the subscriber can control the starting of the asynchronous operation, while by using .ToObservable the subscriber cannot control the starting of the asynchronous operation because that operation has already started.
You can check this behavior by running the following example:
void Main()
{
    Observable.Interval(TimeSpan.FromSeconds(1))
        .SubscribeAsync(number => DoSomeWorkAsync(number));
}

async Task DoSomeWorkAsync(long number)
{
    Console.WriteLine($"DoSomeWorkAsync BEGIN '{number}'");
    await Task.Delay(TimeSpan.FromSeconds(3));
    Console.WriteLine($"DoSomeWorkAsync END '{number}'");
}

#if true
public static class MyObservableExtensions
{
    public static IDisposable SubscribeAsync<T>(this IObservable<T> source, Func<T, Task> onNextAsync) =>
        source
            .Select(number => Observable.FromAsync(() => onNextAsync(number)))
            .Concat()
            .Subscribe();

    public static IDisposable SubscribeAsyncConcurrent<T>(this IObservable<T> source, Func<T, Task> onNextAsync) =>
        source
            .Select(number => Observable.FromAsync(() => onNextAsync(number)))
            .Merge()
            .Subscribe();

    public static IDisposable SubscribeAsyncConcurrent<T>(this IObservable<T> source, Func<T, Task> onNextAsync, int maxConcurrent) =>
        source
            .Select(number => Observable.FromAsync(() => onNextAsync(number)))
            .Merge(maxConcurrent)
            .Subscribe();
}
#else
public static class MyObservableExtensions
{
    public static IDisposable SubscribeAsync<T>(this IObservable<T> source, Func<T, Task> onNextAsync) =>
        source
            .Select(number => onNextAsync(number).ToObservable()) // note ToObservable instead of FromAsync!
            .Concat()
            .Subscribe();

    public static IDisposable SubscribeAsyncConcurrent<T>(this IObservable<T> source, Func<T, Task> onNextAsync) =>
        source
            .Select(number => onNextAsync(number).ToObservable()) // note ToObservable instead of FromAsync!
            .Merge()
            .Subscribe();

    public static IDisposable SubscribeAsyncConcurrent<T>(this IObservable<T> source, Func<T, Task> onNextAsync, int maxConcurrent) =>
        source
            .Select(number => onNextAsync(number).ToObservable()) // note ToObservable instead of FromAsync!
            .Merge(maxConcurrent)
            .Subscribe();
}
#endif

If you use the Observable.FromAsync(...) version then the asynchronous operations do not overlap. However, you use the .ToObservable version then the asynchronous operations do overlap.

@davidnemeti thank you so much for this explanation! Suddenly everything makes sense.
So I would use ToObservable only if I knew, that the Task has already completed such as when using 'Task.FromResult()' for the observable.
Can I buy you a beer somehow? 🍺

@gentledepp, you are welcome. Unfortunately, I don't like beer, so a simple "like" would be enough. :-)
Regarding your question about .ToObservable: of course, you could use it when you know that the Task has already completed, but I think that's not the point.
I think the point is that .ToObservable works on a Task, while Observable.FromAsync works on a Func<Task>.
 But in the overwhelming majority of cases you have more than this: you have Func<Task>, because you have async methods which represent asynchronous operations.
Thus, since you can work with Func<Task>, you can (should) use Observable.FromAsync.
Personally, I do not use .ToObservable, because it prevents me from controlling the starting of the asynchronous operations and thus controlling concurrency. Of course, you could add .Concat and .Merge to the observable (believing that you will control concurrency this way), but it does not really matter which of them you use, because by that time you already lost the possibility to control concurrency.
On the other hand, Observable.FromAsync allows me to control concurrency by adding .Concat and .Merge to the observable, and everything behaves exactly as it should.

@davidnemeti sorry for necroposting, but I am wondering if it is possible, in your first example, to asynchronously handle not every element of the first sequence, but start the task as soon as the element arrives and drop arriving elements while the handler executes?
As soon as it finishes, resume "listening" for the next element to process.

@wh1t3cAt1k sure you can achieve that.
You could use the .Latest() function for that, however it is a blocking function:
Observable.Interval(TimeSpan.FromSeconds(1))
    .Latest()
    .Select(number => Observable.FromAsync(async () => await DoSomeWorkAsync(number)))
    .Concat();

E.g. it blocks until the first element arrives.
A more sophisticated solution could be to introduce our own function which is non-blocking and you can even introduce concurrency if needed:
public static class MyObservableExtensions
{
    public static IObservable<TResult> SelectAndOmit<T, TResult>(this IObservable<T> source, Func<T, IObservable<TResult>> process, Action<T> noProcess, int maximumConcurrencyCount = 1)
    {
        var semaphore = new SemaphoreSlim(initialCount: maximumConcurrencyCount, maxCount: maximumConcurrencyCount);

        return source
            .SelectMany(item =>
            {
                if (semaphore.Wait(millisecondsTimeout: 0))
                {
                    return Observable.Return(process(item).Finally(() => { semaphore.Release(); }));
                }
                else
                {
                    noProcess(item);
                    return Observable.Empty<IObservable<TResult>>();
                }
            })
            .Merge(maximumConcurrencyCount);
    }
}

You can use like this:
Observable.Interval(TimeSpan.FromSeconds(1))
    .SelectAndOmit(
        number => Observable.FromAsync(async () => await DoSomeWorkAsync(number)),
        number => Console.WriteLine($"no work {number}")
    );

With a maximum number of 2 concurrent processes:
Observable.Interval(TimeSpan.FromSeconds(1))
    .SelectAndOmit(
        number => Observable.FromAsync(async () => await DoSomeWorkAsync(number)),
        number => Console.WriteLine($"no work {number}"),
        maximumConcurrencyCount: 2
    );

@davidnemeti thanks a lot!
It looks like there are multiple ways to achieve this, as usual with Rx.
Someone helped me out on StackOverflow, too: see "ExhaustMap" implementation: https://stackoverflow.com/questions/64353907/how-can-i-implement-an-exhaustmap-handler-in-rx-net

I am wondering if the approach suggested by @davidnemeti in this thread is still the best approach you could use when you need to run an async method in a sync context, now that AsyncRX.NET NuGet is released, for .NET Standard 2.0 even (cc @idg10, I am the dude from Reddit comments). I was refactoring my code (currently using ReactiveUI with WinForms) and seeing as ReactiveUI and ReactiveMarbles.ObservableEvents are yet to catch up with AsyncRX.NET it seems that I either

need to use the approach outlined here so that I won't need to deal with IAsyncDisposable and other stuff, or
convert the source-generated events ToAsyncObservable, subscribe asynchronously, and to dispose make a blocking wait for the disposable (I guess I need some IAsyncDisposable -> IDisposable).

From the sound of it, the second option sounds a bit icky to do. Maybe there are other approaches to solving the problem that I am not seeing after a look on the source code?

AsyncRX.NET is currently still experimental, and that's partly because it's not yet clear what the full implications of using it in will be. For example, in the rxnet slack channel, @anaisbetts made this point:

I don't particularly like async Observables because they seem to break IScheduler

which gets to the heart of something that will need to be resolved if AsyncRX.NET is to be used successfully in scenarios such as this.
I haven't fully understood exactly what it is you're doing that has let to you to:

need to run an async method in a sync context

Did Ani already unblock you on the rxnet channel? If not, could you explain what you're doing that needs this?

How to turn Subscribe(onNext, onError, onComplete) into async/await version?

How to turn Subscribe(onNext, onError, onComplete) into async/await version, with 3 different callback?

I would say that, create extension methods.
Something like:
public static IDisposable SubscribeAsync<T> (this IObservable<T> source, Func<T, Task> onNextAsync)
{
    return source.SubscribeAsync(onNextAsync, ex => throw ex, () => { });
}

public static IDisposable SubscribeAsync<T> (this IObservable<T> source, Func<T, Task> onNextAsync, Action onCompleted)
{
    return source.SubscribeAsync(onNextAsync, ex => throw ex, onCompleted);
}

public static IDisposable SubscribeAsync<T> (this IObservable<T> source, Func<T, Task> onNextAsync, Action<Exception> onError)
{
    return source.SubscribeAsync(onNextAsync, onError, () => { });
}

public static IDisposable SubscribeAsync<T> (this IObservable<T> source,
    Func<T, Task> onNextAsync, Action<Exception> onError, Action onCompleted)
{
    return source
        .Select(param => Observable.FromAsync(() => onNextAsync(param)))
        .Merge()
        .Subscribe(
            unit => { },
            ex => onError(ex),
            onCompleted);
}

Correct me if I am wrong.


Seven Ten Views

This should be set up properly, but I am going to double check we are submitting seven-ten classifications correctly. I have confirmed (using two test users) that previous annotations will only appear if a user is in the collaborative variant.
Edit: Switching this to WIP while I confirm seven-ten is queried correctly.
Closes #70

This should be all set now. A couple changes needed to made to seven-ten, particularly PR https://github.com/zooniverse/Seven-Ten/pull/20#pullrequestreview-69351399

Yes, that does make sense about testing, since the frontend will now be restricted to whatever group you're in. I'm thinking to add a toggle button that's available if the user is an admin? I'd probably make it small and inconspicuous, off to the side somewhere.

So, an admin toggle has been added, but I'd like some feedback on it. Perhaps it should only show up in development? It's pretty basic, and I'm open to suggestions.

I like what you did there - putting the the toggle as an admin-only viewable option is good with me. I'm also keen on seeing this on both Development and Production, because we might need that option to debug in the future.
Only two caveats at the moment:

this only allows admins to who have been assigned B (solo) to view see the A (collab) view, but not vice versa.

While visually and functionally the same at the user level, (i.e. the only practical difference is whether you see prev annotations or not) the override should affect the code at the logical ("selecting A or B switch") level.

there's still no indicator if we're on the A or B split.

That said, I'm OK to 👍  this PR and merge it, but if I have time in the future (hahaha) I'd like to revisit the A/B split logic into something like this...
function showAorB() {
  if (adminOverride === "on") {
    if (adminOverrideChoice === "A") { return "A"; }
    else if (adminOverrideChoice === "B") { return "B"; }
  }

  if (userIsLoggedIn) return sevenTen_getAorBSplit();

  return "B";  //Default
}

/*  A tiny, discreet icon on the screen would help us debug a user's situation - when they report an issue, we can ask them to submit a screenshot.
 */
class TinySplitIndicatorIcon extends React.Component {
  ...
  if (showAorB === "A") return <Icon width="5px" height="5px" color="red"></Icon>
  else if (showAorB === "B") return <Icon width="5px" height="5px" color="blue"></Icon>
  ...
}

The seven-ten split is currently only active on staging, so another test would need to be set up. However, I am seeing calls to seven-ten being made in the network tab with staging.

This makes sense about switching between collaborative and non-collaborative. I'll open an issue for it.