added missing spaces

PR #7680 made think I could use some regexes to find a few more missing spaces

Thank you @rpetrusha for your thorough review. I've addressed your comments with the exception of the IIS topic with the reason stated earlier.


pre-allocate spaces

When I try moving a window to a space that currently doesn't exist I get the error below:
could not locate space with mission-control index '8'

is there a way to somehow pre-allocate windows 1 to 9?

You can use the query system to check for existing spaces during startup, and if your desired setup is not present, create new spaces until it is. A user did this and posted his script in #1352


Goal: Asset Pipeline

We currently store assets in a optimized format KTX + compression. This is only optimal for specific platforms. On platforms which don't support these optimizations, it can take several minutes to load textures. Textures may also loose quality if using lossy compression
Being compressed and wrapped, assets are also much harder to work with and change. We cant simple swap a texture out or replace textures.
One solution too this is taking components and building a small executable which loads all available assets and performs specific optimizations for specific platforms. This can all be triaged by CMake at compile time.

Search assets for a assets.json file
Optimize assets according to this files requirements per platform
Cache sha256 of each file processed so that we do not rerun on each asset every build
Save to a folder known by the build system
Have assets which load as fast as possibly for a given platform
VKB_OPTIMIZE_ASSETS

might look something like this

{
    "configurations": {
        "android-default": {
            "*.png": {
                "type": "image",
                "compression-priority": ["ASTC"],
                "mipmaps": true,
                "archive": "KTX"
            },
            "*.gltf": {
                "type": "model",
                "optimize-images": true,
                "generate": ["NORMALS", "TANGENTS", "BITANGENTS", "AABB"]
            }
        },
        "windows-default": {
            "*.png": {
                "type": "image",
                "compression-priority": ["ASTC"],
                "mipmaps": true,
                "archive": "KTX"
            },
            "*.gltf": {
                "type": "model",
                "optimize-images": true,
                "generate": ["NORMALS", "TANGENTS", "BITANGENTS", "AABB"]
            }
        }
    },
    "files": {
        "scenes/sponza/Sponza01.gltf": {
            "android": "android-default",
            "windows": "windows-default"
        }
    }
}

compression-priority could potentially check for ASTC device support before optimizing (if running on the host)

Do we still want to do this? Is there a use case where this is required? If not, maybe we should just close it.


add jupyter notebook

--- adds a jupyter notebook

-- removed notebook

--- adds has_coldkey and other key file functions to the wallet
--- adds checks to the metagraph to ensure it does not fail without a wallet
--- kills the process in the executor gracefully using self.wallet.assert_coldkey ...


Add: Information about image registries, artifact management and rela…

Hello Ivan,
after our insightful discussion yesterday on container runtimes, I'm excited to share with you information about various registry projects. I've compiled some details from your image tooling chapter, so please excuse any duplications.
Life with happiness and yesterday was the happy earth day...
Peter

What a great contribution! Thanks a lot, @rossbachp! Merging momentarily 🚀

Many thanks!

I believe it was beneficial to include the new chapter in the README.md index:)

Great idea! The ./DISTRIBUTIONS.md link is now included in the README file.


[DO-NOT-MERGE][DSTREAM] Fatal error (e.g. OOM) during generating streaming jobs should fail the streaming context

What changes were proposed in this pull request?

Currently Spark streaming uses Scala.Try to catch NonFatal exception during generating streaming jobs in JobGenerator.generateJobs. But if any fatal error (e.g., OutOfMemoryError) happens, it won't be caught. Normally just error should propagate up and finally crash Spark runtime, but Spark streaming does this in separate thread.
If the fatal error crashes entire JVM, it also can bring Spark down. But for some customer cases, the OOM is thrown by ByteArrayOutputStream when it cannot grow up to new capacity (new capacity is overflow on Int range, see Java source code). So it is not actually running memory in JVM and JVM won't be down by that. Eventually the OOM is ignored by Spark streaming and next batches are continuing to execute. Such behavior causes unnoticed data loss for the streaming application. We should propagate all failures including fatal ones.
Why are the changes needed?

See above.
Does this PR introduce any user-facing change?

If user's streaming application contains fatal error during streaming job generation, previously it will be ignored. Now it will bring down the application.
How was this patch tested?

Added unit test
Was this patch authored or co-authored using generative AI tooling?

No

The real thing is that I have no idea about DStream. I am not qualified to review this code change, honestly.

Maybe there is a huge misunderstanding in the community about me hence I've been pinged for DStream PRs.
When I started my contribution in Spark, DStream is already almost inactive (while we call stable) and my job was to make contributions with SS. I never had any reason to work for DStream. I just let the PR for DStream go, not only because we want to strongly deprecate DStream, but also because I have zero idea on the detail.

That's fine. I have no idea who otherwise should be pinged for DStream change. 😄
I think as the project is deprecated now, I'm just not sure if we can make any change to it. I think this should be correct and want to see if we have a chance to merge it and also open this for some visibility.


📦 Package managers for cooltt

We don't really need a package manager now, but it's fun to create trolling PRs (for good reasons).

Cool Package Manager UI Proposal v0.1.0
Initialization
coolpkg init --root=/path/of/root

Creating an empty anchor file as in https://github.com/RedPRL/cooltt/blob/main/test/cooltt-lib
Mounting
coolpkg mount --option "<JSON string as router arguments>" <router name> <mount point>

Adding a new entry to the anchor file
Unmounting
coolpkg umount <mount point>

Remove an entry from the anchor file
Listing
coolpkg list-mounted

List mounted libraries
Synchronization
coolpkg sync

Printing out the string 🤔 Why? Don't you know synchronization is automatic?


Manually create a routing so that Demo app can be viewed from the outside world

Describe the issue
This is important because our EA teams need to be live and reachable from the internet. We need to understand how to do this using the structure of LZ2.
Which Sprint Priority is this issue related to?
The 'Milestone' should clearly list Sprint Priorities - which one is this issue related to?
Additional context

There is a markdown file describing the required steps in the SEA GitHub repo.
consultation with AWS technical expert

Definition of done

Procedure documented in LZ Ops repo and
demo app reachable from the outside world
clarification  should include DNS config to allow access via a URL
clarification should include configuration of TLS for internal traffic path from perimeter network to app ALB

Included this in Sprint 15 as it is closely tied the dpeendent ticket and it doesn't make much sense to do that one without this one.

The documentation for exposing a public facing workload are available here: https://github.com/aws-samples/aws-secure-environment-accelerator/blob/master/docs/installation/guides/public-facing-workload-via-fortigate.md

PR#1 documents the steps in the CloudOps repo
Internal TLS is handled by the defualt self-signed certificated that is added to the internal ALB using terraform


Taking System credentials automatically during Login to CRM in IE (Private browser)

How to disable SSO. Even in IE Private window CRM is not asking for User Name and Password and directly takes the system credentials because of this we are unable to login to CRM with given credentials in app.config. We are able to login using CRM User credential in IE Private window if we do it manually only facing this issue when doing the same with selenium Automation.

I'm facing this issue too in Chrome. I've tried these BrowserOptions and it still automatically logs in as the user running the browser process, rather than the ones in Credentials:
private static readonly BrowserOptions BrowserOptions = new BrowserOptions
{
    BrowserType = BrowserType.Chrome,
    FireEvents = true,
    Headless = false,
    UserAgent = false,
    CleanSession = true,
    Credentials = new BrowserCredentials(BrowserDriverUserName, BrowserDriverPassword),
    PrivateMode = true,
    StartMaximized = true
};

I hoped that a combination of CleanSession, Credentials and PrivateMode would do it, but no luck.

Thank you @amars4u and @MDendura for providing feedback on this issue.
This is similar in nature to Issue #152 and we've observed it internally. I am not immediately aware of a fix outside of utilizing a non-domain computer accounts in the IE scenario.
I'll add an item to the backlog to review this scenario and see if we can determine both why this behavior is occurring and if we can implement a proper solution. In the past this had not been an issue with IE, but something in the last few months has changed behavior.
I'll update again once we have more information to share and/or if a fix has been identified.
Best Regards,
Tyler

@MDendura - Could you provide some details about which code branch and Dynamics org you are testing in addition to Chrome browser version and which Selenium WebDriver, Support driver, and Chrome driver you are using?
Thanks
Tyler

I've just done a quick manual test using InPrivate mode in Edge and I can log in with a different account as I'm prompted by both login.microsoftonline.com and my company's ADFS server 👍
An InPrivate session in IE11 behaves the same way as Chrome - I can enter my test account credentials for login.microsoftonline.com but the redirect to ADFS means that the user running the IE process is the one that gets logged in 👎
@markleavesley Signing out and then signing back in with IE results in the same undesirable behaviour, but thanks for the suggestion. It's the ADFS step that's causing me problems, I think.

@TYLEROL Here's the extra details you asked for:

EasyRepro version is v<IP_ADDRESS>, referenced from NuGet
The version string from my D365 org is "Version 1612 (<IP_ADDRESS>0) (DB <IP_ADDRESS>8) online"
The URL for the org is xrm90.crm11.dynamics.com
Selenium.Chrome.WebDriver is at v2.40.0
Selenium.WebDriver is at v3.13.1
Selenium.Support is at v3.13.1

@TYLEROL Is there any news about this issue?

Hey @MDendura - We have not had any success in identifying the root cause driving this behavior.
I tested again today, and the problem for me is now only affecting Internet Explorer 11 InPrivate Mode. I've listed each of my browsers below if you're able to do a comparison:
- Working

Chrome(v 69.0.3497.100)
Firefox (v 63.0.1)
Edge (v 44.17763.1.0) / (EdgeHTML v 18.17763)

- Not working

Internet Explorer 11 (v 11.55.17763.0) (Update v 11.0.47)

I'll reach out to members of our Internet Explorer team and see if I can get a more concrete determination as to why this issue may be occurring. It's not specific to EasyRepro, as I can replicate the problem launching InPrivate as a normal user.
The only workaround that we have right now is using non-domain user accounts / machines off the corporate network. Understandably these are not ideal.
Thanks,
Tyler

I have an opposite scenario. I want to reuse the cached profile to open chrome browser or reuse existing on e to not have to get authenticated which would additional windows pop ups that Selenium can't handle . Details is in https://github.com/microsoft/EasyRepro/issues/510#issue-448195713. The test sample always opens a new browser in private/incognito and auto enter the username and password defined in app.config thus it fails to handle a windows popup requiring enter Smard card . How can we not let it open a private window and use my cached chrome profile stored in Appdata\Local\Chrome?

Any solution for this? We are getting this issue in chrome and unable to use test user for testing automation. We have SSO enabled through OKTA for office 365 apps.

Any updates on this? I am testing it in Chrome, with Azure AD SSO and I get through the "Taking you to your organization's sign in page..." redirect, it goes to a blank screen and then pops a native browser dialog for credentials. At that point, the test bombs. If I put in creds, it passes through and authenticates correctly.  Seems like I should be able to code around that?

I'm getting similar issue on chrome version 80.0.3987.100. I have to login to Azure using test credentials, but selenium is using default user credentials to auto login in chrome.
If there were any solutions to this issue, could anyone share it

Im getting similar issues for the past few months at the moment. Hoping to hear back from the team as well.


Logo

Why does the right foot in the logo have 6 fingers?

it represents an anomaly

That's exactly what the Gopher is investigating :)

why is the Gopher angry?

Because there is an outage in progress.


Error when evaluating expression with v-if and v-for

{{ slot.name }}

        {{ wrestler.team }}

okay

        {{ wrestler.team }}

It throws a warning Error when evaluating expression "wrestler.team". The initial wrestler.team is shown and so is the word "okay", but the second wrestler.team throws that warning. Is this expected behavior?

Do you actually have a variable named true? Can't see that ending well.

No, true is just true.
The problem and I'm not sure if this is by design, but passing a  within a v-for on a  tag will cause the error to be thrown. Simply changing the v-if to a  tag or any other non- tag will not throw the error.

your template is incorrect

{{ slot.name }}

            {{ wrestler.team }}

okay

            {{ wrestler.team }}

the outer div is not enclosed

No, I just forgot to copy that ending div tag over.

You can't have paragraph tags nested like that. It could be the browser just handling it funny.

Really? I did not realize that. Thanks @crswll


Expose setup hook (closes #33)

This PR exposes a useful setup hook introduced to nixpkgs in https://github.com/NixOS/nixpkgs/commit/a85a699e40708b452d36c70725396ff67dc48e66 mentioned in #33.
This required updating the nixpkgs input from 23.05 to 23.11 and adding a meta section.

One more thing, this CI is trying to evaluate every package that exists in this repo, most of which end up failing due to old sources being unavailable. Can this be changed to only cover, say, the latest release and last 3 nightlies?

Note that there's few issues with the nixpkgs hook https://github.com/Cloudef/zig2nix/blob/master/flake.nix#L17-L21


Option to disable showing the info screen when alarm goes off

Add this option for only Android 5.0+ users.  If info screen disabled, a heads up notification will show when alarm goes off instead.  The options on the notification will be Open (to go to the info screen) and Dismiss.


PHP - highlight correct nav item

Once the user has navigated to a new page (i.e. by clicking Bennu Mapper link), the Home nav item is still highlighted.  The nav item that corresponds to the current page should instead be highlighted.

Already generating the nav links from the database but apparently forgot to comment accordingly. That fix is in api_build, currently

Fix merged into master on 27 Jun 2020


.build/node_modules symlink vs. npm

Hey,
I've just recently started trying out serverless framework and this plugin just today. So sorry in case things are a bit inaccurate :)
My problem is that, together with the typescript-plugin, the core's package plugin fails to properly exclude development dependencies from the build/package. This way even a simple "hello world" function has a footprint of roughly 15MB, where it should be just about 2 kB.
Problem seems to be, that package plugin just calls npm ls --dev=true ... and npm ls --prod=true ... and diffs the output of those two. With this plugin these shell calls are done from the .build folder, yet npm ls --dev=true ... fails to properly list all the dependencies as it stumbles over the symlinking.
Seems like it doesn't expect deduped/flattened dir structure then.
Replacing both symlinkSync calls by copySync immediately fixes the problem so :)
... yet I'm unsure if there's a better way to do that ...
regards

Could you elaborate a bit more on the implications of replacing symlinkSync with copySync?

So my current playground setup looks like this ...
excerpt from package.json:
{
  ...
  "devDependencies": {
    "@types/aws-lambda": "0.0.18",
    "@types/node": "^8.0.47",
    "aws-sdk": "^2.141.0",
    "serverless": "^1.23.0",
    "serverless-plugin-typescript": "^1.1.3"
  },
  "dependencies": {}
}

(so only dev dependencies, serverless installed locally)
With the replacement mentioned above in place, .build folder has a node_modules which is a real directory + package.json a copy of the file from the main folder.
Within .build folder npm ls just works as it should:
<EMAIL_ADDRESS>/home/stesie/Projekte/budgetier/.build
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>deduped
│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>deduped
│ │ └──<EMAIL_ADDRESS>deduped
│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├──<EMAIL_ADDRESS>deduped
│ │ │ └──<EMAIL_ADDRESS>deduped
│ │ ├──<EMAIL_ADDRESS>│ │ └─┬<EMAIL_ADDRESS>│ │   ├──<EMAIL_ADDRESS>│ │   └──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├──<EMAIL_ADDRESS>deduped
... and so on ...

yet without the change/fix in place both node_modules and package.json are symlinks to the parent's folders files.  In this case npm ls fails like this:
<EMAIL_ADDRESS>/home/stesie/Projekte/budgetier/.build
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>extraneous
├─┬<EMAIL_ADDRESS>│ ├── UNMET DEPENDENCY<EMAIL_ADDRESS>│ ├── UNMET DEPENDENCY<EMAIL_ADDRESS>│ ├── UNMET DEPENDENCY<EMAIL_ADDRESS>│ ├── UNMET DEPENDENCY<EMAIL_ADDRESS>│ ├── UNMET DEPENDENCY<EMAIL_ADDRESS>│ ├── UNMET DEPENDENCY<EMAIL_ADDRESS>│ ├── UNMET DEPENDENCY<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├── UNMET DEPENDENCY<EMAIL_ADDRESS>│ └── UNMET DEPENDENCY<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>extraneous
... again many more ...

Notice how it reports unmet dependencies with the actual dependencies + extraneous directories in node_modules folder. This seems to be due to some special handling in npm for symlinks...
Regarding the downsides of this change I think it's just about copying around lots of data instead of creating a two symlinks. Even in my simple case it's copying around 100 MBs on each build, be it invoke local or deploy. Yet with SSDs and the OS's file system cache it shouldn't hurt much. At least on my laptop it's negligible
In a way this would also "solve" issue #23 :-)

@pmuens could you share your view on this?

Maybe a good solution is to use copy when deploying and linking while building for development?

Also having issues with this.

I'm experiencing the exact same issue described by @stesie.
I came across a workaround in a different issue that fixes this for me (https://github.com/prisma/serverless-plugin-typescript/issues/89#issuecomment-398754969) – set NODE_PRESERVE_SYMLINKS=1 in your env.
I'd still love to see this get fixed though!

I have the same issue: the "excludeDevDependencies" feature in Serverless is not working, because npm ls cannot properly read the node_modules from the symlinked dir in the .build dir.
This is not a Windows issue; npm ls will give incorrect results on a unix box when run inside the .build dir as currently created by serverless-plugin-typescript.
This largely breaks excludeDevDependencies. I am getting most of my dev dependencies included in my final zip output. It seems that NODE_PRESERVE_SYMLINKS=1 does fix this though.
I'm not sure what the best fix is from serverless-plugin-typescript's point of view is. It seems to me that the way excludeDevDependencies is coded up in zipService.js in base serverless is a bit fragile. And fixing npm itself seems a bit out of scope for serverless-plugin-typescript. Some ideas:

Maybe serverless-plugin-typescript should just do a cp -r rather than a symlink for now? That would be a more expensive build than the current symlink approach
Maybe serverless-plugin-typescript should explicitly copy only prod dependencies? That would complicate things here and might lead to different problems later.
Maybe serverless-plugin-typescript can convince the dependency examination code in zipService.js to run in the "real" node_modules dir, then use the outputted exclusion paths against the .build node_modules dir later?

I think the title of this issue should be changed to be more like "excludeDevDependencies broken by .build/node_modules symlink vs. npm". This issue is hard to find at the moment.


Release-v0.12.0-rc.1

Yay release

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 5 files lost coverage.
Overall coverage increased (+0.1%) to 85.416%

Files with Coverage Reduction
New Missed Lines
%

src/connectors/impls/http/meta.rs
1
88.24%

src/connectors.rs
1
85.89%

src/connectors/sink.rs
1
83.23%

src/connectors/source.rs
1
80.55%

src/pipeline.rs
1
76.39%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
39347

Relevant Lines:
46065

💛  - Coveralls


Validation messages not appearing on UI

I just setup a test meteor project. I added autoform 6 and the new simpl-schema node package. I created a basic form with a few fields, they fields seem to be validated properly but I don't have any feedback on the ui. Is there something new that needs to be added with autoform 6 from previous versions?

I figured this out. I scanned through the docs a bit more. With the new node-simple-schema  package there is an extra configuration requirement needed when creating the schema. See here for specifics on enabling meteor reactivty.

One last thought, the meteor-autoform docs could be a bit clearer on this requirement, or perhaps have this as a default option.


Displaying banner with Maui Blazor

I got the interstitial to work inside a Maui Blazor application. However my banner ad is displaying a black bar. Is there anyway to display the banner ad without using XAML? How do you attach the MTAdView to your layout if you are using HTML?

You can add a banner programmatically.
Create the banner view in you c# code and attach it to a container like a
stacklayout or a grid or anything you like.
On Sun, 24 Jul 2022, 16:36 Jeroen, @.***> wrote:

I got the interstitial to work inside a Maui Blazor application. However
my banner ad is displaying a black bar. Is there anyway to display the
banner ad without using XAML? How do you attach the MTAdView to your layout
if you are using HTML?
—
Reply to this email directly, view it on GitHub
https://github.com/marcojak/MauiMTAdmob/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUIPIUSVHG5XFHD5XJB2RTVVVPGZANCNFSM54P2XTHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@marcojak Is this also possible with an HTML element?
I could add the banner to MainPage.xaml using XAML, but I want to be able to position the banner underneath my navigation bar
`

<StackLayout>
    <admob:MTAdView></admob:MTAdView>
    <BlazorWebView HostPage="wwwroot/index.html">
        <BlazorWebView.RootComponents>
            <RootComponent Selector="#app" ComponentType="{x:Type local:Main}" />
        </BlazorWebView.RootComponents>
    </BlazorWebView>
</StackLayout>

`

I've done a quick search and apparently Admob can only be used in an Android and iOS app using the official SDK (no support for web), so it's not possible in Blazor.
Adsense is not allowed in an app.
If you don't want to show the ads over the navigation bar, an option is to use a Grid and then build your website considering the position of the AdView.

@marcojak That should work too. Thanks for the tip!


Add yaseer-13

The PR is open for a while. Do the suggested changes to get it merged.
If you need any kind of help, do let us know.

Closing due to long inactivity. Feel free to reopen.


force file with bad state to be removed during checkout

Hi osmosis maintainer,  I had found the following issue in osmosis:
When removeUnknownFiles flag is specified during checkout stage, but file is in "bad state"
(for example symlink that points to itself, which can be created with ln -s link link), [ this is just an example though i had encountered probably because of some file system errors]
The thing is  osmosis fails to complete checkout, it tries to check  symlink istatus for examoke in boost::filesystem::symbolic_link_exists, and fails.
According to boost documentation this error state means that it was impossible
to determine file attributes and whether file actually exists or not. In this case it is better to dispose this file,as it cannot be used anyhow. (see http://www.boost.org/doc/libs/1_44_0/libs/filesystem/v3/doc/reference.html#status).

Thanks!

@eliran-stratoscale  cool thanks


Fix nonfatal but bad practice nits creeping in

[x] in console DRR: TypeError: Cannot read property 'creator' of undefined
[x] in console validateDOMNesting(...): <p> cannot appear as a descendant of <p> in BalanceDisplay
[x] It looks like there are several instances of 'styled-components' initialized in this application. This may cause dynamic styles not rendering properly, errors happening during rehydration process and makes your application bigger without a good reason.

seems to be mostly gone at this point


Rust GitHub Actions in actions-rs seem unmaintained

This is not a Rust crate, but tooling used by many Rust projects. Is this correct place to discuss and maybe take action on informing community about the issue?
The actions-rs GitHub Actions from GitHub (https://github.com/actions-rs) is used by many Rust projects.
However, the actions don't see much love, there's discussion about the maintenance status here: https://github.com/actions-rs/meta/issues/43
As these actions are not maintained a known vulnerabilities might start to pile up and things might start to break because GitHub is deprecating support for some thing (e.g. https://github.com/actions-rs/audit-check/issues/227).
Pinging @svartalf since he's the (only?) owner of the GitHub organization.

I think we could probably put together a much simpler action for people to use, which doesn't depend on e.g. Node.js in any way

Some time ago, I wrote https://github.com/actions-rust-lang/audit to move off actions-rs. It is written as a composite action with the main logic in Python instead of JavaScript. It supports maintaining issues and writes a workflow summary with the findings. It doesn't really provide more features than audit-check except for an explicit input argument to ignore IDs.
I am happy to talk more about it in case you find it of any interest.

This action now uses node16 and dependencies have been bumped so should be perfectly usable.
It's fairly simple in the end - can continue altearntives in discussions if there is really need.


Minour IP4 "isValid" or "construct fromString" issue

Strictly speaking, this one is still correct:
 isValid { _address.count == 4 && _address.all {|n| n is Num && n >= 0 && n <= 255 }}
But the way how the array is filled assumes all parts of the string are in decimal format:
construct fromString(s) {
    _address = s.split(".").map { |n| Num.fromString(n) }
  }

However an IP address like this: <IP_ADDRESS> is not decimal. The 3 parts starting with a 0 are supposed to be interpreted as octal numbers.
So strictly speaking you construct a "valid IP address" but it is not the one, a proper IP stack would construct.
No worries: that usage is extremely rare ... and there are (plenty?) IP stacks that do it wrong, too :P

A patch would be accepted, but right now I'm not sure when I'll ever find the time to finish that branch.


osbuild returns zero status code with an ill-formed manifest

See the following example for osbuild 12:
# cat manifest.json
{
  "osbuild": {
    "state": "awesome",
    "but": {
      "input-validation": 0
    }
  }
}

# osbuild manifest.json
tree id: None
output id: None

# echo $?
0

Yeah, I know, this is a bummer. Should be fixed when #219 is ready and goes in.

Fixed with #219 and #427


point cloud alignment

@ir0
Dear Iro,
I am having a problem, which I thought you could help me with.
I have used panoramas and their corresponding depth images to create some 3D point clouds. However the point cloud that is achieved it this way, is not having the same alignment as the point cloud that already exists in the dataset!! I probably need to use the pose information and perform some transformation. I did try to apply camera_rt_matrix to the point cloud formed using panoramas, and I played a bit around with camera pose infos and point cloud transformation, but I couldn't solve the problem.
Do you have any idea how I should map these two point clouds?

The problem is solved as explained in issue #16


Status Code based redirection

What version of Traefik are you using (traefik version)?
v1alpha1
What is your environment & configuration (arguments, toml...)?
defaultEntryPoints = ["http","https"]
[entryPoints]
  [entryPoints.http]
  address = ":80"
	[entryPoints.http.redirect]
	  entryPoint = "https"
  [entryPoints.https]
  address = ":443"
	[entryPoints.https.tls]
	  [[entryPoints.https.tls.certificates]]
	  CertFile = "**.crt"
	  KeyFile = "**.key"
[web]
address = ":8080"
ReadOnly = true

What did you do?
We have configured Traefik as Load Balancer and for https redirection
What did you expect to see?
I would like to implement a status code based redirection using Traefik. Similar to how it can be implemented in Nginx now. For instance, if  a entry point return 404/500 i would like to redirect it to our application's custom error page.
What did you see instead?
Could not find any documentation/ examples to do status code based redirection
If applicable, please paste the log output in debug mode (--debug switch)

Close in favor of  #1634 - Feel free to subscribe there for updates.


Unable to use presigned url created by assumed role token in credentials

Getting the following error:

Originally posted by @pritamrungta in https://github.com/aws-samples/sigv4a-signing-examples/issues/10#issuecomment-2162191721

@pritamrungta - which example are you trying to run?
I've just tested using the Python example and it works as expected using temporary credentials as long as the AWS_SESSION_TOKEN environment variable is set:
from sigv4a_sign import SigV4ASign
import requests

service = 's3'
region = '*'
method = 'GET'
url = 'https://mt1k8sfqb8iom.mrap.accesspoint.s3-global.amazonaws.com/sample_data.csv'

headers = SigV4ASign().get_headers_basic(service, region, method, url)
r = requests.get(url, headers=headers)
print(f'status_code: {r.status_code} \nobject text: {r.text}')

Hey @jhart0
I tried the above script with AWS_SESSION_TOKEN set in environment and still get the same error message.
Headers:
{'host': '***.mrap.accesspoint.s3-global.amazonaws.com', 'X-Amz-Security-Token': '***', 'X-Amz-Date': '20240620T042415Z', 'X-Amz-Region-Set': '*', 'x-amz-content-sha256': 'e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855', 'Authorization': 'AWS4-ECDSA-P256-SHA256 Credential=ASIA5VAA23GQCPWHIFME/20240620/s3/aws4_request, SignedHeaders=host;x-amz-content-sha256;x-amz-date;x-amz-region-set;x-amz-security-token, Signature=304502205d5b141c8bd5d91dc95f93f63841b03476025f9c39d1cb20b0cb56c4fa400b74022100c794cd4914bc210e527a8148882d341c28de784844753319383f03fbbc1b9104'}

Response:
status_code: 400
object text: <?xml version="1.0" encoding="UTF-8"?>
<Error>UnsupportedSignature<Message>The provided request is signed with an unsupported STS Token version or the signature version is not supported.</Message><RequestId>CB32HXXYSAFWDTDK</RequestId><HostId>pKzqb+Tr7qTRJ3hwEOgogmtDZFZ7RujiQV/1qh1YFeZif9x1+6xNXYozSfWqDTJmY+iWp+GkZgg=</HostId></Error>

Note: The assumed role has admin privileges and it works with plain aws cli commands.

Thanks for testing that @pritamrungta. Can you confirm what version of the awscrt & botocore packages you are using please? If it is not the latest, please try updating to the latest version and testing again.

Here are my versions (which seem to be almost the latest ones):
python = 3.12.3
awscrt = 0.20.11
botocore = 1.34.130

Okay, I have been able to recreate your issue. It is the sts endpoint being used to generate the temporary credentials.
See here for details: https://docs.aws.amazon.com/AmazonS3/latest/userguide/MultiRegionAccessPointRestrictions.html
To use SigV4A with temporary security credentials—for example, when using AWS Identity and Access Management (IAM) roles—you can request the temporary credentials from a Regional AWS Security Token Service (AWS STS) endpoint. If you request temporary credentials from the global AWS STS endpoint (sts.amazonaws.com), then you must first set the Region compatibility of session tokens for the global endpoint to be valid in all AWS Regions. For more information, see [Managing AWS STS in an AWS Region](https://docs.aws.amazon.com/IAM/latest/UserGuide/id_credentials_temp_enable-regions.html) in the IAM User Guide.

If I change the sts endpoint (https://docs.aws.amazon.com/IAM/latest/UserGuide/id_credentials_temp_enable-regions.html#sts-regions-manage-tokens) to use V1Token, I get the same error as you. When I change it back to V2Token, the error is resolved using the global sts endpoint.

I see. So, if I switch STS to generate V2Token for presigning s3-global multi-region access points, will it still work with plain s3 bucket presigning which currently uses Sig V4 using HMAC-SHA256?

Yes, but as with any account-level change you should test thoroughly in a development environment first.

Sure. Thanks for your help.
That note in the docs did go unnoticed by me 😅


Travis: test builds against PHP 7.4 & work around for PHPUnit 8

Travis: test builds against PHP 7.4
Nightly has become PHP 8.0 since PHP 7.4 has been branched, so to continue to also test against PHP 7.4, it needs to be added separately.
Refs:

https://twitter.com/nikita_ppv/status/1089839541828112384
https://twitter.com/nikita_ppv/status/1094897743594770433

Travis: work around PHPUnit 8.x on PHP >= 7.2 images
As of recently, the Travis images for PHP 7.2+ ship with PHPUnit 8.x.
The PHPCS native test framework is not compatible with PHPUnit 8.x and won't be able to be made compatible with it until the minimum PHP version would be raised to PHP 7.1.
So for the unit tests to be able to run on PHP 7.2+, we need to explicitly require PHPUnit 7.x for those builds.
As for nightly: there is no PHPUnit version which is currently compatible with PHP 8.
As that either means that the builds for nightly would always fail or - if the unit tests would be skipped -, the only check executed on nightly would be linting the files, I've elected to remove build testing against nightly for the time being.
For more details about PHPUnit vs PHPCS vs PHP 8, see squizlabs/PHP_CodeSniffer#2416

I've added one additional commit to this PR:

Travis: don't allow failures on PHP 7.3
As the minimum required PHPCS version is now PHPCS 3.3.1, builds against PHP 7.3 should no longer be allowed to fail.

Once this is merged can we release 0.2.0? All that is needed is a changelog I think.

@dingo-d If you think it's a good idea, of course we do a release.
Looking at the milestone though, there aren't really any functionality changes, so should the release be 0.2.0 or 0.1.1 ?

I'm fine with 0.1.1 if there aren't any new functionality added 🙂


Nested objects schemas are not initialized without explicitly calling defaultRealm

Goals
It is a simple issue (odd that it was not raised yet). Without explicitly calling [RLMRealm defaultRealm] nested objects schemas are not initialized and exception is thrown trying to initialize object with value.
Actual Results
https://github.com/viktorasl/Realm-RLMLinkingObjects/blob/master/Realm-RLMLinkingObjects/ViewController.m#L42
Let's say there is a class EventFooType which objects have declared properties of RLMObject subclass EventToken. When trying to initialize [EventFooType<EMAIL_ADDRESS>exception is thrown that EventToken is not persistent in realm.
I added a breakpoint in the - (RLMObjectSchema *)objectForKeyedSubscript:(__unsafe_unretained id<NSCopying> const)className method and printed out _objectSchemaByName so if [RLMRealm defaultRealm] was not explicitly called then schema is missing nested objects schemas:
{
    EventFooType = "EventFooType {\n\tidentifier {\n\t\ttype = int;\n\t\tobjectClassName = (null);\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = YES;\n\t\tisPrimary = YES;\n\t\toptional = NO;\n\t}\n\tname {\n\t\ttype = string;\n\t\tobjectClassName = (null);\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = YES;\n\t}\n\tfooIntProperty {\n\t\ttype = int;\n\t\tobjectClassName = (null);\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = NO;\n\t}\n\ttoken {\n\t\ttype = object;\n\t\tobjectClassName = EventToken;\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = YES;\n\t}\n}";
}

if it was explicitly called then:
{
    EventBarType = "EventBarType {\n\tidentifier {\n\t\ttype = int;\n\t\tobjectClassName = (null);\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = YES;\n\t\tisPrimary = YES;\n\t\toptional = NO;\n\t}\n\tbarIntProperty {\n\t\ttype = int;\n\t\tobjectClassName = (null);\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = NO;\n\t}\n\ttoken {\n\t\ttype = object;\n\t\tobjectClassName = EventToken;\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = YES;\n\t}\n}";
    EventFooType = "EventFooType {\n\tidentifier {\n\t\ttype = int;\n\t\tobjectClassName = (null);\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = YES;\n\t\tisPrimary = YES;\n\t\toptional = NO;\n\t}\n\tname {\n\t\ttype = string;\n\t\tobjectClassName = (null);\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = YES;\n\t}\n\tfooIntProperty {\n\t\ttype = int;\n\t\tobjectClassName = (null);\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = NO;\n\t}\n\ttoken {\n\t\ttype = object;\n\t\tobjectClassName = EventToken;\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = YES;\n\t}\n}";
    EventToken = "EventToken {\n\tidentifier {\n\t\ttype = int;\n\t\tobjectClassName = (null);\n\t\tlinkOriginPropertyName = (null);\n\t\tindexed = YES;\n\t\tisPrimary = YES;\n\t\toptional = NO;\n\t}\n\tfooEvents {\n\t\ttype = linking objects;\n\t\tobjectClassName = EventFooType;\n\t\tlinkOriginPropertyName = token;\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = NO;\n\t}\n\tbarEvents {\n\t\ttype = linking objects;\n\t\tobjectClassName = EventBarType;\n\t\tlinkOriginPropertyName = token;\n\t\tindexed = NO;\n\t\tisPrimary = NO;\n\t\toptional = NO;\n\t}\n}";
}

Version of Realm and Tooling
Realm version: 1.0.1
Xcode version: 7.3.1
iOS/OSX version: 9.0
Dependency manager + version: Cocoapods 1.0.1

Thanks a lot for the feedback @viktorasl! We probably could handle this sort of use case a bit more nicely. I'll assign this to our backlog so we can consider a way to avoid causing an exception.


Product Name Typo Fix in Docs

Description

closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/).
[ ] Added relevant godoc comments.
[ ] Provide a conventional commit message to follow the repository standards.
[ ] Include a descriptive changelog entry when appropriate. This may be left to the discretion of the PR reviewers. (e.g. chores should be omitted from changelog)
[ ] Re-reviewed Files changed in the GitHub PR explorer.
[ ] Review SonarCloud Report in the comment section below once CI passes.

Thanks, @zadkiel88. Can I mark as ready for review or you plan to push more changes?

Thank you, please leave it as a draft for now as I'll return and push more
changes later.
On Fri, Apr 26, 2024 at 10:12 PM Carlos Rodriguez @.***>
wrote:

Thanks, @zadkiel88 https://github.com/zadkiel88. Can I mark the PR as
ready for review or you plan to push more changes?
—
Reply to this email directly, view it on GitHub
https://github.com/cosmos/ibc-go/pull/6229#issuecomment-2079477143, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AZJG3EKT5L6C7PKWPJ6CE5LY7JOEJAVCNFSM6AAAAABG2LR34WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZZGQ3TOMJUGM
.


Scratchpad windows are not shown when pressing SUPER + STRG + Space

I have installed Ubuntu 22.04 LTS and regolith-desktop and ilia/unknown,now 0.11.6-jammy-1
Further on, the terminal WezTerm is also not shown even though it is not on the scratchpad.
Starting ilia -p windows via terminal creates this message: Gtk-Message: Failed to load module "xapp-gtk3-module"

Hi @golmanosocks , is WezTerm the only app that you notice this problem with?  The message Gtk-Message: Failed to load module "xapp-gtk3-module" should not have any functional impact on ilia.

Sorry, I just noticed that I wasn't really specify in my issue. The problem is as follows. Wezterm is not shown in the navigate window by name pane (CTRL+SUPER + Space) even though it is open (on WS1 and WS2). This issue is Wezterm specific. Further on, all applications that have been moved to the scratchpad (SUPER+CTRL+M) are not shown in the navigate window by name pane.

This will be fixed in Regolith 2.1


Change the name of eks-anywhere-test to hello-eks-anywhere

[x] create the repository
[x] mirror the hello-eks-anywhere repo maybe if we need to
[x] rename the folder in build tooling
[x] upstream our changes to hello-eks-anywhere
[x] add the helm chart to hello-eks-anywhere
[x] create new prow job
[x] Delete the old prow job
[x] Delete the old project
[x] Update all-projects in the build tooling Makefile to add the hello app

I asked Abhay if hello should be in the all projects, but no response yet

Done


Provide separate configuration to set readTimeout on OkHttp

Description
Right now, the client configuration only exposes a single timeout config, connectionTimeout. That is then used for both the connection and read timeout configs on the underlying OkHttp client: https://github.com/typesense/typesense-java/blob/master/src/main/java/org/typesense/api/ApiCall.java#L52-L53
There should be separate configs for these two values, so that we can more granularly control the behavior of the API client.
Steps to reproduce
Not applicable.
Expected Behavior
The expectation would be to have two separate configuration values for the connection and read timeouts.
Actual Behavior

Fixed in v0.3.0


Provide baseDir to allow for proper caching.

Providing this baseDir will allow for changes/upgrades in ember-cp-validations to invalidate template compilation caches.
This is roughly inline with https://github.com/babel/broccoli-babel-transpiler/pull/89 which I am porting to ember-cli-htmlbars and ember-cli-htmlbars-inline-precompile now.

@rwjblue 👍 . Ill keep this PR open so we can track the build / test cases

https://github.com/ember-cli/ember-cli-htmlbars/pull/90 adds support for this upstream (and issues a deprecation if baseDir is not provided.

https://github.com/ember-cli/ember-cli-htmlbars-inline-precompile/pull/37 should fix the immediate issue here (I think).  Would love it if you could test/confirm.

@rwjblue I made some comments on your PR. After I made them locally, all the test cases passed! 🎉

Awesome, thank you! I'll get that updated shortly.

OK, those changes were made and released under<EMAIL_ADDRESS>I believe once updated this should pass...

Just pushed a commit with that version bump.

@rwjblue issue seems to be resolved. Tests are failing due to #275

I run

"ember-cli": "2.7.0",
"ember-cli-htmlbars": "^1.1.0",
"ember-cli-htmlbars-inline-precompile": "^0.3.5",

Even now i'm getting deprecation message when starting the ember server !

DEPRECATION: ember-cli-htmlbars-inline-precompile is opting out of caching due to an AST plugin that does not provide a caching strategy: v-get.
DEPRECATION: ember-cli-htmlbars is opting out of caching due to an AST plugin that does not provide a caching strategy: v-get.

Why would those versions prevent this deprecation?
The fix for the deprecation (landed in this PR) hasn't been released in a published version yet (I think).


🐛 Bug Report: Multiple Merge Request creation to same Gitlab Repo fail with 400 Bad request

📜 Description
Hi,
We are using the action: publish:gitlab:merge-request in our Merge Request templates and generating the MR to an existing repo in Gitlab. First MR will be successful, however any subsequent MR with same branch name or different branch name fails with 400 Bad request  error.
Please let me know how can this be resolved?
👍 Expected behavior
The template should generate subsequent MRs successfully as long as the branch name is new.
👎 Actual Behavior with Screenshots
The publish action fails with below error.
Commit to branch failed

👟 Reproduction steps

Create a software template with publish step using - action: publish:gitlab:merge-request
Try to publish the MR to an existing Gitlab repo. Once successful, merge the MR and close it.
Try to create another MR to the same repo, it fails with above error.

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Tried to debug the code to check the branch name passed to the commit action as in below snippet. Even though branch name is new and non-existing but still commit fails.
ctx.logger.info('repoID' +repoID + 'branchName' + branchName + 'ctx.input.title' + ctx.input.title );
await api.Commits.create(repoID, branchName, ctx.input.title, actions);

@sharadpattanshetti perhaps there's a bit more information available on the error object from the GitLab API if we log it without stringifying?  I think the best way to figure out what's going on here is to narrow down the error that we're getting back from GitLab.

@Rugvip I tried that, this is what the gitbreaker api is returning in the error object.

Hmm, there is a commitAction option that you can set to 'update' or 'delete' instead of the default 'create'. That should let you update files rather than create new ones. If you do need something more granular than that I think that might be capabilities that we'd need to add to the GitLab merge action.
Btw, would be awesome if you want to ship the change of forwarding the exact error coming back from GitLab in a PR, that would make troubleshooting easier for everyone! 🙏

Thanks @Rugvip that really helped me. What I thought is to ask user to select create or update action in template ui depending upon whether the feature already exists in target repo or not.
Created 2 publish actions for create and update and conditionally triggering them.
We are testing all the scenarios like - change in files, new file added etc for both create and update.

@Rugvip I also met with the same issue. Can you please reopen it and assign it to me? I will implement returning of error
It also looks like mixed actions are not supported now. (Create new file and update some old)

@sharadpattanshetti For now this action only can create new files. Diffing and updating of existing aren't supported. You can use git actions for it

Thanks @sergeyshevch. Is this more of a limitation from @gitbreaker library we are using?
Because we were trying to explore the @isomorphic-git library and most of the actions like branch creation, commit on existing file update etc are working fine.

@sharadpattanshetti it's a limitation of the GitLab APIs, which gitbeaker uses. The workaround is to work directly with the git protocol and data, which isomorphic-git does.

We also have a problem with not being able to do updates and creates on the same MR. Is there any plan to support this?

@oreonl Currently I'm out of capacity for this issue. Feel free to open PR if you want

@sharadpattanshetti could you elaborate a bit more on what you tried?
@oreonl as far as we can tell we'll need to move over to using git operations directly rather than the GitLab APIs. There's currently no capacity to work on this from any of the maintainer teams so we're relying on community contributions.

@Rugvip finally what worked for us was a custom action to create the MR which will actually generate the modified/changed files.

In the action we used the isomorphic-git lib for the git operations like new branch creation, branch checkout, git commit and git push. Also, used Gitlab API to create a MR since isomorphic-git had some issue creating the MR. (all these operations should be in same order)
After branch checkout we have logic to merge the updated folder and files using the @alumna/reflect lib. This lib helps you replace the updated directory contents in the target directory.
This custom publish action syntax is same as the backstage built in action.
It is not that neat because of multiple libs being used but there were problems using one single lib for all the operations.

Hope this helps you, let me know if you have any other questions.

@sharadpattanshetti could you elaborate a bit more on what step 2 does?

Hi @Rugvip,
I will try to explain, hope it will be helpful :
In the 2 step the issue we were facing was that - Initially we used the fs-extra lib's writeFileSync and other functions, the MR is created in the Gitlab it was not showing the diff or the updated changes only. It was showing contents of each file as addition. Hence we chose @alumna/reflect.
The @alumna/reflect lib's reflect method helps us shift all the updated files from the source repo to the destination repo using the below snippet.
let { res, err } = await reflect({                 src: ${ctx.workspacePath}/${directory},                 dest: ${templateDir}/${directory},                 // (OPTIONAL) Default to 'true'                 recursive: true,                 // (OPTIONAL) Default to 'true'                 // Delete in dest the non-existent files in src                 delete: true,                 // (OPTIONAL)                 // Array with files and folders not to reflect                 // exclude: ["skip-this-file.txt", "skip/this/directory"]             });
This worked beautifully if the source repo has single folder which gets updated. However, if there is more than one folder updated or a new folder added additionally then it is not working. We are looking at fixing that issue.
Please let me know if you have better suggestions on this.

Hi, I have the same error.
Is there any update?
Thanks!!

@sharadpattanshetti ah alright, yep that sounds like a good approach for step 2. Beyond that you'll want to use isomorphic-git to do a shallow clone of the existing repo, then you sync over all the files with @alumna/reflect (or similar method). Once the new files are in the target repo you add all files, equivalent of git add --all, create a new branch and commit all changes, then push the new branch to GitLab. At this point you switch over to calling the GitLab API to create a new MR using the branch that you pushed.

Yes exactly @Rugvip. We identify the status of the sub folders and files isomorphic-git lib's status-metrics and then handle modified, delete scenarios. It is bit complicated but able to get almost all scenarios working.
Please let me know if you need more info.

Thanks @sharadpattanshetti - sounds great. Would you be up for contributing a fix for this bug? :)

sure @tudi2d . Occupied a bit with urgent deliverables now, will try in few days definitely.
Before that I wanted your and @Rugvip  opinion if this is the best way to achieve that functionality?

sure @tudi2d . Occupied a bit with urgent deliverables now, will try in few days definitely. Before that I wanted your and @Rugvip opinion if this is the best way to achieve that functionality?

That sounds like a good idea 👍

I faced with the same problem. @sharadpattanshetti Can you share a code snippet for isomorphic-git which you use?

Is there any update about this?
Thanks!

Also looking for updates

Read integration config and other required details
Clone the template repo - this might have happened using cookiecutter action in the first step in template.yaml
Clone the component repo using below command, execSync is from built in nodejs lib :

 require("child_process").execSync(git clone https://oauth2:${token}@${cloneUrl} ${templateDir} --depth 1)
4. Serialize the template directory({{cookiecutter_componentId}}) content as in the publish:gitlab-merge-request action
```
let fileRoot: string;
if (sourcePath) {
fileRoot = resolveSafeChildPath(ctx.workspacePath, sourcePath);
} else if (targetPath) {
// for backward compatibility
fileRoot = resolveSafeChildPath(ctx.workspacePath, targetPath);
} else {
fileRoot = ctx.workspacePath;
}
const fileContents = await serializeDirectoryContents(fileRoot, {
gitignore: true,
});
const fileContentsIntermediate = await serializeDirectoryContents(intermediateDir, {
gitignore: true,
});
5. Now the fileContents dir has all the serialized template dir content and fileContentsIntermediate  is having all the serialized component content(where the merge request is going to be created)
6. Create local git branch using @isomorphic-git lib
7. Read complete folder tree from the Repo before doing sync
          ```
  const getAllDirectoryNames = fs.readdirSync(ctx.workspacePath, { withFileTypes: true })
                .filter((item) => item.isDirectory())
                .map((item) => item.name);
            const getAllFileNames = fs.readdirSync(ctx.workspacePath, { withFileTypes: true })
                .filter((item) => item.isFile())
                .map((item) => item.name);
            getAllFileNames.map(async (file: any) => {
                fs.copySync(`${ctx.workspacePath}/${file}`, `${templateDir}/${file}`);
            })

Call @alumna/reflect to sync each folder of template repo to the component repo

await Promise.all(
                getAllDirectoryNames.map(async (directory_test) => {
                    await getDirectory(templateDir, ctx.workspacePath, directory_test);
                })
            )

Get modified files using the status using the @isomorphic-git lib

// GET ADDED/MODIFIED FILES
            await Promise.all(
                fileContents.map(async (file: any) => {
                    let status = await git.status({ fs, dir: `${templateDir}`, filepath: `${file.path}` })
                    if (status === '*added' || status === '*modified' || status === 'absent') {
                        ctx.logger.debug(`my-gitlab-merge-request : Step 2 - FILES ADD/UPDATE: ${file.path}:${status}`)
                        await git.add({ fs, dir: `${templateDir}`, filepath: `${file.path}` })
                    }
                })
            )

Get Deleted file @isomorphic-git lib

//GET DELETED FILES
           await Promise.all(
               fileContentsIntermediate.map(async (deletedFile: any) => {
                   let status = await git.status({ fs, dir: `${templateDir}`, filepath: `${deletedFile.path}` })
                   if (status === '*deleted') {
                       if (fs.existsSync(`${templateDir}/${deletedFile.path}`)) {
                           await git.remove({ fs, dir: `${templateDir}`, filepath: `${deletedFile.path}` })
                       }
                       else {
                           await fs.copy(`${intermediateDir}/${deletedFile.path}`, `${templateDir}/${deletedFile.path}`);
                           await git.remove({ fs, dir: `${templateDir}`, filepath: `${deletedFile.path}` })
                           ctx.logger.debug(`my-gitlab-merge-request : Step 2 - FILES DELETED: ${deletedFile.path}`)
                       }

                   }
               })
           )

Do a commit and push using @isomorphic-git lib
Create MR using gitbreaker lib
Additionally if you want to create target branch/verify in between these steps etc then use gitbreaker lib

Any news about this?
Thanks!

ticket is up for grabs @ichasco-heytrade!

GitLab API supports different actions per file, so the scaffolder action could be modified to make it per file/path, for example:

    - id: gitlab_merge_request
      name: publish:gitlab:merge-request
      action: publish:gitlab:merge-request
      input:
        repoUrl: gitlab.com?repo=backstage-playground&owner=agavlyukovskiy
        title: "Test MR"
        branchName: my-mr-branch
        commitActions:
          - path: CODEOWNERS
            action: 'update'
          - path: my-new-dir
            action: 'create'
          - path: dont-like-dir
            action: 'delete'

instead of current approach where we use the same commitAction for all files

@ichasco-heytrade if I understand you correctly, it would require using GitLab API for every file. I guess it's technically possible, but I'd avoid that due to rate limiting.

Is there any news?

Want to link this issue here too: #22244

GitLab API supports different actions per file, so the scaffolder action could be modified to make it per file/path, for example:

    - id: gitlab_merge_request
      name: publish:gitlab:merge-request
      action: publish:gitlab:merge-request
      input:
        repoUrl: gitlab.com?repo=backstage-playground&owner=agavlyukovskiy
        title: "Test MR"
        branchName: my-mr-branch
        commitActions:
          - path: CODEOWNERS
            action: 'update'
          - path: my-new-dir
            action: 'create'
          - path: dont-like-dir
            action: 'delete'

instead of current approach where we use the same commitAction for all files

This would absolutely solve the exact issue I'm having. Trying to make a GitLab MR where we're creating a file and updating an existing file at the same time.

Is anyone working on this?

I'm getting the exactly same issue in gitlabRepoPush. There is any update?

We're also seeing this.

In my case, using the action gitlab:repo:push I had to specify a separate init file and then the rest of New files in this action. Because when I used commitAction: update apply to all files.  Implement next approach would be good for many scenarios:

GitLab API supports different actions per file, so the scaffolder action could be modified to make it per file/path, for example:

    - id: gitlab_merge_request
      name: publish:gitlab:merge-request
      action: publish:gitlab:merge-request
      input:
        repoUrl: gitlab.com?repo=backstage-playground&owner=agavlyukovskiy
        title: "Test MR"
        branchName: my-mr-branch
        commitActions:
          - path: CODEOWNERS
            action: 'update'
          - path: my-new-dir
            action: 'create'
          - path: dont-like-dir
            action: 'delete'

instead of current approach where we use the same commitAction for all files

@ichasco-heytrade if I understand you correctly, it would require using GitLab API for every file. I guess it's technically possible, but I'd avoid that due to rate limiting.

For getting files, one api call is enough (cf https://docs.gitlab.com/ee/api/repositories.html#list-repository-tree)


go.mod, vendor: pin golang.org/x/* packages to tagged versions

The Go project recently started tagging releases on the golang.org/x/* packages, see [1] for more information. Use these versions where appropriate which will allow these dependencies to be updated by dependabot going forward.
[1] https://groups.google.com/g/golang-dev/c/5TazuUXQyHY

/test

The "runtime" failure is fixed on master branch by https://github.com/cilium/cilium/pull/22043 and the "k8s-1.16-kernel-4.9" failure is https://github.com/cilium/cilium/issues/21735. All other tests passed and review is in. Marking as ready to merge.
All other tests passed, marking as ready to merge. Thanks for your contribution @dlapcevic.


Update networking-virtual-datacenter.md

Changing title, per conversation with Jon Ormond.

#Sign-off


Question: how do I open the ipa file

How do I open and install the ipa file

Check out my video https://www.youtube.com/watch?v=3DHv4g-Xt9g, it will show you how to install an IPA using Windows or Mac.


update footer

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the code change
the footer section is not properly arranged please assign me @pr

Proposed solution
i enhance the footer section which looks clean and clear
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

can i work on this


Investigate alternatives for Puppeteer usage

Follow-up from the discussion on #146.
It would be good to know for what exactly Puppeteer is needed and if it could be replaced with something else that doesn't have a strict requirement for a given browser.
@sidvishnoi maybe you could please explain? I'm happy to help on this issue.

@whimboo what's the use case? We could use puppeteer with WebDriver BiDi + Firefox if you need basic JS evaluation to a basic type.

Puppeteer is used in following cases presently:

Getting processed shortname from ReSpec document (after JS evaluation): https://github.com/w3c/spec-prod/blob/bc93d69901e2c76b0e5147c12263acf6099506d3/src/prepare-build.ts#L188-L193
Extracting bikeshed metadata pre.metadata to get shortname: https://github.com/w3c/spec-prod/blob/bc93d69901e2c76b0e5147c12263acf6099506d3/src/prepare-build.ts#L218-L237
Some query selector processing to extract metadata from published pages: https://github.com/w3c/spec-prod/blob/bc93d69901e2c76b0e5147c12263acf6099506d3/src/prepare-build.ts#L263-L266
Full page content parsing, JS/CSS evaluation with subresources to extract all directly and indirectly referenced assets from a page that need to be copied for deployment: https://github.com/w3c/spec-prod/blob/bc93d69901e2c76b0e5147c12263acf6099506d3/src/build.ts#L169


Broken by an Express types update

G'day, it looks like Express has updated their type definition of NextFunction to:
export interface NextFunction {
    (err?: any): void;
    /**
     * "Break-out" of a router by calling {next('router')};
     * @see {https://expressjs.com/en/guide/using-middleware.html#middleware.router}
     */
    (deferToNext: "router"): void;
}

Which means it is incompatible with the NextFunction definition in this module's index.d.ts.
I'm working around this myself by explicitly importing NextFunction from Express to type the wrapper function.
Cheers!

Sounds like the same(?) issue as https://github.com/blakeembrey/async-middleware/issues/19. Feel free to submit a PR, unfortunately I'm not particularly eager to follow @types/express and mirror those changes since they aren't related directly to this package - that interface should be compatible with the one defined here so you don't have issues.


response for video hotplug event

Hi , @mpromonet. Thanks for your great project.
I have a question, can you make your project v4l2rtspserver be response for device hotplug event, like as /dev/video8.
when I tested your project APP, everything is OK, but when I plug out and plug in /dev/video8, the rtsp-stream stopped, and
nothing can proceed.  I don't want to restart this process and re-play rtsp-client app . can you do or can you tell us how to deal with this situation.
Thanks very much.

Hi,
There is probably some works to support removal and recreation of capture device. Maybe a workaround could be to use a v4l2loopback virtual device (that will always exists and feed v4l2rtspserver) and the hotplug event start a script that copy frames for the hotplug device to the loopback device.
Best Regards,
Michel.


Add Categorizer object.

This PR adds a dedicated Categorizer object that is meant to produce masks for defining categories. In particular, the changes are:

Added the new Categorizer as a TaskArrayFunction subclass.
Added the exposed attribute to all CSPs and added exceptions to the mixins in case they unexposed CSPs are requested from the CLI. This was only done for selectors before but is a good generic approach.
Categorizers are supposed to return (events, mask).
All existing core selectors now return (events, results).
In the category_ids producer the new categorizer object is used instead of selectors.
Template analysis:

Use categorizes in category definitions.
Updated the example selectors to always return (events, results).

Updated the create_analysis.sh script for easier local debugging.

Thanks!
Since this one contains core changes, maybe we still wait for input from @mafrahm .


cast call option: timestamp

Component
Cast
Describe the feature you would like
cast call has a --block flag but as best as I can tell it has no way of overriding the timestamp which means trying to simulate swaps in the past to uniswap ~always fail from getting executed past their deadline
relatedly, it would be very nice to be able to get the timestamp given a block and the closest block given a timestamp
Additional context
No response

fyi @DoTheBestToGetTheBest this should be super simple

@wminshew fyi there's
cast find-block:
find-block             Get the block number closest to the provided timestamp [aliases: f]

fyi @DoTheBestToGetTheBest this should be super simple

thanks, taking this

@wminshew just to clarify, should this find the block that corresponds to the timestamp or use the BlockOverrides.timestamp in the request?

@DoTheBestToGetTheBest I'm not sure the .call functions currently supports the BlockOverrides parameter, so it's possible that this requires a few extra changes

@wminshew just to clarify, should this find the block that corresponds to the timestamp or use the BlockOverrides.timestamp in the request?

can't believe I missed find-block -- apologies. But yeah the reason I opened this was the latter: adding a --timestamp override for cast call

Confirmed this isn't resolved yet. This is a feature which is available in Tenderly but not in cast call.
I have a block.timestamp dependent contract function call which I wish to reproduce for an old block number (as of the time that block was mined). However it's not possible to set the timestamp.
From @ gniculae (TG), cast call doesn't support eth_call block overrides and could be a bigger change to accomodate it.


hexgui: unstable-2023-1-7 -> 0.10-unstable-2024-11-03

Things done

cc @ursi

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I am the maintainer of this package and I approve


Update API discovery cache even when incomplete

Previously, if there were any APIs having issues on the cluster, no new API information could be used by the controller. Now, partial results can still be updated. Additionally, API discovery errors during startup will not prevent the controller from functioning at all.
Refs:

https://issues.redhat.com/browse/ACM-5491

I was considering reporting the controller as unhealthy if the API discovery was having an error, but I realized that restarting the container (which is what kubernetes will do when it's unhealthy) won't be likely to fix anything. It would be easy to add that in though, see https://github.com/JustinKuli/config-policy-controller-1/tree/5491-incomplete-server-apis.
Additionally, @mprahl suggested that we take a more similar approach to the API discovery as in the dependency-watches library: https://github.com/stolostron/kubernetes-dependency-watches/pull/8. I think that is a good longer-term solution, but this controller shares the API information with the templates library, and I would rather have a smaller fix in sooner, rather than a bigger one in multiple places that could be tricky to coordinate.


tools/omnissm - Use GSI for registrations table queries

This changes from scanning DynamoDB to querying Global Secondary Indexes for IsTagged and IsInventoried.  This requires using int types instead of bool types since DynamoDB does not support GSIs for its Bool data type.  This uses C style truthiness, in that 0 is false and another greater than 0 is true.

is there anything that needs to go in the example cft to make this valid on the gsi side?


125 upgrade to reactslingshot last version

Changes Unknown when pulling 448e12fc6eb8e1e2062841a10a24aba2f42446fc on 125-upgrade-to-reactslingshot-last-version into ** on master**.

LGTT


[Tests-Only]run latest core changes in ocis

Description

Related Issue

Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

:boom: Acceptance tests Core-API-Tests-ocis-storage-9 failed. The build is cancelled...

should be done by #2262

:boom: Acceptance tests Core-API-Tests-ocis-storage-4 failed. The build is cancelled...

:boom: Acceptance tests Core-API-Tests-owncloud-storage-5 failed. The build is cancelled...


[AMBARI-23091] Zeppelin Notebook SSL credentials in Ambari UI are in …

…plain text rather than being hidden
Added property type and attribute for password fields to be real password fields, not text.
(Please fill in changes proposed in this fix)
Installed fresh cluster and tested existing cluster upgrade.
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/857/
Test PASSed.


Resolve conflicts in fix of Issue #10: proofs did not depend on hashes

This should combine all our suggestions with the new documentation.

@ashleyniemerg I'm eager to merge this in

@dvf Sorry, I had to look it up! You can inherit from 'object' implicitly according to the python 3.6.3 documentation. Someone on stack overflow pointed out that implict vs. explicit can make a difference though: https://stackoverflow.com/a/45893772/8334823
Thanks for the tutorial btw.


How to submit jobs with different JobPriority ?

Organization Name:USTC

Short summary about the issue/question:
I want to submit some jobs with different JobPriority,how to use the rest api to do?Could you give me some hints? Thanks!
Brief what process you are following:

OpenPAI Environment:

OpenPAI version:v1.8.0

In detail,how to edit the yaml to submit job if i want to adapt the JobPriority?

extras:
  hivedScheduler:
    jobPriorityClass: oppo
    taskRoles:
      taskrole:
        skuNum: 8
        skuType: V100

Here is a example. Please add hived config in extra field. And change the sku according  to your cluster


Fix Google search results

Google search results claim that https://www.thedigitalstandard.org/ might have been hacked:

How to fix: https://support.google.com/websearch/answer/190597?p=ws_hacked&hl=en&sa=X&ved=0ahUKEwiso5vnzvvZAhWOPN8KHcPxAhkQpwgILDAA&visit_id=1-636571702554337762-2881092052&rd=1

Yeah, I noticed that earlier today! Flagged for Casey who's gonna give Google the info they need.

We reviewed the site infrastructure and didn't find any problems and the security section of the Google console doesn't list any issues. We've contacted Google support and will keep you guys posted on their response.

I'm no longer seeing the warning that the Digital Standard has been hacked, so I'm closing this issue.

So strange. Thanks @j-br0.


fix Cmake FetchContent  build bug

Why
when use  Cmake FetchContent  to import ylt, compile error:
fatal error: iguana/json_reader.hpp: No such file or directory

What is changing
Example

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.guanshoukui seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Tags <util.player_is_valid[]> isn't recognized as a tag.

Also applies to <util.npc_is_valid[<npc>]>  (this tag worked in build 1552, but not in 1558+).

How about on the dev builds?

Erm, I thought the Jenkins builds were dev builds...? (the ones betweem 1552 and 1558 are failed builds...)

http://ci.mineconomy.org/job/Denizen_Developmental/
These are super-duper-developmental.
The main Citizens CI ones are pretty much release builds, we only build every 10+ commits, and only after confirming stability.
(The failed builds were a jenkins error)

Ahhh...okay, it first fails in dev build #58...

... I just remembered, Util tags are broken and fixing that is on my todolist. From several days ago @.@
I need to hurry up and do that.

Should be fixed in the new latest dev build.


Update identity-provider-twitter.md

The documentation says to use all lower case letters but our example has a capital A midway through.  Adjusting to all lower case.

@da5is : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@da5is : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

#sign-off


Not working if (Memcached) Cache offline

Hi.
Python 3.8.x
django 2.2.x
django_dynamic_models 0.3.0
I set settings.CACHE to use memcached. If the memcached service is offline for any reason, simplest things like creating e FieldSchema fail. As soon as I start the memcached service, things work.
I do not understand exactly how the missing cache backend can affect things. I see this library makes use of cache, but it should be just for performance reason.
from dynamic_models.models import ModelSchema, FieldSchema

car_schema = ModelSchema.objects.create(name='Car')
car_model_field = FieldSchema.objects.create(model_schema=car_schema, name='test', data_type='character')  ### raise error

Traceback
Traceback (most recent call last):
  File ".\test.py", line 31, in <module>
    FieldSchema.objects.create(model_schema = model_schema, name = "test", data_type = 'character')
  File "C:\Users\.....\lib\site-packages\django\db\models\manager.py", line 82, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "C:\Users\.....\lib\site-packages\django\db\models\query.py", line 422, in create
    obj.save(force_insert=True, using=self.db)
  File "C:\Users\.....\lib\site-packages\dynamic_models\models.py", line 95, in save
    self._schema_editor.update_column(model, field)
  File "C:\Users\.....\lib\site-packages\dynamic_models\schema.py", line 39, in update_column
    self.add_column(model, new_field)
  File "C:\Users\.....\lib\site-packages\dynamic_models\schema.py", line 44, in add_column
    editor.add_field(model, field)
  File "C:\Users\.....\lib\site-packages\sql_server\pyodbc\schema.py", line 635, in add_field
    if field.many_to_many and field.remote_field.through._meta.auto_created:
AttributeError: 'NoneType' object has no attribute 'many_to_many'

Thanks for help

The caching has been buggy in the past and needs to be removed / reworked 🙂
That was the last thing I wanted to do before the next release.


one question regarding the replay buffer in DQN atari project

In the class "replay buffer", i found there is "states = np.transpose(np.asarray(states), axes=(0, 2, 3, 1))" in the def get_minibatch.
The problem is why transpose needed here? It looks like the original sequence is already correct so don't need transpose operation here.
Many thanks!

The transpose is just to take the state array from shape (32, 4, 84, 84), to (32, 84, 84, 4). The code could be programmed to work with either one, but in this case I decided to adopt (batch_size, height, width, stacked_frames) as the norm.
(Copied from my Medium response)


feat: add automocking in vitest

react-native-mmkv already has automocking for Jest, but not for Vitest.
This commit adds automocking for Vitest test environments just by checking if the VITEST_WORKER_ID environment variable is present.
This new functionality was also documented in the README, in the "Testing with Jest or Vitest" section, formerly the "Testing with Jest" section.
This way, MMKV will be mocked in both Jest and Vitest test environments.

Interesting, thanks for your PR!
Changes look good, but do people actually use Vitest in RN apps?

Changes look good, but do people actually use Vitest in RN apps?
This package was created to integrate React Native and Vitest, so I am using them together.

About the error during CI, this appears to be an error in the integration between Eslint and Prettier. I will fix this.

Oops! I'll see what went wrong and fix it.

Thank you!


facets dive demo no longer builds

The facets dive demo no longer seems to build.  Here is my best attempt in a Dockerfile:
FROM ubuntu:focal

RUN apt-get update && apt install -y apt-transport-https curl git gnupg

RUN curl -fsSL https://bazel.build/bazel-release.pub.gpg | gpg --dearmor >bazel-archive-keyring.gpg
RUN mv bazel-archive-keyring.gpg /usr/share/keyrings
RUN echo "deb [arch=amd64 signed-by=/usr/share/keyrings/bazel-archive-keyring.gpg] https://storage.googleapis.com/bazel-apt stable jdk1.8" | tee /etc/apt/sources.list.d/bazel.list

RUN apt update && apt install -y bazel

WORKDIR /opt

RUN git clone https://github.com/PAIR-code/facets

WORKDIR /opt/facets

RUN bazel build //facets_dive/demo

The result I see is:
#13 70.61 WARNING: Download from https://zlib.net/zlib-1.2.11.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
#13 70.62 ERROR: An error occurred during the fetch of repository 'net_zlib':
#13 70.62    Traceback (most recent call last):
#13 70.62       File "/root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/bazel_tools/tools/build_defs/repo/http.bzl", line 125, column 45, in _http_archive_impl
#13 70.62               download_info = ctx.download_and_extract(
#13 70.62 Error in download_and_extract: java.io.IOException: Error downloading [https://zlib.net/zlib-1.2.11.tar.gz] to /root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/net_zlib/temp4907857517448745203/zlib-1.2.11.tar.gz: GET returned 404 Not Found

try adding to the WORKSPACE file:
http_archive(
name = "zlib",
build_file = "@com_google_protobuf//:third_party/zlib.BUILD",
sha256 = "c3e5e9fdd5004dcb542feda5ee4f0ff0744628baf8ed2dd5d66f8ca1197cb1a1",
strip_prefix = "zlib-1.2.11",
urls = [
"https://mirror.bazel.build/zlib.net/zlib-1.2.11.tar.gz",
"https://zlib.net/zlib-1.2.11.tar.gz",
],
)

Doesn't seem to fix it, unfortunately.  After adding that to the WORKSPACE file I still get:
Starting local Bazel server and connecting to it...
WARNING: Download from https://mirror.bazel.build/github.com/bazelbuild/bazel-skylib/archive/e9fc4750d427196754bebb0e2e1e38d68893490a.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://mirror.bazel.build/github.com/bazelbuild/rules_webtesting/archive/0.2.2.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://mirror.bazel.build/github.com/google/protobuf/archive/v3.7.1.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://mirror.bazel.build/repo1.maven.org/maven2/com/google/template/soy/2019-03-11/soy-2019-03-11.jar failed: class java.io.FileNotFoundException GET returned 404 Not Found
INFO: Repository net_zlib instantiated at:
  /opt/facets/WORKSPACE:57:21: in <toplevel>
  /root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/io_bazel_rules_closure/closure/repositories.bzl:175:13: in closure_repositories
  /root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/io_bazel_rules_closure/closure/repositories.bzl:1001:17: in zlib
Repository rule http_archive defined at:
  /root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/bazel_tools/tools/build_defs/repo/http.bzl:372:31: in <toplevel>
WARNING: Download from https://zlib.net/zlib-1.2.11.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
ERROR: An error occurred during the fetch of repository 'net_zlib':
   Traceback (most recent call last):
        File "/root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/bazel_tools/tools/build_defs/repo/http.bzl", line 132, column 45, in _http_archive_impl
                download_info = ctx.download_and_extract(
Error in download_and_extract: java.io.IOException: Error downloading [https://zlib.net/zlib-1.2.11.tar.gz] to /root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/net_zlib/temp7042427168180112346/zlib-1.2.11.tar.gz: GET returned 404 Not Found
ERROR: /opt/facets/WORKSPACE:57:21: fetching http_archive rule //external:net_zlib: Traceback (most recent call last):
        File "/root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/bazel_tools/tools/build_defs/repo/http.bzl", line 132, column 45, in _http_archive_impl
                download_info = ctx.download_and_extract(
Error in download_and_extract: java.io.IOException: Error downloading [https://zlib.net/zlib-1.2.11.tar.gz] to /root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/net_zlib/temp7042427168180112346/zlib-1.2.11.tar.gz: GET returned 404 Not Found
INFO: Repository remote_java_tools instantiated at:
  /DEFAULT.WORKSPACE.SUFFIX:363:6: in <toplevel>
  /root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/bazel_tools/tools/build_defs/repo/utils.bzl:233:18: in maybe
Repository rule http_archive defined at:
  /root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/bazel_tools/tools/build_defs/repo/http.bzl:372:31: in <toplevel>
ERROR: /opt/facets/WORKSPACE:57:21: //external:zlib depends on @net_zlib//:zlib in repository @net_zlib which failed to fetch. no such package '@net_zlib//': java.io.IOException: Error downloading [https://zlib.net/zlib-1.2.11.tar.gz] to /root/.cache/bazel/_bazel_root/f0edbeac36261ac6bdca9830d0450fbe/external/net_zlib/temp7042427168180112346/zlib-1.2.11.tar.gz: GET returned 404 Not Found
ERROR: Analysis of target '//facets_dive/demo:demo' failed; build aborted:
INFO: Elapsed time: 44.528s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (155 packages loaded, 4038 targets configured)
    Fetching https://mirror.bazel.build/bazel_java_tools/releases/java/v11.9/java_tools-v11.9.zip; 26.1 MiB (27,385,648B)

I tried removing https://zlib.net/zlib-1.2.11.tar.gz from the urls section because it's a 404 but that didn't work either.

Sorry, Facets is not an actively supported project so updates to it should not be expected.
One recommendation is to check out https://pair-code.github.io/lit/ which includes a facets-dive-like visualization as part of its suite of things it can do.


Squid plugins

This way allows to write plugins, which adds configs to /usr/local/etc/squid/pre-auth/ and /usr/local/etc/squid/post-auth/
No templates intrusions to original squid.conf needed.
The auth plugins can be done by same way.
Developer must pay attention to removing generated plugin config file from /../etc/squid/.. folder on uninstall procedure and  accurately regenerate plugin config on enabling/disabling plugin feature.
I think that
{% include "OPNsense/Proxy/squid.user.pre_auth.conf" ignore missing with context %}
and
{% include "OPNsense/Proxy/squid.user.post_auth.conf" ignore missing with context %}
is not convinient way for squid plugins.

Hi Evgeny,
Your right, the include * is definitely better here. We should however also cleanup the old method when adding this one, 17.1 is probably a good timing for that.
I'm keeping this open, until I can change the old part and check our existing plugins for usage.
Regards,
Ad

note that for consistency the old and new positions for pre/post differ, I don't know which ones are more suitable/correct
and if we can, please let us avoid dummy files (include context missing maybe works too)
also note that this could be prone to racing with multiple files. is there really a need for multiple squid plugins (that can cooperate nicely?)

@fichtner good point, the old order seems to be the correct one:

local configured auth helpers
pre scripts
internal acl's
post scripts

Franco, squid directive 'include' wants to find any file at specified path. Dummy file exists for case, when no squid plugins installed. If directory is empty, squid start fails with error.
This include method can be used for more than one plugin. Plugin configs can be ordered by numeric prefixes (for ex., 100-ipcheck.conf, 200-super-acl.conf).


Option to add notconnected interfaces

It would be nice if docker-topo had an option to create not connected interfaces to a container.
I'd think this could be implemented e.g. using a driver_opt = notconnected or driver = notconnected.
Reason for this feature is to simulate non-complete networks, while still being able to do complete configuration simulation. I've seen configurations of mpoint networks using veth as driver, resulting in a lot of veth connections that could result in unexpected data-plane behavior.
For now the workaround is to use the macvlan driver and connect to an not-connected interface - I guess this is okay but not really the "nice" way.

can you elaborate the use case, because I don't think I understand it.

Sure,
User wants to run a simulation to test if their configuration provisioning tools work correctly. This provisioning tooling typically deploys to production, where they have configuration set on ports e1-48.
The simulated network might be smaller and not have all 48 ports connected. Say, user just likes to simulate devices on peer endports 1-8, and not having resources to spin up all 48 peer devices - yet they still would like to validate whether their production configuration is validated by the CLI parser correctly.
To do so, it would be helpful if they can spin up a container with et1-48 "active" from a ceos-lab perspective, while only having ports 1-8 connected to other devices.

I think you should be able to attach all of those interfaces to a single bridge and then bring the bridge down from the hostOS.

That sounds reasonable enough; I think we can close this for now.


stat_changed callback script

When a player stat has changed, receive a callback with the PlayerEntityObject, the old stat value, and the new value. This provides a free event system for many player actions, without a lot of work.

implemented!


Document testing process

It took me a while to figure it out, so now it won't for someone next :)

@samedii documented in main README


Broker dashboard_url's can be null

With relatively recent change we broke brokers that return a dashboard_url field with null value. This change reverts that behaviour for the provision and aligns the behaviour for update.


Switch out dependency 'RPostgreSQL' for 'RPostgres'

This package is only used once to create a connection in insertPostgisGeometry(), but any future use of non-DatabaseConnector functions should use the RPostgres package which has been tested to be much faster.

Preliminary testing indicates that this is not a simple switch.
Inserting a geom to PostGIS from R turned out to be difficult. After many attempts, RPostgreSQL::insertPostgisGeometry() seemed to be the only function that worked for us.


Whitespace Cleanup

I know, I know, whitespace - who cares? It was just really bothering me that there was so much inconsistency, so I did my best to resist frivolous changes and stuck to mostly running:
Editor > Structure > Re-Indent
With 4 spaces as my indent. It's about as close as we can get to gofmt and seemed reasonable. Most of the files I saw matched this format anyway, many just had a mistake here or there.
Then I just went through and removed some bonus newlines. Ahhh, much better!! :sunglasses:

Oh I know!  I wish we had a standard swift format tool like gofmt but we don't have that yet.
https://github.com/exercism/xswift/issues/10
I am not sure what the stand is for exercism as a project on whitespace only PRs but this being a swift language track I am going to close this for now until we have an official swift linter.
References:
https://github.com/apple/swift/pull/275
https://github.com/apple/swift/pull/963

I think that if we are internally inconsistent, we could probably choose some sort of rule-set just for the repo, and be consistent with that (in which case whitespace-only PRs would totally be a good thing). I think the question is--are we inconsistent? If so, what sort of consistency is important? If we make it consistent, is there some sort of mini-linting that we could do to make it easy to keep things consistent (and also something that we could run on CI)?

I would love a CI solution that makes a lint commit. . I have heard of something like that but I am not sure how to get it set up. I know travis can check lint rules but I don't know about actually making a "Lint" commit.  https://www.bignerdranch.com/blog/continuous-linting/

Makes sense, I'll try to make progress on #10 instead. Thanks!! :D


using 'normal' instead of 'normal!'

This will cause user mappings to modify result of the script which is BAD!
for example using a plugin which maps 'm' to move causes jumplist not to be filled.
I'll commit a fix for that in a minute.

https://github.com/lyuts/vim-rtags/pull/72

Thank you for the fix. It is now merged.


Grammar

It's "its", not "it's".

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

ID: 8c1b9a06-b0b3-22b9-1bfb-5839c7647225
Version Independent ID: 450e75ed-b1c3-9593-26f6-866947fc2e50
Content: Tar and Curl Come to Windows!

—

Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/Virtualization-Documentation/issues/1827#issuecomment-1428610611,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVXAOX7NZU3T73HWPCAGJ3DWXKIMXANCNFSM6AAAAAAS5M6B5U
.

Show my disc to your mother and your sister, son

—

Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/Virtualization-Documentation/issues/1827#issuecomment-1428610611,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVXAOX7NZU3T73HWPCAGJ3DWXKIMXANCNFSM6AAAAAAS5M6B5U
.
You are receiving this because you are subscribed to this thread.Message
ID: <MicrosoftDocs/Virtualization-Documentation/issues/1827/1428610611@
github.com>


Hover Card background is not transparent

When mousing over the cards in your deck list or the cards your opponent has played, the hover card is not transparent.
I'm running on a Macbook Pro OSX 10.10.3
HSTracker 0.16 (769)

Duplicate of #364

I apologize for not catching that previous issue! If it's okay, there are a couple of other things I'd like to report such as the tracker not working after HS crashes or closes and reopens, or when you make a change on the deck manager for your current deck it could automatically update itself.

Yeah, feel free to add issue ;)


Doc: Add in code documentation for  RunnableEach

Privileged issue

[X] I am a LangChain maintainer, or was asked directly by a LangChain maintainer to create an issue here.

Issue Content
Add in code documentation for RunnableEach and Runnable Each Base
https://github.com/langchain-ai/langchain/blob/main/libs/core/langchain_core/runnables/base.py#L2810-L2810
https://github.com/langchain-ai/langchain/blob/main/libs/core/langchain_core/runnables/base.py#L3685-L3685
Here's an example of in code documentation PR for Runnable Parallel:
https://github.com/langchain-ai/langchain/pull/16600/files
And example documentation for Runnable Binding:
https://github.com/langchain-ai/langchain/blob/main/libs/core/langchain_core/runnables/base.py#L4014-L4014

I am taking this.


flint: Always create proper so file, support blas

Motivation for this change
Flint non-deterministically sometimes does not produces the proper .so.major file. This adds a patch (from usptream) to prevent this).
While doing that I also made blas support optional and enabled tests.
Pinging maintainer @7c6f434c.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: arb
Partial log (click to expand)
mkdir -p /nix/store/rnygmzknlyxk31621mzax31hly89y8gb-arb-2.8.1/include
cp  fmpr.h  arf.h  mag.h  arb.h  arb_mat.h  arb_poly.h  arb_calc.h  acb.h  acb_mat.h  acb_poly.h  acb_calc.h  acb_hypgeom.h  acb_modular.h  fmprb.h  bernoulli.h  hypgeom.h  fmpz_extras.h  partitions.h /nix/store/rnygmzknlyxk31621mzax31hly89y8gb-arb-2.8.1/include
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/rnygmzknlyxk31621mzax31hly89y8gb-arb-2.8.1
shrinking /nix/store/rnygmzknlyxk31621mzax31hly89y8gb-arb-2.8.1/lib/libarb.so.1.1.1
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/rnygmzknlyxk31621mzax31hly89y8gb-arb-2.8.1/lib
patching script interpreter paths in /nix/store/rnygmzknlyxk31621mzax31hly89y8gb-arb-2.8.1
checking for references to /build in /nix/store/rnygmzknlyxk31621mzax31hly89y8gb-arb-2.8.1...
/nix/store/rnygmzknlyxk31621mzax31hly89y8gb-arb-2.8.1


Added Identification header

Identification and Verification both are capabilities of Identity Verification so it does not make too much sense to include one and not the other.

@lucia-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@PatrickFarley - Could you please look at this PR?

#sign-off


[Question] Pub packages white list process

Hi,
I know you only just started supporting Pub packages, thanks for the amazing work ! In #901, you mentioned that you would be using a whitelist to support only specific packages import.
My questions is: How can a package hope/ask to be on the whitelist?

This process is now explained here: https://github.com/dart-lang/dart-pad/wiki/Adding-support-for-a-new-package


Fixed bolt protocol disconnect packet not sent upon object destroy

Hello @zsoltpuskas,
Thank you so much for catching this bug! I have merged it into main and tag and test it properly later this week.


Add typing overloads to CloudPath.open()

Currently, the return type of CloudPath.open() is always IO[Any]. This differs from pathlib.Path.open(), which returns more specific types like TextIOWrapper, FileIO, BufferedRandom, BufferedWriter, BufferedReader, BinaryIO, or IO[Any] depending on arguments such as mode, buffering, and encoding. The more specific return types are useful for catching typing issues where a library expects a file handle in binary or text mode, specified as IO[bytes] or IO[str], respectively. With the current behavior, passing a file handle returned by CloudPath.open() often requires using typing.cast() or # type: ignore.
Here’s an example illustrating the current behavior compared to pathlib.Path.open():
from pathlib import Path
from cloudpathlib import CloudPath

cloud_path = CloudPath("s3://bucket/key")  # type: ignore
local_path = Path("test.txt")

with local_path.open("w") as f:
    reveal_type(f)  # Revealed type: io.TextIOWrapper

with local_path.open("wb") as f:
    reveal_type(f)  # Revealed type: io.BufferedWriter

with cloud_path.open("w") as f:
    reveal_type(f)  # Revealed type: typing.IO[Any]

with cloud_path.open("wb") as f:
    reveal_type(f)  # Revealed type: typing.IO[Any]

Thanks @0xbe7a, this is released now in v0.19.0 if you want to upgrade and test your use case