Unable to run project due to python error

Description
Tried running the project locally, received the error below.
I have tried installing Python as it says below (not sure why I need Python, I've been running this project for over a year without it). However, that hasn't fixed the issue.
Steps to reproduce
Node modules is already installed and updated, however it prompted me to run 'yarn' to install dependencies.
Ran yarn, and the following error below happened.
Expected result
The project should run.
Actual result
The project doesn't run. Instead, this error displays:
yarn install v1.19.1
warning package-lock.json found. Your project contains lock files generated by tools other than Yarn. It is advised not to mix package managers in order to avoid resolution in
consistencies caused by unsynchronized lock files. To clear this warning, remove package-lock.json.
[1/4] Resolving packages...
[2/4] Fetching packages...
info<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>- 3".
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "netlify-cms ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "netlify-cms ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "netlify-cms ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "netlify-cms ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "netlify-cms > netlify-cms-app > netlify-cms-core ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>|| ^16.0.0".
warning "netlify-cms > netlify-cms-app > netlify-cms-core > redux-notifications ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^3.0.0".
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^4.0.0".
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^4.0.0".
[4/4] Building fresh packages...
[7/7] ⠈ node-sass
[-/7] ⠈ waiting...
[-/7] ⠈ waiting...
[-/7] ⠈ waiting...
error C:\Users\MyName\Desktop\development\infused-gridsome\node_modules\sharp: Command failed.
Exit code: 1
Command: (node install/libvips && node install/dll-copy && prebuild-install) || (node-gyp rebuild && node install/dll-copy)
Arguments:
Directory: C:\Users\MyName\Desktop\development\infused-gridsome\node_modules\sharp
Output:
info sharp Using cached C:\Users\MyName\AppData\Roaming\npm-cache_libvips\libvips-8.7.0-win32-x64.tar.gz
info sharp Creating C:\Users\MyName\Desktop\development\infused-gridsome\node_modules\sharp\build\Release
info sharp Copying DLLs from C:\Users\MyName\Desktop\development\infused-gridsome\node_modules\sharp\vendor\lib to C:\Users\MyName\Desktop\development\infused-gridsome\node_mo
dules\sharp\build\Release
prebuild-install WARN install No prebuilt binaries found (target=12.16.1 runtime=node arch=x64 libc= platform=win32)
C:\Users\MyName\Desktop\development\infused-gridsome\node_modules\sharp>if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin\..
..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
gyp ERR! find Python
gyp ERR! find Python Python is not set from command line or npm configuration
gyp ERR! find Python Python is not set from environment variable PYTHON
gyp ERR! find Python checking if "python" can be used
gyp ERR! find Python - "python" is not in PATH or produced an error
gyp ERR! find Python checking if "python2" can be used
gyp ERR! find Python - "python2" is not in PATH or produced an error
gyp ERR! find Python checking if "python3" can be used
gyp ERR! find Python - "python3" is not in PATH or produced an error
gyp ERR! find Python checking if the py launcher can be used to find Python 2
gyp ERR! find Python - "py.exe" is not in PATH or produced an error
gyp ERR! find Python checking if Python is C:\Python27\python.exe
gyp ERR! find Python - "C:\Python27\python.exe" could not be run
gyp ERR! find Python checking if Python is C:\Python37\python.exe
gyp ERR! find Python - "C:\Python37\python.exe" could not be run
gyp ERR! find Python
gyp ERR! find Python **********************************************************
gyp ERR! find Python You need to install the latest version of Python.
gyp ERR! find Python Node-gyp should be able to find and use Python. If not,
gyp ERR! find Python you can try one of the following options:
gyp ERR! find Python - Use the switch --python="C:\Path\To\python.exe"
gyp ERR! find Python   (accepted by both node-gyp and npm)
gyp ERR! find Python - Set the environment variable PYTHON
gyp ERR! find Python - Set the npm configuration variable python:
gyp ERR! find Python   npm config set python "C:\Path\To\python.exe"
gyp ERR! find Python For more information consult the documentation at:
gyp ERR! find Python https://github.com/nodejs/node-gyp#installation
gyp ERR! find Python **********************************************************
gyp ERR! find Python
gyp ERR! configure error
gyp ERR! stack Error: Could not find any Python installation to use
gyp ERR! stack     at PythonFinder.fail (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\find-python.js:307:47)
gyp ERR! stack     at PythonFinder.runChecks (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\find-python.js:136:21)
gyp ERR! stack     at PythonFinder.  System Windows_NT 6.3.9600
gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\MyName\Desktop\development\infused-gridsome\node_modules\sharp
Environment
Windows.

Possibly duplicate: https://github.com/gridsome/gridsome/issues/585

@tkhquang tried everything in that thread, none of the solutions worked.
Anyone have any idea?
I've had this project for over a year running totally fine. Suddenly, for the last week, I haven't been able to run the project at all.

Update: solved.
Managed to solve the issue by going into the project and running by going into the project and:

rm -rf node_modules
yarn upgrade --latest
yarn install

Then I could run the project.

Anyone still trying to get this legacy project to work, i had another solution to add to https://github.com/gridsome/gridsome/issues/585 that worked for me.


Uncasted false env vars fix

Fix for uncasted false env vars

Version
:warning: You probably forgot to update the package version.
Changelog
:pencil: Please include the changes in CHANGELOG.md file
Lint
:tada: No lint warnings were reported!

Summary
Value

:octocat: Commit
8046c02bd8fc0a73ec92cc0e7bb1e8ee6e3b090b

Comparing against
develop branch

Build number
17

Job number
17.1

System
linux

Version
:tada: Package version has been updated properly.
Changelog
:tada: Changelog has been updated properly.
Lint
:tada: No lint warnings were reported!

Summary
Value

:octocat: Commit
373248ddcd893cc367007d4c92cabf72a9fd0bb0

Comparing against
develop branch

Build number
18

Job number
18.1

System
linux


[BUG] RapidsConf.scala has some un-consistency for spark.rapids.sql.format.parquet.multiThreadedRead

Describe the bug

Fixed by #875


Add alternating colour mode

Enhancement:
Add a new 'colour' option - alternating mode.
Drawing will invert the colour of the existing pixel (black --> white / white --> black).
If the pixel is set to transparent, set to white on the first draw.
This is very useful in the pencil drawing mode. It saves time avoiding manual colour changes.
When the background has been set (flood fill), we can just draw the foreground details. If a mistake is made, a second click will restore to the background colour.
Various other pixel drawing programs act in this way.
Re. icons - I'd suggest using the current 'invert' icon - as the square form will fit well under the other colour boxes.
A new icon for the global invert tool, could incorporate a round element (which is common). e.g. https://www.flaticon.com/free-icon/invert-colors-button_61136

My first suggestion was essentially: 1st click 'on', 2nd click 'off'. This duplicates the behaviour of bitsy and other editors. I'm not too sure how well that fits with the overall design of your program?...
Alternative: Keep single colour draw on left click unchanged, just add right click for colour sampler/ selection. This will act to speed up drawing, as we can change colours without having to move the mousing from the editor to the palette. This approach is used in PICO-8.


Localize issue

Having a site with 2 languages (wagtail==5.1.1,wagtail-localize==1.5.1,wagtail-metadata==4.0.3) no loner redirects to default locale. e.g.
https://example.com/ - throws an Internal Server Error.
https://example.com/en/ - okay
https://example.com/cy/ - okay
Disabling  wagtail-metadata redirects to /en/ as expected.

Can you pop a stack trace in here?

hm that doesn't really help, is  DEBUG=True set in your settings? That might give a more robust error message

Same issue here, with Debug = True the site shows no errors, which is very confusing.
With Debug = False we get an Internal server error.
Using:
wagtail-localize==1.4
wagtail-metadata==4.0.3

Upgraded and still the same issue:
wagtail==5.0
wagtail-localize==1.5.2
wagtail-metadata==5.0.0


SQL queries generated with brackets by LINQ

Hi!
When we use the GetTable function of a DataContext (LINQ) the generated SQL query has brackets, which are not accepted by DuckDB, causing errors. The function is based on the SQL Server standard. As a suggestion, the brackets could be handled and replaced in the CommandText.
Thanks!

Why should this library accept commands for a different db? That's out of scope of this library. If you need, you can do the replacement yourself before assigning it to the CommandText

Hi!
I can explain in more detail, maybe I wasn't clear. The goal is to use IQueryable with Linq, to perform on-demand queries. Queries are automatically generated by the mentioned function and cannot be modified, only within the connector. This Linq library uses the SQL Server pattern, with square brackets. For example, the MySQL connector has a parameter in the connection string to handle this: "sqlservermode".
It's important for OLAP cubes too, using SSAS objects with AMO.

https://www.connectionstrings.com/mysql-connector-net-mysqlconnection/allow-square-brackets-around-symbols-instead-of-backticks/
[Allow square brackets around symbols (instead of backticks)]
Server=myServerAddress;Database=myDataBase;Uid=myUsername;Pwd=myPassword;sqlservermode=True;
This enables Visual Studio wizards that bracket symbols with [] to work with Connector/Net. This option incurs a performance hit, so should only be used if necessary.

Thanks!

That's what I suspected too and this is how you can solve it: You need to write a new "dummy" implementation for DbConnection and DbCommand classes (maybe others too) that won't actually do anything but call the DuckDB.Net counterpart. In other words, you need to create a decorator for DuckDBConnection and DbCommand classes. These classes can do any type of modification to the command text that you need before passing it to DuckDB.Net. Register this new provider in the tool you use and everything should work fine.
This is a little bit more work on your side but will let this project stay clean without implementing third-party tool requirements.


Adding prepublish script

Fixes #10
My idea is to running the tests and build before publish a new version of package.
What do you think?

Our main problem is when we forget to commit the build before tag/publish. I did the same as yours, but won't fix this issue...=/
What we have to do is:
build =>
verify if we have differences between builds =>
commit =>
tag =>
publish.
I don't know if I'm clear about it....

@Nipher

I'm not sure which step would be better to add the build procedure...
Actually I was thinking about creating a precommit that:

Runs tests / lints
Builds
Runs git add (we can do a very simple js file to execute it)

And then we wouldn´t need to include these steps on prepush/prepublish anymore as we just push/publish what is commited, am I right?
And imho this is a MUST before commiting...
What do you guys think?

And you mentioned a tag step for prepublish right?
do you want to automate this? @willianjusten
because if so, it would be nice to sync between tag and package,json version...
Then in my mind, it would become something like this:

get semver version from user (?)
Updates package.json
git add and git commit ( with a default message for new version release ?)
git tag
publish

If we put tests/lint before every commit, will be very very annoying...

I don't want to create something very complicated for users also...=/

Guys guys!! I was reading a little bit more about npm cli and...they do exactly what we need!
npm version command
They change the package.json, commit and create a tag.

Omg... I'm so fu#%$ng dumb... :joy:
Awesome then... could you define what will be done with prepublish?
build -> npm version command -> publish ??

I'll test if they just add the package.json in the commit or whole modifications. Let me check this.

Meh...if the working directory has files to be committed yet, the command break =/

Hey guys, sorry for delay.
Hmm, I understand the problem.
It's possible to use the npm version command within another script and and call it on prepublish.
In this way, the script can commit the files.

@willianjusten and @Nipher I will be close the P.R for discuss the implementation at #10


Start Parallel processing on nextTick()

Instead of an explicit start of the parallel processing, delay the processing until the nextTick (or wait for an explicit call to then, catch or subscribe). This allows to bundle as many call as possible for the moment but does not delay the execution until then is explicitly called.
Potentially add a configuration option that allows to disable this behavior and waits for an explicit then, catch or subscribe call

Implementing this would hardly increase the complexity. Additional, what is the expected behavior of:
parallel.from([1, 2, 3]).map(x => x * 2).run().filter(x => x % 5 === 0)

Would the second chain set up on the intermediate result or recalculating the whole chain? Therefore, a more explicit api is preferred. But considering a continuation api that may also support chaining multiple parallel jobs (where continuation is also supported from the scheduling perspective)
parallel.from([1, 2, 3]).map(x => x * 2).run().continueWith(chain => chain.filter(x => x % 5 === 0))

Die extra run()-Methode finde ich künstlich, zumal es hier ein deklaratives Konzept ist (wie Reactive).


feat(AccessCode): Add ability to set time-based codes

Partially closes #236
Spin off from #269
Screenshots
https://github.com/seamapi/react/assets/11449462/0956f986-e20b-4322-b38f-ffb9ce2ee840

@razor-x updated to use native pickers in https://github.com/seamapi/react/pull/282/commits/ecf1aeb738136b67b0334a3144dadc4ef0c68546, and https://github.com/seamapi/react/pull/282/commits/5f1f861b1d41381cccd3f928f1438899911a54d4.
Other than the obvious lack of styling, here are a couple other missing features from the design:

Can't force time picker to show / hide seconds, it seems to be tied to the browser's preference.
Can't modify time picker intervals. There's a step option but that's the increment/decrement size when clicking on arrows, not the option intervals.

https://github.com/seamapi/react/assets/11449462/cf675719-e135-401b-8812-67c6b29dc95b

@razor-x I believe all threads have been addressed. Let me know if I've missed any!

Nice! I need to test a few things first, but I'll plan to merge this today!


Continue import if import of separate records fails

OrientDB Version: 2.2.22
Previous versions of OrientDB allows less strict rules for data as more recent versions. For example. In previous versions of OrientDB negative links can be kept in documents.  But if import of single record is failed does not mean that the import of the rest of the records should be stopped. Users can add missed records latter when whole import will be finished.

Fixed


Sample operator

Sample operator implementation
http://reactivex.io/documentation/operators/sample.html

Ongoing PR: https://github.com/ReactiveX/RxGo/pull/199

Implemented thanks to @MarekLabuz


option --time-out is needed.

Hi Brent,
Thanks for share this cool tool, which is better than parallel for commands with nested quotation.
If find that, in some cases, e.g., downloading a web page using some tool, the process may hang and never end. This  will not return finish or fail signal, a timeout mechanism could help.
Merry Christmas and Happy New Year!
Wei

I agree. I will have a look at this as I have time and I welcome pull-requests. Could be done with a context.

I'll read the code and try to add this feature.

I suspect the code is not readily amenable to this change due to my short-sightedness, but it's doable, I'm sure.

I can't follow your code, maybe I can reinvent this wheel :( :joy:

why don't you look at replacing exec.Command with exec.CommandContext? https://golang.org/pkg/os/exec/#CommandContext

Sound easy, can you please replace it when you free? I try to digest all the code, but it needs more time. And I want to invent a new wheel :)

Dear @brentp , I made a similar tool rush borrowed some idea of gargs and GNU parallel, which has similar [performance](https://github.com/shenwei356/rush/releases as gargs).
During the development, I finally understood your code :smile: .
The first version v0.0.1 is out, could you please try and give some advices?


Add before_reload callback

added before_ready callback capabilities. before_ready can store a long running workflow that needs to run before the final build and reload!

Hey, I did some changes on your PR, nothing major, mostly formatting (you had set the file in the wrong place).
If you're interested, GitHub has some nice docs on using PRs

My bad I am new to this.
On Wed, Jun 26, 2024 at 1:27 PM Pedro Alves @.***>
wrote:

Hey, I did some changes on your PR, nothing major, mostly formatting (you
had set the file in the wrong place).
If you're interested, GitHub has some nice docs on using PRs
https://docs.github.com/en/pull-requests
—
Reply to this email directly, view it on GitHub
https://github.com/pta2002/gleam-radiate/pull/6#issuecomment-2192260319,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2UKQFATJ5GKWCV33YWUPZLZJL2WTAVCNFSM6AAAAABJZVDFCKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJSGI3DAMZRHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


use concurrent file reading and flatten structure

Hey @nopeless thanks for these they look really good! I've given it a test locally (rebased with the latest master) and I'm seeing some instability in the tests. Every 3rd to 5th run is producing a Failed to launch the browser process! error from Puppeteer. I'm not sure why this is happening and I can't really see how these changes could be causing it but I'm unable to reproduce the issue on master at all.
I'm wondering if we're not awaiting something to be resolved by accident but can't pick it out at first glance. I'll try and dig when I get some time but if you have the capacity to take a look that would be really appreciated!

Hey @nopeless thanks for these they look really good! I've given it a test locally (rebased with the latest master) and I'm seeing some instability in the tests. Every 3rd to 5th run is producing a Failed to launch the browser process! error from Puppeteer. I'm not sure why this is happening and I can't really see how these changes could be causing it but I'm unable to reproduce the issue on master at all.
I'm wondering if we're not awaiting something to be resolved by accident but can't pick it out at first glance. I'll try and dig when I get some time but if you have the capacity to take a look that would be really appreciated!

can you tell me details about this error?

Sure, sorry that was silly of me!
Error: Failed to launch the browser process!

TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/master/docs/troubleshooting.md

    at onClose (/Users/elliotblackburn/code/mdpdf/node_modules/puppeteer/lib/Launcher.js:750:14)
    at Interface.<anonymous> (/Users/elliotblackburn/code/mdpdf/node_modules/puppeteer/lib/Launcher.js:739:50)
    at Interface.emit (node:events:402:35)
    at Interface.close (node:readline:586:8)
    at Socket.onend (node:readline:277:10)
    at Socket.emit (node:events:402:35)
    at endReadableNT (node:internal/streams/readable:1343:12)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)```

This was being spat out, from what I can see it looks like it might be only happening on the test `Convert CLI When custom style is passed HTML file contains the default styles when --gh-style is passed` although I'm trying to dig into it a bit deeper and run a few more iterations to be sure.

I manually dug through puppeteer's source code and later realized that the code I have written has nothing to do with the browser. Can you run a test on the commit before this and see if all tests pass on that? (can you test on the code before this change)

Hm I've taken another crack and can't seem to recreate the issue again, no idea why. I've run the tests 30 times and no issues so I'll go ahead and merge.


Unable to shutdown on Close(): infinite loop

socket.Recv() does not ends after shutdown socket closed leading to infinite loop in read()
patch to solve:
diff -uwrd old/socket.go new/socket.go
--- old/socket.go	2017-10-11 15:06:17.553303464 +0300
+++ new/socket.go	2017-10-11 15:08:10.601014898 +0300
@@ -7,8 +7,11 @@
 	"net"
 	"strings"
 	"sync"
+	"errors"
 )

+var ErrClosed = errors.New("socket closed")
+
 // Socket holds the socket client connection data.
 type Socket struct {
 	conn     net.Conn
@@ -70,7 +73,7 @@
 				return buffer.String(), nil
 			}
 		case <-s.shutdown:
-			return buffer.String(), nil
+			return buffer.String(), ErrClosed
 		}
 	}
 }

By the way, thanks for the lib! As far as I can see it has chances to be best amongst competitors.

First of all,
Thank you ! For me it's a pleasure this library is helpful for someone.
Well, instead creating ErrClosed, I would like add as return io.EOF.
Maybe it sounds more reasonable though.

Thanks for reacting so fast! Very good. I think you're right about io.Err, but I'm working with Go for less than one week so I have not much experience yet, honestly =)


redeclared in this block

please I follow the step wey, butcan't build the project success, can u tell me why?

You've got two versions of Go installed, you have to delete one and its files.


Contributor Rights Request

Would like to get contributor rights so that I can assist with the release process.

You've contributed a lot to the project, so I have no reservation with this request.  @davidjgonzalez ?  @joerghoh ?

Interpreting no objections as consent. ;P

👍  from me! (sorry - was on vaca!)


Make consensus layer compatible with GHC 8.10

All changes are backwards-compatible with 8.6

Replace * with Type
Remove imports that GHC now considers redundant
fail is no longer a method of Monad, but of MonadFail. See
https://gitlab.haskell.org/ghc/ghc/-/wikis/proposal/monad-fail#adapting-old-code
Remove record wildcard matches when no fields are
used (-Wunused-record-wildcards)

I'm extracting this from #2539 because that PR currently has some test failures and I'd like to merge this ASAP to avoid merge conflicts.

bors merge

bors merge

bors merge

Forgot to mention: https://github.com/input-output-hk/ouroboros-network/pull/2540/commits/f07b7eef67ccd69171e1e88e927a4b154fab4ad6 will be needed to really make it compatible with 8.10, but I'd rather merge that after we have switched to 8.10.


Fix A typo as your first pull request :)

Hey, you want to try contributing to an open source github project? Then do this super simple issue.
The Issue:
Let's make this non difficult - There's a typo in the file 'allanPage.html'. The word 'Button' is mistyped as 'Buttton'. So, find the spelling mistake and fix it!
You can find a guide for how to contribute to github projects, on the main page of this project.
If you need help / do not understand something, find me in the discord under 'NomadAllan'.
Good luck!

I'd like to work on this.


Add more fields in telemetry

Signed-off-by: Harshita sharma<EMAIL_ADDRESS>
What this PR does / why we need it:
Add more fields in telemetry
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #12
Special notes for your reviewer:
Kubermatic configuration name has been kept a sting in place , but please suggest if the name can be fetched from a kubermatic code constant.
In case the kc name is changed to smtng else in future

Also please suggest if you would like to have any other field be added too.


:sparkles: feat(bundler): Handle watching entire module graph

Description
Watch entire dependency graph for reloads
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #350

PR #351   👈

PR #352

This comment was auto-generated by Graphite.


Move East Slovak Romani 3 Project to Next Stage in Pipeline

Team member needs project moved to next stage in the pipeline.
Field Staff:
The East Slovak Romani 3 project is in the Pipeline. Earlier today I received the consultant's (a non Seed Co consultant) endorsement so I wanted to move this to the next stage in the Pipeline. When I did so I received the following error message below. My Supervisor, Matt Merritt who is Eurasia Regional Director, got the same message and was also unable to move this along in the Pipeline. Can you please correct this as soon as possible? I'm working on this transition now and this needs to be approved within the month. Thanks,

There's a bug here in the data. Looks like a migration problem. Looking into it.

Yes, I believe Seth said we already have a fix for this system wide in the backlog. This is a data fix in the meantime.

I think this might be something he's not aware of. It's data that wasn't in V2 and so I deleted the place where it's held in V3. Now the API is trying to find that node in the DB and can't so it gives an error. I wasn't aware of that ramification.

This issue should be fixed now if the team member wants to try updating the status again.

Confirmed, thanks @blakekrammes


distinguishing families of tests

The test families base and hybrid were introduced (86892f30f3b38731b97e537d45531cd3d91fefa6) at a time when polytope required cvxopt, so tests depending on polytope were not included in the family hybrid:
https://github.com/tulip-control/tulip-control/blob/86892f30f3b38731b97e537d45531cd3d91fefa6/run_tests.py#L26-L30
because polytope and cvxopt were optional dependencies:
https://github.com/tulip-control/tulip-control/blob/86892f30f3b38731b97e537d45531cd3d91fefa6/setup.py#L328-L336
Since then, polytope >= 0.1.2 requires only scipy (not cvxopt, https://github.com/tulip-control/polytope/issues/15), and the same holds for abstraction operations in tulip (#154, #168, #122).
The current description of base tests refers to a default installation of tulip, with required dependencies:
https://github.com/tulip-control/tulip-control/blob/6b596491796d8bfa353eef2e209a57464eb64480/run_tests.py#L34-L38
so we may consider moving tests from hybrid to base, for those tests that work with a default installation.
On the other hand, in absence of cvxopt, tests in hybrid run slower, so they would all have to be marked as slow. So the alternative to moving tests to base would be to revise the description of the test family base (in particular the mention of required dependencies, or to explicitly mention that polytope-dependent tests are in hybrid).

Did you run numerical trials to estimate how much slower tests in hybrid run without cvxopt?


Fix missing terms

Fixes #64
Now the output code is:
(function(){
  var _HelloWorld$main = 1;
  return _HelloWorld$main;
})()

Sorry, I noticed this and fixed it at the same time as you, as I was updating the documentation.


publish2Subscribers(payload, topic, qos);   catch  Exception    java.lang.NullPointerException: null

According to log tracking, after running for a period of time, it cannot be repeated. When pushing messages, it appears in publish2Subscribers(payload, topic, qos);
Method, can capture the broker will be Java. Lang. NullPointerException: null, but there is no stack information, and the MQTT client connection is not connect
public void internalPublish(MqttPublishMessage msg) {
    final MqttQoS qos = msg.fixedHeader().qosLevel();
    final Topic topic = new Topic(msg.variableHeader().topicName());
    final ByteBuf payload = msg.payload();
    LOG.info("Sending internal PUBLISH message Topic={}, qos={}", topic, qos);

    publish2Subscribers(payload, topic, qos);
    LOG.info("publish2Subscribers internal PUBLISH message Topic={}, qos={}", topic, qos);
    if (!msg.fixedHeader().isRetain()) {
        return;
    }
    if (qos == AT_MOST_ONCE || msg.payload().readableBytes() == 0) {
        // QoS == 0 && retain => clean old retained
        retainedRepository.cleanRetained(topic);
        return;
    }
    retainedRepository.retain(topic, msg);
}

Hi @Tong-07 thank's for reporting this. Without a stack trace or a debug log it's impossible to understand where the NPE is triggered.

publish2Subscribers(payload, topic, qos); NPE thrown out in this method, but there is no stack information, so I did not analyze the specific where, through the log, you can know is captured in a function

Capture the information is a Java. Lang. NullPointerException: null


parsing: support dot(s) in object keys

Closes #10348
Closes #10501
@thabarbados sorry for stepping on your pr, but as I already put some time to it, I figured its less time spend for everyone if I post this.
This performs consistently a little bit better than master on the default parsing case (uPlot dataset):
pr:

50 runs done in 3389ms. Average: 67ms, min: 57ms, max: 127ms, variation: 70ms.

master:

50 runs done in 3602ms. Average: 72ms, min: 62ms, max: 135ms, variation: 73ms.

@kurkle No problem! It's great that you found a faster solution!

@kurkle, what do you think about adding performance tests? I think it would help new contributors. If you don't mind, I can try to realize it.

@kurkle, what do you think about adding performance tests? I think it would help new contributors. If you don't mind, I can try to realize it.

That would be great!

@kurkle awesome! Can you share with me your large database and how you checked performance? I think this will be a good starting point.
If you don't mind, you can email it to<EMAIL_ADDRESS>

I've been usin uPlot bench dataset, available at https://github.com/leeoniya/uPlot/blob/master/bench/data.json
The data needs some pre-prosessing though. (See the chart.js bench at uPlot, that one is a good starting point)


Old image not deleted after updating image field

Laravel Version: 5.5
Voyager Version: 1.*
PHP Version: 7.1

The question was asked previously and you say it will be solve in the new version for 1.* otherwise we should repost again.
old issue #1309

+1

This is an old post, but it doesn't seem like it's being addressed.  I'm currently working on a project that Voyager is perfect for.  The problem is that at the end of the day, I need to download the active images to a local machine for processing.  It's a lot of extra work to determine which images in the directory should be downloaded.  If anybody has a good work around for this, It would be greatly appreciated.

This was addressed in https://github.com/the-control-group/voyager/pull/3240.
It's gonna be included in 1.1.11

+1

+1
a work around that may help you:
in the model page make a boot method and inside make a listener for deleting/updating/creating events and in the closure you can delete the stored image
the problem with me that i have multiple image upload field and i want to trigger a function on column update from the admin panel to delete only the photo that is removed from the column .. any idea how to do that ??


Allow using an arbitrary hash_fn for generation

Solves part of #88 by providing a separate function to customize the hashing function during generation. That way, one can avoid usage of the default SipHash as well as avoid having a PhfHash bound on the keys. This custom hash function can be used in combination with get_index directly.

Sorry for the delay, thank you!


bug(COMPONENT): mat-grid-list does not show mat-grid-tile inside component

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
"mat-grid-list" list only shows "mat-grid-tile" if it is directly inisde it. It does not show it, if it is put into a child components html file.
This is inconsistent to how for example the "mat-nav-list" and "mat-list-item" works, which does show the "mat-list-item" even if it is put into a child component.
Reproduction
Steps to reproduce:

https://stackblitz.com/edit/components-issue-21e7dn?file=src/app/example-component.html
see example component html and child component html

Expected Behavior
Do show the "mat-grid-tile", even if the tile element is put inside a child component.
Actual Behavior
Shows the "mat-grid-tile" only if directly inside the "mat-grid-list" element
Environment

Angular:12.2.9,  13.0.0 and other versions aswell
CDK/Material:13.1.3
Browser(s): Chrome
Operating System Windows

This is more of a side-effect of how Angular queries and content projection works, rather than an issue with the grid list setup. The ContentChildren inside the grid list is only going to pick up tiles that are defined directly within the template, it isn't going to traverse down the component tree. Closing the issue since we can't do much about it.


Voucher additions

This PR is related to the voucher additions proposed in #3569.
Changes are in-progress and not ready to merge.
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Closing this PR due to no activity.


Bulbasaur bug

On multiple occasions i would see a bulbasaur on the minimap and walk towards it, once i get close it turns into another mon. Just recently one turned into Meowth and another turned into Magikarp.

This is a rendering issue. When Pixelmon knows that a space is to be taken up by a pokemon, but it doesn’t yet know what that pokemon is - or that pokemon has yet to be determined, the default fallback is Bulbasaur. It’s the same reason the statue chisel defaults to Bulbasaur when you use it, or when there’s an occasional visual glitch in your party and some mons turn into Bulbasaur briefly. Unfortunately, there’s nothing to be done about it


How SDK knows, when it  deregistred from the alexa app.

How the AVS DEVICE SDK knows it is de-registered from the alexa app.
As I have seen in Echo/Echo plus, After de-registration, If user interrupts the device Then it is going to setup mode.
How this kind of behavior can be handles in AVS DEVICE SDK.
If user de-register the SDK from alexa app, The sdk is not responding to user and giving authorization errors if user interrupts.
Can i get the de-registration status from this authorization errors.
What is the expected behavior?
AVS DEVICE SDK should know the status of de-registration.
What behavior are you observing?
Some messages are received by avs sdk when user press the de-register button from alexa app. But they are not clarifying that the user is de-registered from the alexa app.
Provide the steps to reproduce the issue, if applicable:
Register the SDK with amazon account.
Open alexa app and de-register the above device.
What version of the AVS Device SDK are you using?
AVS device sdk version 1.11
Tell us what hardware you're using:

[ ]  Laptop

Tell us about your OS (Type & version):

[ ] Linux

Hi @vijay464,
I believe the information in #1179 will help you.  Thanks!

Hi  kclchan,
Thank you for the information. I can follow that and implement. We can close this ticket. If I need any further info, I will try reopen the ticket.
Thanks
Vijay

Hi  kclchan,
Thank you for the information. I can follow that and implement. We can close this ticket. If I need any further info, I will try reopen the ticket.
Thanks
Vijay


Cannot delete parts of a sprite

When editing the sprite, click the scissor tool. Click on the part you want to delete (e.g. feet), the whole sprite is deleted not just the feet. Should ScratchJr have this capability?

Moved from LLK/scratchjr-private#160
/cc @chrisgarrity

Product backlog


Closer on search but still a bit broken

We are now getting this far, an improvement :smile:
ElkArte\Search\API\Util' not found in /sources/subs/Search/API/SphinxqlSearch.class.php on line 401
Looks like a simple missing \ in front the the Util::   just noted so we check all the various search classes for other namespace issues.

LOL
Yeah, all global space that need the \ before... :-\ will add this evening. ;D

Would you mind testing the last PR I sent?
It should fix all these namespace issues (at least here :P).

quick test looked fine, so i'm going to merge it in.

Cool! :D


New version: AquaSecurity.Trivy version 0.49.1

Pull request has been created with komac v2.0.3 :rocket:
Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run


[FIX,PROFILING] Add USE_PAPI configuration to config.cmake

I forgot to add USE_PAPI to config.cmake.
@mbrookhart @areusch

The PR looks good to me. Just retriggered the CI

Oh No! We have a merge conflict :(

Thanks guys for the review! It is merged


update the fujilupe-v1.0 example notebook

Once the files are in their final location, we need to update the notebook. See this comment for more details.

Done in https://github.com/moustakas/desi-photometry/commit/ea1466e3a14405236d87bdd9e74b685d932d008c.


provide a windows-compatible solution to run pre-processing

The pre-processing script is a bash shell. Hence, it will not be possible to run it from a Windows computer (at least not for a common windows user). It would be good to provide a means to easily run the commands for a Windows user, for instance by

providing a copy of this script, written in powershell
or encapsulating the whole process in a single docker image
or simply writing the commands to run, in the readme, without using any variable.

For instance, something like this seems to work on Powershell

$Env:outputDir="data"
$Env:MAPILLARY_ACCESS_TOKEN="myaccesstoken"
mkdir -p $Env:outputDir
mkdir -p ${Env:outputDir}/images_new

docker run --rm -v ${PWD}:/mnt/data/  `
    -e MAPILLARY_ACCESS_TOKEN="${Env:MAPILLARY_ACCESS_TOKEN}" `
    -it jeanpommier/devseed-geokit:python.latest mapillary `
   get_mapillary_points `
      --input_aoi=-61.493225,15.204024,-61.232986,15.642874 `
      --organization_ids=276431331814934 `
      --timestamp_from=1672531200000 `
      --only_pano `
      --output_file_point=${Env:outputDir}/mapillary_points_panoramic__pano.geojson `
      --output_file_sequence=${Env:outputDir}/mapillary_sequences_panoramic__pano.geojson

docker run -v ${PWD}:/mnt/  `
    -e MAPILLARY_ACCESS_TOKEN="${Env:MAPILLARY_ACCESS_TOKEN}" `
    -it jeanpommier/devseed-spherical2images:v1 `
   clip_mapillary_pano `
      --input_file_points=${Env:outputDir}/mapillary_points_panoramic__pano__pano.geojson `
      --image_clip_size=1024 `
      --output_file_points=${Env:outputDir}/mapillary_points_panoramic_process_new.geojson `
      --output_images_path=${Env:outputDir}/images_new `
      --cube_sides=right,left

Complete via https://github.com/developmentseed/housing-passports-v2/pull/39


Running locally

Hey man, great work with this project.
I have a Python CLI project with a similar setup and I'm wondering how you test changes locally since you're importing the package pls_cli instead of importing the local files?
Hope that makes sense!
Thanks

Thank you very much! Be welcome! 😃
I don't understand your question very well, can you elaborate more? 🤔

For example:
In please.py you import from pls_cli.utils.quotes import get_rand_quote
What happens if you want to test a change to utils/quotes.py? When please.py runs, it will import the package stored in python/site-packages/pls-cli rather than your local file ./utils/quotes
My question is, when developing pls-cli do you have a special method to get around this problem?
Hope that's clearer 😅

If I understand the problem correctly, I use poetry to run locally ( And manage my environments ) when I'm developing, in this case it will get the local files and not the ones installed on my python machine.
I use that command when I'm developing:
poetry run python please.py

Perfect, thanks a lot


Update peer dependency for  typescript 3.0

Hi, I'm using typescript 3.0, and I got a warning.
and I think it's time to update the peer dependency.

warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>

Should be done after #344


Version mismatch error after upgrade

When running ddev start with an outdated set of config files, the "version" stanza in docker-compose.yaml is automatically updated to the latest. In this case, it's updated from '3' to '3.6'. If one has additional docker-compose files for add-on services, e.g. solr, created at the earlier release level, ddev start does NOT automatically change the version there to match the version in docker-compose.yaml. That causes ddev start to fail with a version mismatch error:
Failed to start jnjhwsch: Failed to run docker-compose [-f /Users/jeff/ddev/jnjhwsch/.ddev/docker-compose.yaml -f /Users/jeff/ddev/jnjhwsch/.ddev/docker-compose.solr.yaml up -d], err='exit status 1', stdout='', stderr='Version mismatch: file /Users/jeff/ddev/jnjhwsch/.ddev/docker-compose.yaml specifies version 3.6 but extension file /Users/jeff/ddev/jnjhwsch/.ddev/docker-compose.solr.yaml uses version 3.0'
To Reproduce
Steps to reproduce the behavior:

Run ``ddev config` on a site with ddev 1.0.0
Create a docker-compose.solr.yaml file at the same version level as docker-compose.yaml ('3')
Run `ddev start`` etc and bring up system
Update ddev to 1.1.1
Change .ddev/config.yaml APIVersion from v1.0.0 to v1.1.1
Run ddev start

Expected behavior
Version is updated in all of the configuration's docker-compose...yaml files, and ddev containers/system launch successfully
Screenshots
If applicable, add screenshots to help explain your problem.
Version and configuration information (please complete the following information):

Host computer OS and Version: macOS Sierra 10.12.6
Docker version information:
Client:
Version:           18.06.0-ce
API version:       1.38
Go version:        go1.10.3
Git commit:        0ffa825
Built:             Wed Jul 18 19:05:26 2018
OS/Arch:           darwin/amd64
Experimental:      false

Server:
Engine:
Version:          18.06.0-ce
API version:      1.38 (minimum version 1.12)
Go version:       go1.10.3
Git commit:       0ffa825
Built:            Wed Jul 18 19:13:46 2018
OS/Arch:          linux/amd64
Experimental:     true

ddev version information (use ddev version)
config.yaml contents for the misbehaving project
Do you have any custom configuration (nginx, php, mysql) in the .ddev folder? Solr

Yes, we don't think it's right to update your custom docker-compose.*.yaml so we put that in the release notes hoping people would see it. But I know lots of people wouldn't see the release notes.

I'd suggest, then, when there's a potential mismatch, that ddev start updates NONE of the version numbers and fails fast, with an error message to the effect that there is a mismatch and it should be resolved manually, because a partially automatic change without any accompanying instructions at runtime is a recipe for confusion (and more bug reports)

Our general strategy is to ask people to disable custom config during upgrades. I know that's not always expected, but we ask for it on nearly every release.  In this release there was not even a way to not update the docker-compose.yaml version, because it was invalid in version 3.
Definitely sorry to frustrate you on this. It's kind of the tradeoff between writing lots of custom one-time code for people who customize, or trying to make it simple and hope it works out for people using the release notes.

Who do you know who reads release notes when it's a point release?

I would, though, reiterate my earlier suggestion - either fail fast and report it in a meaningful way, or do what's needed not to fail, and report what was done. In either case, it's not left to the user to decipher an at-best semi-comprehensible error message.

We do know that people who are smart enough to extend ddev are going to have to battle out of some of their own issues, and we'll continue to do our best to notify people when that may be needed. Maybe next time an explicit tweet or something about the exact requirement will help with this.


Update PR coverage comment step to continue on error

ok @lukas-h this finally seems to work, needed the step in the job to continue on fail, and the job itself to continue on fail, so to break out a coverage comment failure vs an actual test failure I broke these up into two jobs in the workflow

I have an idea: Let's move all of the steps after the step "Run tests with coverage enabled" to the new job comment-on-pr, so we have a clearer separation what the two jobs should do:

Job 1 (test): Run the dart tests
Job 2 (comment-on-pr): Convert to LCOV, generate HTML, and finally comment

By doing that, we would make sure that the unit tests finish. Independently if anything of the coverage fails or not.
lmk what you think

I agree in separating the jobs more cleanly. I'll push some attempts at getting the artifact paths correct to have the tests give the the raw coverage .json files to the coverage job.

Also, I am learning more about the different common workflow triggers, and there appears to be some best practices (of course there are) to handle things like commenting coverage results on PRs.
From https://securitylab.github.com/research/github-actions-preventing-pwn-requests/
..., a new trigger workflow_run was introduced to enable scenarios that require building the untrusted code and also need write permissions to update the PR with e.g. code coverage results or other test results. To do this in a secure manner, the untrusted code must be handled via the pull_request trigger so that it is isolated in an unprivileged environment. The workflow processing the PR should then store any results like code coverage or failed/passed tests in artifacts and exit. The following workflow then starts on workflow_run where it is granted write permission to the target repository and access to repository secrets, so that it can download the artifacts and make any necessary modifications to the repository or interact with third party services that require repository secrets (e.g. API tokens).

While I'm at it, I'll test this flow out ^^^

should work in theory, but probably can't run the new workflow until this is merged in, so it's kind of a catch-22 to debug this more :-\

until this is merged in

Let‘s test it in main. It won‘t mess with our releases so it‘s okay.


refactor: code based on ratatui component template

Summary by CodeRabbit

New Features

Introduced keybindings for quitting, suspending, refreshing, and setting locale to English.
Added CLI struct for customizing tick and frame rates.
New UI components: List and Map for improved user interaction.
Enhanced configuration management with key bindings and styles parsing.

Bug Fixes

Updated .gitignore to exclude .sqlite and .log files properly.

Documentation

Added environment variable exports for configuration settings.

Refactor

Major restructuring for asynchronous operations and event handling.
Enhanced error handling for better clarity and efficiency.

Chores

Updated dependencies and removed outdated ones in Cargo.toml.
Introduced build script for generating version info and Git metadata.

@coderabbitai review

@coderabbitai review

@coderabbitai review


[Spotify Controls] Favorite currently playing song

Extension – Spotify Controls
Author: @thomaspaulmann
Describe the feature and the current behavior/state.
Allow favoriting the currently playing song. At the moment there's no control over favoriting of the song
Who will benefit from this feature?
This will skip switching to application just like other commands do 😉
Any Other info.
I am referring to favoriting the song not toggling the favorite. So, If you are listening to some song and you liked it then just add it to your liked song playlist not remove it from your favorite list if already a favorite.
Thanks for RayCast & this awesome extension ❤

I don't use spotify player, I only use spotify controls created by you.


Add 'refresh' command support

Add 'terraform refresh' support.
Fixes #93

@charleszipp And another one, by popular demand (ours included) :)

Hmm, not sure why pipeline failed on both PRs. But I returned the version placeholders here. Let's see what we do with decorating it with withAzureRmProvider.

Hmm, not sure why pipeline failed on both PRs. But I returned the version placeholders here. Let's see what we do with decorating it with withAzureRmProvider.

I reran, and seems to pass now. My guess is something on the agent got stuck as it ran for almost an hour before the task was cancelled. I reran locally and worked fine as well. I will add a change to the marketplace docs to get it to retrigger the ci check.


Integrity GitHub hook is incompatible with multiple projects

We have several projects listed in our Integrity app. We have a GitHub post-receive hook set up to notify after a push to each one. Unfortunately, every time Integrity received the POST from GitHub, it tries to build every project with the commit hash it received from GitHub. This breaks all builds except for the one that actually got the push.

I have a similar setup with multiple projects using the same prefix.  I use the /github/:api_token hook since I was unable to properly authenticate the project specific hook, and it is working properly for me.
When using the project specific /:project/builds hook this may still be an issue though I was unable to test that setup.

So commit 5773fa8 looks like it would fix this. What are the chances of having that merged in?

@rud according to a comment above, "the matcher is insufficiently specific". The fix then should make the matcher more specific?

It used to be a wildcard, but that means it'll match too much. The commit 5773fa8 fixes the issue as reported. Currently a "LIKE" query is used, with the fix it just uses an exact match.

If you make a pull request with these two changes:

Change<EMAIL_ADDRESS><EMAIL_ADDRESS>to<EMAIL_ADDRESS><EMAIL_ADDRESS>Add test coverage

I will be more than happy to merge it.

I don't expect to be contributing a PR for this as I've picked a different CI solution for the project. In other words, this is up for grabs for anyone wanting to help out with a simple pull-request.


Add email trigger type to ExportedHandler

This PR adds the type of EmailExportedHandler to ExportedHandler, as was done with other triggers.
Resolves #504

I have read the CLA Document and I hereby sign the CLA


Match gets asked by apple for security questions

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
I know this is not a strictly related to fastlane and/or match, but out of nowhere apple asks for security questions during match auth phase.
Maybe some of you know how to fix it.

Account was used for fastlane well for over a year
Account has no 2FA enabled
Account was made for CI only, it's not used for anything else.

Question Description
The related part of logs is:
[11:14:14]: Verifying that the certificate and profile are still valid on the Dev Portal...
Service key is empty
+------------------+---------------+
|           Lane Context           |
+------------------+---------------+
| DEFAULT_PLATFORM | ios           |
| ENVIRONMENT      | master        |
| PLATFORM_NAME    | ios           |
| LANE_NAME        | ios hockeyapp |
+------------------+---------------+
[11:14:20]: Invalid 2 step response {"securityQuestions"=>{"questions"=>[{"id"=>141, "question"=>"Who was your favorite singer or band in high school?", "number"=>1, "userDefined"=>false}, {"id"=>143, "question"=>"What was the first name of your first boss?", "number"=>2, "userDefined"=>false}]}<EMAIL_ADDRESS>"crResetEnabled"=>false, "resetSecurityQuestionsSupportLink"=>"http://support.apple.com/kb/HT6170"}```

@atomicus We did swap from using the https://itunesconnect.apple.com API to https://appstoreconnect.apple.com API so this might be a weird side effect of that 🤔 Can you try a few more times for me and see if this goes away? (I know this sounds weird 😬 )

@joshdholtz have the same issue, tried a lot of times already :)

😔 Going to close this issue and move this over to #13281 to keep all conversation in one place


:reycle: formatted and refactored code into new files

Regression tested everything except encryption since I don't have a key set up
Closes #17

LGTm


adding support for claude, cohere, llama2, azure

Hey @stoyan-stoyanov ,
Noticed you're only calling OpenAI. I'm working on litellm (simple library to standardize LLM API Calls - https://github.com/BerriAI/litellm) and was wondering if we could be helpful.
Added support for Claude, Cohere, Azure and Llama2 (via Replicate) by replacing the ChatOpenAI completion call with a litellm completion call. The code is pretty similar to the OpenAI class - as litellm follows the same pattern as the openai-python sdk.
Would love to know if this helps.
Happy to add additional tests / update documentation, if the initial PR looks good to you.

also curious to understand - why did you implement your own abstraction classes vs. just using langchain?

Hey @stoyan-stoyanov - thanks for the quick revert. Totally understandable concerns. Any chance you could do a <15min chat sometime this/next week? Would love to understand concerns better and see if we could help

Sure, I am pretty packed at work this week and next week I am traveling but we can have a quick chat the week after

Works - attaching my calendly to make scheduling easier, feel free to grab any time that works
https://calendly.com/d/4mp-gd3-k5k/litellm-1-1-onboarding-chat

Hi @stoyan-stoyanov, Krrish and I work on LiteLLM together.
Noticed you recently added Claude-2 - I'd love your feedback on why LiteLLM was not good enough for LLMFlows. We'd really appreciate even a quick 10min call to hear your thoughts

Hey, @ishaan-jaff , @krrishdholakia sure - let's have a call :)

Sounds good - followed up on LinkedIn @stoyan-stoyanov
Also attaching our calendly here if that's helpful - https://calendly.com/d/4mp-gd3-k5k/litellm-1-1-onboarding-chat


I18N, option to define all text labels

In order to support I18N, we should allow defining all texts. Related to #49

Fixed by #53


Makefile for easier docker commands

On some projects, I've created a Makefile in order to easily run various docker commands, such as make start to run docker compose and startup the containers, make wp to run bash and allow WP-CLI commands to be executed, etc.  If this sounds like it would be useful, I can submit a PR with a Makefile for running commonly-used commands.

Can you explain a bit more the benefits that we'd gain over just running the docker-compose commands (make start vs docker-compose up, etc) directly? I'd hate to add another dependency for people to install just to get a working env.

@petenelson glad you found the bin! Do you have specific suggestions for any other scripts or commands?

@kopepasah here's a PR for additional bin helper commands https://github.com/10up/wp-local-docker/pull/129

Suggest using WP Local Docker V2, which has all kinds of helper commands like this :)


Transaction would exhaust the block limits

hi
i try to deploy a.wasm file on substrate using function system.setCode.
i got the limit error below.
code byte for a.wasm is just 91,594 bytes.
what do i modify for this successful? MaximumExtrinsicWeight -> percent up?
parameter_types! {
pub const ExtrinsicBaseWeight: Weight = 0;
pub const BlockHashCount: BlockNumber = 2400;
/// We allow for 2 seconds of compute with a 6 second average block time.
pub const MaximumBlockWeight: Weight = 2 * WEIGHT_PER_SECOND;
pub const AvailableBlockRatio: Perbill = Perbill::from_percent(75);
/// Assume 10% of weight for average on_initialize calls.
pub MaximumExtrinsicWeight: Weight = AvailableBlockRatio::get()
	.saturating_sub(Perbill::from_percent(10)) * MaximumBlockWeight::get();
pub const MaximumBlockLength: u32 = 20 * 1024 * 1024;
pub const Version: RuntimeVersion = VERSION;

}
code: Bytes
a.wasm (91,594 bytes)
-error-
system.setCode
1010: Invalid Transaction: Transaction would exhaust the block limits

You need to overwrite the weight of the transaction and that can be done from polkadot-js.

can you explain it in details?
(if you share the link, it is ok)

Please read the tutorial: https://substrate.dev/docs/en/tutorials/upgrade-a-chain/sudo-upgrade#upgrade-the-runtime


@sock.before_request functionality

For my app's HTTP API endpoints, I use @blueprintname.before_request() to easily perform common actions automatically for all requests (e.g., authorization validation). How might I do the same with flask-sock when a client first connects?
I didn't have any luck using @sock.before_request and @sock.bp.before_request().
I also tried looking at the source code to see if I could help add this functionality myself. Can someone point me in the right direction?
Thanks!

If you have an app-level before request handler, it will also be invoked for your websocket routes.
Or are you thinking in a before request handler that is specific to WebSocket routes? That doesn't seem that useful to me, since the whole point of a before request handler is to avoid repeating code across endpoints. This is normally not the case with WebSocket, since you usually have a single WS route in your app.

@miguelgrinberg Interesting. My application has several dozen REST API endpoints and related UI views for viewing a list of users, list of tasks, a specific task, etc. What I'm interested in doing is "upgrading" some of the UI views to use websockets so the content gets updated in real-time (without polling or a page refresh).
My thought was to have separate websocket endpoints similar to my REST endpoints:

/ws/users
/ws/users/int:user_id
/ws/tasks
/ws/tasks/int:task_id
...

Then when a UI view loads (I'm using Vue.js), it would start a websocket connected to the relevant endpoint to get data specific  for that view. The connection would persist until the user navigates to a different UI url - in which case a new WS connection would start related to that view.
Is that approach different from what you've typically done? Otherwise, with a single WS endpoint each client would need to somehow specify what kind of data it wants to receive when it connects (e.g., items list page only wants to receive items list data - not task data).
Appreciate your input and efforts with flask-sock!

@dsposito-ubtech To me that is a strange approach, because you have an SPA, so you can have a single WebSocket connection that provides data to all your views. Closing and reopening a WebSocket is inefficient and somewhat approaches HTTP.

Otherwise, with a single WS endpoint each client would need to somehow specify what kind of data it wants to receive when it connects

In your solution the client still specifies what data it wants to receive in the connection URL. How is that any different than indicating what it wants in a WebSocket message?

Fair enough - that does seem more efficient! :+1:
I'll do some more research to figure out how best to do this. My initial thoughts: after each websocket client connects, it will send a message to the server to tell it which data "topics" it wants to subscribe to. So as the SPA url/view changes it would send a new
websocket message to the server to update its list of subscriptions (e.g., "I'm now on the Tasks list page - send me that data and no longer the Users list data).
I'm also planning to look into Socket.io to see if that may help with managing the connections, topics/events, etc.
Thanks again for your input on this - really appreciate all that you do for the Python community!

@dsposito-ubtech the subscription style usage is actually fairly common and has a solid foundation in the Observer pattern. That sounds like a great way to do what you need. 👍


No reliable data transmission with RDP

Hi all,
I was modifying the csp_server.c and csp_client.c files to test whether transmitting data over a lossy connection with RDP actually is reliable. I implemented a proxy that drops 10% of the packets to see if this is the case.
Each packet contains an increasing counter to check for dropped packets. With the code files attached below I could not get a reliable connection to work. Additionally I had  to always create a new connection as there is, from what I could find, no way to tell if a connection is still alive. Same goes for checking if a message has actually been sent as the return type of csp_send() is void.
Am I missing something on how to properly get RDP to work?
Any guidance would be greatly appreciated.
Thank  you
files.zip

Your client spawns 5 new connections per second, one for each packet.
With RDP you are supposed to create one persistant connection and call csp_send() in a loop on the client, to continously stream data through.
Your server is correct in that it accepts a connection, and within this connection it starts a csp_read() loop.
Opening so many connections wont work because there is a conn close timeout of several seconds.

/* Client task sending requests to server task */
void client(void) {

	csp_print("Client task started\n");

	unsigned int count = 0;

	while (1) {

		usleep(200000);

		/* Send data packet (string) to server */

		csp_conn_t * conn = csp_connect(CSP_PRIO_NORM, server_address, SERVER_PORT, 1000, CSP_O_RDP);
		if (conn == NULL) {
			/* Connect failed */
			csp_print("Connection failed\n");
			continue;
		}

		/* 2. Get packet buffer for message/data */
		csp_packet_t * packet = csp_buffer_get(100);
		if (packet == NULL) {
			/* Could not get buffer element */
			csp_print("Failed to get CSP buffer\n");
			return;
		}

		/* 3. Copy data to packet */
        char msg[512];
        memset(msg, '\0', 512);
        sprintf(msg, "HelloWorld:%d", count);
        int len = strlen(msg);
        printf("Len: %d\n", len);
        memcpy(packet->data, msg, len + 1);

		/* 4. Set packet length */
		packet->length = len + 1; /* include the 0 termination */

		/* 5. Send packet */
		// if (csp_rdp_send(conn, packet) != CSP_ERR_NONE) {
        //     csp_print("Could not send packet\n");
        // }
        csp_send(conn, packet);
        csp_print("Sent packet\n");

		/* 6. Close connection */
		csp_close(conn);
        count++;
	}

	return;
}
/* End of client task */

Having only one connection it works. Thank you
As a follow-up I added packet corruption and enabled the CSP_O_CRC32 flag. Despite using CRC after a while the connection is dropped or packets arrive corrupted.
Is there any way to know, from the client side, if a certain connection is still open?
Any ideas why some packets still arrive corrupted? I am currently corrupting 1% of the bytes, which should be handled just fine by CRC32.

What version of CSP are you using?
From 2.1 and onwards (current develop branch) the default behaviour is to protect both the header and the frame contents with the CRC. Prior to this, in version 1.6 only the payload is protected. And CSP was relying only on lower layer checksums.

I am using the latest commit of the development branch. I think I found out why it was not working:
Given that the corruption of packets would require more ACKing and retransmission of packets, and the timeout of csp_read() was quite short, the call would timeout. Then, given how the example code is structured, the connection would be closed. Simply increasing the timeout of csp_read fixes the problem.
One question remains: How can I know, when using RDP, if a connection has been closed by the other party? Is there any function aiding with that?


which version i should use as windows 10 user

i have
winows 10 pro
version: 1909
os build: 18363
cpu: i3

Please provide specific processor model

Please try the following version
https://github.com/Alex313031/Thorium-Win/releases/tag/M117.0.5938.157
If the above version doesn't work, try using the SSE3 version
https://github.com/Alex313031/Thorium-SSE3/releases/tag/M117.0.5938.157-1

the SSE3 version ,work for me thnx

the SSE3 version ,work for me thnx

https://github.com/Alex313031/Thorium-SSE3/releases/tag/M117.0.5938.157-1


Stacked create issue with stroke width

Bug report
I've a graph with stacked true, and I've 3 series, with a stroke: [5,0,0], but  the 2nd bar is losing 5px.
Codepen
https://codesandbox.io/s/vue-basic-example-bu8ne
Explanation
As you can see on the codepen, I tried with an bar type and line type, both with different problems.
In my project I am using the type line, and the downside is that the bar 2nd serie (green bar) is losing width of 5px.
if I put stroke with all 0, it works fine. But I want to add stroke to the 1st serie.
EDIT: This bug on the type line was introduced on the last version 3.17.0

I confirm the issue.
It will be addressed soon.


Running Morpheus on a 32 Bit Windows

I'm trying to run Morpheus on a 32 Bit Windows 7 SP1 and get the following errors. Is it a 32 Bit issue? Can Morpheus be run on a Win32 system?
Thanks
Initializing...
Starting C:\MorpheusPostAcquisitionSearch\UF_Top15_QEPlus_Test.RAW...
Extracting and preprocessing MS/MS spectra...
System.Runtime.InteropServices.COMException (0x80040154): Retrieving the COM class factory for component with CLSID {1D23188D-53FE-4C25-B032-DC70ACDBDC02} failed due to the following error: 80040154 Class not registered (Exception from HRESULT: 0x80040154 (REGDB_E_CLASSNOTREG)).
   at System.RuntimeTypeHandle.CreateInstance(RuntimeType type, Boolean publicOnly, Boolean noCheck, Boolean& canBeCached, RuntimeMethodHandleInternal& ctor, Boolean& bNeedSecurityCheck)
   at System.RuntimeType.CreateInstanceSlow(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
   at System.RuntimeType.CreateInstanceDefaultCtor(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
   at System.Activator.CreateInstance(Type type, Boolean nonPublic)
   at System.Activator.CreateInstance(Type type)
   at Morpheus.TandemMassSpectra.Load(String rawFilepath, Int32 minimumAssumedPrecursorChargeState, Int32 maximumAssumedPrecursorChargeState, Double absoluteThreshold, Double relativeThresholdPercent, Int32 maximumNumberOfPeaks, Boolean assignChargeStates, Int32 maximumThreads)
   at Morpheus.TandemMassSpectra.Load(String rawFilepath, Int32 minimumAssumedPrecursorChargeState, Int32 maximumAssumedPrecursorChargeState, Double absoluteThreshold, Double relativeThresholdPercent, Int32 maximumNumberOfPeaks, Boolean assignChargeStates, Boolean deisotope, MassTolerance isotopicMZTolerance, Int32 maximumThreads)
   at Morpheus.DatabaseSearcher.DoSearch()
Finished!

Hi somakray,
Morpheus works on 32-bit Windows. It sounds like you just need to install Thermo MSFileReader.
-Craig


Multi-architecture support #1238

#1238

pushes container basis to quarkus v<IP_ADDRESS>
implements multi-arch (arm64 and amd64 only since thats whats provided upstream @ quarkus)
no cleanup of old 'arch' build yet, since I don't know what's dependent upon it.

This PR may be imperfect as I don't yet know how to run a test suite here and am hoping PRs have automated QC actions.
Release Note
- pushed quarkus version to v<IP_ADDRESS>
- implemented multi-arch for amd64/arm64

So I just 'manually' added qemu support. This isn't the best way! The cleanest way is through github actions, as described here: https://github.com/marketplace/actions/docker-setup-buildx
It seems though that the build architecture/setup for this project is a bit 'manual' and I don't want to pull all that refactoring into this (intented to be) simple, small fixup PR.

I'm currently not able to test this in my local branch due to a dependency on kind-registry which cannot be resolved. I'm pretty sure thats environment-specific so please start the pipelines here again so we can see how it works here. Thanks!

@tobiasoort Can you squash the commits into single one?  It's too many for the simple fix.
@phantomjinx What do you think about the fix?

We can "squash and merge" instead of "rebase and merge". I think however the parameter setting for the platform is a must before going.

Do we want the built arch(s) in the VERSIONFILE?

I'll work on this PR in my fork and create a new one because this mess is only causing useless notifications for you.


V3.0: Making Factor.io Agentless

Making Factor.io Agentless means that Factor.io runtime will not be dependent on the Connector Service. Instead, it will load the factor-connector-* gems dynamically and use the service directly instead of using WebSockets to connect to the Connector Service.
factor-io/factor

Update factor registry to update Gemfile, but not connectors.yml
Runtime to use gems not web sockets (this is the gist of the work)
Remove any association with connectors.yml.

factor-io/connector-api

Cut out any complexity with Celluloid if unnecessary.

factor-io/docs

Remove mentions of connectors.yml
Remove mentions of using the connector service
Rename "connectors" to "plugin-ins"

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/8560649-v3-0-making-factor-io-agentless?utm_campaign=plugin&utm_content=tracker%2F4030409&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F4030409&utm_medium=issues&utm_source=github).

Making progress on this. Here are the items left on the todo:

Add back on_fail handler (ExecHanlder)
Make calls to listener and run actually async (currently have intentionally blocking code for testing)

A little useful article I found on instance mehtod meta programming: http://www.leighhalliday.com/ruby-metaprogramming-creating-methods


Bug: HorizontalRuleNode breaks TableCell in some case

Lexical version:
Steps To Reproduce

insert a HorizontalRuleNode in table.
select above blank paragraph before HorizontalRuleNode and press delete key
the empty paragraph will deleted, keep press delete key the TableCell will deleted too, see the video on detail.

https://github.com/user-attachments/assets/decd08b6-0fe7-4897-9ce2-852f11d45e69

This no longer reproduces in the same way, but there are still some edge cases here because once the divider is deleted you can enter text directly into the TableCellNode, it does not create a wrapping ParagraphNode.


grafana: 11.2.1 -> 11.2.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for grafana is: Gorgeous metric viz, dashboards & editors for Graphite, InfluxDB & OpenTSDB
meta.homepage for grafana is: https://grafana.com
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 11.2.2 with grep in /nix/store/2qzp053izm31dzbljg6is6gymw0sczqv-grafana-11.2.2
found 11.2.2 in filename of file in /nix/store/2qzp053izm31dzbljg6is6gymw0sczqv-grafana-11.2.2

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

grafana

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/dd09921858285d0e345de0d6aefd86237387adf0#grafana

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

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

1 package built:

grafana

Maintainer pings
cc @offlinehacker @fpletz @WilliButz @globin @Ma27 @Frostman for testing.

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

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

For anyone with merge permissions: please leave this for now.
There's a tag, but no release and I don't know if their process broke somewhere in the middle or something. I'd like to clear this up with upstream first (didn't have time so far), before I'll merge this.

There's a release now and the relevant commit (grafana/grafana@086b47f) got apparently applied twice to both 11.2.1 and 11.2.2. So all good from that side. Testing and merging.


Display default search provider first in search bar

This issue is about displaying the default search provider in the search bar of the discover pane first. (When there is no default selected, the default order is OB1, Duo, etc. The order is only updated when the user revisits the discover pane the next time.)
Showing the default search provider first in the search bar is only natural from a user experience point of view, especially as the list of providers grows over time. It is also a step towards fair handling of the search providers in the reference client (to avoid constantly prioritising OB1) and was agreed upon in a private discussion with @hoffmabc on August 31st.

Maybe I'm confused but I'm not sure how this would be done differently right now.

Clearly none is selected and no default selected which is what we agreed to is it not?

He means that DUO icon always stays on the right side of OB1 icon, even if it has been selected as default.
I think that when there is no default selected yet, their order of display should be randomized (just like web browsers do when they show their available search providers).

We've already discussed the randomization piece. So the order will remain fixed. I'm not sure I see the value of moving the icons after selection since we already have a selected indicator.

Icons are clearly greyed out and the default one is brought to forefront. Seems very confusing if the icons are switching around.

I agree that randomization on search page is confusing.
It would be better to do as web browsers do: user selects his default search provider on onboarding, and the onboarding list of providers is randomized.

Just to catch you up the idea at this point in time is that realistically there are only two search engines and only two reliable ones at the moment so randomizing especially once we start adding in lower tier less trustworthy sources would be bad for users.
We won't rehash the argument about leveling the playing field for all comers over OB1. We've already done this and the short-term compromise was to do it this way.
My only issue here with this request is that it seems silly to make a big deal of switching two icons and kind of a waste of time to develop this when there are more important pressing issues.

The issuis is about what happens once a selection has happened, not before the selection has happened. Here is the relevant part of the private Slack discussion with @hoffmabc. (Happy to share more about this discussion if the screenshot does not provide enough context.)

I understood that the search bar would follow the user's selection once the selection has been made.

I'm not a UX guy but having the selected provider first in the list seems most natural to me as a user. Gray the others out too, but have the selected one first. Just my 0.00000002btc

@jjeffryes just change it to whatever he wants so we can move on. I'm tired of the little change requests to this.

I think the current design is fine. We can put considering changes to it in the backlog, and schedule that along with other features we need to work on.


Several BD compatibility fixes

This fixes

BD plugins using module.exports
Saving/loading bd plugin settings
Unable to interact with setting panels

Also closes #124

The version of ed_settings in this PR completely removes the settings button from all plugin configs. bd_shit in this PR doesn't allow you to click the settings button (only tested when old ed_settings was in plugins folder, since as mentioned before settings won't even show if new ed_settings is present). Saving plugin settings when only using dom_shit from this PR works fine though.

Correct, the button should be gone, but they would show as separate tabs on the left hand side instead. This is intentional.

Thanks, this fixed essentially every issue I was having.


Cosmos read Database, Container and Item APIs throw error on unsuccessful read

Is your feature request related to a problem? Please describe.
When reading Database, container or item from cosmosClient, if any of this is not found, the API throws an error. Because of this behavior, customer has to wrap the API with a try / catch block. Since these are common APIs and are used at a lot of places in customer's code, it is not a good user experience.
Describe the solution you'd like
At least these common APIs should return empty Mono, Flux if database, collection or item is not found.

Some comments from other folks on this issue:
"Cosmos DB used to return a response code in the item response object when readasync(). Now it throws an exception which means callers need to use try catch in order to check for common use cases.
Perform a query with the cosmos SDK and look for an item that doesn't exist via the Read method (by Id) and notice exception instead of result response."
"The Cosmos DB SDK design decision increases friction for common scenarios by throwing an exception for items not found. This means that every retrieve call now requires a try/catch method, which is easy to omit and unnecessarily clutters code. The previous implementation was more elegant."

This is done.


Support for Image conversion and comperssion

Any chance of adding Image Conversion and Compression support?
Been looking through the code and it seems it would be simple enough since the resulting image is already loaded into a canvas and exported from there. So it would basically just require an additional setting specifying the output format you want and the quality.
Then in the imageCropper, instead of using
this.croppedImage.src = this.cropCanvas.toDataURL("image/" + this.fileType);
you would end up with someting like
if (this.settings.OutputFormat){
  //Since compression would only work on jpg and webp
  if (this.settings.OutputFormat == "image/jpg" | this.settings.OutputFormat == "image/webp"){
    this.croppedImage.src = this.cropCanvas.toDataURL(this.settings.OutputFormat,this.settings.OutputQuality);
  }
 else {
   this.croppedImage.src = this.cropCanvas.toDataURL(this.settings.OutputFormat)
 }
}
else {
 this.croppedImage.src = this.cropCanvas.toDataURL("image/" + this.fileType);
}
Im assuming that this might break your circular image transparency?

Pull requests are always appreciated.
More info:
https://help.github.com/articles/about-pull-requests/
as for now, it's not a core function for "cropping images", Neither is it an issue, as you can always do this after obtaining the base64 yoursel.


refactor(metrics): standardize Prometheus metrics

What this PR does / why we need it:
This PR should be considered as a breaking change.
Refactored the metrics so that they meet the K8 Instrumentation Guideline, the changes include:

Removed redundant labels.
Added a component label to all metrics; it takes the value of metric namespace.
Renamed according to the naming convention.
The Histogram type request duration metric use standard bucket sizes (which produce much more data, which will put more pressure on Prometheus, but is also more precise).
Changed the unit of the request duration metric from milliseconds to seconds.

As a result, the metrics should now look a lot like the APIServer request metrics.
Also, removed the Summary type request latency metric: Histogram metrics function similarly to Summary metrics and are generally considered as the superior option, since it supports all quantile values instead of just 0.5, 0.9, and 0.99. This should balance out the extra data volume added by the refactored Histogram metric.
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @supereagle @ddysher @iawia002

Release note:

NONE

I'm submitting this PR just so that we can start discussing it. It does not mean that we should merge it immediately. It is a breaking change and it causes Nirvana to put more pressure on Prometheus under most circumstances (to make the Histogram metrics more precise and meaningful).

What are the benefits of meeting Kubernetes guideline for Nirvana, apart from consistency?

Consistency by itself is a major benefit. In any production setup, one would face hundreds if not thousands of metrics. There is no way you would read the description on each and every one of them. By following the convention, you can understand the metrics by looking at their names.
For example, the first word of a metric name is usually its targeting component, and if a metric end with a _total postfix, then it's a counter.
If also allow you to do something like  {__name__=~"(.+)request_duration_(.+)"} to pull out request latency metrics from all components.
Non-naming conventions, on the other hand, are usually good for performance or/and precision. For example, the removed redundant labels save memory and make queries faster. The added Histogram buckets make histogram_quantile function much more predictable and precise.

Are we actively using those Summary-based metrics now?

We are not using any Nirvana metrics right now. Even if we are, Summary is totally replaceable by Histogram so it doesn't matter. As for Kubernetes, almost all Summary-typed metrics have been depreciated as of v1.17.0. Other open-sourced projects are slower and more unwilling to adapt.

The code LGTM, but since I'm not familiar with Prometheus, I can't give more comments about this. And if there is no more comment, I think we can merge it into the next release (0.3.x).

LGTM
@lichuan0620 thanks for the detailed explanation!

LGTM

/lgtm

@iawia002 make sure to make it explicit in the release note.

This PR should be considered as a breaking change.

@iawia002 make sure to make it explicit in the release note.

This PR should be considered as a breaking change.

Will do


List the "CATARACTS Semantic Segmentation 2020" variant of CATARACTS

https://cataracts-semantic-segmentation2020.grand-challenge.org/

This has already been considered. The dataset linked contains all of the challenges.


post handler for event and announcement

Looks ok have you tried testing these endpoints using postman? Sometimes binding to Time type is a little wacky.

No, I haven't yet... I'm trying to figure out postman.


Add note about having to access with https

From https://discord.com/channels/880681510661926982/1184884581644636292/1184884581644636292

Thank you @arctixdev


[giti] Add an option to checkout a branch by typing the name

The main user flow to checkout a branch is giti branch checkout.
Add an option to type in the branch name as 4th cli argument:
giti branch checkout main

In some cases the user get's blocked when they try to checkout another branch while they have staged, but not committed changes:


Illegal instruction: 4

and then also:
*** Skipped building ZipArchive due to the error:
Dependency "ZipArchive" has no shared framework schemes

I'm not sure if these are Carthage bugs ... or ... computer bugs ...

I think you should be using --use-submodules --no-build, in which case you should never see that message.

Ohhhh right :+1:

FYI, I've been dabbling on a real fix for those module warnings/errors in phatblat/objective-git#1. The change is to build libgit2 as its own "git2" module instead of the "ObjectiveGit/git2" submodule that we're maintaining now. The result is git2.framework but I haven't gone as far as including OpenSSL and libssh2 yet.

That'll be :sparkles: if it works!

There are some other changes in #516 to get the build to work on Travis. Specifically, a newer version of xctool is necessary (e95cd59f5).

Indeed :disappointed:
I opened https://github.com/facebook/xctool/issues/605 to see when they expect a new release.

Cool, thanks. I was starting to think we should just live with xctool at HEAD and open an issue to fix it later. If they can push out a release that will be even better.

@joshaber thanks for opening that issue, looks like it's coming "in a week or so" https://github.com/facebook/xctool/issues/605#issuecomment-147765541
I've been keeping watch for the new release; I'll update #516 as soon as I see it's released

This is ready now :pray:

:ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship: :ship:


Bump Drupal image version to 8.2.3-r0

Bump Drupal image version to 8.2.2-r2
Fixed security issues (Ref DRUPAL-CORE-2016-005).

@k8s-bot ok to test

I signed it!

I signed it!

Manually tested and this lgtm

@k8s-bot ok to test

@k8s-bot e2e test this


disable other input forms when 'no' is selected for 'include this operator'

@hanbyul-here can you assign this to one of the milestones to indicate whether it should happen before initial release or after?


fix(github): rename service to github-actions

The coveralls API refuses to accept submissions with service=github, but accepts
submissions with service=github-actions.

I fear that your commit docs(circleci): token is now required has broken your circleci tests. They all fail their last stage, sending data to coveralls with:
CoverallsException: Not on TravisCI. You have to provide either repo_token in .coveralls.yml or set the COVERALLS_REPO_TOKEN env var.
ERROR: InvocationError for command /root/project/.tox/coveralls/bin/coveralls --verbose (exited with code 1)

Looks like I just forgot to check the button for having CircleCI pass secrets to PR builds -- tests pass upon re-running.
This LGTM as well -- I'll get this deployed as v1.9.2. Let me know if you continue to have any issues!

Thanks, v1.9.2 works like a charm now.
If you ever run across why the coveralls API rejects service='github' let me know, I'm curious.

@ArturKlauser you may be interested to know that the Coveralls API seems to now be accepting service='github' -- as of my testing of #216, I've found that this now works perfectly fine.
I still have no idea whatsoever... but (just guessing here) maybe it was a bug they finally fixed?

Interesting.

As far as I can tell, the service_number is only parsed for parallel builds -- I tried a non-parallel build with service='github' and everything seemed to work just fine. May have missed something, of course!


MudTextField's EventCallbacks [OnKeyDown, OnKeyUp or OnKeyPress] doesn't work on My Mac(MacCatalyst), while it works correctly on Safari(16.4)

Bug type
Component
Component name
MudTextField
What happened?
MudTextField's EventCallbacks [OnKeyDown, OnKeyUp or OnKeyPress] doesn't work on My Mac(MacCatalyst), while it works correctly on Safari(16.4)
Expected behavior
MudTextField's EventCallbacks [OnKeyDown, OnKeyUp or OnKeyPress] should work on both My Mac(MacCatalyst) and Safari(16.4)
Reproduction link
none
Reproduction steps

...
Relevant log output
No response
Version (bug)
6.1.9
Version (working)
No response
What browsers are you seeing the problem on?
Safari
On what operating system are you experiencing the issue?
Mac OSX
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

same here. android and pc works, but mac app doesn't.

已收到，冯唯一。

@AveiShriety @williamlee1982 Would either of you be able to provide some form of reproduction that I can debug with?

@AveiShriety @williamlee1982 Would either of you be able to provide some form of reproduction that I can debug with?

<MudTextField AdornmentColor="Color.Secondary" Adornment="Adornment.Start"<EMAIL_ADDRESS>Label="@T("Lot Id")" @ref="LotIdTextField" OnKeyPress="btnView_Clicked" Clearable="true" @bind-Value="@LotId" Variant="Variant.Outlined" />
OnKeyDown, OnKeyUp or OnKeyPress , none of them worked on Mac app.

I understand this, but I have never used Catalyst and it is not feasible for me to start from scratch with it to debug this issue. I assume it is a Maui app?

Hello,
I would like to emphasize the importance of providing a reproduction repository when addressing issues. If it is not feasible to reproduce the problem on trymudblazor. Creating a project from scratch can be time-consuming, sometimes even surpassing the time required for the actual fix. Unfortunately, the contribution team may not be able to dedicate their valuable free time to do the reproduction project. Having a reproduction repository readily available greatly saves time and effort for everyone involved.

Hello,
After a long time searching about this problem for Mask issue on Android...
KeyPress is obsobete and KeyboardEventArgs are not returning values por Key and Code too.
More Information and problem reproduction at:

https://github.com/MudBlazor/MudBlazor/issues/6472#issuecomment-1552146630

https://try.mudblazor.com/snippet/cEwnuzPChgTGgaBK

Hi @elyelrubens,
It appears that you may have posted your comment in the wrong issue. The current issue is specifically related to MacCatalyst and is not associated with Mask.

Hi @elyelrubens,
It appears that you may have posted your comment in the wrong issue. The current issue is specifically related to MacCatalyst and is not associated with Mask.

I removed the comment, sorry.

Hi @ScarletKuro
I know the importance of reproduction. But now my issue is the basic component MudTextField's EventCallbacks. The problem occurs just between MacCatalyst and Safari(16.4)

I understand this, but I have never used Catalyst and it is not feasible for me to start from scratch with it to debug this issue. I assume it is a Maui app?
"I assume it is a Maui app"  ---->   MacCatalyst is just equal to Window Machine.  MacCatalyst is provided by Visual Studio 2022 for Mac.
Oh....... so that's no way to fix it? Will any other one deal with this issue?

Ok, can you create a demo MacCatalyst project that exhibits the issue and link the repo here?

Ok, can you create a demo MacCatalyst project that exhibits the issue and link the repo here?

Demo Repo
I've create a demo repo

I'm slightly confused here because @Mr-Technician asked a concrete yes-no question about whether it's a MAUI application or not. From the answer given, I understood that it's just a typical wasm application. However, in the reproduction repository, I see a MAUI app and there is already an issue regarding this in MAUI repository.
No offense here to MAUI and everyone involved, but we most likely won't investigate this, because based on our past experiences, we have found that when we invest a significant amount of resources into investigating why certain MudBlazor functionality doesn't work on MAUI, it often turns out to be a MAUI-specific problem rather than an issue with MudBlazor itself. Therefore, I would like the community to first check with Microsoft to determine whether the problem lies on their side. Since there are no special requirements for a Blazor library to work on MAUI, if Microsoft confirms that the problem is not on their side, it would make sense to create an issue here.
I hope this clarification helps. If you have any further questions or concerns, please let me know.

This seems relevant: dotnet/maui#13934
Linked in the thread: https://developer.apple.com/forums/thread/721141
It appears that this is not an issue with MudBlazor, Blazor, or MAUI, but rather with Apple's WKWebView. (See Eilon's repro on Mar 22 in the dotnet/maui thread reproducing the issue without using .NET or MAUI.)

emmmm....I could not accept your statement slightly. Because it works fine on MacOS Safari(16.4). If it's really the mistake of Apple's WKWebView, I assume Safari will not take effect.

This is a bit weird am seeing this WebViewSource displaying the charts while using the local html . However when I have the WebViewSource source as html string, we Are not seeing  chart  key /scroll event works. If its working with a local html file why not with a html string ?

已收到，冯唯一。


Implements interrupt-safe transaction log

When an interrupt happened on a thread during writing the transaction
log, the underlying FileChannel was closed as a result. Any further
use (even from another threads) threw a
java.nio.channels.ClosedChannelException rendering the transaction
manager unusable since the transaction logs were never reopened. This
commit changes this behavior, log file operations reopen the file if
it was closed by an interrupt of another thread.
fixes bitronix/btm#45
Commit by Tibor Billes, Miklós Karakó, Balázs Póka

Unfortunately, by introducing synchronized methods this pull request reverts the disk journal to a sequential write model -- only one thread can write at a time.
To see why this was an issue in 2.1, you can read here.
In 3.0 (SNAPSHOT), the disk journal was re-architected to allow a concurrent write model, with large performance gains.  You can read about it here.
Unfortunately, this leaves #45 as yet unresolved, but you now have context if you want to have another go at addressing the issue.

Palacsint, please attach results of performace tests described on https:/hod?/github.com/brettwooldridge/bitronix-hp/wiki/Disk-Journal-Improvements page.
Why don't you make simply patch of original code only with  reopenFileChannelIfClosed() method?

@krokodylowy, actually we have abandoned this patch and the idea too for a custom SecurityManager which delays interrupts in our application, so interrupts never reach the FileChannel.

Thanks for response. Did you use your patch in any enterprise solution with heavy load? Do you get any loss of performace?
ClosedChannelException is serious gap in Bitronix. It can be 'fixed' by better design of user application but CCE still can occur in any moment and can hang enterprise system.


Register InstrumentModules separately from ChannelLists

[x] Register channels and channel lists in two separate dicts. This is done on the basis of the ABC to avoid a circular import

Open questions

[ ] Deprecate submodules dict (yes but not now)
[ ] Adapt snapshot etc to use the channels rather than (again not now)
[x] Better names for the new dicts (channels is not an option since many instruments uses that as the name of the channel list)

Bors merge


Text only renders before cursor when launched from Powershell

Built on Windows and opened the same file (kod.go) in both Visual Studio Code and kod launched from Powershell.  The screenshot is from the builtin terminal in VS Code, but I get the same result in a standalone Powershell window.
In the following screenshot the cursor is in the same position in both panes, on line 16 (15 in kod).  You can see that the rest of the file past the cursor does not render in kod.  The entire file is rendered on first load, but as soon as the cursor is moved the rest of the file disappears.
Additionally, there is noticeable flicker whenever the cursor moves.

Using ^s to save the file will cause another screen flicker, but will show the entire file as though the editor was relaunched.  Moving the cursor again removes the rest of the file.

This is related to issue #7 and has something to do with the linecache's ApplyUpdate method. The core has finished/changed the protocol for line invalidation so i imagine this is where the problem lies.