Pull request: Nullable float

Hello, gentlemen!
I propose we add pandas' nullable float as a valid type to pandera's engine.
Here is more information on that specific datatype: https://pandas.pydata.org/pandas-docs/stable/whatsnew/v1.2.0.html?highlight=float64dtype#experimental-nullable-data-types-for-float-data
Please let me know what you think about this suggestion.
Regards,
Vladimir

Thanks for the contribution @vovavili ! For this PR to be complete it should add unit tests associated with this change

also be sure to setup pre-commit https://pandera.readthedocs.io/en/stable/CONTRIBUTING.html#set-up-pre-commit
looks like there are some linting issues in the PR

also, for reference those unit tests should live in this file: https://github.com/pandera-dev/pandera/blob/master/tests/core/test_dtypes.py

@vovavili friendly ping, let me know if you need any help getting this through the finish line. Basically the source changes are done, just need to add unit tests

this also needs to be rebased on to dev and we can change the PR target branch to dev as well

@vovavili made some updates to the tests, should be pretty much good to go!
would you mind rebasing your changes onto the dev branch and re-target this PR to dev?

@vovavili made some updates to the tests, should be pretty much good to go!
would you mind rebasing your changes onto the dev branch and re-target this PR to dev?

Done :)

looks like there are test failures across the CI matrix, did you run the unit tests?
locally you can't get the OS part of the CI matrix, but you can use nox to run on different versions of python and pandas
nox -l

# output
...
* tests-3.7(extra='core', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='strategies', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='hypotheses', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='io', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='koalas', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='modin', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='modin-ray', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='modin-dask', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='dask', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='mypy', pandas='1.1.5') -> Run the test suite.
...

Then run a specific set of unit tests
nox -s "tests-3.7(extra='core', pandas='1.1.5')"

if you use conda you can do something like:
nox -s "tests-3.7(extra='core', pandas='1.1.5')" -db conda

looks like there are test failures across the CI matrix, did you run the unit tests?
locally you can't get the OS part of the CI matrix, but you can use nox to run on different versions of python and pandas
nox -l

# output
...
* tests-3.7(extra='core', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='strategies', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='hypotheses', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='io', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='koalas', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='modin', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='modin-ray', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='modin-dask', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='dask', pandas='1.1.5') -> Run the test suite.
* tests-3.7(extra='mypy', pandas='1.1.5') -> Run the test suite.
...

Then run a specific set of unit tests
nox -s "tests-3.7(extra='core', pandas='1.1.5')"

if you use conda you can do something like:
nox -s "tests-3.7(extra='core', pandas='1.1.5')" -db conda

Okay, will do.

@vovavili I'm overhauling CI infra in https://github.com/pandera-dev/pandera/pull/724, will keep you updated.

thanks for the contribution @vovavili ! 🚀


Add spatialite-tools and its missing dependency (readosm)

@KrzyStar Some of the tools seem to segfault...

Some of them don't, though. Merged and fixed licenses

@7c6f434c shall this be closed?

Seems there's no point keeping it open.


Only one MTR probe is working although multiple MTR targets are configured

Hi,
I've configured multiple MTR targets:
conf:
refresh: 15m
nameserver: <IP_ADDRESS>:53
icmp:
interval: 3s
timeout: 1s
count: 10
mtr:
interval: 3s
timeout: 500ms
max-hops: 30
count: 3
tcp:
interval: 3s
timeout: 1s
http_get:
interval: 15m
timeout: 5s
targets:

name: google-dns1
host: <IP_ADDRESS>
type: ICMP
name: google-dns2
host: <IP_ADDRESS>
type: MTR

However, in the exporter, i can see only one MTR. also MTR count metric is 1

If the above is your current config, this is normal you have only defined one MTR target.
For this target you will see the traceroute path in the metrics mtr_rtt_seconds..

Sorry, this is the correct configuration:
conf:
refresh: 15m
icmp:
interval: 3s
timeout: 1s
count: 10
mtr:
interval: 3s
timeout: 500ms
max-hops: 30
count: 3
tcp:
interval: 3s
timeout: 1s
http_get:
interval: 15m
timeout: 5s
targets:

name: google-dns1
host: <IP_ADDRESS>
type: MTR
name: google-dns2
host: <IP_ADDRESS>
type: MTR

See attached.
ping_exporter_metrics.txt

Looking at this I found a labeling issue with the mtr_hops metric but it should not be related.
Can you please try running the 1.3.1 version with the logging level at debug mode --log.level=debug and when the output + the metrics

it works perfectly on 1.3.1:
mtr_targets 2
see attached.
To run on debug mode, can i add parameter to docker run or should i change the Dockerfile and build?
Do you need still the debug log?
Thanks!!
metrics1_3_1.txt

From the image it looks ok now.
If you want you can use the same images, you just need to pass the params:
docker run --rm -p 9427:9427 -v $PWD/ping_exporter.yml:/ping_exporter.yml:ro --name ping_exporter syepes/ping_exporter /app/ping_exporter --log.level=debug


I CANT COMPILE ON 4.0

The code shows erros on some parts of code when i try to compile with 4.0 .
if i wrong i need download de source code for 4.0 of another repo? thanks

Hi.  There is a net40 branch.  No nuget, you must compile yourself.  Evey so often I update it from master.  See:

https://github.com/ButchersBoy/MaterialDesignInXamlToolkit/issues/31
https://github.com/ButchersBoy/MaterialDesignInXamlToolkit/tree/net40

Please note the net40 branch still requires VS2015 though for C#6 features.

Thanks for help guy


Play/pause issues on Big Sur

Play/pause button becomes unclickable like it doesn't exist. The same happens to the blue dragging thingy that sets the time. Once the timer is set and running. It cannot be changed nor paused.
Further there is now a white border at the bottom. (See screenshot)
(On Big Sur beta running on Macbook Air 2017)

Open the timer
Set the timer
Let it run for 5 sec then try to pause. Nothing happens
Try to drag the blue thingy around. Nothing happens

Desktop

OS version: macOS 11 beta]
Version: 1.5.5

Thanks @espresso-math for reporting this. We'll need people to test the new OS for us.
No good that the app breaks within it.

Always happy to do what I can to help. Cannot stress how useful this app has become to me. I use it everyday.

@espresso-math I'm glad. I'll install Big Sur soon to see what the issue is.
@robinstewart do you happen to have Big Sur installed?

@robinstewart Yes, I updated to it a few days ago.

@robinstewart  I have a new observation. The play/pause works as long as I don't try to move the blue slider. As soon as I try to move the blue slider the whole timer window moves with he cursor. After that the play/pause doesn't work and I have to restart the timer or create a new one. Once it gets stuck, if I make a new timer I drag it and set the timer but the same problem appears if I try to drag the slider around change the time.

Thanks @espresso-math. Unfortunately no I don't have Big Sur installed yet. I guess it's not too surprising something broke, given the significant visual changes in Big Sur.
If anyone else wants to look into this, Xcode > Debug menu > View Debugging > Capture View Hierarchy has been helpful to me in the past...

When running this locally on Big Sur, the mouseDown event is never being hit so didDrag has no way of being reset to false. The other events seemed to be firing reliably.

It is working for me now, as expected, with the new Timer 1.6 on Big Sur 11.1.

It is working for me now, as expected, with the new Timer 1.6 on Big Sur 11.1.

Hate to bring this issue up again, but @peasarechic’s issue is still very much alive

As soon as I try to move the blue slider the whole timer window moves with he cursor.

This is on MacOS 11.1 (on a Macbook Air M1, to be exact).

Hate to bring this issue up again, but @peasarechic’s issue is still very much alive

As soon as I try to move the blue slider the whole timer window moves with he cursor.

This is on MacOS 11.1 (on a Macbook Air M1, to be exact).

@phortuin can you share what version of the app you're using?

@phortuin can you share what version of the app you're using?

@karbassi 1.6.0

@karbassi 1.6.0, installed via Homebrew

To pin-point the issue, could you uninstall and reinstall
brew uninstall --zap michaelvillar-timer
brew install --cask michaelvillar-timer

Done, behaviour hasn't changed:
$ brew uninstall --zap michaelvillar-timer
==> Implied `brew uninstall --cask michaelvillar-timer`
==> Backing App 'Timer.app' up to '/usr/local/Caskroom/michaelvillar-timer/1.6.0
==> Removing App '/Applications/Timer.app'
Warning: No zap stanza present for Cask 'michaelvillar-timer'
==> Removing all staged versions of Cask 'michaelvillar-timer'

$ brew install --cask michaelvillar-timer
==> Downloading https://github.com/michaelvillar/timer-app/releases/download/1.6
Already downloaded: /Users/redacted/Library/Caches/Homebrew/downloads/51057c934a474f9b0fc5d58ce5e51828e593d0c3f234c2f82f478b09d38bf8f5--Timer.app.zip
==> Installing Cask michaelvillar-timer
==> Moving App 'Timer.app' to '/Applications/Timer.app'
🍺  michaelvillar-timer was successfully installed!

Specs have changed a bit since last time: MacOS 11.2 on a MacBook Air M1, using an external 4k screen at 1920×1080 (2× pixel ratio)

Update: I've taken about 2 hours to look into this but with no luck. It is broken in Big Sur (11.2.1) from my testing and probably broken in 11.2.x.
@robinstewart do you have time to take a look?

This is still broken in 14.1.2.  It forces me to create a new timer every time I need to change an existing timer with the blue handle in the lower part of the window, and pause just cannot be used.


☂ Remove accentColor and accentTextTheme from ThemeData et al.

The following proposal is from the "Deprecate/remove the accent properties" section of flutter.dev/go/material-theme-system-updates.
The Material Design spec no longer includes a color called ‘accent’. The ‘secondary’ color scheme color is roughly equivalent. The overall theme’s accent properties are rarely used within Flutter itself. For example the accentIconTheme is only used by FloatingActionButton and in that case, it’s only used (in one expression) for the sake of backwards compatibility. The accentColorBrightness property is only used by GridTile. We plan to replace component dependencies on  the accent properties with dependencies on component themes, and the overall theme’s color scheme and text theme.
Color accentColor;
Brightness accentColorBrightness;
TextTheme accentTextTheme;
IconThemeData accentIconTheme;

The process for removing these properties will take a considerable amount of time because they’ve been around since early versions of Flutter.

Remove dependencies - but not definitions - from Flutter.
When all of the accent properties dependencies have been removed, send a breaking change announcement.
Deprecate the properties and constructor parameters.
Remove the deprecated properties and constructor parameters.

accentColor
The following classes still depend on accentColor (checked items have been updated):

[x] BottomNavigationBar PR: https://github.com/flutter/flutter/pull/76810
[x] CheckboxListTile PR: #76908
[x] ExpansionTile:  PR: https://github.com/flutter/flutter/pull/79829
Original PR was reverted: https://github.com/flutter/flutter/pull/77933
Added #78281 to enable migrating existing apps that depended on accentColor.
Added #81241 - use primary instead of secondary for expanded icons/titles.
[x] FloatingActionButton #78296
Some of this work was completed, see flutter.dev/go/remove-fab-accent-theme-dependency and #46923.
[x] GridTileBar PR: #78574
[x] InputDecorator PR: #77999
[x] ListTile PR: #77004
[x] MaterialApp PR: #77652
[x] ProgressIndicator PR: #77976
[x] Added a ProgressIndicatorTheme. PR: #81075
[x] PaginatedDataTable PR: #77744
[x] RadioListTile PR: #76906
[x] RefreshIndicator PR: #77884
[x] Stepper PR: https://github.com/flutter/flutter/pull/77732
[x] SwitchListTile PR: #76909
[x] TabPageSelector PR: #77997

accentTextTheme
The following classes still depend on accentTextTheme (checked items have been updated):

[x] TimePicker PR: #78771
[x] DayPicker PR: #78576
This class is deprecated.
[x] Typography PR: #78371

The only remaining references to accentColor and accentTextTheme are for backwards compatibility in ThemeData, ColorScheme, and related tests.

This is under "Not yet released to stable" but InputDecorator PR: #77999 is on stable channel.
Other classes may be affected as well.
Was this intended?


Use new erlcmd-server to create exec gen_server

Depends on:

https://github.com/lfex/erlcmd-server/issues/1

This ticket was created in the wrong repo.
Correct ticket:

https://github.com/ut-proj/undermidi/issues/48


java.nio.file.Paths.get not working

I am having a bug running Java code with your package :
java.lang.UnsatisfiedLinkError: /opt/local/conda/envs/scijava/jre/lib/amd64/libnio.so: libnet.so: cannot open shared object file: No such file or directory
	at java.lang.ClassLoader$NativeLibrary.load(Native Method)
	at java.lang.ClassLoader.loadLibrary0(ClassLoader.java:1941)
	at java.lang.ClassLoader.loadLibrary(ClassLoader.java:1845)
	at java.lang.Runtime.loadLibrary0(Runtime.java:870)
	at java.lang.System.loadLibrary(System.java:1122)
	at sun.nio.fs.UnixNativeDispatcher$1.run(UnixNativeDispatcher.java:573)
	at sun.nio.fs.UnixNativeDispatcher$1.run(UnixNativeDispatcher.java:571)
	at java.security.AccessController.doPrivileged(Native Method)
	at sun.nio.fs.UnixNativeDispatcher.<clinit>(UnixNativeDispatcher.java:571)
	at sun.nio.fs.UnixFileSystem.<init>(UnixFileSystem.java:67)
	at sun.nio.fs.LinuxFileSystem.<init>(LinuxFileSystem.java:39)
	at sun.nio.fs.LinuxFileSystemProvider.newFileSystem(LinuxFileSystemProvider.java:46)
	at sun.nio.fs.LinuxFileSystemProvider.newFileSystem(LinuxFileSystemProvider.java:39)
	at sun.nio.fs.UnixFileSystemProvider.<init>(UnixFileSystemProvider.java:56)
	at sun.nio.fs.LinuxFileSystemProvider.<init>(LinuxFileSystemProvider.java:41)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at java.lang.Class.newInstance(Class.java:442)
	at sun.nio.fs.DefaultFileSystemProvider.createProvider(DefaultFileSystemProvider.java:48)
	at sun.nio.fs.DefaultFileSystemProvider.create(DefaultFileSystemProvider.java:63)
	at java.nio.file.FileSystems$DefaultFileSystemHolder.getDefaultProvider(FileSystems.java:108)
	at java.nio.file.FileSystems$DefaultFileSystemHolder.access$000(FileSystems.java:89)
	at java.nio.file.FileSystems$DefaultFileSystemHolder$1.run(FileSystems.java:98)
	at java.nio.file.FileSystems$DefaultFileSystemHolder$1.run(FileSystems.java:96)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.nio.file.FileSystems$DefaultFileSystemHolder.defaultFileSystem(FileSystems.java:96)
	at java.nio.file.FileSystems$DefaultFileSystemHolder.<clinit>(FileSystems.java:90)
	at java.nio.file.FileSystems.getDefault(FileSystems.java:176)
	at java.nio.file.Paths.get(Paths.java:84)
	at org.scijava.jupyter.commands.InstallScijavaKernel.installLanguage(InstallScijavaKernel.java:140)
	at org.scijava.jupyter.commands.InstallScijavaKernel.lambda$run$0(InstallScijavaKernel.java:79)
	at java.util.ArrayList.forEach(ArrayList.java:1249)
	at org.scijava.jupyter.commands.InstallScijavaKernel.run(InstallScijavaKernel.java:77)
	at org.scijava.command.CommandModule.run(CommandModule.java:198)
	at org.scijava.module.ModuleRunner.run(ModuleRunner.java:167)
	at org.scijava.module.ModuleRunner.call(ModuleRunner.java:126)
	at org.scijava.module.ModuleRunner.call(ModuleRunner.java:65)
	at org.scijava.thread.DefaultThreadService$2.call(DefaultThreadService.java:191)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)

Any idea ?

Ping @johanneskoester @sodre

The libs are there, but it cannot find them because the zulu build puts them in a nonstandard location. It might be necessary that we set the LD_LIBRARY_PATH in the activate script? Maybe somebody from @conda-forge/core has an opinion what would be the canonical way to proceed?

Adding LD_LIBRARY_PATH is not the conda way. I'll try to patch the build.
Patrick
Sent from my iPhone

On Apr 19, 2017, at 5:18 AM, Johannes Köster<EMAIL_ADDRESS>wrote:
The libs are there, but it cannot find them because the zulu build puts them in a nonstandard location. It might be necessary that we set the LD_LIBRARY_PATH in the activate script? Maybe somebody from @conda-forge/core has an opinion what would be the canonical way to proceed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@hadim,
I was not able to replicate your issue with conda-forge/openjdk. However, if I used continuum.io's openjdk then the problem comes up.
Can you make sure that you are using conda-forge's openjdk? Here is my output for conda list...
(openjdk) root at f8895893-26a7-68bb-8f4d-d10d39b9c4e0 in ~/openjdk-feedstock/recipe/test-nio on rpath*
# conda list
# packages in environment at /root/envs/openjdk:
#
openjdk                   8.0.112          zulu<IP_ADDRESS>_2    conda-forge
(openjdk) root at f8895893-26a7-68bb-8f4d-d10d39b9c4e0 in ~/openjdk-feedstock

@johanneskoester
if @hadim is really using continuum's io openjdk, we should still add the test case for NIO. In that case we might as well take the time to upgrade Azul to the latest version. I can amend my PR #8 to do that.
Would you also be okay with me an additional maintainer?

Of course, please add yourself! I am not using java and have almost no experience with it. I just needed it for the R migration. Hence, I would be very happy to have an additional maintainer.

Working now. Thanks for the help both of you.

FYI I think that the bug I saw was using the openjdk from the default channel and not the conda-forge one.


Using snapToOffsets instead of the custom snap code

I noticed that there's some sort of loop happening when doing very small momentum scrolls within the header.
I'm not sure if this reproduces outside of my project, and I also use react-navigation via a WIP PR I'm working on for #7
What seems to be happening is that onMomentumScrollEnd triggers via the user gesture, then syncScrollOffset calls scrollScene which then also triggers onMomentumScrollEnd and then there's a battle between the two triggering a loop.
This seems to have also been reported here: https://stackoverflow.com/questions/61715531/abrupt-scrolltooffset-calls-in-onmomentumscrollend-react-native
However, it seems to be possible to use https://reactnative.dev/docs/scrollview#snaptooffsets with values of [0, headerHeight] to achieve the same behavior as the custom scroll snapping. Is there any reason not to use snapToOffsets?
A current workaround is to set disableSnap to true and add snapToOffsets to my own list views

I guess one reason for using the custom snap code is the ability to override the snapThreshold.
The native snapToOffsets prop has improved behavior and stops momentum scrolling at the snap points, so the header doesn't peek through before snapping back, which may be preferable.

May need to verify that it doesn't break cases when the tab content slightly overshoots the window size by less than the snap offset, so it doesn't get stuck offscreen with the list snapping back and not allowing access to it.

The snapToOffsets and snapToInterval indeed are a better solution for the ScrollView, but won't work for the FlatList. The current implementation is a simple way to handle both cases at the same time/same way.

I did a quick test on the examples folder, adding snapToOffsets={[0, HEADER_HEIGHT]} to all scrollable components and running on the Collapsible Tab View no snap demo, wich has disableSnap={true}, and it looks like working fine for scroll view, but does not work for flatlist.

Weird. It works with FlatList in the app I'm working on with no issues. I didn't test the examples folder though, something must be different.

@PedroBern make sure to also add snapToEnd={false} otherwise the snap points keep repeating for the rest of the list, resulting in weird scrolling behavior.

@PedroBern make sure to also add snapToEnd={false} otherwise the snap points keep repeating for the rest of the list, resulting in weird scrolling behavior.

😝  forgot, I will try it again later

@andreialecu Can you make a review of #11, please? There was one thing I didn't understand, after fixing #8 and moving to snapToOffsets if I didn't add this line, the albums tab would double in height.

I was able to fix the glitch in #12. I propose we keep the custom snapping code and if someone wants to override they can use disableSnap: true and their own custom snap offsets.


Add verbose option

Amber gives a lot of option to ignore things for different use cases which is great.
However it also comes with cost that there is a high chance that amber will run differently on different environments. And that could become hard to debug what things are being ignored and why.
In my use case is that running amber on Github Actions and somehow the target files being ignored and I just can't find why and the nature of Github Actions makes it hard to debug. Which is not a scope for Amber of course but I think having verbose option to see what amber is processing and skipping or ignoring and why would be awesome nonetheless.

I've added --verbose option.

Fantastic, thanks!


Chaining multiple methods that use the spread operator breaks this value

Here's a minimal reproduction: https://buble.surge.sh/#class Test {
	constructor(){
      this.list %3D []%3B
    }
  	add(...items){
      this.list.push(...items)%3B
      return this%3B
    }
}

let numbers %3D [1%2C2%2C3]%3B
let letters %3D ["A"%2C"B"%2C"C"]%3B

%2F%2F OK%2C uses correct ref
new Test()
    .add(...letters)%3B

%2F%2F NOT OK%2C first .add() will have undefined `this` value
new Test()
	.add(...numbers)
  	.add(...letters)%3B
When chaining:
new Test()
	.add(...numbers)
  	.add(...letters);

The transpiled output:
(ref$2 = (ref$1 = new Test())
	.add.apply(ref$2, numbers)) // This will throw
  	.add.apply(ref$1, letters);

Uses ref$2 as the this value for the first chained call before it has a value.
I'm guessing the desired output would be something like?
(ref$3 = (ref$2 = (ref$1 = new Test()))
  	.add.apply(ref$2, numbers))
  	.add.apply(ref$3, letters);

Note that this bug only appears to apply when chaining directly off the construction of the object, if you save it to an intermediate variable first, like:
let x = new Test();

x.add(...numbers).add(...letters);

Then the results are OK.


[api] Adding set_ownership action to services.

Add set_ownership action to services.
Add generic set_ownership_resource method
Support owner by id, href, name or userid
Support group by id, href  or description
Adding specs

https://trello.com/c/BlB4OVnO

Enhancement to add the new set_ownership action to services. Unlike set_owner to VMs which implemented the SOAP's set_owner, this action implements what the UI functionality support, both owner and group of CI's.
Implemented as a generic action, so adding this action to other CI's would simply be updating the api.yml.
Owners can be specified by href, id, name or userid attribute.
Groups can be specified by href, id or description.
Usage:
POST /api/services/:id

{
  "action" : "set_ownership",
  "resource" : {
    "owner" : { "href" : "http://localhost:3000/api/users/:user_id" },
    "group" : { "href" : "http://localhost:3000/api/groups/:group_id" }
  }
}

Other example POSTs
{
  "action" : "set_ownership",
  "resource" : {
    "owner" : { "userid" : "admin" },
    "group" : { "description" : "EvmGroup-super_administrator" }
  }
}

{
  "action" : "set_ownership",
  "resource" : {
    "owner" : { "id" : :user_id }
  }
}

Note, that ownership to multiple services can be performed by targeting the collection as follows:
POST /api/services

{
  "action" : "set_ownership",
  "resources" : [
    { "href" : "http://localhost:3000/api/services/101", "owner" : { "userid" : "admin" },
    { "href" : "http://localhost:3000/api/services/102", "owner" : { "userid" : "admin" },
    { "href" : "http://localhost:3000/api/services/103", "owner" : { "userid" : "admin" },
     ...

  ]
}

Responses are the standard API action results as follows:
{
  "success": true,
  "message": "setting ownership of services id 104 to owner: admin, group: EvmGroup-super_administrator",
  "href": "http://localhost:3000/api/services/104"
}

@gtanzillo @imtayadeway please review.

:+1: LGTM

@abellotti Please rebase and push again


Issues downloading parquet file from huggingface

Hi, I was following the instruction from this link, which is for downloading laion5b_h14 embeddings for building local backend.
But I am stuck with step 6, which is downloading embeddings in parquet file from huggingface. (I was currently downloading only en-embeddings , without other embeddings), because the next step gives me an error message that the input data(parquet file) is empty.
The file seems weird since the format does not end with <.parquet> as well.

How can I solve this issue? Thanks in advance.

Relevant Issue: https://superuser.com/questions/1661649/how-to-stop-aria2-from-renaming-html-files
Solution 1. Specify the output filename with -o option.
for i in {0000..2314}; do aria2c -x 16 -o metadata_$i.parquet https://huggingface.co/datasets/laion/laion2b-en-vit-h-14-embeddings/resolve/main/metadata/metadata_$i.parquet; done

Solution 2. Use another downloader like wget. You can simply replace
for i in {0000..2314}; do aria2c -x 16 https://huggingface.co/datasets/laion/laion2b-en-vit-h-14-embeddings/resolve/main/metadata/metadata_$i.parquet; done

with
for i in {0000..2314}; do wget https://huggingface.co/datasets/laion/laion2b-en-vit-h-14-embeddings/resolve/main/metadata/metadata_$i.parquet; done

However, it is about 3x slower in my case.


A completely new version

as mentioned in the comments I suggest some some refactoring, but other than that I think this is ready to be merged. I would send my PR afterwards so we don't make this beast even bigger.
 Great job! :)


Add semester to Course and Queue collections

H/T Judah Schvimer

I can't think of particularly strong reasons to do this. Courses can live on and become active/inactive when necessary. Closing for now.


[path_provider] Bump kotlin to 1.7.10

Fixes https://github.com/flutter/flutter/issues/109397 after large gradle bump in https://github.com/flutter/plugins/pull/6189

Landing to fix tree

@GaryQian FYI the waiting for tree to go green label has been deprecated, autosubmit is the new hotness.

Ahh! Ok, I've been seeing Autosubmit, was not aware of the formal switch to it. Thanks!


Initial flash screen

Change the initial startup flashscreen to black. My Bookwyrm theme is set to Dark so the bright white flash is very annoying.
Android version : 8.1.0
App version : 1.3.1 (8)

Thanks for reporting!
Fortunately it was an easy fix.
I will release it maybe later today. Or tomorrow.

For future reference :

Fixed with https://github.com/PrivacyDragon/Bookwyrm_Android/commit/8037938adfa988e272453b0e6e3a986e68801b7c

Thanks


Fancy Zones: Layout Needs Scrolling - Apply is Off Screen

ℹ Computer information

PowerToys version: 0.18.1
PowerToy Utility: Fancy Zones
Running PowerToys as Admin: Yes
Windows build number: [run "winver"] 19041.450

📝 Provide detailed reproduction steps (if any)

Open Fancy Zones > Custom Themes
Add enough custom themes that they no longer fit on your screen
See no Scroll option and delete themes so you can get to the Apply button

✔ Expected result
What is the expected result of the above steps?
Ability to scroll down to Apply
❌ Actual result
What is the actual result of the above steps?
No way to see Apply without deleting layouts. Apply goes off screen.

I move my computer between multiple monitor configurations and locations, and I have layouts for each so having to delete is not ideal.

📷 Screenshots

Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

Duplicate of https://github.com/microsoft/PowerToys/issues/738


Add fastrtps (Fast-DDS) as supported dds middleware

As requested in this discussion https://github.com/mvukov/rules_ros2/discussions/145
This is an attempt to add Fast-DDS as a supported dds middleware to this repo. I am using versions compatible with ROS2 Humble.
Added repo dependencies:

foonathan_memory 0.7-3
fastcdr 1.1.0
fastrtps 2.6.2
rmw_fastrtps 6.2.3
rosidl_typesupport_fastrtps 2.2.1

I have added a build flag dds_vendor, which will be cyclonedds by default. It controls a switch in rmw_implementation.BUILD.bazel.
This is how you can set it to use fastdds instead:
# To use FastDDS, invoke Bazel with `--config=fastdds`.
build:fastdds --@com_github_mvukov_rules_ros2//ros2:dds_vendor=fastdds

I tried to fix the typesupport generation in interfaces.bzl, but getting errors of detail__* and time__* files not generated. Typesupport for fastrtps doesn't have those.

Switched to rmw_fastrtps_dynamic_cpp.
Getting a new, a bit more descriptive error:
Executing tests from //ros2/test:generic_publisher_tests
-----------------------------------------------------------------------------
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from TestGenericPublisher
[ RUN      ] TestGenericPublisher.WhenPublisherDeletedInThread_EnsureCleanNodeCleanup

>>> [rcutils|error_handling.c:108] rcutils_set_error_state()
This error state is being overwritten:

  'Unknown typesupport identifier, at external/ros2_rmw_fastrtps/rmw_fastrtps_dynamic_cpp/src/type_support_registry.cpp:93'

with this new error message:

  'create_publisher() failed to get message_type_support, at external/ros2_rmw_fastrtps/rmw_fastrtps_dynamic_cpp/src/publisher.cpp:181'

rcutils_reset_error() should be called after error handling to avoid this.
<<<
unknown file: Failure
C++ exception with description "failed to initialize rcl node: error not set, at external/ros2_rcl/rcl/src/rcl/publisher.c:116" thrown in the test body.
[  FAILED  ] TestGenericPublisher.WhenPublisherDeletedInThread_EnsureCleanNodeCleanup (5 ms)
[----------] 1 test from TestGenericPublisher (5 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (5 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] TestGenericPublisher.WhenPublisherDeletedInThread_EnsureCleanNodeCleanup

Hi, this is somewhat likely related to https://github.com/mvukov/rules_ros2/pull/2 -- I suspect you'll need to patch rmw_fastrtps_dynamic_cpp in a similar way we patched rmw_cyclone_dds.

Aha! I actually just came to the exact same conclusion haha :smile:
https://github.com/ros2/rmw_fastrtps/blob/b45e73500bc0a2056fdc31197a6f94434b327d55/rmw_fastrtps_dynamic_cpp/src/type_support_common.cpp#L32
== does not work for c strings. It passed when I used strcmp instead!

@mvukov tests are passing and chatter example ran with fastdds :rocket:

When I run tests, I get errors related to
==================== Test output for //ros2/test:test_rclcpp:
terminate called after throwing an instance of 'rclcpp::exceptions::RCLError'
  what():  failed to initialize rcl init options: failed to load shared library 'external/ros2_rmw_fastrtps/librmw_fastrtps_dynamic_cpp.so' due to dlopen error: external/ros2_rmw_fastrtps/librmw_fastrtps_dynamic_cpp.so: undefined symbol: shm_open, at external/ros2_rcutils/src/shared_library.c:99, at external/ros2_rmw_implementation/rmw_implementation/src/functions.cpp:88, at external/ros2_rcl/rcl/src/rcl/init_options.c:75

Please check what shm_open expects and document it. Perhaps add SHM under a flag, like we do for iceoryx.

Strange, I do not get this error when running the tests. I will investigate a bit.

You should not manually edit repositories/ros2_repositories_impl.bzl, but edit repositories/ros2_repo_mappings.yaml and then run bazel run //repositories/private:resolver (if all up-to-date git diff must be empty for this the _impl.bzl file). If needed, you can override a package version from the yaml file.

I ran this command and then the urls were removed from the http_archive rules.

@mvukov fixed comments

You should not manually edit repositories/ros2_repositories_impl.bzl, but edit repositories/ros2_repo_mappings.yaml and then run bazel run //repositories/private:resolver (if all up-to-date git diff must be empty for this the _impl.bzl file). If needed, you can override a package version from the yaml file.

I ran this command and then the urls were removed from the http_archive rules.

OK, I will investigate this.

Strange, I am getting the same error as you when running a regular listener node with fastdds active:
[micro_ros_listener_example_task-2] terminate called after throwing an instance of 'rclcpp::exceptions::RCLError'
[micro_ros_listener_example_task-2]   what():  failed to initialize rcl init options: failed to load shared library 'external/ros2_rmw_fastrtps/librmw_fastrtps_dynamic_cpp.so' due to dlopen error: external/ros2_rmw_fastrtps/librmw_fastrtps_dynamic_cpp.so: undefined symbol: shm_open, at external/ros2_rcutils/src/shared_library.c:99, at external/ros2_rmw_implementation/rmw_implementation/src/functions.cpp:88, at external/ros2_rcl/rcl/src/rcl/init_options.c:75

It works when enabling shared memory transport, but not without...

@mvukov I have learned that we should link all binaries and tests dynamically, see https://github.com/ros2/rmw_fastrtps/issues/703.
This is because some pointers are shared between libraries and static linking creates separate object files which breaks this:
https://github.com/ros2/rmw_fastrtps/issues/703#issuecomment-1659448044
linking dynamically means we do not need to patch this for example:
https://github.com/mvukov/rules_ros2/blob/main/repositories/patches/rmw_cyclonedds-fix-typesupport-conditions-bug.patch

Okay so this works
ros2_cpp_test(
    name = "generic_publisher_tests",
    srcs = ["generic_publisher_tests.cc"],
    linkstatic = False,
    idl_deps = [
        "@ros2_common_interfaces//:std_msgs",
    ],
    deps = [
        "@com_google_googletest//:gtest",
        "@ros2_rclcpp//:rclcpp",
    ],
)

since @ros2_common_interfaces//:std_msgs is loaded as a plugin.
However, it still doesnt work when @ros2_common_interfaces//:std_msgs is used as a cpp library for a node, e.g.:
ros2_cpp_binary(
    name = "publisher",
    srcs = ["publisher.cc"],
    linkstatic = False,
    deps = [
        "@ros2_common_interfaces//:cpp_std_msgs",
        "@ros2_rclcpp//:rclcpp",
    ],
)

Okay so this is strange,
When I run bazelisk run //chatter:chatter --config=fastdds without sestting linkstatic = False, it throws the same error about
'Unknown typesupport identifier, at external/ros2_rmw_fastrtps/rmw_fastrtps_dynamic_cpp/src/type_support_registry.cpp:93'

But if I do set linkstatic = False for both talker and listener, it works!
However, the test does not work, it throws the same error despite linkstatic = False:
bazelisk test //chatter:tests --config=fastdds

I'm afraid linkstatic = False won't be sufficient, but, you can try out whether mechanism like in https://github.com/mvukov/rules_ros2/blob/main/repositories/class_loader.BUILD.bazel#L6 would work for rosidl_typesupport_introspection_*.
Interesting discussion in https://github.com/ros2/rmw_fastrtps/issues/703. FWIW, my take: relying on dynamic linking for correction operation of the code if fragile. Are there other places where == -> strcmp could be applied?

@henriksod Any plans to finish this up?

Sorry I haven't had time. Best to put it on hold if no one else wants to take it up.

Now that #299 is merge I hope adding fastrtps support will be way easier.


Set up diagnostic flags for MCMC convergence

Goal
In a production pipeline, it is nice to have some automated flags thrown when there are convergence criteria that are not met, to alert that these model fits are probably worth manually reviewing.
Would consider some variation of:

acceptable percent of divergent transitions
acceptable percent of parameters with high rhat
acceptable E-BMFI
acceptable maximum tree depth

Could consider the thresholds used here or open to discussion of other thresholds/additional metrics https://github.com/CDCgov/ww-inference-model/blob/9dd766b8da3cd661f7daeb5f6f6127786e4db5ec/R/model_diagnostics.R#L51

This could be implemented as an "overview" page in the diagnostic report.


[Suggestion]The modification of insurance

Describe the idea of new feature
Hope There is a feature could edit the in progress insurance like the return time, retain time or something else.

@Levin-7 This will be a very specific and little used feature. I don't have much time to develop the editor, so I can't devote time to it yet. However, if you or someone else studies the how insurance in the profile file works, provides test files and editing examples, perhaps this function will go into development


Bump KBS to v0.8.0

Let's wait guest-components to be released and then change the rev insode tools/client.
Step 10 in https://github.com/confidential-containers/confidential-containers/issues/145

Guest components is tagged now btw so we can move forward here

looking into the image buiding error


Any method/api to get ip address of access point if its gateway ip is <IP_ADDRESS>

I am connecting my iPhone with an wifi access point. And that access point returns getway ip as nil by getgateway ip method. As that access point gateway ip was <IP_ADDRESS> only so how I can get that access point device ip address in my iPhone.
Can you please suggest any method to get ip address of access point if its gateway ip was <IP_ADDRESS>?

How I can get ip address of dhcp server iOS Swift?


Add linear-gradient() to NameFormat whitelist.

This change adds the CSS function linear-gradient() to the NameFormat whitelist. Since it's not a Sass function it shouldn't be throwing a lint warning.

Thanks!
Merged in 5cb125bc60cba34c8cad409f6061a0e869a50799


Support for Gemma 2?

Hi,
I am wondering whether this support training for second gen Gemma based llava.
When we trained with the new Gemma with this repo, we ran into an error of:
TypeError: '<' not supported between instances of 'NoneType' and 'int'
And debugging lead us to a weir huggingface past key cache bug. Wondering whether you guys have any idea how to solve it.
Thanks!

Hi, I am not sure about this bug. We currently have no plans to test Gemma2 for the llava_gemma project.


thank you for probably the best mod ever

even more than SMART, this mod changed my game so fundamentally, i can honestly say it's the only mod i truly miss in the new Satisfactory 1.0 release.  so very sorry it's been discontinued; i'd hoped you'd update it for 1.0.  regardless, a loooong overdue Thank You for this brilliant mod.  had i the chops, i'd fork a request and carry on your legacy.
happy trails, my dude.  🥲

you absolutel legend!  you made my friggin' year, bro!  never expected to see this working again.  fan-friggin'-tastic!  :D  thank you!  🥳


V#3 Scroll no work

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x ] 3.x
I'm submitting a ...  (check one with "x")
[ x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

The scroll does not work on the device and on the browser when it is clicked but it works with the mouse scroll.
Sorry for my bad English
Ionic info:
Your system information:

Cordova CLI: 6.5.0
Ionic Framework Version: 3.0.1
Ionic CLI Version: 2.2.2
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.3.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.5
Xcode version: Not installed

Have exactly the same issue. I was previously on ionic 2.0.0.
Your system information:

 Cordova CLI: 6.5.0
Ionic Framework Version: 3.0.1
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.3.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 7
Node Version: v7.1.0
Xcode version: Not installed

I think this is related to #11112
One workaround is to remove the (tap) or (press) from the items and use only (click).
In my case it is not enough because I´m using press and tap together.

@ConradoCarvalho Is correct, remove the (press) and work !, thanks, with which I replaced the functionality of (press) is that I need to use it

Hello all! I am going to close this issue as a duplicate of #11112 .


libvirt

Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build libvirt

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

1 package failed to build:

nixops_unstable

27 packages built:

collectd
diffoscope
docker-machine-kvm
docker-machine-kvm2
gnome.gnome-boxes
libguestfs
libguestfs-with-appliance
libvirt
libvirt-glib
minikube
minishift
ocamlPackages.ocaml_libvirt
perl532Packages.SysVirt
perl534Packages.SysVirt
python310Packages.guestfs
python310Packages.libvirt
python39Packages.guestfs
python39Packages.libvirt
rubyPackages.ruby-libvirt
rubyPackages_3_0.ruby-libvirt
rubyPackages_3_1.ruby-libvirt
vagrant
virt-manager
virt-manager-qt
virt-top
virt-viewer
virtlyst

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

1 package marked as broken and skipped:

libvmi

27 packages built:

collectd
diffoscope
docker-machine-kvm
docker-machine-kvm2
gnome.gnome-boxes
haskellPackages.libvirt-hs
libguestfs
libvirt
libvirt-glib
minikube
minishift
nixops_unstable
perl532Packages.SysVirt
perl534Packages.SysVirt
python310Packages.guestfs
python310Packages.libvirt
python39Packages.guestfs
python39Packages.libvirt
rubyPackages.ruby-libvirt (rubyPackages_2_7.ruby-libvirt)
rubyPackages_3_0.ruby-libvirt
rubyPackages_3_1.ruby-libvirt
vagrant
virt-manager
virt-manager-qt
virt-top
virt-viewer
virtlyst

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

1 package failed to build:

nixops_unstable

28 packages built:

appvm
collectd
diffoscope
docker-machine-kvm
docker-machine-kvm2
gnome.gnome-boxes
libguestfs
libguestfs-with-appliance
libvirt
libvirt-glib
minikube
minishift
ocamlPackages.ocaml_libvirt
perl532Packages.SysVirt
perl534Packages.SysVirt
python310Packages.guestfs
python310Packages.libvirt
python39Packages.guestfs
python39Packages.libvirt
rubyPackages.ruby-libvirt
rubyPackages_3_0.ruby-libvirt
rubyPackages_3_1.ruby-libvirt
vagrant
virt-manager
virt-manager-qt
virt-top
virt-viewer
virtlyst

@GrahamcOfBorg build perlPackages.SysVirt

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

1 package marked as broken and skipped:

libvmi

1 package failed to build:

nixops_unstable

27 packages built:

appvm
collectd
diffoscope
docker-machine-kvm
docker-machine-kvm2
gnome.gnome-boxes
haskellPackages.libvirt-hs
libguestfs
libvirt
libvirt-glib
minikube
minishift
perl532Packages.SysVirt
perl534Packages.SysVirt
python310Packages.guestfs
python310Packages.libvirt
python39Packages.guestfs
python39Packages.libvirt
rubyPackages.ruby-libvirt (rubyPackages_2_7.ruby-libvirt)
rubyPackages_3_0.ruby-libvirt
rubyPackages_3_1.ruby-libvirt
vagrant
virt-manager
virt-manager-qt
virt-top
virt-viewer
virtlyst


Click on the menu bar in Visual Studio and select "Extensions and Updates..."

It's better to say: Click "Tools"on the menu bar in Visual Studio and select "Extensions and Updates..."

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

ID: 8b4e9eb5-5185-b93f-361a-0c9247d3f570
Version Independent ID: 2b15cd11-4de9-4ca5-c1f8-c420262f46f2
Content: Quickstart article for Visual Studio Tools for Machine Learning on Azure
Content Source: articles/machine-learning/desktop-workbench/quickstart-visual-studio-tools.md
Service: machine-learning
GitHub Login: @ahgyger
Microsoft Alias: ahgyger

@xiashaohua2010 Thanks for the feedback! We are currently investigating and will update you shortly.

@xiashaohua2010 Thank you very much, I have made the change and it should be fixed shortly.


Please adapt the bootloader to work with different flash-sizes of the ATSAME51 family.

The page size across all E5x/D5x is 512 byte.
But the page size is also available in the NVMCTRL register PARAM as PSZ[2:0].
This supposedly has a value of 0x6 across all members of the family which means 512 byte.
And I just check with the debugger, both the E51J19A and the E51J18A have a 6 in these bits.
Regarding the total flash size, NVMCTRL register PARAM has NVMP[15:0]
which is the amount of pages.
And correctly so the E51J18A has a value of 0x200 in these bits and the E51J19A has a value of 0x400.
So the E51J20A should show 0x800.

I added support for this feature on the master branch. Since I only have a SAME51J19A board available for testing, I couldn't validate if the code works with other boards.
You could try the firmware built from the master branch sources to see if it works for you.

I just tried it with an E51J18A and after the usual trickery with Zadig since I recently re-installed m Win10, it appears to be working just fine, at least on the USB side of things, still need to test the CAN but this is more releated to the board itself.
I read back the flash with Microchip Studio and there is indeed code at 0x20000 or the upper 128k - nice.

Excellent, please let me know how it goes.

I flashed all four interfaces I currently have here and two of these have an E51J18A, the other two have E51J19A.
And three of these work just fine.
The fourth one is the one I finished soldering yesterday and which has the E51J18A and alternative digital isolators populated.
So either I messed up on the CAN side or the digital isolators are not working as expected.
Very nice.
Is the new bootloader also supporting the E51J20A?

Yes, it should. The bootloader / CAN app code gets build for the smallest of the chips which is the E51J18A. To my knowledge, the bigger chips simply have more RAM/ROM.


Removes the DEPLOY_KEY secret

As far as I can see this is no longer used.

Codecov Report

Merging #389 (3e5f9ad) into main (eaff46b) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #389   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           67        67
  Lines         1838      1838
=========================================
  Hits          1838      1838


max_new_tokens as a parameter to answer_question() and default=256 instead of 512

max_new_tokens is added as a parameter to answer_question() and default is set to 256 instead of 512.
Reasons:

In order to reduce the delay when the model enters in loops generating .............. and taking a lot of time (I found such cases with images with text) and to control it.
In test generations I found about 200 was enough for complete text (probably less), but 100 was a bit short.

Also, for avoiding the repetition loops, can some sort of an interrupt be added, as a callback like in GPT4All and a constraint when capturing repetition of tokens, unusual sequences like these dots etc.: repetition_penalty etc.? Also: a streaming mode? (I had a glance on the code, I may try to figure it out.)
Sample repetition loop:
Z:\LMSYS-Free-GPT4-Claude-15-4-2024-2024-04-15_18-33-46.mp4_snapshot_05.28.266.png
The image is a screenshot of a website called "LMSys Chatbot Arena". The website is predominantly white with blue and orange accents. The main focus is a banner at the top of the screen, which is white with red and blue text. The text reads "Free GPT4, claude, llama, code llama, mixral-of-experts, command-r-plus..........................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
The picture itself had "..." in the text.

I'll add this to my list to update for next version, but I'm not sure this PR will work -- syntax looks wrong?
max_new_tokens=max_new_tokens=,

I'll add this to my list to update for next version, but I'm not sure this PR will work -- syntax looks wrong?
max_new_tokens=max_new_tokens=,

Right: https://github.com/vikhyat/moondream/pull/86/commits/cd39aab786fe41042fc49fc1ab5fa3a3b2af2973
I think I edited it in place in a Notebook installation, then I did it too quickly in the cloned repo without testing it, because the Huggingface sandbox happened to be more convenient for the moment.
@jakedahn Thanks for the support. :)
@vikhyat  A wonderful model!


Themes: Dynamic theme installation needs polish

Pre-req: Settings Sync on

Create brand new empty profile
Run Preferences: Color Theme
Pick Browse Additional Color Themes
Pick a theme

🐛 Settings sync asks this strange question:

Additionally:

Cancel this dialog

🐛 Theme is still applied but not installed

@aeschli were there specific steps to repro this?

I can verify that cancelling the dialog no longer leaves the theme applied, thanks @aeschli.
@sandy081  I can still reproduce the main issue though. Why would I not want to synchronize the theme I just picked for my brand new profile, given the profile is being synced?

I think @aeschli is using the general extension install command that will ask user to sync or not before installing. If it is in core, one can use the API directly to install.

I think we still want an install confirmation dialog, but I agree that there should be no question about syncing.


Layout

The only things left to do now  are some power traces and ground stitching.

That's really exciting ! I

Hey @perigoso . this looks good to me, but let me know if you are ready to merge ?

You can merge


R119 bcfix

fixes/improvements for various things

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/57/


chore: Turn URL comments into properties

This is just an idea that I only implemented at a single example to clarify: How do @doubleopen-project/team think about turning URL comments into properties, so they become machine readable / actionable in rules?

Excellent idea IMO! Does ORT give errors for license classifications that are used but not defined?

I also think this is an excellent idea.

Does ORT give errors for license classifications that are used but not defined?

Yes, that's why currently the CI check fails. But this actually is a blocker here: We'd not want to define a category for each different URL... not sure how to solve that yet.

If the check for unknown categories was "relegated" to Evaluator rules from ORT's license classification parsing logic, that would allow only checking for categories that aren't prefixed by url, for example. If possible, a low-friction way could be that the evaluator rules had a default logic that checks for unknown categories as it is know, but it could be overriden in evaluator.rules.kts. Do you think it would be possible?

Do you think it would be possible?

It's surely technically possible, but I believe there would be push-back from members of the ORT community for such a breaking change. I'll clarify on that.


⚠ Bofrost.nl has degraded performance

In 839a555, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 814 ms

Resolved: Bofrost.nl performance has improved in 9dff5c0.


feat: ecAdd / ecMul

Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Resolves: #856
What is the new behavior?

Wrap ecAdd / ecMul
Does this introduce a breaking change?

[ ] Yes
[x] No

This change is

blocked by https://github.com/foundry-rs/starknet-foundry/issues/2397

Some note : if ec_add or ec_mul returns None, it shouldn't be interpreted as (0,0) which a valid way of encoding the point at inifinity. It should be a precompile error

Some note : if ec_add or ec_mul returns None, it shouldn't be interpreted as (0,0) which is a valid way of encoding the point at inifinity. It should be a precompile error
See https://www.evm.codes/precompiled  If the input is not valid, all gas provided is consumed and there is no return data, indicating a precompile contract error. Also, if not enough gas was given, there is no return data.
The Option::None case is reflecting the input is not valid .

Thank you for spotting this !
Will make sure to take this into account when working on it again.
This draft PR is not ready as it does not compile due to the blocking issue.

@obatirou https://github.com/foundry-rs/starknet-foundry/releases/tag/v0.30.0


[BUG] Unable to retrieve Private key for an Certificate with exportable keys

Describe the bug
Using the KeyVaultClient, we are unable to retrieve the private key from the Key Vault.
Exception or Stack Trace
2022-11-11 16:05:55,344  INFO | main | example.certificate.UserMgmtTest   | Key Properties exportable ?: null
2022-11-11 16:05:55,344  INFO | main | example.certificate.UserMgmtTest   | Private Key ?: false
2022-11-11 16:05:55,344 ERROR | main | com.azure.security.keyvault.keys.models.JsonWebKey      | java.security.NoSuchAlgorithmException: no such algorithm: RSA for provider AzureKeyVault
java.lang.IllegalStateException: java.security.NoSuchAlgorithmException: no such algorithm: RSA for provider AzureKeyVault
at com.azure.security.keyvault.keys.models.JsonWebKey.getRsaPublicKey(JsonWebKey.java:583)
at com.azure.security.keyvault.keys.models.JsonWebKey.toRsa(JsonWebKey.java:741)
To Reproduce
Create a self signed certificate in Azure Key Vault, with Advanced option to enable export of keys.
And try to download the private keys using the KeyVaultClient
Code Snippet
        KeyClient keyClient = new KeyClientBuilder()
                .credential(credential)
                .vaultUrl(kvURL)
                .buildClient();

        KeyVaultKey keyVaultKey = keyClient.getKey(keyId);
        log.info("Key Properties exportable ?: {}" , keyVaultKey.getProperties().isExportable());

        JsonWebKey webKey = keyVaultKey.getKey();
        log.info("Private Key ?: {}" , webKey.hasPrivateKey());

Expected behavior
Should be able to extract the private key in this case.
Screenshots
N/A
Setup (please complete the following information):

OS: [e.g. iOS]
IDE: [e.g. IntelliJ]
Library/Libraries: com.azure:azure-security-keyvault-keys:4.5.1
Java version:17
App Server/Environment: [e.g. Tomcat, WildFly, Azure Function, Apache Spark, Databricks, IDE plugin or anything special]
Frameworks: [e.g. Spring Boot, Micronaut, Quarkus, etc]

If you suspect a dependency version mismatch (e.g. you see NoClassDefFoundError, NoSuchMethodError or similar), please check out Troubleshoot dependency version conflict article first. If it doesn't provide solution for the problem, please provide:

verbose dependency tree (mvn dependency:tree -Dverbose)
exception message, full stack trace, and any available logs

Additional context
While creating the certificate, the Private key was marked as exportable

Access permissions are set as

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Bug Description Added
[ ] Repro Steps Added
[ ] Setup information Added

@vcolin7 could you please follow up with @vtapadia on this issue?  thanks

Last I checked you cannot export keys in Key Vault instances, only in Managed HSMs. Is this right @jlichwa?

@vtapadia you should use new SDK versus deprecated KeyClient SDK.
Either way, it is certificate, certificate can be retrieved as PFX/PEM with private key using GetSecret API/SDK. I don't see sample in java but in C#:
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/keyvault/samples/getcert/Program.cs#L71

@jlichwa He's using the new KeyClient (the deprecated one is called KeyVaultClient). @vtapadia one thing to note is that the code sample shared only works for getting a certificate's private key.
@jlichwa, do you know if a KeyVaultKey's private key (not part of a certificate) can even be exported from a Key Vault? I thought we only supported this for Managed HSMs.

While we wait for an answer from Jack, if I recall correctly, you will need to set a release policy for a key before being able to gain access to its private key, even if you set the exportable property to true. You can find more information on how the release operation works here and about the release policy itself here.
You can find how we do this for one of our tests here.
Adding @heaths to the conversation just in case, please feel free to correct me if I'm wrong about this.

@vcolin7 it is different key. You think about cryptographic key, certificates does not have release policy (based on screenshot it is certificate).

The private key material is never part of the certificate but is stored as a secret. Java might consider implementing a downloadCertificate methods on CertificateClient like I did in .NET; though, customers can do it now similar to: https://learn.microsoft.com/samples/azure/azure-sdk-for-net/get-certificate-private-key/

Thanks and sorry for the late response.
We are indeed looking for the Private key which was part of the key pair used to create the certificate. From the certificate, we get the key ID. Using the Key ID, I would expect that I could export the private key programatically for future usage if during creation, I marked the private key as exportable.
In general, we do not need to extract the private key and we don't either. But its a specific use case where the library depends on a file location for the keystore file. So we will really have to extract it.
I still feel that the exportable parameter value being null could indicate that this value is not set properly and might be the main reason for this not working as expected with the Java SDK but did not spend time to dive deep into the SDK.

@vtapadia that key id is only for public key. If you need private key you need to get entire certificate in pfx or pem which is available with getSecret API. If you don't see private key when you get secret (secret download) please let us know.
"Downloading as certificate means getting the public portion. If you want both the private key and public metadata then you can download it as secret."
https://learn.microsoft.com/en-us/azure/key-vault/certificates/how-to-export-certificate?tabs=azure-cli#export-stored-certificates

Wouldn't it be easy to support the way I have mentioned.
At least as it is mentioned as exportable, I would expect the easiest way to get the private key. Doing this as secret would need to again load it and parse it separately.

Also, is it possible to get some sample on how to read the key from the secret in Java. We tried few ways using BouncyCastle but those does not seam to work.

Hi @jlichwa
I would argue that even the usage for TLS is not complete. I would be divulging from the original question but might be good to mention here.
I could not find a way to upload a certificate with our own CA to be uploaded to the Certificate for Truststore (without Private Key) use case. It fails when we try to upload a certificate stating that we also need private key.
Also, when are using it as a keystore with our client certificate, it seems it is not sending the complete chain. We are investigating this issue as well.
Maybe you have solutions or hints for these as well. Otherwise I will raise another issue for those as well.

@vtapadia I asked to understand your use case. Today our use case is either download entire cert with private key or without for TLS offload - which does not scale well. This is to answer why our API is designed that way.
So getting just private key without certificate is likely that you are using for some specific scenario, which maybe something we did not account for.
Other than that, the solution stays, you will either do in code like in c# example (we don't have example for Java) I provided https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/keyvault/samples/getcert/Program.cs#L100
Third-party tools like openssl should also help.
For you TLS issue, yes we require to import certificate with private key. Separate issue/question has to be created.


Release dataset on HF (with gated access)

Hi @yixchen,
Niels here from the open-source team at Hugging Face. I discovered your work through ECCV, it was featured on daily papers here: https://huggingface.co/papers/2401.09340 (feel free to claim it with your HF account). I work together with AK on improving the visibility of researchers' work on the hub.
It'd be great to make the dataset available on the 🤗 hub, we can add tags so that people find them when filtering https://huggingface.co/datasets, so that people can do:
from datasets import load_dataset

dataset = load_dataset("your-hf-username-or-organization/sceneverse")

See here for a guide: https://huggingface.co/docs/datasets/loading.
There's then also the dataset viewer which allows people to see the first few rows in the browser: https://huggingface.co/docs/hub/en/datasets-viewer.
This would make the dataset easier accessible, and also discoverable. We can then also link the dataset to the paper page.
Gating mechanism
We support gating (similar to how this works for llama/mistral models), so that you can manually review who can access the dataset: https://huggingface.co/docs/hub/en/datasets-gated. See FineVideo (a video dataset created by HF) as an example: https://huggingface.co/datasets/HuggingFaceFV/finevideo.
Let us know if you need any help.
Cheers,
Niels
ML Engineer @ HF 🤗

Sure! Thanks for reaching out, we will look into it :-)

Hi, just curious, if we want to still hold all the data access license on HF Datasets (which is now provided in the google form), how should we properly do it when uploading to HF?

Hi, this is explained here: https://huggingface.co/docs/hub/en/datasets-gated.
Usually people start by uploading the data as a private dataset repository, then make it public when gated access is enabled


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3541

Questions ?
Please refer to the Backport tool documentation


Quill table

Is it available or not. If the table is available, please update the documentation. If not, then it would be better if we added a table like CKeditor, so the developer does not have to install extra plugins. These features would be good for all. I am facing the issue right now.

Check this example

@osx11 In React, it is not available as a prop.

Try it demo.

@attoae without definig button include in toolbar

Use this module quill-table-better.


Upgrade to Tox 4 and use editable installs

This makes tox significantly faster to run.

Depends on some other PRs to be merged first. Also, I want to ensure that our requirements are picked up for the type environment, once we get a Python package setup.

@naglepuff Let me know if this works for you.


Off by one bug in win32 display driver

The win32 simulation draws a single pixel white line on the right and bottom of the window. To fix, append a -1 to the following lines.
https://github.com/azure-rtos/guix/blob/890c37ab95df9a87f169ab157cf1ce5e40d4e68b/ports/win32/src/gx_win32_display_driver.c#L1202-L1203

Fixed for <IP_ADDRESS> release.


Releasing the GIL during Thrust sorts

As most thrust calls (including these sorts are blocking), we can release the GIL to allow concurrent CuPy threads to make progress. Fixes weak scaling for the three sorting functions mentioned in https://github.com/cupy/cupy/issues/7759.
Very small modification, and it passes sorting function tests on my local machine.

Thanks @dialecticDolt! This looks good to me. Let me have some time to try this patch on my local.
As a side note: Thrust functions will need to allocate a workspace from CuPy's memory allocator, and GIL will be acquired for a short period during that allocation.
https://github.com/cupy/cupy/blob/ae82162be14de46e69b3cd2ab3327fbc94d3eb65/cupy/cuda/cupy_thrust.cu#L398-L399

Thanks for the note.
Yeah, I have also noticed that the memory allocations do not release the GIL (even on different devices & streams)  either with or without the new async mempool. It's generally only noticeable before the mempool is large enough to avoid new allocations but it can add up.
I'm still not very familiar with the CuPy codebase, but would it be possible/safe in the default mempool to release during the cudaMalloc itself? Iirc the default mempool is separate per device (and per stream). Or is the locked time coming from something else?
In a similar test script to the one linked issue for cp.sort, for a cp.empty call I'm seeing scaling for a new allocation like the following on RTX Quadro 5000s (n=1000000000 int64):
Default Mempool:
1 GPU: 0.17 s
2 GPU: 0.34 s
4 GPU: 0.50 s
Async Mempool:
1 GPU: 0.05 s
2 GPU: 0.15 s
4 GPU: 0.29 s

hmm, nevermind, it seems like it should be. So I'm not sure what is going on in the above timings.
https://github.com/cupy/cupy/blob/d8e970ec0403376471ac0f71e006f9623569aa34/cupy_backends/cuda/api/runtime.pyx#L473

After looking into this further, it seems mostly driven by a system/driver lock on ioctl preventing mallocs and frees in the same process from executing in parallel even if they are on different devices/threads.

I think what you see is because of CuPy's mempool. Assuming you're still using the test script from https://github.com/cupy/cupy/issues/7759, you create temporary streams and discard them right after. There's nothing wrong in terms of correctness or semantics, but it does have a performance implication. Specifically, CuPy caches allocated memory based on stream IDs, but since the streams are discarded, the cached memory would not be reusable immediately, causing CuPy asking for more memory from the CUDA runtime.
Try allocating a pool of streams before entering the busy loop, and reusing the allocated streams, it might help.

You can also try to switch to the MemoryAsyncPool (backed by cudaMallocAsync) following the instruction:
https://docs.cupy.dev/en/stable/user_guide/memory.html#changing-memory-pool

re: the side discussion above - yep, using a smaller stream of pools (so memory is reused before being cleared) or setting async mempool helps avoid cudaFree / cudaAlloc being serialized within the same process at the driver / ioctl level
re: this PR specifically - Are there any other changes needed for the GIL release patch on the Thurst sorts?

Hi @dialecticDolt, apologies for keeping you waiting :bow:
Let me proceed to merge.

/test

/test mini


Add support to query liberty attributes through get_property

The request is to add support to query liberty attributes through tcl.
Example:
set liberty_lib_cell [get_lib_cells -of_objects $cells]
set area [get_property $liberty_lib_cell area]

Any update on this?


Silence unused_mut warning on non-unix platforms

Thanks. And I was thinking about how to make a variable mut conditionally and then gave up.... it's so simple!


Increased zhmcclient to 1.2.1 (scsi-dump fix)

No review needed.

Coverage remained the same at 32.582% when pulling adc33959c1f71a75d61234a8015705d7d7b6bfeb on andy/fix-secure-boot into be73d704a47e2b2fa259c747b8c4c6563bb4f43b on master.


Fix IxMonadCont.shift, add MonadCont instances, and bump version bounds.

The current type for IxMonadCont.shift makes it impossible to actually use the provided continuation, since the argument to 'shift' can't know anything about the type of 'i'.  On the other hand by moving the 'forall' one level deeper, the continuation can be used anywhere that doesn't need to change the index type.  The new type is equivalent to the type with no 'forall', except that the continuation can be used at multiple types.

Fixed build on GHC 7.8.3 -- it needs some help to instantiate a forall that GHC 7.10.2 can do automatically.

Thanks for the PR and sorry about the delay in reviewing.

I think the type makes it precise, since the 'b' is universally quantified -- it's impossible to make such an IxContT that does call its callback [with anything other than bottom].  As for equations, maybe:
callCC (\k -> f (k a >>>= g)) === callCC (\k -> f (k a))
(Which also implies that >>>=ing to undefined, const undefined, or any other argument, are all the same, by applying the law to both sides to erase both >>>=s.)


Install promptfoo on kubernetes

Run promptfoo on a container platform by using the following step that will build an image is not working .
While using import from git strategy it was able to build and push the image , however the pod that runs promptfoo is showing error in the log as shared below
Need to identify the change that are required so it can run on a kubernetes cluster

npm ERR! code EACCES
npm ERR! syscall open
npm ERR! path /opt/app-root/src/.npm/_cacache/tmp/de2575e0
npm ERR! errno -13
npm ERR!
npm ERR! Your cache folder contains root-owned files, due to a bug in
npm ERR! previous versions of npm which has since been addressed.
npm ERR!
npm ERR! To permanently fix this problem, please run:
npm ERR!   sudo chown -R<PHONE_NUMBER>:0 "/opt/app-root/src/.npm"

npm ERR! Log files were not written due to an error writing to the directory: /opt/app-root/src/.npm/_logs
npm ERR! You can rerun the command with `--loglevel=verbose` to see the logs in your terminal

Hey @rrbanda, thanks for the report. I think this has to do with how k8s handles permissions.  I have a potential fix in #1257, although I have not tried it on k8s myself.  If you can give it a try and let me know, that would be great.

@rrbanda We just cut a release with the new docker image. I also confirmed that I can run it ok k8s. Please check it out and let us know if you run into any other issues.

@mldangelo thank you , do you have any docs or link to image ? i can try deploying and share feedback

or i need to build and push image from promptfoo git repo on my end ?

@rrbanda you can check out the library and build from source. I'll work on getting an image published for you today.

@mldangelo just deployed with newly built image from source but still getting an error while the pod is getting created

@rrbanda what's in docker-entrypoint.sh?  Our dockerfile creates entrypoint.sh. Did you create your own?

@mldangelo i have not created one but is it generated as part of the build process ?

i tried a different way and still got an error

is there a helm chart for promptfoo ?

is there a helm chart for promptfoo ?

@tamdoEH authored this Helm chart for promptfoo. Note that I haven't tested it yet. If either of you (@didlawowo or @tamdoEH) would be open to contributing it directly into promptfoo — whether by working on a pull request, testing, or providing feedback - we’d greatly appreciate the help.

Hi @rrbanda, thanks again for your patience and for all the feedback along the way. Since we’ve made several updates to the Docker image and there’s been some progress on Kubernetes support, I’m going to close this issue.
If you’re still running into issues or need further assistance, feel free to reopen the issue or start a new one. Also, a community-contributed Helm chart can be found here: Gekko0114/kube-promptfoo-controller if that’s helpful for your setup.
We really appreciate your help in improving promptfoo!


Authentication examples

First of all, thank you, this is an extremely useful project!
I am wondering if there are any examples to demonstrate how to use the generated libraries. E.g. a small cli app that lists events from a Google calendar or something similar. 🙏

I see there are some CLI apps in the generated repository, but it's not obvious how to setup authentication/authorization.

Yeah authentication is a tricky part of the setup. I can see if I can add an example, but in the meantime, we use  version ^3of yup-oauth2. Most likely you want to use a service account file to authenticate.

All that being said, I just noticed you wanted a Google calendar app, which isn't a Google Cloud API so I am sure the auth API will be different. My bad! 🙃 You will probably want to do an oauth2 flow for authenticating https://developers.google.com/identity/protocols/oauth2

Hey William, thank you for the information. Will check it. Google Calendar was just an example, even though I think it is a Google API. In any case, will check yup-oauth2 as you suggested.

This example no longer compiles:
error[E0308]: mismatched types
   --> src/main.rs:11:9
    |
11  |         yup_oauth2::ServiceAccountAccess::new(credentials).build(),
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct `yup_oauth2::authenticator::Authenticator`, found opaque type
    |
   ::: /home/bcow/.cargo/registry/src/github.com-1ecc6299db9ec823/yup-oauth2-3.1.1/src/service_account.rs:234:27
    |
234 |     pub fn build(self) -> impl GetToken {
    |                           ------------- the found opaque type
    |
    = note:   expected struct `yup_oauth2::authenticator::Authenticator<_>`
            found opaque type `impl yup_oauth2::types::GetToken`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0308`.
error: could not compile `gcp-test`.

How would an API that only needs an API key work?

It would be very useful to have a reference implementation of one of the generated (e.g. Cloud) APIs to provide guidance to those of us noobs who struggle to use this repo.
I'm also affected by the auth example in this issue no longer compiling.
I tried to understand how drmesser's example here
It appears that yup_oauth2 no longer provides access_token but access and ... IIUC... this borks google_api_auth's GetAccessToken trait.


Implement dangling TODOs

Related to #42

Actually, this really is just a duplicate of #42 as a TODO message will make a credo scan fail. Closing


Sign in fail issue

Issue #, if available:
Fixes #589
Description of changes:
Allow users to sign in without getting credentials
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #655 into master will decrease coverage by 0.14%.
The diff coverage is 23.07%.

@@            Coverage Diff             @@
##           master     #655      +/-   ##
==========================================
- Coverage   86.93%   86.79%   -0.15%
==========================================
  Files          71       71
  Lines        3345     3354       +9
  Branches      634      634
==========================================
+ Hits         2908     2911       +3
- Misses        420      426       +6
  Partials       17       17

Impacted Files
Coverage Δ

packages/aws-amplify/src/Auth/Auth.ts
83.91% <23.07%> (-0.8%)
:arrow_down:

Continue to review full report at Codecov.

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


Add visual indicator that a screenshot has been taken.

I'd prefer that this worked seamlessly with whatever application is running. In such a way that the indication goes away after a certain amount of time.