Investigate GNU/Posix MacOS distribution issues

Task Description
We are seeing the following issue with MacOS builds:
ERROR] Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:3.1.1:single (default) on project strongbox-distribution: Execution default of goal org.apache.maven.plugins:maven-assembly-plugin:3.1.1:single failed: group id ‘960648751’ is too big ( > 2097151 ). Use STAR or POSIX extensions to overcome this limit -> [Help 1]

@forbob contributed #1008, which I had to revert (as part of #1068), because it was causing the strongbox-web-core-1.0-SNAPSHOT-spring-boot.jar to have it’s name truncated in the strongbox-distribution to something like strongbox-web-core-1.0-SNAPSHOT, inside the strongbox-distribution ‘s tar.gz.
We need a permanent solution that works both on Linux, Windows and MacOS, which doesn’t break the strongbox-distribution*tar.gz (I believe it’s a bug in the maven-assembly-plugin.
Relates to #1007, #1008, #1068.
Acceptance Test

[ ] The tar.gz file produced by the strongbox-distribution contains proper file names which aren't truncated. for example, the strongbox-1.0-SNAPSHOT/lib/ should contain strongbox-web-core-1.0-SNAPSHOT-spring-boot.jar, not a truncated file name like strongbox-web-core-1.0-SNAPSHOT-spring).
[ ] Building the code and running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[ ] Building the code and running the strongbox-distribution from a zip and tar.gz still works.
[ ] Running the strongbox-web-integration-tests still run properly.

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@fuss86

I reproduced this problem.  On MacOS, users with very large UIDs / GIDs are not able to create the tar archives, the mvn install failing with the message reported above.  Users with smaller UID / GID are able to build successfully.
I discovered much more about the issue where filenames were truncated.  As reported above, configuring the maven-assembly-plugin to use POSIX tar file format extensions allows large UID / GID to be recorded in the tar file.  The file truncation reported above occurred when the tar file was opened in mc on Linux.  The problem is actually a manifestation of this issue in mc:
https://midnight-commander.org/ticket/1952
Midnight Commander does not support POSIX tar file extensions, resulting in the files being displayed with truncated names.  We would generally prefer to keep the tar file extensions using the GNU format, because it's so widely supported.  The only option to fix the UID / GID issue would be to specify a specific UID / GID when creating the tar archive.  The GNU tar command line utility allows us to do this using the --owner and --group options.  However, this is not supported by Plexus Archiver, which is used by maven assembly plugin to create the tar archives.  This was previously reported under the following issue, but I wasn't able to find an issue logged against Plexus Archiver:
https://issues.apache.org/jira/browse/MASSEMBLY-772
This leaves us needing to choose between having to build with a sufficiently small UID / GID user, and distributing tar files that are not supported by mc.  I suggest we close this issue, with the workaround that MacOS users would need to either create a local user (large UID / GID are typical artifacts of a Mac joined to a Windows domain or some other managed account service) with a small UID / GID to build, or build using a Linux VM.

Thanks for the detailed investigation and explanations! Closing as advised!


Create binarytree_to_doublyLinkedList.cpp

Why:

Closes #ISSUE_NUMBER
What's being changed:
Please merge this and give hacktoberfest accepted label to make it successful PR

Hey @iamvpbhaskar
Sorry, but we are not accepting dsa contributions right now, we are setting up the repo for dsa 🕦
Till then, you can contribute to any good first issue in this repo 😃

Note Don't forget to comment !assign on the issue you want to contribute to, to be auto assigned
!assign


Baseline modeling of annotations to allow data validation

Unfinished. What's left:

[ ] Validate on serialization and handle errors appropriate
[ ] Form-level validation in Annotations UI
[ ] Testing

@degrammer: please take a look. I'd like us to use zod for any data interaction with the user. This PR is a prototypical implementation. It might be wise to separate the getAnnotations function from the validation and only use the latter in the web-component that exposes the annotations form. Please make recommendations how you'd like to complete this implementation.

Any chance you could rebase this against main please @degrammer? I fixed some issues with cell associations when editing annotations. Just to be sure I'm testing from a known good state.

Any chance you could rebase this against main please @degrammer? I fixed some issues with cell associations when editing annotations. Just to be sure I'm testing from a known good state.

Done, rebase against main

I believe my initial schema had the now bools modeled as strings because that's what the parser expects:

@degrammer does serialization/deserialization work for you? it does not work for me

I believe my initial schema had the now bools modeled as strings because that's what the parser expects:

@degrammer does serialization/deserialization work for you? it does not work for me
There's also a chance that the code converted the bools to string onChange to conform with the parser. Either way, that seems to currently not work unless I'm overlooking something.

It works for me, probably I don't have some settings you have, let's discuss via Zoom

Fixed the issue:

Worked. Good to merge @degrammer 👍 . Let's do a edge/pre-release on it right after so we can dogfood it for a few days.


When i use docker to run a java application, it runed normally, but a fatal error in JVM happended suddenly.

When i use docker to run a java application, it runed normally, but a fatal error in JVM happended suddenly.
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007ff7c2d44c31, pid=1, tid=16
#
# JRE version: OpenJDK Runtime Environment (11.0.7+10) (build 11.0.7+10)
# Java VM: OpenJDK 64-Bit Server VM (11.0.7+10, mixed mode, tiered, compressed oops, g1 gc, linux-amd64)
# Problematic frame:
# V  [libjvm.so+0xab7c31]  PhaseIdealLoop::build_loop_late_post(Node*)+0x171
#
# Core dump will be written. Default location: Core dumps may be processed with "/usr/share/apport/apport %p %s %c %d %P" (or dumping to /usr/app/core.1)
#
# An error report file with more information is saved as:
# /usr/app/hs_err_pid1.log
#
# Compiler replay data is saved as:
# /usr/app/replay_pid1.log
#
# If you would like to submit a bug report, please visit:
#   https://github.com/AdoptOpenJDK/openjdk-support/issues
#

[error occurred during error reporting (), id 0xb, SIGSEGV (0xb) at pc=0x00007ff7c375f8f0]

Please run 11.0.12+7 and try again.


file farsing failure with quoted field names

I'm fairly new to PySD and Vensim. My model failed to load when the filed names were quoted.
A Vensim .mdl file created with Vensim 7.3.5 failed to parse and threw the following error
Rule 'rules' matched in its entirety, but it didn't consume all the text. The non-matching portion of the text begins with ...
However, after manually inspecting the mdl file and removing all quoted field names and removing special characters in the names the file was parsed successfully.
e.g.
"Av. purchase costs EV"= 	((70000-Subsidies)*(1-(TREND(EV Fleet, 1 , 0 ))/3)) 	~ 	~		|
Was changed to
Av purchase costs EV= 	((70000-Subsidies)*(1-(TREND(EV Fleet, 1 , 0 ))/3)) 	~ 	~		|
I hope this is helpful. It took a while for me to work this out but hopefully, it can be fixed relatively easily.
Note: This is a great project, keep up great work. This is is my first issue post. Please let me know if I could do a better job of writing it.

Thanks for this post. It was very helpful.

This is alreqady solved in the last version

This is alreqady solved in the last version


Fether shouldn't download Parity if Parity was not found on the machine but is still running

We should try isParityRunning before getParityPath

There's one problem: how to get the signer token?

Considering Fether is meant for general use, I think this issue is trivial, but I think it would be best, if Parity is running but we cannot locate the executable, to ask the user to run parity signer new-token manually.
Downloading Parity just to run parity signer new-token wouldn't make much sense since it would most likely not share the same config folders as the running instance of parity (typically in tunnelling use cases)


"portable" sensor reading

Provide example application(s) showing off our high level sensor API.

Should "and it is (securely of course) networked" something to load into this milestone item, or should I track it separately?

Should "and it is (securely of course) networked" something to load into this milestone item, or should I track it separately?

doesn't hurt to track seperately!


Don't silently skip missing files on download

We are currently silently skipping missing or unreadable files on download. As @kofalt points out, this is fundamentally opposed to our data management mission and must be corrected asap.
However, considerable edge cases exist. For example, how do we handle files being legitimately deleted, in parallel to a long-running download operation?
Also, we don't want to inform the user of a (potentially irrelevant) problem 3 hours into a large download and proceed to abort and purge all downloaded data.
We used to check for files during the download preflight, but that was deemed to slow. Let's again explore our options here. This pattern would allow us to mark files as being part of a download and delay physical removal until the download completes.
@kofalt, @ryansanford: Happy to hear your thoughts.
@rentzso: Let's take this on together next week.

:+1:
So, at the point of download, the browser is fully in control and the user interface, for all intents and purposes, has ceased to exist. I will be pretty adamant that at that stage in the game the only thing we should do is abort on any error.
Before that, are options are wider. Since we currently have this ticket acquisition round trip, I would support making an existence check part of the ticket return, and send back detailed error information on failure. This would allow an interface to alert the user (modal perhaps?) and offer the option to download only the existent data. Choosing that would request a new ticket with fewer files, repeating the process.
For more extensive fiddling, we veer into what basically amounts to disaster recovery, and I'm not currently convinced we need to spend much time building an API for that use case, because at that point somebody should be up to their elbows in the system in question and hands-on diagnosing the issue.

Joining this late, so maybe I'm missing something. But in cases of very large downloads, I've seen systems that allow a user to select what they want to download and submit the task to build their download. Then, when the download package is ready, they are notified (e.g., via email) and provided a link to initiate the download. Would that be a viable strategy for downloads of a size that can not be quickly pre-verified?

It definitely could be, but I hope that it won't be necessary in the vast-majority case. I'd be interested to see what (if anything) is slow in an existence check, because that should be very fast.
One pattern I've seen often is where a user interface will show a spinner immediately, then if a process hasn't completed in some short interval it adds additional context - "preparing your download", etc. Something like that should satisfy the 99% use case, I imagine. I would rather not have someone request a tarball of, say, a million files, unless there were no other options; a fuse mount or CLI would probably be a better approach.

I'm actually pretty happy with our on-the-fly tarball generation. It's a nice user experience and also avoids duplication overhead and cleanup. Ultimately, we want to build a system that largely obviates the need for raw data downloads by providing online processing capabilities.
@rentzso did the timing analysis when he added the ability to filter download packages by filetype and other parameters. I don't remember the numbers, though. Conceptually, every time the user makes or changes their selection, a new round trip and file check results. And hitting the filesystem to check for physical file existence is, of course, orders of magnitude slower than simply querying the database.
I could imagine delaying the filesystem check until the last possible moment, that is, the user hitting the download button. And at that time, after confirming the existence of all requested files, we either start streaming or go into an error dialog.
@kofalt: Don't forget that you only have a toy-dataset with grossly truncated scans. The real use-case is multi-gig downloads, containing some hundreds of files. We recently had a request for the entire Simons dataset, which is several hundreds of gigabytes. Clearly indicating to the user what and how much they are about to request is key.

I agree that on-the-fly makes for a nice user experience. Apart from the disaster scenarios mentioned (which I agree aren't worth a lot of UI work, since they will necessitate manual intervention), what else could cause 'corrupt' downloads? Some data are deleted mid-stream? A computation is re-run, replacing some results files? Metadata is altered? Permissions are changed? If all these are expected to be rare edge cases, then I think it's fine to abort the download with an error. It may piss off a user on the rare occasion that it happens, but it's a worthy price to pay for assuring data integrity.
And I agree with @kofalt-- we absolutely need a cli method for large data transfers. Even for small transfers, this is much more convenient for the (very common) use-case of a user wanting the data to end up on a remote compute server.

@gsfr

The real use-case is multi-gig downloads, containing some hundreds of files.

I don't think the size (in bytes) of an average download changes my position - I'm only proposing an existence check, which is N stat calls. A few thousand should definitely take less than a second. To be clear I'm not proposing we verify their integrity or contents...

Clearly indicating to the user what and how much they are about to request is key.

That's a good idea. Maybe for bulk downloads we use the ticket-round-trip information I proposed to populate that data. "You are about to download 32 sessions, which is 500 files and ~5 GB."

I could imagine delaying the filesystem check until the last possible moment

I assert the last possible moment is the ticket request (ref my first comment upthread). The actual download itself is too late - the UI no longer exists nor has any introspection into that request.
@rfdougherty

what else could cause 'corrupt' downloads?

Good question: I think only an incomplete or malfunctioning filesystem. Files are stored in a CAS - a custom, in-house CAS that needs some TLC, but still a CAS - and we punt on corruption as the filesystem + sysadmin's problem, currently. I would expect any problems whatsoever to be an extreme edge case.


[Feature Request/Suggestion] Depth Map Export

I realize this may be a huge ask and not feasible, but since you're digging-around like this figured I'd ask anyway.
There is great potential in using a depth map to get more control in post-processing, specifically in the application of depth-of-field effects (since Cyberpunk 2077's built-in one is pretty terrible in quality, and has truly horrifying interactions with things like hair).

You can use reshade for that :) With the display depth shader you can visualize the depth buffer and e.g. take a screenshot.

You can use reshade for that :) With the display depth shader you can visualize the depth buffer and e.g. take a screenshot.


Add separator for Dropdown Blade components

Description
This pull request introduces a new separator component designed for use within Filament's Blade Dropdown component. This component provides a visual distinction between groups of options or individual items, enhancing the organizational structure and readability of dropdown menus.

Code style
Simply add <x-filament::dropdown-list.separator /> to add the separator.

[x] composer cs command has been run.

Testing

[x] Changes have been tested.

Documentation

[x] Documentation is up-to-date.

<x-filament::dropdown.list> exists for this purpose.
Grouping items separates groups with a visual divider.
This ensures semantic grouping and allows for alternative themes that e.g. group using additional whitespace instead of borders.


Add Windows Terminal on the Win+X menu

Summary of the new feature/enhancement

Like #1060 but

just starting windows terminal. (not starting from the directory I am in.)
add at windows start button right-click context menu, not explorer right-click context menu.
(Seems to be likely under Windows Powershell or replace Windows Powershell button.)

Sorry that the language is not english, because of windows bug, I couldn't change language at this build of windows.

off: inability to change system display language is not a bug for CoreSingleLanguage system SKU.

@arseny92 I just can't get to language setting menu by the bug.

I agree. I also would like to see an option to have Terminal in the Power User menu.

There's a magic folder that contains shortcuts that show up in the Win+X menu, actually. It's somewhere in %APPDATA%\Microsoft!

We're tackling this in a different way; see #1466

There's a magic folder that contains shortcuts that show up in the Win+X menu, actually. It's somewhere in %APPDATA%\Microsoft!

Just %APPDATA% is for Appdata\Roaming
The folder is at %LOCALAPPDATA%\Microsoft\Windows\WinX (Not recommended to touch default groups, to not mess menu order). Just do a Group4 folder, put in a shortcut to shell:AppsFolder\Microsoft.WindowsTerminal_8wekyb3d8bbwe!App
CMD/Powershell Taskbar Settings toggle sets dword at HKCU\SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\Advanced , DontUsePowerShellOnWinX and is pretty much not useful for this case

There's a magic folder that contains shortcuts that show up in the Win+X menu, actually. It's somewhere in %APPDATA%\Microsoft!

Just %APPDATA% is for Appdata\Roaming
The folder is at %LOCALAPPDATA%\Microsoft\Windows\WinX (Not recommended to touch default groups, to not mess menu order). Just do a Group4 folder, put in a shortcut to shell:AppsFolder\Microsoft.WindowsTerminal_8wekyb3d8bbwe!App
CMD/Powershell Taskbar Settings toggle sets dword at HKCU\SOFTWARE\Microsoft\Windows\CurrentVersion\Explorer\Advanced , DontUsePowerShellOnWinX and is pretty much not useful for this case. If only we could override that...

There is a tool for this but beware the site is full of ads and lingering links
https://winaero.com/comment.php?comment.news.30
Download: https://winaero.com/request.php?21

For an alternative idea, check out  #1594 - which is about keeping the existing items in Win+X, but making them open using a preferred terminal app, that could be set to Windows Terminal (much like we set preferred web browser or preferred email client).

Or possibly #492, which is what I think @DHowett-MSFT meant

The folder is at %LOCALAPPDATA%\Microsoft\Windows\WinX (Not recommended to touch default groups, to not mess menu order). Just do a Group4 folder, put in a shortcut to shell:AppsFolder\Microsoft.WindowsTerminal_8wekyb3d8bbwe!App

I tried doing the above, because that sounded awesome.  Created a Group4 folder, and attempted to add some shortcuts.  Noticed a couple of things straightaway:

Attempting to paste the shell:AppsFolder link into a shortcut (using right-click | New | Shortcut) results in the message that the file cannot be found.  That was on my home PC.  On my corporate PC, I think it said something to the effect that I couldn't use that file type or some such.  Both PC's are running Windows 10 1909.
So, I went to shell:AppsFolder via Win-R, found terminal, right-clicked and saved a shortcut to the desktop, then copied that in.
Added a shortcut to Notepad for good measure.  Just to see if something would show up.
Rebooted.  Logged back in.  Hit Win-X.  Absolutely no change.  Nothing shows up from "Group4".  Does this no longer work, or some such?

I am reluctant to try the WinAero thing, but I guess I can give that a go.

all suggestions around Win+X or default apps etc should be redirected to https://github.com/microsoft/terminal/issues/492


Drawing: Misaligned content of a node if there is node has no inputs

Issue:
If a node has no inputs, then the content of the body has a weird off-set before it is drawn. ( see node 2 on image below )
Expected:
To be aligned to the left side of the node
Reproduction steps:
struct SomeEntryNodeInputs : public ImFlow::BaseNode
{
    explicit SomeEntryNodeInputs()
    {
        setTitle( "Some entry node" );
        addOUT<bool>( "Body" )->behaviour( [ this ]()
        {
            return true;
        } );

        addOUT<bool>( "Left hand" )->behaviour( [ this ]()
        {
            return true;
        } );
        addOUT<bool>( "Right hand" )->behaviour( [ this ]()
        {
            return true;
        } );

        addIN<bool>( "In", false, ImFlow::ConnectionFilter::SameType() );
    }

    void draw() override
    {
        Text( "Awesome description of a node that is awesome" );
    }
};

struct SomeEntryNodeNoInputs : public ImFlow::BaseNode
{
    explicit SomeEntryNodeNoInputs()
    {
        setTitle( "Some entry node no inputs" );
        addOUT<bool>( "Body" )->behaviour( [ this ]()
        {
            return true;
        } );
        addOUT<bool>( "Left hand" )->behaviour( [ this ]()
        {
            return true;
        } );
        addOUT<bool>( "Right hand" )->behaviour( [ this ]()
        {
            return true;
        } );
    }
    void draw() override
    {
        Text( "Awesome description of a node that is awesome" );
    }
};

I believe the issue is caused by the call to the SameLine() function in ImNodeFlow.cpp
        // Inputs
        ImGui::BeginGroup();
        for (auto &p: m_ins) {
            p->setPos(ImGui::GetCursorPos());
            p->update();
        }
        for (auto &p: m_dynamicIns) {
            if (p.first == 1) {
                p.second->setPos(ImGui::GetCursorPos());
                p.second->update();
                p.first = 0;
            }
        }
        ImGui::EndGroup();
        ImGui::SameLine(); <- Offender

I'm down to open PR for this, but what would be the preferred way to go about this, not calling SameLine() if nothing was drawn or simply skipping the group drawing if there is no ins and dins?
edit: With same line not being called it behaves as expected:

I'd look into skipping the entire group, make sure that it still works as intended and if so you are welcome to open a PR

Alright, i'll implement it on my end since i am gonna be playing around with this quite a bit and if everything is okay i'll open a PR


Video generator

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
India
Provide a description that is worded well enough to be understood
Video generator is not working
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No

Complete Verbose Output
No

Yes, because I have no idea what you're talking about


Raw string's scope error

Environment data
Version: 1.68.1 (Universal)
Commit: 30d9c6cd9483b2cc586687151bcbcd635f373630
Date: 2022-06-14T12:52:13.188Z
Electron: 17.4.7
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 21.4.0
Code Snippet

import re

from cv2 import imread

img = imread(r"paper\cell_b.png")

print(r'paper\cell_b.png' == 'paper\\cell_b.png')

r'paper\cell_b.png' == 'paper\\cell_b.png' is evaluated as True in Python.
Raw string has "r" before it, and it means string without backslash escape in Python. It's especially useful for handling paths in Windows.
the origin issue: https://github.com/Binaryify/OneDark-Pro/issues/749

@Binaryify and @erictraut I am not a frequent Python user, so please correct me if I've misunderstood something, but I think this is a feature: https://github.com/MagicStack/MagicPython/issues/114#issuecomment-349115524 (that's the repo that VS Code pulls our Python syntax highlighting grammar from.

@Haoyu-UT

@alexr00 Thank you for your comment. I now come to understand that for vscode (and MagicPython) r and R are interpreted differently. I'm sorry for my ignorance.
@Binaryify Thank you for your support. Please kindly close the original issue.


FEAT: Force conference region

Summary
This feature provides adds the ability to set the conference region which is used by Flex when a call task is accepted to establish a conference. By default the conference is established based on the incoming call Region/Edge which is generally the lowest latency to the caller.
Checklist

[X] Tested changes end to end
[X] Updated documentation
[X] Requested one or more reviewers

@dremin Would you be so kind as to review

@dremin RTM?


How to load a local npy file using npyjs in React?

Loading a 2D array does not seem to work despite the code compiling. In the console I get the error -
Uncaught (in promise) SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data

The code that I have is -
import ndarray from "ndarray";
import npyjs from "npyjs";

const NP = function(){

let n = new npyjs();

n.load("./embeddings.npy").then(res => {
    // res has { data, shape, dtype } members.
    const npyArray = ndarray(res.data, res.shape);
    console.log(npyArray);
});
}
export default NP;

If I replace the file path with https://rawcdn.githack.com/aplbrain/npyjs/ba60a3a529f3210dd07d2ed05ab628939e18b6a7/test/data/4x4x4x4x4-float32.npy then it seems to work...is there a way to load from local file paths?

I think I got it working using -
import embeddings from "./embeddings.npy";
``

Awesome, glad you got it working, and thank you for sharing your solution! Please let me know if you run into any other trouble :)

@j6k4m8 I am actually trying to use your library to read npy files and use the extracted data to render scatter-plots using regl (react gl scatterplot), but yeah, I think I am stuck at the last step...been at it for two days...I think its just a conversion/type issue...but being new to web dev, it is taking longer than usual. So far I have made the progress as here - https://github.com/flekschas/regl-scatterplot/issues/83

@j6k4m8 I was wondering, if it is possible to preserve the "topology" of the numpy array when it is read by npyjs/ndarray?
What I mean is, right now when I load a 2D matrix, npyjs reads it as a 1D array. For e.g: 5x2 matrix is read as a 1x10 array.
What I want is to keep it as a 5x2 array, do you think that's possible?

It is definitely possible!
I made a deliberate design decision on this project to NOT make any assumptions about how the user wants the data stored; there is no widely-accepted way of storing tensors in JS, unlike the widely used numpy python library.
People frequently use this npyjs library in conjunction with ndarray, so you could do the following:
const ndarray = require("ndarray");

...
let myTensor = ndarray(new Float64Array(res.data), res.shape)

But some people also use this library with TensorFlow.js, in which case you'd do something like this:
const myTensor = tf.tensor(res.data, res.shape);

By staying agnostic to the framework you are using, we are trying to stay as flexible as possible to your use-case. But I agree that it adds a little extra work :)
If you absolutely must not import another library, you could do something like this:
function indexIntoArray(array, shape, ndShapedIndex) {
    let linearIndex = 0;

    for (let i = 0; i < shape.length; i++) {
        linearIndex += ndShapedIndex[i] * shape[i];
    }

    return array[linearIndex];
}

And then access the $M_{i,j}$th element of your tensor like this:
indexIntoArray(res.data, res.shape, [i, j])

For example, to get the data in row 4 and column 6, you could do:
indexIntoArray(res.data, res.shape, [4, 6])

Alternatively, you could convert the res.data into a nested list of lists... But I do not recommend this, if you can avoid it!

@j6k4m8  Talking about -
const ndarray = require("ndarray");

...
let myTensor = ndarray(new Float64Array(res.data), res.shape)

I think its still the same as what I had described earlier right? myTensor here would have data and shape properties where data would again be 1D...this exactly is the issue for me not being able to use the array to pass to scatterplot.
I think I am missing something very silly...forgive me for taking so much of your time...but let me show this again -
According to your documentation , I do -
import embeddings from "./2DEmbeddings.npy";

let e = new npyjs();
e.load(embeddings).then(res => {
                console.log(`You loaded an array with ${res.data.length} elements and ${res.shape.length} dimensions.`);
                console.log(res.data); // this is 1D...when I wish it was same as numpy file, i.e 2 or 3D
                console.log(ndarray(new Float64Array(res.data), res.shape)); // this is same as above - i.e 1D
                // return res.data;
            });

I think the load function itself transforms the matrix into 2D...that is the problem.
I don't mind using Tensorflow.js if it helps me...haven't tried the following solutions...I am on it now...hopefully something works.

I don't know the ndarray package particularly well, but I believe that it has n-dimensional indexing and slicing operators; what do you need it to do?
From your issue on flekschas/regl-scatterplot#83, it looks like you need a nested list-of-lists where each point is a list of [x, y, color]. The easiest way to do this is:
function convertToXYC(data1d) {
    let points = [];

    for (let i = 0; i < data1d.length; i += 3) {
        points.push([data1d[i], data1d[i + 1], data1d[i + 2]]);
    }
    return points;
}

...if your data are in row-first order in the npy file, or
function convertToXYCRows(data1d) {
    // The transpose of the above function.
    let points = [];
    let lengthOfRow = data1d.length / 3;
    for (let i = 0; i < lengthOfRow; i++) {
        points.push([data1d[i], data1d[i + lengthOfRow], data1d[i + 2 * lengthOfRow]]);
    }
    return points
}

...if your data are in column-first order in the npy file.
(If your dataset is a set of x and y without color values, then replace the 3's with 2's and remove the last element of each push line.)


Depends on "github.com/RangelReale/osin"

Why does this depend on "github.com/RangelReale/osin"?
ref: https://github.com/openshift/osin/search?q=RangelReale
https://github.com/RangelReale/osin has been archived. This means that it will no longer be maintained. On the other hand, I think this repository will be maintained. For this reason, I would like to use "github.com/openshift/osin" instead of "github.com/RangelReale/osin".
However, "github.com/openshift/osin" imports "github.com/RangelReale/osin", so even if you replace "github.com/openshift/osin" with "github.com/RangelReale/osin" Cannot be removed.

/close
fixed in https://github.com/openshift/osin/pull/200

/close
fixed in https://github.com/openshift/osin/pull/200


Public API to change video source

Is your feature request related to a problem? Please describe.
In version 2 it was possible to control the currently selected video source for changing quality. It could be used by page application to switch into lower QT when browser connectivity changed. Or mounting other elements on the webpage to control quality options.
Describe the solution you'd like

List available video sources player.getSources();
Check currently played video source player.getActiveSource(); // => [{isHD: true, url: "http://.."}]
Switch source (should switch only if a new source is part of available source) player.setActiveSource(source);
An event called when the source is changed (currently there is no way to show HD/nonHD icon) player.on('sourceChange', ...);

Describe alternatives you've considered
After hiding private variables of the player there is no way to control sources.

@skallet - Can you please confirm if the issues persist or is resolved

El mar., 30 de noviembre de 2021 7:19 a. m., HariPriyaKuberSingh <
@.***> escribió:

@skallet https://github.com/skallet - Can you please confirm if the
issues persist or is resolved
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fluid-player/fluid-player/issues/495#issuecomment-982628570,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWJU5B4D566NDJC2ZPYNI4LUOTFNBANCNFSM4OWVDHSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@skallet - Can you please confirm if the issues persist or is resolved

Moving the ticket to closed since there is no input from the reporter .


add f1_samples metric

This metric is useful for multilabel classification

Will be added in future versions as a kwarg option for metrics


fix all unhandled errors

handle type revision on equity/fund/etf to check if input value is instance of str (just works on Python 3.x)
handle start and end date check to make sure that end date is earlier than start date
handle input dates on funds & etfs since data is just available from 2010 or 2011 until now
handle IndexError raise when interval scraping has not finished via flags

fixed on commit: 920d127


Update for luarocks 3.0.0 release

Updated on master branch but something is broken on Windows with executable scripts installed by LR, needs investigation.


Add option to only set target properties if value is different than the current value.

Rel. https://github.com/riok/mapperly/discussions/269
Add an option to the Mapper attribute whether to only map a property if the value to be set is different then the current value on the target. This probably should also be configurable on a per property basis. API tbd.

Add an option to overwrite the comparer in use. The API surface could be implemented similarly as for IFormatProviders (just for IComparer<T>). The compared value should always be the one after the conversion.

Hi @latonz
If it's not too much trouble, could you please tell me where in the code it would be best to add this check?
I looked at the code but couldn't find the right place.
Also, how should I handle IQuerable. I'm sure the SQL translation won't be able to handle IEqualityComparer calls correctly

I believe this is quite a significant undertaking. Upon a quick review, I think it could be implemented in certain parts of ObjectMemberMappingBodyBuilder.BuildMemberAssignmentMapping, MemberMappingBuilder, and MembersContainerBuilderContext. Additionally, it likely requires the creation of new Mapping* classes.
Regarding the queryable projections, I believe some EF Core providers can handle certain well-known equality comparers (such as StringComparer.OrdinalIgnoreCase). In the initial iteration, we can probably disregard them for queryable projection mappings.

I know for sure that the Postgres provider doesn't support StringComparision. I asked about support in their repository a while ago, and they said that it's possible to use EF.Functions.ILike for Postgres specifically, but generally the correct approach is to use collations. I don't know about other adapters, maybe it's supported for no-sql DBs like Mongo.


Add manila-csi-plugin to make build

What this PR does / why we need it:
manila-csi-plugin is missing from make build, similar to #655.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

/ok-to-test

/lgtm

/approve


refactor(cicd): change publishConfig to use npm

Current dependencies on/for this PR:

next

PR #8   👈

This comment was auto-generated by Graphite.


Java 8: UnsupportedClassVersionError: ZeebeSpringTest

Running a test with Java 11 shows the following error:
UnsupportedClassVersionError: ZeebeSpringTest has been compiled by a more recent version of the Java Runtime (class file version 61.0), this version of the Java Runtime only recognizes class file versions up to 55.0

Okay, I had to use another dependency for Java 8:
		<dependency>
			<groupId>io.camunda</groupId>
			<artifactId>spring-zeebe-test-testcontainer</artifactId>
			<version>${spring-zeebe.version}</version>
			<scope>test</scope>
		</dependency>

Jepp. I wanted to add a note in the docs - but it is there already :-)
https://github.com/camunda-community-hub/spring-zeebe#writing-test-cases


Usage of deprecated mimetype parameter crashes Django 1.7+

On the support/3.0.x branch, mimetype should be content_type for Django 1.6+. On develop where we don't need support for Django 1.4/1.5, we don't need conditional logic at all.
cms/utils/admin.py:        return render_to_response(template, context, mimetype="text/html; charset=utf-8")
cms/admin/settingsadmin.py:            return HttpResponse(json.dumps(""), mimetype="application/json")
cms/admin/settingsadmin.py:        return HttpResponse(json.dumps(request.session.get('cms_settings', '')), mimetype="application/json")

Apparently tests for these code paths also need to be added. Are the code paths that use content_type unconditionally also untested as it seems that should fail on Django 1.4/1.5? (e.g. a1650839)

@timgraham shouldn't it be content_type for Django 1.5+ (https://docs.djangoproject.com/en/1.5/topics/http/shortcuts/#django.shortcuts.render_to_response)?
I don't think we have any test specific for these functions
Cleanup on develop will happen after 3.0.8 relese

Correct, sorry I got the version wrong on that.

No worries :)
Fixing the 1.4 issue (I bet we don't have much users on 1.4, though)
I'll leave this open anyway for 3.1 to help writing more tests for such functions


[Do not merge] Empty commit for testing

For testing a regex literal printer SPI.

Please test with following PR:
https://github.com/apple/swift-experimental-string-processing/pull/678
@swift-ci Please build toolchain macOS Platform

Please test with following PR:
https://github.com/apple/swift-experimental-string-processing/pull/678
@swift-ci Please build toolchain macOS platform

Please test with following PR:
https://github.com/apple/swift-experimental-string-processing/pull/678
@swift-ci Please build toolchain macOS platform

Please test with following PR:
https://github.com/apple/swift-experimental-string-processing/pull/678
@swift-ci Please build toolchain macOS platform


Max Visible being used to determine if onAdapterAboutToEmpty is called

In SwipeFlingAdapterView.java line 116 the line of code is:
if(adapterCount < MAX_VISIBLE) mFlingListener.onAdapterAboutToEmpty(adapterCount);
Shouldn't it be:
if(adapterCount < MIN_ADAPTER_STACK ) mFlingListener.onAdapterAboutToEmpty(adapterCount);
It looks like the MAX_VISIBLE variable is being used to determine if onAdapterAboutToEmpty is called instead of MIN_ADAPTER_STACK.

@dwang999  yes, you are right, Please see #22


feat: added loader, layout, error

added vitest ui stuff
removed non-needed semantic-release npm's
removed the rest of jest stuff

Closes #47  #48

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: call IsSwimming() vs InLiquidSwimDepth()

InLiquidSwimDepth() appears to have been removed from Player or made private on Character.

fixes #12


Separate element definition and parsing: Image

Rename ImageElement to ImageParser
Extract general stuff into element class that is supposed to be re-used by other import / export modules.
Also removed the original image size, which was only used for debugging (therefore the parsed reference doc changed).

+1

+1


Cells shrink on screen rotation

Table view cell gets distorted when changing orientation from portrait to landscape

Still not resolved

Hi @diptikas, @anad7
This problem has been fixed. With this implementation, TableView retains and restore the scroll positions and selections.

However, I haven't released the related version yet. Because other preferences of TableView like hidden row/column list etc. should be saved as well.
See you

Thanks!! So when are you planning to release?

Probably, this weekend.

0.8.7 version has been released. you can test your case using the newest version.
Let me know the result.

Hi,
I checked your new version and as per my usecase when I don't have rowheaderRecyclerview or when it is null , app is crashing because of null pointer onOrientationChanges in ScrollHandler.java class inside
getRowPositionOffset() method when you are fetching child.getLeft .

Hi @diptikas,
When the configuration is changing, isn't your tableView visible?

Hi,
My tableview is visible but I am doing this removeView(m_jRowHeaderRecyclerView); . So My mRowHeaderRecyclerView will be null.


Support not only OpenShift

Would you be up for adding/changing/supporting a non-OpenShift-only helm chart?
As it is today, I guess it's only the SecurityContextConstraints resource that isn't portable to vanilla Kubernetes (?), i.e:

https://github.com/zscaler/zpa-openshift/blob/main/templates/securitycontextconstraints.yaml
kind: SecurityContextConstraints
apiVersion: security.openshift.io/v1

Hi @MPV,
My thoughts exactly, that is why I added https://github.com/zscaler/zpa-openshift/pull/1 to make the flag .Values.zsglobal.securityContextConstraints.create work. I've installed it on a local vanilla Kube cluster and for me it seems to work.

Now that the changes coming in #1 are on the way, as it will thus support not-only OpenShift, could this repo also be renamed (for discoverability) to reflect that the helm chart could work with vanilla Kubernetes as well?
Also considering the documentation on the Docker page saying Kubernetes isn't supported:

"Docker support is not available for Kubernetes."
https://help.zscaler.com/zpa/app-connector-deployment-guide-docker

Would you also say/recommend that the zsglobal.openshiftEnvironment.value value should be set to some (which?) other value than enabled?
https://github.com/zscaler/zpa-openshift/blob/5b08eb732045948e711ac3ad9f2b253be80af64a/values.yaml#L144-L147
https://github.com/zscaler/zpa-openshift/blob/5b08eb732045948e711ac3ad9f2b253be80af64a/templates/deployment.yaml#L49-L50


Add data connector example

I think we should not commit this bin folder

Removed, good callout

Nit: Do you like this file name being camel case?

Renamed


Fix returning null or undefined from ExtendScript with evalTS

Reproduction Steps:
Create an ExtendScript function which returns null or undefined, and call it using evalTS
Expected:
evalTS should resolve its promise with the value of null or undefined
Actual:
evalTS rejects its promise with the value of null or undefined, because the line
if (parsed.name === "ReferenceError") {

... attempts to evaluate the .name property or null or undefined, which raises a TypeError. This error is caught by the surrounding try/catch, which then rejects the promise with the null or undefined value.
Fix: Use safe navigation operator to access parsed.name so that a TypeError is not raised

An alternative fix would be to add a line
if (res === "null") return resolve(null);


Just says dotnet isnt found..

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
when i type in dotnet new console in visual studio code i get this error message
It was not possible to find any installed .NET Core SDKs
Did you mean to run .NET Core SDK commands? Install a .NET Core SDK from:
https://aka.ms/dotnet-download
but i downloaded .net

Can you paste the results of running dotnet --info?


Add getPayloadBodies and getPayloadBodiesByRangeV1 Methods

https://github.com/ethereum/execution-apis/pull/218
https://github.com/ethereum/execution-apis/pull/146

These linked specs have been obviated by this latest revision:  https://github.com/ethereum/execution-apis/pull/352/files
The corresponding pr in #4980 will need to be updated to take this new version into consideration.


Failed to resolve: com.github.ozodrukh:CircuralReveal:1.3.1

Seems like you need to update version of CircuralReveal

Hi, I had missed one instruction for integration.
Please check wiki page again where you might need to update your project level gradle file.
Try and tell me if it works or not :)


Update AboutUs information for Kaiwei

Update Kaiwei's information for AboutUs

Codecov Report

Merging #25 (394d83b) into master (482eb68) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #25   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 482eb68...394d83b. Read the comment docs.


Inspect appears to do nothing

The Inspect button seems to have stopped doing anything.
@Powell-v2 Do you know what's going on here?

This was definitely working when I filed #1425 on September 4, right after I merged the change to move the inspect area.  So it's something since then, meaning that I probably broke it.

That was my bad. By not importing anything from the codeExplorer module I cut off the reception of incoming messages.


bug: Enforce protobuf oneof constraints

Motivation
When protobufs have fields that are defined oneof, the compiler doesn't enforce that exactly one of the fields is set. We need to manually validate these
Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[X] PR title adheres to the conventional commits standard
[X] PR has a changeset
[X] PR has been tagged with a change label(s) (i.e. documentation, feature, bugfix, or chore)
[ ] PR includes documentation if necessary.
[X] All commits have been signed

PR-Codex overview
The focus of this PR is to enforce protobuf oneof constraints in the validateMessage function and ensure only one body can be set in a message.
Detailed summary

Enforced protobuf oneof constraints in validateMessage
Added validation for setting only one body in a message

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Yeah, we'll have to run validateOrRevoke. I can probably make a list of affected fids and just run the job for them as a migration. Let me do that in a follow up PR


Command list should sorting commands by use frequency

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:18:32.269Z)
OS Version: Darwin x64 16.5.0
Extensions:

Extension
Author
Version

EditorConfig
EditorConfig
0.9.3

material-icon-theme
PKief
2.0.6

box-comment
canna
0.0.1

vscode-eslint
dbaeumer
1.2.11

python
donjayamanne
0.6.4

docthis
joelday
0.4.4

vscode-colorize
kamikillerto
0.5.1

vs-color-picker
lihui
0.6.0

debugger-for-chrome
msjsdiag
3.1.1

vetur
octref
0.6.10

advanced-new-file
patbenatar
0.4.3

The command list(default shortcut: Cmd+Shift+P) is now sorting by characters, I think it will be more convenience if VSCode could sorting by the frequency of use.
For example, I often expect when I enter "install", the first command of the list is "Extensions: Install Extensions", but now is "Extensions: Disable All Installed Extensions", that really weird...

https://github.com/Microsoft/vscode/issues/13080


"Client::AUTH_HTTP_TOKEN" authentication method is deprecated

INFO   | PHP    User Deprecated: Using the "Client::AUTH_HTTP_TOKEN" authentication method is deprecated in knp-labs/php-github-api v2.15 and will be removed in knp-labs/php-github-api v3.0. Use "Client::AUTH_ACCESS_TOKEN" instead.
what do you think if we upgrade the client version?

IMHO we can try to upgrade the client version.

it has been fixed! 🚀


Realtime-notification app

#3294 Real-Time-Notification
Close #3294
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

@TusharKesarwani I am done with my work please review it.


Can't play .opus or .webm from HTTP location when running lavalink in a container

Description
When using Lavalink from a linux container I cannot seem to play .webm or .opus files with opus audio from a HTTP url.
I do not get any audio from the bot, even though it seems to be playing.
Other formats (.mp3 and .flac) from the exact same location do seem to work.
I have tried containers on Docker using WSL2 and on Kubernetes (DOKS).
Log from the moment the song is queued:
2021-12-29 08:37:25.467  INFO 1 --- [  XNIO-1 task-5] l.server.player.AudioLoaderRestHandler   : Got request to load for identifier "https://xyz.com/xyz.opus"
2021-12-29 08:37:25.669  INFO 1 --- [   XNIO-1 I/O-2] lavalink.server.io.SocketServer          : {"op":"voiceUpdate","guildId":"xyz","sessionId":"xyz","event":{"token":"xyz","guild_id":xyz,"endpoint":"xyz.discord.media:443"}}
2021-12-29 08:37:25.700  INFO 1 --- [ader-2-thread-1] lavalink.server.player.AudioLoader       : Loaded track Unknown title
2021-12-29 08:37:25.736  INFO 1 --- [   XNIO-1 I/O-2] lavalink.server.io.SocketServer          : {"op":"play","guildId":"xyz","track":"xyzxyzxyzxyz","startTime":0,"noReplace":false}
2021-12-29 08:37:25.819  WARN 1 --- [ntLoopGroup-2-1] i.n.u.concurrent.AbstractEventExecutor   : A task raised an exception. Task: moe.kyokobot.koe.codec.udpqueue.UdpQueueOpusFramePoller$$Lambda$589/0x0000000801188040@665526a5

java.lang.ArrayIndexOutOfBoundsException: null

I do not see that warning every time though.
It does seem to work when I run Lavalink from Windows with zulu13, but then the sound gets rather choppy and I tons of the following errors like these:
2021-12-28 21:14:00.128 ERROR 57012 --- [tLoopGroup-2-20] m.k.k.codec.netty.NettyOpusFramePoller   : Sending frame failed

java.lang.ArrayIndexOutOfBoundsException: null

My config file is like the example: https://github.com/freyacodes/Lavalink/blob/master/LavalinkServer/application.yml.example
Version info
Client used: Lavalink4net
Output of java -version:
$ java -version
openjdk version "13.0.2" 2020-01-14
OpenJDK Runtime Environment Zulu13.29+9-CA (build 13.0.2+6-MTS)
OpenJDK 64-Bit Server VM Zulu13.29+9-CA (build 13.0.2+6-MTS, mixed mode, sharing)

Images I have tried it with:

fredboat/lavalink:master-v3.4
fredboat/lavalink:3deccd43 (latest that I could find)

Can you test with the 3.5-rc1 build and report back the results? It looks like your issue stems from Koe, the library used by Lavalink to establish connections and send audio to Discord. It was refactored a bit recently and the new version is used in this build so it may or may not solve your issue.

I have completely switched to using the mp3 format for my usage, so I'm afraid I don't have the means to test it anymore.
Perhaps someone else who had this problem could test it to verify, or this issue can just be closed.

As there doesn't appear to be anyone with the same/similar issue, I'm going to close this as a one-off issue, or fixed.


Example Poject template relies on dependencies

Hi,
The template example yaml to create a project won't work as a standalone example since it relies on an existing shared vpc host project.
Possible fixing:

either mentioning that a shared vpc named 'test-vpc-host-project' should exist first.
modify the yaml file to create an autonomous project with its own default network

Cood call. I am extending this Feature request to create 2 additional examples:

Standalone Project
VPC host Project

Additionally extending the example readme/comment to explain the dependencies.

Working on this.


chore: update extension json

Current dependencies on/for this PR:

l/refactor-p-image-container

PR #215

PR #224

PR #225   (this PR)

This comment was auto-generated by Graphite.


Revert "[SDPA-5217] ordered list spacing"

Reverts dpc-sdp/ripple#1000

This got merged by mistake in this 1.36.0 release. So just reverting the changes


Support netd in GCE bootstrap.

We should cherrypick this to 1.11, because netd support is added in Kubernetes 1.11. https://github.com/kubernetes/kubernetes/pull/64591
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

/cc @jingax10 @cadmuxe

/lgtm


UPdate for Google Drive

Hello, Can you update it for google Drive API  too  please?

Hello,
I'm not using Google Drive much, but when I have time I'll look into it! 👍
I'd also appreciate the pull requests if anyone wants to contribute 🎉

@hmrserver sure! Any guides, or would adding #hacktoberfest to repo description suffice?

I guess adding "hacktoberfest" label to this issue would be enough (like you added enhancement one).

@hmrserver Aha, thanks for the heads-up! I can see you've implemented the folder finding method on your script, I'll definitely look for this soon.
By the way, any reason you're downloading gdrive from Google Drive directly?

I had copied the link from the main gdrive repo (I guess): https://github.com/prasmussen/gdrive you can see it's readme.
Which I felt would be nice enough.

Check my implementation https://github.com/Ardakilic/backmeup/pull/6

This is implemented with PR #6 , thanks to @islander . Please see the master branch for this implementation.


Error when running Gemma inference on GPU

When I run
docker run -t --rm \
    --gpus all \
    -v ${CKPT_PATH}:/tmp/ckpt \
    ${DOCKER_URI} \
    python scripts/run.py \
    --device=cuda \
    --ckpt=/tmp/ckpt \
    --variant="${VARIANT}" \
    --prompt="${PROMPT}"

It returns the error:
docker: Error response from daemon: could not select device drit device driver "" with capabilities: [[gpu]].
while if I run on CPU with command:
docker run -t --rm \
    -v ${CKPT_PATH}:/tmp/ckpt \
    ${DOCKER_URI} \
    python scripts/run.py \
    --ckpt=/tmp/ckpt \
    --variant="${VARIANT}" \
    --prompt="${PROMPT}"

It works out OK.

What model variant did you use and what GPU did you use?
One guess is that you may run out of GPU memory if you try to run the 7B un-quantized model on a 16GB GPU. You can either try the 7B quantized model or a 2B model and it should work.

Hi @LarryHawkingYoung,
Could you please confirm if this issue is resolved for you with the above comment ? Please feel free to close the issue if it is resolved ?
Thank you.

Hi @LarryHawkingYoung,
Closing this issue due to lack of recent activity,  Please feel free reopen if this is still a valid request.
Thank you!


Add more logging on the server to see HTTP request coming in

It would be good to discuss what level of detail we would like to capture in the logging. There is still some work to do on the draft PR to set the logging path and frequency as an environment variable.


Better handling of local peer addresses

We currently have ignore_peers in the peer resource and some hardcoded fact names in the volume resource. We should use a common local_peer_addresses parameter for both.

:+1:


added module for monitoring transmission-daemon via rpc

note: this commit is dependant on transmissionrpc, documentation will
need to be updated to reflect installation, etc.
$ easy_install transmissionrpc

Ha, this is nice. There is already a pull request for this but I guess yours is fully working and might be best.

throw the transmissionrpc folder inside the external folder and it will work without installing transmissionrpc

Don't worry this should be done with the merge. The problem is that master is not yet independent so it does not have the structure yet, but the next thing to go to master will definitely be the independent libraries.

Any chance you can update this and fix any conflicts?

I have tested this patch-set against mrkipling/maraschino@master and can confirm it applies cleanly.
Can you handle the transmissionrpc module inclusion after the merge?
On Thursday, 5 January 2012 at 5:30 PM, Gustavo Hoirisch wrote:

Any chance you can update this and fix any conflicts?

Reply to this email directly or view it on GitHub:
https://github.com/mrkipling/maraschino/pull/15#issuecomment-3366179

We can but if you have the chance to do this before we get the chance to merge it would be great. If you want to have a look at the structure, just see either the experimental branch here or pull request #32.
Are you sure it merges cleanly? Git says it doesn't and I'm pretty sure I had to modify a couple of (minor) things to get it going properly but I also merged other branches before (on the experimental branch) so it might have happened from other merges. This is already included in there as well as all open pull reuqests btw.

commit 62cbd99 is ready for integration, merged successfully with master@latest and test OK.
note(s):
    - transmissionrpc 0.8 included in lib/transmissionrpc.
    - marachino.py now sources libaries from lib/* using code snatched
      from current experimental branch.

... marco ... ?

polo...

All clear for integration?
Sent from my phone
On 14/01/2012, at 7:57 PM, Gustavo<EMAIL_ADDRESS>wrote:

polo...

Reply to this email directly or view it on GitHub:
https://github.com/mrkipling/maraschino/pull/15#issuecomment-3491021

Think so. As far as my tests went, all is good but @mrkipling is the one to make the merge tho and he's been sick the last few days and quite busy before that.

Sorry this took so long to review - I've just merged it in. I don't use Transmission myself and haven't tested it thoroughly, but looking at the code it seems straight-forward enough (and gugahoi has informed me that it worked nicely for him). I'll hand it over to Transmission users on the XBMC forums to test.
Thanks for the contribution!

ghuntley, i had a problem where the module would vanish while a torrent was initialising. refer https://github.com/mrkipling/maraschino/pull/69. i was unable to get the total eta to work after this 'fix' any chance you could take a look?

Sure.
I'm on holidays, when I get back (2 weeks) I can have a stab at it
Sent from my phone
On 09/02/2012, at 7:09 PM, David<EMAIL_ADDRESS>wrote:

ghuntley, i had a problem where the module would vanish while a torrent was initialising. refer https://github.com/mrkipling/maraschino/pull/69. i was unable to get the total eta to work after this 'fix' any chance you could take a look?

Reply to this email directly or view it on GitHub:
https://github.com/mrkipling/maraschino/pull/15#issuecomment-3884219


Add keepalive_interval as a persistent configuration option for c8y_bridge.conf

Is your feature improvement request related to a problem? Please describe.
Working with a customer that is utilizing cellular devices.  They have strict bandwidth requirements where even the frequency of keep-alive messages can impact the negotiated contract with their cellular provider.  The proposed option would allow them to limit the number of MQTT keep alive messages being sent by their device and ultimately the amount of bandwidth that passes through the cellular connection.  This could impact any device that is utilizing a cellular connection.
Describe the solution you'd like
Would like the option to either be set through the tedge cli and saved as a tedge configuration option.  Could also be a persistent entry in the c8y_bridge.conf file.  I have found that when I manually add it to the configuration file and restart Mosquitto service through sysctl, I can see the behavior I want.  The problem with this is that the configuration file gets recreated when a tedge reconnect c8y command is run and the change needs to be reapplied.
Describe alternatives you've considered
The workaround that we are investigating in the short term is scripting the modification of the bridge configuration file and then restarting the Mosquitto service.  This is a bit cumbersome and technically may not be feasible given the additional start up latency that it will introduce. We have also considered creating a new configuration file and storing it in the /etc/mosquitto/conf.d directory (this is already referenced in the mosquitto.conf file, similar to /etc/tedge/mosquitto_conf) .   I think this fails because the bridge seems to be created by some logic in thin-edge where it builds out the connection and the keepalive_interval needs to be in the context of that connection.  Because it is outside of that in this config file we have created, it prevents Thin-Edge from starting.
Additional context
I don't think there are any screenshots that are needed.  I'm happy to jump on a call and show any of things I mentioned above if that is helpful.

@jmoo900 It makes sense to have the mqtt keep alive interval configurable, however playing nicely with the cellular network is only half of the picture, the keep alive interval would have also be compatible with the Cumulocity IoT MQTT broker, as I believe there is an upper limit here before the platform thinks the connection is dead (but maybe I'm wrong here).
Have you already experimented with the some ping intervals against Cumulocity IoT and have any idea of what upper limit would be sustainable?

Hey @reubenmiller,
You bring up a good point.  I have really only been focused on the device side of the equation.  The configurations that I have been trying have been in the 5 minute range and based on some wireshark outputs, it appears to be working as expected.  I haven't noticed any MQTT connection issues with my thin-edge.  If I remember correctly, the customer that I am working with was looking at around 15 minute interval for the MQTT keep alive.  On the Cumulocity side, there is the concept of a "required interval" that allows you to set the expected communication frequency of a device, but I think that just drives the availability/online status of the device.  I will have to do some digging on the actual connection to see if there are any timing limitations for the MQTT broker.

On the Cumulocity side, there is the concept of a "required interval" that allows you to set the expected communication frequency of a device, but I think that just drives the availability/online status of the device.

Yes, you are correct. The "required interval" has no influence on the MQTT level (as the feature works for any kind of device, not just MQTT devices).

I will have to do some digging on the actual connection to see if there are any timing limitations for the MQTT broker.
That would be a great contribution :) But we can also help if you need some guidance with anything.

I reached out to the data in motion team to try to get some clarity on this.  Specifically asked them if there were any limitations on what an MQTT keep alive could be set to.  Initial feedback was:
"@Jake Moody what do you mean by the MQTT timeout interval? In core mqtt by default we have 10 seconds time set for the connect, meaning after the TCP connection is established client must send connect message within those 10 seconds. In the connect message there is a keep alive interval that doesn't have any restriction, it has to non-negative value and then server uses this and sets the timeout to 1.5 of this value"
I tried to futher clarify if there could be situations where the TCP and MQTT keep alive were out of sync based on what was set here and received the following:
"I see, so if the TCP connection will be closed, for example by OnTopLb, server should also terminate it after reaching keepAlive*1.5 as it won't be receiving the PING message within this time. Client in such case should also notice that the connection is closed and it should try to reconnect, but I would say that it really depends from the client implementation"
So my limited technical take on this is that as long as its a positive numerical value, Cumulocity will consider the MQTT keep alive to be 1.5 times this.

Did some basic testing on my thin-edge device.  Set the keepalive_interval setting to 5 minutes and monitored the bridge health over a span of 2 hours.  Seems like the bridge health stayed at 1 for the duration.  I also ran a test with an MQTT keep alive set to 10 minutes and monitored the bridge health over a span of around 2 hours.  Noticed some differences with this second test.  Saw the bridge health value go to 0 before being set back to 1 within a second or so.  Not sure exactly what to make of this.  Should I proceed with higher intervals?


Add example: Combobox with list store

Add an example that utilizes Liststore in Combobox.

Apart from my comments and the clippy lint, looks good to me.

the clippy lint

@GuillaumeGomez That's from old code and a new clippy lint, and because we always use latest beta this now triggers for an unrelated PR. Want to send a PR to fix the clippy lint?

Apart from my comments and the clippy lint, looks good to me.

thanks for the review

@bilelmoussaoui Opinion applied. thanks for the review

@GuillaumeGomez @bilelmoussaoui  Can i merge?

Thanks!


Update MLPerf RN50 parser to show filesystem path under test

Closes #28
Signed-Off-By: Joe Handzik<EMAIL_ADDRESS>

Example new output:
MLPerf Results:
Log directory name: /
Filesystem test path: /mnt//datasets/imagenet/recordio:/data
Number of iterations: 5
Nodes tested: 4
Epoch 0:
Speed: 77578.531 images/second
Average time: 16.385 seconds
Overall:
Speed: 78440.737 images/second
Average time: 12.307 minutes


OSSM-3669: initial changes for interop pipeline support

We need an image which can be run on OCP. This is first change which is required. It contains also some minor improvements.

There will be probably more changes, we just need this one merged to be able to start with the interop openshift CI config draft.


[WIP/HOLD] Update Ember-CLI to 2.2.0

This PR tracks the changes to the 2.2.0 beta of Ember-CLI and will be finished once the official 2.2.0 ember-cli has been released.

Sweet, looking forward to this, lots of exciting stuff happening in ember-cli land at the moment!

Rather than go and fix all the merge conflicts, I'm just gonna raise a new PR.


[firebase_auth] Firebase Auth does not seem to pass iOS bundle id information during Phone Verification reCAPTCHA workflow.

Describe the bug
Firebase Auth does not properly pass iOS bundle id information to backend during Phone Verification reCAPTCHA workflow. Because of this, phone verification with reCAPTCHA workflow always get rejected, if API key is restricted to specific app identified by bundle id.
To Reproduce
Steps to reproduce the behavior:

Check out FirebaseAuth sample app and run it. (Any firebase app that uses FirebaseAuth could show this problem) https://github.com/FirebaseExtended/flutterfire/tree/master/packages/firebase_auth/firebase_auth/example
Go to google cloud console's credentials page, and restrict the API key from GoogleService-Info.plist to be only usable with iOS app with bundle id 'io.flutter.plugins.firebaseAuthExample'.
Attempt to sign in with PhoneAuthProvider in iOS simulator.
App will redirect you to reCAPCHA flow because you are using iOS simulator. However, callback will be returned with error information.

flutter: verifyPhoneNumberError - {"error":{"code":403,"message":"Requests from this ios client application <empty> are blocked.","errors":[{"message":"Requests from this ios client application <empty> are blocked.","domain":"global","reason":"forbidden"}],"status":"PERMISSION_DENIED"}}
Above error indicates that some form of id or token wasn't passed, so auth handler could not identify the caller and<empty> was returned instead.
Since the above error does not happen when there is no bundle id restriction in API key, I think that FirebaseAuth isn't properly parsing or passing app id/token information to the Firebase Auth backend.
This behavior seems to be auth specific, as I can still use Firestore or other Firebase services without any issue after restricting the API access, and I am having this empty app id problem only in Firebase Auth Phone verification. (Even email auth works fine)
Expected behavior
App identification info should be passed correctly, and VerifyPhoneNumber with reCAPCHA flow should end successfully even when API key is restricted to a specific iOS apps.
Additional context
Add any other context about the problem here.
flutter doctor -v
[✓] Flutter (Channel stable, v1.12.13+hotfix.7, on Mac OS X 10.15.3 19D76, locale en)
    • Flutter version 1.12.13+hotfix.7 at /Users/user/Dev Tools/flutter/flutter-1.12.13+hotfix.7
    • Framework revision 9f5ff2306b (7 days ago), 2020-01-26 22:38:26 -0800
    • Engine revision a67792536c
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/user/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3.1, Build version 11C504
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 43.0.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] Connected device (1 available)
    • iPhone 8 • E8B94077-0C63-4CAD-A6FC-225654F5AD11 • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-3 (simulator)

• No issues found!

FirebaseAuth version: 0.15.4
FirebaseCore version: 0.4.3

any news on this topic? I seem to have the same issue

Same issue

same issue

Having the same problem, when using API restriction to be used from app only I get 403's

I having the same issue.

I having the same issue.

Hey 👋
Our rework of the firebase_auth plugin as part of the FlutterFire roadmap was published over a week ago with a ton of fixes and new features.
Please could you try the new version and see if this is still an issue on the latest release? If it is then please could someone submit a new up to date GitHub issue so it's easy to track.
For help migrating to the new plugins please see the new migration guide: https://firebase.flutter.dev/docs/migration


Request: Mystic Spells Absent Even When Selecting The "Search All Spellbook Option" Is Active

Recently a new feature allow for a far better management of spellbooks in the game by allowing to add any "normal" spell to any spellbook, however unique Mythic Spells (Demon, Angel, etc.) are unavailable even with this option selected.
I request the possibility to include this unique spells to the selection.

Resolved in upcoming 1.3.9 :)


Implement documentation for the project

Wiki add:

https://github.com/ita-social-projects/Forum/wiki/License:
https://github.com/ita-social-projects/Forum/wiki/Tech-documentation
https://github.com/ita-social-projects/Forum/wiki/Project-Launch-Instructions
Format readmy.txt


Add support for Cluster cache deployment

Add support for Cluster cache deployment

lgtm, merged. Thanks @ryanemerson !


[Bug]: Flappy bird game is not showing on the website

Give a brief about the bug discovered✍
Hey @raj03kumar I found that the flappy game was merged in the repo but it is not showing on the website. Please assign me this issue. I can fix it.
What is the expected behavior?
Show the flappy bird game on the website.
Give a brief description to get the expect behaviour.
Add flappy bird games in the routes
Are you contributing for a program?
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hi @sumitkr2000, I assigned this issue to you. Please complete the work within a week and go through the readme and contributing.md for successful merge.

Yeah sure!

Hey @panwar8279 this game is made with html css and js. So can't include it in the router file. What should I do???

Your description says that all the games should be made with react.


Add bot locking and updating Conda and Pip packages

This PR adds a bot for creating and updating locks for Conda and Pip packages. It consists of a GitHub Actions workflow which handles git and creates a Pull Request and update_conda_lock.py script, which creates or updates locks.
Basically, it creates Conda environment based on environment.yml (which shouldn't be locked, but locked will work too) and creates locks based on Conda and Pip packages installed.
Currently the workflow added can only be run manually and that's how I tested it. It can be easily modified to work regularly on schedule of course. For now there are some simple constants used for PR title, user creating commit and commit title but these are easily configurable. For testing purposes I set them to be set as an input of the manual workflow.
To to use locks during building the scripts will need to be slightly modified. Instead of using conda env create --file environment.yml it needs to be conda create --file conda.lock -n env followed by installing pip packages by the pip from such environment, eg. conda run -n env python -m pip install -r pip.lock.
The script update_conda_lock.py can be tested locally after setting required environment variables:

BOT_ENV_NAME -- custom name for the temporarily created environment (such environment cannot exist),
BOT_ENV_YML -- path of environment.yml file, relative to the current directory (in CI it's repository root),
BOT_CONDA_LOCK -- path of the conda lock being updated or created, eg. conda.lock,
BOT_PIP_LOCK -- path of the pip lock being updated or created, eg. pip.lock.

It doesn't call git or create a PR so it's safe to use locally. Of course its requirement is that conda command is available.

@mithro @litghost @HackerFoo there's a question we need to address and would appreciate your opinion.
The bot is a github action that creates a PR. Unfortunately, such PRs, when created with the standard GITHUB_TOKEN, don't trigger further GitHub actions - so we don't know if our update works or not.
This is a pain, but there are workarounds described here: https://github.com/peter-evans/create-pull-request/blob/master/docs/concepts-guidelines.md#triggering-further-workflow-runs
TL;DR:

manually close and reopen a PR to trigger CI. Very easy, but manual
create a personal token and push using it instead of GITHUB_TOKEN. There is a symbiflow-bot account that could be used for that, but it would require someone with proper accesses to set things up (who would that be? @mithro ?). I'd recommend this option.
Create a dedicated GitHub App - an overkill imho
Use deploy SSH keys - would work only on push events, not pull requests, so it's not really good

@PiotrZierhoffer - I think a dedicated GitHub App is the long term solution (the goal is that this bot should be eventually usable by anyone who wants to roll conda-lock files).
In the short term I can set up the personal token. What do you want the environment variable to be called (so it is different from the repository provided GITHUB_TOKEN)?

To be honest the name doesn't really matter as it will be passed to the bot as a GitHub Action with:  variable so every repository using that Action in a workflow can pass it's own variable.
If this is going to be the symbiflow-bot's token then maybe SYMBIFLOW_TOKEN is a reasonable name?

There were significant changes after thorough testing with symbiflow-arch-defs repository. Previously it's been tested during development on symbiflow-examples which eg. lacks local pip packages which caused many problems.
The Action has been separated from the workflow and made much easier to use on any repository. The Action itself is now the subject of this PR: https://github.com/SymbiFlow/actions/pull/5
The workflow update_conda_lock has failed because of permission problems: https://github.com/SymbiFlow/symbiflow-arch-defs/runs/1628574436?check_suite_focus=true#step:3:858 and it seems to be a cross-repository PR problem. Running this workflow after merge the same token should work. Definitely after changing token to the PAT it should work too but also should trigger the CI on the created PR.
Pushing and creating a PR from this PR's latest commit has been successful on the Antmicro's fork: https://github.com/antmicro/symbiflow-arch-defs/runs/1628574276?check_suite_focus=true#step:3:856 and the PR adding a lock to the master branch from that workflow is: https://github.com/antmicro/symbiflow-arch-defs/pull/23

It seems update_conda_lock workflow failed because of the cross-repository restrictions of the GH Actions tokens.
The git push result was:

remote: Permission to SymbiFlow/symbiflow-arch-defs.git denied to github-actions[bot].

It works well in a fork, though: https://github.com/antmicro/symbiflow-arch-defs/runs/3374983149?check_suite_focus=true#step:3:991

BTW Are we using https://github.com/conda-incubator/conda-lock ?

BTW Are we using https://github.com/conda-incubator/conda-lock ?

No, not at all. I've tested it and I don't even remember all of the issues but certainly it didn't include PIP packages at all.
Conda's built-in conda env export is used to capture Conda packages instead.

Can we figure out how to collaborate with the conda-lock project?

@mithro sure, it's probably quite specific, but we can think about that

All of the kokoro CI went green in https://github.com/SymbiFlow/symbiflow-arch-defs/pull/1885/commits/79ff9e9c0fa75aba889f9a919dafeaca4e33b8f7 and https://github.com/SymbiFlow/symbiflow-arch-defs/pull/1885/commits/13c4e835bc1874733d214ca5c67d6456744895b3 only changes the conda lock CI to not be triggered on PRs and to be executed only on the default branch. Merging


IGNITE-20026 Java thin: Remove port range limitation

Remove the port limitation to support other dynamic port reverse proxys. Like testcontainers[1]
[1]
https://stackoverflow.com/questions/76746998/why-ignite-tcpclientchannel-port-can-not-larger-than-49151

@ptupitsyn What can I do for this CI failure?

@Shoothzj this is unrelated flaky failure, I've restarted CI


hide task with no errors

I wonder if the tasks that do not have errors at all should or should not appear in the console ; maybe they should not by default, and have a "show all tasks" button somewhere

Addressed on https://github.com/CMSCompOps/wtc-console/pull/19


ci: use custom github token in release

Summary

Issues

Coverage remained the same at 100.0% when pulling ab848d2a5fb66621a0b7e0bf90d31c8015069b64 on release-fix into c67662fb74e21194482caea08fbe5261daed6f71 on main.


"@elastic/ecs-pino-format" has a deprecated transitive dependency "string-similarity"

"@elastic/ecs-pino-format" depends on "@elastic/ecs-helpers", which depends on an old version of "fast-json-stringify", which depends on "string-similarity" which is deprecated:
@elastic/ecs-pino-format > @elastic/ecs-helpers > fast-json-stringify ><EMAIL_ADDRESS>Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.

Newer versions of "fast-json-stringify" do not depend on "string-similarity" any more.

See discussion at https://github.com/elastic/ecs-logging-nodejs/pull/145
This'll require a major ver bump to raise the min supported Node.js to v14.17.0.
I hope to do so relatively soon.

This'll require a major ver bump to raise the min supported Node.js to v14.17.0.

Update from #155: this will not require a major version bump. Releases v1.5.0 and later will have this fix.


initial dump for validators

Still WIP, but works for validators right now.
Waiting on Mysten-Infra PR: https://github.com/MystenLabs/mysten-infra/pull/90/files
Next steps:

Allow for read of individual entries
Allow for remote debugging
Support Narwhal & Index tables

Example:
mbp> ./sui dump ~/.sui/sui_config/authorities_db/686681184798e9638fd14d44bc4719b8bf14278a1bf5a3a1af000c661a9e79f0/store  parent_sync

"(0x0000000000000000000000000000000000000001, SequenceNumber(1), o#0649cbd0604acbc6a6bd6591dbcd8cc21ffc445d9a904e607786cb635137bf42)": "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA="
"(0x0000000000000000000000000000000000000002, SequenceNumber(1), o#beb14d5d1d1f934992cd9a414b0a5fa83c407189b5ca6212e0fe4fcd37fc55c6)": "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA="
"(0x0000000000000000000000000000000000000005, SequenceNumber(1), o#687b3f34e451e92380e1b2dccc4b6b69dea60133931999963740ef018b091f08)": "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA="
"(0x018d82bca606525e9702682557d540f34498f17a, SequenceNumber(0), o#1b559b6a1c08fc6b4636ff53c0c3d49907bc4a52046ae3485a98ad20cb1ec70e)": "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA="

Closing in favor of https://github.com/MystenLabs/sui/pull/3188


add linting through travis, rename exposeConsolidate, add Faq

I think for the test we need a practical test with an engine that can't be configured globally (for which you'd need the requires option). Then we could use that to test whether consolidate will compile everything correctly.

makes sense.
I won't have much time during the next two weeks so I will continue working in two weeks.


Bug: Submission Report

Issue:
QI name is blank in report
Steps to Recreate:

Checkout QA Test Data for Potter config 3
Import test file 2023-Potter-1660-3-RATA-fe8a5a6f-5f42-472c-83dd-67d2a4ec7dd3.json
Evaluate and Submit
View report in CDX Submission History (the QI Name is blank)

https://usepa.sharepoint.com/sites/CAMDCVPTeam/Shared Documents/Forms/AllItems.aspx?id=%2Fsites%2FCAMDCVPTeam%2FShared Documents%2FECMPS Beta Testing%2F2023-10-27 Screenshots%2F1660_TEST_DETAIL.html&parent=%2Fsites%2FCAMDCVPTeam%2FShared Documents%2FECMPS Beta Testing%2F2023-10-27 Screenshots

QI name is displayed
Private Zenhub Image


Now that Go Modules have been default for a long time, assume GO111MODULE=on by default.

What version of Go are you using (go version)?

$ go version
go version go1.15.8 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="~/.cache/go-build"
GOENV="~/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="~/go/pkg/mod"
GONOPROXY="github.com/chewxy/graphqlsp"
GONOSUMDB="github.com/chewxy/graphqlsp"
GOOS="linux"
GOPATH="~/go"
GOPRIVATE="github.com/chewxy/graphqlsp"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build810152107=/tmp/go-build -gno-record-gcc-switches"

What did you do?
$ go get<EMAIL_ADDRESS>
The package is just an example. Any package with @version will fail with the same errors.
What did you expect to see?
No errors
What did you see instead?
go get output
$ go get<EMAIL_ADDRESS>go: cannot use path@version syntax in GOPATH mode
$ echo $GOPATH
$ echo $GO111MODULE

This of course is easily fixed by
$ export GO111MODULE=on

It's 2021. Shouldn't we be assuming that $GO111MODULE=on by default now? I'm aware of #37755 and I do think there might be some benefits for keeping GOPATH mode (which is very useful for working on interdependent packages with independent modules), but by default Go should always assume $GO111MODULE=on even when the environment variable is not set.

This is already happening in 1.16 https://tip.golang.org/doc/go1.16#go-command

Closing as dup of #41330.


Inconsistent code highlighting

Location
https://docs.swift.org/swift-book/documentation/the-swift-programming-language/guidedtour/
Description
The section regarding three quotation mark multiple line strings has highlighting after the /() that isn't consistent with the previous code color scheme.  The issue isn't seen in the GitHub preview, only on the webpage.
Correction
No response

Moving from the TSPL repo to the DocC Render repo — the book doesn't do its own syntax highlighting, so we need to fix the issue in the tooling.

This highlighting issue might be specific to multiline string literals.  Looking at the String Interpolation section, those examples look like they're getting the expected highlighting.


Minor RaiseInventoryAlloc improvements

This'll now hook the mem_alloc call inside MessageControl::stageInit to change the pG + 0x3C pointer, instead of hooking the whole stageInit func and changing it afterwards, this should now let the MessageControl::loadStageFont call inside stageInit act on our new buffer instead, maybe will help with inventory issue, not sure.
Also memsets to 0 & aligns the SubScreen buffers to 4096 bytes, and in debug builds adds a SafetyBuf between them, which will cause an exception if anything writes to it - would probably be good to have this in release too, but 0x4000000 bytes is a lot to waste on it...
Sig seems to work fine across 1.1.0/1.0.6/debug.

Nice! I'll see if I can get the inventory issue to show up again, and also send a test build to the people in Nexus. Thanks!


Gate requires in chef-utils and chef-config as well

Update the cop config to handle those as well
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

busted unit tests

Hmm. looks like tests are failing due to uninitialized constant ChefUtils::DSL which may be related.

Fixed things, but excluding specs and using ChefUtils::CANARY


[1.11] Avoid port conflicts in the networking tests.

High-level description
Marathon randomly assigns host ports. In rare cases it causes port conflicts in the networking integration tests. We shouldn't rely on this feature and have to assign unique port numbers manually.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS-46220 test_networking.test_vip can fail because Marathon says Constraints for run spec [xxx] not satisfied.

Related tickets (optional)
Other tickets related to this change:

DCOS_OSS-3736 Update marathon app configuration to match Marathon 1.5 syntax

Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change: integration tests
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: none
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot label [Ready For Review]


Can not read external file from s3 private bucket

Create a private bucket from AWS S3, set environment variables.
Run by dbt run -s ./models
Got error Python model failed: HTTP Error: HTTP GET error on 'https://bucket.s3.amazonaws.com/ms/distribution/csv/orders.csv' (HTTP 403)
My codes:

profiles.yml:
default:
  outputs:
    dev:
      type: duckdb
      path: db/dbt.duckdb
      extensions:
        - httpfs
        - parquet
        - aws
      settings:
        s3_region: "us-east-1"
        s3_access_key_id: "{{ env_var('S3_KEY') }}"
        s3_secret_access_key: "{{ env_var('S3_SECRET') }}"
  target: dev

sources.yml:
sources:
  - name: csv
    meta:
      external_location: "s3://bucket/ms/distribution/csv/{name}.csv"
    tables:
      - name: orders
      - name: customers

models.py:
def model(dbt, session):
    orders = (
        dbt.source("csv", "orders").filter("status='PENDING_PAYMENT'").set_alias("o")
    )
    customers = dbt.source("csv", "customers").set_alias("c")
    pending_orders = orders.join(customers, "o.customer_id=c.id")
    return pending_orders

Some findings: If I persist the s3 secret to duckdb home folder - "~/.duckdb/stored_secrets/" then the dbt run can work without error. It seems the dbt-duckdb did not actually feed s3 credentials to duckdb.

It would be surprising if the problem was that we weren't feeding the s3 credentials to DuckDB given that so many people run with those settings (that is, I feel like I would have heard that complaint from lots of people if that was broken.)
The most common problem I see with these settings is that the region isn't configured correctly (i.e., the data lives in an S3 bucket defined in a region that is different from the one defined in s3_region.) Of course, if you have a way to make this work using the CREATE SECRET construct (which it sounds like you do), you can just run that in a macro that is configured using the on-run-start hook and be on your way.

@jwills Thanks for your reply, I double checked my s3_region and I am pretty sure the value is correct. I also did some testing by adding below config to write files to s3 bucket, it worked without using CREATE SECRET construct. So it proved that my s3 settings were correct. I also don't believe DuckDB has problem with reading files from s3, but could be something  from dbt-duckdb adapter caused s3 settings not proper configured on DuckDB so caused reading from s3 bucket not working.
Below is the config I used so write table to external files on S3 bucket. Again, writing worked without the CREATE SECRET construct, but reading was not working.
Model.py
dbt.config(
        materialized="external",
        location="s3://bucket/parquet/duckdb-sample/pending_orders.parquet",
        format="parquet",
        options={"partition_by": "customer_id", "overwrite_or_ignore": True},
        glue_register=True,
        glue_database="duckdb_sample",
    )

def model(dbt, session):
    orders = (
        dbt.source("csv", "orders").filter("status='PENDING_PAYMENT'").set_alias("o")
    )
    customers = dbt.source("csv", "customers").set_alias("c")
    pending_orders = orders.join(customers, "o.customer_id=c.id")
    return pending_orders

Stumbled upon the same issue for some time, not sure if it also fixes it for you (and I realise it's a few months later).
However simply rewriting the 'settings' in your profiles.yml file to:
secrets:

type: s3
region:
key_id:
secret:

did the trick for me!


routing broken

Expected Behavior

routes should be setup consistently and point to the one node hosting a block.
all blockaffinities should have distinct spec.cidrs

Current Behavior
sometimes, a blockaffinity is created with the same cidr as an existing one:
$ kubectl get blockaffinity node-15.company.com-10-148-158-64-26 node-14.company.com-10-148-158-64-26
NAME                                                      AGE
node-15.company.com-10-148-158-64-26   105s
node-14.company.com-10-148-158-64-26   8d

IPAMBlock object shows the correct affinity:
$ kubectl get ipamblock 10-148-158-64-26 -ocustom-columns=AFFINITY:spec.affinity
AFFINITY
host:node-15.company.com

routes(on all other nodes) point to both nodes(.12 and .13 are node-14 and node-15):
$ ssh node-1.company.com ip -o r | grep <IP_ADDRESS>/26
<IP_ADDRESS>/26 proto bird \	nexthop via <IP_ADDRESS> dev eth0 weight 1 \	nexthop via <IP_ADDRESS> dev eth0 weight 1

pings to pods in affected subnets time out.
Possible Solution
Workaround is to delete the old blockaffinity.
Context
We create dev environments as namespaces, spinning up 50-150 pods at once.
Your Environment

Calico version: v3.20.3
Orchestrator version (e.g. kubernetes, mesos, rkt): k8s v1.22.11
Operating System and version: ubuntu focal
Link to your project (optional):

We run with kdd and typha(2 replicas), bgp mesh, and some nodes peer with routers; each cluster is 50 nodes, blockSize: 26
We see this problem multiple times a day.  Could I get you to post some of the node logs to see if there's anything we can rule out from there? Might also be good to get the logs of the kube-controllers as well since they should be handling IPAM GC too.

will provide the logs of our next case. One interesting thing we noticed is that sometimes the routes become consistent (the blockaffinity object gets deleted) after a long time, e.g.
node-19.company.com-10-148-144-64-26             105d
node-35.company.com-10-148-144-64-26             99m

...so at least after 99 minutes

I noticed that ipamblock is created together with the new blockaffinity:
$ kubectl get ipamblock/10-148-4-64-26 blockaffinity/node-14.company.com-10-148-4-64-26 blockaffinity/node-48.company.com-10-148-4-64-26
NAME                                             AGE
ipamblock.crd.projectcalico.org/10-148-4-64-26   37m

NAME                                                                                AGE
blockaffinity.crd.projectcalico.org/node-14.company.com-10-148-4-64-26   37m
blockaffinity.crd.projectcalico.org/node-48.company.com-10-148-4-64-26   14d

both are 37 minutes old.
So I checked for blockaffinities that don't have an ipamblock and found some. That doesn't sound healthy.
I checked the age of all ipamblocks and found that the bulk of them were created 14 days ago, and the rest is more recent.
14 days ago we tried to fix the(a?) routing problem by upgrading calico to v3.22.4 and downgrading back to 3.20.3 and rebooting everything, because we encountered a whole lot of https://github.com/projectcalico/calico/issues/5779 , which was worse than routing trouble.
Now I believe that we had a different routing problem before up- and downgrade, the only trace of that other problem is how it manifested on some nodes:
root@node-45.company:~# ip r get <IP_ADDRESS>/32
RTNETLINK answers: Invalid argument

...so I think we should close here because too many things have happened to this clusters

OK, @fungusakafungus, thanks for the reply. We'll close this, please let us know if you find issues again, but I'd advise you to use a more recent calico version (v3.20.3 is from Nov 2021; unfortunately the fix for https://github.com/projectcalico/calico/issues/5779 did not make it to v3.22.4, but it should be in the next patch release; it is present in v3.23 and v3.24 though).


new error

Traceback (most recent call last):
File "/home/rexxxo/tiktok-bot/bot.py", line 132, in start_service
WebDriverWait(self.driver, 5).until(
File "/home/rexxxo/tiktok-bot/venv/lib/python3.11/site-packages/selenium/webdriver/support/wait.py", line 105, in until
raise TimeoutException(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message:
Stacktrace:
#0 0x555ee1678863
#1 0x555ee136e8c6
#2 0x555ee13b9618
#3 0x555ee13b96d1
#4 0x555ee13fc744
#5 0x555ee13db5cd
#6 0x555ee13f9c19
#7 0x555ee13db343
#8 0x555ee13ac593
#9 0x555ee13acf5e
#10 0x555ee163c84b
#11 0x555ee16407a5
#12 0x555ee162a571
#13 0x555ee1641332
#14 0x555ee160f87f
#15 0x555ee1667728
#16 0x555ee16678fb
#17 0x555ee16779b4
#18 0x7f3e549123ec <unknow

Checking it

Couldn't replicate the error. Can I get a screen recording?

A new version has been pushed and should fix all raised bugs. Please try it and feel free to reopen an issue for any other bugs.


SLSA Provenance Attestation

Proposal:
Witness should use information in the attestation context to create an SLSA Provenance document.
https://slsa.dev/provenance/v0.1

This will be completed as a optional output attestor that scrapes data from the other attestors

whats the status of this?

👋 Also interesting in getting Provenance with witness.

Closed by https://github.com/in-toto/witness/pull/381 and include in Witness v0.4.0 and later.


Fix yaml

Let me know if that worked. It should pull into the catalog now (I think you need a release)


Npm action fix

What was changed

Why?

Checklist

Closes

How was this tested:

Any docs updates needed?

Please ignore, accidentally opened a PR from my fork


Add request hook

Add a request hook that fires when a request is sent. This resolves kennethreitz/requests#5037.

@Holzhaus I remember the hooks are removed in some old commit https://github.com/kennethreitz/requests/commit/ee6c638e9cfdbdf39b9e499d95c353460365f42c#diff-ab3246c102b4e3c7381cf4d1ea4b2bea for a reason, I don't know why though.

Seems like there was a conflict and the fix did not make it into the branch.
A similar issue ( #5037 ) states that it is fixed by this merge - but unfortunately does not seem to be in because of that conflict.
Would be great if this could be fixed. Any ETA ?

the current hook we have is the only necessary one, i determined. hence, the other ones were removed during the big refactor.