Rename several DEM types and parameters

I've spent a few hours researching some DEM types and these changes are based on the things I've found.
I've named some of the types based off of names that were found in the executable

something seems to have gone awry here. 16 files changed?


🐞 Null returned after message edited

What happened?
Edited message

Edited Callback

Original Mesage

Reproduction Steps
-

Relevant log output
No response
Screenshots or Additional Context
No response
Application Version
v4.21.0
Verification

[X] I have checked for existing closed issues and discussions.

https://github.com/aldinokemal/go-whatsapp-web-multidevice/pull/221


Input Box align is not proper

Can't replicate this on Firefox 56 on Mac. Could you try disabling your add-ons and see if it's still a problem?


feature commands:show/list/register

/CC:@burtbiel, @tjprescott , @derekbekoe
FYI, resource providers typically use this mechanism to provide public/private preview for new features prior to making them generally available.
This is not that much used.

After the table heading change, it LGTM :shipit:


Support transfer data from public dataset in S3

User Scenario
There are a few open source dataset on https://registry.opendata.aws/. Customers want to transfer those data into their own AWS account for further study.
Or transfer those public dataset from standard regions to China regions/GovCloud.

Already implemented in v2.4.0.


Update ammonite to 3.0.0-M2-9-88291dd8

About this PR
📦 Updates com.lihaoyi:ammonite from 3.0.0-M2-8-ba4429a2 to 3.0.0-M2-9-88291dd8
Usage
✅ Please merge!
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 (3.0.0-M2-8-ba4429a2).
You might want to review and update them manually.
website/docs/reference/cli-options.md

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.lihaoyi", artifactId = "ammonite" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.lihaoyi", artifactId = "ammonite" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, old-version-remains, commit-count:1

We may need a workaround on our side for https://github.com/com-lihaoyi/Ammonite/issues/1504 before merging this (either that, or we need it fixed in Ammonite).


Unwanted Black Border

Before you start
Please take a look at the FAQ as well as the already opened issues! If nothing fits your problem, go ahead and fill out the following template:
Describe the bug
On Mac, a black border is added to a clean screen shot.
(Aside - thanks so much for doing this project... it's saving my bacon on DZone)
To Reproduce
Steps to reproduce the behavior:

Shift command 4 on browser
Squoosh
Download

Expected behavior
No border
Version:

OS w/ version: MacOS 12.3.1
Browser w/ version Edge 99.0.1150.55
Node version:v14.17.4
npm version: 6.14.14

Is your issue related to the quality of image compression?
Please attach original and output images (you can drag & drop to attach).

Original image

Output image from Squoosh

Additional context, screenshots, screencasts
Add any other context about the problem here.

This happens because your original PNG image has a border of transparent pixels, but JPG doesn't support transparency, so Squoosh simply transforms them into black pixels. If you choose any other compression format, you can preserve the transparent pixels. Alternatively, you can edit the original image to remove or fill the transparent areas.

https://github.com/GoogleChromeLabs/squoosh/issues/1233#issuecomment-1090493239 is correct. Use a different format if you want transparency.


ComplexMethod: exclude local functions

Expected Behavior
Complexity of f() below is 2.
Maybe behind a toggle? By default?
Observed Behavior
The function f appears to be too complex (6). Defined complexity threshold for methods is set to '3'
Steps to Reproduce
    @Test fun f() {
        assertThat(ComplexMethod(TestConfig(mapOf("threshold" to 3))).compileAndLint(
            """
                fun f(): String {
                    fun local1(flag: Boolean) = if (flag) "a" else "b"
                    fun local2(flag: Boolean) = if (flag) "c" else "d"
                    if (Math.random() < 0.5) {
                        return local1(true) + local2(false)
                    } else {
                        return local2(true)
                    }
                }
            """.trimIndent()
        )).isEmpty()
    }

Context
There are legitimate cases for defining local functions, this rule discourages any of those because it reports them as ComplexMethod, by default, this means that those local functions be it however irrelevant in the surrounding class, file, or package, are forced to be one level higher than necessary so they are littering global or local namespace.
@arturbosch put it very nicely:

Hm, one can argue that local functions help to declutter and structurize your code, removing nestings etc removes complexity. Like an algorithm which implementation is a whole function with many little helper local functions. I would't call this code complex but rather an architectural choice to build it this way :)? -- https://github.com/detekt/detekt/pull/1365#issuecomment-442138661

Here's an example, where I argue that verySimilarProblem and merge are two extracted local functions that make the algorithm below way more readable. I could lift them up to file level as private functions, but at that point I would actually open them up for usage from other functions in the file, even though they're strict implementation details of the collapseFile function. Regarding complexity, I don't think those local functions count towards the collapseFile complexity in understanding, because they're pure(ish, is new-ing an object pure?).
Your Environment

Version of detekt used: 1.21.0
Link to your project (if it's a public repository): https://github.com/TWiStErRob/net.twisterrob.gradle/blob/956016bc8bf5d282c6eb91b8ba64ff199d7a7f76/quality/src/main/kotlin/net/twisterrob/gradle/quality/report/html/ViolationsCollapser.kt#L21-L84

please change bug to feature, used wrong template.

Interesting, I thought we have such a flag.
I propose to count a local function as +1 with the new flag as one does count +1 for each declared function in the CyclomaticComplexity calculation.
One has to at least study the local function's signature which increases the complexity - so Imo +1 with the flag but don't visit the local function.

@TWiStErRob Are you implementing this rule or is this up for grabs, so to speak? I'm looking to get involved but haven't contributed yet.


Enhancement of the Navbar of the page

Improving the navbar involves making it more intuitive by streamlining navigation links, using clear labels, and ensuring it is mobile-friendly. Adding dropdowns for better organization and a search function can enhance usability. This helps users find what they need faster, improving overall site experience.

@snehaapratap this issue was mentioned early and they have created the pull request.
But for the encouragement, pls look for other issues in website like improvements.
I am closing this issue.


DocumentLoader with vision

We need a complete refactor of the Document Loader, when vision=true, documentLoader laways generate image of each page.

For now, it is only capable to do with pdf and image, the rest needs a dedicaded coverter


[Markup] Add common theme constants library (Uno.Toolkit.WinUI.Themes.Markup)

Much like Uno.Themes.WinUI.Markup project, we should have a base Markup library that contains helper classes/properties for the Toolkit Material/Cupertino/etc. styles that are following the shared semantic style naming pattern
Example
public static class Theme
{
	// ...
	public static class Styles
	{
		// ...
		public static class TabBar
		{
			public static Action<IDependencyPropertyBuilder<Style>> Top => StaticResource.Get<Style>("TopTabBarStyle");
			public static Action<IDependencyPropertyBuilder<Style>> Top => StaticResource.Get<Style>("BottomTabBarStyle");
		}
		public static class Chip
		{
			public static class Assist
			{
				public static Action<IDependencyPropertyBuilder<Style>> Default => StaticResource.Get<Style>("AssistChipStyle");
				public static Action<IDependencyPropertyBuilder<Style>> Elevated => StaticResource.Get<Style>("ElevatedAssistChipStyle");

			}
			public static class Input
			{
				public static Action<IDependencyPropertyBuilder<Style>> Default => StaticResource.Get<Style>("InputChipStyle");
				public static Action<IDependencyPropertyBuilder<Style>> Elevated => StaticResource.Get<Style>("ElevatedInputChipStyle");

			}
			public static class Filter
			{
				public static Action<IDependencyPropertyBuilder<Style>> Default => StaticResource.Get<Style>("FilterChipStyle");
				public static Action<IDependencyPropertyBuilder<Style>> Elevated => StaticResource.Get<Style>("ElevatedFilterChipStyle");

			}
			public static class Suggestion
			{
				public static Action<IDependencyPropertyBuilder<Style>> Default => StaticResource.Get<Style>("SuggestionChipStyle");
				public static Action<IDependencyPropertyBuilder<Style>> Elevated => StaticResource.Get<Style>("ElevatedSuggestionChipStyle");

			}
		}
		// ...
	}
	// ...
}

Question is, should Uno.Toolkit.WinUI.Markup have a dependency on this new Uno.Toolkit.WinUI.Themes.Markup package?
We don't have a specific Material or Cupertino Markup lib right now. The only thing that would be helpful with a Material/Cupertino specific Markup package is it could include the helper methods of initializing the Material/Cupertino resources through Markup extensions for App.xaml


Move marker to a new position

If I already have a mark on the map and that every 30 seconds it chooses its new position, how can I reuse the previous mark? to move the icon to its new position and not have to create a new brand and delete the previous one.

Have you solved the problem?


[api] Add random filename generation

close: #1

Codecov Report

Merging #6 (904f6a2) into master (3c4bed2) will increase coverage by 0.29%.
The diff coverage is 64.70%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   62.56%   62.85%   +0.29%
==========================================
  Files          18       18
  Lines         195      210      +15
  Branches       30       33       +3
==========================================
+ Hits          122      132      +10
- Misses         64       65       +1
- Partials        9       13       +4

Flag
Coverage Δ

api
58.97% <64.70%> (+1.83%)
:arrow_up:

front
65.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

api/src/main.rs
58.97% <64.70%> (+1.83%)
:arrow_up:


Android - Gradle - Build Tools - Gradle plugin 2.2.x to 2.3.x requires build tools 25.x

Hi!, sorry I could not find any other tickets related to the build tools version for Android. Feel free to close if this has been discussed somewhere else.
Basically Im trying to upgrade the version of Gradle (and it's plugin) by doing so I get constrained to use android build tools 25.x and react-native-fabric specifies build tools 23.x therefore android won't compile complaining that the build tools that react-native-fabric is using are 'too low'
Actual message:

Any suggestions on how to upgrade the build.gradle configuration of react-native-fabric in order to make it play nice with newer versions of gradle.
Thanks!

this is a problem with android, not this lib itself

I am experiencing the same issue when trying to use this package.
I will submit a PR for this and post a link here.

follow this https://github.com/react-native-community/react-native-camera/pull/1380

fixed on master

https://github.com/corymsmith/react-native-fabric/pull/177


Setting Environment variables for Debugger itself not just target

I have multiple toolchains for different ARM platforms and so on.
To start GDB I have to specify LD_LIBRARY_PATH to toolchain's lib's directory otherwise it will not even start.
Variables in "envirnoment" settings are passed to debugged application and are not set up before starting gdb, so I can't use it to specify toolchain's library path.
Right now I'm using wrapper (export LD_LIBRARY_PATH  and exec real gdb) that is called in miDebuggerPath instead of real debugger. It would be nice to do it without it.

Maybe this should be moved to MIEngine?

This is a feature request/question for an individual debugger extension.


64bit Windows always supports the W functions, so no need to check for them

64bit versions of Windows always support the W versions of functions, so no need to check for them at runtime.

Merged.


Eventhubs new EventProcessor

The new EventProcessor for preview 2.

Get all partition ids from eventhubs
Retrieve ownership list of partitions from data store via PartitionManager
Claim ownership of all partitions and add / update this ownership into data store via PartitionManager
Start all partition consumers on the claimed partitions. Starting checkpoint(offset) is retrieved from data store via PartitionManager
Stop the EventProcessor by sending cancellation tokens to all partition consumers
Sqlite3PartitionManager implements the PartitionManager with sqlite3, which is in Python stand library. It works both with a file and in memory

Can one of the admins verify this patch?


Reveal Logo to be removed from Future Roadmap

Logo has already been revealed so can be removed from the list

Thank you @shivaylamba :D
This is the new link: https://github.com/sansyrox/robyn/blob/main/integration_tests/base_routes.py
Can you please update it?

@shivaylamba The file is available at docs/roadmap.md in case you want to open a PR.

This was already fixed so closing this issue.


[SFTTrainer] Make sure to not conflict between transformers and TRL implementation

What does this PR do?
As per title, https://github.com/huggingface/transformers/pull/27141 is going to be merged in transformers, therefore this PR makes sure SFTTrainer's NEFTune does not conflict with transformers Trainer one
cc @lvwerra

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


About autoplay in iOS platform

Hi CookPete,
I have a problem about iOS9 with attribute 'autoplay' in html5 tag .
When I use <ReactPlayer url={path/to/mp3} fileConfig={{autoplay: true}} />, the autoplay attribute seems not add attribute 'autoplay' to .
And iOS has limitation to block autoplay to prevent wasted bandwidth from user's quota. But we want to make a repeat function with playlist in iOS. So I try to run these code after componentDidMount:
....
var audio = document.querySelector("audio");
audio.autoplay = true;
audio.load(); //iOS9 need do this first
audio.play(); //after iOS 5.x do this to trigger audio to play URL resource.
....
It can work on simple sample without ReactPlayer, just use audio tag and set resource and do that steps which I run after componentDidMount. But ReactPayer cannot, because it create two audio tag in the same time. First player is SoundCloud, second is FilePlayer. I cannot select correct audio tag, because document.querySelector always only return the first target it found(not array, I don't why).
So my solution in function componentDidMount is :
....
var audio = document.querySelector("audio");
audio.remove(); //I know this is point to  which use for SoundCloud Player
audio = document.querySelector("audio"); //re-select audio again, this is point to FilePlayer which I want.
audio.autoplay = true;
audio.load(); //iOS9 need do this first
audio.play(); //after iOS 5.x do this to trigger audio to play URL resource.
....
I know this solution is not good, so I hope you can provide a method or prop for user(developer) to decide which player they really want to use only, not all player which can current URL resource.
Here is committed log that you did in 15 Dec 2015.
You add a map to generate each player which can play url resource.
Maybe you have some consideration, I really hope you can help me to solve this problem.
This library is very useful for me, so I hope it can be better than past.
Thanks!
Regards
Kevin.Chien

Hi, @KevinChien did you tried the document.querySelectorAll() method? It should return more than the first match. (Not that this solve the problem you address, but this might already help :) )

Hi @denisflorkin,
document.querySelectorAll() is good for me, but I still need to do compare to confirm which player I want. I still hope @CookPete can help me to solve this issue. Thanks a lots!

Hi @KevinChien,
You can add any attribute you like to the correct audio tag by passing an attributes object into the fileConfig prop, like this:
<ReactPlayer
  url='file.mp3'
  playing
  fileConfig={{ attributes: { autoplay: true }}
/>

There is also refs.player exposed within ReactPlayer for the player that is currently active, so if you wanted to get the <audio> element specifically for FilePlayer, you could do something like:
componentDidMount () {
  // This assumes that `FilePlayer` is the currently active player
  const audio = this.refs.reactPlayer.refs.player
  audio.load()
  audio.play()
}
render () {
  return (
    <ReactPlayer
      ref='reactPlayer'
      url='file.mp3'
      playing
      fileConfig={{ attributes: { autoplay: true }}
    />
  )
}

That should do the trick for now. If the load() and play() hack fixes autoplay behaviour on iOS9 then perhaps it is worth sniffing the user agent and doing this by default in ReactPlayer on iOS devices.

I've just realised that fileConfig isn't actually passed down properly, so this solution won't work just yet. How embarrassing.. I'll push a fix shortly.

Also note that the autoPlay attribute must use a capital P, due to React's DOM differences.

Hi @CookPete,
I test the solution that you committed 4 days ago. It works.
And something I want to let you know as below:
componentDidMount () {
  // This assumes that `FilePlayer` is the currently active player
  const audio = this.refs.reactPlayer.refs.player
  /*
    If you set autoplay = true, you won't to run load() and play() of audio or video instance.
   And you set autoplay=false then run load() and play(), the src path will be undefined just like
   src="http://localhost/undefined"
   Maybe something not ready, I have no idea about this. Just remaining the operation flow if anyone is
   using this function. :)
  */
  audio.load()
  audio.play()
}
render () {
  return (
    <ReactPlayer
      ref='reactPlayer'
      url='file.mp3'
      playing    //It should be playing = {true | false} right?
      fileConfig={{ attributes: { autoPlay: true }}
    />
  )
}

The solution works both on android and iOS(at least 9.3.x)
But I found the volume cannot work as other platform, if change volume in browser of  iOS.(Such like safari or chrome, etc.)
All information I searched is "Volume is readonly in iOS"
See also here
BTW, anyone know the way if we hope to change the volume when we using the  in iOS with browser (not using UIWebView container.), if it is really a limitation of iOS, I think I will use the native API to do volume change controller.
At last, @CookPete thanks for you solution, it is useful for me!
Thanks.

BTW, anyone know the way if we hope to change the volume when we using the in iOS with browser

From the Safari dev docs:

On iOS devices, the audio level is always under the user’s physical control. The volume property is not settable in JavaScript. Reading the volume property always returns 1.

Also:

playing    //It should be playing = {true | false} right?

See boolean attributes. <Component test /> is the same as <Component test={true} />

@CookPete thanks again!

Solution at https://github.com/CookPete/react-player/issues/76#issuecomment-227371045 doesn't seem to be working for me. I keep getting audio.load() is not a function error. Has anyone able to get it to work with recent releases?

Hi @jainanshul,
Maybe you can try to print instance to make sure the instance is created.
I guess it is a generation process issue. Or you can post your key point code of your problem here.
Hope this helpful for you. :)
//Depend on sample code that @CookPete provide
componentDidMount () {
  // This assumes that `FilePlayer` is the currently active player
  const audio = this.refs.reactPlayer.refs.player
  //Of course, if you can use develop tool of chrome to set check point, you can watch the point via code executing.
  console.log("this.refs.reactPlayer", this.refs.reactPlayer);
  console.log("this.refs.reactPlayer.refs.player", this.refs.reactPlayer.refs.player);
  audio.load()
  audio.play()
}
render () {
  return (
    <ReactPlayer
      ref='reactPlayer'
      url='file.mp3'
      playing
      fileConfig={{ attributes: { autoPlay: true }}
    />
  )
}

I keep getting refs.player  is  unidefind ;
How to implement automatic playback audio；
//Depend on sample code that @CookPete provide componentDidMount () {   // This assumes that FilePlayer is the currently active player   const audio = this.refs.reactPlayer.refs.player   console.log("this.refs.reactPlayer", this.refs.reactPlayer); //unidefined   console.log("this.refs.reactPlayer.refs.player", this.refs.reactPlayer.refs.player);   audio.load()   audio.play() } render () {   return (     <ReactPlayer       ref='reactPlayer'       url='file.mp3'       playing       fileConfig={{ attributes: { autoPlay: true }}     />   ) }

@struggle-lulu Since version v0.10.0 the way to access the <audio> tag would be this.refs.reactPlayer.player.player. The breaking commit is https://github.com/CookPete/react-player/commit/6c16ba0f44d0403ae3e089ca1dafed0077b095e7
See an example jsFiddle using the new ref callback function. It is a bit weird but I'm looking at making this clearer in version 1.0.

I tried in iOS 10.3.3 safari it doesn't work.
https://jsfiddle.net/99ee3atp/19/
I think iOS 10 needs user gesture to autoplay anyway..or is there any solution about this?

I think iOS 10 needs user gesture to play audio element anyway..

This would not surprise me. This has always been an issue and iOS will continue to fix the hacks that people come up with to autoplay media on iOS devices.

@CookPete Thank you very much.
componentDidUpdate(){         console.log(this.rp.player.player);         const audio = this.rp.player.player;         audio.load();         audio.play();     }
I solved the problem through this


debug some error

Describe the bug
C:/Users/ijais/.gradle/caches/transforms-4/4f0d8418c72cae040d9e187f194de809/transformed/play-services-measurement-api-22.0.0-api.jar!/META-INF/java.com.google.android.gmscore.integ.client.measurement_api_measurement_api.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.9.0, expected version is 1.7.1.
Expected behavior
while building the app following error are occurring...while you can see in the image it is implemented correctly
Screenshots
Yes

This can be resolved by changing     'org.jetbrains.kotlin.android' version '1.7.10'  to   'org.jetbrains.kotlin.android' version '1.9.0'

Can you assign it to me ?

@Vishwajith-Shettigar assigned


Activity class does not exist.

Installing APK 'react-native-vector-icons-debug-androidTest.apk' on 'Pixel_2_XL_API_28(AVD) - 9' for react-native-vector-icons:debugAndroidTest
Installed on 1 device.
BUILD SUCCESSFUL
Total time: 29.091 secs
Running adb -s emulator-5554 reverse tcp:8081 tcp:8081
Starting the app on emulator-5554 (adb -s emulator-5554 shell am start -n com.rnmapboxardemo/com.rnmapboxardemo.MainActivity)...
Starting: Intent { cmp=com.rnmapboxardemo/.MainActivity }
Error type 3
Error: Activity class {com.rnmapboxardemo/com.rnmapboxardemo.MainActivity} does not exist.

Use this variant:
react-native run-android --variant=gvrDebug

Use this variant:
react-native run-android --variant=gvrDebug

App is stock in loading from localhost:8081

Solved this by doing the following:

remove node_modules
use yarn from root and inside the sample app RNMapboxARDemo
react-native run-android --variant=gvrDebug


Keycode always null onKeyPress event

http://screencast.com/t/y3uxNO5R

You are probably trying to use event object outside of the event handler.


Add favicon and logo to rtd

The files are scaled from the 700x700 png - 175 is 700/4 and is quite suitable resolution from my testing.

@martinhoyer the image file is sourced from another project, see https://github.com/teemtee/tmt/blob/main/docs/Makefile#L28. So I suppose both new files should land there, and then be fetched when building docs so we can keep images together.

@martinhoyer the image file is sourced from another project, see https://github.com/teemtee/tmt/blob/main/docs/Makefile#L28. So I suppose both new files should land there, and then be fetched when building docs so we can keep images together.

I'm counting with it. Just thinking that we can see what works here and then put the final files there.

Switching to remote URL deserves a cleanup of LOGO-related bits in https://github.com/teemtee/tmt/blob/main/docs/Makefile#L30, plus no file to ignore anymore, https://github.com/teemtee/tmt/blob/main/.gitignore#L17

Switching to remote URL deserves a cleanup of LOGO-related bits in https://github.com/teemtee/tmt/blob/main/docs/Makefile#L30, plus no file to ignore anymore, https://github.com/teemtee/tmt/blob/main/.gitignore#L17

Yeah, I've just noticed that things have changed in docs. I'm unable to render them with a different theme as well. Need to catch up. /me missing mkdocs

Switching to remote URL deserves a cleanup of LOGO-related bits in https://github.com/teemtee/tmt/blob/main/docs/Makefile#L30, plus no file to ignore anymore, https://github.com/teemtee/tmt/blob/main/.gitignore#L17

Yeah, I've just noticed that things have changed in docs. I'm unable to render them with a different theme as well. Need to catch up. /me missing mkdocs

TMT_DOCS_THEME (https://tmt.readthedocs.io/en/stable/contribute.html#docs) does not work?

TMT_DOCS_THEME (https://tmt.readthedocs.io/en/stable/contribute.html#docs) does not work?

Not sure about this, but /latest is not using renku and my furo PoC is broken after rebase. Will investigate.

TMT_DOCS_THEME (https://tmt.readthedocs.io/en/stable/contribute.html#docs) does not work?

Not sure about this, but /latest is not using renku and my furo PoC is broken after rebase. Will investigate.

Test failures unrelated to the PR (centos repo availability)

Documentation-only, merging.


[Bug]: arm_neon.h inclusion causing issues

Describe the issue
Builds using nvcc driving either clang or gcc have issues with the arm_neon.h inclusion. This seems to underpin issues like this: https://github.com/tensorflow/tensorflow/issues/62490
Steps to reproduce the problem
sample code:
#include "absl/container/flat_hash_map.h"

int main() {
  return 0;
}

compiled like so against a recently git clone'd version of absil-cpp into /tmp:
nvcc -gencode=arch=compute_70,\"code=sm_70\" -I /tmp/abseil-cpp/ /tmp/absl_test.cu

yields errors like:
/dev/shm/aweits/spack/opt/spack/linux-rhel9-neoverse_v2/gcc-12.2.0/gcc-12.3.0-myhz7oqexkiskxlu4mgms4zljmzpum6a/lib/gcc/aarch64-unknown-linux-gnu/12.3.0/include/arm_neon.h(40): error: identifier "__Int8x8_t" is undefined
  typedef __Int8x8_t int8x8_t;

nvcc -allow-unsupported-compiler -ccbin=`which clang++` -gencode=arch=compute_70,\"code=sm_70\" -I /tmp/abseil-cpp/ /tmp/absl_test.cu

yields errors like:
/dev/shm/aweits/spack/opt/spack/linux-rhel9-neoverse_v2/gcc-12.3.0/llvm-18.1.1-gzg6frbu43t7iizefsjmlnxciy5gvxyy/lib/clang/18/include/arm_neon.h(66280): error: identifier "__builtin_neon_vrnd32xq_f64" is undefined
    __ret = (float64x2_t) __builtin_neon_vrnd32xq_f64((int8x16_t)__p0, 42);

clang++ --cuda-gpu-arch=sm_80 -I /tmp/abseil-cpp/ /tmp/absl_test.cu

works just fine.
changing the header like so avoids all errors:
diff --git a/absl/base/config.h b/absl/base/config.h
index 5fa9f0ef..503a77af 100644
--- a/absl/base/config.h
+++ b/absl/base/config.h
@@ -962,7 +962,7 @@ static_assert(ABSL_INTERNAL_INLINE_NAMESPACE_STR[0] != 'h' ||
 // https://llvm.org/docs/CompileCudaWithLLVM.html#detecting-clang-vs-nvcc-from-code
 #ifdef ABSL_INTERNAL_HAVE_ARM_NEON
 #error ABSL_INTERNAL_HAVE_ARM_NEON cannot be directly set
-#elif defined(__ARM_NEON) && !defined(__CUDA_ARCH__)
+#elif defined(__ARM_NEON) && !defined(__NVCC__)
 #define ABSL_INTERNAL_HAVE_ARM_NEON 1
 #endif

What version of Abseil are you using?
0941ce7fd95104ba5f4678c41802a3229db1d5eb
What operating system and version are you using?
RHEL9 / upstream kernel
Linux gh-01.rc.rit.edu 6.1.85-1.el9.aarch64 #1 SMP PREEMPT_DYNAMIC Wed Apr 10 15:38:29 EDT 2024 aarch64 aarch64 aarch64 GNU/Linux
What compiler and version are you using?
gcc 12.3.0 / clang 18.1.1 / cuda 12.4.0
What build system are you using?
none
Additional context
No response

As a clarification, the primary issue here is that when nvcc calls the c++ preprocessor arm_neon.h gets included and that brings along definitions of types that only the respective c++ compiler will understand. nvcc then calls cudafe++ on the resultant file and errors out. This is confusing and the output might lead folks to misunderstand where the error is originating from. Running nvcc with the --dryrun flag will show all the individual steps being run in the background, and incrementally executing these steps allows one to see what's going on.

The above is probably overly broad (!defined(__NVCC__)), and should be scoped to this (!defined(__CUDACC__)) per https://docs.nvidia.com/cuda/cuda-compiler-driver-nvcc/index.html
diff --git a/absl/base/config.h b/absl/base/config.h
index 5fa9f0ef..503a77af 100644
--- a/absl/base/config.h
+++ b/absl/base/config.h
@@ -962,7 +962,7 @@ static_assert(ABSL_INTERNAL_INLINE_NAMESPACE_STR[0] != 'h' ||
 // https://llvm.org/docs/CompileCudaWithLLVM.html#detecting-clang-vs-nvcc-from-code
 #ifdef ABSL_INTERNAL_HAVE_ARM_NEON
 #error ABSL_INTERNAL_HAVE_ARM_NEON cannot be directly set
-#elif defined(__ARM_NEON) && !defined(__CUDA_ARCH__)
+#elif defined(__ARM_NEON) && !defined(__CUDACC__)
 #define ABSL_INTERNAL_HAVE_ARM_NEON 1
 #endif

Are there any updates on this issue? This bug is preventing us from building TensorFlow and JAX on aarch64.


Documentation around static preferences

Update README with static preferences, also see https://github.com/solidusio/solidus_stripe#usage
est: 0.5h

This has been fixed in #56 and merged in master.


Revert "Ratings"

Reverts hbharmal43/Supernotes#2

Reverting ratings to see if SuperNotes working or not


Passing Target ID from ECS to ALB Module

Hi support ,
I would like to provide the input for the target_id in the ALB (Application Load Balancer) module from the ECS (Elastic Container Service) module. Could you please let me know which value I should retrieve from the ECS module and how to retrieve it?
module "ecs" {
  source  = "terraform-aws-modules/ecs/aws"
  version = "5.0.1"
  cluster_name = "myecscluster"
   fargate_capacity_providers = {
    FARGATE = {
      default_capacity_provider_strategy = {
        weight = 50
      }
    }
    FARGATE_SPOT = {
      default_capacity_provider_strategy = {
        weight = 50
      }
    }
  }
   container_definitions = {
    myfront_end = {
      cpu       = 256
      memory    = 512
      essential = true
      image     = "650623132949.dkr.ecr.ap-south-1.amazonaws.com/myapp:latest"
      port_mappings = [
        {
          name          = "myapp"
          containerPort = 8080
          hostPort      = 8080
          protocol      = "tcp"
        }
      ]
      # Example image used requires access to write to root filesystem
      readonly_root_filesystem = false
      memory_reservation = 100
    }
  }
    service_connect_configuration = {
    namespace = "myapp"
    service = {
      client_alias = {
        port     = 8080
        dns_name = "testcase.lmapps.io"
      }
    }
  }
}

##ALB module
module "alb" {
  source  = "terraform-aws-modules/alb/aws"
  version = "8.0"
  name = "my-alb"
  load_balancer_type = "application"
  vpc_id             = "vpc-075fcd4310a0f65c0"
  subnets            = ["subnet-0f06b2cbbb7b9b4bf", "subnet-070fdb57f5e2fef8b"]
  security_groups    = [module.sgroupalb.security_group_id]
  target_groups = [
    {
      name_prefix      = "pref-"
      backend_protocol = "HTTP"
      backend_port     = 80
      target_type      = "ip"
      targets = {
        my_target = {
          target_id = "???????????"
          port = 80
        }
      }
    }
  ]
  https_listeners = [
    {
      port               = 443
      protocol           = "HTTPS"
      certificate_arn    = "arn:aws:iam::123456789012:server-certificate/test_cert-123456789012"
      target_group_index = 0
    }
  ]
  http_tcp_listeners = [
    {
      port               = 80
      protocol           = "HTTP"
      target_group_index = 0
    }
  ]
  tags = {
    Environment = "Test"
  }
}

see the examples we provided https://github.com/terraform-aws-modules/terraform-aws-ecs/blob/dee59b733b805f9c16495bf65cc193260f537e47/examples/complete/main.tf#L115-L121
and there is also ECS Blueprints which shows patterns that utilize this module


HotChocolate.Templates.Server does not compile

Describe the bug
The HotChocolate.Templates.Server template produces a project that does not compile because Startup.cs is missing a using statement.
To Reproduce
Steps to reproduce the behavior:

Make sure you have installed the latest template via dotnet new -i HotChocolate.Templates.Server, uninstalling first if necessary.
Follow the steps described at https://hotchocolate.io/docs/dotnet-cli to create a new project.  An example follows:

C:\Dev>mkdir graphql-demo

C:\Dev>cd graphql-demo

C:\Dev\graphql-demo>dotnet new graphql
The template "HotChocolate GraphQL Server" was created successfully.

C:\Dev\graphql-demo>dotnet restore
  Restore completed in 527.57 ms for C:\Dev\graphql-demo\graphql-demo.csproj.

C:\Dev\graphql-demo>dotnet run
Startup.cs(22,22): error CS1061: 'IServiceCollection' does not contain a definition for 'AddDataLoaderRegistry' and no accessible extension method 'AddDataLoaderRegistry' accepting a first argument of type 'IServiceCollection' could be found (are you missing a using directive or an assembly reference?) [C:\Dev\graphql-demo\graphql-demo.csproj]
Startup.cs(25,33): error CS0103: The name 'SchemaBuilder' does not exist in the current context [C:\Dev\graphql-demo\graphql-demo.csproj]
Startup.cs(25,22): error CS1061: 'IServiceCollection' does not contain a definition for 'AddGraphQL' and no accessible extension method 'AddGraphQL' accepting a first argument of type 'IServiceCollection' could be found (are you missing a using directive or an assembly reference?) [C:\Dev\graphql-demo\graphql-demo.csproj]

The build failed. Fix the build errors and run again.

C:\Dev\graphql-demo>

Expected behavior
The dotnet run command compiles without errors.
Desktop (please complete the following information):

OS: Windows
Version 10 Professional

Additional context
Adding a using HotChocolate; statement to Startup.cs fixes the compilation issue.  It would also be helpful if the docs at https://hotchocolate.io/docs/dotnet-cli told you to visit https://localhost:5001/playground/.  I had to search around for a while before I figured that out.

Thanks for reporting this.

I can confirm same problem on Mac OSX, the merged PR does fix it. Thanks!

This one is fixed on 11 but not on 10. @sporeservant do you still want to fix that? The branch for version 10 is version_10_0_0_master. We want to close version 10.5 next week.

This one is fixed ....


forward_pass/feature request

Very simply, it would be nice if there were a feature similar to forward_pass, but would forward the password unmodified.  Example use case:
auth    [success=2 default=ignore] pam_unix.so forward_pass
auth    [success=1 default=ignore] pam_krb5.so try_first_pass
This allows unix or Kerberos login.  Same config with pam_google_authenticator.so will not work, perhaps because it is expecting to strip a pin.   (Eventually works, but first password prompt will always fail if you are trying krb5)
Thank you your work on this project.

What's the scenario you want to support, exactly? Order of checks, etc? E.g. I don't expect someone who doesn't have an OTP to have a password matching the pattern of password+otp, right?

Why would that be desirable? It seems like it would be a pretty serious detriment to the security of the system, since instead of ending up with two-factor auth for the system you'd end up with multiple parallel single-factor auth options for an attacker to pick between.

Indeed, without fail2ban or some other brute force protection, that could be worse.  That said, my use case is not necessarily for system login nor my choice.  Thank you for your consideration.

I guess I can theoretically see use cases. Maybe some people have the password, some have to use OTP, and it's something like a VPN software client that can't do custom prompts in keyboardinteractive.
Still, in those cases I'd probably want a list of users who should go one or the other way. And then just don't have GA config for the users who'll use password.
But if not that, can you not place GA last and use use_first_pass?

Please forgive my tardy response.
A good point but, alas, I can do that.  Consider: If a user is told to use pin, but that username also happens to exist in krb5, the krb5 server will see repeated failed logins and decide to lock the account!  Hence, I am in a pickle as I require pin to be second.  Thank you again for the consideration.

And what about pam_listfile.so? Do you have users that sometimes will use a PIN, sometimes a password? If not then pam_listfile.so. Or not have a config, like I said.
If these don't work: I may accept well-written pull requests for this feature, but don't expect it to be made on its own.


bug fix from 20220624

file platform logo
blurb for origin and location not inside!

filelogo.png

displayName: File-based
name: file
logoUrl: /assets/platforms/filelogo

for ingestapi, names with a / will end up creating a folder when ingest-api persist the request to json. (which fails)

force ingest-api to write a new log file everyday. currently it doesnt, on days it doesnt have traffic

metadata-service/factories/src/main/java/com/linkedin/gms/factory/telemetry/DailyReport.java can't disable but can extend the duration between reporting

modify gms' logback.xml to incorporate:

for org.elasticsearch.client, org.elasticsearch.client.RestClient, org.elasticsearch.client.sniffer
https://stackoverflow.com/questions/51865873/disable-trace-logging-for-tracer-package-for-elasticsearch-java-rest-client

logging for 30 days, needs to be put in

put a note in editing schema for users to add a " " in native type to increment the version change if they only change the field type.

ingest-api has issue if name has / as it cannot save the mcps to file

create_dataset needto check if there is a proposed container. if no container, do not create aspect else GMS will return garbage.
edit-parent container ui


Fix setbufline()

Problem
When applying setbufline() with lines list to the buffer which sets foldmethod=expr, Vim sets lines into unintented buffer.
Repro steps (by @thinca)
reproduce.vim:
setlocal foldmethod=expr foldexpr=0
new
call setbufline(1, 1, [1, 2, 3])

$ vim --clean -S reproduce.vim
expected: lines [1, 2, 3] is into bufnr=1 buffer
actual: only 1 line [1] is into bufnr=1 buffer, lines [2, 3] is into bufnr=2 buffer (current buffer)
Solution

Change/restore curwin

Remaining problem: when setbufline() to hidden buffer, curwin keeps actual current window, this may cause unexpected buffer-state.

Codecov Report

Merging #2293 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2293      +/-   ##
==========================================
+ Coverage   74.42%   74.42%   +<.01%
==========================================
  Files          90       90
  Lines      132130   132139       +9
  Branches    30730    29017    -1713
==========================================
+ Hits        98341    98349       +8
+ Misses      33764    33761       -3
- Partials       25       29       +4

Impacted Files
Coverage Δ

src/evalfunc.c
83.44% <100%> (+0.03%)
:arrow_up:

src/if_xcmdsrv.c
83.99% <0%> (-0.18%)
:arrow_down:

src/if_py_both.h
76.09% <0%> (-0.15%)
:arrow_down:

src/window.c
80.79% <0%> (-0.07%)
:arrow_down:

src/eval.c
80.77% <0%> (-0.05%)
:arrow_down:

src/channel.c
82.44% <0%> (ø)
:arrow_up:

src/gui_gtk_x11.c
47.65% <0%> (ø)
:arrow_up:

src/gui.c
47.16% <0%> (ø)
:arrow_up:

src/ex_cmds.c
78% <0%> (+0.22%)
:arrow_up:

Continue to review full report at Codecov.

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


Multiple installations (with different/same versions) are not supported for deep-packages

If a package is already installed that is in dependencies, I get this error
"message": "Multiple packages with name '@l4legenda/capacitor-contact-name-payload' exist."

At the moment, the code does not support installation in this case. We are working on solution to this problem. It requires to update deep.id first, it should support loading of active version or specific version.
Anyway, it is a good idea to update error message, for example like this:
"message": "Other version of '@l4legenda/capacitor-contact-name-payload' package is already installed."

How can we support multiple installations? Where is the problem with it? Only with deep.id? Can we add optional parameter "version" to it?

@FreePhoenix888 https://github.com/deep-foundation/deeplinks/issues/259 - not only await deep.id, also await deep.name and so on, we have to careful adapt the system.


Add GitHub Action to automatically publish docs

This commit adds a GitHub action to automatically publish doc
changes to the gh-pages branch, which will be rendered nicely
on https://goole.google.io/iree. This includes both docs directly
checked into the docs/ directory and docs auto-generated from
TableGen definitions. For now the former only contains IREE
Design Roadmap; the hope is to refresh existing docs and publish
them one by one.

Would be nice to put the sh + py files in the same place build_tools/docs/ seems good (mirroring the docs/ root name)
(I don't like the top level scripts directory - should have been under build_tools)

Not all scripts are build tools :-P

Would be nice to put the sh + py files in the same place build_tools/docs/ seems good (mirroring the docs/ root name)
(I don't like the top level scripts directory - should have been under build_tools)

Not all scripts are build tools :-P

This seems involve a larger discussion over file organization outside of this PR then? Should we have the discussion separately?


Camera screen turns black while opening camera and go to another activity and came back to camera activity in Redmi Note 8

Please follow the following instructions before filing a bug:

Github issues under this project are only for sample-related issues. If you have modified the sample code in any way, then this is probably not the appropriate place to report a bug.
If you are trying to report a bug related to the CameraX library then you can do so at the Android Issue Tracker: https://issuetracker.google.com/components/618491.
Include the specific sample where you found the issue, in addition to device model and specific Android build number -- which can be found under Settings > About phone.
Include error logs from logcat, if there are any.
If you can, also attach a bug report-- but please keep in mind that personal information can sometimes be found in those, so double check before attaching.

I am only facing this issue in Redmi Note 8, in another devices it's working fine. I am also not finding any logs as this device is not with me.

Thank you for the reporting the issue! Which sample you were using (or all samples have the same behavior)? Please also add the steps to replicate the issue.

This looks important topic, could you forward this one to also to g.co/camerax/developers (please join that group, need your cameraX experience besides this issue), thank you. It is so device dependent, I will go find a device in the mean time.

I am not able to access mentioned page @ggfan

@Shivani3192 the link is https://g.co/camerax/developers, the actual location is at: https://groups.google.com/a/android.com/g/camerax-developers, may you try it again?  Once you are there, clicking on the "Ask to join group" should do it. After getting in, not only share your thoughts, please also help answer some (or all) questions :-).  thank you so much!
The previous link I put there was incomplete(missed the "https://" part, then github prepends its own header), sorry.

@ggfan Thanks for the help. I have created new conversation in mentioned group please help me to resolve this issue

great, use that group fully: maybe beyond its limit to get best quality for your app and the lib :-). For I close this from here for now, if needed, I will pick it from that forum.


Add chinese remainder theorem

A solver for the chinese remainder theorem.
Created with Lazar

Closes #757

Add your algorithm to the README file, please

Add your algorithm to the README file, please

Done! Thanks for the feedback.

LGTM, just update README in alphabetical order.

LGTM, just update README in alphabetical order.

Fixed it


Crash on new update

Crashes on clicking 'Activate' button in beacon menu
MissingFieldException: Field 'Pickupable.OnTake' not found.

UserMenu+<>c__DisplayClass15_0.<AddButton>b__0 () (at <2880ca5af9af4f30949bc3be2db675f2>:0)
KButton.SignalClick (KKeyCode btn) (at <45caf0b640914e4ea2d2d9b064b504df>:0)
KButton.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) (at <45caf0b640914e4ea2d2d9b064b504df>:0)
UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) (at <e9635660f69b4d9d84c085aeda3be353>:0)
UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction`1[T1] functor) (at <e9635660f69b4d9d84c085aeda3be353>:0)
UnityEngine.EventSystems.EventSystem:Update()

Build: U50-581979-S

Please upgrate this mod by Mod Updater, the new version is 1.1.3.

MissingFieldException: Field 'Pickupable.OnTake' not found.

UserMenu+<>c__DisplayClass15_0.<AddButton>b__0 () (at <2880ca5af9af4f30949bc3be2db675f2>:0)
KButton.SignalClick (KKeyCode btn) (at <45caf0b640914e4ea2d2d9b064b504df>:0)
KButton.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) (at <45caf0b640914e4ea2d2d9b064b504df>:0)
UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) (at <e9635660f69b4d9d84c085aeda3be353>:0)
UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction`1[T1] functor) (at <e9635660f69b4d9d84c085aeda3be353>:0)
UnityEngine.EventSystems.EventSystem:Update()

Build: U50-581979-S

Same with the latest version.

Are you kidding me? It is ont the latest version.

It is recommended to use Mod Updater for updates.


4D (time evolving) depth S-grids

In this PR, we provide a Field.from_netcdf that allows to read vertical grids evolving in time. The depth dimension must be Field.depthfield where 'depthfieldis the name of the field on which the data of the depth are provided, such thatU.grid.depth= depthfield.data`

Some small issue:
Since we work with dimension['depth'] = 'not_yet_set', we can't compare dimensions anymore to check if fields are on the same grid. So the quick check performed in the fieldset does not work anymore and dimensions need to be loaded for every field with not_yet_set flag (this wasn't occurring with dimension Field.XXX... So we have some cost here, but we  gained a lot in clarity)

I seem not authorised to merge the master into the origin/branch but locally I have merged since the update for the memory leak.  There is still the same memory problem as discussed in #703 #711 and #668 and even with the master as it was this morning, I still seem to have it (in 1h I ran out of my RAM). It seemed solved and tested for other users so I was wondering @erikvansebille or @CKehl, could you merge the master in this branch as there are no conflicts (yet) and then if we're all on the same page we could hopefully fix where the memory issue still seems to persist.

Hi @cjongedijk. I have just merged master into this PR. To be honest, I'm not really sure what the status of this PR is; it was mostly a discussion between you and @delandmeterp, right?
We have done (and are still doing) quite some optimisations of memory usage, so it may be that this updated version fixes your RAM issues. See also http://oceanparcels.org/faq.html#performance
Let us know how it goes!

Ho - so, I also encounter still some problems with long-running simulations, but not due to the memory but because there are file access conflicts, which I intent to resolve soon. Can you confirm (via the procedure in http://oceanparcels.org/faq.html#performance) that your error actually comes from memory overflow ?

Also: I am right now encountering some issues with 4D data myself when compiling the MPI documentation notebook for 4D simulations, so we're working very much on the same issue, I believe.

@erikvansebille thanks for the merge. @CKehl are you using this branch to simulate with 4D data? I haven't ran into any problems yet but @delandmeterp and me worked on it indeed but nothing has changd since I have not had any issues and could not really do anything or check any performance differences to the 3D fields with memory running out so fast on my longer sims. I should have flagged earlier that I cannot merge myself but I have locally been merging all the time and since I was understanding being the only user of this branch didnt bother pushing back to the repository. I use 4D fields for all of my simulations, however only the ones where the fields are large I run into memory problems, not for ones that have small fields and run for a longer time. I have monitored my memory over two simulations I did this week, one that has run (and is still running) with small fields and one that ran out of memory in one hour with large fields (similar pset sizes). I will post here in a minute how it looks and we can continue from there.

so this is how it looks like for me. I randomly exported the memory during the last few days sorry for the intermittent signal compared to the graphs @CKehl showed in the other threads , working on an automated memory tracker for python processes only but haven't done that yet. approximately 2 days in my slowly aggregating first (light) simulation, I had started one that gives me lots of troubles. I swapped after about 1 hour and then I killed the simulation. I had the long running one on purpose in the background because I saw there was a memory drop earlier whilst simultaneously running sims. This seems to indeed the case around 48 h into the sim where I kill the trouble simulation and the other one continues on a lower memory load.

with 'one field size' I mean the size of the netcdf file containing all variables/field information for one time step.

updated: after the weekend my 'light' simulation linearly increased the memory uptake as expected, and with netdata I tracked yesterday where my local system was only doing the parcels run with similar memory behaviour as in #703  https://github.com/OceanParcels/parcels/issues/668#issuecomment-576667920 tomorrow morning I'll have to kill it to free up space for other projects.
Fig 1:

Fig 2: netdata output of my total RAM used for 24h.

Also, following the conversation in #740 today is that testing for _fromnemo specifically or also generally for _fromNetCDF?  It seems like that data format of NEMO (sigma grids 3d+time, C grid) generally is very similar to mine.

Hi Cleo - sorry to come back to you that late. Indeed this memory issue was keeping me up now the last 2 weeks, cause when I fixed it initially, I ran all tests just with 2D+time data. In 3D+time, I ran into massive issues because I wasn't converting the field_chunksize things properly for those cases - they are more 'involved'. Also, there was a separate issue popping up very often when running Erik's Galapagos case - conflicting file access, because with the lazy loading now actually implemented properly, the locking mechanism needs to be managed. As you already saw in #740 , those issue should be resolved now. I tested #740 by running the full galapagos case (the backward simulation) as it is in the repository (https://github.com/OceanParcels/GalapagosBasinPlastic) successfully on the cluster in a job system without crashes, using auto-chunking. Thus, if we can rebase this branch (update it to the current master), then it would be worth testing again. I am also running the python notebook (https://nbviewer.jupyter.org/github/OceanParcels/parcels/blob/master/parcels/examples/documentation_MPI.ipynb) now on a cluster with the 3D data, regenerating the images. It would be very kind if you were running you simulation with the updates again. Conversly, I'll now browse through your changes in this branch, make a review, and get some inspiration from your changes were I may still need to adapt things.

BTW: if you wanna see how to define field_chunksize for 3D fields, have a look on #731 , the file parcels/examples/documentation_MPI.ipynb. There, in the second part of the file, you have the function set_cmems_fieldset_3D(cs), where cs=field_chunksize. That is for how to do that for general Field[set].from_netcdf() access - for NEMO and the others, that step is done automatically (though only tested in detail with NEMO, via the Galapagos case).


Make it so when you die the script automatticly restarts itself

in the tittle

@MineMooreALT why do u need that?

@MineMooreALT @l3mpik good idea
@mayed505 cuz it's annoying

@Lok0n wtf am i the only one that my bots rejoin the new server follows me?

@Lok0n @MineMooreALT i can add it if u guys want it :3

yes. PR please -

pullrequest ? @mayed505

@Lok0n i changed my mind i wont do it since people would sell it...

thanks for your useless help @mayed505

bruh dude i hate having to refresh ever time i wanna get bots again

same i hope @l3mpik add this new feature soon

@Lok0n l3mpik already have add this feature.... Every time you change server the bots disconect from old server  and connect to new one


Adjust DKG frequency

Issue #1545 specifies that the DKG frequency should be of 1/20 ~ 1/30. We will have this situation with DKG contribution margin set to 5% only when the actual gas is the same as gas price ceiling used for DKG contribution fee estimation.
With gas price set to 1Gwei (same as Ropsten) we have DKG starting:

10 entries after genesis
5 entries after the previous DKG
1 entry after the previous DKG

See #1653

5% gives 1/20 behavior in the worst-case scenario where all submitters use the maximum gas. If submitters use less gas we generate more groups and become time-constrained faster. Assuming money spent on ticket submission uses the same average gas price as DKG submission, this doesn't have unexpected adverse effects on staker finances which are calculated on a max-gas basis.
Normal gas costs tend to be in the range of 6~12 gwei, meaning that 1~2% would give expected behavior most of the time but increase the entry frequency required for the system to stay up without intervention by 2.5~5x in the worst-case. However, I don't think we necessarily need to change it as this higher rate of DKG just means that the average-case robustness of the beacon is better than the worst-case the parameters were calculated with.


move_cursor = false doesn't work when adding surrounds

If this is a general question about the plugin or a point of confusion on a feature, please check the wiki before opening a new discussion post.
Checklist

[x] Have you updated the plugin to the latest version on main branch?
[x] Have you checked the Breaking Changes issue?
[x] Have you read through :h nvim-surround to see if there might be any relevant information there?

Describe the bug
When adding surrounds with move_cursor = false, the cursor position goes to the beginning of surround. This only happens when adding. Changing and deleting works as expected.
To Reproduce

nvim -nu minimal.lua
try to add surrounds

local on_windows = vim.loop.os_uname().version:match("Windows")

local function join_paths(...)
	local path_sep = on_windows and "\\" or "/"
	local result = table.concat({ ... }, path_sep)
	return result
end

vim.cmd([[set runtimepath=$VIMRUNTIME]])

local temp_dir = vim.loop.os_getenv("TEMP") or "/tmp"

vim.cmd("set packpath=" .. join_paths(temp_dir, "nvim", "site"))

local package_root = join_paths(temp_dir, "nvim", "site", "pack")
local install_path = join_paths(package_root, "packer", "start", "packer.nvim")
local compile_path = join_paths(install_path, "plugin", "packer_compiled.lua")

local function load_plugins()
	require("packer").startup({
		{
			"wbthomason/packer.nvim",
			"kylechui/nvim-surround",
		},
		config = {
			package_root = package_root,
			compile_path = compile_path,
		},
	})
end

_G.load_config = function()
	require("nvim-surround").setup({
		keymaps = {
			insert = "sa",
			insert_line = "saa",
			visual = "s",
			delete = "sd",
			change = "sc",
		},
		highlight_motion = {
			duration = 0,
		},
		move_cursor = false,
	})
end

if vim.fn.isdirectory(install_path) == 0 then
	vim.fn.system({ "git", "clone", "https://github.com/wbthomason/packer.nvim", install_path })
	load_plugins()
	require("packer").sync()
	vim.cmd([[autocmd User PackerComplete ++once lua load_config()]])
else
	load_plugins()
	require("packer").sync()
	_G.load_config()
end

Expected behavior
when adding surrounds with move_cursor =  false, the cursor should keep its original position
Additional context
Documentation:
https://github.com/kylechui/nvim-surround/blob/2200f7275e19aea608f33872680e25ee687cfcb3/doc/nvim-surround.txt#L225

Thanks for the bug report! I appreciate you taking the time to make this project better


Patch CVE-2023-46218 in cmake

cmake uses vendored curl. The CVE is for curl.

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Patch CVE-2023-46218 in cmake
Change Log

CVE-2023-46218

Does this affect the toolchain?

NO
Associated issues

NA

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2023-46218

Test Methodology

Unified Buddy Build results are here

I have reran the unified buddy build after updating the manifests files. Here are the results.


ReadHandler for Domain and Repository should make policyDocument available

Is your feature request related to a problem? Please describe.
Without it, GetAtt cannot get resource policy for resources
Describe the solution you'd like
Translator should populate policyDocument into the resource model using getPermissionPolicy APIs

Users can set the policyDocument for domain and repository and is NOT a readOnly property . There seems to be no need to have these as part of GetAtt. Hence closing this issue.


Legacy of Kain: Soul Reaver function won't compile on MASPSX

mcassert is a function that is currently 100% matched using inlined asm, however the solution while compiling with PSYQ 4.4 won't work with the current maspsx preset for Legacy of Kain: Soul Reaver on decomp.me as can be seen in the following scratch:
https://decomp.me/scratch/WumtK

Closed via https://github.com/mkst/maspsx/commit/ef30355e943f7b66ada13b3422500880100d5f38


Add a 'Add New Tool ' button for tool inclusion requests

Is your feature request related to a problem? Please describe.
https://sbombenchmark.dev currently lists the most popular SBOM generators, but it is not a comprehensive list, and might not be aware of new SBOM generators being built.
Describe the solution you'd like
Add a button that allows users to submit a request for new SBOM generators.
Describe alternatives you've considered
Following this github repo
Additional context
N/A

https://sbombenchmark.dev/

We should probably link to  https://github.com/interlynk-io/sbombenchmark.dev/issues/new/choose
as the current redirect is confusing.


"Iron Banner Gear Equipped" not updating in sync with reset estimate

In the reputation dashboard, the reset estimates (wins until reset, etc.) are updating when the app refreshes, but "Iron Banner Gear Equipped" and "Iron Banner Emblem" are not updating. The correct values are shown when opening the detail view.
Dashboard showing updated estimates, but stale gear info:

Detail view showing all info up to date:

Fixed in 2.0!


Fix loopargs_t object duplication into ASYNC context

Code was relying on an implicit data-sharing through duplication of
loopargs_t pointer-members made by ASYNC_start_job().
Fix of issue raised in PR #1462

you tested this? :)

[me@localhost openssl]# ./apps/openssl speed -elapsed -async_jobs 1 dsa512
You have chosen to measure elapsed time instead of user CPU time.
Doing 512 bit sign dsa's for 10s: 72481 512 bit DSA signs in 10.00s
Doing 512 bit verify dsa's for 10s: 115496 512 bit DSA verify in 10.00s
OpenSSL 1.1.0-pre6-dev  xx XXX xxxx
built on: reproducible build, date unspecified
options:bn(64,64) rc4(16x,int) des(int) aes(partial) idea(int) blowfish(ptr)
compiler: gcc -DDSO_DLFCN -DHAVE_DLFCN_H -DOPENSSL_THREADS -DOPENSSL_NO_STATIC_ENGINE -DOPENSSL_PIC -DOPENSSL_IA32_SSE2 -DOPENSSL_BN_ASM_MONT -DOPENSSL_BN_ASM_MONT5 -DOPENSSL_BN_ASM_GF2m -DSHA1_ASM -DSHA256_ASM -DSHA512_ASM -DMD5_ASM -DAES_ASM -DVPAES_ASM -DBSAES_ASM -DGHASH_ASM -DECP_NISTZ256_ASM -DPOLY1305_ASM -DOPENSSLDIR=""/usr/local/ssl"" -DENGINESDIR=""/usr/local/lib64/engines-1.1""
sign    verify    sign/s verify/s
dsa  512 bits 0.000138s 0.000087s   7248.1  11549.6

Because of this block of code, you can't launch all the tests in a row :
    if (doit[D_CBC_128_CML]) {
        for (testnum = 0; testnum < SIZE_NUM; testnum++) {
            print_message(names[D_CBC_128_CML], c[D_CBC_128_CML][testnum],
                          lengths[testnum]);
            if (async_jobs > 0) {
                BIO_printf(bio_err, "Async mode is not supported, exiting...");
                exit(1);
            }

it annoying, i will push another PR...

If you decide to take this pull request instead of #1462 , could you please ping me or @stevelinsell?
We would like to run a few tests with our engine - which is really asynchronous - to make sure everything is working as expected.
Thanks!

Please take a look, as I prefer this version.

ping @agrandi @stevelinsell  please take a look, as I prefer this to #1462.

Will do! Thanks @richsalz

I have only had a chance to run a functional test and the code works OK with the async engine.
Tomorrow I'll run a performance test and I'll let you know if we see any change compared to the previous implementation.

+1 from me subject to an ok from @agrandi

Bad news!
We have done some more test and we discovered that this version causes a seg fault in sync mode.
Here is the back-trace:
Starting program: /bin/taskset -c 4 apps/openssl speed rsa2048
process 130699 is executing new program: /root/agrandi/beta3_bugfix/openssl-worldread/apps/openssl
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
Doing 2048 bit private rsa's for 10s:
Program received signal SIGSEGV, Segmentation fault.
0x0000000000459529 in RSA_sign_loop (args=0x6b3940) at apps/speed.c:915
(gdb) p tempargs
$1 = (loopargs_t *) 0x0
(gdb) bt
#0  0x0000000000459529 in RSA_sign_loop (args=0x6b3940) at apps/speed.c:915
#1  0x0000000000459c08 in run_benchmark (async_jobs=0, loop_function=0x45950e <RSA_sign_loop>,
#2  0x000000000045ce2c in speed_main (argc=1, argv=0x7fffffffe540) at apps/speed.c:2337
#3  0x0000000000437174 in do_cmd (prog=0x6b2810, argc=2, argv=0x7fffffffe530) at apps/openssl.c:471
#4  0x0000000000436900 in main (argc=2, argv=0x7fffffffe530) at apps/openssl.c:177

The first part of run_benchmark (used in the sync case) should be changed to pass the right type of pointer:
    if (async_jobs == 0) {
        return loop_function((void *)loopargs);
    }

We have tested the performance of the async mode and the patch does not have an impact.
I think you can merge this, as long as you fix the sync mode.
Would it make sense in your opinion to add a few tests cases to the continuous integration tools to make sure that speed is working OK?

Ideally "yes", although we don't really have any tests for the command line utils. We should do. However this close to 1.1.0 release that is a low priority. More important is to get this fixed.

@mattcaswell +1 to this MR, and +1 for you to make the change i marked with my thumbs-up.

My bads...
Fixed !
openssl ca ...is tested at least , I'm sure ;)

My +1 reconfirmed. Ping @richsalz for reconfirm.

+1.  you merging it @mattcaswell ?

Yes

Merged. Thanks.


Typescript Intellisense fails to reference inherited methods after private property in class

From @FadeZhanger on August 29, 2018 16:26

VSCode Version: 1.27.0-insider (latest commit on Aug 29)
OS Version: Windows 10 build 17134
Typescript version: 3.0.1

Steps to Reproduce:

Create a .ts or .tsx file with a class that extends another class (example below)

class Parent {
  protected shouldWork() {
    console.log();
  }
}

class Child extends Parent {
  private hmm: string;
}

Give the child class a private property.
Try referencing the inherited method from the parent class below the private property in the child class.

Expected: Intellisense autocomplete for the inherited method.
Actual: Nothing shows up. Changing the private property to a public or protected property causes Intellisense to appear again.
Video: http://recordit.co/3AXtreY6pc
tsconfig:
{
  "compilerOptions": {
    "allowJs": true,
    "allowSyntheticDefaultImports": true,
    "checkJs": false,
    "experimentalDecorators": true,
    "jsx": "react",
    "lib": [
      "es2015",
      "es2016",
      "es2017",
      "dom"
    ],
    "module": "es2015",
    "moduleResolution": "node",
    "noImplicitAny": true,
    "noImplicitReturns": true,
    "noImplicitThis": false,
    "outDir": "dist",
    "strictNullChecks": true,
    "target": "es5"
  },
  "exclude": [
    "node_modules",
  ],
}

Does this issue occur when all extensions are disabled?: Yes
Copied from original issue: Microsoft/vscode#57523

Tested with 3.1.0-dev-20180901
To summarize, the issue is trying to trigger suggestions at the | in the example:
class Parent {
    protected shouldWork() {
        console.log();
    }
}

class Child extends Parent {
    private hmm: string;

    sh|
}

An entry for shouldWork is not included in the completions response

Tested with TS 4.0 nightly builds and confirmed this is now working:


[Feature] Dapr - Azure extensions + Redis configuration

.NET Aspire issue link
No response
Overview

Dapr–Azure Integration

Simplifies the configuration of Azure resources for Dapr.
Uses Aspire’s resource builder pattern to configure the necessary infrastructure when you publish your application.
Eliminates the need to maintain additional Dapr component manifests.

Azure Redis Extension : Builds on top of the Dapr–Azure integration to configure Redis as a Dapr state store.

Usage example
var builder = DistributedApplication.CreateBuilder(args);

var redisState = builder.AddAzureRedis("redisState").RunAsContainer();

// This currently only effects publishing
// local development still uses dapr redis state container
var daprState = builder.AddDaprStateStore("daprState")
    .WithReference(redisState); // <= This instructs the application to configure dapr to use the redis state store

// API does not provide any functional example of Dapr - it simply demonstrates referencing the dapr state
var api = builder.AddProject<Projects.CommunityToolkit_Aspire_Hosting_Dapr_AzureRedis_ApiService>("example-api")
    .WithReference(daprState)
    .WithDaprSidecar();

builder.Build().Run();

Additional context
I have this on a branch ready to go into a PR
Help us help you
Yes, I'd like to be assigned to work on this item

I'm unfamiliar with how Dapr works, but if I'm understanding, this is a proposal to add this API:
var daprState = builder.AddDaprStateStore("daprState")
    .WithReference(redisState);

And that gives you a resource you can provide to another resource, such as the API project in the above sample, to have an opinionated pre-configured Dapr integration. Is that right?

Feels like these should go with the dapr integration

I guess there are two parts to the proposal the first part (CommunityToolkit.Aspire.Dapr.AzureExtensions) adds a set of functions that can be used to configure the dapr resources directly from the apphost rather than requiring supplementary yaml files for the dapr config
The second part is a specific implementation as above for redis (CommunityToolkit.Aspire.Dapr.AzureRedis) which adds the
.WithReference(redisState);
i.e
 var daprState = builder.AddDaprStateStore("daprState")
already exists as part of Aspire.Hosting.Dapr but requires additional configuration based on yaml files. This uses withreference to configure the dapr state based on the
 var redisState = builder.AddAzureRedis("redisState")
above.
Note the current implementation has no bearing on local development, only on publishing
Two readme files might help
https://github.com/FullStackChef/CommunityToolkit-Aspire/blob/dapr-azure-hosting-ext/src/CommunityToolkit.Aspire.Hosting.Dapr.AzureExtensions/README.md
https://github.com/FullStackChef/CommunityToolkit-Aspire/blob/dapr-azure-hosting-ext/src/CommunityToolkit.Aspire.Hosting.Dapr.AzureRedis/README.md

Feels like these should go with the dapr integration

Couple of things I considered here:

Pretty certain that would simplify parts of the integration.
The current Dapr integration as I understand it is not Azure specific - (I think my integration is - or at least I haven't considered non azure since I live in azure)
Perhaps it's not one or the other? Maybe AzureExtensions should sit with Dapr integration while specific implemenations are toolkit? The reason I created this as two libraries is that otherwise you have to reference every possible azure hosting package in the toolkit where as this approach allows you just to install redis for redis if that makes sense

Cool, that all makes sense.
The question on whether this should be in the Aspire repo (likely as new packages due to the Azure dependency) or the Community Toolkit is something that we'd need the Aspire team to weigh in on.


Challenge http://www.freecodecamp.com/challenges/waypoint-import-a-google-font has an issue

When I submit my code,
It will not allow me to pass, despite fulfilling all the criteria.
<link href="http://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.

I'm not going to do anything for the best way to do with the page all day I
have not been on the page at all thank you for being there for me
Gary M Barker
On Feb 19, 2016 12:46 PM, "Ryan Yim"<EMAIL_ADDRESS>wrote:

When I submit my code,
It will not allow me to pass, despite fulfilling all the criteria.
.red-text { color: red; } h2 { font-family: "Lobster"; } } p { font-size:
16px; font-family: Monospace; } CatPhotoApp
Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching
attack your ankles chase the red dot, hairball run catnip eat the grass
sniff.
Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere
rip the couch sleep in the sink fluffy fur catnip scratched.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7147.

Thanks for the issue. But, unfortunately your code is not correct.
 h2 {
    font-family: "Lobster";
  }

  }     //<<-------------------------- Delete this extra bracket.
  p {
    font-size: 16px;
    font-family: Monospace;
  }

If you are still stuck please ask your queries at Help Chat.

Thank you for this, amateur mistake  :(


Typo on this page "producvitivy score"

[Enter feedback here]

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

ID: bfe3e20f-ec8b-1c49-2fb8-25a900533ade
Version Independent ID: ed13a2a7-ef92-bc3c-d051-c2cfa22b2415
Content: Microsoft 365 Network Insights (preview) - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/office-365-network-mac-perf-insights.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@MSFTEfeedback Thank you for your feedback. We have updated the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/4880
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @MSFTEfeedback Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


CMake support for restclient-cpp

Hi Guys,
We like to know CMake support for this project. currently, we are trying to build this project via Visual Studio Compiler unable to do it. can you please guide me. How to do this.
-Dhanasekaran.

Closing this for lack of details about the problem. Please feel free to reopen if you can provide more information.


Broken OpenAI LLM implementation for node version 18

Issue
When running the vanilla OpenAI LLM function, it returns a 400 for node version 18.
To reproduce
The following code was used to create this error.
import { OpenAI } from "langchain/llms/openai";

export const run = async () => {
  const model = new OpenAI({ modelName: "gpt-3.5-turbo"});
  try {

    const resA = await model.call(
      "What would be a good company name a company that makes colorful socks?"
    );
    console.log({ resA });
    } catch (e: any) {
      console.log(e!.response!.data)
    }
};

run()

Error from the response body
{
  error: {
    message: 'you must provide a model parameter',
    type: 'invalid_request_error',
    param: null,
    code: null
  }
}

As seen, even though a model parameter is provided it still returns an error asking to provide one.
Additional notes
This issue is not present in node v19 and v20.

I've been struggling with that one, thanks for the issue report.

@DarylRodrigo
I just run this code using v18.15.0 and it works. The only change to the code you provided was adding openAIApiKey to the constructor.  Do you have any more details on this?

@DarylRodrigo @JeremyFabrikapp I just ran the exact code snippet above using node v18.16.0 and it works. Can you confirm the exact version of Node you're using? And any more details about your setup

@nfcampos it seems like the issue doesn't occur on every 18.x version, i've tried the followings and so far found only one incompatibility :

Node v18.0.0 (npm v8.6.0)  ✅
Node v18.1.0 (npm v8.8.0)  ❌
Node v18.4.0 (npm v8.12.1) ✅
Node v18.16.0 (npm v9.5.1)  ✅

I'm using NVM.
I can push a fix to .nvmrc by setting specifically 18.16.0 rather than v18 or 20.

Using Node v18.16.0 and still getting same issue
Error: Request failed with status code 400 and body {
    "error": {
        "message": "you must provide a model parameter",
        "type": "invalid_request_error",
        "param": null,
        "code": null
    }
}

@dqbd another one to look into, I cannot reproduce this

@jacobcwright can you confirm it's working on your side with node 20.x ?
Also are you using a static Node install or nvm ?

I'm using nvm
It does indeed work with Node v20.1.0

The issue stems from fetch implementation found in Node 18.1.0 adding an additional suffix to the Content-Type when passing a string as a body. The Content-Type header will thus be application/json, text/plain;charset=UTF-8 instead of application/json.
The issue seems to be gone in LangchainJS 0.0.71 and later, where the fetch adapter is not used when NodeJS is detected #1144 and the issue goes away when using other NodeJS 18.x versions. Consider upgrading either of them to fix this issue.
Closing this issue for now.

Note: if the issue occurs again, it might make sense to patch axios-fetch-adapter to convert body into an Uint8Array instead, this will fix the issue even in NodeJS 18.1.0, but I'm not sure if this might break other environments as well.

Thanks @dqbd for the coverage!


Move Projects filtering to Projects page

The "Projects" filtering bar on the left side of the homepage would make more sense if it were shifted onto the [Projects page] (https://codeforphilly.org/projects).

[ ] Add Projects link to top navbar
[ ] Move Projects filtering sidebar to Members page
[ ] Remove Projects filtering sidebar from home page

Code for Charlotte's already done this with their laddr instance, might be a good starting point.

@bennovack this has already been added, right?
Currently:

Projects link in top navbar
projects filtering added to /projects
members filtering added to /people

@schlos yep, again good catch, thanks!


The pdi-pur-plugin included in the war file is not loaded

When webSpoon is deployed to the Apache Tomcat shipped with BI server (CE or EE), it will search plugins in the pentaho-solutions/system/kettle/plugins folder and will find the pdi-pur-plugin.
However, when deployed to the vanilla Tomcat or Jetty, the pentaho-solutions/system/kettle/plugins folder is not there and it will not look for the plugins folder extracted from the war file; consequently, the pdi-pur-plugin is not loaded.

This can be resolved by setting the environment variable: KETTLE_PLUGIN_BASE_FOLDERS properly.


Tooltip affects width of trigger

Current Behavior
In skeleton v3, <Tooltip> appears to have no class attribute, making it difficult to control the width of the trigger that it wraps.
Since it's converted into what appears to be a <span data-testid="tooltip">, it ends up affecting the width of any block-level trigger elements it wraps: the trigger's w-full starts filling up the inline <span> rather than the actual container of the <Tooltip>.
Component not wrapped into a tooltip (intended width):

Component wrapped into a tooltip (unintended, shorter width):

Expected Behavior
No response
Steps To Reproduce
No response
Link to Reproduction / Stackblitz
No response
More Information
No response

@sssilver it looks like there's an issue with our documentation tool that's not picking up the style property classes available to our popover features. There definitely is available classes. See the source here for a quick reference:
https://github.com/skeletonlabs/skeleton/blob/next/packages/skeleton-svelte/src/lib/components/Tooltip/Tooltip.svelte#L14
You would want to use triggerClasses in this case. By default the wrapping element is a span, which are inline. So you may want to set block like so:
triggerClasses="block"

We'll keep the ticket present until the documentation issue is resolved. Thanks.

@Hugos68 FYI - it looks like we have a gap in Necroparser.

My mistake, we need additional root element props available to the parent <span>. Without that, this change will not be possible.
Note to myself: let's investigate this for all four Svelte popover components.

Why not remove it? 🤔 The span doesn't seem to do anything

@Hugos68 I think we covered this on Discord - but just for the record, the wrapping parent element removes the trigger/popover from the flow of the DOM. So if you have space-y-4 this doesn't affect the these elements. They remain relative to one another.

@sssilver FYI I've made a similar change to the props here where I've added base and classes to the root element for the Svelte Popover, Tooltip, and Modal components. The Combobox component already had these.
See: #3031


error of processing all videos in upload folder

when running
!python inference_realesrgan_video.py --input upload -n RealESRGANv2-animevideo-xsx2 -s 4 -v -a --half --suffix outx2
I'm getting :
Traceback (most recent call last):
  File "inference_realesrgan_video.py", line 199, in <module>
    main()
  File "inference_realesrgan_video.py", line 108, in main
    if mimetypes.guess_type(args.input)[0].startswith('video'):  # is a video file
AttributeError: 'NoneType' object has no attribute 'startswith'

I'm unable to run the inference on the videos inside the upload folder one after the other.
Hope someone can help fix this issue.

I want to run inference on all files in a folder , not one file at a time manually.

@xinntao hope you can have a look at this.
I'm using the anime video colab notebook.

@GeorvityLabs Script is made to input one video file. Video gets extracted into frames then upscaled and finally rendered back to video. Also if you run it on colab you might run out of storage space when upscaling multiple videos.


chore: combine waci issuance flow with share flow

Signed-off-by: Firas Qutishat<EMAIL_ADDRESS>

We are issuing a University Degree Credential, but verifying a PR Card, so the verifier cannot find the credential it is looking for - can we bring this in accordance?


exception thrown on first and every refresh() when using 1.54" monochrome EPaper display

Looks like Adafruit started shipping the 1.54" monochrome display with the SSD1681 chipset as of June 7th 2021. https://www.adafruit.com/product/4196
For some reason I can get an initial image on the display but cannot refresh without throwing an exception ("refreshed too soon!"). Is anyone else having issues with this? I can post code but I imagine I'm not the only one having this issue. I have tested with the STM32 feather and the Feather M4 with the same results. I've tried using the busy pin and not with the same results. I've been following the tutorials for CircuitPython but was only able to get the display to refresh by using a different micropython driver..

@caileighf what does your code look like that you are running? By default, you can't refresh more than once every 180 seconds. You may want to try running simpletest if your code isn't working. I just verified the sample code (and this library) are working fine with the latest release of CircuitPython.

Whoops, just noticed you posted about the monochrome display and I tested the tri-color.

Hi @makermelissa I actually just came to the same conclusion as you and found your issue (https://github.com/adafruit/circuitpython/issues/2137) really helpful! I got the code working when I used these lines:
display.show(None)
display.show(splash)
display.refresh()

I also needed to add a shorter seconds_per_frame value to the adafruit_ssd1681.SSD1681 constructor.
I'm using the Adafruit Feather M4 (https://www.adafruit.com/product/3857) and the 1.54" Monochrome EPaperDisplay (https://www.adafruit.com/product/4196) with the Adafruit SSD1681 displayio driver. The Feather is running CircuitPython 7.0.0.
Currently I'm able to refresh and not have issues with the displayio objects showing up offset from their original position (I can't find that issue but I think you were on that thread? I'll keep looking and post here if I find it. It was for an older version of CircuitPython 5.0.0 I think? Essentially, after updating an instance of type label.Labels .text attribute, it would show up on the display offset vertically from the original location and the x, y position for the object hadn't changed). I think display.show(None) was needed before each display.show(splash) and subsequent display.refresh() following your init/deinit logic in the issue.
The inability to refresh() issue is solved by passing a seconds_per_frame value to the adafruit_ssd1681.SSD1681 constructor BUT, the additional display.show(None) and subsequent display.show(splash) solve a different issue (offsetting issue which always occurred for me even when refresh() was working).

I apologize for the delay! I need to post my code. I'll get to that ASAP. Need to cut it down to a minimal example.

It turns out I didn't have the correct display on hand to test anyhow, so it should be here in a couple days. My monochrome ones were both older ones. Although it shouldn't normally need that code to work, it might in its current state because I probably never updated the driver to work with the monochrome display since I didn't have one.

I really appreciate it @makermelissa ! Thanks!


update coverage badge

badge in readme wasn't properly displaying the coverage

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.032%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
483

Relevant Lines:
499

💛  - Coveralls


fixed View Profile button text position on center

Fixes Issue
Fixed View Profile buttion text on center.
Closes #9504
Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots
Before:

After:

Note to reviewers
Thanks for reviewing

@AbdullahWins This issue is in triage and not assigned to you nor ready for dev.  Please read our contributing guide before working on anything so that you do not spend time on PR's we cannot accept. Closing this issue.
How to contribute


Raw data question for Maze reaching

Hi, I've recently been looking at the DANDI release ( https://dandiarchive.org/dandiset/000070/draft ) for the reaching dataset and have two questions.

I notice a drastic base firing rate difference in the datasets here and in another maze dataset released for the neural latents benchmark. I plot the rasters from the two datasets here, for example:

This DANDI release has average firing 50Hz, while the benchmark has average firing ~5Hz.  Even accounting for sorting, it seems unlikely that the MUA in this release should be 10x. Is there some filter I am missing? This DANDI raster seems to have some line noise, perhaps?

Are there raw (non-NWB) datasets available for the maze datasets? I ask because I saw some funky non-ascending/overlapping spike times stored in some of the NWB files (the dataset producing the above image was fine in this regard) and was wondering whether the conversion process might have introduced a bug.

cc @Saksham20 , who made these files

Hi @joel99 thanks for bringing this to our attention and sorry for the delayed response. I'm just getting back from holiday break. I'll look into these files and see if we see similar FR in the source files.

Thanks! Gentle query if you've gotten the chance to look at this yet?

Hi @joel99, I'm looking into this now. Thanks for your patience.

First of all, thank you for raising this issue. It's great to see a DANDI user doing a comparative analysis between these types of datasets from different labs.
I plotted the spike times from the original data and got a similar raster:

I get a mean of ~38 Hz, which is roughly consistent with your analysis. We aren't looking at the same trial, so you wouldn't expect the numbers to match perfectly.
One idea why this might be so different from the Sabes dataset might be that the Shenoy data is unsorted. They don't got through the processing step of separating out spikes and just report threshold crossings per channel. The Sabes dataset might instead examine each spike waveform and cluster them (which would divide the avg fr by the # of groups per channel) and remove waveforms that cannot be grouped (which would further reduce the avg fr).
If you want to do a good comparison between different labs, I think you'll need to do the same preprocessing, e.g. filtering, spike sorting, etc. Those types of decisions can have a big impact on the statistics of the resulting data, and I would expect these types of issues to continue to arise.

Are there raw (non-NWB) datasets available for the maze datasets? I ask because I saw some funky non-ascending/overlapping spike times stored in some of the NWB files (the dataset producing the above image was fine in this regard) and was wondering whether the conversion process might have introduced a bug.

You can see from the conversion script that we take the spike times as written and copy them to NWB:
https://github.com/catalystneuro/shenoy-lab-to-nwb/blob/0603061fdbcff8e5dad71ce38981cc4c2e352cc1/maze_task/matextractor.py#L29-L40
There's nothing fancy going on here, so any issues you have with the spike times in the NWB files I would expect you to also find in the source files.

Thanks for checking. OK, good to know! Do you have thoughts on the mismatch in the shape of ir and jc columns in the Reggie/Jenkins .mat files?

A reminder ping here?

Sorry @joel99, I've been swamped. I believe they are using: csc_matrix notation: https://docs.scipy.org/doc/scipy/reference/generated/scipy.sparse.csc_matrix.html#scipy-sparse-csc-matrix
See the last argspec:

csc_matrix((data, indices, indptr), [shape=(M, N)]):
is the standard CSC representation where the row indices for column i are stored in indices[indptr[i]:indptr[i+1]] and their corresponding values are stored in data[indptr[i]:indptr[i+1]]. If the shape parameter is not supplied, the matrix dimensions are inferred from the index arrays.

I think what they are doing is csc_matrix((data, ir, jc)). So the length of jc would be the number of columns and the length of ir and of data should be the number of non-zero datapoints.

Brilliant, thank you! Sorry for the spam.


missing FileDialog MODE_OPEN_ALL

I need a way to set FileDialog.MODE_OPEN_DIR and FileDialog.MODE_OPEN_FILE
I can't see a flag for it. Setting no mode results in a save dialog I need a open dialog.
Can someone confirm that this feature is not implemented?
Looks easy to implement.
https://github.com/godotengine/godot/blob/e46e43d2aaa6339a1675eb989f41885e745bf5c3/scene/gui/file_dialog.cpp
https://github.com/godotengine/godot/blob/e46e43d2aaa6339a1675eb989f41885e745bf5c3/scene/gui/file_dialog.h

It looks like those constants are bound, unless I'm missing something?
https://github.com/godotengine/godot/blob/e46e43d2aaa6339a1675eb989f41885e745bf5c3/scene/gui/file_dialog.cpp#L710-L713

@bojidar-bg I am not sure, but I think he mean a mode to open both files and directories.


This has been fixed in master for a while, and will be part of 2.2.0
https://github.com/kubernetes/helm/pull/1788


Storage Adapter LevelDB

I'm gonna code a new storage adapter for levelDB, using LevelUP. :+1:

Brilliant, thanks!

There we are: roccomuso/keyv-leveldb
I have @keyv/leveldb as package name. The storage adapter is tested against keyv-test-suite and it's fully API compliant if you wanna publish it under the @keyv namespace. I've also add you as mantainer of the repo.
To pass the tests the line leveldb: '@keyv/leveldb' should be added to keyv:index.js#L7
Probably I'll work on a storage adapter also for etcd.

Thanks for this, it looks really great!
Pretty busy with another project atm but would love to add this as an official adapter after I've had a chance to check it over.

@lukechilds thanks :)

@roccomuso - going to close this out. If you plan to publish it just let me know and happy to add it as a 3rd party library.


Commonize webhook validations at the some points

/kind feature
Based on this discussion (https://github.com/kubeflow/training-operator/pull/2051#discussion_r1559967156), after we complete the implementation of the webhook validations here (#1993), we should commonize duplicated validations like the podTemplate.

/remove-lifecycle stale

/remove-lifecycle stale
/good-first-issue

/assign


"More VMs and related solutions" option not available at described place.

"Select the arrow next to Create, then select More VMs and related solutions."
this step describes something that does not seem to be available in the regular portal UI in the "Virtual Machine" area.

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

ID: 6ebd7f86-9641-a031-bea0-b37eb5400586
Version Independent ID: 25812e0e-3c0f-ab85-2b0a-671b1ad9200f
Content: Build infrastructure and deploy workloads using Microsoft Copilot for Azure (preview)
Content Source: articles/copilot/build-infrastructure-deploy-workloads.md
Service: copilot-for-azure
GitHub Login: @JnHs
Microsoft Alias: jenhayes

@phwecker
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for raising this @phwecker, I'll work on getting this updated to match the current UX.

Thanks again @phwecker - we appreciate your taking the time to point this out. I've updated this article to link directly to the page, rather than provide a screenshot, since the options may vary as you mention. I'll close this issue now, but please let us know if you have additional feedback.
#please-close


[CELEBORN-449] repair the hdfs path regex

What changes were proposed in this pull request?
change the hdfs path regex
Why are the changes needed?
Path protocols are all started with xxx://, and is unnecessary to restrict the content after that. Actually, it makes an error when write shuffle files which like "xxx://abc/shuffle/hadoop/rss-worker/shuffle_data/spark-0de72e2ce2e24f6db69c2228dd12a514/0/0-0-0"
Does this PR introduce any user-facing change?
How was this patch tested?
has been tested in testing environment within the company

ping @FMX please take a look

xxx://abc/shuffle/hadoop/rss-worker/shuffle_data/spark-0de72e2ce2e24f6db69c2228dd12a514/0/0-0-0

Thanks, this pull request is necessary.

Simply allow other Hadoop FS compatible file systems is not sufficient, more tests is need @FMX, maybe we should log warning when detect non-HDFS protocol

@FMX I see, let's add UT for the regex, then get it in.

@lianneli Hi I added UT and created a PR(https://github.com/lianneli/incubator-celeborn/pull/1) at your repo.

@FMX you should have permission to push the PR branch directly

@FMX you should have permission to push the PR branch directly

Yes you're right. This PR updated.