eureka: improve duel support

config option for duels now plays only on CEs before duels
different coloring for duel CEs
cooldown timer for duels (and Castrum/Dalraida) shown on map

This should help some for folks chasing duels.

My ACT Eureka encountered an abnormal display problem after this update. After reading the commit history, I suspect that a change was introduced in this commit.

Before the update, if the weather for a NM does not arrive, it will show red color in the map.
However after the update, if the weather for a NM does not arrive and it didn't die during my exploring, it shows white color and the time is how long it is before we can trigger the NM.

I find it inconvenient because I need a second sight on the weather panel to distinguish whether the number means "Oh I need to wait XXX if I want to trigger NM" or "Oh we still have XXX time for triggering, hurry up".


feat(async) replace ngx.timer_at(0) and ngx.timer.every with kong.async

Summary
By default it creates one forever running timer per worker. Each timer by default starts 100 light threads. kong.async:run adds work to a jobs queue and posts to semaphore.
local PluginXHandler = {
  PRIORITY = 500,
  VERSION = "1.0.0",
}
function PluginXHandler:log(conf)
  kong.async:run(function(premature)
    if premature then
      return
    end
    ngx.log(ngx.ERR, "hello")
  end)

  kong.async:run(function(premature)
    if premature then
      return
    end
    ngx.log(ngx.ERR, "world")
  end)
end

In this example the plugin on log phase runs two async functions. It is not guaranteed that "hello" is logged before "world".
This is a drop in replacement to ngx.timer.at(0, func, arg1, arg2...) == kong.async:run(func, arg1, arg2...).

if creating buckets then everything behind a slow job in the same bucket will still be delayed.
I'd say create a single bucket/queue, and have all workers just grab the next item from that queue.

if creating buckets then everything behind a slow job in the same bucket will still be delayed.

No. Slowness means usually yielding, which means that other threads on that bucket can run. If it is blocking code, then we cannot do anything about it.

I'd say create a single bucket/queue, and have all workers just grab the next item from that queue.

This would mean that they also edit shared structures. A source of many bugs.

acme uses ngx.timer.every - should we change it?
zipkin is one that uses ngx.timer.at, so we need to check the plugins that are not in Kong/kong too
the EE plugins, a lot of them use timers as well

@Tieske I added another commit that just uses workers.

i like it. it's unfortunate that unpack() and table.remove() are NYIs, but in all any coroutine boundary is already a hard compile barrier.

I changed it to fixed size array, so no need for table.remove.

acme uses ngx.timer.every - should we change it?

This only deals with ngx.timer.at(0, ...) case. The recurring timers we need different solution.

zipkin is one that uses ngx.timer.at, so we need to check the plugins that are not in Kong/kong too

That is correct!

the EE plugins, a lot of them use timers as well

Yes, let's see how this goes. It might improve things somewhat, that is it exchanges scarce resources (timers) to light threads. Light threads are also lighter construct in general. This implementation can queue up to 100000 functions, until it starts returning errors (if the functions are not processed as fast as they are added).  It is more than what we have for timers:
lua_max_running_timers 4096;
lua_max_pending_timers 16384;

The thing that I do not know is that what would be good number of threads to start (currently it starts 100 per worker), also what would be good size of the queue (now it is fixed size of 100.000 per worker).

acme uses ngx.timer.every - should we change it?

There is now kong.async:every too, so we can use that. The thing is that this is implemented by just calling ngx.timer.every. The benefit is that it groups each delay to a bucket and tries to minimize the usage of timers this way. Of course if every kong.async:every(delay, func, ...) uses different delay, then there is no benefits, except one. The timer that this thing creates, does not execute the functions, but passes them to kong.run:async. This makes these every timers always lightning fast and non-yielding. It is an indirection, so it might add some additional latency, but that is usually not an issue with timer.every type of recursion.

superceded by #6552


GOROOT missing bin/go

Issue description
In order for some commands to work with the go packages in nix, you must set GOROOT. It must be set to ${pkgs.go}/share/go (correct me if I am wrong...?)
However, the go packages do not install the go binary to $GOROOT/bin/ . This causes tools which expect the go binary to be in $GOROOT to fail. (vscode-go is an example).
I propose a copy or symlink of the go binary in /nix/store/-go-/share/go/bin
PR incoming.
Steps to reproduce
$ nix-env -i go
$ ls /nix/store/-go-/share/go/bin
... no go binary ...
Technical details

System: 16.03.948.a96c308 (Emu)
Nix version: nix-env (Nix) 1.11.2
Nixpkgs version: 16.03.948.a96c308

(triage) status?

 I'll see if this is still a problem. Most likely it's a non-issue

Still an issue. For go 1.7, the change is simple (listed below and waiting to finish building/testing). For 1.6 and below the change is a little more complicated. We may just want go 1.6 and below to use the same preFixup, install, and configure phases.
diff --git a/pkgs/development/compilers/go/1.7.nix b/pkgs/development/compilers/go/1.7.nix
index 0f78646..4da9d4d 100644
--- a/pkgs/development/compilers/go/1.7.nix
+++ b/pkgs/development/compilers/go/1.7.nix
@@ -138,7 +138,7 @@ stdenv.mkDerivation rec {

   preFixup = ''
     rm -r $out/share/go/pkg/bootstrap
-    mv $out/share/go/bin $out/bin
+    ln $out/share/go/bin $out/bin
   '';

New PR for 1.7 #19413

What is the reason of using $out/share/go instead of simple $out? If you look at binary package provided from golang.org it looks simple like that.

@kamilchm GOROOT was always "$pwd/" and at e58b3517 this was added, effectively making GOROOT share/go . Can't say 'why' but it's been a convention since go 1.0.2.
+  # I'm not sure what go wants from its 'src', but the go installation manual
+  # describes an installation keeping the src.
+  preUnpack = ''
+    mkdir -p $out/share
+    cd $out/share
+  '';

To compare: archlinux has /usr/lib/go and /usr/share/go. GOROOT is /usr/lib/go and share/go is the merging of both of those.
Additionally, my system has a ~/.nix-profile/share/go. if we install directly to $out then ~/.nix-profile would get the folders in GOROOT . ~/.nix-profile and /var/run/current-system/sw seem to mimic standard unix folders and NOT putting GOROOT in share or lib might break that pattern.

PR for this was closed. GOROOT=${pkgs.go}, $GOROOT/bin/go now works for go 1.7


devcontainer.json

What's changing?
How's this tested?
Closes [related issues]

what files have changed? KAREN


Create dijit for navigating through the workflow

As a user of this widget I need to be able to intuitively navigate between the various tasks involved in the workflow.
The goal of this dijit is to allow the user to quickly navigate forward, backward or to a home location. Internally it will accept a collection of pages for each of the key parts of the workflow.

[ ] Leverage the 'jimu/dijit/ViewStack' dijit somewhat like the tab container does
[ ] Support navigation in a forward or backward direction
[ ] Support navigation to a defined "home" page

It should look like this:

@NikkiGolding @jessnoo

The way I have started this is leveraging the 'jimu/dijit/ViewStack' somewhat like the tab container does. It will allow for a modular design. Logical chunks of code and html will be created for the key parts of the workflow:

Start Page (for browse, drag drop, or manual input)
Location Type (for choosing between Address input or Coordinate input)
Addresses (for options associated with single or multi field address types)
Coordinates (for options associated with coordinate value input)
Field Mapping (for review or definition of the appropriate field mapping)
FeatureList (for reviewing the lists of features that are matched, un-matched, or potential duplicates )
Feature (for review of mapped field information or feature geometry edit)

Awhile back I wanted to ensure that I had not overlooked an existing dijit that would already handle this and reached out to the AWAB team. They confirmed that one does not currently exist and asked if we would add to the jimu library.

This should be good and I think new issues associated with proper flow should be logged seperately from this.

Verified- Enhancements logged to refine.


Add Claim Metrics/Status Support

What problem are you facing?
The new metrics framework currently does not include the ability to view metrics or status for claims. This limitation makes it challenging to monitor the health and usage of claims, which is an essential aspect of managing Crossplane resources effectively.
How could Crossplane help solve your problem?
When using x-metrics in the past there were several metrics we were able to key off of things such as Number of claims, Number of Synced MRs per Claim, Number of MRs by Control Plane and Number of Ready MRs per Claim.

Thanks for your input on this area of metrics that would be valuable for your team @mitchelldavis44 - it's something that has come up before so there is demand for it 😉
I've gone ahead and added this issue to the task list in https://github.com/crossplane/crossplane/issues/4620 for tracking purposes so it doesn't fall through the cracks.
Some links to previous related conversation:

request for claim level metrics: https://github.com/crossplane/crossplane/issues/4620#issuecomment-2475015550
dangers of high cardinality for individual claim metrics, but support for aggregate claim metrics: https://github.com/crossplane/crossplane-runtime/pull/683#discussion_r1550609074

it sounds like your request is for higher level aggregate metrics about claims though, as opposed to details for every single claim, which is discouraged in the discussion above ^


Add ability to change path for binder

Change from Kernel to Session to support the path argument.
This adds the ability to specify path in kernelOptions so the current working directory is changed.
Fixes #114.

Excellent, thanks!


Increment the "accessed" counter for secrets that have been revealed

This was blocked by #39, and should be working now ...

No ... output and debug while update is happening indicate everything's working as expected ... but checking afterwards shows that the counts have not been incremented.

Now it is :-D


Appintro show for first time

How to display Appintro for once , when Application first run?

https://github.com/apl-devs/AppIntro/wiki/How-to-Use#show-the-intro-once


linear_int4_kernel for XPU

Pure SYCL path for. int4 gemm
Benchmark results on PVC-1100, clpeak BW ~600GB/s. The remaining gaps are lack of usage of 2D load.

M
K
N
Dimension 3
SrcT
WeiT
DstT
Bandwidth (BW)

1
4096
4096
31
float16
float16
float16
354.24

1
4096
11008
11
float16
float16
float16
344.50

1
4096
16384
7
float16
float16
float16
394.02

1
12288
4096
10
float16
float16
float16
464.00

Besides PVC, the kernel can achieve
~88GB/s bandwidth on MTL (clpeak ~95GB/s)
~373GB/s bandwidth on A750 (clpeak ~440GB/s)

@liangan1 CC

@sunjiweiswift for the perf benchmarking, please include other configs expect M=1. This would serve as a reference of final decision making. I expect that big M would have worse perf, but that's fine, we still need to know the numbers.

@xytintel not only this PR but the latest several CI all failed, could you check?
2024-12-23T02:49:25.9617774Z /home/sdp/actions-runner-1/_work/torch-xpu-ops/pytorch/third_party/torch-xpu-ops/src/ATen/native/transformers/SDPUtils.cpp: In function ‘bool sdp::can_use_mem_efficient_attention(sdp::sdp_params, bool)’:
2024-12-23T02:49:25.9624103Z /home/sdp/actions-runner-1/_work/torch-xpu-ops/pytorch/third_party/torch-xpu-ops/src/ATen/native/transformers/SDPUtils.cpp:37:7: error: ‘array_of’ was not declared in this scope; did you mean ‘c10::array_of’?
2024-12-23T02:49:25.9625568Z    37 |       array_of<bool (*)(sdp_params const&, bool)>(
2024-12-23T02:49:25.9626012Z       |       ^~~~~~~~
2024-12-23T02:49:25.9626381Z       |       c10::array_of
2024-12-23T02:49:25.9627538Z In file included from /home/sdp/actions-runner-1/_work/torch-xpu-ops/pytorch/third_party/torch-xpu-ops/src/ATen/native/transformers/SDPUtils.cpp:2:
2024-12-23T02:49:25.9629144Z /home/sdp/actions-runner-1/_work/torch-xpu-ops/pytorch/c10/util/Array.h:14:23: note: ‘c10::array_of’ declared here
2024-12-23T02:49:25.9630059Z    14 | inline constexpr auto array_of(T&&... t) -> std::array<V, sizeof...(T)> {
2024-12-23T02:49:25.9630604Z       |                       ^~~~~~~~
2024-12-23T02:49:25.9631919Z /home/sdp/actions-runner-1/_work/torch-xpu-ops/pytorch/third_party/torch-xpu-ops/src/ATen/native/transformers/SDPUtils.cpp:37:16: error: expected primary-expression before ‘bool’
2024-12-23T02:49:25.9633177Z    37 |       array_of<bool (*)(sdp_params const&, bool)>(
2024-12-23T02:49:25.9633623Z       |                ^~~~
2024-12-23T02:49:25.9635453Z /home/sdp/actions-runner-1/_work/torch-xpu-ops/pytorch/third_party/torch-xpu-ops/src/ATen/native/transformers/SDPUtils.cpp:50:18: error: expected primary-expression before ‘bool’
2024-12-23T02:49:25.9636852Z    50 |         array_of<bool (*)(sdp_params const&, bool)>(
2024-12-23T02:49:25.9637304Z       |                  ^~~~
2024-12-23T02:49:25.9638691Z /home/sdp/actions-runner-1/_work/torch-xpu-ops/pytorch/third_party/torch-xpu-ops/src/ATen/native/transformers/SDPUtils.cpp:63:18: error: expected primary-expression before ‘bool’
2024-12-23T02:49:25.9639958Z    63 |         array_of<bool (*)(sdp_params const&, bool)>(


DocsArchive-1.0.20136.1Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/MicrosoftDocs/globalization-docs-archive/pull/5 is merged into live branch.
Auto archive content to https://github.com/MicrosoftDocs/globalization-docs-archive.git

Docs Build status updates of commit 16dabf6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

globalization/testing/the-role-of-test-tools.md
:white_check_mark:Succeeded
View

globalization/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

This PR was created to test archive feature and hence no need to merge.


onPrepare not being passed to tests

Yesterday I upgraded to v0.10.0 to take advantage of onPrepare goodness. Unfortunately I'm getting an error when trying to take advantage of it (sorry it's another log in test, but that's the easiest file to update). Selenium is running when these tests are started.
protractorConf.js
exports.config = {
  seleniumServerJar: './selenium/selenium-server-standalone-2.35.0.jar',
  seleniumPort: 4444,
  chromeDriver: './selenium/chromedriver',
  seleniumArgs: [],
  specs: [
    './.grunt/js/spec/login_spec.js'
  ],
  capabilities: {
    'browserName': 'chrome'
  },
  baseUrl: 'http://localhost:3000/web/agent',
  rootElement: 'body',
  onPrepare: function() {
    var ptor = protractor.getInstance();
    ptor.elem = ptor.findElement;
    ptor.elems = ptor.findElements;
    global.by = protractor.By;
    global.ptor = ptor;
  },
  jasmineNodeOpts: {
    onComplete: null,
    isVerbose: true,
    showColors: true,
    includeStackTrace: true,
    defaultTimeoutInterval: 30000
  }
};

login_spec.coffee
describe 'Logging in', ->
  # ptor

  # beforeEach ->
  #   ptor = protractor.getInstance()

  it 'sets URL to /web/agent', ->
    ptor.driver.get '/'
<EMAIL_ADDRESS>    ptor.driver.findElement(protractor.By.css('input[name="user_session[password]"]')).sendKeys('Testing123')
    ptor.driver.findElement(protractor.By.css('.btn-toolbar .btn')).click()
      .then () ->
        ptor.driver.getCurrentUrl()
      .then (url) ->
        expect(url).toContain '/web/agent'

Error
UnknownError: unknown error: unhandled inspector error: {"code":-32603,"message":"Cannot navigate to invalid URL"}

Other options I've tried but end in the same result:

bootstrapping Angular on the page and removing .driver
set baseUrl: "http://localhost:3000" and use ptor.get '/web/agent'

this.baseUrl still doesn't seem to override the one that was set earlier on

@btm1 Please open a new issue


Fix netperf docker image build failure

What this PR does / why we need it:
What

Update netperf container base from debian:jessie to debian:bullseye
Add CFLAGS=-fcommon to build of netperf binary

Why

The default image sirot/netperf-latest used by the netperf test crashes when given the -testFrom and -testTo parameters. These parameters cannot be turned off, but are implicitly set when omitted by user; introduced by kubernetes/perf-tests#2144.
debian:jessie container on docker hub is ~2 years old: apt repo keys are expired, thus preventing dependency installation and container build
After upgrade to debian:bullseye, with a newer compiler, netperf needs -fcommon flag for building (as workaround); see HewlettPackard/netperf#46

Relevance/Value
Updating the container is necessary to run the tests at all, fixing a regression introduced by the additionally passed parameters from kubernetes/perf-tests#2144
Which issue(s) this PR fixes:
NONE

Special notes for your reviewer:
NONE

What type of PR is this?
/kind bug
/kind regression

Could you take a look at it Dorde?
/assign @dlapcevic

Thanks @bitmeal.
/approve

/lgtm


chore: remove version from monorepo package.json

Changes proposed in this pull request

We don't want to publish the Rafiki packages on npm. Hence, the version is optional. I propose removing it as long as all runs in docker.
Context

Checklist

[ ] Related issues linked using fixes #number
[ ] Tests added/updated
[ ] Documentation added
[ ] Make sure that all checks pass
[ ] Bruno collection updated

We have a problem in that the admin UI displays the version and gets it from the package.json. Any ideas how else to get it?
Or should a release PR template just check that the version was increased?

@sabineschaller
Hmm, need to think this through, my first thought was something like pulling the git tag into and ENV variable that the admin UI could use?

I have removed the version from the home page now.


Add ip, ipv4 and ipv6 assertions

Fixes #44

AppVeyor is failed but it does not seem to be related with PR.

@Nyholm done!


Show failure descrition rather than name from Build Failure Analyzer

Currently, Build Monitor shows name of the failure from the Build Failure Analyzer. IMHO showing description would be more useful, since it uses placeholders which can be used to pass a message with more detailed info to the monitor.

Thanks for your contribution, @Witos!
@patrickdepinguin - what's your view on that?

@Witos @jan-molak  I think this best would be a configurable option: in my case the build monitor screen is relatively small and even with ~10 jobs we are already running against space issues such that we need to carefully design the job display names, hide culprits, ...
So in our case, showing the description would become too much, but I can see the value of the description for other users.

OK, will make it an option.

+1 for optional. I would go further and say the default should remain the name. When I added BFA support, I looked at what is presented for BFA elsewhere in Jenkins and decided the name field only has value if it is meant to tersely summarise the fail. This fits with the concept of a heads-up monitor.
We have full text in our descriptions that serves as instructions on how to resolve certain environmental issues. No way that is intended for the monitor.

+1 for this MR.
Do you have any plans to merge it in master?

I've just noticed BFA can now parameterise the Description field with regex capture elements from the log scraping. This makes the description much more powerful and maybe I would now like to use it as the primary display and move the "how to resolve" text into the comment section.
+1

Cool, sounds good


Upgrade build scripts to Albacore 2.0

Hi,
You've been users of albacore for a long time; I suggest you to upgrade to v2.0 and bring back this project to the fold. We're very open to pull requests and it would be great to have the scrutiny of another large popular .Net project that compiles for both Windows and Mono.

Can't build even with older version of  Albacore (1.0)
used this command for instlallation: gem install albacore --version "~>1.0.0"

... but the build tries to resolve Release|HPD configuration wihin VS and can't find this configuration.
When using the current Albacore 2.3 I can't even run the script:

We use nuget packages to include project dependencies but we are currently unable to use them because Nancy 0.2x series had a missing feature that successfully. Luckyly this feature arrived in 1.0 but Nancy is also unable to serve static files in the 1.0 release (worked in 0.2x).
So we have these options:

Try to find and solve the problem... then build the packge (=> currently the rake scripts not working so we have no packages)
Integrate Nancy as Lib without Nuget (=> don't like this solution, did that in the past)
Try to grab a latest package from other CI servers which can build the package including the fix for static files (where to get the "latest build" nuget packages?)
Build a workaround using OWIN for static files => some said it's working this way... but why, what about the super duper happy path ;-)
...

From major version 1.0 to version 2.0 the API is different, so you'd have to change the script first. See albacore's wiki, but I think there are already some progress towards using 2.0 in a PR somewhere by Daniel.
As for the first error, it's not an albacore error, but an MsBuild error. See what MsBuild command line is being invoked and fix that error.

I think there were some major pains to upgrade to v2.x so we've not really bothered (bigger fish to fry, not enough gain and things like that). We would ofc be happy to take a pull-request that performed the upgrade (we would in the long run have to update all other repos as well since we use RAKE for them as well)

is anyone tackling this? I'd love to take it on. I am a fairly big fan of Ruby and Albacore

@tparnell8 nope, based on what @thecodejunkie said, you're free to do it. :)

Just bear in mind that Nancy.Build has the repo with the main build script of doom in it 😊

Alright I'd be down for a challenge.

So I started working on this a little, but I keep getting side tracked by life, and other things. So I'm going to have to throw this back, because otherwise its going to take me another month or so before I will have a PR. Sorry everyone!

I guess this is invalid now, given #2484?

It is invalid. Closing.


only jdk1.8 or higher?

it  seems  tikv-client-lib-java  must 1.8

Yes. The client part of project uses lambda and some stream api only available on JDK 1.8 and newer.  And start from Spark 2.2, JDK 1.7 support is dropped. So we consider it not a big problem.
Are you using jdk 7 in your current spark cluster?

yes .we use  Cloudera Manager,All services run 1.7 now

For upgrading, here is a reference.
https://www.cloudera.com/documentation/enterprise/5-3-x/topics/cdh_cm_upgrading_to_jdk8.html
Or you might want to use a standalone cluster for test and decide if to use it and upgrade later.


pull request for e2e test at 12/12/2016 19:56:03

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


.get() Does not assign value to a variable

I want to save the returned value to a variable, but it returns undefined,
var mytoken
      this.storage.get('token').then((token)=>{
        mytoken = String(token)
     });

     console.log("mytoken: "+ mytoken)

in console i have mytoken: undefined
what am I doing wrong?
pd: i am noob :(
Your system information:

Cordova CLI: 6.5.0
Ionic Framework Version: 3.0.1
Ionic CLI Version: 2.2.2
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.3.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Linux 4.10
Node Version: v7.7.3
Xcode version: Not installed

I managed to make it work!
Instead of assigning the data to a variable, I call a function
this.storage.get('token').then((token)=>{
        this.getData(token);
     });

Overload the value and take the data:
this.storage.get('txt_key').then((val) => {
this.apikey = ('&' + val).substring(1, val.length+1);
});

I managed to make it work!
Instead of assigning the data to a variable, I call a function
this.storage.get('token').then((token)=>{
        this.getData(token);
     });

can u please elaborate :(
I am facing same issue but unable to figure it out

@harshm90 - what does your code look like?
The OPs code was this:
var mytoken;
this.storage.get('token').then((token)=>{
  mytoken = String(token)
});

console.log("mytoken: "+ mytoken)

But the problem there is simply a poor understanding of how promises work.
Obviously mytoken will be undefined where it is used in the console.log() since the callback given to the .then() will not execute until at least the next loop through the task queue, perhaps even later.

@harshm90 - what does your code look like?
The OPs code was this:
var mytoken
this.storage.get('token').then((token)=>{
  mytoken = String(token)
});

console.log("mytoken: "+ mytoken)

But the problem there is simply a poor understanding of how promises work.
Obviously mytoken will be undefined where it is used in the console.log() since the callback given to the .then() will not execute until at least the next loop through the task queue, perhaps even later.
This would make much more sense:
let mytoken;
this.storage.get('token').then((token)=>{
  mytoken = String(token);
  console.log("mytoken: "+ mytoken);
});

Or using async/await...

Method :- 1
getData() {
        return this.storage.get('products')
          .then(res => {
            return this.cart = res;
          });;
      }

This gives me output as Undefined

Out of context, it is hard to determine what is going on, but a few things you can try:
First, make sure storage is ready... This is rarely an issue, but it could be:
getData() {
  return this.storage.ready()
    .then(() =>
      this.storage.get('products')
      .then(res => {
        console.log('res:', res);
        return this.cart = res;
      })
    );
}

I also added a console.log() to examine the actual return just to see that the problem is actually there and not elsewhere.
Double check the set() call to make sure you are using exactly the same key. Better yet, create an enum of all of your keys and use that when calling these functions.
Make sure you have actually set something... If you are running this in a browser, open your dev tools and check whichever storage mechanism you happen to be using and see that the value you expect is actually set properly.
If all else fails, post a sample app that has the same issue and I will have a quick look. I use the storage module in a few apps and have never had an issue, so you likely just missed something. I would be happy to have a look at it to help.


Feature request: Disable or Delete a template

Issue in help/analyze/analysis-workspace/build-workspace-project/starter-projects.md
This page should explain how to remove a template from the list of custom templates. Currently only appears that you can create a template and share it. Unshare or remove template from broad distribution would be helpful.

I have added this to the templates documentation:
"These templates can be created, shared, or deleted by users with admin rights or by non-admins, provided they have been granted the "Analysis Workspace: Save as Template" permission in the Admin Console. Learn more...
Should be published later today.


@ConditionalOnEnabledFilter seems to be ignored

Describe the bug
I'm trying to annotate a @Bean in a @Configuration with @ConditionalOnEnabledFilter. However it seems to be completely ignored. My custom Condition works as expected on the same bean.
Sample
Works: (only instantiated if CustomCondition is met)
@Bean("TokenRequestClient")
@Conditional(CustomCondition.class)
 TokenRequestClient tokenRequestClient(@Value("${CLIENT_ID}") String clientId) {
           return new TokenRequestClient(clientId);
 }

Does not work:
@Bean("TokenRequestClient")
@ConditionalOnEnabledFilter(TokenGatewayFilterFactory.class)
 TokenRequestClient tokenRequestClient(@Value("${CLIENT_ID}") String clientId) {
           return new TokenRequestClient(clientId);
 }

My applycation.yml:
spring:
  cloud:
    gateway:
      routes:
        - id: test-route
          uri: http://localhost:${wiremock.server.port}
          predicates:
            - Path=/service/**
          filters:
            - LoggingPreFilter
            - LoggingPostFilter
            - Token

It seems to ignore if I set - Token in the application.yml or not and always instantiate the bean. Therefore my tests without - Token fail because they don't have CLIENT_ID set and should not instantiate the client.
I hope this description is good enough, it's my first time submitting a bug on Github :)

As your config :
...
      routes:
        - id: test-route
          uri: http://localhost:${wiremock.server.port}
          predicates:
            - Path=/service/**
          filters:
            - LoggingPreFilter
            - LoggingPostFilter
            - Token
...

That means add a seriles of filters(LoggingPreFilter,LoggingPostFilter and  Token) to test-route Route.
If you want to disable TokenGatewayFilterFactory bean (or other beans like TokenRequestClient), you should config property like this:
spring:
  cloud:
    gateway:
      filter:
        token:
          enabled: false

By the way, you can refer to the SCG source code：
org.springframework.cloud.gateway.config.GatewayAutoConfiguration.java:
	@Bean(name = PrincipalNameKeyResolver.BEAN_NAME)
	@ConditionalOnBean(RateLimiter.class)
	@ConditionalOnMissingBean(KeyResolver.class)
	@ConditionalOnEnabledFilter(RequestRateLimiterGatewayFilterFactory.class)
	public PrincipalNameKeyResolver principalNameKeyResolver() {
		return new PrincipalNameKeyResolver();
	}

	@Bean
	@ConditionalOnBean({ RateLimiter.class, KeyResolver.class })
	@ConditionalOnEnabledFilter
	public RequestRateLimiterGatewayFilterFactory requestRateLimiterGatewayFilterFactory(RateLimiter rateLimiter,
			KeyResolver resolver) {
		return new RequestRateLimiterGatewayFilterFactory(rateLimiter, resolver);
	}

So basically the @ConditionalOnEnabledFilter only listens to
spring:
  cloud:
    gateway:
      filter:
        token:
          enabled: false

and not whatever filters I have in the individual routes?
I think I even stumbled upon the source code at some point during my research but did not understand where the config comes from.
Thanks for explaining!!


Smaller text & Internal Links in Section Overview open in same tab

So I've made the text smaller and I think it looks alright but I encourage people to test it out on different browsers / OS.
https://github.com/user-attachments/assets/5ad6ae34-29b3-48ec-985f-b001d97cd31d

Where can I preview how it looks?


accept Ctrl-C in cri log viewer

Fixes: #1956
Signed-off-by: zhaojizhuang<EMAIL_ADDRESS>

cc @AkihiroSuda @djdongjin @fahedouch

updated @djdongjin

/test

CI failed retest it? @djdongjin

@zhaojizhuang sorry I'm a reviewer and don't have write permission to restart CI 😅. The PR will be merged by a committer who will restart the CI.

You can try close & reopen the PR to force CI rerun.

This method scares me.

updated @fahedouch


Update print-battleground-state-changes to cap hurdles at 100%

The hurdles will grow beyond 100%. Should we cap them there or at 1000%? Currently this should do it at 100%. Simple change. Sorry for being noob, this is my first pull request.

I can see you put some effort into this pull request, but we're really not interested in changing the way the hurdle rate is represented. I don't mean this to be discouraging, given that you proposed this and implemented the change in earnest. We discussed this internally and want to err on the side of presenting more information rather than less, even if that means numbers like the hurdle rate become (humorously) large.

Yeah I think more information is good. Maybe the cap should not be at 100%. But the information has to be capped at some point right? Otherwise it will blow up off the charts. You could set the cap at 1000% or 10000% or whatever you want. You can use my code to do that if you decide that's what you want to do in the future. Obviously all you'd need to change is the 1 to a 10 or 100 and change the wording to "1000%" or "10000%." I had fun making this PR, it's okay if it doesn't get used.


oh-my-zsh: 2016-09-24 -> 2016-10-08

Motivation for this change
Update to latest revision
Things done

[X] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

thanks!


Add top spacing to headings

The DS and census website both add margin-top in their own overriding style sheets - but we should probably just add default top padding/margins to headings used within the main container so content is correctly spaced by default.
See an example of govuk frontend 'contextual adjustments': https://github.com/alphagov/govuk-frontend/blob/48d1a5230b470a49888d6e14ef7cef550a280e47/package/govuk/core/_typography.scss#L157

Part of spacing work - not currently a priority

Relates to #1934. Included this ticket in the typography work required on the UCD Board: https://jira.ons.gov.uk/browse/CU-64.

This issue is currently parked on the UCD Board.


[#282] Hide and disable sponsorship application

This is a bit cleaned, i've put back the green button on the homepage and only remove it when we are deciding to disable the sponsorship application.
Also, by default everything is enable, an admin should disable it explicitly.

Hello! Thank you so much for your contribution to the PyCon website!
Every year, the site sees many small changes that build up to quickly overwhelm the limited resources spent on the codebase.
I'd encourage you to hold onto the ideas in this Pull Request and determine how they fit once again when the 2018 website has launched, don't worry as you'll be reminded here once that happens.


TypeScript: InferInput always infers intersect as 'never'

For any valibot schema that uses intersect, InferInput always infers intersect as type never
For the below code, the v.InferInput infers typeof eventSchema as never
This started in the first pre-release v0.31.0-rc.0 and still happens in the latest pre-release v0.31.0-rc.4
import * as v from 'valibot'

const eventSchema = v.intersect([
  v.pipe(
    v.object({
      name: v.pipe(
        v.string(),
        v.trim(),
        v.maxLength(128),
      ),
      startDate: v.date(),
      endDate: v.date(),
    }),
    v.check(({ startDate, endDate }) => startDate < endDate),
  ),
  v.variant('hasCelebrity', [
    v.object({
      hasCelebrity: v.literal(true),
      celebrity: v.literal('Fabian Hiller'),
    }),
    v.object({
      hasCelebrity: v.literal(false),
      celebrity: v.null_(),
    }),
  ]),
])

type EventSchemaT = v.InferInput<typeof eventSchema> // infers as 'never' (all the time)

Screenshot from VSCode Intellisense

Thanks for reporting this. I will fix it and release a new version.

v0.31.0-rc.5 is available

Many Thanks @fabian-hiller
You're a super star ⭐


Add a function to open a random Zettel

Adds a function to open a random zettel. I occasionally find this useful to explore my Zettelkasten.
I haven't mapped any keys — but happy to add gzr (that I use in my config) if that's acceptable.

@oberblastmeister does adding that sound good?

Sounds good


Add more details contrasting fromString and jsonParser

Description of the change
I recently found fromString when searching pursuit for String -> Json, and misunderstood how it's different from jsonParser. I assumed jsonParser just frontloads some error-checking that would otherwise be handled by decodeJson. This change adds some clarifying details.
Checklist:
Is changelog necessary for a documentation edit?

[ ] Added the change to the changelog's "Unreleased" section with a link to this PR and your username

Not applicaple

[ ] Linked any existing issues or proposals that this pull request should close
[ ] Updated or added relevant documentation in the README and/or documentation directory
[ ] Added a test for the contribution (if applicable)

Is changelog necessary for a documentation edit?

Not necessary, but welcome. Usually it'd go in the "Other improvements: " section under "Docs: ".


New feature flags for Fan platform

In HA 2024.8, new flags are being introduced for FanEntityFeature.TURN_ON and TURN_OFF.
As with the climate platform when these were introduced there, there is an ugly hack of setting an internal attribute when the change has been made to avoid poorly implemented warnings.
https://developers.home-assistant.io/blog/2024/07/19/fan-fanentityfeatures-turn-on_off

Amazing, thank you!
Unfortunately turning it off doesn't work without issue; its always displayed at 20% (even when off), and using the fan switch reports that the action fan/turn_off couldn't be run and is not supported by the device. Maybe another issue with #2243?

It's controlled with the speed settings 1 through 4, and 0 turns it off (in addition to the main off switch). That is "supporting it", I think?
Maybe there needs to be a mapping to speed 0 being the "turn off" function? It worked on the older HA / plugin version, unfortunately I'm not sure when exactly it stopped working.

Probably it is related to changes made around


Add Unique Booking List and Booking Class

simple skeleton, will add more methods if needed

can't believe checkstyle fails builds

can't believe checkstyle fails builds

ikr, btw i think we can use your UniqueBookingList, just need to move into the booking package instead of person, and looks like for Booking also need to change the attributes to individual classes instead of LocaleDateTime/String from what i see in Person


Support dpp built-in processors as processing steps

This pull request fixes #14

able to pass dpp processors with the same syntax that it would be defined in pipeline-spec.yaml
add pipeline_steps (if any) of required artefacts to run before they are streamed further

Coverage decreased (-1.06%) to 84.512% when pulling b878180f412104febe5847bfd615b5c07a5da67f on feature/run-built-in-processors-on-demand into 82f0fc7d7f2d4c536feae429bb1ee0c62406fb0a on master.


How to improve the mask-shift results

Hi authors, thanks for releasing this great work!
I am trying to use mask-shift technique on my image-to-image translation task in medical images. I trained my diffusion model with crop data augmentation. But I can’t get the smooth results like yours. Is there any way to improve it?
The following are the results of my task.
This is the generated result of the first patch.

Then, I shift the patch right and leave a half of overlapped regions with the previous patch.

This is the result of the step 2.

Hi, you may check these points:

Can your framework do normal-size inpainting tasks?
Is the masked region correctly placed?


driver-kafka - Waiting for consumers to be ready never ending

In WorkloadGenerator.java there is a infinite loop that is pretty dangerous.
private void ensureTopicsAreReady() throws IOException {
        log.info("Waiting for consumers to be ready");
        // This is work around the fact that there's no way to have a consumer ready in Kafka without first publishing
        // some message on the topic, which will then trigger the partitions assignment to the consumers

        int expectedMessages = workload.topics * workload.subscriptionsPerTopic;

        // In this case we just publish 1 message and then wait for consumers to receive the data
        worker.probeProducers();

	while (true) {
            CountersStats stats = worker.getCountersStats();

            if (stats.messagesReceived < expectedMessages) {
                try {
                    Thread.sleep(100);
                } catch (InterruptedException e) {
                    throw new RuntimeException(e);
                }
            } else {
                break;
            }
        }

        log.info("All consumers are ready");
    }

I have suffered this infinite loop because for whatever reason the consumer does not respond to the message sent, so the while loop never ends and the run is stuck there. I would suggest to add a timeout and continue in that case. What do you think?
I have tested this change:
long start = System.currentTimeMillis();
long end = start + 60 * 1000;
while (System.currentTimeMillis() < end) {
            CountersStats stats = worker.getCountersStats();

            if (stats.messagesReceived < expectedMessages) {
                try {
                    Thread.sleep(100);
                } catch (InterruptedException e) {
                    throw new RuntimeException(e);
                }
            } else {
                break;
            }
 }

and it works:
08:34:27.626 [main] INFO - Workers list - consumers: [http://<IP_ADDRESS>:8080, http://<IP_ADDRESS>:8080]
08:34:27.631 [main] INFO - --------------- WORKLOAD : 1 producer(s) / 1 consumers on 1 topic(s) --- DRIVER : Kafka---------------
08:34:30.437 [main] INFO - Created 1 topics in 439.890699 ms
08:34:30.585 [main] INFO - Created 1 consumers in 142.21151 ms
08:34:30.694 [main] INFO - Created 1 producers in 107.7444 ms
08:34:30.694 [main] INFO - Waiting for consumers to be ready
08:35:31.150 [main] INFO - All consumers are ready
08:35:31.195 [main] INFO - ----- Starting warm-up traffic (1m) ------
08:35:41.491 [main] INFO - Pub rate     4.9 msg/s /  0.0 MB/s | Cons rate     3.7 msg/s /  0.0 MB/s | Backlog:  0.0 K | Pub Latency (ms) avg: 53.9 - 50%: 35.6 - 99%: 375.2 - 99.9%: 375.2 - Max: 375.2 | Pub Delay Latency (us) avg: 145.6 - 50%: 147.0 - 99%: 195.0 - 99.9%: 195.0 - Max: 195.0

The topic is ready despite the loop is not ended as expected.

The change looks ok, though we should be printing that we timed out waiting, instead of saying "all consumers are ready"

Should I create a PR with the change?


[core] Fix the issue of table copy

Linked issue: close #2038

@miomiocat, could you add test case to verify this bugfix?

@miomiocat, could you add test case to verify this bugfix?

added, PTAL

See test failed in: https://github.com/miomiocat/incubator-paimon/tree/fix_path

See test failed in: https://github.com/miomiocat/incubator-paimon/tree/fix_path

Done. PTAL


BAMS & NPO to CL

We should start importing terms from BAMS into CL
see https://bams1.org/cells/list.php
ROBOT template is probably a way to go
Need to create a cycle between @tgbugs and @shawntanzk to handle this

Things to think about:

What does the thing we want to import look like?
Missing: annotations with citations of evidence
properties to be mapped
Relax equivalent classes? (NPO)
BAMS -> table needs to be mapped to owl file ID? but we should use table

Will set up a new repo to do mapping
@tgbugs will make an owl file to start working with.

Hi @tgbugs - shall we use this board to manage task and keep track of this project? https://github.com/orgs/obophenotype/projects/14/views/1
We can add tickets to this repo and use the board to manage this overall project :)

Thanks, @shawntanzk! I'll follow up with Tom.

After discussion with @tgbugs, I'll start a ROBOT table with BAMS cell types and annotate with the information I can gather from source. When that's ready, I'll bring it to a CL meeting for discussion. We will coordinate on NPO later in the spring.


Refactor: Improve migration world representation

During migration planning, we construct a local and remote representation of the world, in order to diff them and generate a migration plan.
The current code is at:
https://github.com/dojoengine/dojo/blob/c515e9e839c848e132506085be6eed67195cc758/crates/dojo-cli/src/migrate.rs#L150
It is pretty hacked together right now and can be improved by using the manifest generated by the build step.
https://github.com/dojoengine/dojo/blob/c515e9e839c848e132506085be6eed67195cc758/crates/dojo-lang/src/manifest.rs#L67
In particular, the component hashing logic can be moved to the build step:
https://github.com/dojoengine/dojo/blob/c515e9e839c848e132506085be6eed67195cc758/crates/dojo-lang/src/compiler.rs#L70
The manifest can then be extended to include a from_rpc method that generates a manifest based on the remote source.
Finally, we can introduce migration planning logic which takes a local + remote manifest and generates a strategy to transition the remote state to match the local.

Does this imply creating another manifest file specifically for remote? or contained both in a single file? eg. :
struct Manifest {
    local: LocalManifest,
    remote: RemoteManifest
}

We can have a single Manifest struct and create a remote and local instance. The local is loaded by parsing the manifest.json outputted during the build and the remote one is loaded from the chain using the rpc provider.
So we should implement a method on Manifest like
fn from_remote(rpc...) -> Self {

}


GitHub Actions Required workflows - GA

Summary
The importance of enforcing a set of quality standards to continuously deploy in a consistent and predictable way can’t be underestimated. Implementing these standards without the duplication of CI/CD configuration code is a challenge many organizations face today. How can workflows solve these problems and allow the ability to “push things down” or enforce requirements from organization down to its repositories?
We are building the controls that allow an organization to require a workflow file (or list of workflow files) to pass before code is merged into any of its repositories. Requiring a workflow to pass before merging will be available via Rulesets and will allow access to controls already available to branch rulesets, such as:

enforcement status
bypass rules
repository targeting

By requiring a workflow to pass before merging will allow organization admins to specify:

which workflow file to run from any of the organization’s repositories
a specific branch or tag for the workflow file
an exact commit sha to pin to the required workflow (optional)

Required workflows can help companies with the following use cases:

Correctness and Compliance: Ensure that all code meets an enterprise’s quality standards before merging.
DRY: Reducing duplication of CI/CD configuration code

🚢 This has shipped: https://github.blog/changelog/2023-10-11-requiring-workflows-with-repository-rules-is-generally-available/
Leaving open to track for GHES release!

🚢 This has shipped with GHES 3.12<EMAIL_ADDRESS>Closing as complete!


Stream with audio while microphone is being used

Hello, Is there a way to stream video and audio, while the microphone is already being used? Maybe streaming the system audio or be able to open the microphone again? (It is already open, by our video chat based in Tokbox)
Thanks!

Hi,
You can try use secondary microphone (Normally all device have 2 microphones). If you only have a microphone you can't open it 2 times you only can use a instance at time.
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/encoder/src/main/java/com/pedro/encoder/input/audio/MicrophoneManager.java#L52
Try replace "MediaRecorder.AudioSource.DEFAULT" with "MediaRecorder.AudioSource.CAMCORDER"
The other way is capture audio buffers from the other library and send it to my library (You need modify my library)


Default rules for scientific notation and decimal, with tests

We've decided that:

Decimal will always show all digits.  As it is an exact representation and limited to 29 digits (either big or small) this seems most appropriate.
Floating point scientific notation will use 1E+01 notation with a capital E to be consistent with Excel.
The point at which a floating point value switches to scientific notation will be platform dependent.  We will guarantee that 1e15 to 1e-4 will be decimal.

✅ No public API change.

✅ No public API change.

✅ No public API change.


11.6 Problem

Is there a way to make this work on 11.6? Is there something I'm missing? I wanted to help my friend out with this and I've hacked my 3DS before but after 11.6 my soundhax stopped working (they probably patched it...). Is there still a way to use this application? Please someone respond soon!

Get an NTR cartridge and install Luma3DS


Assets and routes

I try to bind my image src attribute and use a path that depends on the current locale:
<img :src="'~/assets/image-' + $i18n.locale + '.svg'">
However it does not work and the output is:
<img src="~/assets/image-en.svg">
Another issue is with video posters, my paths don't work. Any idea?
<video poster="~/assets/poster.png">...</video>

You cannot use "dynamic paths" that change at runtime because webpack evaluates them before.
An easy solution for this is to move the images in the static folder and link to them with :src= "'/image-' + $i18n.locale + '.svg'"


Doesn't work after playing 10-15 songs in a playlist

Experienced just now. Hard to debug/reproduce though.

We might need to watch player. And use mutation observers for performance.

This issue is fixed. Haven't seen happening after the latest release.


Final Individual Review

video: https://github.com/daniel-s-lee/dslee/assets/142445258/c5de1122-69d8-4190-af91-362cf0879210
College Board Requirements:

instructions for input from user actions that trigger events: functions for the user to input actions such as moving in certain directions using the arrow keys and taking user info of how long it takes for them to complete the level.

use of lists: list (array) used to represent the structure of the maze in each level

at least one procedure: the createMaze() function generates the maze based on the selected level. It iterates over the maze array of the selected level and creates the corresponding HTML elements to show the maze. Additionally, it sets the position of the rat and the food, initializes the start time for the game, and prepares the game environment for the player.

algorithm within the procedure that includes sequencing, selection, and iteration:

sequencing - The function executes a sequence of steps to generate the maze and initialize the game environment. (creating HTML elements to represent the maze structure, setting the position of the rat and the food, and initializing the start time for the game)
selection - Within the nested for loops used to iterate over the maze array, there's a conditional selection based on the value of each cell in the maze array. It checks whether the cell value is 0 (representing a wall), and if so, it adds a specific CSS class to the corresponding HTML element to style it as a wall.
iteration - The function iterates over the maze array using nested for loops to generate the maze dynamically based on the provided array for the selected level.

call to the procedure:

output: displays 'you win' once you reach the end and also displays the amount of time taken to finish the level.

4.7/5 I see your contribution to the project, I also like how you clearly follow the CPT guidelines. But I think you more focused on following the guidelines rather than engaging in creating your webpage, but you did well overall.


Allow building from source without downloading

This is prompted by #326, but I think it's a legitimate feature idea even if #326 was not getting in my way.
I would like to avoid downloading libsodium from scratch when building libsodium-sys. Of course this is possible using the SODIUM_LIB_DIR environment variable, and it's a relatively easy scenario for Windows because a pre-compiled lib is actually provided by the libsodium project.
However it seems that for Unix-like systems to avoid downloading the source code during the build, they would need to manually build libsodium separately to generate their own lib files, which is a bit more of a hassle. Especially for cross-platform projects that need to build on multiple operating systems.
It would be awfully nice to be able to specify an environment variable like SODIUM_SRC_DIR or something like that, and have the build script use that directory instead of downloading the code every time or forcing users to supply their own lib files. This would enable me to take advantage of Git submodules to retrieve the libsodium source, or a local mirror of the libsodium project, for example.
I might be able to submit a PR myself, if you think this is a good idea.

Ah. Should have checked the list of pending pull requests. #300 would address this.

Closing this as duplicate. Thanks


[#7654] Async support SDK

How about renaming the package name for the java SDK?

Maybe I will do it like below.
import com.navercorp.pinpoint.sdk.v1.concurrent.xxx;

I plan to use the delegate pattern as much as possible.
    public void test1() {

        ExecutorService executorService = TraceExecutors.wrapExecutorService(Executors.newSingleThreadExecutor());

        executorService.execute(TraceRunnable.wrap(new Runnable() {
            @Override
            public void run() {
                // action
            }
        }));
    }

However, ForkJoin class has a problem that cannot be delegation
    public class ForkJoinPool extends AbstractExecutorService {
              public <T> ForkJoinTask<T> submit(Callable<T> task) {
              }
    }

There is also a way to use Annotation. This does not require any SDK dependency.
// If the annotation class name is configurable, no SDK dependencies are required.
profiler.trace.asynctrace.classname=x.y.z.AsyncTrace

@AsyncTrace
public class MyRunnable implement Runnable {
     public void run() {
         // action
     }
}


before_create may not be called before create

I'm just looking at the code for create and noticed that if the object is valid it will have already been created when the before_create hook is called.
Would it make sense to change the first create to a new, then call before_create, then valid? and then save?

I'd like to add you to my professional network on LinkedIn.

Brian

Brian  Kierstead
Information Technology and Services Professional
Ontario, Canada
Confirm that you know Brian  Kierstead:
https://www.linkedin.com/e/s3n0cw-gui8q7et-45/isd/4773961256/3YOYdPgI/?hs=false&tok=1H9COMwiSJM4Y1
--
You are receiving Invitation to Connect emails. Click to unsubscribe:
http://www.linkedin.com/e/s3n0cw-gui8q7et-45/q3QKptP3H3Rt4HMY4e9LHzc6zffppF53Fsgi_d3W2lNypo008ccnmu3YtBfChuSULQgVq7Y6v4fJvU0AHPRVq7JG-ysh-uS6CKRGCz/goo/reply%2Bi-900800-5fb0d9042fe07df675de012af24f7086f0cf6ed1%40reply.github.com/20061/I1659929324_1/?hs=false&tok=1ppXaxw5aJM4Y1
(c) 2011 LinkedIn Corporation. 2029 Stierlin Ct, Mountain View, CA 94043, USA.

sure, why not..


Fix bugs with write and read FixedString

@long2ice I found bug with FixedString. Please review it and release)

@long2ice ping


Apple's Hypervisor Framework check

there is a way to check HAXM (-accel-check command) but nothing mentioned for Hypervisor.
is there anyway to be sure that macOS is using Hypervisor virtualization technology for android device?
and does it work with x86_64 or not (only x86) ?

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

ID: 4e37d642-2051-8d70-9968-dfb484122a6a
Version Independent ID: 178ea80c-2b99-0c44-eeb6-9db319439b33
Content: Hardware Acceleration for Emulator Performance (Hyper-V & HAXM) - Xamarin
Content Source: docs/android/get-started/installation/android-emulator/hardware-acceleration.md
Product: xamarin
Technology: xamarin-android
GitHub Login: @conceptdev
Microsoft Alias: crdun

@Goldenstarc There are two ways:

Use emulator -accel-check"or emulator-check accel which will indicate what hypervisor is installed and usable.

i.e.
HAXM:
accel:
0
HAXM version 7.5.1 (4) is installed and usable.
accel

Hypervisor.Framework:
accel:
6
HAXM is not installed on this machine
accel

How to double check? Start an emulator with the -verbose argument. There's a message indicating the hypervisor that is being used (if one).
emulator: CPU Acceleration: working
emulator: CPU Acceleration status: Hypervisor.Framework OS X Version 10.13

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


Fix ITKv5 Otsu compatibility

From the ITKv5 migration guide:

Otsu filters now return correct threshold (bin's maximum value instead of mid-point) by default. To keep old behavior, use filter->SetReturnBinMidpoint( true );. This change should be relevant only in tests.

If add the CMake flag ITKV4_COMPATIBILITY the old Otsu behavior will remain.


📝 Host read-the-docs for STGraph

Host the official read-the-docs website for STGraph. Have a basic site ready with the following pages

[ ] Home Page
[ ] Installation Page
[ ] Running a STGraph program

Currently successfully built and hosted at STGraph Docs. It is however under my personal readthedocs account. Might create one for bfGraph and host it there instead


Rename StarshipEngine to RemoteEngine, starship CLI to sf

This is the last change to be merged into the demo branch, before demo is merged into master.

One comment before merge: updating the CHANGELOG. :) As starship appeared in there a couple of times, it would explain earlier PR entries in there to the users.

One comment before merge: updating the CHANGELOG. :) As starship appeared in there a couple of times, it would explain earlier PR entries in there to the users.

Good catch. I'm thinking it makes sense to accumulate those entries into one entry, summarising the new RemoteEngine and related functionality, since this will be a single squash commit into master. We did a similar thing when we released the apps layer

Note: tests are failing due to an API change in the new release of pytest. I'm going to temporarily pin travis to the previous pytest version, and then the test suite can be upgraded in a subsequent PR (as it will be quite a lot of changes)


Experiment with debugging in V8

JavaScriptEngineSwitcher added support for V8 config in https://github.com/Taritsyn/JavaScriptEngineSwitcher/issues/8, including the ability to use the debugger. Experiment with what it would take to use this for debugging server-side rendering.
Reference: https://pandeysk.wordpress.com/2014/05/25/debugging-with-clearscript/

Is it possible to debug the JS code executed with V8 with VS?

Not at the moment. I'm not quite sure how to implement that :)

Better debugging support would be great. Since nobody's actively working on it though, I'm closing this old issue and tracking future work in #655


Output to System Log

It would be nice to have an option to output in device logs to be able to debug when the app is not attached to the debugger.
I know I could use a file but it's not as convenient as doing a simple device log reading.

Just an FYI, when I have a chance I'll add this option. Most likely for a version 2.0 (Swift 1.2).

Having an option on XCGConsoleLogDestination to tell it to use NSLog instead of print would be great. In most of the apps we build we like to have info/warn/error go to the device's log and then verbose/debug only enabled when we're running/debugging from xcode.

OK, thanks for the heads up.

FYI, Gilt Groupe recently published a project called CleanroomASL that provides a Swift API for writing to and reading from the Apple System Log.
You can use CleanroomASL to send messages to the Apple System Log so that they show up in the device console. (This is what NSLog() does that Swift's print() doesn't do, and it explains why the former shows up in the console and the latter does not.)
Note that things behave a bit differently between the simulator and the device. I spent some time banging my head against ASL and interacting with it from Swift, so if I can answer some questions, let me know.

I've added a 'XCGNSLogDestination' log destination that uses NSLog and will thus write to the console.

Awesome! Thanks!
Sent from my iPhone

On 12 Aug 2015, at 05:41, Dave Wood<EMAIL_ADDRESS>wrote:
Closed #33.
—
Reply to this email directly or view it on GitHub.


add explicit pick plan exists constraints for all picking related CSPs in pybullet

see https://github.com/tomsilver/multitask-personalization/pull/107


Bug fix: Prevent fully defined links name from being reformatted

Summary
Currently fully define markdown links which include ticks are being reformatted by
https://github.com/astral-sh/ruff/blob/861998612300520f2c18dbc7b8a6226300ceb508/crates/ruff_dev/src/generate_docs.rs#L105-L114
[`name`](link) -> [`name`][name](link)
For example: https://docs.astral.sh/ruff/rules/typed-argument-default-in-stub/
This PR excludes the open parentheses from the regex, so that these types of links won't be reformatted.
Test Plan
Extended the regression test.

Does the example in https://docs.astral.sh/ruff/rules/typed-argument-default-in-stub/ now render correctly? I think this hack exists because MkDocs wasn't respecting backticks within URLs.

It does work. Although looking again at #280 I'm not sure why.

Is it possible that something changed in MkDocs?

Wait actually this always worked even in https://github.com/Python-Markdown/markdown/issues/280

>>> markdown.markdown('[`hello`](world)')
'
hello
'

What doesn't work is if the ref is not in parantheses.

>>> markdown.markdown('[`hello`][]\n\n[`hello`]: world')
'
[hello][]
'

What doesn't work is empty references with code formatting, so the proposed fix is reasonable.

>>> markdown.markdown('[`hello`][]\n\n[`hello`]: world')
'
[hello][]
'

Oh right, I think the motivating case was specifically about reference links.


Refactor to split two controllers to handle klusterlet deployment and cleanup

Signed-off-by: zhujian<EMAIL_ADDRESS>

/assign @qiujian16 @zhiweiyin318

/hold

/approve

/lgtm

/hold cancel


⚠ Bofrost.lu has degraded performance

In 381f1e3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 986 ms

Resolved: Bofrost.lu performance has improved in a2817de after 9 minutes.


Date with year 1988 doesn't work correctly when setting year

Description of the Issue and Steps to Reproduce:
When I have a moment object of which the year is 1988 (e.g. moment("1988-02-10")) and I change the year (e.g. date.year(2017)). Then the day of month is always changed to the last day of the month.
Environment:
Chrome Version 61.0.3163.100 (Official Build) (64-bit) on MacOS Sierra Version 10.12.6.
Examples can be run on the javascript console on the momentjs homepage.
Some incorrect examples::
moment("1988-02-10").year(1999).toString() // => "Sun Feb 28 1999 00:00:00 GMT+0100"
moment("1988-09-25").year(2017).toString() // => "Sat Sep 30 2017 00:00:00 GMT+0200"

Some good working examples:
moment("1989-02-10").year(1999).toString() // => "Wed Feb 10 1999 00:00:00 GMT+0100"
moment("1987-09-25").year(2017).toString() // => "Mon Sep 25 2017 00:00:00 GMT+0200"

It happens with all leap years, not just 1988.

so the original use case was that we wanted to calculate the next birthday of the user. It yielded incorrect results

Dup #4258.  Will track there.  Thanks.

Actually, dup #4024 and fix pending in #4255.


Support png/base64 encoding optionally

Hello there,
Is it possible to make tmx-parser only require zlib optionally? Maybe make it a default to automatically require it but add a switch to avoid it. It adds much code to a feature that is optional on the TMX format (compression).
If you think it's a good idea I can try to write an appropriate merge request.
Thanks!

I think nobody cares about this any more. If somebody cares they can open a new issue.


Fix update-contributors handling of missing github_username

Addresses error seen in https://github.com/pyOpenSci/pyosMeta/actions/runs/9770061315/job/26970644921.

This now shows the warning:  UserWarning: No github_username found for automata.

likely to conflict with https://github.com/pyOpenSci/pyosMeta/pull/174 which also changes name handling

likely to conflict with https://github.com/pyOpenSci/pyosMeta/pull/174 which also changes name handling

I hadn't seen #174, closing!


Fixed name of app module (and link on metacpan)

There was a typo in the name of the module, which also led to a broken link in metacpan.

Thanks for the PR.


AutoEscapedFunctions: add get_submit_button()

Looks like this function correctly escapes everything for output and does not have a filter which could undo things.
Ref: https://developer.wordpress.org/reference/functions/get_submit_button/

Normally, yes. However, I could do something evil and pass bad $other_attributes like either of:
$other_attributes = '><script>alert("EVIL")</script><';

or
$other_attributes = array(
    '><script>alert("EVIL")</script><' => '',
);

This would be unlikely, but we wouldn't be able to guarantee where $other_attributes is coming from statically.

Cricky. You're right of course, though only when $other_attributes is passed as a string as for an array it's correctly escaped.
The above would result in broken HTML though, but I suppose it could be rewritten as onLoad="runEvilScript();", so you're point is valid (though chances are very small).

@jrfnl actually even when $other_attributes is passed as an array it is vulnerable because only the array values get escaped. The attribute names do not:
        foreach ( $other_attributes as $attribute => $value ) {
            $attributes .= $attribute . '="' . esc_attr( $value ) . '" '; // Trailing space is important
        }

So as I see it, a key of '><script>alert("EVIL")</script><' would get injected.

Closing this PR for now. I do think it would be a good idea for a Trac ticket to be opened for this function to be improved in Core, though I suspect it won't be accepted what with that it would break some plugins doing really funky (and very stupid) stuff.....


chore(docs): add Windows wmic information to README

Relates to #384

Issue
start-server-and-test uses the npm package<EMAIL_ADDRESS>with cross-platform support, to implement the shut down server functionality.
As shown in the ps-tree > README > Windows section, ps-tree makes use of the wmic utility on Windows. This is the "Windows Management Instrumentation command-line" utility executable, also referred to as WMIC (upper case).
WMIC has however been deprecated by Microsoft since Aug 2016 and is removed by default in Windows 11 24H2 and Windows Server 2025.
There is currently no higher version of ps-tree available which removes the dependency on WMIC. There are two associated open PRs https://github.com/indexzero/ps-tree/pull/59 and https://github.com/indexzero/ps-tree/pull/62 which have not received any maintainer feedback.
Change
Add information into the README > Install section for Windows 11 24H2 and Windows Server 2025.
References

Features Removed or Deprecated in Windows Server 2012 Aug 30, 2016

WMI providers

The WMI command-line tool (Wmic) is deprecated. Use PowerShell cmdlets instead.

Deprecated features for Windows client Nov 14, 2024

Windows Management Instrumentation command-line (WMIC) utility.
The WMIC utility is deprecated in Windows 10, version 21H1 and the 21H1 General Availability Channel release of Windows Server. This utility is superseded by Windows PowerShell for WMI. Note: This deprecation applies to only the command-line management utility. WMI itself isn't affected.
[Update - January 2024]: Currently, WMIC is a Feature on Demand (FoD) that's preinstalled by default in Windows 11, versions 23H2 and 22H2. In the next release of Windows, the WMIC FoD will be disabled by default.

WMI command line (WMIC) utility deprecation: Next steps

How to install WMIC Feature on Demand on Windows 11

At this time, I'm not aware of any usable workaround for the GitHub Actions runner image windows-2025, so the instructions in this PR are incomplete. See also https://github.com/bahmutov/start-server-and-test/issues/398


deployment_head stuck after reopen allocation

After reopen allocation deployment_head metric didn't update value(stuck on latest value before close allocation)
In db subgraph is healthy and latest_ethereum_block_number is equal to ethereum_chain_head_number
deployment,latest_ethereum_block_number,last_healthy_ethereum_block_number,health,synced from subgraphs.subgraph_deployment;                    deployment                   | latest_ethereum_block_number | last_healthy_ethereum_block_number | health  | synced ------------------------------------------------+------------------------------+------------------------------------+---------+--------  QmUghXvKf5cVjtayNNRHCd3RbHEwfbGBQ95s9vheJjN5hH |                     12780253 |                                    | healthy | t
This happened for multi subraphs on each reopen.
Index node restart helps.
Using graph-node:v0.23.1

Confirming that I also see the same behaviour, and restarting the index nodes seems to resolve the issue.

Thanks @SpontaneousOverthrow and @cryptovestor21 . @fordN is there any reason that allocations should impact the deployment head metric in graph-node?

Confirming I am seeing the same behaviour. Not sure if it was reallocations that broke deployment_head. In theory, a reallocation may cause indexer-agent to ensure the deployment is made, and maybe those interactions with the graph-node API are causing the metric to break? Happy to help debug

I confirm this issue regarding graph-node monitoring endpoint
Just to clarify. I am using GraphQL query for getting deployment statuses.
Here is the provided shema: https://github.com/graphprotocol/graph-node/blob/master/server/index-node/src/schema.graphql
Here is my Query:
req := graphql.NewRequest(`
		query {
			indexingStatuses {
				subgraph
				synced
				health
				entityCount
				node
				fatalError
				nonFatalErrors
				chains {
					network
					chainHeadBlock { number }
					earliestBlock { number }
					latestBlock { number }
					lastHealthyBlock { number }
				}
			}
		}
	`)
	```

Here is log messages:

```json
  "graph": {
    "indexing_statuses": {
      "QmNRkaVUwUQAwPwWgdQHYvw53A5gh3CP3giWnWQZdA2BTE": {
        "synced": true,
        "health": "failed",
        "fatal_error": {
          "message": ""
        },
        "chain_head_block": {
          "block_number": 13009543
        },
        "latest_block": {
          "block_number": 12730208
        }
      },

Is the deployment_head metric populated from  latestBlock field?
It might  be a root cause for the incorrect deployment_head value
I also worry if it is a more general graph-node syncing issue so the monitoring is just a symptom of the underlying problem


Dev

To fix tests failing, switch arguments passed to function around product then sum instead of sum then product.


How do I use the text concatenate and text as input to the sampler?

This keeps on giving me an error and Im pretty sure I am missing something simple.

Question 1 - Can i get some guidance here with an example or correction? It would be much appreciated, ty.
Question 2 - I want to have a text prompt that says a mouse {in the room | in grass | in a tree} and be able to reuse that so that the choice is "fixed" across the graph when it is referenced, and concatenate that into other prompts like {sunny day|late evening} etc..

Tensor.__contains__ only supports Tensor or scalar, but you passed in a .

File "E:\ComfyUI\execution.py", line 151, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "E:\ComfyUI\execution.py", line 81, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "E:\ComfyUI\execution.py", line 74, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "E:\ComfyUI\nodes.py", line 1207, in sample
return common_ksampler(model, seed, steps, cfg, sampler_name, scheduler, positive, negative, latent_image, denoise=denoise)
File "E:\ComfyUI\nodes.py", line 1177, in common_ksampler
samples = comfy.sample.sample(model, noise, steps, cfg, sampler_name, scheduler, positive, negative, latent_image,
File "E:\ComfyUI\custom_nodes\ComfyUI-Impact-Pack\modules\impact\hacky.py", line 22, in informative_sample
raise e
File "E:\ComfyUI\custom_nodes\ComfyUI-Impact-Pack\modules\impact\hacky.py", line 9, in informative_sample
return original_sample(*args, **kwargs)
File "E:\ComfyUI\comfy\sample.py", line 82, in sample
models, inference_memory = get_additional_models(positive, negative, model.model_dtype())
File "E:\ComfyUI\comfy\sample.py", line 56, in get_additional_models
control_nets = set(get_models_from_cond(positive, "control") + get_models_from_cond(negative, "control"))
File "E:\ComfyUI\comfy\sample.py", line 50, in get_models_from_cond
if model_type in c[1]:
File "C:\Users\rook2\AppData\Local\Programs\Python\Python310\lib\site-packages\torch\_tensor.py", line 999, in __contains__
raise RuntimeError(```

closed - solved by other issue https://github.com/comfyanonymous/ComfyUI/issues/1372


Release 1.0.3

Fixes https://github.com/webNeat/ctrl-keys/issues/8

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
201

Relevant Lines:
202

💛  - Coveralls


Add default parameter to select options.

Also enable the use of other default types (callable)!

Thank you! I like the concise implementation and that it also works for other  decorators. While running example.py multiple, I did encounter a small issue:
raise ValueError(
ValueError: Invalid `default` value passed. The value (`['Mangoes', 'Oranges', 'Pears', 'Raspberries']`) does not exist in the set of choices. Please make sure the default value is one of the available choices.

I'll go ahead and merge this and will make a quick fix for this.


Incorrect module resolution when running custom TS transformers

I ran into a problem when using custom TS transformers that add new import declarations to a source file programatically. The problem is that the newly added symbols won't be handled correctly by fuse-box:
Let's take the following code snippet:
import { flow } from "mobx";
flow();

If it was manually written, the transpiled code looks something like this:
var mobx_1 = require("mobx");
mobx_1.flow();

But if I generated the code with a transformer, here's what it ends up looking like:
var mobx_1 = require("mobx");
flow();

The latter fails at runtime with a flow is not defined error. For a minimal repro, please see: https://github.com/Gustorn/fuse-box-transformers-issue

@Gustorn FuseBox doesn't do anything special with it. It's all a default behavior. You should take a look at your transformer.
I am not really sure what you are trying to do there, but maybe auto import will help?

Indeed, the transformer was actually wrong, it just happened to work with bundlers that use ES2015 modules as their native format (webpack, rollup). Sorry for the false alarm!


If class doesnt exist, redistribute samples to others

Currently, if a class doesnt exist in a subject, samples for that one wont be extracted, so from that subject there will be less samples extracted.
Instead, just redistribute those samples to the other classes.
Easily done if I move:
`get_num_samples_per_cat_per_subj(...)'
in load_subj_and_get_samples(...)' right after the call of: sampling_type.logicDecidingAndGivingFinalSamplingMapsForEachCategory(...)'
Make it a function of sampling_type. Pass the sampling-maps to the function to check existence of classes. And re-assign internally if one class cant be sampled..
Change get_num_samples_per_cat_per_subj(...) to get argument:
n_samples_per_subj = n_samples_per_subep / n_subjects_for_subep

Solved in commit 6e670b4 , currently on dev branch


Update limitation about non-published s3 image

s3 image should be published, it' a bug, not an intentional limitation.
https://hub.docker.com/repository/docker/fluent/fluentd-kubernetes-daemonset/tags?page=1&ordering=last_updated&name=s3
amd64 image for s3 is published but arm64 image isn't published.
This is why the tag v1.16-debian-s3-1 isn't created. We need both amd64 image and arm64 image to generate multi arch image.

https://hub.docker.com/repository/docker/fluent/fluentd-kubernetes-daemonset/builds
s3 build settings don't exist.
We already reach the limitation of build config number, so some needed configuration seems dropped implicitly.
At least we need to revive s3 and syslog images, so we should drop some needless images.

Thanks, I've overlooked that situation.


Fixes issue with creating multiple workspaces with the same machine.

Fixes #2743

There are tests for the CM, it is quite testable, would be nice to add one for this case.

👍


feat: repository documentation and updates and asyncmy tests

Adds tests coverage for the repository contrib for the MySQL using the asyncmy.
PR Checklist

[x] Have you followed the guidelines in CONTRIBUTING.rst?
[x] Have you got 100% test coverage on new code?
[ ] Have you updated the prose documentation?
[ ] Have you updated the reference documentation?

Since these database backed tests are quite slow, can we maybe add a pytest marker to skip them?

can we run the DB tests only on main? or maybe using a schedule (github action cron basically).

Hmm. I think we generally do want to run those in our CI and locally (maybe optionally). It's not like they take ages to run (the asyncpg tests take 5 seconds once the container is set up). That's not too bad on its own, but if we start adding many other dbs, it will somewhat add up. For me, just having a marker to be able to skip them if I want would be good enough. For the CI, it definitely doesn't matter since we have steps that take significantly longer than our whole test suite.

can we run the DB tests only on main? or maybe using a schedule (github action cron basically).

Hmm. I think we generally do want to run those in our CI and locally (maybe optionally). It's not like they take ages to run (the asyncpg tests take 5 seconds once the container is set up). That's not too bad on its own, but if we start adding many other dbs, it will somewhat add up. For me, just having a marker to be able to skip them if I want would be good enough. For the CI, it definitely doesn't matter since we have steps that take significantly longer than our whole test suite.

ok

@cofin is this currently blocked by something?

No, it's not blocked by anything, and I need to wrap this up. Based on the last status, I need to add the Pytest marker and merge the latest changes.

Okay, cool, just checking in!

I've been having issues with the asyncmy library, and it looks like there's an existing issue that references the problem: https://github.com/long2ice/asyncmy/issues/39

I've excluded these mysql tests by default.  I can de the same with the asyncpg tests if that makes sense?

I've excluded these mysql tests by default.  I can de the same with the asyncpg tests if that makes sense?

I.e., exclude from the general CI tests?

I've excluded these mysql tests by default.  I can de the same with the asyncpg tests if that makes sense?

I.e., exclude from the general CI tests?

Correct, i've added -m 'not sqlalchemy_asyncmy' to the pytest arguments in pyproject.toml.
We may need to actually remove the asyncmy support if we can't get a resolution to the above issue though.

So, it looks like we have some issues with the mysql driver that may take a bit to resolve.  I'd like to merge this sooner rather than later.  I've left these mysql tests disabled and added the following features:

Basic repository docs.  A more comprehensive usage guide will be completed when the Service and DTO patterns have been merged
get_or_create optionally accepts a match_fields argument.  This lets you lookup a model using a subset of the kwargs you've provided.  If the remaining kwargs are different from the retrieved model's stored values, an update is performed.
Renamed base_select to statement.

If the remaining kwargs are different from the retrieved model's stored values, an update is performed.

I'm not so sure about this. Sounds like a rather unexpected side effect of a get_or_create method to also perform updates. Maybe this could be its own method with a distinct name?


Services, PVCs and ingresses

We need to enable the creation of services, pvcs and ingress resources related to an influx, kapacitor or telegraf deployment.
When the services are available we can even populate deployment variables from the spec to allow interconnection of components via resource creation.

We need to add secrets support too and ConfigMap for confgurations

Yea, thanks for pointing that out


Change txlist to only incrementally call getTransactions with larger indices

Before this change getTransactions() would be called with the full range from index 0 to the full range necessary for scrolling. Now it only asks for the range of transactions it doesn't already have.
This required moving a lot of logic that was in the TransactionList component into the action.js file. This includes state of the tx list and validation on what has changed.

Merge after v1.1.3 is merged to master


Move "smarts" from get_filterset_parameter_form_field to multivalue_field_factory

Followup to #2348
Proposed Changes
Currently multivalue_field_factory() is used to generate and attach form field classes to various filter types in a fairly simple/naive way. We then (after #2348) have get_filterset_parameter_form_field() which takes these form fields and modifies them in various ways to make them more suitable for filter form usage. We should evaluate whether it makes sense to combine these two functions in some way for simplicity and reusability.

Duplicated by #2782.


Avisynth input?

Is it possible to use an Avisynth script for input? FFmpeg can decode them, but I'm not sure if raw data can be input directly for analysis or if the module always expects an actual video clip. I'd like to use PySceneDetect for replacing scene change detection by ffmpeg, which is quite flaky it seems.

I did a quick test, and you can use the MoviePy backend to do this, as it uses the ffmpeg executable on your system for this purpose.  Right now it will print some warnings about being unable to determine aspect ratio, but these can be ignored for most purposes.  This requires you to have MoviePy installed on your system as well as the latest version of PySceneDetect.
This option isn't documented right now since it's under development, but you can try it out as follows:
scenedetect -i script.avs -b moviepy detect-content

Aside from the warnings, it seems to work well with the save-images and split-video commands.  Feel free to try it out and let me know how it works out for you, and if you run into any other issues.

Thanks for the information!
I did a quick test as well, and got it running. I got a couple of other errors but at least it's scanning right now. In fact that first line is the same error I got when trying to use the API with Avisynth script as video source.
<EMAIL_ADDRESS>global cap_ffmpeg_impl.hpp:1237 open Could not find decoder for codec_id=192
<EMAIL_ADDRESS>global cap_ffmpeg_impl.hpp:1286 open VIDEOIO/FFMPEG: Failed to initialize VideoCapture
<EMAIL_ADDRESS>global cap.cpp:166 cv::VideoCapture::open VIDEOIO(CV_IMAGES): raised OpenCV exception:

OpenCV(4.8.0) D:\a\opencv-python\opencv-python\opencv\modules\videoio\src\cap_images.cpp:253: error: (-5:Bad argument) CAP_IMAGES: can't find starting number (in the name of file): f:\temp\captures\withnail.avs in function 'cv::icvExtractPattern'

Pixel aspect ratio is queried using OpenCV when the video loads since MoviePy doesn't support it yet, so the same messages will show up currently.  In the next release I'll make it so this is only done when required (for save-images).  I'll also look if suppressing these warnings is feasible so a better error message can be printed:

[ ] Calculate aspect ratio on demand
[ ] Suppress OpenCV warnings when this happens

Thanks for the report!

One more question regarding the detection: does detect-adaptive do a 2-pass process internally or is it supposed to scan the source twice? I tried it shortly and it (visibly) ran one pass and then printed out the scene change list. This test source I'm using has some luma flicker between frames and triggers false detections easily so I thought the adaptive method might be a best general option to use.

I guess that the automatic calculation of kernel-size is also not working, seems to be None in the debug output.

detect-adaptive used to be a two-pass algorithm, but I made it a single pass in v0.6.2.  On the command line, the final scene list is always printed at the end once processing is complete.  Regarding kernel size, None means kernel size will be calculated automatically.  I should probably update the API to allow disabling it however.

I possibly found a bug while testing my script.
scenedetect -i f:\temp\captures\malone.avs -b moviepy -o c:\temp detect-adaptive -t 2 -m 64 list-scenes -f test.csv creates a 28 frames long scene as the first one.
Timecode List:,00:00:01.168,00:00:09.051,00:00:27.110,00:00:51.635,00:00:58.392,00:01:59.786,00:02:23.268,00:02:28.940,00:02:34.988,00:02:40.577,00:02:47.125,00:04:38.653,00:05:04.095,00:05:08.725,00:05:17.943,00:05:30.121,00:05:37.295,00:05:43.593,00:05:46.346,00:05:49.224,00:06:03.113,00:06:05.865,00:06:09.369,00:06:16.585,00:06:23.800,00:06:33.560,00:06:38.440,00:06:41.776,00:06:45.697,00:06:48.950 Scene Number,Start Frame,Start Timecode,Start Time (seconds),End Frame,End Timecode,End Time (seconds),Length (frames),Length (timecode),Length (seconds) 1,1,00:00:00.000,0.000,28,00:00:01.168,1.168,28,00:00:01.168,1.168 2,29,00:00:01.168,1.168,217,00:00:09.051,9.051,189,00:00:07.883,7.883 3,218,00:00:09.051,9.051,650,00:00:27.110,27.110,433,00:00:18.060,18.060 4,651,00:00:27.110,27.110,1238,00:00:51.635,51.635,588,00:00:24.524,24.524 5,1239,00:00:51.635,51.635,1400,00:00:58.392,58.392,162,00:00:06.757,6.757 6,1401,00:00:58.392,58.392,2872,00:01:59.786,119.786,1472,00:01:01.395,61.395 7,2873,00:01:59.786,119.786,3435,00:02:23.268,143.268,563,00:00:23.482,23.482 8,3436,00:02:23.268,143.268,3571,00:02:28.940,148.940,136,00:00:05.672,5.672 9,3572,00:02:28.940,148.940,3716,00:02:34.988,154.988,145,00:00:06.048,6.048 10,3717,00:02:34.988,154.988,3850,00:02:40.577,160.577,134,00:00:05.589,5.589 11,3851,00:02:40.577,160.577,4007,00:02:47.125,167.125,157,00:00:06.548,6.548 12,4008,00:02:47.125,167.125,6681,00:04:38.653,278.653,2674,00:01:51.528,111.528 13,6682,00:04:38.653,278.653,7291,00:05:04.095,304.095,610,00:00:25.442,25.442 14,7292,00:05:04.095,304.095,7402,00:05:08.725,308.725,111,00:00:04.630,4.630 15,7403,00:05:08.725,308.725,7623,00:05:17.943,317.943,221,00:00:09.218,9.218 16,7624,00:05:17.943,317.943,7915,00:05:30.121,330.121,292,00:00:12.179,12.179 17,7916,00:05:30.121,330.121,8087,00:05:37.295,337.295,172,00:00:07.174,7.174 18,8088,00:05:37.295,337.295,8238,00:05:43.593,343.593,151,00:00:06.298,6.298 19,8239,00:05:43.593,343.593,8304,00:05:46.346,346.346,66,00:00:02.753,2.753 20,8305,00:05:46.346,346.346,8373,00:05:49.224,349.224,69,00:00:02.878,2.878 21,8374,00:05:49.224,349.224,8706,00:06:03.113,363.113,333,00:00:13.889,13.889 22,8707,00:06:03.113,363.113,8772,00:06:05.865,365.865,66,00:00:02.753,2.753 23,8773,00:06:05.865,365.865,8856,00:06:09.369,369.369,84,00:00:03.503,3.503 24,8857,00:06:09.369,369.369,9029,00:06:16.585,376.585,173,00:00:07.216,7.216 25,9030,00:06:16.585,376.585,9202,00:06:23.800,383.800,173,00:00:07.216,7.216 26,9203,00:06:23.800,383.800,9436,00:06:33.560,393.560,234,00:00:09.760,9.760 27,9437,00:06:33.560,393.560,9553,00:06:38.440,398.440,117,00:00:04.880,4.880 28,9554,00:06:38.440,398.440,9633,00:06:41.776,401.776,80,00:00:03.337,3.337 29,9634,00:06:41.776,401.776,9727,00:06:45.697,405.697,94,00:00:03.921,3.921 30,9728,00:06:45.697,405.697,9805,00:06:48.950,408.950,78,00:00:03.253,3.253 31,9806,00:06:48.950,408.950,10000,00:06:57.083,417.083,195,00:00:08.133,8.133
It might also be better to use 0 as the first frame since that is probably the most common way.


Not working on 3.7.2

It seems that the script isn't working on sketch 3.7.2. I've made all the changes as I usually do (with previous versions), but neither the text or the screenshots were changes on step 5. Of course when I try to export I get all the images with the placeholders.

me too

me too

Turns out you just have to hit the refresh button (2 bended arrows) on the right (next to the element name) after you change an image so that the changes are made.


Update package-lock.json

Change due to security failure(Changed Hoek version from 4.2.0 to 5.0.3)

But this could resolve into a major depenency error when there were major changes that other depenencies aren't prepered for.

Well, if that's the case, we can always not upgraded the dependency that costing problems

According to the Travis, it's passing, and if the package upgrades is a problem, we can freeze the version here so

Please do minor API Testing, (old Login etc.), when nothing breaks will merge. :)
Sent from my iPad

On 30. Apr 2018, at 18:12, Akatsuki Levi<EMAIL_ADDRESS>wrote:
According to the Travis, it's passing, and if the package upgrades is a problem, we can freeze the version here so
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or mute the thread.

The resolving patch has been moved to the following commit at the other branch
👇
https://github.com/AminoJS/Amino.JS/commit/2424e3cd912f7a47ca85008b578c0b826d5d8958
Thanks @akatsukilevi 😉


Enhancement: Mirror ChatGPT's equation rendering.

What features would you like to see added?
Add \(...\) and \[...\] delimiters for equation rendering, as in ChatGPT.
More details
First, I'd like to express my sincere gratitude to the developers for their exceptional work on LibreChat. As a STEM guy who frequently interacts with mathematical equations, I've recently encountered some unexpected behavior in this area. Upon closer examination, I discovered that:

ChatGPT (chat.openai.com) pruduces equations surrounded by \(...\) and \[...\] by default.  The Pictures section showcases how these equations are flawlessly displayed within ChatGPT's web interface.
When called from the API, GPT 3.5 Turbo uses an unusual default format for equations. But upon request, it outputs equations using the same \(...\) and \[...\] delimiters as ChatGPT. To ensure LibreChat compatibility, GPT 3.5 Turbo's API must be explicitly prompted to use $...$ and $$...$$ for equation formatting, as dipicted in the Pictures section.

While adjusting the system prompt offers a workaround, \(...\) and \[...\] allow equations to coexist peacefully with the dollar sign's primary use as a currency symbol. That's likely why both ChatGPT and Org-mode embrace these delimiters. Since LibreChat claims to have a UI matching ChatGPT, matching equation rendering capabilities would be a fantastic improvement.
Additional information: Upon further investigatgion, I found that equation rendering is provided by rehype-katex and remark-math in LibreChat. rehype-katex renders math contents via KaTeX, while remark-math converts contents surrounded by dollar signs into math content. While exploring the issue list of remark-math, I stumbled upon this issue where the developer declined to implement \(...\) or \[...\] delimiters within the plugin. This may make the enhancement I propose harder or even impractical to implement. Therefore, I would appreciate the opportunity to discuss the significance and feasibility of the proposal with you.
Which components are impacted by your request?
UI
Pictures
Equation rendering within ChatGPT:

Equation rendering within LibreChat:

Code of Conduct

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

I'm looking into it. The dev says to "just" fork the package yet it's not super straight-forward as it uses several different packages all published by him lol I'll dive deeper into the runtime to figure what's what

To be honest, I've encountered the same issue as well. My users and I deal with a large number of formulas every day. Although the developers of remark-math seem somewhat irresponsible, they cannot be blamed too much, as the format is not a standard latex formulas. I need a solution that is seamless for the users.
Here is my solution:

Enforce the addition of a system prompt, requiring the use of $ to wrap the formulas. However, this needs to handle the compatibility with user-defined system prompts, which means adding it after the existing promptPrefix in OpenAIClient.js.
Also in OpenAIClient.js, replace [ in the reply string with $, and do the same for others. This may lead to issues such as needing the entire message to be output before it displays, and it may cause other problems.
Look for other libraries that support the rendering of formulas in the chatgpt format.

To be honest, I've encountered the same issue as well. My users and I deal with a large number of formulas every day. Although the developers of remark-math seem somewhat irresponsible, they cannot be blamed too much, as the format is not a standard latex formulas. I need a solution that is seamless for the users.
Here is my solution:

Enforce the addition of a system prompt, requiring the use of $ to wrap the formulas. However, this needs to handle the compatibility with user-defined system prompts, which means adding it after the existing promptPrefix in OpenAIClient.js.
Also in OpenAIClient.js, replace [ in the reply string with $, and do the same for others. This may lead to issues such as needing the entire message to be output before it displays, and it may cause other problems.
Look for other libraries that support the rendering of formulas in the chatgpt format.

@SailFlorve Thank you for your solutions. Here is my opinons on each of them:

Compatibility is an issue. As a user with basic programming knowledge, I prefer the developer suggesting me to append the instruction (use $ to ...) to my prompt, instead of doing this in a hidden way and causing surprises.
That makes the chat bot clunky to use. People will be less likely to use your UI, I guess. However, if the application is not a chat bot, such as an AI-powered content generator, that will be an viable option.
Since remark is a very popular markdown implementation, there might not be many alternatives with equal quality. Forking remark-math might be an option.

Of course, this is just a stopgap measure, not a complete solution, but a temporary patch.

@SailFlorve @zyxir

Best way with the current state is to use a preset on a per-user basis in my opinion
Using regex to replace the unsupported delimiters would not be a bad approach and probably the quickest fix. It would be best to wait until the response is done to do so.
Forking remark-math would be the cleanest solution but is also the most complex. the unified/remark plugin framework is not easy to learn. I dove into it for about an hour yesterday and didn't get much out of it.

i tried to use a preset prompt but some equations didn't work

like the ( \frac{6}{8} \times h )and the

what prompt should I use

i tried to use a preset prompt but some equations didn't work

My prompt is simple but works great:
Latex inline: $x^2$.
Latex block: $$e=mc^2$$.

i tried to use a preset prompt but some equations didn't work

My prompt is simple but works great:
Latex inline: $x^2$.
Latex block: $$e=mc^2$$.

thanks for the reply. I tried your prompt and find out it is very useful in English, but not work very well with Chinese. but still  thanks

i tried to use a preset prompt but some equations didn't work

My prompt is simple but works great:
Latex inline: $x^2$.
Latex block: $$e=mc^2$$.

Latex inline: $x^2$.
Latex block: $$e=mc^2$$.
use English to process math equations and other Latex format
this prompt works well with Chinese. other languages may need this prompt too

Just an update here. I think some simple parsing on the frontend would help and close this issue once and for all. I might make it optional but enabled by default to begin, as it can potentially impact performance especially on mobile and longer threads of conversation.
Raw text:

Post-parsing and Render:

if this is satisfactory, I will move forward

Looks Great. Is it instantaneous or have to wait until the entire response is done? I'm looking forward to seeing the new commit.

Looks Great. Is it instantaneous or have to wait until the entire response is done? I'm looking forward to seeing the new commit.

it's instantaneous as soon as LaTeX is detected!

https://github.com/danny-avila/LibreChat/assets/110412045/43c30748-bd50-4739-9040-9be4aaabe8a6

What a nice implementation! Thank you for your hard work.


Move deployment and tools from Makefile to autonomy CLI

Is your feature request related to a problem? Please describe.
See this. Several command tools like make build-deploy or make run-oracle-dev live inside the Makefile instead of the autonomy CLI. This forces a framework user to download our repo to have it. The CLI should be enough for the users to have all the required functionality, and the Makefile should only contain functionality related to the framework development.
Describe the solution you'd like

Implement those make commands as CLI tools.
Update the docs to match the new commands

Mostly addressed by now


Kannada language finetuned model

Hello Team AI4Bharat,
Could you please look into including the kannada language finetuned model. Couldn't find the kannada model in the section on finetuned models here
Reference:
https://github.com/AI4Bharat/IndicWav2Vec#download-models
Thank you,
Manu

Hi [manualsh]
We have the Kannada model trained, and we will be open-sourcing it very soon.

Thank you @tahirjmakhdoomi
Could you please link to the model in Readme table.
It's still  missing in the above table.
https://github.com/AI4Bharat/IndicWav2Vec#download-models
Could you please  look into this?

@tahirjmakhdoomi It's been 1.5 years since your last comment about making the Kannada model open source & it is yet to be released. Can you kindly revert?