Instructions for Debug module in launch mode for C# do not work in latest VS Code

VS Code version 1.58.2
I'm trying to follow the instructions in section "Debug module in launch mode" for C#, but they need to be updated for the latest VS Code, as the menus seem to have changed.

"Navigate to the Visual Studio Code Debug view by selecting View > Debug. Select the debug configuration  Local Debug (.NET Core) from the dropdown."

Under the View menu, there is no "Debug" option, only "Debug Console".
If I select Debug Console, I don't see any way to select a debug configuration from a dropdown.

If I instead click on Run and Debug (Ctrl+Shift+D) on the left, I can then click on "create a launch.json" file. This generates a new launch.json file which I must then fill to specify the DLL I want to debug and its path. When I then run this debug configuration, I have an exception saying "Environment variable IOTEDGE_WORKLOADURI is required." (as others have encountered in the following issue: https://github.com/Azure/iotedge/issues/584). The solution to the issue I'm referencing was to add the following lines to the .csproj file, but it's not applicable in my case becaue these lines were already there:

<ItemGroup>
  <ProjectCapability Include="AzureIoTEdgeModule" />
</ItemGroup>

I'm guessing that the issue is that I'm not launching the DLL correctly. Either using launch.json is not the right way to start debugging an IoT Edge Module, or it needs some customizations to work with the IoT Edge simulator.
My launch.json file is:
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "program": "${workspaceFolder}/path/to/my/project/bin/Debug/netcoreapp3.1/mydll.dll",
            "args": [],
            "cwd": "${workspaceFolder}/path/to/my/project/bin/Debug/netcoreapp3.1",
            "console": "internalConsole",
            "stopAtEntry": false
        },
        {
            "name": ".NET Core Attach",
            "type": "coreclr",
            "request": "attach"
        }
    ]
}

(I created it manually from the file that's generated by default)

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

ID: 5a3f2f86-40bf-19f2-4f56-9dc71ed47fa2
Version Independent ID: 20e8912c-4a10-68b9-c46a-e945a62421d6
Content: Develop and debug modules for Azure IoT Edge
Content Source: articles/iot-edge/how-to-vs-code-develop-module.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@ckarrasalithya Thank you for your query . We will investigate and update the thread.

@ckarrasalithya Adding author and SME @kgremban To review this further and update the doc as appropriate.

Hi @ckarrasalithya, thank you for pointing this out. You're right that the debug interface has changed since this article was last updated, and you found the correct steps to go to the Run + Debug view from the lefthand menu. I will update the article to reflect this change.
As for the error that you're encountering, did you follow the steps in the previous sections to start the IoT Hub simulator? Either Set up IoT Edge simulator for IoT Edge solution or Set up IoT Edge simulator for single module app? If I tried to debug without starting the simulator first, I got the error that you describe. Once I started the simulator, I was able to follow the debug steps successfully.

@ckarrasalithya Doc has been updated. Let us know if you have any other queries.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@kgremban Looks like I am in the same situation following the current documentation, and yes I followed the Set up IoT Edge simulator (with option 1, simulate an IoT Edge solution). Looks like simulatore got configured properly:
PS C:\Path\To\Solution> & "c:\Users\me\.vscode\extensions\vsciot-vscode.azure-iot-edge-1.24.3\iotedgehubdev\0.14.9\iotedgehubdev" setup -c "HostName=myhub.azure-devices.net;DeviceId=me-dev;SharedAccessKey=SomeAccessKeyValue" -i "HostName=myhub.azure-devices.net;SharedAccessKeyName=iothubowner;SharedAccessKey=SomeAccessKeyValue"
Setup IoT Edge Simulator successfully.

Looks like I have no additional docker container running after this, should I?

@AshokPeddakotla-MSFT I have the same problem even after iot edge simulator is setup successfully.
Project file already contains

and launch.json local debug settings are below
{
"name": "CSharpModule Local Debug (.NET Core)",
"type": "coreclr",
"request": "launch",
"program": "${workspaceRoot}/modules/CSharpModule/bin/Debug/netcoreapp3.1/CSharpModule.dll",
"args": [],
"cwd": "${workspaceRoot}/modules/CSharpModule",
"internalConsoleOptions": "openOnSessionStart",
"stopAtEntry": false,
"console": "internalConsole",
"env": {
"EdgeHubConnectionString": "${config:azure-iot-edge.EdgeHubConnectionString}",
"EdgeModuleCACertificateFile": "${config:azure-iot-edge.EdgeModuleCACertificateFile}"
}
}
Can you look into it?

@konichi3 Can your team help troubleshoot this?

Yes, one of the members can take a look at the issue. Can someone please file an issue in GitHub repo here? This item is already closed and it's probably better to track in our repo.
https://github.com/Azure/iotedgehubdev/issues


How to submit a PR to the SWC Plugins site

I developed a swc-plugin-canyon. How can I add it to the SWC Plugins collection on the site?

You can send a PR to https://github.com/swc-project/pkgs/tree/main/apps/swc-plugins

Sorry, I misguided you. You can add one to https://github.com/swc-project/crawl-core-version/tree/main/pkgs/plugins


Render From specific viewpoint

Hello, and thank you for your excellent work on these 4DGaussians. I really appreciate this method and code. I want to ask about the rendering process for the Dynerf dataset. In the dynerf dataset (i.e., cut_roasted_beef), if I render it using this commandpython render.py --model_path "output/dynerf/cut_roasted_beef/" --skip_train --configs arguments/dynerf/cut_roasted_beef.py  & I will get a result that looks like yours on the project web: https://guanjunwu.github.io/4dgs/static/videos/cut_roasted_beef_2_30.mp4.
My question is: how do I render the train result so that I can only view it from a specific angle? For example, I want to render the result from the left perspective. I expect that the output should closely resemble the cam10.mp4 found in the original cut_roasted_beef data set. Is it possible to do it this way?

Hi，you can select a training view and rotate it? just modify the code in here

According to https://github.com/hustvl/4DGaussians/issues/137#issuecomment-2187862008, does view like get_spiral affect to train result or only render output ?


Make the structs in all iter_sent.hpp lower case

I would Iike to work on this, please.

I would Iike to work on this, please.

I would Iike to work on this, please.

Please go ahead!

I would Iike to work on this, please.

Please go ahead!


Fix Travis CI builds

https://travis-ci.org/github/intercept/intercept/builds/677071561
Seems like Travis CI status reporting is not working as it should, perhaps the integration token must be updated?


unable to augment class definition

Search Terms
typescript unable to augment class
Suggestion
I keep running into cases where I want to augment a class that I import from another module.
I know it isn't best practice, but in some cases it is necessary.
Use Cases
F.e. extending a prototype
Examples
At the moment, I'd like to do something like
import {MyClass} from './MyClass';

declare module './MyClass' {
  interface MyClass {
    newMethod(s: string): void;
  }
}

MyClass.prototype.newMethod = (s: string) => { /* ... */};
  // ^---- ERROR, 'MyClass' only refers to a type, but is being used as a value here. ts(2693)

But it won't let me. When I try to do this, tsc complains
'MyClass' only refers to a type, but is being used as a value here. ts(2693)

I always run into this issue when attempting to augment a class.
Oddly, this is the method prescribed over at Module Augmentation, but every time I try it it simply does not work.
Also the other most visible StackOverlow answer by @basarat says it isn't supported with classes.
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

In that example you apparently haven't imported the class constructor.  So what is it actually importing?  This is where I'd ask for a minimum reproducible example but suddenly I realize this is not Stack Overflow.   Your code works for me so this may be an issue you're having in your environment as opposed to a deficiency in TypeScript itself?

The actual class I'm importing is Object3D from the three module, which is class like
//<EMAIL_ADDRESS>
export class Object3D extends EventDispatcher {
  // ...
}

The following works:
import {Object3D} from 'three'

Object3D.prototype.updateMatrix = function() {
  // override implementation
}

But when I try to augment the Object3D class like the following,
import {Object3D, Vector3} from 'three'

declare module 'three' {
    interface Object3D {
        pivot: Vector3
    }
}

Object3D.prototype.updateMatrix = function() {
  // ^---- ERROR here

  // override implementation (uses this.pivot)
}

I get
'Object3D' only refers to a type, but is being used as a value here. ts(2693)

Seems like I'm importing a class.

Classes/constructor functions have two interfaces, they have the "static" interface and they have instance interface.  There are two ways to fix this problem:
import {Object3D, Vector3} from 'three'

declare module 'three' {
    interface Object3D {
        pivot: Vector3
    }

    interface Object3DConstructor {
        new (): Object3D;
    }

    const Object3D: Object3DConstructor;
}

Object3D.prototype.updateMatrix = function() {
  // override the implementation (uses this.pivot)
}

or
import {Object3D, Vector3} from 'three'

declare module 'three' {
    class Object3D {
        pivot: Vector3
    }
}

Object3D.prototype.updateMatrix = function() {
  // override the implementation (uses this.pivot)
}

In the first, you keep the two interfaces seperate, and this is the way a lot of the built in classes are modelled in typings and better models constructor functions, the latter uses the class keyword, which fuses the two interfaces together, just like they are at runtime.
I don't think TypeScript can really provide a clearer message, because it can't read your mind of what your intent it, it can only state the facts.

I think we would benefit from having a link to a web IDE with the relevant dependency already configured, like maybe this.
Here, I have the following code in index.ts, which errors as @trusktr is reporting:
import { Object3D } from 'three';

declare module 'three' {
  // comment the following out to see error go away
  interface Object3D { }
}

const object3D: Object3D = new Object3D(); // error!
object3D.updateMatrix(); // error!

But I also have the following code, which works just fine:
import { Object7D } from './seven';

declare module './seven' {
  interface Object7D { }
}

const object7D: Object7D = new Object7D(); // okay
object7D.updateMatrix(); // okay

Now seven.d.ts is just a made-up declaration file I've placed inside my own project, with the following contents:
export class Object7D {
  constructor();
  updateMatrix(): void;
}

But the configured three dependency is similar, with code like
export class Object3D extends EventDispatcher {
	constructor();
// ... snip ...
	updateMatrix(): void;
// ... snip ...
}

So, what's going on?  What's the difference between seven and three here? (don't say four pls)  I don't understand why in the case of seven, the compiler understands that the static side of Object7D still exists as a value named Object7D, while in the case of three the compiler immediately forgets that Object3D was a class with an instance and an object side.
I don't know enough about TS dependency management to know if this is a bug, design limitation, or user error, but it seems fishy to me.

What's the difference between seven and three here?

four. :trollface:

@kitsonk Your first suggestion,
declare module 'three' {
    interface Object3D {
        pivot: Vector3
    }

    interface Object3DConstructor {
        new (): Object3D;
    }

    const Object3D: Object3DConstructor;
}

results in all other properties not existing on instances of Object3D. So things like obj.rotation result in Property 'rotation' does not exist on type 'Object3D'.
The second example,
declare module 'three' {
    class Object3D {
        pivot: Vector3
    }
}

results in the same errors about all other properties missing, but also with additional errors inside the Object3D.prototype.updateMatrix override:

I'm not able to reproduce the problem.
// ./world.d.ts

export class Foo {
    bar(): void;
}

// ./hello.ts
import {Foo} from "./world";

declare module "./world" {
    interface Foo {
        blah(): void
    }
}

Foo.prototype.blah = function() {

}

@DanielRosenwasser  Seems to be a problem with the three/Object3D definition. Here's a reproduction: https://repl.it/repls/QuarterlyWellinformedArchitect

I updated the previous comment's broken link (in case you were reading email).

@trusktr Did you resolve this issue. Currently experiencing the same thing while augmenting classes imported from three
The module augmentation seems to simply overwrite the declaration. I cant add properties that I need.

Please open a new issue with a concrete repo if you're encountering problems.


fix tests

Our refactor has broken the tests.  I assume the imports are silly.
We can also set up the tests to:
-- test the services and their construction.  this would involve mocking them.

Yes, I think it broke when we did the type refactor and TS didn't complain since it's not checking those testing files. Will fix it.


Fix/multiple mappings

Coverage remained the same at 100.0% when pulling d650de3cff35a2447eb4a6789203fda69222a746 on fix/multiple-mappings into 788e80f4e412413fe857e2a39a806979914b8095 on develop.


Move generic workflow out of repo

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

Comparison is base (d1dc1c8) 89.65% compared to head (f1fd10a) 89.09%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main       #8      +/-   ##
============================================
- Coverage     89.65%   89.09%   -0.57%
  Complexity       22       22
============================================
  Files             2        2
  Lines            58       55       -3
============================================
- Hits             52       49       -3
  Misses            6        6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Parallel execution of uptodate-properties

As File::exists, just like obtaining a file's modification date, both are potentially blocking operations, huge file sets can slow down the uptodate-property and uptodate-properties Mojos.
A potential solution is to use several threads, so while the first thread is still blocked, another thread can take over the next uptodate-check. With multi-core CPUs being ubiquitous these days, such a rather small change can improve the build time considerably, in particular for huge projects with thousands of files and / or slows disks (like typically on NTFS). For single-core CPUs the drawback is neglectable.

@slachiewicz @slawekjaranowski @khmarbaise Kindly requesting your review. :-)


Make directory structure of Android version of react-native-app-hotels Node.js package Android compliant

The current directory structure of the deployed react-native-app-hotels Node.js package for Android is:
node_modules\
  @kiwicom\
    react-native-app-hotels\
      android\
        fonts\
        drawable-xxxhdpi\
        drawable-xxhdpi\
        drawable-xhdpi\
        drawable-mhdpi\
        drawable-hhdpi\
        hotels.android.jsbundle.meta
        hotels.android.jsbundle

We'd like to change this structure to be compatible with Android and Gradle:
node_modules\
  @kiwicom\
    react-native-app-hotels\
      android\
        assets\
          fonts\
          hotels.android.jsbundle.meta
          hotels.android.jsbundle
        res\
          drawable-xxxhdpi\
          drawable-xxhdpi\
          drawable-xhdpi\
          drawable-mhdpi\
          drawable-hhdpi\

@Blackhex Thanks, this is a great report.
@tbergq Could you please also delete the build folder before every new build? We are not doing it now but it's wrong.


[MM-34309] migrate to typescript components/admin_console/jobs entire folder

Summary
This PR migrate to typescript the folder: components/admin_console/jobs.
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/17241
Release Note
NONE

Thanks for participating in Hacktoberfest! You can claim your sticker set here: https://get.printfection.com/hacktober21/4144583267 @ArcaneDiver

Please tag both @hmhealey and I once you are ready to have your PR re-reviewed. Thank you.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

@ArcaneDiver Did you have any additional questions from us for this PR? Please let us know :)

done in https://github.com/mattermost/mattermost-webapp/pull/9491


[Fix](paritial update) Fix the case of partial update failing in cloud mode

Proposed changes
Issue Number: close #xxx

run buildall

run buildall

remember to update documents in repo:doris-websit

run buildall

run performance

run performance


X-Hub-Signature does not match blob signature

using my own function works but it crash with github-webhook-handler:
    const createComparisonSignature = (body) => {
      const hmac = crypto.createHmac('sha1', process.env.GITHUB_SECRET);
      const self_signature = hmac.update(JSON.stringify(body)).digest('hex');
      return `sha1=${self_signature}`; // shape in GitHub header
    }
    const compareSignatures = (signature, comparison_signature) => {
      const source = Buffer.from(signature);
      const comparison = Buffer.from(comparison_signature);
      return crypto.timingSafeEqual(source, comparison); // constant time comparison
    }
    const { headers, body } = req;
    const signature = headers['x-hub-signature'];
    const comparison_signature = createComparisonSignature(req.body);
    if (!compareSignatures(signature, comparison_signature)) {
      return res.status(401).send('Mismatched signatures');
    } else {
      console.log("signature match")
      const { action, ...payload } = body;
      req.event_type = headers['x-github-event']; // one of: https://developer.github.com/v3/activity/events/types/
      req.action = action;
      req.payload = payload;
      next();
    }
    return next()

please let me know if there is any solution

I have the same issue too. But the project seems to be obsoleted because the issue exists for a long time and there is no response from the maintaining team.

I have forked a repository git-webhook-handler, it works will.


[Request]: Focus Editor

Prerequisites

[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.

Criteria

[X] Reasonably well-known and widely used (e.g. if it's a GitHub project, it should have at least 100 stars and/or 50 forks)
[X] English interface (or at least English documentation)
[X] Latest stable version
[X] Full version (i.e. not a trial version)
[X] Fairly standard install (e.g. uses a version-specific download URL, no elaborate pre/post install scripts)

Name
Focus Editor
Description
A simple and fast text editor
Homepage
https://github.com/focus-editor/focus
Download Link(s)
https://github.com/focus-editor/focus/releases/download/0.3.0/focus.exe
Some Indication of Popularity/Repute
Written in Jai, the focus editor is a minimalistic vscode.
Jonathon Blow trying it: https://youtu.be/7pGsRz48yjM?si=SMUwNsa-893gJE3B
Tsoding trying it: https://www.youtube.com/watch?v=AhPm8pv6dEs

I created a manifest for the focus-editor.
Currently it's provided in my scoop bucket:
https://github.com/danielpauldd/scoop-bucket.git
If desired, I can file a PR for integration into this bucket.

@danielpauldd I think your implementation is stelar. (great idea to start focus once, so it generates configs.)
{
    "version": "0.3.0",
    "homepage": "https://github.com/focus-editor/focus",
    "description": "A simple editor whose goal is to get out of your way and let you do work.",
    "license": "GPL-3.0",
    "architecture": {
        "64bit": {
            "url": "https://github.com/focus-editor/focus/releases/download/0.3.0/focus.exe",
            "hash": "788bf5fdf79249e59e3a94488001f4dc680657d4bac8c7d51134283a98aeb77c"
        }
    },
    "installer": {
        "script": [
            "#start editor once, to create default files and folders",
            "$app_process = Start-Process \"$dir\\focus.exe\" -PassThru -WindowStyle Hidden; start-sleep 2; $app_process | stop-process -force"
        ]
    },
    "shortcuts": [
        [
            "focus.exe",
            "Focus-Editor"
        ]
    ],
    "persist": [
        "global.focus-config",
        "projects"
    ],
    "checkver": "github",
    "autoupdate": {
        "architecture": {
            "64bit": {
                "url": "https://github.com/focus-editor/focus/releases/download/$version/focus.exe"
            }
        }
    }
}


In the Number input, when the Label text is long, its showing in multiple lines.

Before submitting a bug:

[yes] Is this reproducible in the latest version of Shidoka?
[yes] Is this reproducible in the Shidoka Storybook?
[ ] Is this reproducible in a Sandbox?
If it's not reproducible in any of the above scenarios, please do not open a bug report as it is an application-level issue.

Describe the bug
A clear and concise description of what the bug is.
In the Number input, when the Label text is long, its showing in multiple lines.
To Reproduce
Steps to reproduce the behavior:

Go to Shidoka Storybook
Click on Number input
Scroll down to 'Label' - enter a long text
See the label is showing in multiple lines

Expected behavior
The text must show in one line
Screenshots
If applicable, add screenshots to help explain your problem.

Environment

OS: [e.g. iOS] Windows
Browser [e.g. chrome, safari] Chrome, Edge, Firefox
Version [e.g. 22]

Additional context
Add any other context about the problem here.
Cross checked for Text area, but the text box is also expanding based on the label provided.

@kyndryl-design-system/designers need to know if the number input should have a maximum width or something before proceeding.


Map rotation

At least 3 users asked to allow rotating the map. Guess it needs to be done. The rotation needs to be preserved between modes.

Done in 0.4.


Fixed #4258

Fixed #4258, by adding missing space, otherwise "error: corrupt patch at line 12"
It seems that the original patch file is not generated by git diff > path or it has been edit with a text editor and saved without trailing whitespace.

Closing and replacing with #4288 since nGraph cannot merge from fork.


CB-18752 Allow multiAz when StackType is TEMPLATE

See detailed description in the commit message.

retest this please

CC @biharitomi

Closing.


Restrict notification rules so that can't be run more than once a minute.

as a user
when i create a notification rule
then i would like to not be able to schedule it to run more than once per minute.

First step is for @e-dard, @stuartcarnie and @aanthony1243 to diagnose the latency. We don't want to set a 1m minimum interval now.

@121watts hold off on implementing this until we have more information about the root cause of the query issue described above


Scrolling is the SLD editor is confusing

Scrolling in the SLD editor can be very confusing. When the mouse if over the single line diagram, obviously the pan zoom is using the scroll to zoom in and out. When the mouse is outside the single line diagram, scrolling leads to disappearing tabs. I think in the single line diagram we should only allow scrolling for pan zoom and block it otherwise.

As an engineer
I want of have a fixed tabs bar in case of navigation
So I don't have to look for the tabs
Acceptance criteria:
The tab bar should stick when scrolling down.
The scrollbar in combination with the zoom function is the issue here:

@ca-d can you check if this now reflects our discussion of this morning?

Thanks @Sander3003 , it does.
This may even be as simple as a different overflow css property on the containing element.


Add trailing slash to a few more URLs in the Get Started index page.

Proposed changes
Adds a trailing slash to the URLs in the Get Started layout page. You can get to this page from PRODUCT (nav item) > (under Migrate to Pulumi header) click on AWS CloudFormation or Terraform. Scroll-down the "Get Start with Pulumi" section.
Unreleased product version (optional)
N/A
Related issues (optional)
N/A

Mind changing these to relref while making changes here?

Yep. I'll accept your suggestions. I thought about using relref's but wasn't sure if these were specifically not using relref for some reason.


Make Bicep extensibility production ready (phase I)

Creating the issue to track the tasks for making Bicep extensibility production ready.
Plan Items
Legend of annotations:

Mark
Description

🏃
work in progress

🔵
more investigation required to remove uncertainty

[x] 🏃♂Publish a NuGet package for the first-party providers which will be consumed by the extensibility host

[x] 🏃♂Finalize the Kubernetes provider
[x] Set up official build CI for signing the NuGet package
[x] Publish the NuGet package

[x] 🏃♂Migrate the extensibility host to ASP.NET Core application running on a Service Fabric cluster

[x] Update EV2 to configure load balancer and test it
[x] Update ARM config to route extensibility requests to the host (test in DF first)
[x] Set up auth (ARM -> host)
[x] Set up MDM and MDS
[x] Set up IcM

[ ] 🔵Make What-If work for extensible resources (to be discussed)
[ ] 🔵Simplify the way types are generated for first party providers (to be discussed, needs feedback)

Adding partner asks for extensibility that need to be triaged:

#6822
#6825
#6824
#6731
#6652
#6628
#6627

Some notes from the discussion today:

To enable the feature, we will allowlist for everyone on the ARM side, but we want to set the expectation that it's going to be experimental

We're OK with experimental warning only being shown with Bicep CLI (the current approach)

We will continue to use 1.9-experimental as the languageVersion
AzCLI works fine, but we should add a test to make sure it continues to work with extensible resource deployments
AzPowershell hits an error. We will need to fix up code to work with symbolic names
When announcing the feature, we should make it clear that symbolic name support will also be enabled
@alex-frankel will reach out to the Portal team to see if they want to catch & display a user-friendly error message when seeing languageVersion 1.9-experimental (rather than throwing an unintelligible JS error)

The kubernetes provider is now available to everyone!
Relevant docs are here:

QuickStart with AKS
Import keyword
Kubernetes provider "How To"

Given this is closed, how can we use this functionality to extend Bicep?

@alex-frankel when can we expect MS Graph/AAD support to arrive for bicep?

The kubernetes provider is now available to everyone!
Relevant docs are here:

QuickStart with AKS
Import keyword
Kubernetes provider "How To"

I can confirm deployment works nicely.
We tested web app pulled from docker registry in other tenant and WAF terminating and exposing HTTPS.
Bicep import of YAML Kube config files works nice in VSCode as well.
Good job out there!

@alex-frankel any ETA on this feature?
Interestingly, I did a bing chat search, and this is what I got -
https://sl.bing.net/dtJMMoQ1bUG

Here's an example of a Bicep script that sets an Azure AD service principal app role assignment using Microsoft Graph ¹:
param tenantId string
param appId string
param appRoleName string
param principalId string

resource appRoleAssignment<EMAIL_ADDRESS>= {
  name: guid(principalId, appRoleName)
  properties: {
    principalId: principalId
    resourceId: appId
    appRoleId: '[${appId}/appRoles[?value==\'${appRoleName}\']].id'
  }
}

You can use this script as a starting point and modify it according to your needs.
Let me know if you have any other questions!
Source: Conversation with Bing, 4/28/2023
(1) Create Azure RBAC resources by using Bicep. https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/scenarios-rbac.
(2) Quickstart: Assign an Azure role using Bicep - Azure RBAC. https://learn.microsoft.com/en-us/azure/role-based-access-control/quickstart-role-assignments-bicep.
(3) azure active directory - How do I use Bicep (or ARM) to create an AD .... https://stackoverflow.com/questions/69120936/how-do-i-use-bicep-or-arm-to-create-an-ad-app-registration-and-roles.
(4) azure-docs/how-to-assign-app-role-managed-identity-powershell ... - Github. https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/active-directory/managed-identities-azure-resources/how-to-assign-app-role-managed-identity-powershell.md.
Is this something you guys are working on? we tried it out and the resource type is not yet supported in bicep.

If you are asking about support for the Graph provider, this is tracked here:
#7724


Add volumes type - emptyDir

Quite strange, you have an initContainer, but no volumes with the emptyDir type. Please consider adding it.
volumes:
  - type: emptyDir
     name: app
     sizeLimit: 500Mi
     medium: Memory

...

volumes:
  - type: emptyDir
     name: app

Hi, @abix5.
You don't need to use typed volume if you need emptyDir. Just use extraVolumes.
extraVolumes:
  - name: cache
    emptyDir: {}
  - name: cache-limited
    emptyDir:
      sizeLimit: 500Mi
      medium: Memory

btw why not to add this option
    volumes:
    - name: uuid
      type: emptyDir
      # sizeLimit: 1Mi
      # medium: Memory

see MR #46

What advantage would be gained if we used a typed volume, for this type, instead of an extra volume?

Because it strange.. Do not use and not support basic things for emptyDir and support all other common options:
    - name: secret-files
      type: secret
    - name: app-pvc
      type: pvc

What method is easy with or without extraVolumes? I'm sure if all volumes will be defined in one section, would be easy to maintain it.
@randreev1321 Please approve the MR


Arabic Language Support

Description:
The current STORM code supports generating Arabic articles. I verified this by integrating the Bing API and GPT-4. The following command was used to generate Arabic articles:
python examples/run_storm_wiki_gpt.py \
    --output-dir $OUTPUT_DIR \
    --retriever bing \
    --do-research \
    --do-generate-outline \
    --do-generate-article \
    --do-polish-article

After inputting an Arabic title, The output was  Arabic article, and I verified that Arabic articles function in the UI version too (screenshot attached).

However, there are some remaining challenges and areas for improvement:

Arabic Input Support: The deployed version currently lacks support for Arabic input.
UI Enhancements: Adjustments are needed to handle Right-to-Left (RTL) languages, resolving text alignment and display issues.
Improvement of Arabic Article Quality: There is a need to further enhance the overall quality of the generated Arabic articles.

Thanks for trying it out! It's very interesting to see you have an initial working version for Arabic.

The deployed version currently lacks support for Arabic input.

We don't plan to deploy multilingual version at present.
@DulayelNotaifi, Do you know any forum that we could probably reach out to more developers who know Arabic?

Unfortunately I don't know a specific forum


Add adoption variant to run minimal without ceph

Add a variant of the cifmw-adoption job to run with a swift backend and
run the test-minimal test-case.
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/1330

recheck

recheck


Add proxy support -- I saw many  proxy request had been closed

This tool is so good, however , I hope this tool can use proxy for network communication.
Without proxy support , I almost can not connect to udemy to download course ,or just very slowly.

@tlsalex considering this for next version so closing the issue without any issue :P

Bump this issue to get more attention. my pc can get 3MB/s+ to download a udemy course if using a paid proxy, but without proxy only 20~KB/s


Fix inception v1 preprocessing steps example

Set up imports and print statements to conform with python3 syntax. Print statement syntax was a mix between python2 and python3 syntax

Thank you for the PR.
It works now with python 3 now but gives an error with python 2. (No module named request).
Can you update the PR so that it checks the python version and then imports accordingly (urllib or urllib.request)

See PR #217
Declining this one now.


Add create feedstock PR workflow

Use a github action to instantly create a feedstock pull request when our library is release on PyPI
Uses this workflow action: https://github.com/alteryx/create-feedstock-meta-yaml

@ParthivNaresh

To trigger a workflow in a repository, the user should be a collaborator with Write permission in the repository. Normally the external users can’t trigger workflows in the repository.

Source


best way to handle ServerError and ServerUnavailable Exception

I usually have a queued job to process sending a push notification. When the job gets a ServerError or ServerUnavailable Exception, it will retry immediately until it runs out of the number of tries.
Ideally, if the job gets the ServerError or ServerUnavailable. It should wait for 30s before trying again. I am not sure how to do this? Any suggestions?

Is the retryAfter behavior in the job class perhaps what you‘re after?
https://laravel-news.com/job-based-retry-delay

@jeromegamez I ended up using
       try {
            app('firebase.messaging')->send($message);
        } catch (NotFound $e) {
            $this->device->delete();
        } catch (ServerError $e) {
            $this->release($this->attempts() * 10);
        }

I have some weird issue with retryAfter where the retryAfter time does not increment by the number of attempts.
This brings me to the next question: how to try and catch for all app('firebase.messaging')->send($message); without copy and paste try/catch where I  use app('firebase.messaging')->send($message);

You can catch the MessagingException interface (https://github.com/kreait/firebase-php/blob/main/src/Firebase/Exception/MessagingException.php).


warning in cache.cc

In the Chrome build:

third_party/leveldb/util/cache.cc(115) : warning C4018: '<' : signed/unsigned
mismatch

Code looks like:

    memset(new_list, 0, sizeof(new_list[0]) * new_length);
    uint32_t count = 0;
    for (int i = 0; i < length_; i++) {   // L115
      LRUHandle* h = list_[i];
      while (h != NULL) {

length_ is a uint32_t.  Probably just need to change i to uint32_t.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 13 Jul 2011 at 11:46

Original comment by<EMAIL_ADDRESS>on 17 Jul 2011 at 5:35

Changed state: Accepted

Evmar, these should be fixed in r47.  Should get rolled into chromium when
http://codereview.chromium.org/7522008/ is committed.

Original comment by<EMAIL_ADDRESS>on 6 Aug 2011 at 12:27

Changed state: Fixed

Some additional warnings of the same flavor:

third_party/leveldb/db/version_set.cc(105) : warning C4018: '<' :
signed/unsigned mismatch
third_party/leveldb/db/version_set.cc(266) : warning C4018: '<' :
signed/unsigned mismatch
third_party/leveldb/db/version_set.cc(297) : warning C4018: '<' :
signed/unsigned mismatch
third_party/leveldb/db/version_set.cc(435) : warning C4018: '<' :
signed/unsigned mismatch
third_party/leveldb/db/version_set.cc(526) : warning C4018: '<' :
signed/unsigned mismatch

Original comment by<EMAIL_ADDRESS>on 13 Jul 2011 at 11:47


MWPW-135447 Use .html for all bacom links

Set the useDotHtml flag for all of bacom.
Requires this milo pr: https://github.com/adobecom/milo/pull/1148
Resolves: MWPW-135447
Test URLs:

Before: https://main--bacom--adobecom.hlx.live/?martech=off
After: https://usedothtml--bacom--adobecom.hlx.live/?martech=off

Codecov Report

Merging #144 (446e5fa) into main (6776c48) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #144   +/-   ##
=======================================
  Coverage   97.57%   97.57%
=======================================
  Files           8        8
  Lines         702      702
=======================================
  Hits          685      685
  Misses         17       17

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Status of the project

Hi!
I would like to know if there are any development plans for the project or has it reached a stable state?
I found it looking for alternatives to Katacoda and despite having used play with docker a lot in the past, it never occurred to me that it was open source, besides that it seems to me extremely useful, even able to compete with Katacoda itself as an open source solution.
I'm reading the code and playing with my infrastructure, I'm still having some problems (login / authentication, dns and metrics for now, but I'm investigating besides studying the DIND itself) and I would like to know, if the project is still under development, is there anything I can contribute to?
What I noticed is that the documentation would need an update (I managed to create the infrastructure by reading some old issues), besides that, do you have something in mind?
Other things I thought about was login based on LDAP.

I would like to know if there are any development plans for the project or has it reached a stable state?

Nope, we're still adding features but a very slow pace. You can see from the commit history that it's still being supported (https://github.com/play-with-docker/play-with-docker/graphs/contributors) even though it's been a while that I haven't worked in any major features due to lack of time and contributors.

even able to compete with Katacoda itself as an open source solution.
Katacoda tries to compete with PWD  :P

and I would like to know, if the project is still under development, is there anything I can contribute to?

There are several issues opened and also users asking for documentation. There's several places where contributions are welcome

Other things I thought about was login based on LDAP.

Happy to accept PR's. We already have some Oauth login mechanisms and I believe that if there's interesting in adding other ones it'd be a good moment to start thinking about integrating a generic authentication provider that could potentially handle multiple providers like LDAP, Oauth, etc.

I would like to know if there are any development plans for the project or has it reached a stable state?

Nope, we're still adding features but a very slow pace. You can see from the commit history that it's still being supported (https://github.com/play-with-docker/play-with-docker/graphs/contributors) even though it's been a while that I haven't worked in any major features due to lack of time and contributors.

even able to compete with Katacoda itself as an open source solution.
Katacoda tries to compete with PWD  :P

and I would like to know, if the project is still under development, is there anything I can contribute to?

There are several issues opened and also users asking for documentation. There's several places where contributions are welcome

Other things I thought about was login based on LDAP.

Happy to accept PR's. We already have some Oauth login mechanisms and I believe that if there's interesting in adding other ones it'd be a good moment to start thinking about integrating a generic authentication provider that could potentially handle multiple providers like LDAP, Oauth, etc.

I'm sorry for the delay in responding.
I am happy to know that the project is active, in the short term maybe I will do a PR with infrastructure instructions (a little more robust than the one that exists today in README.md).
I believe it would be interesting to disable or remove the old setup described on the wiki.
I will also take a look at what I can contribute with code, I will probably see the LDAP issue because it may be a future need of mine.

Cool, thx. I'll be around if you come up with any issues.

Cool, thx. I'll be around if you come up with any issues.

Thanks!


Styles

Not sure any of this still makes sense with the button and import-dat input moved into dat-header.
Might want to just keep this one: https://github.com/juliangruber/dat-desktop/commit/5b297d491c13790d8236749b574ecd99305fefc9

i think it's smart to move them into smaller/different components since you said they'll diverge from eachother

what, i did not add that in progress label oO

IMO this still makes sense, this is the app, this has the app specific styles

@Kriesse what's the current state of this, with all the changes in dat.land?

@juliangruber Yeah let's ditch this one.


Inquiry: Making a programming language for sparkscript

I've been trying to work on my own programming language for DiamondFire. Still, it's really going nowhere, so I was thinking of collaborating with sparkscript so I already have a foundation to work off of.
I was also thinking of making a package manager for people to publish their actions sort of like the code vault, I already have an idea for packages, to save space they will just be a folder on supabase with an actions.json file and a spark.yaml file with package data, such as version and name
Packages cannot depend on other packages because it would cause versioning issues, I might add this later but at the time being packages are built dependency-less

hello sorry for the late reply i haven't looked at the repo page in a while.
that sounds like a neat idea, you could have a wrapper around sparkscript to make stuff like that
you're free to make whatever you want with sparkscript, as long as its not with malicious intent.
sparkscript will probably not be lightweight any time soon because i have to bundle the actiondump file with it to extract types and type intellisense for convenience.
and it would be almost done if not for the fact that DiamondFire has duplicate actions for separate codeblocks 😑

@mavthedev should probably send a mention..

talk to me on Discord because i'll have to close this issue

@GeorgeRNG @silvxrcat
you too @RedVortexDev

make it dfonline
I think we would have a system to authenticate users which I have tried to make with dfonline once, and I could see if I can get dfonline to have proper system for packages

so first off, the programming language itself is a large obstacle. we'll have to assign keywords and feed that into a tokenizer, then parse the output & make it stack oriented so that we can handle operations correctly. we should find a syntax we like.

~make it dfonline~ I think we would have a system to authenticate users which I have tried to make with dfonline once, and I could see if I can get dfonline to have proper system for packages

if dfonline is what we're going to use to host the packages, that's perfectly fine, but i'm getting some errors loading a template.

we should make sure to give an error if the template fails to load

Interesting, a programming lang for df with a pm stored on dfonline, truley the most epic collaboration of all

Though this could be of use: craftinginterpreters.com

A tutorial for making a programming language with tokenizers in java

lets keep the topic about the package manager
the programming language is a bit out of scope

the package manager sounds like a pretty sick idea
being able to import library templates and having intellisense and autocomplete
as for defining the library templates themselves i have thought about that

the package manager sounds like a pretty sick idea
I tried to make a language with a package manager, I didnt really finish it bc I got bored, but it used supabase and gzipped the package files

being able to import library templates and having intellisense and autocomplete
That's way more work 🫤

https://craftinginterpreters.com

pretty sure we all know Java
AHHEM 😬

Also there should be some sort of export keyword that is ignored at compile time but is used by the pm to check what should be accesible.

being able to import library templates and having intellisense and autocomplete

That's way more work 🫤

i was referring to sparkscript not the language
you could essentially do something along the lines of npx sspkg add library_name
and it would import the code for that library and in js do something like:
const df = require("sparkscript").uses("library_name");
const template = df("example", e => {
    e.action.someActionInTheLibrary(/* args */);
});

as for the language, idk

being able to import library templates and having intellisense and autocomplete

That's way more work 🫤

i was referring to sparkscript not the language
you could essentially do something along the lines of npx sspkg add library_name

Ahhh makes sense 👍🏾

i forgot to mention this but i will allow the ability to compile code from other languages such as Spark (till v2 comes out)  and Kindling and Blackstone


code refactorings

Unfortunately, class properties are not ES6 and even less a standard yet, it's only a proposal at stage 2.
I would not recommend using this until stage 3 at least.

arrow functions looks better than .bind()
there are a lot of people using class properties on react native projects, we can easily change this if something goes wrong and it did not reach stage 3

I understand what you mean and you're right, arrow functions looks better than .bind().
However, a lot of people doing something doesn't necessarly justify it. We're talking about a router library and for the sake of stability I'd like to not rely on babel quirks. Issues are growing up since few months and few contributors have time to help, adding this to the pipeline without a proper description, nor even a test plan seems a bit crazy to me.
Using this in a project? Go ahead.
In a library that many users rely on? Sorry, but that will be a no for me.
@stereodenis Anyway, thank you for your contribution it's highly appreciated.
I'll let @aksonov close or merge this pull request, it's his library after all.

@aksonov ?

@charpeni @aksonov I had rebased branch

@charpeni I don't understand advantage of this refactoring...

@aksonov

readability
new modern syntax
less code

Sorry, but
renderLeftButton = (navProps) => { is not more 'readable' than renderLeftButton (navProps) {
The only one reason to do it is to avoid manual binding, but it doesn't make code more readable...

@aksonov yep, https://daveceddia.com/avoid-bind-when-passing-props/

Way #4: Class Instance Field With an Arrow Function (good, ES8+)
Even though this is supported by Babel, there’s a (small) risk that this feature could be taken out of the spec and require some refactoring, but a lot of people are using it so it seems likely that it’ll stay put.

Way #5: Binding in the Constructor (good, ES6)

As I said in my previous message, for the sake of stability I'd like to not rely on babel quirks. (ES8+).


Code order: tool, signal, variable, function, etc...

https://github.com/Poobslag/turbofat/wiki/guidelines#standard-guidelines
Reordered code to conform to our guidelines. Ran a series of regexes to catch problems throughout our entire codebase.

Most of these ordering issues are trivial code readability things, but some of them actually cause compilation errors when converting to Godot 4 (the tool script declaration)


Font primary script utilities

This adds two things:

a primary_script function in gftools.utils which finds the primary script of a font, optionally ignoring Latin characters. It does this by categorising all encoded glyphs by their script and also applying the same script to any other glyphs produced by substitutions. It also rewrites the autohint_script_tag function to use that new function.
a gftools-set-primary-script command line utility which walks a directory tree looking for METADATA.pb files and sets the primary_script field of the METADATA to the primary script of the font.

I have this error
(env3) MacBook-de-Rosalie:fonts rosalie$ gftools set-primary-script ofl/ibmplexsansdevanagari
Traceback (most recent call last):
  File "/Users/rosalie/Google/env3/bin/gftools-set-primary-script.py", line 55, in <module>
    main(args.directory)
  File "/Users/rosalie/Google/env3/bin/gftools-set-primary-script.py", line 37, in main
    script = primary_script(TTFont(first_font), ignore_latin=False)
  File "/Users/rosalie/Google/env3/lib/python3.9/site-packages/gftools/utils.py", line 530, in primary_script
    g = classifyGlyphs(lambda uv:list(ftunicodedata.script_extension(chr(uv))), ttFont.getBestCmap(), gsub=ttFont.get("GSUB"))
  File "/Users/rosalie/Google/env3/lib/python3.9/site-packages/ufo2ft/util.py", line 284, in classifyGlyphs
    glyphSets.setdefault(key, set()).add(glyphName)
TypeError: unhashable type: 'list'

You have an old ufo2ft.

@simoncozens doesn't seem to work on Tiro Indic fonts (except Tiro Telugu) + Some IBM Plex (hebrew, thai and thai looped)

This is hard to know how to detect. For example, Tiro Tamil contains 168 Tamil glyphs and 723 Latin glyphs. It's definitely a Latin font with a subsidiary Tamil. :-)

I guess if the font name contains the script name, we can assume the primary script?

The script is setting "Hani" for all CJK families, but it doesn't seem to work in sandbox. I gonna test "Jpan" for Japanese instead

As a result of my test,
Japanese fonts (full set) : "Jpan"
Hangul: "Kore"
For now we don't have traditional chinese families other than Noto, but I would expect "Hant". I tested "Hans" in simplified chinese fonts but dev-sandbox is not updating, so I don't know.
"Hina" (for SC and TC), "Hira" (for Hiragana) and "Kana" (Katakana only fonts) don't work.

@m4rc1e I'd still like this to be merged, especially since primary_script is now becoming more of a thing.

+1 for merging this. I'd like to implement a fontbakery check for primary_script and wouldn't want to duplicate the code, rather use gftools directly to generate primary_script and see whether it's missing from METADATA.pb

I tested the bare primary_script() method on a number of Arabic and South Asian fonts and found it to work reasonably well. It only failed to produce a primary script for Alkatra because it contains a total of four scripts incl. Latin, which I guess is expected after studying the code.

I have another wish:
Can a third result type be added to primary_script() where it determines that no script comes out as a clear winner, but the overwhelming majority of glyphs are non-Latin?
I want to use this method directly in the upcoming related fontbakery check to avoid code duplication, and I wouldn't want it to fail silently (primary_script() returning None) when it’s clearly a non-Latin font, like Alkatra (linked above), which supports three South Asian scripts in addition to Latin.

Update: I just realized that gftools is not a dependence of fontbakery, and I’m guessing you want it to stay like that, so I would have to duplicate the code into fontbakery anyway. So never mind, I can probably add my desired feature myself then.

I don't mind getting this across the line today.
I'm wondering, do we expect every new family to use the primary_script field? If so, I think we need to add it gftools add-font as well.

Only the new families that are non-LCG

Awesome thanks. I've just hooked this up to gftools add-font. If the family's metadata doesn't have a primary_script field, it'll add one if the primary isn't Latin, Greek or Cyrillic.


feat: update family group urls to household

Describe your changes
2350: https://pins-ds.atlassian.net/jira/software/c/projects/ASB/boards/166?selectedIssue=ASB-2350
Update family group urls to household.
Type of change 🧩

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

Checklist before requesting a review

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

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

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

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


Does not work with Debian 9

Hey everyone! @sprotheroe
I've created a repo that creates a Vagrant VM with Debian 9 with some additional packages. The repo is https://github.com/pedrofurtado/vagrant-workspace
Following the instructions, the VM will be created successfully. But, the disk does not have the size defined by plugin.
You can check this typing df -h, inside the vagrant VM, and see that /dev/sda does not have the size configured.
Any ideas about why this happens with Debian? Because here at company we have a very similar repo that uses Ubuntu, and it works well.

I am using Vagrant 2.1.3, Debian 9 and plugin at latest version (at the moment of this issue).

and virtualbox 5.2.16

I think I see the issue. The Debian9 images includes a swap partition at /dev/sda2, right in front of the the filesystem partition at /dev/sda1. (By comparison the Ubuntu18 image only has a single /dev/sda partition that takes up the whole disk.)
The Vagrant plugin does resize /dev/sda on the image. (This can be verified with fdisk /dev/sda). However /dev/sda1 (the root partition) cannot be resized to take advantage of this added space, because /dev/sda2 is sitting in front of it.
This suggests one of two approaches to fix this problem going forward. One's from the users' side, one's from the plugin side. The first is just to have the user remove the swap partition, then re-rerun the plugin. (Even better would be to have the Debian image maintainer remove swap to begin with. It's mostly useless to have swap space in a VM to begin with. If a VirtualBox user needs more memory, they can just allocate more to the guest, and let their host machine take care of virtual memory allocation.)
The second is to enhance the logic in this plugin. Quite rightly it doesn't go around reformatting hard disks and moving partitions, because of the risk of nuking data. But a swap partition, is inherently ephemeral. There's no risk to repartitioning the disk and moving swap to the end of disk, because there's no persistent data to risk losing. The plugin could check partitions for type, and move non-persistent partitions around by default, if needed to make room for resized root partitions.

I agree that it does not make sense to have a swap partition here. Following https://app.vagrantup.com/debian a bug should be reported at https://tracker.debian.org/pkg/cloud.debian.org

As a workaround, I changed my repo to use Ubuntu 18.04 LTS, that creates the disk with correct size :+1:

The Ubuntu image has its own issues (slow startup, weird serial port thingy) so I'd rather like to have this fixed for Debian.

+1 for debian 10


[FEATURE] Provide an API documentation

Should be good for developers to provide an API documentation similar to this: https://xstate.js.org/api/
Intended to describe objects, types, functions, expected parameters and so on.

Thanks for the suggestion, updating our documentation is an on going process.


Generate enum casting helper functions (extensions) from prisma enums

Thanks for the great library!
The biggest drawback we currently face using kysely, this extension, postgresql and prisma is, that it is required to wrap every Postgresql enum before inserting/filtering. If we forget to do so, the query throws.
As this article shows, it is quite easy to add extensions to kysely by for example wrapping a sql tag around a variable to cast something.
Like this: sql${ProductStatus.ARCHIVED}::product_status,
Wouldn't it be super helpful if a set of helper functions would directly be generated based on the Prisma schema's enums?
We are currently building these wrappers manually which is prone to erros, especially when renaming an enum.

Hey! I don't think I've ever come across this issue if I'm totally honest. But I'd totally be willing to consider a pull request that provides a clear example and an implementation 😄

Using kysely for the first time and ran into this same issue. Very possible I'm doing something incorrectly but seems that raw Prisma Enums cannot be used in SQL without an explicit cast. My example...
This query:
await prisma.$kysely.deleteFrom("SomeTable")
        .where("relatedEntityType", "=", EntityType.User)
        .execute();

Causes this error at runtime:
Raw query failed. Code: `42883`. Message: `ERROR: operator does not exist: "EntityType" = text
HINT: No operator matches the given name and argument types. You might need to add explicit type casts.`

Changing this query to:
await prisma.$kysely.deleteFrom("SomeTable")
        .where("relatedEntityType", "=", sql<EntityType>`${EntityType.User}::"EntityType"`)
        .execute();

Works just fine but is verbose and means we have to be super vigilant about not using enums directly. Is there a better way? I'd be happy to contribute a PR if what @luap2703 suggests is the best path forward.

Indeed, same here.
Another common use case is relying on the DB to generate UUID for id columns.
It means, each time you want to filter a query by id, you'll need to cast the id to UUID:
await prisma.$kysely.selectFrom("MyTable").where("id", "=", sql`${entityId}::uuid`

Hey guys, I feel like this issue isn't directly related to prisma-kysely. It probably belongs in the https://github.com/eoin-obrien/prisma-extension-kysely repo (which uses prisma-kysely under the hood). The prisma-kysely package only really relates to generating Kysely schema definitions from Prisma schemas, and as of right now doesn't at all touch runtime code.
Helpers and such should probably be suggested on the prisma-extension-kysely repo.
Thanks :D


[ADD] Change Fisher to purely PyTorch

Review after #142, #145, #146, #147.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

5 of 11 (45.45%)  changed or added relevant lines in 2 files are covered.
70 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-3.2%) to 89.542%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

curvlinops/_torch_base.py
0
1
0.0%

curvlinops/fisher.py
5
10
50.0%

Files with Coverage Reduction
New Missed Lines
%

curvlinops/examples/utils.py
10
31.25%

curvlinops/_torch_base.py
20
89.61%

curvlinops/fisher.py
40
25.76%

Totals

Change from base Build<PHONE_NUMBER>1:
-3.2%

Covered Lines:
1524

Relevant Lines:
1702

💛  - Coveralls


[executor] idempotent commits

Motivation
the implementation of idempotent commits support described in #1590
Test Plan
unit test added.
Related PRs
Need to rebase once #1620 lands.

:umbrella: The latest upstream changes (presumably 9d510ae3717935dbd3f89cf6f880a7dca9ca80c4) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r=dmitri-perelman

:pushpin: Commit 0725986 has been approved by dmitri-perelman

:hourglass: Testing commit 072598603e3151a14c06745f920ecf82d5d1cf57 with merge 11893f812523c855343bd57754b3637f6be9b090...

@bors-libra r=dmitri-perelman

:pushpin: Commit 3687915 has been approved by dmitri-perelman

:hourglass: Testing commit<PHONE_NUMBER>c6b17b9ff94093e67058f2b457b480 with merge 19f051796e97ca6cd22b9563b0a1d2932c3ad585...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: dmitri-perelman
Pushing 19f051796e97ca6cd22b9563b0a1d2932c3ad585 to master...


Add Bower support

It would be great if this plugin could be installable using Bower.

Thank you for the suggestion.
I don't know about Bower, but I'll take care about your issue when the plugin will be stable.
Alain

Thanks, Alain.

Added Bower support in https://github.com/ninsuo/symfony2-collection/commit/72582c4bbf5ac9f7453fa4584d930dac59f28ed8
Thank you daFish!
Alain


Add Remove instances for Set and Map

Adds Remove instances for Set and Map. I have tried to follow the same pattern that's used for the Append instances, with the exception that for Map we only remove the keys, like in the stdlib methods.

Hi @DavidGregory084,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:
https://www.lightbend.com/contribute/cla

@eed3si9n just waiting for our legal folks to get back to me about the CLA 👀

Sorry for the long delay on this one @eed3si9n, my request to review the CLA got lost in somebody's inbox for a while 😆

ok. It's all green, so I am going to merge it.


METRON-979 markdown errors in site-book, part 3

Contributor Comments
Due to an integration error on my part, one editorial glitch remains, in the form of an unresolved merge fragment in metron-deployment/Kerberos-manual-setup.md. This PR is to fix this.
Pull Request Checklist
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes: NA
For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book?

+1 by inspection

Thanks, @JonZeolla .


fix: add unpack logic after container commit to images.

May fix this: https://github.com/containerd/nerdctl/issues/827

test commands and results:
nerdctl -n k8s.io commit 555dd4b423742 lingdie/commit:dev
WARN[0000] Image lacks label "nerdctl/platform", assuming the platform to be "linux/amd64"
sha256:55920a9d373f7778901f57501cf15cb5233f69ceadc38ceae827bc7404455643

nerdctl -n k8s.io inspect lingdie/commit:dev
[
    {
        "Id": "sha256:55920a9d373f7778901f57501cf15cb5233f69ceadc38ceae827bc7404455643",
        "RepoTags": [
            "lingdie/commit:dev"
        ],
        "RepoDigests": [
            "lingdie/commit@sha256:3684882f267f612a499de1786a627e216b8db0510fede8d817429606e64f546d"
        ],
        "Parent": "",
        "Comment": "",
        "Created": "2024-08-01T08:07:58.833742114Z",
        "DockerVersion": "",
        "Author": "",
        "Config": {
            "AttachStdin": false,
            "Env": [
                "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
            ],
            "Cmd": [
                "/bin/bash"
            ],
            "WorkingDir": "/root",
            "Labels": {
                "org.opencontainers.image.ref.name": "ubuntu",
                "org.opencontainers.image.version": "24.04"
            }
        },
        "Architecture": "amd64",
        "Os": "linux",
        "Size": 87519232,
        "VirtualSize": 87519232,
        "RootFS": {
            "Type": "layers",
            "Layers": [
                "sha256:a30a5965a4f7d9d5ff76a46eb8939f58e95be844de1ac4a4b452d5d31158fdea",
                "sha256:5f70bf18a086007016e948b04aed3b82103a36bea41755b6cddfaf10ace3c6ef",
                "sha256:97b66fcb143832ac7d953ff66d09c77c2ba31290d3aa422b2bc27762f55e0a73"
            ]
        },
        "Metadata": {
            "LastTagTime": "0001-01-01T00:00:00Z"
        }
    }
]

nerdctl -n k8s.io push lingdie/commit:dev
INFO[0000] pushing as a reduced-platform image (application/vnd.docker.distribution.manifest.v2+json, sha256:3684882f267f612a499de1786a627e216b8db0510fede8d817429606e64f546d)
manifest-sha256:3684882f267f612a499de1786a627e216b8db0510fede8d817429606e64f546d: done           |++++++++++++++++++++++++++++++++++++++|
config-sha256:55920a9d373f7778901f57501cf15cb5233f69ceadc38ceae827bc7404455643:   done           |++++++++++++++++++++++++++++++++++++++|
elapsed: 4.4 s                                                                    total:  2.5 Ki (579.0 B/s)

It's not easy to add tests. The environment corresponding to the issue is to commit the pod container in k8s. I cannot create a test environment and test cases from the existing tests.
It would be great if you could provide me with more information.
And please rerun ci, some ci tests failed because of docker.io 502 error 😢...

It's not easy to add tests. The environment corresponding to the issue is to commit the pod container in k8s. I cannot create a test environment and test cases from the existing tests.
It would be great if you could provide me with more information.

Ah!
Yep, we should probably have a kube testing environment.
Do you have an opinion on what we should go with to help with that? kind?
I recently cleaned-up that part in nydus, so, maybe we could reuse some of that stuff to spin-up our env:
https://github.com/containerd/nydus-snapshotter/tree/main/tests/helpers

And please rerun ci, some ci tests failed because of docker.io 502 error 😢...

I cannot do that, but @AkihiroSuda can :-).

@lingdie now we can test for Kube.
Here is an example:
https://github.com/containerd/nerdctl/blob/main/cmd/nerdctl/container_commit_linux_test.go#L28-L70

sudo /usr/local/bin/nerdctl  -n k8s.io push kubesharkwork
FATA[0000] failed to create a tmp single-platform image "docker.io/library/kubesharkwork:latest-tmp-reduced-platform": content digest sha256:a58ecd4f0c864650a4286c3c2d49c7219a3f2fc8d7a0bf478aa9834acfe14ae7: not found

@ouyangningdong https://github.com/containerd/nerdctl/issues/827#issuecomment-2300594324


auto apply_theme to mo.ui.data_explorer based on the notebook display theme

Description
The figures in mo.ui.data_explorer are always light.

Suggested solution
Change the plotting theme to dark when the display theme is dark
Alternative
No response
Additional context
No response

hey @metaboulie - i agree, we can make this darktheme compat. should be 2 lines to change, and only in the frontend.

ill see if GitHub Copilot Workspace can actually fix this


arhena.read_sql_query() returns rows  ordered randomly

Data
my dataset is here.
data.gz
Query
my query is here. (replace the <your-database-name> and etc with yours)
returned rows should be ordered by timestamp column. (please check sql last line)
SELECT stockcode, price, volume, timestamp, isBull as is_bull FROM (
  SELECT row_number() over() AS rn, * FROM (
    SELECT * FROM <your-database-name>.<your-table-name>
      WHERE  "$path"='s3://<your-s3-bucket-name>/data.gz'
      AND (stockcode='057030')
      AND (true)
      ORDER BY from_iso8601_timestamp(timestamp), sortkey
  )
) ;

Bug Description
I executed wr.athena.read_sql_query() with above query and  every time I've got rows which is ordered randomly.

but the query works well in athena console. it is ordered by timestamp well as I intended

Hi @seyoongit, pass ctas_approach=False to turn off CTAS docs or use the solution described in #785 to force a single query result file.


Changing internal mem unit from bytes to Mb or Gb?

Right now when user specified mem='6G', SoS converts it to bytes 6000000 and users can do some calculation in the template file and say
#BSUB -M {mem // 10**9}G

It does not really matter but all the systems I am working on require memory in Gb, so it is more convenient if we provide mem in Gb or Mb.

I'm neutral to the change, considering the template is something one might not change once it is written. So users only have to think of bytes for just one time.

The problem here is inconsistency, as we provide walltime as HH:MM:SS (string) and mem as int (bytes). For walltime, one of the systems asks for HH:MM so I had to use walltime.rsplit(':', 1)[0], and for mem, one system asks for xxG and one system asks only xx in GB.
Whereas the current situation is certainly tolerable, I am wondering if we should make walltime and mem some objects so that users can do something like
walltime.format('HH:MM')

and
mem.format('Gb')

and
queue = { 'long' if walltime > '12h' else 'medium' }

The problem here is that whereas the existing rules are rudimentary and inconsistent, they are easy to remember, and the new interface can be confusing as they add something else to remember.

Well, although I appreciate the problems and the appeal of solutions, I still think to users it should not be a big deal since these properties in job templates are not going to be changed once created. Same arguments holds for voting to create those objects you proposed which are flexible; and the additional learning curve is totally tolerable again because they'll not be changed.

ok, let us keep things as they are. If STAR accepts parameters such as --whatever<PHONE_NUMBER>0 (I was counting zeros yesterday), perhaps it is not too bad to write #BSUB -M {mem //<PHONE_NUMBER>}G.


Add plugin to show trailing whitespace

It would be helpful to visualize trailing whitespace by highlighting it in red, maybe as an overlayed color or as an underline using lint+.

What do you think about lite-xl/lite-xl#908? Any other option you'd like to see?
Keep in mind, that PR only applies to the future 2.1 version and up.

drawwhitespace is very good with improvements 908, but the options are hidden in the code. It would be nice to have those options in the config editor.

I'd say open that issue on the primary lite-xl repo if you want to persue it @Titousensei .


BESM4eExtras v1.4.13

Changes / Comments
BESM4eExtras.html
In some cases, such as taking an attribute over level 6, will cause the attribute's cost to be double counted when applied to the group cost.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


[Bug]: 502 Bad Gateway causes project to deploy anyways

What happened?
Sometimes you get a 502 when deploying.
If you deploy again you might notice that it is waiting for file lock...
The first errored deploy turns out that it is actually building!
Fixing the 502 altogether would be nice, but having the deploy continue when the command exits immediately is confusing.
Version
v0.18.0
Which operating system(s) are you seeing the problem on?
In deployment
Which CPU architectures are you seeing the problem on?
In deployment
Relevant log output
$ cargo shuttle deploy
2023-06-07T23:21:07.645341Z ERROR cargo_shuttle::client: failed to connect to websocket
Error: could not connect to websocket

Caused by:
    HTTP error: 502 Bad Gateway

$ cargo shuttle deploy

... (normal build output, "Blocking waiting for file lock on package cache")
^C

$ cargo shuttle deployment list

╭─────────────────────────────────────────┬──────────────┬─────────────────────────╮
│              Deployment ID              ┆    Status    ┆       Last updated      │
╞═════════════════════════════════════════╪══════════════╪═════════════════════════╡
│ dbb0ebcd-7adb-40ae-a124-42be5960c9a5    ┆   building   ┆   2023-06-07T23:23:08Z  │
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┤
│ 49e009c2-d831-4c9d-832e-bc78636893c1    ┆   building   ┆   2023-06-07T23:22:34Z  │

Duplicate declaration

[X] I have searched the issues and there are none like this.

Closing due to being solved on the shuttle.dev platform.


Implement registry patching mechanism and use it to enforce pulling core from std source

Stack:

#358
#357
#350
#356
#355
#354
#353 ⬅
#352

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #355


fix(topology): Change toolbar text and spacing

Fixes:
https://github.com/janus-idp/backstage-plugins/issues/200
https://github.com/janus-idp/backstage-plugins/issues/199
Screenshot:

LGTM!


project badges api missing parameters

Greetings
GitLab Project badges API describes the field name that is missing in this project.  I was thinking to use this library but the name field is so important for me.
I think it's easy to do on the project_badges.go file. With your help, I'd like to fix this issue if you assign this to me.

By all means feel free to create a PR for it 👍🏻


Mirage files are loaded on development environment even when ember-cli-mirage is disabled

Version: 2.4.0
Given ember-cli-mirage is disabled on development environment :
// config/environment.js
if (environment === 'development') {
  ENV['ember-cli-mirage'] = {
    enabled: false
  };
}

And the Mirage config file :
// mirage/config.js
import myUtils from "../tests/routes/mirage-routes";

The EmberJS development environment is broken, the following error appears on every page :

Uncaught Error: Could not find module myapp/tests/utils/routes/get-entries imported from myapp/mirage/config
at missingModule (loader.js:247:1)
at findModule (loader.js:258:1)
at Module.findDeps (loader.js:168:1)
at findModule (loader.js:262:1)
at Module.findDeps (loader.js:168:1)
at findModule (loader.js:262:1)
at requireModule (loader.js:24:1)
at r (loader.js:176:1)
at resolveInitializer (index.js:10:1)
at registerInitializers (index.js:31:1)

This is probably because Ember does not load tests/ directory in development environmment.
I believe ember-cli-mirage should do the same with its directory when ember-cli-mirage is disabled.
I discovered this because I want to share route handlers functions between mirage/config.js file and my tests ( see https://github.com/miragejs/miragejs/issues/1022 ) like this :
// mirage/config.js
import getPosts from "../tests/utils/routes/get-posts.js"
function routes() {
  this.get("/posts", getPosts);
}

// tests/acceptance/application-test.js
import getPosts from "../utils/routes/get-posts.js"
// ...
test("shows current post title", function() {
  // ...
  this.server.get("/posts", function (schema, request) {
    if (request.queryParams["filter[current]"] === "1") {
      return post;
    } else {
      return getPosts(schema, request);
    }
  });
});

It indeed works in test environment but completely breaks the development environment.
Am I missing something ?
Thanks !

Given that getPosts accepts schema and request, I would say that is very much a mirage concern and suggest that you move the util from the tests directory and put a util in the mirage directory. I think would that would solve the problem.
Now for some questions. If you have mirage disabled in dev.  Is so, I would use this environment variable https://www.ember-cli-mirage.com/docs/advanced/environment-options#excludefilesfrombuild instead of enabled. Think that would also solve your problem because then even config.js would not be included in you dev build.  (although I do not know how this is going to look in an embroider world)
Given that you defined the route in the mirage config, is that a default, and in the test you are overriding that definition for that one specific for the test? If that route is used ONLY in that test, the you do not really need to define it in the routes if you are going to dynamically add it in the test.

Hi !

Given that getPosts accepts schema and request, I would say that is very much a mirage concern and suggest that you move the util from the tests directory and put a util in the mirage directory. I think would that would solve the problem.

No sure why it didn't work when I tried it, but it indeed works now :+1:

Now for some questions. If you have mirage disabled in dev.  If so, I would use this environment variable https://www.ember-cli-mirage.com/docs/advanced/environment-options#excludefilesfrombuild instead of enabled. Think that would also solve your problem because then even config.js would not be included in you dev build. (although I do not know how this is going to look in an embroider world)

This is indeed what I was looking for because I never use mirage in dev env, and it works, thank you !

Given that you defined the route in the mirage config, is that a default, and in the test you are overriding that definition for that one specific for the test? If that route is used ONLY in that test, the you do not really need to define it in the routes if you are going to dynamically add it in the test.

Yes, I have some default route handlers defined in mirage/config.js which is used by the majority of my tests, but for some cases, I override those handlers in specific tests. So yep, I need to define route handlers in the mirage config file too.
Thank you very much for your detailed answer, I can close this issue then :slightly_smiling_face:


chore: regular update

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

Project coverage is 89.13%. Comparing base (969153a) to head (bb39475).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #160      +/-   ##
==========================================
- Coverage   89.24%   89.13%   -0.11%
==========================================
  Files          15       15
  Lines         948      948
==========================================
- Hits          846      845       -1
- Misses        102      103       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Feature request: net_plugin needs to monitor peers activity

As experienced by EOSUSA, there are sometimes network conditions that keep the outgoing TCP connections open, but all incoming traffic is blocked in those sessions. So, nodeos considers the peer as connected and waits forever. Then the Linux kernel closes the socket after standard 15 minutes  timeout.
net_plugin needs to monitor the messages it receives from each peer, and if nothing has arrived within X seconds, it should close the connection. If it's an outgoing connection, it should start reconnecting. The period X should be configurable. As the blocks should arrive 2 times a second, default X=3 looks reasonable.

This was fixed in 2.1.


UI Minor bug

https://user-images.githubusercontent.com/1178856/215720054-c4174a09-d3e1-46af-8951-5e395af3fb4e.mp4

This happens when a user with a long NIP05 note is on the timeline or profile.
Ex: https://snort.social/p/npub1gw2veenjhjk0afmha9jslmlksc37svlp2rkagupp3nshdlpx8zfsf4nvxa


Add RefreshTokenLifetime setting to OpenIdSettings

By default refreshtoken lifetime used by OpenIddict library is 6 hours.
We need to add RefreshTokenLifetime setting to OpenIdSettings to allow users to set this value

By default refreshtoken lifetime used by OpenIddict library is 6 hours.

Ooops, sorry, the wiki page where you probably found this information was obsolete. The default refresh token lifetime was re-evaluated and was changed to 14 days: https://github.com/aspnet-contrib/AspNet.Security.OpenIdConnect.Server/issues/146

Milestone changed, as it's not critical for 1.0: the default value is quite reasonable and though not super easy, it's possible to change it by adding some custom configuration code in a custom module.

@kevinchalet any guidence on what and where we would put this custom code?
thanks


Fixes #525: Corrected Repo Header Link to go to Homepage

Fixes #525: Fixed the Repo Header link to go to home page.

I signed it!

@nworden Please review this

Hey Yash! Thanks for this fix!
I'll talk with our UXer about whether we ought to have a back button on the homepage at all: during crises, people often follow links from social media, etc. that go directly to the repo homepage rather than the global homepage, so maybe we ought to leave the back button off the homepage altogether, but that's a UX decision. If we keep the back button, having it go to the global homepage makes more sense to me than having it go nowhere, so I'll merge in this PR.

Sound's good @nworden . Also, I would like to know the perspective on static pages as well. Like should I create a HOC for all these components or create an interface from which they all get extended.

Just FTR, I continued the discussion on issue #478.


MINOR: Revert to Gradle 8.5

Rollback to Gradle 8.5.
When upgrading to Gradle 8.6, Scala incremental compilation seems to be broken. See https://github.com/apache/kafka/pull/15404#discussion_r1526739190
Committer Checklist (excluded from commit message)

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

@jlprat Thanks for the PR. It looks like the PR contains unwanted files.

Oops, you are right @dajac I don't know what happened. I'll check

I accidentally committed the core/data files from some failed test run.
@dajac feel free to review now. Thanks!

I managed to reproduce the issue with Gradle 8.6 once, but now that I'm trying to reproduce it again, it seems that incremental compilation works as expected (I tried it on trunk and modified kafka.admin.AclCommandTest.scala as well).
I think we should

Hi All,
Update on my original comment: issue with incremental compilation seems to happen only when changes made from IntelliJ.
Would anybody mind trying to reproduce it on your local environment? Steps to reproduce:

Open kafka.admin.AclCommandTest in IntelliJ Idea. I'm using 2023.3.5 Ultimate
Add println("a") at the beginning of testAclCliWithAuthorizer method
Run in terminal: ./gradlew core:test --tests "kafka.admin.AclCommandTest.testAclCliWithAuthorizer"
Result: all tests in core module are re-compiled

If test modified with vim, only single test class is re-compiled

Thanks @pasharik ! I was able to reproduce it using the steps you mentioned above.
I've a potential fix. Can you please try updating the zinc version in gradle/dependencies.gradle:166  to 1.9.6 and try the steps once again? It seems to fix things on my end.

@gaurav-narula I've tried zinc 1.9.6 with gradle 8.5 and 8.6, but getting the same issue with all tests re-compiled.
Probably one radical solution to this incremental compilation issue with Scala, can be re-writing the entire AclCommandTest in Java? :smile: I'm looking to add KRaft support for this test, so there will be lot of changes in it anyway, as I understand.  I see there are already some Java tests in core module

I'm restarting the build. The previous run I started this morning (CET timezone) got hung towards the end (it was running for over 6 hours).

Open kafka.admin.AclCommandTest in IntelliJ Idea. I'm using 2023.3.5 Ultimate
Add println("a") at the beginning of testAclCliWithAuthorizer method
Run in terminal: ./gradlew core:test --tests "kafka.admin.AclCommandTest.testAclCliWithAuthorizer"
Result: all tests in core module are re-compiled

I can't reproduce it and my Idea is 2023.3.6  Ultimate. I add some changes to testAclCliWithAuthorizer and then run ./gradlew core:test --tests "kafka.admin.AclCommandTest.testAclCliWithAuthorizer". Only file AclCommandTest.class get recompiled.

any updates here? gradle 8.7 has been released. we can update gradle to 8.7 if the issue is nonexistent

any updates here? gradle 8.7 has been released. we can update gradle to 8.7 if the issue is nonexistent

Update to 8.7 is ok with me. For now, I'm compiling my test with scalac as a workaround anyway

Update to 8.7 is ok with me. For now, I'm compiling my test with scalac as a workaround anyway

@jlprat thanks for response. ticket: https://issues.apache.org/jira/browse/KAFKA-16490

Closing this one, in favour of whatever comes out of https://issues.apache.org/jira/browse/KAFKA-16490


Governance Decluttered Webpage

Hi @karntrehan @prtkjakhar @riya-2206, creating this ticket for the creation of the governance decluttered webpage.  The wireframe is linked below:
Desktop: https://www.figma.com/design/A1hSB1v5szqlRrq7O3Ca7Q/Samagra-website?node-id=95-3003&t=jjbGLHmhB7lHLITX-4
Mobile: https://www.figma.com/design/A1hSB1v5szqlRrq7O3Ca7Q/Samagra-website?node-id=44-2264&t=jjbGLHmhB7lHLITX-4

Questions:

[ ] Where is the following going to come from?

[ ] Are these counts to be updated?

[ ] Is this to update on Instagram? If so, we would need to link to instagram. Is that high ROI?

Hey @karntrehan, responding to the questions:

We will provide the bifurcations of the reel in these categories.
Yes, the counts are to be updated
We would ideally like to link instagram here, as we do envision the website to become a high-frequency touchpoint for our TG.

On 2 & 3, would like to understand the effort and challenges on linking instagram with the website with count updations and actions on website getting updated on instagram. If it is a significant effort, we can reconsider. Please let us know, thanks!

Hey @karntrehan, for pt 1 above, providing the categorization in this sheet: https://docs.google.com/spreadsheets/d/1fr8FXk9R5UiuO8qxF_SB2G-KUppCkqD5_JMtcfmfA7U/edit?usp=sharing

Option 1: Manual Embed

OT create a post
OT adds Thumbnail
OT adds category
OT add embed link as well: https://developers.facebook.com/docs/instagram/embed-button

Pros:

OT can decide which reel to post on website.
OT can also post non reel content
Updated likes on website itself.
Medium dev effort
Support to play reel on website itself.

Cons:

Manual effort from OT

Option 2: Auto Oembed

Eng builds the OEmbed app and gets it approved from Meta
OT creates post
Oembed auto pulls post

Pros:

Auto population of reels - No manual effort.

Cons:

Posts will not contain categories
OT cannot decide which reel to post on website.
Highest dev effort
No play of reel on website.

Option 3: Redirect to Instagram

OT create a post
OT adds Thumbnail
OT adds category
OT adds caption
OT adds post link

Pros:

OT can decide which reel to post on website.
OT can also post non reel content
Smallest dev effort

Cons:

No updated likes / views on website.
No play of reel on website.

Load more to be clarified from OT. Option 2 is a go-ahead.

To show more than the first 9 videos in the grid, we can have a functionality similar to the button below, to ensure the same amount of vertical space is taken by the grid of videos at each point in time.

Hey @aayush-malik ,
thanks for the update. Can you also provide the link of the instagram profile which needed to be linked.
cc @karntrehan

Hey @riya-2206, link to IG: https://www.instagram.com/samagragovernance/

Feedback:

[ ] Move from video file to youtube link
[ ] Make full item clickable
[ ] Make 4 columns on web view
[ ] Add menu item under assets when page ready
[ ] Move categories to free text from a dropdown

@aayush-malik , will make our assets and decluttered page live, also shall I add decluttered page in the navbar and footer?

@riya-2206, please do add it in the nav bar & footer under 'Our Assets'. Thank you.

Hey @aayush-malik,  done.


get basemap/blue marble via MeshArrays

https://www.ncei.noaa.gov/thredds seems down, which leads to failed CI


sm3_mb:implement sm3_mb with sm3 extension

Implement SM3 with sm instructions

@gbtucker , please review

I'll take a look.

@haoyu01, can you rebase the sm3 series and resolve the conflicts?

@gbtucker  Okay

@gbtucker , the conflict was resolved . Please review


Test Error. KeyError: 'bninception_caffe'.

Hi @hkkevinhf, I was dealing with the same error,  I guess you used pip (seem a few commits behind master). I solved this by cloning mmcv (https://github.com/open-mmlab/mmcv) and install it in development mode. Cheers!

Hi @hkkevinhf ,
(Seems that some guy has answered this issue but the comment is gone.)
Are you installing mmcv from pip? The models are new and haven't been published in the latest release. Please clone mmcv from https://github.com/open-mmlab/mmcv and build it from dev mode. Thanks!


Latin squares is not enough

https://github.com/aidevnn/FastGoat/blob/3b0cf275f506371d3ce73ed46ada81e0dc9695e3/FastGoat/Group.cs#L190-L204
Associativity is not verified.
Latin square is cheap, complexity is O(n2) when Associativity is more costly O(n3)

UserGroups and their elements are by construction associatives. Writing a custom Loop of order 5 which isnt a Group is possible and fixing this issue isnt the priority at this moment.


Implement fleet naming style change in hm-diag

Acceptance Criteria

All fleet references must be compatible with https://github.com/NebraLtd/helium-miner-software/issues/385
Frequency reference has to be removed from root / endpoint (Region info will be kept)
Error related to missing FREQ var has to be removed from /initFile.txt endpoint
Remove status report for FREQ from /json endpoint
Remove all FREQ references including tests

Think we can close this now @KevinWassermann94 unless I'm mistaken?


Canvas drawing failed for the Sightlayer

Just made a new world. There is only Perfect Vision and lib-wrapper enabled and I'm getting this error:

This is looks like a cache issue. Follow the following instructions (CTRL+F5 should to the trick). Let me know if that fixes it.

Caching Issues
There is a known issue for users updating from V8 to V9 related to caching which causes some errors, UI oddities, and trouble with canvas rendering which can be addressed quickly and immediately without support assistance. If a simple CTRL+F5 (CMD+SHIFT+F5 for Mac) does not resolve the issue:
Steps to fix caching issues:

Open the DevTools using the F12 key (or cmd-shift-i for macOS)
Click the settings ⚙ icon
Below the heading 'network' check the checkbox for 'Disable Cache (when DevTools is open)
Leave the DevTools window open and refresh (f5) the Foundry VTT window

Note: Firefox users will see this checkbox under 'Advanced Settings' -> 'Disable HTTP Cache (when toolbox is open)'

Yes, it does fixes it. Thank you!


[Bug]: Type initializer for 'ReactiveUI.RxApp' throws TypeLoadException (DispatcherScheduler)

Describe the bug 🐞
Hi,
I am maintaining a WPF project (.NET 4.7.2), which is currently on ReactiveUI and ReactiveUI.Wpf version 9.22.1 and using System.Reactive version 4.3.2. I didn't change anything compared to the last successful build/release (which is ~8 months ago). I now need to add some minor bugfixes, but I am no longer able to compile the solution. I get an exception:
System.TypeInitializationException: 'The type initializer for 'ReactiveUI.RxApp' threw an exception.'
Inner Exception:
TypeLoadException: Could not load type 'System.Reactive.Concurrency.DispatcherScheduler' from assembly 'System.Reactive, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=94bc3704cddfc263'.
I tried a lot of things, also copied an implementation of DispatcherScheduler to my project, but the exception remains. I have to say, that I am using paket for package management and I see that it always adds references to netcore versions of System.Reactive. But even if I manually replace those with net46 assembly, the exception remains. I don't have an idea how to proceed with that at the moment. I thought, maybe I drop paket and switch to nuget (which was improved over the years, so not sure, if all arguments to use paket will still hold) in first instance. But this is quite a big monolith having around 70 projects, so this is some work, which I would only do, if there is a chance that it works afterwards. Sorry for not being able to provide more details, please request any missing information, I am happy to provide that.
Step to reproduce
Currently only happening in my solution, so no repro.
Reproduction repository
No response
Expected behavior
Not even sure, if ReactiveUI isn't behaving as expected and it's just my fault, it doesn't work.
Screenshots 🖼
No response
IDE
Visual Studio 2022, Visual Studio 2019
Operating system
Windows
Version
11
Device
PC
ReactiveUI Version
9.22.1
Additional information ℹ
No response

check your app.config to see if you have an assembly binding redirect for System.Reactive that is pointing to 4.3.0 as you have mentioned you're on 4.3.2 and that version of ReactiveUI requires 4.1.6. if that's the issue then the redirect should read something like
            <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>.0" newVersion="4.3.2"/>

https://learn.microsoft.com/en-us/dotnet/framework/configure-apps/file-schema/runtime/bindingredirect-element

Good catch, the app.config had a binding redirect to 4.3.0, but even after changing that, the error persists. DispatcherScheduler class is not available - I verified by trying to set RxApp.MainThreadScheduler = DispatcherScheduler.Current.
Any other ideas? I still suspect paket to add wrong dependencies, but I cannot build even with setting the dependency manually. As I was able to build the same thing 8 months ago, has there been any change in VS that would cause such issue? Sorry, if I cannot provide better error details at the moment, I am still confused why it won't just work.

If it's the DispatcherScheduler feels almost like paket is not including the windows system.reactive package.
I converted a company recently from paket to nuget with 1000 projects. The way I did it is read the paket.lock/paket.references file using the paket.core nuget package. Then using Microsoft.build nuget package I used msbuild to add PackageReference. You can then just delete the paket files.

@glennawatson Thanks for the hint to these two nuget packages. I was able to hack sth together that was converting paket.references to PackageReferences., I also converted the .NET Framework 4.7.2 projects to .NET 7 and I am now also able to use the latest bits of ReactiveUI again. It was quite some work to convert all ReactiveList occurrences to DynamicData, as I found some things behaving differently, but all in all I am quite happy that everything is working again. Thanks a lot for your help!


⚠ Bofrost.be has degraded performance

In b11f28d, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1454 ms

Resolved: Bofrost.be performance has improved in b9d8049.


Improve column width in Users menu

In the Users menu, if the organization has long names and/or email addresses, the Role column doesn't get enough space:

We should fix it so that the role can always be displayed on one line, even if that causes names and/or emails to get wrapped more.
If it will help get us a bit more space, we can also rename "Last active" -> "Active", as the meaning of that column should be clear enough from context.

@sahil839 would you be up for picking up this one?
I think Deactivated users is OK because we don't show a "Last active" column, but it would be worth double-checking.

This issue has been resolved for English, but still exits for Russian:

Reopening it as lower priority.

I will improve the pull request already made to make it ok for Russian and also make the email column a bit wider, @zulipbot claim

@zulipbot claim

Completed in #30832.


[2.9] Create cluster form is distorted/confusing when the broser window size is reduced

Setup

Rancher version: v2.9-d2fa8a7d6116c48a814ee7983af522d614461e49-head
Rancher UI Extensions: master df60f44
Browser type & version: Firefox on openSUSE Leap 15.5 115.8.0esr (64-bit), Google Chrome Version 124.0.6367.201 (Official Build) (64-bit)

Describe the bug

The Create cluster form is distorted when the the browser size is reduced to half.
To Reproduce

Go to Create AKS cluster form, fill details until the entire page loads.
Resize the browser size to half.

Result
The form values are distorted.
Expected Result

The form must be clearly visible.
Screenshots

Sample expected result from Rancher 2.8:

Additional context

We need a general pattern for reactivity. I thought we had move issues like this, but after quick search could only find #932


Add ioredis v5 support

Fixes https://github.com/microfleet/ioredis-lock/issues/20
Main changes:

Updated ioredis dependency to v5
Fixed types to work with ioredis v5
Replaced bluebird with setTimeout
Updated Readme to mention ioredis v5
Updated comments that were wrong

P.S. I'm not a pnpm user, so after installing latest pnpm and updating ioredis to v5, pnpm simply decided to ignore pnpm-lock file (due to old lock file version) and updated all the deps together with ioredis... :-/  I hope that's fine.

@AVVS Since this would require a major version bump, I can also open a PR to replace uuid with Node's randomUUID to make this package dependency-free.


Solr Only - Integration impact assessment

What do we lose if we stick with the current setup.
Considering:
Currency conversion
Codelist integration
Aggregated transaction data