Activate selected directory's environment within a multi-root workspace with different environments

Looked around for open issues related to my problem; I could only find #690 -- which has been closed.
To summarize, I have several different projects mapped using [workspace name].code-workspace and in each there is a .vscode/ containing settings.json with:
{
    "python.pythonPath": "venv\\Scripts\\python.exe"
}

Environment data

VS Code version: 1.49.3
Extension version (available under the Extensions sidebar): N/A
OS and version: Windows 10.0.18362 Build 18362
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8.5 64bit
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: N/A
Relevant/affected Python-related VS Code extensions and their versions: N/A
Value of the python.languageServer setting: N/A

Expected behaviour
On integrated prompt launch activate project's venv using its ./.vscode/settings.json.
Actual behaviour
Uses the first project's venv.
Steps to reproduce:
See this comment for a really good example.
Reproducing my specific example:

Set up multi-root workspace.
Add venv to each folder in workspace using python -m venv venv from within the folder or pointing it to the root of the directory.
Close all files but leave workspace open.
ctrl+shift+" ` " to launch new terminal.
Select a directory other than the first one (gif).

NOTE: as an update (I've been playing around). The expected behavior occurs by doing the following:

Create the multiple-root workspace in vscode (1 & 2 from my specific example above).
Open a file within the directory you intend to activate your environment automatically by launching the terminal.
f1-> Terminal: Create New Terminal (In Active Workspace).

Logs
NOTE: could not figure out how to get the logs for the integrated terminal/workspace automation.

Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.

Try the new cross-platform PowerShell https://aka.ms/pscore6

PS D:\github\ortools-pyinteractive> & d:/github/solverstack/solverstack-crud/venv/Scripts/Activate.ps1
(venv) PS D:\github\ortools-pyinteractive>

We have marked this issue as "needs decision" to make sure we have a conversation about your idea as we don't view this as a bug (it's really hard to infer what the user wants in a multi-root workspace with differing interpreters, e.g. did you mean to have that file open when you launched the extension or did it just happen to be open from when you last quit VS Code?). We plan to leave this feature request open for at least a month to see how many 👍 votes the opening comment gets to help us make our decision.
If the automatic activation is not working for you it is possible to disable it so you can manually activate whichever environment you prefer.

Thanks for the response. My specific setup might be a bit different than the comment I linked, but it's relatively similar with the main difference being that my virtual environment is python venv located within the individual roots (each folder). I guess maybe I should have been more descriptive of what I was expecting given my specific setup.
On launching New Terminal (crtl+shift+ " ` ") I am prompted with "select a current working directory for the new terminal". If configured via ./.vscode/settings.json, at that specific directory I selected, it would activate an environment using that python.pythonPath setup. I believe this is how it works if it is in its own workspace (using ${workspaceFolder}/path/to/python.exe).
Maybe this is just a user error and there is a way to do this. I'll play around.

it's really hard to infer what the user wants in a multi-root workspace with differing interpreters

This definitely sounds fair, maybe there is a way to not infer anything if it is not setup within a root-specific .vscode/settings.json, i.e. launch a regular prompt with default integrations. I'm not familiar with how some users may use this feature outside of the python venv world, so I totally understand if there's tons more complexity behind this than I realize.

did you mean to have that file open when you launched the extension or did it just happen to be open from when you last quit VS Code

What extension are you referring to?

If the automatic activation is not working for you it is possible to disable it so you can manually activate whichever environment you prefer.

This could be ideal for now. Thanks!
I also would like to rename the title of this issue to something other than "Activate the environment for the open file in a multi-root workspace with differing environments" since that's not specifically my issue. A better title might be "Activate selected directory's environment within a multi-root workspace with different environments".

Here is a gif of the use-case. d:/github/solverstack/solverstack-crud/venv/Scripts/Activate.ps1 is the incorrect environment and I do not have any file open.

Updated the original post.

What extension are you referring to?

Our extension: the Python extension for VS Code.

Any updates on this?

@nikogeg nope; we would post if we had anything to share.
If someone wants to propose a solution, discuss it with us, and the submit a PR we would be happy to have such a discussion.

I have a similar use case. We have a workspace with 5 different projects, each with it's own venv. Everytime we open a new terminal window (without explicitly selecting which working directory), it opens up the terminal for the first directory on the workspace, even though the terminal itself is on the current working directory, which makes things inconsistent: the current working directory terminal with the first workspace directory venv.

To be honest I've moved away from doing this since this issue.

Reading through this thread after experiencing this issue and I have what I hope is a potential solution @brettcannon, but likely not enough knowledge of TypeScript to implement it myself.
It seems like the big disconnect here boils down to the extension's behavior when receiving the Terminal: Create New Terminal command, vs the Terminal: Create New Terminal (In Active Workspace) command. If no active terminals exist, the latter command is what is called by e.g. the Ctrl + ` shortcut.
Terminal: Create New Terminal activates the environment associated with that directory's selected interpreter. If it has none, it will activate the workspace interpreter's env. If the workspace also has none, it will not activate any environment at all.
Terminal: Create New Terminal (In Active Workspace) activates the environment associated with the first folder in the workspace (even if the working directory doesn't have an interpreter selected and the workspace itself does!)
Especially given that last point, I really think the current (In Active Workspace) behavior is undesirable and not what one might expect. At the very least, I think that this should be user-configurable to one of two behaviors when opening a terminal in a multi-root workspace (perhaps adding a true/false option called python.terminal.activateInWorkspace or something):

true: All new terminals created will use the working directory to select an interpreter (i.e. behavior of Terminal: Create New Terminal (In Active Workspace) will mirror the current Terminal: Create New Terminal -> <selected directory> behavior.
false: Current behavior, the environment activated by the python extension will always default to activating the environment of the interpreter associated with the first folder in the workspace, regardless of the working directory or the workspace interpreter settings.

It's on our iteration plan to take a fresh look at our terminal story and the multi-root part will play into it. But beyond that there are no updates.

Awesome, definitely looking forward to this as well!

Closing as dup of https://github.com/microsoft/vscode-python/issues/15522 which fixes the issue. Feel free to let us know if you still face the issue and we'll be happy to have another look.


Feature/reconciler

Fixes #126
This adds a first pass at a controller. It follows Knative's lead with the way it builds controllers, logs, and reconciles.
The space definition is still in flux, but once it's complete we'll add unit-tests similar to Knative Serving's where each sets up one particular cluster status, then tests the controller's behavior.

Created concourse pipeline for testing...

Created concourse pipeline for testing...


Find a way to remove dist from import paths

It would be nice to cleanly not have to include dist in the import paths. I wonder if the export paths functionality might handle this, although I'm not sure if this made it into the stable release.

@adobe export issue to Jira project CAI


Add both Meetups in Chennai

Add both Meetups in Chennai

Let's add this if this meetup is active, but I don't see any past or upcoming events on the meetup page?


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 861 ms

Resolved: Bofrost.de performance has improved in 85a15f1.


Replace the legacy java.util.Date with Instant or LocalDate

Suggestion: https://twitter.com/p_chmielowski/status/1560027370874085376?s=20&t=DDL0ltrfh6q-JieM_KESIw

Hi @waseefakhtar, Please assign this to me, if this issue is still open

@waseefakhtar
Raised a P.R for this
Please review


chore: removed v1beta2 samples

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

I'm assuming these are obsolete now?

Correct, they have been removed from documentation


HUD moves to accomodate keyboard #2

i made a new pull request according to the previous pull request @MendyK because it was a bit outdated.
https://github.com/pkluz/PKHUD/pull/179
check it out and merge if possible

Thanks

@PGLongo thank you for the PL. should i add move buttons when keyboard shows up in another PL?


Redesign of internals of SiteSet

This redesign changes the Site types (which label the "physics" or semantic meaning of a site) to no longer store an Index inside. Instead the SiteSet just stores an array of (Index,Site) tuples.
The goals of this redesign are to simplify the Site types and to allow generic code to update the indices in a site set. A notable future use case is removing the QNs from indices in a site set.
I was going to use Pair instead of Tuple for the SiteSet storage but Pair is also immutable, like Tuple, so I didn't see the benefit.

Instead of the method siteType for getting the site object stored with the Index, would a better name be siteObject? The reason is that technically siteType does not return a type, but an instance of a type.

Codecov Report

Merging #95 into master will decrease coverage by 0.2%.
The diff coverage is 96.85%.

@@            Coverage Diff            @@
##           master     #95      +/-   ##
=========================================
- Coverage    94.1%   93.9%   -0.21%
=========================================
  Files          26      26
  Lines        2495    2493       -2
=========================================
- Hits         2348    2341       -7
- Misses        147     152       +5

Impacted Files
Coverage Δ

src/physics/sitesets/tj.jl
98.48% <83.33%> (-1.52%)
:arrow_down:

src/mps/siteset.jl
97.14% <95.65%> (-0.16%)
:arrow_down:

src/physics/sitesets/spinhalf.jl
95.91% <96.15%> (-4.09%)
:arrow_down:

src/physics/sitesets/electron.jl
98.78% <98%> (-1.22%)
:arrow_down:

src/physics/sitesets/spinone.jl
98.75% <98.14%> (-1.25%)
:arrow_down:

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 75c3d23...5d9a1ab. Read the comment docs.

Looks like a pretty minimal change, it seems pretty natural to design it this way.
Is there a reason why the function that gets overloaded for the SiteTypes is called operator but the one for SiteSets is op?
Also, it is kind of small, but I prefer the signature operator(::SiteType,::Index,::Int) instead of operator(::Index,::SiteType,::Int). It feels more natural to have the type that is used for dispatch first, I think that is fairly common in Julia.
Regarding siteType vs. siteObject, can we just use the types instead of the instances, or are we anticipating types that might have other data in them?

Another question about interface. Say that I have my own SiteSet that has all SpinOnes, and then later on I want to add a SpinHalf at the edge. Is this the interface?
sites = SpinOneSites(N)
sites[1] = Index(2,"Site,SpinHalf,n=1")
setSite!(sites,1,SpinHalfSite())

If so, I am wondering if we can do:
sites = SpinOneSites(N)
sites[1] = (SpinHalfSite(),Index(2,"Site,SpinHalf,n=1"))

(I think we can do this with multiple dispatch, not sure if it is a huge improvement but I am just thinking out loud about what we can do with this design).

Another question, how do we deal with default operators like we do in C++ (i.e. the "Id", "F" and projector operators)?

Closing this for now, as we're probably going to pursue a different design of this system.


Remove irrelevant change from tutorial

In 01-svelte/07-lifecycle/02-ondestroy the boolean open switches value from false to true.
While solving the tutorial, I had to use the "solve" functionality in order to figure out what I was missing. This change of value seems irrelevant to the lesson itself and might bring confusion to others.

thanks!


Support multi typeclass which refers to each other

For two typeclass which involes between each other, there seems no way to define derivation rule.
With shapeless I would probably define something like this.
import shapeless._
import shapeless.labelled._

trait FieldEncoder[A] {
  def encode(fn: String, a: A): Unit
}

trait ObjectEncoder[A] {
  def encode(a: A): Unit
}

implicit def hlistEncoder[K <: Symbol, H , L <: HList ](
  implicit  key: Witness.Aux[K],
   hE: Lazy[FieldEncoder[H]],
  lE: Lazy[ObjectEncoder[L]]
): ObjectEncoder[FieldType[K, H] :: L] = ???

implicit def objFieldEncoder[A](
  implicit fe: ObjectEncoder[A]
): FieldEncoder[A] = new FieldEncoder[A] {
  def encode(fn: String, a: A) = {
    println(fn)
    fe.encode(a)
  }
}

I'm not sure I would consider this to be typeclass derivation, in the strictest sense. It is really a generic program.

@fommil Sounds reasonable. And I found it could be modeled as a single typeclass sometime

@jilen I know the problem you're trying to solve though, it comes up in JSON encoding / decoding. I prefer to allow such cases and have an encoding for them: e.g. by wrapping with an xvalue field. See https://gitlab.com/fommil/scalaz-deriving/blob/master/examples/jsonformat/src/main/scala/jsonformat/JsMagnolia.scala

@fommil Thanks for sharing.


badge string?

Hi,
i can't set badge '+1' on payload. My app can't increase counter.
Can you change type of badge payload like string?

Hey,
APNS doesn't natively support incrementing badge counts.
You'll need to keep track of the badge value within your application and send the correct badge value via APNS each time.


Error when list storage volume names for all projects (client)

(*ProtocolLXD).GetStoragePoolVolumeNamesAllProjects fails with `Unexpected URL path "/1.0/storage-pools/default/volumes/container/{containerName}?project={projectName}".
This is because u.String() is passed into urlsToResourceNames here. Unfortunately u.String() contains all-projects=true so splitting on this always results in an error.
An easy fix for this issue would be to pass u.URL.Path into urlsToResourceNames as the pathMatchPrefix argument. However, this will still omit the project name from the resulting list of resources.
@tomponline @stgraber when a client method is listing resources for all projects, should we include the project name in the resource? If so, should we add it as a query parameter. For example, container/{containerName} will become container/{containerName}?project={projectName}

I'll take a look at this tomorrow and come back to you.

I think we need to fix both parts. Changing the return value of GetStoragePoolVolumeNamesAllProjects to map[string][]string so that the project name can be ascertained, as well as allowing the ability to differentiate between volumes of the same name in different projects.
We can then iterate through the URLs returned from the API, parse them like urlsToResourceNames does, but only use the path part of the URLs to prefix match against the path part of the API URL used.


runtime: TestFreeBSDNumCPU fails

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

$ go version go1.14.2 freebsd/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="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOFLAGS="-v -race"
GOHOSTARCH="amd64"
GOHOSTOS="freebsd"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="freebsd"
GOPATH="/root/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/freebsd_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/data/jails/go/root/golang/go/src/go.mod"
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 -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build792114039=/tmp/go-build -gno-record-gcc-switches"

What did you do?
./run.bash -no-rebuild -run=^go_test:runtime

What did you expect to see?
All tests pass
What did you see instead?
--- FAIL: TestFreeBSDNumCPU (0.01s)
    numcpu_freebsd_test.go:13: expected "OK\n", but got:
        runtime.NumCPU() expected to 25, got 24 when run with CPU list 0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23 []string{"0", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12", "13", "14", "15", "16", "17", "18", "19", "20", "21", "22", "23\npid 42890 domain policy: first-touch mask: 0", "1"}
FAIL
FAIL    runtime 10.246s

Clear from the above its a test bug as I have 24 cores not 25.

This was previously raised here: https://github.com/golang/go/issues/25924


Problem using this library with the LSM9DS1 IMU.

Hello, I'm having a problem.
I'm trying to use the velocity example that you provide with the data from the LSM9DS1 IMU. I am gathering the accelerometer and gyroscope data thanks to the Arduino_LSM9DS1 Library, as it can be seen on the script below.
According to the documentation and my previous experimentations, the functions readAccelerometer and readGyroscope return float values for each axis, just like the imu data that you can get from the basicMPU6050.
I'm kind of disappointed, because the only thing I changed compared to your example is to put my values instead of yours in fusion.setup, fusion.update and fusion.updateVel functions, and I'm getting "nan nan nan" in the Serial Console for vel.x, vel.y and vel.z.
Moreover, I tried to send static values (i.e send {2.0,2.0,2.0} for gyro and {1.0,0.5,0.5} for accel), and it still gives the same result.
Do you have any ideas where the problem comes from ?
/*
 This sketch shows to integrate acceleration to obtain an estimate of velocity
*/

/* NOTE: The accelerometer MUST be calibrated to integrate its output. Adjust the
   AX, AY, AND AZ offsets until the sensor reads (0,0,0) at rest.
*/

#include <Arduino_LSM9DS1.h>
#include <accIntegral.h>

// ========== IMU sensor ==========

// Gyro settings:
#define         LP_FILTER   3           // Low pass filter.                    Value from 0 to 6
#define         GYRO_SENS   0           // Gyro sensitivity.                   Value from 0 to 3
#define         ACCEL_SENS  0           // Accelerometer sensitivity.          Value from 0 to 3
#define         ADDRESS_A0  LOW         // I2C address from state of A0 pin.   A0 -> GND : ADDRESS_A0 = LOW
                                        //                                     A0 -> 5v  : ADDRESS_A0 = HIGH
// Accelerometer offset:
constexpr int   AX_OFFSET =  552;       // Use these values to calibrate the accelerometer. The sensor should output 1.0g if held level.
constexpr int   AY_OFFSET = -241;       // These values are unlikely to be zero.
constexpr int   AZ_OFFSET = -3185;

// =========== Settings ===========
accIntegral fusion;
                                            //  Unit
constexpr float GRAVITY = 9.81e3;           //  mm/s^2          Magnitude of gravity at rest. Determines units of velocity.
constexpr float SD_ACC  = 200;              //  mm/s^2          Standard deviation of acceleration. Deviations from zero are suppressed.
constexpr float SD_VEL  = 200;              //  mm/s            Standard deviation of velocity. Deviations from target value are suppressed.
float acceleration[3];
float gyrom[3];

void setup() {
  Serial.begin(38400);

  if(!IMU.begin())
    while(1);
  if(IMU.accelerationAvailable()){
      IMU.readAcceleration(acceleration[0],acceleration[1],acceleration[2]);
      Serial.print("acceleration : ");
      Serial.println(acceleration[0]);
    }
  // initialize sensor fusion
  fusion.setup(acceleration[0],acceleration[1],acceleration[2]);   // set initial heading
  fusion.reset();                                 // zero velocity estimate
}

void loop() {
  /* NOTE: The heading must be updated along with the velocity estimate for accurate results.
           Use the following steps to ensure proper integration */

  // 1. measure state
  if(IMU.accelerationAvailable() && IMU.gyroscopeAvailable()){
      IMU.readAcceleration(acceleration[0],acceleration[1],acceleration[2]);
      IMU.readGyroscope(gyrom[0], gyrom[1], gyrom[2]);

  vec3_t accel = { acceleration[0],acceleration[1],acceleration[2] };
  vec3_t gyro = { gyrom[0], gyrom[1], gyrom[2] };

  // 2. update heading

  fusion.update( gyro, accel );

  // 3. update velocity estimate

  vec3_t vel_t = {0,0,0};   // Known measured velocity (target state)
  vec3_t vel;               // placeholder variable

  fusion.updateVel( gyro, accel, vel_t, SD_ACC, SD_VEL, GRAVITY );
  vel = fusion.getVel();

  // Display velocity components [view with serial plotter]

  Serial.print( vel.x);
  Serial.print( " " );
  Serial.print( vel.y);
  Serial.print( " " );
  Serial.print( vel.z);
  Serial.println();
  }
}

Hi Hyflow,
I suspect the problem lies in how you're updating the imu values. Mainly, the gyroscope and accelerometer measurements are not initialized to a particular value. Rather, they take on random values.
In theory, this clause
 // 1. measure state   if(IMU.accelerationAvailable() && IMU.gyroscopeAvailable()){       IMU.readAcceleration(acceleration[0],acceleration[1],acceleration[2]);       IMU.readGyroscope(gyrom[0], gyrom[1], gyrom[2]);
Should update the values before being sent to the Filter. However, right after the setup() function, this if statement is likely being ignored. Try modifying the variable declarations as such:
float acceleration[3] = {0,0,1}; float gyrom[3] = {0,0,0};

Hello RCmags,
I tried your idea but the problem does not seem to come from here. I tried to find exactly where in the code the problem happens, and from my experimentations I think the issue comes from the updateVel function
Indeed, by adding these prints to the accIntegral.cpp :

I am getting this.

That's why I think the problem might come from the rotate function from quaternion_type.cpp file.
As I said, even when i try to send a constant value for the acceleration, like {0,0,0}, this is still giving me "nan"  for the vel_local.x value after the Remove centrifugal step in accIntegral.cpp.

If the quaternion is messing up, it may be the heading is not being updated properly. The fusion.update() function may be at fault. This function expects the angular velocity to be in radians/second, and the acceleration to be normalized with respect to 1g (earth gravity). That being said:
Is the output of the acceleterometer normalized? ( (0,0,1) output at rest )
What units does the gyroscope output have?
Try checking the output of the heading using the "heading" example sketch. If its working, you should be able to read the pitch/yaw/roll angles of the sensor. If this fails, then we're closer to identifying the problem.

Any updates on this issue? How did it go?

Sorry, I didn't check your answer since Friday.
Indeed, I wasn't sending radian/second but degree/second. I changed that. For the acceleration, I am sending in g, but it can go up to +-4g, I don't know if this may cause a problem ?
According to your advices, I have modified the heading example with my inputs and it goes like this :
/*
 This sketch shows to rotate the heading (yaw angle) of the estimated orientation.
 */

#include <Arduino_LSM9DS1.h>
#include <imuFilter.h>

float acceleration[3] = {0,0,1};
float gyrom[3] = {0,0,0};
imuFilter fusion;

void setup() {
  Serial.begin(38400);

  if(!IMU.begin())
    while(1);
  if(IMU.accelerationAvailable()){
      IMU.readAcceleration(acceleration[0],acceleration[1],acceleration[2]);
      Serial.print("acceleration : ");
      Serial.println(acceleration[0]);
    }

  // Initialize filter:
  fusion.setup( acceleration[0], acceleration[1], acceleration[2] );

  // Rotate heading:
  float angle = 45 * DEG_TO_RAD;                // angle in radians to rotate heading about z-axis
  fusion.rotateHeading( angle, LARGE_ANGLE );   // Can choose LARGE_ANGLE or SMALL_ANGLE approximation
}

void loop() {
  // Update filter:
  if(IMU.accelerationAvailable() && IMU.gyroscopeAvailable()){
      IMU.readAcceleration(acceleration[0],acceleration[1],acceleration[2]);
      IMU.readGyroscope(gyrom[0], gyrom[1], gyrom[2]);

  vec3_t gyro;
  gyrom[0] *= DEG_TO_RAD;
  gyrom[1] *= DEG_TO_RAD;
  gyrom[2] *= DEG_TO_RAD;
  // 2. update heading
  fusion.update(  gyrom[0], gyrom[1], gyrom[2], acceleration[0],acceleration[1],acceleration[2] );

  // Display angles:
  Serial.print( fusion.pitch() );
  Serial.print( " " );
  Serial.print( fusion.yaw() );
  Serial.print( " " );
  Serial.print( fusion.roll() );
  Serial.println();
  }
}

However, I am still getting nan for the 3 angles...

Hi Hifrow,
Sorry for the delay in response. Unfortunately, I don't have an answer for your problem. Have you installed the latest version of "vector_datatype"? That's the only thing, code-wise, that I can think of that would cause the quaterion to fail.
Nonetheless, I was able to replicate your (nan,nan,nan) error using the Arduino IDE v1.8.19.

If I had two .ino files opened and I tried to upload the imuFilter code, the aformentioned error would appear.
Once I closed all .ino files, except for the imuFilter code, the error didn't seem to appear any more.

It seems the problem lay while uplaidng the sketch to the board. Try only opening an example, and check your board connection. Maybe this will resolve the problem?
Besides this, I've updated the library to make the accIntegral class simpler. It explicitly mentions the required gyroscope and accelerometer scales. Try upgrading the library. It may help.


I'm having a problem working with GPU in the node

Hi, this is an amazing node, so thank you for it!
I did all the steps in your YouTube video to install Cuda.
I'm having a problem working with GPU in the node, so it is very slow.
I have I9 computer with 32 ram, and RTX 4090.
this is the log error,
Error occurred when executing DeepFuzeAdavance:
C:\ComfyUI_windows_portable\ComfyUI\output\deepfuze\enhanced_17197493769867313.mp4 could not be loaded with cv.
File "C:\ComfyUI_windows_portable\ComfyUI\execution.py", line 151, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ComfyUI_windows_portable\ComfyUI\execution.py", line 81, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ComfyUI_windows_portable\ComfyUI\execution.py", line 74, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-DeepFuze\nodes.py", line 1425, in lipsyncgenerate
return load_video_cv(filename,0,'Disabled',512,512,0,0,1)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-DeepFuze\nodes.py", line 543, in load_video_cv
(width, height, fps, duration, total_frames, target_frame_time) = next(gen)
^^^^^^^^^
File "C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-DeepFuze\nodes.py", line 462, in cv_frame_generator
raise ValueError(f"{video} could not be loaded with cv.")

@gilushon With 4090 it should work so fast, this error you shared it seems it's a cuDNN and CUDA installation problem. Also for some reason when you install CUDA one of the dll files in the bin folder is missing. I added information in the error section on how you can fix that and there is a CV error section as well. If none of those worked follow the YouTube video for fully uninstalling your CUDA and reinstalling it.

Sam, Thanks for the quick reply, I did what you wrote about the fix under RERRORS and it solved the problem, now the rendering is 20 times faster.
(Missing zlibwapi.dll error: Search for NVIDIA zlibwapi.dll file, download it and copy it in C:\Program Files\NVIDIA (GPU Computing Toolkit\CUDA\v11.8\bin\zlibwapi.dll
Just one more thing, I think the frame enhancers don't work, because you don't see any differences between them, the same goes for the enhancer, you don't see any differences between them.

@gilushon i checked it, enhancers are working

Wow This is the ultimate Tool !!!!
very good quality, Thank you !!!
yaa enhancers are working, I initially missed the DeepFuze directory inside the output directory.

Just one suggestion, if there are already frame enhancers, maybe you could make an option to upscale using the frame enhancers where it would be possible to choose a resolution for upscaling?

@gilushon thabk you, there are already many great upscale nodes you can upscale before passing it to lipsync if you have a powerful GPU or upscale after lipsync.


[Polymer - build]: problem with minification

Hello,
I've been developing plugins for one system.
Project uses:
"dependencies": {
"@polymer/iron-icon": "^3.0.1",
"@polymer/iron-icons": "^3.0.1",
"@polymer/iron-pages": "^3.0.1",
"@polymer/paper-tabs": "^3.1.0",
"@vaadin/vaadin-button": "^2.2.0",
"@vaadin/vaadin-checkbox": "^2.2.8",
"@vaadin/vaadin-custom-field": "^1.0.1",
"@vaadin/vaadin-date-picker": "^3.3.4",
"@vaadin/vaadin-form-layout": "^2.1.2",
"@vaadin/vaadin-grid": "^5.4.2",
"@vaadin/vaadin-icons": "^4.3.1",
"@vaadin/vaadin-notification": "^1.4.0",
"@vaadin/vaadin-ordered-layout": "^1.1.0",
"@vaadin/vaadin-progress-bar": "^1.1.0",
"@vaadin/vaadin-radio-button": "^1.2.0",
"@vaadin/vaadin-select": "^2.1.1",
"@vaadin/vaadin-tabs": "^2.1.1",
"@vaadin/vaadin-text-field": "^2.4.3",
"@vaadin/vaadin-time-picker": "^1.2.2",
"@webcomponents/webcomponentsjs": "^2.2.10",
"lit-element": "^2.1.0",
"pwa-helpers": "^0.9.1",
"redux": "^4.0.1",
"redux-thunk": "^2.3.0"
}
I used polymer-cli for bundling and minifying
polymer build --js-minify --js-compile --js-transform-modules-to-amd --css-minify --html-minify --bundle
Last functional bundle was with version polymer-cli 1.9.6
New versions (1.9.7 - 9) seems to corrupt bundled file.
When I look at network activity I see that index.html is not minified in new versions
When I deploy plugin to the system I can see only this:
App name ``` iron-list must be given a `
My question is what is the main difference between those versions?
I looked at changelog and first change was treeshaking.
What else coulde be doing this?
Thanks for any hints.

I have the same issue but with a completely different setup etc.  Does not minify correctly.  I was using polymer-cli 1.9.10. Moved to version 1.9.6 and it now works correctly.


dummy commit - JUST FOR TESTING

Summary

The "what"; a concise description of each logical change
Another change

The "why", or other context.
Test plan
Issues

"THING-1234" or "Fixes #123"

Coverage increased (+29.7%) to 98.153% when pulling 42b3b2f9577904a7c0bc4fac53059280663274d0 on sohail/testpr into 138fff50735c390779654e45cb72d438b7e4fb08 on master.


Fleek installs outdated version

Using ujsut fleek and installing fleek via gum, will install fleek v0.9x instead of the latest which is 0.10x
The docs state to use
nix run "https://getfleek.dev/latest.tar.gz" -- init
...
<make edits to .fleek.yml>
...
nix run "https://getfleek.dev/latest.tar.gz" -- apply

I see where to edit this in ujust files, but was curious the implementation you would prefer. Until the user runs nix run "https://getfleek.dev/latest.tar.gz" -- apply fleek wont appear in their $PATH

@HikariKnight thanks!

@bayou-brogrammer np!


Function1: Calculating on "Close" price or "Adj Close" Price?

For function1 calculation, do we better calculate on "Close" price or "Adj Close" Price?

"The adjusted closing price will give you a better idea of the overall value of the stock and help you make informed decisions about buying and selling, while the closing stock price will tell you the exact cash value of a share of stock at the end of the trading day." By giving this definition, is it better to use "Adj Close" price?

I think we should use regular "Close" option, as it is not clear how they did that adjustment. At least with that example it makes more sense to use "Close", with price 225.24 similar to Open/High and Low prices with 225.0/225.8/223.88. While "Adj Close" price is equal 206.1, which is far off from actual trading price during that day


Update Helm to V2.0.1

In line with https://review.openstack.org/#/c/406681/, this commit updates Helm to V2.0.1

Yup, noticed this over the weekend too! Thanks @intlabs


Fixes STIX file loading and lightcurve/spectrogram plotting

This PR fixes the functions for handling STIX spectral and SRM files, this means that the Sunxspex class will be apple to correctly load in the new/final version of STIX files. I'm mainly fixing formatting of keywords and data, change of time units, format of energy bins and counts that are given in the spectral files. I have also fixed the incorrect plotting of background and event time-ranges for STIX lightcurves and spectrograms (examples in issue #97).

Hi @natsat0919.  Thanks so much for contributing this PR.  Your changes appear to be breaking a lot of the tests.  Can you clarify why this is?  Is it possible you've made changes in certain places that a valid for STIX but not other instruments?

Hi @DanRyanIrish, I'm missing a changelog file that I'll add. I'm still trying to figure out why the other tests are failing, but I think that it's because of changing the fundamental loading function for STIX files and the tests aren't adjusted to that? The changes that I have done were only for STIX (mainly in the stix_spec_code.py), so it shouldn't affect any other instruments

Following up on our discussion @natsat0919 and @KriSun95: I think we should be careful and explicit as to which kind of "spectrogram" we load in, i.e. the one produced by IDL or the one from the data center, and pay particular attention to the statistical errors that are being used. I would be happy to review some of the code in this PR and see how we can make sure that distinction is clear. Or we can merge the PR and do that later. Thoughts?

@settwi I agree, we definitely need to get that sorted as a priority. I think right now is the best to get this out as the basic working version (just updating what we already have) and build up from there? Also the STIX srm and spectral files tests are outdated (hence I got so many fails) so I'm sorting that out as well. Meanwhile, we can think what's the best way to implement the "distinction" when loading in?

Codecov Report
Patch coverage: 33.33% and project coverage change: +0.08 :tada:

Comparison is base (22935b5) 55.01% compared to head (a8b76b8) 55.10%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #98      +/-   ##
==========================================
+ Coverage   55.01%   55.10%   +0.08%
==========================================
  Files          19       19
  Lines        3119     3125       +6
==========================================
+ Hits         1716     1722       +6
  Misses       1403     1403

Impacted Files
Coverage Δ

sunxspex/sunxspex_fitting/instruments.py
21.84% <0.00%> (-0.06%)
:arrow_down:

sunxspex/sunxspex_fitting/stix_spec_code.py
23.80% <0.00%> (ø)

sunxspex/sunxspex_fitting/io.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

This looks great, thanks @natsat0919 ! I've had a play about with the changes and it works fine for me. Playing about with it has pointed out a small but important bug elsewhere (that I need to fixed) but I can't see anything wrong with this PR. I definitely agree with both you and @settwi about moving on to think about how to separate loading in of the two STIX file formats for the spectra (data centre and IDL).
I'll return to this later today, add a review for the PR, and then move on to approve the merge if there are no other objections.

Do we consider this PR ready to merge?

@DanRyanIrish yes, I just did some very minor refactoring changes after Kris's approval. It should be ready to merge


Plugin not working on Android 4.1.2

Hello,
The plugin not working on Android 4.1.2.
Any suggestion ?
Thank you.

Not enough info. Be more specific, it is working for you in another versions? What did you try to?

First of all, thank you for your quick response.
The plugin working fine on Android 5.0 or above.
The first function "window.DatecsPrinter.listBluetoothDevices" lock the APP.
My code:
    window.DatecsPrinter.listBluetoothDevices(
        function(devices) {

            var impresora = 0;
            var bt = 0;
            for (var i = 0; i < devices.length; i++) {
                if ( devices[i].name.includes("DPP-") ) {
                    bt = i;
                    impresora = 1;
                    break;
                } else if ( devices[i].name.includes("Printer") ) {
                    bt = i;
                    impresora = 1;
                    break;
                }
            }

            if ( impresora == 0 ) {
                app.showNotification('Impresora no reconocida');
            }

            app.showNotification(devices[bt].name);

            window.DatecsPrinter.connect(devices[bt].address,
                function() {

                    window.DatecsPrinter.printText('Hola mundo', 'ISO-8859-1');
                    window.DatecsPrinter.disconnect();
                },
                function() {
                    alert(JSON.stringify(error));
                }
            );
            window.DatecsPrinter.disconnect();
        },
        function(error) {
            alert('error');
            alert(JSON.stringify(error));
        }
    );

Lets break this into baby steps to easily find what's going wrong. I guess it's some implementation issue, cuz I can confirm it's working well on Android 4.1.2.
First of all, be sure to only have the printer paired to the device.
Then try the example, I'll paste here:
window.DatecsPrinter.listBluetoothDevices(
  function (devices) {
    window.DatecsPrinter.connect(devices[0].address,
      function() {
        alert("connected!");
        printSomeTestText();
      },
      function() {
        alert(JSON.stringify(error));
      }
    );
  },
  function (error) {
    alert(JSON.stringify(error));
  }
);

function printSomeTestText() {
  window.DatecsPrinter.printText("Print Test!", 'ISO-8859-1',
    function() {
      alert("finished!");
    }
  );
}

I'm very sorry, but not working.
If the bluetooth is down, it show the error correctly.
However, if the bluetooth is up does nothing.
Thank you.

what is the return of listBluetoothDevices?

Nothing, no error too.
Tomorrow I will prove with other mobile.

Closing due to long time without activity.


Fix jsdocs and regenerate scaffolding.

@lukesneeringer I'm not sure how the streamingRecognize unit tests ever worked...

@jmdobry They did work before; these test failures are due to breaking changes in autogen .

Codecov Report

Merging #3 into repo-migration will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           repo-migration     #3   +/-   ##
=============================================
  Coverage             100%   100%
=============================================
  Files                   2      2
  Lines                  28     28
=============================================
  Hits                   28     28

Impacted Files
Coverage Δ

src/index.js
100% <ø> (ø)
:arrow_up:

src/helpers.js
100% <ø> (ø)
:arrow_up:

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 fca1548...8932e6b. Read the comment docs.


Avatar Picture Upload State on Developer Profile Create fails to update

Describe the bug
When creating a new developer account, if you attempt to save the profile without an avatar picture the form raises an error as expected. Subsequently when an avatar is uploaded, the 'avatar required' error sticks around on the page persisting through a successful upload, and the avatar circle (place where I would've expected an avatar preview) does not populate with a preview.
To reproduce
Steps to reproduce the behavior:

Go to 'https://www.railsdevs.com'
Click on 'Register', follow confirmation steps (click email confirmation link)
'What brings you to railsdevs' click: 'Im looking for work'
Fill in all required profile information, omit an avatar/profile picture
Click 'save'
Raises prohibited this developer from saved: 'Avatar is not attached'
Upload a photo

Expected behavior
Error from top of page should change to a confirmation 'Avatar uploaded', and button too 'Upload is attached'. Possibly with 'success' green coloring
Actual behavior
Raised error remains at top of page, and avatar button remains 'Upload is not attached' while saving profile is not possible as avatar has been uploaded
Screenshots

I can also reproduce when trying to upload an image before submitting an invalid form.


[shapes] DrawLineBezier*() functions redesign

As stated in this PR (https://github.com/raysan5/raylib/pull/2721), DrawLineBezier*() functions do not work as expected, leaving gaps between the lines. Solution requires a redesign of those functions using triangle-strip.
DrawLineBezier() calls DrawLineEx() that calls DrawTriangleStrip(). That last function should actually manage the gaps properly because it's a triangle strip.
DrawLineBezier() shouldn't use DrawLineEx() to draw the bezier in small pieces but use a custom DrawTriangleStrip() implementation to define all required points while drawing the triangles connection between them.

hi @raysan5, please assign me this issue, I would can work on this.

@kgpjay You got the issue assigned, feel free to ask if you have any question. Remember to check the required coding conventions.

@kgpjay Are you working on this feature? Any progress?

Working on this, will open a pull request soon.


MudOverlay doesn't show in MainLayout and App.razor

Bug type
Component
Component name
MudOverlay
What happened?
I have this MudOverlay :
<MudOverlay ZIndex="9999" DarkBackground="true" AutoClose="false">
    <MudProgressCircular Color="Color.Secondary" Indeterminate="true" Size="Size.Large" />
</MudOverlay>

It shows in page components but doesn't show in MainLayout and App.razor files!
I tested MudProgressCircular component alone in MainLayout and App.razor files and it shows.
<MudProgressCircular Color="Color.Secondary" Indeterminate="true" Size="Size.Large" />
But MudOverlay doesn't show in MainLayout and App.razor files!
Also I tested Absolute="true" also with @bind-Visible="showOverlay" and set showOverlay=true from page components but it not show.
Expected behavior
Showing MudOverlay on MainLayout and App.razor files.
Reproduction link
fsd
Reproduction steps

Add a MudOverlay component to MainLayout and App.razor files.
When start the project we see MudOverlay on pages.

Relevant log output
No response
Version (bug)
6.9.0
Version (working)
https://mudblazor.com/components/overlay
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Oh this worked in MainLayout :
`<MudDrawer @bind-Open="drawerOpen" Elevation="2">

JobClock.ir

<MudOverlay @bind-Visible="showOverlay" DarkBackground="true" AutoClose="false">

public bool showOverlay = false;
public void ShowOverlay()
{
showOverlay = true;
StateHasChanged();
}
public void HideOverlay()
{
showOverlay = false;
StateHasChanged();
}`


Missing one key point on authorization spec

Guys, I'm thinking tha we are loosing one key point in authorization/authentication spec.
There is a need for a client identification. My backend should have to register some client to be able to allow connections.
So we would need first generate a new client entitiy, give it a credentials, say what api(services) it will be allowed to call.

This is not necessary. See: https://jwt.io/introduction/


Remove NLP sumarizing

The user should create its own summary - and not a NLP program.
Todo:

Add a summary text area
Upon completion of the description text area, the summary text area gets populated with the n first characters of the content.

Working that way could also avoid having enconding issue (as below)
LookupError at /petition/add
NLTK tokenizers are missing or the language is not supported.
Download them by following command:
python -c "import nltk; nltk.download('punkt')"
Original error was:
**********************************************************************
Resource [93mpunkt[0m not found.
Please use the NLTK Downloader to obtain the resource:
>>> import nltk
>>> nltk.download('punkt')
For more information see: https://www.nltk.org/data.html

This module was not so amazing at the end... Closing this issue without implementing this NLP idea.


Update manual for Deno 1.13

Closes https://github.com/denoland/manual/issues/46
Closes https://github.com/denoland/manual/issues/47

It'd be great to merge some of the prose in the 1.13 release notes into the manual somehow...

@bartlomieju the onerror event has the following signature:
onerror: ((this: WebSocket, ev: Event | ErrorEvent) => any) | null;

So this still is marked as an error console.log('socket errored:', e.message);
Is this going to be like that in the future? Since if yes I think the docs could be like this:
  socket.onerror = (e) => {
    if (e instanceof ErrorEvent) {
      console.log('socket errored:', e.message);
    } else {
      console.log(e.type);
    }
  };

Thanks!


Fix full-screen triangle winding.

If you tried removing culling: none from any post-process materials, everything would be fine in GL but Vulkan would be black because our full-screen triangle was back-facing.
Usually this did not manifest as a discernible issue because tex coords were also flipped.

Hey @bejado, this morning I've updated the PR and it seems to work for Metal but can you double-check for me using the XCode inspector?  To recap:

Metal and OpenGL define clip space as Y-up whereas Vulkan is Y-down.
Metal and Vulkan define tex coords as Y-down whereas OpenGL is Y-up.

Each of these bullets are now expressed as separate #if conditionals in the common postprocess code, and I've tried to simplify all the custom post-process materials accordingly.
The job of the common postprocess code is to compute normalized tex coords (uv), non-normalized tex coords (texCoords), and of course the clip-space coordinate of the vertex (gl_Position).  Users should not make assumptions about how these values are related since it varies according to backend.

@prideout Metal backend looks good 👍 Verified SSAO, tonemapping, and FXAA.

To help make this less confusing, I've renamed the uv field to normalizedUV, but I think I'll let Ben make follow-up PR's if we decide we'd like to clean this up even further.


[Feature Request] Add SwinV2 model

Nice extension, any chance to add swinv2 model?
https://huggingface.co/SmilingWolf/wd-v1-4-swinv2-tagger-v2

@unLomTrois I just made two Pull Requests with the necessary code changes to add the swinv2 tagger model. You can check those requests to see the changes if you want to make them yourself. They're super short and simple.


Fixed the Issue in  utis.py

1 st changes

In this modified code, the spacy_version variable is used to store the version of the SpaCy library. Inside the loop, the code checks whether the SpaCy version is 3.0.0 or higher. If it is, the lemmatization is performed using nlp(combined_texts). Otherwise, for older versions, the lemmatization is done using nlp.tokenizer(combined_texts). This change accounts for potential differences in SpaCy versions and ensures the code can handle them correctly.
#47 Edit spaCy

2 nd changes

Replaced the string slicing with data.endswith() to check the file extension.
Changed the error message to be raised as an exception using raise ValueError() instead of just calling ValueError().
Fixed the indentation of the function and the if-elif-else blocks.

hey @andrewtavis   Thank you noticing the error I will look into it
If any help needed to me I will ask you

Thank you for checking this, @cicada0007!

@cicada0007, can you let me know what your Python version is? I'm thinking I'm going to update kwx to run on 3.8+ :) I might just merge this as the changes do look good, and then I can figure out the conflicts on my end.

@andrewtavis My current version of python is Python 3.11.3

Thank you, @cicada0007. I figured as much :) kwx being written years ago, it makes sense that some changes seem like they’re working for you and then aren’t in the repo. I’ll again merge this at some point soon. This is that the SpaCy check wouldn’t be needed as we’ll only be above a certain version 🤔 I’ll verify this and remove the check if so 😊

If you have interest in helping with the update, then let me know. I did an initial update and test last night and to be expected there are lots of failures, but it seems like what needs to happen is that the test targets needs to be switched as the random seeds are behaving differently at this point :) So when we’re checking that the output of a model is a list of strings that’s always the same, it now needs to be different ones given that the random number generator behind the output is slightly different :)

Ya that's correct

Ok, @cicada0007 :) As I said I’ll merge this soon and try to figure the updates out.


OCPBUGS-37645: Set right endpointSlice port

What this PR does / why we need it:
Manual cherry-pick of https://github.com/openshift/hypershift/pull/4422
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/jira refresh

/lgtm

/jira refresh

/override e2e-azure
/jira refresh

/override ci/prow/e2e-azure
/jira refresh

/jira refresh

/cherry-pick release-4.15


Request - add instructions to connect veth interfaces to p4-ovs

For the recipe in https://github.com/ipdk-io/ipdk/blob/main/build/IPDK_Container/README, can you provide an alternative which does not require spawning VMs? For example, I'd like to instantiate some veth pairs on the host to connect to the switch, similar to how one would connect to bmv2 or tofino-model. That way I can connect SW packet generators running on the host, to the switch. Is this possible? Thanks.

Yeah, the possibility of using veth pairs would be really nice for testing and poc's !!

For now we are supporting vhost-user port type, on further enhancement we are going to support TAP ports. This will enable users to run end-to-end scenarios.


Fix C and C++ keywords with underscores collisions

Convert to snake case before checking for keyword collisions, so C and C++ keywords with underscores will be “escaped” as well.

@primoly maybe it makes sense to add this commit and the one from #941 to https://github.com/bytecodealliance/wit-bindgen/pull/942 which now has the wit for the generator test to catch these scenarios?


Added UTM links

This task is in regards to the Communication team:

Normal links were replaced by UTM links in terms of tracking.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


menu items reload

Hi,
I want to use MenuView in my App.
There's one question: can I set menu.items to be blank, and reset it after I got the items content from the server?  Because I do not know the items's count or content.
Thanks a lot.

@TangSirOnGit that should be possible.
Simply pass an empty array!


Discussion section not visible for the repo

Discussion
@krumutsav I don't see the discussion section for this repo , can you enable it?.

Enabled now


garage: 0.8.2 -> 0.8.3

Description of changes
https://git.deuxfleurs.fr/Deuxfleurs/garage/releases/tag/v0.8.3
https://garagehq.deuxfleurs.fr/documentation/operations/upgrading/

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Technically unreleated to this bump, but pretty important nonetheless:
Could you please fix the typo in https://github.com/NixOS/nixpkgs/blob/812ccee781267a7bcf8cbd04268e1664b31a5cd6/pkgs/tools/filesystems/garage/default.nix#L60
so ofborg runs the VM tests without manually writing
@ofborg test garage
diff --git a/pkgs/tools/filesystems/garage/default.nix b/pkgs/tools/filesystems/garage/default.nix
index b6dbe2e89846..df47d8194377 100644
--- a/pkgs/tools/filesystems/garage/default.nix
+++ b/pkgs/tools/filesystems/garage/default.nix
@@ -57,7 +57,7 @@ let
       "sqlite"
     ]);

-    passthru = nixosTests.garage;
+    passthru.tests = nixosTests.garage;

     meta = {
       description = "S3-compatible object store for small self-hosted geo-distributed deployments";

Thanks!

so ofborg runs the VM tests without manually writing

Yes, definitely. Added another commit with this fix.

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

1 package built:

garage (garage_0_8 ,garage_0_8_2)

Result of nixpkgs-review pr 252142 run on aarch64-linux 1

1 package built:

garage (garage_0_8 ,garage_0_8_2)

Result of nixpkgs-review pr 252142 run on x86_64-darwin 1

1 package built:

garage (garage_0_8 ,garage_0_8_2)

Result of nixpkgs-review pr 252142 run on aarch64-darwin 1

1 package built:

garage (garage_0_8 ,garage_0_8_2)

I’ve successfully upgraded my three node cluster to this version.

@0x4A6F any particular reason you would like to wait until ~RaitoBezarius is back from vacation?

None :p ! (From a boat)
Please proceed if you deem it appropriate
Le mer. 30 août 2023 à 16:17, Emily @.***> a écrit :

@0x4A6F https://github.com/0x4A6F any particular reason you would like
to wait until ~RaitoBezarius is back from vacation?
—
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/252142#issuecomment-1699152734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACMZRAUEKUXLKLULPI25QLXX44O3ANCNFSM6AAAAAA4C6FNM4
.
You are receiving this because your review was requested.Message ID:
@.***>

I verified that x86_64-darwin does actually build successfully. There are clearly no nixosTests for darwin, and looking at the aarch64-darwin build it just runs the basic package build.
It took 27 minutes to run the emulation compile on my M1 machine. The check for this PR timed out at an hour, so maybe it was just resource contention?

hydra.nixos.org managed to build garage 0.8.3 on x86_64-darwin as well, so all good
https://hydra.nixos.org/build/233596067


Cannot compile with CUDA 7.5

CPU miner compiles fine, but fails after enabling:
features=["build-cuda-plugins"]
Compiling with nvcc V7.5.17
Sample errors:
/usr/lib/gcc/x86_64-linux-gnu/5/include/mwaitxintrin.h(36): error: identifier "__builtin_ia32_monitorx" is undefined /usr/lib/gcc/x86_64-linux-gnu/5/include/mwaitxintrin.h(42): error: identifier "__builtin_ia32_mwaitx" is undefined /usr/lib/gcc/x86_64-linux-gnu/5/include/mwaitxintrin.h(36): error: identifier "__builtin_ia32_monitorx" is undefined /usr/lib/gcc/x86_64-linux-gnu/5/include/mwaitxintrin.h(42): error: identifier "__builtin_ia32_mwaitx" is undefined ~/.cargo/git/checkouts/cuckoo-miner-4752934f0f1f2bfe/2a08251/src/cuckoo_sys/plugins/cuckoo/src/mean_miner.cu(361): error: Within a __device__/__global__ function, only __shared__ variables may be marked "static"

Also having issues building with CUDA 7.5

Cuda 7.5 is very old at this stage, is it the default on some distro?
I need to create proper documentation for this package, but it's only ever targeted Cuda 9+. Could you try upgrading and try again?

Hi @bedbc and @emmick4, I had a related C or CXX related issues. You would have the same error by using either gcc-5 or gcc-6. I solved most of these issues by upgrading from gcc-6 to gcc-7.  Give it a try and let me know if it solves your problem.
Had to do a lot of "update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-7 "  to make sure all C and CXX compilers are set to gcc-7. Things like c++, cc, and the likes weren't what they were supposed to be for reasons I can't recall on my machine.


chore: sets owner as a Controller OwnerReference

Description
Add SetControllerReference function to manage controller resources.
This is used for garbage collection of the controlled object and for reconciling the owner object.
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/lgtm

/lgtm
/approve

/retest

/retest

/retest-required

/retest-required

/retest-required


zipkin mysql request timeout

Hey guys
i`ve met a problem when i trying get data from the zipkin-ui.
Cannot store spans [34ddfb7bbb28d076.67dc18dedd736d2b<:67dc18dedd736d2b] due to RuntimeException(java.sql.SQLTransientConnectionException: HikariPool-1 - Connection is not available, request timed out after 30003ms.)
The logs are below：
zipkin.log

You have hit max connections. You might want to monitor zipkin metrics and
possibly increase the connection count above 10
https://github.com/openzipkin/zipkin/blob/master/zipkin-server/README.md if
this is a volume problem you might habe a separate instance for query vs
collector (same code)
Hard to tell what the connections are doing. Some normal practice such as
thread dumps could tell. Check to see if messages are being dropped or
threads are hung. Restart is likely needed.
On 26 Sep 2017 09:26, "eshujiushiwo"<EMAIL_ADDRESS>wrote:

Hey guys
i`ve met a problem when i trying get data from the zipkin-ui.
Cannot store spans [34ddfb7bbb28d076.67dc18dedd736d2b<:67dc18dedd736d2b]
due to RuntimeException(java.sql.SQLTransientConnectionException:
HikariPool-1 - Connection is not available, request timed out after
30003ms.)
The logs are below：
zipkin.log https://github.com/openzipkin/zipkin/files/1331397/zipkin.log
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/issues/1746, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD61_uszC9HXHuOmKircNRt8JtFhAu9ks5smFLNgaJpZM4PjjKh
.

@adriancole
Thats very kind of you. Thanks a lot, ive set MYSQL_MAX_CONNECTIONS to 30.
But still shows that problem .
Should i change the storage to cassandra?

maybe hop onto https://gitter.im/openzipkin/zipkin ?
there are different storage options like cassandra and elasticsearch, and
choices depend on what you are comfortable running. You can try different
ones but you also might need to look at your ingest rate and if you need to
adjust sampling etc.

question answered, mysql is not for high volume sites.


Move algorithms to tag_fallback_invoke

Our experience with disentangling the segemented algorithms by the help of utilizing tag_invoke style overloads has shown that it is helpful to move all existing tag_invoke implementations of the parallel algorithms to use tag_fallback_invoke instead. This allows to use SFINAE on the overloads provided by external specializations.
Here is a list of algorithms that need to be adapted:

[x] adjacent_find (#5201)
[x] all_of any_of none_of (#5224)
[x] copy (#5203)
[ ] copy_if copy_n
[ ] move
[x] count count_if (#5220)
[ ] equal mismatch
[ ] reduce
[x] transform (#5256)
[ ] fill fill_n
[ ] find find_end find_first_of find_if find_if_not
[x] for_each (#5176)
[x] for_each_n (#5212)
[ ] for_loop for_loop_n for_loop_strided for_loop_strided_n
[ ] generate generate_n
[ ] is_heap is_heap_until
[ ] is_partitioned
[ ] is_sorted is_sorted_until
[ ] make_heap
[ ] remove remove_if
[ ] remove_copy remove_copy_if
[ ] search search_n
[ ] set_difference set_intersection set_symmetric_difference set_union includes
[ ] inplace_merge
[ ] merge
[ ] destroy destroy_n
[x] transform_reduce (#5260)

I will try to start working on copy_if copy_n


Bug with YAML handling 2D arrays

Hello! I imported your library to allow for YAML parsing and it's been super helpful so far :)
I noticed a bug while trying to parse 2D arrays
Input YAML
-
  - 1
  - 2
-
  - 3
  - 4

Expected Output
[
  [1,2],
  [3,4]
]

Actual Output
[
 [ 1 ],
 2,
 [ 3 ],
 4
]

I confirmed this using both tabs and spaces
I was also used these source to verify the expected output
https://www.json2yaml.com/
https://stackoverflow.com/a/49725357
I worked around by using the following format, but I figured I'd report it anyway
- - 1
  - 2
- - 3
  - 4

Thanks for reporting this. I'm off on holiday today so won't have a chance to look at this until next week.

https://github.com/JujuAdams/SNAP/commit/4736cff3b512b2288fdc7b9eb1acdf4d0cdd7a05


EISDIR: illegal operation on a directory, read by right-clicking package.xml

Summary
I just got this error, when I retrieve metadata to production org from VSCode(1.32.3) with salesforce cli ver 7.1.4.
Steps To Reproduce:

Triggered the retrieve by right-clicking package.xml

Expected result
Retrieve meta data
Actual result
EISDIR: illegal operation on a directory, read.
Additional information
{"name":"sfdx","hostname":"winmyintoo.local","pid":29085,"level":50,"msg":"EISDIR: illegal operation on a directory, read.","time":"2019-04-01T00:47:58.715Z","v":0}
{"name":"sfdx","hostname":"winmyintoo.local","pid":29085,"level":50,"msg":"[ false,\n  '{\"message\":\"EISDIR: illegal operation on a directory, read\",\"status\":1,\"stack\":\"Error: EISDIR: illegal operation on a directory, read\\\\n    at Object.fs.readSync (fs.js:600:3)\\\\n    at Object.readSync (/Users/owin/.nodebrew/node/v10.0.0/lib/node_modules/sfdx-cli/node_modules/graceful-fs/polyfills.js:137:28)\\\\n    at tryReadSync (fs.js:440:20)\\\\n    at Object.fs.readFileSync (fs.js:481:19)\\\\n    at AggregateSourceElement.decomposeMetadata (/Users/owin/.nodebrew/node/v10.0.0/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/lib/aggregateSourceElement.js:471:39)\\\\n    at AggregateSourceElement.commitMetadata (/Users/owin/.nodebrew/node/v10.0.0/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/lib/aggregateSourceElement.js:312:87)\\\\n    at AggregateSourceElement.commit (/Users/owin/.nodebrew/node/v10.0.0/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/lib/aggregateSourceElement.js:279:18)\\\\n    at sourceElements.forEach.sourceElement (/Users/owin/.nodebrew/node/v10.0.0/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/lib/sourceWorkspaceAdapter.js:193:103)\\\\n    at Map.forEach (<anonymous>)\\\\n    at SourceWorkspaceAdapter.updateSource (/Users/owin/.nodebrew/node/v10.0.0/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/lib/sourceWorkspaceAdapter.js:192:20)\\\\n    at MdapiPullApi._syncDownSource (/Users/owin/.nodebrew/node/v10.0.0/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/lib/mdapiPullApi.js:137:20)\\\\n    at SourceRetrieve.<anonymous> (/Users/owin/.nodebrew/node/v10.0.0/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/lib/source/sourceRetrieve.js:111:56)\\\\n    at Generator.next (<anonymous>)\\\\n    at fulfilled (/Users/owin/.nodebrew/node/v10.0.0/lib/node_modules/sfdx-cli/node_modules/salesforce-alm/dist/lib/source/sourceRetrieve.js:10:58)\",\"name\":\"Error\",\"warnings\":[]}' ]","time":"2019-04-01T00:47:58.717Z","v":0}

SFDX CLI Version(to find the version of the CLI engine run sfdx --version):
7.1.4
SFDX plugin Version(to find the version of the CLI plugin run sfdx plugins --core)
@oclif/plugin-commands 1.2.2 (core)
@oclif/plugin-help 2.1.6 (core)
@oclif/plugin-not-found 1.2.2 (core)
@oclif/plugin-plugins 1.7.8 (core)
@oclif/plugin-update 1.3.9 (core)
@oclif/plugin-warn-if-update-available 1.7.0 (core)
@oclif/plugin-which 1.0.3 (core)
@salesforce/sfdx-trust 3.0.2 (core)
analytics 1.1.2 (core)
generator 1.1.0 (core)
salesforcedx 45.8.2 (core)
├─ force-language-services 45.9.0 (core)
└─ salesforce-alm 45.11.0 (core)

OS and version:
macOS High Sierra ver: 10.13.6

@k010c1232, can you check the permissions? are you using SUDO? I think your user that is logged into CLI does not have permissions. See: https://npm.community/t/global-installs-sudo-npm-i-g-fail-on-mac-after-6-5-upgrade-works-fine-after-6-4-1-downgrade/4082


39 add discord auth integration

Codecov Report

Merging #42 (bca823e) into release-candidate (dabf694) will not change coverage.
The diff coverage is 100.00%.

@@                 Coverage Diff                 @@
##           release-candidate       #42   +/-   ##
===================================================
  Coverage             100.00%   100.00%
===================================================
  Files                     30        37    +7
  Lines                    198       242   +44
  Branches                  17        21    +4
===================================================
+ Hits                     198       242   +44

Impacted Files
Coverage Δ

src/discord/discord-auth.constants.ts
100.00% <100.00%> (ø)

src/discord/discord-auth.controller.ts
100.00% <100.00%> (ø)

src/discord/discord-auth.guard.ts
100.00% <100.00%> (ø)

src/discord/discord-auth.module.ts
100.00% <100.00%> (ø)

src/discord/discord-auth.routes-resolver.ts
100.00% <100.00%> (ø)

src/discord/discord-auth.strategy.ts
100.00% <100.00%> (ø)

src/discord/specs/mock.discord-auth.service.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add support to spatie/crawler v7

Hello @insign
Thank you for your PR. Have you tested this further? Are you using the package in production?
Cheers,
Peter

Sure. Worked fine to me.


"Airspeed sensor failure detected (1, -1)" after landing

When running the PX4 JSBSim bridge via make px4_sitl jsbsim on Ubuntu 18.04 and with QGroundControl connected, I'm seeing the PX4 console output  a poll timeout 0, 22 error after completing a mission and landing, as well as a Airspeed sensor failure detected (1, -1) dialog inside QGroundControl. The view inside of FlightGear goes black and PX4 is unresponsive at this point and needs a restart before uploading another mission.

@zimmy87 Thanks for reporting, I will have a look

@zimmy87 Turns out this is not something coming from the simulator, but was a real issue on the flight controller side.
This was reported in https://github.com/PX4/PX4-Autopilot/issues/18437 and fixed by https://github.com/PX4/PX4-Autopilot/pull/18442
Can you confirm that the issue has also been fixed?

Even i was facing same issue. I guess the reason was bcz of steady airspeed sensor value getting published. PX4 considers this as a sensor fault. I have resolved it by changing the value of wait time from 1 second to 10second in /src/modules/airspeed_selector/airspeed_selector_main.cpp
Try and let me know.

@rrajnidhi Are you sure your firmware includes the fix in https://github.com/PX4/PX4-Autopilot/pull/18442 ?


feat: ZeroMorph

Adds ZeroMorph functionality (without zk) required to prove and verify batched multilinear evaluation claims, including efficient handling of shifts. For now, the protocol is fully laid out in a test. Incorporation into honk will be done in a separate PR.
The best resource for understanding this PR is this companion doc. The full batched protocol description therein matches the implementation.
Note: The goal of this first pass was to make the implementations as clear as possible. There are many simple optimizations that could be introduced to make things more efficient.
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

This looks great! but the doc linked in the comment and PR (ref) isn't public 😢

This looks great! but the doc linked in the comment and PR (ref) isn't public 😢

Working to make it public - will keep you posted!

@huitseeker Should be public now


go.mod: update golang.com/x/net package

What
This PR updates the golang.com/x/net package and related.
Why
The update is because of security alerts from dependabot.
https://github.com/stellar/stellar-disbursement-platform-backend/security/dependabot/1
https://github.com/stellar/stellar-disbursement-platform-backend/security/dependabot/2
Known limitations
N/A
Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).
[x] This PR's title starts with the name of the package, area, or subject affected by the change.

Thoroughness

[ ] This PR adds tests for the new functionality or fixes.
[ ] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[x] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[x] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[x] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[x] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[x] This is not a breaking change.
[x] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr98.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr98.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr98.previews.kube001.services.stellar-ops.com


Missing license

Hi, I noticed that this repo doesn't have a license.
I'm trying to include your library in a project of mine but I need to know if it's oss.
Thanks!

I've added the LICENSE, please go ahead and use it.


Replace green-drain map symbol with a blue-wave map symbol

Objective:

Integrate the symbolism of new adopt-a-drain logo into the map symbolism

Strategy:

replace the tosewer.png with a new symbol.
Keep the shape,
keep the shadow,
change green color to blue,
replace the round drain with the round blue waves from the adopt-a-drain.png.
keep the name tosewer.png.

Review Instructions

look at map, find blue map symbol with wave logo.

If found and is satisfactory then move issue #84 to Done in waffle.io.


我遇到了import whisper这个问题，怎么解决

已经按照教程下载环境和模型
D:\anaconda\envs\cosyvoice\python.exe C:\Users\zyg\Desktop\CosyVoice_For_Windows-main\webui.py
2024-08-05 16:21:07,422 - modelscope - INFO - PyTorch version 2.0.1+cu118 Found.
2024-08-05 16:21:07,424 - modelscope - INFO - Loading ast index from C:\Users\zyg.cache\modelscope\ast_indexer
2024-08-05 16:21:07,545 - modelscope - INFO - Loading done! Current index file version is 1.15.0, with md5 1e11f8386aec41c41543daa60d3f8d92 and a total number of 980 components indexed
Traceback (most recent call last):
File "C:\Users\zyg\Desktop\CosyVoice_For_Windows-main\webui.py", line 41, in
from cosyvoice.cli.cosyvoice import CosyVoice
File "C:\Users\zyg\Desktop\CosyVoice_For_Windows-main\cosyvoice\cli\cosyvoice.py", line 20, in
from cosyvoice.cli.frontend import CosyVoiceFrontEnd
File "C:\Users\zyg\Desktop\CosyVoice_For_Windows-main\cosyvoice\cli\frontend.py", line 18, in
import whisper
File "D:\anaconda\envs\cosyvoice\lib\site-packages\whisper.py", line 69, in
libc = ctypes.CDLL(libc_name)
File "D:\anaconda\envs\cosyvoice\lib\ctypes_init_.py", line 363, in init
if '/' in name or '\' in name:
TypeError: argument of type 'NoneType' is not iterable

never meet this, install environment again exactly according to readme

我也遇到这个问题，请问怎么解决的呃？


[fix](jdbc catalog) Fix incorrect format of FullDriverUrl under Windows

When debugging in a Windows environment, the jdbc_drivers_dir format is D:\xxx\xxx.
The getFullDriverUrl() will return file://D:\xxx\xxx,

which does not comply with URI format specifications and will prevent the JDBC jar from being loaded in Windows.
The correct FullDriverUrl format should be file:///D:\xxx\xxx.

run buildall

Could you please help review this? Thank you all. @morrySnow @xiaokang @dataroaring


Update peerDependency for Grunt

Attempting to update dependencies to enable Greenkeeper on our project, we get this error:
$ npm install
npm ERR! node v4.8.7
npm ERR! npm  v2.15.11
npm ERR! code EPEERINVALID

npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>
Could you update the peerDependencies section of your package.json tagged with
"grunt": ">=0.4.0" to allow for a more recent version of Grunt?
See Grunt-1.0.0 release notes concerning their peer dependency recommendation.

@mazerte what's the delay for this 30 sec change?


Refs already exist

On the project i'm working on i get the following exception:
2018-11-27T14:53:25.4275277Z Updating'Microsoft.AspNetCore' from 2.1.0, 2.0.3, 2.1.2, 2.1.4 to 2.1.6 in 10 projects 2018-11-27T14:53:25.4300247Z Updates failed NameConflictException : failed to write reference 'refs/heads/nukeeper-update-Microsoft.AspNetCore-to-2.1.6': a reference with that name already exists.
Might be related to: https://github.com/GitTools/GitVersion/issues/864 but i'm not sure yet.

The workaround is usually to find the existing branch in git, and merge or delete it.
If there's a ref but not a branch then that's something new.

That's what I thought initially, only the branch is already deleted. Maybe because of multiple project with the same package upgrade?

That's odd. I'm not a git expert, I don't know how a /refs/head/foo differs from a branch called foo.
I thought that they were the same

Strange i don't have it on my testscase at: https://github.com/MarcBruins/nukeeperrefissue

Issue is solved magically? Maybe ADO still had some reference to the branch somewhere after all. Closing it for now

I'm getting similar error when using NuKeeper extension for Azure Devops.
Here's a bit censored tail of detailed logs:
Nuget restore on <my solution> complete
Updating 'Newtonsoft.Json' from from <...> to <correct version> in <...> projects
Git checkout 'origin/feature/NuKeeper'
Using branch name: 'nukeeper-update-Newtonsoft.Json-to-<correct version>'
Git checkout new branch 'nukeeper-update-Newtonsoft.Json-to-<correct version>'
Updates failed NameConflictException : failed to write reference 'refs/heads/nukeeper-update-Newtonsoft.Json-to-<correct version>': a reference with that name already exists.
   at LibGit2Sharp.Core.Ensure.HandleError(Int32 result)
   at LibGit2Sharp.Core.Proxy.git_branch_create_from_annotated(RepositoryHandle repo, String branch_name, String targetIdentifier, Boolean force)
   at LibGit2Sharp.BranchCollection.Add(String name, String committish, Boolean allowOverwrite)
   at NuKeeper.Git.LibGit2SharpDriver.<>c__DisplayClass13_0.<CheckoutNewBranch>b__0() in <...>/NuKeeper.Git/LibGit2SharpDriver.cs:line 123
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot)
--- End of stack trace from previous location where exception was thrown ---
   at NuKeeper.Engine.Packages.PackageUpdater.MakeUpdatePullRequests(IGitDriver git, RepositoryData repository, NuGetSources sources, SettingsContainer settings, IReadOnlyCollection`1 updates) in <...>/NuKeeper/Engine/Packages/PackageUpdater.cs:line 75
   at NuKeeper.Engine.Packages.PackageUpdater.MakeUpdatePullRequests(IGitDriver git, RepositoryData repository, IReadOnlyCollection`1 updates, NuGetSources sources, SettingsContainer settings) in <...>/NuKeeper/Engine/Packages/PackageUpdater.cs:line 46
Attempted 3 updates and did 0

I'm sure that branch doesn't currently exist on remote.
I thought it may be an issue with LibGit2Sharp, so I've ran it providing my own git cli path and then the result was (blank lines as originally printed in logs):
Nuget restore on <my solution> complete
Updating 'Newtonsoft.Json' from from <...> to <correct version> in <...> projects
In path <repo root path>\.git\, running command: <git path>\git.exe checkout -b origin/feature/NuKeeper origin/origin/feature/NuKeeper
Command <git path>\git.exe failed with exit code: 128

fatal: this operation must be run in a work tree

Using branch name: 'nukeeper-update-Newtonsoft.Json-to-<correct version>'
In path <repo root path>\.git\, running command: <git path>\git.exe checkout -b nukeeper-update-Newtonsoft.Json-to-<correct version>
Command <git path>\git.exe failed with exit code: 128

fatal: this operation must be run in a work tree

Updates failed NuKeeperException : Command <git path>\git.exe failed with exit code: 128

fatal: this operation must be run in a work tree

   at NuKeeper.Update.ProcessRunner.ExternalProcess.Run(String workingDirectory, String command, String arguments, Boolean ensureSuccess) in <...>/NuKeeper.Update/ProcessRunner/ExternalProcess.cs:line 63
   at NuKeeper.Git.GitCmdDriver.StartGitProzess(String arguments, Boolean ensureSuccess) in <...>/NuKeeper.Git/GitCmdDriver.cs:line 92
   at NuKeeper.Git.GitCmdDriver.CheckoutNewBranch(String branchName) in <...>/NuKeeper.Git/GitCmdDriver.cs:line 52
   at NuKeeper.Engine.Packages.PackageUpdater.MakeUpdatePullRequests(IGitDriver git, RepositoryData repository, NuGetSources sources, SettingsContainer settings, IReadOnlyCollection`1 updates) in <...>/NuKeeper/Engine/Packages/PackageUpdater.cs:line 75
   at NuKeeper.Engine.Packages.PackageUpdater.MakeUpdatePullRequests(IGitDriver git, RepositoryData repository, IReadOnlyCollection`1 updates, NuGetSources sources, SettingsContainer settings) in <...>/NuKeeper/Engine/Packages/PackageUpdater.cs:line 46
Attempted 3 updates and did 0

where 2 things seems incorrect. First is running checkout in .git\ subdirectory and the second one is doubled origin in that checkout command.
In both situations I've run extension task with targetFolder = src because my project is structured like <repo_root>/src/, but NuGet restore and whole packages-to-update search process went well, so I don't think it's an issue.
Also I saw that in both situations logs are starting with:
<azure devops git path>\git.exe config --global user.name NuKeeper
<azure devops git path>\git.exe config --global user.email<EMAIL_ADDRESS>
so it's not using both gitclipath and LibGit2Sharp there, but uses it in latter operations.
I can provide more info if needed.

Is there a branch in the branching tab on azure devops?

There's a feature/NuKeeper branch for which this pipeline was run. Branch named nukeeper-update-Newtonsoft.Json-to-<correct version> doesn't exist on Azure Devops under feature/ or anywhere else.

Should I create separate issue or maybe that one could be reopened?

I'm running into the same issue on Azure DevOps:

Updating 'EntityFramework6.Npgsql' from 6.4.0 to 6.4.1 in 5 projects
Updates failed NameConflictException : failed to write reference 'refs/heads/nukeeper/nukeeper-update-EntityFramework6.Npgsql-to-6.4.1': a reference with that name already exists.

There exists no such branch.

I'm running into the same issue on Azure DevOps. Did this get solved?

@AnthonySteele I'm running into the same issue too. Azure have no such branch, but issue still appear.
@greengumby @Bouke, have you fixed that somehow?

Using branch name: 'nukeeper-update-4-packages-01C74E6642BE3E6B46C76535EDDA791B'
Git checkout new branch 'nukeeper-update-4-packages-01C74E6642BE3E6B46C76535EDDA791B'
Updates failed NameConflictException : failed to write reference 'refs/heads/nukeeper-update-4-packages-01C74E6642BE3E6B46C76535EDDA791B': a reference with that name already exists.
at LibGit2Sharp.Core.Ensure.HandleError(Int32 result)
at LibGit2Sharp.Core.Proxy.git_branch_create_from_annotated(RepositoryHandle repo, String branch_name, String targetIdentifier, Boolean force)
at LibGit2Sharp.BranchCollection.Add(String name, String committish, Boolean allowOverwrite)
at NuKeeper.Git.LibGit2SharpDriver.<>c__DisplayClass13_0.b__0() in d:\a\r1\a_NuKeeper PR Build\drop\NuKeeper.Git\LibGit2SharpDriver.cs:line 123
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot)
--- End of stack trace from previous location where exception was thrown ---
at NuKeeper.Engine.Packages.PackageUpdater.MakeUpdatePullRequests(IGitDriver git, RepositoryData repository, NuGetSources sources, SettingsContainer settings, IReadOnlyCollection1 updates) in d:\a\r1\a\_NuKeeper PR Build\drop\NuKeeper\Engine\Packages\PackageUpdater.cs:line 75    at NuKeeper.Engine.Packages.PackageUpdater.MakeUpdatePullRequests(IGitDriver git, RepositoryData repository, IReadOnlyCollection1 updates, NuGetSources sources, SettingsContainer settings) in d:\a\r1\a_NuKeeper PR Build\drop\NuKeeper\Engine\Packages\PackageUpdater.cs:line 46


Fix crash when deallocating named colors map

Fixes #18216.
Not sure why it doesn't happen on 3.0, but it can be cherrypicked there as well.
The code is a bit of a hack, unfortunately.The code is a bit of a hack, unfortunately.

Thanks! Might be worth PR'ing for master and then cherry-picking for 3.0 too, just to be sure.


Add option to disable resharper suppression comments

For users that don't use Resharper it will be nice to disable generation of suppression comments.
Default is to generate them so current functionality behaves the same.

Fixed by another change that created suppressions as a file instead.


support .bz2 FASTA file with FeatureData[AlignedSequence]

Improvement Description
It could be handy to support .bz2 FASTA files with FeatureData[AlignedSequence] in addition to the current support for uncompressed FASTA files.
References
forum x-ref

Closing this one out since @ebolyen submitted a more general issue addressing the same functionality request here.


feat(auth): add admin privilege

This PR adds a new permission domain specifically for API methods that require admin privilege, like add_task or del_entity

r? @PhotonQuantum


Expose JsonWriter.Dispose(bool) to derived classes.

Change to protected and virtual. Add GC.SuppressFinalize to IDisposable.Dispose().
See discussion on #746.
Also a micro-opt on JsonReader.Dispose(bool), changing a && to a & as the right-hand operand is fast to access and for most calls both will be true anyway, so removing the branch is a slight win on short-circuiting in that branch.

Ah. @JamesNK got there ahead of me.


Diagonal matrix not reported as positive semi-definite

There seems to be an issue with expressing diagonal matrices via cp.diag(constant_vector) -- namely it does not implement its own is_psd() / is_nsd() methods (https://github.com/cvxgrp/cvxpy/blob/master/cvxpy/atoms/affine/diag.py#L45), so it inherits from AffAtom (https://github.com/cvxgrp/cvxpy/blob/master/cvxpy/atoms/affine/affine_atom.py#L83), whose implementation, as far as I can tell, is asking if the underlying constant_vector is psd, which doesn't make sense.
I believe the fix would be to implement those methods on the diag_vec class. Here is a complete piece of code illustrating the behavior.
import cvxpy as cp
import numpy as np

# Here is a super simple problem:
# Minimize x0^2 + 2x1^2

x = cp.Variable(2)
w = np.array([1.0, 2.0])

# Two equivalent ways of expressing the objective:

# Using quad_form
objective1 = cp.Minimize(cp.quad_form(x, cp.diag(w)))

# Using sum_squares
objective2 = cp.Minimize(cp.sum_squares(cp.multiply(x, np.sqrt(w))))

# Here is the issue:
objective1.is_dcp()  # == False (?!)
objective2.is_dcp()  # == True  (as it should be)

# objective1 fails to be dcp because quad_form asks if its matrix is psd
# (https://github.com/cvxgrp/cvxpy/blob/master/cvxpy/atoms/quad_form.py#L64)
# and the following is observed:
cp.diag(w).is_psd() # == False, should be True

This is fixed on master.


Removing characters in the prompt takes a long time on Linux

in between the prompt displaying Console.WriteLine("world hello"); and Console.WriteLine("hello world"); an up arrow character was pressed, which caused the REPL to remove every individual character, and then insert the Console.WriteLine("hello world"); text.
Using Ubuntu 18.04, gnome-terminal 3.28.2 and .NET Core 2.1.602

This looks like a problem in https://github.com/tonerdo/readline which is used for this GNU style navigation.
What it should do, it should provide you a way to navigate the command history with the up arrow.
Is the normal delete also slow?
Also, if the line is empty, is the up arrow navigation working correctly (toggling through the history of your commands)?

Reproduced the problem by using their library directly. Gonna see what I can do there.
Yes, regular delete is slow too. Up arrow navigation works correctly, except that their implementation revolves around removing all the text first (slow), and then adding the new text (fast).


feat: injective continuous linear map with closed range is a continuous linear equivalence

Added analogue of LinearEquiv.ofInjective but for ContinuousLinearEquiv on Banach spaces.
Added analogue of LinearMap.rangeRestrict but for ContinuousLinearMap.
Also I updated a docstring that had the old name of a theorem.

I added ContinuousLinearMap.rangeRestrict and a simp lemma from rangeEquiv to it. Thanks for the quick review.

@winstonyin this is delegated, you can now merge it yourself. No need to mark it awaiting-review. I'll let you merge in case there is some reason you were waiting.

FYI: I renamed to equivRange because I realized it's actually that which we have in other places in the library (which makes more sense because the symbols literally occur in the order ≃, then range). Sorry for miscommunicating before.

bors merge


Integrate with the Test Explorer

One of the best features I would like of this plugin would be to run and debug tests directly fom the test explorer.
The Go extension provides this and I can say that this makes the dev's life much easier.

Related #122


etcdmain: var log -> plog

So the variable name doesn't mess up with standard package name.

Can you also clean up the printf (use infof)

@xiang90 sure. added. PTAL

addressed all comments.

LGTM


add caribbean currencies

If I understand correctly, this change only adds price data to the UI? It does not allow users to denominate their account in another currency?
Also, do you know if this backfills with historical price data for each of these currencies? or does this just allow querying for current price data?

correct. its just a display currency. Their wallets will always be denominated in USD or BTC.
It does not backfill for historical price, this is only for real-time conversion on the Home Screen. The historical chart is in USD.

I don't think I have the format correct for realtime.config

What documentation have you seen for this? How did you know to update Values.realtime.config with the values provided?

What documentation have you seen for this? How did you know to update Values.realtime.config with the values provided?

From Arvin at Galoy:
Yup that's a price server thing
https://t.co/oajegdSByI
From github.com
This is the relevant config here in the repo
https://t.co/Pi4QPw16Nz
From github.com
And I think we inject all our currencies via an override of this entry in charts
❤

thx man


Domain activation is Failed - Unable to add custom domain

App Id: d3inyt16c4mcj0
Region: eu-west-1
Step: Domain activation
Status: Failed
Issue/question
I'm trying to add a custom domain. I added DNS records properly but we receive an error.
Error message
I'm receiving this message: One or more aliases specified for the distribution includes an incorrectly configured DNS record that points to another CloudFront distribution. You must update the DNS record to correct the problem. For more information, see https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/CNAMEs.html#alternate-domain-names-restrictions
Additional information

Refer to our documentation to troubleshoot domain setup.
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html#i-get-a-cnamealreadyexistsexception-error
Let us know if you're still facing issues.


Double check what extensions are in the README.md versus the environment.yml [45, 30, 30]

Fixing discrepancies between README and .yml file

6/20:

Added -latex, -sidecar, -code_formatter to the environment file.
Spellchecker and -vim library are on our separate branches that we created when we did the tutorial exercise
-Included in envmt.yml file but not in README file is jupyterlab-unfold and jupyterlab-git


don't rely on internal-apis to get your external IP address

maybe use the DHT?
returning the port in addition to the IP will also help with NAT holepunching

APIs recently went down and no one was able to start the SDK.


Disable v-wave on "disabled" state for buttons

Hello, I think adding the possibility to disable v-wave on "disabled" state when the v-wave directive is applied to the buttons could be great. I don't know if it should be a default behavior because it could introduce a breaking change, but maybe add a boolean to the global config ?
I know it's currently possible to do it by copy / paste the same code used into "disabled" button attribute, but it's a lot of duplication, especially when you get many buttons

Love the idea! I'll look into it!

Purposed option:

name
default
type

respectDisabledAttr
false
boolean

Wave will be disabled if the disabled attribute is present on the button (can be set globally):
<button disabled v-wave="{
  respectDisabledAttr: true
}">Click me!</button>

Ideally this would've been included with the 2.0.0 release and defaulted to true but I didn't want to delay Nuxt 3 support any longer than it already had been.

@justintaddei Im using the directive like this and doesnt work for me, there is no description about this attr and usage in the documentation
import VWave from "v-wave";
const { vWave } = VWave.createLocalWaveDirective({
respectDisabledAttr: true
});


Resources and citations of search

How can I make the vector store contains the page number and source of the content in metadata(able to get the title which is pdf's name), can some help me with this

Hi @Nalin879 ,
You might want to check out our more fully featured samples
https://github.com/Azure-Samples/azure-search-openai-demo/tree/main
https://github.com/Azure-Samples/chat-with-your-data-solution-accelerator


Single request on different post_type with conditional operation on custom_field_values

Hi,
I know, just reading the title is getting you already tired :)
Well, I have a search engine which goes on two post_type at the same time BUT I would like my request to sort directly with some custom field values. I have 2 problems with, the first one is I do not know how to use comparison signs with custom fields values ( like > < != etc ... ) and the second one is this custom field is only for one of the two post types. Which means I have to manipulate my query in a way that the part where I try to sort / select the correct values only goes for the correct post type.
I do not know if it's clear and if anyone might have this issue but I'll be really glad to get an answer.
Thanks,

I'm not sure what you mean but I try anyway. I think I had the same use case two. I wanted to sort the posts of a post_type by some value defined via custom field value.
ccfr_table_name = CamaleonCms::CustomFieldsRelationship.table_name.freeze
result = @post_type
             .the_posts
             .published
             .joins("INNER JOIN #{ccfr_table_name} ON cama_posts.id = #{ccfr_table_name}.objectid")
             .includes(:metas)
             .where("#{ccfr_table_name}.custom_field_slug = 'weekly_amount' AND #{ccfr_table_name}.value::float > 0")
             .reorder("#{ccfr_table_name}.value::float DESC")

That query gets all posts of that post_type that have a custom field 'weekly_amount' greater than 0. Note: This is for Postgres. I suppose in MySQL the query would look a little different. The includes(:metas) prevents n+1 queries. The value::float is a Postgres shorthand and casts the the string to float.

Thanks @haffla .
That answers to one part of my problem already and that is already pretty good.
I changed a little bit my models to get only one post_type where my search engine has to look into.
Now my deal is to use elasticsearch indexing the Post model looking into custom fields value ;)
But that's another story. Again, if you're familiar with that, I won't say know to a little help.
Thanks again.


Use updated FieldValidationOption API

In the PR I have adjusted code base for the new FieldValidationOption API introduced in base#446.

@armiol PTAL.


Fix posts not loading (fixes #62)

Fixes #62 on Keystone's end. Thanks @bjwyse for the suggested fix!

Thanks! I think @JedWatson needs to deploy a new version?


[BUG] Update of AAS DetailsCard

Describe the bug
AAS DetailsCard is not updated after selecting another AAS
To Reproduce
Steps to reproduce the behavior:

Open aas-gui (in my case http://localhost:3099/)
Selecting an AAS and open the AAS Details Card (click on (i) icon)
Selecting another AAS
AAS Details Card is not updated with respect to the new selected AAS

Expected behavior
If the AAS DetailsCard is shown it will be updated with respect to the current selected AAS
BaSyx (please complete the following information):

BaSyx Version v2.0
AAS Version DotAAS V3, see example files

Screenshots to Reproduce
Step 1:

Step 2:

Step 3 + 4:

Expected behavior (after step 3):

Used Asset Administration files (.aasx/.json/.xml)
AAS examples.zip
BaSyx configuration
Default configuration of aas-environment, aas-registry, sm-registry, aas-discovery and aas-gui (see BaSyx Starter Kit) (the latest docker image version)
Additional context
I'm not sure if it's a bug, or if it's intentional that the user should click on the (i) icon of an AAS to update the DetailsCard.
Personally, as I said, I would expect the DetailsCard to be updated every time an AAS is selected.
If this is a bug, I will be happy to provide a PR to fix.

Addition
Describe the bug
AAS DetailsCard is not updated after jumping to another AAS (e.g. from the SM BOM plugin)
To Reproduce
Steps to reproduce the behavior:

Open SM BOM visualization (DetailsCard of selected AAS is shown)
Click on Jump Button
AAS Details Card is not updated with respect to the new selected AAS

Screenshots to Reproduce
Step 1 + 2:

Step 3:

Expected behavior (after step 3):

Hi @seicke,
that's definitely a bug. If you like to I would be happy if you provide a PR with a fix for this :)