Problems installing brackets for Win

Hi there
When i try to install brackets, this happens. If i click cancel, it gives me error 2755. Works fine on my laptop. Cant find a solution. Please help.

Please specify which version of Windows you are using. What's the location of the installer and installation folders (local drive? network drive? removable drive?)

Can you check whether you have any of the issues listed here? https://support.microsoft.com/kb/834549?wa=wsignin1.0
Also, this page says having Trend Micro Anti-Virus can cause this problem.  If you have that app installed, try temporarily disabling it.
If none of those help, please follow these steps on the command line to generate an installer log file:

cd to the folder containing the installer
Run msiexec /i "Brackets.Release.1.2.msi" /l*v .\brackets-install.log
Upload the generated log file to something like Gist, pastebin, etc. and post the link to it here

I`m using win 8.1, location of the installer is on my D: drive. I left the installation folder on default, so c:/program files, and no i dont have trend anti-virus. Will try to generate an installer log...

Just solved the problem. I just had to move the setup file from d to my c partition. My C partition is actually an SSD, while my D is a regular drive. I dont know why trying to install from D caused problems but it did. I`m just glad its working now.

Great, glad to hear you got it to work!


fix: [M3-6141] - Use our custom dialog instead of MUI's

Description 📝
Close button for dialog was collapsing to new line on mobile / and small breakpoints.
Changes:

Use our custom dialog component to fix issue

You'll notice the overlay is nicer and button is styled to our typical design

Preview 📷

Before
After

How to test 🧪

Go to http://localhost:3000/linodes
Scroll to bottom and click "Monthly Network Transfer Pool" link

Looks good, but while we're here I think there is some clean up that could be done

Some classes likepoolUsageProgress could probably be deleted
Remove React.FCs

The new dialog is also much wider than the previous. Maybe we should add the  maxWidth="sm" prop to make it more similar to what we had before?

Thanks for review, was able to remove a lot of unused classes! Fixed the width of the dialog as well.


[feat] Create new branch could default to checking it out as well

Is your feature request related to a problem? Please describe.
When I create a new branch, I then need to use bl to find it in the list and check it out.
Describe the solution you'd like
When I create a new branch, the branch gets checked out automatically.
Describe alternatives you've considered
An interactive list like lazygit's branch TUI. Note that an issue exists for that as well https://github.com/TimUntersberger/neogit/issues/289

Do you create a new branch using bc or any other command.
bc should create and checkout a branch
bn just creates a branch, which follows the git behaviour git branch vs git checkout -b
I personally only use bc and work is underway to auto deduce the name from a remote branch to make it easier to checkout branches which aren't local

If I use bc I get a branch selection UI. I don't see a possibility of entering a new branch name there. Maybe I'm missing something?

It prompts for a branch to base it on, so that you can create a local branch for an upstream branch.
bn creates a new branch based on the current branch.
It is somewhat inconsistent, do you think that bn should prompt for the base branch as well, or bc should use the current branch (which makes checking out remote branches difficult).
An alternative is to make the bb screen auto setup a local branch if you attempt to checkout a remote branch, rather than the current behavior of landing in detached head state

... I know I pretty much keep saying this, but I overhauled the branch popup and the behaviour is much more straightforward (and pretty much identical to Magit's ux)
Coming soon to a repo near you..!

To the above point, I flipped the order around for bc so first you give it a name, then pick the base. That always annoyed me too.

:+1:
@CKolkey I've been trying to use your fork but it's very buggy on my setup. Happy to help debug. I was going to post something but Issues are disabled there.
Either way, looking forward to some of that getting upstreamed :tada:

@fnune oh, really? Yeah, I'd love to know whats going wrong. Lemme see if I can turn issues on over there


@graphql-codegen/client-preset-swc-plugin adds imports before "use client"

Which packages are impacted by your issue?
@graphql-codegen/cli, @graphql-codegen/client-preset
Describe the bug
@graphql-codegen/client-preset-swc-plugin inserts imports at the top of modules where you define queries. This breaks modules that are defined as React client components using the "use client" directive, which must appear at the top of the file - because the inserted imports end up before the directive.
This means you cannot define GraphQL queries in the same module as a client component when using the Next.js app directory.
Please see the example repository for a reproduction.
Your Example Website or App
https://github.com/victorandree/graphql-code-generator-swc-app-dir
Steps to Reproduce the Bug or Issue

Create a client component in a Next.js app using the "app routes", requiring the use of a "use client" directive at the top of the module.
Define a GraphQL query in this module.
Set up GraphQL code generator with the "client preset" and using the SWC plugin.
Try to build the Next.js app

The build will fail with a message like the following:
$ npm run dev

[...]
- error ./app/QueryComponent.tsx
ReactServerComponentsError:

The "use client" directive must be placed before other expressions. Move it to the top of the file to resolve this issue.

   ,-[/graphql-code-generator-swc-app-dir/app/QueryComponent.tsx:1:1]
 1 | 'use client';
   : ^^^^^^^^^^^^^
 2 |
 3 | import { useQuery } from '@apollo/client';
 4 | import { graphql } from '../gql';
   `----

Import path:
  ./app/QueryComponent.tsx
  ./app/page.tsx

Expected behavior
Since "use client" is required to be at the top of the file by the React Server Components RFC, I expect the SWC plugin to insert its imports after this directive.
Expressed differently: I expect to be able to define GraphQL queries in client components.
Screenshots or Videos
No response
Platform

OS: macOS
NodeJS: 16.19.1
graphql version: 16.6.0
@graphql-codegen/cli: 3.3.1
@graphql-codegen/client-preset: 3.0.1
@graphql-codegen/client-preset-swc-plugin: 0.2.0

Codegen Config File
import { CodegenConfig } from '@graphql-codegen/cli';

const config: CodegenConfig = {
  schema: 'schema.graphql',
  documents: 'app/**/*.tsx',
  generates: {
    'gql/': {
      preset: 'client',
      plugins: [],
    },
  },
};

export default config;

Additional context
A workaround is to define queries in a separate file, and import them in the client component. This is a bit annoying.
I'm not super-experienced in reading Rust, but I'm pretty sure that the imports are inserted by the plugin by module.body.insert(0, [...]) in lib.rs:183. I'm not sure if there's a simple way to insert it "a bit later" if there's a "use client" directive there.
Note: The SWC plugin crashes with the latest version of Next (13.4.3).

I tried it out with using 1 as index and it works - There might be edge cases though

@BowlingX By "1 as index", do you mean passing a 1 as the first argument to module.body.insert, instead of 0?
Yeah, that should probably insert the import statement after the first ModuleItem. However, the first ModuleItem may not be a "use client" directive, so it's probably better to only apply this logic conditionally.
The code should probably check module.body, to find the index where the import declaration should be inserted. It should be possible to identify "use client" directives, or other module items, that should be placed before an import, and then insert imports after that index.


HostConfig Mounts: Settings not Updating

As pointed out in https://stackoverflow.com/a/56414070/4577212, if a volume is host mounted (HostConfig) and associated with a container, the settings become set / unchanging.
The current solution to this is to delete the container, delete the volume, then allow for everything to rebuild. Does IoT edge do anything to streamline this?
For example:
"createOptions": {
 "HostConfig": {
   "Mounts": [
     {
       "Target": "/alt_data",
       "Source": "samba-data-connection",
       "Type": "volume",
       "ReadOnly": false,
       "VolumeOptions": {
             "DriverConfig": {
             "Name": "local",
             "Options": {
               "type": "cifs",
               "device": "device connection info",
               "o": "usernames and passwords"
           }
         }
       }
     }
   ]
 }
}

I might want to change the volume username, password, and ip address to:
  "Options": {
     "type": "cifs",
     "device": "NEW device connection info",
     "o": "NEW usernames and passwords"
 }

If I deploy this to the device, nothing will happen. I will see the updated deployment manifest in the Azure Portal as reported by the device, but those actual volume settings are not changed. This is going to be a major problem if we have 100 devices and need to manually do this. How does IoT edge handle this?

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

ID: ad3731a8-f0df-f1a1-d1a1-e89a769a0908
Version Independent ID: 50a5190e-eafa-7dde-c18f-be9505b00a0a
Content: Declare modules and routes with deployment manifests - Azure IoT Edge
Content Source: articles/iot-edge/module-composition.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

Hi @josiahlaivins Thanks for trying IoT Edge. Would you please post your question on the IoT Edge repository? https://github.com/Azure/iotedge/issues

I went ahead and posted at iotedge
However, I was posting here because I was assuming that there might be a feature that already exists to solve this.

@josiahlaivins That makes sense and thank you for taking the time to provide your valuable feedback. A way to push for this functionality would be to create an entry in the Azure IoT Edge User Voice if it gets enough upvotes it will help the engineering team prioritize. https://feedback.azure.com/forums/907045-azure-iot-edge


preventOverflow should allow root boundaries to be set to window

Feature description
One use case for the preventOverflow modifier would be to allow the popper to be constrained to the boundaries of the window. When the user is zoomed in at high magnifications (200/400%) or has reduced their viewport size, the window may extend past the edge of the viewport; the requested behaviour would allow the user to move the viewport around the window without the popper 'following' them.
Currently in popper v2, rootBoundary can only be set to viewport or document. rootBoundary: 'document' isn't helpful for this case, as at high zoom levels the document boundaries are a viewport-sized bounding box in the top left of the page, so a reference elements sitting outside of the document boundary are positioned as if the reference is off-screen. With rootBoundary set to viewport the popper will stick to the edges of the viewport as you scroll around; a big enough popup may obscure large parts of the screen until the reference is completely off-screen.
An example for experimenting with this behaviour in popper v2 is in this codesandbox: https://codesandbox.io/s/small-hill-lpc3i?file=/src/index.js
Zoom in so that the reference element is partially out of the viewport on load, and move around. There doesn't appear to be a configuration of preventOverflow that allows the popper to sit directly below the reference and not shift to sit the boundaries of the viewport or document, without disabling preventOverflow entirely and losing overflow prevention on the edges of the window.
This behaviour was supported in popper v1 via the boundariesElement option:
https://codesandbox.io/s/holy-wood-jmw9u?file=/src/index.js
Why should this feature be part of the Popper's core?
Allowing the popper to behave independently of the viewport at high zoom settings or viewport positions is useful as it can allow the user to scroll around a page without the popper 'following them', while also keeping the popper from overflowing the window and being partially obscured.
This makes sense as core behaviour as it's restoring functionality to a core modifier, preventOverflow, and that modifier has a rootBoundary option where this option could logically be added (similar to boundariesObject in popper v1)

This is fixed in v2.4.2, if you update the version in the CSB it appears to work

Oh that's great news, thanks for the quick response 😄
Follow-up question (I can raise a ticket around it if it's not supported); is there a way to allow hide to only occur when the popper leaves the window?

I don't think so; I think it should allow all the detectOverflow options to be passed though. You can create a custom modifier to add the functionality yourself tho

Thanks for the help, I'll mark this as resolved then


Panic when an animated webp image is loaded

Describe the bug
Whenever an animated webp image is loaded it will lead to a panic.
This happens in BoilR
https://github.com/PhilipK/BoilR/issues/203
To Reproduce
Steps to reproduce the behavior:

Download an animated webp image, example: https://cdn2.steamgriddb.com/file/sgdb-cdn/logo/42cd4e870371f116e5695912004e1b5a.webp
Load the image using ColorImage::from_rgba_unmultiplied using the example in the documentation:

fn load_image_from_path(path: &std::path::Path) -> Result<egui::ColorImage, image::ImageError> {
    let image = image::io::Reader::open(path)?.decode()?;
    let size = [image.width() as _, image.height() as _];
    let image_buffer = image.to_rgba8();
    let pixels = image_buffer.as_flat_samples();
    Ok(egui::ColorImage::from_rgba_unmultiplied(
        size,
        pixels.as_slice(),
    ))
}

fn load_image_from_memory(image_data: &[u8]) -> Result<ColorImage, image::ImageError> {
    let image = image::load_from_memory(image_data)?;
    let size = [image.width() as _, image.height() as _];
    let image_buffer = image.to_rgba8();
    let pixels = image_buffer.as_flat_samples();
    Ok(ColorImage::from_rgba_unmultiplied(
        size,
        pixels.as_slice(),
    ))
}

This will not load or give a Result::Err it will instead panic with the following message:
thread 'ui::defines::tests::test_image_load_animated_webp' panicked at 'source slice length (19292) does not match destination slice length (50176)', C:\Users\Philip\.cargo\registry\src\github.com-1ecc6299db9ec823\image-0.24.3\./src\codecs\webp\extended.rs:459:21

Expected behavior
The image is loaded as a static image
Desktop (please complete the following information):

OS: Windows & Linux
Version: Windows 11, Arch Linux, Steam OS

Additional context

This has been tested with multiple animated webp images, the one example is just a very simple/small example that causes a panic.
Tested with both master (as of this post) and version 0.19

This might actually be a bug in the image crate, instead of epaint, I will close this and create an issue there instead.


Error message during startup "general unexpected critical error"

Error message I get when starting bitcoinz-wallet.exe

bitcoinz.conf

Please download the latest release zip file again.  We've updated it.

I also don't understand what is this?
Vào 00:21, T.4, 2 Th6, 2021 Shai0Hulud @.***> đã viết:

Error message I get when starting bitcoinz-wallet.exe
[image: image]
https://user-images.githubusercontent.com/82765139/120365018-3ae55a80-c30e-11eb-86ef-240839f567cd.png
bitcoinz.conf
[image: image]
https://user-images.githubusercontent.com/82765139/120365234-7718bb00-c30e-11eb-9070-e2a3147007a9.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/btcz/bitcoinz-wallet/issues/61, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOUFYMWRW6TK6FW4354OZYDTQUJKTANCNFSM455C7Q2A
.

Please redownload the latest release again as already stated here: https://github.com/btcz/bitcoinz-wallet/releases/tag/v2.0.7
I'm locking this conversation as this issue has been addressed.


Not able to Filter the tag data

Not able to read the tag with reserved memory -
There is function in Android APP
///
/// Setup scanning filter.
///
/// Filter start address
/// Filter data length
/// Filter data
/// Permanent
/// True is success, false is failed
public bool SetFilter(int ptr, int cnt, string data, bool save)
{
https://github.com/stuarthallows/chainway-c72-rfid-scan/blob/7a1ed20a2eb5dfce860ab4bab0401e5e447a7378/C72Scan/C72Scan/C72Scan.Android/Services/RfidService.cs#L63
Not able to understand how to pass the following parameter
PTR
Length
STart add
data
Can you please help with this

My plugin doesn’t work for Android 11 so it is better to add the suggestion for the following plugin
https://pub.dev/packages/rfid_c72_plugin


⚠ Bofrost.de has degraded performance

In 48c547c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1688 ms

Resolved: Bofrost.de performance has improved in aa365ac after 20 minutes.


New loading modes, in progress

@skavish I plan to finish this one at some point, so I'll leave it here.


Replace deprecated Renderer with Renderer2.

From @angular/core's api.d.ts:
/**
 * @deprecated Use the `Renderer2` instead.
 */
export declare abstract class Renderer { ... }

Hello, Michael.
Thank you for your pull request. Yes, the Renderer class is deprecated in v4, but it's still actual for v2. I have updated the repo to version 4+.


Only use console.tron.log in dev mode

Fixes #302

:tada: This PR is included in version 4.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Bug] easymde loading content only when clicking into it

Bug report
What I did
I have an easymde field added to the CRUD controller as a subfield of a repeatable:
[
    'name' => 'content',
    'label' => __('Content'),
    'type' => 'easymde',
],

I am using Tabler-Theme. The field is on and additional "pages" tab.
What I expected to happen
When editing an existing entry, the content should be shown in the editor.
What happened
When editing an existing entry, the content seams to be empty, until clicked into the editor. (In dark and lite mode).
It seems it works, when opening the tab directly .../1/edit#pages

What I've already tried to fix it
Nothing.
Is it a bug in the latest version of Backpack?
After I run composer update backpack/crud the bug... is it still there?
Yes.
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
### PHP VERSION:
PHP 8.2.10 (cli) (built: Aug 30 2023 09:46:43) (NTS Visual C++ 2019 x64)
Copyright (c) The PHP Group
Zend Engine v4.2.10, Copyright (c) Zend Technologies

### LARAVEL VERSION:
<IP_ADDRESS>

### BACKPACK PACKAGE VERSIONS:
backpack/basset: 1.2.1
backpack/crud: 6.2.4
backpack/generators: v4.0.2
backpack/pro: 2.0.16
backpack/theme-tabler: 1.1.1

Hey @BernhardK91,
Thanks for putting this bug in notice. I have submitted a fix here.
Once it's tested! We will have it in the coming updates.

Hey @BernhardK91 I've merged the fix in backpack pro 2.0.20 just now. In a few moments our private repository will pick it and you should be able to do a composer update backpack/pro to get the latest version.
Let me know if you are still experiencing the issue after the update and we can re-evaluate the issue.
Cheers


[Feature] [Bugfix] [Refactor] Fix shadows and refactor measurement protocols

Closes #39, #48 and #47.
We introduce the concept of MeasurementManager to keep in memory the measurement data that can be reused to reconstruct states or calculate over quantities of interests. We also separate each functionality into a different class method for the sake of clarity and we improved the speed of execution.

Thanks for the feedback @RolandMacDoland, made the requested few changes.

Hi @RolandMacDoland, eventually I change the Measurements so that they can be initialized and more bound to the input model, states, and parameter values as it makes more sense (note for tomography, the observables should be provided to be defined, but for a shadow not as the observable only plays a role in the expectation - yet it can be provided directly). We are also more transparent with the manager by allowing accessing several methods from Measurements directly.
Also we use now a dataclass MeasurementData with two fields.


Create component for user account page

Description:
Create in Angular component for user's account view to hold all components related with current user account.

@OtisRed


Add support for custom sounds

node-pushover currently has hardcoded the list of sounds available, but the Pushover API also supports custom sounds that users may upload to their accounts and then use for Pushover notifications.
To support custom sounds, Pushover suggests the following:

In most cases, applications choosing to override the default sound should offer the user the option to pick a sound from the list above. Rather than hard-coding the list of sounds (since more may be added in the future), an API call may be made to retrieve the list of current sounds and their descriptions by sending a GET request to:
https://api.pushover.net/1/sounds.json?token=(your app token)
Include your application's token as the token parameter. This API call returns a sounds hash with each key being the actual sound parameter to store for the user and send to our API, with its value describing the sound.
Note: In addition to the list of sounds from our API, your application must provide a blank option to the user that will not send the sound parameter to our API (or send it with a blank value). This will allow the user's default tone to play, rather than being overridden by your application.

As an example, I have added a custom sound named "doorbell" to my account (see: screenshot), and the API call above returns this JSON which includes the custom sound in the list along with the built-in Pushover sounds:
{"sounds":{"doorbell":"Doorbell chime x2","pushover":"Pushover (default)","bike":"Bike","bugle":"Bugle","cashregister":"Cash Register","classical":"Classical","cosmic":"Cosmic","falling":"Falling","gamelan":"Gamelan","incoming":"Incoming","intermission":"Intermission","magic":"Magic","mechanical":"Mechanical","pianobar":"Piano Bar","siren":"Siren","spacealarm":"Space Alarm","tugboat":"Tug Boat","alien":"Alien Alarm (long)","climb":"Climb (long)","persistent":"Persistent (long)","echo":"Pushover Echo (long)","updown":"Up Down (long)","vibrate":"Vibrate Only","none":"None (silent)"},"status":1,"request":"7bf9fdb2-66c2-404b-85a7-9d62cc5ad9a0"}
Can this be implemented so that it can be used in upstream dependent packages like homebridge-messenger?

sigh I just noticed the updateSounds method... My apologies - ignore this issue.


OpenVPN: makeOVPN.sh causes client ccd file error

Discussed in https://github.com/pivpn/pivpn/discussions/1613

Originally posted by liasom September  4, 2022
New installations (today and yesterday) of OpenVPN on two separate Raspberry Pi computers. One is Pi Zero 1.3 with 32-bit Raspberry Pi OS lite and the other is Pi Zero 2 W with Pi OS 64-bit. This error happens on both. When I create a client ovpn profile the corresponding ccd file contains an error in having no space between the ip address and subnet. This content pasted in from one of the ccd files.
ifconfig-push <IP_ADDRESS><IP_ADDRESS>

Faulty line: https://github.com/pivpn/pivpn/blob/master/scripts/openvpn/makeOVPN.sh#L478
Missing space after ${i}

Thanks :pray:


Make hint an optional field for data table actions

Still have no idea what a hint is. Couldn't really find anything about hints at https://www.lightningdesignsystem.com/components/buttons/#hint as specified across multiple locations in docs and code. Is this a deprecated thing?
Either way it's messing with the styling of the dropdown icon for me, and there's no way to override it.

It's a "hover hint" and is when the color of the down arrow on the button changes from gray to black on hover of the table row.

Please add prop to comments so that it's listed on the website for folks to see like so. Probably something like /* Changes the color of SVGs in child nodes due to the mouse hover on ancestor nodes. */ if that makes better sense.
If true is the default, we need to make it noHint so we can be consistent with not having defaults that are true

Lol what happened here :)

I don't know. 😃


Budget-App-Completion

This Budget app have following things :

Splash screen

A simple page with the name of your app (yes, you need to choose one), and links to the sign up and log in pages.

Sign up and log in pages

The user should be able to register in the app with full name, email and password (all mandatory).
The user can log into the app using email and password.
If the user is not logged in, they can't access pages that require the user to be logged in (all the pages described below).

Home page (categories page)

When the user logs in, they are presented with the categories page.
For each category, the user can see their name, icon and the total amount of all the transactions that belongs to that category.
When the user clicks (or taps) on a category item, the application navigates to the transactions page for that category.
There is a button "add a new category" at the bottom that brings the user to the page to create a new category.

Transactions page

For a given category, the list of transactions is presented, ordered by the most recent.
At the top of the page the user could see the total amount for the category (sum of all of the amounts of the transactions in that category).
There is a button "add a new transaction" at the bottom that brings the user to the page to create a new transaction.
When the user clicks on the "Back" button (<), the user navigates to the home page.

"Add a new category" page

The user fills out a form to create a new category, indicating their name and icon (both mandatory).
The user clicks (or taps) the "Save" button to create the new category, and is taken to the home page on success.
When the user clicks on the "Back" button (<), the user navigates to the home page.

"Add a new transaction" page

The user fills out a form to create a new transaction with:

name (mandatory)
amount (mandatory)
categories (mandatory at least one)

The user click (or taps) the "Save" button to create the new transaction, and is taken to the transactions page for that category.
When the user clicks on the "Back" button (<), the user navigates to the transactions page for that category.

Thank you ,
Bhushan .

Hello sir/mam,
I hope this app fulfills all the requirements but still if you find any errors it will be more helpful to me if you give me hint to solve it .
Thank you ,
Bhushan .


get_max_spacing in clustering.py

Is get_max_spacing getting the min space in the code?

It's getting the min space between the clusters. For more clarity have a look at this lecture


Observing Scroll

Hello there:
I have several sections with unique id's, How could I keep track on my controller which one is the currently selected one? The end result that I am trying to achieve is to have a swipe function included so that if I swipe down from the 3rd section it will scroll down to section 4 perfectly.
I tried:
    var container = angular.element(document.getElementById('container'));
    container.on('scroll', function() {
      console.log('Container scrolled to ', container.scrollLeft(), container.scrollTop());
    });

With no success, of course changing the 'container' id to a unique one. Nothing seems to work...
Of course your exemple does, so I know that it is installed and set up right.
Thank you for your help.
Cheers,
Jules

Found a solution:
      $rootScope.$on('duScrollspy:becameActive', function($event, $element){
        console.log( $element, $element[0].id);
      });


Tensorflow 1.15 or 2.0 support?

Does RKNN-toolkit support Tensorflow 1.15 or 2.0 ?

I'd be interested in the same. Or at least a way to load onnx models.

Any updates?

This project's good and I've been using RK3399Pro for 2 years
But maintainer has not been able to responding to upgrade TF for a long long time.
and customer service and other supports are unkind.
I'd rather use Nvidia.


Add a linkify component

Can you elaborate on what this would look like?

You know to be honest, I can't remember what I was thinking when I created this issue. So, I think for now I'll just close this as it works well currently as a helper.


Make IRC->Discord notifying without '@' prefix behavior configurable

This was the behavior introduced in #470, but doesn't mesh well in communities where people are using both IRC and Discord; users talking to each other in IRC will receive a bunch of Discord notifications when one highlights the other. There should be a setting to have IRC->Discord highlights always require the @ prefix.
<Discord> <Link> this new bot allows irc to discord @highlights
<Austin> i mentioned that
<Austin> and i dont like that it doesnt require @
<Discord> <Link> Try it on me
<Austin> Link, no
[appears in Discord as "@Link no"]
<Discord> <Link> Was that without @?
<Austin> yes
<Discord> <Link> Hmm

Hey! Just to add an update to this: I'm now only maintaining discord-irc for bug fixes and vulnerabilities, as I'm no longer an active user of the project. Accordingly, this enhancement and others are unlikely to be solved in the project anytime soon. I hope you can find a solution regardless.

is it possible to get a fix for this ASAP many users of my relay would like to have this toggled off


Point stat choice

Resolves #1817
Motivation
Allow a user to select which point statistic they want displayed in the Marginal 1D tool: mean or median.
Changes proposed

Added method to calculate the median of an array in JavaScript.
Updated the Marginal 1D tool to use the new method.
Updated the Marginal 1D interfaces to include a new widget component.
Updated the Python side of the tool to render the new widget.
Updated Python TypedDict objects to include docstrings for inclusion in the documentation in the future.
Updated the coin flipping tutorial to use the new point statistic button.

Test Plan

Updated the coin flipping tutorial, and visually inspected the tool worked.
Built the documentation locally for testing the new docstrings in Sphinx.
Built the Docusaurus documentation locally to ensure the new widgets worked.

new-widget-check.webm
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] 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 change)

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

add the Continuous Ranked Probability Score stat as well


Jiggle dependencies to fit available python deb packaging

Note that some dependencies have been rolled back and we'll need to test for
regressions and incompatibilities.
Bug: T107972

I think we should just pin them to specific versions than specify a range. Upgrading on the hosts will require us to build a new package and roll it out, so we'll always know the exact version...

I don't want to tie 'revscoring' to our installation context so strongly.  Others should be able to use this module in their own context.

I don't see how specifying exact versions prevents other people from using
it?
On Tue, Sep 1, 2015 at 3:43 PM, Aaron Halfaker<EMAIL_ADDRESS>wrote:

Merged #175 https://github.com/wiki-ai/revscoring/pull/175.
—
Reply to this email directly or view it on GitHub
https://github.com/wiki-ai/revscoring/pull/175#event-397816254.

--
Yuvi Panda T
http://yuvi.in/blog


Subscriber

Codecov Report

Merging #103 into master will decrease coverage by 4.41%.
The diff coverage is 27.62%.

@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
- Coverage   75.25%   70.84%   -4.42%
==========================================
  Files          86      101      +15
  Lines       13481    14795    +1314
==========================================
+ Hits        10145    10481     +336
- Misses       2553     3504     +951
- Partials      783      810      +27

Impacted Files
Coverage Δ

utils/env.go
0% <ø> (ø)
:arrow_up:

subscriber/common/job/streaming_processor.go
0% <0%> (ø)

subscriber/common/job/retry_failure_handler.go
0% <0%> (ø)

subscriber/common/job/processor.go
0% <0%> (ø)

subscriber/common/message/json_decoder.go
100% <100%> (ø)

subscriber/common/rules/transformation.go
100% <100%> (ø)

subscriber/common/job/driver.go
13.82% <13.82%> (ø)

subscriber/common/job/controller.go
19.02% <19.02%> (ø)

subscriber/common/message/message_parser.go
22.95% <22.95%> (ø)

subscriber/config/service_config.go
36.36% <36.36%> (ø)

... and 23 more

Continue to review full report at Codecov.

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

coverage drops by  -4.42% ... ideally we should keep same level if not higher


fix agency poll api redirects

Scope & Purpose
Fix the agents' poll API to redirect to leader

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[ ] :book: CHANGELOG entry made

Backports:

[ ] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket number:
[ ] Design document:

Testing & Verification
(Please pick either of the following options)

[ ] This change is a trivial rework / code cleanup without any test coverage.
[ ] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[ ] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[ ] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

Link to Jenkins PR run:
https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/12967/
Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the HTTP API
[ ] Added Swagger examples for the HTTP API
[ ] Updated license information in LICENSES-OTHER-COMPONENTS.md for 3rd party libraries

External contributors / CLA Note
Please note that for legal reasons we require you to sign the Contributor Agreement
before we can accept your pull requests.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13007/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13021/

http://jenkins.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13025/

Tests blue.


Unable to load transpiler to transpile

When importing ng2-handsontable into the module file I get this error "Unable to load transpiler to transpile http://localhost:52607/node_modules/ng2-handsontable/index.js".
"(SystemJS) XHR error (404 Not Found) loading http://localhost:52607/traceur/traceur.js
	Error: XHR error (404 Not Found) loading http://localhost:52607/traceur/traceur.js
	    at XMLHttpRequest.wrapFn [as __zone_symbol___onreadystatechange] (http://localhost:52607/node_modules/zone.js/dist/zone.js:1242:29) [<root>]
	    at Zone.runTask (http://localhost:52607/node_modules/zone.js/dist/zone.js:166:47) [<root> => <root>]
	    at XMLHttpRequest.ZoneTask.invoke (http://localhost:52607/node_modules/zone.js/dist/zone.js:420:38) [<root>]
	Error loading http://localhost:52607/traceur/traceur.js
	Unable to load transpiler to transpile http://localhost:52607/node_modules/ng2-handsontable/index.js
	Error loading http://localhost:52607/node_modules/ng2-handsontable/index.js as "ng2-handsontable" from http://localhost:52607/app/app.module.js"

I used the quickstart files for Angular 2 to make sure it wasn't our own app that was conflicting with anything. Then npm install ng2-handsontable.
All I'm doing up to this point is adding the import to the module file.
import { NgModule }      from '@angular/core';
import { BrowserModule } from '@angular/platform-browser';
import { AppComponent }  from './app.component';
import {HotTableModule} from 'ng2-handsontable';

@NgModule({
  imports:      [ BrowserModule, HotTableModule ],
  declarations: [ AppComponent ],
  bootstrap:    [ AppComponent ]
})
export class AppModule { }

I did try configuring traceur in another test to resolve the transpile error but it would lead to more errors in handsontable about $defineProperty is not a function.
Error: (SystemJS) $defineProperty is not a function
	TypeError: $defineProperty is not a function
	    at Object.createClass (http://localhost:52607/node_modules/handsontable/dist/handsontable.full.js:22329:5)

Which quick start files did you use? Does it use Webpack or SystemJS for bundling?
ng2-handsontable should work with Angular-CLI or the official Angular2 seed and TypeScript out-of-the-box. I haven't tested it with Traceur yet.

Thanks for the reply. I'm using this seed: https://github.com/angular/quickstart linked from the documentation. It's using SystemJS.
I added 'ng2-handsontable': 'npm:ng2-handsontable/index.js' to the system config files as well as the import to the module file.
I'll test out the one you linked. If commonjs works instead of systemjs, I don't think it'll matter for our project to switch.

@renehamburger I used the Angular 2 seed you linked and was able to use ng2-handsontable. The styling didn't work but I can deal with that. Not sure if the issue is with using SystemJS or TypeScript compiling. Our projects are within Visual Studio so if webpack works then I'll look into converting our projects.

You will need to include Handsontable's css manually for now from node_modules/handsontable/dist/handsontable.css, e.g. as one of your component's styleUrls. I'll consider including that in ng2-handsontable straight away.

I was able to include ng2-handsontable with the current version of angular/quickstart (which does not include traceur). The one additional thing that is necessary with SystemJS which is to include 'ng2-handsontable': 'node_modules/ng2-handsontable/bundles/ng2-handsontable.umd.js' in the systemjs.config.js.

@renehamburger That works wonders. Thanks for the help!


I'm a noob when it comes to forked libraries. Sorry, in advance if this closed pr causes any confusion. It was all a mistake. Well, all except the part where I deleted it.

Nice amended PR title. :wink: And no worries; happens to most of us every once in a while.

I enjoyed reading the amended README.md as well. Thanks for the chuckle! 😃


Text Size instead of Text Scale

What happens is that our builds grow along the day, like everyone else.
Then, if I set a proper font size for our TV, at the start of the day, showing lets say 4 builds, at the end of the day, after new 20 builds, nobody can read things anymore, since the font size decreases as much as new builds came in.
How can I set a font size and stick to it, no matter the number of builds the dashboard shows up?
Tks
Marco

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/37387116-text-size-instead-of-text-scale?utm_campaign=plugin&utm_content=tracker%2F580790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F580790&utm_medium=issues&utm_source=github).

Hey Marco! From what I've seen so far, in most cases the number of builds that the Build Monitor displays remains relatively constant throughout the day (usually when people have 20 projects that they want to display, they'll have 20 projects displayed on screen all the time).
Can you tell me a bit more about your scenario? What makes the new projects appear on screen throughout the day? A screenshot of how your display looks in the morning (with few projects) and in the evening (with 20 projects) would also be helpful!
Best,
Jan

Sure! This is a typical morning dashboard.

For each repo, we use 3 jobs, one to build from master, one to build for develop, and one to build a Pull Request prior to accept a merge.
And this is my config (perhaps this will explain some aspects):

Explaining, I only want to show lastest builds (someone from other geography may commit something at mi night, so I wanna see it on my morning), and I want to have all problematic builds on the top, permanently. I have 156 jobs, so I dont care OK builds from two days ago or older, they are history. They are solved.
Thats why, along the day, the number of builds do increase, up to 4 times.
Below I did a simulation, increasing 16h to 160h to look back.
This is how I would see it at the end of the day:

Best!

This is a FRAME on a HTML. What I just noticed is that at a certain width, a scroll bar do show up, and the font stops getting smaller. Using this width, the plugin tries to render everything on the same HTML area. Perhaps this can solve everything...

Hey @lovato and thanks for the thorough explanation and the examples! Could you please elaborate a bit more on your last message?

This is a FRAME on a HTML.

Do you mean that you display the Build Monitor within a frame, or am I misunderstanding?

What I just noticed is that at a certain width, a scroll bar do show up, and the font stops getting smaller. Using this width, the plugin tries to render everything on the same HTML area.

Yes, Build Monitor will show the scroll bars if it can't fit all the items within the screen, and will only decrease the font size until a threshold.

Perhaps this can solve everything...

Not sure if I follow? What are you thinking?

Yes, it do show up inside a frame, since the other frame I list all PullRequests opened.
I said about solving because if I enlarge a little more the width for Build Monitor, it will probably looks better.
Will try it and return back to you.

Cool, thanks. I designed Build Monitor to be displayed on large displays and radiate the information across the office floor. Since this is its primary purpose, the support for embedding within other dashboards is not really there ...


Settings: view state lost when switching tabs

Fixes https://github.com/Microsoft/vscode/issues/16434
@sandy081 I noticed a couple of issues in side by side editor:

position was not forwarded to editors in setEditorVisible and changePosition
setVisible was called only after setInput was called
clearInput was not forwarded to the embedded editors

All of these issues basically meant that the file editor for settings.json was unable to store and restore its view state.

@bpasero Thanks for fixing this.
LGTM


Public key does not exist in jwk_uris for the AD B2C user flow

I've created a user signup for an AD B2C tenant and followed the examples above to create an authorization code which I then used to generate an access token. However, when I try to get the public key to verify the access token I can't find it.
Although I can match the key_id of the JWT from the 'jwks_uri' of the B2C tenant's user flow metadata the response I receive does not contain the public key information:
{'kid': 'X5eXk4xyojNFum1kl2Ytv8dlNP4-c57dO6QGTVBwaNk', 'nbf':<PHONE_NUMBER>, 'use': 'sig', 'kty': 'RSA', 'e': 'AQAB', 'n': 'tVKUtcx_n9rt5afY_2WFNvU6PlFMggCatsZ3l4RjKxH0jgdLq6CScb0P3ZGXYbPzXvmmLiWZizpb-h0qup5jznOvOr-Dhw9908584BSgC83YacjWNqEK3urxhyE2jWjwRm2N95WGgb5mzE5XmZIvkvyXnn7X8dvgFPF5QwIngGsDG8LyHuJWlaDhr_EPLMW4wHvH0zZCuRMARIJmmqiMy3VD4ftq4nS5s8vJL0pVSrkuNojtokp84AtkADCDU_BUhrc2sIgfnvZ03koCQRoZmWiHu86SuJZYkDFstVTVSR0hiXudFlfQ2rOhPlpObmku68lXw-7V-P7jwrQRFfQVXw'}

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

ID: c54adceb-9510-aeb5-0c61-c0d052459cda
Version Independent ID: 0ff6b6b2-2802-9534-8963-796b9c4a2a76
Content: Web sign-in with OpenID Connect - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-reference-oidc.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

@mliudev Thanks for your feedback! We will investigate and update as appropriate.

Hey @mliudev, what do you mean by, the "Response you receive does not contain the public key information"? If the KID's match then it should be correct. You can follow this guide on how to easily check if the JWT Token has been properly signed : https://blogs.aaddevsup.xyz/2019/03/using-jwt-io-to-verify-the-signature-of-a-jwt-token/

Hey @mliudev I'm following up on this, can you please respond in regards to this issue? If there's no response by end of day, I'll be closing this issue out, if you have anymore questions please reopen the git issue

Hi Frank,
The kids match however as I've shown in the response there is no 'x5c' key in the response dictionary. I can't verify the signature because there is no public key in the response.

@mliudev I see, please see the stack post here : https://stackoverflow.com/questions/44664510/difference-between-azure-ad-and-azure-ad-b2c-tokens
Per the post : "it is same to verify the tokens from Azure AD B2C and normal Azure AD. We can generate the public key using the exponent(e) and modulus(n). But the keys endpoint is different, we need to using the link like below to retrieve the keys for the Azure AD B2C:"
I do believe we should include this in the documentation however as it is not clear how to properly validate the id token. Thank you for your feedback, and we'll go ahead and make those changes.
Please let us know if you have anymore questions within the scope of the thread,

Hey @mliudev as this issue has been addressed I will be closing out this git issue now. If there's anymore issues please reopen the issue and we can continue discussing it.
Thanks again for letting us know about this issue!

Thanks, I didn't know that you could generate the public key from the exponent and modulus. I agree this would be useful information in this documentation.

@FrankHu-MSFT I'd like to say that putting the burden of having to generate the public key on the developer is quite frustrating. Most libraries expect you to already have a public key. I don't see any reason why a developer needs to understand how to generate a public key at this low of a level, especially since the format of the exponent and modulus isn't provided in this documentation.
In my case I had to convert both the exponent and modulus to integers which required several type conversions in Python. For example the modulus had to be encoded as bytes in hex format and then converted to an integer from that hex format. The exponent had to be base64 decoded into bytes and then converted to an integer. No where in the documentation does it say what format the exponent or modulus are in nor are there links pointing to the RSA spec which explains this. Not that I would expect a developer to have to understand that spec in the first place.

@mliudev you're absolutely right, and we're looking into this issue and will get back to you. We appreciate the feedback, and hope to provide a more friendly developer experience. I'll let the product group team know about this.
I also suggest filing this in uservoice here : https://feedback.azure.com/forums/169401-azure-active-directory

@FrankHu-MSFT Hi Frank,
Has there been any progress on updating documentation regarding the generation of a public key? Or better yet, an OOB Azure B2C configuration option for auto-generating this public key?
If I understand the progress of this thread correctly, it ended in 2019 by linking to a StackOverflow which details the code to generate the public key. Unfortunately, I don't have as much experience as @mliudev seems to have - this option sounds out of my realm of expertise.
I am also using a 3rd Party App (Absorb LMS) and trying to integrate Azure B2C as our Identity Provider using OpenIdConnect - and they (Absorb LMS) require the public key be saved in their configuration. Can you please provide updated links/documentation to help aid us in providing Service Providers with the required x5c Key?


Node: fix pubsub IT

Required by #2296
Removing standalone client from a test which checks numsub and shardnumsub together, because shard commands available on cluster only. Those commands are checked in other tests.
fixes: https://github.com/valkey-io/valkey-glide/issues/2327

@Yury-Fridlyand did you make sure this does not happen in other cluster-only tests?

I run all pubsub IT on valkey 8 with this fix on another branch.


When I open Leaguebroadcast,it will get stuck at "Retrieving latest patch info" and crash.

When I open Leaguebroadcast,it will get stuck at "Retrieving latest patch info" and crash.
I have redownloaded but it keeps happening.

Could you send the crash log, available in the Crashes folder within the directory you have LeagueBroadcast?

Crash-2022-11-03-16-31.log

Crash-2022-11-03-18-27.log

This means either your config files are broken or you cannot reach Community Dragon servers. Try deleting Config\Component.json. If that does not work, it must be something with the internet connection

still not working but tks for your help.

Can you tell me what you see when you open https://raw.communitydragon.org/latest/content-metadata.json on the same PC?

{
"version": "12.21.4770420+branch.releases-12-21.content.release"
}

{
"version": "12.21.4770420+branch.releases-12-21.content.release"
}

{
"version": "12.21.4770420+branch.releases-12-21.content.release"
}

same issue in <IP_ADDRESS> version...
This is my crash log.
Crash-2022-11-09-00-25.log

I can open version 1.4.106 today,but i can not open version <IP_ADDRESS> .

Are these issues resolved? As far as I can tell I think I found the issue, but it would be good if you could confirm that for me
@curlydoggy @syuanlyu @thangoghd

Yes,my issues resolved.


Cannot read properties of undefined (reading 'replace')

TypeError: Cannot read properties of undefined (reading 'replace')
at u in src/vs/base/common/labels.ts:410:16
at Wt.db in out/vs/code/electron-main/main.js:82:23744
at Wt.V in src/vs/platform/menubar/electron-main/menubar.ts:652:22
at <anonymous> in src/vs/platform/menubar/electron-main/menubar.ts:503:24
at Array.forEach (<anonymous>)
at Wt.M in src/vs/platform/menubar/electron-main/menubar.ts:478:9
at <anonymous> in src/vs/platform/menubar/electron-main/menubar.ts:484:10
at Array.forEach (<anonymous>)
at Wt.M in src/vs/platform/menubar/electron-main/menubar.ts:478:9
at Wt.N in src/vs/platform/menubar/electron-main/menubar.ts:514:9
at Wt.J in src/vs/platform/menubar/electron-main/menubar.ts:377:9
at Wt.I in src/vs/platform/menubar/electron-main/menubar.ts:270:9
at new Wt in src/vs/platform/menubar/electron-main/menubar.ts:97:8
at I.j in src/vs/platform/instantiation/common/instantiationService.ts:119:18
at I.createInstance in src/vs/platform/instantiation/common/instantiationService.ts:85:18
at M.e in src/vs/platform/menubar/electron-main/menubarMainService.ts:35:17

Go to Errors Site

no longer reporting


Data set not available?

Hey there, i was looking for the data set of whotracks.me for a research for my uni but instead of the data i get this text:
version https://git-lfs.github.com/spec/v1
oid sha256:42f43921beed49e843db30eeeec308209e71669c4a3c650f101fc35596063519
size 219147
On Wednesday it was still available...  Could you help me with that issue?
Thanks and greetings
Lena

We did not have any changes since last Wednesday. The last change was around 12th of November.
git log --all -p -S 42f43921beed49e843db30eeeec308209e71669c4a3c650f101fc35596063519

Points to file whotracksme/data/assets/2018-06/eu/companies.csv (commited Tue Aug 28 15:51:29 2018).
Probably, the download aborted for you at some point. At least, that would be consistent with the error that I now got when trying to download it from scratch:
git clone git@github.com:ghostery/whotracks.me.git
Cloning into 'whotracks.me'...
...
Receiving objects: 100% (16751/16751), 686.46 MiB | 6.90 MiB/s, done.
Resolving deltas: 100% (7209/7209), done.
Downloading static/img/blog/autoconsent/cookie-blocker-after.gif (12 MB)
Error downloading object: static/img/blog/autoconsent/cookie-blocker-after.gif (ddba742): Smudge error: Error downloading static/img/blog/autoconsent/cookie-blocker-after.gif (ddba742eb728b4cd59fbd5b1ca7f136f68b9d5f298511cc96e238275f0c545da):
batch response: This repository is over its data quota.
Account responsible for LFS bandwidth should purchase more data packs to restore access.

So, looks like we exceeded the bandwidth.

@LenaLawless  We increased the quota now. You should be able to get the data again.

Thanks a lot! It works again and I'll download it directly.
Philipp Claßen<EMAIL_ADDRESS>schrieb am Mo., 23. Nov. 2020,
12:06:

@LenaLawless https://github.com/LenaLawless We increased the quota now.
You should be able to get the data again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ghostery/whotracks.me/issues/218#issuecomment-732091121,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARY3Z2UF2C2FZQHUJQT3GRTSRI63PANCNFSM4T5Y3YBQ
.


[Bug] Codeunit 80 ModifyTempLine / Codeunit 90 ModifyTempLine

At the codeunits 80 and 90 there is a bug at the function ModifyTempLine.
In some cases you will get the wrong temprecord values and also get an error that you relate to an old entry.
Our Team made a fix of it:
Codeunit 90:
Standard was:
TempPurchLineLocal.MODIFY;
PurchLine := TempPurchLineLocal;
PurchLine.MODIFY;
New Code:
TempPurchLineLocal.MODIFY;
IF PurchLine.GET(TempPurchLineLocal.RECORDID) THEN
PurchLine.DELETE;
PurchLine.INIT;
PurchLine := TempPurchLineLocal;
PurchLine.INSERT;
Codeunit 80:
Standard:
TempSalesLineLocal.MODIFY;
SalesLine := TempSalesLineLocal;
SalesLine.MODIFY;
new Code:
TempSalesLineLocal.MODIFY;
IF SalesLine.GET(TempSalesLineLocal.RECORDID) THEN
SalesLine.DELETE;
SalesLine.INIT;
SalesLine := TempSalesLineLocal;
SalesLine.INSERT;

@sharkphoenix
Could you provide repro steps leading to the error?
With the suggested fix we have +2 additional SQL requests comparing to the original state:
GET + DELETE + INSERT instead of one MODIFY.
Having a failing scenario would help a lot to understand how to fix it optimally.
I would try to keep MODIFY but reread the record and replace the assignment with TRANSFERFIELDS:
TempPurchLineLocal.MODIFY;
PurchLine.GET(TempPurchLineLocal.RECORDID);  // to refresh the timestamp
PurchLine.TRANSFERFIELDS(TempPurchLineLocal,FALSE); // here we don't copy PK and the timestamp
PurchLine.MODIFY;

Would it work right and faster?

yeah it works :)
in our case we modified a lot of times the line and in one special case it crashes because of the timestamp. But this fix helps. Could you add it to the next CU :)?

Thanks for submitting this issue. We will publish a fix for it in the next minor update for release 15.


Add new size for Switches and add the hover effect

See Figma update 1.2.19 dated 9/25/2023. New selected focus and hover states, label size adjustments etc.
https://www.figma.com/file/wyfVJUHWRMkeCfdB38HFEE/Modus---Web?type=design&node-id=1346-9&mode=design&t=O8SUl2p4FBPLRbCR-0

@ElishaSamPeterPrabhu This is completed, right? Can we close this one?

@ElishaSamPeterPrabhu - can you squash the commits in that PR please? The PR should just be a single commit ideally (right now it's 13).

@coliff What does this mean in the PR? I thought this will automatically squash the commits during merge

I think that's different. I just checked the last 4 PRs which have been merged and none have them were squashed to a single commit.

@ElishaSamPeterPrabhu - can you squash the commits in that PR please? The PR should just be a single commit ideally (right now it's 13).

@coliff  unable to squash commits since merge commit exists in between , maybe we can apply squash and merge?


CLI tools are downloaded for the wrong architecture on AMD Ryzen and Linux Mint

Are you accessing the CLI from the default port :4280 ?
I can't access the CLI from any port.

Make sure you are accessing the URL printed in the console when running swa start!

No URL is printed.
Describe the bug
Wrong CLI tools appear to be downloaded for my CPU arch.
To Reproduce
Steps to reproduce the behavior:

Open a repo on AMD Ryzen
Type in command 'npx @azure/static-web-apps-cli start --verbose=silly'
View the error message containing SIGTERM and [api] /bin/sh: 1: /home/jaypratt/.swa/core-tools/v4/func: Exec format error

Expected behavior
The npm package for the CLI tools should download (or build) a version of the tools for the relevant OS and CPU.
Desktop (please complete the following information):

OS: Linux Mint 21.1 Cinnamon
Version of SWA: latest from npm (i.e. running via npx)

Additional context
Logs:
npx @azure/static-web-apps-cli start --verbose=silly

Welcome to Azure Static Web Apps CLI (1.1.2)

Getting config file options from swa-cli.config.json...
Changed directory to /home/jaypratt/skfltech/skfl
Using configuration "skfl" from file:
  /home/jaypratt/skfltech/skfl/swa-cli.config.json

***********************************************************************
* WARNING: This emulator may not match the cloud environment exactly. *
* Always deploy and test your app in Azure.                           *
***********************************************************************

Checking if localhost:4280 is accepting TCP connections...
Port 4280 is available. Use it.
Resolved port number: 4280
appDevserverUrl provided, we will try connect to dev server at dist
Api Folder found: /home/jaypratt/skfltech/skfl/api
Trying to read workflow config with values:
 - appLocation: /home/jaypratt/skfltech/skfl
 - outputLocation: http://localhost:5173
 - apiLocation: /home/jaypratt/skfltech/skfl/api
Found a SWA workflow file: /home/jaypratt/skfltech/skfl/.github/workflows/azure-static-web-apps-lively-pond-0f8c8b91e.yml
 - app_build_command: <undefined>
 - api_build_command: <undefined>
 - app_location: /
 - output_location: dist
 - api_location: api
 - data_api_location: <undefined>
Validating user workflow config (BEFORE):
 - appLocation: /home/jaypratt/skfltech/skfl
 - outputLocation: http://localhost:5173
 - apiLocation: /home/jaypratt/skfltech/skfl/api
Validating user workflow config (AFTER):
 - appLocation: /home/jaypratt/skfltech/skfl
 - apiLocation: /home/jaypratt/skfltech/skfl/api
 - outputLocation: http://localhost:5173
 - dataApiLocation: <undefined>
Workflow configuration:
- config:
  - appBuildCommand: <undefined>
  - apiBuildCommand: <undefined>
  - appLocation: /home/jaypratt/skfltech/skfl
  - apiLocation: /home/jaypratt/skfltech/skfl/api
  - outputLocation: http://localhost:5173
  - dataApiLocation: <undefined>
 - files:
   - 0: /home/jaypratt/skfltech/skfl/.github/workflows/azure-static-web-apps-lively-pond-0f8c8b91e.yml
User workflow config:
 - appBuildCommand: <undefined>
 - apiBuildCommand: <undefined>
 - appLocation: /home/jaypratt/skfltech/skfl
 - apiLocation: /home/jaypratt/skfltech/skfl/api
 - outputLocation: http://localhost:5173
 - dataApiLocation: <undefined>
- files:
  - 0: /home/jaypratt/skfltech/skfl/.github/workflows/azure-static-web-apps-lively-pond-0f8c8b91e.yml
Running echo 'No Data API found'. Skipping
Starting the SWA emulator with the following configuration:
- ssl:
  - 0: false
  - 1: <undefined>
  - 2: <undefined>
- env:
  - SWA_RUNTIME_CONFIG_LOCATION: /home/jaypratt/skfltech/skfl
  - SWA_RUNTIME_WORKFLOW_LOCATION: /home/jaypratt/skfltech/skfl/.github/workflows/azure-static-web-apps-lively-pond-0f8c8b91e.yml
  - SWA_CLI_DEBUG: silly
  - SWA_CLI_API_PORT: 7071
  - SWA_CLI_APP_LOCATION: /home/jaypratt/skfltech/skfl
  - SWA_CLI_OUTPUT_LOCATION: http://localhost:5173
  - SWA_CLI_API_LOCATION: /home/jaypratt/skfltech/skfl/api
  - SWA_CLI_DATA_API_LOCATION: <undefined>
  - SWA_CLI_DATA_API_PORT: undefined
  - SWA_CLI_HOST: localhost
  - SWA_CLI_PORT: 4280
  - SWA_CLI_APP_SSL: false
  - SWA_CLI_APP_SSL_CERT: <undefined>
  - SWA_CLI_APP_SSL_KEY: <undefined>
  - SWA_CLI_STARTUP_COMMAND: npm run dev
  - SWA_CLI_VERSION: 1.1.2
  - SWA_CLI_SERVER_TIMEOUT: 60
  - SWA_CLI_OPEN_BROWSER: false
- commands:
  - swa: node<EMAIL_ADDRESS>  - api: cd "/home/jaypratt/skfltech/skfl/api" && "/home/jaypratt/.swa/core-tools/v4/func" start --cors "*" --port 7071
  - dataApi: <undefined>
  - run: cd "/home/jaypratt/skfltech/skfl" && npm run dev
[api] /bin/sh: 1: /home/jaypratt/.swa/core-tools/v4/func: Exec format error
[api] cd "/home/jaypratt/skfltech/skfl/api" && "/home/jaypratt/.swa/core-tools/v4/func" start --cors "*" --port 7071  exited with code 126
--> Sending SIGTERM to other processes..
[run] cd "/home/jaypratt/skfltech/skfl" && npm run dev exited with code SIGTERM
--> Sending SIGTERM to other processes..
[swa] node<EMAIL_ADDRESS>exited with code SIGTERM
✖ SWA emulator stoped because SWA emulator exited with code SIGTERM.

Sorry, mixed up AMD and ARM. Updated comment and title

I attempted to use the azure-cli installation from https://learn.microsoft.com/en-us/cli/azure/install-azure-cli-linux?pivots=apt and this also failed (it seems https://packages.microsoft.com/repos/azure-cli vera Release does not have a release file).
Not sure how to proceed.

Hmm, removing the config file lets me use swa start ./dist but running with swa ./dist --api-location api fails.
It appears it is ~/.swa/core-tools/v4/func that is the wrong format, not swa itself.

@Cypher1 , I think this is the same issue I am seeing, just tagging here to associate the bugs.
https://github.com/Azure/static-web-apps-cli/issues/703

Thanks @kevinrs5855
Looks similar but for /Users/XXXX/.swa/dataApiBuilder/0.6.14/Microsoft.DataApiBuilder: cannot execute binary file rather thank the v4 API. Interesting. Not sure if the contributors will prefer to consider these separate or duplicate bugs. Keen to stay in contact about it though

Thanks for reporting this, we will investigate

Any news? We're keen to get started building with Azure. Would be nice to have at least an idea of the resolution path :)

Found a workaround. It seems that the issue was that swa was installing the azure-functions-core-tools itself and doing something wrong in the process.
We found that manually installing https://github.com/Azure/azure-functions-core-tools via the npm method bypassed this broken process and we were then able to run with the local api server AND the local server (which was previously).
Still learning how to get the two working together neatly (currently I'm coordinating the port numbers manually) but we're well on the way. Thanks for the tooling, looking forward to the out of box experience being as good as the dev experience (which I'm already enjoying).
Cheers

@nogic1008 Yup! We're hitting this now too. @thomasgauvin Any news?

The fix was found and awaiting release

The bugs on both azure-functions-core-tools and DataApiBuilder are caused by unzipping package "unzipper" with node 18. We are happy to inform you that we have released a new version that fixes the issue. Please update swa-cli to v1.1.4 and let us know if you encounter any problems.


OOM Exceptions on heavy Usage

Describe the bug
We tried using Garnet in our systems and encountered a unexpected behaviour.
Our usage consist of writing and reading a lot of keys with TTLs. The TTLs never get renewed, but are let to be expired in which case new keys are written (the keys have timestamps in them).
The value sizes are from some bytes up to almost 100MB per key.
We only use the String objects so object store is disabled.
When running the system under load it starts to use incresingly more RAM, up to the point that it OOMs due to Kubernetes Memory Limits.
There isnt much in the logs besides an endless stream of OOM Exceptions
Parameters are "--memory=1g", "--index=1g", "--page=512m", "--no-obj=true", "--metrics-sampling-freq=1"
Garnet uses over 3.5GB in those cases.
As reference we switched to Redis in the same application, the max storage required was around 300MB, with no OOM issues what so ever.
Steps to reproduce the bug

Have garnet running
Cyclically write a lot of keys of variable sizes with TTLs
on expiration of keys, write new ones of aproximately the same size
See Memory rise
wait for Garnet to OOM

Expected behavior
Garnet should reuse the memory pages it has when keys expire or have a way of compacting its memory so a high TTL-Key rate does not lead to a memory leak.
Screenshots
No response
Release version
1.0.18
IDE
No response
OS version
K8s
Additional context
No response

Thanks for the report. I don't know why an OOM happens in this case. Do you have a repro client app you can include here?

Ping

@hr-alebel along with OOM allocating stack. It would be good to know the container memory provisioned for the Pod. Have you configured  - DOTNET_GCHeapHardLimitPercent? The default value is 75% which means you need to add some overhead or increase this value

@msft-paddy14 the provisioned memory was around 3.7GB
The data stored is around  300MB max (also thats around the size of memory redis uses now)
@TedHartMS Sry for the late response. I had some stress fixing other stuff too, will try to make a app that repliactes the issue today

@TedHartMS https://github.com/hr-alebel/GarnetOOMIssue
I created a small repo. it isnt the most beutiful one, but it gets the job done for me. let it run for a bit and see the memory rise.
Garnet ran in docker desktop using docker run --name garnet-service  -p 6379:6379 ghcr.io/microsoft/garnet --page=512m --memory=1g --index=1g --no-obj=true --metrics-sampling-freq=1

I think the issue might not actually be the writing, but reading. Will do some more tests right now and update the repo

Updated the repo with using "MEMORY USAGE", this basically instantly OOM garnet. Not sure if some part of our software uses that, its a giant PHP Symfony project and there is a lot of blackbox behaviour, but regardless, getting the size of keys shouldnt lead to OOM exceptions.

FYI: I tried to reproduce issue. It took 50 seconds till Exception. Garnet took 3.2GB. Exception message:
ProcessMessages threw an exception:
    System.OverflowException:
        Arithmetic operation resulted in an overflow.
        at System.GC.AllocateNewArray(IntPtr typeHandle, Int32 length, GC_ALLOC_FLAGS flags)
        at System.GC.AllocateArray[T](Int32 length, Boolean pinned)
        at Garnet.server.ScratchBufferManager.ExpandScratchBuffer(Int32 newLength)
        at Garnet.server.ScratchBufferManager.FormatScratch(Int32 headerSize, ReadOnlySpan`1 arg)
        at Garnet.server.StorageSession.GET[TContext](ArgSlice key, ArgSlice& value, TContext& context)
        at Garnet.server.StorageSession.MemoryUsageForKey[TContext,TObjectContext](ArgSlice key, Int64& memoryUsage, TContext& context, TObjectContext& objectContext, Int32 samples)
        at Garnet.server.GarnetApi`2.MemoryUsageForKey(ArgSlice key, Int64& memoryUsage, Int32 samples)    at Garnet.server.RespServerSession.NetworkMemoryUsage[TGarnetApi](TGarnetApi& storageApi)
        at Garnet.server.RespServerSession.ProcessMessages()
        at Garnet.server.RespServerSession.TryConsumeMessages(Byte* reqBuffer, Int32 bytesReceived)

I've disabled MEMORY USAGE loop and no Exceptions after 5 mins. GarnetServer memory usage MAX 1.3GB.

Considering the max Stored Data is ~300MB thats still a lot of overhead, but at least not as critical.


HDDS-11759. Remove LegacyReplicationManager

What changes were proposed in this pull request?
Due to HDDS-11516 release note, the LegacyReplicationManager would be removed in the  Ozone 2.0.0.
What is the link to the Apache JIRA
Please take a look at HDDS-11759 for more details.
How was this patch tested?
It can tested by CI, also pass the tests in local.

Thanks @chiacyu for the patch.  Please enable workflows (at least build-branch) in your fork.

Thanks @chiacyu for updating the patch.  Looks like the checkstyle issue was missed: https://github.com/chiacyu/ozone/actions/runs/12356830790/job/34483652628

Thanks @chiacyu for the patch, @sodonnel for the review.


problem with orderBy and enableAjaxTable

I would like to order by created_at my list view.
So i have add
$this->crud->orderBy('updated_at','desc');
I need Ajax table, so i have also
$this->crud->enableAjaxTable();
I've read in the doc that date and datetime columns won't be sortable anymore with enableAjaxTable :-(
Maybe this is why I have this error :
QueryException in Connection.php line 770:
SQLSTATE[42000]: Syntax error or access violation: 1140 Mixing of GROUP columns (MIN(),MAX(),COUNT(),...) with no GROUP columns is illegal if there is no GROUP BY clause (SQL: select count(*) as aggregate from `personnes` order by `updated_at` desc)

How can I order by created_at with ajax enable ? Any way to do it ?

Same problem here

Hey,
This is a known issue. The issue is actually with a third party library used for the ajax datatables. If you have the time and will you can overwrite the AjaxTrait from the package as well as the CrudController and (to reflect the new AjaxTrait) or you can wait for a solution.
What I did was overwrite the AjaxTrait and that helped fix part of the issue, but since im still using that 3rd party library(https://github.com/LiveControl/EloquentDataTable) I also still have the sort by date issue.

Same problem here and it is happening because of $this->crud->orderBy('updated_at','desc');
It is mentioned not use orderBy with Date & Datetime while using ajax

date and datetime columns won't be sortable anymore

But the problem is orderBy is not working with any datatype. i.e; id,name, etc.

This seems to work when adding $this-crud->orderBy('ANY_FIELD');.  I am trying to order on an indexed string field and getting this error.

I just want to throw in my two cents.  I have backpack running a fairly high-end server.  We start to see 20 second page loads with as few as 2000 records, displaying only 5 fields.
We had to completely disabled the "views" area of debugtoolbar because the site wouldn't even load.
$this->crud->enableAjaxTable(); won't be an option for use if the sorting doesn't work.  I think our only option will be to write a custom index blade for the setup method of CRUD.
I want to see backpack succeed, but I can't pitch it to clients as it stands.

Do you mean that you can't set the data tables default ordering?

@OwenMelbz  When you use ajax table, the search function that retrieves the data uses this library: https://github.com/LiveControl/EloquentDataTable. The order by date actually works, but this library takes the next column when you order by date so there's where the problem comes from.
@ChrisThompsonTLDR  that's what I'm worried about as well, but as a quick solution you can overwrite the AjaxTrait in backpack inside your project and change the search function to behave normally. I will let you know when I have time to work on a solution

@indra1 would it be worth you publishing the rewritten AjaxTrait so it can be used in a PR? as it might be quicker to get implemented? Open Source projects need community contributions as well as feedback!

@OwenMelbz I will post here what I did. I'm reluctant to publish it. It's a quick hack and I need to go back to it and do it properly. I can in no way do a pull request with that

Closing this as its a known issue which needs addressing, have added it to the consolidated list.

$this->crud->query->orderBy('updated_at', 'DESC');
worked for me !


'with_format' is extremely slow when used together with 'interleave_datasets' or 'shuffle' on IterableDatasets

Describe the bug
If you:

Interleave two iterable datasets together with the interleave_datasets function, or shuffle an iterable dataset
Set the output format to torch tensors with .with_format('torch')

Then iterating through the dataset becomes over 100x slower than it is if you don't apply the torch formatting.
Steps to reproduce the bug
import datasets
import torch
from tqdm import tqdm

rand_a = torch.randn(3,224,224)
rand_b = torch.randn(3,224,224)

a = torch.stack([rand_a] * 1000)
b = torch.stack([rand_b] * 1000)

features = datasets.Features({"tensor": datasets.Array3D(shape=(3,224,224), dtype="float32")})
ds_a = datasets.Dataset.from_dict({"tensor": a}, features=features).to_iterable_dataset()
ds_b = datasets.Dataset.from_dict({"tensor": b}, features=features).to_iterable_dataset()

# Iterating through either dataset with torch formatting is really fast (2000it/s on my machine)
for example in tqdm(ds_a.with_format('torch')):
    pass

# Iterating through either dataset shuffled is also pretty fast (100it/s on my machine)
for example in tqdm(ds_a.shuffle()):
    pass

# Iterating through this interleaved dataset is pretty fast (200it/s on my machine)
ds_fast = datasets.interleave_datasets([ds_a, ds_b])
for example in tqdm(ds_fast):
    pass

# Iterating through either dataset with torch formatting *after shuffling* is really slow... (<2it/s on my machine)
for example in tqdm(ds_a.shuffle().with_format('torch')):
    pass

# Iterating through this torch formatted interleaved dataset is also really slow (<2it/s on my machine)...
ds_slow = datasets.interleave_datasets([ds_a, ds_b]).with_format('torch')
for example in tqdm(ds_slow):
    pass

# Even doing this is way faster!! (70it/s on my machine)
for example in tqdm(ds_fast):
    test = torch.tensor(example['tensor'])

Expected behavior
Applying torch formatting to the interleaved dataset shouldn't increase the time taken to iterate through the dataset by very much, since even explicitly converting every example is over 70x faster than calling .with_format('torch').
Environment info

datasets version: 2.16.1
Platform: Linux-6.5.0-15-generic-x86_64-with-glibc2.38
Python version: 3.11.6
huggingface_hub version: 0.20.3
PyArrow version: 15.0.0
Pandas version: 2.2.0
fsspec version: 2023.10.0

The "torch" formatting is usually fast because we do zero-copy conversion from the Arrow data on your disk to Torch tensors. However shuffling seems to do data copies that slow down the pipeline, and shuffles python objects instead of Arrow data.
To fix this we need to implement BufferShuffledExamplesIterable.iter_arrow() (same as regularBufferShuffledExamplesIterable.iter() but yields Arrow tables)
https://github.com/huggingface/datasets/blob/b7d854b7fd3e9a330e21b76ee8421d4a7ebb4a7a/src/datasets/iterable_dataset.py#L968-L974


How can i get code without build?

It hard for me for edit a little thing on blockly game code

Hi Teeratat,
You can get the Blockly Games code using the green "↓Code" button on the Blockly Games repo homepage.  (You'll need to have git and/or a GitHub client installed.)
After that you don't need to build—just point your browser at the file blockly-games/appengine/index.html and away you go.

@cpcallen: I'm afraid it doesn't work this way. (I downloaded the zip file with code and unzipped it into c:\blockly-games) It seems like there is hardcoded some link to "generated" files. Am I doing something wrong?

I don't think @cpcallen got this one right.  There are three approaches:

If you want to run Blockly Games locally, and don't want to make any modifications, then download the offline versions.  Everything is right there.  https://github.com/google/blockly-games/wiki/Offline  But this pre-compiled version cannot be edited.
If you want to run Blockly Games locally, and you do want to make any modifications, then download the sources as described above.  Then visit blockly-games/appengine/admin.html and choose "Debug mode".  Then you can go to index.html and play.  Edits to the src files will work fine.  But this doesn't work well over the Internet, since debug mode doesn't build/compress the files, meaning the load times will be very large.

Thank you for your help, but I think there is still some misunderstanding.
I'm trying the option 2 (to be able to make modifications).
In offline version there is no admin.html.
In github sources there it is, so I can switch to debug mode (which I can see in console "Uncompressed mode activated.  Happy hacking!") but when I go to index.html then I get very similar errors.

I feel the problem is somewhere in boot.js around here:
  // Load the chosen language pack.
  var script = document.createElement('script');
  if (debug) {
    script.src = 'generated/msg/' + lang + '.js';
  } else {
    script.src = appName + '/generated/msg/' + lang + '.js';
  }
  script.type = 'text/javascript';
  document.head.appendChild(script);
  // Load the code bundle for the chosen game.
  var script = document.createElement('script');
  if (debug) {
    script.src = appName + '/generated/uncompressed.js';
  } else {
    script.src = appName + '/generated/compressed.js';
  }
  script.type = 'text/javascript';
  document.head.appendChild(script);

There is nothing like generated or msg or uncompresed in the repository. Seems like I still have to build it at least once. (?) (I hoped to avoid getting python or java just for making javascript from javascript.)


libwhb: Rename render-related functions to be less ambiguous

I renamed the following functions so that it's more obvious what they do:

WHBGfxBeginRender -> WHBGfxWaitForSwap
WHBGfxFinishRender -> WHBGfxSwapScanBuffers (automatically calls WHBGfxWaitForSwap)
WHBGfxClearColor -> WHBGfxClearColorDepthStencil (this is what the function actually does)
WHBGfxBeginRender{DRC/TV} -> WHBGfxMake{DRC/TV}ContextCurrent
WHBGfxFinishRender{DRC/TV} -> WHBGfxCopy{DRC/TV}ColorBufferTo{DRC/TV}ScanBuffer

Since these existed for a long time, I deprecated them instead of removing them to retain compatibility with existing homebrew.
I also added the following functions for convenience:

WHBGfxClearColorOnly (this should have been the real WHBGfxClearColor)
WHBGfxClearDepthStencil
WHBCopyTVColorBufferTo (useful for duplicating TV output to DRC/Gamepad)
WHBCopyDRCColorBufferTo (^^^)

I do not think I agree with this change.
Yes you renamed the functions to describe what GX2 functions they call, but this isn't necessarily an improvement.
The WHBGfx stuff is supposed to be a simple library to help get something on the screen. We do not need to bleed implementation details into the function names, like WHBGfxBeginRender and WHBGfxEndRender are just functions you call when you start rendering and end rendering, and internally they will do whatever they need to do to make this happen.
I think a user who wants to know WHBGfxBeginRender is actually waiting for a swap or WHBGfxBeginRenderDRC is just setting the current context to the one correctly initialised for DRC rendering is better off just using more GX2 APIs directly over WHBGfx rather than renaming all the WHBGfx APIs to describe their internal implementation details.

I agree with @exjam.
Another thing to point out is that there are certain codebases (such as SDL) which currently depend on libwhb and which would be needlessly broken by this set of changes.
On top of that, I consider libwhb to be quite limited, containing several things that are flawed by design, or that don't belong in wut. For this reason I would not recommend its usage regardless. This leads me to prefer deprecating/replacing libwhb altogether with new wrapper APIs that are more generally useful and flexible.

I agree with fincs, a more advanced wrapper API would probably be the best option to fix WhbGfx's flaws.
Just my thoughts about those changes, and why I had approved them:
Libraries like GLFW and EGL also work with buffer swap calls and contexts, which in my opinion would make it easier for someone to understand coming from that side.
glfwMakeContextCurrent -> WHBGfxMake{DRC/TV}ContextCurrent
glfwSwapBuffers -> WHBGfxSwapScanBuffers
But especially the WHBGfxBeginRenderTV / WHBGfxBeginRenderDRC seem kind of confusing to me.
Someone might as well do something like this outside of the rendering loop:
    // Enable depth testing
    WHBGfxBeginRenderTV();
    GX2SetDepthOnlyControl(TRUE, TRUE, GX2_COMPARE_FUNC_LESS);
    WHBGfxBeginRenderDRC();
    GX2SetDepthOnlyControl(TRUE, TRUE, GX2_COMPARE_FUNC_LESS);

There is now nothing going on related to begin rendering a frame, but the function names suggest that.
Regarding breaking codebases: This PR seems to be compatible with older codebases, by only deprecating the older functions.

Libraries like GLFW and EGL also work with buffer swap calls and contexts, which in my opinion would make it easier for someone to understand coming from that side.

I had in mind something inspired by the fencing/presentation model used in APIs such as Vulkan, which seems to suit GX2's workflow better. I.e. a WaitForPresent function that waits for a flip, and a Present function that copies the specified color buffers to the TV/DRC's scan buffers + applies a scan buffer swap. Context management would be handled user-side. Either way your point is taken.

Regarding breaking codebases: This PR seems to be compatible with older codebases, by only deprecating the older functions.

While the code in this PR attempted to preserve source-level compatibility, it introduced ABI changes that render existing compiled code (i.e. pacman packages) incompatible.


Interceptor access to WebSocket Connection

Feature Request :
when use web socket, I want to get dns ip result which tcp connect success

This kind of thing is available on the connection in an interceptor for regular calls, but not for web sockets.

+1
Or at least make it possible to set http headers on the request that opens the socket

Would love this as well - trying to make a HttpLoggingInterceptor equivalent seems impossible at this point.

Yes, pleaaase. I would love this for logging

Did this ever get released, I'm looking to log my websocket traffic in a similar fashion to my http traffic.

BTW Interceptor and Logger are probably two different features.  I suspect we should implement Logger first.

Yeah, would be nice to have some kind of logging

Related issue https://github.com/square/okhttp/issues/5833

This would be great to have +1

+1

+1

Closing as low priority. Not implemented after almost 5 years.


GTO trade has ridiculously high value

Trade with tx hash 0x1e0b26077435656a42d3e648fb9dbf5d1422c63d435cf4126066ef1e1bfba0e7 appears to be valued as a huge amount when it looks like it shoudln't be anymore than $18.

This is because the converter has a small amount of BNT and a minuscule amount of GTO.
I've fixed the high value on the swap and hard coded the volume for now, this will all change upon final adoption of the API.

This is because the converter has a small amount of BNT and a minuscule amount of GTO.
I've fixed the high value on the swap and hard coded the volume for now, this will all change upon final adoption of the API.


Polish the Terraform script and Ansible playbook

Allow using both remote SkyWalking binary package or locally built package.
Add AWS access_key and secret_key to Terraform variables so users do not need to configure the AWS credentials locally for this repo instead of globally.

FYI @rabajaj0509


Spec out Yearn vault

Determine how a yearn Vault can address some of our needs with respect to:

making usage of Hyperdrive easier
creating a bond wrapper on PTs that can be used as collateral

Hyperdrive could benefit from a vault in a variety of ways
Four use cases for Hyperdrive vaults:

wrap PT tokens
make Hyperdrive easier to use (automate logic, rolling positions, LP only when favorable?)
create a new Hyperdrive yield source (could then be used to hedge borrow)
demonstrate the benefit of fixed rate borrows in combination with other positions

Use case 1: wrap PT tokens
Motivation:
Since PT tokens are ERC1155, they are harder to integrate into the DeFi ecosystem.
One way to increase composability is by wrapping these tokens into an ERC20 wrapper, which is then easier to integrate.
A very simple vault can achieve this, by taking PT deposits and hanging out LP tokens.
Inputs:
PT tokens (ERC1155)
Outputs:
LP tokens (ERC20)

Based on yesterday's discussion, we're going forward to spec out an implementation of a "Funge hyperdrive positions" vault, now being tracked in https://github.com/delvtech/hyperdrive/issues/1027


Preview 1.80.12 Hotfix Release

What's New? - 1.80.12

[Fix] Threading exception on certain UI components, causing the launcher to crash, by @shatyuka
[Fix] Sophon Installer crashed due to invalid type of Language String to Locale Code converter, by @neon-nyan
[Fix] Honkai: Star Rail Game Repair and Cache Update issues, by @bagusnl and @neon-nyan

Adding fallback method to lookup values in Honkai: Star Rail gateway API if the metadata KVP is not updated/unmatched
Updating metadata's Key-Value pairs.
Adding debug information for the Gateway API parser under Debug build
Check for End-Of-File on the buffer
Fix the location of certain patch-marked files under Persistent Folder
Fix FileNotFound error on certain file marked under Persistent but found under StreamingAssets by hardlinking the file

[Imp] Add new game settings for Honkai: Star Rail, by @bagusnl

Character Shadow in Map Exploration: self explanatory
Half Resolution Transparency: Controls resolution of objects behind transparent plane (window, etc.)

Templates

Changelog Prefixes
  **[New]**
  **[Imp]**
  **[Fix]**
  **[Loc]**
  **[Doc]**

I don't think the block except Console.WriteLine stuff belongs inside the DEBUG flag, no?
https://github.com/CollapseLauncher/Hi3Helper.EncTool/blob/2fc3be162c840c607192e7dfa919d89bb47344fb/Parser/Proto/src/StarRailGatewayStatic.cs#L188-#L208

I don't think the block except Console.WriteLine stuff belongs inside the DEBUG flag, no?
https://github.com/CollapseLauncher/Hi3Helper.EncTool/blob/2fc3be162c840c607192e7dfa919d89bb47344fb/Parser/Proto/src/StarRailGatewayStatic.cs#L188-#L205
Edit: fixed in ab6f276<EMAIL_ADDRESS>

Please do not mark this inside DEBUG flag. This is being used for fallback if proto ID is invalid or out-of-date.

I don't think the block except Console.WriteLine stuff belongs inside the DEBUG flag, no?
https://github.com/CollapseLauncher/Hi3Helper.EncTool/blob/2fc3be162c840c607192e7dfa919d89bb47344fb/Parser/Proto/src/StarRailGatewayStatic.cs#L188-#L205
Edit: fixed in ab6f276<EMAIL_ADDRESS>

Please do NOT flag it as DEBUG feature, I intentionally make it as a fallback if proto static is failing to get the value.
I beg you. HoYo is being annoying recently :pepeSob:


invalid covariant return type of LTISystem

I'm getting invalid covariant return type compiler error when builidng a LTISystem:
std::shared_ptr<ct::core::ControlledSystem<state_dim, control_dim> > Dynamics(
        new ct::core::LTISystem<state_dim, control_dim>(A, B));

error: invalid covariant return type for ‘const Eigen::Matrix<double, STATE_DIM, STATE_DIM>& ct::core::LTISystem<STATE_DIM, CONTROL_DIM>::getDerivativeState(const ct::core::StateVector<STATE_DIM>&, const ct::core::ControlVector<CONTROL_DIM>&, double)

I think the problem comes from the incoherency with StateMatrix StateVector ControlMatrix ControlVector types, and maybe solved as suggested in #39

Dear @yangcyself
thanks for reporting, I will prioritize this bug in the next release!
Best

Will be fixed v3.0.3, see also commit 5f73b9e on the devel branch.


raidboss: p4s part two headmarkers and safe spot triggers

Currently WIP. I may add additional triggers for the acts here.
Currently this may call out Act 1 and Act's "north/south first" or "east/west first" safe spot during the tethers going out, but relies on combatant order being what determines the position of the thorns which may or may not be static. Not yet working, but the maps should be correct.

Currently these don't work. I am guessing the headmarker that is first in the instance is wrong on top of the combatant numbers are off that it registered into the array as there were no matches.

I'm not sure how we want to handle curtain call. I think the generic requirement is that no back-to-back of the same role will be followed, but not sure if it's possible to have tank/healers at 12 and dps at 6 or some other strange timing strategy. Right now it just does dps tank/healer dps tank/healer etc.

I'm tested lastest version. No Safe Sports appears. Thorns Collector seems like not working

Below is yesterday version

Thank you so much!!!

I somewhat recall there being discussion that the callOverlayHandler stuff doesn't work in raidemulator... but maybe I am misremembering. I don't fully understand what the overlayhandler does other than we can get more up to date information from it regarding combatants than we do in the network logs. I have written with it a few times now, so I kind of know what sort of things it can do and/or things it would be required for.
For the thorns, the alternative approach would be for cactbot to record the addcombatants outside of combat and use that in the next combat (before instance reset), but I am not aware of a feature in cactbot to do that. It may have that option, I just haven't used it.

based on what i am seeing from output, this condition is bugged

EDIT: 2nd callout for curtain call works, but the first call out is still double calling somehow

Ok, thanks.  I'll take a look in raidemulator and see if I can see anything obvious.

Is it that I left out matches in the function that Conditions.targetIsYou bugs out?


Add microseconds to waveform name if necessary.

[WIP, just found the tests folder]
This pull request fixes #44 by checking for a start and end time within the same second, then adding microseconds if necessary. PR's to ASDF_definition and asdf_validate address this potential change to the ASDF file.
I don't think it's necessary to add the additional check
if s == e:
    raise ValueError("Some sensible error message")

The goal was to avoid rejecting separate waveforms that matched timestamps in seconds but not microseconds. If the start and end still match in microseconds that probably just means an npts=0 trace was added. Maybe someone has a reason to do this?

Coverage increased (+0.1%) to 89.024% when pulling b32f11616bb5a89ee0ee45d523ed8f76b9208dec on jes-p:master into c976927284efdec1264d652d47e4d0048dfe6f5c on SeismicData:master.


action_sheet: Redesign bottom sheet

Matches the design of the bottom sheet with the Figma design https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=3481-27050&m=dev.
Made the bottom sheet take as much space as needed and when it takes more than a screenful, the option button list is scrollable.

Before
After

After (more than a screenfull)
After - Dark

Screen recording (screenful with top and bottom shadows)
https://github.com/user-attachments/assets/71880a57-3270-4278-925f-8c79c2f8d3f2
Fixes: #90

Thanks @chrisbobbe for the review! Changes pushed. PTAL.

Thanks for the comments @gnprice. Pushed the new revision. This time only checked for the "Star message" and "Cancel" buttons to test if the action sheet is shown.

Thanks @chrisbobbe for the review! Changes pushed.

Thanks @chrisbobbe for the review and the proposed approach! I prefer yours and there is no need for the "dynamic padding".
Changes pushed. Please have a look!

_InsetShadowBox was previously implemented with a SingleChildScrollView wrapped by a NotificationListener.
The current implementation got a lot simpler because I realized that we always have some paddings such that the content does not get covered by the gradient (and thus the shadow is not visible without scrolling). It seems that we also have the paddings here, it should be a straightforward refactor later to reuse the widget.
That PR will probably land later than this one, but you can cherry-pick ccf2cf76bac8e012f60458be5d7fab64d2e81cb3 to use InsetShadowBox here.

Thanks @chrisbobbe and @gnprice for the reviews. Changes pushed.

Thanks for the review @gnprice. Changes applied, PTAL.

All looks good; and I did that manual testing of the scrolling behavior, and that looks good too. Merging.
Thanks again @sm-sayedi for all your work on this!


Gc safe embedding api changes

Most of the changes stem around from removing usages of MonoImage, MonoAssembly, and Domain in native. Also implemented exception_from_name_msg and mono_class_get_name among the other changes.
zVerificationCoreCLR passes with these changes.

It would be nice to have managed tests for everything so that we have the option to delete the native tests if we want too.


to import csv file to a table in mysql, should it already exist?

i am trying to build a dynamic table uploaded by a user using odo but the table is not being created dynamically. is there any inbuilt method i can use or should i resort to manually creating the table and then inserting it.
datashape = discover(resource(csvfilepath))
uri=mysql+pymysql://'+username]+':'+password+'@localhost/'+databasename+"::"+tablename
x=odo(csvfilepath,uri,dshape=datashape)
this gives me an error that database.tablename doesnt exist

With a simple three-column CSV I have no trouble doing this with SQLite:
>>> odo.odo('demo.csv', 'sqlite:///demo.db::demo')
Table('demo', MetaData(bind=Engine(sqlite:///demo.db)), Column('0', Text(), table=<demo>), Column('1', BigInteger(), table=<demo>, nullable=False), Column('2', Text(), table=<demo>), schema=None)

This creates the SQLite database file and table as expected. The process would be the same for MySQL.
Please double check that the database you are trying to insert into exists, and the user that odo is running as has permission to create tables in it. Those could be causes of the error message you're seeing.

So the tables that are being created are dynamic generated based on the user input csv file, the user connecting the database is root. @necaris  I am looking for a generic solution

@kunchamharsha It's not clear to me what you mean. Which part of the table is dynamic? The name? The schema? The existence of the table? All three?
If you had a program which would accept a user's CSV file, then based on its name create a corresponding table in the database, then what you describe should work providing that the target database exists.  Of course, I would never advise connecting to a database as root unless you really have to, and especially not from a script -- consider creating a user that has permissions only on the database you want it to use.

so this is user flow, user uploads csv file, we analyse the csv and create a csv schema and then create the table and upload the content. I am currently using pandas dataframe but it is pretty slow.

If you have a smaller sample of the code that is not working and a full traceback, please provide them.


operator project-quay (3.9.0-nightly.20230207)

Add 3.9.0-nightly.20230207 to community operators

/merge possible

/merge possible


Automated cherry pick of #2415: webconsole: update aliyun vnc base url and uniq vnc session

Cherry pick of #2415 on release/2.12.
#2415: webconsole: update aliyun vnc base url and uniq vnc session

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@8e1f581). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.12   #2423   +/-   ##
==============================================
  Coverage                ?   7.09%
==============================================
  Files                   ?     456
  Lines                   ?   71666
  Branches                ?       0
==============================================
  Hits                    ?    5083
  Misses                  ?   66165
  Partials                ?     418

Flag
Coverage Δ

#aFlag
7.09% <ø> (?)

Continue to review full report at Codecov.

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

/lgtm

/approve


Fix pieces init

Proposed Semver Increment Bump: PATCH
Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)

Fix for pieces initialization

(If Applicable) What Issue does it fix?

Removed command init from loadSingleCommand.js (now in initialize.js)
Fixed loadProvider.js not using getfileListing return value correctly
Fixed initialize.js to properly loop, AND await init of every piece.

Integrated into #118


*Making discount factor intuitivly accurate

Rename fee discount factor to fee subsidy factor (synonomous, but encourages the necessary configuration change).
We make all calculations based on (1 - old_value) so that the term itself makes sense.
Now we have "Fee subsidy - 0 means no subsidy"
Renamed a few variables and included validation at runtime.

Note this change requires a configuration change.

Note this change requires a configuration change.

Yep, renamed the PR with an * and will make corresponding config change.


de-duplicate otlpserver.mapToKVString and otelcli.flattenStringMap

As discussed in #120, @edw-eqix noticed that there was a copy of this function in the otlpserver package and we could merge. I think the one in otelcli/helpers.go has tests and all we need to do is make it public. otelcli.flattenStringMap is a terrible name, I don't know what this @tobert person in the blame was thinking. A new, clear name should be chosen.
The helper version also implements a custom empty return value so you can return e.g. {} when the map is empty. Calls to it in otlpserver will need both name and arguments updated, probably to add {}, but also try "" and see what the tests do.
So, the steps follow. Each step can/should be a commit. We can squash (or not) at merge time.

rename the helper

starts with a capital letter
please come up with a better name

refactor otelcli package references to use the new name

tests should pass again at this point

replace mapToKVString in otlpserver with otelcli.flattenStringMap

test suite exercises this so if tests pass you're done

Will be a non-issue when #221 merges, closing.