LDAP auth for mysql protocol connections

https://github.com/youtube/vitess/pull/2440 adds support for the mysql binary protocol. This should allow users to connect to the VTGate using tools like SequelPro, mysql workbench, etc.
We'd like to support LDAP auth for those client connections. We use LDAP for both authentication and authorization, and would like to be able to enforce this at the VTGate as well. For instance, we would bind to LDAP with the passed username/password, then do an LDAP search using the username and some configurable query pattern. Depending on the result, the user would be allowed or denied access to the requested keyspace.
We were planning on building this into g10's mysql proxy initially, until this new mysql protocol work began which negates the need for the proxy. Now the logic makes senses to live in the VTGate.
This issue is to gauge the viability of such a system in VTGate. We'll be looking to begin implementing this in the very near future, and would obviously prefer to get it merged. Our plan would be to make a pluggable auth package, with an implementation for LDAP that is configurable to our needs (i.e. the queries described above).
Thoughts?

Let me start with the way MySQL native auth works: the server sends a challenge to the client, and then both client and server transform that same challenge using the password and the same transformation. The client sends its transformed version, the server compares with its transformed version, if they match then the user is allowed. What is worth noting is the server never sees the client password, but only the transformed challenge. And the transformation is not reversible (it's a couple layers of checksuming).
In this page:
https://dev.mysql.com/doc/refman/5.5/en/cleartext-authentication-plugin.html
it is also explained that clear text password can be sent up, and used by the server. This is used by PAM plugin for instance. I think we would need this here too.
But this is not secure at all, if we are not using SSL. If using SSL, the SSL negotiation happens before the password exchange, so it is secure. Without SSL, the password is sent in the clear. So we should at least echo a warning if clear passwords are used, and not SSL.
That being said, setting up the plug-in mechanism and implementing one plug-in that uses a simple JSON file with users and passwords in it is easy enough, I can do that. I actually need to do that so vtgate's server connector can recognize more users than just the hard-coded one.
As far as table ACLs are concerned, we can use the user used to log in as the VTGateCallerID (AKA immediate caller id), maybe with a prefix. So if you log in to vtgate's mysql binary protocol connector with 'user1', then 'user1' is passed to vttablet, and does table ACL checks on that. (note: here we can add a level of indirection, and map mysql user name to vttablet ACL name. Is that necessary? In the file case, it would be an extra field, with default to user name. In the LDAP case, it could be a user attribute).

Thanks for the detailed investigation/explanation @alainjobart. I hadn't realized that is how the challenge system works.
Can the vtgate mysql implementation work with SSL? Because I agree we're going to want that rather than sending the password in clear text. I think we'd even want to just support SSL for our case.
I think what you mentioned sounds like a good first start. In terms of your question towards the end "Is that necessary?": for us, yes. Like you suggested, we'd want to be able to use a user attribute as the immediate caller id.  For us, every LDAP user has a "group" field, which defines the engineering team their on. So I'd like to define an ACL like "this keyspace can only be queried by hs-foo", where hs-foo is a group/team name.
If you can provide the plugin setup, and thoughts on the SSL configuration, we can write an ldap plugin. I would think such a plugin would loosely be configurable with an ldap host to bind to, and a field or search query to use as the immediate caller id.

A few answers:

I am working on SSL with our MySQL implementation. The main problem there is tests (create certs, keys for unit tests) and I am reworking this now. Should be ready soon.

Next on my list is the plugin interface for the server side, and a file-based implementation. I will probably make it generic enough so it supports both the challenge-based method, and the clear text password method. It will allow the plug-in to set the immediate caller id. The file-based version will just use an extra field for that, and if not set, use the user name.

For test purposes, I may have two versions of the file-based auth, one with the challenge, one with clear-text passwords.

For clear text support, I'll make sure there is a check to only allow that over SSL. And it will probably be on by default.

I'll let you work on your LDAP plugin. It should be fairly straightforward, as you can use any flag you want, and link it in vtgate.

Thanks @alainjobart, this sounds great!

SSL support is added in this PR: https://github.com/youtube/vitess/pull/2567

This is done.


Option to limit price scale range

I have a a set of baseline series that are always positive and I'd like to be able to zoom in on a narrow price range but don't want to allow the price scale to get into negative territory.
The ideal solution would be to have minimum and maximum options on the price scale that act like fixLeftEdge and fixRightEdge except that they are specified with a constant value instead of being based on the range of the data.

When the chart is automatically adjusting the price scale min and max range values, this is called autoScale and what it is doing is automatically adjusting the scale to ensure that all the data fits within the chart.
You can disable autoScale on the Price scale with: https://tradingview.github.io/lightweight-charts/docs/api/interfaces/PriceScaleOptions#autoscale
You could also use the autoscaleInfoProvider option (which can be applied to any series via the series options) which allows to you customise the scaling. So if you used autoscaleInfoProvider on all the series, and made it return the minimum and maximum values which you would prefer then you could achieve the desired effect.

https://tradingview.github.io/lightweight-charts/docs/api/interfaces/SeriesOptionsCommon#autoscaleinfoprovider

@SlicedSilver Thanks for the idea. My feature request might not have been clear enough that the limit is about how far the user can adjust the scale (by dragging on the axis or chart area). The autoscale settings don't limit this, so I don't think it will help in this case.

You are correct. AutoScaleInfo won't help for that case.
I'll mark the issue as a feature request.

Solution?


veleroDeployment: Create a Service and ServiceMonitor

Create Service and ServiceMonitor so that metrics can be scraped

/assign

Rebased on the latest master branch revision and squashed in all suggestions except for the service refactoring.

/lgtm
/approve

Reapplying  lgtm since I just squashed @cblecker's suggestions into the 2nd commit.  No other changes.


chore: pause emit pause reason

I don't see where we are setting the pause reason for manual pauses. Can we add that?

The way @tomkit implemented this, the notification / email path is only triggered for autopaused workflows. Doing this would require a much more significant change. (I agree with you though)

I don't see where we are setting the pause reason for manual pauses. Can we add that?

The way @tomkit implemented this, the notification / email path is only triggered for autopaused workflows. Doing this would require a much more significant change. (I agree with you though)

We do send webhooks on manual pauses though. It would be good to have the reason there. (That's what's powering the slack notifications)

I don't see where we are setting the pause reason for manual pauses. Can we add that?

The way @tomkit implemented this, the notification / email path is only triggered for autopaused workflows. Doing this would require a much more significant change. (I agree with you though)

We do send webhooks on manual pauses though. It would be good to have the reason there. (That's what's powering the slack notifications)

Done.


longest palindromic substring in given string

Related Issue
#145
Checklist

[X] ✅ My code follows the code style of this project.
[ ] 📝 My change requires a change to the documentation.
[ ] 🎀 I have updated the documentation accordingly.
[X] 👀 I have read the CONTRIBUTING document.
[ ] ✨ I have added tests to cover my changes.
[ ] 🚩 All new and existing tests passed.
[ ] 🌟 ed the repo

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Br
[ ]

eaking change (fix or feature that would cause existing functionality to change)

isuue #145

i am closing this pr please merge i will open it again after my another pr will be merged


german support

just a few code formatting issues. Looks good to me but someone else must approve :)


Keeping MV3 extension service worker alive using CDP?

I needed to migrate a private MV2 extension to MV3, but instead I tried emulating it using this repo. Got most of it working, but the extension uses the chrome.proxy api, which I couldn't find a way to replicate using the CDP, so I had to do that part in a MV3 extension.
But then I encountered this issue: https://bugs.chromium.org/p/chromium/issues/detail?id=1152255 (tried a few of the workaround from there but didn't seem to work)
I know I could just launch chrome with the --proxy-server flag, but I'd like to be able to set and change the proxy without having to restart chrome.
Since opening the extension service worker devtools window manually on chrome does keep the sw alive, I was wondering if there's is a way to keep it alive using the CDP? I do see the sw target using Target.getTargets(), and tried sending a few random commands, but it still becomes inactive after like 30 seconds of inactivity.

Since opening the extension service worker devtools window manually on chrome does keep the sw alive, I was wondering if there's is a way to keep it alive using the CDP?

Hmm, there shouldn't be in principle any difference between DevTools and a script using the CDP. You can try to see what DevTools is doing under the hood and try to replicate that in your script, maybe there's something that allows the SW to stay alive.
To do that you can inspect DevTools itself.


Gvim stable version 8.2.2824 colorscheme is broken.

Steps to reproduce

On Windows, download vim stable using scoop:

scoop install vim

Using this minimal vimrc:

call plug#begin('~/vimfiles/plugged')
Plug 'https://github.com/sainnhe/edge.git', {'as': 'edge'}
call plug#end()
set termguicolors
colorscheme edge

Open Gvim and see the result

Expected behaviour
Display right colorscheme, not white background
Version of Vim
8.2.2824
Environment
OS:  Windows 11
Term: Windows Powershell
Logs and stack traces
I thought this is a bug cause by this colorscheme but it's not. Here is the link to my question

okay, can you please minimize this: Does it work without scoop? Can you simply download the colorscheme so we can test without installing Plug? Is that in the terminal or in the gui? Not clear from your description.

okay, can you please minimize this: Does it work without scoop? Can you simply download the colorscheme so we can test without installing Plug? Is that in the terminal or in the gui? Not clear from your description.

No, it didn't work with version 8.2.2824. I tried to download .exe file from vim-win32. It was broken both terminal and gui vim in window but you can see it easier with GUI

can you please provide a screenshot and mark what exactly is wrong?

I checked it out locally:

with :set bg=dark

with :set bg=light

Apart from the fact that the light is very light and hard to read, what exactly is wrong?

I opened GVim and run :e $myvimrc to show you minimal vimrc, and also version of vim. It's should be dark edge default theme not the light screen.

Possible duplicate of https://github.com/vim/vim/issues/8150

Then, closing.

default background for gvim --clean is light, so please use :set bg=dark

Here is your :set bg=dark. This is the error with this version, not duplicate or something like that.

I found exactly what happening in here. It caused by scoop install vim. It went wrong in all version with scoop install vim or vim-nightly. .exe file still working. It seems problem with scoop.


[Feat]: IP drone Python Script

Is your feature request related to a problem? Please describe.
Ipdrone is a simply python script, which can be used to Ip lookup and to get information of particular target Ip. This tool works on both rooted Android device and Non-rooted Android device.
Describe the solution you'd like.

Real live location
Updated maintainence
Ip lookup
Easy For Beginners

Describe alternatives you've considered.
No alternatives
Show us the magic with screenshots
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Already Contributed to Pentesting Repo


number of next word predictions at translation time

Dear nematus community,
It has been couple of months I have started to deal with neural machine translation. I have a question related to nematus code, and I am sorry if it is so primitive for you.
In translation time, in the function gen_sample() under nmt.py, next_w predictions are not one, but it varies "as I have experimented up to now" up to 5, why is it?
So basically, I am getting alignment vector, context vector and next word predictions at translation time under the following loop:
# x is a sequence of word ids followed by 0, eos id
for ii in xrange(maxlen):
But the number of values at each time step at alignment vector, context vector and next word vector is more than one, up to 5.
Why?
Thanks for your time and answer.
Kind Regards,

Hi, I think that you get one alignment vector, context vector and next word vector for each one of the beams in your beam search, try to run the training with -k 3 ( -k is for the beam search's size) for example and if you will find out that instead of 5 of each you get 3 of each then my guess is probably correct

Hi @yftah89 so you mean that the size of the vectors are basically beam search size right?

Yes, this is exactly my assumption, an easy way to verify it is to simply change the beam search default value (5 if I'm not wrong) the different one by adding -k 3 for example and to check if you get 3 of a kind for all of the things you mentioned above.
Regardless, if you use beam search you will have to save all of those values for each beam if not at the point you are talking about than in some other point.


neorv32_cpu_get_cycle() always returns zero

I just created a new project using the script in the boards folder. The bitstream is generated as expected and the bootloader is working. But when testing the blink-example, nothing happens.
After some investigation I realized, that the function
neorv32_cpu_get_cycle();
always returns zero, and therefore the function
neorv32_cpu_delay_ms(200);
used in the blink-example never returns.
The behaviour is the same for the ARTY and the NEXYS examples.

Oh crap!
Seems like my last modifications of crt0.S had some side effects I did not consider...
Right now, ctr0 sets all bits of the mcountinhibt CSR so all counters are stopped until they are explicitly enabled by writing zero to the according mcountinhibt bit.
I will fix that.
Thanks for the hint!

Should be fixed now.

It works again, great. I guess this issue may be closed now.


Credits

Merely opening this pull request for easier editing & also people can comment if something's up or i'm accidentally missing someone.

Will rebase when done, lol.

@electronoob you'll have to add
Credits
 to skins.agariomods.com and connect.agariomods.com

yep just let me know when you want it done.


Implement theming in avocado

Is your feature request related to a problem? Please describe.
Design systems have to provide some way to customize them. Avocado currently works with just one theme (the default). It would be great to enable theming so projects using avocado can look different and unique.
Describe the solution you'd like
A way to specify design tokens that change the look and feel of interfaces built with avocado.
Describe alternatives you've considered
I've considered a few approaches

use theme-ui under the hood and adhere to the spec
use css variables that can be overridden with a config file

Additional context
Theming is a core part of my experiment with avocado. Of course, I could leave this out for now and focus on building more components, but I think it's something foundational and should be handled now.

I've spent a few hours researching this and I think theme-ui is the way forward.
This was hard to conclude. Majorly because I didn't like the extra props theme-ui would add to my components (sx,mr etc.). But this is the only logical way to achieve it.
So I'm going to be working on incorporating theme-ui on another branch.

After a few days of experimenting here are a few conclusions I've come to
I don't really need theme-ui: It took a full day of experimenting with theme-ui to know I didn't need it. A few things theme-ui offers

ThemeProvider: to pass theme props to all components of the app
merge: merge is a function that allows you to combine themes
The sx prop that allows you do inline theming (example below). I really don't like this API and it's part of the reason I hesitated to go with theme-ui.
A few hooks for managing modes.

// `sx` props example
export default (props) => (
  <div
    sx={{
      fontWeight: 'bold',
      fontSize: 4, // picks up value from `theme.fontSizes[4]`
      color: 'primary', // picks up value from `theme.colors.primary`
    }}>
    Hello
  </div>
)

If I want to be honest with myself, I only need the ThemeProvider and the merge function. I don't even need the merge function if I can do some merging myself.
Since I'm using emotion which already exposes a ThemeProvider that I can use to pass a theme value to other components. As for merging themes, I can do something like this

import React from 'react'
import { theme, ThemeProps } from 'avocado-ui'
import { ThemeProvider } from '@emotion/react'

import { Header } from './components'

// extend the theme from the default theme.
// There's a current problem where we have to ensure the type matches ThemeProps
// We have to do this because there's no way to combine themes incrementally like what merge does in theme-ui or system-ui
// FIXME: Implement a merge function to merge themes together without doing all the extra work

const customTheme: ThemeProps = {
  ...theme,
  colors: {
    ...theme.colors,
    red: {
      ...theme.colors.red,
      '100': 'blue'
    }
  },
  components: {
    ...theme.components,
    buttonTheme: {
      ...theme.components.buttonTheme,
      variants: {
        ...theme.components.buttonTheme.variants,
        error: {
          default: 'blue',
          hover: 'red'
        }
      }
    }
  }
}

const App = () => {
  return (
    // Pass custom theme to theme provider
    <ThemeProvider theme={customTheme}>
      <Header />
    </ThemeProvider>
  )
}

export default App

With this approach, I don't need theme-ui and will have my own custom theme specification.

This approach didn't work. Commit experiment here.
It worked for custom components the user of the design system would create, but not for components exported from the design system. So I guess I have to go back to theme-ui since this part worked flawlessly with it.

So I thought of this differently.
There's a lot of moving parts when working with theming and it's not something I have all the time to implement. I'll come back to this, but let's see how far we can go with building new components first.
I have a feat-theming branch where I'll be doing the experiments.


Add progress bar to kusk install

When running the kusk install I wasn't sure if my terminal hanged or the process failed as it looked like the following

Could we add a spinning wheel? 🙃

Could use something like this https://github.com/janeczku/go-spinner

I would like to work on this if no one is working right now @kylehodgetts

@gunishmatta by all means! We would be happy to receive your contribution!

@gunishmatta echoing Kyle, please feel free to do it!! Thanks a lot for the interest :)
As a note, the file that is responsible for the command is https://github.com/kubeshop/kusk-gateway/blob/main/cmd/kusk/cmd/install.go , you'd start looking there if you want to add a spinner :)
Please let us know if you decide to start on it!

I am starting on it.
@aabedraba

@gunishmatta were you able to give it.. a spin? :)
Please let me know if you'd like me to help you get this done!


No support for indels

The pathogenic genotypes of some relevant SNPs (like rs387906510 and rs398124239) are defined by deletions, which are not currently supported by SNPeek.

I've implemented this in a branch.


Create auto_catchup.yml

Part of https://github.com/WideChat/Rocket.Chat/issues/486
Merge after https://github.com/WideChat/Rocket.Chat.Apps-cli/pull/1

Thanks!


Reusing mock definitions across multiple services

This is a feature request more than a bug, unless this is already possible in which case I would love to know how!
I'm tweaking the generated mocks via the orval config file. This works great for the most part, but I have some common patterns/helpers that I would love to be able to reuse effectively, e.g. a faker definition for various types that are used across the application.
Is there a way to include these in an external file and reuse them across definitions? I tried simply using imports in the orval config file, but as the imports do not carry over into the generated files, it's not possible to use the generated code.

Hello @SoftMemes, can you provide a little repo with an example?


Unbreak the initialization of /etc/hosts and /etc/resolv.conf

The unary logical negation operator (ie. !) was getting associated with
the 'cd /etc' instead of the entire sequence. As a result, neither
/etc/hosts nor /etc/resolv.conf were getting symlinked.
Fallout from 8b84b5e4604921fad818ede5267591de7f0993e6

Thanks @jamescassell


No "include" folder in 20160216_win64_gpu.7z

Hello,
I'm trying to install MXNet with GPU support in R. As per the installation instructions I need to take the include folder from 20160216_win64_gpu.7z and paste it in mxnet/R-package/inst. I tried using the include folder from the previous version but was having trouble. Am I missing something?
Thank you!

Sorry about that... I just re-uploaded the packages. It is ok now.
On Thu, Feb 18, 2016 at 1:09 AM, dave5556<EMAIL_ADDRESS>wrote:

Hello,
I'm trying to install MXNet with GPU support in R. As per the installation
instructions I need to take the include folder from 20160216_win64_gpu.7z
and paste it in mxnet/R-package/inst. I tried using the include folder from
the previous version but was having trouble. Am I missing something?
Thank you!
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/issues/1501.

--
HONG Chuntao
System Research Group
Microsoft Research Asia

Thank you


roxterm: 3.14.3 -> 3.15.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/roxterm/versions.
meta.description for roxterm is: A highly configurable terminal emulator
meta.homepage for roxterm is: https://github.com/realh/roxterm
meta.changelog for roxterm is: https://github.com/realh/roxterm/blob/3.15.0/debian/changelog
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.15.0 with grep in /nix/store/6a4csg3pgsnz0nv97aifk8zh5mws7ilk-roxterm-3.15.0
found 3.15.0 in filename of file in /nix/store/6a4csg3pgsnz0nv97aifk8zh5mws7ilk-roxterm-3.15.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

roxterm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6a4csg3pgsnz0nv97aifk8zh5mws7ilk-roxterm-3.15.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A roxterm https://github.com/r-ryantm/nixpkgs/archive/880754ba3a403376f9d5e3ddde35854985cacf34.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/880754ba3a403376f9d5e3ddde35854985cacf34#roxterm

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6a4csg3pgsnz0nv97aifk8zh5mws7ilk-roxterm-3.15.0
ls -la /nix/store/6a4csg3pgsnz0nv97aifk8zh5mws7ilk-roxterm-3.15.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

roxterm

Maintainer pings
cc @AndersonTorres for testing.

In light of the problems solved at https://github.com/NixOS/nixpkgs/pull/296778, please someone with enough power put this on draft.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/roxterm/versions.
meta.description for roxterm is: A highly configurable terminal emulator
meta.homepage for roxterm is: https://github.com/realh/roxterm
meta.changelog for roxterm is: https://github.com/realh/roxterm/blob/3.15.2/debian/changelog
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.15.2 with grep in /nix/store/pa87py9m9xyyq76l34llc2mk88h1384r-roxterm-3.15.2
found 3.15.2 in filename of file in /nix/store/pa87py9m9xyyq76l34llc2mk88h1384r-roxterm-3.15.2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

roxterm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pa87py9m9xyyq76l34llc2mk88h1384r-roxterm-3.15.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A roxterm https://github.com/r-ryantm/nixpkgs/archive/3b90f52765df9a875e01ee14fa67873a14ff677a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/3b90f52765df9a875e01ee14fa67873a14ff677a#roxterm

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pa87py9m9xyyq76l34llc2mk88h1384r-roxterm-3.15.2
ls -la /nix/store/pa87py9m9xyyq76l34llc2mk88h1384r-roxterm-3.15.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

roxterm

Maintainer pings
cc @AndersonTorres for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4694

Please someone undraft


LastPass TOTP Seed

In my enterprise LastPass test account I can add a TOTP seed to a Account item in my vault. I can also see this TOTP seed exported with my CSV from LastPass.

I would like to add it to the LastPass Account object here, but am having a hard time finding it in the LastPass parser. I have tried adding more ReadItem() calls instead of skipping (thinking one of the skip calls was skipping the TOTP), but I have not been able to locate a value that matches what I have stored in LastPass. Ex:
            var notes = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);
            var favorite = ReadItem(reader).ToUtf8();
            // SkipItem(reader);

            // Looking for TOTP seed
            var maybeseed = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);

            var username = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);
            var password = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);
            // SkipItem(reader);
            // SkipItem(reader);

            // Looking for TOTP seed
            var a = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);
            var b = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);
            var c = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);
            var d = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);
            var e = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);
            var f = Util.DecryptAes256Plain(ReadItem(reader), encryptionKey, placeholder);

Any tips on how to locate this TOTP seed property in the LastPass payload?

It's possible that there's no room in this chunk for this and it's encoded elsewhere. Unfortunately I do not have an enterprise account to check. I sent you an email about this to the address from the git commit.


5.0 Update fails to find nested subdomain route53 zones

Description
When attempting to update from < 5.0 to 5.0, I get new errors relating to route53 hosted zone discovery that did not exist prior to the update. It appears that the code to lookup the hosted zone for the gateway has changed when using the domain_name parameter when using single subdomained domains, i.e.,
Zone domain: my.domain.here.io
Subdomain/domain of the api gateway: testing.my.domain.here.io

[x] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]: 5.0

Terraform version: Terraform v1.3.5 or v1.7.4

Provider version(s):

provider registry.terraform.io/hashicorp/aws v5.52.0
Reproduction Code [Required]
provider "aws" {
  region = "us-east-1"
}

data "aws_region" "current" {}

data "aws_route53_zone" "verify" {
  name = "my.domain.here.io"
}

module "gateway" {
  source  = "terraform-aws-modules/apigateway-v2/aws"
  version = "5.0.0"

  name          = "test-endpoint"
  protocol_type = "HTTP"

  # Custom domain
  domain_name = "testing.my.domain.here.io"

  # Routes and integrations
  routes = {
    "GET /" = {
      integration = {
        uri                    = "arn:aws:lambda:us-east-1:${data.aws_region.current.name}:function:my-function"
        payload_format_version = "2.0"
      }
    }
  }
}

Expected behavior
I should get a successful terraform plan back from this specifying a new API gateway creation
Actual behavior
│ Error: no matching Route 53 Hosted Zone found
│
│   with module.gateway.data.aws_route53_zone.this[0],
│   on .terraform/modules/gateway/main.tf line 134, in data "aws_route53_zone" "this":
│  134: data "aws_route53_zone" "this" {

Terminal Output Screenshot(s)

Additional context
If one comments out the module, clears the .terraform directories and re-inits/plans, the aws_route53_zone data module finds the zone just fine. My assumption is that the use of the domain_name var has changed without clear guidance on how to replicate previous behavior. Apologies if this is in the new documentation or migration guide, I couldn't find it.

I think you would have:
  ...
  domain_name = "my.domain.here.io"
  subdomains  = ["testing"]
  ...

If you plan to route multiple sub-domains to this in the future, you can use:
  ...
  domain_name = "*.my.domain.here.io"
  subdomains  = ["testing"]
  ...

Ref https://github.com/terraform-aws-modules/terraform-aws-apigateway-v2?tab=readme-ov-file#multiple-subdomains

That does appear to at least clear the errors. Is that something that can be added to the migration document? It seems a change from previous behavior that may not be clear to others referencing that document rather than the primary readme.

Looks like this does resolve my errors, so I'll close. It does appear the final output differs pretty significantly due to the resource reorganization under the hood, which I suppose is expected for a major version number change. Appreciate the quick feedback!


Avoiding name shortening; explicit is better than implicit or derived.

I used to be a big fan of word shortening for naming. img instead of image and so on. Now, more often than not, I feel they are a bad idea. The benefits disappear, since we & users have autocomplete.
nistats.utils.pos_recipr  # Not the best idea.
nistats.utils.pos_reciprocal  # Better idea.

In the above example, pos_reciprocal feels so much better to me.
When I talk about this (such) functions with someone, I automatically say "pos  reciprocal", not "pos recipr". Cutting the word mid-way is a greater cognitive burden for me, and I think, others.
I think it will be better if we consistently use the more informative, slightly longer version; one that agrees with the way we tend to think and speak.

You're right on that !


Issue: Login with GitHub is not working on production deploiment

Description
Trying to login after the deployment of the GH login feature on production results on a redirect (with no authentication) to the / index page.

Expected behavior
User is redirected to GH oAuth2 login page for the current application.
Screenshots
n/a
Additional context
n/a

LGTM now


re-write Riemann solver to use HLLC-LM

implement HLLC-LM Riemann solver and use it by default. Ref: Journal of Computational Physics 423 (2020) 109762
Does not appear to have enough dissipation for very low-amplitude linear waves, as used in the HydroWave test.

With 3d Sedov test, there are instabilities in low-Mach regions of the flow, at least with PVRS states. Probably not viable.

Still does not work with linear waves. Very strange!


Credential Rotation feature stuck in "CredRotationInProgress" with "unsupported protocol scheme" message

Context
Trying to apply "credentials rotation" feature https://github.com/SAP/sap-btp-service-operator#credentials-rotation for the existing ServiceBinding.
SAP BTP Service Operator version: 0.2.7
Service Binding:
---
apiVersion: services.cloud.sap.com/v1
kind: ServiceBinding
metadata:
  name: ****
  namespace: ****
spec:
  serviceInstanceName: ****
  parameters:
    credential-type: X509_GENERATED
  credentialsRotationPolicy:
    enabled: true
    rotationFrequency: '5m'
    rotatedBindingTTL: '1m'

Issue
Service Binding (and underlying Secret) are not rotated.
Status
Status:
  Binding ID:  bc782b64-6c89-4ac0-9ea9-b393135f64c1
  Conditions:
    Last Transition Time:  2022-07-18T07:01:43Z
    Message:               ServiceBinding provisioned successfully
    Observed Generation:   1
    Reason:                Created
    Status:                True
    Type:                  Succeeded
    Last Transition Time:  2022-07-18T07:01:43Z
    Message:
    Reason:                Provisioned
    Status:                True
    Type:                  Ready
    Last Transition Time:  2022-07-23T19:12:10Z
    Message:               Patch /v1/service_bindings/bc782b64-6c89-4ac0-9ea9-b393135f64c1: unsupported protocol scheme ""
    Observed Generation:   2
    Reason:                Preparing
    Status:                True
    Type:                  CredRotationInProgress
  Instance ID:             d38b79c9-ddfd-4a26-b2f9-d49e2a5c0fe5
  Observed Generation:     1
  Ready:                   True

Question
What would latest status message mean?
Patch /v1/service_bindings/bc782b64-6c89-4ac0-9ea9-b393135f64c1: unsupported protocol scheme ""

Hi, I would like to check the binding in Service Manager, which landscape is it?

Hi, I would like to check the binding in Service Manager, which landscape is it?

Canary landscape.

Thanks! Can you please also share the operator's log?

Thanks! Can you please also share the operator's log?

I had to revert the changes back that day and do not have additional information, unfortunately.

Can you provide steps to reproduce and which service was it?
Which version of the operator are you using?

Can you provide steps to reproduce and which service was it?
Which version of the operator are you using?

As stated in the description, I ma using SAP BTP Service Operator version: 0.2.7
It is IAS (Identity Service)
Steps to reproduce:

Extend existing ServiceBinding spec with credentialsRotationPolicy
Apply K8s deployment manifest (with kubectl), so that ServiceBinding is reconfigured
Wait for 5th minute (as configured) in ServiceBinding
Check if respective servicebinding of secret K8s resources changed. -> did not happen

A short update.
"unsupported protocol scheme" issue seems to be generic and specific to latest version(s) of the Operator 0.2.7
I can reproduce it independently from "credential rotation" option. It is simply not more possible to create service bindings (at least for IAS).
All worked fine before Operator has been updated from version 0.1.18
Logs of the respective Operator's pod are not very helpful:
Defaulted container "kube-rbac-proxy" out of: kube-rbac-proxy, manager
I0801 03:36:02.143627       1 main.go:180] Valid token audiences:
I0801 03:36:02.143760       1 main.go:284] Generating self signed cert as no cert is provided
I0801 03:36:03.072364       1 main.go:334] Starting TCP socket on <IP_ADDRESS>:8443
I0801 03:36:03.073871       1 main.go:341] Listening securely on <IP_ADDRESS>:8443

Recent statuses of the service binding (new one) is "CreateInProgress" -> "NotProvisoned":
        "conditions": [
            {
                "lastTransitionTime": "2022-07-25T07:00:12Z",
                "message": "Error sending request GET /v1/service_bindings?attach_last_operations=true\u0026fieldQuery=name+eq+%27webshop-backend-ias-binding-temp%27+and+context%2Fclusterid+eq+%2732ce97c4-39ea-450a-a8b7-c7862423bbd4%27+and+context%2Fnamespace+eq+%27webshop%27\u0026labelQuery=_k8sname+eq+%27webshop-backend-ias-binding-temp%27: Get /v1/service_bindings?attach_last_operations=true\u0026fieldQuery=name+eq+%27webshop-backend-ias-binding-temp%27+and+context%2Fclusterid+eq+%2732ce97c4-39ea-450a-a8b7-c7862423bbd4%27+and+context%2Fnamespace+eq+%27webshop%27\u0026labelQuery=_k8sname+eq+%27webshop-backend-ias-binding-temp%27: unsupported protocol scheme \"\"",
                "observedGeneration": 1,
                "reason": "CreateInProgress",
                "status": "False",
                "type": "Succeeded"
            },
            {
                "lastTransitionTime": "2022-07-25T07:00:12Z",
                "message": "",
                "reason": "NotProvisioned",
                "status": "False",
                "type": "Ready"
            }
        ],
        "observedGeneration": 1,
        "ready": "False"
    }

Do you have some hints for us? What might be wrong?

An update for my previous comment https://github.com/SAP/sap-btp-service-operator/issues/185#issuecomment-1201031518
After downgrading from 0.2.7 to 0.1.18 (previously used) it is now again possible to create service bindings on our cluster w/o issues (no "unsupported protocol scheme """ occur )
However with this older version uses older CRDs which do not allow us to test "credentials rotation".
Question

Could you please confirm our findings? Are there any issues with recent version(s) of the Operator?

I was not able to reproduce the issue with version 0.2.7, I created a binding with rotation and it was created successfully

Can you please schedule a meeting for us and reproduce the issue?

Issue was caused due to invalid url to service manager.
helm value "url" was changed to "sm_url" in version 0.2.3


ValueError: buffer source array is read-only with apply_ufunc

I am trying to use scipy.signal.find_peaks function within apply_ufunc but fails with ValueError: buffer source array is read-only.
MCVE Code Sample
import numpy as np
import xarray as xr
from scipy.signal import find_peaks

# Generate waveform
x = (np.sin(2*np.pi*(2**np.linspace(2,10,1000))*np.arange(1000)/48000)
     + np.random.normal(0, 1, 1000) * 0.15)

# Find peaks non-xarray way
peaks, _ = find_peaks(x, prominence=1)
print(peaks)

# Cast waveform to xr.DataArray
x = xr.DataArray(x, dims='time')

# Duplicate data along a new dimension
rep = xr.DataArray(range(11), dims='repeat')
x = (x.broadcast_like(rep).assign_coords(repeat=rep))

def process_peaks(arr):
    # Apply find_peaks
    peaks, _ = find_peaks(arr, prominence=1)
    return peaks

# Apply function to array
results = xr.apply_ufunc(
    process_peaks, x,
    input_core_dims=[['time']],
    output_core_dims=[['peaks']],
    vectorize=True
)

# Should show repeats of peak results
print(results)

Expected Output
In the MCvE above, there are two print statements, The first print out the results of the peak finding without involving xarray.
The second print statement prints out the xr.DataArray of same peak results repeated along a dimension repeat.
Problem Description
The function scipy.signal.find_peaks threw an error ValueError: buffer source array is read-only. See the full traceback below
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-43-1a80f67560a6> in <module>
     35     input_core_dims=[['time']],
     36     output_core_dims=[['peaks']],
---> 37     vectorize=True
     38 )
     39

c:\users\at17\repos\xarray\xarray\core\computation.py in apply_ufunc(func, input_core_dims, output_core_dims, exclude_dims, vectorize, join, dataset_join, dataset_fill_value, keep_attrs, kwargs, dask, output_dtypes, output_sizes, *args)
   1040             join=join,
   1041             exclude_dims=exclude_dims,
-> 1042             keep_attrs=keep_attrs,
   1043         )
   1044     elif any(isinstance(a, Variable) for a in args):

c:\users\at17\repos\xarray\xarray\core\computation.py in apply_dataarray_vfunc(func, signature, join, exclude_dims, keep_attrs, *args)
    230
    231     data_vars = [getattr(a, "variable", a) for a in args]
--> 232     result_var = func(*data_vars)
    233
    234     if signature.num_outputs > 1:

c:\users\at17\repos\xarray\xarray\core\computation.py in apply_variable_ufunc(func, signature, exclude_dims, dask, output_dtypes, output_sizes, keep_attrs, *args)
    599                 "apply_ufunc: {}".format(dask)
    600             )
--> 601     result_data = func(*input_data)
    602
    603     if signature.num_outputs == 1:

~\.conda\envs\asi\lib\site-packages\numpy\lib\function_base.py in __call__(self, *args, **kwargs)
   2089             vargs.extend([kwargs[_n] for _n in names])
   2090
-> 2091         return self._vectorize_call(func=func, args=vargs)
   2092
   2093     def _get_ufunc_and_otypes(self, func, args):

~\.conda\envs\asi\lib\site-packages\numpy\lib\function_base.py in _vectorize_call(self, func, args)
   2155         """Vectorized call to `func` over positional `args`."""
   2156         if self.signature is not None:
-> 2157             res = self._vectorize_call_with_signature(func, args)
   2158         elif not args:
   2159             res = func()

~\.conda\envs\asi\lib\site-packages\numpy\lib\function_base.py in _vectorize_call_with_signature(self, func, args)
   2196
   2197         for index in np.ndindex(*broadcast_shape):
-> 2198             results = func(*(arg[index] for arg in args))
   2199
   2200             n_results = len(results) if isinstance(results, tuple) else 1

<ipython-input-43-1a80f67560a6> in process_peaks(arr)
     27
     28     # Finally execute find_peaks
---> 29     peaks, _ = find_peaks(arr, prominence=1)
     30     return peaks
     31

~\.conda\envs\asi\lib\site-packages\scipy\signal\_peak_finding.py in find_peaks(x, height, threshold, distance, prominence, width, wlen, rel_height, plateau_size)
    937         raise ValueError('`distance` must be greater or equal to 1')
    938
--> 939     peaks, left_edges, right_edges = _local_maxima_1d(x)
    940     properties = {}
    941

_peak_finding_utils.pyx in scipy.signal._peak_finding_utils._local_maxima_1d()

~\.conda\envs\asi\lib\site-packages\scipy\signal\_peak_finding_utils.cp37-win_amd64.pyd in View.MemoryView.memoryview_cwrapper()

~\.conda\envs\asi\lib\site-packages\scipy\signal\_peak_finding_utils.cp37-win_amd64.pyd in View.MemoryView.memoryview.__cinit__()

ValueError: buffer source array is read-only

I have a conversation with some people on Gitter about it and a workaround was proposed, adding this line of code before find_peaks
arr = arr.copy()

I am not sure if a read-only array is a view or not.
This appears to be a known problem to some people but there are no documentation on this limitation of apply_ufunc. It may be unlikely that a copy operation will be added to apply_ufunc to deal with this issue internally. So the way forwards is to add documentation to apply_ufunc about this issue and the workaround.
Output of xr.show_versions()
INSTALLED VERSIONS
------------------
commit: None
python: 3.7.6 | packaged by conda-forge | (default, Jan  7 2020, 21:48:41) [MSC v.1916 64 bit (AMD64)]
python-bits: 64
OS: Windows
OS-release: 10
machine: AMD64
processor: Intel64 Family 6 Model 78 Stepping 3, GenuineIntel
byteorder: little
LC_ALL: None
LANG: None
LOCALE: None.None
libhdf5: 1.10.5
libnetcdf: 4.7.3

xarray: 0.14.1+50.ge0fd4805
pandas: 0.25.3
numpy: 1.17.3
scipy: 1.3.1
netCDF4: 1.5.3
pydap: None
h5netcdf: 0.7.4
h5py: 2.10.0
Nio: None
zarr: None
cftime: <IP_ADDRESS>
nc_time_axis: None
PseudoNetCDF: None
rasterio: None
cfgrib: None
iris: None
bottleneck: None
dask: None
distributed: None
matplotlib: 3.1.2
cartopy: None
seaborn: None
numbagg: None
setuptools: 45.0.0.post20200113
pip: 19.3.1
conda: None
pytest: None
IPython: 7.11.1
sphinx: None
```

@DancingQuanta someone pointed me towards this issue — did you happen to make any progress on this / post an issue on scipy?
As per your issue, while this isn't an ideal workaround, this does work:

def process_peaks(arr):
    # Apply find_peaks
    arr = arr.copy()        <- # added
    peaks, _ = find_peaks(arr, prominence=1)
    return peaks

I think we can close as upstream, please reopen if that's not the case


Update argonaut to 6.3.5

Updates io.argonaut:argonaut from 6.3.3 to 6.3.5.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (6.3.3).
You might want to review and update them manually.
README.md

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.argonaut", artifactId = "argonaut" } ]

labels: library-update, semver-patch, old-version-remains

Superseded by #257.


Apple Silicon

Hi,
This installs and runs perfectly on my intel Mac, but it does not install on my Apple Silicon Mac
Collecting tensorflow-serving-api<3.0.0,>=2.1.0
Downloading tensorflow_serving_api-2.8.0-py2.py3-none-any.whl (37 kB)
Pip subprocess error:
ERROR: Could not find a version that satisfies the requirement tensorflow<3.0.0,>=2.8.0 (from aizynthfinder) (from versions: none)
ERROR: No matching distribution found for tensorflow<3.0.0,>=2.8.0
failed
CondaEnvException: Pip failed
I was wondering if anyone had managed to install perhaps using the details here?
https://makeoptim.com/en/deep-learning/tensorflow-metal

Hello,
Thanks for reaching out. Unfortunately, we developers have no way to trying out the software on all platforms. For instance, we have not access to an Apple Silicon Mac. I just know that installations are problematic.
Currently, I see this as a niche use case and therefore I suggest that you try to find the solution on e.g. Tensorflow homepage or on Stack Overflow.

I will close this because of inactivity.


Add managed scale-down for SolrClouds

Resolves #559
This also adds cluster-operation locking for SolrClouds, as described here: #560
TODO:

[x] Implement scale-down of SolrClouds, moving replicas when needed
[x] Add cluster-operation locking for SolrClouds, but do not move other cluster-operations logic. That can be done separately.
[x] Add option to do managed scale-downs, this should be enabled by default
[x] Add integration tests for scaling down
[x] Add documentation & upgrade notes
[x] Add changelog entry

Review request: @radu-gheorghe and @tflobbe. I'll finish this up today or Monday

Cool, docs and e2e tests are pretty good to go. There might be optimizations that can be made down the line, but I think this is complete for now!
Overall the only thing that is in my mind is that we might want to change the autoscaling section to just be called scaling. Not exactly sure though. Would welcome opinions.

Awesome! I will test it with HPA then, so I can get a jump start on that tutorial as well.

@tflobbe thank you for the wonderful suggestions! I think I have addressed all of them, I'll merge early next week in case you want to take another look, but no worries if you don't have time.


Remove version: reMarkable.reMarkableCompanionApp version <IP_ADDRESS>0

Vanity URL, version superceded by newer release
Microsoft Reviewers: Open in CodeFlow

URL: https://downloads.remarkable.com/latest/windows
Response Error: The remote server returned an error: (405) Method Not Allowed.
(Automated message - build 720)


gadgets: Enable/disable columns according to flags

There are some cases in which the columns we want to print are dependent on the flags passed to the gadget:

snapshot process --threads  -> enable the TID column (https://github.com/inspektor-gadget/inspektor-gadget/pull/1396#issuecomment-1489250861)
trace tcpconnect --latency -> enable the latency column (https://github.com/inspektor-gadget/inspektor-gadget/pull/1574#discussion_r1177659199)

The purpose of this issue is to implement this support to bring a better user experience.
Implementation options
I think this is a gadget specific logic, so each gadget should decide which columns to enable / disable based on this configuration. One option is to pass the gadget parameters to Parser() to let each gadget hide / show columns.
Questions
Should it be allowed to change the gadget config based on the set columns. For instance, snapshot process -o columns=container,pid,tid should implicitly enable the --threads flag?
/cc @flyth

Regarding the implementation for the first part: as long as we're only enabling features (and therefore columns by bool params), we could simplify and generalize it by adding for example a column tag:
type Event {
...
	Latency   time.Duration `json:"latency,omitempty" column:"latency,minWidth:8" columnTags:"param:latency"`
...
}

Then we could reduce the default columns automatically when a given param is not true. Something like this could do it without having to add code to gadgets. However, this only works for bool params (we could add a negation easily, like param:!latency and maybe even check for certain values like param:mode=abc, but if we would need more complicated rules, then it would probably be better having dedicated functions inside the gadgets).
Regarding the second part: I'm not really sure whether specifying columns should control feature flags. I'll think more about it.

I think we can implement a very simple logic by using the param tag you suggested. It later on we find out that more complex cases are needed we can rethink it.

Solved in #1587


Deny processing GitLab webhooks without token

Signed-off-by: Matej Focko<EMAIL_ADDRESS>

issue with token should be created only once, so I don't expect possibility of spamming; also I haven't noticed new issue when testing

Would be nice to have some test-case but LGTM otherwise. Thanks, good catch!

extended, we had tests for valid, invalid secrets/tokens in headers, but not for missing

recheck

recheck


Update deps

This updates webstub to the version that doesn't cause spec errors and a general update of dependencies.

Eyyyy tests passing! that's awesome!

Yup! An old version of webstub was the culprit.


Loss curve

Hi, I am currently training a wavernn, however, it's hard to train. Have you ever trained a good model? Would you mind to share your loss curve at training phase.

Hi, which version of WavRNN are you using? I trained on Fatcord's original WavRNN and it had difficulty converging as well, however his modified version the preliminary results are better.
I am porting his code from jupyter notebook and will add training curves when training on LJ speech is done. I'm trying to see if I can get gaussian output working too (from clarinet)

Inspired by Fatcord's modified version, I use two gru to train the model described in paper (dual softmax). It runs much faster than but equivalent to Fatcord's original version (I think they are equal).
My local condition is upsampled mel only. Because I think 1d residual module is similar with upsample module.
Currently, my CrossEntropy training loss is ~2.7. However it seems infered audio (incrementally inference) is not good.
Once you get good result with LJSpeech, please tell me. I also use LJSpeech to train model.
Thank you.

Ok, I will update you once the model is trained well. Would you mind sharing some audio samples of your wavernn vocoder for reference?

My model failed to infer good audio. I though there are some bugs in my code. I'm curious about gaussian output you mentioned before. Is that similar to mixture logistic loss used in wavenet?

That sounds good. Is that training data? Or just a testing data.

These are from the test data, about 50 sentences were held out for testing.
I'm currently finishing up training the text-to-mel part (DCTTS) in a few hours. I'll generated some samples and use the vocoder to generate some outputs. I'll post samples here once done.

here's audio with new sentence, the static is still an issue, probably with the 9 bit audio.
new_sent.tar.gz
how is your models coming along?

Your new sentence sounds good except some noise. But it sounds natural.  Is that generated by DCTTS and WaveRNN together?
Do you have some ideas about processing text for better attention alignments ?
I'm working on Tacotron2, spent about one month on it. However the result is not good.


How to upload both images and files?

According to the setting, I can upload image successfully but cannot upload pdf file. It will show "Unexpected mime type error". I also check the $this->file_type inside LfmController, it always equals 'iamge'. I am not understand the part in CKEditor. How to change the url among the 4 urls?
Thank you very much.

What are the exact steps you make that give you this error? Please give some screenshots.

Thank you for your reply.
I just click upload and upload a pdf file, it gives the error.

Be sure to set mime types you want in config. See here.

Thank you for your help. I understand much more about my problem.
Furthermore, how can I change the upload request url from '/laravel-filemanager?type=Images' to '/laravel-filemanager?type=Files' ?

Doc here, you can see the urls as you described and change all 'Images' to 'Files'.

Thank you for your reply.
I can upload pdf files without error message by hard code the type with 'Files'

However, the image features which are resize, crop and thumbnail preview had gone.

This is the screen shot for the original which is the type equals image

Are there any solutions to keep the image features such as thumbnail and crop, at the same time to upload pdf files?
Thank you for your attention.

You can modify this blade in resources/views/vendor/laravel-filemanager, to fix the broken file icon in image view.

It would be great to have a global array with supporting Mime types. In my case I would like to have thumbnails (and preview) for images  and a generic one for any other type ( zip, doc, txt, pdf etc.)

v1.7-alpha has been released. The solution will be to simply use "File mode".

"File mode" where location folder?  @g0110280

@hermawanheri66 See configs here.


Host selector test

Q
A

Test Features
Yes

Do not forget to add notes about your changes to CHANGELOG.md

Closing. No work going on.


Don't allocate new strings when re-casing a string which didn't need it

This is my first ChakraCore pull request. As you can probably guess from the code I'm not a very experienced C++ developer. I am not even sure this code actually improves performance, since it makes it longer, but I wanted to see what you guys thought about the idea.
Being a Microsoft Employee I don't think I need to sign a CLA or something, but if I am wrong let me know.

Hi @FremyCompany, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@FremyCompany if you're a member of the Microsoft GitHub organization, you don't need to sign the CLA. Otherwise you do. The @msftclas bot indicates that you aren't (otherwise it would have tagged this PR as cla-not-required). Even if you're an MS employee, no harm in signing the CLA anyway. As for getting your GitHub account added to the MS organization on GitHub, you can send our team email and we'll send you instructions.

As for the code, no need to be too harsh of a critic on yourself. We'll evaluate this PR and if it doesn't hurt performance, we can help you get it where it needs to go. To see whether this improves performance, you should first convince yourself that is helps (or at least doesn't hurt) by doing many iterations of a test with many loops to exercise the types of cases you're trying to optimize. Run that test before and after your change and measure the performance change.
Note that longer code doesn't mean longer execution time. As a general rule, you make a program faster by it do fewer things (fewer iterations, less wasteful buffer copies, and so on). Adding more checks to remove unnecessary loop iterations and shortcuts is sometimes how we do that. Often, a faster program has more source code.
The only thing to worry about is whether your extra checks introduce a lot of overhead to the cases which do not hit your optimization path. In that case the situation you're trying to improve might get better but average program performance would get worse. That's the sort of thing you should try to evaluate before we consider accepting this change.
If you run some tests, please provide your test harness, method of testing, and results here in the comments.
Thanks!

I was thinking about this earlier today, wondering how to make it fast without having to visit all characters in the string an extra time.  Is the GC allocation costly versus visiting the entire string an extra time?  Probably not.  If the Upper/LowerCharBufW APIs told us whether they changed the string or not we could decide to abandon the new allocation and return the original string, letting the new one get collected on the next GC, but would that help performance?  Perhaps not, unless there is a scenario that frequently calls toLowerCase() or toUpperCase().
A change like this should be motivated by a real world scenario, either a poor performing code path in a real website (or product hosting the engine), or a poor performing path in a benchmark.
Without a real scenario in hand to test against there is no way to judge whether a change like this is making a difference.
I microbenchmark can help reveal a difference in performance, but making sure a microbenchmark models real scenarios can be tricky.

Yes, it is a bit sad the function does not tell you if it changed anything. I agree with you the frequent calls are likely to cause a slowdown.
My use case was a data collection script which called toLowerCase() on a big string which actually was already lowercased very often, and which is actually not used after toLowerCase() as it is the result of a computation.
I figured out it would be a good idea to try to optimize this, but I didn't find a good way to fix this without trying to use our own function to "upper" or "lower" chars... What if we build a quick and dirty isProbablyLower and isProbablyUpper function, which would work for ASCII and return false for things outside ASCII or that are of the wrong case? That would allow to check the string efficiently, what do you think?

@FremyCompany, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

That might help.  If it is the toUpper case you could loop through and see if all the characters are ascii and outside the lowercase ascii range.  If you reach the end then there is no work to do and return the string as is.  Vice versa for toLower.  If you break out of this loop early, then you have a pointer to the character where you need to start checking, so you could call Upper/LowerCharBufW with this offset into the destination string.

Ok I'm going to try to implement that this weekend and see if it improves the perf on some test cases.

Rad, look forward to seeing your next update

Thank you for your pull request.  Here is my opinion:
Advantages of current code:
•	Simple (less code).
•	Performs better on strings which are not of intended case.
Advantages of new change proposed:
•	Performs better on strings which are already in the intended case.
I would not change the current built-in unless there is sufficient data to prove the new change is beneficial at the cost of simplicity (which helps maintainability). It would be great if you can collect that data before the code change. Something like 70% of the strings in the web are of intended case then this makes worthwhile effort. Anything less than 50% is not.
Also there is a bug in the code as it doesn’t check for lowercase conversion.

@FremyCompany  Any update on this PR?

Yes please don't close it just yet, I had to skip working on this last weekends due to other work, but I'm definitely going to work on it this weekend.

This discussion moved to #468 to start anew now that I have a better idea of what I am doing.

@FremyCompany is this PR superseded entirely by your new one?  If so could you close this PR please?


Getting YYYY-MM-DD ddd to render as a date property

I use this format YYYY-MM-DD ddd in other Obsidian templates and such but I cannot get modified-date to render it as a link. To render this as a link, how do I tell modified-date to use list property to render this format YYYY-MM-DD ddd. What am I missing?

Like this:
https://github.com/alangrainger/obsidian-frontmatter-modified-date?tab=readme-ov-file#date-format
[[[]YYYY-MM-DD ddd[]]]

I had been using your wonderful plugin for a while, so all the  properties were Date type. Soon as I changed to the type to list, all the notes now relfect the link format. Many thank yous!


Add example with new features

Hello,
I add a new sample "greenHover". Thanks for the idea.
I change the package name to customizable-radial-menu. I think we could publish this projet on npm.

Cool

Cool


nothing to commit!

hi there,
I had this issue when I using this package.
"alt +c" --> pop-up window --> "No files were changed or add ...."
in my project folder, there are changes.
I fund that the "commit to:" is empty, not read the ".svn". look like is not read my repo info on the folder.
can you help?
thanks

I am using "win 7"

I am working on a fix for this. I have a PR that I am trying to finesse a bit more.
The issue is with the space in the folder name of your repository.
A workaround is to change the "my sites" folder to "my_sites" (or anything without spaces).

Jason, thanks for the reply. yes, I could workaround by changed name of my folder. :)


beast::get_lowest_layer throws exception

I am facing a really weird issue. I have the following code in my two apps which run on the same system.  In one app it runs perfectly fine but it crashes at "beast::get_lowest_layer(stream).connect(results);" continuously in the second app.
Following is the same code present in two apps:

    net::io_context ioc;
    ssl::context ctx(ssl::context::tlsv12_client);
    ctx.load_verify_file(strCertFile.c_str());
    ctx.set_verify_mode(ssl::verify_peer);

    tcp::resolver resolver(ioc);
    beast::ssl_stream<beast::tcp_stream> stream(ioc, ctx);

    // Set SNI Hostname (many hosts need this to handshake successfully)
    if (!SSL_set_tlsext_host_name(stream.native_handle(), strHost.c_str()))
    {
        beast::error_code ec{ static_cast<int>(::ERR_get_error()), net::error::get_ssl_category() };
        throw beast::system_error{ ec };
    }

    // Look up the domain name
    auto const results = resolver.resolve(strHost, strPort);

    // Make the connection on the IP address we get from a lookup
    beast::get_lowest_layer(stream).connect(results);

It looks like in VC\Tools\MSVC\14.26.28801\include\memory file, it is going in loop in following function:

template <class _Ty1, class _Ty2>
_NODISCARD bool operator!=(const shared_ptr<_Ty1>& _Left, const shared_ptr<_Ty2>& _Right) noexcept {
return _Left != _Right;
}

The complete call stack is below: (pasted only bottom entries. Its a very huge call stack with possible loop.)

TestAuth.exe!std::operator!=<std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>,std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>>(const std::shared_ptr<std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>> & _Left, const std::shared_ptr<std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>> & _Right) Line 1302	C++
TestAuth.exe!std::operator!=<std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>,std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>>(const std::shared_ptr<std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>> & _Left, const std::shared_ptr<std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>> & _Right) Line 1302	C++
TestAuth.exe!std::operator!=<std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>,std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>>(const std::shared_ptr<std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>> & _Left, const std::shared_ptr<std::vector<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>,std::allocator<boost::asio::ip::basic_resolver_entry<boost::asio::ip::tcp>>>> & _Right) Line 1302	C++
TestAuth.exe!boost::asio::ip::basic_resolver_iterator<boost::asio::ip::tcp>::equal(const boost::asio::ip::basic_resolver_iterator<boost::asio::ip::tcp> & other) Line 172	C++
TestAuth.exe!boost::asio::ip::operator!=(const boost::asio::ip::basic_resolver_iterator<boost::asio::ip::tcp> & a, const boost::asio::ip::basic_resolver_iterator<boost::asio::ip::tcp> & b) Line 154	C++
TestAuth.exe!boost::asio::connect<boost::asio::ip::tcp,boost::asio::executor,boost::asio::ip::basic_resolver_iterator<boost::asio::ip::tcp>,boost::asio::detail::default_connect_condition>(boost::asio::basic_socket<boost::asio::ip::tcp,boost::asio::executor> & s, boost::asio::ip::basic_resolver_iterator<boost::asio::ip::tcp> begin, boost::asio::ip::basic_resolver_iterator<boost::asio::ip::tcp> end, boost::asio::detail::default_connect_condition connect_condition, boost::system::error_code & ec) Line 235	C++
TestAuth.exe!boost::asio::connect<boost::asio::ip::tcp,boost::asio::executor,boost::asio::ip::basic_resolver_results<boost::asio::ip::tcp>>(boost::asio::basic_socket<boost::asio::ip::tcp,boost::asio::executor> & s, const boost::asio::ip::basic_resolver_results<boost::asio::ip::tcp> & endpoints, boost::system::error_code & ec, void * __formal) Line 123	C++
TestAuth.exe!boost::asio::connect<boost::asio::ip::tcp,boost::asio::executor,boost::asio::ip::basic_resolver_results<boost::asio::ip::tcp>>(boost::asio::basic_socket<boost::asio::ip::tcp,boost::asio::executor> & s, const boost::asio::ip::basic_resolver_results<boost::asio::ip::tcp> & endpoints, void * __formal) Line 113	C++
TestAuth.exe!boost::beast::basic_stream<boost::asio::ip::tcp,boost::asio::executor,boost::beast::unlimited_rate_policy>::connect<boost::asio::ip::basic_resolver_results<boost::asio::ip::tcp>,void>(const boost::asio::ip::basic_resolver_results<boost::asio::ip::tcp> & endpoints) Line 596	C++
TestAuth.exe!OAuthImpl::RequestToken() Line 113	C++ // This is my function

Any help on this is much appreciated.
Regards,
Ganesh
PLEASE DON'T FORGET TO "STAR" THIS REPOSITORY :)
If you rather keep your project secret, feel free
to email the author at<EMAIL_ADDRESS>any
private correspondence is treated as confidential.
When reporting a bug please include the following:
Version of Beast
You can find the version number in the file <boost/beast/version.hpp>,
or by using the command "git log -1" in the local Beast repository
directory.
Steps necessary to reproduce the problem
A small compiling program is the best. If your code is
public, you can provide a link to the repository.
All relevant compiler information
If you are unable to compile please include the type and
version of compiler you are using as well as all compiler
output including the error message, file, and line numbers
involved.
The more information you provide the sooner your issue
can get resolved!

My boost version is 1.73. I have also tried with 1.72 and have seen the same issue.

Are you getting a crash or an exception?

Getting unhandled exception.

I am seeing an unhandled exception in debug build. In release build, it just hangs. So I suspect this is some infinite loop (maybe related to string arguments) but I am not sure why it doesn't happen with other app with the same code and parameters.

Have you caught the exception and printed it?

Honestly, it sounds like there is some code that you are not showing me that is causing undefined behaviour.

Ok. Which part of the code do you want? Right now Host, post, and cert file name all are hardcoded in the code.
Again pasting the code block with hardcoded value. In the initial configuration only the exceptions are seen.
std::string strCertFile = "F:\cacert.pem";
std::string strHost = "login.microsoftonline.com";
std::string strPort = "443";
    net::io_context ioc;
    ssl::context ctx(ssl::context::tlsv12_client);
    ctx.load_verify_file(strCertFile.c_str());
    ctx.set_verify_mode(ssl::verify_peer);

    tcp::resolver resolver(ioc);
    beast::ssl_stream<beast::tcp_stream> stream(ioc, ctx);

    // Set SNI Hostname (many hosts need this to handshake successfully)
    if (!SSL_set_tlsext_host_name(stream.native_handle(), strHost.c_str()))
    {
        beast::error_code ec{ static_cast<int>(::ERR_get_error()), net::error::get_ssl_category() };
        throw beast::system_error{ ec };
    }

    // Look up the domain name
    auto const results = resolver.resolve(strHost, strPort);

    // Make the connection on the IP address we get from a lookup
    beast::get_lowest_layer(stream).connect(results);

    // Perform the SSL handshake
    stream.handshake(ssl::stream_base::client);

This code is taken from a published Beast sample on GitHub.

can you catch the exception, determine its type and print its message, and it relevant, error_code?

Exception details from debugger:
Unhandled exception at 0x00007FF77D2D8D45 in TestAuth.exe: 0xC00000FD: Stack overflow (parameters: 0x0000000000000001, 0x000000DC85C03F80). occurred
The following try catch block doesn't catch this exception:
try
{
beast::get_lowest_layer(stream).connect(results);
}
catch (std::exception const& e)
{
std::cerr << "Error: " << e.what() << std::endl;
return false;
}
Visual Studio caught the exception (not the above catch block).
Like I have mentioned in my first comment the stack is probably getting full because of the infinite loop.

Hmm. Did you copy the example exactly? I see that you set the TLS ext_host_name before connecting the underlying socket.

I am referring this: https://github.com/boostorg/beast/blob/develop/example/http/client/sync-ssl/http_client_sync_ssl.cpp
And the flow seems similar.

OK, can we try a couple of things?
first let's put in a loop to print the endpoints discovered by the resolver to be sure there's no infinite loop:
for(auto entry : results) { std::cout << entry.endpoint() << std::endl; }
Then, let's move the tls_ext_host_name until after the tcp connect.

The for loop is throwing the stackoverflow exception.
for(auto entry : results) { std::cout << entry.endpoint() << std::endl; }
So looks like something wrong is fetched as the output of resolver. Is there any particular field from the results you want me to check?
Relevant code block with try-catch block.
        try {
        // Look up the domain name
        auto const results = resolver.resolve(strHost, strPort);

        for (auto entry : results) { std::cout << entry.endpoint() << std::endl; }

        // Make the connection on the IP address we get from a lookup
        beast::get_lowest_layer(stream).connect(results);
    }
    catch (std::exception const& e){
        std::cerr << "Error: " << e.what() << std::endl;
        return false;
    }

LEt me know if there are any suggestions to debug and resolve this issue.

C:\Users\Administrator>nslookup login.microsoftonline.com
Server:  UnKnown
Address:  <IP_ADDRESS>
Non-authoritative answer:
Name:    www.tm.a.prd.aadg.akadns.net
Addresses:  <IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
Aliases:  login.microsoftonline.com
a.privatelink.msidentity.com
prda.aadg.msidentity.com
Nslookup output from the system on which I am seeing this issue. Is there anything we can do to debug this further?

I am interested to know whether there is anything unusual about your setup. It's very strange that the resolve return result seems valid when we look at the debugger and yet you get a crash when you try to inspect it.
Something like a difference in the way two compilation units or libraries have been compiled? e.g. shared c++ runtime library vs static?


[UBS_Prod. Orders] Wrong sum in the field 'Amount due' of the table orders

Environment: Windows 11 Home 64 bit, Chrome Version 110.0.5481.97 (Official Build)
Reproducible: always
Build found: 3/5/2023
Preconditions

Log in as admin to https://www.pick-up.city/#/ubs-admin/orders
Goto UBS Admin, Orders Page
At least one order is present in the system

Steps to reproduce

Goto order and check the amount due
Check the column 'Amount due' if the same sum is displayed

Actual result

The sum in the column 'Amount due' (Сума до оплати) of the table is not equal to the amount to be paid in the order

Expected result

The sum in column 'Amount due' (Сума до оплати) of the table should be equal to that one in the order

E.g.: "User story #3184
Test case"
Labels to be added
"Bug", Priority ("pri: "), Severity ("severity:"), Type ("UI, "Functional"), "API" (for back-end bugs).

Has it been fixed at https://www.testgreencity.ga/#/ubs. There will soon be a merge at pick-up.city

Merged into prod.


Filter account authentication by domain

The Azure tenancy includes other state agencies which shouldn't necessarily have access to the application. This affects all applications that use SOG authentication.

See https://github.com/gaepdit/complaint-tracking/pull/735 for similar work done.


[DEV] dont show swap button in tab bar even when swap feature is enabled unless in debug mode

So it doesn't appear in TestFlights or accidentally in app store builds

Force-pushed after rebase master


Problem with compilation

Hi, im using react native 0.59.8 and when try to run I get the following error
SpanView.java:1156: error: cannot find symbol
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) {
^
symbol:   variable P
location: class VERSION_CODES
/Users/sensordata/Proyectos/sensordata-app-android/node_modules/react-native-svg/android/src/main/java/com/horcrux/svg/TSpanView.java:1157: error: no suitable method found for create(Typeface,int,boolean)
typeface = Typeface.create(typeface, weight, isItalic);
^
method Typeface.create(String,int) is not applicable
(actual and formal argument lists differ in length)
method Typeface.create(Typeface,int) is not applicable
(actual and formal argument lists differ in length)
2 errors
FAILURE: Build failed with an exception.

Can you try setting compileSdkVersion = 28


[Bug]: Chromadb import error. Dependency issue: Install pydantic-settings package

What happened?
What happened?
There was a PydanticImportError when trying to import chromadb. The error occurred because the BaseSettings class has been moved to the pydantic-settings package.
What did you expect to happen?
I expected to import chromadb successfully without any errors.
Versions
Python 3.11.3
Relevant log output
import chromadb

---------------------------------------------------------------------------
PydanticImportError                       Traceback (most recent call last)
Cell In[1], line 1
----> 1 import chromadb

File G:\RajArun\os\chroma\chromaenv\Lib\site-packages\chromadb\__init__.py:1
----> 1 import chromadb.config
      2 import logging
      3 from chromadb.telemetry.events import ClientStartEvent

File G:\RajArun\os\chroma\chromaenv\Lib\site-packages\chromadb\config.py:1
----> 1 from pydantic import BaseSettings
      2 # from pydantic_settings import BaseSettings
      3 from typing import Optional, List, Any, Dict, TypeVar, Set, cast, Iterable, Type

File G:\RajArun\os\chroma\chromaenv\Lib\site-packages\pydantic\__init__.py:206, in __getattr__(attr_name)
    204 dynamic_attr = _dynamic_imports.get(attr_name)
    205 if dynamic_attr is None:
--> 206     return _getattr_migration(attr_name)
    208 from importlib import import_module
    210 module = import_module(_dynamic_imports[attr_name], package=__package__)

File G:\RajArun\os\chroma\chromaenv\Lib\site-packages\pydantic\_migration.py:279, in getattr_migration.<locals>.wrapper(name)
    277     return import_string(DEPRECATED_MOVED_IN_V2[import_path])
    278 if import_path == 'pydantic:BaseSettings':
--> 279     raise PydanticImportError(
    280         '`BaseSettings` has been moved to the `pydantic-settings` package. '
    281         f'See https://docs.pydantic.dev/{VERSION}/migration/#basesettings-has-moved-to-pydantic-settings '
    282         'for more details.'
    283     )
    284 if import_path in REMOVED_IN_V2:
    285     raise PydanticImportError(f'`{import_path}` has been removed in V2.')

PydanticImportError: `BaseSettings` has been moved to the `pydantic-settings` package. See https://docs.pydantic.dev/2.0.2/migration/#basesettings-has-moved-to-pydantic-settings for more details.

For further information visit https://errors.pydantic.dev/2.0.2/u/import-error

Python 3.9 installed with conda also got this err.

hey @husy8 this solution doesn't works for me, and I'm using the same version.

it doesnt for me @husy8 i have a python 3.9.1 and installed a pip install pydentic==2.0

Downgrading to an even older version of pydantic seems to do the trick (as far as workarounds go): pip install pydantic==1.9.0

with this downgrade I was able to solve the problem and run the application, I will wait for a definitive solution, thanks @dmahlow.

FastAPI recently released v0.100.0 where they started using Pydantic v2, which does not work with ChromaDB.
Installing fastapi==0.99.1 fixed the issue for me.
A short-term fix could be setting the FastAPI dependency in ChromaDB to fastapi >=0.85.1, <=0.99.1. Settingpydantic >=1.9 , <=1.10.11 might also do the trick.

We are looking into this, thanks everyone!

Hi folks, we have temporarily hotfixed this issue by hardpinning to known good versions of fastapi and pydantic. We will in the future address #785 and fix it entirely.

Thank you for the quick fix! Love your product - keep up the good work!

For those langchain users - chroma is cutting a new release momentarily to fix the pydantic version conflict - https://github.com/chroma-core/chroma/pull/799

Python3.7 and Python3.10 also meet this import problem. This error should be fixed! I solve this by installing pedantic-settings and modify the first line of config.py file to import pedatic-settings.

Hi, I have the same issue that chromadb is using pydantic 1.10.12 but it clash with elevenlabs which use pydantic 2.30.  Beside chromadb  I use gradio and langchain. I see that gradio is using fastapi also... How did u solve it?

For those langchain users - chroma is cutting a new release momentarily to fix the pydantic version conflict - #799

Hey did this issue get resolved just tried to setup embedding with langchain and chroma today and im getting the same conflicts with pydantic.

We have a PR landing today alongside a new release which should address this. You can follow along here - https://github.com/chroma-core/chroma/pull/1174

hi @inlima which version did you use? I downgraded pydantic to version 1.9.0 but it is not compatible with python 3.10


[stacked_generator] Could not resolve annotation for class App

Hey, I tried running build_runner watch but I am getting this error.
[SEVERE] injectable_generator:injectable_builder on lib/app/app.dart (cached):

line 1, column 53 of package:health_educational/app/app.dart: Could not resolve annotation for `class App`.
  ╷
1 │ @StackedApp()
  │ ^^^^^^^^^^^^^
  ╵
[SEVERE] Failed after 148m

can you please post your full file here

import 'package:stacked/stacked_annotations.dart';
@StackedApp()
class App {}

This is all in the file.

I guess I figured it out. The error went away as soon as I put the routes argument in the StackedApp annotation.
import 'package:stacked/stacked_annotations.dart';
@StackedApp(routes:[])
class App {}

This works.


Revert "Do not add unnecessary experiment use_multiple_sdk_containers."

Reverts apache/beam#13475
Reason: for streaming jobs new codepath using UW (https://github.com/apache/beam/blob/9f97585160fae644d9b4a7f0dca6558c82c29723/sdks/python/apache_beam/runners/dataflow/internal/apiclient.py#L276) is not yet triggered by default.
So, at this time Dataflow first need to change the behavior to not require use_multiple_sdk_containers for streaming pipelines, or we need to use UW codepath for all streaming pipelines in Beam.

R: @chamikaramj

LGTM


Added my name to the readme.md

Related issue

What changes does this PR do?
Screenshots (If any)
Questions

sooo......happy lol
This is actually my contribution in open source

updated my name in alphabetical order


Bump client to 2.6.0

Description
2.6.0 version and some regular post release tasks.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks @VachaShah !


[feat]: Sheet compatibility with Resizable

Feature description
I need a Sheet that is Resizable by the user, I know we have both components, but couldn't make them work together.
Affected component/components
Sheet, Resizable
Additional Context
Additional details here...
Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues and PRs

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.

I think it is relevant enough


NaN when sharre.php is enabled?

Im recieving a NaN on all my counters when I have sharre.php linking up correctly? Remove it and its fine but doesnt count google plus obviously.

Do you have a current "not working" example ?


#635 Update properties for two way binding to EntryView

Updates issues with setting properties not applying to EntryView child
Formats bindable properties to a standard format

I'll try if everything works when I find some time.
And I'll add some new test cases that covers the two way binding logic. (If you have time, you can add one or two test method to test two-way binding logic in this file https://github.com/enisn/UraniumUI/blob/develop/test/UraniumUI.Material.Tests/Controls/TextField_Test.cs)


Trouble reading backed objects

Hey. I've run into a couple issues with reading in backed objects with a raw representation.
The first is just the case of reading in an object with a raw attribute:
import scanpy.api as sc
import numpy as np

adata = sc.AnnData(X=np.random.binomial(100, .01, (100, 100)))
adata.raw = adata.copy()
sc.pp.log1p(adata) # Just so they are different
adata.write("./tmp.h5ad")
sc.read("tmp.h5ad", backed="r")

 traceback
AttributeError                            Traceback (most recent call last)
<ipython-input-2-7e0cdbc773a6> in <module>()
      3 sc.pp.log1p(adata) # Just so they are different
      4 adata.write("./tmp.h5ad")
----> 5 sc.read("tmp.h5ad", backed="r")

/usr/local/lib/python3.6/site-packages/scanpy/readwrite.py in read(filename, backed, sheet, ext, delimiter, first_column_names, backup_url, cache, **kwargs)
     73         return _read(filename, backed=backed, sheet=sheet, ext=ext,
     74                      delimiter=delimiter, first_column_names=first_column_names,
---> 75                      backup_url=backup_url, cache=cache, **kwargs)
     76     # generate filename and read to dict
     77     filekey = filename

/usr/local/lib/python3.6/site-packages/scanpy/readwrite.py in _read(filename, backed, sheet, ext, delimiter, first_column_names, backup_url, cache, suppress_cache_warning, **kwargs)
    274     if ext in {'h5', 'h5ad'}:
    275         if sheet is None:
--> 276             return read_h5ad(filename, backed=backed)
    277         else:
    278             logg.msg('reading sheet', sheet, 'from file', filename, v=4)

/usr/local/lib/python3.6/site-packages/anndata/readwrite/read.py in read_h5ad(filename, backed)
    407     if backed:
    408         # open in backed-mode
--> 409         return AnnData(filename=filename, filemode=backed)
    410     else:
    411         # load everything into memory

/usr/local/lib/python3.6/site-packages/anndata/base.py in __init__(self, X, obs, var, uns, obsm, varm, layers, raw, dtype, shape, filename, filemode, asview, oidx, vidx)
    681                 layers=layers,
    682                 dtype=dtype, shape=shape,
--> 683                 filename=filename, filemode=filemode)
    684
    685     def _init_as_view(self, adata_ref: 'AnnData', oidx: Index, vidx: Index):

/usr/local/lib/python3.6/site-packages/anndata/base.py in _init_as_actual(self, X, obs, var, uns, obsm, varm, raw, layers, dtype, shape, filename, filemode)
    895                     self,
    896                     X=raw['X'],
--> 897                     var=_gen_dataframe(raw['var'], raw['X'].shape[1], ['var_names', 'col_names']),
    898                     varm=raw['varm'] if 'varm' in raw else None)
    899

AttributeError: 'NoneType' object has no attribute 'shape'

Additionally the reader doesn't clean up after itself if it errors. In the same session:
adata.write("./tmp.h5ad")

 traceback
---------------------------------------------------------------------------
OSError                                   Traceback (most recent call last)
<ipython-input-3-e04b48216e1b> in <module>()
----> 1 adata.write("./tmp.h5ad")

/usr/local/lib/python3.6/site-packages/anndata/base.py in write(self, filename, compression, compression_opts, force_dense)
   1887
   1888         _write_h5ad(filename, self, compression=compression, compression_opts=compression_opts,
-> 1889                                                              force_dense=force_dense)
   1890         if self.isbacked:
   1891             self.file.close()

/usr/local/lib/python3.6/site-packages/anndata/readwrite/write.py in _write_h5ad(filename, adata, force_dense, **kwargs)
    218         d['X'] = adata.X[:]
    219     # need to use 'a' if backed, otherwise we loose the backed objects
--> 220     with h5py.File(filename, 'a' if adata.isbacked else 'w', force_dense=force_dense) as f:
    221         for key, value in d.items():
    222             _write_key_value_to_h5(f, key, value, **kwargs)

/usr/local/lib/python3.6/site-packages/anndata/h5py/h5sparse.py in __init__(self, name, mode, driver, libver, userblock_size, swmr, force_dense, **kwds)
    139             userblock_size=userblock_size,
    140             swmr=swmr,
--> 141             **kwds,
    142         )
    143         super().__init__(self.h5f, force_dense)

/usr/local/lib/python3.6/site-packages/h5py/_hl/files.py in __init__(self, name, mode, driver, libver, userblock_size, swmr, **kwds)
    310             with phil:
    311                 fapl = make_fapl(driver, libver, **kwds)
--> 312                 fid = make_fid(name, mode, userblock_size, fapl, swmr=swmr)
    313
    314                 if swmr_support:

/usr/local/lib/python3.6/site-packages/h5py/_hl/files.py in make_fid(name, mode, userblock_size, fapl, fcpl, swmr)
    146         fid = h5f.create(name, h5f.ACC_EXCL, fapl=fapl, fcpl=fcpl)
    147     elif mode == 'w':
--> 148         fid = h5f.create(name, h5f.ACC_TRUNC, fapl=fapl, fcpl=fcpl)
    149     elif mode == 'a':
    150         # Open in append mode (read/write).

h5py/_objects.pyx in h5py._objects.with_phil.wrapper()

h5py/_objects.pyx in h5py._objects.with_phil.wrapper()

h5py/h5f.pyx in h5py.h5f.create()

OSError: Unable to create file (unable to truncate a file which is already open)

I get these errors using v0.6.10 and the current master branch.

Thank you for pointing this out.
Yes, no tests for the backed mode when the .raw attribute is set. The whole backed mode is still in development, even though essential things work (incremental PCA on a backed data, plotting a backed object, etc.)...
@Koncopd: do you have bandwidth for looking into this? You're the one that worked on the backed mode most recently.

@falexwolf
Yep. i'll try to look.

@ivirshup
Should work now...
https://github.com/theislab/anndata/commit/415502ceaa8478ab915489eeeef261e2dd257667

Thanks!


Adds scaffolding and mock support for PATCH endpoints

Description

Adds scafolding for PATCH endpoints
Adds stub for patching RegisteredModels
Fixes a bug where the location header in the response of POST/PATCH endpoints did not return the correct URL

How Has This Been Tested?

Run in mock mode
Run new PATCH request in README via curl
Verify response is in correct envelope format
Verify location header does not contain an unparsed template placeholder

Merge criteria:

All the commits have been signed-off  (To pass the DCO check)

[x] The commits have meaningful messages; the author will squash them after approval or in case of manual merges will ask to merge with squash.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work.
[x] Code changes follow the kubeflow contribution guidelines.

If you have UI changes

[x] The developer has added tests or explained why testing cannot be added.
[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Verify that UI/UX changes conform the UX guidelines for Kubeflow.

/lgtm


Integration Details - AdminLTE v3.2.0

Only a few pages are provided in this first version:

All pages required by Django.contrib.AUTH
Pages for the Django Admin Section (superusers)
LIVE Configurator (the collapsible panel on the right)
Pages for common users:

https://adminlte.io/themes/v3/index.html
https://adminlte.io/themes/v3/pages/widgets.html
https://adminlte.io/themes/v3/pages/charts/chartjs.html
https://adminlte.io/themes/v3/pages/forms/general.html
https://adminlte.io/themes/v3/pages/calendar.html
https://adminlte.io/themes/v3/pages/examples/profile.html

All other pages will be migrated in 2nd iteration.
ty!

Hello @mominur-helios
Ty for your work. Just installed the theme and the layout is not usable.
For tests, you can use this playground project, already configured
https://github.com/app-generator/django-admin-adminlte_p
The dist directory is not shipped .. and probably other assets.
Please take a look. Ty!

Validated. Closing.
Ty @mominur-helios . Great work


HTMLparser module

ModuleNotFoundError: No module named 'HTMLParser'
[152/21839] CXX obj/base/base/base_jumbo_5.obj
ninja: build stopped: subcommand failed.

Not sure why am I getting this issue. HTMLParser is a python 2 module, but I guess it should work with python 3 with 2to3.
Did anyone have a same issue?

Sounds like a problem with tools/metrics/histograms/extract_histograms.py. Unfortunately, that's not the only Python 2 script that needs to be fixed so it runs on Python 3.
Right now, I have a patch in my fork (on the python3-support branch) that can fix those scripts, but it's incomplete. Progress on Python 3 migration on the Chromium Bug Tracker is currently a bit slow-going.

Thanks for a reply!
Yes, the histogram file was causing a problem.
Since it's a known issue, how do you ever build it with python3?

It works by setting python3 as py and python2.7 as python in path.
Here is the latest version to download.

Don't worry. I still use the old build instructions to build, since that's the only way to get this browser built at the moment.

@tangalbert919 If that's the case, we should update the README. Otherwise, it'll be frustrating for users who are trying to build.

Don't worry. I still use the old build instructions to build, since that's the only way to get this browser built at the moment.
Can you share that old build instructions and old build.py?

Fixed in #19


tests(hostcalls) drop url dependency; improve load time

Removing the url crate, reduces hostcalls load time in Wasmtime by 25%.

Awesome!


to fix probelm in single upload hander

when uploading a single chunk file (smaller than maximum chunk size)
a error is thrown
so i fixed it by removing the config parameter from the method header and added $this before the config parameter in singleSave

please fix it asap :)

Thanks for the catch! Missed that after refactoring changes.


Make ChannelHandler removal cheaper.

Motivation:
Any version of ChannelHandler removal that does not have a
ChannelHandlerContext already in hand is currently excessively
expensive. This is because it allocates a promise and a callback for
finding the context, despite already having a promise in hand for users
to complete.
We can remove a pair of allocations here by jumping to the event loop
directly and then running our operations synchronously.
Modifications:

Rewrite removeHandler(name:promise:) and removeHandler(_:promise:) to
jump directly to the event loops and then work synchronously.

Result:
Cheaper code

I'm waiting to see if the alloc counter tests are affected by this. If they aren't, I'll write one.

For future readers: yes, this patch saves 10 allocations from the use-case where we do not already have a ChannelHandlerContext, more than 50% of the allocations on that operation.

great stuff!

50% win, super cool, AHC will thank you :)


Question about the sphere sample

Hi, could you please point me to some references (e.g., paper or blog) on the sphere sampling which is used in s3dis_sphere.py? I tried to google it but couldn't find anything about it.

@ZhengGongzi Thank you for the reply. I find it hard for me to understand s3dis_sphere.py. I would greatly appreciate your help in understanding it!
First, could you please explain the difference in pre-processing performed by s3dis.py and s3dis_sphere.py?
Second, why does Pix4Point not adopt the same data pre-processing as PointNeXt (s3dis.py)?"

@linhaojia13 s3dis_sphere.py involves some not difficult but trivial mechanisms, and it will be more convenient for us to discuss them together through WeChat. I have sent my WeChat account to your email.


Sijliu/test

@bradenrc sijliu/test branch is used for my experimental purpose only, there is no guarantee it works...


Port lib files

@mhegazy No longer updated. Can you port and publish it to check it before the release of the next version? You've less than 2 weeks left before the release.

We are waiting on some legal issues to be resolved now that these are generated from whatwg/w3c which have their own licences..

So never reflects https://github.com/Microsoft/TSJS-lib-generator? Some fixes should be ported.

the current state bits in the master of https://github.com/Microsoft/TSJS-lib-generator is based on w3c and whatwg specs, and that needs some attribution before we use it.
We will not be porting anything until we have legal signoff on the generation process.

Any tracking issues about the legal issues?

i am afraid not. that is an internal MS process.

okay, thanks.