Adding a big bench task- debate_judge

Delete request

@sinhadhruv Unfortunately I don't believe that we can delete a PR (only close them), see here for example.


Deezer not working anymore

What version of Music Assistant has the issue?
2023.6.0b3
The problem
Deezer plugin is be able to setup, load the playlist but keep loading when playlist opened and needs to show the numbers. Other plugin (radio) is working as it should be.
Reinstalled the plugin, reconfigured Deezer but it didnt solve the problem.

How to reproduce
Install latest version and try it out with Deezer
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2023.07.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
ODROID

Restore backup (previous version) did fixed te controls via Music Assistant. The Service mass.quee_command is still broken

the issue is confirmed and a fix is on the way

This should be fixed in the new beta 60 :slightly_smiling_face:

Thank you guys! Appreciate all the hard work. I am gonna try Beta 60 when its available.

I can confirm that Beta 60 fixed the issue (and 2023.6.0b3 fixed the command issue). Great work!


duplicate sort options in filter view

duplicate 'sort by' radio buttons at the top and bottom of the filter view now - don't think the ones up top do anything anymore(although there is a mechanism that offers to set it default for all, so maybe port that to the bottom set before removing it)

Just verified and they are working as design.
The top one's are for sorting new on top, or new on bottom
the bottom is for sorting in the mangadex sort order, or the chapter number
so if i did chapter number, and oldest then it will start at chp 1
if i did chapter number and newest then it will start the list at the last chp


Error in render: "TypeError: _vm.$i18nForDatatable is not a function"

It seems that $i18nForDatatable is not defined.

https://github.com/OneWayTech/vue2-datatable/blob/master/src/main.js#L7

I am using Vue js and webpack. My code wasn't able to get this definition on line 7 of main.js. When I added this line to my own code, it complained what "locale" was. I then used this line:
Vue.prototype.$i18nForDatatable = srcTxt => srcTxt

Sorry for replying so late (cursing Chinese Great Firewall)
Did you find the solution?

any solution for this ?

@paolog22 i've got no idea...

could you tell me your Webpack version? 1.x or 2.x?

how about:
import Vue from 'vue'
import Datatable from 'vue2-datatable-component/src/main'

Vue.use(Datatable)

@bsamadi I need your help. May I have your screenshots of code?

I need to figure out what causes the problem on earth

@kenberkeley i am using webpack 2

@paolog22 have you tried https://github.com/OneWayTech/vue2-datatable/issues/30#issuecomment-346539652?

yes.. same error as this

is it alright that you create a repo for me to debug?

technically speaking, is for reproducation

implemented successfully! thanks :)

？？？。。。

you're right i made a mistake at initializing.. now i am looking for how to render row data with custom html.

sorry, but i have this mistake ($i18nForDatatable is not defined) too

[Vue warn]: Property or method "$i18nForDatatable" is not defined on the instance but referenced during render. Make sure to declare reactive data properties in the data option.
found in
--->  at node_modules/vue2-datatable-component/src/index.vue
<Pages/index.vue> at pages/index.vue
 at .nuxt/components/nuxt.vue
 at layouts/default.vue

//nuxt.config.js
build: {
    vendor: [
     ...
      'vue2-datatable-component'
    ],
  },

@Defcoun You have to register it globally, not locally


update path building for HelperLambda CLTransformer to remain relativ…

Re-opening previously closed PR
Issue #, if available: Please see https://github.com/awslabs/aws-centralized-logging/pull/32
Description of changes:
Update the way paths are being build from assets to consider the current directory instead of process.cwd()
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This has been addressed with release v4.0.1


Improve function lookup perfomance

WIP. This is intended to address the remainder of https://github.com/rust-lang/backtrace-rs/issues/232#issuecomment-517578935.
It currently requires a new low level DIE parser in gimli, but I want to do more investigation to see if that can be avoided.

This is all covered by other PRs now.


Add yaml configuration provider

Hello.
Microsoft.Extensions.Configuration assemblies contain many configuration providers including ini, xml and json configuration. Also the json configuration is actually used as one of default config sources when creating hosts.
However, I propose to also add the yaml configuration provider. In my experience yaml is easier to read, understand and edit, is more human readable for the specific case of configuration files.
Json editing feels more error prone, for the reason of possible missing or extra commas or brackets, need to quote every name including json object keys, etc. My example is editing /etc/docker/daemon.json.

See https://github.com/aspnet/Extensions/issues/2659, https://github.com/aspnet/Extensions/issues/1904, etc.


Converts entire text to lowercase instead of just command

yeah that, double check the command parsing. Either check the command text itself, or send the raw data to the command.

Fixed with https://github.com/SpiderlingStudio/Commandler/commit/f38ee89eb2bb69121ed7fedfc0b55d2723b2da92


removeEventListener is deprecated

On the newer versions of React Native, the removeEventListener is deprecated.
https://reactnative.dev/docs/appstate#removeeventlistener
In connection_monitor:
AppState.removeEventListener("change", @visibilityDidChange)

I think it needs to be something like this (I've never done coffeescript before):
  start: =>
    unless @isRunning()
      @startedAt = now()
      delete @stoppedAt
      @startPolling()
      @eventListener = AppState.addEventListener("change", @visibilityDidChange)
      @log("ConnectionMonitor started. pollInterval = #{@getPollInterval()} ms")

  stop: =>
    if @isRunning()
      @stoppedAt = now()
      @stopPolling()
      if @eventListener
        @eventListener.remove()
      @log("ConnectionMonitor stopped")

Published new version with the fix


[SPARK-16192][SQL] Add type checks in CheckAnalysis

What changes were proposed in this pull request?
CollectSet cannot have map-typed data because MapTypeData does not implement equals.
So, this pr is to add type checks in CheckAnalysis.
How was this patch tested?
Added tests to check failures when we found map-typed data in CollectSet.

Test build #61170 has started for PR 13892 at commit 5110474.

Test build #61170 has finished for PR 13892 at commit 5110474.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61170/
Test PASSed.

@cloud-fan Could you check this?

@maropu could you implement checkInputDataTypes and do the validation in that method?

Test build #61211 has started for PR 13892 at commit 73dc6e7.

Test build #61212 has started for PR 13892 at commit 681b993.

Test build #61213 has started for PR 13892 at commit dd7e233.

Test build #61216 has started for PR 13892 at commit 4924984.

Test build #61217 has started for PR 13892 at commit 1a445c8.

Test build #61211 has finished for PR 13892 at commit 73dc6e7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61211/
Test FAILed.

LGTM - pending jenkins

Test build #61212 has finished for PR 13892 at commit 681b993.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61212/
Test PASSed.

Test build #61213 has finished for PR 13892 at commit dd7e233.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61213/
Test PASSed.

Test build #61216 has finished for PR 13892 at commit 4924984.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61216/
Test PASSed.

Test build #61217 has finished for PR 13892 at commit 1a445c8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61217/
Test PASSed.

Merged build finished. Test PASSed.

Merging to master/2.0. Thanks!


Hold notes are broken

Describe your bug here.
The hold notes for me are broken. Every time I add one, when I save and come back to the song I'm charting, the hold note's tail shortens. Why is this happening?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I did not edit anything.

It’s the same thing as https://github.com/ShadowMario/FNF-PsychEngine/issues/9524
Also what version are you using

I'm not even using Change BPM, they're just broken for me. And I'm using 0.5.2h.

I'm not even using Change BPM, they're just broken for me. And I'm using 0.5.2h.

Then update to action build that fixes it https://github.com/ShadowMario/FNF-PsychEngine/actions/runs/2864975591
And close the issue

That's the EXACT build I'm trying to avoid because of another issue that I put up.

You're using an old version, close this

Ok, thanks for the help. Wish me luck as 0.6 or 0.6.1 will probably crash as well.

Nope, 0.6 doesn't fix the issue.

quit opening and closing the issue

Sorry, sorry. This issue just isn't being fixed and I can't figure out what to do. The thoughts in my mind are "Should I reopen this?" "Should I make a new one?" "What should I do?" This problem is really getting to me.


Apply patch to make boost::thread on_tls_callback work with VS2017

Boost 1.63.0 has a bug when using the VS2017 toolchain that causes a wrong TLS function signature to be used. The result is that when statically linking Boost the application can't initialize correctly because instead of an int a void is being returned. It has been fixed in the current development version of boost, but until a new release, this continues to be an issue.
Original discussion can be found here:
https://github.com/boostorg/thread/pull/114

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks for the PR!


[Question] How to select  between date on daterange using e2e test.

Existing Component
Yes
Component Name
DateRange
Description
I am a e2e tester. and i have no idea about how to click select date when attibute 'editable' is false.
Please suggest a way how can i select between date on daterange when attibute 'editable' is false.
Or other way.
Thank in advance.

Do you mean to open the dropdown when editable is false? I think clicking the icon will do the job.


Remove explicit secrets from jobs

Changes

Remove unnecessary secrets passing

/approve
/lgtm

Required status check "cla/google" was not set by the expected GitHub app.

Looks like prow persisted the override with the branch - closing


Create Lyrics “utwór-jeden”

Automatically generated by Netlify CMS

:construction_worker: Deploy Preview for thar-ai processing.
:hammer: Explore the source changes: 1c70fbbda45a034458a7abec9c8f945942c2fea0
:mag: Inspect the deploy log: https://app.netlify.com/sites/thar-ai/deploys/60c6120f3a80f300072b3ef0


[Go] pqarrow.NewSchemaManifest creates wrong schema field for array object fields

Describe the bug, including details regarding any error messages, version, and platform.
I'm using the arrow V11 package for reading parquet files.
Calling pqarrow.NewFileReader creates the schema manifest using pqarrow.NewSchemaManifest.
When dealing with objects array fields, the record readers create an array of primitives of the first field in the object. for example:
[[“a”:1, “b”:2},[“a”:3, “b”:4}] -> [1,3]
diving into the code I found this in pqarrow/schema.go -> listToSchemaField:
listGroup := listNode.(*schema.GroupNode)
if listGroup.NumFields() == 1 && (listGroup.Name() == "array" || strings.HasSuffix(listGroup.Name(), "_tuple")) {
	// list of primitive type
	if err := groupToStructField(listGroup, currentLevels, ctx, out, &out.Children[0]); err != nil {
		return err
	}
} else {
	if err := nodeToSchemaField(listGroup.Field(0), currentLevels, ctx, out, &out.Children[0]); err != nil {
		return err
	}
}

At this stage, the listGroup is the inner dictionary. Assuming the above example listGroup.NumFields() returns 2 and the schema node is converted to a primitive field taking the first field in the group (else clause).
It also seems like the groupToStructField function is more suitable for this case.
changing the condition solves the problem and the field is being read as expected.
listGroup := listNode.(*schema.GroupNode)
if listGroup.NumFields() != 1 && (listGroup.Name() == "array" || strings.HasSuffix(listGroup.Name(), "_tuple")) {
	if err := groupToStructField(listGroup, currentLevels, ctx, out, &out.Children[0]); err != nil {
		return err
	}
} else {
	// list of primitive type
	if err := nodeToSchemaField(listGroup.Field(0), currentLevels, ctx, out, &out.Children[0]); err != nil {
		return err
	}
}

Can you take a look and fix it? the current state is misleading as the user might think this is the real value
Component(s)
Go, Parquet

@zeroshade currently I can't share such files due to restricted data.
this is the schema of the problematic field:
optional group field_id=-1 answers (List) {
    repeated group field_id=-1 array {
      optional byte_array field_id=-1 type (String);
      optional byte_array field_id=-1 rdata (String);
      optional byte_array field_id=-1 class (String);
    }
  }

is it something you can work with? if not I'll try to create a sample file like this

Thanks @orianED I was able to track down the issue and filed a PR with the fix for it. The PR is linked here.


Script for regenerate auto-generated conversions.

Part of this PR is forked from #8381
cc @thockin @smarterclayton @lavalamp

cc @justinb

Assigning to @thockin since you were reviewing the original, feel free to reassign.
/cc @bgrant0607 who commented on the original.

LGTM, but what this is missing is the part where it doesn't need any arguments at all and just regenerates the output into pkg/api/${version}/conversion_generated.go for each version :)

@thockin - sorry - I forgot to do git add for the main file :)
PTAL

Thanks for doing this.  LGTM but 1 nit.

#8507 sent out for review.


Indenting is incorrect inside a list or outline

Given a simple list like:
- ## Fruit
  - Apple
  - Orange
  - Banana

If I try to increment the "Fruit" line heading level, the heading prefix is added before the bullet point, which is incorrect.
# - ## Fruit
  - Apple
  - Orange
  - Banana

When it should be done inside the list item, like this:
- ### Fruit
  - Apple
  - Orange
  - Banana

I think you have open the issue in the wrong plugin. This plug doen not change the interraction in source mode


The timing of the displayed text and the audio is inconsistent between the "Play Timeline" option and calling Dialogic.start() ingame

The problem
Describe the bug
I am using the [pause], [speed] and other timing features of dialogic's text event to time the text with the voice of a voice event. To do this I greatly appreciate the "play timeline" button because it is trial and error until you get the timing right, especially when you are looking for a perfectly timed text on a scene that benefits from it for a comedic or dramatic purpose. The problem is that when I finish timing it perfectly and I start the timeline ingame using Dialogic.start("timeline"), the timing is not perfect anymore, it is not consistent with the timing I got using the "Play Timeline" button.
If those two ways of starting the timeline don't have a consistent timing, I wonder if there are more factors that could mess with the timing, but so far I have only noticed this one.
I really think it's crucial that the audio timing is always super consistent to use Dialogic with a game with voice acting, so I really hope that can be fixed because Dialogic is very cool.
Godot 4.0.3, Dialogic 2.0 Alpha 8, Windows 10.
Possible fixes
I'm sorry I never provide possible fixes, I just know I don't have the programming level to contribute to a serious project like Dialogic with code so I try to help reporting bugs and my experiences as a begginer user.

Don't worry if you don't have a solution when reporting!
This seems like a bug but just to be sure we are talking about the same thing: you mean the typing sounds are not conistent? Or what audio are you refering to?

Don't worry if you don't have a solution when reporting!
This seems like a bug but just to be sure we are talking about the same thing: you mean the typing sounds are not conistent? Or what audio are you refering to?

I mean the voice audio events. I use them to add voice lines to the dialogues and I used .wav format by the way, in case it is relevant (I heard Mp3 is harder to process because it is compressed and can introduce some delays).
The timeline is specifically: a characater join event, then a voice event with the wav audio (5-10 seconds, nothing big), then a text, a character leave event and an end timeline event.

Hmm... I've tried to recreate this, but was unable to notice the difference. Would it be possible for you to upload a audio file + timeline script with which you notice this?
Have you tried spawning this (with Dialogic.start()) in a pretty empty scene or a more complex one? (trying to find out if the scene you put it in has any influence on this).

Sorry I didn't see this response. I'm bad at github and I'm not sure why I never get any notifications.
This timing difference is probably hard to notice with the default letter by letter text display. I noticed it while using my workarround to display text word by word tweaking dialogic text settings, described here: https://github.com/coppolaemilio/dialogic/issues/1613
I displayed the text very precisely timed with the voice, and that's how I noticed that the timing was inconsistent.
I can't share this specific scene but I will try to create one when I can.

Done! Now I understand the problem better and updated the issue with more precise information.
With the new information you may not need it but I have a reproduction project ready, the problem is that I have tried every compression mode and I can only get it to 27 MB, being 25MB the maximum allowed to upload. How should I do it?

Thank you! I will look into it once I'm back home.


Update root route to follow Endpoints standard

from the endpoints example repo:
{
  "authors": "/v1/authors?include={books,books.chapters,photos}&filter[{id,name,alive,dead,date_of_birth,date_of_death,born_before,born_after}]",
  "books": "/v1/books?include={chapters,firstChapter,series,author,stores,photos}&filter[{author_id,series_id,date_published,published_before,published_after,title}]",
  "chapters": "/v1/chapters?include={book}&filter[{book_id,title,ordering}]",
  "photos": "/v1/photos?include={imageable}",
  "series": "/v1/series?include={books,photos}&filter[{title}]",
  "stores": "/v1/stores?include={books,books.author}"
}

Another idea would be to split these URLs out into objects. The idea is that folks would be able to programmatically handle the root endpoint easier if they didn't have to parse the URL. For instance,
[
  {
    resourceName: "authors",
    url: "/v1/authors",
    include: ["books", "books", "chapters", "photos"],
    filter: ["id", "name"]
  }
]

However, it makes it less at-a-glance readable for humans.

This should perhaps instead just use the json api schema, with a links field.


Generic initialiser type mismatch causes crashes

Problem
The following code compiles and causes a crash
let somePromise = Promise<String> {
     return Optional<Void>.none
}

somePromise.then { (v: String) in
  // causes crash
  print(v)
}

Explanation
Here is a piece of code that causes this crash. The naming of the initialiser generic type Value can be easily confused with the class generic Value type, so I'll use two different names for my explanation below.
https://github.com/google/promises/blob/afa9a1ace74e116848d4f743599ab83e584ff8cb/Sources/Promises/Promise%2BDo.swift#L26-L38
The way the current convenience initialisers in Promise+Do.swift are written, they allow the user to create a promise of some type OneValue and pass a function that returns a type SecondValue. This is because the convenience initialisers contain their own generic type SecondValue which does not have the match the OneValue type of the class Promise. Since the generic type is erased to ObjCPromise<AnyObject> when any class methods are run that cast the resolved to the class OneValue type
class Promise<OneValue>
  convenience init<SecondValue>(on queue: DispatchQueue = .promises, _ work: @escaping Do<SecondValue) {
    let objCPromise = ObjCPromise<AnyObject>.__onQueue(queue) {
      /*....*/
    }
    self.init(objCPromise)
    objCPromise.__addPendingObject(self)
 }

Is there any reason why there are generic initialiser types used in the project like these?
https://github.com/google/promises/blob/afa9a1ace74e116848d4f743599ab83e584ff8cb/Sources/Promises/Promise%2BDo.swift#L26
https://github.com/google/promises/blob/afa9a1ace74e116848d4f743599ab83e584ff8cb/Sources/Promises/Promise%2BDo.swift#L45

I also encounter this issue. Is there any good idea about this?


hubot not working with systemd

I'm copying this bug here because I don't know whether this bug belongs to hubot or hubot-slack https://github.com/github/hubot/issues/1152
I have tried to configure hubot according to the example[1], but the service does not connect.
Here is what happens when I run hubot manually:
https://gist.github.com/marceloslacerda/13b8f27621b8939b694f
Notice how the "Connecting..." info is displayed correctly.
So I tried to configure hubot for systemd. This is the service file:
https://gist.github.com/marceloslacerda/cf2e24df1aad1af95acf
And the log after running # systemctl daemon-reload; systemctl start hubot
https://gist.github.com/marceloslacerda/ef1ed465a18d647011b8
Notice how the "Connecting..." message does not appear and how hubot-slack throws an error, but I don't know what is it about.
What am I doing wrong?

Apparently you're not providing the correct service token. The setup should work fine, we're running it using a similar service file.

As a workaroud we were used a bash script as a bootstrap script for hubot and put that in our systemd service file.
https://gist.github.com/marceloslacerda/fab1225f28a28a4a27aebf41b6916db6

Closing as resolved.


Add Sentry Mode command

Adds support in the Ruby library for enabling/disabling sentry mode.
Also adds/updates fields across the responses to various data requests.

@timdorr Can this be merged?

Yeah, not sure how I missed this one. I don't have a sentry mode compatible car (AP2.0), so I can't test it myself, but it certainly looks right.


npm install doesn't install @mapbox

Scenario - after cloning my repo and running npm install, this is the only package not being installed. after manually running npm install @mapbox/polyline it works just fine.
`Module does not exist in the module map or in these directories: /path/to/MyReactNativeProject/node_modules/@mapbox``
Cheers

This is very likely an issue with npm and not specific to polyline. Please make sure you're running a recent version of npm, as older versions did not support organizations (@mapbox prefix).


Add Simulation Setup

Add the following to the monorepo:

CARLA and CARLA ROS Bridge
CARLA Viz Tool
Jupyter Notebook Server with CARLA dependencies
Python sample node for CARLA

Is this ready for rereview?

Is this ready for rereview?

Not yet, been a bit busy I'll try and get it done this weekend.

Done addressing the PR comments @Edwardius . Please take a look when you have a chance


options.moduleName does not seem to be respected (probably since 9d88d89)

(note: this is from debugging transpiled code which is monkeypatched by angular, it's very hard to make sense of this)
It looks like the angular transpiler patches are passing a non-filename as the moduleName option, but as far as I can tell, this never seems to be used anymore.
This deleted block of code https://github.com/google/traceur-compiler/commit/9d88d8929309064b10db24a630419704eb45691f#diff-654825a71bbcba648d7bf62497eaaec7L152 seems to have been used to allow overriding a module name, but as far as I can tell, the actual option is never used at all anymore. If that's the case, should this option be removed, or should that block be undeleted?

meh, I guess it's just another API change that goes completely undocumented, fair enough


examples demo_proxy fails

I am trying to run the demo_proxy example from the README
I am using an infura node as my ethereum rpc provider.
However, I keep geting
`web3.exceptions.BadFunctionCallOutput: Could not transact with/call contract function, is contract deployed correctly and chain synced?
What am I doing wrong?
`

How do you solve this mistake?

I was running a ropsten node but the contract is actually deployed on the kovan network!
Switched to kovan network and it worked perfectly!

It's ok, great, but how my address join this contract channel ?


⚠ Chatwoot has degraded performance

In 7c999f8, Chatwoot (https://app.chatwoot.com) experienced degraded performance:

HTTP code: 200
Response time: 5210 ms

Resolved: Chatwoot performance has improved in 1406aee.


Configure the Windows build in Travis CI

Some preliminary research has suggested that the language options supported by Windows Travis CI environments is quite limited and (in many cases) a misconfigured language will result in a build environment that can't even boot up.
Seeing as the Windows build is hanging, I suspect the problem is a matter of setting the language to generic, shell, or bash. All other configurations in the windows yaml recipe should be consistent with a sane python environment.

Appveyor is full-featured for Windows build. Had great success with it in the past.
https://www.appveyor.com/docs/lang/python/


Main email is shown as name, if the account has no name

As in the title, the email is shown on the top of the "My account" section of the app. I'm not sure if this is intended. Could be taken in count the possibility to not show anything, since the email is already shown near the profile pic zone?
Build 20024.

Thanks, I like it even more than my suggestion!


RFC: add take_along_axis to take values along a specified dimension

This RFC proposes the addition of a new API in the array API specification for taking values from an input array by matching one-dimensional index and data slices.
Overview
Based on array comparison data, the API is available across most major array libraries in the PyData ecosystem.
take_along_axis was previously discussed in https://github.com/data-apis/array-api/issues/177 as a potential standardization candidate and has been mentioned in downstream usage. As indexing with multidimensional integer arrays (see https://github.com/data-apis/array-api/issues/669) is not yet supported in the specification, the specification lacks a means to concisely select multiple values along multiple one-dimensional slices. This RFC aims to fill this gap.
Prior art

NumPy: https://numpy.org/doc/stable/reference/generated/numpy.take_along_axis.html
CuPy: https://docs.cupy.dev/en/stable/reference/generated/cupy.take_along_axis.html
Dask: (does not currently implement)

Issue: https://github.com/dask/dask/issues/3663
PR: https://github.com/dask/dask/pull/11076

PyTorch: https://pytorch.org/docs/stable/generated/torch.take_along_dim.html

Named take_along_dim, rather than take_along_axis

TensorFlow: https://www.tensorflow.org/api_docs/python/tf/experimental/numpy/take_along_axis
JAX: https://jax.readthedocs.io/en/latest/_autosummary/jax.numpy.take_along_axis.html

Supports additional mode and fill kwargs

Proposal
def take_along_axis(x: array, indices: array, /, axis: int) -> array

Questions

NumPy and its kin allow axis to be None in order to indicate that the input array x should be flattened prior to indexing. This allows consistency with NumPy's sort and argsort functions. However, the specification for sort and argsort does not support None (i.e., flattening). Accordingly, this RFC does not propose supporting axis=None. Are we okay with this?
NumPy and kin allow keyword and positional arguments. This RFC makes x and indices positional-only and allows axis to be both positional or keyword. Any concerns?
As elsewhere with this specification, presumably PyTorch will be okay aliasing take_along_dim as take_along_axis and dim as axis to ensure spec compliance?

Thanks @kgryte

Accordingly, this RFC does not propose supporting axis=None. Are we okay with this?

That makes perfect sense to me. The along_axis in the name would be a bit meaningless if the operation isn't happening along an axis in the end:)

This RFC makes x and indices positional-only and allows axis to be both positional or keyword. Any concerns?

Looks like a good choice to me.

presumably PyTorch will be okay

That's consistent with the rest of the design, so I don't see a problem here.

It's used quite rarely at least in SciPy and scikit-learn, and it's fairly new even in NumPy (1.15.0). It looks like there's enough justification for adding take_along_axis, but it'd be valuable to spell that justification out @kgryte, as we just discussed.

While take_along_axis may not be used on its own in SciPy, it is commonly used indirectly. For example, as @mdhaber mentioned offline, it is used in rankdata (see https://github.com/scipy/scipy/pull/20639), which in turn appears in a variety of important statistical tests, such as wilcoxon, mannwhitneyu, kendalltau, and various hypothesis tests. In which case, limiting our search to just direct usage may not paint the full picture.
Regardless, I've updated the OP with some additional justification. Namely, replicating take_along_axis with just fancy indexing is harder to get right and more verbose. IMO, there are definite ergonomic benefits to take_along_axis.


Keywords breaking highlighting in OMeta

Literals of certain words (found out that I can reproduce this bug with a rule such as keyword = ``interface'', ) mess up with large chunks of the grammar's syntax highlighting immediately after the keyword is used. As a curiosity (and workaround, too), know that you can use a commented opening curly brace (//{) in the lines preceding the broken syntax section to bring back colorization.
Tested in Visual Studio Code 1.8.1, PEGjs 0.1.0 and an OMetaJS .ojs grammar.

I am not fully sure what you mean... could you post some lines of code as an example showing the problem?

Sure I can : ).
1.- Messed syntax highlighting due (probably?) to said problem with the keyword.

2.- Same code showing properly, without the keyword.

3.- Commented braces workaround.

I'm pretty sure that bug occurs due to other keywords too (I'm suspicion of "class"), but I haven't checked out that too.
Hope that helps!

Hmm, I probably are making a logical mistake here, but for me it seems, that you are try to highlight a OMetaJS grammar... which is - as far as I can see - not related to peg.js in any way (though I don't know OMetaJS). This plugin is for highlighting peg.js grammars which are different.


v4.0.1 beta.1 - [merged]

In GitLab by @ltenorio on Jul 18, 2024, 11:52
Merges v4.0.1-beta.1 -> master

In GitLab by @ltenorio on Jul 18, 2024, 11:58
approved this merge request

In GitLab by @ltenorio on Jul 18, 2024, 11:56
added 1 commit
780a75a9 - add colors in change status modal #114
Compare with previous version

In GitLab by @ltenorio on Jul 18, 2024, 14:28
mentioned in commit df097f5090cf1e67f90a9dc228dd3e4a345606eb


[Storage] Return ledger consistency proof in update_to_latest_ledger

Motivation
Since the client passes in client_known_version, we will return a proof that shows the latest ledger is derived from the ledger at that version.
Have you read the Contributing Guidelines on pull requests?
.
Test Plan
Not much logic change in this diff.
Related PRs
None.

:umbrella: The latest upstream changes (presumably ce63efd5881f8a2f141a8fc2ca634134d7bac642) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra delegate
I think you'll need to resolve some minor merge conflicts, but otherwise looks good.
One thing - this definitely breaks wire protocol, is that a breaking change?

@zamsden if we add a field to a struct, the old client should still be able to parse the object?

@zamsden if we add a field to a struct, the old client should still be able to parse the object?

IIRC, that should be okay.

@bors-libra r=zamsden

:pushpin: Commit 1c71161 has been approved by zamsden

:hourglass: Testing commit 1c71161dc950035c56b0c606baaea4b84882855a with merge dcd5d48a72e897b518272dd7fc6d17e82de3efb1...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: zamsden
Pushing dcd5d48a72e897b518272dd7fc6d17e82de3efb1 to master...


Invalid credential error at LDAP authentication cause of case sensitive comparison of database contact_ldap_dn and LDAP contact info

Module version
centreon.noarch            19.04.20-4.el7.centos (after checking the concerned file it doesn't seem fix on the last version neither)
Operating System
CentOS 7
Browser used

[x] Google Chrome
[ ] Firefox
[ ] Internet Explorer IE11
[ ] Safari

Version: 111.0.5563.148
Additional environment details (AWS, VirtualBox, physical, etc.):
Virtual machine
Description
At LDAP authentication users may have "invalid credentials" error when the case of the contact_ldap_dn from LDAP is different from the Centreon database one.
On LDAP side this information is case insensitive so it can change at a LDAP configuration, migration or other maintenance step and it is causing authentication problem even if the user is the correct one.
Steps to Reproduce
To reproduce the issue :
--Log out from Centreon
--Change the user DN case on Centreon database table "contact", field "contact_ldap_dn"
Ex: Change CN=TEST,OU=FR,OU=TEST,o=test,C=FR to CN=TEST,OU=FR,OU=TEST,O=TEST,C=FR
--Try to login again > you should got "invalid credential" error
Describe the received result
"invalid credential" error
Describe the expected result
Be able to login
Logs
PHP error logs
"[17-Apr-2023 17:03:14 Europe/Paris] PHP Notice:  Undefined index: ldap_auto_sync in /usr/share/centreon/www/class/centreonLDAP.class.php on line 962"

### Additional relevant information (e.g. frequency, ...)

This bug can be fix by adding function strtolower to make the comparison case insensitive on the /www/class/centreonAuth.LDAP.class.php
--line 117 for 19.04 version:
$this->ldap->findUserDn($this->contactInfos['contact_alias']) !== $this->contactInfos['contact_ldap_dn'] replaced by strtolower($this->ldap->findUserDn($this->contactInfos['contact_alias'])) !== strtolower($this->contactInfos['contact_ldap_dn'])
--line 116 On the master branch:
$userDn !== $this->contactInfos['contact_ldap_dn'] replaced by strtolower($userDn) !== strtolower($this->contactInfos['contact_ldap_dn'])

Issue opened on the wrong repository. Closing it


how can I use ag like "cat test.txt | grep -f reg_file"?

If the file "test.txt" and "reg_file" are huge, I should use grep like "cat test.txt | grep -f reg_file".
But I cannot find a way to use ag like that in the mannual of ag. And obviously, it will takes lots of time if I use ag with a for-loop.
I think I am a careless user,so that I cannot find the correct parameter.Could you do me a favor?Thanks a lot.

unlike grep, ag doesn't support searching multiple regexes at once anyway, so an equivalent of grep's -f option doesn't make much sense.
Instead, join the regexes to search with | characters using a set of tr/sed/awk/perk/python commands of your choice, such as ag "$(tr '\n' '|' <patterns.txt | sed 's/|$//')" tosearch.txt
It ain't the prettiest, but at least there's only one ag invocation.


feat(PageHeader): Adding secondary sub title content option

Adding option to pass sub title content to Page Header

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


System information graph / window shows wrong graph value for network statistics

Brief description of your issue
Send and receive graphs heigh / scale magnitude are not same
Steps to reproduce (optional)
No response
Expected behavior (optional)
No response
Actual behavior (optional)
seems receive graph scale magnitude 2 times higher than send graph scale.
that makes receive data looks lower while actual values are close to each other.

Environment (optional)
No response

receive data looks lower while actual values are close to each other

The graph scaling is proportional:
https://github.com/winsiderss/systeminformer/assets/1306177/b637618b-b244-487f-84b6-9912fdda03bb

seems receive graph scale magnitude 2 times higher than send graph scale.

No because both values would be drawn at the same location and inaccurate activity/usage. For example:
Are both the upload and download values the same (90%) or is upload 90% and download 10%?

Are both the upload and download values the same (90%) or is upload 90% and download 10%?

Send and receive graphs heigh / scale magnitude are not same

You're requesting all graphs are displayed like the first image. If both values are identical then which graph is more accurate?

That would be good to make both graphs half transparent or special color for graphs overlay area then.
Or make one of the graphs without filling inside, just a line on top on top layer.
I was looking few minutes what is draining my data with such incredible speed 0.5GB/s.
Until I realize that scale is different for TX and RX.


[ERROR]When Running utils/s3dis_utils/s3dis_gen_h5.py

The following error will occur at the beginning of the program:
/root/autodl-tmp/JSNet/data/indoor3d_ins_seg_hdf5/Area_1_conferenceRoom_1.h5: (80, 4096, 9), (80, 4096), (80, 4096)
Traceback (most recent call last):
File "utils/s3dis_utils/s3dis_gen_h5.py", line 56, in
data_dtype, label_dtype)
File "/root/autodl-tmp/JSNet/utils/data_prep_util.py", line 111, in save_h5ins
h5_fout = h5py.File(h5_filename)
File "/root/miniconda3/envs/tf1/lib/python3.6/site-packages/h5py/_hl/files.py", line 427, in init
swmr=swmr)
File "/root/miniconda3/envs/tf1/lib/python3.6/site-packages/h5py/_hl/files.py", line 190, in make_fid
fid = h5f.open(name, flags, fapl=fapl)
File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
File "h5py/h5f.pyx", line 96, in h5py.h5f.open
OSError: Unable to open file (unable to open file: name = '/root/autodl-tmp/JSNet/data/indoor3d_ins_seg_hdf5/Area_1_conferenceRoom_1.h5', errno = 2, error message = 'No such file or directory', flags = 0, o_flags = 0)
Error performance:

room_filelist.txt only outputs the conferenceRoom of Aere1
missing area_1_conferenceRoom_1.h5 in indoor3d_ins_seg_hdf5

SOLUTION
After trying, I found that if I change h5py.file in the function save_h5ins to mode='a', it will run successfully
data_prep_util.py, line 111
h5_fout = h5py.File(h5_filename, mode='a')
END
Hoping that other beginners like me will not be troubled by this error.

Using ln to create a symbolic link and link to the data, see to readme file.

The following error will occur at the beginning of the program: /root/autodl-tmp/JSNet/data/indoor3d_ins_seg_hdf5/Area_1_conferenceRoom_1.h5: (80, 4096, 9), (80, 4096), (80, 4096) Traceback (most recent call last): File "utils/s3dis_utils/s3dis_gen_h5.py", line 56, in data_dtype, label_dtype) File "/root/autodl-tmp/JSNet/utils/data_prep_util.py", line 111, in save_h5ins h5_fout = h5py.File(h5_filename) File "/root/miniconda3/envs/tf1/lib/python3.6/site-packages/h5py/_hl/files.py", line 427, in init swmr=swmr) File "/root/miniconda3/envs/tf1/lib/python3.6/site-packages/h5py/_hl/files.py", line 190, in make_fid fid = h5f.open(name, flags, fapl=fapl) File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper File "h5py/h5f.pyx", line 96, in h5py.h5f.open OSError: Unable to open file (unable to open file: name = '/root/autodl-tmp/JSNet/data/indoor3d_ins_seg_hdf5/Area_1_conferenceRoom_1.h5', errno = 2, error message = 'No such file or directory', flags = 0, o_flags = 0)
Error performance:

room_filelist.txt only outputs the conferenceRoom of Aere1
missing area_1_conferenceRoom_1.h5 in indoor3d_ins_seg_hdf5

SOLUTION After trying, I found that if I change h5py.file in the function save_h5ins to mode='a', it will run successfully data_prep_util.py, line 111 h5_fout = h5py.File(h5_filename, mode='a')
END Hoping that other beginners like me will not be troubled by this error.

Can you leave me a contact number? I have a question for you


overflow problem when inside popup

Hi I just started to using this awesome component in my application. I noticed that component doesn't work correctly with modals, screenshot below

I would be very happy if you could fix this.

im having this problem as well inside a jquery ui dialog

my question is - can the popup appear at the top of the input box rather than the bottom?

i added a css so that it appears on top instead of the bottom, its not a real fix but it would to for now
.evo-pop {
    bottom: 0;
}

although it would only be useful if the color input box is at the bottom part of the dialog modal.

So it this fixed now?

Sorry, no, it's not fixed but I decided not to fix it.
It's not trivial and outside of the normal usage.

ok, thanks for that info need to find another library for this :(

You can modify the following function in evol.colorpicker.js source code.  You could try converting it a prototype function or extend without changing the source code.
  	_create: function() {
            ..............................
             // change the following code to pass the evt to show palette
	if(showOn==='both' || showOn==='button'){
				e.next().on('click', function(evt){
					evt.stopPropagation();
					that.showPalette(evt);
				});
			}
                     }
             }
              ....................

  showPalette: function (e) {
	if(this._enabled){
		this._active=true;
		$('.colorPicker').not('.'+this._id).colorpicker('hidePalette');
		if(this._palette===null){
		    this._palette = $('#YOUR_DIALOG_DIV_ID')
                .after(this._paletteHTML()).next()
                .css({ top: e.clientY + 'px', left: e.clientX + 'px', position: 'absolute' })
				.on('click',function(evt){
					evt.stopPropagation();
				});
			this._bindColors();
			var that=this;
			if(this._isPopup){
				$(document.body).on('click.'+that._id, function(evt){
					if(evt.target!=that.element.get(0)){
						that.hidePalette();
					}
				}).on('keyup.'+that._id, function(evt){
					if(evt.keyCode===27){
						that.hidePalette();
					}
				});
			}
		}
	}
	return this;
},

@Kenneth-Chan nice one, you should make a PR about this with extra option appendToBody: true which changes behaviour to this.


[hover style] The social media icons in the footer section lack a hover style.

What's the style idea?
The social media icons in the footer section lack a hover style.
Please assign me this issue, I will add this style to footer section social media icons .
@CSEdgeOff  please assign me this issue under GSSoC'24.
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSoC'24 contributor.

Code of Conduct

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

@Durgesh-Vaigandla Can you please assign me this issue?


Upgrade to OpenShift 4.15.x

Description
OpenShift 4.15.x has been officially announced, and rosa list versions shows that 4.15.2 is available.
Discussion
No response
Motivation
It would be good to upgrade to see if we have any regressions there.
Details
No response

Hi @ryanemerson - depending on the HCP efforts, this might be included or not included.
Maybe you want to make up your mind about this one, before we talk about it in the planning on Monday.


Convert IConfiguration use to Options pattern

This is better for refactoring (goodbye magic strings) and testability
(see TestOptions).

@BillWagner  I'm reviewing this one now.

@rynowak Can you address the merge conflicts, and we'll merge this in.

@MisterJames @BillWagner  updated


Patch Fix BlenderTools dev server option server.fs.deny can be bypassed when hosted on case-insensitive filesystem

Update 👾 Describe The Sumarry:
Affected of this project frontapp/front-chat-sdk are vulnerable to Access Control Bypass via the server.fs.deny option. An attacker can gain access to sensitive files by requesting raw filesystem paths using case-augmented versions of filenames. This is only exploitable if the server is hosted on a case-insensitive filesystem, including those used by Windows. This bypass is similar to CVE-2023-34092 with surface area reduced to hosts having case-insensitive filesystems.
Details
Since picomatch defaults to case-sensitive glob matching, but the file server doesn't discriminate; a blacklist bypass is possible. See picomatch usage, where nocase is defaulted to false:
 _setInternalServer(_server: ViteDevServer) {
      // Rebind internal the server variable so functions reference the user
      // server instance after a restart
      server = _server
    },
    _restartPromise: null,
    _importGlobMap: new Map(),
    _forceOptimizeOnRestart: false,
    _pendingRequests: new Map(),
    _fsDenyGlob: picomatch(config.server.fs.deny, { matchBase: true }),

PoCs By IAP ZeroDay:
npm run dev -- --host <IP_ADDRESS>
Created dummy secret files, e.g. custom.secret and production.pem
export default { server: { fs: { deny: ['.env', '.env.*', '*.{crt,pem}', 'custom.secret'] } } }

🥷 According CVeScores:
Users with exposed dev servers on environments with case-insensitive filesystems Files protected by server.fs.deny are both discoverable, and accessible
CVE-2024-23331
CWE-178
CWE-200
CWE-284
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:N/A:N

Hi @lamrecognitions ,
Thank you for opening this Pull Request. Just a heads-up that while we are open to Issues being submitted on this repository, we are not actively monitoring or merging Pull Requests at this time.
For the vulnerability flagged in this PR, it seems that it should be okay because we are not actively building or deploying front-chat-sdk anywhere, it is purely meant to be an example repository. If it is a more serious issue than I suspect, our team can look into pushing the recommended change.


error in last step

Error in quantile.default(x, seq(from = 0, to = 1, length = n)) :
'na.rm'如果设为FALSE的话不允许有遺漏值和NaN
Calls: Mclust ... mclustBIC -> qclass -> unique -> quantile -> quantile.default
停止执行

Could you please confirm if your input file contains only one sample? Additionally, to better understand the issue, could you provide the code you are using?


[Core] Subclass ModelRunner to support cross-attention & encoder sequences (towards eventual encoder/decoder model support)

This PR is a step towards encoder/decoder model support. This PR creates a specialized ModelRunner subclass for encoder/decoder models; it differs from the base ModelRunner class primarily in that it (1) expects each SequenceGroup to have an encoder sequence, and (2) it properly constructs the AttentionMetadata structure to support both self- and cross-attention.

A quick overview of the plan for supporting encoder/decoder models in vLLM:

Architectural assumptions:

The encoder/decoder model comprises one non-autoregressive encoder module and one autoregressive decoder module.
A single inference call to the model consumes an encoder prompt and a decoder prompt. The model output is the result of decoder inference against the decoder prompt, conditional on the encoder hidden states which result from applying the encoder to the encoder prompt. The encoder hidden states are not part of the overall model output
Thus, encoder inference is a prerequisite for decoder inference. The decoder consumes encoder hidden states via cross-attention, which is not present in decoder-only models.
It is assumed that these architectural details are handled inside the model definition; however, to support the inference process for such models, vLLM core must be changed to accommodate cross-attention

Encoder/decoder inference process & cross-attention:

Prefill phase: (1) Non-autoregressive encoder inference yields encoder hidden states in a single pass; no KV caching occurs. (2) decoder prefill yields first-token-prediction & cached KVs. Within the decoder, cross-attention layers cache the KVs derived from encoder hidden states:

Key_{cross-attn, layer-n} = W_{K, cross-attn, layer-n} x (Encoder hidden states)

Value_{cross-attn, layer-n} = W_{V, cross-attn, layer-n} x (Encoder hidden states)

Note that all cross-attention layers consume the same encoder hidden states; however each cross-attention layers' keys and values differ because each layer has unique W_{K, cross-attn, layer-n} and W_{V, cross-attn, layer-n}. Therefore, the cross-attention KV cache must store KVs for each decoder layer, even though these KVs are all derived from a single set of encoder hidden states.

Note that self-attention layer behavior is unchanged compared to what it would be in a decoder-only model (cache KVs computed from the previous decoder layer outputs.)

Decode phase: during each iteration of the autoregressive decode process,

Each self-attention layer appends the last predicted token's KVs to the KV cache, and then utilizes cached KVs for next-token prediction (again, this is unchanged compared to a decoder-only model)
Each cross-attention layer has read-only access to cross-attention KVs, to use for next-token prediction. The cross-attention KV cache is never modified after prefill

To implement the above encoder/decoder inference process, the following functionality will be added to vLLM over the course of multiple PRs:

Support cross-attention KV cache & memory management (allocate/swap/free) in block manager

See PR: https://github.com/vllm-project/vllm/pull/4837

Invoke cross-attention operation via the Attention wrapper & Attention metadata data structure

See PR: https://github.com/vllm-project/vllm/pull/4888

(This PR) Modify ModelRunner to construct input tensors & Attention metadata structures for cross-attention
Small changes to LLM engine & scheduler so that vLLM requests can include an encoder input prompt

Note 1: because this PR makes an incremental contribution (cross-attention KV-caching and memory management), this PR will not enable end-to-end encoder/decoder support (this will rely on later PRs.)
Note 2: the best effort is being made to ensure that encoder/decoder models are compatible with existing vLLM features. At this time, encoder/decoder models are unlikely to be compatible with the following vLLM features:

Speculative decoding
Chunked prefill
Automatic prefix caching
Sliding window
Flash attention
CUDA graph

INCREMENTAL FIX TOWARDS #187
BEFORE SUBMITTING, PLEASE READ THE CHECKLIST BELOW AND FILL IN THE DESCRIPTION ABOVE

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

/ready

/ready

Once again thanks @sroy745 @njhill @maxdebayser  @DarkLight1337 @WoosukKwon @khluu  @robertgshaw2-neuralmagic for all of your excellent help in landing this PR!

Usage example: examples/offline_inference_encoder_decoder.py
Correctness test against HF: tests/models/test_bart.py
Parallel test: tests/distributed/test_basic_distributed_correctness_enc_dec.py


JSON encoding fails on attr argument in alert modifier

When using the alert modifier with an isPresented:
alert(attr("title"), isPresented: attr("presented"), actions: :actions, message: :message)

Compilation fails with the following error:
** (ErlangError) Erlang error: {:unsupported_type, {:isPresented, {:__attr__, [file: "core_components.swiftui.ex", line: 1, module: "", source: "alert(attr(\"title\"), isPresented: attr(\"presented\"), actions: :actions, message: :message)"], "presented"}}}
        (stdlib 6.1.2) json.erl:199: :json.do_encode/2
        (stdlib 6.1.2) json.erl:244: :json.list_loop/2
        (stdlib 6.1.2) json.erl:241: :json.do_encode_list/2
        (stdlib 6.1.2) json.erl:244: :json.list_loop/2
        (stdlib 6.1.2) json.erl:244: :json.list_loop/2
        (stdlib 6.1.2) json.erl:241: :json.do_encode_list/2
        (stdlib 6.1.2) json.erl:241: :json.do_encode_list/2
        (stdlib 6.1.2) json.erl:257: :json."-do_encode_map/2-lc$^0/1-0-"/2

I don't think stylesheet compilation errors should ever prevent an app from compiling. Instead the styles should be ignored.

Can you pull down main on stylesheet and try again? I am not able to reproduce this locally.
Also, I agree with ignoring errors, however I want to wait until we get closer to 0.4 so that compilation issues are more in our face and we can quickly address them.

@carson-katri can you confirm if this is stil broken?


Adding post detail functionality and commenting

Minor changes of name replacements to ease the use of the 'Post' model
in the project [Post/migrations, Post/models.py][1]
Added a form file [commenting_system/forms.py][2] for the commenting functionality
Added a view function to [Post/views.py][3] to support comment
creation and saving to DB process (using registration only premission
and approved by adminn only)
Updating routing of Post app to be external to support access URL of
specific post by its id (serial auto generated number- exp. post #1
has id = 1) - change in [picATrip/urls.py][4]
Added [Post/templatetags][5] for supporting access to a Dictionary value
by key (since it's not provided by Django)
Added nice visability to comments tags
Used a 404 page redirection to prevent the site from crashing when
acessing a comment id page that doesn't exist

closes #63
post detail and adding a comment functionality
instructions:

Log in to system as a user (doesn't matter if super or not)
go to localhost:8000\postList
go to post # (replace the # with the post number you'd like to see)-
meaning : localhost:8000\postList# and behold
add comments as you desire
go to localhost:8000\admin and approve comments (you can choose
multiple comments and choode in the action dropdown menu- 'Approve
comments')
repeat until stage 2 (include) and see your comments with colorful
tags if chosen
Logout and try to add comments ad explained (it suppose to redirect
the user to login URL)
try to repeat the start and give post # that doesn't exist (it
suppose to redirect to a 404 error page)
Attached image of tags for illustrations:

Working on solving the fact I changed the migration initial files that already have been merged

I fetch the changes from your branch and I ran 'vagrant provision'.
I got many errors and I can't access our website.

@Shoval-Ovadia try removing your db.sqlite3 file before running vagrant up and it should work, I believe it's because the test_data file trying to add new users that are already in your DB and then it raises an exception due to the problem with the uniqueness of users in system (as demanded in django)

It is still not working for me.
Did you delete the template that I did?
This is what appears when I am trying to access 'localhost:8000/postList'  :


Bump up<EMAIL_ADDRESS>

Codecov Report

Merging #24 (dc14580) into master (b4bdd71) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head dc14580 differs from pull request most recent head 3fd8aad. Consider uploading reports for the commit 3fd8aad to get more accurate results

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   89.86%   89.86%
=======================================
  Files          18       18
  Lines         829      829
=======================================
  Hits          745      745
  Misses         65       65
  Partials       19       19

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 b4bdd71...3fd8aad. Read the comment docs.


[WIP] Zarr-Python 3 compatibility

[x] Closes https://github.com/zarr-developers/zarr-python/issues/1953
[x] Tests added / passed
[x] Passes pre-commit run --all-files

We have a few more things to fix in Zarr before merging this. I'll circle back once this is ready for a review.

@jrbourbeau -- this is just about ready to go. Any idea what is going on with the upstream test build?

Hmm looking now...

Alright, @jhamman merging main should fix CI

So close here. We're down to just one error:
ERROR dask/tests/test_distributed.py::test_zarr_distributed_roundtrip - Failed: 4 thread(s) were leaked from test

@jrbourbeau - do you have thoughts on how to debug this? This is a part of the Dask test suite I don't quite understand.

@martindurant - also pinging you here because I assume you have sorted something similar in the past w/ fsspec. Looking at the logs, I see 3-4 threads:

zarrIO - our global IO thread where we run our IO loop
asyncio_{1,2,...} - unclear where these are coming from.

zarr.core.common.to_thread creates a hidden ThreadPoolExecutor and stuff gets assigned to run there. If you wanted to maintain this pool, you would have to create and close it in order not to be caught by dask's thread checker.
It is used throughout LocalStore, by the way, which makes no sense.
It is also used for offloading en/decoding, which makes more sense, as these can involve considerable CPU. However, dask already has a pool running, so I would tentatively suggest that dask should take care of parallelism, and zarr should have a mode in which it never makes new threads.
There is even an argument that the ioloop should not be created in its thread since a (distributed) dask worker already has one in the main thread, but fsspec does this too. On a worker, you may well then have three asyncio loops running on different threads.

Heads-up, @jrbourbeau has been reassigned to baby-duty for the next month.  He's unlikely to be responsive here.  Your best bet may be @fjetter and colleagues.

zarr.core.common.to_thread creates a hidden ThreadPoolExecutor and stuff gets assigned to run there. If you wanted to maintain this pool, you would have to create and close it in order not to be caught by dask's thread checker.

Makes sense. I'm fairly confident that none of the threads here are from the ThreadPoolExecutor but I agree that we should probably be looking to manage this ourselves.

It is used throughout LocalStore, by the way, which makes no sense.

+1, I agree with this

There is even an argument that the ioloop should not be created in its thread since a (distributed) dask worker already has one in the main thread, but fsspec does this too. On a worker, you may well then have three asyncio loops running on different threads.

I'm curious there are ways to detect this and utilize an existing thread (or loop) in a general way.
Also, it just occurred to me that this test may also raise an error with fsspec but it is probably not exercised through the distributed test suite today. I think I can rig that up pretty easily...

A leaking thread is not a huge deal. For instance, we know that the offloading thread (I think Martin mentioned this earlier) is not cleaned up. We're initializing this on import time, see https://github.com/dask/distributed/blob/36020d6abe4506af08bae2807d78477b7916c8aa/distributed/utils_test.py#L112 such that the thread is already up and running before the test runs. So, if there is a way to initialize that hidden threadpool once before the tests run, that would be the cleanest solution

@fjetter and co -- this is ready for a real review. The leaked thread issue was mostly fixed upstream, now Zarr includes a cleanup utility for its internal threads.


Add onReady and onErr optional params

Users could potentially miss something without them (possible race condition).

This project is no longer being updated or maintained.


imagesLoaded + Masonry when there are no images

I'm using Masonry for a new project, and it's been awesome. The loading images were messing with the layout, so I added imagesLoaded (with progress()). However, occasionally the grid will load contents that have no images, and so Masonry never lays out the grid because images were never done loading. I could use a jQuery selector to check if there are no images, but that feels a bit heavy. What's the best way to still run Masonry even if there are no images to load?

imagesLoaded() still triggers always with 0 images. Checking for images before using imagesLoaded might be the best solution.

Thanks @desandro! Checking for images first now and everything's working brilliantly. 🙌


Create 2013-Detect-Squares.cs

Create 2013-Detect-Squares.cs
File(s) Modified: 2013-Detect-Squares.cs
Language(s) Used: csharp
Submission URL: https://leetcode.com/submissions/detail/812131566/

Thanks for contributing!


Summary/Question/Quotes should be Callouts

For the Book Summary: https://github.com/groepl/Obsidian-Templates/blob/main/Templates/Book Template.md
It should be a callout: https://help.obsidian.md/Editing+and+formatting/Callouts
Since this is an officially supported field:
Abstract
Aliases: summary, tldr

Similarly, the Question and Quote templates should use their appropriate callouts since this is default Obsidian functionality.

Great idea.
I've just changed "Summary" to a callout formatting.


Cannot run multiple topologies on a single LocalCluster object

Fixed for 0.7.0
The workaround for pre-0.7.0 is to use a LocalCluster object per topology.


Chart loading issue on html iframe

Hi,
I am trying to load chart and dashboard inside iframe.
Displaying an error inside iframe.
Here, one chart is loading and other charts getting error inside iframe.
Loading charts to anonymous user inside iframe issue:

Also showing cookie related issues:

Also used this SESSION_COOKIE_SAMESITE = "None" in superset/config.py.
But not worked.
But same url, at external window for anonymous user, all charts are displaying correctly:

can someone please help to resolve charts loading issue inside iframe.
Thanks

Do you have domain sharding enabled? if so, you may need to configure cross-site cookies for the other domains you shard to as well

Looks like this thread has an open question that's gone unanswered for over a year, so we'll call this one closed. Please update the context on the issue if you'd like to revisit/reopen the issue around a more current version of Superset. Thanks!


Deprecated: preg_match() Passing null in getClientIp()

I'm seeing this on PHP 8.1 using v4:
Deprecated: preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated
Coming from here (omnipay/sagepay/src/Message/DirectAuthorizeRequest.php in Omnipay\SagePay\Message\DirectAuthorizeRequest::getClientIp at line 138)
I'm not sure why the IP can be null at that point, but here we are.

Thank you. It looks like parent::getClientIp() is returning null, which is understandable if running tests and the tests don't explicitly set the IP address.
It needs to skip the regex check if the IP address is already null, or is not a string.

Strange thing is, this wasn't the result of running tests. This happened in production.

Just checked the core and I completely forgot that it is the application that supplies the IP address. I'm guessing your application isn't providing the client IP. A quick workaround would be to pass in anything that is not a valid IPv4 and not a null. An empty string would work.
$request->setClientIp('');

Just for some background, what the regex does, is just make sure the IP address is IPv4, and blanks it out if it isn't. Sage Pay would error if the IP wasn't in the a.b.c.d format. So it is not essential the IP address is sent (though for 3DS v2 maybe it is, but not at this point?)

Yeah, PR looks good. It was PHP 8.1 that has brought this deprecation notice - what you are [not] setting is good too. I was just trying to think of a quick workaround that would fix the problem for you.

Great. My confusion was that the other place I'm running this code is in fact PHP 7.4. So yeah, in PHP 8.1 this notice is a thing if not setting the IP.

I don't think I can assist with refactoring that. I believe a lot of the Server code extends the Direct code?

Thanks again - I've merged and made a patch release.


Add missing proto lock content

What changes are proposed in this pull request?
Add mission proto lock content.
Why are the changes needed?
#16578 Modified the proto file but not the proto.lock file.
Does this PR introduce any user facing changes?
No user facing changes.

alluxio-bot, merge this please


Fix tab change & reload page issue

References
Add references/links to any related issues or PRs. These may include:

Fixes #68 (if this fixes an issue ticket)

Description
Fix tab change & reload page issue
Instructions for Reviewers
Please add a more detailed description of the changes made by your PR. At a minimum, providing a bulleted list of changes in your PR is helpful to reviewers.
List of changes in this PR:

Fix Open a researcher profile → Choose the CRIS tab for Publications → Select a search filter → Switch to a different tab for Projects will display incorrect search data issue.
Fix Open a researcher profile → Choose the CRIS tab for Publications → Select a search filter → Reload webpage will display incorrect search data issue.

Include guidance for how to test or review your PR. This may include: steps to reproduce a bug, screenshots or description of a new feature, or reasons behind specific changes.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes ESLint validation using yarn lint
[x] My PR doesn't introduce circular dependencies (verified via yarn check-circ-deps)
[ ] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[ ] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[ ] If my PR includes new libraries/dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[ ] If my PR includes new features or configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

Hello @oye816
from which branch have you created your working branch related to this PR.
It seems you started from main-cris but this PR is against dspace-cris-2023_02_x. if so we can't merge it, could you please create a new branch starting from dspace-cris-2023_02_x with only the commits to fix the issue? thanks

Ok, I will create new PR from dspace-cris-2023_02_x


[DOC] correct plutus-playground.rst

According to the documentation, the 'Hello World' app should display ten Lovelace. The playground however shows us one hundred Lovelace. This is just a quick fix.

Thank you!


Bugfix - application validator test

Resolves #4639
Notes:

Tried switching to using the words method in faker. However, we'll need to get Travis back up (or switch to Git Actions) before finding out if it worked.

Well, this passed the tests the first time at least. Probably good to merge.


add clarity tracking

Add clarity tracking

Could you please make it configurable via config.yml, and disabled by default when no key? Otherwise the analytics will be distorted by the local development environments and preview/staging once we have them set up

Sorry Oleg I was too quick on the merge. I'll create a PR shortly that will add that.


Final versions of F5 approved README, CONTRIBUTING, SUPPORT and TERMS_OF_USE files

Once we have final approval of the required files for the meta files for the project we need to ensure that we have them in place and correct.  This must be done prior to opening the repository to the public.

This is a duplicate of #41


Added Loading Animation

https://github.com/TechNodes2-0/ElectiveHub/assets/101019545/104b043a-2726-481e-be22-e0e498821ece

Thanks You @adityakhattri21  for your Contributing look for such more good contribution from you ✨

I wanted to inquire about how will I get points in SLOP for this PR . That acts as a motivator 😅


Typo in nodes.md

Fixing abnormal new line char

Hi @dedronek,
please sign EasyCLA first, which will allow us to review your PR and accept the contribution.
You can find about EasyCLA here - https://docs.linuxfoundation.org/lfx/easycla
and the steps to sign EasyCLA are here - https://docs.linuxfoundation.org/v2/easycla/contributors/individual-contributor
You can start your signing EasyCLA from here - https://api.easycla.lfx.linuxfoundation.org/v2/repository-provider/github/sign/18706487/51478266/38056/#/?version=2

Thanks for this PR @dedronek,
related doc - https://kubernetes.io/docs/concepts/architecture/nodes/#non-graceful-node-shutdown

@dedronek : Please sign the CLA & consider reopening the PR whenever you're ready. We're closing the PR due to a lack of response from your end.
/close


[YUNIKORN-41] Update YuniKorn main README.md to make user can better get focus of the project

PR for that. Please review!

fixed by commit 351207c


[SPARK-15954][SQL] Disable loading test tables in Python tests

What changes were proposed in this pull request?
This patch introduces a flag to disable loading test tables in TestHiveSparkSession and disables that in Python. This fixes an issue in which python/run-tests would fail due to failure to load test tables.
Note that these test tables are not used outside of HiveCompatibilitySuite. In the long run we should probably decouple the loading of test tables from the test Hive setup.
How was this patch tested?
This is a test only change.

The diff is a lot smaller when ignoring whitespaces: https://github.com/apache/spark/pull/14005/files?w=1
Most of the changes are just some indentation change.

Test build #61579 has started for PR 14005 at commit e090304.

Great approach, I mentioned I had a similar approach available in the other PR https://github.com/apache/spark/pull/13737#issuecomment-229766815 to fix this (adding a flag to disable loading the tables) (although mine used a lazy val and only had the if around register) but looks functional equivalent.
Note: I ran the scala hive tests locally and they failed, I think that the default should be to load the tables. You can fix it here or I can push my version. But other than that LGTM pending tests passing.

ah yes default should definitely be true. let me fix that.

Test build #61579 has finished for PR 14005 at commit e090304.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61579/
Test FAILed.

Test build #61582 has started for PR 14005 at commit 5982972.

LGTM pending tests. I'll go ahead and close my original PR. cc @MLnick and @sameeragarwal .

Test build #61582 has finished for PR 14005 at commit 5982972.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61582/
Test PASSed.

Merging in master/2.0.


Redefine or Remove the User type alias

Goals
Please rename the User typealias. After running pod update, I needed to namespace our User class to remove the collision. If I'm using both RealmSwift and my own homegrown models, I shouldn't have to deal with obvious collisions here.
Expected Results
I should be able to use the User object in my own code. If I'd like to type alias it, should be able to.
Actual Results
Compilation errors when using RealmSwift and an object called User
Steps to Reproduce
N/A
Code Sample
N/A
Version of Realm and Tooling
Realm version: 2.0.0
Xcode version: 8.0 (8A218a)
iOS/OSX version: N/A
Dependency manager + version: CocoaPods 1.1.0.rc.2
Swift 3.0

To add to this, it's not so much that you're using a typealias, it's just that User is such a common name, probably used by many users of Realm.
So, when you do a typealias on something that so likely to be used by other people, it's going to be a pain point. Thanks for any reconsideration of this.

Sorry, not to belabor the point (but I guess I will), you didn't do the same for RLMSyncManager and RLMSyncSession. Those were just typealiased to SyncManager and SyncSession, respectively. Couldn't this just have been named SyncUser?

It's an oversight that User was not prefixed. We've been bitten by Swift's lack of namespacing a few times recently so it's unfortunate that we didn't catch this case.

Fixed in #4181.


sendMultipleTracking problems

Hello.
When I trying to use sendMultipleTracking from your example the code are hanging forever in
Racecore\GATracking\Client\Adapter\Socket@readConnection method.
When I use simple sendTracking it work correct. any ideas?

Did you use the example code 1:1? Or could you please post your current code which results in this error.

it's enough to get example code and change async flag.
async => false

Thanks @psychowico! There was an issue with reading the correct byte length. If you like to test the upcoming changes - replace this file temporary
https://github.com/ins0/google-measurement-php-client/blob/fix/26/src/Racecore/GATracking/Client/Adapter/Socket.php

it works quite random now.
when I run script few times, sometimes work properly, sometimes I got
Server closed connection unexpectedly.
I believe that when I send more that 2 tracking at once, it throw this error often.

Ok give it a second try - optimized some things

Hello there, I'm having the same problem with sendMultipleTracking() (without the mentioned fix in Socket.php) with v2.1.0.
I'm sending a total of 3 tracking - ecommerce transaction, ecommerce item and an event, but none got delivered. And sometimes, not always, I get the Server closed connection unexpectedly error.
After implementing the mentioned fix in Socket.php, it seems to work fine.
Will this fix be merged back to master or be included in a tagged version?

My bad, i will merge this as soon i'm back home! Sry!
Am Freitag, 18. März 2016 schrieb Catherine Koo :

Hello there, I'm having the same problem with sendMultipleTracking()
(without the mentioned fix in Socket.php) with v2.1.0.
I'm sending a total of 3 tracking - ecommerce transaction, ecommerce item
and an event, but none got delivered. And sometimes, not always, I get the Server
closed connection unexpectedly error.
After implementing the mentioned fix in Socket.php, it seems to work fine.
Will this fix be merged back to master or be included in a tagged version?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/ins0/google-measurement-php-client/issues/26#issuecomment-198515202

--

Marco Rieger
Flachsstraße 16
32369 Rahden
Germany
0171-5540818
https://racecore.de http://racecore.de/

@psychowico @catherinek merged - sry totaly forget this issue - ty for reminding me :+1:

thanks for you work!

Thank you! Much appreciated :+1:


font-weight on First row looks different

Minor CSS deal in material proj.

I have the same issue but i don't think it's due to CSS.
The first row is repeating twice (see capture : 2 rows and 3 elements).
.

Oh wow, I saw this when prototyping some perf improvements but thought it was just something I did!

Actually this is not just the first row.
When scrolling is involved, this happens to items beyond the initial view. This became more clear when I changed the css for cell-content

Also it seems that the rows are getting re-rended again and again on every scroll

Test latest, this is fixed

Thanks. That did fix the first row. But this is still happening to rows that are beyond initial visible scroll area.

Can you create a demo that shows that plz?

Will do
On Wed, Aug 19, 2015 at 1:12 PM, Austin<EMAIL_ADDRESS>wrote:

Can you create a demo that shows that plz?
—
Reply to this email directly or view it on GitHub
https://github.com/Swimlane/angular-data-table/issues/22#issuecomment-132698312
.

Code Demo for this bug. Scroll all the way to the bottom to see the issue.

Fixed in 0.3.9

Thank you. Although this is fixed only when scrollbarV is true. When it is false This is hapening.

I will try to get a demo

here is the demo pen

Retest in latest plz

Lastest version not working for me in Safari but works in Chromium (see capture for Safari bug).

This should be resolved too now.

This is still happening in the following situation

scrollbarV: false
dt element has no height
the container element has a height
no paging

rows outside the container scroll area are double rendered. see this demo pen

I see it, but its not due to same issue as before.  I wonder if its because of the css translate?


Show also how to autogenerate documentation from sources

Added a link in #123 which discusses this. Otherwise I think it's too specialized to go into it during the lesson.


Rework path management to be more explicit

Adds the configs folder to gitignore, and makes the path selection for the same more organized.
Potentially relevant to @emanuele3d @oniatus @eviltak

Hooray Jenkins reported success with all tests good!

I made the change here because it makes the most sense with this PR (oniatus wanted that change to go in a separate PR either way). So yeah, it would be great if you could revert.

For the configs thing in #3227 watch out for Git getting any funny ideas. If it is added here, then also added + deleted in that other PR, when it gets merged it may try to delete it instead. Rebase fixes but brings in its own hassle. Yay Git! :-)

Great point, @Cervator, but I do not think it should be a problem in this case as the change in #3227 is on a separate line. Nonetheless, to avoid any problems the best way out, in my opinion, is handling this through the merge conflict which will result on trying to merge #3227 after this PR.

Added in the merge on top of #3227 and manually resolved the one thing in the .gitignore - no technical conflict just had to remove one of the configs lines :-)

Yay!!! Congrats everybody! And in particular @eviltak! Thank you for sticking to it!


Data refresh transition

You could probably use chart.load to refresh data columns like I did in plnkr below. That would also allow transition animations.
http://plnkr.co/edit/2z9Quc94vTBGDelftlLx?p=preview
Thanks, for your simple and not overly complex solution. Especially using a service for passing javascript API was something I haven't seen before and will use that some day.

:+1: Looks great, thanks. I used your implementation that looks much cleaner.

Hi, I have added data update watching, so it will be handled (and animated) properly. Check out version 0.0.8 (particularly commit: 5259b19ab805a0895f110f795c28e497356b169e).


Centralize side effect management

Leave the side effect (lu/lg/qna) management into the caller side (form\visual) is an architecture is fragile and very error-prone architecture. In the past, we've fixed many issues, regressions on those operations. See blow of the list.
Letting shell take control the side effect management will lead us to a more stable, testable architecture which have a higher level guarantee of correctness of the application.
This would include a generic change detection mechanism in shell and a pub-sub mechanism to call certain actions under certain changes. This would also make the application more extensible for further customization.
Note: this ticket is a pure refactoring, only cover the centralization part, without creating any new v property.

FYI, some previous issue on areas that generate side effect.
undo/redo

undo / redo support for nodes with lu and lg https://github.com/microsoft/BotFramework-Composer/issues/2212

copy/paste

Cross-file copy of LU / LG resources https://github.com/microsoft/BotFramework-Composer/issues/2207
Unable to copy actions across dialog files https://github.com/microsoft/BotFramework-Composer/issues/2199

concurrency

Shell - debouncedUpdateLg cannot handle paralleled lg update requests https://github.com/microsoft/BotFramework-Composer/issues/2247

other entrance

Manually added json node doesn't have designer id and cannot edit lg text https://github.com/microsoft/BotFramework-Composer/issues/1776
Delete trigger should also delete LU intents and LG templates https://github.com/microsoft/BotFramework-Composer/issues/2209
Auto-generated Bot Response templates are not removed when entire dialog (trigger) is deleted https://github.com/microsoft/BotFramework-Composer/issues/2150
Multi nodes deletion should also delete the lg template https://github.com/microsoft/BotFramework-Composer/issues/2210

Close this for now, since it's out of date.


z_get_next_switch_handle() is not enough

This is an attempt at formally distinguishing and supporting the case
described in 40795 where an architecture doesn't preserve/restore the
complete thread state upon entering/exiting interrupt exception state.
The corresponding ARM64 changes are also included. RISC-V is concerned
too, but pending patches have to be merged first.

This seems reasonable.  I think if we're going to respecify the API, we might go a little farther though:

The name isn't great, it's really not just a predicate returning the next swtich handle, the scheduler uses this as the EOI hook where it knows it needs to set things up for the next context.

It seems like the z_ prefix for internal names is in less favor now, and this is clearly an architecture API.

We really don't want a third scheme in the API.  Let's fix all the architectures in one PR.

So maybe arch_isr_complete(), with the existing semantics but no switch handle argument, and arch_isr_context() which must be called at some point after that with a complete/valid switch handle?

Though thinking on it, the operation of arch_isr_context() is just to set the switch_handle field on the thread.  That's trivially inlined in C but the relevant code is almost certainly going to be in assembly on most architectures.  So really we'd just be specifying it as a field assignment anyway, maybe with a callable symbol just for clarity/documentation.

I think if we're going to respecify the API, we might go a little
farther though

Hmmm OK. I was under the impression that I had gone too far already.
More on that below.

The name isn't great

I totally agree.

It seems like the z_ prefix for internal names is in less favor now,
and this is clearly an architecture API.

Well, I thought arch_* is used for kernel common to architecture calls
whereas architecture to kernel common calls always uses z_*. Here we have
the later case.

We really don't want a third scheme in the API.  Let's fix all the
architectures in one PR.

Not sure I understand. What third scheme?

So maybe arch_isr_complete(), with the existing semantics but no
switch handle argument, and arch_isr_context() which must be called
at some point after that with a complete/valid switch handle?

Hmmpfff. I'd need an example to understand how this could work.

Though thinking on it, the operation of arch_isr_context() is just
to set the switch_handle field on the thread.  That's trivially inlined
in C but the relevant code is almost certainly going to be in assembly
on most architectures.  So really we'd just be specifying it as a field
assignment anyway, maybe with a callable symbol just for
clarity/documentation.

Well, when I said I might have gone too far above... It is about
the fact that we could simply formalize the definition of
z_get_next_switch_handle() where the passed handle value could
have 2 values. Either:

The interrupted thread's handle, in which case this function returns
the handle for the next thread to execute, which might be the same
handle as the one passed in. The thread context must be completely parked
when this is called.

A NULL indicating that we want to know what the next thread to run is
without parking the whole thread state just yet. If there is no other
thread to run then NULL is returned. It is the responsibility of the
caller to assign the k_thread_t.switch_handle of the interrupted thread
with its handle value once the thread context is fully parked.

The beauty of it all is that the above already just works without any change to
the code, even with !SMP. Suffice to formalize it.
And with the explicit field assignment we won't need that tripple-pointer
argument.
And yes, the assignment is most likely done in assembly. And I would avoid
a callable symbol as this forces you to preserve active registers across
the call when a straight forward offsetable store (one instruction) can
do the job.

I pushed the simpler version. Unless you convinced me otherwise,
I don't see why we need to make it more complicated.


Client Allocation Request for: Enjin

Client Allocation Questions
Core Information

Name: Enjin
Website / Social Media:  https://enjin.io/
Region: Asia (excl. Greater China)
DataCap Requested: 30TiB
Addresses to be Notarized: f14jgqqieillwuus45qsa5nyu5gy7xefkgctxj5lq
Notary Requested: IreneYoung

@faopngineg Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

@IreneYoung Hi could you take some time to check this request? Thx

Hi @faopngineg  - Thanks for your application. Please answer the following questions to let us know about you, your company, and further information about the DataCap allocation.
Basic Info
What's your name and your position in your company?
Please introduce yourself and your team briefly. How many team members do you have in your R&D or core team? Could you provide several Github accounts or email addresses of them?
For your organization, Please describe its business, its history since inception, registration country, scale, employee counts, operation regions, named executive officers, relevant projects, industry reputation, etc.
Please show us some of your web pages, Github, social media accounts such as Twitter, Facebook, Naver blog, WeChat official account, etc. Please include as many relevant details to provide contextual information (with external links) to show your Web3 reputation or external reputation.
Please provide your company's business registration license. If you have any presentation documents about your business, please provide them to us. And also could send me via email for privacy concerns. My contact<EMAIL_ADDRESS>How long have the team core members been engaged in development?
What types of data do you plan to store with the DataCap and how many do you need? Could you please send us some of your sample data?
Regarding the data you intend to store, What’s the original source of those data, and do you have any proof of the ownership or authorization?
Where is your data stored now? Whether the data is stored on a server or a cloud service provider?
Track Record
Has your organization obtained DataCap before? if so, please answer the following sub-questions.
1.1 Please share all the links and recordings of your applications before.
1.2 How about your DataCap usage? Is there any remaining? Is there any information on the deals you've done and the miners you've allocated your DataCap to?
1.3 What was your allocation strategy plan then and how did you execute it?
Are there any disputes open against you from your previous DataCap allocations? If any, please describe any of your honors or disputes in the field of Filecoin.
Allocation Plan
Please describe your plan to use the DataCap as detailed as possible, including use case, amount, timespan, geography, storage providers, partners, etc.
How many regions do you plan to store the data in?
How many storage providers you intend to make dealing with?
Do you plan to record your allocation decisions and execution details in public? And how to realize it?
Do you agree to use the DataCap to only store data that abide by local regulations and is in compliance with the recipient miner’s terms of service?
We may further ask some questions and then give an accurate score to the application. Thank you for your cooperation!

@faopngineg  The initial score of your DataCap request is 5.
30 TiB DataCap are being granted to you. Congratulate!
See more details here: https://docs.google.com/spreadsheets/d/1qOiteXrTKTrO1FCCR0TRkSjc8j-h6lk_AVUlXFAF96k/edit#gid=0

@IreneYoung Hi really appreciate your prompt reply and support! Many thanks!

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzacec7hypey72i2ozh46wuje2baob4vghbcefd7sh52hebgrra27emxk

Address

f14jgqqieillwuus45qsa5nyu5gy7xefkgctxj5lq

Datacap Allocated

30.00TiB

Signer Address

f1d4gmpqz3execjj2wvrxuuhvbms5mzh7t7yqrviq

You can check the status of the message here: https://filfox.info/en/message/bafy2bzacec7hypey72i2ozh46wuje2baob4vghbcefd7sh52hebgrra27emxk


:initial style is not applied during SSG build - Vitesse

Here's repro: https://github.com/Idered/vitesse-motion-ssg-bug

    Motion

is compiled to

Motion

but it should be

Motion

Hello @Idered :)
I am currently working on SSR support for @vueuse/motion!
Expect updates on this issue really soon!


[Snyk] Security upgrade com.fasterxml.jackson.core:jackson-databind from 2.12.3 to <IP_ADDRESS>

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

generated/src/main/resources/pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

712/1000  Why? Currently trending on Twitter, Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5.9
Denial of Service (DoS) SNYK-JAVA-COMFASTERXMLJACKSONCORE-3038426
com.fasterxml.jackson.core:jackson-databind:  2.12.3 -> <IP_ADDRESS>
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

Included in https://github.com/dockstore/swagger-java-zenodo-client/pull/14/files