FW-1910

The Approve/Reject drawer was displaying UI text for current visibility and requested visibility in the wrong spots.

Appreciate the review @gmcauliffe


GhostedRecall

After following the instructions on this page, I checked the Access policy. (Honestly, I did it after setting up the Azure File Sync, per instructions here: https://docs.microsoft.com/en-us/azure/storage/files/storage-sync-files-deployment-guide?tabs=azure-portal
The Access policy has an odd Identifier setup that says "GhostedRecall"
That was not part of any ACL or access property I setup on our local AD. I have Azure AD installed and Azure AD connect setup.
I believe the documentation needs to mention what GhostedRecall is, where it came from, what is it for, and why we would or would not want to leave it.
I also have a lot of questions about OnPrem ACL's and the Access policy here, and how to verify what ACL's are in place, but I'm not sure if this is the right document that should have addressed this. I'll keep checking.
In any event, can we shine some light on "GhostedRecall"?
Thanks!
    == John ==

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

ID: ad2c9646-a4bc-aa28-de27-5fce4d4b4e44
Version Independent ID: 1288f4c1-e871-8bc7-9693-2e62d5074a09
Content: How to create an Azure file share
Content Source: articles/storage/files/storage-how-to-create-file-share.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

@jgwinner Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@roygara

@jgwinner thanks for reaching out, I'm not too sure I follow, though. Would you mind clarifying what you mean by access policy?

I'm more wondering what GhostedRecall is, who has that right, and why it's there.

My client is an ISO company, and we need to know who has access to what.

@jgwinner I understood that part but, you're saying ghostedRecall, is coming up as part of your access policy. I don't understand what you mean by access policy, that text isn't in the article. Would you please clarify what you mean by that so we can look into this?
@Adam-Smith-MSFT FYI

@jgwinner Is this what you mean by access policy? https://docs.microsoft.com/en-us/windows-server/networking/technologies/ipam/create-an-access-policy

No, that's under IPAM; what I mean is the Access Policy under the File properties under the File Share.
Go to portal.azure.com
Pull up the storage account
(Breadcrumbs are Home > Storage account > File shares (pick a share) > (Share name)
Click on a file, go to "Access Policy" on the left panel.

@jgwinner Thanks for clarifying, I've looped the product team into this and am waiting to hear back. It might not be until after Ignite, though. I'll let you know as soon as I hear anything.

@jgwinner I heard back from the product team. That Access policy is set automatically, it's so that the SAS can be recalled. It's only used if you use GetSharedAccessPolicy.
Would you mind sharing why you were trying to configure SAS? I didn't find much of anything in the article besides a small mention of ensuring your SAS is valid by not deregistering your server.

I wasn’t configuring SAS, at least to my knowledge.
I did a very, very basic setup.
Brand new O365 client.
Fairly vanilla On-Prem AD.
Setup Azure File Sync.
After setting up the Azure File Sync, went to see what permissions were on the file.
Saw GhostedRecall.
I’m not using any custom code, c# or otherwise. Just the vanilla control panels.
    == John ==

@jgwinner Ah, "Access policy" on that screen configures SAS. "Access control" configures the Azure AD.
Are you enrolled in the limited private preview for Azure AD DS? You wouldn't be able to use an on-prem AD otherwise. Only an Azure AD with domain services.

Well, we're using an On Prem AD by Azure File Copy, but not via Azure files, so I understand.
We'd be VERY interested in the private preview. How can I get access?
    == John ==

Oh, sorry - regardless, it's somewhat odd looking and I think it would be helpful to be documented. I mean, you didn't even know what it was! :)
    == John ==

@jgwinner To get access to the preview, fill out this survey - http://aka.ms/AzureFilesADSignup
As always, you should keep in mind Azure's preview terms of use, before signing up for any of our previews. https://azure.microsoft.com/en-us/support/legal/preview-supplemental-terms/
I don't think we should add this information to either of these documents though. Neither of them contain any mention or information on Access Policy (hence my confusion). Do you happen to recall what sent you in this direction? I could have overlooked something that could use a second look.

@jgwinner did you have any additional questions on this particular issue? If not, that's fine, please let me know. If I don't hear back from you I will close this out in two days.
Again, thanks for reaching out. :)

#please-close

We still get "Weird things" in the control panel that have no explanation. Either, the control panel needs to change, or something should explain what "Ghosted Recall" is. That's the purpose of documentation. So no, I do still have questions on this issue.
Sorry for the delay, it's frustrating when I run into things even the documentation people don't know what it is, and see no possibility of improvement. Even if it's my mistake, it's a mistake that's easy to make, so it should be clarified via documentation.


Drawing of wrong circles  on some android devices

Description
On some android devices the circles are drawn on the screen visibly lengthened. To obtain a visibly acceptable circle you must draw an oval with a ratio of 10 to 8, then with rx=100 pixels and ry=80 pixels.
Code
       var info =  e.Info;
       var surface = e.Surface;
       var canvas  = surface.Canvas;

        // CLEARING THE SURFACE
        canvas.Clear(SKColors.White);

        // DRAWING SHAPES

        // create the paint for the outline of the circle
        SKPaint paint = new SKPaint
        {
            Style = SKPaintStyle.Stroke,
            Color = SKColors.Red,
            StrokeWidth = 25
        };

        // draw the outline of the circle
        canvas.DrawCircle(info.Width / 2, info.Height / 2, 100, paint);

Expected Behavior
I expect a circle with the same height and width.
Actual Behavior
On some Android devices the circle shown on the display is extended vertically.
Basic Information

Version with issue:  1.60.0
IDE:  Visual Studio Community 2017 (15.6.7)
Platform Target Frameworks:

Android: 6.0

Screenshoot

This problem occurs only with the device "ASUS - Thinker board/2GB - Android 6.0.1" (Part. No. 90MB0QY1-M0EAY0) connected to an "elo Touch" monitor model ET1093L.
The defect could be due to the fact that the Asus board has an output resolution of 1280x720px while the monitor resolution is 1280x800px ?

That might be it... Are any other controls stretched? Also, what is the pixel dimensions of the canvas (e.Info) and the view dimensions?

It is difficult to determine if there are any other stretched controls, with a circle, as with a square, the difference between height and width is evident.
For the information that you have requested I enclose the following sreenshoot:

produced by the following code:
        SKImageInfo info = e.Info;
        SKSurface surface = e.Surface;
        SKCanvas canvas = surface.Canvas;

        canvas.Clear(SKColors.White);

        SKPaint paint = new SKPaint
        {
            Color = SKColors.Black,
            TextSize = 40
        };

        float fontSpacing = paint.FontSpacing;
        float x           = 20;               // left margin
        float y           = fontSpacing;      // first baseline
        float indent      = 100;

        canvas.DrawText("SKCanvasView Height and Width:", x, y, paint);
        y += fontSpacing;
        canvas.DrawText(String.Format("{0:F2} x {1:F2}", canvasView.Width, canvasView.Height), x + indent, y, paint);
        y += fontSpacing * 2;
        canvas.DrawText("SKCanvasView CanvasSize:", x, y, paint);
        y += fontSpacing;
        canvas.DrawText(canvasView.CanvasSize.ToString(), x + indent, y, paint);
        y += fontSpacing * 2;
        canvas.DrawText("SKImageInfo Size:", x, y, paint);
        y += fontSpacing;
        canvas.DrawText(info.Size.ToString(), x + indent, y, paint);

Tomorrow, with this accessory elo touch cable OSD I try to inhibit screen resizing on the monitor.
Thank you very much for your interest.

Another thing that you could try is to do the same drawing without SkiaSharp and just use the native Android APIs.
This looked like an interesting case, so I have created a sample app that tries to draw using Xamarin.Forms, SkiaSharp and Android.Graphics: https://github.com/mattleibow/AndroidStretchingTests
Let me know if there are any differences between drawings - they should all be exactly the same.

By running the AndroidStretchingTests on my device I get the three views exactly the same, all three stretched out in the same way along the vertical axis.
I apologise for the delay, but I have only now resumed the project.
In view of the outcome, I consider this issue to be closed.

Ok, looks as if the screen/android is stretching the entire view/app.


[Help] unable to import Postman dump into Insomnia

Unable to import the dump file generated from postman into insomnia

Insomnia Version:  5.12.4
Operating System: 5.5.0

Generated the dump file from
Postman >> Settings >> Data >> Download
Tried importing using the insomnia import option
Insomnia >> Preferences >> Data >> Import from File
received the error message "No Importers found for file"
Below is the gist to the json schema from the json dump generated
https://gist.github.com/gokulraja/8f5628805ff110a81b184b3acf135a29
A sample of the initial json is added as a comment to the gist

Ah yes, right now Insomnia only supports Postman collection v2 format. To export in this format, you must use the Export Collection menu from the collection dropdown in the sidebar.

Thank You !!

😄

Oh Dude..Hi Five!!

Hi there! Postman is deprecating Scratchpad soon which will make it impossible to work in offline mode. This is a great opportunity for Insomnia to capitalize on since many companies don't want to sync their private endpoints & secrets to Postman's cloud. For this reason, is there any effort to support importing the Postman collection v2.1 format soon?

Hi team, for now, follow @gschier'step, still meet the same issue. Not sure if it's because the latest version of postman made some changes, after all, it's been years.

Hi team, My postman was updated and collections are lost. I am able to export data from scratchpad version. But json dump export is of collection version 1. Is there a way we can convert v1 to v2.1 or workaround to v1?
Thanks in advance!


Offset when second click

When I'm drawing on the canvas, the curve on the canvas follows the mouse perfectly when I click for the first time;but when I click for the second time, the curve produced on the canvas is offset from the mouse position.
Why does this happen, and how can it be solved?

can you create a jsfiddle or codepen that can reproduce this?

I found it's caused by CSS:
#signature {
    width: 6.5rem;
    height: 2.5rem;
    margin-left: 0.25rem;
    margin-top: 0.3rem;
    background-color: #f3f3f4;
}

Why?

If you can create a way for us to reproduce it we can try to help debug it. Can you try to create a div around the canvas an add the styles to that instead?

Canvases have two widths and heights. They have CSS width and height and they have canvas width and height . If those are not the same, strange things can happen. See https://github.com/szimek/signature_pad#handling-high-dpi-screens

thanks for your reminder


CLI support for listing out current app resources

With corresponding provider names. Fixes #1136

Thanks! The commit message needs a cli:  prefix.

Should I amend it to be this then? cli:Add support for listing out current app resources, with corresponding provider names.

The first line should be cli: Add provider list command, you can describe in more detail starting on the third line. The commit should be amended and force-pushed to the same branch.

Looks good! Please squash the last commit and rebase on the latest master and I'll merge.

Done and ready for merge.

Thanks!


Field label language fallback doesn't work

If the panel user has a language that is not included as a translation for this plugin, the field labels shows the label keys instead of a fallback translation.
To reproduce:

For the currently logged in user, set the user language to a language that is not supported by this plugin, i.e. Swedish.
Go to a page and show the SEO tab.
All field labels shows the label key: global-meta-headline, meta-title-template, meta-title-template-help, meta-description etc

I would rather see that if the user language is not supported, the fallback would be shown with English translation.

This doesn't seem like a plugin issue but rather an issue of core. Surely it should work by default?
https://github.com/getkirby/kirby/issues/1188

I was also thinking it might be a Kirby issue. Forgot to mention that. But because of this issue I took the time to make a PR with a swedish translation. So something good came out of it. 😄


add support for Github app

We need to add code to be able to deploy Repairnator as a Github app.
The application is registered at the organization or at the user level.
It's NOT a Github webhook or a Travis webhook.
Activation
Visit https://github.com/apps/repairnator-bot aand click on the green button "Install"
How it works?
If the build is failing and is in Java, it pushes the build id to an ActiveMQ. Then, the default pipeline worker with ActiveMQ can pick it.

Documentation: https://developer.github.com/apps/

Luc has created the app: https://github.com/apps/repairnator-app

Based on the repairnator's arch diagram (from henry)

There are two ways:

NodeJS Server feeding failed travisCI builds to BuildRainer.java, like tdurieux/travis-listener
Java Server pushing failed travisCI builds to MQ directly, equivalent to RTScanner.java

It seems very recommended to use JS for building GitHub app, so I will take the first way.

It seems very recommended to use JS for building GitHub app

That's a valid option. Although a JVM language (Java/Scala/Kotlin) would be a preferred option for
providing unified maintenance and deployment.

I have been familiar with the work for building GitHub app. I will turn to the second way.

Maybe I will still use the first way, because current/popular solution relies on Probot, one framework for building GitHub Apps. (or else we have to implement some necessary parts in that framework, such as receiving and validating webhooks, and doing authentication, etc.)

that's an excellent idea to reuse a dedicated framework such as probot.

app implementation and WebSocket demo:
https://github.com/jianguda/repairnator-bot
one repo for testing:
https://github.com/jianguda/repairnator-bot-test

Per our discussion, we don't need websockets, we don't need BuildRainer.
Actions:

push to ActiveMQ
move the code from https://github.com/jianguda/repairnator-bot to a WIP PR here
Run the app on the KTH Box in 1548 (See with @javierron)

@gluckzhang is deploying the worker, and see also doc at https://github.com/eclipse/repairnator/pull/1021.

Hi @jianguda ,
The messages your github app pushes to ActiveMQ will be handled by a repairnator pipeline. Could you document the format of these messages?

What type: byte array / plain text?
What format: JSON?

Thank you!

Note: we need to keep full backward compatibility with the format already implemented in the
BuildRainer.

Both BuildRainer and RepairnatorScanner only push build ids to the queue. How about

adjusting github app to push build ids in plain text as well?
OR enhancing PipelineBuildListener.java to check whether a message is formatted as JSON. Now the JSON string pushed by github app contains the source of a build id (travis.org / travis.com), but I am not sure if this is useful for the pipeline.

Which idea do you prefer? @monperrus @jianguda

I'd go for Option 2 (enhancing PipelineBuildListener.java). Pushing structured JSON data is more
extensible, we will likely enrich this format later.

Agree. I'll enhance the PipelineBuildListener and propose a PR later :)

Hi @monperrus , please take a look at #1026 :)
@jianguda, FYI, now PipelineBuildListener supports a JSON string like {"buildId":"648902893","CI":"travis-ci.org"}.

Hi @monperrus , please take a look at #1026 :)
@jianguda, FYI, now PipelineBuildListener supports a JSON string like {"buildId":"648902893","CI":"travis-ci.org"}.

Nice! Thanks!

done per #1009 we will create new issues for bugs


build: add codecov task to CI actions

Description
Adds missing codecov config. That has landed in the upstream template repository after this repository was created.
Checklist
Ensure you completed all of the steps below before submitting your pull request:

[ ] Added natspec comments?
[ ] Ran forge snapshot?
[x] Ran yarn lint?
[ ] Ran forge test?

@3esmit CI is failing currently because main is failing. Once that is resolved this PR should be gtg.

@3esmit rebased this PR as we've rolled back on some of the non-compiling changes.


Convert HandleData into Skeleton in Asset function

This PR moves the skeleton creation to a function in the Asset namespace instead of the Animation plugin.

Actually, are changing the IO management via the PR #209.
It would be nice of you to review the PR #209 in order to validate the architecture, and then update your present PR accordingly once the PR #209 is accepted.


Have you updated for Swift 3?

Have you updated for Swift 3?

This package still requires a lot of work. It hasn't been updated for Swift 3 and still relies on NSURLRequest.
Best,
Scott Gauthreaux
Developer
www.scottgauthreaux.com

On Apr 20, 2017, at 08:20, Shyamal S. Chandra<EMAIL_ADDRESS>wrote:
Have you updated for Swift 3?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


checkfiles step for shell scripts

After we move the integration test to its own script, we can add shellcheck to the testing pipeline/workflow to check for shell script errors.
For this step, I've used the shellcheck github actions from marketplace: https://github.com/marketplace/actions/shellcheck
Test for intentional mistake: https://github.com/hellofresh/kangal/pull/26/checks?check_run_id=1181914579

+1


cargo-sort: 1.0.6 -> 1.0.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/devinr528/cargo-sort/releases.
meta.description for cargo-sort is: "A tool to check that your Cargo.toml dependencies are sorted alphabetically"
meta.homepage for cargo-sort is: "https://github.com/devinr528/cargo-sort"
meta.changelog for cargo-sort is: "https://github.com/devinr528/cargo-sort/blob/v1.0.7/changelog.md"
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/n14p7jxnw6hc73qh2d8mcxh1l5ysfanz-cargo-sort-1.0.7/bin/cargo-sort passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 1.0.7 with grep in /nix/store/n14p7jxnw6hc73qh2d8mcxh1l5ysfanz-cargo-sort-1.0.7
found 1.0.7 in filename of file in /nix/store/n14p7jxnw6hc73qh2d8mcxh1l5ysfanz-cargo-sort-1.0.7

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
cargo-sort
docbookrx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/n14p7jxnw6hc73qh2d8mcxh1l5ysfanz-cargo-sort-1.0.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-sort https://github.com/r-ryantm/nixpkgs/archive/dc6187bba1994c4e83a735b6d425af2b1fa53c71.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/n14p7jxnw6hc73qh2d8mcxh1l5ysfanz-cargo-sort-1.0.7
ls -la /nix/store/n14p7jxnw6hc73qh2d8mcxh1l5ysfanz-cargo-sort-1.0.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cargo-sort

Maintainer pings
cc @figsoda for testing.

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

1 package built:

cargo-sort


fix(comment-url): hold fragment if there's any

for comment url https://web-next.matters.news/p/30000#comments
for particular comment url https://web-next.matters.news/@smog_again/30000#Q29tbWVudDoyNzg5Njc-Q29tbWVudDoyNzkwMDY
for article owner's ?mode=edit to enter edit mode

also add utm tracking code, for better computing links source&destination before or after visiting a link
should also resolve https://github.com/thematters/matters-web/issues/2587

should be already in 86a446a0d ; without debugging log lines


nixos/acme: add s3Bucket option

Description of changes
Use case: serving on an anycasted IP, with the well-known/acme-challenge being backed by an S3 bucket.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

This is AWS s3 exclusive, as lego does not expose a way to specify a custom s3 endpoint, right?

Yes, I have a patch locally:
File: ../nixfiles/pkgs/lego.diff
diff --git a/providers/http/s3/s3.go b/providers/http/s3/s3.go
index d8fc4246..84bf3c46 100644
--- a/providers/http/s3/s3.go
+++ b/providers/http/s3/s3.go
@@ -5,6 +5,7 @@ import (
    "bytes"
    "context"
    "fmt"
+   "os"
    "strings"

    "github.com/aws/aws-sdk-go-v2/aws"
@@ -33,7 +34,11 @@ func NewHTTPProvider(bucket string) (*HTTPProvider, error) {
        return nil, fmt.Errorf("s3: unable to create AWS config: %w", err)
    }

-   client := s3.NewFromConfig(cfg)
+   client := s3.NewFromConfig(cfg, func(o *s3.Options) {
+       o.EndpointResolver = s3.EndpointResolverFromURL(os.Getenv("AWS_ENDPOINT_URL"), func (e *aws.Endpoint) {
+           e.HostnameImmutable = true
+       })
+   })

    return &HTTPProvider{
        bucket: bucket,

For reference also on PR, this blocks evaluation and thus channels (for unstable-small already).
nixos.tests.acme.aarch64-linux: error:
       … while calling the 'head' builtin

         at /nix/store/8vm2qc8gqki6i8avyrwy3sjhvc100nib-source/lib/attrsets.nix:850:11:

          849|         || pred here (elemAt values 1) (head values) then
          850|           head values
             |           ^
          851|         else

       … while calling the 'deepSeq' builtin

         at /nix/store/8vm2qc8gqki6i8avyrwy3sjhvc100nib-source/lib/customisation.nix:286:7:

          285|     in if drv == null then null else
          286|       lib.deepSeq drv' drv';
             |       ^
          287|

       (stack trace truncated; use '--show-trace' to show the full trace)

       error:
       Failed assertions:
       - Exactly one of the options
       `security.acme.certs.http.example.test.dnsProvider`,
       `security.acme.certs.http.example.test.webroot`,
       `security.acme.certs.http.example.test.listenHTTP` and
       `security.acme.certs.http.example.test.s3Bucket`
       is required.


What happened to the with-redux-saga example directory?

Is there a specific reason this was removed from the examples?

It looks like the examples in v3-beta are still in that branch instead of master 🤔
cc @timneutkens @arunoda

Oh, thanks!

It seems like no files were committed and just the package.json version was bumped? It seems like no v3.0.1 features are actually available. @rauchg

It's fixed now, v3-beta examples are now in master.


SIWA on android refactor

SIWA on android refactor changes split from #5 , merge #7 first.

Closing in favor of addressing in cleanup task.


:art:Consultancy page hero section #74

Description
This Pull Request (PR) addresses the following issues and introduces some enhancements:

Fixes #74
dependant on #11 , #1

The changes in this PR aim to improve the user experience on our website. I have added a hero section in the consultancy page. It done in /src/features/pages/consultancy-page.html
Type of change

[x] New feature (non-breaking change which adds functionality)

Screenshot (optional)

On smaller screens

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Good attempt closing this pr as it is already done, please pick another issue


Stolen work from you!

Hi,
your repo here,
Has been stolen & re-uploaded by the GitHub user: https://github.com/5R33CH4
Here is their repo in which they've stolen your work, and attributed it to themselves: https://github.com/5R33CH4/Anime4K-
They also have a GitHub org, where they likely also have some stolen work: https://github.com/tesseract-org.
Checkout their all their personal repos...
I'm sorry to bring you this bad news, this user has stolen >50 repos so you should consider reporting them to GitHub.
Galen

Thanks for the report, currently it looks like their repo was removed.


Update ready_made_pipelines.mdx

Capitalized first letter in params of code examples in keeping with  https://github.com/deepset-ai/haystack/issues/1748
, not sure if there are other places where this change has to be made too

Hi @nishanthcgit , thanks for pointing this out! I just opened a PR on the pipelines documentation some different changes. For simplicity of merging, I've incorporated your changes into my PR and acknowledged your contribution. Hope that's alright!


Replace checkNotNull with requireNonNull

I'll squash the commits together before pushing, but left them separate to make review easier. Each commit was creating using the "inline" action in IntelliJ to inline that specific signature

I'm not sure the format() change is an improvement, otherwise looks good


Crash on right clicking Warp Core in Space Station

AR 1.0.19 in Age of Engineering Modpack
Forge 2316
MC 1.10.2
https://pastebin.com/UCZ3AEdy
Someone built the Warp Core on the Space Station on Earth and send it into space. In the Space Station he continuosly crashed on accessing the Warp Core by right clicking. Even breaking the Warp Core and replacing it, led to this.

Does this occur in the 1,1.x+ builds?

I haven't heard from anyone having this issue since then, but I also added a note into the guide that you should only form the multiblock in space, just in case. I won't update the pack to the newer AR versions though because so many things changed, including things that break progression and not sure if I want to invest the time figuring out if I can change recipes/configs to counteract that.
Btw, I'm glad you are back as I heard you took a break. Looking forward to 1.12 versions for my upcoming packs. You don't have a discord or something similar if I have questions etc?

Is this still a issue? Please comment if it is and i will reopen if if you are unable? We are closing old issues to see what is still relevant and needs to be fixed.


Improve KeywordIdentifier detection in symbol table using

IsIdentifierLike()
The overlapping keywords and identifiers are the result of
mis-classification in the SymbolTable by simply looking
at SymbolIdentifier; instead, use the IsIdentifierLike()
predicate function.
Also, while at it, reduce the number of special cases by
only keep the KeywordIdentifier tokens that are also used
in other places of the grammar.
Long term, we need to make sure to properly handle
KeywordIdentifiers in the whole stack; it seems like there
are other subtle bugs lurking.
One approach would be to change the token tag to
SymbolIdentifier if we happen to encounter it in
KeywordIdentifiers, but currently there are hidden assumptions
that this is not happening.
Fixes #917
Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

Ok, looks like using IsIdentifierLike() as an indicator to treat a leaf as identifier results in more issues surfaced by the smoke test. So for now, just reduce the number of unnecessary keywords.

Codecov Report

Merging #1226 (efaae9d) into master (f280847) will increase coverage by 0.00%.
The diff coverage is n/a.

:exclamation: Current head efaae9d differs from pull request most recent head b038c5c. Consider uploading reports for the commit b038c5c to get more accurate results

@@           Coverage Diff           @@
##           master    #1226   +/-   ##
=======================================
  Coverage   92.66%   92.67%
=======================================
  Files         338      338
  Lines       23314    23286   -28
=======================================
- Hits        21605    21580   -25
+ Misses       1709     1706    -3

Impacted Files
Coverage Δ

verilog/analysis/symbol_table.cc
98.70% <ø> (ø)

verilog/formatting/verilog_token.cc
100.00% <ø> (ø)

verilog/parser/verilog.lex
96.29% <ø> (-0.05%)
:arrow_down:

verilog/parser/verilog.y
82.96% <ø> (+0.02%)
:arrow_up:

verilog/parser/verilog_token_classifications.cc
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

all the removed keywords were only used in the KeywordIdentifier, not anywhere else in the grammar. In which way could we see failures here ?

Files with syntax errors detected in the ivtest (200 out of 2789) is the same for

source head CI https://github.com/chipsalliance/verible/runs/5366850371?check_suite_focus=true#step:7:969
This pull request https://github.com/chipsalliance/verible/runs/5366870155?check_suite_focus=true#step:7:907


shebang command not found

Using scala-cli version 0.0.8 on macOS, I get a shebang: not found error when trying to run the shebang examples on scala-cli.virtuslab.org/docs/guides/scripts. This is what I get when I run the command at the command line:
$ scala-cli shebang
shebang: not found

I’m not sure if I’m reading that document correctly, but that’s what I get. (I notice that 'shebang' is not in the help text, so maybe I’m doing something wrong?)

After further testing, I see that the script approach works with just this header:
#!/usr/bin/env -S scala-cli

So now I’m guessing that the shebang command isn’t required any more?

It is actually required, it's just not in the current release (it'll be in the upcoming one). Ideally, it should have appeared on the website only after the next release. We're working on a way to ensure new features only land in the documentation upon release to prevent that kind of thing from happening.

The first approach to update website with the release: https://github.com/VirtusLab/scala-cli/pull/433

shebang is present in freshly released 0.0.9 version


Convert to timestamp from milliseconds does not work

Hi Everyone,
The statement - To convert milliseconds from epoch to a date or timestamp, use toTimestamp(). If time is coming in seconds, multiply by 1,000.
toTimestamp(1574127407*1000l)
written in this page- https://docs.microsoft.com/en-us/azure/data-factory/concepts-data-flow-expression-builder#find-time-from-epoch-or-unix-time
is not working inside Data Flow, getting
Error:
Cannot convert to number: number would be truncated
Please help.

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

ID: 0566a65b-8e82-4de1-0568-a6f28ae33d03
Version Independent ID: eb158bc3-f017-32e3-18ca-a20efe306d86
Content: Expression builder in mapping data flow - Azure Data Factory
Content Source: articles/data-factory/concepts-data-flow-expression-builder.md
Service: data-factory
GitHub Login: @kromerm
Microsoft Alias: makromer

@shawon017  Since this issue is not related directly to the doc, I would recommend you to create a thread on the forums - Microsoft Q&A or Stack Overflow
Once you post your issue on forums, it will have visibility across the community which is a better suited audience for such types of issues.

@shawon017  We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Can't install module

Tried installing in my existing (and also new) project and I'm getting
'RCTView.h' not found
'RCTViewManager.h' not found

Getting the same here:
In file included from /Users/username/reponame/node_modules/lottie-react-native/lib/ios/LottieReactNative/LottieAnimationViewManager.m:9:
/Users/username/reponame/node_modules/lottie-react-native/lib/ios/LottieReactNative/LottieAnimationViewManager.h:9:9: fatal error: 'RCTViewManager.h' file not found
#import "RCTViewManager.h"
        ^
1 error generated.

I'll push out an update that's compatible with RN 0.40. Sorry about that!

Fixed by #28


RPM Build Failures for bpfman v0.5.2

The builds for F39 and F40 succeeded, but F41 and rawhide failed with the following error:
+ /usr/lib/rpm/redhat/brp-mangle-shebangs
*** ERROR: ./usr/src/debug/bpfman-0.5.2-1.fc41.x86_64/vendor/typed-path/src/lib.rs has shebang which doesn't start with '/' ([cfg_attr(feature = "std", doc = include_str!("../README.md"))])

RPM build errors:
error: Bad exit status from /var/tmp/rpm-tmp.51Gkr8 (%install)
    Bad exit status from /var/tmp/rpm-tmp.51Gkr8 (%install)

This seems to be coming from the brp-mangle-shebangs script running on the library as it's marked as an executable. I've submitted a PR to it in order to change. https://github.com/chipsenkbeil/typed-path/pull/33
Will also check a workaround for us for the time being.


Try to parse a JSON with "type" key

Hello,
I am trying to parse a JSON like this:
{
  "trains": [
    {
      "origdest": "LE HAVRE",
      "num": "3132",
      "type": "INTERCITES",
      "voie": "22",
      "heure": "21:41"
    },
    {
      "origdest": "LE HAVRE",
      "num": "3134",
      "type": "INTERCITES",
      "voie": "27",
      "heure": "22:10"
    }
  ]
}

but one of the key is named "type".
I define the following structures:
#[derive(Serialize, Deserialize, Debug)]
struct JsonSncf {
    trains : Vec<TimeLineSncf>
}

#[derive(Serialize, Deserialize, Debug)]
struct TimeLineSncf {
    origdest: String, //	"LE HAVRE"
    num: String, //	"3132"
    type: String, //	"INTERCITES" <--- type is a reserved keyword of Rust language
    voie: String, //	"22"
    heure: String, //	"21:41"
}

The compiler give the following error:
error: expected identifier, found keyword type``
The Json source is from the site (so I am not generating JSON myself ) :
French train company : Paris Saint Lazare station

My first idea is to use a workaround is to replace all "type" by another key in the JSON, but it is a kind of dirty hack.
Another possibility to find a way to link my struct field with "type" key of JSON.

Thanks for your support.

Check out the rename attribute and other possible attributes to customize behavior of generated Serialize and Deserialize impls.

Perfect, your answer is what I expected.
Thank you.


[WIP] prompt - update instruction for commands

Description
Update system instruction related to command execution.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.46%. Comparing base (5573278) to head (ba9c12e).
Report is 16 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1177   +/-   ##
=======================================
  Coverage   95.46%   95.46%
=======================================
  Files          56       56
  Lines        2380     2381    +1
=======================================
+ Hits         2272     2273    +1
  Misses        108      108

Files
Coverage Δ

ols/src/prompts/prompts.py
100.00% <ø> (ø)


Documentation

Make documentation separate from the program itself.

This is partially done. More documentation is needed.

The scope is going to be as follows:
Jumpstart docs: readme.md
Full documentation: wiki
Script execution docs (Included in the wiki): Get-Help .\Convert-ARF.ps1 -full


Support LaTeX syntax

Through JS lib or as pegdown extension

Which pegdown extension you talked about?

write by hand:)


Added 'numberStr' option to colDef.type make use of sortNumberStr functi...

As per discussion with PaulL1

Excellent, thanks.


Can't change icon family

node, npm, react-native, react and native-base version
node: v8.7.0
npm: 5.6.0
react-native: 0.53.0
native-base: 2.3.1
Expected behaviour
Passing type as a prop into an Icon component should set the font family.
Actual behaviour
filter is a valid part of the Feather family:

However when I create a component like:
<Icon type="Feather" name="filter" />
I get:

The only icon family that I can use it seems is the default Ionicons

NativeBase is on 2.6.1
You are using 2.3.1, quite old one

Go through release notes to use this feature of Icon

Ha! That fixed it. Thanks for your prompt response @SupriyaKalghatgi


Move overload checks from handle_certificate to handle_transaction

We already check transaction manager pending queue and per-object dependencies during the handle_transaction in AuthorityState. It makes more sense to not sign the transaction in case the system is overloaded, than to sign the transaction and refuse handling the certificate.
Description
Describe the changes or additions included in this PR.
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Ok, thanks for the input! Let me unify the load shedding checks for both phases, and we can reconsider this policy once we add txn prioritization during txn signing.

Updated the logic and moved these checks to the ValidatorService: could you take another look please?


Imported "testing" outside of a "_test.go" file

Because of this file/line, any executable that imports "go-travis" ends up importing "testing" as well, which adds the following to -help output:
  -test.bench string
    	regular expression to select benchmarks to run
  -test.benchmem
    	print memory allocations for benchmarks
  -test.benchtime duration
    	approximate run time for each benchmark (default 1s)
  -test.blockprofile string
    	write a goroutine blocking profile to the named file after execution
  -test.blockprofilerate int
    	if >= 0, calls runtime.SetBlockProfileRate() (default 1)
  -test.count n
    	run tests and benchmarks n times (default 1)
  -test.coverprofile string
    	write a coverage profile to the named file after execution
  -test.cpu string
    	comma-separated list of number of CPUs to use for each test
  -test.cpuprofile string
    	write a cpu profile to the named file during execution
  -test.memprofile string
    	write a memory profile to the named file after execution
  -test.memprofilerate int
    	if >=0, sets runtime.MemProfileRate
  -test.outputdir string
    	directory in which to write profiles
  -test.parallel int
    	maximum test parallelism (default 8)
  -test.run string
    	regular expression to select tests and examples to run
  -test.short
    	run smaller test suite to save time
  -test.timeout duration
    	if positive, sets an aggregate time limit for all tests
  -test.trace string
    	write an execution trace to the named file after execution
  -test.v
    	verbose: print additional output

Thanks @spencerhakim for the report!
And thanks @shurcooL for the fix.
Sorry for the very long reply, will work on improving that in the future :)


wlroots, sway: make xwayland optional

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

[ ] Fits CONTRIBUTING.md.

I also think it's not necessary to test this variant via a NixOS VM test (at least not for now).
I'm also not sure if this PR is even really necessary in the first place as using xwayland disable in the configuration seems more convenient (so most people probably use that instead; it's also what I'm using for a few months now) but I also don't mind it since Nix supports such configurability (even though it's still not discoverable, etc.).


ios (iPhone6) and iPhone XS tiles_scroll_perf tests

For https://github.com/flutter/flutter/issues/31086.
Also test if our mac8 with iPhone XS is working
(https://github.com/flutter/flutter/issues/34590).

LGTM


Feature/pdct 633 add timeout exception to search api

Description
Please include:

Added timeout error handling
Added max_results query param and set default to 500
Updated family search to return 200 with empty response instead of 404 when no matching families found
Linear ticket PDCT-633

Type of change
Please select the option(s) below that are most relevant:

[ ] Bug fix
[x] New feature
[ ] Breaking change

How Has This Been Tested?
Unit & integration tests added.
Reviewer Checklist

[ ] The PR represents a single feature (small drive-by fixes are also ok)
[ ] The PR includes tests that are sufficient for the level of risk
[ ] The code is sufficiently commented, particularly in hard-to-understand areas
[ ] Any required documentation updates have been made
[ ] Any TODOs added are captured in future tickets
[ ] No FIXMEs remain

Would be good to chat, as it may also be an ideal to set the statement_timeout when we call create_engine here to something like 10s to improve the user experience. See https://stackoverflow.com/a/44936982

Addressed. Link to docs describing how we've configured this here https://www.postgresql.org/docs/current/libpq-connect.html#LIBPQ-CONNECT-CONNECT-TIMEOUT


Document how to build / run locally the tests

Is there a notion page, @nyospe ?

Is there a notion page, @nyospe ?

Was tucked away in https://www.notion.so/espressosys/Kick-off-fe0bc3b62afb4232bf2048edccb58cc8#7e050d7a85814820b46a94fffd54b5b6
Creating a new page for it here:
https://www.notion.so/espressosys/Build-and-run-op-leader-election-497429c789ef4862a5d29a88c0517868


Use Hex colours where theme colours are currently supported

You should be able to code eg #AAAAAA where today you have to code eg BACKGROUND 2.

Coded.

Shipped in v3.7.


Cleanup news

See https://github.com/our-city-app/mobicage-android-client/pull/208
TODO

we should first cleanup all apps that use this
TBD block registration of apps older then x
TBD block registration of apps that use the old news

logs that show usage of getNews: https://console.cloud.google.com/logs/query;query="\"f\":\"com.mobicage.api.news.getNews\","
protoPayload.resource%3D"%2Fjson-rpc";timeRange=P7D?project=rogerthat-server
so it's mostly the ios version of threefold


Add new field in ahead connector

[ ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @tjkb,
I hope you are doing well.
Congratulations, your pull request is approved and merged. Please submit the files to ISV Studio or update your submission using your latest commit ID, please allow us up to 1-2 weeks to review your submission, and our Engineers will notify you in the “Activity Control” area in ISV Studio. Please make sure that the icon meets all the requirements, that it’s size is 230x230, icon background is not white or transparent and it matches the iconBrandColor property in apiProperties.json file.
After making the submission in ISV Studio, please attach an intro.md file in the activity control section. This file is required and it's different from readme.md. You can find an example of intro.md file on this page at step 6.
Please Create an environment in the Preview region. You will use this environment later to test your connector after Microsoft is done with the functional verification for your connector.
We expect all tests to be completed within 1-2 weeks.
If your connector passes all tests, the deployment process begins, and it typically takes up to 3 to 4 weeks to deploy your connector to all of our regions.
Please let me know if you have any questions or run into any problems while creating or updating your submission.
Thank you very much for working with us.

@Amjed-Ayoub thanks for approving but the pr is not merged. Please merge it so that I can proceed with the submission in the ISV studio, thx

@Amjed-Ayoub thanks for the help, now it should be all correct. Please take another look

@tjkb Please submit the files to ISV Studio or update your submission using your latest commit ID


Add PHPStan

and fix static analyser issues

will be great if Travis CI testing can be enabled for this repo

@willemstuursma can you please merge this one too and fix Travis so it is working?

@willemstuursma can you please merge this one too and fix Travis so it is working?

Thanks @mvorisek.
I'll look into the Travis issues some time in the future, they closed for open source projects so I'll have to migrate to Github actions or a different CI tool.

Thanks @mvorisek.
I'll look into the Travis issues some time in the future, they closed for open source projects so I'll have to migrate to Github actions or a different CI tool.


Feature gate bevy dependencies

This crate can still be useful in non-Bevy contexts.
Add a feature flag to make sure that non-Bevy users don't have to pull in Bevy.

There isn't enough here to be useful to non-Bevy users. Closing out. If you want this, feel free to open an issue and argue your case though.


Feature/348 hide y axis x axis and point on char

👋 Thanks for creating a pull request! 🚀 Checkout the storybook we created for it here.


Does Google Analytics in docs capture hits correctly?

Hi again,
Thanks again for all of your work.
In recently trying to configure Google Analytics to work with Phenomic for myself, and using the doc site as an example. In the "Realtime Overview" (leftmost circle) I noticed GA's "Active Page" (middle circle in screenshot) always appeared to be the last page that was "hard-loaded" (e.g. by hitting enter on the URL bar), even though ajax-y "soft-loads" ("virtual pageviews", e.g. by clicking a ) were still showing up in the "per-second totals" (right circle).

Is anyone else observing this behaviour? Is it what is expected?
Lines 41–43 of phenomic/docs/web_modules/GoogleAnalyticsTracker/index.js
  if (isProduction) {
    ga("send", "pageview")
  }

Based on Google's Advice for using Analytics with SPAs, setting the page before sending seems to show the correct "Active Page", even after soft-loading.
  if (isProduction) {
    ga('set', 'page', window.location.pathname);
    ga("send", "pageview")
  }

Does this seem reasonable? It appears to work for me.
It's not a big deal (or hard to fix), I just thought I'd let you know. Thanks again for everything.
Cheers,
Ezra.

This is probably a huge bug. Thanks for letting me know. I will try your recommendation and see if stats are changing.

FYI I think it's best to go to Behavior > Site Content > All pages.

So not sure if the line you added will change something. A better option might be to rely on #384 .

@MoOx Behavior > Site Content > All pages doesn't provide accurate information for this bug. As @ezralalonde said, if this is true, Google Analytics only record the last full page reload.

@ezralalonde I did some test on my website and I can't reproduce this. (probably because I'm using a different GA wrapper)
My component https://github.com/thangngoc89/blog/blob/2108b8d514ce8655c8c1ef11180311c77ce39ab5/web_modules/components/GATracker/GATracker.js

I think I made some tests in the past but I am not sure.
We can probably create an hidden page and access it via another one to see using real time tracking. That might be a way to ensure there is a bug. Did you do that @ezralalonde ?

I did the tracking on the main page, since the site I was testing has ~0 traffic. In this sense it is "hidden".
My test method was manual variations on the following:

Find a page XYZ with 0 active users,
Visit XYZ using a hard-loading method, such as typing it in the browser bar,
Observe XYZ show up in "Active Pages"  with 1 active user, and the per-second visit registered,
Navigate away to ABC using a soft-loading method, such as clicking a ,
Observe that XYZ still has 1 active user, and the per-second visit registered. ABC's value is unchanged

I agree this is easier on a site with 0–1 users than hundreds. I wasn't able to "fake" this by manipulating the URI by appending # or ? strings.
Checking real-time reports is the suggested way to verify a GA installation. I believe these reports are the simplest and most reliable way to track the behaviour, especially if you're dealing with an extremely low-traffic site, as I am.
Like I said, I observed the following behaviour originally:

All page views would register (ie. all views would show up in per-second views),
"Virtual" Page views would never update the "Active Pages" metric.
Reloading the page would always update the "Active Pages" metric.

After adding the one "pathname" line, I noticed that "Virtual" Page views would accurately update the "Active Pages" metric.
For @thangngoc89, your site uses a different GA library (react-ga instead of react-google-analytics) and it does appear to supply the URI to the library for logging.
I think that my test site is substantially similar enough to the Phenomic docs site, though they obviously differ, that it may have existed in both. Still, the bug may be particular to my implementation, or some other factor, since they are not identical.
It's not a critical, show-stopping, life-changing, stop-the-presses issue; I just thought I'd share what I noticed. I only noticed the potential issue because I learn by browsing source of the sites in the showcase, which you have both graciously shared with me.
Thanks again!

@ezralalonde thanks for the detail information. It's great to read your comment. @MoOx patched this as you suggested https://github.com/MoOx/phenomic/commit/5e4d4dd15e510bb77e05d67090612804b271e326

FYI, I am not sure this change really changed something.
See below the number of pageview for last mont (I integrated the change the 16 june).

The total number of pageviews was always being recorded correctly. Certain pages were being counted twice, while others were not being counted at all.
As you know, with isomorphic/universal rendering in React, there are two types of pageviews:

Pages viewed by directly loading an HTML file
Pages viewed by loading and rendering a JSON file

The second type is what was not being recorded correctly in my experience. I was able to test it easily because it was on a low-traffic site, so I could easily correlate the Active Pages/Active Users with actual pageviews. I agree that it would be very hard to tease this data or gauge the effect from actual usage statistics on a website with multiple concurrent users, but that it's very easy when you have perfect knowledge of all visits.
Imagine the following actions:

User uses address bar to navigate to example.org/
User clicks link to /able.
User refreshes current page, /able.
User clicks link to /bravo.

With the bug present, this would register two pageviews for / (steps 1 & 2) and two for /able (3 & 4). With the bug fixed, it would register one for / (1), two for /able (2 & 3), and one for /bravo (4). You can verify this by inspecting "Active Pages" under Google Analytics with and without the bug, and comparing it to the page that is actually open in the browser.
If you are really not sure, I'd recommend putting together a private site with a new tracking ID, and testing the effect in isolation. It's going to be very hard to make sense of a general effect using measurement of real traffic patterns.
I hope that makes sense.

Thanks again for the explanation. I will trust you on this.


Feed fix

This pull request removes unneeded iTunes portions from the main feed as they can cause issues in podcast posts like https://github.com/StackExchange/stack-blog/commit/120d184653d20de39506deaab6bc6ab0a2bdcb51 (this was resolved in the podcast feed, but can still occur here in the main feed--I decided to remove rather than fix since this one isn't for podcast readers, so it doesn't need those extra tags).
It also removes the outdated category/podcasts/feed/index.xml file since that is redirected at the server level, and could cause confusion in the future.

Defunct.


Can't insert a Tab

Platform

OS: iOS
Device: iPhoneSimulator

Describe the bug
Trying to adapt the SillyBottomTabsPage in the SillyCompany demo, I ran into this problem, which seems to be a problem with the base Sharpnado Tabs functionality, not the SillyCompany code. I could be wrong about that of course.
I created an AddTab(...) method in the view holding the tabs and the views and the switcher, whose body is basically:
     TabHost.Tabs.Add(tabItem);
     Switcher.Children.Add(view);

And this seems to work well.
However I tried an InsertTab(...) variation, which basically did the same, only like this:
     TabHost.Tabs.Insert(index, tabItem);
     Switcher.Children.Insert(index, view);

But when I did this, it inserted the icons properly but had the labels in the wrong place.
It just added the label on the end instead of inserting it where it should have gone.
To Reproduce
Steps to reproduce the behavior:

use the SillyBottomTabsPage in the SillyCompany demo
create a new BottomTabItem and a new View
Use Insert(...) to insert the BottomTabItem to TabHost and the View to the Switcher, each using the same index, making sure the index is somewhere before the last items already in both those arrays.

Should work now with version 2.0!


fix typos in CONTRIBUTING.md

My contribution
-Check if the your issue exits or not if it exits
+Check if your issue exist or not. If it exist

Thanks, but you found a typo in that file but didn't read it? I don't know how this works😅

yes you're right. What i was thinking i should open an issue first

yes you're right. What i was thinking i should open an issue first

great!


Standing Inside Door Causes Damage

I first noticed this with MalisisDoors' High-Tech Doors. While on a space station with an oxygen vent supplying oxygen to a room, if I stand next to a closed high tech door or inside an open one, i'll take suffocation damage.
I've also noticed this happening with the Small Airlock Doors from AdvancedRocketry. If the door is placed in a way to put the door on the edge farthest away from the room with oxygen, if i stand in the doorway itself with the door closed, i take suffocation damage.

Part of that is due to the fact that AR's atmosphere's stop whenever it encounters a solid block like a door.  I'm not really sure how to fix this without changing the logic for seal checking.  So a fix may take a bit of time.

Would changing the model for the door to be solid on two parallel sides be a  workable fix?
Changing the accessibility rather then the seal checking.

I could do that, but the airlock door is not the only block with this problem.  So I'd just have a solution for a single block.  A Universal solution would be much better.

AR's atmosphere's stop whenever it encounters a solid block like a door.

Would it not be possible to simply add +1 range to the check? To include the immediate surrounding blocks?
I'm still learning basics of coding so feel free to ignore my suggestion.

I'd have to do something similar to that, but the reason that's difficult is the algorithm does not allow air to occupy the same space as solid blocks and uses whether or not a block is solid to halt its propagation.  If I make it just +1, then I have to also make sure it doesn't try to propagate out the other side of any 1 block thick wall.

might also do something on the player side, looking for air in adjacent blocks before suffocation starts.

That could work, but it would have to be on the server side (suffocation is calculated only on the server to prevent cheating) and the additional checks every tick would increase load slightly.  I'll consider it as an option though

bug is still present...
well, what about just prevent any player inside a sealed door from taking damage? Thats a very bad fix indeed, but we are able to walk inbetween spacestations and don't get hurt constantly....
greetings
cad435

Is this still a issue? Please comment if it is and i will reopen if if you are unable? We are closing old issues to see what is still relevant and needs to be fixed.


bugfix XamlParseException on Windows 7

Please allow this in! This fixes a bug that's causing one of my programs to crash for some users.


Strange pullsync/reserve growth

Context
2.0.0
Summary
Starting a new node into the sepolia testnet.   The reserve seems to grow on a non-linear curve.  I would expect it to climb directly up to the current swarm reserve level and then flatten, not asymptotically approach that level.

Expected behavior
Expected a steady climb at a constant pull rate until all reserve has been filled.
Actual behavior
See the graph above, it doesn't make sense to me.
Steps to reproduce
The sepolia testnet is in a unique position right now to see this.   There is no data going into the swarm, and the current radius is still zero so every single node in the swarm should have the full contents.   Completely pullsyncing from any single node should completely fill the reserve of the new node.   This just doesn't seem to be happening.
Any issue with pullsync like that being seen in a radius zero swarm could explain the mismatched reserve contents being indicated by the differing hashes being revealed in the mainnet Schelling game.
Possible solution
Don't have one yet, sorry.   But fire up a brand new sepolia testnet node and watch your own reserve metrics to see if you see the same thing I'm seeing.

It also seems kind of pointless for a node that is freshly pullsyncing to be offering all of the "new" chunks to the other peers, only to have them (as expected) not wanted since they already have them.   The yellow lines are the new node.

And no, the host is not CPU, Disk, Memory, nor network bandwidth bound.

Every node in the testnet swarm should have all of the chunks, but will the new node ever get the final few?  Film at 11.

It did finally finish and get up to the swarm's reserve level, but that last 15 minutes was agonizing to watch.

And just to complete the picture, here's the final reserve growth graph.   I'm working to fire up another new node and may enhance the logging in the pullsync area to maybe get a better idea of what is happening.

Is there possibly an off-by-one error in there somewhere?   Or something that can cause individual chunks to be skipped over or not offered?  I know every node may have the chunks in different orders and/or in different bins, but something is definitely not working as I would expect.

I just discovered the bee_puller_synced_chunks of historical vs live, so here are those graphs:

And zooming in on the final 15 minutes shows that they were historical pulls that delivered the final needed chunks.

I may understand a bit about the nature of the reserve growth.   Bins are pulled in parallel from a single peer, and all peers are started at the same time.   Higher numbered bins have fewer chunks in them, so they finish first.   Lower numbered bins have more chunks (fewer bits to match), so they take longer to finish.   As bin/peer pullers finish, the pull rate reduces and the number of chunks arriving into the reserve also is reduced.
But that understanding doesn't explain the long delay in getting those final few chunks.   Hopefully my increased logging in the pullsync/puller area will shed some light on it when this peer (a nuked copy of the one that is described above) finishes it's pull.   Currently at 800K out of a 1.3M target and beginning to taper off in the growth rate.


MSRV

Internally we run nsncd on Debian Stretch, which has rustc 1.41, until Stretch LTS EOL mid-next year.
Our tests and benchmarks don't work on 1.41, though the program itself works:
# cargo +1.41.0 test
   Compiling nsncd v1.2.0 (/root/nsncd)
error[E0277]: can't compare `&[u8]` with `std::vec::Vec<u8>`
   --> src/protocol.rs:216:9
    |
216 |         assert_eq!(header.as_slice(), expected);
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ no implementation for `&[u8] == std::vec::Vec<u8>`
    |
    = help: the trait `std::cmp::PartialEq<std::vec::Vec<u8>>` is not implemented for `&[u8]`
    = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

# cargo +1.41.0 bench
   Compiling criterion v0.3.5
error[E0658]: use of unstable library feature 'matches_macro'
   --> /root/.cargo/registry/src/github.com-1ecc6299db9ec823/criterion-0.3.5/src/stats/univariate/outliers/tukey.rs:227:9
    |
227 |         matches!(*self, HighMild | HighSevere)
    |         ^^^^^^^
    |
    = note: for more information, see https://github.com/rust-lang/rust/issues/65721

Should we try to keep at least tests working on 1.41.0? If so, let's adjust our CI to handle that, too.

This was fixed by #30 I think, but not closed out.

I think this ticket was about the fact that test/bench didn't work on 1.41 but the binary did.
Since #36 the binary doesn't work either (some dependencies have been upgraded to use edition 2021). While we've dropped Stretch internally, we're still using Buster (oldstable), whose packaged rustc seems to also be 1.41. So we should downgrade some dependencies, and maybe also reopen this if we feel like it's important for tests to run on oldstable.

Ugh, let me know if you want to revert #36, but hopefully you get CI working.

Sorted out in #46. For the question of tests, it doesn't seem terribly easy - Criterion's MSRV policy is "Criterion.rs supports the last three stable minor releases of Rust. ... Future versions of Criterion.rs may break support for such old versions, and this will not be considered a breaking change." And it brings in dependencies that in turn seem to not treat MSRV bumps as breaking. So let's leave this closed as WONTFIX.


README should use npm install ui-select

The issues forum is NOT for support requests. It is for bugs and feature requests only.
Please read https://github.com/angular-ui/ui-select/blob/master/CONTRIBUTING.md and search
existing issues (both open and closed) prior to opening any new issue and ensure you follow the instructions therein.
Bug description:
If you follow the readme, running npm install angular-ui-select, you get some older, unmaintained npm repo which is not compatible with angular 1.5+. Most newcomers will naively run npm install angular && npm install angular-sanitize && npm install angular-ui-select, which results in version incompatibilities.
Link to minimally-working plunker that reproduces the issue:
plunkr
Version of Angular, UI-Select, and Bootstrap/Select2/Selectize CSS
Running naive install with npm gives:
Angular: 1.5.5
UI-Select: 0.12.1
Bootstrap/Select2/Selectize CSS (if applicable): N/A

Whoops, I must have copied the bower one when I updated the README, can you submit a PR?


Enable Travis CI builds for the iOS version of AffdexMe

This should help with pull requests and give users a starting place for continuous integration with their apps.
I had to add a shared scheme to make this work because xctool, like xcodebuild, isn't able to automatically create schemes.
For example output see here:
https://travis-ci.org/j4y/ios-sdk-samples

That's correct Jay. Schemes need to be set to shared so that they can be committed into source control and seen by the builder. Are we going to be building AffdexMe on Travis now?

I think that we'll continue building the AffdexMe that we ship on our private Jenkins instance because we need to inject a license file.
The motivation behind also building it with Travis CI is that it provides a good example for getting continuous integration going for other IOS SDK app developers.  It also will run builds for pull requests on Github and show a green check mark or a red X depending on the results.
It has already been added for the Android AffdexMe app:
https://travis-ci.org/Affectiva/affdexme-android/builds/111343030

It has already been added for the Android AffdexMe app:

...and it found some legit bugs in our new build process.  Definitely worthwhile.

Does Travis support OS X as a build platform?
Merçi.
Boisy Pitre | Mobile Visionary
(781) 996-3037 x201 | Affectiva | affdex.com http://www.affdex.com/
https://www.facebook.com/Affectiva https://twitter.com/Affectiva http://www.affdex.com/buzz/blog/ https://www.linkedin.com/company/affectiva_2 https://plus.google.com/115885368219797754125/posts https://www.youtube.com/user/affectiva

On Feb 26, 2016, at 7:20 PM, toby cabot<EMAIL_ADDRESS>wrote:
It has already been added for the Android AffdexMe app:
...and it found some legit bugs in our new build process. Definitely worthwhile.
—
Reply to this email directly or view it on GitHub https://github.com/Affectiva/affdexme-ios/pull/3#issuecomment-189546250.

Does Travis support OS X as a build platform?

Mais oui: https://docs.travis-ci.com/user/osx-ci-environment/

Sacre bleu!
Merçi.
Boisy Pitre | Mobile Visionary
(781) 996-3037 x201 | Affectiva | affdex.com http://www.affdex.com/
https://www.facebook.com/Affectiva https://twitter.com/Affectiva http://www.affdex.com/buzz/blog/ https://www.linkedin.com/company/affectiva_2 https://plus.google.com/115885368219797754125/posts https://www.youtube.com/user/affectiva

On Feb 26, 2016, at 7:27 PM, toby cabot<EMAIL_ADDRESS>wrote:
Does Travis support OS X as a build platform?
Mais oui: https://docs.travis-ci.com/user/osx-ci-environment/ https://docs.travis-ci.com/user/osx-ci-environment/
—
Reply to this email directly or view it on GitHub https://github.com/Affectiva/affdexme-ios/pull/3#issuecomment-189547165.

merged manually


Utilizing Kademlia for Faster Block Propagation

I have an idea for improving the throughput and latency of propagation of blocks and transactions throughout the network. As I understand this has a large effect on the overall performance of the system. I research compilers rather than distributed systems at MIT (though I do have some experience building blockchain systems) so please let me know if something I say is completely off.
As I understand it, right now Cardano uses Kademlia for establishing a peer to peer network with a small diameter. However to broadcast a block, a node must send an INV message to all peers to inform them that there is a new block and then if a peer has not already heard about this block from another peer it must send a REQ message to this node. Finally, this REQ message is answered with the larger DATA message. These INV and REQ messages are relatively cheap, but there are a large number of INV messages sent (# of peers per node * # of nodes). For a 1M node network with K=7, this means that each node receives approximately 140 INV messages per block.
In the following scheme, INV messages can be removed entirely and the minimum number of DATA messages will still be sent. This also means that each propagation step skips a network round trip for the INV/REQ part of the protocol. More importantly, it also ensures that every node sends a similar number of DATA messages, which keeps the propagation tree balanced. This is not true if you send INV messages to all of your peers and allow them to select the first node they hear from.
This scheme is much clearer when considering a DHT implementation with a circular hash space such as Chord (which Kademlia is based off of), but I believe this extends to Kademlia's XOR distance metric as well since it satisfies the triangle inequality.
Basically the idea is to propagate the block according to the tree rooted at the origin of the block or transaction. In effect this is using the ability for Kademlia to achieve a log(n) table lookup to instead broadcast information with log(n) latency. The root node sends messages to its F (fan-out) peers of farthest distance that are in different k-buckets. Each of these messages include the block or transaction as well as a distance (relative to the root node). The peer that receives this message is responsible for distributing the message to all nodes that are further than itself from the root node, but closer than the distance passed. This peer will tend to have more nodes closer to itself (due to the k-bucketing) and therefore can fan out this message among these nodes.
As each node is given a distance range to fan out to that is separate from any other node's range, it ensures that no duplicate DATA messages are sent. The closest peer to the root node must also be instructed to propagate to all nodes between itself and the root node. As each node is guaranteed to know at least one node in each of its subtrees, each node is guaranteed to eventually receive the message (just like how in Kademlia every table lookup is guaranteed to eventually succeed).
For an example, I will consider a consistent hashing space like Chord since it is more easily understood. Consider a hash space from 0-99 with nodes numbered as such. I am assuming that the space is full of 100 nodes (though this extends to when it is not full). If node 0 creates a new block, it sends the following messages to the following nodes:
To node 50: Send this message to nodes 51-99
To node 25: Send this message to nodes 26-49
To node 12: Send this message to nodes 13-24
To node 6: Send this message to nodes 7-11
To node 3: Send this message to nodes 4-5
To node 1: Send this message to node 2
Node 50 then repeats this procedure, sending to nodes 75, 62, 56, 53, 51. Node 25 sends to nodes 37, 31, 28, 26. And this continues until all nodes have received the message.
There are some additional complexities here when you consider malicious peers that are incentivized to not propagate your message. However, I believe that if good peers were to randomly check if peers downstream of a potential malicious peer eventually heard their message, you could then track backwards to determine which node failed to propagate the network and then ban this node from the network. Constructing a scheme like this might actually ensure stronger guarantees than the current approach since in the current network a malicious peer that only transmits its own mined blocks can not be discovered. Of course before implementation, it would be necessary to formally prove what these guarantees are, but I wanted to post this proposal here to raise some initial discussion and get feedback on it.
Some questions:

Is something like this already being done in the blockchain space?
Does this idea generally sound feasible / is there something that I am missing?
If this idea was to be implemented into Cardano what steps would need to be taken?
Will the improvements I mention above result in a noticeable change in performance or is there another bottleneck in Cardano?

Is there a better place to post this suggestion? Would appreciate any ideas from those who are more familiar with how the cardano development effort is structured.

At the moment Cardano is running in a federated setting and is not utilizing Kademlia . New network related work for Cardano's decentralized release is going on over at https://github.com/input-output-hk/ouroboros-network . Work on peer-discovery hasn't been pushed but there are no plans to use Kademlia for it.

Ok, thank you for the information so I assume that this document https://cardanodocs.com/technical/protocols/p2p/ is out of date then? Do you have a link where I could read more on the proposed peer discovery method? As long as peer discovery is still done in a structured method then I think an adapted form of this scheme should still work.

Yes, the document at https://cardanodocs.com/technical/protocols/p2p/ is out dated. The peer discovery method hasn't been made public yet.

Ok, thanks for letting me know. Once the protocol is made public do you mind linking it here (also do you know approximately when I should expect it to be made public)? I do feel like there is a pretty significant performance benefit to structuring the network as I suggested above, but would like to compare to whatever peer discovery mechanism ends up getting used.

I will make a post here when it is pushed.


I can't reproduce the result for rnnoise Pesq 2.9

I run your github code with following command:
python test.py --clean_wav_folder=clean_testset_wav/ --noisy_wav_folder=noisy_testset_wav/ --tflite_path=rnnoise_INT8.tflite
The document said Avg pesq is 2.9  but I on get 2.664

I download test dataset with provided script file.

Another question is why the final_denoised_audio is append with previous denoise result?  It caused pesq calculation with different shape and accumlate.

Hmm strange I have just re-ran the scripts and i get the following output:

pesq score for audio before de-noising: 1.9782992604288083
INFO: Created TensorFlow Lite XNNPACK delegate for CPU.
pesq score for audio after de-noising with ../rnnoise_INT8.tflite is: 2.9194513495685985

One issue I did run into is the requirements.txt file no longer works correctly so I can post a new one that should work.
For your question on appending, RNNoise runs only on a small number of samples from the audio clip (480 if I remember correctly). You need to run the model many times for one audio file to get the full denoised audio out. Therefore we append after each inference until we have the complete denoised audio.
We then calculate the pesq score for that audio clip and add it to the running total before averaging at the end.

Using Python=3.7.16
The packages I tested it with working just now:

librosa==0.8.1
tensorflow==2.11.0
numpy==1.21.6
pesq==0.0.4
soundfile==0.12.1

thanks for your reply. I try it later. Maybe I confused is that why final_denoised_audio in the code doesn't  clear for each wav file. I think each clean_wav has their own final_denoised_audio.
![image](https://github.com/ARM-software/ML-zoo/assets/23479581/e880079d-f66f-4d0d-ae7b-55bef4986a1b

as the picture show. I think it should clear the final_denoised_audio for each iter in loop after pesq for this step when is calculated.

Oh yes that is a good point I understand now! I will check this, re-run and let you know thanks.

I rerun the script again. from clean vritual environment based on provided env and new git clone.

I got the very high result.
With command
python test.py --clean_wav_folder=/dataset/rnnwave/clean_testset_wav/ --noisy_wav_folder=/dataset/rnnwave/noisy_testset_wav/ --tflite_path=rnnoise_INT8.tflite

I found that the order of input wav file caused the different result pesq in the end. As example
I add sorted() in load wav function

It means load wave file in order based on file naming.
And I rerun the code.
It got different pesq result for each wav file

I print each iter avgpesq for check

After looking yes that was a bug and we should have been clearing the list after each wav file has been processed, thank you for spotting this!
I have re-run and get an average pesq of 2.465 which I think is more in line with the results in the RNNoise paper now at least.

Interesting about the order affecting your overall result though, I wouldn't have thought that should make a difference as we take an average. I will add sorted() and run myself again to check.
One thing I have spotted is the states are not currently reset to 0's between wav files, which should be happening and could affect things maybe.

With the fix moving final_denoised_audio to the correct place and properly resetting states to 0 before each inference I now get a consistent pesq of 2.43. Order of loading WAVs doesn't matter now.


Are mixed-integer constraints allowed inside a disjunct?

This isn't standard, but is currently allowed in the CHR function

This is relevant for nested disjunctions

See Sections 3.2-3.4 in http://public.tepper.cmu.edu/jnh/milpmodeling2.pdf

Yes. Allow them.


Transitive dependency on log4j

Hello!
This project is very solid work but I suggest you to mark dependency on slf4j-log4j12 to "provided" or even "test" scope in your sbt configuration. Thanks!

Well, interesting I can't actually recall why it's set as "compile" or even there...:)
Most likely just a copy/paste thing from some other project.
As it looks I'm not even using log4j in testing...should be removed completely.
Thx for notifying me.

Closing the issue as the issues has been solved and released in v1.1.1


Render problem

Hi!
I have installed open-wc successfully by following your instructions. I can see the initial page.
Next, I add in the same file (right below), the 01-basic-setup component. I can see the extra Hello world!. So far, so good!
Next, I add in the same file again, the 02-manage-properties component. Unfortunately I do not get the extra lines:
The message is: Hello world, count is: 0
The reversed message is: dlrow olleH

In general, any component more complicated than 01-basic-setup fails to get rendered in any way either as additional in the initial page or individually as a separate project.
Looking forward reading your advice.
Tia

It's a little unclear to me what your setup is, do you have a repo I could take a look at?

@thepassle Thanks for your interest and help. Please have a look at https://github.com/GHNewbiee/temporary. I have included only main files. webpack files (index.js and index.html) are missing since they do not play any important role, do they? I concluded in that, since the initial two components (open-wc-yo.js and open-wc-logo.js) are normally displayed despite of the content of those webpack files. Tia

You'll still need something to resolve your imports, you can use the owc-dev-server for this, instructions: https://open-wc.org/developing/owc-dev-server.html
And as described in: https://github.com/open-wc/open-wc/issues/244 as well. If you don't mind, i'll close this in favor of #244

@thepassle owc-dev-server is already installed. There is also "start": "owc-dev-server --open" at package.json scripts. As regards the other server options, I suspect that the default one is also correct since the initial two components (open-wc-yo.js and open-wc-logo.js) are normally displayed. What I just do is to add additional components in the end. Anyway, #244 instructions gave me a "working" env. Thanks for your help.


MEDS 0.3 Release Candidate

For the label schema, I think we need a bit more clarity. In particular,

Is prediction_time the time at which the prediction should be made? The last timestamp at which point data is allowed to be read in (and if so, inclusive or exclusive)? The inclusivity/exclusivity point we may need to add another column to specify.
Are we comfortable with this omitting the possibility of specifying tasks that occlude some parts of an event from the input window but not others? E.g., given everything up to a visit and the data of that visit that aren't meds, predict the meds prescribed.
Do we maybe want to replace "_value" with "_label"?

I assume there is no reason we don't want a TypedDict version of the split schema and will add one.

Is prediction_time the time at which the prediction should be made? The last timestamp at which point data is allowed to be read in (and if so, inclusive or exclusive)?

Yep. The idea is that it's legal to use all features up to and including the timepoint of the prediction time (inclusive). In my experience inclusive makes things a bit easier.
The documentation for this could perhaps be improved.

The inclusivity/exclusivity point we may need to add another column to specify.

I think that's unnecessary complexity at this point.

Are we comfortable with this omitting the possibility of specifying tasks that occlude some parts of an event from the input window but not others? E.g., given everything up to a visit and the data of that visit that aren't meds, predict the meds prescribed.

I think we should avoid trying to cover those sorts of complex labels with this schema.

Do we maybe want to replace "_value" with "_label"?

I don't think that's a good idea. I think that would lead to confusion since then the word "label" would be overloaded a bit. I think we want "label" to be the pair of a value + prediction_time.

I have a slight concern about omitting an explicit quantifier for inclusive or exclusive. If we don't allow people to specify it, and someone wants to use that, the next default would be to add or subtract a small amount from the prediction time, and that feels brittle to me if timestamps are binned or the granularity is reduced at all for a dataset.
I also think there are reasonable settings where inclusivity or exclusivity matters. E.g., if you are trying to predict something about a patient at a particular time without observing that value, then the natural way to express that is to say that you have an exclusive prediction time of the time at which you want the thing predicted, which enables models to leverage both the observations up to that prediction time without including the answer and the knowledge of when the target value that is being predicted is projected to happen -- this is much harder to express without a notion of exclusivity.

I think the simplicity gains of only having one type of label outweigh the minor annoyance of people having to subtract 1 microsecond every now and then for labels that need exclusivity.

I have a slight concern about omitting an explicit quantifier for inclusive or exclusive. If we don't allow people to specify it, and someone wants to use that, the next default would be to add or subtract a small amount from the prediction time, and that feels brittle to me if timestamps are binned or the granularity is reduced at all for a dataset.

Our datasets have a specified granularity of "[us]" so brittleness shouldn't be a problem. Subtracting one microsecond is always correct.
Exclusivity does matter, but I think it's easier to just let labelers subtract the microsecond when they need to rather than force a lot more complexity throughout the entire setup.

I suspect we'll eventually want to revisit this but for now I'm fne dropping the issue of label exclusivity. WIth the documentation changes in #35, I'm happy with this to go in.


Add Google.Cloud.ServiceDirectory.V1Beta1

We'll need to provide a product URL before releasing the first package, but this gets most of the way there.

Ah, no - it was a failure to rerun generateprojects. Repushing; will merge on green.


Prevent require executing some scripts (fixes brew typecheck --update)

Loading test.rb will execute code that will terminate processes at the end of the file. This also terminated Sorbet during a typecheck update.
These files are designed to never be loaded via require, so this PR prevents that from happening.

Review period skipped due to critical label.


Apply sorting to overview outputs

I do deployments containing more than 30 resources. With this amount, the output of the overviews is not easily comparable between runs so these don't serve a whole lot.
In my opinion, the following outputs should by alphabetically sorted:

Discovering templates
Checking initial resource statuses
Deploying resources
Result: Successful resources

I'll do this just as soon as https://github.com/Shopify/kubernetes-deploy/pull/424 is merged, because they will conflict.


Rewrite in hyprland/window

Is it possible to rewrite by using properties such as class or initialTitle obtained by hyprctl activewindow?

this would be awesome! without initialTitle i can't filter for spotify 😭


How to Start!!

Hello,
First of all this is one of the greatest products out there...
We are developing a custom Dashboard (Dashboard using a single webresource),
and we need to add grids and graphics with the same look and feel of CRM 2013.
We download your solution, the examples are amazing. The thing is.... were can we start building our one html grid?
we are lost
Can you please help us getting started?

Thanks for the encouragement. I quick start on how to create a simple grid is work in progress - in the meantime, the best place to look at is the Quote Editor Example.

Check out the tutorial videos!


A StackFrame.hint 'subtle' is ignored if source attribute is missing

VS Code Insider 1.14
Setting StackFrame.hint to subtle has no effect if the stackframe has a falsy source attribute.
The hint is a pure presentation hint that should not be coupled with any other attribute.

fyi @roblourens @gregg-miskelly we are now handling the presentationHint exacty as specified in the debugProtocol.d.ts. That is a presentationHint of subtle, label and normal will only be respected on StackFrame. And the presneationHint of emphesize and deemphesize will only be respected on Source.


chore(telemetry): sprinkle a few telemetry spans into connlib

DNS resolution is a critical part of connlib. If it is slow for whatever reason, users will notice this. To make sure we notice as well, we add telemetry spans to the client's and gateway's DNS resolution. For the client, this applies to all DNS queries that we forward to the upstream servers. For the gateway, this applies to all DNS resources.
In addition to those IO operations, we also instrument the match_resource_linear function. This function operations in O(n) of all defined DNS resources. It should be fast enough to not create an impact but it can't hurt to measure this regardless.
Lastly, we also instrument refresh_translations on the gateway. Refreshing the DNS resolution of a DNS resource should really only happen, when the previous IP addresses become stale yet the user is still trying to send traffic to them. We don't actually have any data on how often that happens. By instrumenting it, we can gather some of this data.

Draft because I still need to do some performance measurements but conceptually ready for review @conectado @jamilbk.


Manifest in release v0.0.5 contains v0.0.4 image tag

The v0.0.5 release contains the wrong image tag inside the manifests/dragonfly-operator.yaml file.
When viewing the file as in tag v0.0.5 (https://github.com/dragonflydb/dragonfly-operator/blob/v0.0.5/manifests/dragonfly-operator.yaml#L463) the following shows: https://github.com/dragonflydb/dragonfly-operator/blob/f4cc58c3d7a7bd3d84d021e615ec0041c886ecd1/manifests/dragonfly-operator.yaml#L463

Hey @MarnixBouhuis, Thanks for raising the issue.
Actually, The plan for manifests was to have the latest image manifests available that folks could use from the repo, and not tie it per commit (like you are expecting). This is why we also didn't release these as part of the release Github artifacts.
But, I agree with your expectation here and I will make it so that we update the manifests as a release process step from the next release.

So, We strarted following this pattern and the last two releases (atleast) have the matching versions in the files. So, Closing this.


Authentication secret annotations leak tekton abstractions

https://github.com/qu1queee/build/blob/4d47b776da6cb1c6c6f8080d5f2d2c1ec543ef96/docs/development/authentication.md#authentication-for-git
For providing git/registry authentication credentials, a user is asked to provide the authentication information in a Secret
apiVersion: v1
kind: Secret
metadata:
  name: secret-git-ssh-auth
  annotations:
    tekton.dev/git-0: github.com
    tekton.dev/git-1: gitlab.com
type: kubernetes.io/ssh-auth
data:
  ssh-privatekey: <base64 <~/.ssh/id_rsa>

Having to specify annotations with "tekton.dev" is a leakage of abstraction.
Perhaps,

The user should be able to omit the annotations, and
The BuildRun controller should generate a temporary copy of the secret with whatever tekton-specific decoration is needed.
After the build is complete, the secret could be deleted?  ( similar to sas created on the fly )

The reason I propose the generation of a temporary secret instead of annotating the existing one is that the user may not expect her Secret to be decorated with annotations, let alone understand where it came from.

This isn't really a bug per se - feel free to re-label, but to a user the ask for annotations might be odd  ;)


Feature: ASB-2081 - Withdrawn cases - project stage update

Describe your changes
https://pins-ds.atlassian.net/browse/ASB-2081?atlOrigin=eyJpIjoiNTcwYTMwOTVmOGY1NGNiZjk1OTYzZWU0NDE4ZTA1MDUiLCJwIjoiaiJ9
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[ ] I have made corresponding changes to the documentation
[ ] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[ ] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

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

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

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