feat: working typings for Paragon, better types for  component

Description
This represents my version of the minimal changes to Paragon to start getting (just a few!) useful type definitions shipping with it.
Before this PR, I wasn't seeing any types for Paragon imports, either in MFE code that uses it nor even in the www subproject of this repo, which nominally uses TypeScript.
After this PR, you still won't see many types, but TypeScript will at least know which component exist and which don't (it can check the imports). And a few components - <Icon>, <Chip>, <ChipCarousel>, and <Bubble> will have types because they are written in TypeScript (or in the case of Icon because there's a .d.ts file).
I deliberately didn't go any further than that because I want to keep the changeset small enough that we can confidently review and merge it. From that point on, it will be much easier to incrementally add types to individual components in separate PRs.
This PR:

Creates a src/index.d.ts file that makes all of the top-level exports visible to TypeScript. Without this fix, types were not being found in repos that import Paragon by path, such as the www subproject. It will be necessary to manually keep this file in sync with src/index.js. To make that as easy as possible, I made sure the files have a 1:1 correspondence line-by-line.
Example: www/src/components/IconsTable.tsx

Before: <Icon> has no types
After: <Icon> is type checked

Adds "types": "dist/index.d.ts", to package.json. Without this change, types were not being found in MFEs that import Paragon via NPM.

Renames /src/Icon/Icon.test.jsx to be a .tsx file so that the <Icon /> component tests will use/check TypeScript.

Fixes incorrect types in src/Icon/index.d.ts. Now that Icon.test.tsx is enabled for type checking, we see that it contained type errors, because the .d.ts file was incomplete:

Adds type information to the icons sub-package. Now you get auto-completion and compile-time validation of the icon names. The new file, icons/es5/index.ts is completely auto-generated so there's no need to maintain the types manually.

Adds some compile-time (TypeScript-only) tests to Icon.test.tsx to avoid regressions.

Renames a few .jsx files in the documentation site (www subfolder) to .tsx because otherwise the linter was complaining that ~paragon-react couldn't be found. This is because a .jsx file cannot import Paragon's src/index.ts anymore after the change in step 1. This may not be an issue anymore after I changed the approach from step 1 above, but it doesn't hurt :)

Fixed the tsconfig.json (see inline comment) and separated out the build vs. dev tsconfigs.

How to test in an MFE

Have the MFE and paragon in folders next to each other on your host. This example will assume that frontend-app-course-authoring and paragon are peer folders on your host.
Make sure the MFE's dependencies are up to date (e.g. tutor dev run course-authoring npm install to do this for course-authoring)
In Paragon, edit package.json and change the line "@types/react": "17.0.0", to "@types/react": "../frontend-app-course-authoring/node_modules/@types/react", (otherwise you'll get weird conflicts with this package). Substitute the appropriate folder name if you're using a different MFE.
Run npm install in the paragon folder.
Mount paragon into the MFE's container: tutor mounts add course-authoring:/Users/yourname/path/to/paragon:/openedx/paragon (adjust this command as needed)
In the MFE's package.json change "@openedx/paragon": "^21.5.7", to "@openedx/paragon": "file:../paragon",
Link in the local paragon copy using tutor dev run course-authoring npm install --force (warnings about "Conflicting peer dependency" are expected; ignore them)
In VS Code, open any .jsx file in the MFE that has // @ts-check on its first line (or add that line in!), and you'll see type checking. Note that <Icon> should have correct typing, and other Paragon components like <Stack> should not show any type errors.

Example, showing the types appearing in VS Code for Course Authoring MFE's ImportTagsWizard.jsx:

Next Steps
Once this PR is merged, I think we should follow up with additional PRs to add typing to individual components (e.g. one component per PR, or similar). That can be done either by adding a .d.ts file for the component, or by converting the component implementation to TypeScript. In either case, I would encourage people to follow the example in this PR and make sure that the tests for the component are also converted to TypeScript (or some other mechanism is used to check that the types are working).
Once everything has a type (or close to it), the index.js file can be renamed to index.ts and the index.d.ts file can be deleted :)
Deploy Preview
The code for this page was changed, but you should not be able to see any changes:
https://deploy-preview-3016--paragon-openedx.netlify.app/foundations/icons/
Merge Checklist

[x] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[x] Does your change adhere to the documented style conventions?
[x] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[x] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[x] Were your changes tested in the example app?
[x] Is there adequate test coverage for your changes?
[x] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

I tested this out in frontend-app-authn by replacing "@openedx/paragon": "^22.0.0", in package.json with "@openedx/paragon": "file:../paragon",, and adding // @ts-check to the top of EnterpriseSSO.jsx. I then looked at this line https://github.com/brian-smith-tcril/frontend-app-authn/blob/2d37b8b0bfd8d289dc4da409d5d9c749b6c01f2b/src/common-components/EnterpriseSSO.jsx#L62 and saw

I'm not sure if this is a "task failed successfully" moment or if this means there's more that needs to be explored here.

@brian-smith-tcril That happens when you have two different versions of @types/react installed - Paragon has its own version that's different from what's in your MFE. It's a side effect of the way you added paragon as a local folder dependency. I ran into the same problem, and found a workaround, which is in the "How to test in an MFE" in the PR description.
Also, once this is merged and installed via NPM, its deduplication will make sure that doesn't happen. But here you're joining together two separate sets of npm installed packages and two different node_modules folders, rather than one big one.

@bradenmacdonald thanks for clarifying! I wasn't sure what you meant by "weird conflicts with this package" in that section. After following that step everything seems to work as expected!

@bradenmacdonald 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


Did some refactoring

Hi., Thanks for creating this. It was exactly what I needed as my developers are using Nest.js for some projects and that thing has too many decorators.
Here are the changes as the code flows:
New tsconfig loader
There wasn't support for tsconfig file that had extends set in there.
The answer was to use Typescript's own parser to get the current config data.
There is now a function parseTsConfig that is based on [rollup-plugin-typescript2][https://github.com/ezolenko/rollup-plugin-typescript2/blob/42173460541b0c444326bf14f2c8c27269c4cb11/src/parse-tsconfig.ts#L48]
Bail out early
I changed the logic a little so that if the options are not there, we don't load the typescript file. That was a file read that would happen every time, even when we didn't need it.
Regex
Switched to using a regex expression to look for a decorator hit.
Test coverage
I threw together some tests to cover various cases. I did them very fast, but I think I covered all the cases that I could see.

Oh, isn't it a bit ironic that this is a plugin for typescript, not written in typescript :)

@thomaschaaf please merge this gem. Thanks!

@andrestone  I couldn't wait for a merge myself, and I have a few teams needing this.
So I went ahead and refactored my own and published it.
I'll be keeping it updated as it will be in active use across a few fortune 500 companies.
https://www.npmjs.com/package/@anatine/esbuild-decorators

@andrestone  I couldn't wait for a merge myself, and I have a few teams needing this.
So I went ahead and refactored my own and published it.
I'll be keeping it updated as it will be in active use across a few fortune 500 companies.
https://www.npmjs.com/package/@anatine/esbuild-decorators

Thanks for sharing!

@Brian-McBride the regex is a lot slower than the current implementation. And will fire if an email address or an @ like with require('@org/repo') is included. If you'd revert that part I could go ahead and merge this.

@Brian-McBride I'd gone ahead and provided you with write access to the repo so that the project is maintained in the future :) we could also improve the module further by using the tsconfig from https://esbuild.github.io/plugins/#build-options

Hrm. Good catch on the email. Things to add to test cases.
That is true about pulling in the same tsconfig reference. I wish that the options exposed the traversed Tsconfig options.
 I mean, the regex is only looking for the first positive match too.

const shouldNotCatchEmail =<EMAIL_ADDRESS>That passed the unit tests. The decorator detection requires a space before the @
However, this will catch incorrectly
const thisIsATrickyCase = 'A string where we talk about @decorator value'
That is tricky. And, honestly, this decorator metadata is such a pain, I'm not sure it would make a huge difference if a file with this string was parsed by TSC. That said, maybe there is a good way to remove all strings from the file before detecting like the removed comments.

I am not sure if the regex is correct. Why does there need to be a space before the @?
@testDecorator()
class example {}

I looked at this https://stackoverflow.com/a/54538979/19929 when writing the code but the conclusion is actually that it doesn't matter for longer strings, which source code usually is.

It is actually zero or more white spaces before the @
Meaning, this something@else is not a decorator.
However, the string thing has got me.
Looking at performance. (which I agree is an issue)

Loading each typescript file is the biggest hit. I suppose it would be possible to do some detections on the stream itself. Unfortunately, files that don't have decorators would still stream completely through.
The comment remover code runs a lot of detectoins that are not needed. I'm tempted to just copy the core code and have it only run the javascript and HTML patterns. Maybe it could be modified to also remove any strings from the file text too.
And I looked around too. It does seem that Regex becomes a better option the longer the string you are searching is.

released as v0.2.0 :) would love to get you on board with write rights so that the package is maintained by two people.


Tormenta20

Hi! The module is very cool. Could you make it compatible with Tormenta20 system?

hi dude, ty for work, but is possible add suport to PM (Mana Points) too?


Bluetooth switch position isn't saved

It's always on, resulting in a permanent "Bluetooth disabled" banner above the input fields. v1.7.0

Hi, thanks for the report. Unfortunately I cannot reproduce this.
To clarify:

you disabled this switch:

Bluetooth input is shown regardless:

Could you export the settings that cause this?

It remembers the switch position during the same session. But as soon as I eg. swipe Bpm out of the app switcher and the look at the Bluetooth setting the switch is always on. (Apologies for not answering your questions exactly and without screenshots, I'm in a hurry)

Thank you for taking the time to respond, you answered exactly what I needed to know.
I still can't reproduce your issue, although it helped catch another problem with resetting this setting when deleting settings (bcde09dd04de8a1942e61bea2106f3c28455cd85). I will investigate your issue further on other devices/installation sources. Do you still have the Fairphone 3 with the app from F-Droid?

It should be fine without :)

Not much effort with the DevStat app:
Screen Metrics
Width: 1080 px
Height: 2022 px
Display density: 460 dpi
Drawable density: Unknown DPI: 460
Screen size: Normal

Software
Android version: 13
SDK Int: 33
OpenGL ES version: 3.2
Google Play Services version: 24.23.34<PHONE_NUMBER>40581) [242334038]
Build Number: 6.A.029.0

Hardware
Manufacturer: Fairphone
Device model: FP3
Device: FP3
Brand: Fairphone
Board: FP3
Host: jenkins-agent-01
Product: FP3
Memory class: 192 MB
Large memory class: 512 MB
Max memory: 192 MB
Free space: 35,3 GB
Telephony: Enabled
SD Card: Present: true, Emulated: true
ABIs: arm64-v8a, armeabi-v7a, armeabi
Online Processors: 8

Could you export the settings that cause this?

Would it help to send you my settings db?

That's what I meant, yes.

For whatever reason I've had to zip it in order to upload it. Yet it seems not to be attached to this post. :/

You've got mail

Curiously your settings are still in v1.6.4 format. even the app version number update isn't updated (this is one of the first things that happen on app launch).
I'm also getting very interesting results freshly installing the f-droid build on a real device: The add measurement button doesn't work (although for me the settings saved).
#344 also showed some results that shouldn't have occurred from the code alone and are not reproducible on an emulator. To me this feels like either an bug in the f-droid build script or in flutter.

I think I found and fixed the issue. Since there has been quite a few issues with this release I will publish a new release.

I still couldn't reproduce the issue myself, so please give me feedback once f-droid builds.

please give me feedback once f-droid builds.

Will do. Thanks!

Out if curiosity: is your fix aimed at switch positions in general? The reason why I'm asking is, my "Compact measurement list" switch position isn't saved across app restarts as well.

@uli-on the fix is aimed at all settings in general. It appears like your settings file doesn't write changes at all (I hardened the auto-save setup logic).
Thinking about this again: Do measurements save between restarts? If not this might be some problem on your end (SD-card issues, others).

Do measurements save between restarts?

Thankfulky they do. Phew! :)

Thanks for the link, I click/read through several pages from there and found that I currently do have the most current build for my device.

The F-Droid build was delivered today. The compact view issue (#341) is solved, thanks! The forgetful switches issue is not, sorry. But the "sticky" position for compact view is my favourite (compact on) and I can live with the Bluetooth banner on top of the measurement form. Yet I (have to ) hope there won't be more switches added to the prefs 😄)

I don't like jumping into this closed issue but I don't really wanna open a new one for now, as you are already aware:
Every restart of the app, it resets to the modern/ new layout here as well on two devices, one storing on sd and f-droid build, one storing on internal and Google play, both running android 10.

Don't worry about the issue being closed. Is this only true for switches or do other settings(time format, color) also fail to save?

Tested with time format (first item in settings) and yes, it also fails to save
Because you mentioned it above: new entries are being saved / stored reliably.

I'm facing the same issue. Using Android 11 and v1.7.2 of the application. The switches reset to the previous value after closing the app. Measurement info saves normally.
Please let me know if there's any other information I can provide. Thank you.

@uli-on @Der-Flip @kp8888 Can someone who experiences this issue install the app from the relaible-settings branch and check whether the issue is fixed?
Please note that this WILL DELETE ALL EXISTING APP DATA. So please make BACKUPS!
After reading the disclaimer: cd app && flutter run
I can provide apk builds, please email me about them, since I don't want them to be mistaken for real releases in the wild, as they are also going to delete your data.

Hi, sorry I didn't get back to you. I wanted to test it but I couldn't find the time. First vacation, I returned from last week and now I have some family health stuff to deal with. Is the testing still necessary or has it been shipped already? If it still needs testing I will try to find a quiet minute this week, though.


Remove question marks, equal signs and hashes

Those were not removed if multibyte was allowed.

Could you please include a test in URLSegmentFilterTest with setAllowMultibyte(true) and some of those characters?

Ok, I did that, but apparently it came up as a new pull request #4963. I thought that GitHub would combine the pull requests when creating a new one while having another still open. Anyway, is this good now?

Are you able to please, maybe, try putting them into the same PR? Otherwise the tests won't pass with the fix. You'll need to switch to your patch-1 branch in the github editor and put the changes there.

see #4973


release artifacts for release v1.2.9

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/retest

/retest

/retest

/retest

/retest

/lgtm cancel

/lgtm


Could not connect to streams! PLease Help!

Whenever i try to connect to any twitch streams it says connection timed out! I am a noob please help!

I can't help you without any useful informations. Even if you consider yourself a noob, you should provide at least some kind of info.
What kind of timeout are you talking about? Do you get a timeout error during the livestreamer validation when trying to launch a stream or are you talking about a timeout while viewing the stream in your video player?
Does it work when using the Livestreamer CLI? Anything "Livestreamer Twitch GUI" unrelated doesn't belong here. Also, this is an issue tracker, personal issues don't belong here either.

I am having the same problem. Funny thing is that it was working well a couple of hours ago so i dont know what happened. As soon as i click on a stream, the window pops up but after a couple of seconds i get the timed out message. I can provide a screenshot if thats gonna be any help.

Closing this. See #301


fix path handling

This should fix an issue with deep linking introduced by the changes to support branches. There may still be issues with branch handling, but this should at least restore previous functionality.

:+1:


Unsupported major.minor version 52.0

Hi, I love this plugin, but I noticed that the version published (2.10) is compiled with java8 raising an "Unsupported major.minor version 52.0" exception when running the application.
Can you please publish the plugin with java 7 compatibility?
Thank you,
Enrico

Thanks! Try 2.11 and let me know if that doesn't work.

Great! It works, thank you!


Region coordinates should not be floored

Commit 68442c1565d3dd4b058f5caadfd638b8d287e98e added floored vectors to regions and parsers.
Coordinates should be real numbers and rounding them breaks some mechanisms, like circles and cylinders centered on the middle of a block:

This region cannot work if its center is floored, as it will just be pushed to the lower corner of the middle block (tested and working on build 111 but not on build 112).

Thanks


Format syntax errors always with column range.

If the token mentioned in a syntax error covers more than one character,
print syntax error with the full column range.

Codecov Report

Merging #1271 (b7c38e8) into master (bda7f9a) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1271      +/-   ##
==========================================
- Coverage   92.80%   92.80%   -0.01%
==========================================
  Files         339      339
  Lines       23527    23524       -3
==========================================
- Hits        21835    21832       -3
  Misses       1692     1692

Impacted Files
Coverage Δ

common/analysis/file_analyzer.cc
91.39% <100.00%> (-0.61%)
:arrow_down:

common/analysis/lint_rule_status.cc
96.55% <100.00%> (-0.06%)
:arrow_down:

common/strings/line_column_map.cc
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


⚠ Bofrost.lu has degraded performance

In 7ea32ae, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 852 ms

Resolved: Bofrost.lu performance has improved in 3044aa2 after 9 minutes.


[FLOC-1392] Array APIs now get documented

https://clusterhq.atlassian.net/browse/FLOC-1392
To review visually, compare documentation for GET /v1/configuration/datasets and GET /v1/state/datasets in build results and in http://doc-dev.clusterhq.com/advanced/api.html

Thanks @itamarst . Please address the minor comments and merge.


vulnerable against reflection, replay, reordering, and nonce-reuse

Each packet in the stream is encrypted with its own, randomly generated nonce. This can lead to nonce re-use (when using AES GCM) if encrypting more than 2^32 packets. That would mean that keys need to be rotated that often. I doubt a user of the library would do that.
Also, because there is no sense of direction, an attacker could reflect a packet sent by a client back to the client and the client would mistake it as a response from the server.
There is also no authenticated packet counter, meaning that a man in the middle (MITM) attacker could replay and re-order packets without detection by the recipient.
Overall, I think it'd be fair to say that this library achieves confidentiality in presence of an eavesdropper, but no security in presence of a MITM.

@t0b0 These issues should be fixed by #8. In the future we can add native support for noise framework for easier usage.

Closed for inactivity. Feel free to reopen it if needed


Remove GetHash from the games

It doesn't get used at all. Probably good to remove to cut down on confusion.

Fixed in #14


C.3 Create a web service interface (Vol. 2)

Create a web service interface (design the URIs) to the annotation data. This should
implement the full set of CRUD operations for all the resources. Again, imagine the
needs of the user interface and use the use cases/user stories from A to grasp the
requirements. Document the interface.

Does this mean, that we dont have to implement the standard StackOverflow API, such as getting a post?


Implement Enumerable#filter_map

[#19]

Also, can you word your commit message in imperative from? i.e. “Implement Enumerable#filter_map”
Thanks!

Sorry to nag, but you did it again: in your commit message, "Avoiding" should be "Avoid".
Side note: Ideally this PR would just be one commit since it's one logical chunk of work, but I'd settle for imperative voice in the commit message subject. Thanks!

Thanks!


Realm: incorporate MPI-based active message implementation as option

This'll require some restructuring on my part to make the activemsg implementation somewhat pluggable, and then probably somebody else's work to dust off the code in gitlab PR 151 (https://gitlab.com/StanfordLegion/legion/merge_requests/151) to make the peg's shape fit the hole.

The first part of this (the modularization of the network backend) is there with commit d607d30.  Next up is making an MPI network module as an alternative to the current GASNet-1 module.

The MPI network layer is merged in as of commit 05a66970.  Thanks to @hzhou for doing most of the work on this.  This network layer is enabled by setting REALM_NETWORKS=mpi for the make-based build or Legion_NETWORKS=mpi for the cmake build.
The existing "gasnet1" (i.e. GASNet-1 or GASNet-EX using its GASNet-1 compatibility API) network layer is requested either by the new settings: REALM_NETWORKS=gasnet1 and Legion_NETWORKS=gasnet1 or the existing USE_GASNET=1 (for make) and Legion_USE_GASNet=ON (for cmake) for backwards-compatibility.

How does the performance compare?

How does the performance compare?

Unknown.  Please try it out on applications of interest and report back.  Let's open new issues for
any performance-related concerns though.

I'm seeing undelivered messages with this network. I'm running the mpi_with_ctrl_repl example in the control_replication branch with mpirun -n 2 ./mpi_with_ctrl_repl -ll:force_kthreads so both processes are running on the same node and intentionally leaving off the --bind-to none option to force as much randomness on the thread scheduling as I can. If you crank it up to -n 16 you'll get a hang every time at least on sapling.

Worth noting that I'm also running in debug mode. The "interesting" backtraces show no evidence of a virtual channel hang, only messages with no preconditions on unordered virtual channels that haven't been delivered.

@lightsighter I'll investigate.

@lightsighter I am trying to reproduce what you described. Is this supposed to run on a single node with the command line:
mpirun -n 16 ./mpi_with_ctrl_repl -ll:force_kthreads

and you were seeing it hangs?
I am trying it on a ubuntu work station and it seems to complete ok. Were you running it in docker? Which mpi library were you using?

Yes, two processes on a single node. I'm seeing it hang in lots of different ways. The common theme seems to be a user event that is created and waited on in one process and then sent to another process where it is triggered. I can see that the event is being triggered on the remote node, but the event trigger and corresponding wake-up of the thread that waited on the event is not happening on the first node.
I'm using OpenMPI 3.1.3. I'm not running it inside docker, just standard Ubuntu Linux (although a very old machine).

I just tried with OpenMPI 4.0 -- one just happened to be available on my machine -- and it seems to run ok. I'll try 3.1.3 later.
A difference I noticed that with OpenMPI, it shows following warnings:
[0 - 7f0bab898e80] {4}{threads}: reservation ('DMA request queue') cannot be satisfied
[1 - 7f001e1bbe80] {4}{threads}: reservation ('DMA request queue') cannot be satisfied

I am not sure whether this is relevant. I recall this might due to mpi-processes being bound-to certain cores -- probably OpenMPI's mpirun does that by default.

OpenMPI 3.1.3 is ok on my side as well. I am using a self-compiled version.

I tried to compile Legion into both debug and release mode with Open MPI 3.1.3, and I did not see any issues when running the following in single node:

mpirun -n 16 ./mpi_with_ctrl_repl -ll:force_kthreads

The default process binding of Open MPI is -bind-to core, I tried -bind-to none, and I still can not reproduce the bug. One way to check the installation of Open MPI is ompi_info, which tells configurations of Open MPI, although I do not think our MPIs will have significant differences.

I still see it 1 in 1000 runs. If I run the Legion test suite in the background it comes down to 1 in 100 runs. If you have access to sapling I can point you at a pair of frozen processes.

A difference I noticed that with OpenMPI, it shows following warnings:

You should be getting those warnings. I intentionally said when I first reported the issue that you should be using --bind-to core (the default MPI setting) in order to force as much random thread scheduling as possible.

@lightsighter I just repeated 10000 runs. I don't have access to sapling. Do you think you could reproduce it in docker and share with me the docker file (or image)?

@lightsighter can you try a few thousand runs with --mca btl ^vader on the mpirun command line?  Best I can tell, these packets are getting lost inside of the ompi implementation.

10K runs and no hangs with that flag. @streichler care to enlighten us as to why that makes the problem go away?

Closing this tracking issue.  Any new functional or performance concerns go into new issues.


Add CHANGELOG.md

Adds a changelog, fixes #121.
Contains all changes I could find in the "Releases" section, all the way back to 0.6.0.

Awesome, thanks!


pt-BR

[ ]

What's the screenshot mean?

What's the screenshot mean?

Happens this here. No category as result.


Wasm Backend for Ergo

Issue #727
Changes

New WASM backend
New execution engine
Refactor logic manager and script manager

Coverage decreased (-1.5%) to 94.069% when pulling a864a1b1960d5762a63780f4b22be0abad027f10 on jeromesimeon:wasm into 9e64c949d539aa13421f309594f1da815cc6a2d5 on accordproject:release-1.0.


Bump PHPUnit to v9

This fixes https://github.com/webonyx/graphql-php/pull/819. Sorry about the timing 🙃

I still think #819 is fine. While in a PR, declining coverage is actionable. Once merged, the failing status is only a nagging eyesore.

🚢


Cloud credentials support for phoenixNAP driver

Proposed changes

Types of changes

Linked Issues

Further comments

cancelling this pull request new one will be created


Flappy Bird game implementation using HTML, CSS, and JavaScript with basic physics (30th mini-project)

Created game structure with HTML
Styled components with CSS
Implemented game mechanics and logic in JavaScript

Added bird character with jump functionality
Included moving pipes as obstacles
Developed collision detection
Implemented scoring system

Integrated gravity and bird movement physics
Added game over and restart functionality

https://github.com/dhairyagothi/50_days_50_web_project/assets/140265777/28657e6a-9f94-43e3-98de-8a3e86d1d06f

Assigned @Yaser-123
make sure to add correct Day first Pull the changes
Level of PR is assigned after checking it
Go Ahead


Qt.py Linter

Goal
Discover binding incompatibilities in your code through linting.
$ Qt.py --lint my_module.py
#24:my_widget = QtGui.QWidget(): Should be QtWidgets.QWidget()
#103:header.setResizeMode(header.Fixed): setResizeMode has been renamed setSectionResizeMode in Qt 5.
#244: ...

Motivation
In lieu of warning messages due to technical constraints, how about an external process that can either be run as-is on your new or existing code, or integrated into a realtime linter, such as SublimeLinter?
Implementation wise it would be rather straightforward; simply look for common patterns where something is used that shouldn't. We'd have to keep in mind more esoteric use, such as import QtWidgets as MyCustomStuff which would complicate things, but at least we can help a majority. (And eventually do handle such cases gracefully).

This would be a killer feature.

I looked into doing something like this a few different ways.  One is as a pylint-plugin of sorts.  pylint uses ast, which breaks up the code into nodes using Python's builtin grammar engine, then walks the grammar tree.  That's probably the 'right' way to do something like this.  But for the use case, it might be overkill, and a series of simple regular expressions that look for common issues might be a better solution for this feature.
At least in my mind, this linter is mainly going to be used for converting Qt4 code to a Qt5 compliant API.  With that in mind, I see this as powerful, but seldom used conversion tool.  But, a good way to catch obvious problems, and to flag less obvious problems, like using QString and QVariant objects, deprecated methods, etc. eq. value = self.model().data(0, Qt.UserRole).toPyObject()  The type of problems you will encounter at run time and can only be discovered by exercising the code, or by carefully examining the code.

Your insights are greatly appreciated @melevine as at least myself has never tried to write a linter. In case you feel like taking a stab at it, feel free to do so :)

One way to achieve this is to provide PEP484-compatible pyi stub files with the project or within the official python typeshed.  Then you run a program like mypy to do static type checking.  It checks for missing objects/attributes and also incorrect types, where possible.
PyCharm also supports pyi files as of version 2017.  Support in other IDEs is still lagging, but now that type hints are an official part of python -- including syntax support added in python versions 3.5 and 3.6 -- I think you'll see the community converging on this format.   The mypy project is headed up by Guido Van Rossum, btw.  And yes, pyi stubs work in python 2.
PyQt provides them as a build option for python3 but it turns out that these are not working quite right.  The author of QtPy is working on pyi stubs for their project.   You may be able to "borrow" them.
I should point out that, for better or for worse, mypy will check for far more than just incorrect usage of Qt.py, so when a user runs it against their code it will likely generate many additional errors, unrelated to Qt.py.  The plus side is that those might be real errors, and the typing info is super useful for realtime feedback within IDEs.
Btw, while I have you, where can I find a description of the differences between Qt.py and QtPy?

Thanks @chadrik.

Btw, while I have you, where can I find a description of the differences between Qt.py and QtPy?

There's a summary here, but nothing more in-depth exists at the moment.
https://github.com/mottosso/Qt.py#projects-similar-to-qtpy


Error in getting storage account

Hello
When I run $storageAccount = Get-AzStorageAccount -ResourceGroupName $resourceGroupName -Name $storageAccountName, I get an error that Get-AzStorageAccount : Resource group 'training' could not be found.
The problem could be that I have access to multiple subscriptions and when I run Login-AzAccount, it outputs a subscription name that I am not using. I have also set my default subscription but it does not login with default subscription.

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

ID: c12f1ea9-6446-be01-22fe-885fb945d22c
Version Independent ID: 634f8ace-1a8c-6811-627b-fdfbdd1942a3
Content: Use an Azure file share with Windows
Content Source: articles/storage/files/storage-how-to-use-files-windows.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

@thecpshah Thanks for your question. We are checking on this and will respond to you soon.

@thecpshah I'd recommend posting your question in https://social.msdn.microsoft.com/Forums/en-US/home?forum=windowsazuredata  since it's not directly related to the documentation. We'll follow up there. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Fix update targets on windows

This pr fixes #10

oops accidentally deleted the branch xD

Thanks @Tyderion


New version: StratiGraphics v0.1.1

Registering package: StratiGraphics
Repository: https://github.com/juliohm/StratiGraphics.jl
Created by: @juliohm
Version: v0.1.1
Commit: 95cac89c1ab61849094995d853e238fdbeff5e6f
Reviewed by: @juliohm
Reference: https://github.com/juliohm/StratiGraphics.jl/commit/95cac89c1ab61849094995d853e238fdbeff5e6f#commitcomment-33987289

Depends on #1423

Updated.


Socket Hang Up and Keep alive

Hi there,
Thanks for this great project !
We have done some stress tests and sometimes during load (randomly) we get those kind of errors :
Error: socket hang up\n    at createHangUpError (_http_client.js:331:15)\n    at Socket.socketCloseListener (_http_client.js:363:23)\n    at emitOne (events.js:121:20)\n    at Socket.emit (events.js:211:7)\n    at Socket.wrapped [as emit] at TCP._handle.close [as _onclose] (net.js:567:12) code: 'ECONNRESET'
This error means that the connexion (tailor server to fragment endpoint) has been closed while the response isn't complete. One workaround was to increase the fragment timeout but it's not our wish. I precise that the remote server is AWS edge with a lot of cache so the response time is very fast everytime (below 20ms).
We tried to use the keep alive parameter to avoid recreating http connexions, but it doesn't seems to work, node is not able to reuse the connexions, we observe a saturation of server sockets.
Did you experience those kind of issues ? Did you find workarounds ?
Regards
Fred.

Hey, internally we use a extended version of tailor by implementing our own requestFragment function
We use https://github.com/node-modules/agentkeepalive which helps with keep-alive agents and you can configure both TCP and HTTP keep alive timeouts there.
The other solution we did before was using a custom Agentpool logic

Use a queue to maintain the list of Agents (Maximum 3 agents in our case).
Create a new HTTP or HTTPS Agent and mark this new agent as the current Agent.
Serve all the traffic through the current Agent for specified intervals.
Follow step 2 and 3 and mark the old Agent as inactive.
Do this till we hit our maximum 3 agents and recycle the old agents.


Fixed issue and created new style.css file

I've improved the negative values issue
improved some hovering effect on logo
added default values to input fields
and some little bit improvements here and there
Happy hactoberfest-2022 😊✌

@SukarayamJanjua A CSS file already exists in the project. Update your code in that and avoid creation of multiple files.

Well...did that ✌..updated my css code in existing app.css and deleted the new style.css

@Alok5102R Well ...how come ? I didn't interfere with home page animation.Indeed its working just perfect on my pc !!

@Alok5102R Have a look now

@SukarayamJanjua
Sorry wrong issue was written previously.
Issue with your code is:
When you set default values in choose.html it's creating issues and not showing results.
To fix this you need to update script file also.
Also, please try to resolve conflicts by making changes in latest commits.

Thanks for your time @SukarayamJanjua
Since, you didn't responded I need to close this.
But, you can contribute in other issues.


LepIndex - Can the TaxonNo field be migrated as a data attribute field to the taxon name record

It would be a good idea to have the LepIndex TaxonNo field migrated as a data attribute field to make it easier for us to tie up legacy records once the migration is complete.

Done, previewable on sandbox now.


Remote Event Receiver which are registered with the option 'LegacyAuthProtocolsEnabled' set to false are not triggered/fired

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
When we attach one or more SharePoint Remote Event Receivers, RER, to a list or library we would expect that these RER`s are fired/triggered and the call can be handled by the RER service. No matter if the option 'LegacyAuthProtocolsEnabled' is set to true or false.
Observed Behavior
When the Office365 Tenant option 'LegacyAuthProtocolsEnabled' is set to false it is not possible to use any newly registered RER on a list or library. The RER is not triggered/fired at all, so we do not see any request come in within the RER service.
Already existing RER which were registered before setting the option 'LegacyAuthProtocolsEnabled' to false are still working correctly.
Also when the option 'LegacyAuthProtocolsEnabled' is set back to true and a new RER is registered it works fine, but the RER`s which are registered with the option 'LegacyAuthProtocolsEnabled' set to false are still NOT working.
All the above observations are done with multiple RER`s to make sure it is not due to a particullar RER.
We also have seen this behaviour at 3 different tenants, so it is reproducable at our side. The only downside is that when setting the option 'LegacyAuthProtocolsEnabled' it takes about 2 to 3 hours before it is pushed to the entire tenant! So it is not possible to switch this option and do a quick test after that, first wait 3 hours before doing that test.
Steps to Reproduce
Correct Situation
First check if the RER is working in the correct situation in which the option 'LegacyAuthProtocolsEnabled' is set to true.
This can be done by registering a new RER by using the beneath PowerShell:
#Region #### Variables ####
#The RER url can be used for testing. This is a free website within Azure, so could be that is sleeping, so please call this in the browser before testing it in SP
$RERUrl = "http://ctbrerhelloworld.azurewebsites.net/service.svc"
#The url of the site in which the list or library exists at which the RER must be attached
$siteUrl = "https://{TenantName}.sharepoint.com/sites/MySite"
#The name of the list or library at which the RER will be attached
$listName = "Shared Documents"

#endregion

#Connect to SP (in this case using username/pwd, can also be using Appid/secret/certificate but this will make it a bit more difficult to explain)
$credentials = Get-Credential
Connect-PnPOnline -Url $siteUrl -Credentials $credentials
$connection = Get-PnPConnection

#Add event receivers
Add-PnPEventReceiver -List $listName -Name "RER-HelloWorld-ItemAdding" -Url $RERUrl -Connection $connection -SequenceNumber 10000 -EventReceiverType ItemAdding -Synchronization Synchronous
Add-PnPEventReceiver -List $listName -Name "RER-HelloWorld-ItemUpdating" -Url $RERUrl -Connection $connection -SequenceNumber 10000 -EventReceiverType ItemUpdating -Synchronization Synchronous

#Verify if the events are added (which is the case)
Get-PnPEventReceiver -List $listName -Connection $connection

When these RER`s are registered please add an item within the list or library, which should result in an error message with the text 'Hello World!!!'
This means the RER is registered and working correctly.
Fail Situation
Set the 'LegacyAuthProtocolsEnabled' tenant property to false by using the following PowerShell:
#Connect to Office365
Connect-SPOService -Url https://{TenantName}-admin.sharepoint.com
#Check the property LegacyAuthProtocolsEnabled which is currently set to true
Get-SPOTenant
#Set the property LegacyAuthProtocolsEnabled to false
Set-SPOTenant -LegacyAuthProtocolsEnabled $false

#Wait for at least 2 to 3 hours before continuing, while the setting of this property can take up-to 3 hours!

After a couple of hours the property LegacyAuthProtocolsEnabled is set correctly to false in the entire environment.
At this time we can register a new RER by executing the same PowerShell as specified in the Correct Situation. Please make sure either or both the site and or list name differs from the first test, so this RER is attached to another site and/or list.
When now adding an item to the list or library it succeeds which means the RER is not fired/triggered.
When adding an item to the list or library of the previous correct siuation check it still gives an error which indicates that these RER`s are still working correctly, which also eliminates the question whether the RER is still running or is correct, yes it is ;-).
As specified above, we can reproduce this (with a lot of patience) at 3 different Office365 tenants.
Hopefully someone can fix this problem so we can continue to use the RER`s while this is our main trigger for our solutions at our environments and our customer environments.
In the mean time, does anyone have idea's how to work around this issue? We are thinking in using WebHooks or Flow, but any other idea's are more than welcome!
Beneath the code of our Hello World RER which is very complicated ;-)
public class Service : IRemoteEventService
{
    SPRemoteEventResult IRemoteEventService.ProcessEvent(SPRemoteEventProperties properties)
    {
        SPRemoteEventResult result = new SPRemoteEventResult();

        result.Status = SPRemoteEventServiceStatus.CancelWithError;
        result.ErrorMessage = "Hello World!!!";

        return result;
    }

    void IRemoteEventService.ProcessOneWayEvent(SPRemoteEventProperties properties)
    {
        throw new NotImplementedException();
    }
}

I hope someone can help us while this is blocking the main of our solutions at all of our customers which have disabled the LegacyAuthProtocolsEnabled Office365 option...
Thanks in advance for looking into this!

Hello you all,
I have an update regarding this issue.
We have started a Microsoft Premier support call and found out some interesting stuff.
It turns out that the tenant option 'LegacyAuthProtocolsEnabled' is not the real problem but the actual problem is the usage of App-Only when registering the RER`s on a list.
Let me explain the reason behind this which Microsoft told us.
When registering a RER it is not only saving the name, url, order, etc but also the context of the user or in this case the app which added the RER is saved as owner. So registering the RER works fine, expect in our case the user context which is normally saved is now using the app only context as owner of the RER.
When adding or updating a list item the first thing which is done is to check which RER`s should be executed. In our case also our RER.
The second thing which is done is to check if the owner of the RER has the correct permissions to execute the RER. In normal situations a user has added the RER and that user is available in SharePoint and therefor can execute the RER. But in our case the owner is the App, which only exists within Azure AD as an App and not within SP, so in this case the permission check fails while SharePoint is only checking the internal permissions and not the Azure AD App permissions....
The workaround is to also give the App permissions within SharePoint by using the appinv.aspx page within the tenant admin. When doing so (after some delay >30min) it started to work and the added RER`s are now working correctly.
This is offcourse not the perfect solution while in this case you have to specify permissions within the Azure AD App (which is OK) and you have to specify also permissions within SP (which is not OK). This SP permissions are not hard to monitor which app has which permissions and you have to give full control to the whole SP tenant.
So in the end it would be very nice if Microsoft can fix this issue and the RER is also working with App Only in which the app is not even known in SP itself!
I guess at least the documentation should be updated so everyone knows that if using App-Only the App should have enough permissions within SP to let the RER work.

We face the same problem, as we switch from SharePoint App Only to AzureAd App Only authentication. So we would like to avoid appinv.aspx as well.
Any progress in using RERs created from AzureAd App Only authenticated code?

Hi @EinmalIM. No change unfortunately. As Microsoft stated they will not fix this issue so I guess we are stuck with this behaviour

Too bad. Do you have a link where Microsoft stated that they won't fix it?

Hi @EinmalIM, I do not have a link for you. Microsoft has stated this within the Premium Support ticket we had opened. After long testing and jumping from one team to another they came to the conclusion that the 'old school' Remote Event Receiver is not working when used in this scenario and they will not update this while this would have too much impact :-(


[FLINK-7190] Activate checkstyle flink-java/*

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[ ] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[ ] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[ ] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

This is the last commit of series. It enables the stricter checkstyle. So should be merged last.

What if we first split the suppressions files as with suppressions-runtime.xml? I think we could do this in one ticket for all of core, java, and optimizer.

@greghogan I agree, expanding the suppression files would be nice.

The switch statements can be modified to be checkstyle compliant with few changes.
NestedMethodAnalyzer#naryOperation:
...
case INVOKESTATIC:
case INVOKESPECIAL:
case INVOKEVIRTUAL:
case INVOKEINTERFACE:
	isStatic = insn.getOpCoed() == INVOKESTATIC;
...

NestedMethodAnalyzer#copyOperation:
...
case DUP2_X2:
	if (isTagged(value) && tagged(value).isInput() && tagged(value).isCallByValue()) {
		return tagged(value).copy();
	} else {
		return super.copyOperation(insn, value);
	}
...

merging.


Fix for #94 - Sticky header jump

Now when the stuck class is added to header#banner, the top property is animated via css.

Alright, this looks good to me! Let's ship it.


fix: reloading breaks interface refreshing

Anything holding this up?

Order is going to matter more as the source grows, but this is a good start.  Another of my branches doesn't work without more explicit ordered because when you start to use the objects more freely you run into issues about the order in which the objects are imported into different files.

Lost track of this one.  Merging.

Thanks!


[Impeller] Implement path based text rendering.

@chinmaygarde can you describe the scope of this goal?
I've been working with @slightfoot for some time on custom glyph rendering in Flutter because Paragraphs were too heavy for us.
We've explored using standard curves to paint individual glyphs. We've explored pre-painting fonts to textures and then drawing glyphs with drawAtlas(). And now we're exploring doing our own tessellation and drawing with drawVertices().
If Flutter is going to bring more granular control to text painting, I'd really like for our use-case to make it on the roadmap, and ensure that we can use any such new Flutter APIs.

This issue doesn't track modifying the existing Flutter APIs in any way. It just that Impeller's text rendering today is extremely rudimentary in that it only uses glyph atlases with glyphs in them rendered at-scale. This works in most cases but is slow or takes up more memory than necessary (especially relative to the benchmark) for extremely large glyphs (when they are zoomed in). This issue along with a related on for SDF based rendering rounds out text rendering strategies necessary for ideal text rending support with Impeller.

This might also lead to the fix of the infamous emojis rendering issue #28894

@jonahwilliams implemented this in https://github.com/flutter/engine/pull/41402 to get numbers and also did a comparative study of Skias text rendering strategies. The finding was that that this isn't something that is used today for any point size.
While it's good to know that we could use this approach if we needed to, it doesn't have a current use-case. Closing this as wont-fix and putting the patch in the icebox.

Yes, two important points:

Tessellation of the geometry across various point sizes was often times more expensive than software rendering of the same text. I don't expect compute shader based tessellation approaches to change this as text paths are often concave or self intersecting in ways that defeat common optimizations.

generally 4x MSAA is not sufficient anti aliasing for text at any point size, though it was harder to tell if the text was particularly heavy.


March 28th, 2020 Retro

We need a summary of all that we have accomplished collectively, what is working for us and what still needs more work.

Here is a summary of what we have accomplished, what is working, and what we need to work on as of today.
COVID19_Helpline Retrospec.docx

Taking care of automating the share of this document weekly.


Please, provide a changelog

Hi.
Wearing my downstream hat, it would be nice to have a changelog that I can ship to users.
I have one that asked me how I provide the Debian changelogs on the Debian packaging and I replied "by reading the source code", which after thinking a bit, is some duplicated effort for other distributions that don't benefit from my work.
Can you provide something as a changelog? Even something like a git shortlog --no-merges would be helpful and (hopefully, not too noisy), which is, BTW, what Linus Torvalds himself uses to announce newer kernel releases.
Thanks in advance,
Rogério.

oh goody, this is/would be good, exactly what I was gonna ask for, thanx rbrito :)

please create it or assist me creating it

Added in 35aa6c538f593eb47169e6b06ef383ad12b45930.


multiple og:type attributes

Add ability to pull multiple og:type in open-graphs.
Add sub-types such as image:width into the open-graph dictionary

@dlrobertson this helps get closer to #84

Please note that there are still some changes that should likely be made:

We need to handle cases where the same tag is found multiple times. The following from this page has multiple article:tag elements. Currently, we only keep the final one.

<meta property="article:tag" content="Python">
<meta property="article:tag" content="strings">
<meta property="article:tag" content="tuples">

Yeah we don't currently support OGP arrays.

OK, then I think this resolves #84; we can add the OGP arrays in another issue.


Prediction interval for ES-RNN

Thank you for making this implementation available. Sorry if I missed It; Is there an easy way to obtain prediction intervals with your ES-RNN implementation?

Hi @marijnhazelbag,
For the moment we are only estimating quantiles using the quantile regression like in the paper and main M4 competition.
We have implemented a multi-quantile loss function here:
https://github.com/Nixtla/nixtlats/blob/master/nixtlats/losses/pytorch.py#L317
I imagine, it could be easy to adapt the final layer of the ES-RNN to output multi-quantiles.
https://github.com/kdgutier/esrnn_torch/blob/master/ESRNN/utils/ESRNN.py#L285

Thank you for the swift response! I will try and let you know if I run into problems :)

I am reaching a friend who might have already done it to point you to his work.

Hi, @marijnhazelbag,
Thanks for your interest. We have written the multi-quantile version of the ESRNN model: MQESRNN; you can find it in our new library for time series forecasting using Deep Learning named nixtlats, in particular here. You can import it using:
from nixtlats.models.esrnn.mqesrnn import MQESRNN

You pass a list of percentiles to obtain prediction intervals; for example, if you want a 90% prediction interval, you could use training_percentiles=[5, 50, 95]. Under the hood, the model trains the ESRNN model using the multiquantile loss.

Thank you, gentlemen! I really appreciate it.


Update link to kured yaml

Kured has moved to docker hub now and updated the location of the yaml file. Changed the link so it goes to the updated version

@PixelRobots : Thanks for your contribution! The author, @iainfoulds, has been notified to review your proposed change.

Thanks for the contribution, @PixelRobots. LGTM.
#sign-off


LaTeX parser does not support the quantifiers on the MathLive keyboard

Description
If one creates an expression using MathLive, and uses either the universal or existential quantifiers (∀ as \forall, ∃ as \exists), the resulting LaTeX cannot be parsed into MathJSON.  These quantifiers appear on the MathLive keyboard, and thus appear as if they are supported.
Steps to Reproduce
Here I show how to reproduce the behavior from the browser console, since that does not require clicking, etc., but the LaTeX used here could be created using the live widget if one wanted. to.
> MathfieldElement.computeEngine.parse('\\forall x, P').json
// => ["Error",["ErrorCode","'unexpected-command'","'\\forall'"],["LatexString","'\\forall'"]]
> MathfieldElement.computeEngine.parse('\\exists x, P').json
// => ["Error",["ErrorCode","'unexpected-command'","'\\exists'"],["LatexString","'\\exists'"]]

Actual Behavior
See above.
Expected Behavior
Well, I'm not sure exactly what the MathJSON convention for bound variables is.  By parsing an integral, I find that it does not seem there is any special indicator that a variable is bound, other than the fact that the operator in question is one that binds variables, like so:
> MathfieldElement.computeEngine.parse('\\int x dx').json
// => ["Integrate","x","x"]
// (as opposed to something like ["Bound","x",["Integrate","x"]])

That's not a criticism, just an observation about the format.  So I suppose what I'd expect is an analogous thing for the predicate logic quantifiers:
> MathfieldElement.computeEngine.parse('\\forall x, P').json
// => ["forall","x","P"]
> MathfieldElement.computeEngine.parse('\\exists x, P').json
// => ["exists","x","P"]

Although some folks use a nonstandard separator between the bound variable and the body (as in \forall x.P) or some other notation (as in (\forall x)P) those are atypical, and the most common thing I would expect to see in a math textbook is the syntax shown above.
Environment
I do not expect that this is a regression; it has probably always been this way.
MathLive version 0.98.5
Operating System macOS 12.4
Browser
Brave v1.61.109, but I don't expect this is browser-dependent.

Awesome, thanks!
One small suggestion:  Your diff shows that you also support exists-unique and forall-unique, but I do not think that forall-unique is a real thing.  https://math.stackexchange.com/questions/502101/for-all-unique-notation

Yes, good point. I've removed it.

Regarding "bound" variables... You can indicate that a variable should not be evaluated by wrapping it in a Hold expression, i.e. ["Hold", "x"]. This can be used to "declare" a variable, as in the first argument of a universal quantifier or integral, or to indicate that an expression should not be evaluated, as in the body of a universal quantifier or integral.
However, functions also have "attributes" that can indicate if the arguments of a function (or some arguments of a function) should be evaluated or not.
The universal quantifiers (and integral function) have their arguments automatically "held" (i.e. not evaluated), so they don't need to be wrapped with a ["Hold"] function, although that won't cause any problem if they are.
If you encounter additional syntactic variants for universal quantifiers that you feel it would be useful to support, let me know.


Update to abigen v1.10.15

This PR updates the generate.sh script to use the latest abigen version (v1.10.15) and includes the changes regarding the different bindings creations.

Closed because I accidentially chose the wrong branch.


include image-pull-policy=Never for kubectl run

default functionality for kubectl run sets imagePullPolicy=Always. This causes an error because the image is local to the machine, rather than in a docker registry.
This PR fixes the example docs by configuring the imagePullPolicy=Never and as such, allowing image to be loaded from local filesystem.

/assign @tengqm

/lgtm

/lgtm
/approve


Q4 Project Comms Plan

Coming out of our Q4 OKRs presentations, a key take away is we have a ton of launches coming up this quarter! To help keep track of the associated project communications, I created this Q4 Comms Plan which tracks all the announcements involved in landing our features and improvements with the community. Next steps are to fill in estimate dates and clarify owners for all of these - and then check in on this regularly in cross-team meetings to ensure we're tracking our progress against our goals.
@daviddias @olizilla @autonome @parkan @Stebalien @terichadbourne and @jessicaschilling - would love your review in particular!

For IPFS docs beta site launch, please note we're tracking this here: https://github.com/ipfs/docs/issues/351


Incorporate new API throttling

Add a plugin to handle the API rate limiting system used by Meetup.com

If anyone's interested, I finally implemented an albeit basic solution, by simply wrapping the client class and interogating all responses and sleeping then nearing the limit: https://github.com/phpsw/phpsw/blob/develop/src/Meetup/API/Client.php

This is also solved by the patch in PR #19

Implemented in #19 and accepted for v2.


Fix for #175.

Please follow this template, if applicable.
Description
Widgets on non existing monitors do not spawn.
Additional Notes
Currently this is only effective for Wayland.

@manderio testing, testing, 1, 2, 3

it works fine for me too, also really like the improved error messages, they look really neat!
Thanks for being epic and fixing this!

^ this - having a way to state "on current screen" would be nice, both on X and on wayland.
Also build fails, check out the error and fix 👀

The default behavior when no screen is defined is current screen with the Wayland backend.

Thanks!


Add flyout image button and flyout status button

Proposed Changes
Further work on the status button for extensions to display in the flyout.

Add a FlyoutImageButton class that inherits from FlyoutButton, but displays an image.
Add a FlyoutStatusButton class that inherits from FlyoutImageButton, but has a setStatus function to be used by the extension to update the button's image.
Add status enums for 'ready' and 'not ready' states.
Add a top-level function to update a status button with a status enum and an id (we'll use the scratch extension id).

One remaining chunk of work for a future PR is to position the status button to spec.

Replacing this with https://github.com/LLK/scratch-blocks/pull/1610


Feature/save raw to comet

Training script now logs raw images as Comet assets. The images are stored as float32 RGBDD (RGB + Depth Prediction + Depth GT).
The images can be loaded using:
np.fromfile(file_name, np.dtype(('f4,4))).reshape(224, 224, 5)

Made this PR by accident, sorry about that.

Made this PR by accident, sorry about that.


update pdf.js to 2.12.313

We want to figure out how to automate updating pdf.js instead.

Implemented in https://github.com/GrapheneOS/PdfViewer/pull/138


Add sharp detection for Next.js

sharp is required to be installed when using next/image in the project to run next start.
next start is used to run the production code in nodejs server, which in turn requires sharp

I guess sharp is not an (optional) peer dependency of next, but it should be?
So your idea is to see whether next/image is referenced in code somewhere, and then report sharp as unlisted if necessary? Sounds pretty custom 😃
Perhaps a task for the Next.js ESLint plugin, although not sure whether that would be possible. Not sure what that's capable of.

Started a discussion in next.js repo

From https://github.com/vercel/next.js/pull/19216#issuecomment-729832697:

We're looking to eliminate Sharp ASAP though, so this shouldn't be a problem for long.

Either way, for such exceptions you can use ignoreDependencies and e.g. sharp will not be reported as unused. Open to better ideas here :)

Alright then.


accept multi line input by shift+enter

Type: Feature Request

we need to input multi line at the same time.

suggestion shift+enter to input new line.
an icon button in tool bar to switch the character of new line,

\r
\n
\r\n

allow to send multi-line text file https://github.com/microsoft/vscode-serial-monitor/issues/7

Those that are interested in this feature should upvote the issue to let us know of your interest, thanks!

Just  do  it √


@ProtoSchema and @AutoProtoSchemaBuilder should always set "service …

Closes #13233

One last test was failing. Should be good now, just need the squash on merge

Thanks @ryanemerson for this contribution


Code Bin Sort algorithm :infinity: in any programming language you like :computer:

Code Bin Sort algorithm :infinity: in the relevant language folder :file_folder: in the main branch.
if language folder doesn't exist you can make folder :file_folder: and add your program in it.
You can find new languages in this Programming Language List.
Give star :star: to this repository.

hi @iParadocx , I think this is still open, would you like to assign it to me?
I'd like to complete this for javascript, c++ and java.

Hello There @iParadocx , I would like to contribute to this Using C++, Could you please assign me?
Thank you

Hi @iParadocx , created PRs for the binsort algorithm in Java and Javascript. I was also going to do for C++ as I asked for that too, but I see that @rishi2802 has done it already. Please review PRs #166 and #167

I would like to contribute to this issue in C. Please assign this task to me.


Edit Forms - disappering values when special characters are included

Current: When a value (actor, property) suggested in the auto-complete includes a slash, after being selected, if the user clicks somewhere else on the page, the value disappears. It doesn’t prevent submitting or saving items, but leads users to think that they have not selected anything.

Expected: When selecting a value that includes a slash, the value should be displayed.
@notify @mkrzmr @KlausIllmayer @vronk

@tparkola do i need to escape the "/" slash character for /api/actor-search?q=?

I would suggest to encode, but as far as I see it is not necessary in this case (parameter).

@tparkola i'm asking because when i search for "DARIAH-GR", i get a match:
curl "http://localhost:8080/api/actor-search?q=DARIAH-GR" | jq

returns:
{
  "actors": [
    {
      "affiliations": [],
      "externalIds": [],
      "id": 52,
      "name": "DARIAH-GR / DYAS"
    }
  ],
  "count": 1,
  "hits": 1,
  "page": 1,
  "pages": 1,
  "perpage": 20,
  "q": "DARIAH-GR"
}

but when searching for "DARIAH-GR / DYAS" i get none:
curl "http://localhost:8080/api/actor-search?q=DARIAH-GR+%2F+DYAS" | jq

returns:
{
  "actors": [],
  "count": 0,
  "hits": 0,
  "page": 1,
  "pages": 0,
  "perpage": 20,
  "q": "DARIAH-GR / DYAS"
}

@tparkola ping :point_up:

Ok, then I think it is related to the backend, I did a branch with some fixes, could you please check it (if it helps you)? see https://github.com/SSHOC/sshoc-marketplace-backend/tree/bugfix/SSHOC-171_actor_search_related_improvements

@tparkola LGTM :+1: - tested the branch locally, and search for "DARIAH-GR / DYAS" works!


KAFKA-15161: Fix InvalidReplicationFactorException at connect startup

In certain very specific cases a constellation can occur when Connect isn't able to get topic metadata when the brokers are being in a rolling restart.
To make both the broker and the java clients more resilient, this change aims to modify the ZkAdminManager to return UNKNOWN_TOPIC_OR_PARTITION error if the metadata cache is not yet initialized when topic metadata is queried from it.
It also adds canControllerHandle() method to MetadataCache if Kafka runs in Zookeeper mode then KafkaApis can return a NOT_CONTROLLER error if the controller is not active (as it did before) and metadata cache isn't initialized (new behavior).
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@urbandan would you please review it as well?

@cmccabe @mumrah would you please review it too?

@mumrah correct, the problem happens because the broker doesn't have a populated metadata cache and it tries to auto-create a topic during a topic metadata request (and it might be that this topic otherwise already exists). The PR is trying to fix this problem too. I thought I'd put the solution to ZkAdminManager so that it's not too different from the KRaft solution as there the broker would forward a topic creation request to the controller and return UNKNOWN_TOPIC_OR_PARTITION to the client.
It's important to note that this isn't the first metadata request that the consumer in the herder sends out but a subsequent one, so it had some connection to a valid broker but lost it and reconnected to another one which was just starting up and didn't have metadata.
In the Jira description I used a single broker because it was easier to reproduce it that way but it can be reproduced in a multi-broker environment too of course. I didn't really consider how would the controller affect things here because as you say they start accepting requests before having metadata.

Captured two screen recordings to demonstrate the reproduction of the problem.
This is the bug: https://drive.google.com/file/d/1Itq_Fv9bwNtRtPsD9KLUnD9I-6BX6Tki/view?usp=sharing
This is how the fix would help: https://drive.google.com/file/d/10Otvj880EF--BOARmhULhvixwWsAhIt9/view?usp=sharing

@cmccabe would you please take a look when you have some time or could you please suggest a reviewer?

@mimaison do you think you can review this?


fix(imap-types): Fix SequenceSetIterNaive

See #411

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
8961

Relevant Lines:
9706

💛  - Coveralls


Update mixin names

There are some mixins that have been renamed to conform with SCSS linting rules that have not been updated in src/components. This fixes errors until the refactor of src/components is completed.

Merge that junks.


EXSWHTEC-296 - Implement negative tests for builtin atomic operations

Depends-on: #292

@mirza-halilcevic : Could you please look into splitting this PR into smaller ones so that it can be reviewed easily?

PR has been merged into develop branch via commit 8f112a639a8163b074ba265ec81bcd4244f944b4.


import redux error

|-src
   |-app.jsx
   |-pages
         |-Home
               |-actions.js
               |-reducers.js
               |-index.jsx
               |-constants.js
   |-store
       |-index.js
|-index.html

index.html
<script type="text/jsx">
import React from 'react';
import ReactDOM from 'react-dom';
import { Provider } from 'react-redux'
import {AppContainer} from 'react-hot-loader';
import configureStore from './store'
const store = configureStore()

const render = () => {
  const App = require('./app').default;
  ReactDOM.render(
    <Provider store={store}>
      <AppContainer><App /></AppContainer>
    </Provider>, document.getElementById('App'));
}

render();
if (module.hot) {
  module.hot.accept(render);
}
</script>

Home/reducers.js
import * as CONST from './constants'

export const usersReducer = (state={
  loading: false,
  total: 0,
  data: [],
  error: ''
}, action) => {
  switch(action.type) {
    case CONST.FETCH_GITHUB_SEARCH_USER_LOADING:
      return {
        ...state,
        loading: true
      }
    case CONST.FETCH_GITHUB_SEARCH_USER_FAILURE:
      return {
        loading: false,
        error: action.error
      }
    case CONST.FETCH_GITHUB_SEARCH_USER_SUCCESS:
      return {
        loading: false,
        total: action.payload.total_count,
        data: action.payload.items
      }
    default:
      return state
  }
}

export const followersReducer = (state={
  loading: false,
  data: []
}, action) => {
  switch(action.type) {
    case CONST.FETCH_GITHUB_USER_FOLLOWERS_LOADING:
      return {
        ...state,
        loading: true
      }
    case CONST.FETCH_GITHUB_USER_FOLLOWERS_SUCCESS:
      return {
        loading: false,
        data: action.payload
      }
    case CONST.FETCH_GITHUB_USER_FOLLOWERS_FAILURE:
      return {
        loading: false,
        error: action.error
      }
    default:
      return state
  }
}

export const followingsReducer = (state={
  loading: false,
  data: []
}, action) => {
  switch(action.type) {
    case CONST.FETCH_GITHUB_USER_FOLLOWING_LOADING:
      return {
        ...state,
        loading: true
      }
    case CONST.FETCH_GITHUB_USER_FOLLOWING_SUCCESS:
      return {
        loading: false,
        data: action.payload
      }
    case CONST.FETCH_GITHUB_USER_FOLLOWING_FAILURE:
      return {
        loading: false,
        error: action.error
      }
    default:
      return state
  }
}

Home/constants.js:
export const FETCH_GITHUB_SEARCH_USER_LOADING = 'FETCH_GITHUB_SEARCH_USER_LOADING'
export const FETCH_GITHUB_SEARCH_USER_FAILURE = 'FETCH_GITHUB_SEARCH_USER_FAILURE'
export const FETCH_GITHUB_SEARCH_USER_SUCCESS = 'FETCH_GITHUB_SEARCH_USER_SUCCESS'

export const FETCH_GITHUB_USER_FOLLOWERS_LOADING = 'FETCH_GITHUB_USER_FOLLOWERS_LOADING'
export const FETCH_GITHUB_USER_FOLLOWERS_FAILURE = 'FETCH_GITHUB_USER_FOLLOWERS_FAILURE'
export const FETCH_GITHUB_USER_FOLLOWERS_SUCCESS = 'FETCH_GITHUB_USER_FOLLOWERS_SUCCESS'

export const FETCH_GITHUB_USER_FOLLOWING_LOADING = 'FETCH_GITHUB_USER_FOLLOWING_LOADING'
export const FETCH_GITHUB_USER_FOLLOWING_FAILURE = 'FETCH_GITHUB_USER_FOLLOWING_FAILURE'
export const FETCH_GITHUB_USER_FOLLOWING_SUCCESS = 'FETCH_GITHUB_USER_FOLLOWING_SUCCESS'

Home/actions.js
import * as CONST from './constants'

// simulate newwork deley
const delay = ms => new Promise(resolve => setTimeout(resolve, ms))

export const searchUsers = (username, page) => {
  return async dispatch => {
    dispatch({
      type: CONST.FETCH_GITHUB_SEARCH_USER_LOADING
    })

    await delay(2000)

    try {
      const response = await fetch(`https://api.github.com/search/users?q=${username}&page=${page}`)
      let data = await response.json()

      dispatch({
        type: CONST.FETCH_GITHUB_SEARCH_USER_SUCCESS,
        payload: data
      })
    }catch(e) {
      dispatch({
        type: CONST.FETCH_GITHUB_SEARCH_USER_FAILURE,
        error: "No This User"
      })
    }
  }
}

export const getFollowers = (username, page) => {
  return async dispatch => {
    dispatch({
      type: CONST.FETCH_GITHUB_USER_FOLLOWERS_LOADING
    })

    await delay(2000)

    try{
      const response = await fetch(`https://api.github.com/users/${username}/followers?page=${page}`)
      let data = await response.json()

      dispatch({
        type: CONST.FETCH_GITHUB_USER_FOLLOWERS_SUCCESS,
        payload: data
      })
    }catch(error) {
      dispatch({
        type: CONST.FETCH_GITHUB_USER_FOLLOWERS_FAILURE,
        error: "No This User"
      })
    }
  }
}

export const getFollowings = (username, page) => {
  return async dispatch => {
    dispatch({
      type: CONST.FETCH_GITHUB_USER_FOLLOWING_LOADING
    })

    await delay(2000)

    try{
      const response = await fetch(`https://api.github.com/users/${username}/following?page=${page}`)
      let data = await response.json()

      dispatch({
        type: CONST.FETCH_GITHUB_USER_FOLLOWING_SUCCESS,
        payload: data
      })
    }catch(error) {
      dispatch({
        type: CONST.FETCH_GITHUB_USER_FOLLOWING_FAILURE,
        error
      })
    }
  }
}

app.jsx
import React from 'react';
import Home from './pages/Home';

export default
class App extends React.Component {
  render() {
    return (
      <div>
        <Home />
      </div>
    );
  }
}

You will need to add the babel plugin that supports the spread operator (...state) to your .compilerc for this to work 👍 It's a non-default language feature

@MarshallOfSound how to wirte ./compilerc , I don't find methods in doc.

.compilerc documentation lives in the electron-compile README.

@malept can I want to rewrite webpack.config.js?

Could you rephrase the question?

@malept  sorry, I want to:

set alias in webpack
open css module
...

so, can I modify or rewrite webpack.config.js?

I'll let someone who knows more about webpack answer the question.

@damaoxianjia electron-compile does not use webpack at all, it uses babel in it's own custom way.  There is no webpack.config.js to customize

@MarshallOfSound ok.
so I want to modify webpack.config.js in electron-forge is impossible? where can I modify webpack.config.js?

Electron Forge does not currently integrate with webpack. However, there's work going on to add webpack support in the next major version.

@malept cool ! look forward to the new verison

@malept
cool ! look forward to the new verison


unresolved dependency: com.iheart#play-swagger_2.13;0.9.1-PLAY2.8

Hi,
added in plugins.sbt
resolvers += Resolver.bintrayRepo("iheartradio", "maven")
also this one
addSbtPlugin("com.iheart" % "play-swagger_2.13" % "0.9.1-PLAY2.8")
All the time
(*:update) sbt.ResolveException: unresolved dependency: com.iheart#play-swagger_2.13;0.9.1-PLAY2.8: not found
Where is the problem?
I only want to get the files via sbt without any locale storage or create a locale sbt.
Thank you

My fault!!!! Needing sbt 1.X !!!


File.file_exists() doesn't take remappings after resource conversion into account

Godot version:
Current master branch, earlier versions too.
OS/device including version:
All OS AFAIK.
Issue description:
This is an issue that was mentioned as the reason for "bugs" people were encountering, but since all those previous issues only came to this conclusion after a lot of discussion, they're not so convenient to keep track of a potential resolution (see #1562 and #8773 for example).
Some resources are being converted automatically or optionally when exporting a project, and either get a new suffix (e.g. scene.tscn becomes scene.tscn.converted.scn, or they even go to the .import folder where they are appended a MD5 sum.
Methods like load() or preload() know how to convert original paths to their new location and name, but File.file_exists() doesn't, and it generates a lot of confusion.
File.file_exists() could maybe be given an optional remap parameter that would involve the resource filenames remappings in its check (defaulting to true).
Steps to reproduce:

Make a project with a res://scene.tscn file. Use a script checking File.new().file_exists("res://scene.tscn"), it should be true when running from the non-exported project.
Export the project, converting .tscn files to .scn resources.
The check should now return false. Same story with assets.

I needed this to get my exported game to work correctly, so I went ahead and implemented it.

Fixed by 8523781.

Ah, I finally edited the commit message. Sorry for the junk messages, there will be no more of them.

Now that Godot 3.1 has been released, we only plan to cherry-pick critical bug fixes to the 3.0 branch, unless there is very strong support in favor of a specific non-critical issue. Removing cherrypick label for 3.0.


next greater element

Implementation of next greater element in c language

🛠 Issue (Number)

Issue no #376
👨💻 Changes proposed
Next Greater Element in C language

✔ Check List (Check all the applicable boxes)

[ ✔] My code follows the code style of this project.
[ ✔] This PR does not contain plagiarized content.

📄 Note to reviewers

📷 Screenshots

Please don't raise spam PRs for single issue


Icon request: NPM

https://www.npmjs.com/

Already mentioned at issue #5

Done
https://github.com/krisu5/aegis-icons/blob/master/PNG/NPM.png


Terraform AKSv2 - Nodepool Error when using the default documented example

Rancher Server Setup

Rancher version: v2.6-head (09/15/2021) e03b203
Installation option: Helm Chart HA

If Helm Chart, Kubernetes Cluster and version: RKE1

Proxy/Cert Details: byo-valid

Information about the Cluster

Kubernetes version: 1.21.4
Cluster Type: local

If downstream, what type of cluster? : N/A - Trying to provision an AKSv2 with Terraform

Describe the bug
While using the AKSv2 example from the rancher2 provider doc I configure the provider with those values but using the kubenet network_plugin
To Reproduce
resource "rancher2_cluster" "iztf1" {
  name = "iztf1"
  description = "Terraform AKS cluster"
  aks_config_v2 {
    cloud_credential_id = rancher2_cloud_credential.foo-aks.id
    resource_group = "iztf1"
    resource_location = "eastus2"
    dns_prefix = "iztf1"
    kubernetes_version = "1.21.2"
    network_plugin = "kubenet"
    node_pools {
      availability_zones = ["1", "2", "3"]
      name = "iztf1pool"
      count = 1
      orchestrator_version = "1.21.2"
      os_disk_size_gb = 64
      vm_size = "Standard_DS2_v2"
    }
  }
}

Result
Waiting for API to be available:min and max node count must be nil for node pool [iztf1pool] for cluster [iztf1], because autoscaling is disabled
Expected Result
AKSv2 cluster provisioned properly and in Active state in Rancher
Screenshots

Additional context
https://github.com/rancher/rancher/issues/31296

Issue reproduced. Submitted PR https://github.com/rancher/terraform-provider-rancher2/pull/744 to fix it.
@izaac TFP built binaries from the PR are available here to facilitate tests

Rancher version: v2.6-head (09/15/2021) e03b203
I checked both fixed node count and auto scaling with the new binaries and it seems to be working as expected now.

This was validated as part of the Provisioning test scope of this issue: https://github.com/rancher/rancher/issues/31296#issuecomment-923169182
Provider version: v1.20.0


meta: move tests from dvc

https://github.com/conda-forge/dvc-feedstock/pull/274

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Trouble to build libxdo - encodings ?

Hello
I Have try to install libxdo in a chroot from a Debian 7.
I use this way :
(debian)edumas@xxx:xxx/libxdo$ tar zxf python-libxdo-0.1.2a1.tar.gz
(debian)edumas@xxx:xxx/libxdo$ cd python-libxdo-0.1.2a1
(debian)edumas@xxx:xxx/libxdo/python-libxdo-0.1.2a1$ python3 ./setup.py build
Traceback (most recent call last):
  File "setup.py", line 11, in <module>
    longdesc = fp.read()
  File "/xxx/python/lib/python3.6/encodings/ascii.py", line 26, in decode
    return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 1718: ordinal not in range(128)
(debian)edumas@xxx:xxx/libxdo/python-libxdo-0.1.2a1$

I also install other python module by this way (untar, setup build, setup install). By example module six is install like this without trouble.
I edit by hand file README.rst and MANIFEST.in by remoev any non ascii character, and now I can install libxdo.
Best regard
Emmanuel

I am experiencing this issue as well, when attempting to install with pip install python-libxdo. I'm on Ubuntu 18.04 using python3.6.


Skip firewall checks if there is no VPC

fixes https://github.com/GoogleCloudPlatform/inspec-gcp-pci-profile/issues/25

/gcbrun

Are we sure these shouldn't be set to non applicable aka impact zero when they skip or is it truly an absent subcomponent

/gcbrun

Are we sure these shouldn't be set to non applicable aka impact zero when they skip or is it truly an absent subcomponent

@aaronlippold would you suggest to rather skip each check with zero impact than skip all at once at the top?

/gcbrun


Remove extra double quote from line 326

Hi @oscerd
Should I close this PR and open another?
Thanks!

Do we have the same problem in the main branch? If so you are welcome to fix it there too and send a new PR


⚠ ClamAV has degraded performance

In 75452ce, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 346 ms

Resolved: ClamAV performance has improved in b003824.


Phone authentication fails when deep link is enabled in ios.

Phone authentication fails when deep link is enabled in ios. Please give solution for this issue.
FlutterDeepLinkingEnabled

Today Ced answered this query.
https://stackoverflow.com/questions/77426466/error-with-go-router-while-doing-a-phone-auth-with-firebase-on-ios


AArch64 idle loop corrupts IRQ state with CONFIG_TRACING

Describe the bug
arch_cpu_atomic_idle on AArch64 with CONFIG_TRACING clobbers X0, resulting in potentially corrupted IRQ state.
To Reproduce
Examine arch_cpu_atomic_idle
Expected behavior
arch_cpu_atomic_idle restores the correct IRQ state on AArch64.
Impact
arch_cpu_atomic_idle on AArch64 potentially corrupts IRQ state / restores the wrong IRQ state, leading to missing interrupts and hangs. This only affects AArch64.
Logs and console output
n/a
Environment (please complete the following information):

OS: Ubuntu 18.04.5 LTS
Toolchain (e.g Zephyr SDK, ...)
Commit SHA or Version used: based off of /u/MrVan's AArch64 SMP branch (#30676) with some additional fixes that I'm in the process of upstreaming. That being said, the references below are from trunk.

Additional context
The current source of arch_cpu_atomic_idle  is as follows:
GTEXT(arch_cpu_atomic_idle)
SECTION_FUNC(TEXT, arch_cpu_atomic_idle)
#ifdef CONFIG_TRACING
	stp	xzr, x30, [sp, #-16]!
	bl	sys_trace_idle
	ldp	xzr, x30, [sp], #16
#endif
	msr	daifset, #(DAIFSET_IRQ)
	isb
	wfe
	tst	x0, #(DAIF_IRQ)
	beq	_irq_disabled
	msr	daifclr, #(DAIFSET_IRQ)
_irq_disabled:
	ret

If CONFIG_TRACING is enabled, the call to sys_trace_idle is allowed to clobber X0. This then results in the check after wfe being unknown, potentially unmasking interrupts when interrupts should not be unmasked, or potentially not unmasking interrupts when interrupts should be unmasked.
arch_cpu_idle has much the same issue, but only as a latent bug (because arch_cpu_idle doesn't use X0.)
The simplest fix here is just to change
#ifdef CONFIG_TRACING
	stp	xzr, x30, [sp, #-16]!
	bl	sys_trace_idle
	ldp	xzr, x30, [sp], #16
#endif

to
#ifdef CONFIG_TRACING
	stp	x0, x30, [sp, #-16]!
	bl	sys_trace_idle
	ldp	x0, x30, [sp], #16
#endif

(This would be unsafe if arch_cpu_atomic_idle required that other arguments were preserved after the call to sys_trace_idle - luckily arch_cpu_atomic_idle only takes one argument.)

Indeed. arch/arm/core/aarch32/cpu_idle.S store/restore r0. I think it should be ok to just store/restore x0 here and add a comment that x0-x3 maybe clobbered?


polars_cli hangs in windows 7

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of the Polars CLI.

Reproducible example
cmd
poloars

Issue description
it shows many empty lines and cannot input anything
C:\Users\LD>polars
Polars CLI version 0.5.1
Type .help for help.
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉
〉----------------blinking

when I using a sql string as parameter, it works

C:\Users\LD>polars "select count(*) from read_parquet('d:/slow3.parquet')"
┌────────┐
│ count  │
│ ---    │
│ u32    │
╞════════╡
│ 100000 │
└────────┘

Expected behavior
it accept some input and run the sql.
Installed version
0.5.1

BTW pipeline doesn't work
C:\Users\LD>echo "select count(*) from read_parquet('d:/slow3.parquet')" |polars
Error: sql parser error: Expected an SQL statement, found: "select count(*) from read_parquet('d:/slow3.parquet')" at Line: 1, Column 1

I tried download https://github.com/pola-rs/polars-cli/releases/download/0.5.1/polars-cli-0.5.1-x86_64-pc-windows-msvc.tar.gz and ungzip the file, then rename it to polars051.exe,then run it, same quesiton as above

FYI: Windows 7 was "end of life" as of 14th Jan 2020 (four years ago), and even the paid/corporate "Extended Security Updates" (ESU) users were only able to receive updates until 10th Jan 2023 (one year ago). I think it's safe to say we won't be actively supporting this version unless it turns out to be a very low effort fix 😅

FYI, r-polars also discovered a problem where it stopped working on Windows 7 at some point, but we closed the problem because it was an effect of something upstream and it was impractical to fix.
pola-rs/r-polars#383


Support resetting the colors via OSC 104, 110, 111...

Description of the new feature/enhancement
These are the counterparts of OSC 4, 10, 11 to reset the default color. (And continue the line with 117, 119 if 17, 19 get implemented, etc.)
Proposed technical implementation details (optional)
In VTE + GNOME Terminal we figured out the best is if the OSC 4, 10, 11 sequences have precedence over the UI / config file settings. That is, for each color slot, if its value is defined via OSC 4, 10, 11 then that one is used and the one in the settings is ignored. If a value hasn't been defined via OSC 4, 10, 11, or has been reset via OSC 104, 110, 111 then the value specified in the terminal emulator's settings is used. This way re-applying the same settings is an idempotent operation. (Previously the two sources were fighting with each other, both overwriting the value in the same slot. That way re-applying the user config (e.g. "altering" a color to the same value) would invalidate a previous OSC, which was bad.)

I've just bounced off this. I'm using a script with OSC 10 and 11 to switch from Solarized Dark to Solarized Light in Administrator PowerShell sessions. However, editing my config then causes all such sessions to have their colours reset, except for a bunch of stuff that seems to keep the colour scheme from the time it was written (under PowerShell, I guess this is PS-Readline at work).
I'm particularly interested in the config layering behaviour, but without OSC 104/110/111, being unable to apply config changes is probably a UX regression.
Are there any particular concerns with the proposed behaviour, e.g. standarisation/commonality? Or is it just a case of implementing the relevant code in this repo?
Or (based on the milestone being a Windows release) does it actually require MS-internal work first to support resetting the colours?

Just speaking for myself, this is quite low down my priority list, because I think it's going to be a bit of a pain. It needs to be implemented twice, because the color palette is one of those things that has to be handled separately by conhost and Windows Terminal. And on the conhost side there really isn't a concept of "default palette" to reset to, because the initial colors can come from a variety of sources (e.g. shortcut links and registry entries), and the source can change at run time.
I also have plans to refactor some of the palette management code, and I'm not keen to add any more palette functionality until that refactoring is out of the way. Not that the refactoring is necessarily a prerequisite for this feature, but I'd personally prefer to get that cleaned up first.
So basically it seems like a lot of work for a fairly obscure feature, and one which I suspect few people will care about. But that's just my point of view. It's possible someone else may consider this more of a priority and decide to take it on sooner.

Makes sense. It's not in any way a blocker for me, so I'm happy to wait for any planned refactoring to land (and perhaps #942), before I start trying to pick-off low hanging fruit.


Marker Animation - Smoothly

hi ,
how i can add Animating  on osmdroid Marker when change GPS coordinates for example animate marker movement from point A to point B smoothly ?
tnx

I believe this was asked the other day. I'm not exactly sure what your requirement what you are referring to but I think you mean when using MyNewLocationOverlay and you and the device are moving at speed and receiving GPS updates at the rate of about 1hz, the screen basically animates to the new location in 1 second chunks. The solution may be to increase the delay (duration) of the animation for animateTo.

closing due to lack of feedback/reply. feel free to reopen if this is in error


Banner Ads cover whole screen in white on Android (when using Ad Mediation)

When using Ad Mediation, setting a Banner Ad visible results in the entire screen being white, with the ad banner at the top (and sometimes top/left).
Everything works fine with regular Ad Mob ads.
The problem does not occur on iOS.
I'm using the Unity Plugin
I'm using MMedia as ad mediation
-------Possible fix --------
I sent MMedia a help request, and they informed me that this problem occurs when your ads do not have explicit size for the layout.
With that in mind, I believe I may have narrowed the problem to a place in com/google/unity/ads/Banner.java, where its calling:
FrameLayout.LayoutParams adParams = new FrameLayout.LayoutParams(
FrameLayout.LayoutParams.MATCH_PARENT, FrameLayout.LayoutParams.WRAP_CONTENT);
I believe the adParams should be explicitly setting the size of the adView, but in the code, it looks like adParams just inherit size from parent (in the unity player, i suspect this would just be full-screen)
Unless I'm mistaken, this would cause the adView to take the entire screen (I can only assume that the reason this doesn't occur with regular Ad Mob ads is because they use a transparent background, while our Mediation ad-source does not)

This issue is being closed due to its age. For bugs in the plugin open a new issue or reach out on our support group at https://groups.google.com/forum/#!forum/google-admob-ads-sdk.

Ram


istioctl -? doesn't work

Describe the feature request
istioctl -? is actively less helpful than other commands.  Please make it a synonym for istioctl -h.
$ istioctl -?
Error: unknown shorthand flag: '?' in -?

$ istioctl flobadob
Error: unknown command "flobadob" for "istioctl"
Run 'istioctl --help' for usage.

kubectl -? behaves as the requester asks.

$ kubectl '-?'
Error: unknown shorthand flag: '?' in -?
See 'kubectl --help' for usage.

Other common commands do not support this
$ ls '-?'
ls: invalid option -- '?'
Try 'ls --help' for more information.

$ helm '-?'
Error: unknown shorthand flag: '?' in -?

Personally, I don't think this is something we should do

Closing as there doesn't seem to be interest or consistency in similar tools


Allow to gracefully disable snapshot assertions

We should add a new terminal operation to the snapshot DSL that should behave like the other terminal operations in terms of advancing the Snapshot instance's internal state. However it

should not create an initial snapshot if one doesn't yet exists
it should not perform any comparison if a snapshot already exists
it should still contribute to orphan detection, so that an existing snapshot is not detected as orphan

In case there is a snapshot with multiple snapshot assertions, this would allow to disable for example only the first one, without disrupting the second one (when relying on automatic snapshot naming):
@Test
void sampleTest(Snapshot snapshot) throws Exception {
    snapshot.assertThat(actual1).asText().diabled();
    snapshot.assertThat(actual2).asText().matchesSnapshotText();
}

Will be released with 1.5.0


[Feature Request] Postfix

Just like prefix except at end. Ideal for including units in a numeric input.

Thanks for the request, unfortunately, we had problem in implementing the postfix with cleave.js.
The cursor moving issue makes it hard to use.
Hence For now, it is not in the plan, will close this.

(Duplicate #42)


[PoC] Remove BasicAer

Summary
Proof of concept to discuss about removal of BasicAer.
It is not intended to be merged unless BasicAer is removed in Terra.
Under discussion at #1003
Details and comments
It replaces BasicAer with qiskit.quantum_info.{Operator, Statevector}.
It removes reset instruction included in initializer by applying transpiler.
Some gates are converted into basis gates that are equal up to global phase and some unit tests fail due to the global phase. So I manually specify the gates not to be unrolled.
TODO

[x] initializer
[x] documentation

Hi Imamichi-san, thanks for investigating how to handle certain cases of removing BasicAer!
However, I think that this is a more involved project for which we should have a clear plan before starting to implement the changes. Some questions I think we need to address in particular are, how to replace passing BasicAer backends to algorithms, i.e. how to update the QuantumInstance and whether or not we need an additional wrapping (I was hoping to not introduce a new object).
I think we should discuss this and come up with a strategy. Maybe you could start the discussion in the original issue #1003 with the places you identified where BasicAer must be replaced?

Sure. This is not intended to merge immediately. It made this PR to share my experimental code to discuss details.

Since opflow and algorithms have been moved to Terra any such changes would need to be done there in the future. I think the thinking around BasicAer evolved too. Either way I see no real reason to maintain this open as I am trying to clean up PRs ahead of the code re-org.

Since opflow and algorithms have been moved to Terra any such changes would need to be done there in the future. I think the thinking around BasicAer evolved too. Either way I see no real reason to maintain this open as I am trying to clean up PRs ahead of the code re-org.


Added Starter App for Streamlit and SurrealDB

Hacktoberfest 2023

This is my contribution for the #35 . This is a Starter App for Streamlit + SurrealDB. It's a simple "content posting" application that contains all the CRUD implementations, thus anyone who want to get started with Streamlit + SurrealDB can find this helpful.
For convenience, here's a link to the template: https://github.com/LuciAkirami/surrealdb-streamlit-starter-kit
Do let me know if there's anything missing or to be added

Well done @LuciAkirami!
Thank you for contributing to SurrealDB during Hacktoberfest 2023 🎉
Kindly send an email to<EMAIL_ADDRESS>to coordinate the delivery of your SurrealDB swag! 😁