Launcher crash on start

Description: Trying to start the launcher, it's been crashing, and giving me this error. I've tried following the instructions on the download page, going to %LOCALAPPDATA%\CDDA Game Launcher\ to delete config.db, but this has not improved the issue.
Version: 1.6.7
OS: Windows-10-10.0.22000-SP0 (64-bit)
Type: <class 'AttributeError'>
Value: 'NoneType' object has no attribute 'split'
Traceback:

  File "C:\Program Files (x86)\CDDA Game Launcher\cddagl\ui\views\main.py", line 1412, in showEvent
    self.refresh_builds()
  File "C:\Program Files (x86)\CDDA Game Launcher\cddagl\ui\views\main.py", line 3266, in refresh_builds
    self.refresh_changelog()
  File "C:\Program Files (x86)\CDDA Game Launcher\cddagl\ui\views\main.py", line 3292, in refresh_changelog
    body = entry['body'].split('####'),

Also having the same problem

I don't know if this helps; but the launcher was working fine this morning (10/6/22), and started crashing every time I try to start it in the evening.

The crash seems to be due to failing to parse the latest game build's changelog. Hence one workaround is to launch without internet - then it can't retrieve the latest builds and so doesn't crash.

If someone wants to get their launcher working before the PR gets merged, go to <launcher directory>\cddagl\ui\views\main.py and change line 3292 in the following way:
-                                        body = entry['body'].split('####'),
+                                        body = (entry['body'] or "").split('####'),

Looks like @RundownRhino already has a fix.. but if you don't wanna wait for the repo owner to merge, you can just copy-paste the change from their PR and slap it onto '\cddagl\ui\views\main.py'
You can also just add like 2 lines to skip the bad entry. Should be around line 3292 on the file just like the log says. Works fine for me eitherway.
        #START - COPY THESE 2 LINES######################
        if entry['body'] is None:
            continue
        #END#######################################
        new_entry = changelog_entry(title =entry['title'],
                                    body = entry['body'].split('####'),
                                    node_id =entry['node_id'],
                                    number=entry['number'],
                                    html_url=entry['html_url']
                                    )

        if new_date in changelog_sorted:
            changelog_sorted[new_date].append(new_entry)
        else:
            changelog_sorted.update({new_date : [new_entry]})

The entry is fine except for not having a description, so I wouldn't skip it. Here's how it looks with my PR applied:

oh. good point.
that's just me being lazy since i don't really try to read the changes too much.

Thank you, RundownRhino! Your fix has the launcher working again.
This opens up a new question. Will I need to undo this change when something updates in the future?

@RundownRhino : You have my thanks also.

@Arcania88 I'm not entirely sure, but my understanding is that the launcher update process will replace the file entirely.

@RundownRhino That makes sense. Thanks again for the help.


Add a LICENSE

Add a LICENSE file to the GItHub repo. This helps people who want to use openkeepass or contribute to it, they can directly see what they are allowed to do.
I see you are using Apache 2.0 on Maven. You should add either the full text of the license (up to "END OF TERMS AND CONDITIONS") or the short version (under "APPENDIX", replace the placeholders) to the repo.
For bonus points add a link to tldrlegal at the top.
I plan to use openkeepass myself and have one or two things I'd like to contribute. :)

Adding a license file to the repo makes sense. However, in the pom file the license is already mentioned and it should be sufficient as it is right now.


Dart 2 compatibility issue / type '_Future' is not a subtype of type 'FutureOr'

Hello,
When using Dart2 on a Flutter project, the signOut() method is not working:
E/flutter (20982): [ERROR:topaz/lib/tonic/logging/dart_error.cc(16)] Unhandled exception:
E/flutter (20982): type '_Future' is not a subtype of type 'FutureOr<Null>' where
E/flutter (20982):   _Future is from dart:async
E/flutter (20982):   FutureOr is from dart:async
E/flutter (20982):   Null is from dart:core
E/flutter (20982):
E/flutter (20982): #0      FacebookLogin.logOut (package:flutter_facebook_login/flutter_facebook_login.dart:150:42)
E/flutter (20982): <asynchronous suspension>

More details in this issue: flutter/flutter#15654

Hi!
Thanks for the info. I'll look into making this Dart 2 compatible during the weekend.

Should be now fixed! If possible, confirm that it does indeed work for you :)


chore(deps): update jx-api version

Update jx-api version so that jx gitops requirements merge pulls new fields from terrafrom requirements

/cc @tomhobson


[SPARK-18200][GRAPHX][FOLLOW-UP] Support zero as an initial capacity in OpenHashSet

What changes were proposed in this pull request?
This is a follow-up PR of #15741 in order to keep nextPowerOf2 consistent.
Before
nextPowerOf2(0) => 2
nextPowerOf2(1) => 1
nextPowerOf2(2) => 2
nextPowerOf2(3) => 4
nextPowerOf2(4) => 4
nextPowerOf2(5) => 8

After
nextPowerOf2(0) => 1
nextPowerOf2(1) => 1
nextPowerOf2(2) => 2
nextPowerOf2(3) => 4
nextPowerOf2(4) => 4
nextPowerOf2(5) => 8

How was this patch tested?
N/A

Test build #68074 has started for PR 15754 at commit 95ad783.

Test build #68074 has finished for PR 15754 at commit 95ad783.

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/68074/
Test FAILed.

The only failure seems to be irrelevant.
[info] StreamingContextSuite:
[info] - stop slow receiver gracefully *** FAILED *** (1 second, 419 milliseconds)

Retest this please.

Test build #68078 has started for PR 15754 at commit 95ad783.

Thank you for review and approving, @srowen .

Test build #3411 has started for PR 15754 at commit 95ad783.

Test build #68078 has finished for PR 15754 at commit 95ad783.

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

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

Merged build finished. Test PASSed.

Test build #3411 has finished for PR 15754 at commit 95ad783.

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

Merging in master/2.1/2.0.


[stable/nginx-ingress] Configurable default-backend port

What this PR does / why we need it:
It add a new value defaultBackend.port for being able to configure the port of the default-backend pod.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

What's the reason for this? It would be kind of inconsistent to only make the port of the default backend configurable.

/assign

Hi @unguiculus
Previous to this PR, the default backend port should always be 8080 and so, we can't use an image which expose a arbitrary port.

It would be kind of inconsistent to only make the port of the default backend configurable.

What do you see that could be configurable?

You will have to bump the chart version.

@unguiculus Version bumped and README updated. Thanks

/ok-to-test

/lgtm


GT links to loc catalog author bio are not full text links

This record links to http://catdir.loc.gov/catdir/enhancements/fy1318/2013434364-d.html
It should not display an Online button
http://findit.library.gwu.edu/item/13318936

BTW, this record looks like it has some other problem-- the author should be García Lorca, as it is in the original GT record, but it's gotten cut off along the way. The problem exists in WRLC Voyager, so it's not a launchpad issue.


Creating a notebook doesn't immediately show

I opened Noterium anew and created a notebook called Development. Clicked okay, waited for the NoteBook to appear in the left panel but did not. Restarted the Noterium and it now shows.
Note: Thanks for the nifty application. Been after something like this for a while!

Hi @wonea, will have a look at this as soon as possible, hopefully sometime this august, not a lot of free time nowadays.
You're quite welcome, really hope it come in handy for you. Don't hesitate to ask for improvements or do a pull request. :)


Bug - Assessment plan titles load incorrectly

Description
The levels and Level descriptions aren't loading correctly.

[ ] This bug requires a unit test.

Steps to Reproduce

Steal Gray's login credentials
Log in as manager
Click on Assessment plan
See error on level titles and descriptions in each skill

Images

Resolved by #881


Take advantage of C# 11

Fixes: My own personal issues.
Commit-by-commit review thoroughly recommended, so you can skip the mechanical change and look at the interesting ones separately.

Draft so I can see what CI thinks of this.. VS preview is happy, and command line build on my machine is happy


Badge request: quay.io

This is a docker image builder service:
They dont't seem to provide public API access, just a svg url, is this enough? For example: https://quay.io/repository/glerchundi/parkeeper/status.
Thanks!

Thanks for pointing it out!
I do rely on SVG scraping for some things, but I tend to de-prioritize implementing that, since it can be brittle and requires more work later. It seems they use Shields.io internally.
If you can ask the quay folks whether they plan on adding an API, I would be very grateful!

Quay dev here: What kind of API do you guys expect? We'd love to be able to be officially supported by shields.io instead of just hosting our own SVGs.

@jzelinskie A common strategy is to download the badge you wish for from http://img.shields.io/badge/insert-text-green.svg and serve it to users.
If you wish the badges to be available under the img.shields.io domain, we need an API, preferably JSON, that gives us enough information to construct the badge.

@espadrine we currently do the former (hosting of SVGs we downloaded from you guys), but we have an API that exposes this info: the phase attribute on this API call http://docs.quay.io/api/swagger/#!//getRepoBuilds

Neat!

Any progress on this? The API of quay.io has been changed, not sure if the method proposed above is still feasible.

Just wanted to put out there that Shields is a community project, and that means you can help make this happen! 👐 We have a great tutorial and welcome anyone who would like to dig into this! Our maintainer team is happy to provide guidance on the API design and provide assistance with the implementation. We've had many first-time contributors who don't use JavaScript and have tried to make the first-time contributing experience as straightforward as possible.

@paulmelnikow Thanks for the prompt response! Yeah, I'd be really excited to contribute to this great project. Specifically on this issue, my concern is that quay.io might not be providing sufficient information through their endpoints to build such a badge 🤔 Let me do some research and see if I could try to make this happen!

Sounds great!
Another thing to keep in mind whether there is an authentication requirement. As a rule we don't allow passing read-write credentials to the server for security reasons. We can install a production API token, though it won't be specific to any particular projects.
Also, if you want to, you could play around with a live implementation using RunKit and our beta Endpoint badge: #2838.

Notice this was tagged good first issue, though I removed that tag since I'm not sure whether we have the upstream API we need.

I would really like to have quay.io badges for image size, pulls, and stars. Should I open another issue?


Add Payara 5.2020.2 as MP 3.3 compliant

@rdebusscher please add 5.2020.2 as MP 3.3 compliant to https://wiki.eclipse.org/MicroProfile/Implementation


[DashboardLayout] Improve header title customizability

Summary
Right now, customization options for the title that shows in the layout header are quite limited.
Even though it might be possible to do it by using the sx prop in DashboardLayout and targeting the correct element, we could find ways to make that title more straightforward to customize, as it might be something many users want more control over.
A simple, flexible solution might just be adding a new slot for that title.
Examples
No response
Motivation
User feedback on docs: "How to style the branding title? Can I change it to a component instead of a text?"
Search keywords: dashboard layout header branding title appbar style

Relates to: https://github.com/mui/toolpad/issues/4450

+1 for this.
title as string is rather limiting, it should be able to accept full ReactNodes or make it work like a slot, ala toolbar.
For the record, passing a ReactNode to title right now works, with only TS throwing a type error

For the record, passing a ReactNode to title right now works, with only TS throwing a type error

This would be my least favourite solution as I could imagine we'd want future use cases where no React host is available. But I could see a slot or a render prop work well.

Oops, @Janpot  I want to issue a correction: I thought this was about the title in the PageContainer, and that's where my comment was applying to!
I'll see if there's a request-issue for PageContainer title


Missing WEBP file icon

A webp file icon appears to be missing.
A webp file icon should appear next to .webp files.
I'm not sure if I should post the issue here or with A File Icon, but I've tried reverting to a fresh state and get the same results – missing .webp icon.
Running Sublime v3, 3143. MacOS High Sierra.

@djmtype Hello!
Thanks! Yes, it's missing now. I've created an issue for that here. It will be added with the next version of A File Icon.

I'm not sure if I should post the issue here or with A File Icon, but I've tried reverting to a fresh state and get the same results – missing .webp icon.

Next time please file your icon-related issues at A File Icon :wink:
P.S. Note, this theme will be deprecated later this year. Consider giving a try to my new theme.


Update LLVM.jl

I'm probably missing some context, but there's no LLVM.jl v8 yet: https://github.com/JuliaRegistries/General/blob/cdbfd773a76c85cbfaf1d32bb11468ad6ce705c0/L/LLVM/Versions.toml?


Weird toString failure printing over sbt.

steps

The version of SBT I am using is 1.5.4
The version of scala I am using is 3.0.0
The OS I am using is Ubuntu 20.04
The terminal I am using is gnome-terminal
My build.sbt looks like this:
val scala3Version = "3.0.0"

lazy val root = project
  .in(file("."))
  .settings(
    name := "scala3-simple",
    version := "0.1.0",

    scalaVersion := scala3Version,

  )
libraryDependencies += "org.typelevel" %% "cats-parse" % "0.3.4"

libraryDependencies += "org.scalameta" %% "munit" % "0.7.25"
testFrameworks += new TestFramework("munit.Framework")

libraryDependencies += "com.softwaremill.magnolia" %% "magnolia-core" % "2.0.0-M9"

I have a file called "LexerGrammar.scala" that contains a function that looks like this:
import cats.parse.Rfc5234._
import cats.parse.{Parser => P, Parser0}
import cats.parse.Parser._

final case class CharLiteral(value: String)

object Grammar:
        val charLexer              : P[CharLiteral] =
		val emptyString = (cats.parse.Parser.char('\'') ~ cats.parse.Parser.char('\''))
		val nonEmptyString =
			(
				P.string("\\n").backtrack |
				P.string("\\\\").backtrack |
				P.string("\\t").backtrack |
				P.string("\\b").backtrack |
				P.string("\\r").backtrack |
				P.string("\\f").backtrack |
				P.string("\\a").backtrack  |
				(P.string("\\u") ~ digit.rep).backtrack |
				P.charIn("!#$%&()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ'[]^_`abcdefghijklmnopqrstuvwxyz{|}~€‚ƒ„…†‡ˆ‰Š‹ŒŽ‘’“”•–—˜™š›œžŸ ¡¢£¤¥¦§¨©ª«¬®¯°±²³´µ¶·¸¹º»¼½¾¿ÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖ×ØÙÚÛÜÝÞßàáâãäåæçèéêëìíîïðñòóôõö÷øùúûüýþÿ"))
					.surroundedBy(cats.parse.Parser.char('\''))

		(emptyString.backtrack | nonEmptyString).string.map(value => CharLiteral(value))

Which I call via the main function that looks like this:
import Kierkegaard.Lexer.Grammar._

@main def hello: Unit =

  println("Running...")

  charLexer.parse("'\n'") match {
    case Right(some) =>
      println(some._2.value)
    case Left(none) =>
      println(none.toString)
  }

problem
When I run that I get a failure to parse that I still do not understand but the problem is that it is failing to print the information on when it failed to parse. More specifically it is failing to properly render "println(none.toString)". When I run this, I get this output:
[info] compiling 1 Scala source to /media/kevin/hello/kierkegaard/target/scala-3.0.0/classes ...                                                    ), InRange(1,,
[success] Total time: 14 s, completed Jul 20, 2021, 3:07:51 PM

Notice the weird text "), InRange(1,," at far right its failure to run "println("Running...")" at all.
expectation
I was expecting something more along the lines of:
[info] compiling 1 Scala source to /media/bradly/hello/kierkegaard/target/scala-3.0.0/classes ...
[success] Total time: 14 s, completed Jul 20, 2021, 3:07:51 PM
Running...
Error should be stated here

notes
I have been using scala3 and had no problem until a few moments ago and I have been using the same version of SBT for some time now as well so I have no clue as to what could be going on. I restarted by terminal, computer, and sbt but the result is always the same. Note that testing prints properly.

Must have been a terminal issue. I switched to xterm and now it is displaying properly.


Add 3D map

✨ Feature Request
Description of the Feature
Add Google map to the 3D-map section
Problem Statement
Missing map in 3D map section
Proposed Solution
Add google map in the 3D map section

Alternatives Considered
None
Additional Context
None
Are you participating in GSSoC?

[X] Yes
[ ] No

do you know how to use MAPBox ??

i was doing with google api

google map api is now free

we are thinking to  make 3d models of station

For Mapbox it is asking for linking bank account I tried but my card is not supporting
do you have account ?

shall i add the map in the navigation in the 3d map section ?


Upstream package sync replacement

The upstream 'sync' functionality that was enabled for the on-prem depot has a number of issues that need to be resolved. The issues range from operational (eg, setup/configuration/management -  transparency of failures, resolving errors, etc) to user facing (eg, synchronous vs. asynchronous results, specific triggers that enable package sync, etc) to missing key functionality (eg, dependent packages or keys are not synchronized).  The overall feature needs to be revisited and redesigned with a more detailed user story and feedback from the on-prem user community.

For my org, something similar to Artifactory's virtual repos would be ideal. Any upstream packages fetches would happen synchronously and transparently when first requested and cached for subsequent requests.

So in the meantime, is there a command which pulls a single package into a depot?

This worked for hab 0.59.0 (regressed recently, see https://github.com/habitat-sh/habitat/issues/5678)
# HAB_BLDR_URL and HAB_AUTH_TOKEN should be set in your env
function on_prem_sync() {
  PKG_IDENT=$1
  hab studio run "hab pkg install $PKG_IDENT --url https://bldr.habitat.sh -z '' && hab pkg upload /hab/cache/artifacts/$(printf $PKG_IDENT | sed 's/\//-/g')-x86_64-linux.hart"
}

@jamessewell Yes. the current plan is to have some helper commands (scripts) available to enable upstream pulls/updates until we get a better sync experience in place.  Those should be available soon (before we pull the feature).

@chefsalim I'd love to see this return a 202 when a package is not yet available. The client would need to be modified to take that 202, add a delay and add to the end of the queue of things to pull down. This would allow the on-prem-builder time to fetch the requested package.
   +--------+            +-----------------+         +------------------+
   | Client |            | On|Prem Builder |         | Official Builder |
   +--------+            +-----------------+         +------------------+
       |                          |                           |
       |          get             |                           |
       |       core/foo           |                           |
       |+------------------------>|                           |
       |                          |                           |
       |          202             |                           |
       |    Hang on a minute      |                           |
       |<------------------------+|            get            |
       |                          |         core/foo          |
  +----|                          |+------------------------->|
  |    | Re-queue                 |                           |
  +--->|                          |<-------------------------+|
       |                          |
       |                          |
       |     Some time later      |
       |                          |
       |          get             |
       |       core/foo           |
       |+------------------------>|
       |                          |
       |      core/foo for you    |
       |<------------------------+|

Please see https://github.com/habitat-sh/on-prem-builder/pull/106 for a replacement script.  This is not intended to be the 'ultimate' solution, rather a stopgap as discussed.

@predominant I'm not sure that is the model that we will ultimately land on. It's not clear that will be a great or transparent user experience to have a 202.  Lets say the background sync attempt fails (upstream is unavailable then for example) - then should the client continue to keep waiting and for how long.  How or whether to give the client status that something failed?  Also there's not just one 'get' for a packages, we have many APIs like getting the latest metadata, search, etc.  There's issues like that that get thorny in this model.

Understood

There is also better, but harder method: you can issue a 302 redirect to a unique "task-page" which returns:
"202"  - while processing
"5xx" - on error
"200" - when ready
Download synchronously, like @HT154 said is also option.
I'm worried about real end-to-end use case.
Let imagine we have:

on-prem https://bldr.local
secret/myproject habitat package (it depends on core/postgresql and on secret/libA)

I can try build the package with public builder -> no secret/libA -> no success.
I can try build the package with private builder -> no package updates (like core/postgresql)
So it is not clear for me how stuff can be automatically updated without explicitly upload them? Allow habitat to specify multiple repositories (like APT/YUM, MAVEN)? Make on-prem builder like supervisor - e.g. listen for updates in upstream builder and download them immediately?
If packages somehow get on-prem builder there will be no point to do "proxy download", "302-202-200 dance" - artifact either exists or not.

One major pain point that this issue would address is the constant missing packages after a core plans refresh. I usually spend a whole day or more dealing with this all over my build system:
↓ Downloading core/nghttp2/1.34.0/20190115160823
↓ Downloading core/nghttp2/1.34.0/20190115160823
↓ Downloading core/nghttp2/1.34.0/20190115160823
↓ Downloading core/nghttp2/1.34.0/20190115160823
↓ Downloading core/nghttp2/1.34.0/20190115160823
✗✗✗
✗✗✗ We tried 5 times but could not download core/nghttp2/1.34.0/20190115160823. Giving up.
✗✗✗

I'm still in "developing-refactoring", but decided to push a scratch commit with hab-depot-sync.
It, probably, buggy, but I was able to incremental sync only latest packages. Syncing doesn't require disk space.
Unfortunately you have to:

have ruby
clone https://github.com/habitat-plans/hab-sdk.git
rake build && rake install

Try hab-depot-sync --help

Work still in progress. Now I have habitat:
sudo hab pkg install -b ya/hab-sdk -c unstable
hab-depot-sync --dest-depot https://bldr.on-prem.domain.local --dest-auth-token $auth_token --origin core --latest-version


Update 'concatenates JavaScript files' on query-string-sort.md

Addresses #13502

Thank you @colorful-tones


make iterate() support arrays

You can now do
bipf.iterate(buffer, pointerToArray, (buf, pointerToItem) => {
  console.log(bipf.decode(buffer, pointerToItem))
})

fix implementation of iterate
make iterate support aborting
replace == with === (it's faster)
added tests
documented in readme

Looks good. Seems like the array was a bit of copy/paste.

Do you have publish rights on npm for this package? npm info says it's only dominic :sweat_smile:

Sadly no ;)

@dominictarr One quick request: could you npm owner add arj03 bipf? :raised_hands:


Clarifying behavior of env, service, and version when extension_layer_version is not provided (ie because lambda docker image is used)

Expected Behavior
Given below code, with lambda_handlers running Docker images,
    datadog = Datadog(
        stack,
        "Datadog",
        add_layers=False,  # Extension provided in Dockerfiles. We don't add layers here.
        # NOTE: Extension layer is NOT used however we do need it to add the tags we're specifying in the construct.
        # https://github.com/DataDog/datadog-cdk-constructs/blob/main/src/env.ts#L127
        **extension_layer_version=None,**
        api_key_secret=datadog_api_key,
        site="datadoghq.com",
        enable_datadog_logs=True,
        enable_datadog_tracing=True,
        enable_merge_xray_traces=True,
        inject_log_context=True,
        log_level="debug",
        capture_lambda_payload=True,
        env=stage_name,
        service=stack.stack_name,
        version=version,
    )
    datadog.add_lambda_functions(lambda_handlers)

the env, service, and version should correspond to the DD_ENV, DD_SERVICE, and DD_VERSION environment variables added to the lambda handlers.
It does seem that passing a dummy extension_layer_version=65 does add them again.
Actual Behavior
DD_SERVICE, DD_ENV, and DD_VERSION the variables seem not to get added if you don't pass extension_layer_version, and these are typically how one may find the relevant traces.
Noted here in the code that the constructs do not add these variables if the extension layer is unspecified: https://github.com/DataDog/datadog-cdk-constructs/blob/main/src/env.ts#L127-L140
It seems this may be intended behavior, but I am not sure why.
https://github.com/DataDog/datadog-cdk-constructs/issues/185 seems related.
Steps to Reproduce the Problem

Create a DockerImage lambda function which includes the datadog extension already.
Create Datadog construct with no extension_layer_version.
Deploy and note whether or not DD_SERVICE, DD_ENV, and DD_VERSION are set.

NOTE: Traces are showing up fine. It's just that the 3 common fields I'd search for in APM Traces are not tagged (ie searching by service, env, or version) so it appears as if it doesn't work.
Specifications

Datadog Lambda Layer version: 65 (installing via COPY --from=public.ecr.aws/datadog/lambda-extension:65 /opt/. /opt/ based on public.ecr.aws/lambda/python:3.10
)
Node version: v20.12.0 (CDK)

datadog-cdk-constructs-v2==1.18.0
Stacktrace
There is no crash; this behavior may be WAI.
Would anyone be able to share?
BTW - thank you Datadog team for a great product. It is so useful :)

Thanks for reporting! I think DD_SERVICE, DD_ENV, and DD_VERSION should be set in this case. I'm going to track this issue internally. For now, you can set a dummy extension_layer_version to have the environment variables set.

Internal ticket: https://datadoghq.atlassian.net/browse/SVLS-5931


⚠ Bofrost.es has degraded performance

In c5e26ec, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1511 ms

Resolved: Bofrost.es performance has improved in 58cfa99 after 6 minutes.


Mogli updates

Allow test suite to run
Relax multi_json version

Opps, raised this on the wrong repo, sorry! This was meant to go to our fork!


MTR Manual App update command incorrect

The powershell command for updating the MTR app does not work.
Add-AppxPackage -Update -ForceApplicationShutdown -Path '\$oem$$1\Rigel\x64\Ship\AppPackages**.appx' -DependencyPath (Get-ChildItem '\$oem$$1\Rigel\x64\Ship\AppPackages*\Dependencies\x64*.appx' | Foreach-Object {$_.FullName})
the"-Update" switch should be removed.

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

ID: fd52fe62-457b-c9e1-ad13-a9b1f24c9c38
Version Independent ID: 6a8c3285-30e1-15a8-6472-f0cfe5b30e20
Content: Microsoft Teams Rooms maintenance and operations - Microsoft Teams
Content Source: Teams/rooms/rooms-operations.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@nachapman Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

This is not a support issue. It's a documentation issue.
The documentation is not correct.
Others have found the same problem with the documentation.
https://www.graham-walsh.com/manual-update-of-the-microsoft-teams-room-system-app/
"The difference from the Microsoft docs and to the actual correct command is that you don’t use -Update in the script."


Basemaps do not show up when zoomed in to southern California

Zoom in to southern California, for example Redlands, and the basemaps do not display. You have to zoom out several levels to get it to display. Instead, it should display at all levels since this is our "new geography" area of interest.

@garys-esri Please refer to following web map for areas that have high resolution caches.
https://nsec.esri.com/arcgis/home/webmap/viewer.html?webmap=31a38062b3194689989bfa4eddc64b15


travis: PHP 7.0 nightly added, cleanup

Coverage remained the same at 96.03% when pulling 378a2bee05d27054bfca8ce1ac972e680ffe29f0 on TomasVotruba:patch-1 into c6ebe6c1337974cf1f14c23051664fef0307f648 on jenssegers:master.

Coverage remained the same at 96.03% when pulling 378a2bee05d27054bfca8ce1ac972e680ffe29f0 on TomasVotruba:patch-1 into c6ebe6c1337974cf1f14c23051664fef0307f648 on jenssegers:master.

:+1:


Fix deployment bug

Fix deployment bug for zip deploy

Superseded by https://github.com/Azure/ai-hub/pull/157


The app is crashing for no reason

So, when I try to open one of the library folders from the sidebar, in that case the images and video folders, the app closes for no apparent reason, and I don't know why

Hi @DustyOldRowe , this is the files-website repository. For feedback regarding the Files App, please create an issue on their GitHub repository.

Hi @DustyOldRowe , this is the files-website repository. For feedback regarding the Files App, please create an issue on their GitHub repository.


Other diarization metrics

Following metrics (from pyannote and dscore) may be implemented:

[x] Diarization error rate (DER)
[ ] Jaccard error rate (JER)
[ ] Purity and coverage
[ ] Bcubed precision/recall
[ ] Goodman-Kruskal Tau
[ ] Mutual information

Are JER/clustering metrics still of interest? I'd be up for adding them if I know the PRs would get accepted.

Hi Neville! Yeah, that would be awesome. JER is top-most on the list, but I can imagine people would be interested in other metrics as well.
(@popcornell and I want to switch from dscore to spyder in CHiME-7 DASR, but it is blocked by JER not being implemented yet.)

Ok, I can add this to the TODO list. I'm in the process of rewriting dscore to eliminate the md-eval dependency and output more detailed reporting. The initial version is based on pyannote.metrics, but between the penalty of Python being an interpreted language and the repeated calls to uemify, it's not particularly quick. So, it's in my interest to get faster implementations of the various metrics and I'd rather contribute to an existing project if possible.

Cool! Your contributions would be very welcome. In my benchmarking, I found pyannote.metrics to be an order of magnitude slower than md-eval.pl --- pyannote is a great tool overall, just not suitable for DER evaluation :)
I'm sure spyder would benefit immensely from your expertise. Please use this thread for any questions/discussions once you get around to implementing the metrics.

That sounds about right. When I benchmarked on the DIHARD III eval (full) condition, just the DER computation (omitting IO and building the Annotation/Timeline instances in memory) averaged over 13 seconds; cf. to 3.5 seconds for running md-eval. Most of this comes from the call to IdentificationErrorRate.uemify that constructs the equivalent of your get_eval_regions. Specifically, this block, which accounts for 10 seconds of that run time.
I've been updating dscore off-and-on for the past week for an LDC internal project and want to finish that work first, but will look into implementing JER in spy-der after. I think it should be relatively straightforward.


incorrect if logic in Speller.replaceRunOnWords()?

This looks wrong, it uses getOutputConversionPairs() if it's empty, shouldn't it be the other way round?
https://github.com/morfologik/morfologik-stemming/blob/dc8dde7da215b51166931f3eb4d49a74397f8a01/morfologik-speller/src/main/java/morfologik/speller/Speller.java#L340-L345
I'm not sure if applyReplacements() makes sense at all. We've just checked that both parts are in the dictionary, should replacements still be applied? The if isn't tested, remove it and testRunonWords() still works.
Ping @milekpl, seems this was originally your code.

@jaumeortola Maybe you can comment on this?

I think you are right. The 'if' can be removed and just do candidates.add( ). But I haven't done any test.
Is this used anywhere? Does it cause any error?

I found this when trying to develop a fix for https://github.com/languagetool-org/languagetool/issues/731#issuecomment-328265948 and I just thought it's probably better to clean it up first.


Upgrade browserify and other dev dependencies

Upgrade browserify to get fixed ecstatic to avoid TypeError('The header content contains invalid characters') for browsers with non-latin headers connected to local frontend:
_http_outgoing.js:360
    throw new TypeError('The header content contains invalid characters');
    ^

TypeError: The header content contains invalid characters
    at ServerResponse.OutgoingMessage.setHeader (_http_outgoing.js:360:11)
    at serve (C:\Install\Garmin\GitHub\osrm-frontend\node_modules\ecstatic\lib\ecstatic.js:204:11)
    at C:\Install\Garmin\GitHub\osrm-frontend\node_modules\ecstatic\lib\ecstatic.js:134:11
    at FSReqWrap.oncomplete (fs.js:123:15)

Original issues:

https://github.com/indexzero/http-server/issues/244
https://github.com/angular/angular-phonecat/issues/303

This also updates deprecated<EMAIL_ADDRESS>to 3.0.3 (#205)

Looks good to me!


Update config and run Detekt during the build

This changeset does the following:

Updates config.
Includes Detekt code analysis into the build and addresses its findings.
Adds in-place pull script.

The version is bumped to 2.0.0-SNAPSHOT.122.

@alexander-yevsyukov @armiol Please take a look.


[SPARK-47847][CORE] Deprecate spark.network.remoteReadNioBufferConversion

What changes were proposed in this pull request?

spark.network.remoteReadNioBufferConversion was introduced in https://github.com/apache/spark/commit/2c82745686f4456c4d5c84040a431dcb5b6cb60b, to allow disable SPARK-24307 for safety, while during the whole Spark 3 period, there are no negative reports, it proves that SPARK-24307 is solid enough, I propose to mark it deprecated in 3.5.2 and remove in 4.0.0
Why are the changes needed?

Code clean up
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Pass GA.
Was this patch authored or co-authored using generative AI tooling?

No

cc @dongjoon-hyun and @mridulm

To @pan3793 , I rethink about this.

I fill the deprecated message with "Not used anymore", to be consistent with existing items
DeprecatedConfig("spark.yarn.am.port", "2.0.0", "Not used anymore"),
DeprecatedConfig("spark.executor.port", "2.0.0", "Not used anymore"),
...

Since remoteReadNioBufferConversion is used in the code, Not used anymore is technically wrong.
Shall we ask the user to report like the following?
https://github.com/apache/spark/blob/1138b2a68b5408e6d079bdbce8026323694628e5/sql/core/src/main/scala/org/apache/spark/sql/execution/analysis/DetectAmbiguousSelfJoin.scala#L101
For example, we can use Please open a JIRA ticket to report it if you need to use this configuration..

BTW, we need to give enough time to report the issue from users. So, we cannot delete this configuration at Apache Spark 4.0.0 because Apache Spark 3.5.2 is not released yet and we need to wait for one whole release after this.

there are no negative reports,

@dongjoon-hyun thanks for your suggestion, updated the deprecated message, and we can consider removing it at 4.1.0 or later

Since this is irrelevant to CI, I verified manually like the following.
$ bin/spark-shell -c spark.network.remoteReadNioBufferConversion=true
WARNING: Using incubator modules: jdk.incubator.vector
24/05/09 22:54:07 WARN SparkConf: The configuration key 'spark.network.remoteReadNioBufferConversion' has been deprecated as of Spark 3.5.2 and may be removed in the future. Please open a JIRA ticket to report it if you need to use this configuration.
24/05/09 22:54:07 WARN SparkConf: The configuration key 'spark.network.remoteReadNioBufferConversion' has been deprecated as of Spark 3.5.2 and may be removed in the future. Please open a JIRA ticket to report it if you need to use this configuration.
24/05/09 22:54:07 WARN SparkConf: The configuration key 'spark.network.remoteReadNioBufferConversion' has been deprecated as of Spark 3.5.2 and may be removed in the future. Please open a JIRA ticket to report it if you need to use this configuration.
24/05/09 22:54:07 INFO SignalUtils: Registering signal handler for INT
24/05/09 22:54:07 WARN SparkConf: The configuration key 'spark.network.remoteReadNioBufferConversion' has been deprecated as of Spark 3.5.2 and may be removed in the future. Please open a JIRA ticket to report it if you need to use this configuration.
Welcome to
      ____              __
     / __/__  ___ _____/ /__
    _\ \/ _ \/ _ `/ __/  '_/
   /___/ .__/\_,_/_/ /_/\_\   version 4.0.0-SNAPSHOT
      /_/

Using Scala version 2.13.13 (OpenJDK 64-Bit Server VM, Java 21.0.3)
Type in expressions to have them evaluated.
Type :help for more information.
24/05/09 22:54:09 WARN SparkConf: The configuration key 'spark.network.remoteReadNioBufferConversion' has been deprecated as of Spark 3.5.2 and may be removed in the future. Please open a JIRA ticket to report it if you need to use this configuration.
24/05/09 22:54:09 WARN SparkConf: The configuration key 'spark.network.remoteReadNioBufferConversion' has been deprecated as of Spark 3.5.2 and may be removed in the future. Please open a JIRA ticket to report it if you need to use this configuration.

Merged to master/3.5.


Duplicate image URLs due to not considering 'EventTypes'

There is a parameter named 'EventTypes' on the get_markers() method. However, It has not been considered when fetching URLs.
Hence, turtle_ids and URLS have been duplicated.

It has been identified that 'EventType' needs to pass when fetching image URLs from API.
It will be fixed in the next release.


Fix mysql vault setup

Mysql pool was created in MysqlVault.open but it should be created in MysqlVault.setup as it's not doing any connection.
Also, MysqlVault.createDatabase and MysqlVault.dropDatabase need the pool when MysqlVault.open is not called.

We could call pool.getConnection to create one connection.


Use of claim crate causes dependency conflicts

See https://github.com/svartalf/rust-claim/issues/9
This makes it impossible to use rgeometry together with nix, for example, which I want to do.
A possible workaround for users is to patch the (seemingly unmaintained) dependency like this:
[patch.crates-io]
claim = { git = "https://github.com/Turbo87/rust-claim.git", rev = "23892a3" }

Switched to claims. Does that solve the issue?

Yeah, looks like it, thanks!


[MSL] Give the FragDepth builtin a type of float.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Adding Prerequisites methods and test and view methods for Course model

This is based on PR #78
Fixes #79
Work in progress:
I have two options going forward with this, and they are split around the sorting issue:

return all courses filtered or sorted in a specific manner as a QuerySet
return all courses filtered or sorted in a specific manner as a list of Course objects

I'm leaning towards 2, since it allows me to followup and use a backend sorting for filtered results,
as otherwise, I see no way to sort a filtered QuerySet.
Would love to hear you opinions!
The methods in models.py are currently mostly the QuerySet variety,
but there is an example of how I intend to treat lists of Course objects (starting at line 58).

@oriAdler I tried altering the sorting functions and now they work with QuerySet too! So the dilemma was resolved

@VeraTs Glad to hear that, so we can enjoy the best of both worlds


Investigate increasing developer productivity with pull requests environments

I would like to make life easier for existing and new Apache Superset contributors including maintainers by implementing Uffizzi previews.
Disclaimer: I work on Uffizzi
Uffizzi is a Open Source full stack previews engine and is free for Apache Superset. This will provide newcomers with previews of their PRs in the cloud, allowing them iterate faster, without having to first set up a complete development environment themselves.
TODO:

[ ] Intial POC

Closing this as it's more of a feature request rather than a bug. If you're willing to provide the effort to wire this up, we might be interested in learning more about this. We have an ephemeral test environment setup in place, but if there's something better/cheaper/free, perhaps we should talk :) CC @geido


Implement noex::Table::AsyncBulkApply

At the end of this task we will have a AsyncBulkApply API in noex::Table that look (roughly) like this:
  template <
      typename Functor, ...>
  void BulkApply(BulkMutation&& mut, CompletionQueue& cq, Functor&& callback) {
  }

The Functor parameter must meet these requirements:
static_assert(
    google::cloud::internal::is_invocable<
              Functor, CompletionQueue&, std::vector<FailedMutation>&, grpc::Status&>

This operation should implement the same retry loop as the synchronous version, but without blocking for any request.

Another dumb question: why do we want it templated rather than just pass an appropriate std::function? Is it the cost of polymorphism?

 Partly to avoid the cost of polymorphism. Partly to avoid the cost of creating a std::function<> (they are not cheap).  Mostly, because this is how the C++ community writes code these days.
If we were writing this for ourselves, and we understood the typical use cases for how the library is used, then we could make the tradeoff of a simpler API (std::function<>) with a slightly higher performance overhead.  Because this is for applications that we do not control, or know, I tend to give them more flexibility.
Not the strongest argument ever, for I am sticking with it.

This was fixed in #1336.


Need Example

Hello sir.
I found your  great job.
Can you provide us an example?

Hello @phuongphally,
You can find a working example taking a look at Musically repo, particularly in playerActions.
Let me know if you need more help, I can provide something more detailed.
Thanks.

@flarocca  can you provide me an example of latest [https://github.com/flarocca/musically] ?
Because I'm can not get current repository working.
Thanks

+1 on example source code.


Changes not vetted upstream?

Let me preface this by saying that I have been watching and involved in this fork for years, just look at the history.
Now for the question, why have none of the changes made here since @prasad-alatkar left the scene been PR'd and vetted upstream with the mynewt team? There has been some good changes that would benefit the upstream repo as well.

Hello @h2zero ,
There have been indeed quite a few fixes which we have pushed upstream and are part of mynewt nimble too .
Some fixes are specific to IDF primarily related to different chipsets , which we cannot move upstream.
But for fixes which are related to Bluetooth as a technology OR any change that is related to C code ( compilation issues , code style etc) , we do push them upstream .
Few of our changes have not been accepted upstream yet with open PRs
Having said that, we would be glad to push any changes that you can point , that we have missed. Pushin changes upstream works even for us as migration becomes easier for us too going ahead and hence we too are aligned to ensure as many changes as we can push upstream .

My apologies @rahult-github, yes there have been quite a few changes pushed upstream, thank you for that, they just weren't always easy to spot, especially when tired :).
I've been bouncing around between the repos and adjusting my library trying to maintain it in line with this one and some of the changes, especially the porting layer, have been difficult to work around. I guess I was mostly concerned with things like the cpfd addition which broke my library and hasn't been accepted upstream yet.
Anyway sorry to bother, my mistake that I didn't see the PR's upstream.


Scalacp includes inaccessible object constructor

The class info type for objects produced by metacp includes the inaccesible object constructor
$ metap -proto $(metacp $(coursier fetch org.scala-lang:scala-library:2.12.4)) | grep "declarations:.*Sorting.*init"
        declarations: "scala.util.Sorting.`<init>`()."

I expect the constructor to be excluded from the declarations list.
PS. There is no symbol information for scala.util.Sorting.().

I'm trying to solve this one. If I understand correctly the filtering is supposed to happen around the scala.meta.internal.javacp.asm.MethodSignatureVisitor. But does the scala.tools.asm.tree.MethodNode hold any information about the fact that it is accessible or not ? Is it possible to retrieve such information from the byte code ?

@lgirault cool! This issue is in Scalacp, which uses scalap to parse Scala signatures in classfiles. javacp is for extracting signatures from Java signatures.
A good place to start would be

reproduce this bug with an expect test by writing code in https://github.com/scalameta/scalameta/tree/da7b6e8baf146e133402f7428d75e6458414e09b/semanticdb/integration/src/main/scala/example and testing with testsJVM/testOnly *ExpectSuite -- -z metacp.expect
look here https://github.com/scalameta/scalameta/blob/da7b6e8baf146e133402f7428d75e6458414e09b/semanticdb/metacp/src/main/scala/scala/meta/internal/scalacp/Scalacp.scala#L269

I removed the "Good first contribution" since this ticket is probably harder than I originally anticipated.

Cool ! thank you for the tips. Actually I think I'm not clear with the list of cli and what they do. Given the line you gave in the initial description of this issue, I thought that  matacp since its argument is a class jar would extract a semanticdb from class files and that metap would transform the output to the protobuf format … as I'm writing it I realize something is fishy since the semanticdb already is in the protobuf format …
Is there a list of executables in the scalameta project and a description of what they do ?

Duplicate of https://github.com/scalameta/scalameta/issues/1541.


TLS error on connecting to CheckWX

Describe the bug
Logfile warns about a TLS error fetching data from CheckWX
2024/12/16 00:07:30 Error getting weather, using default: Error making request to CheckWX: Get "https://api.checkwx.com/metar/LTDA/decoded": remote error: tls: internal error
Expected behavior
I should be able to fetch data from CheckWX

remote error: tls: internal error suggests a TLS issue on the server side (CheckWX) and is not a client issue (your computer/real weather). This was likely caused by a temporary CheckWX outage of some sort. The only thing you can do client side is enable the fallback providers (if using v2.0.0-beta-1 or later) that will use a different API like aviationweather.gov if CheckWX is inoperative.


Feedback: Delta map not showing up

If you go to the delta tab, the map doesnt show

Pls assign

I have almost solved this, Please assign

@braddf are you ok to assign @akshayw1 if this is still an issue

Hey 👋  I'm not aware that this is still an issue, this was from a while ago so I must have just missed it on the last clean-up, but if you've found something @akshayw1 then do put some screenshots/info on here!


[ADLS Gen2] Filesystem - Create support in storage SDK

https://docs.microsoft.com/en-us/rest/api/storageservices/datalakestoragegen2/filesystem/create
Current blocker:
Setting headers from Request to Http client.
Getting headers from Http client to Response.
Http pipeline support.

Scope:
Validation and Documentation API support is included.

Done.


Drag items from inbox into draft and remove form index v2

what is going on here, I thought we were abandoning all this work and restarting from scratch, close the pr and delete the branch?

we resolved #272 and #357 as a new pr .. kindly merge this in the live env

what is going on here, I thought we were abandoning all this work and restarting from scratch, close the pr and delete the branch?

we resolved #272 and #357 as a new pr .. kindly merge this in the live env

you've got merge conflicts, so its not worth me even looking to review it until thats resolved and the tests pass

What the, why are you trashing the package-lock.json
Please don't do that for what should be really super obvious reasons

The merging process of the PR was getting a conflict from your end, so for QA purpose we removed the file package-lock.json, which we have a backup also with us , we will be pushing that from our end to the specific PR

Er what? No.
Please can you look into how to use git properly to solve this, I'm not even sure why you're changing the packages at all in the pr to begin with. Proposing a PR that removes it is not the answer

Er what? No. Please can you look into how to use git properly to solve this, I'm not even sure why you're changing the packages at all in the pr to begin with. Proposing a PR that removes it is not the answer

Yes Chrisns, definitely  we will resolve that issue, by mistake it was deleted for testing purpose in future we will be more careful about that, and we have deleted it from our branch, if we done anything wrong please guide us.

there is no 'my end' there is the main branch and your feature branch, the conflict is from changes in your feature branch.
I'm afraid I don't have the time to guide you on this much, you'll need to figure it out yourselves or go with the previous advice of abandon this branch and create the work afresh, not just another branch with the same commits.

closing pr, deleting branch, hopefully we can just move on now.


Auto-generate a styled TOC for every guide

Is your feature request related to a problem? Please describe
Unless a user navigates to the developer site through a specific link with a fragment identifier, they have to scroll through the content to find a specific section.
Describe the solution you'd like
I'd like to see every guide have an auto-generated (and styled) table of contents.
Describe why this important to you
This is important to me for a few reasons:

Authors craft descriptive and informative headings for sections, sub-sections, and steps. These headings tell the story of what the guide/article is about and allow the reader to quickly understand where the article covers particular material.
A table of contents allows readers to jump to specific content. If there is no table of contents, it's harder for the reader to make use of these headings because they have to scroll through the content anyway.
This would be consistent with the docs site, which has a table of contents on every content page.

Additional context
The style guide mentions creating tables of contents as an option for particular articles and suggests using a tool like Markdown-TOC for automating that process. I think the "optional" nature of this makes it less likely that authors will create one and more difficult for editors to keep them updated. The table of contents that Markdown-TOC produces is also just a bulleted list, so not the best UX.

On the Docs site, we have each page's table of contents in the right nav. For more complicated docs, this allows for a next TOC: https://docs.newrelic.com/docs/query-your-data/explore-query-data/data-explorer/introduction-data-explorer
It's pretty unobtrusive, which I like, but I'm not sure people always notice it's there.
I absolutely prefer an auto-generated TOC over a manually crafted one (especially given the quantity of docs we're dealing with on docs.newrelic.com.)


Setting refresh rate causes DPI scale reset.

Hi, I'm on Fedora 41 and GNOME 47. When I set the refresh rate in quick settings, the DPI scale gets set to 100% as well (I had 175% previously).

Hi! Thank you for reporting the bug, I will investigate and fix it


update room csv for this year

definitely drop

room_owned_by
changeable
change_reason
placement_team_notes
room_notes
paying_guest
department
placement_verified
connected_to
floor

probably drop

art_room
art_room_type
art_name_placed_by
guest_restriction_notes

updated sample CSVs to remove all columns listed except guest_restriction_notes and linked the branch


Plasma-desktop/latte-dock Error loading OMI file

Describe the bug
On latest nixpkgs-unstable, kde-plasma and latte-dock fail to start. Instead, each widget displays:
Error loading OMI file: file///nix/store/****latte-dock-0.9.7/share/plasma/plamoids/org.kde.latte.containment/contents/ui.qml:1627:9: Type applet AppletItem unavailable.

To Reproduce
Steps to reproduce the behavior:

add nixpkgs-unstable channel
sddm/kde enable
add latte-dock

This is fixed by running latte-dock replace which makes me think it might be related to the KDE startup issue in  #79707 and issue #80550
Screenshots
If applicable, add screenshots to help explain your problem.
Metadata

system: "x86_64-linux"
host os: Linux 5.4.20, NixOS, 20.09pre213994.e2b4abe3c8f (Nightingale)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.3
channels(root): "nixos-20.09pre213994.e2b4abe3c8f"
channels(mjlbach): "home-manager, nixpkgs-20.09pre213040.f77e057cda6"
nixpkgs: /home/mjlbach/.nix-defexpr/channels/nixpkgs

Fixed on master

This doesn't appear to have been fixed by any codebase changes; I've been stuck with an identically broken installation for the past few weeks on nixos-unstable. Interestingly, however, running --replace does not fix my installation, so I'm not 100% sure this is the exact same bug, but it seems a strange coincidence considering ye ol' google provides literally only this issue when searching for an appletItem error.
Are you sure you didn't do something external to a switch --upgrade to fix this?

It's latte-dock --cc (double dash)


don't add artificial newline characters in debug console

VSCode Version: 1.10.2
OS Version: Windows 10

Steps to Reproduce:

Create a file.js with the code below:

process.stdout.write('Working...');
process.stdout.write('done!');

Run it and see the words are not on the same line in DEBUG CONSOLE.

Actual output:
Working...
done!

Expected result:
Working...done!

(the same as in windows command prompt or other terminal emulators)

Fixed in latest. Please try out vscode insiders http://code.visualstudio.com/insiders


remove Json dependency and add Ruby 2.3 in the matrix

Coverage increased to 100% but the build failed.


Fix focus visibility in theme settings

The Problem
When using tab to focus light elements on a dark background (as someone might due using a keyboard to navigate a site), the focus visibility of elements is non-apparent.
The Solution
Override the $focus-ring-color SCSS variable and set it to blue instead of $primary in src/scss/_bootstrap-variables.scss.
Then regenerate the theme CSS using ./scripts/compile_styles.sh.
Additional Information
LINK TO ISSUE: https://github.com/CityOfDetroit/COD-Design-System/issues/244

TO-DO AS OF JULY 23, 2024:

[ ] Look into/Research the .btn:focus-visible variable

--bs-btn-focus-box-shadow is not defined:

Any other help you can give for this PR? I am still not able to get the var(--bs-btn-focus-box-shadow) to show as defined @maxatdetroit

NOTES ABOUT btn-focus-box-shadow

$btn-focus-box-shadow: This variable provides the box shadow of the button on focus. By default, it is 0.
It can be customized this variable by overriding it in the custom SCSS file

@sreidthomas
To set the box shadow for buttons on the :focus-visible state in Bootstrap 5.3, you should set the --bs-btn-focus-box-shadow CSS variable. Here's how you can properly set this property:
In src/scss/_host.scss, add the following under both the light and dark mode :host customization blocks:
  --bs-btn-focus-box-shadow: 0 0 0 0.25rem blue;

@sreidthomas
To set the box shadow for buttons on the :focus-visible state in Bootstrap 5.3, you should set the --bs-btn-focus-box-shadow CSS variable. Here's how you can properly set this property:
In src/scss/_host.scss, add the following under both the light and dark mode :host customization blocks:
  --bs-btn-focus-box-shadow: 0 0 0 0.25rem blue;

Note: This is how you would accomplish what you suggested:

This variable can be customized by overriding it in the custom SCSS file

So I'm testing this solution, and when that variable gets added, isn't the dropdown shadow supposed to be blue @maxatdetroit?

So I'm testing this solution, and when that variable gets added, isn't the dropdown shadow supposed to be blue @maxatdetroit?

Yes, it should. If it isn't turning blue, make sure you've regenerated the CSS from the SCSS using your script: ./scripts/compile_styles.sh after updating any scss files.

Also, how did you know it should be the host.scss file?

This is the bootstrap documentation that suggests modifying the :root style selector (which impacts all elements on the page in the light DOM), however since we're using the shadow DOM, we actually need to use the :host style selector style selector to target the shadow DOM styles specifically.
I learned this while reading all the code and reading those links I just shared a while back while getting up to speed on custom elements, bootstrap, and shadow DOM styling.

So I'm testing this solution, and when that variable gets added, isn't the dropdown shadow supposed to be blue @maxatdetroit?

Yes, it should. If it isn't turning blue, make sure you've regenerated the CSS from the SCSS using your script: ./scripts/compile_styles.sh after updating any scss files.

Also, how did you know it should be the host.scss file?

This is the bootstrap documentation that suggests modifying the :root style selector to set bootstrap variables (which impacts all elements on the page in the light DOM), however since we're using the shadow DOM, we actually need to use the :host style selector style selector to target the shadow DOM styles specifically.
I learned this while reading all the code and reading those links I just shared a while back while getting up to speed on custom elements, bootstrap, and shadow DOM styling.

I did run the ./scripts/compile_styles.sh command but its still not blue. The links inside are black @maxatdetroit:

Even after adding it to that file, it still says undefined. Is there anything else I should check @maxatdetroit?

Even after adding it to that file, it still says undefined. Is there anything else I should check @maxatdetroit?

When we update something in the source code, then compile it, and serve the minified/aggregated version in the browser, we know that the breakdown is somewhere in that sequence of events:

Updating the source code. Sounds like you've already checked this. ✅
Compiling the source code. E.g. from SCSS into CSS. Have you checked the compiled CSS to make sure the variable is set in the style sheets there? ❓
Making sure the right version of the file is served to the browser. E.g. downloading the CSS directly from the browser so you can get an exact copy of what the browser is using when applying the styles. Have you tried this? We'd want to check that the variable is set somewhere in the stylesheets. ❓
Making sure there aren't any styling conflicts. E.g. maybe something later in the stylesheets is overwriting the variable and unsetting it? Or maybe the :host selector isn't the right selector due to specificity or something else? Have you checked this? We'd want to use the developer inspector to see where else the variable may be being set or looking at the computed styles to see if it's not specific enough or overridden by a conflicting stylesheet. ❓

I would check out the remaining suggestions above with a ❓ next to them. Let me know if you want some more details about how to go about checking those. My notes are sort of brief above.

TO DO AS OF JULY 29, 2024:

[ ] Update Source Code
[ ] Check to make sure the variable is set in the compile CSS file
[ ] Make sure the right version of the CSS file is being served to the browser
[ ] Make sure there are no conflicting styles in the stylesheet

Making sure the right version of the file is served to the browser. E.g. downloading the CSS directly from the browser so you can get an exact copy of what the browser is using when applying the styles. Have you tried this? We'd want to check that the variable is set somewhere in the stylesheets. ❓

How do I download the CSS file from the browser @maxatdetroit?

Making sure the right version of the file is served to the browser. E.g. downloading the CSS directly from the browser so you can get an exact copy of what the browser is using when applying the styles. Have you tried this? We'd want to check that the variable is set somewhere in the stylesheets. ❓

How do I download the CSS file from the browser @maxatdetroit?

I followed steps from this: https://webdeveloper.com/bounties/how-to-download-css-file-from-a-website/
But I don't see the relevant CSS files

Here's a way to find the relevant stylesheets using a combination of the inspector element picker and the stylesheet tab:
https://github.com/user-attachments/assets/4fd4b5cc-40f1-4fca-971e-c6466f3718f3
Our stylesheets are a little different than most websites because they're all inline stylesheets in the design system (as opposed to external stylesheets with helpful names).

We'd want to use the developer inspector to see where else the variable may be being set or looking at the computed styles to see if it's not specific enough or overridden by a conflicting stylesheet.

Is there an easy way to find where else the variable is being set @maxatdetroit?

Regarding Different Styles in Different Browsers
That is expected.
Default browser stylesheets, also known as user agent stylesheets, are a set of CSS rules that browsers apply to HTML elements by default. These stylesheets provide basic styling and layout for web pages even when no custom CSS is specified by the web developer.
The purpose of default stylesheets is to ensure that HTML documents have a consistent, readable appearance out of the box. They define things like:

Basic text formatting (font sizes, line heights, etc.)
Margins and paddings for various elements
Default colors (e.g., blue for links)
Basic layout rules (e.g., block vs. inline display)

However, these default styles can vary between browsers, which is one reason why web developers often use CSS reset or normalization techniques to create a more consistent cross-browser starting point.
Now, regarding the :focus-visible pseudo-class and why you might see different styles across browsers:

Browser Implementation: The :focus-visible pseudo-class is a relatively new feature in CSS. It was introduced to provide a way to show focus indicators only when they're needed (e.g., when using keyboard navigation, but not when clicking with a mouse). Different browsers may implement this feature slightly differently or may be at different stages of implementation.

Default Styles: Just like with other elements, browsers may have different default styles for the :focus-visible state. Some may apply a more prominent outline, while others might use a subtler indication.

Heuristics: Browsers use different heuristics to determine when to apply :focus-visible styles. These heuristics can vary, leading to inconsistent behavior across browsers.

Browser Versions: Older versions of browsers might not support :focus-visible at all, falling back to the standard :focus behavior or ignoring it entirely.

Operating System Influence: The appearance of focus indicators can also be influenced by operating system settings, which can lead to differences even between the same browser on different OS platforms.

Regarding How to Find Where a CSS Variable is Set
Try using the 'Computed' tab of the style inspector in the dev tools. You can use the filter to filter down to a specific variable name, then expand the notes for that variable and it will tell you all the cascading styles and how they're overriding each other. Here's a video:
https://github.com/user-attachments/assets/eeb77691-b29c-46c6-ba9d-6b45f915e9b4

Since Chrome and Firefox are behaving differently and that's normal, which browser should I focus on as far as this PR goes @maxatdetroit?

@sreidthomas , we should make sure the fix works on both browsers. Ideally, they have the same style behavior too (e.g. both are blue) but as long as there's a focus indicator with sufficient contrast on each browser, then it's not strictly necessary that they're the same exact styles.

NEW NOTES AND FINDINGS AS OF JULY 30, 2024:

The order of the rules (rules lower in the list override those above)
Any crossed-out properties (indicating they've been overridden)

Okay, so I think I found the issue. The .btn class has the btn-focus-box-shadow property set. When I remove it in Developer Tools, the blue highlight around dropdown shows @maxatdetroit, I'm just not sure what file to make the change in. I tried adding this line to _host.scss but it did not work:
.btn {     --bs-btn-focus-box-shadow: none;   }
I shared the video via teams because it won't let me share here.

@sreidthomas very nice investigative work!
Based on the video, it looks to me like the --bs-btn-focus-shadow-rgb is not set in the --bs-btn-focus-box-shadow rule where it's referenced.

Any crossed-out properties (indicating they've been overridden)

This can also mean they've never been set in the first place.
Have you tried setting the --bs-btn-focus-shadow-rgb in the _host.scss file?

--bs-btn-focus-shadow-rgb

I set it to blue @maxatdetroit?

I think it expects an RGB format. blue is a named color with a HEX (i.e. base 16) value of #0000FF. That converts to rgb(0,0,255) in RGB format.

I set that variable in the file but it didn't work. It's only when I remove the whole thing like in the video that the blue highlight works @maxatdetroit

but it didn't work

Would you try using the debugging tools/workflow you've been using to figure out why?

WHERE I BECAME BLOCKED ON THIS ISSUE:
Found how the property --bs-btn-focus-shadow-rgb is being set twice but not sure where/how to fix the issue.
Currently right now, inside the .btn class if you disable the property, the blue shadow box shows but when it is enabled, the same property is being used but with lesser values so the blue shadow box is there but not as visible.
Video was posted on teams demonstrating this

After the most recent commits, this is now working for all elements, including buttons.
@sreidthomas for future reference, the problem with the buttons was that --bs-btn-focus-shadow-rgb was being set in the bootstrap stylesheets after the :host ruleset, effectively overriding whatever we had put in the :host stylesheet.
How did I find that? With the help of the 'computed styles' pane in dev tools, I noticed all the styles were being set in the main bootstrap style sheet. So then I read through the compiled bootstrap styles in themed-bootstrap.css and noticed the variable is set multiple times.
How did I fix it? I moved the variable definition into _detroitmi-style-guide.scss which gets imported at the very end of the main bootstrap style sheet, and thus, overrides the previous variable definitions. We also could have defined the variable in button.css but if we did it there, then it wouldn't take effect for buttons unless they're cod-buttons and since others use our bootstrap stylesheets with basic button elements (not cod-buttons), we need to define it inside the bootstrap style sheets instead of the component style sheet to make sure it takes effect everywhere.

Latest test:
https://github.com/user-attachments/assets/2d326fc9-0453-4fef-b9f8-912c2918aa83

Latest test:
Screen.Recording.2024-07-31.at.3.01.23.PM.mov

The video isn't showing. But I think I understand how you fixed this.


bug: APIs bound through Permission Claims not available when using identityHash

Describe the bug
When specifying another APIExport's identityHash in a permission claim in an APIExport / APIBinding, the APIBinding will create successfully:
E.g.:
apiVersion: apis.kcp.dev/v1alpha1
kind: APIBinding
metadata:
  name: has-binding
spec:
  reference:
    workspace:
      path: root:has
      exportName: has
  acceptedPermissionClaims:
      - group: ""
        resource: "secrets"
        verbs: ["get", "list", "watch"]
      - group: ""
        resource: "configmaps"
        verbs: ["get", "list", "watch"]
      - identityHash: 5d8e0fa5f9a10dbff668aad2cd63777e061c837e3f0a2f1adda15f7acae9fd53
        group: "appstudio.redhat.com"
        resource: "environments"
      - identityHash: 5d8e0fa5f9a10dbff668aad2cd63777e061c837e3f0a2f1adda15f7acae9fd53
        group: "appstudio.redhat.com"
        resource: "applicationsnapshotenvironmentbindings"

with the following status conditions:
  conditions:
  - lastTransitionTime: "2022-08-26T18:24:05Z"
    status: "True"
    type: Ready
  - lastTransitionTime: "2022-08-26T18:24:05Z"
    status: "True"
    type: APIExportValid
  - lastTransitionTime: "2022-08-26T18:24:05Z"
    status: "True"
    type: BindingUpToDate
  - lastTransitionTime: "2022-08-26T18:24:05Z"
    status: "True"
    type: InitialBindingCompleted
  - lastTransitionTime: "2022-08-26T18:24:05Z"
    status: "True"
    type: PermissionClaimsApplied
  - lastTransitionTime: "2022-08-26T18:24:05Z"
    status: "True"
    type: PermissionClaimsValid

But if I run kubectl get environments the command fails, saying error: the server doesn't have a resource type "environments"
Steps To Reproduce
I've created a reporducer here: https://github.com/johnmcollier/kcp-permissionclaims-reproducer
Expected Behaviour
The APIs specified in the permission claims are available.
Additional Context
No response

/assign

@johnmcollier please try adding another APIBinding in your workspace, to the gitops APIExport.
/priority awaiting-more-evidence

@ncdc Yes, if I add another APIBinding the APIs in the gitops APIExport become available. But I thought the permission claims meant that wasn't needed?

Is this #1183 ?

@johnmcollier both bindings are required

This is not a bug

Closing

@ncdc So a controller running in my APIExport's virtual workspace should have the API(s) available there without a separate APIBinding?
If so, my controller still fails on start up since those APIs aren't available in the virtual workspace.

Now that is #1183 😄


Support for default values for optional config

Something along the lines of
config.postgres_port or 5432
config.postgres_host or 'localhost'

I guess I could still do config.values.get('pg_port', 5432), but a simpler syntax would be great.

Thanks for your feedback! I have some mixed feelings about this, because supporting something like:
config.postgres_port or 5432

presents two issues:

None could still be a legitimate value for certain configuration settings
if a user forgets to set at all a certain setting, (and doesn't use the or ... part), the code fails silently

As a side note, even Łukasz Langa recently commented on a similar subject, stating that it's more Pythonic to throw exception if an attribute is missing, rather than returning T | None (https://twitter.com/llanga/status/1522922855335436289).
Of course there are exceptions even in the standard library: Regex match returns a Match object or None - so I take this with a grain of salt.

The original philosophy of this library is to enable composition (something I like in .NET Core and wanted to have also for my Python projects), like:

first read default values from example-settings.yaml / json
then read values from example-test-settings.yaml / json (for the test environment - using an env variable to define the environment)
then - for example - read values from environmental variables
then read values from an Azure Key Vault / AWS Key Management Service

In other words, if you wish to have defaults, the idea of this library is that you would configure them either in memory:
    builder = ConfigurationBuilder()

    builder.add_map({
        "postgres_port ": 5432,
        "postgres_host": "localhost"
    })

    builder.add_source(YAMLFile("./yaml_example_01.yaml"))

    config = builder.build()

or, more likely, in a base app settings file:
{
    "postgres_port": 5432,
    "postgres_host": "localhost"
}

I know by practice that this has the good quality that applications settings are more understandable: it's sufficient to open this base settings file to get an overview of what settings are used by the app, and what are their defaults.
In conclusion, as I am thinking while writing these sentences, I don't think that returning default None would be the best option in this case.

But I like your recommendation, we could add a method similar to what dict offers:
configuration.get("key", value)

What about something like this? https://github.com/Neoteroi/essentials-configuration/commit/368e9a4e0f1a331bff78cbac7f8e2870094dd094

Yes, that works. Thanks


Ros2 improve release process

@KmakD Removed triggering workflow.
This PR needs to be merged as well
https://github.com/husarion/panther-docker/pull/72


All @cycle/dom DOM VNode stream errors silently fail

Code to reproduce the issue:
Any error in the DOM vnode stream
Expected behavior:
It reports an error
Actual behavior:
It just breaks, with no waring
This is because of the rootElement$.addListener() line in makeDomDriver() that ignores all errors. Is this expected behaviour? I just spent over an hour trying to find out why my app silently failed, and froze without any error.
If a had console.error(error) to the observer's error handler in makeDOMDriver() it reports errors.

My bad, it's errors in vdom hooks. All create a new bug.


Replace to-be deprecated popen4 with shellout

Replace deprecated popen4 with shellout

@kaustubh-d can you take a shot at rebasing this off of master?

cc @chef/client-core @chef/client-engineers can you review the code as it is now?

overall a pretty straightforward conversion, :+1: We should look at what ShellOut#inspect does, since that will be in the error messages. If it's something ugly, we may want to use ShellOut#format_for_exception instead. Side note, we could alternatively just use the shell_out! form, but that would change the exception type, unless we went all the way to something like:
def run_some_pkg_command
  shell_out!("package command", returns: [0,1])
rescue ShellCommandFailed => e
  raise Chef::Exceptions::Package, e.message
end

But I think the improvement of just switching to shellout from popen4 is enough to merge without requiring the extra changes.

Overall this looks pretty good :+1:

Marking this as ready-to-merge.  Still needs a rebase and bonus points to looking at Dan's comments, but its a Chef Software, Inc PR and we should get it across the finish line for 12.1.0 even if @kaustubh-d is AFK right now.

sure, working to rebase

Rebased into #2924. It still looks a bit incomplete, like the replacement of Chef::Mixin::Command#handle_command_failures, and I think there's some places where shell_out needs to be shell_out!. I'm feeling like we keep working on that new branch for 12.2.0. Please post any comments on the new PR.


Retain the previous selected subscriptions after re-signing in with same account

Repro Steps:

Sign in to Azure.
Select one subscription.
Sign out, then sign in again with the same account.

Expect:
All subscriptions list.
Actual:
Retain the previous selected subscription.

This is by-design


Make bevy_ecs compile without the bevy_reflect feature

Objective

Make bevy_ecs compile with default-features=false

I was trying to be a good citizen with my library, not bringing in unneeded features
Not that anyone has used bevy_ecs without the bevy_reflect feature, but there has to be a first 👀

Solution

Conditionally compile out the parts which stop compilation.

Superceded by #3097

ho sorry I forgot this one was opened


State that the data cannot be symmetric if it's empty

A RuntimeWarning was emitted by numpy with the following snippet. It was trying to calculate np.nanquantile on an empty series in this case.
import pandas as pd
import hvplot.pandas

df = pd.DataFrame({'x': [], 'y': [], 'color': []})
df.hvplot(kind='scatter', x='x', y='y', color='color')

The only thing I'm unsure of is whether the data variable always has a size parameter. In other words, what type of object could data possibly be in this function?

Coverage decreased (-0.01%) to 72.427% when pulling d50a52cac132c5ab00da8add68e1a4d0f891fb2e on handle_empty_color_column into 17ce0cc18a0393ae82d3a9f6f11b3bc4cad29b30 on master.


FINOS : "Zeno Lee" user detail missing under "Cloud9 Technologies LLC" organization on Finos project

Steps :

Navigate to Signature tab : https://projectadmin.lfx.linuxfoundation.org/project/a092M00001JwBlEQAV/tools/easycla/overview/5564f18f-3252-40fc-a65b-2eef1ad45fcd/signatures
Open Signed CCLAs tab
Search Organization "Cloud9 Technologies LLC"
Click on View All Approved Contributors

Actual Results : "Zeno Lee" user detail is missing

Kindly verify

Missing User is added, So making issue status QA-Done


feat: add outputs to get the preview-url and preview-name

Description
Basically I want to get the preview-url generated by netlify to run some e2e tests and Lighthouse suite to that URL.
Vercel deployment action already does this.

@South-Paw this PR is opened since July....

@South-Paw this PR is opened since July....

I've ended up using mine which is the fork of South Paw.
You can see here https://github.com/verdaccio/verdaccio/blob/master/.github/workflows/website.yml#L92

@T-Philipp Yeah, open source software is like that, sometimes people miss things, sometimes they are busy, sometimes they've chosen to not to use their spare time on free projects and sometimes its just a combination of everything. I have PRs and issues that were raised years ago and never even got a maintainers response - that's just how free software is sometimes.
Now that being said, @semoal I apologize that it hasn't been looked at, I know how frustrating it is but its just one of those things that got lost in my notifications and I haven't been spending my spare time keeping track of.
I'll get it in now.
However, as mentioned in #21 deploy has been removed in a newer version of Netlify's js client and I have been planning to write up a migration path for people who use this action so they can just change over to using the Netlify CLI + a deployments action to replace this action.

perfect.
Thanks a lot both of you 😁👍

@T-Philipp @semoal v1.2.1 is on marketplace now


Add a status to the protocol run record in the empty state

current state:
we display the "last run protocol" on the upload page's empty state regardless of whether or not the run is completed. This could be confusing to users if they canceled it or closed it before running.
Future state:

Add a column to the last run section to include the run's status:
Not started: when this protocol was loaded on to the robot, it was closed before the user ran it
Canceled: when this protocol was loaded on to the robot, it was canceled before the run completed
Complete: when this protocol was loaded on to the robot, it was closed after the protocol run completed

Update the copy of the "how rerunning a protocol works" to include what the statuses mean (copy in figma)

Figma link:
https://www.figma.com/file/JCQNOTTK9tTmYPhRqeOfSj/Milestone-0%3A-Protocol-Upload-Revamp?node-id=9788%3A180017

closing #8954 in favor of this ticket.


Add version: Microsoft.bitsmanager version 1.12

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/115149&drop=dogfoodAlpha

@wingetbot run

Duplicate of #17016


Missing 2 landmarks in between standard and attention FaceMesh model.

Hello!
As I understand from documentation, AttentionModule produces 478 landmars face model, adding Irises to the standard one.
mp_face_mesh.FACEMESH_NUM_LANDMARKS = 468
mp_face_mesh.FACEMESH_NUM_LANDMARKS_WITH_IRISES = 478
However, mp_face_mesh.FACEMESH_IRISES=8.
My  question is what are those two missing landmarks? I have 468 landmarks canonical face model and I want to solve PnP.
I could not find 478 landmarks canonical face model. If You can provide me with such one, it will be great!
Best,
Jan

Hi @janglinko-dac, Could you please elaborate your query with complete details. Thank you!

Hello @janglinko-dac,
We are upgrading the MediaPipe Legacy Solutions to new MediaPipe solutions However, the libraries, documentation, and source code for all the MediapPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions. Thank you