After installing NPM with brew I can't use NPM to install Bootstrap

I get the following error
npm WARN enoent ENOENT: no such file or directory, open '/Users/myusername/package.json'
npm WARN mysername No description
npm WARN myusername No repository field.
npm WARN myusername No README data
npm WARN myusername No license field.
How can I fix this ?

Ask npm, we don't support it.

I'm 99% sure this isn't brew related looking at those errors, so yeah, probably best to check in with npm or their documentation. If they say otherwise, let us know!

Ok, thanks.
On Fri, Aug 5, 2016 at 1:07 PM, Dominyk Tiller<EMAIL_ADDRESS>wrote:

I'm 99% sure this isn't brew related looking at those errors, so yeah,
probably best to check in with npm or their documentation. If they say
otherwise, let us know!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Homebrew/brew/issues/632#issuecomment-237744401, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AL5sdqJNIdFEo5uyrD3zhhVxH4dQJNZTks5qcqj0gaJpZM4JcKrZ
.


[Feature Request] support fuzzy search

💭 Describe the feature

types, scopes and issuePrefixs list support fuzzy search

💡 Proposed Solution
as inquirer plugins util function

After testing, it is found that the use of fuzzy search for type list is not suitable for full content search, only suitable for fuzzy search of short characters, like feat fix ...🤔


Uncaught Error: No protocol method IReset.-reset! defined for type number: 1920

Seems like we're trying to reset! something that isn't an atom:
        window-width      js/window.innerWidth
        handle-window-resize (fn [e]
                               ;; N.B. I don't think this should be a perf bottleneck.
                               (reset! window-width js/window.innerWidth))

Sorry about this, I think it was a merge gone wrong.


Message moderation/hiding

As opposed to or in conjunction with #64
This is possible in 1.19:

Custom servers can hide player chat messages from display via a new network packet

Not sure bungee or velocity APIs will support this (they definitely don't at the moment), or if it's even practical, plus we target 1.16 API rather than 1.19 -- but, maybe?

1.19.3 note: this feature has been adjusted somewhat, the packet now displays an uncustomisible "Message deleted" thing in chat if you do this. There's actually no API provided by anyone to do this afaik, even on spigot or paper -- though if there is I'd love to be proven wrong as it is an interesting new thing.


Working with CSV files in java

Topic Suggestion
Working with CSV files in java
Proposed title of article
Working with CSV files in java
Proposed article introduction
The CSV represents Comma-Separated Values. It is a basic document design which is utilized to store plain information in straightforward text structure, like an accounting page or data set. The documents in the CSV organization can be imported to and sent out from programs (Microsoft Office and Excel) which store information in tables. The CSV document utilized a delimiter to recognize and isolate various information token in a record. The CSV record design is utilized when we move plain information between programs that locally work on incongruent organizations.
Key takeaways

Step by step instructions to make CSV document, utilizing notepand and Microsoft Excel.
Java String.split() technique.
Java scanner class.
Using open CSV API.

Article quality
In this article I will acquaint client with CSV documents utilizing java,am additionally going to take the client through the strategies of making a csv record utilizing note pad and Microsoft Excel, likewise I will disclose to the client how CSV records are perused and deciphered utilizing various techniques. I trust the client will appreciate to know how csv documents work in java.
References
N/A

This topic is already covered by a lot of blog sites - How does your article differ from them? @tesswambui

hello @ahmadmardeni1, I have looked at those articles published in those blogsites and I found them offering very shallow content. Also its hard to understand the content published. In my article am going to dive deep into csv files in java and how they work, I will use a beginner friendly language to deliver the content unlike the already published articles.

Sounds like a helpful topic - let's please be sure it adds value beyond what is in any official docs and/or what is covered in other blog sites. (the articles should go beyond a basic explanation - and it is always best to reference any EngEd article and build upon it).
Please be attentive to grammar/readability and make sure that you put your article through a thorough editing review prior to submitting it for final approval. (There are some great free tools that we reference in EngEd resources.)
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.
Please be sure to double-check that it does not overlap with any existing EngEd articles, articles on other blog sites, or any incoming EngEd topic suggestions (if you haven't already) to avoid any potential article closure, please reference any relevant EngEd articles in yours. - Approved @tesswambui


lodash\mergeWith.js issue

Hi, anybody else receiving this issue when using typescript?
[!] Error: 'default' is not exported by node_modules\lodash\mergeWith.js, imported by node_modules\gqty\Cache\dataCache.mjs
its doing when i try to include gqty into the library
import { graphql } from './gqty';
console.log("graphql",graphql)`

before i do this the build works (tsc, rollup -c rollup.config.js )

With #949 and the new @gqty/utils package this will be 100% fixed


How to get information about a video, such as width, height, and length? I seem to have not found any relevant methods.

Meedu controller.Video controller.value


PowerShell Common Launcher for Windows

Overview
PowerShell Core for Windows should support a common launcher that can start either a specified version of or a "default" version .
Changes Required

An executable or script that supports a -version  parameter and passes cmd line arguments to an instance of PowerShell.exe
The executable or script resides is %ProgramFiles%\PowerShell\powershell.exe
It determines the "default" instance of PowerShell by reading a value from a configuration file in its same directory. File type and format are flexible and should be driven by the ease of parsing the info.

Open Issues

The exact design has not yet been finalized. This is a place holder issue to note that work is likely required to support the desired behavior.
On non-Windows systems, the installer already creates a symlink on the path that points to the latest installed instance of PowerShell. No additional work is likely required in those environments.

Last discussion by @PowerShell/powershell-committee is that using file paths is sufficient and user can use symlinks to change default


Correction for raw affiliation Research Group on Alcohol & Pharmacodependences (GRAP) Centre Universitaire de Recherche en Santé (CURS)  INSERM U1247 University of Picardie Jules Verne Amiens France

Correction needed for raw affiliation Research Group on Alcohol & Pharmacodependences (GRAP) Centre Universitaire de Recherche en Santé (CURS)  INSERM U1247 University of Picardie Jules Verne Amiens France
raw_affiliation_name: Research Group on Alcohol & Pharmacodependences (GRAP) Centre Universitaire de Recherche en Santé (CURS)  INSERM U1247 University of Picardie Jules Verne Amiens France
new_rors: 01gyxrk03;02vjkv261;01c62av11
previous_rors: 01gyxrk03;02vjkv261
works_examples: W2887185890
contact: 1e4bdc297d0adb4d9515bef911217572:4236eb8480efdf640fd2e2810e302d9e @ u-picardie.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


ERROR Error: Uncaught (in promise): Error: Failed to scan code.

Good morning, I have cloned your project, followed every step, built and run on my device, and it worked perfectly, but since 3 days now I can't scan any QR or barcode, simply when I click on FAB, the camera doesn't start.

The only that I remember is that my Android Studio update the Gradle version to 8.0.2
Thanks in advance for your support
Eduardo Allan

Thank you for your request!
I close this as duplicate of https://github.com/capawesome-team/capacitor-mlkit/issues/38


Lack of security check and wrong use of libpcap API

I found an issue when reading source code 'pcap_interface.c'.
This issue is caused by forgetting to check the return value of libpcap API 'pcap_activate'.
According to libpcap document "If pcap_activate() fails, the pcap_t * is not closed and freed; it should be closed using pcap_close()."
So it's safer to check it, and use 'pcap_close()' if an error occurs.
https://github.com/ephracis/pylibpcap/blob/bcab6cd2b27eeae0be1b3899cc1edb7b1f487136/pcap_interface.c#L161-L163

Hi,
Thank you for finding this issue.
This repository is just a fork from https://sourceforge.net/projects/pylibpcap/ so this fix should be submitted upstream. Their bug page (where you can submit bug reports) is located here: https://sourceforge.net/p/pylibpcap/bugs/

Thank you for your remind. I will submit it upstream.


Method setupBundlePath doesn't work on Android.

I tried on android phone using setupBundlePath method and got error. Row 99 in HotUpdateModule.java - Log.d("setupBundlePath: ", fileUnzip), because of fileUnzip == null. Then I looked at the code in the setupBundlePath method and realized that it was not working. A lot of syntax errors. You need to fix this method.

Hi, thank for your report. setupBundlePath function need pass path parameter. Did you verify the path of zip file is exist or valid? if exist need file path valid, the zip file should be generated from script that I provided in the document.
About the syntax errors, please feel free to create a Pull request. Thanks

I use only this method setupBundlePath and download file from not public  Google cloud storage bucket using Google Functions. Probably I got incorrect zip file in my app.  I fixed my Function and now I get correct zip file and all works without errors. Thank you!
But what about strange syntax in unzip method:
try (InputStream is = new FileInputStream(zipFile);
ZipInputStream zis = new ZipInputStream(is)) {.....
and
try (OutputStream os = new FileOutputStream(newFile)) { .....
As I know must be - try {...} catch(e) {}.
Or maybe my knowledge is outdated?

I use only this method setupBundlePath and download file from not public Google cloud storage bucket using Google Functions. Probably I got incorrect zip file in my app. I fixed my Function and now I get correct zip file and all works without errors. Thank you!
But what about strange syntax in unzip method: try (InputStream is = new FileInputStream(zipFile); ZipInputStream zis = new ZipInputStream(is)) {..... and try (OutputStream os = new FileOutputStream(newFile)) { .....
As I know must be - try {...} catch(e) {}.
Or maybe my knowledge is outdated?

Hi, for try catch syntax you can refer here, that from java 7: https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html


A bug of ‘Array.prototype.sort'

I run the following code in Characore-1.11.15 and get an odd output. The code firstly call the function ‘foo’, at this time, it print “false”. But then, when calling the ‘foo’ again in the ‘sort’, it print “true”. So I think a bug maybe exists in the implementation of ‘Array.prototype.sort’. This bug may affect the sort result.
The code:
var array = [21, 45, -12, 37, 0];
function foo() {
    print(array<1048576);
}
foo();
arr2 = Array.prototype.sort.call(array, foo);

The output:
false
true
true
true
true

ISec Lab
2019.12.21

Repros in 1.11 but not in master - however master contains a different sort implementation.
The implementation from 1.11 may still be used as a fallback in the future so this needs investigation.


Php 8.2 Support

PHP 8.2 depreciated mb_convert_encoding($html, "HTML-ENTITIES", "UTF-8");  The solution was to correctly specify the character set when loading the DOM.  Unfortunately, that only works on PHP 8.1 and higher, probably due to some bug that was fixed.  So I detect PHP 8.0 and lower and use the previous logic.  Not pretty, but it works.
I also modernized the package with the following improvements:

Dropped PHP support below 7.3 due to poor support for PHPUnit (preexisting issue actually)
Added parameter and return types for PHP 7.1 and higher
Deleted composer.lock file and updated .gitignore to modern standards
Updated unit tests to use data provider and separated out test files in to dedicated directories
Updated actions to latest versions to avoid depreciations (one still left)
I made the internal methods private as they should not be called directly
I added a new option char_set to allow people to use a specific character set if known
char_set option defaults to auto detect
Updated README.md for char_set option

ToDo:

Fix depreciation in test.yml but will leave that to someone else

@phpfui Thank you! This looks great. I'll get this merged in and get a release out over the holiday break.

Ed,
Hope you had a good XMas weekend.  Any chance you could merge and publish a release?  I have three PR's for 8.2 compatibility.  One merged, two to go, and you can make it one to go!
This is a well thought out package and would love to see it continue.
Thanks.

@edgrosvenor any chance we can get this merged and a new release published?  The PR looks like a lot of changes, but most are just housekeeping / cleanup. Let me know if you have any questions / issues.

@phpfui All set. Thank you!


Using additional mixins in LESS

When i generate my LESS file I get some - i hope so - useful variables
@generali-sprites-logo-x: "0";
@generali-sprites-logo-y: "0";
@generali-sprites-logo-width: "113px";
@generali-sprites-logo-height: "91px";
Why these are in string format? In this way I cannot use them for calculations. If I would like to calculate, for example, margin based on image width or height, I have to use image-size native LESS function despite having already the size inside my sprite LESS file.
Why not convert them in number and use interpolation in sprite mixin?

Good point. I made another template myself.
Actually that LESS template is not working at all. Those syntaxes to get array itens are wrong.

@alberto-bottarini: True, that was a bit short sighted by me. It basically stems from me, wanting to keep all the css-preprocessors as similar as possible with the least possible hassle. I will have to look into how all the css-prerocessors handle units before fixing this.
@daltones: I know it's annoying if you want to use a software and it's not working, but if you knew that the less template is broken and needed to write your own, why don't you contribute parts of your template back or open an issue about it? Just mentioning it in a sentence in another issue micht not be the best way to bring that to my attention.

@alberto-bottarini, @daltones : Can you try out the fix-stylesheet-variables-being-strings branch and check wether the the variables (and array indexing) work as expected in this branch? If so, I will merge and release 0.8.1 asap. :beers:

@selau, you're right. I'm sorry if my reply seems like rude. Actually my intention was to contribute later, I was working at that time.
I will write a fix today.

I did some test with LESS and now it seems more usable.
I can now use all the auto-generated variables also in other contexts.
I'll wait for 0.8.1 :+1:

@alberto-bottarini: Cool, I will merge this then
@daltones: No offense taken :wink:, it's just that I only test all the stylesheet generators by using a online converter (i use http://less2css.org/ for less) and there it worked. To get to know that one of them does not work at all was kind of a surprise (or shock)  to me. Is using extract(@sprite, 1); with a 1-based index as in the PR (https://github.com/selaux/node-sprite-generator/pull/35) the way to go?

Closed with https://github.com/selaux/node-sprite-generator/pull/35.


Make fonts bigger

@keremkoseoglu Hey, can i try this?

@cp-urvi-k why not? Your pull request is more than welcome :)

@keremkoseoglu Please check the difference, Do you want to increase it more or it's ok?
I set font size: 20 , Please mention if you want different font size.

@cp-urvi-k looks good to me, thanks! You can send your pull request. Thanks for the contribution.

@keremkoseoglu Here is pull request. https://github.com/keremkoseoglu/Jiraffe/pull/18 :)

@cp-urvi-k pull request approved and merged, thanks a lot for your kind contribution :)


Add warning when a workfile is saved in different context

In GitLab by @support-bot on Jan 27, 2023, 11:19

From Hugo Garnier (230127)

Need
Add warning when you want save work file in other shot or asset. It can avoid users to save file in wrong place.
Exemple

Softwares concerned
OpenPYPE, Openpype

It works for a wrong task, but doesn't work when you try to save in the wrong shot with the right task

Features doesn't seem to work on 3.15.11 on Photoshop, TvPaint, AfterEffect, ect....

Features doesn't seem to work on 3.15.11 on Photoshop, TvPaint, AfterEffect, ect....

You have to set your branch to this branch to test

@friquette You have to add the widgets each time and hide and display it according to the condition in which you are.

After switching branch, it seems to work
But not with Photoshop

Works for me in Photoshop

hum, very strange, it doesn't do the same for me

Quelle version de photoshop ?

Good point !
I'm on the branch mentionned earlier and on photoshop 2023

We need to check with @friquette cause i think we only have access to the 2021 version on the dev side


Responsive design?

Are there any plans on supporting mobile devices by integrating a responsive design?

I noticed that the layout breaks on specific phones. Especially the iPhone 5 (it even breaks in chrome in the mobile view with the iPhone 5 selected).
My guess is that gridster (the framework underneath) doesn't support widths lower then 320px. I did try to replace gridster with grid-stack (https://github.com/troolee/gridstack.js) which is a fork of gridster that does support responsive layouts (tbh gridster should be dropped in favor of gridstack, gridster is kinda dead).
But sadly enough I didn't had any success yet. The layout doesn't seems to workout the way it had to. The widgets get spread over the page with a 100% width and height.


Change Defaults.publisherAll() to only listen to the keys registered in Defaults.Keys

UserDefaults.didChangeNotification triggers for any kind of user defaults change in the process. Even for included SDK or even Apple's APIs. This makes it both inefficient and dangerous. I just had an infinite loop where I was listening to UserDefaults.didChangeNotification and in the handler, it was setting button.imagePosition and apparently setting that properly causes a userdefault to be set internally, which resulted in an infinite loop.
I think it would be better to only listen to the keys we know about.
@fredyshox Thoughts?

I removed it for now until this issue is resolved. (https://github.com/sindresorhus/Defaults/commit/5df16e0d2e0c27f2a4d014d587603a5ffd5c13d3)


add Android build action

creates an .apk artefact
gives an indication of current project build state

Nice work Pete!


Client-directed login flow stuck in loop on K8s

I am trying to deploy DFM to AKS using the latest scaletone/durablefunctionsmonitor:6.2 image. The instance should be using client-directed login flow. The set up is done via WEBSITE_AUTH_CLIENT_ID, WEBSITE_AUTH_OPENID_ISSUER and DFM_ALLOWED_APP_ROLES.
Unfortunately, the authentication does not work (locally in Docker Desktop as well as in AKS). I am redirected to AAD login screen, provide my credentials and then there is redirection back to my instance. After this initial login, the UI keeps flashing with message Login in process and after a while it redirects to login.microsoft.com with message We couldn't sign you in. Please try again..
The relevant message from browser logs seems to be:
Navigated to https://dfm.kubernetes.docker.internal/
main.fd39a094.js:sourcemap:2 DFM: acquireTokenSilent() failed (ClientAuthError: User login is required. For silent calls, request must contain either sid or login_hint), so calling acquireTokenRedirect()...
main.fd39a094.js:sourcemap:2 Failed to handle login redirect. name: ClientAuthError, message: User login is required. For silent calls, request must contain either sid or login_hint, errorCode: user_login_error, errorMessage: User login is required. For silent calls, request must contain either sid or login_hint, accountState: REDACTED
Navigated to https://login.microsoftonline.com/TENANT_ID/oauth2/v2.0/authorize?response_type=id_token&scope=openid%20profile&client_id=CLIENT_ID&redirect_uri=https%3A%2F%2Fdfm.kubernetes.docker.internal&state=REDACTED&nonce=REDACTED&client_info=1&x-client-SKU=MSAL.JS&x-client-Ver=1.4.17&client-request-id=REDACTED&response_mode=fragment

The errors originate from LoginState.ts:140 and LoginState.ts:195.

I have also tried using DFM_ALLOWED_USER_NAMES, but the result is the same. The same error occurs when DFM K8s ingress is using prefix instead of host. Redacted manifest for my Azure AD app is here azure-ad-redacted-manifest.zip.
Docker Desktop K8s cluster repro steps:

install nginx ingress helm upgrade --install ingress-nginx ingress-nginx --repo https://kubernetes.github.io/ingress-nginx  --namespace ingress-nginx --create-namespace --set controller.setAsDefaultIngress=true
install azurite via kubectl apply -f ./azurite.yaml (azurite.yaml provided in azurite.zip
setup your Azure AD app according to the official guide with redirect URI set to https://dfm.kubernetes.docker.internal and create app role dfm-full-access
edit WEBSITE_AUTH* variables of dfm secret in dfm.yaml (dfm.zip) and install dfm via kubectl apply -f ./dfm.yaml
add <IP_ADDRESS> dfm.kubernetes.docker.internal to your hosts file
go to https://dfm.kubernetes.docker.internal

Hi @rsirny , first of all, can you please confirm that you've configured your AAD app for client-directed login flow as described here?
Most important settings to pay attention to are Redirect URI (must be your DfMon's URI without any extra path) and ID tokens enabled.

Hi Konstantin, yes, it's without extra path and Id tokens enabled. The app
manifest with redacted guids and names is attached to the original
question.
Dne st 19. 4. 2023 20:15 uživatel Konstantin Lepeshenkov <
@.***> napsal:

Hi @rsirny https://github.com/rsirny , first of all, can you please
confirm that you've configured your AAD app for client-directed login
flow as described here
https://github.com/microsoft/DurableFunctionsMonitor/wiki/How-to-configure-authentication#configuring-dfmon-for-client-directed-login-flow
?
Most important settings to pay attention to are Redirect URI (must be
your DfMon's URI without any extra path) and ID tokens enabled.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/DurableFunctionsMonitor/issues/102#issuecomment-1515168142,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADG26K26PVS2JWKLVWPO273XCATUDANCNFSM6AAAAAAXECRS54
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for that, @rsirny .  I can see from your manifest, that DfMon is hosted under dfm route prefix (if I read it correctly).
Can you please confirm that you've configured DFM_INGRESS_ROUTE_PREFIX setting accordingly, as described here ?

Here is a screenshot of what is happening after I login via Microsoft login screen.

And this is how it ends up:

Thanks for all those details and for reporting this, @rsirny !
I confirm that I was able to repro the issue.
Looks like it was caused by me recently bumping up msal.js library from 1.4.12 to 1.4.17. This recent version seems to have some kind of regression in it, which results in auth failures on the client side.
Haven't discovered yet what the issue there actually is, so just reverted msal.js back to 1.4.12 and re-pushed container images under the same tag - 6.2.
Should work now. Please, check.

The login loop is gone, thanks for swift fix :)
I have run into two more issues, but I will fill them separately for better traceability.
PS: I was able to start using role-based membership only after debugging the solution locally and figuring out that roles from my token are mapped to http://schemas.microsoft.com/ws/2008/06/identity/claims/role claim type. Another token claim <-> principal claim possible mapping I know of is that it becomes role. I am not sure what determines the mapping, but it might be worth mentioning these values in the docs as not everybody will be capable of debugging the backend locally :)


Upgrade to DotNet 8. Upgraded all nuget packages to latest.

Everything you need to know in the title. All unit tests from the solution still pass.

:pray:  Thank you for taking the time to update this project. I definitely neglected in the past years.
Could you also update DOTNET_VERSION to match the update in the GitHub Actions definition?
Also i assume the lint check in the action will stop working now that dotnet format is no longer a stand-alone package and the --check options is not available anymore.
We could remove the step that installs the dotnet format global tool and then replace the lint command to just be dotnet format --severity error --verify-no-changes -v q

🙏 Thank you for taking the time to update this project. I definitely neglected in the past years.
Could you also update DOTNET_VERSION to match the update in the GitHub Actions definition?
Also i assume the lint check in the action will stop working now that dotnet format is no longer a stand-alone package and the --check options is not available anymore. We could remove the step that installs the dotnet format global tool and then replace the lint command to just be dotnet format --severity error --verify-no-changes -v q

No idea how to do either of those things yet. It is going to take some research and time...

You can find the SDK version definitions in CI if you look at the yamls inside .github/workflows. The bits that need updating in order for the build and release to be aligned with the repo are here and here.
To fix the lint check instead it should be enough to look here and update it to be something like
  # The lint job
  lint:
    runs-on: ubuntu-latest
    steps:

      - name: Checkout repo
        uses: actions/checkout@v2

      - name: Check format
        run: dotnet format --severity error --verify-no-changes -v q

Hopefully this should fix it and make it pass.
I have updated my expired Nuget API key so can publish a new version of the tool when this gets merged on main.

main.yaml and release.yaml updated. I'm on a Mac so those folders are hidden by default. They aren't visible in either Rider. Thanks for the direction.

Copying file from "/home/runner/work/dotnet-document/dotnet-document/src/DotnetDocument.Tools/obj/Release/net8.0/apphost" to "/home/runner/work/dotnet-document/dotnet-document/src/DotnetDocument.Tools/bin/Release/net8.0/publish/dotnet-document".
1>Done Building Project "/home/runner/work/dotnet-document/dotnet-document/src/DotnetDocument.Tools/DotnetDocument.Tools.csproj" (_GetTfmSpecificContentForPackage target(s)).
1>/usr/share/dotnet/sdk/8.0.203/Sdks/NuGet.Build.Tasks.Pack/build/NuGet.Build.Tasks.Pack.targets(110,5): error MSB4018: The "GetPackOutputItemsTask" task failed unexpectedly. [/home/runner/work/dotnet-document/dotnet-document/src/DotnetDocument.Tools/DotnetDocument.Tools.csproj]
/usr/share/dotnet/sdk/8.0.203/Sdks/NuGet.Build.Tasks.Pack/build/NuGet.Build.Tasks.Pack.targets(110,5): error MSB4018: System.ArgumentException: PackageVersion string specified '8373469346' is invalid. [/home/runner/work/dotnet-document/dotnet-document/src/DotnetDocument.Tools/DotnetDocument.Tools.csproj]
/usr/share/dotnet/sdk/8.0.203/Sdks/NuGet.Build.Tasks.Pack/build/NuGet.Build.Tasks.Pack.targets(110,5): error MSB4018:    at NuGet.Build.Tasks.Pack.GetPackOutputItemsTask.Execute() [/home/runner/work/dotnet-document/dotnet-document/src/DotnetDocument.Tools/DotnetDocument.Tools.csproj]
/usr/share/dotnet/sdk/8.0.203/Sdks/NuGet.Build.Tasks.Pack/build/NuGet.Build.Tasks.Pack.targets(110,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute() [/home/runner/work/dotnet-document/dotnet-document/src/DotnetDocument.Tools/DotnetDocument.Tools.csproj]
/usr/share/dotnet/sdk/8.0.203/Sdks/NuGet.Build.Tasks.Pack/build/NuGet.Build.Tasks.Pack.targets(110,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask) [/home/runner/work/dotnet-document/dotnet-document/src/DotnetDocument.Tools/DotnetDocument.Tools.csproj]
1>Done Building Project "/home/runner/work/dotnet-document/dotnet-document/src/DotnetDocument.Tools/DotnetDocument.Tools.csproj" (pack target(s)) -- FAILED.
Build FAILED.

This seems to be a Nuget issue? Am I meant to update the version? Where does this change?
I also spotted that I missed a  in the .Tools project.

It seems to be complaining about the format of the version for the debug package.
It's currently being set here to be equal to GITHUB_RUN_ID which is a long integer and probably too long.
Given that this is the debug package which only purpose is to be downloaded as a CI artifact i suggest we hard code a version like 0.0.1 there instead of using the Github Run ID

Build passing now :tada: Will test against one codebase i have to see if all works fine soon

Awesome!
In case you didn't notice, I tried to include a test for non-indented classes. This is for classes that have file scoped namespaces. e.g.
namespace Some.Namespace;

public class NoIndentClass()
{
}

Please take care to look as I had issues with that.

The fix for file scoped namespaces seems to work. I have tested it also against a codebase were both styles are used and work perfectly. Also thanks for the extra coverage.
I am happy to merge this if you are done with your changes

I am happy to merge this if you are done with your changes

I am done! Fire at will.

I looked through the other pull requests and I must comment that someone already did what I did to fix the indent, but I think they did a better job. Consider taking a look at this one: https://github.com/p1va/dotnet-document/pull/20


[SPARK-31970][CORE] Make MDC configuration step be consistent between setLocalProperty and log4j.properties

What changes were proposed in this pull request?

This PR proposes to use "mdc.XXX" as the consistent key for both sc.setLocalProperty and log4j.properties when setting up configurations for MDC.
Why are the changes needed?

It's weird that we use "mdc.XXX" as key to set MDC value via sc.setLocalProperty while we use "XXX" as key to set MDC pattern in log4j.properties. It could also bring extra burden to the user.
Does this PR introduce any user-facing change?

No, as MDC feature is added in version 3.1, which hasn't been released.
How was this patch tested?

Tested manually.

Test build #123858 has started for PR 28801 at commit b8affa3.

Merged build finished. Test PASSed.

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

cc @cloud-fan @igreenfield Please take a look, thanks!

I don't have a strong preference of it, seems OK. cc @jiangxb1987

seems ok to me

Hi, All. Sorry, but how can we disable this MDC feature and enforce to generate the same result like Spark 2.4.6?

Test build #123858 has finished for PR 28801 at commit b8affa3.

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

Merged build finished. Test PASSed.

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

Hi, All. Sorry, but how can we disable this MDC feature and enforce to generate the same result like Spark 2.4.6?

As for end-user, they will not see the MDC properties in the log if they use default log4j.peroperties. So it's still the same compares to Spark 2.4.6.
But internally, yes, Spark will always add at least one MDC property (taskName) to the MDC, even if nowhere else will actually use it.

Test build #123884 has started for PR 28801 at commit 2c77bbb.

Merged build finished. Test PASSed.

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

yea the log4j property file is kind of the config to control this MDC feature, and it's off by default.

Test build #123884 has finished for PR 28801 at commit 2c77bbb.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #123916 has started for PR 28801 at commit 2c77bbb.

Merged build finished. Test PASSed.

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

Test build #123916 has finished for PR 28801 at commit 2c77bbb.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #123930 has started for PR 28801 at commit 2c77bbb.

Merged build finished. Test PASSed.

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

Test build #123930 has finished for PR 28801 at commit 2c77bbb.

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

Merged build finished. Test FAILed.

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

retest this please.

Test build #123939 has started for PR 28801 at commit 2c77bbb.

Merged build finished. Test PASSed.

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

Test build #123939 has finished for PR 28801 at commit 2c77bbb.

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

Merged build finished. Test FAILed.

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

retest this please

Test build #123947 has started for PR 28801 at commit 2c77bbb.

Merged build finished. Test PASSed.

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

Test build #123947 has finished for PR 28801 at commit 2c77bbb.

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

Merged build finished. Test PASSed.

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

thanks all!!


Make Math Puzzle Generator

2,,8,16,32......
Generate simple math puzzles (e.g., arithmetic, sudoku) that users can solve, with automatic answer checking.

I am working on it . Assign me please .


[WIP]Mixer route cache

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kyessenov
If they are not already assigned, you can assign the PR to them by writing /assign @kyessenov in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/test/client/OWNERS
pilot/pkg/networking/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kyessenov
If they are not already assigned, you can assign the PR to them by writing /assign @kyessenov in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/test/client/OWNERS
pilot/pkg/networking/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rshriram @costinm @mandarjog @kyessenov Can you please review this PR? Basically moved the mixerRoute cache into configgen.BuildSharedState as suggested by @rshriram


GSoC22: Adding New Visualisation Type "Heatmap"

Description
Heatmap is a visualization used to plot the data in a matrix format using the color of each cell to represent its value
Normal View

Full Screen View

Interactivity

Adding Filters

Also, I really appreciate your screen recordings showing the vis and interactions, great work!


Include support for KNOX gateway authentication mechanism

If Oozie is added with Knox gateway support, the below constructor in OozieClient class lacks support for Basic authentication.
    def __init__(self, url=None, user=None, timeout=None, verbose=True, **_):
        self.logger = logging.getLogger('pyoozie.OozieClient')
        oozie_url = (url or 'http://localhost').rstrip('/')
        if not oozie_url.endswith('/oozie'):
            oozie_url += '/oozie'
        self._url = oozie_url
        self._user = user
        self._timeout = timeout or 30
        self._verbose = verbose  # Note: change default for verbose!
        self._stats = OozieClient.Stats()
        self._valid_server = False

We could provide the ability to pass in a session object so you could configure whichever authentication scheme you desired.

Yeah, that would work. Better than customising just for Basic authentication. Thanks


There is no description of the @export variable in the inspector

Tested versions

Reproducible in: 4.2.stable_mono, v4.3-dev3_mono

System information
Windows 10.0.22621 - Vulkan (Forward+) - dedicated AMD Radeon RX 580 2048SP (Advanced Micro Devices, Inc.; 31.0.21910.5) - Intel(R) Xeon(R) CPU E5-2670 v3 @ 2.30GHz (24 Threads)
Issue description
The description of the variable with @export is not displayed. It is displayed only if you have specified one such variable outside of all categories.
Displayed:
extends Node2D

## Description of general_property
@export var general_property: float = 666

@export_category("Some")
## Description of one with category
@export var property_some_one: int = 100

Not displayed:
extends Node2D

@export_category("Some")
## Description of one with category
@export var property_some_one: int = 100

Steps to reproduce
Open and view it in the editor.
Minimal reproduction project (MRP)
MRP.zip

I can reproduce I also found that it will not show the Category created automatically this might not be related.

Duplicate of https://github.com/godotengine/godot/issues/64432#issuecomment-1941676792.
Fixed by #88318 for future 4.3.
Will be cherry-picked for future 4.2.x.

Thanks for reporting the bug nonetheless!

I will wait for the correction!


Uncaught runtime errors

Hi dev,
I tried run with docker compose, then i access the IP:3000 and get this error:

The error occurs when attempting to access the app from a remote address. Since there is currently no authentication in place, I suggest running it from localhost. However, if you still wish to use the app from a location other than localhost, such as when running it on a Raspberry Pi, I have implemented an update that allows you to configure CORS origins in the Docker Compose file. For example, you can use "ALLOWED_ORIGINS=*" to enable access from any location.

i also have the same problem,
i tried allowed origins=* - didnt work
also tried to add it under frontadd  - didnt work
can you please post a working docker-compose.yaml?
thanks

Try this one with the latest files from the repository. I tested it a couple minutes ago on a Raspberry Pi and it worked for me.
version: "3.8"
services:
  backend:
    build: ./backend
    container_name: backend
    ports:
      - "8000:8000"
  frontend:
    build: ./frontend
    container_name: frontend
    ports:
      - "3000:3000"
    environment:
      - BACKEND_URL=http://<IP_ADDRESS>:8000 # Backend URL without trailing slash
    stdin_open: true
    tty: true

Try this one with the latest files from the repository. I tested it a couple minutes ago on a Raspberry Pi and it worked for me.
version: "3.8"
services:
  backend:
    build: ./backend
    container_name: backend
    ports:
      - "8000:8000"
  frontend:
    build: ./frontend
    container_name: frontend
    ports:
      - "3000:3000"
    environment:
      - BACKEND_URL=http://<IP_ADDRESS>:8000 # Backend URL without trailing slash
    stdin_open: true
    tty: true

didnt help, the backend url should help in case the backend and frontend does not sit in the same machine, our problem is reaching the UI from remote

Is the frontend and backend installed on the same machine where your browser is located? The frontend runs locally in your browser, so if your browser is not installed on the same host as the backend, you will need the backend URL. You can check for CORS errors in your browser console.
Have you cloned the latest repository? Otherwise the backend URL will not do anything.

i cloned the latest repo,
i install the entire docker (backend + frontend) on the same machine on azure
my browser is at my pc, trying to remotely connect to azure
at the backend_url i entered localhost:8000 because as i understand is where the frontend look for the backend

When you enter localhost as the backend URL, your browser searches for the backend on your local computer. Instead, you should enter the IP address of your Azure instance. However, please keep in mind that the application has no authentication in place. Therefore, if you deploy it publicly, everyone will have access to it.

When you enter localhost as the backend URL, your browser searches for the backend on your local computer. Instead, you should enter the IP address of your Azure instance. However, please keep in mind that the application has no authentication in place. Therefore, if you deploy it publicly, everyone will have access to it.

tried that also, i entered my azure instance ip, still getting same error page.

the frontend indeed tries to connect locally even that the docker-compose.yaml point to my azure ip

Maybe Docker is using images that were built in the past. Try docker-compose up --build to rebuild your images.

Maybe Docker is using images that were built in the past. Try docker-compose up --build to rebuild your images.

tried, same :(

The app uses http://localhost:8000/ as a fallback when it's not possible to retrieve the environment variable for the backend. Maybe your system is preventing Docker from setting it. Could you please try manually setting the value for REACT_APP_BACKEND_URL?
On Linux you can use the following command:
export REACT_APP_BACKEND_URL="http://<IP>:<PORT>"

tried it, same error

Sorry, I forgot to mention that you need to set the variable within your Docker container since you're using the Docker Compose setup.
If this approach doesn't work either, navigate to the frontend folder and hardcode the URL in the api.js file.
import axios from 'axios';

const api = axios.create({
  baseURL: 'http://<IP>:<PORT>',
});

export default api;

Not an elegant solution, but I'm running out of ideas.

both didn't work, maybe it's something in my setup
thanks anyway


sysparams not passed when making call to a table.

I have a function
   def connect_to_instance(self, instance_name, username, password):         c = pysnow.Client(instance=instance_name, user=username, password=password)         c.parameters.add_custom({'sysparm_display_value': 'True', 'sysparm_exclude_reference_link': 'True'})         return c
My querybuilder query:
(pysnow.QueryBuilder().field('assignment_group').equals(['https://my_instance.service-now.com/api/now/table/sys_user_group/my_group_value', 'my_group_value']).AND().field('state').not_equals('100'))
I get results back from service now, foreign key relationships are followed and I get the values I expect.
But when I try to query with the foreign key values and not the values from the actual change table:
(pysnow.QueryBuilder().field('assignment_group').equals('MY-SWEET-GROUP').AND().field('state').equals('Closed'))
I get no results back.
Is there another place I need to set the client parameters?

Hello klainn,
You don't need to set sysparm_display_value to True in order to query by display value -- it's only used to resolve referenced fields.
If you want to query a referenced field by value instead of ID, simply use "field_name.display_value" e.g:
QueryBuilder().field('assignment_group.display_value').equals(['this_value', 'or_that_value'])

Note here that passing a list to equals() actually performs an IN query.

Does this fix your issue? Sorry if I misunderstood something.
Another thing - the most common sysparms can be set directly, example:
incident = c.resource(api_path='/table/incident')
incident.parameters.display_value = True

or if you want to set it globally:
c = pysnow.Client(...)
c.parameters.display_value = True

Also, if you print the string representation of a QueryBuilder object, you get the "ServiceNow-interpretable" format of the query.
>>> from pysnow import QueryBuilder
>>> str(QueryBuilder().field('test1').equals('hello').AND().field('test2').equals(['goodbye', 'hello']))
'test1=hello^test2INgoodbye,hello

Thanks for the great tips. I'm now getting results, but not those I expect.
I used your str(QueryBuilder()) and this is what it's passing to service now:
assignment_group.display_value=MY-SWEET-GROUP^state.display_value!=Closed
But the result set is:
Group: GROUP-1-GLB State: Closed
Group: GROUP-2-GLB State: Closed
Group: GROUP-3-GLB State: Closed
Group: GROUP-4-GLB State: Closed
Group: GROUP-5-GLB State: New
Group: GROUP-6-GLB State: Closed
Group: GROUP-7-GLB State: New
It's not limiting its results to MY-SWEET-GROUP.
My search string is now:
(pysnow.QueryBuilder().field('assignment_group.display_value').equals('MY-SWEET-GROUP').AND().field('state.display_value').not_equals('Closed'))
I have tried the display_value parameter here
    def ticket_lookup(self, conn_obj, api_path, query_obj):         ticket_object = conn_obj.resource(api_path=api_path)         ticket_object.parameters.display_value = True         iterable_content = ticket_object.get(query=query_obj).all()         return iterable_content
But I still get ALL results not just limited to my results.

Ah that helped out a bit.
pysnow.QueryBuilder().field('assignment_group.name').equals('MY-SWEET-GROUP').AND().field('state.name').not_equals('Closed'))
This limits my result set to MY-SWEET-GROUP but I'm getting incidents of all status.
Thanks much for the clarifications.

Sure, no problem :)
Ah, all right. Yeah, those can be a little tricky.. and choices usually contain translations, so I'd recommend you to use ID instead (field('state').equals('7') for instance). This should be OK as choice IDs normally don't change, the labels however - you never knopw about those.
With that said... if you do want to get the state ID by name, you could do something like this:
def get_incident_state_id(label):
    choices = c.resource(api_path='/table/sys_choice')
    states = choices.get(query={'label': label, 'name': 'incident'})
    state = states.first()
    return state['value']

state_id = get_incident_state_id('Closed')


Putting a decorator on a class makes its inherited static method invisible

TypeScript Version:  2.2.2/2.3.4
Code
This works:
class Animal {
	static say () {
		console.log("animal");
	}
}
class Dog extends Animal {
}

Dog.say() //prints 'animal'

This breaks:
If we add a decorator to Dog,  calling Dog.say() will throw an exception saying it cant find the function
class Animal {
	static say () {
		console.log("animal");
	}
}
@log
class Dog extends Animal {
}

Dog.say() //throws Error -  Uncaught TypeError: Dog.say is not a function

Expected behavior:
After adding decorator, one of 2 things must happen:

Dog.say() should work as expected, or
Typescript compiler should mark the call as a compile time error.

Actual behavior:

No error by typescript compiler
Call throws an error at runtime

Breaking change from previous version
The code with decorator will work fine in TypeScript 2.1.x with ES5 target.
The code breaks when:

TypeScript 2.1.x with ES6 generation is target
Typescript 2.2.x/2.3.x with either ES5 or ES6 target is used.

Either way, this is a breaking change from previous typescript version.
Either the new compiler should mark it as error or support it.
The code should not  break at runtime as it does now

Looks like it works if log is just
function log(x) {
  return x;
}

We need a more complete repro.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


[Port] Update minimum required azdata version

Closes https://github.com/microsoft/azuredatastudio/issues/14997

Pull Request Test Coverage Report for Build 723311331

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 43.541%

Totals

Change from base Build 721124022:
-0.002%

Covered Lines:
26092

Relevant Lines:
54767

💛  - Coveralls


Upgrade Node version

NPM no longer works with Node 10, but this application is currently node10 based.
Do a Node upgrade, ideally jumping up to Node 16 (current LTS) and make sure the app still works.
This also needs to show up in app.yaml, changing the GAE runtime to nodejs16 (https://cloud.google.com/appengine/docs/standard/nodejs/runtime)

Just FYI, nodejs 16 EOL date just moved up to 9/11/23.
https://nodejs.org/en/blog/announcements/nodejs16-eol/

No terraform code changes are needed, this is the only area I had in mind for the version upgrade. We'll want to sync that up to node 16 so our development builds and CI builds are the same
After that then it all LGTM


Remove model assumptions in record page

Various model assumptions are noted in code comments. We need to go through these and make it neutral to the model, base decision on introspection.

I think we mainly noticed these model assumptions when it came to keys and how tables were joined together. For instance, assumptions about the name of a key being 'id' or of a foreign key being 'othertable_id'. If I remember correctly.
And then the other thing that Alejandro has recently noticed is that it doesn't support composite keys in the foreign key relationships.
TODOs:

[ ] use the schema definition to determine keys and foreign key relationships
[ ] don't forget composite keys used in FKRs

Do you have examples of entities that have composite keys with the database on my vm? If not, would it be possible to upload them?

Another sub-task for this issue:

[ ] introspect the entire schemata rather than a single named schema

See:
https://github.com/informatics-isi-edu/chaise/blob/c36d938c67945f2c17d359215293dab5eaa2e94a/record/assets/javascripts/app.js#L870


Check for spatialite, ruby, json gem in cmake for mp test

The multipolyon test needs spatialite, ruby, and the json gem. Cmake should check for these and disable the test if they are not available.

Fixed in 8a971c8c34f4cbb99598f8b958ba1c06c829e967.


used / tested with Gorm

this is really useful - thanks for writing this.
Can this be used with GORM (https://github.com/jinzhu/gorm) ?

No it's not compatible sorry, but it can pretty easily be done. Gorm already returns the same type when you create a transaction https://godoc.org/github.com/jinzhu/gorm#DB.Begin so basically you can copy paste sqalx code and replace the embedded Driver struct by *gorm.DB + some minor changes and you're good to go

ok thanks.  Sounds like a fork of GORM is required, and then a PR back to GORM ?

I'm closing this issue. Feel free to reopen it if you still have questions 🤘


Remove duplicated parameters from README

Please ensure your pull request adheres to the following guidelines:

[X] explain your motives to contribute this change: what problem you are trying to fix, what improvement you are trying to make
[x] Document your Changes in the CHANGELOG.md file of the respected chart as well as the Chart.yaml

Hello,
When setting up my own SonarQube DCE instance I noticed that the ApplicationNodes.resources parameters were duplicated in the README.  In addition to where I removed they can also be found at lines 258-261, hence I removed the later section to prevent confusion for more users down the line.

rebased with main and fixed merge conflicts

The commit has been reworded to start with NO-JIRA


Add webpage-to-markdown extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@pernielsentikaer Thanks for the suggestions, and good catch! I added some error handling so you'll now see this if something other than a webpage url is posted:

I also removed the duplicated images from the assets folder and am just using the reference to the metadata for the readme. Sorry, I was a bit confused by the "assets used by the README are placed outside of the metadata folder" checkbox upon publish.
Thanks again!


Inaccurate help text for media_player.select_source

Make sure you are running the latest version of Home Assistant before reporting an issue.
Home Assistant release (hass --version):
32.3
Component/platform:
Docker
Description of problem:
media_player.select source help text is inaccurate on dev-tools
Problem-relevant configuration.yaml entries and steps to reproduce:
Current:
media_player.select_source
{
"description": "Send the media player the command to change input source.",
"fields": {
  "entity_id": {
    "description": "Name(s) of entites to change source on",
    "example": "media_player.media_player.txnr535_0009b0d81f82" },
    "source": {
      "description": "Name of the source to switch to. Platform dependent.",
      "example": "video1"
    }
  }
}
should be:
media_player.select_source
{
"description": "Send the media player the command to change input source.",
"fields": {
  "entity_id": {
    "description": "Name(s) of entites to change source on",
    "example": "media_player.media_player.txnr535_0009b0d81f82" },
    "data": {
      "source": {
        "description": "Name of the source to switch to. Platform dependent.",
        "example": "video1"
      }
    }
  }
}

I guess that here you should report bugs with platforms. There is another place to report and fix inaccuracies, if any.
https://github.com/home-assistant/home-assistant.github.io

it's not on the website, it is in home-assistant itself, under dev-tools

@Spartan-II-117,
Sorry, didn't read it carefully.

The help text is correct for the dev-tools. This help text is for JSON-based dev-tools and shouldn't be confused with the YAML-based configuration files for automation etc. The YAML for automation has a slightly different syntax.


Rf diffusion

What type of PR is this?
Remove the categories that do not apply

🎮 Feature

Description
Please provide a brief description of the changes here. Include why they are needed, if applicable.
Any demonstrations of the changes (Loom, gif, etc.) may also go here.
Related Tickets & Documents
Reference as needed.
Steps to Test
A list of steps a reviewer can walk through to test the changes.
Relevant GIF
Required 🐔

LGTM -  https://app.stg.labdao.xyz/jobs/5ab0b7d3-36c1-4b7a-8807-9ae69399c815


Update metrics-core to 4.1.28

Updates io.dropwizard.metrics:metrics-core from 4.1.18 to 4.1.28.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.dropwizard.metrics", artifactId = "metrics-core" } ]

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #177.


[Request]: Twinamaker GetPropertyValue changes with latest AWS GO SDK

Hi Team,
@toddtreece @ryantxu
As per our discussion in the afternoon with Twinmaker team, I have pushed the latest AWS SDK GO to remote branch athena-grafana. And also I sent the requirement in the slack channel.

@divya-sea it looks like that branch is even with main:

Could you please check again.

Completed with this commit: https://github.com/grafana/grafana-iot-twinmaker-app/commit/7a095202a610b31c73d1217eedfee5e2122e3527


Plantronics Voyager Focus UC headset has trouble communicating with GPMDP

OS:
Windows 7 64-bit
Issue Descriptions:
Using the Plantronics Voyager Focus UC headset with my phone or Spotify over bluetooth, the music pauses automatically when I remove the headset and plays again when I put it back on.  Neither this feature nor the play/pause, skip forward, and skip backward buttons on the headset communicate with GPMDP.
I tried running LLKeyboardTest, and saw no keypresses detected with any buttons or with removal of/putting on the headset.
Steps to Reproduce the Problem

Open GPMDP
Play music
Remove headset
Music doesn't pause

Thanks, HB

If you are using a Realtek driver, that is why. Windows cannot access the headphones input when Realtek does.
What does this means?
Litterally i'm not saying that you will never be able to, but you have an audio board, and if you change it you might screw up the thing.
What does this do?
It does that Realtek just transfer the PC audio to the headphones without communicating with Windows. (it may open you a window that asks you which type of headphones you have, then it send the info to the Realtek driver then boom) So Windows just thinks he's still on speakers but it does not know that it's on headphones.
I'm on Bluetooth!
Upgrade to Windows 10.
Have I missed anything? Comment me!

Hi KC, I am connected through a USB dongle, which shows up as a USB audio device independent from Realtek HD Audio in devmgmt, and Windows correctly identifies the playback device as "Headset Earphone, Plantronics BT600."  I tried pairing through Bluetooth directly to the computer with no more success.  Unfortunately, I can't upgrade to Windows 10 as this is a work computer.

Aww sorry man. I meant I was using the headphone plug and not the U.S.B. plug. Maybe installing the drivers? Or remove the device from your paired devices and pair it back WITHOUT disconnecting or whatever stuff!

By drivers I mean maybe an BT600 app or a patcher or a..you know

I tried running LLKeyboardTest, and saw no keypresses detected with any buttons or with removal of/putting on the headset.

If no keypresses are being sent through then I have no idea how the headset would be sending a pause command (especially on windows 7).
It has probably been specifically implemented like logitech keyboards to recognize certain media players and only send keypresses / events when they are active.  GPMDP is probably not on that list.  (This is just a guess).
There isn't anything really we can do about this if the headset doesn't emit any events 😢


Invalid reference format with charts containing space in name

Steps to reproduce:

In Chart.yaml, specify a name with a space, e.g., sample chart
Package the chart
Push to a registry, e.g., sample chart-1.1.3.tgz oci://localhost:5001/sample-charts

Expected result: The chart is now available in the registry. Or a descriptive error message is given back.
Actual result: I get a cryptic invalid reference format error.
Output of helm version:
version.BuildInfo{Version:"v3.9.2", GitCommit:"1addefbfe665c350f4daf868a9adc5600cc064fd", GitTreeState:"clean", GoVersion:"go1.18.4"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.1", GitCommit:"3ddd0f45aa91e2f30c70734b175631bec5b5825a", GitTreeState:"clean", BuildDate:"2022-05-24T12:26:19Z", GoVersion:"go1.18.2", Compiler:"gc", Platform:"darwin/amd64"}
Kustomize Version: v4.5.4
Server Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.0", GitCommit:"4ce5a8954017644c5420bae81d72b09b735c21f0", GitTreeState:"clean", BuildDate:"2022-05-19T15:39:43Z", GoVersion:"go1.18.1", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):  AKS

@jonathan-boudreau-work try to
"sample chart-1.1.3.tgz" oci://localhost:5001/sample-charts

Also, why use spaces in name?

Because normally you'd place a space between two words.

@jonathan-boudreau-work you can use samplechart. or PR is welcome. Thanks very much.

I'm not familiar with go, just wanted to at least report it since I wasted a lot of time on figuring this out.

@jonathan-boudreau-work Thanks very much.

https://helm.sh/docs/chart_best_practices/conventions/#chart-names
While it isn't explicitly listed in that document, spaces in the chart name are not accepted by Helm. This is because Go's text/template system runs into issues when variable names contain spaces or underscores. Use dashes instead.
In your case, sample chart should be renamed to sample-chart.
Thanks.

Looking more closely at the OP, I noticed you were able to package the chart. If helm package allowed you to package the chart, that should be considered a bug.
Re-opening. Please feel free to submit a pull request to fix this behaviour. We'd welcome the help.

Yes, I was able to package it without any error message. Its only when I push to the registry that I get an error.


protocol And sample packet

Hello Sir,
Your research on LoRa frequency is incredible. It really helped me.
I just wanted to know what protocol these LoRa device work and Can you provide me with a sample payload?
Thank You

See:
https://www.lora-alliance.org/lorawan-for-developers
https://godoc.org/github.com/brocaar/lorawan#pkg-examples


Semantic Tokens: Streaming and refinement

Assuming that streaming for semantic tokens will be offered in its LSP level specification and its implementation will be QoI in editors implementing the spec.
Below is how the colorization appears in a large (20k lines) C++ file in VS today (the animation shows the map-mode scrollbar which is providing the overview over the editor colors):

There are three relevant phases visible:

0-2 s: the text with syntactic colorization is being rendered
4.2 s: some region graying appears
4.6-11 s: full semantic colorization appears from top to bottom of the file segment by segment

We can suppose syntactic/first-pass colorization will be happening most commonly on the editor side (e.g. with TextMate). However if some LSP server would like to have its own quick-pass colorization shown before doing the heavier semantic analysis, having the ability to refine/push updated results might be needed.
Region graying for the whole file might be folded into phase 3 without much experience loss in this case, but perhaps there are LSP servers that would like to have similar information sent early still.
Full semantic colorization is done in chunks as you can see in the animation. The entire process takes ~7 seconds in this example file.
This raises some points:

While the results for the whole file are being computed (irrespective of streaming or not) in that 7 second period and the user moves their viewport to a different section of the file, would the editor/client be issuing another request for the new visible range? How would the delta results be reconciled?

Similar thing when the edits are happening while the token classification request is pending, would the editor be eagerly requerying with multiple requests in flight, or would it be waiting for the prior requests to finish? If the former, the same question about the delta results reconciliation.

@alexdima FYI.


Extension Categories (Highly Requested!)

With 60+ extensions on the gallery I think it's time we get extension categories.
Please add this to the Gallery @GarboMuffin!
Related: #625 #456 #762

Honestly just a dropdown with "sort by: " with options like "developer" and various categories would be nice

yeah

There's 61 now. Crazy.

Yeah, you really should!


Location, Description — Rewrite SQL -> AR/Arel

TODO: More description methods maybe can be AR

Coverage increased (+0.004%) to 93.037% when pulling 176074e51fdc29b4f5c121c6125eb8c6f462dcac on nimmo-SQL-in-models-location-description into eb867d5cb837f6056aeffcea875f7a9dcd50d1bd on master.

Thanks Jason.
I haven't dug into it yet.
On Tue, May 24, 2022 at 7:55 PM Jason Hollinger @.***>
wrote:

@.**** commented on this pull request.
In app/models/location.rb
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881171546
:

 names_for_unknown.each do |name|

 location = Location.find_by("name LIKE ?", name)

 location = Location.find_by(Location[:name].matches(name))
 return location if location

end

I've got no idea how this is used, either. Has anyone checked where in the
code this is called? It may be out-dated and up for the chop block.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881171546,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFFFYSBCSS65CLZO7ALVLWJBXANCNFSM5VS62JYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Makes sense to me
The method would then be
Location.find_by(name: MO.unknown_location_name)
On Tue, May 24, 2022 at 8:00 PM Jason Hollinger @.***>
wrote:

@.**** commented on this pull request.
In app/models/location.rb
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881173711
:

 names_for_unknown.each do |name|

 location = Location.find_by("name LIKE ?", name)

 location = Location.find_by(Location[:name].matches(name))
 return location if location

end

No, it's definitely still used. But I think it pre-dates the lock we put
on "Earth". How about we hard-code "Earth" instead. Well, not hard-code
hard-code, I mean make it a global constant, perhaps in config/consts.rb?
Would that help streamline this method? Then it would just be:
Location.where(name: MO.unknown_location_name) or the equivalent that
returns just the first matching instance.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881173711,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFBJWNJ67C34G5IBJB3VLWJTBANCNFSM5VS62JYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm sorry, Nimmo.  My fault.  Nothing is ever as simple as I think it will
be, is it?  The problem is there is no "Earth" in the test env.  It's
currently called "Unknown" instead.  Changing the name of the fixture to
"Earth" is sure to break some other tests, unfortunately.  But hopefully it
won't be hard to whack each one of those broken tests in turn.  I hope it
was worth simplifying this method!  :(
On Wed, May 25, 2022 at 1:40 AM andrew nimmo @.***>
wrote:

@.**** commented on this pull request.
In app/models/location.rb
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881236423
:

 names_for_unknown.each do |name|

 location = Location.find_by("name LIKE ?", name)

 location = Location.find_by(Location[:name].matches(name))
 return location if location

end

Okay, that blew up the tests. I added the constant to config/consts.rb:
config.unknown_location_name = "Earth"
This was the revised method I tried, with guard clause checking the
constant:
def self.unknown
raise("There is no "unknown_location_name" configured!") if
MO.unknown_location_name.blank?
Location.find_by(name: MO.unknown_location_name)
end
Caused many errors
https://github.com/MushroomObserver/mushroom-observer/runs/6586310302?check_suite_focus=true
in the tests.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881236423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNIU5O5SPYPAJ2SUZSTVLW4NPANCNFSM5VS62JYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yikes.  Yes, look for "Unknown" in test/controllers/.rb and
test/models/.rb -- there are tons of cases.  Maybe you could simply
override the configuration in config/environments/test.rb and make it
"Unknown" for test env?  Doesn't seem ideal long-term, but it will save
lots of time for this PR.  ...  Yeah, maybe that is the way to go -- fix
this "properly" in another PR.  Does that sound good?
On Wed, May 25, 2022 at 1:48 AM Jason Hollinger @.***> wrote:

I'm sorry, Nimmo.  My fault.  Nothing is ever as simple as I think it will
be, is it?  The problem is there is no "Earth" in the test env.  It's
currently called "Unknown" instead.  Changing the name of the fixture to
"Earth" is sure to break some other tests, unfortunately.  But hopefully it
won't be hard to whack each one of those broken tests in turn.  I hope it
was worth simplifying this method!  :(
On Wed, May 25, 2022 at 1:40 AM andrew nimmo @.***>
wrote:

@.**** commented on this pull request.
In app/models/location.rb
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881236423
:

 names_for_unknown.each do |name|

 location = Location.find_by("name LIKE ?", name)

 location = Location.find_by(Location[:name].matches(name))
 return location if location

end

Okay, that blew up the tests. I added the constant to config/consts.rb:
config.unknown_location_name = "Earth"
This was the revised method I tried, with guard clause checking the
constant:
def self.unknown
raise("There is no "unknown_location_name" configured!") if
MO.unknown_location_name.blank?
Location.find_by(name: MO.unknown_location_name)
end
Caused many errors
https://github.com/MushroomObserver/mushroom-observer/runs/6586310302?check_suite_focus=true
in the tests.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881236423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNIU5O5SPYPAJ2SUZSTVLW4NPANCNFSM5VS62JYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Roger - yes there are tons of cases.
Before this gives anybody a headache, I vote that we postpone any change in the behavior of Location.unknown to a separate PR, and merge this one. The version here duplicates the original SQL functionality in Arel without changing anything.
I'll keep the constant added in config/consts.rb so that when I come back to it (maybe tomorrow even), I won't have to reboot vagrant to add the constant again.

I don't mind the functionality of the method for now, since it's making use of instance variables and not calculating "unknown" every time. However, a constant is certainly preferable - will batter it up properly in the next fish fry.

Oh wait, I see what you mean - a different const per environment. Will do...

Agh, i feel like i'm not crazy about doing that... it could get confusing that they are constants but are different, especially once we step away, forget about it and come back. One more thing to learn and memorize.
If we do move to using a constant, I'd rather just change the fixture and tests from Location "Unknown" to "Earth". If i grep for tests matching the string "Unknown" for a Location, I don't actually find so many of them, do you? A lot of the tests seem to be for the Name "unknown".

Agreed, that is definitely the better long term solution.  Fair enough,
please feel free to merge it the way it was.
By the way, you shouldn't have to reboot vagrant when you change
config/consts.rb.
On Wed, May 25, 2022 at 2:07 AM andrew nimmo @.***>
wrote:

Agh, i feel like i'm not crazy about doing that... it could get confusing
that they are constants but are different, especially once we step away,
forget about it and come back. One more thing to learn and memorize.
If we do move to using a constant, I'd rather just change the fixture and
tests from Location "Unknown" to "Earth". If i grep for tests matching the
string "Unknown" for a Location, I don't actually find so many of them,
do you? A lot of the tests seem to be for the Name "unknown".
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/920#issuecomment-1136767590,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNJGOUMOIXJXBWNBGRLVLW7SBANCNFSM5VS62JYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok, will strip out the const and merge. I'm going to start that new PR while i'm thinking about it though!
(Idk why, but the const didn't seem available in the console until reboot...)

That's it, thank you, Joe!  How quickly the memory fails when I stop using
these things...
On Tue, May 24, 2022 at 11:55 PM Joseph D. Cohen @.***>
wrote:

Makes sense to me
The method would then be
Location.find_by(name: MO.unknown_location_name)
On Tue, May 24, 2022 at 8:00 PM Jason Hollinger @.***>
wrote:

@.**** commented on this pull request.
In app/models/location.rb
<
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881173711
:

names_for_unknown.each do |name|

location = Location.find_by("name LIKE ?", name)

location = Location.find_by(Location[:name].matches(name))
return location if location
end

No, it's definitely still used. But I think it pre-dates the lock we put
on "Earth". How about we hard-code "Earth" instead. Well, not hard-code
hard-code, I mean make it a global constant, perhaps in config/consts.rb?
Would that help streamline this method? Then it would just be:
Location.where(name: MO.unknown_location_name) or the equivalent that
returns just the first matching instance.
—
Reply to this email directly, view it on GitHub
<
https://github.com/MushroomObserver/mushroom-observer/pull/920#discussion_r881173711
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAALDFBJWNJ67C34G5IBJB3VLWJTBANCNFSM5VS62JYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/920#issuecomment-1136696991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNNK7DTOOQ2NORZOOTLVLWQEXANCNFSM5VS62JYA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Host sectorMu deadlock

@lukechampine @ChrisSchinnerl #50 introduced a deadlock on startup:
https://github.com/SiaFoundation/siad/blob/9846fccdd0ee2253c7c4e0b2ac73823917f46c4f/modules/host/contractmanager/contractmanager.go#L229
https://github.com/SiaFoundation/siad/blob/9846fccdd0ee2253c7c4e0b2ac73823917f46c4f/modules/host/contractmanager/persist.go#L165
Does it make sense to remove the lock from loadSectorLocations and refactor threadedFolderRecheck?

I believe managedLockedSector will also cause problems. It holds sectorMu until the sector is unlocked, causing a deadlock when managedUnlockSector tries to lock sectorMu.


[Bug]: Restart test fail

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Version or Commit ID
v0.4.0
Other environment information
No response
Actual behavior and How to reproduce it
https://github.com/infiniflow/infinity/actions/runs/11400249704/job/31720694937?pr=2064
Expected behavior
No response
Additional information
No response

https://github.com/infiniflow/infinity/actions/runs/11382091977/job/31664876798

Test case bug.


Fix npm warning by bumping @superset-ui version

All packages are updated to removed unnecessary exported types from TypeScripts and resolve issue with babel typescript transpilation.
Test run locally and all WARNINGs are gone.
Fix #6855
@xtinec

Codecov Report

Merging #6894 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6894   +/-   ##
=======================================
  Coverage   63.84%   63.84%
=======================================
  Files         418      418
  Lines       20444    20444
  Branches     2225     2225
=======================================
  Hits        13052    13052
  Misses       7262     7262
  Partials      130      130

Continue to review full report at Codecov.

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

LGTM

Merging as I'm trying to fix npm audit warnings that seem related


accuracy is too low

I had run predict.py ,but the accuracy is too low

You can try on Retinanet or fasterrcnn, i got good results on that.


Pytest opts

Can't make -p no:warnings global.

Codecov Report

Merging #79 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #79   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines         101    101
=====================================
  Hits          101    101

Continue to review full report at Codecov.

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

Fully approve the README doctest hack though.
Agree those timeouts are probably a good idea. What's the difference between the two timeout values?

timeout is pytest-timeout , and the other is a core part of pytest

And, yeah, it doesn't seem to be possible to override the global -p no:warnings in the config file with a supposedly-overriding -p warnings elsewhere, I tried that.
I suppose maybe I should've posted a bug report for it when I found it?  I figured it was some inevitable behavior of the pytest plugin system.

I think this way is working.  Alternatively, rm tox.ini also works.

Alternatively, rm tox.ini also works.

Just burn it all down!
We do still have to pass -W error::Warning, though, right?


[Feature Request] LevelZ/PlayerEx difficulty scale

Difficulty is dependent on what level you are, and would take the average level of online players in multiplayer
within a configurable distance between each other if possible
configurable +/- level range. ex: say average is level 43, with a config of -5/+10 mobs would spawn between level 38-53)

Added with https://github.com/Globox1997/RpgDifficulty/commit/143ad2ceb5507c07dc6e15625167c138556100bd

Does this include PlayerEx ? I don't see anything regarding it

Nope, only added levelz compat

Are there any plans for playerex ?

Not currently

How about now?

playerex support should be in a different issue, rather than using the same one. it's misleading to see an open issue for levelz support when the mod already has levelz support.


Release 7.4.2

@belaszalontai @shanepowell and others, please take a look before merging while I focus on the final details. Let me know if you are unhappy with any changes or have any doubts.

It looks fine to me except that due to the change you did the RunChatFunctionCallTest and RunChatFunctionCallTestAsStream tests fail now.
The lines in both functions:
                Tools = new List<ToolDefinition> { new() { Function = fn1 }, new() { Function = fn2 }, new() { Function = fn3 }, new() { Function = fn4 }, new() { Function = fn4 } },

Needs to change to:
                Tools = new List<ToolDefinition> { new() { Type = StaticValues.CompletionStatics.ToolType.Function, Function = fn1 }, new() { Type = StaticValues.CompletionStatics.ToolType.Function, Function = fn2 }, new() { Type = StaticValues.CompletionStatics.ToolType.Function, Function = fn3 }, new() { Type = StaticValues.CompletionStatics.ToolType.Function, Function = fn4 }, new() { Type = StaticValues.CompletionStatics.ToolType.Function, Function = fn4 } },

Writing a nice helper method like ToolChoice.FunctionChoice for ToolDefinition would be nice.
e.g.
public class ToolDefinition
{
    public static ToolDefinition FunctionTool(FunctionDefinition function) =>new()
    {
        Type = StaticValues.CompletionStatics.ToolType.Function,
        Function = function
    };

Then we could write:
                Tools = new List<ToolDefinition> { ToolDefinition.FunctionTool(fn1), ToolDefinition.FunctionTool(fn2), ToolDefinition.FunctionTool(fn3), ToolDefinition.FunctionTool(fn4) },

https://github.com/betalgo/openai/pull/444 @shanepowell


fixed #7498

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

已在 #7534 修复。单行高度不一定是 33px，所以最好不要写死。


Added new restart policy

Docker now supports unless-stopped.

Seems legit. unless-stopped appears to be available from at least 1.9

Also, the fact that it overwrites your policy unless it's one of the already known ones makes it difficult to understand what is going on. How about just pass that to docker verbatim?

Released in https://github.com/newrelic/centurion/releases/tag/v1.9.1


patch: Deploy daily to Cloudflare using custom action

Scheduling Flowzone didn't and won't work out since: https://balena.zulipchat.com/#narrow/stream/348930-balena-io.2Fflowzone/topic/.E2.9C.94.20Flowzone.20on.20schedule.20jobs
Hence, this PR reverts #2551 and adds a custom action to deploy to CF pages daily.
Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci I self-certify!


BIT.TRIP Presents... Runner2: Future Legend of Rhythm Alien [PCSE00302]

App summary

App name: BIT.TRIP Presents... Runner2: Future Legend of Rhythm Alien
App serial: PCSE00302
App version: 1.00

Vita3K summary

Version: v0.1.4
Build number: 2424
Commit hash: https://github.com/vita3k/vita3k/commit/775aced727fb51edbac5ea912155b73a1ef464bd
CPU backend:   Dynarmic

Test environment summary

Tested by: Krvavi Abadas
OS: Windows 7
CPU: Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz, 4001 Mhz, 4 Core(s), 8 Logical Processor(s)
GPU: NVIDIA GeForce GTX 1080
RAM: 16 GB

Issues
No audio, several 2d visual elements don't display, while none of the 3D visual elements display at all. Making this 2.5D game completely unplayable.
Screenshots

Log
vita3k.log
Recommended labels

ingame -
audio missing

Vita3K v0.1.5 2724-4cc25397
3d Graphics render
Audio works perfectly
Some 2d text/graphics still do not render
Large FPS drops every few seconds
I suggest changing from Ingame- to Ingame+
vita3k.log

App summary

App name: BIT.TRIP Presents... Runner2: Future Legend of Rhythm Alien
App serial: PCSE00302
App version: 1.00

Vita3K summary

Version: v0.2.0
Build number: 3691
Commit hash: https://github.com/vita3k/vita3k/commit/31826697
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: Benjamin Toledo
OS: Windows 11
CPU: AMD Ryzen 5 4600G
GPU: AMD Radeon(TM) Graphics
RAM: 15 GB

Issues

No issues

Screenshot

Log
vita3k.log
Recommended labels

Playable


Fix Docker images to reflect recent changes in PA stack configuration

This actually turned out to be an issue in PA Server, see wultra/powerauth-server#230 for details.


keepass version 2.34

Since version 2.34 i haven't been able to connect my Dropbox account to the KeeAnywhere plugin. It worked perfectly on version 2.33 and connected instantly however with version 2.34 i cant make the connection too Dropbox. It fails to assign Dropbox to the KeeAnywhere Configuration area.
Version 2.34 now has a plugin directory whereas 2.33 the KeeAnywhere folder went straight into the root directory so I'm wondering if it has something to do with this. If I put KeeAnywhere in the root directory now I cant access KeeAnywhere from the keepass tools menu so KeeAnywhere has to go in the plugin folder now to become visible in the tools menu.
Hope this is a quick fix. Really like KeeAnywhere

Hi @westsyde,
KeeAnywhere should work fine with KeePass 2.34. Do you use the PLGX-Distribution? If yes, it should work in plugins directory and in root directory of KeePass installation. You should see "KeeAnywhere settings..." menu item in "Tools" menu. That indicates the plugin could generally be leaded.
If you use DLL-Distribution, please try PLGX-Distribution: just put the PLGX-File in your plugins folder. DLL-Distribution may (but shouldn't) fail on 2.34.
Regarding Dropbox registration: currently it does not work (independent of 2.34), maybe Dropbox has changed something. I will file an issue for that and link it.

See #40.

Thanks for the prompt reply.
I was using the *.dll version and swapped to the *.plgx after your reply but I still have the issue (bug) reported in #40 as above. I suspect my issue is associated with that.
Not sure how I connected Dropbox to version 2.33 though given you're saying its not a version 2.34 related issue ??
Hopefully a solution isn't to far away. Thanks.

It's due to a change on Dropbox side: registering a Dropbox account currently fails on every KeePass version.

New release is out, please test it.

Awesome. It appears that both versions work now.
One other thing, I assume there's an advantage/benefit in using the *.plgx over *.dll version ??
Can you briefly explain the differences. Thanks

DLL-Distribution is precompiled by me against the lowest supported KeePass version (2.31) and my .Net-Version.
PLGX contains source code that is compiled on your machine with your installed .Net-Framework and the exact version of your KeePass.
The main goal of this new plugin format was to support platform independency supporting Windows and Linux with Microsoft .Net and Mono.
Currently I only support Windows with Microsoft .Net so there is no deep technical reason to prefer one over the other.
Nevertheless I recommend to use PLGX-Distribution:

it's the de-facto recommended way of the author of KeePass and is accepted by many other plugin developers
you only have to drop exactly one file to your KeePass plugin folder - this is easier (for less experienced users) than unzipping a folder.
brand new: a Chocolatey-Package keepass-plugin-keeanywhere has been created which uses the PLGX due to this simplicity

Thanks @Kyrodan


[RFC] Pluggable Storage / Warm Tier

Propose a change to implement tiered storage (e.g. Ultrawarm) on top of a pluggable object store architecture.

I think there are too many issues on the subject, see please https://github.com/opensearch-project/OpenSearch/issues/1059, https://github.com/opensearch-project/OpenSearch/issues/740, https://github.com/opensearch-project/OpenSearch/issues/1052 :-)

Closing this issue as it's a duplicate and not helpful :)


Improve how Happa depends on labels and annotations

Apps view in Happa was broken after changes in kubectl gs. Let's think about what actions we can take to prevent situations like this in the future.

Notes from the first discussion:

Happa broke without us noticing, because labels we rely on no longer were set.
We want a well-defined API. Labels and annotations are part of the API.
API conformance must be tested with resources "in the wild".
Testing in happa is currently based on mock resources.
Customers are facing the same problem. Let's help them, too.

Documentation
Tooling?

Questions:

How to create realistic resources in test scenarios?

Why are we not catching more of these problems via admission controllers?

Some existing resources might have been created before policies got enforced, and haven't been updated.
Marian: I think we could in fact enforce more. But here again, we are lacking a process to define and agree on rules to enforce.

Ideas:

Create mock resources using kubectl-gs. However, admission controllers and controllers modify resources after applying.

E2e test based on real resources.

Happa's requirements can be tested even without contract.
How to create the resources to test?

Can we avoid creating cloud provider resources?

Can we use the test suite as a validator for arbitrary installations? (popeye style)
Should be executed as part of the cluster-app/default-apps release process
Not only metadata (labels, annotations) should be testable. Also e. g. string values, anything that goes beyond the specified OpenAPISchema, also referential integritiy and presence of resources.
Ownership?

"Contract" should be encoded in a way that crd-docs-generator, e2e test suite, and customers can make sense of it.

Human readable changes (diffs) and changelog entries would be a benefit.

Notes from Nov 10 sync:
Let's look into CEL - https://github.com/google/cel-spec, to find out if it can be useful for this story

Progress of this issue is tracked in 2 other tickets:

https://github.com/giantswarm/roadmap/issues/1599
https://github.com/giantswarm/giantswarm/issues/24462


Role library now requires non-zero addresses.

The null address (0x00) is now considered a domain error in the Roles library, and all transactions involving it with revert.

@nventuro This seems to be causing some of the SignatureBouncer tests to fail. It may be because of the check added in has.


OnAfterInsertRecord right after commit in CRM sync

Hi,
I noticed that the onafterinsertRecord event on the CRM sync is right after a commit. This is very inconvenient when extending the CRM sync. We use this event to add extra data after the insertion of the record. If something fails after the commit, the record is inserted and all other needed information fails. This makes the solution a bit unsafe for transferring all the data.
Partners need this event before the commit, because in some cases it is necessary that the record needs to be inserted first because of numbering, validation, etc. Of course, workarounds can be thought of, but are really messy and not easy to implement.
Is it possible to make an extra event right before the commit? Or maybe move it before the commit. If something fails, all inserts will be rolled-back and tried again in next sync. Which is also a lot more data safe.

I am talking about object: codeunit 5345 "Integration Rec. Synch. Invoke"

If you need an event, please provide event definition.

Put the even after the applyconfigtemplate function and before the if <> fail. Please use the same parameters as onAfterInsertRecord in the screenshot. My suggestion would be:
OnAfterInsertBeforeCommit(IntegrationTableMapping: Record "Integration Table Mapping"; var SourceRecordRef: RecordRef; var DestinationRecordRef: RecordRef).

@AlexanderYakunin is this up for re-evaluation?

@AlexanderYakunin I am asking again. Is the information i provided enough for you to reevaluate? Is there any wiki how request should be formulated?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 13655.

Availability update: We will publish a fix for this issue in the next update for release 23.
Build ID to track: 13708.


Possibly unhandled rejection closing Modal with ESC key

Bug description:
Closing a modal with ESC end up with "Possibly unhandled rejection: escape key press"
$modalStack.dismiss(modal.key, 'escape key press'); created a rejected promise that is not catched.
here
if (modalWindow && broadcastClosing(modalWindow, reason, false)) {
          modalWindow.value.modalScope.$$uibDestructionScheduled = true;
          modalWindow.value.deferred.reject(reason);
          removeModalWindow(modalInstance, modalWindow.value.modalOpener);
          return true;
        }

Version of Angular, UIBS, and Bootstrap
Angular: 1.6.1
UIBS: 2.2.0
Bootstrap: 3.3.7

Same problem as the finally. It can be taken care of by client with .finally().then(angular. noop, angular. noop);

@Rouche can you explain how to fix it more clearly or send a link for fixing example. I experience the similar bug when clicking outside of the Modal to close Modal.

Maybe its not the same problem, us, was because we used finally() on window.
Here is the complete code (See fix on the last line finally()):
       var instanceBootstrap: ng.ui.bootstrap.IModalServiceInstance = this.$uibModal.open({
            templateUrl: url,
            controller: myCont,
            resolve: {
                model: function() {
                    return model;
                }
            },
            keyboard: options.closeOnEscape,
            scope: myScope,
            windowClass: myCss
        });
        instanceBootstrap.result.finally(
            (): void => {
                    myScope.$destroy();
                }
            }
        ).then(angular.noop, angular.noop);

a better solution: https://github.com/angular-ui/ui-router/issues/2889#issuecomment-237521045

@hikalkan:

Is there a more elegant way to fix this without adding .result.finally(angular.noop).then(angular.noop, angular.noop) at every $uibModal.open() call? Looks like a dirty hack, specially if you have a few modal calls

Try not using finally() Finally creates a new promise, you have no choice to catch it, even if you have nothing to do, and you probably have nothing to do in a modal's finally.then().
Calling open without using finally wont give you the error.

It does for me. Pressing "esc" logs the error on console. This is how I call the modal:
$uibModal.open({
      templateUrl: "views/layout/modals/audienceList.modal.client.view.html",
      controller: "AudienceListModalController",
      size: "lg"
    })

Chaining  .result.finally(angular.noop).then(angular.noop, angular.noop) to the open method fix the issue for me.

Well, its not perfect for sure. But you can wrap it in another service so you dont need to repeat the code everywhere.


Consider remove third-party library "request" ?

Instead of it, use Electron API's "ClientRequest" as a alternative.

thanks to @marlon360 for implementing this :)


Options for filtering

As a user, I want to have access to the available options when filtering the graph so that I know how I can filter it
Acceptance Criteria

[ ] Performance should not be downgraded

[ ] Available filtering options are accessible

[ ] (remove ID as a filter?)

Performance is not a concern as long as the frontend doesnt submit superfluous queries, all the filtering is done on the native backend.


Registration of the default endpoint(s)

ApplicationFrameworkLayer.createDefaultSendingEndPoint() registers 3 endpoints, apparently without any real reason.
The code is weird too because it iterates over a set of 34 clusters and registers as many endpoints as needed (i.e. 3 endpoints) each with 16 clusters as inputs and outputs (the third only contains 2 clusters).
Also, each endpoint is registered specifying a device ID=0 in the Home Automation Profile which is the On/Off Switch in the HA specification.
I believe this default registration should not be hardcoded in the ApplicationFrameworkLayer and delegated to the application (e.g. the ZigBeeConsole).
Hint: the HA Combined Interface described in the HA spec could be a better choice.
Ciao,
Cristiano

Hi Cristiano,
My understanding is that the default sending endpoint(s) implement all the clients to be able to interoperate with other servers of ZigBee devices.

I believe this default registration should not be hardcoded in the ApplicationFrameworkLayer and delegated to the application (e.g. the ZigBeeConsole).

Agree.
Regards,
Brhett

Hi Brhett,
If you plan to create a PR with your fixes to addCustomEndopint, would you also consider the above?
Thanks,
Cristiano

The code is weird too because it iterates over a set of 34 clusters and registers as many endpoints as needed (i.e. 3 endpoints) each with 16 clusters as inputs and outputs (the third only contains 2 clusters).

Actually, the clusters are registered as outputs only. See https://github.com/tlaukkan/zigbee4java/blob/master/zigbee-api/src/main/java/org/bubblecloud/zigbee/network/impl/ApplicationFrameworkLayer.java#L202
I think its original purpose was create a ZigBee On/Off Switch to control the Philip Hue for example. But then it became to a "Combined Interface Device"+. It is used to control other ZigBee devices as many as possible.
In order to be HA specification compliant, I think there should be several specific endpoints being registered to cover the functionalities. e.g. "Combined Interface Device" + "Thermostat" + "Home Gateway" + "IAS CIE" + ... However, some of them have common clusters, which might introduce more complexity.
So I think it's better to delegate the endpoints registration to the application. e.g. ZigBeeConsole