[Bug]: Session Token is null when retrieved by Service

General Info

[x] I checked for similar bug report
[X] I am using the latest version
[X] I checked the troubleshooting page for similar problems

Version(s)
2.5.2
Kotlin Target(s) and their respective versions
JVM8
What happened? (include your code)
I was trying to access accessToken/refreshToken in Service after user login in Activity. However the supabase client return null session.
The issue varies in different phone OS. We found the issue on Harmony OS.
Steps To Reproduce (optional)

Login in activity
In service, when service is setting up, retrieve the session token.

private fun setupService() {
        CoroutineScope(Dispatchers.IO).launch{
            supabase.auth.loadFromStorage()
        }
........

Print the session, we got null

val accessToken = supabase.auth.currentSessionOrNull()?.accessToken
val test = supabase.auth.currentAccessTokenOrNull()
Timber.d("accessToken is $accessToken, test is $test")
// accessToken is null, test is null

Relevant log output (optional)
No response

Two things:

How/Where do you call setupService? Since this method is non-blocking, the loading could happen after the printing
Normally, the session gets automatically loaded from session, but you want to do that manually?

Hi Jan. Thanks for reply.

It's in onServiceConnected() of service. Yeah it's non-blocking. I removed it btw.
Is it by default loaded from storage? I loaded manually just for make sure it's loaded.

A more info about my scenario:
So in my application, there's a global supabaseClient variable to be accessed by other files.
And it seems when I access when service get connected (onServiceConnected) it works and doesn't need loadFromStorage
However when I am emitting a network request from a composable method with Launcheffect, supabase session returns null.
So it becomes:
1. I see tokens are non-null in onServiceConnected, where the whole service spins up.
2. I see tokens become null in launcheffect's http request method.

Btw, my application is a bit different, as I had some composable added to service.
I was trying to use loadFromStorage() just to make sure if

Is it by default loaded from storage? I loaded manually just for make sure it's loaded.
Unless disabled in the config, yes. On Android it will load depending on lifecycle events to ensure that no expired sessions get loaded. You can try out two things:

Switch to your application if Auth#sessionStatus switches from LoadingFromStorage to Authenticated, NotAuthenticated, etc.. sessionStatus is a hot flow.
There is also a helper function Auth#awaitInitialization() which basically does that, suspends the coroutine until the initialization is done.

The initialization is not really noticeable in terms of time usage, but making requests before that or calling currentSessionOrNull before that obviously fails.

Hi Jan.
When I called Auth#awaitInitialization() , application got stuck at this method.
loadFromStorage works as expected.
I'm not super sure if the issue relates to lifecycle management of composable views that added to windowManager.. I can find a time to have a minimum repro codes. Those Composable's lifecycleowner is set to the Service (as service has derived from LifecycleOwner)

If it got stuck, the loading never appeared. The implementation for the automatic loading can be viewed here. If this is related to a lifecycle issue, then maybe manual loading will be your best bet, but then I'd disable AuthConfig#enableLifecycleCallbacks.

I can find a time to have a minimum repro codes.

If you want sure, not certain what exactly the problem is with the lifecycle.

Thanks so much for your help!
Will post a repro tomorrow. (PDT timezone)

Also a QQ: should I have token stored to sharedPreference? Or I can rely on loadFromStorage

You can rely on loadFromStorage. You don't have to worry about saving the session at all.

Closing due to inactivity, feel free to reopen.

There is also a helper function Auth#awaitInitialization() which basically does that, suspends the coroutine until the initialization is done.

This methods saved my day. Thanks for clarifying
Hopefully, this method will be added to the documentation.


Getting Started examples fail to render emoji and some spinners on Windows

Information

OS: Windows 10 Pro
Version: 21H2 19044.1466
Terminal: Windows Terminal, cmd.exe, PowerShell 5.1 & PowerShell 7.2.1

Describe the bug
Tried to render several emojis and didn't find any that worked. The Clock spinner does not work.
To Reproduce
I ran the following code on a Windows Terminal (PowerShell 5.1), cmd.exe console, PowerShell 5.1 console, and PowerShell 7.2.1 console. BTW: Blinking only occurs on Windows Terminal.
    public static async Task<int> Main(string[] args)
    {
      Task COutTask = Console.Out.WriteLineAsync("Hello World!");
      Spectre.Console.AnsiConsole.MarkupLine("[underline red slowblink]Hello[/] World! :globe_showing_europe_africa:");
      await COutTask;
      AnsiConsole.Progress()
        .AutoRefresh(true)    // Turn on auto refresh
        .AutoClear(false)     // Do not remove the task list when done
        .HideCompleted(false) // Hide tasks as they are completed
        .Columns(new ProgressColumn[]
          {
              new TaskDescriptionColumn(),    // Task description
              new ProgressBarColumn(),        // Progress bar
              new PercentageColumn(),         // Percentage
              new RemainingTimeColumn(),      // Remaining time
              new SpinnerColumn(Spinner.Known.Clock),            // Spinner
          })
        .Start(ctx =>
        {
          // Define tasks
          var task1 = ctx.AddTask("[green]Reticulating splines[/]");
          var task2 = ctx.AddTask("[green]Folding space[/]");
          while (!ctx.IsFinished) {
            task1.Increment(1.5);
            Thread.Sleep(20);
            task2.Increment(0.5);
          }
        });
      return 0;
    }

Expected behavior
I expected the code to produce emojis and the clock spinner. If this is not supported on Windows please update the documentation.
Screenshots

Additional context
None for now.

@sldr Unicode functionality is only available in terminals that support that. Terminals on Windows that support Unicode is (for example) Windows Terminal.
You can check whether or not the current terminal supports Unicode via AnsiConsole.Profile.Capabilities.Unicode.
In general, you can never assume that a terminal (especially on Windows) supports things like Unicode and there is nothing Spectre.Console can do about this. We do however try to handle stuff gracefully where we can (such as showing the ASCII-spinner instead of garbage output).


OOD PR Merge pull request #519 from OSC/sshhost_whitelist

Create documentation for https://github.com/OSC/ondemand/commit/14e09f788e95eed9bafeb8c34d92759ae4d75c32

superceeded by #298 .


Failing import produces full screen HTML output

Trying to import the recipe from https://www.oetker.de/rezepte/r/spiralkartoffeln I'm greeted with a fullscreen error, spitting out raw HTML instead of a sensible message like just "Import source is not supported" or so.
See screenshot:

Using:
Nextcloud: 24.0.6
Cookbook: 0.9.15
Cookbook App: 0.7.8

With #171 this won't happen anymore and we just display the status code.
I'll try bringing better error handling into the app


AAD SignIns Insights

I wonder if there would be an interest for such a module. It's essentially a similar concept of what we have for analysts in the entity page available for automation and a bit of what we have in UEBA.
Takes a user and return stats such as:

Last successful logon data (timestamp + other metadata)
Last failed logon data
Usual user-agent-string data
Usual contries/IPs

If there is a cloud-logon-session present in the entities (case of an AAD Protection alert), return all the info about this particular login.
That last one maybe could be added to the AAD Risk Module instead.

I like the idea, would this be a new module or perhaps an extension to the capabilities of the AAD Risks?  It may make it too complex though if we add it... not sure off hand.
One issue, the cloud-logon-session is not passed from the incident trigger so you have to go back into SecurityAlerts to get it.... this is one of the reasons for #205 so you can use the KQL module to lookup the incident easily and work from there

Maybe also return a table of last successfull access per app?

possibilities to include:

Conditional access failures (and the policies that failed)
Named locations the user has been seen from
Device join status of the signins
password resets or other interesting admin actions on account

insights about signin hours

(although we would need to define what baseline could be used to determine out of character behaviors)


Input file and command interface

Hi,
Please let me ask an improvement.
Can you improve the input file format? It's a little bit complicated for who are machine learning beginner like me.
For example, splitting input data both for training and test by using folder structures, and using JSON for graph structures and features.
I tried to find out the spec of the current input data format details by using debugger but not yet cleared.
Please consider or give me your advice to make a pull-request to do it.

Thanks, I appreciate your comment. Unfortunately, I won’t have
time/capacity to work on the data loading pipeline. If you find a way to
simplify the data loader, feel free to make a pull request.
On Mon 3. Dec 2018 at 09:35 John<EMAIL_ADDRESS>wrote:

Hi,
Please let me ask an improvement.
Can you improve the input file format? It's a little bit complicated for
who are machine learning beginner like me.
For example, splitting input data both for training and test by using
folder structures, and using JSON for graph structures and features.
I tried to find out the spec of the current input data format details by
using debugger but not yet cleared.
Please consider or give me your advice to make a pull-request to do it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tkipf/gae/issues/18, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHAcYGV3kgRaW_9IDPNQR_JCdlew6zwCks5u1TahgaJpZM4Y-tQR
.

Got it. Thanks for your reply.


R-hub badges

cc @gaborcsardi (I hope it's fine to have deleted your name even if the badges are still yours!)

thanks


jax.scipy.linalg routines segfault on Mac OS X on scipy 1.2.1 or later but not scipy 1.1.0

I'm not exactly sure why this happens, being unfamiliar with the internal architecture, but on MacOS with Python 3.6.8, the following code segfaults if scipy 1.2.1 is installed (the version that comes by default when you pip install jax jaxlib):
import jax.random as random
import jax.scipy.linalg as linalg

key = random.PRNGKey(42)
# For some reason, matrices smaller than (50, 50) or so do not trigger segfaults
X = random.normal(key, (500, 500))
A = X @ X.T  # Drawn from standard Wishart distribution
linalg.cholesky(A)
print("Success!")

This is a great project by the way--thanks for working on it!
Edit: after further digging, I found the following in the the Scipy 1.2 release notes:

scipy.linalg.lapack now exposes the LAPACK routines using the Rectangular Full Packed storage (RFP) for upper triangular, lower triangular, symmetric, or Hermitian matrices; the upper trapezoidal fat matrix RZ decomposition routines are now available as well.

Perhaps this has something to do with it?
Even more edits: yet more digging has revealed scipy/scipy#9751, which hints that this might be caused by a specific (old) version of XCode. I will report back once XCode is upgraded.

Thanks for reporting this, and for digging into it! (And for the kind words about JAX too!)

This smells like a scipy bug to me, but it's hard to be sure without tracking it down...
(I am increasingly leaning towards not using scipy's LAPACK kernels on CPU, because of issues like this one...)

Turns out upgrading XCode doesn't fix it. I agree that it seems like a scipy bug, perhaps it is because I'm running MacOS High Sierra still (at the request of department IT). In any case pinning the dependency to Scipy 1.1.0 works for now, and deployment on Linux is no problem.
Since I'm the only affected user and the bug isn't showing up on the CI system, it might be reasonable to drop this until other people report the same problem.

One more data point: I tried building scipy from source at version 1.2.1, and the self-built version doesn't segfault. I'm using Mac OS Mojave and XCode 10.0.
So I'm wondering if this is a problem with the PyPI-provided scipy packages.

Another data point:
Segfault with jax.numpy.linalg.solve and scipy>1.1 from PyPI

The same happens for scipy 1.4.1 on macos and python 3.6.6 for the qr decomposition
for matrices where number of columns > 128
import jax.numpy as np
import jax
import numpy as onp
import jax.config as config
config.update("jax_enable_x64", True)

q, r = sp.linalg.qr(onp.random.rand(2000, 128)) #works fine
q, r = sp.linalg.qr(onp.random.rand(2000, 129)) #bus error: 10

I'm seeing a similar problem with Cholesky decomposition on Mac OS 10.15.4, Scipy 1.4.1 and Python 3.7.4 from Anaconda but where it works for a matrix 63 x 63 but gives a Bus error for 64 x 64.

I think I've figured out what's going wrong here, and why it's Mac OS specific.
The problem is that we run out of stack space and crash due to a stack overflow. Mac OS is thread stacks default to 512KiB, whereas Linux defaults to 8MiB stacks. (Since these threads are part of a thread pool, you cannot work around this by changing ulimit, it requires code changes.)
I'm not quite sure what the best way to fix this is at the moment but I'll figure something out.

Thanks for all the hard work! Much appreciated

#2927 should fix this bug; it requires a new jaxlib, so you can either build from source or we will most likely make a release next week.


Syntax error caused by missing angle bracket in type declaration can crash new solver

As of version 0.646, the following code causes an internal compiler error in the new solver, regardless of strict mode being used:
--!strict
local Class = {}
-- missing angle bracket here causes a crash:
-- InternalCompilerError - CG: AstExprLocal came before its declaration?
--          v
type Class<S = typeof(setmetatable({}, Class))

Thank you for the report! I've been working on this on-off for a bit, the root cause is that aliases are processed prior to other statements, so anything of the form:
local A = {}
type B<T = typeof(A)> = unknown

... will be problematic.


feature request: command line version that outputs decode

A command line executable of this that accepted an audio file and outputted a decode to either stdout or to a file would be awesome.
Another nice to have would be the ability to have a lookup table that you could pair codes to characters for different languages.

Hi, thanks for the suggestion!
I just added the example ggmorse-from-file that decodes Morse Code from an uncompressed WAV file:
https://github.com/ggerganov/ggmorse/tree/master/examples/ggmorse-from-file

Much appreciated. I'll give it a go next week!

Thanks for the tests!

You may want to add the following on the main readme as I had to do the same to get it to build. https://github.com/ggerganov/ggmorse/issues/4#issuecomment-950360431

If you use the --recursive option during cloning as specified in the README, you don't need that command.

It would be nice if there was a way to build only ggmorse-to-file and ggmorse-from-file so you don't have to worry about unneeded dependencies if you're not going to use the GUI version.

This is already supported. Use:
cmake -DGGMORSE_SUPPORT_SDL2=OFF ../

I'll update the README to make this more clear.

Your code works well - I have used this to post-process CW Data, however ... as there is a very large discrepancy between QRO stations and QRP stations - some form of AGC would be very nice to have.


Support for bind mounts?

So far as I can see Shiplift does not support bind mounts (https://docs.docker.com/storage/bind-mounts/) only named volumes, is this the case? Would it be possible to add bind mounts support if it is missing?

I think that you can just use the path to file/directory you want to mount:
docker
    .containers()
    .create(&ContainerOptions::builder(image.as_ref()).volumes(vec!["/path/to/file:/path/in/container"]).build())

Ah yep confirmed you can, had assumed that option was for named volumes, will open a PR to add some clarificatory documentation.


[Docs] Feature: Data plane intelligence - Envoy-based WASM filter management

Current State
Meshery has emergent support for filter management, but not documentation published on the set of functionality surrounding filter management.
Desired State
Create a new "Filter Management" page that describes this functionality area.

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation
Layer5 Community Slack

Hey @leecalcote I would like to work on this issue.

Here you go :)


Add access to the model: application.schemas

Refs: https://github.com/metarhia/impress/issues/1573

[x] tests and linter show no problems (npm t)
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/impress/commit/241163263139c9b9baa954ccb2549cd513c7ee81


Fix: mysql8 - Unknown variable "default-authentication-plugin"

This is a fix for  issue  #277 - The description is below.
Proposed changes:
Change default-authentication-plugin=mysql_native_password to mysql_native_password=ON in the mysql8 Dockerfile.
Describe the bug

MySQL crashes with the default-authentication-plugin=mysql_native_password configuration enabled by default in the Dockerfile for mysql8

Which Branch / PHP Version are you using ?

php83

Steps to reproduce

Use mysql8 as database

Expected behavior

MySQL is supposed to start and run.

Error message
bigbase-mysql8      | 2024-05-12T16:59:48.716026Z 0 [ERROR] [MY-000067] [Server] unknown variable 'default-authentication-plugin=mysql_native_password'.
test-mysql8      | 2024-05-12T16:59:48.717540Z 0 [ERROR] [MY-010119] [Server] Aborting
test-mysql8      | 2024-05-12T16:59:50.739542Z 0 [System] [MY-010910] [Server] /usr/sbin/mysqld: Shutdown complete (mysqld 8.4.0)  MySQL Community Server - GPL.
test-mysql8      | 2024-05-12T16:59:50.739616Z 0 [System] [MY-015016] [Server] MySQL Server - end.
test-mysql8 exited with code 0

This just repeats until I kill the container
Desktop (please complete the following information):

OS: Windows
Docker Version: 26.0.0

Additional context

Related to issue #273
But I was unable to reproduce the issue described there with or without my Pull Request, and I tried every version of mariadb included.
I suspect that the issue there is that version of mariadb was changed without cleaning the data folder.

I was about to commit the exact same PR since I ran into this issue and fixed it in the exact same way.
I can confirm: since MySQL 8.4 was released, docker-compose-lamp fails to spin a MySQL 8 container because of this issue.
And I can confirm: this fix works.

it works for me.
hope this can be merged soon!


Implement site-packages getter for Venv

Tag #335 (python-launcher would help here)

Tag #399

Tag #443


waitforx issue

xrdp version
0.10.0
Detailed xrdp version, build options
xrdp 0.10.0
  A Remote Desktop Protocol Server.
  Copyright (C) 2004-2024 Jay Sorg, Neutrino Labs, and all contributors.
  See https://github.com/neutrinolabs/xrdp for more information.

  Configure options:
      --prefix=/usr
      --sysconfdir=/etc
      --libexecdir=/usr/lib
      --localstatedir=/var
      --sbindir=/usr/bin
      --with-systemdsystemunitdir=/usr/lib/systemd/system
      --enable-jpeg
      --enable-tjpeg
      --enable-fuse
      --enable-fdkaac
      --enable-opus
      --enable-rfxcodec
      --enable-mp3lame
      --enable-pixman
      --enable-painter
      --enable-vsock
      --enable-ipv6
      --enable-pam-config=arch
      --enable-rdpsndaudin
      --with-imlib2
      CFLAGS=-march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions         -Wp,-D_FORTIFY_SOURCE=3 -Wformat -Werror=format-security         -fstack-clash-protection -fcf-protection         -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -g -ffile-prefix-map=/home/felix/xrdp/src=/usr/src/debug/xrdp -flto=auto
      LDFLAGS=-Wl,-O1 -Wl,--sort-common -Wl,--as-needed -Wl,-z,relro -Wl,-z,now          -Wl,-z,pack-relative-relocs -flto=auto
      CXXFLAGS=-march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions         -Wp,-D_FORTIFY_SOURCE=3 -Wformat -Werror=format-security         -fstack-clash-protection -fcf-protection         -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wp,-D_GLIBCXX_ASSERTIONS -g -ffile-prefix-map=/home/felix/xrdp/src=/usr/src/debug/xrdp -flto=auto

  Compiled with OpenSSL 3.3.1 4 Jun 2024

Operating system & version
Arch Linux 2024.07.01
Installation method
git clone & make install
Which backend do you use?
xorgxrdp
What desktop environment do you use?
GNOME
Environment xrdp running on
VM
What's your client?
Windows 10 22H2, RDP
Area(s) with issue?
Other
Steps to reproduce
Connect to target client via RDP in Windows.
✔ Expected Behavior
To connect successfully
❌ Actual Behavior
Got the "Can't create session for user - X server could not be started" error.
Anything else?
No response

xrdp-sesman.log
xrdp.log

Solved this issue. Problem was the AUR package for xorgxrdp is currently broken, and installing the glamor version yay -S xorgxrdp-glamor solved this issue for me. Closing this issue.


Find the previous match in the editor help when pressing Shift + Enter

Follow-up to #27716.

Thanks!


tgBoxInfo calculates and incorrect mass, tgBox takes half-extents not full-extents

The current call to the Bullet Physics function that creates a box, line 109 in tgBoxInfo.cpp, passes in the full width and height of a box:
m_collisionShape = new btBoxShape(btVector3(width, length / 2.0, height));
However, the Bullet API says that the btBoxShape constructor takes "half extents", which would be half the width and half the height of the box. As a consequence, our boxes in Bullet are twice as tall and thick as they are according to NTRT. In other words, our constructor for tgBox does not actually take the full width and height, but only half the width and height.
Changing this behavior would drastically affect other folks' code, so it does not make sense to me to fix this issue right now.
However, this does cause one major inaccuracy in our simulations: the getMass() function in tgBoxInfo does not calculate the correct mass, since it assumes the volume is (length x width x height) whereas volume is really ( length x width/2 x height/2).
I plan to correct the getMass() function. Please let me know ASAP if your code depends on the mass of a tgBox.

For what its worth, this was already reported in #123

Thanks Brian, I didn't see that issue. Since no one else seems to care much, I went ahead and implemented the fix - mass is now correct as of commit c4a4c92.
Since we've already been using width and height in this inconsistent way, it would make sense to keep that convention, right? We can just be very clear in the documentation that "length" is the full length, but "width" and "height" are half the width and half the height respectively.


Added requirements for weather/tasks.py.

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


dBeaver issue on macOS:   a driver is needed, but  it didn´t install automatically

Description
It´s a company Laptop. DBeaver is in our Selfservice Tool to install. You can open it, but the drivers will not install automatically so I can´t use DBeaver. And its not possible to install manuell a dbcd driver on this laptop.

DBeaver Version
24.2.5
Operating System
macbook pro / mac OS Sequoia Version 15.1
Database and driver

Steps to reproduce

open ssh tunnel
try to connect Database.
direcly asking for the driver

Additional context
we use ssh tunnel for the connection.

Can you access https://repo1.maven.org/maven2/ from this PC?

I'm closing this issue as it's been a while since there was an update.


Configuring Project ID is a bit confusing

I received the following error even though GOOGLE_PROJECT_ID was set as an environment variable. I assume it’s because a project NUMBER is requested, but the error message does not explain which environment variable should be set.
ERROR:@google-cloud/trace-agent: Unable to acquire the project number from metadata service. Please provide a valid project number as an env. variable, or through config.projectId passed to start(). Error: getaddrinfo ENOTFOUND metadata.google.internal metadata.google.internal:80

So there are two issues here:

The GOOGLE_PROJECT_ID environment variable should work to configure the project ID in the config
The error message should display the name of the env variable

Additionally, is the explanation "through config.projectId passed to start()" clear enough for explaining the following:
start({ projectId: "YOUR_PROJECT_ID"})

Because I found this to be a little unclear.

We support GCLOUD_PROJECT being read from the environment. I can't find any Google-backed documentation for the usage of GOOGLE_PROJECT_ID -- if you have a source please let us know.
It's true that the error message is somewhat outdated (and a little unclear) -- thanks for bringing this to our attention!

Thank you for your quic response, GCLOUD_PROJECT works great!
So then my only ask is to make the error message use "project ID" instead of "project number" and identify GCLOUD_PROJECT as the env var.

Fixed in #700


Material Stepper icon change

Bug, feature request, or proposal:
feature request
What is the expected behavior?
need to the stepper index number to alphabetic
What is the current behavior?
index is coming only in numbers.
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
What is the use-case or motivation for changing an existing behavior?
for sub-steps i need to change the numbers into alphabetics.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
angular 5,  material-5.2.4
Is there anything else we should know?

This will be possible once https://github.com/angular/material2/pull/10516 is released.


Crosshair bug

If I play spy and then choose another class I no longer have any Crosshair by all classes.

https://streamable.com/1nfmv That's how it looks by me (sorry for the laggy video)


Problems with Export-Excel

I regularly create a CSV file where some lines have a string value (always the same) in the last column (G) and others do not. This file is then converted to XLSX using the following command:
Get-Content E:\file_name.csv | ConvertFrom-Csv -Delimiter ';' | Export-Excel E:\file_name.xlsx
This worked for quite a while, but suddenly the conversion started also setting the value for many lines that didn't have it in the CSV. After some rather fruitless testing and changing the SQL script that handles the export to CSV back and forth, the conversion seems to work correctly again for some reason (with the exact same code and configurations as before).
However, as the conversion also has to work when I'm away, I would really appreciate it if you could either fix this or tell me how to prevent it from happening again. (I realise this report may be a little too vague to work from, but one can hope.)
Another problem (or feature?) that I noticed along the way is that if you name the output file "Test.xlsx", the conversion will add information about currently running processes to the result.

More short way:
Import-Csv E:\file_name.csv -Delimiter ';' -encoding default | Export-Excel E:\file_name.xlsx

More short way:
Import-Csv E:\file_name.csv -Delimiter ';' -encoding default | Export-Excel E:\file_name.xlsx

Thanks for the tip.
Now the last two lines (out of about 1100) are repeated at the end.

Thanks for the tip.
Now the last two lines (out of about 1100) are repeated at the end.

Check your CSV with this
Import-Csv E:\file_name.csv -Delimiter ';' -encoding default | out-gridview

Check your CSV with this
Import-Csv E:\file_name.csv -Delimiter ';' -encoding default | out-gridview

Hmm, with
Import-Csv E:\file_name.csv -Delimiter ';' | select-object -last 2 | Export-Excel E:\file_name.xlsx,
the two repeated lines replaced the first two lines instead. The same happened with
Import-Csv E:\file_name.csv -Delimiter ';' | select-object -last 0 | Export-Excel E:\file_name.xlsx
and
Import-Csv E:\file_name.csv -Delimiter ';' | select-object -first 0 | Export-Excel E:\file_name.xlsx.
With
Import-Csv E:\file_name.csv -Delimiter ';' -encoding default | select-object -first 1100 | Export-Excel E:\file_name.xlsx,
it seems to work for now.
Curiouser and curiouser...

Hmm, with
Import-Csv E:\file_name.csv -Delimiter ';' | select-object -last 2 | Export-Excel E:\file_name.xlsx,
the two repeated lines replaced the first two lines instead. The same happened with
Import-Csv E:\file_name.csv -Delimiter ';' | select-object -last 0 | Export-Excel E:\file_name.xlsx
and
Import-Csv E:\file_name.csv -Delimiter ';' | select-object -first 0 | Export-Excel E:\file_name.xlsx.
With
Import-Csv E:\file_name.csv -Delimiter ';' -encoding default | select-object -first 1100 | Export-Excel E:\file_name.xlsx,
it seems to work for now.
Curiouser and curiouser...

It is probably your CSV contains something that corrupts output. Does out-gridview output look correct?

It is probably your CSV contains something that corrupts output. Does out-gridview output look correct?

Yes, it does.

Yes, it does.

BTW, -last 0 and -first 0 are meaningless. The argument is count, NOT index. For index there is parameter -index.
If except the fact that export-excel works correct but the problem persists you can try this
# to rebuild data
import-csv | export-csv
# then
import-csv | export-excel

Play with select-object to find corrupted data.

BTW, -last 0 and -first 0 are meaningless. The argument is count, NOT index. For index there is parameter -index.
If except the fact that export-excel works correct but the problem persists you can try this
# to rebuild data
import-csv | export-csv
# then
import-csv | export-excel

Play with select-object to find corrupted data.

I only used -last 0 and -first 0  to see if the number of repeated lines was in any way dependant on the count (which it was not).
Running
Import-Csv E:\file_name.csv -Delimiter ';' -encoding default | Export-Csv E:\file_name2.csv

Import-Csv E:\file_name2.csv -Delimiter ';' -encoding default | Export-Excel E:\file_name.xlsx

resulted in a CSV file that had all values surrounded separately with quotation marks (but everything was displayed as a single column via out-gridview) and the special characters were messed up despite -encoding default.
Oddly enough, the XLSX file that was generated from that had the same messed up column followed by the correctly displayed values (to the right).
Most vexing...

I only used -last 0 and -first 0  to see if the number of repeated lines was in any way dependant on the count (which it was not).
Running
Import-Csv E:\file_name.csv -Delimiter ';' -encoding default | Export-Csv E:\file_name2.csv

Import-Csv E:\file_name2.csv -Delimiter ';' -encoding default | Export-Excel E:\file_name.xlsx

resulted in a CSV file that had all values surrounded separately with quotation marks (but everything was displayed as a single column via out-gridview) and the special characters were messed up despite -encoding default.
Oddly enough, the XLSX file that was generated from that had the same messed up column followed by the correctly displayed values (to the right).
Most vexing...

So, did you fix something or was this just not enough info to work from?

So, did you fix something or was this just not enough info to work from?


Suggestion

Describe the feature you'd like
A clear and concise description of what you want to have as a new feature.
A sync with Wraith ARS 2X and The Cad
Notifications in game when bolo calls are made in the cad
A sound when attached to a call
A sound when theres a 911 caller
The ability to turn on register button and have admins only create the logins
Remove the top icons when at the login page but when logged in add them back
A syned ALPR system with Wraith ARS 2x and the Cad

Hey!
Thanks for sending these suggestions! I can already tell you some of these suggestions will be added soon :), such as the sounds.
Thanks, Casper.

Thanks 🙏

I've written all the suggestions down, closing issue.
Thanks again for the suggestions


note about registry_package firing multiple times

Why:
Closes #22091

What's being changed (if available, include any code snippets, screenshots, or gifs):

Add a note about a workflow potentially being triggered more than once for the same image and an example for how to mitigate that.
Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

@makkes Thank you for opening a PR and linking it to your issue! ✨
I'll get this triaged for review! :zap:

Why:
Closes #22091
What's being changed (if available, include any code snippets, screenshots, or gifs):
Add a note about a workflow potentially being triggered more than once for the same image and an example for how to mitigate that.
Check off the following:

[x]  I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x]  For content changes, I have completed the self-review checklist.

check****

[ ]

@makkes Thank you for opening a PR and linking it to your issue! sparkles
I'll get this triaged for review! zap

Hi @cmwilson21, thanks for helping getting this in. Since there's been no movement in the PR since I opened it I'd like to understand if there is anything I can do to get it merged.

Thank u

@makkes Thanks for checking in on this! Sorry for the delay, we are currently working through our backlog so we appreciate your patience. 💛
Someone will have eyes on it soon for review 👀


Apple AMX support in LLVM or CLANG

Include aarch64.h where you want to use native AMX commands.
Other stuff is docu, tests and perf meter.

Huh? What is this, does any part of it touch tinygrad?

Don't include all this README crap, it's just the one header file.

No, it's standalone, and just allows you to use AMX commands where you need them.
On 18 Apr 2024 at 13:28 +0300, George Hotz @.***>, wrote:

Huh? What is this, does any part of it touch tinygrad?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Torch and CUDA

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Only build for CUDA >=11.2
Build for torch 1.12 and 1.13

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Should I modify the string number to add the torch version to it? This is more convenience here.

Closing here. See https://github.com/conda-forge/pyg-lib-feedstock/issues/2 for context.

So far the failing builds are related to network error. I will wait for all then to finish before relaunching.

LGTM here.

@conda-forge-admin, please rerender


PyO3 interface for Kira

Related to #90 , I think it'd be useful to have a Python interface for Kira to make implementing sound effects using Python easy. Initially, just providing hooks to transform StaticSoundData would be good enough for my use case.

The main challenge I see is that the frames are interleaved [ (left, right), ... ], which isn't immediately the easiest to work with.
I wonder if this structure would be simpler:
pub struct StaticSoundData {
    pub sample_rate: [u32](https://doc.rust-lang.org/nightly/std/primitive.u32.html),
    pub left: Arc<[f32]>,
    pub right: Arc<[f32]>,
    pub settings: [StaticSoundSettings](https://docs.rs/kira/latest/kira/sound/static_sound/struct.StaticSoundSettings.html),
}

How would that make it easier to work with? Is there something about Python that makes it harder to work with interleaved data?

Yes, generally most math libraries expect de-structured data. For example:
https://numpy.org/doc/stable/reference/generated/numpy.fft.fft.html
Having the structure as [(f32, f32)] means that users will need to split and copy the data into two separate arrays by iterating in python. This prevents the user from directly supplying the array to numpy libraries and forces them to run iterate in the python interpreter unless they implement another way to de-structure the data themselves.

It's easy to make your array library understand interleaved data, because it's the transpose. In numpy the transpose is a view (IIRC) and so is effectively free.
The bigger problem with using high-level languages to process audio is that they are not made to be "real-time safe". Processing audio in real-time needs careful programming and use of algorithms that are themselves real-time safe. Here is a good article about what it means to write real-time audio processing code.
One of the biggest sources of problems in the audio thread is allocations. Asking the OS for some memory means using a system call, at which point the OS may pause the thread (for an unknown amount of time) and do something else, then searching for an appropriate region of memory to allocate to the process, before finally returning control to the audio thread; needless to say, this can make the audio thread miss the deadline for processing audio, which results in audio glitches.
Python allocates memory for every object created, which means you wouldn't be able to do anything really without triggering an allocation; it's therefore not a good candidate for writing real-time audio processing code.
The same argument can be made for every other language with automatic memory management, with stop-the-world garbage collection or even with reference-counting. Especially with games, subpar audio thread performance is definitely going to be noticeable event with only a few effects active.
This is not a "this is what happens in the worse-case scenario" either; even if allocations trigger a pop in 0.1% of cases, that's still once every 10 seconds on average.
The only way to compromise on this is to ask for a big buffer size on the backend, so that each audio callback has more time to complete, and the effects of the scheduler, allocations, and locks are diminished; however this is at the cost of latency.
Python bindings make sense for writing games in higher-level languages, and using the raw power of Rust for the performance, but the reverse is sadly not feasible.

@SolarLiner my use case would be for transforming static assets, not realtime assets. Mostly, I think it would make sense to prototype transformations in Python, then for production code you translate that into Rust (or C, etc.)
I see your point with avoiding inserting python hooks into realtime audio processing though. With that in mind, perhaps a better choice would be to provide some C-style callbacks for extending Kira and letting the user shoot their own foot with python if they would like to.

Kira is a real-time audio engine for games, so transforming static assets is strictly speaking outside the scope (though @tesselode is the final arbiter on this).
If you want to preprocess your audio files with Python there are many libraries available. You can then call the script in build.rs or in a Makefile (or Justfile, or Cargo Makefile, or xtask, etc., pick your poison). If you really want to do it at runtime, you can embed Python so you can run Python scripts from your binary.


Installation

Hi , I would like to use this on a daily basis can you please provide install instructions. I use a Mac at home. I use a Windows machine at work and connect to Linux.
thank you

+1. This would be pretty cool to try out, but I'm not sure it would run on a Mac.

Here are my findings for Windows: https://github.com/AshleyF/VimSpeak/pull/8#issuecomment-713433118

I got pretty close to getting it to work on mac. Here's what I've done so far:

homebrew install mono
homebrew install dotnet
dotnet run

It spits out the following error:
System.IO.FileNotFoundException: Could not load file or assembly 'System.Speech, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies.

I'm assuming because System.Speech doesn't work on mac, however, it looks like it's available in NuGet:
https://www.nuget.org/packages/System.Speech
Haven't figured out how to get that to work though since I don't have much experience with F#.


Update instructions with correct URL

Issue #, if available:
Description of changes:
The URL in the workshop leads to a 404, this adds the correct URL.
Can also update the description to mention the controller being installed, but since the daemonset should still have the same name, everything should still work.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @philnichol ,
I think you need to update the command of kubectl get daemonset calico-node --namespace=kube-system as well.
When using Tigera operator, calico is being installed to calico-system namespace and not kube-system

@tsahiduek good spot! Have updated with correct namespace


luci-mod-network: support dhcp.dnsmasq.ipset

Allow to modify the ipset option in the dhcp.dnsmasq config.
This makes possible to populate IP sets with dnsmasq.
Signed-off-by: Vladislav Grigoryev<EMAIL_ADDRESS>

Can you link to an example usage (and also manual describing this)?

https://openwrt.org/docs/guide-user/firewall/fw3_configurations/dns_ipset
https://openwrt.org/docs/guide-user/base-system/dhcp?s=ipset#all_options

Looks great, thanks!


Add Space glyphs

I want to suggest that we add a group of space glyphs somewhere in the glyph lists.
The argument for having them is that they add functionality and broader text compatibility. Unlike a more typical kind of glyph which has a shape these don't add to the data footprint very much at all.
They are also either logical ( based on existing data ) or even better just require a standardized width (assuming a specific EM is in use). I suspect that their addition to fonts is almost certainly scriptable. The work could be done in Glyphs or in FontBakery or both/either.
I have taken my definitions from the MS typography standards document. Here is the list in the order that Glyphs app puts the new glyphs in. This is not unicode order:
emquad - U+2001 - this is equal to the nominal emspace so in a font using an em of 1000 it should be a width of 1000 units.
emspace - U+2003 - as above. The reason for seeming duplication doesn't seem to be explained in spite of a longer and less matter of fact description. 1000 units
enquad - U+2000 - MS says "equal to the en space and one half the em" FYI the normal en space is also 1/2 of the em space. In a 1000 em font this should be a width of 500 units.
fourperemspace - U+2005 - "equal to 1/4 of the em space." In a 1000 em font this should be a width of 250 units.
narrownbspace - U+202F - In a 1000 em font this should be a width of 167 units
sixperemspace - U+2006 - equal to 1/6 of the em space. In a 1000 em font this should be a width of 167 units
threeperemspace - U+2004 300 units
zerowidthjoiner - U+200D - In a 1000 em font this should be no width or 0 units.
zerowidthnonjoiner - U+200C - In a 1000 em font this should be no width or 0 units.
There are also formulas or logics for many of the non-wordspace spaces that are already in the standard glyph lists Google made. If we make a script for the glyphs above we could make sure these are also treated.
figurespace - U+2007 - The width of the default numeral zero in the font
hairspace -  U+200A-  In a 1000 em font this should be 1/16 width of the or 62 units.
punctuationspace - U+2008 - Standard setting is equal to the advance width of the period U+002e and comma U+002c.
nbspace - U+00A0 - The No Break Space width = the width of the space glyph of the font.
thinspace U+2009 - Standard setting is 1/5 of the em space.  In a 1000 em font this should be a width of 200 units. However MS notes : _When traditionally typesetting the French language a word space is inserted before or after several punctuation characters. These characters are _colon, semi colon, question, exclamation, right guillemets, and left guillemets. Commonly the preferred word space used is a thin space of 1/8 the em. Some French typographers prefer to use a larger space character of 1/4 the em with the colon and some other punctuation characters. OpenType supports character substitution and language dependant variants. I am not sure we want language specific variation for this glyph. Maybe we do. But if we don't and we want to cater to those who seem to use this the most e.g. French typographers we could say let's use 125 units. I would like to chat to a few people about it before hand though.
zerowidthspace - U+200B - In a 1000 em font this should be no width or 0 units.
Note: I am not sure glyph list the spaces I will make a case for should be added to, e.g Core, Plus, Pro or Expert. I am can see arguments a range of choices.

If U+2009 isn't a non-breaking space, I believe U+202F should have the same width.
For the French taste (but probably also in the old English taste), some recommend that the value be the third of the normal space. In this case the value varies. For the glyph, I think 125 or 167 units are acceptable values. 250 (1/4 the em) comes from XPress and has nothing to do with traditional French typography (where the ideal normal space could be 250 or 333 units depending on the choice of composition).


Rename parameter names of erode_labels function

In one of the tier*.hpp files and in the erode_labels.cpp file, we need to rename the parameters of the erode_labels function:

"relabel" to "relabel_islands"

@StRigaud this is a test to see if git-bob could do this.

This message was generated by git-bob (version: 0.9.0, model: gpt-4o-2024-08-06), an experimental AI-based assistant. It can make mistakes and has limitations. Check its messages carefully.
Certainly! Here's a proposal for the change:
// Before
void erode_labels(bool relabel, /* other parameters */) {
    // Implementation
}

// After
void erode_labels(bool relabel_islands, /* other parameters */) {
    // Implementation
}

Additionally, update any place where the function is called:
// Before
erode_labels(relabel, /* other arguments */);

// After
erode_labels(relabel_islands, /* other arguments */);

Which files need to be update for this?

This message was generated by git-bob (version: 0.9.0, model: gpt-4o-2024-08-06), an experimental AI-based assistant. It can make mistakes and has limitations. Check its messages carefully.
Certainly! Here's a proposal for the change in the erode_labels.cpp and tier*.hpp files:
// erode_labels.cpp
// Before
void erode_labels(bool relabel, /* other parameters */) {
    // Implementation
}

// After
void erode_labels(bool relabel_islands, /* other parameters */) {
    // Implementation
}

// Update calls to the function
// Before
erode_labels(relabel, /* other arguments */);

// After
erode_labels(relabel_islands, /* other arguments */);

// tier*.hpp
// Before
void erode_labels(bool relabel, /* other parameters */);

// After
void erode_labels(bool relabel_islands, /* other parameters */);

Update any additional places in the codebase where the function is declared, defined, or called with the new parameter name.

The tier*.hpp files doen't exist. There must be a number instead of the * . In which tier*.hpp file is the definition of erode_labels?

This message was generated by git-bob (version: 0.9.0, model: gpt-4o-2024-08-06), an experimental AI-based assistant. It can make mistakes and has limitations. Check its messages carefully.
To address the request for renaming the parameter in the erode_labels function, you should make the following changes in the appropriate files where the function is defined and declared:
Proposed Changes:

In erode_labels.cpp:
// Before
void erode_labels(bool relabel, /* other parameters */) {
    // Implementation
}

// After
void erode_labels(bool relabel_islands, /* other parameters */) {
    // Implementation
}

// Update calls to the function
// Before
erode_labels(relabel, /* other arguments */);

// After
erode_labels(relabel_islands, /* other arguments */);

In the relevant tier*.hpp file where erode_labels is declared (assuming this is in tier6.hpp):
// Before
void erode_labels(bool relabel, /* other parameters */);

// After
void erode_labels(bool relabel_islands, /* other parameters */);

Ensure that you perform a search through the codebase to update all instances where the erode_labels function is called with the old parameter name and change them to the new name relabel_islands. This will ensure consistency and prevent any compilation errors.


Refactor scitokens stuff into a separate library and request scitokens for xrootd

Create a library/credentials.py library that contains functions for requesting and manipulating credentials (currently scitokens but in the future also idtokens and macaroons); modify test_401_scitokens.py to use this new library.
The primary function in credentials is request_scitoken() which requests a scitoken from demo.scitokens.org and sets values in the core.state and core.config dict appropriately.  Created tokens are cleaned up in special_cleanup.py.
core.state['token.NAME_created'] now really means 'did we create this token' instead of 'does this token exist'; condor tests now use os.path.exists() instead.

VMU run forthcoming...

Yup, fine by me


Update esmf-sdk version

update esmf-sdk version to 2.2.3

PR:139 will close this issue.

Hi @richaashara ,
thanks for your feedback and PR. We will take a look over the PR.

Hi @richaashara ,
Thanks for the PR. This PR is incomplete since unit tests are failing.
We are also working on ESMF SDK vesion update.

Updated to 2.4.2. Issue will be closed.


Use matchMedia for orientation detect

We should be using matchMedia instead of manually calculating the orientation of the screen. Checks on window's width and height can trigger layout unnecessarily (https://github.com/Jam3/generator-jam3/blob/master/templates/react/src/util/detect/index.js#L83).
matchMedia on MDN: https://developer.mozilla.org/en-US/docs/Web/API/Window/matchMedia

I agree that window width / height is not the best solution (especially since it caused problems for Android browsers when the keyboard is opened). I'll have to run some tests on matchMedia to see if it its a better solution to the issue.

I tested it and it works great. The only caveat is no IE9 support.

@jeremenichelli might be related: #279

Looks like matchMedia may just be using browser width / height under the hood. It runs into the same issues with the Android Keyboard. I'll close this and evaluate PR #279.


capture window without border respective to location settings

Hi,
great tool!!!
I have problems fixing the capture screen at a desired location on a secondary monitor. My configuration looks like this with a ~17 inch internal laptop monitor and an external 55 inch monitor connected via dockingstation to the laptop:
                              x=0
                               |
                               |

     -  +----------------------+
     ^  |                      |
     |  |                      +----------------+ --- y=0
     |  |       55 inch        |     17 inch    |  ^
2160 |  |   secondary screen   |  main screen   |  |  1080
 px  |  |   external monitor   | laptop monitor |  v   px
     |  |                      +----------------+  -
     v  |                      |
     -  +----------------------+

        |<-------------------->|<-------------->|
                 3840 px             1280 px

Without any changes in configuration, the tool works very well when starting via npm start:

the render window appears in the middle of the main screen (with 1280x720 size)
the capture window appears in the middle of the main screen (on top of the render window)
the thick red border of the capture window is visible
draging and re-sizing the capture window works fine (e.g. drag to secondary screen)
the render window stays always centered on the main screen. adjusting its size accordingly.
the capture window is always on top of all other windows (e.g. on the secondary screen)

Now, I want to statically set the location of the capture screen to cover a certain area on the secondary screen (at startup of the tool). So, I adjust the following elements within src/index.js:
const initCapRect = {
  x: app.commandLine.hasSwitch("cx")
    ? parseInt(app.commandLine.getSwitchValue("cx"))
    : null,
  y: app.commandLine.hasSwitch("cy")
    ? parseInt(app.commandLine.getSwitchValue("cy"))
    : null,
  width: app.commandLine.hasSwitch("cw")
    ? parseInt(app.commandLine.getSwitchValue("cw"))
    : 1280,
  height: app.commandLine.hasSwitch("ch")
    ? parseInt(app.commandLine.getSwitchValue("ch"))
    : 720,
};
const initRenderRect = {
  x: app.commandLine.hasSwitch("rx")
    ? parseInt(app.commandLine.getSwitchValue("rx"))
    : null,
  y: app.commandLine.hasSwitch("ry")
    ? parseInt(app.commandLine.getSwitchValue("ry"))
    : null,
  width: app.commandLine.hasSwitch("rw")
    ? parseInt(app.commandLine.getSwitchValue("rw"))
    : 1280,
  height: app.commandLine.hasSwitch("rh")
    ? parseInt(app.commandLine.getSwitchValue("rh"))
    : 720,
};

The shown default elements for x, y, width and height for both windows (except for x and y of the render window, as they should be set automatically) are changed to:
for capture

x=-2000  (in order to appear on the secondary screen)
y=0
width=1920
height=1080

for render

width=1920
height=1080

After doing so, and restarting the tool via npm start, the following effect shows:

the render window appears in the middle of the main screen (with 1920x1080 size)
the capture window seems to be placed on the secondary monitor (on the secondary screen)
NO thick red border of the capture window is visible
NO draging and re-sizing of the capture window possible (as the border is missing)
the render window still stays always centered on the main screen
the capture window does not seem to stay on top of all other windows (border not visible)

question
Is there a way to locate the capture window on a defined position of the secondary screen with a size of 1920x1080 without loosing the border of the capture window?

I adjust the following elements within src/index.js

Have you tried adjusting the positions via command line options? i.e.
./screen-area-share.exe --cx=-2000 --cy=0 --cw=1920 --ch=1080 --rx=0 --ry=0 --rw=1920 --rh=1080
or with
npm start -- --cx=-2000 --cy=0 --cw=1920 --ch=1080 --rx=0 --ry=0 --rw=1920 --rh=1080
Be sure not to add the --freeze option
https://github.com/mPyKen/ScreenAreaShare?tab=readme-ov-file#command-line-parameters

Hi @mPyKen,
yes, I tried that but got the following result messages within the terminal:
npm start -- --cx=-2000 --cy=0 --cw=1920 --ch=1080 --rx=0 --ry=0 --rw=1920 --rh=1080

><EMAIL_ADDRESS>start
> electron-forge start --cx=-2000 --cy=0 --cw=1920 --ch=1080 --rx=0 --ry=0 --rw=1920 --rh=1080

√ Checking your system
error: unknown option '--cx'

I thought, I did something wrong by misunderstanding the required format using npm.

@nnako sorry, it must be
npm start -- -- --cx=-2000 --cy=0 --cw=1920 --ch=1080 --rx=0 --ry=0 --rw=1920 --rh=1080
(note the additional -- which is required as electron-forge needs to pass these parameters down to the application)

@mPyKen thanks for the correction.
Unfortunately, the effect is exactly the same as documented in my first post. So, providing startup parameters via npm start and directly changing the startup parameters in src/index.js seem to be equivalent.
As I am no native JavaScript'er and even don't know the Electron application framework at all, I would hardly be able to track down the effect, myself, without specific guidance. But I would be willing to further understand the application's structure and then try to contribute. As I already start to like this project very much ;-) .
Would you recommend any specific source of information for getting on track with Electron and your codebase?

Hmm interesting... 2 things I would try:

Could it be that you are starting the application from the terminal application which is on the laptop screen? Could you move the terminal window to the other monitor and then start the application? My random guess is that the capture window is being placed only on the same screen as the one started from, even if it is outside its monitor area
Could you test it with a setup such that there are no negative coordinates? i.e. coordinate (0, 0) would be on the top left?

I currently use only one monitor, being unable to test this myself

Would you recommend any specific source of information for getting on track with Electron and your codebase?

I am no expert on electron, either, it's my first project with it:) But it's not a big application, it mainly consists of the 3 src/*.js files.

Hi @mPyKen ,
I've tried your suggestions.

Could it be that you are starting the application from the terminal application which is on the laptop screen? Could you move the terminal window to the other monitor and then start the application? My random guess is that the capture window is being placed only on the same screen as the one started from, even if it is outside its monitor area.

I create the terminal windows randomly here and there. But, in fact, mostly on the secondary (big) screen where I want the red border to appear together with the capture window. But, unfortunately it makes no change to the behaviour.

Could you test it with a setup such that there are no negative coordinates? i.e. coordinate (0, 0) would be on the top left?

This is, indeed, one thing I have observed (see my previous message). As soon as I insert a negative value for --cx, the red border vanishes.
Maybe, by changing the sequence of operation to something like this, concerning the capture window, would change the effect?:

first create the window (using positive value for --cx)
then do all the other stuff ...
and at the end, move the capture window to the desired position (applying the negative value for --cx

It would be worth a try. Will try to do it as soon as I find the time...

I just saw that when creating the caption window on the secondary screen, the red border is not vanished but the following effects are visible:

the caption window has, in fact, a red border, but
the caption window stays behind all (or most) other windows so that on a crowded screen, it appears not to be visible
by clicking around, it is not possible to get the caption screen permanently into the foreground, as each other window, once activated, seems to keep priority over the caption window
the transfer of information from the caption window to the render window is still working, though

Could you test it with a setup such that there are no negative coordinates? i.e. coordinate (0, 0) would be on the top left?

This is, indeed, one thing I have observed (see my previous message). As soon as I insert a negative value for --cx, the red border vanishes.

What I initially thought was that maybe you could try rearranging your monitors such that the (0, 0) coordinate becomes the top left corner, like this: (that way, there are no negative values anymore)
       x=0
        |
        |

     -  +----------------------+                  --- y=0
     ^  |                      |
     |  |                      +----------------+
     |  |       55 inch        |     17 inch    |  ^
2160 |  |   secondary screen   |  main screen   |  |  1080
 px  |  |   external monitor   | laptop monitor |  v   px
     |  |                      +----------------+  -
     v  |                      |
     -  +----------------------+

        |<-------------------->|<-------------->|
                 3840 px             1280 px

This part sets the capture window always on top and unfocusable. Could you try it without these lines? Maybe they have something to do with the behavior you described.

Changing my main screen to the secondary one had a positive effect, indeed. The red border and the stay-on-top feature of the capture window works now

Interesting... If everything works as intended with positive numbers but not with negative ones, it sounds like some weird problem on electron / chromedriver to me. The electron version currently in use is not the latest one. You could give it a try by updating the dependencies

at least, there are old and relatively new posts about it being an electron issue:

https://github.com/electron/electron/issues/3126
https://current.workingdirectory.net/posts/2023/electron-doesnt-like-negative-numbers/

Please have a look at pull request #18 . This solves the issue concerning Electron's inability to handle negative capture window coordinates.

Thank you for taking the time to investigate and fixing this! Could it be that we need to do the same with the rendering window? I.e. starting at (0,0) and then moving it to the desired position

yes I can do that. although this would not be necessarily applicable to the render window as it has no border and does not have to stay in forground. Will add a respective commit for that.

Thank you for contributing!


Add  bounds for WorkerLocal's Send and Sync

See https://github.com/rust-lang/rust/issues/81425
We could let the auto impl take care of Send if that would be preferable.

Could you add comments to the code justifying why these bounds are correct?
I agree with the change, but we need to remember the reason. :)

Could you add comments to the code justifying why these bounds are correct?
I agree with the change, but we need to remember the reason. :)

Good point, added a comment justifying the Sync bound. I think Send is fine without one since it's the obvious auto-impl version?

Good point, added a comment justifying the Sync bound. I think Send is fine without one since it's the obvious auto-impl version?

I think Send is fine without one since it's the obvious auto-impl version?

Let's just leave it to auto-impl then, as you proposed earlier.

I think Send is fine without one since it's the obvious auto-impl version?

Let's just leave it to auto-impl then, as you proposed earlier.

Done.

Done.

Hmm, combing over this again -- Debug does allow shared access to all locals. So either we do need T: Sync, or we should not print the actual values in Debug.

Hmm, combing over this again -- Debug does allow shared access to all locals. So either we do need T: Sync, or we should not print the actual values in Debug.

Good catch, I completely looked over the Deubg impl. I've changed it so that it doesn't actually print the underlying value to avoid accessing them concurrently. Instead, it now prints the registry that the WorkerLocal is associated with.

Note that this causes compile failures in rustc_middle currently:
    Checking rustc_builtin_macros v0.0.0 (/home/joshua/rustc3/compiler/rustc_builtin_macros)
error[E0277]: `*mut u8` cannot be sent between threads safely
    --> compiler/rustc_middle/src/ty/mod.rs:2788:14
     |
2788 |             .for_each(|&body_id| f(self.hir().body_owner_def_id(body_id)));
     |              ^^^^^^^^ `*mut u8` cannot be sent between threads safely
     |
     = help: within `rustc_arena::DropType`, the trait `Send` is not implemented for `*mut u8`
     = note: required because it appears within the type `rustc_arena::DropType`
     = note: required because of the requirements on the impl of `Send` for `Unique<rustc_arena::DropType>`
     = note: required because it appears within the type `RawVec<rustc_arena::DropType>`
     = note: required because it appears within the type `Vec<rustc_arena::DropType>`
     = note: required because of the requirements on the impl of `Send` for `RefCell<Vec<rustc_arena::DropType>>`
     = note: required because it appears within the type `DropArena`
     = note: required because it appears within the type `Arena<'tcx>`
     = note: required because of the requirements on the impl of `std::marker::Sync` for `WorkerLocal<Arena<'tcx>>`
     = note: required because it appears within the type `&'tcx WorkerLocal<Arena<'tcx>>`
     = note: required because it appears within the type `GlobalCtxt<'tcx>`
     = note: required because it appears within the type `&'tcx GlobalCtxt<'tcx>`
     = note: required because it appears within the type `context::TyCtxt<'tcx>`
     = note: required because it appears within the type `&context::TyCtxt<'tcx>`
     = note: required because it appears within the type `[closure@compiler/rustc_middle/src/ty/mod.rs:2788:23: 2788:74]`

error[E0277]: `*mut u8` cannot be sent between threads safely
    --> compiler/rustc_middle/src/ty/context.rs:1738:13
     |
1738 |             sync::assert_sync::<ImplicitCtxt<'_, '_>>();
     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ `*mut u8` cannot be sent between threads safely
     |
    ::: /home/joshua/rustc3/compiler/rustc_data_structures/src/sync.rs:420:32
     |
420  | pub fn assert_sync<T: ?Sized + Sync>() {}
     |                                ---- required by this bound in `assert_sync`
     |
     = help: within `rustc_arena::DropType`, the trait `Send` is not implemented for `*mut u8`
     = note: required because it appears within the type `rustc_arena::DropType`
     = note: required because of the requirements on the impl of `Send` for `Unique<rustc_arena::DropType>`
     = note: required because it appears within the type `RawVec<rustc_arena::DropType>`
     = note: required because it appears within the type `Vec<rustc_arena::DropType>`
     = note: required because of the requirements on the impl of `Send` for `RefCell<Vec<rustc_arena::DropType>>`
     = note: required because it appears within the type `DropArena`
     = note: required because it appears within the type `Arena<'_>`
     = note: required because of the requirements on the impl of `std::marker::Sync` for `WorkerLocal<Arena<'_>>`
     = note: required because it appears within the type `&WorkerLocal<Arena<'_>>`
     = note: required because it appears within the type `GlobalCtxt<'_>`
     = note: required because it appears within the type `&GlobalCtxt<'_>`
     = note: required because it appears within the type `context::TyCtxt<'_>`
     = note: required because it appears within the type `ImplicitCtxt<'_, '_>`

error: aborting due to 2 previous errors

OK, I think I have the DropArena fix figured out. I'll publish a new release and open a rust PR with that.
Thanks for the fix, @ammaraskar!


fix(udf): add embedded-python-udf feature and fix dockerfile

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This PR makes embedded-python-udf an optional feature. This feature is not enabled by default so that developers are not required to have Python 3.12 installed. To enable it, you can run ENABLE_PYTHON_UDF=1 ./risedev d or cargo build --features embedded-python-udf. This feature is enabled in production build.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

My concern is that off-by-default features could be fragile to maintain as most of the developers won't realize its existence. 😕 Fortunately, check on CI will run with all features. I hope this can enforce the functionality and quality.
https://github.com/risingwavelabs/risingwave/blob/06e28e8dec8b2f2f2ff16bd32f4bcfd79efeaa85/ci/scripts/check.sh#L24

Hmmm, but if we enable it for binary release, users without python3.12 will not be able to start RisingWave. 🤔

Indeed... We should make it static linked.

Strong +1 for this. 🥺

Also I think it's okay to make it a default feature.

It's not a big deal that developers need to install a new build-time dependency.
If the developer is familiar with cargo and rust-analyzer, they can set --no-default-features to improve DX.
All existing build scripts work well.


Need assistance in understanding tab device permission in Mac and Android device

Need assistance in understanding tab device permission in Mac and Android device

Hello @SushmaSathyanarayana - Could you please share all your queries here, so that we can try to help you further?

Hi Chetan,
Thanks for reaching out.
As per the article Request device permissions for your Microsoft Teams app - Teams | Microsoft Learnhttps://learn.microsoft.com/en-us/microsoftteams/platform/concepts/device-capabilities/native-device-permissions?tabs=mobile%2Cteamsjs-v2%2Cmobile1 and GitHub sample Microsoft-Teams-Samples/README.md at main * OfficeDev/Microsoft-Teams-Samples * GitHubhttps://github.com/OfficeDev/Microsoft-Teams-Samples/blob/main/samples/tab-device-permissions/nodejs/README.md the tab permission should work fine in all the client like iPhone, mac, window and Android.
However when one of our customer is running the sample in mac they are not be notified for prompt to enabled the permission or the app is not been listed in the setting->app permission. I verified that the functionality works fine on Windows client. Need your assistance on iPhone and mac.
Thanks in advance!!
Regards,
Sushma S
From: ChetanSharma-MSFT @.>
Sent: Friday, January 20, 2023 5:21 PM
To: OfficeDev/Microsoft-Teams-Samples @.>
Cc: Sushma S @.>; Mention @.>
Subject: Re: [OfficeDev/Microsoft-Teams-Samples] Need assistance in understanding tab device permission in Mac and Android device (Issue #679)
Hello @SushmaSathyanarayanahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSushmaSathyanarayana&data=05|01|sushms%40microsoft.com|db0b3a7ac352434d0b9a08dafadc92e7|72f988bf86f141af91ab2d7cd011db47|1|0|638098122546679644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=iOkDxMXlslZ5X4Ui3rITlVB8sMxE3nQYASvI3e0xJgo%3D&reserved=0 - Could you please share all your queries here, so that we can try to help you further?

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2FMicrosoft-Teams-Samples%2Fissues%2F679%23issuecomment-1398271690&data=05|01|sushms%40microsoft.com|db0b3a7ac352434d0b9a08dafadc92e7|72f988bf86f141af91ab2d7cd011db47|1|0|638098122546679644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=uytu%2Bs7hIgKHJrknZM37WI0UEX7KhELKtyV1IlWGPc8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAW3MI463L6JGQ6Y6YTSHGD3WTJ3ZRANCNFSM6AAAAAAUBNL4GY&data=05|01|sushms%40microsoft.com|db0b3a7ac352434d0b9a08dafadc92e7|72f988bf86f141af91ab2d7cd011db47|1|0|638098122546679644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=GRSqbm%2FcZ%2FnG%2BdVl%2Bcb67d4rPRu8kD1F2HiYAz3PjVI%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @Sushma,
We will test this in iPhone and update you soon.
You can share your Feedbackhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Faka.ms%2FDevSupportFeedback&data=05|01|v-nivedipasu%40microsoft.com|d7624a16170a4a99ea8808daf33e09d1|72f988bf86f141af91ab2d7cd011db47|1|0|638089745518816907|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=wpgbV0A2Eiy18iVhQmrqdtXlV2KXjiH6q46A3DEwIsE%3D&reserved=0/Escalatehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Faka.ms%2FDevCommunityEscalationForm&data=05|01|v-nivedipasu%40microsoft.com|d7624a16170a4a99ea8808daf33e09d1|72f988bf86f141af91ab2d7cd011db47|1|0|638089745518816907|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=mVD%2B3Lg5sbYqnx1ZjVZnhM76wH4c1WvJqh1DjITF9nk%3D&reserved=0
Thanks,
Nivedipa Suryavanshi
Teams Dev Community Response
From: SushmaSathyanarayana @.>
Sent: Friday, January 20, 2023 5:27 PM
To: OfficeDev/Microsoft-Teams-Samples @.>
Cc: Nivedipa Suryavanshi (Tata Consultancy Services Limi) @.>; Assign @.>
Subject: Re: [OfficeDev/Microsoft-Teams-Samples] Need assistance in understanding tab device permission in Mac and Android device (Issue #679)
Hi Chetan,
Thanks for reaching out.
As per the article Request device permissions for your Microsoft Teams app - Teams | Microsoft Learnhttps://learn.microsoft.com/en-us/microsoftteams/platform/concepts/device-capabilities/native-device-permissions?tabs=mobile%2Cteamsjs-v2%2Cmobile1 and GitHub sample Microsoft-Teams-Samples/README.md at main * OfficeDev/Microsoft-Teams-Samples * GitHubhttps://github.com/OfficeDev/Microsoft-Teams-Samples/blob/main/samples/tab-device-permissions/nodejs/README.md the tab permission should work fine in all the client like iPhone, mac, window and Android.
However when one of our customer is running the sample in mac they are not be notified for prompt to enabled the permission or the app is not been listed in the setting->app permission. I verified that the functionality works fine on Windows client. Need your assistance on iPhone and mac.
Thanks in advance!!
Regards,
Sushma S
From: ChetanSharma-MSFT @.>
Sent: Friday, January 20, 2023 5:21 PM
To: OfficeDev/Microsoft-Teams-Samples @.>
Cc: Sushma S @.>; Mention @.>
Subject: Re: [OfficeDev/Microsoft-Teams-Samples] Need assistance in understanding tab device permission in Mac and Android device (Issue #679)
Hello @SushmaSathyanarayanahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSushmaSathyanarayana&data=05|01|sushms%40microsoft.com|db0b3a7ac352434d0b9a08dafadc92e7|72f988bf86f141af91ab2d7cd011db47|1|0|638098122546679644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=iOkDxMXlslZ5X4Ui3rITlVB8sMxE3nQYASvI3e0xJgo%3D&reserved=0 - Could you please share all your queries here, so that we can try to help you further?

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2FMicrosoft-Teams-Samples%2Fissues%2F679%23issuecomment-1398271690&data=05|01|sushms%40microsoft.com|db0b3a7ac352434d0b9a08dafadc92e7|72f988bf86f141af91ab2d7cd011db47|1|0|638098122546679644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=uytu%2Bs7hIgKHJrknZM37WI0UEX7KhELKtyV1IlWGPc8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAW3MI463L6JGQ6Y6YTSHGD3WTJ3ZRANCNFSM6AAAAAAUBNL4GY&data=05|01|sushms%40microsoft.com|db0b3a7ac352434d0b9a08dafadc92e7|72f988bf86f141af91ab2d7cd011db47|1|0|638098122546679644|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=GRSqbm%2FcZ%2FnG%2BdVl%2Bcb67d4rPRu8kD1F2HiYAz3PjVI%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2FMicrosoft-Teams-Samples%2Fissues%2F679%23issuecomment-1398277338&data=05|01|v-nivedipasu%40microsoft.com|9b228f63af484b37995008dafadd74c3|72f988bf86f141af91ab2d7cd011db47|1|0|638098126335956110|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=75olhyopRVr%2BUZuStd%2FGbBE0YpQYd3nOJIrxgSK7Pt0%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAV7BSNUBOMGB47S4VIVEK43WTJ4RHANCNFSM6AAAAAAUBNL4GY&data=05|01|v-nivedipasu%40microsoft.com|9b228f63af484b37995008dafadd74c3|72f988bf86f141af91ab2d7cd011db47|1|0|638098126335956110|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=FgV5AyDy4IubuYg0kVFF7mWMtz10i%2BVRA1p21tQjo3Q%3D&reserved=0.
You are receiving this because you were assigned.Message ID: @.***>

@SushmaSathyanarayana - We have tried on IOS device, it is working fine from our end.
Sample Link-https://github.com/OfficeDev/Microsoft-Teams-Samples/tree/main/samples/tab-device-permissions/nodejs
IOS version- 16.2
IOS Teams Version-4.23.772022235001/0110
Also If you want to add permissions on desktop you can click on App Permission icon-

https://user-images.githubusercontent.com/93527375/213865862-678b4e02-91b5-4773-9d44-982cd1a5208e.mp4

Thanks Sayali.
I will share an update to cx and get back if there are any questions.
Regards,
Sushma S
From: sayali-MSFT @.>
Sent: Saturday, January 21, 2023 5:32 PM
To: OfficeDev/Microsoft-Teams-Samples @.>
Cc: Sushma S @.>; Mention @.>
Subject: Re: [OfficeDev/Microsoft-Teams-Samples] Need assistance in understanding tab device permission in Mac and Android device (Issue #679)
@SushmaSathyanarayanahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSushmaSathyanarayana&data=05|01|sushms%40microsoft.com|08194f28b0b3431da6b308dafba75f18|72f988bf86f141af91ab2d7cd011db47|1|0|638098993550604733|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=HL7mPl%2F%2BtlsnaoHNyreQWRR9%2FqZyTDrIUwIbdMmfXg8%3D&reserved=0 - We have tried on IOS device, it is working fine from our end.
Sample Link-https://github.com/OfficeDev/Microsoft-Teams-Samples/tree/main/samples/tab-device-permissions/nodejshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2FMicrosoft-Teams-Samples%2Ftree%2Fmain%2Fsamples%2Ftab-device-permissions%2Fnodejs&data=05|01|sushms%40microsoft.com|08194f28b0b3431da6b308dafba75f18|72f988bf86f141af91ab2d7cd011db47|1|0|638098993550604733|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=VtlO8SIbatYXnJ%2B%2BcP0tMNqx5xOX2VetauaFCIkMjkA%3D&reserved=0
IOS version- 16.2
IOS Teams Version-4.23.772022235001/0110
Also If you want to add permissions on desktop you can click on App Permission icon-
[Image removed by sender. tabdevicepermiison]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F93527375%2F213864475-88caee15-cc78-4480-b296-b814f42c8af9.png&data=05|01|sushms%40microsoft.com|08194f28b0b3431da6b308dafba75f18|72f988bf86f141af91ab2d7cd011db47|1|0|638098993550604733|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=88Op0RHjiqJ1TPGUGqVCK1VSNClwghEyF9wD7Uei7%2F4%3D&reserved=0
https://user-images.githubusercontent.com/93527375/213865862-678b4e02-91b5-4773-9d44-982cd1a5208e.mp4https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F93527375%2F213865862-678b4e02-91b5-4773-9d44-982cd1a5208e.mp4&data=05|01|sushms%40microsoft.com|08194f28b0b3431da6b308dafba75f18|72f988bf86f141af91ab2d7cd011db47|1|0|638098993550604733|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=4Up9r7f9gD3dOovVf%2FNWTSCHSlVjg6mWWfa7yARcpns%3D&reserved=0
[Image removed by sender. MicrosoftTeams-image (30)]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F93527375%2F213865896-61458ac5-3b94-443f-bfb3-9670c4f98536.png&data=05|01|sushms%40microsoft.com|08194f28b0b3431da6b308dafba75f18|72f988bf86f141af91ab2d7cd011db47|1|0|638098993550604733|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=xzWaueELJ5OMwzGGY9YVXDNDsR4pYMWmZ%2F57DoVPXn8%3D&reserved=0

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2FMicrosoft-Teams-Samples%2Fissues%2F679%23issuecomment-1399238107&data=05|01|sushms%40microsoft.com|08194f28b0b3431da6b308dafba75f18|72f988bf86f141af91ab2d7cd011db47|1|0|638098993550604733|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=2Zv8RTpP%2FqMbiXuXh13h8XtzIvFcKX6rw9hBMrsakZk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAW3MI45GUUZQR7CXVKSJYC3WTPF5LANCNFSM6AAAAAAUBNL4GY&data=05|01|sushms%40microsoft.com|08194f28b0b3431da6b308dafba75f18|72f988bf86f141af91ab2d7cd011db47|1|0|638098993550604733|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=gqBYfMZ%2BZXGFr%2FSUkELVT%2FDJ%2BDYAyDCca4qWDqrILgI%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Hello @SushmaSathyanarayana  - Hope your queries are resolved.
We are closing this issue for now, please feel free to reopen it, if required.


uib-alert close button not rendered when using ng-bind

Hello ui-bootstrappers
This isn't a major bug, but it had me confused for a while, so might be worth investigating or documenting.
Bug description:
When using ng-bind (or similar) to set the the inner content of a uib-alert, the close button does not get rendered. For instance:
<uib-alert ng-repeat="alert in alerts" type="{{alert.type}}" close="closeAlert($index)" ng-bind="alert.msg"></uib-alert>
http://plnkr.co/edit/DrZzUoAqvDKu4a7NGDI5?p=preview
Angular: 1.5.3
UIBS: 1.3.2
Bootstrap: 3.3.5

@stevie-p, this is because we're using transclude on our template.  if you want to play with this to see if there's a way around it, go ahead.  be mindful of creating unnecessary extra watches.
if you don't care, i'm just going to close this as won't fix.

Thanks @icfantv - let's close the issue.
I might have a look at the source and think about it when I have time...


Refactor monitoring deploying

Problem:

Operator operation is scattering in cluster level monitoring enabling
and altering enabling
Disable cluster-level monitoring but don't withdraw project-levels
project-level monitoring doesn't deploy Prometheus
Don't support multi tanancy for alerting
If user don't open project-level monitoring could not see the project metric

Solution:

Use a separated handler to loop Operator
Modify all projects Spec.EnabledProjectMonitoring to false before
owner-cluster withdraw its monitoring apps
Add Prometheus deploying when installing project-level monitoring app
Use project-level Prometheus for project-level alerting
Use cluster-level Prometheus and add service account when query then could
only get current project data

Issue:

#16932
#17039
#16986

Related:

https://github.com/rancher/system-charts/pull/8

@loganhz , @orangedeng


Does not always work correcly on webkit-based browsers

in setBoxHeights a copy of a element is made, inserted into the DOM, and then the height is calculated. This sometimes fails on webkit-based browsers, because insertAfter may return before the DOM has actually been updated, which results in an incorrect height calculation. Thus, reaqdmore.js only works with very simple content using webkit, when the insertion is fast enough that it has finished before the call to el.outerHeight().

Can you post a Gist with an example of this? I am unable to reproduce.

Have the same issue on Safari 8.0.6 Wrong height of element is calculated. Looks like the images could cause the problem.
Example: http://mobilitaet2050.vcd.org/community/unterwegs/nichtohnemeinfahrrad/
Second element "Georg - Dein Transportrad für Bocholt"
expandedHeight in safari: 887px
expandedHeight in chrome: 1989px (correct)

Yes. Images are making the calculations too low. I tried using imagesLoaded.js to see if that would help the calculations, but no joy.


Attributes are not parsed for record fields

typedef struct {
  long long __clang_max_align_nonce1
      __attribute__((__aligned__(__alignof__(long long))));
  long double __clang_max_align_nonce2
      __attribute__((__aligned__(__alignof__(long double))));
} max_align_t;

Should we be attaching record field attributes to the field's type, or should we be putting them somewhere else?

Adding them to the fields type should be a good starting point. If it isn't correct we'll find out as a part of #42.


Add support for Linode as a node driver

See this PR: https://github.com/rancher/rancher/pull/17848
The Linode team added support for Linode as an optional node driver.
Linode team will handle functional testing of the driver.
I would like our QA team to just assert that it can be turned on and off approprirately, it is off by default, and that the UI loads.
cc @deniseschannon @displague

Verified with rancher:master(02/13/2019)
Linode node driver is available on the UI. Linode node driver is OFF by default and can be turned off and on appropriately. UI loads with the Linode driver when turned ON.


Use multiple gradients on a single ingredient

Problem
Closes #262
Solution
Added support to allow multiple gradients on a single ingredient at the ingredient level.
Type of change

New feature (non-breaking change which adds functionality)

Change summary:

Allow the gradient attribute of an ingredient to be a List instance with multiple gradients.
While creating an ingredient, check if it requires multiple gradients. If yes, add an attribute combined_weight that specifies the probabilities of the grid point to be picked to place the ingredient.
Refactor functions in the Gradient class to be static methods.
Add example recipe and edit config.

Steps to Verify:
Pack a recipe with an X and Y gradient combined:
pack -r cellpack/tests/recipes/v2/test_combined_gradient.json -c cellpack/tests/packing-configs/test_grid_plot_config.json

Screenshots:
Weight map:

Simularium screenshot

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.73%. Comparing base (843dfec) to head (7a880a9).
Report is 11 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #270      +/-   ##
==========================================
- Coverage   98.75%   98.73%   -0.03%
==========================================
  Files          19       18       -1
  Lines         563      553      -10
==========================================
- Hits          556      546      -10
  Misses          7        7

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


TSServer 2.0.6 fails with filenames without an extension

TypeScript Version:  2.0.6
Code
File bin\compile (note the file has no extension)
#!/usr/bin/env node

console.log('')

Opening the file and requesting a language feature (hover, code complete) results in an exception in the tsserver:
[Info  - 12:11:55 PM] Using tsserver from location: c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js
[Trace - 12:11:55 PM] Sending request: configure (0). Response expected: yes. Current queue length: 0
Arguments: {
    "hostInfo": "vscode"
}
[Trace - 12:11:56 PM] Response received: configure (0). Request took 245 ms. Success: true
[Trace - 12:11:56 PM] Sending request: compilerOptionsForInferredProjects (1). Response expected: yes. Current queue length: 1
Arguments: {
    "options": {
        "module": 1,
        "target": 2,
        "allowSyntheticDefaultImports": true,
        "allowJs": true
    }
}
[Trace - 12:11:56 PM] Response received: compilerOptionsForInferredProjects (1). Request took 247 ms. Success: true
Result: true
[Trace - 12:11:56 PM] Sending request: open (2). Response expected: no. Current queue length: 0
Arguments: {
    "file": "p:/mseng/VSCode/vscode-extension-vscode/bin/compile",
    "fileContent": "#!/usr/bin/env node\n\nconsole.log('')",
    "scriptKindName": "JS"
}
[Trace - 12:11:56 PM] Sending request: geterr (3). Response expected: no. Current queue length: 0
Arguments: {
    "delay": 0,
    "files": [
        "p:/mseng/VSCode/vscode-extension-vscode/bin/compile"
    ]
}
[Trace - 12:11:56 PM] Sending request: projectInfo (4). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "p:/mseng/VSCode/vscode-extension-vscode/bin/compile",
    "needFileNameList": true
}
[Trace - 12:11:57 PM] Event received: configFileDiag (0).
Data: {
    "configFile": "p:/mseng/VSCode/vscode-extension-vscode/tsconfig.json",
    "diagnostics": []
}
[Trace - 12:11:57 PM] Response received: projectInfo (4). Request took 1091 ms. Success: true
Result: {
    "configFileName": "/dev/null/inferredProject1*",
    "languageServiceDisabled": false,
    "fileNames": [
        "c:/Program Files (x86)/Microsoft VS Code Insiders/resources/app/extensions/typescript/node_modules/typescript/lib/lib.es6.d.ts",
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>    ]
}
[Trace - 12:13:03 PM] Sending request: quickinfo (5). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "p:/mseng/VSCode/vscode-extension-vscode/bin/compile",
    "line": 3,
    "offset": 11
}
[Trace - 12:13:03 PM] Response received: quickinfo (5). Request took 2 ms. Success: false . Message: Error processing request. Could not find file: 'p:/mseng/VSCode/vscode-extension-vscode/bin/compile'.
Error: Could not find file: 'p:/mseng/VSCode/vscode-extension-vscode/bin/compile'.
    at getValidSourceFile (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:46883:23)
    at Object.getQuickInfoAtPosition (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:48776:30)
    at IOSession.Session.getQuickInfoWorker (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:54807:62)
    at Session.handlers.ts.createMap._a.(anonymous function) (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:54114:61)
    at IOSession.Session.executeCommand (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:55223:28)
    at IOSession.Session.onMessage (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:55243:35)
    at Interface.<anonymous> (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:56300:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:188:7)
    at Interface._onLine (readline.js:232:10)
[Error - 12:13:03 PM] 'quickinfo' request failed with error.
Error processing request. Could not find file: 'p:/mseng/VSCode/vscode-extension-vscode/bin/compile'.
Error: Could not find file: 'p:/mseng/VSCode/vscode-extension-vscode/bin/compile'.
    at getValidSourceFile (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:46883:23)
    at Object.getQuickInfoAtPosition (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:48776:30)
    at IOSession.Session.getQuickInfoWorker (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:54807:62)
    at Session.handlers.ts.createMap._a.(anonymous function) (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:54114:61)
    at IOSession.Session.executeCommand (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:55223:28)
    at IOSession.Session.onMessage (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:55243:35)
    at Interface.<anonymous> (c:\Program Files (x86)\Microsoft VS Code Insiders\resources\app\extensions\typescript\node_modules\typescript\lib\tsserver.js:56300:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:188:7)
    at Interface._onLine (readline.js:232:10)

This worked without problem with TS 2.0.3
@mhegazy @vladima could you please look into a fix. This is a regression and we should look into fixing this for VS Code 1.7 which ships mid / end next week.

@dbaeumer I cannot repro it as-is by just creating file without extension and opening it. However in the log I can see that you also have config file p:/mseng/VSCode/vscode-extension-vscode/tsconfig.json. Can you also paste it here?

The tsconfig.json is:
{
    "compilerOptions": {
        "target": "es5",
        "module": "commonjs"
    },
    "exclude": [
        "node_modules",
        "vscode.d.ts",
        "typings"
    ]
}

You can also clone: https://github.com/microsoft/vscode-extension-vscode

@dbaeumer  We've fixed the original issue in master however at this point 2.0.5 is already locked and it is pretty hard to put anything there. As a workaround on your side you can:

set internal option allowNonTsExtensions to true in compilerOptionsForInferredProjects request
don't send compilerOptionsForInferredProjects at all - default compiler options on the tsserver side already have allowNonTsExtensions being enabled

Should be fixed by  #11922

@mhegazy could this be backported to 2.0.6. We ship 1.7.x this week and this is basically a regression for us.