I-MISALIGN_LDST-01 assumes misaligned data access will trap

I-MISALIGN_LDST-01 assumes misaligned data access will trap. Misaligned data access traps are NOT required by the ISA spec v2.2 nor the privileged spec; it is entirely reasonable for the misaligned access to be handled in hardware.
The test should be written to succeed if either the misaligned access completes correctly with no trap, or traps with the appropriate values in the CSRs.
This issue came up when I was testing a core I designed for the RISC-V soft CPU core design contest, which requires that the core pass all 55 RV32I tests. My core implements misaligned data access in hardware, but I had to add extra stuff to my core to implement the misaligned data access traps to pass the test.

@brouhaha
Yes you are right. This test apart from checking misaligned data-access behavior also checks trap mechanisms. And as you pointed out traps on mis-aligned data-access are not mandatory. I think this test would be better suited for "privilege tests"
I have added a new test to check misaligned data-accesses. You can see them in this pull-request.
Would be helpful if you could confirm that you get the same signature for this test using your HW.
Thanks,

Current implemetation of I-MISALIGN_LDST-01 is testing misalign trap and it should be skipped if misalign memory access is supported in HW. It is same case as I-MISALING_JMP-01 which is valid only for implementations without compressed extension.
@neelgala
In my opinion, the better approach is to add new test for HW support while keeping the original one for traps. Framework should select which one to run for given configuration. I think we should keep tests simple and we shouldn't mix features together.
The reasons why I think it's better:

code maintenance - when someone will change one part of the test (for example adding extra check for MEPC and others CSR registers etc.) he won't break the second part of the test unintentionally (for example when he adds extra signature output to trap handler). Moreover right now there is no way how to prove the HW support part of the test is correct. As far as I know spike does not support this.

there is possibility that GM and tested implementation will execute different code of the test which makes it very hard to debug. I think in general GM should always run same code as tested implementation and we should avoid ambiguous program flow.

makes both tests easily extensible in the future.

Hi Radek
your comment
'Framework should select which one to run for given configuration. I think
we should keep tests simple and we shouldn't mix features together'
yes it does - today in a very simple way, where you select the suite... for
example
make RISCV_ISA=rv32i
this runs RV32 I instructions in machine mode

but it needs much further discussion as there is the other school of
thought where the test explores what is implemented
there are many many configuration options and this means there would need
to be a test for each various cross multiplied configuration set meaning an
explosion in the number of tests and then a framework to select them all
etc the framework then becomes very complex - the effort moves to the
framework and not the tests
or the other approach is a few more complex tests
hence more discussion is needed
it is like the choice of self checking tests or a signature file to compare
with - there are pros and cons of each option
and probably what needs to happen is to carefully document the different
approaches in detailed proposals and then have the community of compliance
concerned companies make a decision - this was done for the choice of
signature or self checking and the decision was signature - which is the
current place/direction we are at.
I personally am in favour of smarter tests that explore the DUT (which can
be done in the mandatory machine mode) and then exercise and check what is
enabled in that device for a specific suite
in this test specific instance (mis-aligned load/store) there is detailed
discussion needed about what is mandatory in hardware and what is optional
etc...
all for future discussions...
Simon
On Mon, Nov 26, 2018 at 8:41 AM Radek Hajek (Codasip) <
<EMAIL_ADDRESS>wrote:

Current implemetation of I-MISALIGN_LDST-01 is testing misalign trap and
it should be skipped if misalign memory access is supported in HW. It is
same case as I-MISALING_JMP-01 which is valid only for implementations
without compressed extension.
@neelgala https://github.com/neelgala
In my opinion, the better approach is to add new test for HW support while
keeping the original one for traps. Framework should select which one to
run for given configuration. I think we should keep tests simple and we
shouldn't mix features together.
The reasons why I think it's better:

code maintenance - when someone will change one part of the test (for
example adding extra check for MEPC and others CSR registers etc.) he won't
break the second part of the test unintentionally (for example when he adds
extra signature output to trap handler). Moreover right now there is no way
how to prove the HW support part of the test is correct. As far as I know
spike does not support this.
2.
there is possibility that GM and tested implementation will execute
different code of the test which makes it very hard to debug. I think in
general GM should always run same code as tested implementation and we
should avoid ambiguous program flow.
3.
makes both tests easily extensible in the future.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-compliance/issues/22#issuecomment-441558918,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJrER215wUAyQ_6U7-cZsMcq8I9SjYNQks5uy6kCgaJpZM4Yxqja
.

--
The information contained in this electronic mail message and any
attachments hereto
is privileged and confidential information intended only for the use of the
individual or
entity named above or their designee.  If the reader of this message is not
the intended
recipient, you are hereby notified that any dissemination, distribution or
copying of this
communication is strictly prohibited.  If you have received this
communication in error
please immediately notify us by return  message or by telephone and delete
the
original message from your mail system.  Thank you.

Update: the riscof framework will have a YAML cofiguration file that describes what the implementation supports. This includes the  extensions defined in the configuration string, and many other things (how CSR WARL fields respond to illegal values, which optional CSRs are implemented, and whether unaligned accesses are supported. This will be parsed, and used to select which tests will be run. There will be two different tests: one that expects HW supported unaligned accesses, and one that expects traps (alternatively, we could have a test that returns 2 different signatures, but the reference model will still have to be configured to support unaligned access or trap in any case)

The latest framework release, riscof, should have fixed this issue.
Could you confirm that this is or is not the case so we can close this?
Note that in order to do this you'll need to install and configure the new framework,
and create the YAML file that describes your hart (and specifically, misalign support or lack of it)

Needs to be coordinated with Sail (& Spike). Should be closed when Sail parsing of YAML file is conplete

It is possible to configure Sail to support or not support misaligned load/store, and the test infrastrucutre should handle this correctly now. It may be necessary to manually add the misaligned support to the Sail command line with -m to force it to be configured for misaligned support.


Make all/most info on a Issue's detail page editable

We should at list make the title and description editable. Ideally more, if the time required to work on it makes sense in the PoC phase.

@gsaslis were we meaning to have this implemented as part of the PoC (I suppose that's why we added #7, right?)?


Chemistry Bottles cannot be placed in chem dispenser, master and syringes can't draw/inject from it

Description
Chemistry Bottles can be drank from and transfer their contents into beakers and large beakers, but they cannot be directly placed in chem-masters, dispensers or drawn from by droppers or syringes. I suspect this is because they don't have the drawable/injectable tag in the code like the beakers do.
Step 1) Go to a Chem Dispenser, put a beaker in and then anything you like in the beaker then alt+click to extract the beaker when done.
Step 2) Put your beaker in the chem-master, transfer to buffer and then create a bottle.
Step 3) Take a syringe and try to draw from the bottle. You won't be able to.
Step 4) Try to put your bottle in chem master or chem dispenser, you won't be able to either.
Screenshots

These are the bottles in question
Additional context
Since Pills can't be labelled or have sprites changed, the only way for a doctor to be able to identify what's in his medkit or medbelt is using bottles and memorising the descriptions. At the moment doctors have to manually force-feed people (which is not as immediate as syringe) or make the chems on the spot. Or... rigorously catalogue and organise with several medkits and pills.

Step 3) Take a syringe and try to draw from the bottle. You won't be able to.

I fixed the injector behaviour in https://github.com/space-wizards/space-station-14/pull/5250.

Step 4) Try to put your bottle in chem master or chem dispenser, you won't be able to either.

I don't actually feel like this is a problem but I am more than happy to be wrong. To me the container that you provide as input to the ChemMaster 4000 and the output of the chemical dispenser is a beaker. A bottle is an output of the ChemMaster 4000 and isn't in the same class as the bottle. Because if you really want to put the bottle into either machine you can dump it into the 'scientific object' of the beaker which works for the other kind of chemist devices like the dropper and syringes. I wouldn't expect a dropper, syringes, or a cup from the bar to be able to be used in the chemmaster without transfer to a beaker that 'fits' the machine's outputs or inputs.

Adding to this, syringes don't seem to be able to draw currently.

can draw and inject, but not fitting in chem machines seems intended


mouse without borders wired connection？

Description of the new feature / enhancement
I have noticed that there is a picture in the settings interface of mouse without borders tool :

seems like this tool supports wired connection, but I cant find any option for that. Is there anyway I can use physical connection instead of Internet connection to improve the quality?
Scenario when this would be used?
Just want to make sure the inter-device operation is smooth.
Supporting information
No response

I'd assume that wire image is more of "wired" vs "wireless" network not direct device to device communication.

The standalone MWB used to support wired connection between two devices directly connected via ethernet cable. I'm still trying to confirm is the new one under Power Toys also does this as I just moved from the standalone version to this one.
As for the ethernet connection between both devices, it will depend if they support it. In the old times an ethernet crossover cable was needed to allow point to point connection between two computers, nowadays most network controllers support point to point connection with standard ethernet cables.


[$500] Android - white border in checkbox while Offline

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

Action Performed:

Go offline
Go to settings > workspace > members > invite
Invite member which is not in list
Go to members
Select that user

Expected Result:
It should not show white border
Actual Result:
white border appears while Offline
Workaround:
Unknown
Platforms:

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

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://github.com/Expensify/App/assets/93399543/b17ffaf2-72c1-4193-a701-12936dd9af32
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695988538728789
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010099c7efd17d5ffa
Upwork Job ID:<PHONE_NUMBER>635340800
Last Price Increase: 2023-09-30

Going to triage this tomorrow.

IMO, it's not worth doing anything to change this. I don't think this is really a bug. It's a minor design inconsistency that isn't really noticeable at all. Closing.


Fix lightbox spinner

Closes #6499
Preface
This is the first time I have ever worked with Expo, React Native, and even used native-esque debug in Xcode. So I apologize if anything I saw here is grossly misconstrued. I am hoping to continue learning the next few months and I thought this might be a good issue to tackle.
Problem statement
I observed a few days ago that we had a redundant logic condition in the image item components are what I believe is loading logic:

Here is what I think this code intends to do reference to docs for this OSS
The first function "prepare" calculates some value (in this case a boolean) based on whether in the current state of the app the view has finished animating and the image has not loaded yet. This defines the "current value" on first run, called show. Per the API docs the previous value defaults to null and is thus falsy.
So for the first stage we can assume the image is NOT loaded yet and the animation is likely not done yet
show = false | prevshow = null (false)
Since both conditions are false showloader remains false
For second stage I believe animation of the Lightbox "settles" however at this point the image may still not be loaded for a variety of reasons (slow connection, etc). Therefore the prepare function will now show true.
show = true | prevshow = false (previous show value)
Per the current logic, this sets the loader to the first if condition, which keeps the setter value in useState to false.
For the third stage the image has loaded. Our prepare function now flips to false since hasLoaded is now true
show = false | prevshow = true
Neither condition in the current logic evaluates to true for the current logic. So the loader remains false.
Proposed solution
I think the intent of this code is follow:
For the second stage I believe what actually should happen is when the animation has "settled" rendering, but we have not finished loading we want to set the boolean from prepare to be true. At the same time we should ensure that the prevShow is false because we really only want to load if the image has never loaded before.
therefore we can consolidate the logic condition to simply

Stage 3 then occurs after and since show becomes false the setter sets the loader back to false
Testing wise
This is where I honestly struggled a bit. I did manage to get expo running on a Xcode simlulator. I scrolled my feed and clicked many images to expand to full screen. I was able to see the image appear and dismiss (by closing the box in the upper right corner)
I hope this was enough detail. If I have misunderstood the intent of this code please let me know. Thank you so much for everything the Bsky team has done so far!

thanks for the PR, i'll have a look. i think we want a guarnatee that runOnJS only happens when something actually changes so unconditionally calling is not right.

ok i tweaked the logic, this should be good. thanks for catching

ok i tweaked the logic, this should be good. thanks for catching

Thanks for taking the time to explain. Actually your solution was one of previous one's I had but I think I was probably just over optimizing haha. Take care.


Add display-mode article

Add an article about display-mode which is already shipping

I signed it!

Why close it?

Ah

@PaulKinlan It was a mistake

Error: Invalid Author 'kenchris' found in  en/updates/posts/2015/10/display-mode.markdown
@PaulKinlan I guess you need to fix this, somehow

@kenchris you need to pull in the PR I have on your repo so that it gets picked up by this repo

OK let me try that


RangeError: Maximum call stack size exceeded

@Tyriar from error telemetry (https://ticino-errors.azurewebsites.net/Errors?bucketIdHash=de687ba5-8cd1-98c8-612d-2bea31e9853e):
RangeError: Maximum call stack size exceeded
at Array.findIndex (native)
at buildProcessTree (\node_modules.asar\windows-process-tree\lib\index.js:12:31)
at rootPid.children.childIndexes.map.c (\node_modules.asar\windows-process-tree\lib\index.js:22:37)
at Array.map (native)
at buildProcessTree (\node_modules.asar\windows-process-tree\lib\index.js:22:28)
at rootPid.children.childIndexes.map.c (\node_modules.asar\windows-process-tree\lib\index.js:22:37)
at Array.map (native)
at buildProcessTree (\node_modules.asar\windows-process-tree\lib\index.js:22:28)
at rootPid.children.childIndexes.map.c (\node_modules.asar\windows-process-tree\lib\index.js:22:37)
at Array.map (native)

This is odd that there's a spike in occurrences since March 8th

This error is weird as it's saying that there's a loop in a process/child process relationship. Regardless I think I'll add a max depth to search for just in case.


ESLint couldn't find the plugin "@trilon/eslint-plugin".

ESLint: 8.56.0

ESLint couldn't find the plugin "@trilon/eslint-plugin".

(The package "@trilon/eslint-plugin" was not found when loaded as a Node module from the directory "/Users/igormakowski/Documents/repositories/rigtch/rigtch-music-api".)

It's likely that the plugin isn't installed correctly. Try reinstalling by running the following:

    npm install @trilon/eslint-plugin@latest --save-dev

The plugin "@trilon/eslint-plugin" was referenced from the config file in ".eslintrc.cjs".

If you still can't figure out the problem, please stop by https://eslint.org/chat/help to chat with the team.

.eslintrc.cjs
{
  plugins: [
    '@typescript-eslint',
    '@trilon/eslint-plugin',
    'import',
    'nestjs',
    'eslint-plugin-import-helpers',
    'prettier',
  ],
  extends: [
    'plugin:@typescript-eslint/strict-type-checked',
    'plugin:@typescript-eslint/stylistic-type-checked',
    'plugin:@trilon/recommended',
    'plugin:sonarjs/recommended',
    'plugin:unicorn/recommended',
    'plugin:nestjs/recommended',
    'plugin:eslint-comments/recommended',
    'prettier',
  ],
}

package.json
{
  "devDependencies": {
    "@trilon/eslint-plugin": "^0.2.0",
  }
}

Thanks! You caught an error in our CI ❤

No problem! I find your eslint plugin very useful and would really like to use it.

I just published v0.2.1 it's still early days for this plugin so if you encounter any more issues, please let us know and we'll fix it


Welcome - Here's the plan.

Introduction
Readify has an fully fledged RSS aggregator of all of it's employees. When a new employee joins part of their first day on the job is to send in a pull-request to our  aggregator (firehose) for inclusion into our blogroll if they so desire.  The PR looks as follows.
    public class GeoffreyHuntley : IAmAReadifarian
    {
        public string FirstName => "Geoffrey";
        public string LastName => "Huntley";
        public string Title => "Lead Consultant";
        public string StateOrRegion => "NSW";
        public string EmailAddress =><EMAIL_ADDRESS>        public string TwitterHandle => "geoffreyhuntley";

        public Uri WebSite => new Uri("https://ghuntley.com/");
        public IEnumerable<Uri> FeedUris { get { yield return new Uri("https://ghuntley.com/atom.xml"); } }
    }

The Proposal

Readify releases the source code under appropriate license (MIT/MS-PL/Whatever that's not GPL)
We work together to create github.com/xamarin/firehose which is hosted at https://firehose.xamarin.com/
It's hosted on Microsoft Azure by Xamarin and merged pull-requests results automatic deployments into production.
When it's unveiled Readify asks for a quick plug in the blogpost  (see http://www.arnnet.com.au/article/583877/interview-readify-managing-director-graeme-strange/ for more info about Readify) - James if this is going to be contentious then please speak with me privately. It's a ask/acknowledgement type thing.

We implement three interfaces that use a contract similar to above.

IWorkAtXamarin
IAmAnXamarinMVP
IAmAComunityMember

We expose a mixture of end-points:

An endpoint that is everyone in the firehose.
An endpoint that is just Xamarin employees
An endpoint that is just Xamarin MVPs
An endpoint that is just Community.

Screenshots

Website
https://firehose.labs.readify.net/

@jfversluis invitation has been sent!
Here's the next steps:

Complete ordering of SSL certificate  #13
Host the website on Azure w/PaaS website #4
Configure Azure PaaS website to automatically deploy from this source code #5

@ghuntley  Do you just need me to do stuff. we should close out all old stuff and assign stuff to me. I can try to finish all this week.

@jamesmontemagno do the stuff in https://github.com/ghuntley/planetxamarin/issues/1#issuecomment-272755848 and then we can "launch".


Web UI or Telegram Bot

Hi.
How about web ui or telegram bot for self-host.
I think it would be great to download music anywhere.
I tried, but I can't make this with my skills. 😢
Thanks.

Bot is already hosted privately through https://jpopsuki.eu/requests.php


Support quantizing to K-quant formats

The new K-quant formats can be inferred with, courtesy of #326, but llm cannot quantize to them yet. I suspect this will require additional work because they use different quantization levels for different tensors.

Not too bad, just tedious: https://github.com/ggerganov/llama.cpp/blob/32c5411/llama.cpp/#L2512-L2559
Also not sure what this would look like for non-LLaMA models.


Update echo middleware to get Hub instance from context

Modified to support getting sentry.Hub instance from parent Context to include extra fields defined in the parent context.
Background -- we use sentry with echo web framework in a AWS lambda environment behind AWS API Gateway. We have top-level sentry instance defined with extra fields.
The purpose of this change is to use the existing hub on context by default.

@j--wong would you have a short example of how you're using this?
I would like to understand what use case this is solving, because if we change the echo middleware we may want to consider the same change for other frameworks.

Hi @rhcarvalho, thanks for the reply.
We are building rest apis in AWS using AWS API Gateway > Lambda > Echo.
We configure logging, tracing (honeycomb) and error handling/monitoring (sentry) at the Lambda layer with extra information.
The key use case for us is we want that same set of extra information to flow to sentry, honeycomb and logs  without having to reconfigure them with every error/log/trace.
We configure a hub instance at Lambda layer and attach it to the context. We'd like Echo layer (ie., sentry-echo middleware) to use the same hub instance.

Without this PR, those extra information is not preserved in sentry-echo middleware as it creates a new Hub instance internally instead of getting existing one from the parent context -- which is what I am trying to address with this PR.
We configure honeycomb the same way.  Honeycomb version of echo middleware looks for existing trace from parent context first, and create a new one only if there is no existing trace in parent context.  As seen here:  https://github.com/honeycombio/beeline-go/blob/d24142d664/wrappers/common/common.go#L45-L57
I believe this should be applicable to other web frameworks as well.  I think it's sensible to use existing hub from context as default.
Maybe you could introduce this behind an option like UseHubFromContext bool or something along those lines if you want to provide a toggle for this behaviour.
Sorry it was a bit long, hope this all makes sense 😄

We configure [...] error handling/monitoring (sentry) at the Lambda layer with extra information.

@j--wong thanks for the description. Do you have a short piece of code to demonstrate how you're putting things together? Particularly interested in the part related to sentry-go.
I'd like to see precisely how you're inserting a hub in the request context before the sentry middleware.
In this example code there is a demonstration of how to add a tag to all Sentry events:
https://github.com/getsentry/sentry-go/blob/af3076c5259eb407873c99da142af3ee29051105/example/echo/main.go#L35-L46
With the same pattern you can configure the request's hub/scope arbitrarily.

I am imagining you have a middleware that is running before Sentry's middleware, that's why you are trying to add a hub to the Echo context manually?!

I am imagining you have a middleware that is running before Sentry's middleware, that's why you are trying to add a hub to the Echo context manually?!

Hi @rhcarvalho, in my case, I already have a Hub instance on context with all extra fields set.  I just wish the sentry-echo middleware could use that instance.
Yes, I got a middleware running before sentry-echo middleware (it's lambda middlware, not echo middleware).  The code looks like this:
return func(next lambda.Handler) lambda.Handler {
	...
	...
	return func(ctx context.Context, payload []byte) ([]byte, error) {
		lc, _ := lambdacontext.FromContext(ctx)

		hub := sentry.CurrentHub().Clone()
		hub.Scope().SetExtras(cfg.Fields) // extra information
		hub.Scope().SetExtra("aws_request_id", lc.AwsRequestID)
		hub.Scope().SetExtra("...",...) // more lines omitted
		ctx = sentry.SetHubOnContext(ctx, hub)
		...
		...
		// invoke next lambda middleware; from this point forward, we have hub instance on context
		return next.Invoke(ctx, payload)
	}
}

The reason I configured sentry this way in Lambda layer is so that Sentry can catch lambda errors too, not just errors from Echo.
The other approach I could take is, like in the example you shared, duplicate above code in another echo middleware func. So I add whatever fields I want to add after sentry middleware.
I guess the key use-case question I'd like to ask is -- do you believe this is a valid/reasonable scenario for sentry/echo users to have Hub instance configured higher up in the chain before echo middleware run?
To me, that's exactly my use case 😄. Maybe not many people are doing Lambda > Echo + Sentry combo for their apis. So this may not be common use case?

This is a reasonable use case. I'll follow up with adding similar behavior to the other integrations. Thanks, @j--wong!
This will be part of the upcoming next release.

Great stuff @rhcarvalho!  Thanks for that 👍


Update supported versions

Drop Ruby 2.7
This is not supported anymore as of March 31st 2023 and I have no
desire to support it indefinitely. Kill it with fire.
Try Windows on CI
Local development makes use of some commands like diff that might
not be portable to windows but the app itself doesn't really use
much beyond basic Ruby. It might be cross-platform.

File path handling probably doesn't work on Windows though.


Conflict when try to require google-translate

Hello guys,
I am happy I can write to you and show you what is my problem.
This tutorial is perfectly working for me: https://dev.to/robertchen/how-to-use-google-translate-api-27l9M
But in my other project when I try to import: import { googleTranslate } from "./utils/googleTranslate"; in App.js it shows me the image i attached.
I think the problem is because I already have some other packages installed, and this is my package.json:
{   "name": "client",   "version": "0.1.0",   "private": true,   "dependencies": {     "@date-io/date-fns": "^1.3.11",     "@material-ui/core": "^4.2.0",     "@material-ui/icons": "^4.2.1",     "@material-ui/lab": "^4.0.0-alpha.40",     "@material-ui/pickers": "^3.2.7",     "@material/auto-init": "^3.0.0",     "@material/card": "^3.0.0",     "axios": "^0.18.0",     "bootstrap": "^4.1.3",     "core-js": "^3.1.3",     "date-fns": "^2.0.0-beta.5",     "google-translate": "^3.0.0",     "instascan": "^1.0.0",     "jquery": "^3.3.1",     "material-components-web": "^3.0.0",     "materialize-css": "^1.0.0-rc.2",     "qcode-decoder": "^0.1.0",     "qr-scanner": "^1.1.1",     "qrcode-reader": "^1.0.4",     "raf": "^3.4.1",     "react": "^16.7.0-alpha.2",     "react-cookies": "^0.1.1",     "react-country-flag": "^1.0.2",     "react-dom": "^16.7.0-alpha.2",     "react-fontawesome": "^1.6.1",     "react-localize-redux": "^3.5.2",     "react-modal": "^3.6.1",     "react-qr-reader": "^2.1.2",     "react-rating": "^2.0.4",     "react-rectangle-popup-menu": "0.0.1",     "react-redux": "^5.1.1",     "react-redux-multilang": "0.0.5",     "react-responsive-carousel": "^3.1.43",     "react-responsive-modal": "^3.5.1",     "react-router": "^4.3.1",     "react-router-dom": "^4.3.1",     "react-router-redux": "^4.0.8",     "react-scripts": "2.1.1",     "react-star-ratings": "^2.3.0",     "react-swipeable-views": "^0.13.3",     "react-transition-group": "^2.5.0",     "reactstrap": "^6.5.0",     "redux": "^4.0.1",     "redux-thunk": "^2.3.0",     "require": "^2.4.20",     "rodal": "^1.6.3"   },   "scripts": {     "start": "react-scripts start",     "build": "react-scripts build",     "test": "react-scripts test",     "eject": "react-scripts eject"   },   "eslintConfig": {     "extends": "react-app"   },   "browserslist": [     ">0.2%",     "not dead",     "not ie <= 11",     "not op_mini all"   ] }
If you need more details about it I can provide it. Thanks!

Actually, when I changed the version of react, it fixed the things:
"react": "^16.12.0",
"react-dom": "^16.12.0",
"react-scripts": "3.3.1",


Mutual Auth: Re-Authentication based on CertificateRotation Events

Since https://github.com/cilium/cilium/pull/24300, the AuthHandler interface provides the possibility to get notified about CertificateRotation events.
The AuthManager should react on these events by re-authenticating the relation between the two corresponding workloads.

Implemented with https://github.com/cilium/cilium/issues/25475


DownloadGitHubRelease cannot download zip or tarball source code assets

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Question / Bug
Enter Task Name: DownloadGitHubRelease
list here (V# not needed):
https://github.com/microsoft/azure-pipelines-tasks/tree/master/Tasks/DownloadGitHubReleaseV0
Environment

Server - Azure Pipelines

Agent - Hosted:

If using Hosted agent, provide agent queue name: Current agent version: '2.158.0'

Issue Description
The current release being created for this particular GitHub project is not creating any particular asset except a zip of the source code for a Django Python project. there are no "assets" created in terms of GitHub and no assets are listed under the GitHub API which this task is using.
when leveraging the API, there is a "zipball_url" for the latest release, but this task is unable to download the relevant zip ball asset.
https://api.github.com/repos/netbox-community/netbox/releases/20617298
Task logs
2019-10-15T05:12:05.5938637Z ==============================================================================
2019-10-15T05:12:05.5938883Z Task         : Download GitHub Release
2019-10-15T05:12:05.5938971Z Description  : Downloads a GitHub Release from a repository
2019-10-15T05:12:05.5939073Z Version      : 0.156.0
2019-10-15T05:12:05.5939159Z Author       : Microsoft Corporation
2019-10-15T05:12:05.5939252Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/download-github-release
2019-10-15T05:12:05.5939374Z ==============================================================================
2019-10-15T05:12:07.4650664Z Downloading assets of release 'v2.6.6' from: https://api.github.com/repos/netbox-community/netbox/releases/20617298/assets
2019-10-15T05:12:07.7037722Z Total Files: 0, Processed: 0, Skipped: 0, Failed: 0, Download time: 1.021 secs, Download size: 0Bytes
Troubleshooting
Validated the zip source code for each release is NOT under the "assets" of the GitHub API, but rather a part of a separate "zipball_url" under the latest release.
Error logs
[Insert error from the logs here for a quick overview]

@houstonjb This is by design. Download GitHub Release task only downloads assets that are uploaded by user. Source Code (zip) and Source Code (tar.gz) are not published by user as part of GitHub release assets. They are automatically attached to release by GitHub.
@ashokirla Please see if we can consider this feedback in next iteration of task.

Download Github releases task is designed to download assets published by user. If you would like to access repository data, you can use Git or use multi-repository resource and checkout.


BlastDB not available

Hi,
I failed to download the Blast database from http://bench.cs.vt.edu/ftp/data/metacomp (Error 404: The requested URL was not found on this server).
Best

Did you figure out the problem? I got the same error.
Best,
Irmarie

I meet this problem too

Same here!

Still BlastDB link is not working and I am not able to download the database. Can anyone had earlier downloaded the database and help me?

Please use this url as an alternative: https://zenodo.org/records/10471551


Advisor to correct command mistypes

Despite the help command, when a player do a mistype, it would be good to have an advisor who suggest the proper spelling for it. The system could suggest the closest to the last command typed by the player.

This can be done using the Levenshtein distance.
https://en.wikipedia.org/wiki/Levenshtein_distance


ImportError: libboost_python-py27.so.1.55.0

When using BLESuite, latest version from GIT (August 1, 2016), in Kali Linux 2017.3, the following error is generated when trying to run any of the default scripts from the "bleSuite" dir:
# python bleSmartScan.py -h
Traceback (most recent call last):
  File "bleSmartScan.py", line 4, in <module>
    from bleConnectionManager import BLEConnectionManager
  File "/Labs/BLE/BLESuite/BLESuite-master/bleSuite/bleConnectionManager.py", line 1, in <module>
    from gattlib import GATTRequester, GATTResponse
  File "build/bdist.linux-x86_64/egg/gattlib.py", line 7, in <module>
  File "build/bdist.linux-x86_64/egg/gattlib.py", line 6, in __bootstrap__
ImportError: libboost_python-py27.so.1.55.0: cannot open shared object file: No such file or directory

After creating a symbolic link from libboost 1.55 to the currently available (latest) libboost version, v1.62, the error message disappears:
# pwd
/usr/lib/x86_64-linux-gnu
# ln -s libboost_python-py27.so.1.62.0 libboost_python-py27.so.1.55.0

Then, a similar issue affects the related "thread" library:
# ln -s libboost_thread.so.1.62.0 libboost_thread.so.1.55.0

However, when a tool that makes use of BLESuite and that requires establishing a BLE connection is run, such as "https://github.com/nccgroup/BLESuite-CLI", a mismatch library error is generated:
# python ./bleSuite-runner.py smartScan --addr 0E:0A:B0:00:23:15
BTLE Smart Scan beginning
Traceback (most recent call last):
  File "./bleSuite-runner.py", line 12, in <module>
    main()
  File "/Labs/BLE/BLESuite/BLESuite-CLI-master/bleSuiteCLI/bleSuiteCLI.py", line 356, in main
    processArgs(args)
  File "/Labs/BLE/BLESuite/BLESuite-CLI-master/bleSuiteCLI/bleSuiteCLI.py", line 191, in processArgs
    args.addrType[0], args.security[0])
  File "/Labs/BLE/BLESuite/BLESuite-CLI-master/bleSuiteCLI/cmdLineToolWrappers.py", line 520, in bleRunSmartScan
    connectionManager = bleConnectionManager.BLEConnectionManager(address, adapter, addressType, securityLevel)
  File "/usr/local/lib/python2.7/dist-packages/bleSuite/bleConnectionManager.py", line 41, in __init__
    self.createRequester()
  File "/usr/local/lib/python2.7/dist-packages/bleSuite/bleConnectionManager.py", line 55, in createRequester
    self.requester = GATTRequester(self.address, False)
Boost.Python.ArgumentError: Python argument types in
    GATTRequester.__init__(GATTRequester, str, bool)
did not match C++ signature:
    __init__(_object*, std::string)
    __init__(_object*, std::string, bool)
    __init__(_object*, std::string, bool, std::string)

Is there any specific reason why BLESuite is linked to libboost v1.55, or could it make use of newer libboost versions?

The libboost references are associated to PyGattlib (from within the BLESuite "PyGattlib" directory):
Binary file ./build/lib.linux-x86_64-2.7/gattlib.so matches
./build/lib.linux-x86_64-2.7/gattlib.so
	       libboost-python1.55-dev,
	       libboost-thread1.55-dev,
./debian/control


ACCUMULO-4758 throw correct exception when meta block absent

Can one of the admins verify this patch?

My approval was merely a thank you for the information so thanks!


Ability to create a fake file from any MIME type

Its good to fake an image with UploadedFile::fake()->image('image.jpg'); but what about other file types?
In my case I want to test when you upload a file with audio/mpeg MIME type. I expected that the create could take a MIME type parameter.
Best

For suggestions and feature requests please use the https://github.com/laravel/internals repo.


feat: packages tracker ux logic

Fixes #170
How to test this version of the CLI?
To install it:
go install github.com/listendev/lstn/cmd/lstn@better-request-tracking

What happens?
A single request is made for every dependency instead of bulking them
The reason behind this is that by making a single request we can

avoid hitting the API timeout of 6 seconds (there is no dependency request that will take more than 200/300 ms round trip)
provide fine-grained progress tracking (this leads to better UX, the user can see things going on instead of waiting minutes for something to happen)
show specific errors when a dependency had an error (which is not one of our pre-defined Problems)

A progress tracker is shown
Now that one of the benefits of doing a single request is to be able to do fine-grained progress tracking, that's what happens!
Screenshots
here are some screenshots of how it looks like
lstn scan
A request is made for every version mentioned in the package.json

lstn to react
This is the most basic use case, only the available versions are requested

lstn to react *`
A request is made for every possible version on npm, the progress is shown like this. You will notice that the semver constraint is used to specify what we are processing

lstn in
lstn in is not included in this. I did the code to actually include it but in is still very unstable for me at the moment until we do #132 . Once this gets merged I'll open an issue to do it.

[x] I have read the contributing guidelines
[x] I have followed the coding guidelines
[x] I have written unit tests
[x] I have made sure that the pull request is of reasonable size and can be easily reviewed

TODO after merge:

[ ] once this gets merged, open an issue to support lstn in

LGTM


Incorrect unit in Home Assistant Energy dashboard for GAS reading

Describe the bug
In home assistant energy dashboard under gas consumption I can see quite good value but incorrect unit. I have used Yearly usage as suggested in the configuration.
To Reproduce
add GAS Yearly sensor to Energy Dashboard in Home assistant configuration
Screenshots

Version
What version of the integration and what version of Home Assistant are you running?
Latest version and latest version of Home Assistant
Debug log
n/a
Additional context
n/a

Are you sure this is not just because it is playing catch-up, with the usage since the start of the calendar year?
Let it run for a day or so and see if it stabilises to a correct value.

It may be but when you look on the daily or yearly usage in Lovelace it reports it correctly (compared with Bright iOS app).

@ColinRobbins you were right that was some caused by "first day" reading. all looks good now.

@ColinRobbins you were right that was some caused by "first day" reading. all looks good now.


Improve sort to handle Latin numbering

Some "Various Vol." music media using Latin numbering which does not get sorted as expected. For example

You would expect the middle entry to appear last

Turns out to be even more difficult as the sort order after the initial load is handled by DataTables. A JS sorting plug for DataTables needs to be written for this to function as expected.


Allow zip file to contain multiple swift source code files

Allow Swift runtime to accept zip files that contain multiple source code files.

Bonus points if you one of those swift files can be Pacakge.swift and so we can pull in dependencies!

@akrabat the cold start times may be killer with full Swift Package Manager support.  However, worth investigating!
The pre-compilation of a Swift Package seems like a better approach.

Good point.


M2 Still not working

Even after the update for M1/M2 iPads. The exploit still kernel panics. IOS 16.3.1

same as you！
iPad pro 2022，still not work

依旧不行

依旧不行

Same iPad Pro 2018 16.4.1

Same.  M2 iPad Pro, iPadOS 16.5.

Same. M2 12.9" iPad Pro, iPadOS 16.5.
Just tried yet.Not work.


Add new logos

Changelog
This PR adds new icons as well as versions with white fonts of our existing logo. It also deletes the outdated vertical-logo-small-font.svg logo.
We should only merge this once we updated the README.mds in all public repositories.

lgtm


Add support for building Android engine for M1 mac

M1 macs currently cannot build the android engine. We see:
Generating GN files in: out/android_debug_unopt
ERROR at //build/config/android/config.gni:55:5: Assertion failed.
    assert(false, "Need Android toolchain support for your build CPU arch.")
    ^-----
Need Android toolchain support for your build CPU arch.
See //build/config/sysroot.gni:27:3: whence it was imported.
  import("//build/config/android/config.gni")
  ^-----------------------------------------
See //flutter/BUILD.gn:6:1: whence it was imported.
import("//flutter/common/config.gni")
^-----------------------------------
See //BUILD.gn:8:5: which caused the file to be included.
    "//flutter",
    ^----------

We should support this as M1 macs become more common across engine developers.

We need to update the NDK to 24+. Instead of ndk-bundle, we can specify the version in tools/android_sdk/packages.txt as ndk;24.0.8215888


Updates are not installing

Hello,
The program downloads the update, but it seems it cannot install it, as when the program is restarted the the programs starts to download the update again.
Best Regards,
Adriyan

Hi Adriyan,
Which OS are you using?
I know it's inconvenient but a fresh install might resolve the issue.

I am using Windows 11 but on Windows 10 the problem was also present. In fact, I am since recently with a Windows 11 so I installed the program also recently. Currently, I am with the newest version and I just installed a fresh copy per your suggestions. Once there is an update I will comment here whether the problem is present or it is gone.
Best Regards,
Adriyan

Hi Adriyan,
I don't have a Windows system to test the update feature so your input is greatly appreciated!  Please let me know if you have any issues during the next update.
Thank you,

Hello,
just now the app tried to updated to 4.4.3, but again unsuccessfully. The app shows a message: "updating Numara", but after that nothing happens.
I am currently with 4.4.2.
Best Regards,
Adriyan

Hi Adriyan,
Thanks for reporting again. I will need to dig a bit deeper into this one.

Hi Adriyan,
I have changed the installer type in v4.5.0 released yesterday.  Please try installing this version and let me know if you still have issues when the next update is released next week.  Thank you!

ok

Hi @1andro1,
Thanks for your patience.  I figured out the problem.  Since the Windows build is not signed with a certificate, it was throwing an error during auto update. It took couple iterations to test but latest version should now fix the issue (Tested on Window 11). You will need to update manually one last time.  Sorry for the inconvenience and please report if you have any other issues. Thank you!

Hi Timur,
now it is working! Many Thanks!


Add German support for typewrite()

From a user's email:
I use PyAutoGUI-0.9.33-py3.4.egg-info in one of my programs. When I use the function pyautogui.typewrite special characters like ö,ä,ü,#,@ ß, do not work.
I found an workaround for me on the following side:
http://stackoverflow.com/questions/13289777/how-can-i-send-keyboard-commands-hold-release-simultanous-with-a-python-script
Maybe you can use this code for windows, so special characters are also supported if it not already work now.
Thank you for your time and thanks for PyAutoGUI :)
Following the code that works for me:
import ctypes
import time
SendInput = ctypes.windll.user32.SendInput
PUL = ctypes.POINTER(ctypes.c_ulong)
class KeyBdInput(ctypes.Structure):
fields = [("wVk", ctypes.c_ushort),
("wScan", ctypes.c_ushort),
("dwFlags", ctypes.c_ulong),
("time", ctypes.c_ulong),
("dwExtraInfo", PUL)]
class HardwareInput(ctypes.Structure):
fields = [("uMsg", ctypes.c_ulong),
("wParamL", ctypes.c_short),
("wParamH", ctypes.c_ushort)]
class MouseInput(ctypes.Structure):
fields = [("dx", ctypes.c_long),
("dy", ctypes.c_long),
("mouseData", ctypes.c_ulong),
("dwFlags", ctypes.c_ulong),
("time",ctypes.c_ulong),
("dwExtraInfo", PUL)]
class Input_I(ctypes.Union):
fields = [("ki", KeyBdInput),
("mi", MouseInput),
("hi", HardwareInput)]
class Input(ctypes.Structure):
fields = [("type", ctypes.c_ulong),
("ii", Input_I)]
def writetext(text_unicode):
def PressKey_local(KeyUnicode):
extra = ctypes.c_ulong(0)
ii_ = Input_I()
ii_.ki = KeyBdInput( 0, KeyUnicode, KEYEVENTF_UNICODE, 0, ctypes.pointer(extra) )
x = Input( ctypes.c_ulong(1), ii_ )
ctypes.windll.user32.SendInput(1, ctypes.pointer(x), ctypes.sizeof(x))
def ReleaseKey_local(KeyUnicode):
extra = ctypes.c_ulong(0)
ii_ = Input_I()
ii_.ki = KeyBdInput( 0, KeyUnicode, KEYEVENTF_UNICODE|KEYEVENTF_KEYUP, 0, ctypes.pointer(extra) )
x = Input( ctypes.c_ulong(1), ii_ )
ctypes.windll.user32.SendInput(1, ctypes.pointer(x), ctypes.sizeof(x))
for c in text_unicode:
PressKey_local(ord(c))
ReleaseKey_local(ord(c))
writetext("Ölgemälde")

When I run this code, I get NameError: 'KEYEVENTF_UNICODE' is not defined.
When I specify these values like this:
KEYEVENTF_UNICODE = 0x0004
KEYEVENTF_KEYUP = 0x0002

I get the following error:
Traceback (most recent call last):
  File "C:\Users\Tim\Desktop\type_cyrillic.py", line 60, in <module>
    writetext('Я олень')
  File "C:\Users\Tim\Desktop\type_cyrillic.py", line 42, in writetext
    PressKey_local(ord(c))
  File "C:\Users\Tim\Desktop\type_cyrillic.py", line 48, in PressKey_local
    ii_.ki = KeyBdInput( 0, KeyUnicode, KEYEVENTF_UNICODE, 0, ctypes.pointer(extra) )
TypeError: too many initializers

Do you have any ideas?

Folding this into issue https://github.com/asweigart/pyautogui/issues/137


Why I can't run OrchardVNext.Demo.Home controller?

Hello I'm just starting with ASP.NET vNext so maybe I'm having more questions than is usual.
I'm at the commit 732852ed16b18ffc932949662dc0250cc4d38480 (last one at entityframework branch).
I have created a wwwRoot folder within OrchardVNext.Web in order to run the app because if it doesn't exists I get an error. To create manually this folder is what is expected?
If I debug the solution as it is I get an "Unable to locate project.json" message within an Error page.
However I'm able to debug the app replacing IIS Express per web in Debug Target properties of OrchardVNext.Web project.
Problem is the app only runs when my url request is http://local.orchardvnext.test1.com or  http://local.orchardvnext.test2.com but nothing happens if I request http://localhost:61562
The only way I've managed to make the solution work in response to a http://localhost request has been adding localhost without port to web command in project.json
--server.urls http://localhost;http://local.orchardvnext.test1.com;http://local.orchardvnext.test2.com
And removing the port from RequestUrlPrefix within App_Data\Sites\Default\settings.txt.
The point is when I call http://localhost/OrchardVNext.Demo/Home/Index middleware is run but Controller is never called.
Is it the normal behavior? I haven't managed to find why the controller doesn't work because  everything looks fine. Any help?
Regarding what I have had to do for making work the solution, is it normal? I imagine the change in the port is because when I use web command instead of IIS Express Visual Studio it is ignoring the port for debugging . But are you able to debug it with IIS Express?

try this: http://local.orchardvnext.test1.com/Home/Index
it must be work.

Yeah, so far we have only 1 route. and that is /Home/Index. So as @nicholaspei mentioned, try navigating to http://local.orchardvnext.test1.com/Home/Index

When I have read your answer I have thought: "I checked this it didn't work"
The point is it works!
I should have tested it at some point when I had other problems
Thank you!


[$500] Public room can't be seen on LHN after opening it from a link

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.3.93-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:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal team
Slack conversation:
Action Performed:

Navigate to https://staging.new.expensify.com/
Log out
Navigate to this link - https://staging.new.expensify.com/r/3504895439653267
Log in with any accounts

Expected Result:
The public room should be displayed
Actual Result:
The public room can't be seen on LHN
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

Window: Chrome

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010cd18e947fb2a8d5
Upwork Job ID:<PHONE_NUMBER>952470016
Last Price Increase: 2023-10-30
Automatic offers:

dukenv0307 | Contributor | 27493079

Proposal
Please re-state the problem that we are trying to solve in this issue.
If the current chat is a public room, it should show in the left hand nav.
What is the root cause of that problem?
Public rooms were removed from the left hand nav in a recent PR:
https://github.com/erquhart/expensify-app/commit/5dc91a13a198bd8e16c5f761b2188be25b3aeb2b
What changes do you think we should make in order to solve the problem?

Revert to fix (tested locally, works).
What alternative solutions did you explore? (Optional)
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

Hmm I'm unable to repro on Staging v1.3.93-0 (Chrome) - LHN is there for me:

@jliexpensify left hand nav is there, but the public channel currently being viewed isn’t in the list.

Ah sorry, my mistake - thanks @erquhart!

@thesahindia I can't seem to find your comment, do you mind linking it again?

So it's actually expected that the participantAccountIDs are empty if you're not a member of the workspace that the public room is a part of.
This is because of privacy concerns. So no BE fixes are needed here.

We just need to update the condition here to be the following:
report.participantAccountIDs.length === 0 &&
 !isChatThread(report) &&
 !isUserCreatedPolicyRoom(report) &&
 !isPublicRoom(report) &&
 !isArchivedRoom(report) &&
 !isMoneyRequestReport(report) &&

@jasperhuangg that's what my existing proposal is, I just didn't check to see what else that PR had. When I say to revert, I only meant reverting the exclusion of the isPublicRoom() condition. I'll go ahead and make an updated proposal.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Public rooms opened via link aren't showing in the left hand nav.
What is the root cause of that problem?
Reports with a visibility property matching PUBLIC or PUBLIC_ANNOUNCE used to be included in the left hand nav via isPublicReport():
https://github.com/Expensify/App/blob/1700ecfa697411588de13b50dc67d9ff7fb70f00/src/libs/ReportUtils.js#L411-L413
This was removed with - based on the PR conversation from #27966 - the understanding that public rooms were expected to be included in isUserCreatedPolicyRoom(), but that function only includes reports with a POLICY_ROOM chat type. isPublicRoom() is still needed to match report visibility against PUBLIC and PUBLIC_ANNOUNCE types.
What changes do you think we should make in order to solve the problem?

Re-add the isPublicRoom(report) statement to this conditional: https://github.com/Expensify/App/blob/deb267f81a771114b9d304f6386d887498320e0f/src/libs/ReportUtils.js#L3277-L3283
What alternative solutions did you explore? (Optional)
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

Got it, since @dukenv0307 had a clearer proposal and was more in line with what I was thinking, let's assign them. cc @neil-marcellini

Cool, I agree that the proposal from @dukenv0307 looks good. Hired!

Hrrrm what happened with the updater here? It looks like this hit production 2 weeks ago right? If so, I should be paying + the checklist should be worked on, right?

If we are going off https://github.com/Expensify/App/pull/30814, here's the Payment Summary:

Contributor: @dukenv0307 $750 (bonus as it was merged in 3 working days)
C+: @thesahindia$750 (bonus as it was merged in 3 working days) - NEW.DOT PAYMENT

Upworks job
is this all correct? @thesahindia could you complete the checklist too?

Paid and job closed. Waiting on checklist to be completed!


test(ui): Add cypress tests for listening endpoints page

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #7021

PR #7024

PR #7054

PR #7072

PR #7092

PR #7111   👈

This comment was auto-generated by Graphite.


Pages: update _redirects limit to 1000

Existing documentation URL(s)

https://developers.cloudflare.com/pages/platform/redirects/

Section that requires update
Please specify which section requires the update.

NOTE: Please name or describe the section as not all page sections have header-links and section IDs may change over time.

What needs to change?
https://github.com/cloudflare/cloudflare-docs/blob/production/content/pages/platform/redirects.md?plain=1#L38
How should it change?
The document says:

A project is limited to 100 total redirects.

However, according to a message in the Discord channel and this tweet, it looks the limit is bumped up to 1000.
That's a great update and docs should mention that!
Additional information
Any other details or screenshots that may be relevant.

Will be resolved in https://github.com/cloudflare/cloudflare-docs/pull/4123


breakpoint does not stop in mac m1

OS:  Apple M1 MacOs Ventura 13.4.1
VSCODE:1.81
CodeLLDB version: 1.9.2
Compiler: llvm16
launch.json
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "type": "lldb",
            "request": "launch",
            "name": "Launch",
            "program": "${workspaceFolder}/main",
            "args": [],
            "cwd": "${workspaceFolder}",
            "stopOnEntry": true
        }
    ]
}

run degguer:

OS:
VSCode version:
CodeLLDB version:
Compiler:
Debuggee:

Verbose log

Why do you think this is a codelldb issue?

Also, please review this: https://github.com/vadimcn/codelldb/wiki/Breakpoints-are-not-getting-hit

Why do you think this is a codelldb issue?

I use debug_info show , outputted path same as the source file.


playground: custom Domain Name

Description

while loading farms the select gateway will be hidden

before deploy

Screencast from 18 يول, 2023 EEST 06:39:03 م.webm
-after deployment only deployment data dialog

Support custom domain name
Changes

change GatewayFQDNModel to allow longer names
add an alert on deployments that use ipv4 with the custom domain in the deployment data dialog
create a new component  for the domain Name
create a new manager to link between farm and gateway nodes
include the manager in select farm components.
update select gateway node component.
change GatewayNode interface
allow delete solution that should have gateways if there is no contract with its name
support DomainName component in some weblets

Related Issues

#670

Checklist

[ ] Tests included
[x] Build pass
[ ] Documentation
[x] Code format and docstrings
[x] Screenshots/Video attached (needed for UI changes)

Changing the farm after selecting a gateway causing the gateway input to reload (which is right) but the deploy button isn't disabled @0oM4R

Changing the farm after selecting a gateway causing the gateway input to reload (which is right) but the deploy button isn't disabled @0oM4R

should be fixed please try again @MohamedElmdary

when trying to deploy funkwhale instance with a custom domain and without public ip

when trying to deploy funkwhale instance with a custom domain and without public ip also the same with public ip

should be fixed

tested weblets:

weblet
gateway
custom without ipv4
custom with ipv4

Freeflow

Freeflow

Freeflow

Freeflow

Freeflow


Stateless Mode Not Compatible With ELEMENTOR

Elementor | V 2.1.2
WP-Stateless | V 2.1.5
When using the element builder and adding an image element:
The only Image size that works properly is the: custom image size
The rest of the image sizes appear to work but once you click away from the element it disappears.
It also breaks other random things like sides and carousels
WP-Stateless Settings |
Mode | Stateless
File URL Replacement | Enabled
Delete GCS File |  Enabled
Bucket | CDN.Domain.com
Domain | CDN.Domain.com
Organization | Disabled
Cache-Busting | Enabled
Server Environment |
Operating System: | Linux |
Software: | nginx/1.13.8 |
MySQL version: | 5.7.22 |
PHP Version: | 7.2.7 |
PHP Max Input Vars: | 1000 |
PHP Max Post Size: | 8M |
GD Installed: | Yes |
ZIP Installed: | Yes |
Write Permissions: | All right |
WordPress Environment |   |
Version: | 4.9.7
Max Upload Size: | 8 MB |
Memory limit: | 256M |
Permalink Structure: | /%postname%/

alimuzzaman,
Appreciate your help here just tried the plugin 2.1.8-beta2 and am still having the same issue.
It appears as tho when I initially add the picture it works fine.
But once I click off to another elementor still breaks.
This holds true with all image sizes besides the thumbnail and custom sizes.
Have added images below to demonstrate.
I know that this has something to do with images being http - https

@TheOneWizard1

I know that this has something to do with images being http - https

Yes, that might be the problem. https on custom domain isn't supported.

Yes, I think so. But I never tried.

The think its possible for you too?

@TheOneWizard1 please check the issue #291


AppImage Build process is failing

The appimage build process fails on a Linux Distro which is not Debian based (a.k.a it doesn't use apt as package manager).
I noticed that the build script is trying to do something with apt-get.
In this case it would be good if you specify in the docs that a debian based Linux distro is needed to build appimages.

Currently according to appimage-builder Docs (which is used under the hood) AppImages can only be built in Debian, Ubuntu and Arch based distros. So if you're on anything other than that you're out of luck although you can always use podman/docker containers
Also, there're some appimage-builder installation instructions in the flutter_distributor docs but I'd recommend you to follow the Official Docs of AppImage Builder to install proper dependencies required for it


Fix Options Descriptions

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2020.03.24
[x] I've searched the bugtracker for similar feature requests including closed ones

Description

What I really wanted to bring light to: The state of the embedded options descriptions.
What I would really appreciate first off is if each of these options descriptions specified the type of the option. E.g. keepvideo (bool): Keep the video file after post-processing. A developer should not have to make assumptions or infer what the type will be. It's not a thinking heavy task so I'd appreciate it. Secondly, you should also include the command line alternative in the description so a developer knows the corresponding command-line option.
This next part is a criticism of you.
Please stop locking conversations left and right with an iron fist, you're wasting your own time (and the time ofo others). You locked my old thread because of the non-important part of it and your comments were not helpful at all! (I know what the cachedir does but obviously that's the only way for me to fix the problem). You provided the CLI solution the 403 Forbidden error but not the embedded solution. It's not enough because you are in charge of the youtube-dl and the README.md gives instructions to use youtube-dl in an embedded fashion. If you don't to support the embedded option, you could've said you don't care about the embedded version or that it's not a priority, that you'd work on it later, or use subprocess instead. Don't just provide a comment of no value.

You can't specify the type for each option since most of them may accept different types and this is explained for each option in particular. For the rest PRs welcome I don't plan to spend time on this any soon.
Are you trolling or what? Again: you don't need the embedded solution for one-time option and there is no such option for that reason.


Revise mapped tuple type instantiation logic

This PR revises and simplifies our mapped tuple type instantiation logic. Previously, when a mapped type was applied to a variadic tuple type, we'd transform the operation into a sequence of operations on manufactured singleton tuple types. This would have surprising effects when observing property key types, as reported in #48856. For example:
type Keys<T> = { [K in keyof T]: K };
type Foo<T extends unknown[]> = Keys<[string, string, ...T, string]>;  // ["0", "0", ...Keys<T>, "0"]

With this PR, the property keys are more consistent:
type Keys<T> = { [K in keyof T]: K };
type Foo<T extends unknown[]> = Keys<[string, string, ...T, string]>;  // ["0", "1", ...Keys<T>, number]

The PR also fixes issues related to instantiation of tuples with required elements following a rest element, as reported in #56888, and it fixes the error reporting issue discussed here.
Finally, the PR reverts #53522. The fix in that PR is incorrect--it effectively leaks a type parameter. With this PR, the repro from #53458 now causes an excessive instantiation depth error, which should have been the behavior in the first place.
Fixes #48856.
Fixes #56888.

@typescript-bot test top100
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this faster

Looks good, left some comments that should be quick to address.

It might be worth rechecking if the recently merged in https://github.com/microsoft/TypeScript/pull/55774 doesn't throw any wrinkles into this.

I checked, and this PR still passes with main merged in (unless you were thinking of something other effect).

Unfortunately, https://github.com/microsoft/TypeScript/pull/55774 got reverted because it weirdly clashed with another PR that was merged recently ( https://github.com/microsoft/TypeScript/pull/57091 ). However, I didn't think that a merge of those would fail any existing test cases but rather that maybe some new test cases should be written in light of this PR. I just had a hunch of sorts that maybe there is something to be re-checked but I didn't have the energy myself at the time to think about the actual test cases.


Port OpenSpending Next site into community site

@danfowler @mintcanary do one of you want to give this a quick once over before we merge?
It adds the content of this site under the next directory.

Actually, sorry, but I'm going to merge myself now as I do want to use this shortly. I hope nothing broke :).

I had a quick look, and it looks good :+1:


Add GuzzleTracingMiddleware as global middleware on Laravel HTTP client by default

This auto-configures the GuzzleTracingMiddleware as a global middleware on the Laravel HTTP client. This feature was introduced on Laravel 10.14 (PR: https://github.com/laravel/framework/pull/47525).

Woops... I need to combine this with our current tracing setup to prevent duplicate spans, back to draft!


Postgres schema

Hey,
Hope you are doing well!
Could you please help me to find the best way to create postgres database/tables.
If it possible may you share a postgres schema?!
Thank you in advance,
Alex

The table names and column names of the two tables can be customized and then configured in the configuration file. See cmd/config/auth-postgresql.yml.
schema reference:
CREATE TABLE mqtt_user (
id serial PRIMARY KEY,
username varchar(32) NOT NULL UNIQUE,
password  varchar(32) NOT NULL,
allow smallint DEFAULT 1 NOT NULL,
created timestamp with time zone DEFAULT NOW(),
updated timestamp
);
allow options:  0 deny, 1 allow
CREATE TABLE mqtt_acl(
id serial PRIMARY KEY,
username varchar(32) NOT NULL,
topic varchar(256)  NOT NULL,
access smallint DEFAULT 3 NOT NULL,
created timestamp with time zone DEFAULT NOW(),
updated timestamp
);
CREATE INDEX mqtt_acl_username_idx ON mqtt_acl(username);
access options: 0 deny、1 publish (Write)、2 subscribe (Read)、3 pubsub (ReadWrite)


bscp for case when destination greater than source

Thanks!

To anyone involved in this issue, please see #16 "Future of bscp".


Additional NoFlo UI integration

This pull request contains NoFloUI registration examples, websocket message compatibility using json:"label" text in structs, network changes to handle multiple components, and dummy components to register for debugging in NoFlo UI.

Closing as outdated


Cannot create azure function with python using vscode from mac

Repro steps:1.2.Action: azureFunctions.createNewProjectError type: ErrorError Message: Failed to run ". .env/bin/activate
Steps of the output window

Running command: "brew tap azure/functions"...
Updating Homebrew...
==> Auto-updated Homebrew!
Updated 1 tap (homebrew/core).
==> Updated Formulae
bitrise
cfssl
dcos-cli
docker
docker-completion
dynare
futhark
gitlab-runner
gmic
jenkins
jenkins-lts
lolcat
meson
nginx
opam
openttd
pegtl
php-code-sniffer
prometheus
putty
sshguard
step
stunnel
swiftformat
ttfautohint

Finished running command: "brew tap azure/functions".
Running command: "brew install azure-functions-core-tools"...
Warning: azure/functions/azure-functions-core-tools 2.5.553 is already installed and up-to-date
To reinstall 2.5.553, run `brew reinstall azure-functions-core-tools`
Finished running command: "brew install azure-functions-core-tools".
Running command: "python3.6 -m venv .env"...
Finished running command: "python3.6 -m venv .env".
Running command: ". .env/bin/activate && func init ./ --worker-runtime python "...
/bin/sh: func: command not found
Error: Failed to run ". .env/bin/activate && func init ./ --worker-runtime python" command. Check output window for more details.

I can still create venv and functions using azure cli from the mac, but not using vscode.

It appears it can't find func in your path. First try restarting VS Code - sometimes environment variables changes aren't picked up immediately.
Second, I suppose you could try uninstalling and reinstalling the func cli. Brew might update the path during that
brew uninstall azure-functions-core-tools

brew install azure-functions-core-tools


ObjectWriter update.

Code sections should have special HasInstructions flag set.
It is done for .text inside objectWriter.

@MichalStrehovsky could you take a look please?
llilc commit .


Show clearer error in browser when fenced code blocks don't have a title

Steps To Reproduce
I created a new page "prod" with the following content:
# Production

Production environment charts and stuff.

```
SELECT COUNT(*) AS prod_builtin_users FROM bq_segment.cloud_prod.user_created AS t1 JOIN bq_segment.cloud_prod.access_code_used AS t2 ON t1.user_id = t2.user_id
```

Note that there's no longer language or title for the code block. I navigated to the new "prod" page and got this error:

I was able to figure out the root issue based on the logs in the terminal:
Error while preprocessing /Users/sean/Code/github.com/glaredb/reports/.evidence/template/src/pages/prod/+page.md - Cannot read properties of null (reading 'toLowerCase')
  untitled running...✗ untitled Queries require a title

And adding the language+title to the code block fixed the issue.
Environment

Node version (node -v): v16.20.2
npm version (npm -v): 8.19.4
OS: macos
Browser: firefox
Database: postgres

Expected Behavior
A clearer error when navigating to the page in the browser.
Actual Behaviour
See above screenshot. I needed to look at the logs to know what happened. And I could see that log message ("Queries require a title") being easy to miss or have it not be obvious that that's what's causing the error.
Workarounds
I read the logs and figured out what caused the error.

This is a regression, thanks for flagging. Untitled queries should show an error as if the SQL failed "queries require a title"


:sparkles: Add verify shutdown state

What type of PR is this?
/kind feature
What this PR does / why we need it:
Adding state to verify that a device is shut down. Also deal with a state where the device is reloading forever. Set a label on that device that it is unhealthy and should not be used in that case.
TODOs:

[ ] squash commits
[ ] include documentation
[ ] add unit tests

didn't we talk as well about code to verify whether a machine is successfully provisioned?

We talked about how to detect if a machine was successfully deprovisioned. We had the idea to create a simple tcpserver which the controller could check. I found a simpler solution. If the machine is on and not reloading and Script does not contain "cloud-init", then the machine is provisioned with the dummy-machine-image.
Since we don't make network connections to the machine up to now, I would rather not create the dummy tcpserver.,


Fix wrong function signature when compiling with HDF5 1.12 series

This is because the function has changed its parameter in HDF5 1.12 releases.
See:
https://portal.hdfgroup.org/display/HDF5/H5O_GET_INFO
https://portal.hdfgroup.org/display/HDF5/API+Compatibility+Macros
Closes issue #23.

thanks for the PR!

Pardon me, I overlooked when supplying the new parameter, it is fixed in PR #25.


GSE5859 available for R version 3.3.1

Is there a GSE5859 available for R version 3.3.1
When running the following command to install the package
biocLite('GSE5859')
I get the following error
Warning message:
package ‘GSE5859’ is not available (for R version 3.3.1)
thanks!

library(devtools)
install_github("genomicsclass/GSE5859Subset")
library(GSE5859Subset)
data(GSE5859Subset)
if that does not work please provide full error trace
also you could do perhaps if interested in the whole series
library(GEOquery)
x = getGEO("GSE5859")
On Thu, Mar 16, 2017 at 5:04 PM, ale07alvarez<EMAIL_ADDRESS>wrote:

Is there a GSE5859 available for R version 3.3.1
When running the following command to install the package
biocLite('GSE5859')
I get the following error
Warning message:
package ‘GSE5859’ is not available (for R version 3.3.1)
thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/genomicsclass/labs/issues/60, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEaOwvEJ6qAOS3OYnsn389hjk1PEGMJlks5rmaPvgaJpZM4Mf4JM
.

library(devtools)
install_github("genomicsclass/GSE5859Subset")
library(GSE5859Subset)
data(GSE5859Subset)
works thanks!!

library(devtools) install_github("genomicsclass/GSE5859Subset")
library(GSE5859Subset) data(GSE5859Subset)


[React] Migrate to React 18

Hello. I would love to contribute to fixing this issue!

Hello. I would love to contribute to fixing this issue!

@OloyedeTobi
Hi, feel free to open a PR.
If there are any issues with migration you can ask here.
Cheers 👍

@OloyedeTobi Have you made any progress?


[Language]How to perform CRUD Operations on Python dictionary

Topic Suggestion
How to Perform CRUD Operations on Python dictionary
Proposed article introduction
In this article, we will learn what Python Dictionary is and how to generate it. I will also explain into details how to create, remove, update, delete and search elements in a dictionary object.
A dictionary in Python is a scrambled collection of objects. Unlike other data types such as a list or a set which has a single value field, the dictionary type stores a key along with its value. The keys can be of any data type varying from integer to string which makes it more flexible.
This article will give the learner a step-by-step guide on how to generate data in the python dictionary.
Key takeaways
By the end of the article, the readers will have learnt:

Python Dictionary Overview
Creating Dictionary
Retrieving data from the dictionary
Updating data
Deleting data from the dictionary
Iterating data

Article quality
In this article, I will show novices how to perform CRUD  on python dictionary. Unlike the existing articles on the same my article will have a complete script that ties everything together. Real life demonstration with output and also include comments to make the learners understand and follow.
References
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Hi @cathy-254
Thanks for submitting your article suggestion. However, after a short google search it seems your article has been widely covered on the internet. Due to this, we cannot proceed with your article as it will raise red flags during the plagiarism checks. Please feel free to explore other topics of interest.

@LinusMuema hello sir, I believe that knowledge can never be exhausted, and when I look at nearly similar online content  are not beginner-friendly and are not as engaging as they should be. Of course, this article will be one-of-a-kind, give live demo on every part of the operation and it will address these issues as gracefully as possible in order to maintain quality and add value to the community.I think it will be unique. Can you have a look at it again. Thanks


Add support for sync_file_range on Linux

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+
Thanks!

:pushpin: Commit 37552b7 has been approved by alexcrichton

Just realized I had defined the SYNC_* constants as signed when they should be unsigned.

@bors: r+
sounds good to me

:pushpin: Commit ae6cbcd has been approved by alexcrichton

:hourglass: Testing commit ae6cbcd with merge 574b293...

:sunny: Test successful - status-appveyor, travis
Approved by: alexcrichton
Pushing 574b29310ac4716505b2971ac668db1811bd8ebd to master...


Add plugin API version.

This version is across all plugin types and it versions the actual RPC
API between the daemon and the plugin.
Signed-off-by: Anusha Ragunathan<EMAIL_ADDRESS>

ping @vieux PTAL


[rest] Request OOB proof and Offer OOB credential should use d_m instead of c_i for encoded message

Currently the urls returned by the created OOB proof and offer OOB credential use c_i (which stands for  Connection Invitation). Although there is not a strict standard, we should use d_m (which stands for DIDComm Message I believe). This is also what Trinsic uses

@janrtvld

Sorry to be late on this, but for OOB, shouldn't the query parameters defined here in the RFC be used?  Or is this at a different level?

Yes for the OOB protocol it does, but we don't support the OOB protocol yet. This is what Trinsic and other wallets are using for the pre OOB protocol way of doing OOB. This way it would be easy for us to detect what protocol to use (oob = OOB protocol, d_m = AIP OOB, c_i = AIP 1.0 connection)


[Sync] Execute worker requests under foreground service

Fixes #430
Things done:

All workers create their separate notifications instead of reusing the same id. This is to prevent any running worker from getting interrupted by another worker.
Call setForegroundAsync to execute worker request under foreground service
Moved worker for tileDownload under persistence/sync

Notifications are canceled automatically by the system once the request completes.
Tested manually:

Turn off wifi
Open app
Select project Plastic Frog Finder
Create a new feature by using '+' button. Then create a new observation
Fill some random data and also attach a photo.
Click Save
Killed the app from recent apps drawer
Turn on wifi
Notifications for photo and data sync are visible

Sorry for the delay! A few questions:

All workers create their separate notifications instead of reusing the same id. This is to prevent any running worker from getting interrupted by another worker.

Why would the notification id interrupt the worker? Do you mean "to prevent existing notifications from getting interrupted" rather than "running worker"?
General question - data sync workers use ExistingWorkPolicy.APPEND, so they wouldn't get interrupted. What is our sync policy for other workers?

Call setForegroundAsync to execute worker request under foreground service

How many notifications are generated? One for each sync type, one for each event, or a single notification continually updated? The latter would be preferable.

Why would the notification id interrupt the worker? Do you mean "to prevent existing notifications from getting interrupted" rather than "running worker"?
General question - data sync workers use ExistingWorkPolicy.APPEND, so they wouldn't get interrupted. What is our sync policy for other workers?

Yes, to prevent it from getting interrupted.
Let's assume that we started downloading offline satellite imagery for a large area. So, a notification would start for that. Now, if we also add a new feature or observation, the same notification would be updated. So, we'd see disappearing of download progress which might confuse users.

How many notifications are generated? One for each sync type, one for each event, or a single notification continually updated? The latter would be preferable.

One for each type. When the task gets finished, then the notification also disappears.


Update robphoenix in maintainers.json file

While I realise I haven't done much in the way of maintaining this track, I
have a new job, and so have to step down as an official maintainer. Thanks &
all the best @ryanplusplus

No worries, best of luck in the new position!


docs: document permission values (#164)

Related to #164 .

I have read the CLA Document and I hereby sign the CLA


Finisher not showing up in the All for One pack

According to the 9 April changelog it should be there?

Thanks for pointing that out, we didn't notice that Finisher was already in the Silent pack. In the next update, we'll replace Finisher in the Silent pack with Leg Sweep.


Update a few components of the website

Add a favicon
Update theme and Font Awesome
Point Ci at Travis, VPCI appears mostly broken

*  External link https://voxpupuli.org/favicon.ico failed: 404 No error
Didn't #44 fix this?

No, #44 only fixes it for posts and nav links, not any random asset. I'm working on that one next.

OK, that seems like the only non-problem, so here we go!


Docs change: "HMSET" is depreciated

Just a heads up: HMSET (https://redis.io/commands/hmset/) has been depreciated in favor of HSET (https://redis.io/commands/hset/) as of Redis 4.0.0. At some point the docs should be updated, but it's not a big deal. The same command syntax should work.

Sorry for taking so long to fix this. I am working on a new version, and I am thinking of yanking this version.


Misnamed and probably pointless function

Describe the bug
In semver_index.py._coerce_suffix, there's this code:
for component in match.group(1)[1:].split('.'):
  if not component:
    pre_components.append('-')
  elif component.isdigit():
    pre_components.append(_remove_leading_zero(component))

The components are split on '.', so no component can contain a '.'.
Yet _remove_leading_zero's job is removing leading '.'s:
def _remove_leading_zero(component):
  """Remove leading zeros from a component."""
  if component[0] == '.':
    return '.' + str(int(component[1:]))
  return str(int(component))

The function is misnamed, because it has nothing to do with zeroes.
To Reproduce
Steps to reproduce the behaviour:

Look at the code.

Expected behaviour

The function name should be changed to _remove_leading_period.
The function probably should be deleted because it has no effect.

@cuixq FYI: Keep this in mind when reviewing code for readability


Add Apollo federation spec compliance tool in CI

Adds a copy of the product subgraph implemented in apollo-federation-subgraph-compatibility repo.
Also includes a Github action to run on each pull request which checks compatibility against Apollo Federation Subgraph Specification and posts the results to the PR.
Targets #215

Thanks a lot! I'll have a look later.

Thanks a lot for your contribution!
Your help is very valuable. I've allowed myself to add you to https://github.com/orgs/sangria-graphql/teams/contributors so that you can have more permissions to contribute if you want. If you disagree, just tell me, and I'll remove you.

Thanks a lot for your contribution!
Your help is very valuable. I've allowed myself to add you to https://github.com/orgs/sangria-graphql/teams/contributors so that you can have more permissions to contribute if you want. If you disagree, just tell me, and I'll remove you.

Thanks. I'd like to help out if possible.  Let's see how much time I can spend on the project.


Feature request: making the Electricity Capacity accessory more useful

https://github.com/ebaauw/homebridge-p1/commit/ff35d99663a7d343db05dc4ca789985d657a16f4 implemented capacity tariff support. As promised in https://github.com/ebaauw/homebridge-p1/pull/69#issuecomment-1427784275, I've been testing and am now reporting back 😊

The most important information is not yet exposed via Electricity Capacity: 1.4.0 (current window's average consumption), 1.6.0 (current month's consumption peak) and 98.1.0 (previous months' consumption peaks). I posted a detailed analysis + proposal at https://github.com/ebaauw/homebridge-p1/pull/69#issuecomment-1425648028 previously.
No detailed logging for Electricity Capacity as already exists for Electricity? That'd be super valuable while refining this functionality! (Especially combined with #71.) If I'd be able to observe 1.4.0 in real-time in the log, that'd be very powerful.
The Eve History support for Electricity Capacity is either wrong or misleading? 😅 ~Should I multiply the values by 6?~ I should, you wrote that previously at https://github.com/ebaauw/homebridge-p1/pull/69#issuecomment-1427035635! I agree that showing a line would be more helpful, perhaps as Air Pressure. I can confirm that the current result is useless:

1.4.0 is exposed as Current Consumption on the Electricity Capacity accessory.  1.6.0 and 98.1.0 are parsed, but not yet exposed.  If you set Log Level to 2, you should see debug messages for the parsed telegrams, where they are combined under avg_power_peaks.
You should see messages in the log for this characteristic, when Log Level >= 1.
Eve history for Total Consumption is very confusing; it took me several iterations to understand it and get the code right.  The entries, as reported by the accessory, actually contain (average) Current Consumption since the previous entry.  Current Consumption isn't consumption (in Wh) at all, but rather power (in W).  The bar chart does show consumption, which is computed by the Eve app.  In the hourly view, the graph shows a bar per 10 minutes.  With an average power of 1000W, the consumption over 10 minutes = 166.67 Wh.  So yes, if you want the (average) power, you need to multiply that value by 6.  In the daily view, the graphs shows a bar per hour.  Which an average power of 1000W, the consumption over an hour = 1000Wh.  You can use the value directly, but you no longer have the capacity per 15 minutes.
Hence my suggestion to expose 1.4.0 as Air Pressure, whose history would show as a line chart, without any power to consumption conversions.  It would also eliminate the need for an extra accessory, as I can combine the Current Consumption and Air Pressure history in one accessory.

Note that there's a similar conversion for Gas and Water.  The Total Consumption value is in m3 (but Eve displays kWh).  The history entry values are "power" (in litres per hour), and the bar graphs shows consumption in litres (as Wh) or m3 (as kWh).  I don't expose Current Consumption as the P1 only reports gas or water consumption every 5 minutes.  I don't think seeing the average "power" (in litres per hour) over 5 minutes would bring any value, and it would add to the confusion.

Could you try beta v1.3.4-0?
This version no longer exposes a separate ElectricityCapacity accessory, but adds Air Pressure to the Electricity accessories, incl. history.  The value is the peak capacity (from 1.4.0) in W, as reflected in the log, but, obviously not in Eve.
Also includes the following changes:

All accessories are now exposed as Outlet (since Eve doesn't like Stateless Programmable Switch services combined in an accessory with other services and history).  The mandatory On characteristics is Off by default (so it won't show in room status in Eve), and reverts to Off when set from HomeKit.
Gas and Water accessories now expose Current Consumption.
Fixes Log Level bug, see #71.

The value is the peak capacity (from 1.4.0) in W, as reflected in the log, but, obviously not in Eve.
Would it be possible to expose this as an additional characteristic?

Could you try beta v1.3.4-0?
Given a7bb3de's change to Eve history support … is it safe for me to update? What will the consequences be for the Eve app? Any special steps I need to take?

Would it be possible to expose this as an additional characteristic?

With exactly the same value?

is it safe for me to update? What will the consequences be for the Eve app? Any special steps I need to take?

Yes, each history entry includes info on which values it contains.  The graph for Air Pressure will simply show no values for the entries created under a previous version,  without a value for Air Pressure.
Only if you un-expose and re-expose an accessory, you might need to delete the history already downloaded by Eve.  I still don't know how Eve accessories tell the Eve app that history has been re-initialised on factory reset.

Did that. The reboot in action:
[2/26/2023, 1:32:45 PM] Registering platform 'homebridge-p1.P1'
[2/26/2023, 1:32:45 PM] [P1] Loaded homebridge-p1 v1.3.4-0 child bridge successfully
[2/26/2023, 1:32:45 PM] Loaded 4 cached accessories from cachedAccessories.0E981C41ADC8.
[2/26/2023, 1:32:45 PM] [P1] homebridge-p1 v1.3.4-0, node v18.14.0, homebridge v1.6.0, homebridge-lib v6.3.9-0
[2/26/2023, 1:32:45 PM] Homebridge v1.6.0 (HAP v0.11.0) (P1) is running on port 54199.
[2/26/2023, 1:32:45 PM] [P1] hardware: Mac mini (M1, 2020)
[2/26/2023, 1:32:45 PM] [P1] os: macOS Monterey 12.3.1 (21E258)
[2/26/2023, 1:32:45 PM] [P1] restored 4 accessories from cache
[2/26/2023, 1:32:47 PM] [P1] connected to <IP_ADDRESS>:23
[2/26/2023, 1:32:47 PM] [P1] warning: latest version: homebridge-p1 v1.3.3
[2/26/2023, 1:32:47 PM] [P1] warning: ignoring invalid telegram
[2/26/2023, 1:32:48 PM] [P1] FLU5\253770234_A v5.0
[2/26/2023, 1:32:48 PM] [P1] Electricity: Name: set to "Electricity"
[2/26/2023, 1:32:48 PM] [P1] Electricity: Configured Name: set to "Electricity"
[2/26/2023, 1:32:48 PM] [P1] Electricity: On: set to false
[2/26/2023, 1:32:48 PM] [P1] Electricity: Air Pressure: set to 0 W
[2/26/2023, 1:32:48 PM] [P1] Electricity: Log Level: set to 2
[2/26/2023, 1:32:48 PM] [P1] Electricity: Total Consumption: set to 764.283 kWh
[2/26/2023, 1:32:48 PM] [P1] Electricity: Total Consumption Normal: set to 372.378 kWh
[2/26/2023, 1:32:48 PM] [P1] Electricity: Total Consumption Low: set to 391.905 kWh
[2/26/2023, 1:32:48 PM] [P1] Electricity: Tariff: set to "low"
[2/26/2023, 1:32:48 PM] [P1] Electricity: Current Consumption: set to 0 W
[2/26/2023, 1:32:48 PM] [P1] Electricity: Electric Current: set to 1.79 A
[2/26/2023, 1:32:48 PM] [P1] Electricity: Voltage: set to 242.3 V
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: Name: set to "Electricity Delivered"
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: Configured Name: set to "Electricity Delivered"
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: On: set to false
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: Total Consumption: set to 380.298 kWh
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: Total Consumption Normal: set to 275.588 kWh
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: Total Consumption Low: set to 104.71 kWh
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: Tariff: set to "low"
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: Current Consumption: set to 148 W
[2/26/2023, 1:32:48 PM] [P1] Gas: Name: set to "Gas"
[2/26/2023, 1:32:48 PM] [P1] Gas: Configured Name: set to "Gas"
[2/26/2023, 1:32:48 PM] [P1] Gas: On: set to false
[2/26/2023, 1:32:48 PM] [P1] Gas: Total Consumption: set to 570.752 m³
[2/26/2023, 1:32:48 PM] [P1] Electricity Capacity: remove stale ElectricityCapacity v1.3.3-0 accessory 1SAG1105302451C
[2/26/2023, 1:32:48 PM] [P1] Electricity: warning: remove stale service 00000011-0000-1000-8000-656261617577. (Service)
[2/26/2023, 1:32:48 PM] [P1] Electricity: warning: remove stale service 00000089-0000-1000-8000-0026BB765291 (StatelessProgrammableSwitch)
[2/26/2023, 1:32:48 PM] [P1] Electricity: warning: remove stale context 00000011-0000-1000-8000-656261617577.
[2/26/2023, 1:32:48 PM] [P1] Electricity: warning: remove stale context 00000089-0000-1000-8000-0026BB765291
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: warning: remove stale service 00000011-0000-1000-8000-656261617577. (Service)
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: warning: remove stale service 00000089-0000-1000-8000-0026BB765291 (StatelessProgrammableSwitch)
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: warning: remove stale context 00000011-0000-1000-8000-656261617577.
[2/26/2023, 1:32:48 PM] [P1] Electricity Delivered: warning: remove stale context 00000089-0000-1000-8000-0026BB765291
[2/26/2023, 1:32:48 PM] [P1] Gas: warning: remove stale service 00000011-0000-1000-8000-656261617577. (Service)
[2/26/2023, 1:32:48 PM] [P1] Gas: warning: remove stale service 00000089-0000-1000-8000-0026BB765291 (StatelessProgrammableSwitch)
[2/26/2023, 1:32:48 PM] [P1] Gas: warning: remove stale context 00000011-0000-1000-8000-656261617577.
[2/26/2023, 1:32:48 PM] [P1] Gas: warning: remove stale context 00000089-0000-1000-8000-0026BB765291
[2/26/2023, 1:32:49 PM] [P1] Electricity: Electric Current: set to 1.78 A (from 1.79 A)
[2/26/2023, 1:32:49 PM] [P1] Electricity Delivered: Current Consumption: set to 173 W (from 148 W)

I see an Air Pressure service on the Electricity accessory. It currently reads 0 hPa, because solar power provides all electricity, so 0W is imported 👍 I will monitor and report back!
P.S.: I updated beyond 1.3.4-0 — I'm running https://github.com/ebaauw/homebridge-p1/commit/155ac8c54d5d1761865afa953d466d3e398905a9 already (I'm comfortable with git and npm link, so as far as I'm concerned, you don't need to tag pre-release versions — I'm happy to test a specific commit you ask me to test!).
P.P.S.: the [2/26/2023, 1:32:47 PM] [P1] warning: ignoring invalid telegram [2/26/2023, 1:32:48 PM] [P1] FLU5\253770234_A v5.0 in there is pretty curious … is that logged somewhere?

[2/26/2023, 1:32:47 PM] [P1] warning: ignoring invalid telegram

That's Homebridge P1 connecting to the P1 port while it's sending a telegram, causing it to receive only a part of the telegram.  Pretty common for DSMR 5, which sends a telegram every second.

[2/26/2023, 1:32:48 PM] [P1] FLU5\253770234_A v5.0

That's a seconds later, after the first telegram has been received in full.

you don't need to tag pre-release versions

Beware that there might also be changes to homebridge-lib.

That's Homebridge P1 connecting to the P1 port while it's sending a telegram, causing it to receive only a part of the telegram. Pretty common for DSMR 5, which sends a telegram every second.

Very interesting, and confirmed just now:
[2/27/2023, 7:28:35 AM] [P1] connected to <IP_ADDRESS>:23
[2/27/2023, 7:28:35 AM] [P1] warning: latest version: homebridge-p1 v1.3.3
…
[2/27/2023, 7:28:36 AM] [P1] warning: ignoring invalid telegram

👍

Beware that there might also be changes to homebridge-lib.
Yep! Checking those too 😊

I've now been running this for a while. So I started looking at the data, to check if it makes sense. And it does not … at least not at first sight — I'm hoping you can help me understand 😊

☝ This is last night, in the middle of the night. It sometimes goes down to 14 hPa … which AFAIK means "14 Watt used on average for the past 10 minutes" (since you're aligning to Eve's 10-min resolution, not to Fluvius' 15-min resolution).
But … I only wish my idle power consumption in the middle of the night would drop to 14 hPa 😅 That's never actually the case!
So perhaps it should be multiplied by 6? 6*14 = 84 W. That is possible.
However, if that is accurate, then something else does not make sense:

The highest peak in the last week is 1489 hPa. Multiplied by 6 yields 8934W. It's absolutely impossible for me to reach that level of consumption. I'd need to run a hairdryer, washer, dryer, oven, stove, microwave, toaster, coffee machine all at the same time and then I'd maybe get there. (That scenario definitely didn't happen 😅)
So … any idea what's going on here? How should I interpret the numbers reported by homebridge-p1?


Use System.text.Json and multitarget to be able to omit System.Text.Json for supported platforms

Once this gets merged, I'd love to see a beta release.

Yes, this is the straight-forward apporach to use System.Text.Json ... but did you run the unit-tests?
At least for me the license list is not properly load. There is a base class and there are interfaces - two things that work smoothly with Newtonsoft.Json, but not with System.Text.Json.
I will spend the next two days some more hours to find a solution.

I found a solution and a merged that branch.
This PR is no longer needed.