Pressing F1 does nothing

Describe the bug
I want to learn the keyboard shortcut to compute the square root. It's neither r nor s, so I'm out of guesses.

I press F1 to get the manual, like in the good old days when Microsoft documented their software.
I hover over the square root button, but there's no tooltip, even though I'm using a desktop computer with a mouse.
I right-click the square root button, but nothing happens.
In the hamburger menu, I scroll down to the settings, as that is the only menu item that might contain documentation or further information.

Expected behavior

It's possible to discover the keyboard shortcuts for the arithmetic operations.

Device and Application Information

OS Build: 10.0.22621.0
Architecture: X64
Application Version: 11.2210.0.0
Region: de-DE
Dev Version Installed: False

Requested Assignment
I'm just reporting this problem.  I don't want to fix it.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Hey! I think microsoft has removed the manuals feature. But please someone else correct me, if I am wrong.


Can I change the notification priority/importance?

First of all thanks for this amazing simple notification package, the local_notification looks way too overcomplicated for my simple use case. Anyways is there any support to change notification property? My notifications aren't that important I was thinking if I could give them the lowest priority so they just silently slide into the notification tray.

Thank you so much!
That's the idea. Offers a simple to learn and really flexible solution to send notifications on Flutter, which abstracts the complexity to send notification in different platforms and allows the developers  to think more on "what to do" than on "how to do".
A: Yes, you can. And the priority is converted automaticaly between the platforms and operational versions without you need to worry about whats the name of it in iOS, or if on web is different, etc.
Those configurations are applied over the notification's channel and there is two properties that i believe will be intersting to you:

Importance: is the importance to delivery and alert the user about a new notification. When the importance is higher, the OS will delivery your notification in any case, playing sounds and plays even if your device is low battery.

Default: shows everywhere, makes noise, but does not visually intrude.
Higher: shows everywhere, makes noise and peeks. May use full screen intents.
Low: Shows in the shade, and potentially in the status bar (see shouldHideSilentStatusBarIcons()), but is not audibly intrusive.
Min: only shows in the shade, below the fold. This should not be used with Service#startForeground(int, Notification) since a foreground service is supposed to be something the user cares about so it does not make semantic sense to mark its notification as minimum importance. If you do this as of Android version Build.VERSION_CODES.O, the system will show a higher-priority notification about your app running in the background.
None: A notification with no importance: does not show in the shade.

Privacy: is the privacy level of your notifications in cases when your device is on lock screen, etc:

Public: Show this notification in its entirety on all lockscreens
Secret: Do not reveal any part of this notification on a secure lockscreen
Private: Show this notification on all lockscreens, but conceal sensitive or private information on secure lockscreens

Example of how to define those two parameters in a NotificationChannel object:
      NotificationChannel(
          channelKey: 'basic_channel',
          channelName: 'Basic notifications',
          channelDescription: 'Just a basic notification channel',
          defaultColor: Color(0xFF9D50DD),
          ledColor: Colors.white,
          defaultPrivacy: NotificationPrivacy.Private,
          importance: NotificationImportance.Max
      )

@rafaelsetragni Although I was set importance: NotificationImportance.Max but alert not shows, only shows the icon on status bar.

You need to reinstall your app to test new channels configuration.


Enforce boost-python was found by CMake

Motivation
Ensure the CMake preparation fails when the correct Boost-Python library is not found. With the current behavior, the build goes through even when not found, and then the resulting _pulsar.so will not be linked with boost-python and will fail to load at runtime.

run cpp tests


display lastRunMessage and lastRunStatus in the UI

as a user
when my tasks are failing to run
then i would like an indication in the UI
then i would like to view the last error message returned.
this will be enabled after https://github.com/influxdata/influxdb/issues/15153 is complete.

Status should be straightforward, that was in the original design for the feature. I'll have to play around with it and see what the best way to display the message is

@desa this is unblocked

I believe this is done


Standalone tests

Change tests so they can run on an empty db (for example by running migrations and injecting mocked data if not present).
Maybe see if possible to mock lcd calls too (less problematic than db calls for ci runs)
I will help CI or external contributors running tests without relying on existing infrastructure.

blocks #3

Given development is shifting to mantle, it might not be necessary to invest time on this


socks.SOCKS5Error: 0x06: TTL expired

After fixing #9 this came up, seems like a PySocks bug: #todo report it.

Exception happened during processing of request from ('<IP_ADDRESS>', 48068)
Traceback (most recent call last):
  File "<path2venv>/ahmia-crawler/lib/python3.6/site-packages/socks.py", line 851, in connect
    negotiate(self, dest_addr, dest_port)
  File "<path2venv>/ahmia-crawler/lib/python3.6/site-packages/socks.py", line 497, in _negotiate_SOCKS5
    self, CONNECT, dest_addr)
  File "<path2venv>/ahmia-crawler/lib/python3.6/site-packages/socks.py", line 578, in _SOCKS5_request
    raise SOCKS5Error("{0:#04x}: {1}".format(status, error))
socks.SOCKS5Error: 0x06: TTL expired

Currently polipo remains the stable proxy solution for the crawler.

Deprecated by torfleet, new native python tor proxy


Deliver OpenAPI commandline tool for slicing of openAPI descriptions based on a set of Graph resources

When a user selects the set of Graph resources they would like to export ie could be in the form of postman collection, this can be provided as input to DevX API to generate OpenAPI descriptions.
The OpenAPI description can be input to kiota to generate SDK subset. The task details are tracked on  OpenAPI.NET repo

This resulted in Hidi commandline tool

This was completed -resulted in Hidi command-line tool


Conflict in Style Guide between app structure and demo

In the Style Guide, the Module and Structure section (https://angular.io/docs/ts/latest/guide/style-guide.html#!#app-structure-and-angular-modules ) shows the app.component.ts file in src/app
But the demo code for app.module.ts, which resides in the root of src has this line of code
import { AppComponent }   from './app.component';
Either the app structure should be changed or the line of code should be
import { AppComponent }   from './app/app.component';

Please reference the specific place.
I'm not following you. IfAppModule and AppComponent source files are siblings, then import { AppComponent } from './app.component'; is correct.

https://angular.io/docs/ts/latest/guide/style-guide.html#!#app-structure-and-angular-modules
Sorry.  I cut and pasted from an issue incorrectly posted in the main angular project.  Misses the most important part

Ah. Ugh

Closed by #2535


WIP Added synchronization for shared memory reference counter.

It will be goot to have this as alternative way to secure our ref counters. Can you add this as unused but exists alternative, so user can switch between ref counters if he needs?


Font Smoothing Property is Not Changed By Autoprefixr

Semantic-UI uses "font-smoothing: antialiased" in six different places: https://github.com/Semantic-Org/Semantic-UI/search?q=font-smoothing&type=Code&utf8=✓
There is no such CSS property, neither is such a property planned to be included in CSS in any near future.
To affect font-smoothing you need to use the vendor-specific "-webkit-font-smoothing: antialiased;" and "-moz-osx-font-smoothing: grayscale;".

We use autoprefixer which should be adding the vendor prefix for us. It looks like from perusing our dist/ folder that font-smoothing does not automatically translate to -webkit-font-smoothing

font-smooth property is not part of standard:

Though present in early (2002) drafts of CSS3 Fonts, font-smooth has been removed from this specification and is currently not on the standard track.

It is very bad thing. Autoprefixer could not add prefixes for non-standard properties, because CSS will not be useful in future.

postcss/autoprefixer#763
So it looks like there's no plans for autoprefixer to support this property, nor an assurance it will continue to exist, meaning it should probably be removed altogether.

@jlukic This seems to be a well know bug for font-smoothing: antialiased;.
How come it still not be change to -webkit-font-smoothing: antialiased;?

To clarify, this is not a bug. font-smoothing is a non-standard feature of certain browsers.
https://developer.mozilla.org/en-US/docs/Web/CSS/font-smooth

Non-standard
This feature is non-standard and is not on a standards track. Do not use it on production sites facing the Web: it will not work for every user. There may also be large incompatibilities between implementations and the behavior may change in the future.

Though present in early (2002) drafts of CSS3 Fonts, font-smooth was removed and this specification is not on the standards track.

You could probably use https://github.com/morishitter/postcss-font-smoothing as an alternative and recommend this in the README or something.

Hi, when I hover the boxes the white text is changing smoothing automatically and getting it fixed automatically as well, not sure why font smoothness is being changed. I don't know ?


Show basic error message details when boarding fails

As per feedback from @steipete, it's time-consuming, and more difficult to open up the server log every time something fails. This way we can show the error message without the stack trace here.
This is just an idea, ideally we might even want to show the stack trace somewhere hidden?
Do we want a way to disable this, as it might reveal more sensitive information?
Feel free to append commits directly to this PR 👍

Awesome - thanks! As long as the exception doesn't reveal other email addresses, this is perfect.

It shouldn't, the only thing I could think of is that

if the login fails, it might reveal your Apple ID
if it can't find the app, because iTC is not available, it might reveal the App Identifier

Both are probably not very sensitive, and maybe we don't show the Apple ID on failure

I don't see any issues with the Apple ID - but as that one's known it could probably be filtered out easily, just in case.

I don't have too much context for this, but the change seems good! If you have some sample output it would be great to see that, otherwise, merge away! 🚀

Basically the error message is shown for the user to be visible, this way when they report a problem, it includes the kind of error 👍

🚀 ❗


scheduler settings dont correspond to cron activity

after setting the snapshot time as follows:
"snapshot": {
      "scheduler-enabled": true,
      "bucket-name": "my-bucket",
      "cron-schedule": "@every 4h"
    },

The operator node appears to still run every 15 minutes.. this is from the pod log for the operator.
time="2017-05-18T13:42:34Z" level=info msg="-----> Init scheduler"
time="2017-05-18T13:42:34Z" level=info msg="Cron is set to @every 4h"
time="2017-05-18T13:45:58Z" level=info msg="Created snapshot repository!"
time="2017-05-18T13:45:58Z" level=info msg="About to create snapshot..."
time="2017-05-18T13:45:59Z" level=info msg="Created snapshot!"
time="2017-05-18T14:04:50Z" level=info msg="Created snapshot repository!"
time="2017-05-18T14:04:50Z" level=info msg="About to create snapshot..."
time="2017-05-18T14:04:51Z" level=info msg="Created snapshot!"

Cron updates are fixed since moving to k8s cronjobs.


Template engine

Implement some kind of template engine (eg. Handlebars)

see also #25 slot functionality


Should have a method for listing index models.

Currently there is a method for listing indices on a collection, which returns a Result<Cursor<bson::ordered::OrderedDocument>>. This is great, however it would be awesome to have a method which returns Result<Cursor<IndexModel>> for interoperability with the other index methods dealing with IndexModel.
I'm happy to open a PR for this as well.

Feel free to open a PR! Our cursor type isn't generic (as Iterator types in Rust use associated types rather than generics), so Cursor isn't quite the right type, so I think the right way to go might be to create an IndexModelIterator type that wraps cursor.


Group Sentry issues

by throwing the log warning at the common level of the Execute call and not at multiple places relying on it.
Related to #589

Once we merged and deployed this issue, we should probably follow-up and sort / merge some of the Sentry issues, too. Can you please do so (since you know better which ones are related)?

I just deployed the latest version of Poseidon to production and staging, allowing us to take care of the Sentry issues (merging, ...).

I just resolved POSEIDON-3W and POSEIDON-3V. This allows us to notice when these high-level issues reoccur because of a different low-level error.
I hope to see another unexpected EOF error soon creating its own Sentry issue so we can verify that the error check of this PR is working.

Thanks! As mentioned in the ticket #589, we need to make some further adjustments before resolving POSEIDON-3W again (since it was regressed).


Prototype Update - Extensions

Title of ticket: Prototype Update - Extensions
Description
Updating Prototype based off of changes to the Extension Section
Dependencies
Are there any dependencies?
DOD

[x] Discuss design updates with Technical Team - Why is this better?
[x] Update Prototype
[x] Share out Link to Dev Team for Reference

Link to updated prototype - https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/812308852


glowstone biome appearing in overworld

Describe the Bug:
Glowstone biome generating in overworld
How to Reproduce:
I had added 1 mod to the pack I am playing on
Expected Behavior:
I expected nether biomes to remain in the nether
Screenshots:
If applicable, add screenshots to help explain your problem.
Version Information:

Operating System: Windows 10
Mod Version: 2.2.0
Minecraft Version: 1.16.5
Forge Version: 36.2.2
Other mods that were running when the crash happened:

Additional Context:
Add any other context about the problem here.

pleasrle list any other mods you have installed

Sorry for the late reply. This pack is currently being run on a server, and all the mods played well together until we attempted to update and adding Refined storage, I'm unsure if there is a bug there but removing it and replacing it with AE2 seemed to work.
Full mod list:
BetterEnd
Guard Villagers
Selene
Supermentaries
Tiny Redstone
Abnormals Core
Alex's Mobs
AppleSkin
Applied Energistics 2
AstikorCarts
Biomes 0' Plenty
Carry On
Chisels & Bits
Citadel
Comfortable Nether
Create
Croptopia
Damage Tilt
Dynamic Trees & Dynamic Trees BoP
Easier Sleeping
Extra Boats
Farmers Delight
Flywheel
Immersive engineering
Infernal Expansion
Iron Chests
JourneyMap
JEI
JournyMap
Macaw's  Roofs
Minecoprocessors
Mouse Tweaks
MrCrayfish's Furniture Mod
Neat
Pams HarvestCraft 2
Paragliders
silent lib
spartan shields
spartan weaponry
swing through grass
travelers backpacks
upgrade aquatic
upgraded netherite
valhelsia core
valhelsia structures
waddles


cannot find symbol class GlobalData

when i run this application it show GlobalData class is not found.

have you solve the GlobalData issues?

no

can you?? author did't response our issue


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_0fi6jrsgqu/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_0fi6jrsgqu/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_0fi6jrsgqu/clone/recipe/meta.yaml:41: missing_description: The recipe is missing a description
Errors were found


Flow <-> EVM Interoperability

ABI encoding/decoding functionality for Cadence
Grant category
Please select one:

[x] Open source maintenance
[x] Developer tools / services
[ ] Educational material

Description
The Axelar cross-chain integration work needs the Ethereum ABI.encode() and ABI.decode() functionality made available, ideally as a native Cadence language function owing to the difficulty of implementing the same logic directly in Cadence code due to the byte level handling. This need will doubtless also arise for other aspects of EVM integration in future since ABIs are very common.
Problem statement
ABI (Application Binary Interface) is the standard for interacting with Ethereum smart contracts. Data is encoded based on its type, and because the encoded result doesn't contain type information, it is necessary to indicate their types when decoding.
Proposed solution
Introduce two natively-implemented methods on the EVM contract, for ABI encoding/decoding:

EVM.encodeABI
EVM.decodeABI

A sample usage of the two above functions can be seen below:
// Encode & decode `String`, `UInt64` and `Bool`
var data = EVM.encodeABI(["John Doe", UInt64(33), false])
let types = [Type<String>(), Type<UInt64>(), Type<Bool>()]
var values = EVM.decodeABI(types, data: data)

// Assert the decoded values
assert(values.length == 3)
assert((values[0] as! String) == "John Doe")
assert((values[1] as! UInt64) == UInt64(33))
assert((values[2] as! Bool) == false)

We make use of Cadence type-system, and map each supported type to its Solidity counterpart.
Impact
These functions would allow developers to create data packages that would be portable between Flow and EVM as well as generate EVM function calls/signatures on Flow.
Milestones and funding

Milestone
Deliverables
Timeline
Risks
USD proposal

1 - Functions Implementation
Merge https://github.com/onflow/flow-go/pull/5024
3 weeks
-
4500 $

1 - Metering
Properly meter the computation of these 2 functions, based on the encoded/decoded payload
2 weeks
-
2500 $

Total funding proposed: 7000 $
Team

Name
Role
Bio
Contact

Ardit Marku
Back-end Engineer
A seasoned back-end engineer with a focus on problem solving and designing robust software systems. Over 9 years of experience building high-traffic search engines, marketplaces, exchanges and in-house analytics solutions. Led cross-functional teams to deliver both web and mobile apps, participating in all steps of the software development lifecycle. Open source lover and advocate.
<EMAIL_ADDRESS>m_peter

@franklywatson Both milestones of this grant have been completed:

1st Milestone: Completed with the approval & merging of https://github.com/onflow/flow-go/pull/5024
2nd Milestone: Completed with the approval & merging of https://github.com/onflow/flow-go/pull/5146

Could you please review the completion? If it is accepted, we can close this :pray:


TODO: Run neo-python in ARM CPU architecture using Docker

Current behavior
Currently we only have docker containers to run in x86/x64 CPU
Expected behavior
Run a normal docker container in ARM architecture CPU like the Raspberry Pi board and others.
Objective
Build a Dockerfile that can build a container and installed seamlessly

What is the issue here in particular? You can run the current Docker image on the Raspberry Pi, no? What kind of change in the Dockerfile do you think about?

No, the current docker don't build directly on ARM/Raspbian. We need some extra dependencies.
I have done lots of tries and installed many things. I can get the smaller list of dependencies I can and post here.
Maybe those dependencies are not exclusive of ARM raspbian and we can add them to the default dockerfile(s) we currently have. That way the same Dockerfile will build on all x86/x64 and arch.
If you feel this is not important right now, just close this issue and we can move on. No problem :)

What are you trying to do -- run the neo-python Docker image on a Raspberry, or build it there? I don't see a real advantage of building the image on a Pi.

What I've done was trying to run the normal docker image on a Pi. First you need to build it there (the image itself).
So, after that, the container runs, but neo-python doesn't, by lack of dependencies.
Then, I've fiddled around and installed many things inside the running container. And now neo-python prompt is working without problem.
In resume, I will do it again and see what really helped the lack of dependencies and propose 1 or 2 scenarios:
1- The normal Dockerfile can accept those new things without interfering with x86/x64 arch and we can use the same file for all.
2- We need a separated Dockerfile for ARM/raspbian.
The goal is obvious, the first scenario.
My question to the community is: Should I bother spending time with this? Just because I don't want to spend time for a PR that is not needed.

As discussed in chat, this can be a fun hobby project, but not prioritised. Closing this issue. @nunojusto if you want to spend a few hours on this as hobby/side project, feel free to :)


Revert "添加随机整型数组的生成方法"

revert bad dep

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

utils/utils.go
100.00% <ø> (ø)

... and 6 files with indirect coverage changes


docs(changelog): Slight cleanup

What does this PR try to resolve?
For strip, I felt the nuance of where this change applies isn't as obvious and might give people the wrong impression, so I tried to make that front and center.
For the panic, the fix was backported to 1.77 (#13393) which is when it got introduced (#13172).
How should we test and review this PR?
Additional information

Nice! Thank you.
@bors r+

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

:hourglass: Testing commit 0d2ebbb5b8554f1046baaa9b1f46da1d45b3365a with merge 7bc317b45177384de4fd74be8cb559efe98bdced...

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


Repo Reorg + monolithic python package

Will also involve rewriting our Makefile based build process with a dedicated python command line utility

Addressed by #273 and #283


upgrade folder structure of grid-studies-processing

this pull request makes every grid-studies-processing file live in their own folder,
as is the standard for Processing 3, which is the same proposal of https://github.com/monome/grid-studies-processing/pull/4,
and which i also mentioned at https://github.com/monome/grid-studies-processing/pull/5

hi @tehn, i fixed the folder structure for the grid studies in processing,
and with that we can go ahead and delete or archive the repo "grid-studies-processing", since all the content is hosted on this docs/ repo now, best!


Screen Class

Object oriented implementation of screen
Packagist: https://packagist.org/packages/screen/screen
Heroku App: https://stormy-waters-5943.herokuapp.com/

Hi, very nice work
I will test on local and merge if it works.
Do you want me to add you as a project maintainer?
This project receives some attention and i don't have the time to add more features.
If someone is willing to take care of maintaining this project i would be very glad.

Hello Peter,
I would very much like so.
I'll be awaiting for the privilegie.
Em 11/01/2016 21:19, "Peter Ivanov"<EMAIL_ADDRESS>escreveu:

Hi, very nice work
I will test on local and merge if it works.
Do you want me to add you as a project maintainer?
This project receives some attention and i don't have the time to add more
features.
If someone is willing to take care of maintaining this project i would be
very glad.
—
Reply to this email directly or view it on GitHub
https://github.com/microweber/screen/pull/34#issuecomment-170692919.

hi @MASNathan i have added you as a project admin

Hello @peter-mw
Sorry for the delayed reply, I was on my honeymoon.
In the next couple of days I'll do some stuff


There are no commands defined in the "datatables" namespace

Hello i have this error when run :
There are no commands defined in the "datatables" namespace

I already publish vendor using this  command :
php artisan vendor:publish --provider="yajra\Datatables\DatatablesServiceProvider"

I'm using Laravel 5.1

@fanjavaid what version of datatables are using? Please use dev-develop branch if you want to use DataTable as Service. See this tuts for ref: http://datatables.yajrabox.com/service.
Also please note that I have changed the namespace earlier https://github.com/yajra/laravel-datatables/issues/265 from yajra\Datatables to Yajra\Datatables (capital Y). Thanks!

Ooh :D okay it works now. Thank you.


Create phar not responding!

when i try to put a .zip file from my ios device it doesn’t respond.

What do you mean by "it doesn't respond"?

by that i mean it doesn’t upload the file to the website to create the phar file when i select the zip file i want to make into a .phar file

when i upload a .zip file it doesn’t show the file is uploaded and doesn’t allow me to tap the “create phar” bottom.

You must be on desktop.

Bug fixed.


Update Set-PnPStructuralNavigationCacheWebState.md

Before creating a pull request, make sure that you have read the contribution file located at
https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md
Type

[ ] Bug Fix
[ ] New Feature
[x] Sample
[x] Parameter description

Related Issues?
N/A
What is in this Pull Request ?
Parameter and example description

Well spotted, thanks!


Implement function to resolve/normalize pipeline.Sources

The task here is to implement the following function from the PR with exhaustive unit-tests.
// expands takes a list of sources (./*, ./, ...) and current pkg path
// and returns list of package paths that this pkg depends on.
func (p *pkg) resolveSources() []string {

/cc @Shell32-Natsu @natasha41575 @frankfarzan

/cc @Shell32-Natsu @natasha41575 @frankfarzan


fix: successfully generate documentation

Issue #, if available:
Description of changes:
Tile server was failing to correctly link the generated HTML documentation.  Updated doc generation configs, comments,  docstrings, and structure to better intact with sphinx.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM. I noticed a drop in the unit test coverage %, any idea why?

Yes, I added an init.py to the lambda submodule so now the test saw cleanup_dlq.py but there is no test for it.  We should add a test but that was out of scope for this task.


Add extra user controls to CSVFieldMapper

Delete existing records before import

It would also be useful if users could set CSV delimiters, enclosures etc.

I've added in delete records.


node badges are missing descriptions #4871

Fix #4871

Warnings

:warning:
This PR modified service code for node but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @regevbr!

  Generated by :no_entry_sign: dangerJS against 2f5f49aa1af31a61e805be7b48fb616d91ff4163

@calebcartwright I added descriptions to the node badges, which we forgot to add in #4810
I'm not too good with texts, so please feel free to suggest a more user friendly description and I will change accordingly.

What do you suggest to do about the left align?

What do you suggest to do about the left align?

That's the alignment context of the help text, and not something that can be changed at this time, so you'll need to ensure that the text added for these badges looks decent in that context.
I'd still suggest removing, or at least trimming, some of the existing text which should help. You can also split content into multiple paragraphs

You can also include HTML tags in there. For example, I think you could wrap each paragraph in a <p>. Although these really are just sentences, and I think it would be better to combine it into one paragraph.

For example, I think you could wrap each paragraph in a . Although these really are just sentences, and I think it would be better to combine it into one paragraph.

That's what I was suggesting as well. "paragraphs" in the markup sense, not the grammar sense :smile: I think the main driver is the length of that second sentence which has to be wrapped, and that ends up making the whole block of content look off (at least to my eye)

Thanks guys for the comments. I gave it another shot, but as I said, expressing my self in writing was never a strong suite...

@calebcartwright I applied your suggestions.

Thanks @calebcartwright for helping out here


Emacs like keybindings

Thanks for developing Tofi!
As I use Emacs I added some of my own keybindings to reduce friction and thought they might be useful to others.
Tofi already has C-p and C-n for forwards and backwards, however when displayed horizontally C-f and C-b make more sense.
Additionally, I added C-g to exit Tofi.

Sorry merging this took so long, but it looks good to me - thanks for the PR!


Fix cannot assign requested address

We did not close the http.POST response Body in our calls. This results in the cannot assign requested address error after a couple of hours, thus the reconciliations get stuck.
{"level":"WARN","time":"2022-02-23T14:35:59.076Z","message":"Heartbeat failed to communicate status update 'failed' to mothership-reconciler: Post \"[http://reconciler-mothership-reconciler:80/v1/operations/1093--463a0d26-5d1d-4654-9bf9-1a0b33003438/callback/1093--6679cab7-79a1-4452-939e-ebce9539601d](http://reconciler-mothership-reconciler/v1/operations/1093--463a0d26-5d1d-4654-9bf9-1a0b33003438/callback/1093--6679cab7-79a1-4452-939e-ebce9539601d)\": dial tcp <IP_ADDRESS>:80: connect: cannot assign requested address","correlation-id":"1093--6679cab7-79a1-4452-939e-ebce9539601d","component-name":"CRDs"}
{"level":"ERROR","time":"2022-02-23T14:35:59.077Z","message":"Remote callback handler failed to send HTTP request: Post \"[http://reconciler-mothership-reconciler:80/v1/operations/1064--bcbfd0b9-cfce-461f-b1cd-edeb4a65d9e4/callback/1064--baa250ec-9a84-4b57-8a8b-bd9d7c9b3788](http://reconciler-mothership-reconciler/v1/operations/1064--bcbfd0b9-cfce-461f-b1cd-edeb4a65d9e4/callback/1064--baa250ec-9a84-4b57-8a8b-bd9d7c9b3788)\": dial tcp <IP_ADDRESS>:80: connect: cannot assign requested address","correlation-id":"1064--baa250ec-9a84-4b57-8a8b-bd9d7c9b3788","component-name":"certificates"}
{"level":"WARN","time":"2022-02-23T14:35:59.078Z","message":"Heartbeat failed to communicate status update 'failed' to mothership-reconciler: Post \"[http://reconciler-mothership-reconciler:80/v1/operations/1064--bcbfd0b9-cfce-461f-b1cd-edeb4a65d9e4/callback/1064--baa250ec-9a84-4b57-8a8b-bd9d7c9b3788](http://reconciler-mothership-reconciler/v1/operations/1064--bcbfd0b9-cfce-461f-b1cd-edeb4a65d9e4/callback/1064--baa250ec-9a84-4b57-8a8b-bd9d7c9b3788)\": dial tcp <IP_ADDRESS>:80: connect: cannot assign requested address","correlation-id":"1064--baa250ec-9a84-4b57-8a8b-bd9d7c9b3788","component-name":"certificates"}
{"level":"ERROR","time":"2022-02-23T14:35:59.083Z","message":"Remote callback handler failed to send HTTP request: Post \"[http://reconciler-mothership-reconciler:80/v1/operations/1023--7076ead9-0724-43ec-b992-80e44b58e3ef/callback/1023--3a829264-c65a-4a2e-8dc3-fcb1675edfc2](http://reconciler-mothership-reconciler/v1/operations/1023--7076ead9-0724-43ec-b992-80e44b58e3ef/callback/1023--3a829264-c65a-4a2e-8dc3-fcb1675edfc2)\": dial tcp <IP_ADDRESS>:80: connect: cannot assign requested address","correlation-id":"1023--3a829264-c65a-4a2e-8dc3-fcb1675edfc2","component-name":"cleaner"}
{"level":"WARN","time":"2022-02-23T14:35:59.083Z","message":"Heartbeat failed to communicate status update 'failed' to mothership-reconciler: Post \"[http://reconciler-mothership-reconciler:80/v1/operations/1023--7076ead9-0724-43ec-b992-80e44b58e3ef/callback/1023--3a829264-c65a-4a2e-8dc3-fcb1675edfc2](http://reconciler-mothership-reconciler/v1/operations/1023--7076ead9-0724-43ec-b992-80e44b58e3ef/callback/1023--3a829264-c65a-4a2e-8dc3-fcb1675edfc2)\": dial tcp <IP_ADDRESS>:80: connect: cannot assign requested address","correlation-id":"1023--3a829264-c65a-4a2e-8dc3-fcb1675edfc2","component-name":"cleaner"}
{"level":"ERROR","time":"2022-02-23T14:35:59.083Z","message":"Remote callback handler failed to send HTTP request: Post \"[http://reconciler-mothership-reconciler:80/v1/operations/1059--3e6588c3-53df-4852-8e15-2197ceccf30f/callback/1059--8c29b015-d1e3-44df-b5a6-b200fbe84040](http://reconciler-mothership-reconciler/v1/operations/1059--3e6588c3-53df-4852-8e15-2197ceccf30f/callback/1059--8c29b015-d1e3-44df-b5a6-b200fbe84040)\": dial tcp <IP_ADDRESS>:80: connect: cannot assign requested address","correlation-id":"1059--8c29b015-d1e3-44df-b5a6-b200fbe84040","component-name":"rafter"}

/LGTM


[BUG - Release] mods.txt parsing

Branch or Release
Release and main
Game and Engine Version
Any
Describe the bug
First line of mods.txt is read as having additional characters.
To Reproduce
Print the first mod's name to console or otherwise.
Expected behavior
No extra characters
Screenshots, UE4SS Log, and .dmp file

Can be fixed by simply recreating our mods.txt, but this problem is likely pervasive since last release with anyone's files. Unclear when it became corrupted without more testing.  So, it should probably be fixed programmatically if at all possible.

Duplicate of #486

Corrupted mods.txt was introduced between 2.5.2 and 3.

Corrupted mods.txt was not introduced as part of a commit.  Possibly related to our release action's method of copying instead.
https://github.com/UE4SS-RE/RE-UE4SS/commit/63ce8b427c7afca8e09f501035fd706eb0198104 - does not have the issue.

Solution would be to skip ï»¿ chars / EF BB BF hex when reading mods.txt for the foreseeable future.

Something like this should probably go into our File module to use when opening file streams.  I don't feel like doing that at the moment though.
    // BOM sequences
    const std::string BOM_UTF8 = "\xEF\xBB\xBF";
    const std::wstring BOM_UTF16_LE = L"\xFEFF";
    const std::wstring BOM_UTF16_BE = L"\xFFFE";

    // Function to check for BOM and reopen the file as a wide stream
    std::wifstream openFileSkippingBOM(const std::wstring& filename) {
        std::ifstream file(filename, std::ios::binary);
        if (!file.is_open()) {
            throw std::runtime_error("Failed to open the file.");
        }

        char bom[3] = {0};
        file.read(bom, 3);
        if (std::string(bom, 3) == BOM_UTF8) {
            // UTF-8 BOM detected
            file.close();
            std::wifstream wfile(filename);
            wfile.seekg(3, std::ios::beg); // Skip BOM
            return wfile;
        } else {
            // Check for wide BOM
            wchar_t wbom;
            file.seekg(0, std::ios::beg); // Rewind to start
            file.read(reinterpret_cast<char*>(&wbom), sizeof(wchar_t));
            file.close();
            if (wbom == BOM_UTF16_LE[0] || wbom == BOM_UTF16_BE[0]) {
                std::wifstream wfile(filename);
                wfile.imbue(std::locale(wfile.getloc(), new std::codecvt_utf8_utf16<wchar_t>));
                wfile.seekg(sizeof(wchar_t), std::ios::beg); // Skip BOM
                return wfile;
            } else {
                // No BOM detected
                file.close();
                return std::wifstream(filename);
            }
        }
    }

Fixed by #556 and #540


__typename was not executed because no suitable resolver could be found

Whenever I try to post a mutation via the library I get the error message
"_typename was not executed because no suitable resolver could be found - this indicates a resolver or validation bug. Please let us know by filing an issue."
I minimized my code for a suitable test example:

code example
class APIClient {
  String token = 'XXXXXX';

  GraphQLClient graphQLClient() {
    final HttpLink _httpLink = HttpLink('http://<IP_ADDRESS>:8080/graphql');
    final AuthLink _authLink = AuthLink(getToken: () async => 'Bearer $token');
    final Link _link = _authLink.concat(_httpLink);

    final Policies _policies = Policies(
      fetch: FetchPolicy.networkOnly,
    );

    return GraphQLClient(
      link: _link,
      cache: GraphQLCache(),
      defaultPolicies: DefaultPolicies(
        watchQuery: _policies,
        query: _policies,
        mutate: _policies,
      ),
    );
  }
}

const String mtAddTest = r'''
mutation addDance($input: [AddTestInput!]!) {
  addTest(input: $input) {
    test {
      name
    }
  }
}
''';

void test() async {
    final MutationOptions options = MutationOptions(
      document: gql(mtAddTest),
      variables: <String, dynamic>{
        "input": [
          {"name": "temp"}
        ]
      },
    );

    QueryResult result = await APIClient().graphQLClient().mutate(options);
    if (result.hasException) {
      print(result.exception.toString());
    }
  }

At first I thought the db at fault, as subscriptions were still working (not in the code example, for simplicity reasons), but then I completely started from scratch.
Here my db schema for dgraph if that matters.
type Test {
    name: String!
}

In the meantime I can execute the mutation via insomnia quite fine.
I work with the library graphql: ^4.0.0-beta.5   but also tried ^4.0.0-beta.2 for testing sake. It was all the same.
The server is run locally and accessed over the local ip address.
Also Flutter is run on the same computer using an emulator.

flutter doctor -v

[flutter] flutter doctor -v
[✓] Flutter (Channel stable, 1.22.4, on Linux, locale en_US.UTF-8)
    • Flutter version 1.22.4 at /opt/flutter
    • Framework revision 1aafb3a8b9 (5 weeks ago), 2020-11-13 09:59:28 -0800
    • Engine revision 2c956a31c0
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /home/owner/Android/Sdk
    • Platform android-30, build-tools 30.0.0
    • ANDROID_HOME = /home/owner/Android/Sdk
    • Java binary at: /opt/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Android Studio (version 4.0)
    • Android Studio at /opt/android-studio
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Connected device (1 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86 • Android 10 (API 29) (emulator)

If someone has the same issues, or has some input for me that would be great,
as I am reaching for straws.

Just tried the query option.
It works when the __typename is also used in the query (like #785), otherwise the same error occurs.

gql adds  __typename to every selection set recursively, so I don't know how manually adding __typename would change things. The error is being thrown on the server side – resolvers are a server side graphql concept and the client has no real awareness of them.
manual addition of __typename changing the results is very strange if you're using the gql helper bundled with the client.

Thank you for your input, it proved to be quite helpful.
The issue with the __typename in the query resolved itself somehow, so that the __typename specification is no longer needed. ¯_(ツ)_/¯
For the mutation the error messages comes from one __typename that is inserted by gql:
mutation addTest($input: [AddTestInput!]!) {
 __typename                 <- just this
 addTest(input: $input) {
   __typename
   test {
     __typename
     name
   }
 }
}

@micimize, you are right, as the error message is coming from the server.
My question then changes into: is the upmost __typename the intended graphql syntax, and what return value is expected for that. I am assuming 'Mutation' as that would be the type.
{
 "data": {
                  <- I assume then you'd expect to have "__typename": "Mutation" here?
   "addTest": {
     "__typename": "AddTestPayload",
     "test": [
       {
         "__typename": "Test",
         "name": "john"
       }
     ]
   }
 },

As this seems to be an error on the dgraph service and I don't want to waste your time, I'll close this issue.
For everyone else having the same problem, please refer to the dgraph issue

As the dgraph service does not allow the upmost __typename, I'd like to know if there is a way to disable that in gql and if not if there is documentation to change gql myself accordingly.
thanks for your help, even though it is quite a special case

@noobloser that's interesting – it should be fairly easy to do. gql is really just:
DocumentNode gql(String document) => transform(
      parseString(document),
      [AddTypenameVisitor()],
    );

where AddTypenameVisitor is from normalize. I think the following will do what you want:
import 'package:normalize/utils.dart';

class AddNestedTypenameVisitor extends AddTypenameVisitor {
  @override
  visitOperationDefinitionNode(node) => node;
}

DocumentNode gql(String document) => transform(
      parseString(document),
      [AddNestedTypenameVisitor()],
    );

@noobloser I'm going to close this as something like the above should work, but comment if it doesn't and I'll try and help troubleshoot.
Dgraph is actually spec-noncompliant here, so I'd recommend opening an issue there:

Type Name Introspection should be supported against any Object type
All Root Operation Types are also Object types.


shoot

The output of /version on my server is: Paper 632 (MC 1.13.2)
The output of /version denizen on my server is: Denizen dev build 898
The issue is:
When using the shoot command and looking straight down a console error is thrown. The entity just falls to the ground. (tested with arrows and fireball, on the above version, and Paper 115 (Mc 1.14.3)
https://one.denizenscript.com/paste/54975

Use the speed: argument when using the shoot command.


Bootstrap sampling

Bootstrap sampling (see kuchaale/grl_2020/code/montecarlo_composites_script-faster.py) where we could consider error estimates _err in the sampling.

@gast-ben, can you open pull request here so that I am able to continue with your code?

Pull request was created in the new branch 'bootstrap' #26


"--enable-local-huggingface-models" also seems broken

Seemingly in a different manner from #2056
I've tried a bunch of permutations, but I can't get it to pick up a local model. Basing things on #1505
run_specs.conf:
entries: [
  {description: "gsm:model=huggingface/llama2_gsm8k_lora", priority: 1},
  #{description: "gsm:model=huggingface/meta-llama_Llama-2-7b-hf", priority: 1},
  #{description: "gsm:model=stanford-crfm/BioMedLM@main", priority: 1},
]

commands:
helm-run --conf-paths run_specs.conf --suite v1 --max-eval-instances 10  --enable-huggingface-models /scr/biggest/dlwh/llama2_gsm8k_lora/ --enable-local-huggingface-models /scr/biggest/dlwh/llama2_gsm8k_lora/

helm-run --conf-paths run_specs.conf --suite v1 --max-eval-instances 10 --enable-local-huggingface-models /scr/biggest/dlwh/llama2_gsm8k_lora/

both fail with
  File "/nlp/scr/dlwh/miniconda3/envs/dlwh-helm/lib/python3.8/site-packages/helm/proxy/models.py", line 923, in get_model
    raise ValueError(f"No model with name: {model_name}")
ValueError: No model with name: huggingface/llama2_gsm8k_lora

cc @julian-q

@yifanmai fwiw i tried this with v0.2.3 (referenced at https://github.com/stanford-crfm/helm/pull/1505#issuecomment-1648352798) and it doesn't work there either. could be i'm misconfiguring?

This should be working now on v0.4.0.


bumping vendered dependancies

updating versions of vendored code as identified by dependabot
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

good to go @Luap99 , thanks for the review

LGTM

/lgtm


use ic-repl on staging environments

I need to use ic-repl with my staging environment
i thought the command would look something like this:
#!/usr/bin/ic-repl -r staging
identity staging "./staging.pem";
import blah = "zzzzz-oqaaa-aaaan-ql32q-cai";
call poop.setPooWasm(file("./poo.wasm"));
and then
ic-repl -r staging local_upload.sh
because my dfx.json looks like this
"networks": {
"staging": {
"providers": [
"https://icp0.io/"
],
"type": "persistent"
},
any idea how to do this??

ic-repl doesn't read dfx.json. Looks like your staging network still points to the mainnet, so either ic-repl -r https://icp0.io or ic-repl -r ic would work.


Define the specs of notarization files

Done, need review. see here: https://github.com/o1-labs/zk-oracles/pull/1


Publish the relase version of v3.0.0

Currently, it's just a pre-release.
https://github.com/toji/gl-matrix/releases/tag/v3.0.0-0
https://www.npmjs.com/package/gl-matrix?activeTab=versions

The main thing that has changed is:
https://github.com/toji/gl-matrix/pull/339
https://github.com/toji/gl-matrix/pull/343
https://github.com/toji/gl-matrix/pull/345

https://www.npmjs.com/package/gl-matrix?activeTab=versions

Yay!


PR: Fix Ctrl+Tab to cycle files on the editor

Description of Changes

[x] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

As per https://bugreports.qt.io/browse/QTBUG-8596, the action needs to be added to a menu for this to work on OSX. Created a "hidden" menu to try this out and it is working :-). now... where should we add these actions? We cannot disable or hide them or this will cease to work

This PR now updates the default shortcut on MAC to use the control key as chrome firefox and similar applications do on OSX.
There is a separate issue on the Spyder Preferences related to setting shortcuts on OSX that use tab and control, which currently do not seem to work.
Issue(s) Resolved

Fixes #6055
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @goanpeca

Hello @goanpeca! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/editor/plugin.py:

Line 2750:5: E303 too many blank lines (2)

where should we add these actions?

I think they can go in the File menu.

Like this @ccordoba12

?

Let's put them at the end, i.e. after Symbol switcher and before Restart. That's because I think most people will use the shortcuts instead of the actions.

Let's put them at the end, i.e. after Symbol switcher and before Restart. That's because I think most people will use the shortcuts instead of the actions.

👍

Ready @ccordoba12

Hi everyone. Did this eventually got fixed? I've downloaded and installed the latest version of Spyder 5.1.1. for Mac and can't see these newly added options to the file menu


Is it possible to co-exist two plugins which are add-adv and sspi, in one mptcpd?

Hello, MPTCP developer.
I'm a newbie to understand MPTCP daemon.
Can I ask for some question?
According to before question https://github.com/multipath-tcp/mptcpd/issues/287,

If yes, mptcpd can work with both, it depends on which path manager plugin you use, e.g. add-adv will setup the in-kernel path manager, while sspi will require the userspace PM.

'add-adv' uses in-kernel path manager and,
'sspi' uses Userspace path manager
Q1. Is it possible to co-exist two plugins which are add-adv and sspi, in one mptcpd ?
I think, It's not possible because 'add-adv' uses in-kernel path manager and 'sspi' uses Userspace path manager. so, to use them, net.mptcp.pm_type should be changed flexibly.
Q2. If yes, When pm_type will be changed?
Thanks.

Q1. Is it possible to co-exist two plugins which are add-adv and sspi, in one mptcpd ?
I think, It's not possible because 'add-adv' uses in-kernel path manager and 'sspi' uses Userspace path manager. so, to use them, net.mptcp.pm_type should be changed flexibly.

Indeed, that's not possible with a single mptcpd. But note that the path-manager can be different per network namespace (netns). You can then have one netns with pm_type=0 and one instance of mptcpd using the add-adv plugin, and another netns with pm_type=1 and another instance of mptcpd using the sspi plugin.
Also, changing the pm_type will only affect connections created after the modification. The ones created before will continue to use the previous path-manager. (So yes, on the same netns, you can have connections using different path-managers, but the goal is not using the two at the same time.)

Thank you.
Additionally, When I checked sspi code,
if MPTCP_EVENT_CREATE event occurs, by sspi plugin, mptcpd will call 'send_add_addr" with MPTCP_PM_CMD_ANNOUNCE including an address for specific interface. And, I think sending MPTCP_PM_CMD_ANNOUNCE will lead to trigger to create a subflow.
Q1. Why do sspi call MPTCP_PM_CMD_ANNOUNCE, instead of MPTCP_PM_CMD_SUBFLOW_CREATE?
is there any reason for that?
Q2. sspi is focusted on the client side ? or both (server and client) ?
Thanks.

Q1. Why do sspi call MPTCP_PM_CMD_ANNOUNCE, instead of MPTCP_PM_CMD_SUBFLOW_CREATE?
is there any reason for that?

This plugin is just a demo one, to give an example of what can be done with the plugins. Here, it announces addresses, but it doesn't create any subflows.

Q2. sspi is focused on the client side ? or both (server and client) ?

Maybe more the server side. But again, that's just a demo. People can create a plugin that is specific to their use-cases. Typically, when such use-cases are not handled by the in-kernel PM.
I guess this repo can also host new plugins with generic use-cases, e.g. a _single-subflow-per-interface` path-manager, more focussed on the client side. Or add options, but that requires #210.

Thank you!!


render names on buildings

Style change requested
building names can have significance, and should be rendered when present
Where this is visible
Pavilion names in NY World's Fair ground, 1964-1965
https://www.openhistoricalmap.org/#map=17/40.74617/-73.84290&layers=O

it seems they do get rendered for certain types of buildings, e.g. schools and churches. See https://www.openhistoricalmap.org/#map=18/47.51480/9.42904&layers=O

@geohacker, it appears to me that the buildings that currently have icons and labels are in the Source Layer amenity_areas, rendered in the points_of_interest_fromareas and points_of_interest_fromareasz14 layers. We're also going to show the name labels of buildings from the Source Layer buildings, rendered in the points_of_interest_frombuildings layer. Does that sound right to you?

@tsinn that sounds right to me, yes. It's ok if you discover we might want to split those into different source layers, I can make that happen.

I'm not sure I follow how this is being resolved - I do see building names on some buildings, so has this already been done?

I'm not sure I follow how this is being resolved - I do see building names on some buildings, so has this already been done?

When we label based on the name of the polygon, we run into the same issue we have/had with labeling the polygons of states and countries. We'll have to determine if this is okay, or if we want to have nodes for labels.

I don't mind labels a ton and there are good reasons for using points for building information, but... this wasn't a problem with raster tiles. I'm really surprised that this is a capability step backward in rendering.


[QUESTION] - Is there a way to add label to the connecting lines?

View https://github.com/jerosoler/Drawflow/issues/18#issuecomment-991167477
And issue:

#18


Feature Request: Drush Aliases use Project Folder name when "getting"

If I run
$ platform get 3eelsfv6koejw lp

And then I
$ cd lp

And run
$ platform drush-aliases

I would love to see
@lp.master
@lp.develop

Instead I see
@3eelsfv6koejw.master
@3eelsfv6koejw.develop

This was mentioned before so I finally implemented it v2.4.0. So update to the latest version v2.4.3 and it should work for you - re-open this if not!


Fan Speed configuration missing

Not sure when this has changed but i can't find the optuion to change the fan speed

config_entry-maestro_mcz-466d8e4de749e610e22c8bce7783edad.json(3).txt

MCZ changed the config for FAN 1, I've also noticed this some time ago. I'll look into fixing this.

Hello,
I have the same problem with my stove which has two fans.
Only the second fan is visible in home assistant.
Thanks

config_entry-maestro_mcz-df57f039dff96c4ad058338ada8306e3.json.txt

UPDATE: I'm currently working on a fix, it's already working for my model, but I need some testers for the M1+ models as well.
This is harder than I initially thought since the fan configurations are now embedded in the configuration of the current mode the stove is in. This means I had to make a mapping foreach mode to the right configuration. I did this foreach mode I know of, but hopefully this will be the same for other models. That's actually the hardest part, I can't test on other models (e.g. hydro, ...).

happy to do some beta-testing

@VaillantHassIo, I'll soon post a PR linked to this item.

Happy to do some beta testing to. 👍
Is it solving the other problem with fan?

here is the log:
`Logger: homeassistant.components.websocket_api.http.connection
Source: components/websocket_api/commands.py:238
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 13:35:52 (5 occurrences)
Last logged: 20:27:10
[547504673856] MczFanEntity.async_turn_on() takes 1 positional argument but 3 were given
[547604844352] MczFanEntity.async_turn_on() takes 1 positional argument but 3 were given
[547506757056] MczFanEntity.async_turn_on() takes 1 positional argument but 3 were given
[547641341504] MczFanEntity.async_turn_on() takes 1 positional argument but 3 were given
[547507256384] MczFanEntity.async_turn_on() takes 1 positional argument but 3 were given
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 238, in handle_call_service
response = await hass.services.async_call(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/core.py", line 2149, in async_call
response_data = await coro
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/core.py", line 2186, in _execute_service
return await target(service_call)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 272, in handle_service
return await service.entity_service_call(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 882, in entity_service_call
single_response = await _handle_entity_call(
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 952, in _handle_entity_call
result = await task
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/fan/init.py", line 334, in async_handle_turn_on_service
await self.async_turn_on(percentage, preset_mode, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: MczFanEntity.async_turn_on() takes 1 positional argument but 3 were given
`

and the diagnostics files:
config_entry-maestro_mcz-04a794c9c7eed4e9f7e8ecaf83460238.json.txt

and of course I give you a star. You did a great job with this integration.
Thanks a lot.

@VaillantHassIo, @Leon3310, @lalek54,
You guys can start BETA testing with this change.
Official release will follow after testing.
Meanwhile, I'll keep making improvements on this.
Cheers,
Robbe

Great! HACS doesnt seem to find an updated beta version. Do you want us to load the code directly or did you release a new beta?

@VaillantHassIo , I can release a new pre-release if you want ?

Update: pre-release v0.4.4-beta is now available !

Great work!

Wow quick update. Thanks a lot.
seems a lot better: fan 1 came back, I was able to change fan 3 mode to silent.
but now I me unable to command fans.
nothing happen et still the same error.

Wow. Crazy fast answer. Thanks. It s working except an error message but that’s fine.
i can’t set « auto » mode because it s not defined as preset mode.
am I able able to add it somewhere tonuse it?


Update Device.cs to add read memory

Added command to read memory
This adds some overloaded copies of function from the base class which aren't accessible due to them being private (which isn't pretty) - however it may give others a way of doing this.

These changes should be done on https://github.com/IntergatedCircuits/DeviceProgramming/blob/master/Dfu/Device.cs so you don't need to copy existing functions.
Where does the 1024 max transfer size come from?


STANDARDS.md update

Fixes #396.
Updates KEM encoder limitation statement.

Merging will fail until #423 has landed, so waiting with this for now.


Refactor to avoid barrel files

Looks like at some point a barrel file was introduced.
export * as Accordion from './accordion';
export * as Alert from './alert';
export * as Avatar from './avatar';
export { Badge, type BadgeProps } from './badge';
export { Button, type ButtonProps } from './button';
export * as Card from './card';
export * as Carousel from './carousel';
export * as Checkbox from './checkbox';
export * as Clipboard from './clipboard';
export { Code, type CodeProps } from './code';
export * as Collapsible from './collapsible';
export * as ColorPicker from './color-picker';
export * as Combobox from './combobox';
export * as Dialog from './dialog';
export * as Drawer from './drawer';
export * as Editable from './editable';
export * as Field from './field';
export * as FileUpload from './file-upload';
export { FormLabel, type FormLabelProps } from './form-label';
export { Heading, type HeadingProps } from './heading';
export * as HoverCard from './hover-card';
export { IconButton, type IconButtonProps } from './icon-button';
export { Input, type InputProps } from './input';
export { Kbd, type KbdProps } from './kbd';
export { Link, type LinkProps } from './link';
export * as Menu from './menu';
export * as NumberInput from './number-input';
export * as Pagination from './pagination';
export * as PinInput from './pin-input';
export * as Popover from './popover';
export * as Progress from './progress';
export * as QrCode from './qr-code';
export * as RadioButtonGroup from './radio-button-group';
export * as RadioGroup from './radio-group';
export * as RatingGroup from './rating-group';
export * as SegmentGroup from './segment-group';
export * as Select from './select';
export * as SignaturePad from './signature-pad';
export { Skeleton, type SkeletonProps } from './skeleton';
export * as Slider from './slider';
export { Spinner, type SpinnerProps } from './spinner';
export * as Splitter from './splitter';
export * as Switch from './switch';
export * as Table from './table';
export * as Tabs from './tabs';
export * as TagsInput from './tags-input';
export { Text, type TextProps } from './text';
export { Textarea, type TextareaProps } from './textarea';
export * as Toast from './toast';
export * as ToggleGroup from './toggle-group';
export * as Tooltip from './tooltip';
export * as TreeView from './tree-view';

This is causing a whole host of problems tree shaking the bundle and also the build times have substantially increased.
Normally, I would just make sure that the file is not referenced anywhere in my codebase (or just delete it). But the problem is that the barrel file is also referenced within the component library.
I ended up rewriting all references in the generated code, e.g. avatar.tsx:
- import { Avatar as ArkAvatar } from '#app/components/ui/primitives/avatar';
+ import * as ArkAvatar from '#app/components/ui/primitives/avatar';

However, I would greatly appreciate if you consider dropping this pattern.

Ended up writing a post generation script to fix this.

@lilouartz
This is an interesting topic. I understand that there are some downsides to using asterisk exports like:
export * from './avatar'

Since the compiler/transpiler needs to resolve each export individually. However, I thought this kind of import should be fine as the compiler knows what it resolves:
import * as Avatar from '~/components/ui/avatar'

We use this approach to abstract away whether the user should use * as or a normal import syntax:
import * as Avatar from '~/components/ui/avatar'
// or
import { Avatar } from '~/components/ui/avatar'

Could you provide some numbers to verify your claim about tree shaking and bundle size?

@cschroeter wow, this latest update is great! Thank you so much ❤


Vector Maps

Hi,
Since the work is based on Autoware and it makes use of vector maps for additional information like cross walk, stop lines. However, I see no such information about vector maps in Aslan. Is there any reasons to not include it in project Aslan? Had it not been more beneficial to have vector maps?
Thank you.

Hi @tekrajchhetri ,
This release of Aslan is tested and proven stable with point cloud maps only. On the latest Changelog you can find all the features in the aslan stack as well as the features added on other open source stacks and the differences between the original ones.
If you would like to add a new feature in Aslan please create a new feature request and we can discuss the specifics there.

@EfimiaPanagiotaki-StreetDrone I checked the change log, could not find there, will create a feature request.


Multiplying the value of dynamic chartExtents

I currently have the chartExtents set up to be dynamic.
          chart-extents='{
            "x": ["dynamic", "dynamic"],
            "y": ["dynamic", "dynamic"]
          }'

Is there a way to make the default chartExtents the max and min values * 1.1 or something?
As seen in this pic, the charted line is literally at the top of the graph because the numbers vary so little.

No, there is no way to currently do this other than manually passing in the calculated value into chartExtents.


Crash on resize on master

@iphydf I'm not sure which of your recent changes could have impacted this, but when I attempt a resize I get this crash pretty consistently. Would you be willing to look into this?
matterhorn: vty internal failure: this value should not propagate to users
CallStack (from HasCallStack):
  error, called at src/Graphics/Vty/Input.hs:161:21 in vty-5.34-inplace:Graphics.Vty.Input

Here's some relevant code, but I haven't looked into how this might have arisen as a consequence of your changes:
      let pokeIO = Catch $ do
              let e = error "vty internal failure: this value should not propagate to users"
              setAttrs
              atomically $ writeTChan (input^.eventChannel) (EvResize e e)
      _ <- installHandler windowChange pokeIO Nothing
      _ <- installHandler continueProcess pokeIO Nothing

Will take a look at that now.

Or alternatively maybe I can avoid the bottom value altogether. I'll have a look at the code.

I guess my reaction to the code above is, why is a lazy EvResize being written to that channel in the first place? Is there a better way to deal with whatever that code is for?

I agree. I'm trying to figure out where that event goes and where it's being evaluated.

(I.e. the initial draw)

Sending any other event works too (e.g. EvGainedFocus). I don't know if it always works, though.

Not sending any event does not work (resize becomes delayed until the next real event happens), so yeah that seems reasonable.

Looking at the code, if this is just for recovery after signals, then would it be possible to provide the actual window dimensions in the EvResize event that the handler is producing? That seems like a way to make this behavior legitimate.

It is received here, ignoring the bottom values in the constructor arguments: https://github.com/jtdaugherty/vty/blob/e4fb394cdf9565673697af6aba9c6427337a9b05/src/Graphics/Vty.hs#L215
and then it's fixed up to become a real resize event with the new bounds. It doesn't seem like we can access the bounds in the sender context. This also means that sending anything other than EvResize bot bot will not be quite correct (even though it does seem to work - I'm not sure when it would not).

Also correct would be to send EvResize 0 0 or anything else, but the current code would catch the introduction of a bug where that 0 value accidentally ends up in user code. The current code is better than sending 0 0.

Okay, that makes sense. Thank you for looking into this! I'll merge the PR you created, but I also want to think about improving the internal event-handling so that a constructor for this specific case could be added without polluting the event type that applications handle. I don't know how much of an overhaul that would require, but I'd like to have the cake and eat it too (make resizes strict, and not use dummy resize constructors as they are used here, overloading the Event type in this way).

I had some time and this turned out to be pretty straightforward. Let me know what you think, @iphydf -
https://github.com/jtdaugherty/vty/commit/2cd98a862c88069e3863ad0ad989dfa0e9081d29

(I didn't make the EvResize fields strict in that branch yet, but will if these changes are otherwise reasonable.)

Also, a note on the API breakage: I very much doubt that the majority of Vty users are in the habit of customizing their Inputs, and since that's the only spot where the new type affects the user-facing API, I suspect that the risk of this causing anyone any trouble is very low.

LGTM. Yeah, that's a good way of doing it. Thanks for the better fix :).

Okay, thanks for taking a look. It's now merged, and I'll remove the laziness annotations from EvResize shortly.


brasilia lifecycle integration

In addition to the events implementation (see #3), there are other expectations of a Brasilia function, such as initial registration with the host.  The Brasilia adapter will therefore need to also communicate with the host on instantiation, performing any handshake necessary to register itself as the running worker, with the function process (expected to be started or starting on a well-known local port) to be the implementation, essentially decorated by Grid.

Some WIP code: https://github.com/matejvasek/boson-adapter


Invalid parameters appear when nested parameters are present in action input

Describe the Bug with repro steps

[Internal] Open sid-sap-testbox.ui-test in BTS4
Create [RFC] Call function in SAP in-app action
Set the Input Type to Custom
Then select the optional RFC Name, and set it as STFC_CONNECTION
A new dynamic textbox appear REQTEXT, enter Hello in that
Check save. - This experience is great.

Now refresh the page, or go to code view and then back to designer
Open the same action - A new parameter body will appear that was not part of the input when the action was first authored

What type of Logic App Is this happening in?
Standard (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
{
    "definition": {
        "$schema": "https://schema.management.azure.com/providers/Microsoft.Logic/schemas/2016-06-01/workflowdefinition.json#",
        "actions": {
            "[RFC]_Call_function_in_SAP": {
                "inputs": {
                    "parameters": {
                        "body": {
                            "REQUTEXT": "Hello"
                        },
                        "inputBodyType": "Custom",
                        "outputBodyType": "XML",
                        "rfcGroupFilter": "*",
                        "rfcName": "STFC_CONNECTION"
                    },
                    "serviceProviderConfiguration": {
                        "connectionName": "sap",
                        "operationId": "callRfc",
                        "serviceProviderId": "/serviceProviders/sap"
                    }
                },
                "runAfter": {},
                "type": "ServiceProvider"
            }
        },
        "contentVersion": "<IP_ADDRESS>",
        "outputs": {},
        "triggers": {
            "When_a_HTTP_request_is_received": {
                "kind": "Http",
                "type": "Request"
            }
        }
    },
    "kind": "Stateful"
}

Screenshots or Videos
No response
Browser
Edge
Additional context
No response

@siddharth-ms - The manifest is incorrect and that is why it behaves weirdly with the dynamic inputs.

You cannot have x-ms-dynamic-properties defined and also have some static properties for the same object. if any property has x-ms-dynamic-properties defined, then the dynamic call should result in replacing that whole property schema. This is the same case both for manifest based operations and swagger based operations.
You can defined another property and mark that as dynamicproperties and it should be working fine. You might have to change the location of your inputs for runtime... but that is how we have all manifests defined today.
If you need to discuss more, please set up time with me. I am closing with this comment


python3Packages.shellingham: use pyproject format

Motivation for this change
closes #89475 #89448
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

failures broken on target branch:
https://github.com/NixOS/nixpkgs/pull/89497
1 package failed to build:
python27Packages.poetry

14 packages built:
cmake-language-server doitlive python27Packages.shellingham python37Packages.aria2p python37Packages.click-completion python37Packages.poetry python37Packages.shellingham python37Packages.toggl-cli python37Packages.xdg python38Packages.aria2p python38Packages.click-completion python38Packages.poetry python38Packages.shellingham python38Packages.xdg


System.writeAll

A minor omission but the System.writeAll method is currently undocumented. As you'd expect it does the same as System.printAll except that it doesn't print a newline at the end.

It would be nice if you were sending PRs for missing documentation instead of raising an issue. Writing a documentation for a method takes less time that opening an issue. Do you know how to send PR for the docs? I use GitHub editing feature for that, not even messing with git, vscode or whatever.

I wanted to make sure that I was right about these apparent omissions before submitting PRs.
As you said yourself in one of the other issues, these methods may be private or might even be removed before 0.4.0 (though I hope they aren't as they're all useful).
Anyway,  if there are no objections, I will submit PRs for all of them tomorrow.

First, please submit separate PRs. I think @ruby0x1 will agree with that.
Second, GitHub and a cool (maybe unknown? 😛) feature that let you close PRs without merging them. When the change is such tiny, it's easier to send a PR then discuss about it, or close it, than to open an issue before.
Third, and somewhat off-topic, I'm not aware of any methods removed in Wren ever. Especially in 0.4.0. Remember that removing a public method is a breaking change.

OK, I've submitted separate PRs for each of the 'undocumented' issues I've raised (4 in total).

Third, and somewhat off-topic, I'm not aware of any methods removed in Wren ever.

Good as I dislike breaking changes unless they're absolutely necessary.


Suggestions to improve comparison pseudo-operators.

From the manual:

Octo also provides pseudo-ops for using <, >, <= and >= to compare two registers or a register with a constant
...
These are implemented by using the subtraction instructions -= and =- and querying vf, and will destroy the contents of a temporary register as well as vf. By default this temporary register will be ve, but defining an :alias named compare-temp can reassign it to any register but vf.

There is no reason why vF can't be used (in fact, there is no reason to use any other register)
if v0 > v1 then ...

This code is currently compiled to (written here in pseudo-code):
vE = v1
vE -= v0
skip if vF == 01

Which is not ideal. First, of course, it uses that temporary register rather than vF (using vF is ok since the flags are set after the subtraction is preformed)
Additionally, it's recommended to always compare vF with 0 rather than 1, because some interpreters set it to values other than 1 in some situations.
I believe a better result would be this:
vF = v1
vF -= v0
skip if vF != 0

Urm, there are only matter when quirks of "arithmetic results write to vf after status flag." are turned.
But yeah, that's okay when the code is compiled to preferred result when that quirks were turned off.

If you know it isn't a problem in your program, you can reassign the register used by assigning an :alias to compare-temp:
:alias compare-temp vF

There are method to compare both register without requiring register changes but vF.
v0 =- v1  v0 =- v1 if vF != 0 then equivalent as if v0 > v1 then or if v1 < v0 then
v0 =- v1  v0 =- v1 if vF == 0 then equivalent as if v0 <= v1 then or if v1 >= v0 then
But, for the constant, which I don't sure...
vF := n  vF =- v0  if vF != 0 then equivalent as if v0 > n then or if n < v0 then
vF := n  vF =- v0  if vF == 0 then equivalent as if v0 <= n then or if n => v0 then
This is trouble for arithmetic results write to vf after status flag quirk.

@12Me21 having come to the decision to deprecate the vf order quirks flag, there's no reason left not to implement this suggestion. Comparison against zero is also a safer choice. Done!


[🚀] Fix up project template README

🚀 Feature Request
Too much information and icons.
Reduce them and place them in further information after creating the project.
Some work is done in #1

Continues in #10


feat: add next/prev keyboard navigation to svelte inspector and improve a11y

this PR follows up on #436 to complete keyboard only navigation and a11y improvements
cc @geoffrich

i also cleaned up the changeset files, hoping it creates a cleaner changelog for 1.1.0. given inspector is still experimental and is going to move into a separate package at a later stage, should we use patch instead?

I think it'd be fine if we stick as a patch 👍


TraceLoggingEnhancer sets trace ID in the wrong field

The trace ID of a log entry is being set in the wrong field (label).
In order for the log entries to be linked to the trace, the TraceLoggingEnhancer needs to look something like this.

Use setTrace(), not setLabel()
Use a project/myProjectId/traces/myTraceId format.

Pull request: https://github.com/GoogleCloudPlatform/google-cloud-java/pull/3301


Fix Sightglass' Lucet flags

specifically, pass correct form of "as optimized as possible please" flag to lucet

Curious: Why not just speed?

The simple answer: because once upon a time this was "best" and "speed_and_size" these days maps to "ok cranelift, do the best you can do".
As for why we'd pick speed_and_size rather than speed when we're measuring... speed: so far as I know, speed_and_size won't do anything that would regress speed in favor of size, but does get cranelift to run through instruction squeezing that can do things like reduce offset sizes from four to one (or no!) byte, remove prefixes that aren't necessary, and other optimizations of that nature. This probably doesn't make a measurable impact, but if we're doing the best we can vs the best gcc can, it doesn't hurt to try.


Android - BA - One moment loader appears after every step while entering Personal info

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

Version Number: 1.4.71-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Precondition: Workspace created and enables Workflows in More features

Launch ND app on Android device
Go to Workspace settings >> Workflows >> select Connect Bank account
Select Connect manually
Enter Routing & Account number
Enter first & last name as "First" & "Last" name & tap Next (Note: One moment loader appears before DOB page)
Enter DOB ((Note: One moment loader appears before SSN page)
Enter SSN ((Note: One moment loader appears before address page)
Enter address (Note: One moment loader appears before confirm page)

Expected Result:
One moment loader should not appear after every step while entering personal info
Actual Result:
One moment loader appears after every step while entering Personal info
DOB page appears briefly after providing FN/LN, then immediately one moment loader is seen & this happens after DOB, SSN & address steps
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/1056aba6-8797-47f7-bf8d-9c201dcad5b6
View all open jobs on GitHub

@MitchExpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

Proposal
Please re-state the problem that we are trying to solve in this issue.
The one moment loader appears every time completing a sub-step of personal info. This happens in company/business info too.
What is the root cause of that problem?
This happens after this PR where we save each sub-step data. Every time we save a substep, we set the loading data to true.
https://github.com/Expensify/App/blob/e0b6a06d2f34f35cda96ccc4d05252ede111c732/src/libs/actions/BankAccounts.ts#L137-L139
The loading data will then show the one moment loading indicator as shown below.
https://github.com/Expensify/App/blob/e0b6a06d2f34f35cda96ccc4d05252ede111c732/src/pages/ReimbursementAccount/ReimbursementAccountPage.tsx#L391-L396
The reason this only happens in Android is because shouldReopenOnfido is true for Android but false for other platforms. The comment above explains why.
https://github.com/Expensify/App/blob/e0b6a06d2f34f35cda96ccc4d05252ede111c732/src/libs/shouldReopenOnfido/index.android.ts#L3
What changes do you think we should make in order to solve the problem?

If we are saving a sub-step data, don't set the loading data to true. We can pass the isConfirmPage from updatePersonalInformationForBankAccount to getVBBADataForOnyx.
https://github.com/Expensify/App/blob/e0b6a06d2f34f35cda96ccc4d05252ede111c732/src/libs/actions/BankAccounts.ts#L286-L297
Then, set the loading state to true if isConfirmPage is true.
https://github.com/Expensify/App/blob/e0b6a06d2f34f35cda96ccc4d05252ede111c732/src/libs/actions/BankAccounts.ts#L139
function getVBBADataForOnyx(currentStep?: BankAccountStep, shouldSetLoading = true): OnyxData {
    return {
        optimisticData: [
            {
                onyxMethod: Onyx.METHOD.MERGE,
                key: ONYXKEYS.REIMBURSEMENT_ACCOUNT,
                value: {
                    isLoading: shouldSetLoading,
                    errors: null,
                },
            },
        ],

We should probably wait for the performance issues to die down before resting this but I agree the copy is a bit weird regardless of the loader showing after every step

@MitchExpensify hi, I believe this is a buggy behavior.

This only happens in Android
The loading appears after the next sub-step is shown which gives a bad UX. (as pointed out in the 2nd actual result)

I think the loading should only show when moving between steps 1 -> 2 just like it was before https://github.com/Expensify/App/pull/37680

Reopening to investigate further

@MitchExpensify looks like it gets closed again unintentionally.

I reproduced this but I just don't feel like its worth fixing right now


Fix #4413: Handle JS symbols and bigints in systemIdentityHashCode.

By spec, for values that are not Scala objects nor primitives corresponding to a hijacked class, x.hashCode() resolves to j.l.Object.hashCode(), which delegates to System.identityHashCode().
Previously, the implementation of the latter would jump back to j.l.Object.hashCode() for JS bigints, and crash with a TypeError for symbols.
The main cause of these issues is that we thought we could delegate back to j.l.Object.hashCode() for all primitives, because they are supposed to be covered by hijacked classes. We had forgotten symbols, though, and anyway it does not work because not all JS primitives are covered by a hijacked class.
In this commit, we break this cycle. systemIdentityHashCode never delegates back to j.l.Object.hashCode() anymore. Instead, it separately handles every type of primitive in JavaScript with a dedicated hash code implementation. For strings and numbers, we reuse j.l.String.hashCode() and j.l.Double.hashCode(), because we can. For other values, we have an inline implementation.

In JavaScript, for ES 2015, this yields:
var $lastIDHash = 0;
var $idHashCodeMap = new WeakMap();
function $systemIdentityHashCode(obj) {
  switch ((typeof obj)) {
    case "string": {
      return $f_T__hashCode__I(obj)
    }
    case "number": {
      return $f_jl_Double__hashCode__I(obj)
    }
    case "bigint": {
      var biHash = 0;
      if ((obj < BigInt(0))) {
        obj = (~obj)
      };
      while ((obj !== BigInt(0))) {
        biHash = (biHash ^ Number(BigInt.asIntN(32, obj)));
        obj = (obj >> BigInt(32))
      };
      return biHash
    }
    case "boolean": {
      return (obj ? 1231 : 1237)
    }
    case "undefined": {
      return 0
    }
    case "symbol": {
      var description = obj.description;
      return ((description === (void 0)) ? 0 : $f_T__hashCode__I(description))
    }
    default: {
      if ((obj === null)) {
        return 0
      } else {
        var hash = $idHashCodeMap.get(obj);
        if ((hash === (void 0))) {
          hash = (($lastIDHash + 1) | 0);
          $lastIDHash = hash;
          $idHashCodeMap.set(obj, hash)
        };
        return hash
      }
    }
  }
}


style(go): fix

Codecov Report

Merging #206 (c1a425d) into design/event-driven-modular-monolith (32c58e2) will not change coverage.
The diff coverage is n/a.

@@                          Coverage Diff                          @@
##           design/event-driven-modular-monolith     #206   +/-   ##
=====================================================================
  Coverage                                 89.28%   89.28%
=====================================================================
  Files                                         1        1
  Lines                                        28       28
=====================================================================
  Hits                                         25       25
  Misses                                        3        3

Flag
Coverage Δ

unittests
89.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 32c58e2...c1a425d. Read the comment docs.


feat: tsdx improvements keychain

@aulneau @hstove what's the latest here?

I need to rebase and clean up this branch, as it got a bit out of date.


fix(improvements): Improve the usability of karmor probe by detailing…

… error messages
When KubeArmor is running in systemd mode, the following error may occur when executing "karmor probe".
> karmor probe
probe.go:380: error when getting kubearmor daemonset Get "http://localhost:8080/apis/apps/v1/namespaces/kube-system/daemonsets/kubearmor": dial tcp <IP_ADDRESS>:8080: connect: connection refused

Didn't find KubeArmor in systemd or Kubernetes, probing for support for KubeArmor

Host:
  Observability/Audit: Supported (Kernel Version 5.15.0)
  Enforcement: Full (Supported LSMs: lockdown,capability,landlock,yama,apparmor)
To get full probe, a daemonset will be deployed in your cluster - This daemonset will be deleted after probing
Use --full tag to get full probing

When "karmor probe" is executed, "systemctl status kubearmor" is executed internally, but the systemctl command may require sudo. In this case, without sudo, an error occurs.
In this commit, to make it easier to understand the root cause of such an error, error messages have been modified to be more detailed.

@haytok 🤔I don't think we should handle it this way, as whenever someone will run karmor probe for k8s it will log this error. WDYT?