fix kernel and server labels when not connected

For #9933

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #9965 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #9965      +/-   ##
==========================================
- Coverage   61.18%   61.18%   -0.01%
==========================================
  Files         564      564
  Lines       30077    30079       +2
  Branches     4550     4550
==========================================
  Hits        18404    18404
- Misses      10642    10644       +2
  Partials     1031     1031

Impacted Files
Coverage Δ

src/client/common/utils/localize.ts
95.12% <100%> (+0.01%)
:arrow_up:

src/client/common/utils/platform.ts
64.7% <0%> (-5.89%)
:arrow_down:

...datascience/jupyter/liveshare/hostJupyterServer.ts
11% <0%> (-0.12%)
:arrow_down:

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 ad70988...0c6a1f3. Read the comment docs.


CSRF Support

Since this topic covers security and Spring Security has quite a bit of out of the box security features, I am surprised to see CSRF specially disabled in SecSecurityConfig.java.
I have tried to disable it by changing csrf().disable() to just csrf(), but when testing the application goes consistently states session invalid after trying to login and then I finally received the following error:
There was an unexpected error (type=Forbidden, status=403). Invalid CSRF Token 'null' was found on the request parameter '_csrf' or header 'X-CSRF-TOKEN'.
What is your reason for turning off this feature? Have you tried to implement it and could update this application to support it? Maybe a future topic for your site?
Per spring.io:

When should you use CSRF protection? Our recommendation is to use CSRF protection for any request that could be processed by a browser by normal users. If you are only creating a service that is used by non-browser clients, you will likely want to disable CSRF protection.

@tedwards-rogue That is a good point.
The reason it's disable is that this particular application is following a series of articles. And, in the articles we didn't yet reach CSRF protection.
Hope that makes sense.
Cheers,
Eugen.

I appreciate the update. The content of the articles and support code is well written. I have some other tasks first, but I will try to implement CSRF in the near future. Definitely using this application as an example.

Sounds good Ted. If you'd like to pick this one up yourself, I'd be happy to publish it.
Of course there's a process until that happens.
Cheers,
Eugen.


E2: Comment(s) after #include directive results in invalid E2

Hello,
Just yet another day with Expression 2 scripting..
It is worth noting that this issue exists even in Workshop version, and still exist at this point (on master branch). @TomyLobo, thanks for confirming that this issue exists in Workshop version.
For example:
#include "path/to/lib/file"  # Include some E2 lib.

Screenshot:

Output:

Unknown character found (.) at line 1, char 51

By removing `.` from the comment:
#include "path/to/lib/file"  # Include some E2 lib

Screenshot:

Output:

Left parenthesis (() must appear after function name, variables must start with uppercase letter, at line 1, char 30

By removing the comment, like this:
#include "path/to/lib/file"

Screenshot:

Output:

Validation successful

This is in fact a bug, not a feature. C allows placing comments on #includes

@oldmud0

This is in fact a bug, not a feature.

Of course this is a bug.

C allows placing comments on #includes

You mean after include statement. Sure it does, why would not it. (< is not a question)

C also allows things like:
/* now */ # /* we */ include /* all of */ <dlfcn.h> // so we can use its functions

This is because comments are replaced with spaces before any other tokenization or preprocessor directive parsing.

@AbigailBuccaneer E2's preprocessor is supposed to get rid of comments as well, it just missed these because include starts with a #

Actually a line with an include should not include anything else.
First of all we need a proper error message here and then we can sit down and look at whether it makes sense to allow anything else there.

Why wouldn't it make sense to allow comments everywhere? (except inside strings)

like i said, that discussion is premature

Include is a directive and directives are designed to input the whole line. #ifdef or whatever suffers the same issue.

Will need a new issue asking to allow directives to not consume the whole line following # if this is to change.

Or, just provide a correct error message and stick with the current design.
Simpler?

The error message is already correct

The error message is already correct

Left parenthesis (() must appear after function name, variables must start with uppercase letter

How is that correct?!

Because that's what happens when you try tokenizing invalid input.

#include acts like a token so you could probably do something like #include getInclude()

#include acts like a token so you could probably do something like #include getInclude()

Nooot.
@persist Include:string
function string getInclude() { return "path/to/lib" }
Include = getInclude()
#include getInclude()
#include Include

The same error on line 4th and 5th:

include path (string) expected after include

#include "path/to/" + "lib"

Identity operator (+) must not be succeeded by whitespace at line 1, char 21

Guess not then. Go reverse engineer it yourself. https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_expression2/base/compiler.lua#L916
I'm done messing with it.

Probably need code from https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_expression2/base/compiler.lua#L815
if you want it to do that


Short solution needed: "How to add noise" (python-pillow)

Please help us write most modern and shortest code solution for this issue:
How to add noise (technology: python-pillow)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to add noise


[Roadmap] Q1S2

Priorities for Q1 Sprint 2
Virtual Cards
We would like to prototype how we could give Cards to Collectives to spend their budget. We’re considering using the Privacy API because we’re already using the service internally and it seems possible to automatize. Some challenges are:
issuing credit cards from our API, and our interface retrieve the list of transactions from Privacy to invite collective admins to fill expenses, because we need their paperwork enforcing limits based on the collective budget
Engineers: @kewitz + @znarf
Kick Off Notes: Google Doc
Project board: [Board] (https://github.com/opencollective/opencollective/projects/49)
Status:
Implemented features needed for project.
Ready for internal use
Design: Done
Pending:
User interface to create, apply for and manage cards.
External Test
PayPal Recurring
PayPal is not handled for recurring contributions and that is frustrating people and preventing some to contribute. We anticipate it will be more complex and different from our Stripe setup but we still want it. We would like to see how a basic implementation looks like and identify blockers.
Engineers: @Betree + @kewitz
Kick Off Notes: Doc
Project board: Board
Project issue: https://github.com/opencollective/opencollective/issues/3986
Status:
Feature ready for public release but still want to deploy improvements in the interface (like subscriptions page)
Ready for Beta Test.
Next Steps: Beta Test & iterate
Embed Contribution Flow
Engineers: @Betree
Currently in beta-testing
Bug fixes as needed based on feedback.
Status: ongoing
Need to onboard more collectives
Next steps: document, release it and close it.
Close project.
Open Stocks
Engineers: @znarf
Project Owner @piamancini
Deliverable: website with a checkout flow for stock donations for OSC
Status: in development
Expense Spam button
Engineer @znarf
Project Owner: @piamancini @alanna
Deliverable: user facing flag to mark an expense as spam
Status: Done
Pending: documentation
Whitelabeling
SCA apply flow project
Design: done
Engineer: @znarf + contributor
Deliverable: Pricing & proposal
Project Owner: @piamancini
Status: Done
Pending: Follow up on reply.
Updates
1 day: fixing issues on Updates
Engineer @Betree
Done
Fiscal Sponsorship Static Page
Design: @Memo-Es
Owner: @piamancini
Project issue: https://github.com/opencollective/opencollective/issues/4118
Status: Design and Content done
Pending: Illustrations
Accessibility issues
Design: @Memo-Es
Project issue: https://github.com/opencollective/opencollective/issues/4045
Status: In implementation
Pending: global navigation + standards documentation
Changelog page
Design: @Memo-Es
Dev Support: @znarf
Status: scoped, but delayed because of Virtual Cards.
Dev Website
Goal: review good dev websites for proposal
Design: @Memo-Es
Dev support: @znarf
Status: Pending

This looks good! Do we also want to carve out some time for the team to squash a bunch of small bugs and issues? Such as a three day sprint or something.
I notice many of these do not have their own project issues. I think the project issue template has been working well for us and we should stick with it, and link the project issues here.

Review Document: https://docs.google.com/document/d/1mFArzqaMeAY68G4dXpgoLScaM7zMfzuid9k8EwFI2eU/edit#


Refactor management commands to use common base classes

We have some base classes that a few of our management commands use. The others (with a couple of exceptions, probably) should be rewritten to use these.

Looks good to merge after addressing the last two comments. Thanks!

Worth mentioning that...

VumiApis are now only created in BaseGoCommand
VumiUserApis are still created in quite a few commands, but there are far fewer than before

:+1:
Great work, thanks.


Live Dash DRM on webOS buffering

Have you read the Tutorials?
yes
Have you read the FAQ and checked for duplicate open issues?
ues
What version of Shaka Player are you using?
yes
Please ask your question
What can be a reason for shaka player to work fine with live dash drm on chrome but same code on webOS will get into buffering / playing state constantly
we tried buffering goal config which did not help
we saw that disabling audio helped for that issue but it is not a solution
please advise.
thanks!

Can you test with v4.3.4 and main branch? Thanks!


sctp transport implementation

Hi Trustin,
Followings are available with this pull request,

sctp client & server transport implementations
multi streaming support
sctp socket options
sample echo application

I have tested with my linux box with lksctp 1.0.11, it worked fine :)
(Most of the classes are copied and modified form nio socket , not nio datagram. Since sctp uses socket, it was easy to modify nio socket classes)
The current code has faire amount of duplicates, looking forward for your suggestion to improve it.
Jestan.

Seriously, this is awesome.  Would you join the team and continue your work on the SCTP transport?  Merging this pull request without letting you contribute directly doesn't make much sense in my opinion. :-)

Nice work...
Anyway I would prefer to move all the "implementations" out of the core library. What about splitting netty into multiple jars. Something like:
netty-core
netty-scpt
netty-http.
...
...
This will allow us to keep the core slim in the future ;)
WDYT ?

Welcome to the team, and please feel free to work in the upstream as long as you don't break anything. :-)

thanks :) . I will start merging with upstream soon.

Is the com.sun.nio.* package only in java 7 ? As I have compile errors with java 6

Yes its only available in Java 7, shall we move this to a branch?

Yes I think this will make sense. We should move it back once we have multi-module support as we could just require java7 for it then.


turbo-tasks this calls

Description
Make this argument in turbo-tasks function calls explicit
Testing Instructions

#8737
#8736
#8735  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sokra and the rest of your teammates on  Graphite

Merge activity

Jul 17, 5:07 PM EDT: @sokra merged this pull request with Graphite.


Replace documents in array using positional $ operator

With positional $ operator, Mongo allows not only to update but to replace entirely an element of an array. Morphia doesn't seem to support it.
For example if I have a document like this:
{
    "_id" : 1,
    "grades" : [
        { "value" : 80, "rank" : 12 },
        { "value" : 85, "rank" : 5 }
    ]
}

Suppose I want to change the first subdocument in the array "grades". The following update works in Morphia:
datastore.update(
                datastore.createQuery(Students.class)
                        .filter("_id", 1)
                        .filter("grades.value", 80) ,
                datastore.createUpdateOperations(Students.class)
                        .set("grades.$.value", 82) // change "value" only.
        );

But the following should also work:
Grade grade = new Grade(82, 10);
datastore.update(
                datastore.createQuery(Students.class)
                        .filter("_id", 1)
                        .filter("grades.value", 80) ,
                datastore.createUpdateOperations(Students.class)
                        .set("grades.$", grade) // replace completely the matching subdocument
        );

But it fails with the following error:
org.mongodb.morphia.query.ValidationException: The given path is invalid: grades.$
	at org.mongodb.morphia.internal.PathTarget.resolve(PathTarget.java:110)
	at org.mongodb.morphia.internal.PathTarget.getTarget(PathTarget.java:92)
	at org.mongodb.morphia.query.UpdateOpsImpl.add(UpdateOpsImpl.java:279)
	at org.mongodb.morphia.query.UpdateOpsImpl.set(UpdateOpsImpl.java:223)
 ..............

For the record, Mongo supports this very well:
db.students.update(
   { _id: 1, "grades.value": 80 },
   { $set: { "grades.$" : { value: 82, rank: 10} } }
)

Looks like simultaneously hit #1127 along with @ckarmann.  I'll take a look at his PR and then your test case but I have a feeling they're the same issue.

confirmed.  fixed by #1127


Youtube videos playing on background when device is inactive. Which cause playstore to reject app

Environment
Flutter version: 1.17.0
Plugin version:  flutter_inappwebview 3.3.0
Android version: 9.0
iOS version:---
Xcode version:  ---
Device information: techno
Description
Expected behavior:
Current behavior:
Steps to reproduce

This
Than that
Then

Images
Stacktrace/Logcat

For Android, you can use InAppWebViewController.android.pause and InAppWebViewController.android.resume to pause and resume WebView.
You should implement WidgetsBindingObserver in your Widget and check AppLifecycleState state through didChangeAppLifecycleState() method.
If you need to pause/resume also JavaScript execution you can use InAppWebViewController.pauseTimers/InAppWebViewController.resumeTimers.
Check the API docs if you need more detail.
Howevert, here is an example with a YouTube video:
import 'dart:async';
import 'dart:io';
import 'package:flutter/material.dart';
import 'package:flutter_inappwebview/flutter_inappwebview.dart';

Future main() async {
  WidgetsFlutterBinding.ensureInitialized();
  runApp(MyApp());
}

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => new _MyAppState();
}

class _MyAppState extends State<MyApp> with WidgetsBindingObserver {
  InAppWebViewController webView;

  @override
  void initState() {
    WidgetsBinding.instance.addObserver(this);
    super.initState();
  }

  @override
  void dispose() {
    WidgetsBinding.instance.removeObserver(this);
    super.dispose();
  }

  @override
  void didChangeAppLifecycleState(AppLifecycleState state) {
    print('state = $state');
    if (webView != null) {
      if (state == AppLifecycleState.paused) {
        webView.pauseTimers();
        if (Platform.isAndroid) {
          webView.android.pause();
        }
      } else {
        webView.resumeTimers();
        if (Platform.isAndroid) {
          webView.android.resume();
        }
      }
    }
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(
          title: const Text('InAppWebView Example'),
        ),
        body: Container(
            child: Column(children: <Widget>[
          Expanded(
              child: InAppWebView(
            initialUrl: "https://www.youtube.com/watch?v=NfNdXgJZfFo",
            initialHeaders: {},
            initialOptions: InAppWebViewGroupOptions(
              crossPlatform: InAppWebViewOptions(
                debuggingEnabled: true,
              ),
            ),
            onWebViewCreated: (InAppWebViewController controller) {
              webView = controller;
            },
            onLoadStart: (InAppWebViewController controller, String url) {},
            onLoadStop: (InAppWebViewController controller, String url) {},
          ))
        ])),
      ),
    );
  }
}

inAppWebview flickering anyway to go around that?

What do you mean? Can you make an example/screen record?

I mean it flickers when keyboard show  ups any way to go around that

Unfortunately no. This is related to how Flutter embeds Android native views inside its widget tree.
I know that somebody is working on another way to embed Android native views.
I hope it will be better than what we have right now!

Ok awesome thank very helpfull

For Android, you can use InAppWebViewController.android.pause and InAppWebViewController.android.resume to pause and resume WebView.
You should implement WidgetsBindingObserver in your Widget and check AppLifecycleState state through didChangeAppLifecycleState() method.
If you need to pause/resume also JavaScript execution you can use InAppWebViewController.pauseTimers/InAppWebViewController.resumeTimers.
Check the API docs if you need more detail.
Howevert, here is an example with a YouTube video:
import 'dart:async';
import 'dart:io';
import 'package:flutter/material.dart';
import 'package:flutter_inappwebview/flutter_inappwebview.dart';

Future main() async {
  WidgetsFlutterBinding.ensureInitialized();
  runApp(MyApp());
}

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => new _MyAppState();
}

class _MyAppState extends State<MyApp> with WidgetsBindingObserver {
  InAppWebViewController webView;

  @override
  void initState() {
    WidgetsBinding.instance.addObserver(this);
    super.initState();
  }

  @override
  void dispose() {
    WidgetsBinding.instance.removeObserver(this);
    super.dispose();
  }

  @override
  void didChangeAppLifecycleState(AppLifecycleState state) {
    print('state = $state');
    if (webView != null) {
      if (state == AppLifecycleState.paused) {
        webView.pauseTimers();
        if (Platform.isAndroid) {
          webView.android.pause();
        }
      } else {
        webView.resumeTimers();
        if (Platform.isAndroid) {
          webView.android.resume();
        }
      }
    }
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(
          title: const Text('InAppWebView Example'),
        ),
        body: Container(
            child: Column(children: <Widget>[
          Expanded(
              child: InAppWebView(
            initialUrl: "https://www.youtube.com/watch?v=NfNdXgJZfFo",
            initialHeaders: {},
            initialOptions: InAppWebViewGroupOptions(
              crossPlatform: InAppWebViewOptions(
                debuggingEnabled: true,
              ),
            ),
            onWebViewCreated: (InAppWebViewController controller) {
              webView = controller;
            },
            onLoadStart: (InAppWebViewController controller, String url) {},
            onLoadStop: (InAppWebViewController controller, String url) {},
          ))
        ])),
      ),
    );
  }
}

yes, worked for video, but not working with mp3, please help me!

@thang510023  have u ever fixed bug when playing mp3 ?

This solution is working on Android but still have the issue on iOS

Unfortunately no. This is related to how Flutter embeds Android native views inside its widget tree.
I know that somebody is working on another way to embed Android native views.
I hope it will be better than what we have right now!

I must say, you guys have done an amazing job.

Is that issue has build in fix?


Implement basic event tracking

@peduarte: wanna checkout that branch so I can see if it's showing up? Cheers!

Yep, works! :facepunch:

:dancer:


python311Packages.hahomematic: 2023.8.13 -> 2023.8.14

Diff: https://github.com/danielperna84/hahomematic/compare/refs/tags/2023.8.13...2023.8.14
Changelog: https://github.com/danielperna84/hahomematic/releases/tag/2023.8.14
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 252709 run on x86_64-linux 1

2 packages built:

python311Packages.hahomematic
python311Packages.hahomematic.dist

Result of nixpkgs-review pr 252709 run on x86_64-linux 1

2 packages built:

python311Packages.hahomematic
python311Packages.hahomematic.dist


Fishing Icons displayed twice

Fishing icons are getting displayed twice. Not sure what's causing this, it's being displayed on fishing spots I am not fishing at as well as ones that I am fishing at.
It seems to last up until my inventory is full, at which point it fixes itself.

Those are tiles where two fishing spots exist at the same location, it is expected behavior.

Oh, didn't realise that was even a thing. Should two icons be displayed then if it's the same type of fishing spot then? Kind of redundant to show multiple.

Duplicate of #1392


feat: Find matching prerelease before bumping a new one

Closes #284
In order to do not re-tag prerelease for the same channel referencing the same commit.
I created a new function find_last_prerelease in git.rs, but changing find_last_prerelease could be a good solution too.
Let me know if you prefer another implementation.

looks good. Thanks.


changing secureTextEntry results in changing font in iOS < 10

When setting the secureTextEntry property of JVFloatLabeledTextField on and off (password-like textField) and using some custom font it eventually gets overriden by the system font. Happens in iOS 8, iOS 9, not in iOS 10.
Test project with single JVFloatLabeledTextField and button changing its secureTextEntry property:
https://github.com/catchakos/JVFloatLabeledTextField_secureTextEntry.git

seems to be a bug in UITextField, fixed in iOS 10.
Any possible solutions?

@catchakos Did you find any solution for this? I'm having the same issue.


Global csrf enable causes session creation on all GET requests

Node version: 12.16.1
Sails version (sails): 1.4.0 (still applies to master)
Session driver: connect-redis 3.2.0

When CSRF protection is enabled globally (security.csrf === true and session.saveUnitialized === false) all GET requests are passed through csurf middleware, which causes new session creation on every request when requesting client (external monitoring / metrics service in our case) does not support cookies.
CSRF middleware can't be disabled for these endpoints due to explicit check disallowing csrf adjustment in GET endpoints.
Additionally this could be partially worked around by adjusting csurf configuration (changing its default mode from session storage to cookie) but that's not exposed via application configuration.
Currently we ended up switching default to false and manually applying csrf: true to all routes as a workaround, but we'd prefer security features be enabled by default for all endpoints in the long term.
Reproduction tip:

Create a new app
Declare any test GET endpoint in config/routes.js that doesn't touch req.session
Set config.session.saveUnitialized to false
Response from GET endpoint above should not contain sails.sid cookie
Set config.security.csrf to true
Response from GET endpoint above will contain sails.sid cookie - a new session has been created

Another workaround I just found is declaring a route in config/routes without specifying its method.
{
// This doesn't work:
// "GET /healthz": {controller: "InternalController", action: "healthz", csrf: false},

// This works fine with csrf enabled globally
"/healthz": {controller: "InternalController", action: "healthz", csrf: false},
}

Hi @Informatic,
Thank you for bringing this issue to our attention and posting your workaround. We'll take a closer look when time allows.
Has anyone else encountered this behavior?


vardict.pl - use of uninitialized value

Good Evening,
I was trying Vardict (perl version), and I was facing the same problem as other regarding the creation of 36 columns output file and a teststranbiais.R script that was accepting only 34 or 38 columns. This problem is solved, and I thank you for that.
However, despite the fact that the program seams to work well, using vardict.pl script I noticed a message that arose several times:
Use of uninitialized value in hash element at /home/alflag/miniconda3/envs/vardict/bin/vardict line 1158, <SAM> line 708.
I would like to know the reason(s) of such answers, and if it will compromise the output file.
Thanks to all.
Regards,
Alex

@ALFLAG, hello!
Such errors can compromise the output sometimes. So we have to investigate them closer.
Could you please upload the small part for BAM and the command you used, which causes such an error? We will try to reproduce it.

Hello @PolinaBevad !
I am terribly sorry for this very late answer.
unfortunatly, I will not be able to communicate my data, even a part of it.
For the record, I tried to use vardict multiple times on different datasets, and I was not able to replicate the warning message, excepted of course, with the one I evoked in the first post.
I removed the problematic lines in my bam file (only four reads were problematic), and the problem disappeared.
Thank you for your time.
Regards,
@ALFLAG

@ALFLAG, thank you for the answer!
I looked at this part of the code and didn't see any values that can be unitialized on 1158 line of vardict.pl. Maybe you use an old version of Vardict?
I will close the issue, but you can re-open it if you will be able to upload part of other problematic bam to found out the reason.
Thank you again.


Missing Path for autoloader throws RuntimeException

When installing the composer package, the autoloader can not be found, as the path you are trying to resolve for the autoload, has a additional depth of 2.
Adding another line within the $paths array, within vendor/aeliot/php-cs-fixer-baseline/bin/pcsf-baseline in my case, helped me out:
$paths = [
        __DIR__ . '/vendor/autoload.php',
        __DIR__ . '/../vendor/autoload.php',
        __DIR__ . '/../../vendor/autoload.php',
        __DIR__ . '/../../../../vendor/autoload.php',
    ];

Hi @sfaujour,
Version with fix of this issue released: v1.0.1
Could you try it?

Sorry for late answer

@Aeliot-Tm Hi and sorry, was on holidays :) I'll try it this week 🙂


Improving how dependencies for themes are handled: enabling dependenc…

…ies (modules) for themes too, handling this also when themes are just previewed and disabling themes when their dependencies are disabled.
Fixes the tightly connected following issues: #4336, #5004, #4667

Closing because during the Thursday meeting we decided to push it to dev. Will merge it into dev.


An Iterator is mutable and cannot be reused but it is simple and it offers both efficient iteration and laziness.

 Is that due to the use of mutable variables? It seems valid to me to use local mutable state

it seems too simple.
it returns a mutable object. I guess it is fine, we are used to consuming Iterator only once.

Mainly I am waiting for someone to tell me it is a bad idea but it seems valid to me too.

Well it seems fine to me 👍


build: remove factory /usr/share/man deletion

Closes https://github.com/cypress-io/cypress-docker-images/issues/1184

Issue
The Global Cleanup phase of the factory build process defined in factory/factory.Dockerfile deletes the Debian system directories:

/usr/share/doc
/usr/share/man

According to https://github.com/cypress-io/cypress-docker-images/issues/1184 the full default empty directory structure of /usr/share/man is necessary for installation of the Debian package default-jre.
Background
The cypress/factory image is built on debian:12-slim which includes a description:

Image Variants
debian:<suite>-slim
These tags are an experiment in providing a slimmer base (removing some extra files that are normally not necessary within containers, such as man pages and documentation), and are definitely subject to change.

Variant size comparison

Directory
debian:12
debian:12-slim

/usr/share/doc
8.7M
1.5M

/usr/share/man
5.3M
20K

$ docker run -it --rm debian:12
root@bd2d433bb2e0:/# du -sh /usr/share/doc
8.7M    /usr/share/doc
root@bd2d433bb2e0:/# du -sh /usr/share/man
5.3M    /usr/share/man
$ docker run -it --rm debian:12-slim
root@d5237aa3e3ee:/# du -sh /usr/share/doc
1.5M    /usr/share/doc
root@d5237aa3e3ee:/# du -sh /usr/share/man
20K     /usr/share/man

Conclusion
debian:12-slim includes only a skelton directory structure under /usr/share/man and it has already removed the 5.3M files that occupy this directory structure under debian:12. This means that an additional purge of this directory by the Cypress Docker build process is not necessary for debian:12-slim.
Change
Remove the following instruction from factory/factory.Dockerfile:
rm -rf /usr/share/man

This leaves the empty /usr/share/man directory structure intact:
4.0K    /usr/share/man/man5
4.0K    /usr/share/man/man1
4.0K    /usr/share/man/man8
4.0K    /usr/share/man/man7
20K     /usr/share/man

Verification
cd factory
docker compose build factory
docker run -it --rm cypress/factory
du -sh /usr/share/man # shows 20K
apt update
apt install default-jre -y # should be successful

@MikeMcC399 Hmm, this failed miserably after I made a small commit. 🤔

@jennifer-shehane

Hmm, this failed miserably after I made a small commit. 🤔

I had the same problem with the initial commit. CircleCI was having network issues. I've just pushed an empty commit to retrigger CI.

@jennifer-shehane
Thanks for your approval and documentation change! CircleCI was successful after re-triggering.

@jennifer-shehane
Please ignore the previous request in https://github.com/cypress-io/cypress-docker-images/pull/1185#issuecomment-2275030450 to re-run a failed job.

The problem has been solved as a by-product of merging PR https://github.com/cypress-io/cypress-docker-images/pull/1187


Caret for settings menu doesn't rotate appropriately.

Issue Description
The caret for the settings dropdown should point up when clicked, but it always points down.
Issue Type

[ ] New Feature
[ ] Enhancement
[x] Bug
[ ] Other

Expected Behavior
Caret rotates appropriately when menu is opened vs closed.
Current Behavior
Caret does no rotating.

@JakeLaCombe This should be logged as a bug for terra-application-utility in the framework repo.

Yeah it gets a bit fuzzy. 😄At this point, almost all of terra-dev-site is supplied via terra-application-layout and configuration.
See https://github.com/cerner/terra-framework/issues/158 for information regarding this issue. Will close this issue once resolved in terra-framework.

Since we closed in the terra-framework as not applicable, we can close this


Hello

Enable authors to switch fonts within document (e.g., some equations use STIX, others Neo Euler perhaps depending on the CSS of their context (e.g., textbook problems vs solutions))

thk

Duplicate of https://github.com/mathjax/MathJax/issues/1121


fix: Improve CodeMirror rendering performance using idleCallback

Description
Lazy load code editor in property pane.

Load a dummy input widget.
Load the CodeEditor component upon focus

https://github.com/appsmithorg/appsmith/issues/13112
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix-10379

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
56.74 (0.03)
38.71 (0.07)
36.28 (0.05)
56.99 (0.04)

:sparkles: :new:
app/client/src/components/editorComponents/CodeEditor/LazyCodeEditorWrapper.tsx
87.01
72.37
72.22
85.48

:red_circle:
app/client/src/components/editorComponents/CodeEditor/codeEditorUtils.ts
73.85 (0)
52.94 (-2.94)
55.56 (0)
78.43 (0)

:red_circle:
app/client/src/components/editorComponents/CodeEditor/index.tsx
76.97 (0)
57.96 (-0.45)
65.22 (0)
78.46 (0)

:green_circle:
app/client/src/utils/WorkerUtil.ts
89.76 (0.78)
72.55 (1.96)
100 (0)
93.33 (0.95)

/ok-to-test sha=5dc05c8

/ok-to-test sha=5dc05c8

/ok-to-test sha=d9cecfe

/ok-to-test sha=1e4cd56

/ok-to-test sha=3194318

/ok-to-test sha=1195f9c

/ok-to-test sha=2eac476

/ok-to-test sha=ef895a4

/ok-to-test sha=8c32d7d

/ok-to-test sha=d0ecd3a

/ok-to-test sha=9a1bf83

/ok-to-test sha=d2c20c7

/ok-to-test sha=d2c20c7

/ok-to-test sha=3c5279f

/ok-to-test sha=3ee97e1

/ok-to-test sha=7fb8d69

/ok-to-test sha=7065bd9

/ok-to-test sha=9e5b3e5

/ok-to-test sha=eee6bff

/ok-to-test sha=4b686cd

/ok-to-test sha=5b0e291

/ok-to-test sha=27a58fa

/ok-to-test sha=0a64c24

/ok-to-test sha=85f88c6

/ok-to-test sha=7a174aa

/ok-to-test sha=7a174aa

/ok-to-test sha=674722d

/ok-to-test sha=0e743dc

/ok-to-test sha=0e743dc

/ok-to-test sha=151a55a

/ok-to-test sha=151a55a

/ok-to-test sha=bcf673a

/ok-to-test sha=632d292

/ok-to-test sha=15dee5b

/ok-to-test sha=867e132

/ok-to-test sha=4e336ff

/ok-to-test sha=1fe4356

/ok-to-test sha=dfbb5c0

/ok-to-test sha=dfbb5c0

/ok-to-test sha=2e28097

/ok-to-test sha=daae18c

/ok-to-test sha=e4be1cb

/ok-to-test sha=3173a67

/ok-to-test sha=dfb4dde

/ok-to-test sha=4b2cdff

/ok-to-test sha=3efa51a

/ok-to-test sha=60a9a77

/ok-to-test sha=fed1bc0

/ok-to-test sha=974935e

/ok-to-test sha=32aa1f1

/ok-to-test sha=fb760bf

/ok-to-test sha=161755d

/ok-to-test sha=c8c518b

/ok-to-test sha=4ce9130

/ok-to-test sha=606b3a6

/ok-to-test sha=7dc1211

/ok-to-test sha=7798cd5


:hammer: fix : swapped comments referencing prebuilt and xe

Hi,
the comments above the prebuilt and xe secrets seem to have been accidentaly swapped. This proposal is a change to make it right.
Please accept my Pull-Request 🍀

Thanks @NebojsaKrtolica for your contribution


Configuring middleware in the component

Description
New doc for configuring middleware
Add new middleware node under 'Developing Applications/Components"
Issue reference
PR will close: #3917

Thanks for the pr @hhunter-ms . let some comments

consider also adding these points --
We are adding 2 new metadata options (all keys and values are handled as case-insensitive):
pipeline can be set to httpPipeline or appHttpPipeline (we'll have aliases including http-pipeline and http for the first; app-http-pipeline and app for the second)
priority is an integer from -MaxInt32 to +MaxInt32
Both metadata options are required for the component to be enabled in a pipeline. For backwards-compatibility, there's no default priority.
Components with lower priorities are executed first
Priorities don't necessarily need to be sequential. In the docs, we will document as best practice the use of values like "100", "200", "300", etc - so changing the order is less complex


The last backup information returned is different from actual. (similar case reported on #5477 earlier)

Background of the case, the following command is used.
$allservers = Get-DbaCmsRegServer -SqlInstance carmd-iv-sqlmgt -Group PROD
$allservers | Get-DbaLastBackup  |
Select-Object *, @{L='CollectDate';E={Get-Date -Format 'MM/dd/yyyy HH:mm:ss'}} |
ConvertTo-DbaDataTable |
Write-DbaDataTable -SqlInstance carmd-iv-sqlmgt -Database DBA_Monitor -Table PS_LastBackupHistory -AutoCreateTable
Environmental information

PowerShell Version        : 5.1.14393.3471
dbatools latest installed : 1.0.115
Culture of OS             : en-US
SQL Server:
/* REPLACE WITH output of @@VERSION */

/* REPLACE WITH output of @@LANGUAGE */

us_english
Report

Host used

[ ] powershell.exe

[ ] ISE

[ ] VS Code

[X] Other (please specify)
SQL Agent job running the following command.
powershell.exe -ExecutionPolicy Bypass -File D:\dbatools_script\Job_LastBackupHistory\LastBackupHistory.ps1
If anything other than powershell.exe was used, please confirm that you can duplicate the issue with powershell.exe

[ ] Still buggy in powershell.exe

Errors Received
Shown in the log file.
WARNING: [22:10:03][Get-DbaBackupHistory] Found backups from multiple recovery
forks for MISBMM64 on carmd-iv-sql171, this may affect your results

PSMessageDetails      :
Exception             : System.Management.Automation.RuntimeException: Unable to find type
[Microsoft.PowerShell.Commands.PowerShellGet.Telemetry].
at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext,
Exception exception)
at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
TargetObject          : Microsoft.PowerShell.Commands.PowerShellGet.Telemetry
CategoryInfo          : InvalidOperation: (Microsoft.Power...llGet.Telemetry:TypeName) [], RuntimeException
FullyQualifiedErrorId : TypeNotFound
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at , C:\Program Files\WindowsPowerShell\Modules\PowerShellGet\<IP_ADDRESS>\PSModule.psm1: line 678
at , : line 1
PipelineIterationInfo : {}
Steps to Reproduce

powershell.exe -ExecutionPolicy Bypass -File D:\dbatools_script\Job_LastBackupHistory\LastBackupHistory.ps1
Expected Behavior
I am expected the latest backup information captured.
Actual Behavior
The result is false, the backup date is not the actual value.

The code looks good, so there is maybe an edge case in the data. Can you provide the information that Get-DbaDbBackupHistory returns for that database?
Or can you reproduce this on a clean instance with a series of Backup-DbaDatabase commands that we can use to build that situation in our labs?

Hi Sqlcollaborative/Dbatools,
Kindly find the screenshot from the table below which is the result wrote
into the table,
The date highlighted are not the true value of the DB backup corresponding
to the DB 'MISBMM64'.  The actual backup date regarding the DB is shown in
the screenshot below which is captured from SSMS via connected to the
corresponding SQL Instance.
[image: image.png]
Actually the DB backup is run on schedule (i.e. daily on DIFF; weekly on
FULL) and job run successfully; this is why I do not understand why
"Get-DbaDatabase" did not return the true value of backup date for this
specific database.   Frankly speaking, I rely on the record returned and
scan for any DB is missing backup currently for taking appropriate action.
However, the false positive is really challenging to rely on this command.
[image: image.png]
[image: image.png]
[image: image.png]
Thanks,
Ben
On Tue, Aug 11, 2020 at 10:39 PM Andreas Jordan<EMAIL_ADDRESS>wrote:

The code looks good, so there is maybe an edge case in the data. Can you
provide the information that Get-DbaDbBackupHistory returns for that
database?
Or can you reproduce this on a clean instance with a series of
Backup-DbaDatabase commands that we can use to build that situation in our
labs?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/issues/6730#issuecomment-672601908,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIYD7Y27O2SESBI4TCQ57BLSAITKVANCNFSM4P2FKECA
.

Hi, I have done the suggested exercise today as listed below.
Backup-DbaDatabase -SqlInstance CARMD-IV-SQLMGT -Database MISBMM64
Backup the database 'MISBMM64' from carmd-iv-sql171 using the above command and restore the database 'MISBMM64' into carmd-iv-sqlmgt.  The two SQL instance are the same SQL version, then run the below command and got the output log as shown in the screenshot.  and I got the correct backup date value from CARMD-IV-SQLMGT, but still not getting the correct value in CARMD-IV-SQL171.  Do you see any reason why the date return are incorrect from CARMD-IV-SQL171?
Get-DbaLastBackup -SqlInstance carmd-iv-sql171 -Database MISBMM64 | Out-Gridview
Get-DbaLastBackup -SqlInstance carmd-iv-SQLMGT -Database MISBMM64 | Out-Gridview

Can you run the following command and verify if it returns the proper backup:
Get-DbaDbBackupHistory -SqlInstance CARMD-IV-SQLMGT -Database MISBMM64 -LastFull
Get-DbaDbBackupHistory -SqlInstance carmd-iv-SQLMGT -Database MISBMM64 -LastFull

I think you mean these
Get-DbaDbBackupHistory -SqlInstance CARMD-IV-SQLMGT -Database MISBMM64 -LastFull | Out-Gridview
Get-DbaDbBackupHistory -SqlInstance carmd-iv-SQL171 -Database MISBMM64 -LastFull | Out-Gridview
But I still getting the incorrect date return from CARMD-IV-SQL171.

And if you don't use -LastFull? So showing all full backups?

It will be like this as shown before the last.

Get-DbaLastBackup only gets the last backup.
Get-DbaDbBackupHistory -LastFull gets the last full backup.
What I want to see is the complete list of all backups, or better all full backups. So please use Get-DbaDbBackupHistory -Type Full to get these. There should be the one that is reported as the last but also others. I would like to see the reason for that backup beeing reported as last backup.

Yes, here are all the history on SQL171 regarding this database.

The wrong last backup is the first in the list, right?
Lets add the recovery fork and the LSN: place a | Select-Object -Property Start, End, LastRecoveryForkGuid, FirstLsn before the | Out-GridView. (Have no live system - there maybe typos in the command)
A lot of ordering is done with last_lsn. Maybe you have old records from an other database with the same name in the msdb, that had greater lsn. Then a way to solve the problem would be to clean these records from the backup-tables in msdb.

Yes, the incorrect date is the first record of the list.  I will try the suggested command first, thanks.
Per suggested "A lot of ordering is done with last_lsn. Maybe you have old records from an other database with the same name in the msdb, that had greater lsn. Then a way to solve the problem would be to clean these records from the backup-tables in msdb."
If you mean housekeeping the related record rows with database='MISBMM64'  from following tables in msdb resided in the SQL Instance carmd-iv-sql171.  Would that be any harm because it is a production SQL instance.

Here is the command I just run.
PS C:> Get-DbaDbBackupHistory -SqlInstance carmd-iv-SQL171 -Database MISBMM64 -TYPE Full |
Select-Object -Property SqlInstance, Database, Type, DeviceType, Start, Duration, End, LastRecoveryForkGuid, FirstLsn |
Out-Gridview

Questions:
1, Can I use 'order by' in the select statement in this commandlet
2. Frankly, my daily run command is actually using get-DbaLastbackup, not get-DbaDbBackupHistory, so the incorrect value is also returned from this command.

It's 10 PM local time in germany, so I'll have a closer look in about 12 hours. But I think we can solve the case soon. Will answer your questions tomorrow.

Really appreciate for your help on the case Andreajordan.🙏

First of all, there is a very good comment in the code:
https://github.com/sqlcollaborative/dbatools/blob/784387f8a066d69fb9c3dba92a3a0468858b4ff7/functions/Get-DbaDbBackupHistory.ps1#L474-L484
But it does not work as designed, I did a demo:
$SqlInstance = 'WIN10\SQL2016'
$DatabaseName = 'Test01'

$server = Connect-DbaInstance -SqlInstance $SqlInstance

$server.Databases['master'].ExecuteNonQuery("CREATE DATABASE $DatabaseName")
$database = $server.Databases[$DatabaseName]

$database.ExecuteNonQuery("CREATE TABLE dbo.test (x char(1000) default 'x')")
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName
1 .. 100 | ForEach-Object -Process { $database.ExecuteNonQuery("INSERT INTO dbo.test DEFAULT VALUES") }
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName

Get-DbaDbBackupHistory -SqlInstance $SqlInstance -Database $DatabaseName | Format-Table -Property End, LastRecoveryForkGuid, FirstLsn
<#
End                   LastRecoveryForkGUID                          FirstLsn
---                   --------------------                          --------
8/14/2020 12:21:58 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>7200042
8/14/2020 12:20:05 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>3600179
#>

$database | Remove-DbaDatabase -Confirm:$false

$server.Databases['master'].ExecuteNonQuery("CREATE DATABASE $DatabaseName")
$database = $server.Databases[$DatabaseName]

$database.ExecuteNonQuery("CREATE TABLE dbo.test (x char(1000) default 'x')")
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName
1 .. 100 | ForEach-Object -Process { $database.ExecuteNonQuery("INSERT INTO dbo.test DEFAULT VALUES") }
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName

Get-DbaDbBackupHistory -SqlInstance $SqlInstance -Database $DatabaseName | Format-Table -Property End, LastRecoveryForkGuid, FirstLsn
<#
End                   LastRecoveryForkGUID                          FirstLsn
---                   --------------------                          --------
8/14/2020 12:21:58 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>7200042
8/14/2020 12:24:18 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>7200042
8/14/2020 12:20:05 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>3600179
8/14/2020 12:24:16 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>3600179
#>

# I deleted the database (without deleting the historie) and created a new one with the same name.
# So the old backup history should not be included - have a look at the last part of the comment in the code.
# But it is - so there must be a bug there.

1 .. 1000 | ForEach-Object -Process { $database.ExecuteNonQuery("INSERT INTO dbo.test DEFAULT VALUES") }
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName
1 .. 1000 | ForEach-Object -Process { $database.ExecuteNonQuery("INSERT INTO dbo.test DEFAULT VALUES") }
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName
1 .. 1000 | ForEach-Object -Process { $database.ExecuteNonQuery("INSERT INTO dbo.test DEFAULT VALUES") }
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName
Get-DbaDbBackupHistory -SqlInstance $SqlInstance -Database $DatabaseName | Format-Table -Property End, LastRecoveryForkGuid, FirstLsn
<#
End                   LastRecoveryForkGUID                          FirstLsn
---                   --------------------                          --------
8/14/2020 12:26:43 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>0000042
8/14/2020 12:26:41 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>2800042
8/14/2020 12:26:39 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>4000042
8/14/2020 12:21:58 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>7200042
8/14/2020 12:24:18 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>7200042
8/14/2020 12:20:05 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>3600179
8/14/2020 12:24:16 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>3600179
#>

1 .. 1000 | ForEach-Object -Process { $database.ExecuteNonQuery("INSERT INTO dbo.test DEFAULT VALUES") }
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName
1 .. 1000 | ForEach-Object -Process { $database.ExecuteNonQuery("INSERT INTO dbo.test DEFAULT VALUES") }
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName
1 .. 1000 | ForEach-Object -Process { $database.ExecuteNonQuery("INSERT INTO dbo.test DEFAULT VALUES") }
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName
Get-DbaDbBackupHistory -SqlInstance $SqlInstance -Database $DatabaseName | Format-Table -Property End, LastRecoveryForkGuid, FirstLsn
<#
End                   LastRecoveryForkGUID                          FirstLsn
---                   --------------------                          --------
8/14/2020 12:27:56 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>7200042
8/14/2020 12:27:54 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>5200042
8/14/2020 12:27:52 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>7200042
8/14/2020 12:26:43 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>0000042
8/14/2020 12:26:41 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>2800042
8/14/2020 12:26:39 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>4000042
8/14/2020 12:21:58 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>7200042
8/14/2020 12:24:18 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>7200042
8/14/2020 12:20:05 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>3600179
8/14/2020 12:24:16 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>3600179
#>

# Now I restore the database to an earlier backup to get a new recovery fork
Get-DbaDbBackupHistory -SqlInstance $SqlInstance -Database $DatabaseName | Where-Object FirstLsn -EQ '36000000124000042' | Restore-DbaDatabase -SqlInstance $SqlInstance -WithReplace

# Lets take a backup and have a look
$null = Backup-DbaDatabase -SqlInstance $SqlInstance -Database $DatabaseName
Get-DbaDbBackupHistory -SqlInstance $SqlInstance -Database $DatabaseName | Format-Table -Property End, LastRecoveryForkGuid, FirstLsn
<#
End                   LastRecoveryForkGUID                          FirstLsn
---                   --------------------                          --------
8/14/2020 12:27:56 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>7200042
8/14/2020 12:27:54 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>5200042
8/14/2020 12:27:52 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>7200042
8/14/2020 12:26:43 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>0000042
8/14/2020 12:26:41 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>2800042
8/14/2020 12:33:28 PM 46920f9e-367c-4e7f-91ac-e3592b225f7a<PHONE_NUMBER>3600081  <==
8/14/2020 12:26:39 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>4000042
8/14/2020 12:21:58 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>7200042
8/14/2020 12:24:18 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>7200042
8/14/2020 12:20:05 PM 230d92b7-1a6e-4cda-a968-65bd6a63b63e<PHONE_NUMBER>3600179
8/14/2020 12:24:16 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>3600179
#>

# I marked the new backup with <==
# Because the backups are sorted by FirstLsn, my last backup is not the "LastBackup"
Get-DbaDbBackupHistory -SqlInstance $SqlInstance -Database $DatabaseName -LastFull | Format-Table -Property End, LastRecoveryForkGuid, FirstLsn
<#
End                   LastRecoveryForkGUID                          FirstLsn
---                   --------------------                          --------
8/14/2020 12:27:56 PM ec32997b-c70b-4bf4-affd-014aef1c7b00<PHONE_NUMBER>7200042
#>

First of all, how can @bmwluk2012 solve the problem:
You could just delete all entries in the backup tables in msdb not related to your current recovery fork. The data in these tables are only there to help a restore on that instance. But there a constraints, so it's not a one liner.
But you can change the database name in the backupset table:
UPDATE msdb..backupset SET database_name = database_name + '_invisible' WHERE database_name = 'Test01' and last_recovery_fork_guid != '46920F9E-367C-4E7F-91AC-E3592B225F7A'

The data is still there, so no record deleted in a production instance, but kind of marked "invisible".
And now to the moriginal editors ( @potatoqualitee / @Stuart-Moore ): Would you like me to rewrite the sql to address this issue? Should we open a new issue for that or use this one? Or am I missing something and this is no bug?

What is your proposed change to the code to address the issue?

Because I like SQL I played a little bit with the query.
I think we can change these lines:
https://github.com/sqlcollaborative/dbatools/blob/784387f8a066d69fb9c3dba92a3a0468858b4ff7/functions/Get-DbaDbBackupHistory.ps1#L566-L575
to this:
                        JOIN (
                        SELECT database_guid, last_recovery_fork_guid
                        FROM (SELECT database_guid, last_recovery_fork_guid, ROW_NUMBER() OVER (ORDER BY backup_finish_date DESC) AS rn
                              FROM msdb..backupset
                              WHERE database_name = '$($db.Name)'
                             ) AS bs
                        WHERE rn = 1
                        ) AS last_guids ON last_guids.database_guid = backupset.database_guid AND last_guids.last_recovery_fork_guid = backupset.last_recovery_fork_guid
                    WHERE 1=1 $whereCopyOnly

That would get the last database_guid (is maybe not needed if last_recovery_fork_guid is unique over all databases) and the last used last_recovery_fork_guid for the database to filter the possible last backups. Then they can savely sorted by LastLSN.

@Stuart-Moore authored and owns our backup/restore process so he gets the review and final call on this one.

What is your proposed change to the code to address the issue?

Regarding the suggested workaround, I have tried and it works perfectly now, no more false positive value returns, see my actions as shown following
Run the TSQLs ...
use msdb;
/Identified the related/required last_recovery_fork_guid,/
--CARMD-IV-SQL171	MISBMM64	Full	Disk	8/12/2020 9:45:26 PM	00:00:01	8/12/2020 9:45:27 PM	1cc9f673-d6d4-437b-9220-e374079ff33a<PHONE_NUMBER>6300037
/Action/
Begin Tran T1
update backupset
set  database_name = database_name + '_invisible'
WHERE last_recovery_fork_guid<>'1cc9f673-d6d4-437b-9220-e374079ff33a' and database_name = 'MISBMM64'
--(1904 rows affected);   Completion time: 2020-08-14T08:23:43.1447451-07:00
Commit Tran T1	--Completion time: 2020-08-14T08:25:44.0602198-07:00
SELECT * from backupset WHERE last_recovery_fork_guid<>'1cc9f673-d6d4-437b-9220-e374079ff33a' and database_name = 'MISBMM64_invisible'	--1904, not the latest backup set
Then run the DBATool commandlet as ....
PS C:> get-DbaLastBackup  -SqlInstance  carmd-iv-sql171 -Database MISBMM64 | Select * | Out-Gridview
And the result set has return as expected ...

Thanks for the help @andreasjordan 🙏

gosh I love the fact that I was the one commenting on that query or I'd have an headache trying to reconstruct why we do that.
Theoretically it seems that we wanted to prevent a problem that was fixed but not on all cornercases. Though I'm still unsure why the current query (which doesn't consider fork_guid) doesn't solve the problem at all. It SEEMS to select correctly the database_guid which has the most recent finish-date.
My notes from two years ago are only a minimal corner-case where DST happens (or timezone in the server is set to a different one) and you created a new db in between ...
Query-wise, ROW_NUMBER() on 2005 surely works (do not ask where, though).
But I'll try nonetheless to reproduce the error and spot where the current query breaks

@niphlod The database_guid only helps to filter out different databases with the same name. When you have a database with a very high LSN and you have to restore a backup with a low LSN (so going way back in history - why that ever is usefull - but was done here in this case in this issue) the database_guid stays the same, only the recovery_fork_guid changes and the LSNs are back to the low value. Then sorting just by LSN (which is very good and the reason for that is very good documentated in the code - thanks for that!) gets the backup with the high LSN from the old recovery fork that is not active at the moment. So I introduced the filter to the current recovery_fork. If you want to replay it, have a look at my demo: https://github.com/sqlcollaborative/dbatools/issues/6730#issuecomment-674019765

@andreasjordan : aaargh, gotcha.
Note to self (to read in about 2 years lol): I remember now why we added that part.
It was when you restore dbs with the same name over and over BETWEEN environments, i.e. a refresh from prod to test. In that case database_guid is different hence it works. Here instead we're restoring in-place and it bombs, given database_guid doesn't change.
The snippet was incredibly useful. But .... we can totally avoid ROW_NUMBER() (and IMHO get a more legible query) with a rather simple
JOIN msdb..backupset AS backupset ON backupset.media_set_id = mediaset.media_set_id
                        JOIN (
                            SELECT TOP 1 database_guid, last_recovery_fork_guid, backup_finish_date
                            FROM msdb..backupset
                            WHERE database_name = '$($db.Name)'
                            ORDER BY backup_finish_date DESC
                        ) AS last_guids ON last_guids.database_guid = backupset.database_guid AND last_guids.last_recovery_fork_guid = backupset.last_recovery_fork_guid
                    WHERE backupset.database_name = '$($db.Name)' $whereCopyOnly

@niphlod Thanks for reminding me that we are on SQL Server and not on Oracle. Most SQL stuff I do, I do on Oracle databases so I often forget the nice TOP 1 feature from Microsoft. I will update the pull request first thing tomorrow morning - and now go back to my friends and my glas of wine.

@andreasjordan : noprobls. Maybe add a comma to that comment, I feel like it'll still help down the line to retrace the steps we took (and the thinking behind).
if you don't want to add a regression test to Get-DbaDbBackupHistory.Tests.ps1 I'll be happy to punch that in reusing your snippet.
to all who partecipated in this issue: thanks for the patience and the hints (and pushing for a real-case scenario even if I think I proved that remembering things is a hard job )

Thank you Gents. for sorted out the issue and the refinement.  I am looking forward the new release of 'get-DbaLastBackup'.  Cheers.


Compiler option for CoreMark

I have tried running CoreMark on the simulator with below options provided by Syntacore
O3 -funroll-loops -fpeel-loops -fgcse-sm -fgcse-las
However, the performance is up to 4.1 CM/MHz
Is there any suggestion about compiler option for better performance?
Besides, is there any suggestion about hardware configuration or simulation mechanism for better performance?
Here is my hardware configuration and simulation mechanism

Pre-load instruction in ICCM (Default size)
I-cache enabled(Default configuration)(Using provided assembly code)
Pre-load data in DCCM (Default size)

Thanks!

Lance, there are a couple of things to check to make sure you get the right CoreMark score from rtl simulation.
First thing is exactly how are you measuring cycles for one CoreMark loop iteration?
We use perf_start and perf_end labels as shown below in the C source:
/* perform actual benchmark */
start_time();

__asm("__perf_start:");

#if (MULTITHREAD>1)
if (default_num_contexts>MULTITHREAD) {
default_num_contexts=MULTITHREAD;
}
for (i=0 ; i<default_num_contexts; i++) {
results[i].iterations=results[0].iterations;
results[i].execs=results[0].execs;
core_start_parallel(&results[i]);
}
for (i=0 ; i<default_num_contexts; i++) {
core_stop_parallel(&results[i]);
}
#else
iterate(&results[0]);
#endif
__asm("__perf_end:");

stop_time();
total_time=get_time();

When we measure from perf_start to perf_end in rtl simulation with the following compiler flags “-O3 -funroll-all-loops fgcse-las”, we get a CM/MHz of around 4.6 for the default configuration of swerv core.  Our small cmark2.c example has 2 iterations, so calculation is (cycle_perf_end - cycle_perf_start) / 2 for 1 iteration of CoreMark.
What do you get?
If I reconfigure the core to have more instruction cache, larger predictor
./swerv.config -btb_size=512 -bht_size=2048 -icache_size=32

the score improves to around 4.9 CM/MHz.

Hi Lance,
We compiled core-marks with gcc 7.2.0 using the following flags: -O3 -funroll-all-loops -fgcse-sm -fgcse-las -mabi=ilp32 -march=rv32imc
We put all the source code of coremarks into one file.
Joe

btw, I think we need to figure out CoreMark before we switch over to Dhrystone.
Robert

Lance, if you don't mind email me at<EMAIL_ADDRESS> Thanks.
On Thu, Mar 14, 2019 at 10:36 AM jrahmeh<EMAIL_ADDRESS>wrote:

Hi Lance,
We compiled core-marks with gcc 7.2.0 using the following flags: -O3
-funroll-all-loops -fgcse-sm -fgcse-las -mabi=ilp32 -march=rv32imc
We put all the source code of coremarks into one file.
Joe
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/westerndigitalcorporation/swerv_eh1/issues/14#issuecomment-472923293,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ArVBndFB6BGDfTwRg8niMo177QSbeHzKks5vWmyYgaJpZM4bksEi
.

Robert,
That would be helpful, I will contact you via email.
Thanks!

The cause of mismatch is version of gcc
When I switch my gcc version from 8.2 to 7.2, the performance is improved!
In original config: ~4.6CM/MHz
Larger i-cache and larger BTB/BHT : ~4.9CM/MHz
Thanks a lot!

Lance,
I would also quite appreciate if you publish your findings as some sort of
PDF "Application note" document, for future attempts on Coremark reqsult
replication.
Super thanks,
Z.
On Sun, Mar 17, 2019 at 8:16 PM lance91633<EMAIL_ADDRESS>wrote:

The cause of mismatch is version of gcc
When I switch my gcc version from 8.2 to 7.2, the performance is improved!
In original config: ~4.6CM/MHz
Larger i-cache and larger BTB/BHT : ~4.9CM/MHz
Thanks a lot!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/westerndigitalcorporation/swerv_eh1/issues/14#issuecomment-473755246,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHVI2FtJOFRXUGU3iGB1TpUXs2pun3bks5vXwT6gaJpZM4bksEi
.


Fix intake v2 trace logging

TRACE logging could corrupt the intake v2 events previously

Codecov Report

Merging #616 into master will decrease coverage by 11.57%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             master     #616       +/-   ##
=============================================
- Coverage        74%   62.43%   -11.58%
+ Complexity     1385     1358       -27
=============================================
  Files           130      189       +59
  Lines          5375     7115     +1740
  Branches        566      819      +253
=============================================
+ Hits           3978     4442      +464
- Misses         1151     2406     +1255
- Partials        246      267       +21

Impacted Files
Coverage Δ
Complexity Δ

.../apm/agent/report/serialize/DslJsonSerializer.java
81.84% <0%> (ø)
132 <0> (ø)
:arrow_down:

...c/main/java/co/elastic/apm/agent/util/IOUtils.java
56.94% <0%> (-40.68%)
10% <0%> (ø)

...ration/source/PropertyFileConfigurationSource.java
17.72% <0%> (-22.28%)
7% <0%> (ø)

...tion/source/SystemPropertyConfigurationSource.java
15.38% <0%> (-17.95%)
2% <0%> (ø)

...va/co/elastic/apm/agent/impl/transaction/Span.java
66.66% <0%> (-10.46%)
19% <0%> (-21%)

...apm/agent/impl/transaction/TraceContextHolder.java
53.19% <0%> (-9.31%)
12% <0%> (-9%)

...lastic/apm/agent/impl/transaction/Transaction.java
88.23% <0%> (-5.89%)
21% <0%> (-13%)

...c/apm/agent/impl/async/SpanInScopeBaseWrapper.java
58.33% <0%> (-4.63%)
6% <0%> (ø)

...va/co/elastic/apm/agent/impl/ElasticApmTracer.java
80.4% <0%> (-1.1%)
59% <0%> (-30%)

...m/agent/impl/async/SpanInScopeRunnableWrapper.java
100% <0%> (ø)
5% <0%> (ø)
:arrow_down:

... and 62 more

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 3cb729c...c79c416. Read the comment docs.


Simplify steps by using powershell on both platforms to install built package.

Pro:
1 step instead of 2 for all platforms. Very consistent. The steps file is getting a bit bloated.
Con:
People don't usually use Powershell as their shell on macOS and Linux. The fact it is preinstalled on all Azure Pipeline runners on all the platforms is quite unusual.
I don't know what decision to make. It's something I noticed.

Looks good. Thanks.


Configuring a course

My builders made parkour and in order to complete it you need speed 1 can you please add a feature to where when you start the parkour you can have speed 1 or jump boost 2.
Thank you

This has been achieved in the form of the new ParkourMode "Potion", when having the ParkourMode conversation via "/pa setmode (course)" the "potion" option will allow the user to enter a PotionEffect name, followed by an optional duration and amplifier, then an optional join message.


About TeamTalkSDK

I was looking at the new versions of TeamTalkSDK, and the library was all refactored in version 8.12. The problem that I couldn't run it on TTMediaBOT, because of the acertion errors.
I would like to hear from the developers, if they intend to switch to this SDK or if they will continue on TeamTalkSDK 5.8.
I suggest that if they are going to continue in TeamTalk 5.8, that TTMediaBOT distributes its own TeamTalkSDK, as it has no way of implementing the missing functions of the library, such as _DoMoveUser, as they will be refused by BearWare, since they are too old.

As I remember we could run TTMediaBot on latest TeamTalk 5.12 and it worked normally.
Maybe there are some breaking changes but we didn't find them yet.
And I am not sure about Linux because I have tested it only on Windows and without actually playing any audio. It was just launch test and some commands.

Hello. I have several modifications in the TeamTalkSDK and I would like to send them to TTMediaBot. Added the missing _DoMoveUser function and may add some others in the future. But for that, TeamTalkSDK would need to be distributed with TTMediaBot. It's a suggestion, but if you accept it I can send my changes to the SDK that you add to the BOT, since what is being used here is 5.8 and that's what looks best in my opinion.
I would take advantage and also send a very nice change that I made to My TTMediaBot. A command that a user can use to pull a bot into a channel, and also functionality that can be turned off and on with an admin command. If activated and no user is in the channel the bot entered, it automatically returns to the initial channel.

@cyrmax

i believe this is a good function, and i woould be very hapy if it goes to official ttmediabot. i see in several teamtalks persons loadin 10 or 20 ttbots, and requesting to admins to move it, or create one for their channels. if this functionalty goes, the user can find for example in root channel all available bot, request one to his channel, and after using, bot returns to the root channel, not keeping to waste resources unecessary in the server or playing audio for noone.

20 TTMediaBot's? if the bot didn't have memory leaks, it would be possible, but when you using the youtube service, memory leaks, and can go to 600 mb per bot

it is possible if you have a server that can handle


Detect if an external authentication provider was used at sign in or not

From @gabosistvanpwc on December 12, 2017 14:33
Hi,
Is there any way to tell after login if the current user was signing in by using the regular authentication flow or an external authentication provider?
Thank you
Copied from original issue: aspnet/Identity#1530

From @grahamehorner on December 12, 2017 14:54
the claims for the user identity should have the login provider information?

From @gabosistvanpwc on December 12, 2017 15:12
Unfortunately it doesn't.
I'm using the basic ASP.NET Core Web Application project template with the Individual User Accounts authentication option. As the external provider I have configured the WsFederation middleware as follows:
services.AddAuthentication()
.AddWsFederation(options =>
{
options.Wtrealm = "xxxx";
options.MetadataAddress = "xxxx";
})
.AddCookie();
After the request is going through the usual steps

AccountController.ExternalLogin
Identity provider login page
AccountController.ExternalLoginCallback (and here _signInManager.ExternalLoginSignInAsync method)

and the user is authenticated the claims from the User.Identity have the same "default" entries as if the user was authenticating directly through the application.
I would like to tell at any point when accessing an action /controller which requires authentication if the user was logging in the system through the regular sign in flow or through an external authentication provider.
Probably I'm missing here something....

@Tratcher Does WS-Fed not embed this info like OIDC does? Can we ensure it matches?

They're dropping all provider specific information at step 3 above. This is more of an identity / template question on how to preserve external claims when transitioning to a local user.


the idea of the golden setting should be improved

In config.py now, it is forced to bundle with the settings for FB15k.
Should make it more flexible to have different golden settings for different known datasets.

done


Shrink images that are too large

This is necessary eventually. However it also comes with a few downsides such as longer compile times, larger binary size and longer parsing times of splits due to the shrinking. This introduces the image-shrinking feature that allows you to choose whether you want to compile this in. It is activated by default. The plan is to introduce this to the original LiveSplit and Splits I/O before this lands in LiveSplit One, so that we can reduce the amount of actively used splits in the wild that still need to be shrunk once LiveSplit One gets the feature.
 I think natively compiled it does the conversion a bit faster, and we want to do it there as soon as possible anyway, so I'd rather wait a bit.


Revert "Cooper/develop"

Reverts qingfengzxr/gdscript-web3#5

Roll back the messy commit caused by forgetting to rebase


Deliver api doc with server

enable api doc page under /swagger-ui.html

This PR contains swagger info that we don't even have implementations for (query lookup by user, query deletion etc.).
If everyone is cool with it, we can merge this anyway, move the project to a new organization and file a new issue over there in order to revise it.

I think we should put it on hold until the corresponding functions are implemented.

Closing due to the latest comment.


Unblock releases given Conda issue blocking tiledbsoma 1.9.5

Tracks https://github.com/TileDB-Inc/tiledbsoma-feedstock/issues/126 which per our established procedure prohibits us from doing any releases until that problem is resolved.

Underlying third-party issues we were trying to work around have been resolved here: https://github.com/TileDB-Inc/tiledbsoma-feedstock/pull/129#issuecomment-2047776530
Conda feedstock pre-check PR is green https://github.com/TileDB-Inc/tiledbsoma-feedstock/pull/131 is green
Manual wheel-build pre-check is green https://github.com/single-cell-data/TileDB-SOMA/actions/runs/8646791632

[sc-44854]


Source maps for the documentation

When using the styles in the documentation, source maps are not available so all styles come from one CSS file, here post-external.css and it's no longer possible to evaluate where exactly a certain line of code is originating from.

Is there a possibility to load source maps?

Closing as obsolete in favor of the local styles dev build


Code Climate Fixes

Description
The .yml file for code climate needed some improvements on how it should run through our code. This will help it fit better with JavaScript.
Fixes # (issue)
.yml file was too simple causing fake positives.
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)

Change Status

[X] Complete, but not tested (may need new tests)

Checklist

[X] I have performed a self-review of my own code
[X] There are no merge conflicts

 Fix Code Climate to Work Better


Peer dependencies not fully specified

After upgrading to DaisyUI V2, I got this message from Yarn

The reason that this is an issue is because it violates Yarn's hoisting behaviour. Yarn documentation on missing transitive peer dependencies and blog post that the documentation references

Yes, if you don't provide resolution for the dependencies yourself (which for autoprefixer, is certainly the case)

@JamesYeoman Please install daisyui version 2.0.7 and check if this fixes the issue

Yep, no more peer dependency message. Thanks a lot for taking the time to address it


CLOUD-933: Remove generate from doctor, create only compose-yaml for migration

@akanto or @lalyos pls review...

I don't see that 'cbd doctor' would be used: https://github.com/sequenceiq/docker-cbdb/blob/master/create-db-tgz.sh
The reason behind doctor calls generate is: that it will check that generate would change anything.
so i'm hesitant to remove it.
The original issue is about a side effect: "certs directory is created". How about simply deleting this dir in the clean() function: https://github.com/sequenceiq/docker-cbdb/blob/master/create-db-tgz.sh#L46


Unable to edit form after error when creating a community

Describe the bug

If a picture doesn't say 1000 words, then: If you get a form submission error on /create/community then you can no longer edit the form (since it looks like the fields are disabled during submission).
To Reproduce
Steps to reproduce the behavior:

Go to https://opencollective.com/create/community
Fill in the form, using the collective URL of your username (in my case thisismissem)
Submit the form
See error
Try to correct error

Expected behavior
I should be able to correct the error and resubmit the form
Actual behavior
I am not able to edit the form and resubmit it.
Screenshots
See above.
Desktop (please complete the following information):

OS: MacOS 14.6
Browser: Firefox
Version: 128, with enhanced tracking protection (the little shield icon in the URL bar)

Additional context
n/a

Triage Template (core team only)
This template is for members of the team to triage for prioritisation. For more guidance see https://www.loom.com/share/369ab467fbc64dec848085d38ff57ca0:
P1 high frequency, high impact
P2 low frequency, high impact
P3 high frequency, low impact
P4 low frequency, low impact
Examples of high impact - a problem affects users during an essential process (expenses and payments > onboarding and registration > contributing) with no workaround.
High frequency - >10% of users affected (measured as a proportion of total potential users for this case).

@ThisIsMissEm Thanks for the detailed report! We'll fix it with https://github.com/opencollective/opencollective-frontend/pull/10598.


Add RangeHelper

Addresses aspnet/Mvc#3702
cc @Tratcher @pranavkm @Eilon

@jbagga,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@Tratcher Updated


How to serve output of flutter build web --wasm.

Steps to reproduce
I am taking a wasm build of my application by running  flutter build web --wasm and trying to serve this using my dart server.
Since this creates dual-compatible output for both dart2js and wasm, it picks the js bundle by default.
I also tried the following config in my index.html file.
config : {"compileTarget":"dart2wasm","renderer":"skwasm","mainWasmPath":"main.dart.wasm","jsSupportRuntimePath":"main.dart.mjs"} inside _flutter.loader.load
This throws an error FlutterLoader could not find a build compatible with configuration and environment.
Actual results
Looking for a way to set the config to pick wasm file when I serve the output of flutter build web --wasm.
Flutter Doctor output

Doctor output

[✓] Flutter (Channel stable, 3.24.4, on Ubuntu 22.04.5 LTS 5.15.0-117-generic, locale en_US.UTF-8)
    • Flutter version 3.24.4 on channel stable at /usr/local/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 603104015d (3 weeks ago), 2024-10-24 08:01:25 -0700
    • Engine revision db49896cf2
    • Dart version 3.5.4
    • DevTools version 2.37.3

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/to/linux-android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /chrome

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 14.0.0-1ubuntu1.1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/to/linux-android-setup for detailed instructions).

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.5 LTS 5.15.0-117-generic
    • Chrome (web)    • chrome • web-javascript • Chromium 122.0.6177.0

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

Hi @ebala96
How do you serve your web? Please check this to see if it helps: https://docs.flutter.dev/platform-integration/web/wasm#serve-the-built-output-with-an-http-server
Please also check https://github.com/flutter/flutter/issues/154541 if it's similar to your case.

@huycozy, that section of the docs has since been removed in https://github.com/flutter/website/pull/11367
@sfshaza2, that section was marked as "not needed" since Flutter run works by default, but that doesn't address the issue of serving a release build from your own web server. This page of Chrome Developers seems to suggest the config is still very much needed.
Can we consider re-adding it?


Updated "Frequently asked question" section | #233

Hey, i just updated the "Frequently asked question" section, take a look!
Issue number: #233
https://github.com/user-attachments/assets/f5f98391-f18f-4255-b508-1a516dd79fd3
I added the appropriate answer for the corresponding questions and made sure its responsive too!
And added a "Frequently asked question" heading at the center.
Hope you like it, Please label me under gssoc and hacktober fest and a level label too.
Thank You.🙏

Hey, can you please label my contribution!
Label me under gssoc-ext and hacktober fest and a level too.Thank You🙏

#233

Hey, can you please label this pull request !
Label me under gssoc-ext and hacktober fest and a level too. Thank You🙏

Hey, can you please label this pull request !
Label me under gssoc-ext and hacktober fest and a level too. Thank You🙏

@07sumit1002 Hey, can you please label this pull request !
Label me under gssoc-ext and hacktober fest and a level too. Thank You🙏


Minor fix of Javadoc formatting

This PR addresses a minor issue in Javadoc formatting. This issue prevented the previous Spine version from being published to the repository (see #558).
Summary:

format the reference to Enricher's own method without using a generic parameter.

The library version left intact.

@alexander-yevsyukov PTAL.


[Cub] Input element for entering episode number does not have a label

Description
The input element for entering episode number does not have a visible text label. According to the WCAG specification, an interactive element must be labeled. This is because it is intended for users to interact with. The label will help those using assistive technologies to become aware of the type of interaction required with the focused element.
To  Reproduce
On firefox v81.0.2, right click the input element with the placeholder that says "Enter Episode Number" and click inspect accessibility properties and you will see that this element has a text label accessibility issue.
Expected Behavior
The input element should have a label that announces to the user the kind of interaction expected, in this case, to enter episode number.
Browser information
Firefox for Windows 10 v81.0.2
Possible Solution
A visible text labeling the input element does not fit into the design in this project. Therefore, the input element can be wrapped inside a label element. The label element should not be given a visible text, rather the aria-label attribute should be added to the input element and given the same value as the placeholder. Ordinarily, the voice-reader would read the text in the placeholder and that could serve as the name of the input element but it is generally advised not to use placeholder for assistive technology users because the text disappears when the user starts to interact with the element and that could cause some problems.

Closed by #38


bug: in discord-reply the role color attribute is still roleColor instead of role-color

Is there an existing issue for this?

[X] I have searched the existing issues

Description of the bug
The html attribute to set the role color of a reply is roleColor instead of role-color.
Steps To Reproduce
See this: https://stackblitz.com/edit/github-syurak?file=index.html
Expected behavior
role-color to set the role color of a reply
Screenshots

Additional context
No response

Thanks for catching this!

released in v4.0.0-alpha.36


[Bug]: Latest versions of llama-index-multi-modal-llms-gemini and llama-index-llms-gemini are not compatible with each other

Bug Description
llama-index-multi-modal-llms-gemini (0.1.7) depends on llama-index-llms-gemini (>=0.1.9,<0.2.0)
and src depends on llama-index-llms-gemini (0.2.0), llama-index-multi-modal-llms-gemini is forbidden.
So, because src depends on llama-index-multi-modal-llms-gemini (0.1.7), version solving failed.
Version
0.10.62
Steps to Reproduce
pip install llama-index-llms-gemini = "0.2.0"
llama-index-multi-modal-llms-gemini = "0.1.7"

Relevant Logs/Tracbacks
llama-index-multi-modal-llms-gemini (0.1.7) depends on llama-index-llms-gemini (>=0.1.9,<0.2.0)
 and src depends on llama-index-llms-gemini (0.2.0), llama-index-multi-modal-llms-gemini is forbidden.
So, because src depends on llama-index-multi-modal-llms-gemini (0.1.7), version solving failed.

This is still broken with the new release on llama-index-multi-modal-llms-gemini.
I am getting llama-index-multi-modal-llms-gemini (0.1.8) depends on llama-index-llms-gemini (>=0.1.9,<0.2.0)

It was a pretty straightfoward fix, would have welcomed a pr 🤷🏻


Add pipeline schedules trigger

closes #870

trynna add tests. Any reason to not use testify assert?


Clarify date primitive type in PROTOCOL.md

@zsxwing @scottsand-db clarified semantics based on the review, please take another look


Implemented MPI_Waitany

I needed the MPI_Waitany for a personal project so I decided to implement it as a function in the Request module. This is the first time i work with ffi and unsafe code so I tried to keep things as simple as possible.
I'm not sure if using a Vec of Request as input parameter and removing the completed Request from it is the best way to go but it was the simplest I could find.

By the way, stay strong. America stands with Italy!

Thank you Andrew, I hope the situation there in America don't get as bad as here.
I've fixed the problems you have spotted in the implementation.


Update roadmap and add option for revert BaseAspect.aj

Other than Valeria's comment, I think this looks great!

Looks good to me.


Grid friendly position and rotation

I've ran into a problem with how pivot/origin of the transformation are handled. I'm trying to position various custom collision shapes (bodies) onto a grid and rotate them in place around the center. They all have same bounding box size (100×100), but different inner shapes, for rounded corners etc.
I understand that for the purposes of physics interaction it is natural to move around the center of mass (centroid), but for exact placement of various shapes on a grid, I need to devise a way to use center of the whole bounding box for these operations.
Here is a simple demo to illustrate this with two triangles: http://jsfiddle.net/Eskel/hxdzo0ud/6/
For my use case (and I hope I'm not alone in this), the bounding boxes of those triangles should still occupy the exact same space even while rotated 90/180/270 degrees. It also hinders exact positioning, since the shape itself displaces the origin of the position attribute.
Any idea how could I solve this? Advice on what to modify, or any temporary work-around would be most appreciated.

here is an example of the positioning issue http://jsfiddle.net/Eskel/cce3pxo6/1/

I've added a method Vertices.boundsCentre that solves it by finding the boundry box center for the Body.setVertices method. It seems to work fine for my needs. Should I make a pull request?
It's not pretty, but it's working well. Maybe someone can help me refine it.
    /**
     * Returns the centre of the set of vertices based on bounds.
     * @method boundsCentre
     * @param {vertices} vertices
     * @return {vector} The centre point
     */
    Vertices.boundsCentre = function(vertices) {
        var average = { x: 0, y: 0 };
        var minX = 0;
        var minY = 0;
        var maxX = 0;
        var maxY = 0;

        for (var i = 0; i < vertices.length; i++) {
            if (vertices[i].x < minX) {
                minX = vertices[i].x;
            }
            if (vertices[i].x > maxX) {
                maxX = vertices[i].x;
            }
            if (vertices[i].y < minY) {
                minY = vertices[i].y;
            }
            if (vertices[i].y > maxY) {
                maxY = vertices[i].y;
            }

            average.x = (maxX + minX) / 2;
            average.y = (maxY + minY) / 2;
        }
        return average;
    };

Just found out it doesn't work for concave shapes that are made of many smaller shapes.

After some hours of delving into the library, I managed to make it work with parts and simlified the function to this:
    Vertices.boundsCentre = function(vertices) {
        bounds = Bounds.create(vertices);
        var centre = Vector.add(bounds.min, bounds.max);
        centre = Vector.div(centre, 2);
        return centre;
    };

it has to be used in Body.setVertices, Bodies.fromVertices instead of Vertices.centre.
I'm gonna polish it some more and open a pull request.

I had the same problem. I think exact positioning of objects should be a very basic feature if you are building a game or a simulation of some kind. I used this approach https://github.com/acasaccia/matter-js/commit/6ff103103ddc0b4a7bda0499c783512971418598

Hi @acasaccia , I'm facing the same problem and I can't see the commit you've mentioned. How did you solve it?
The function proposed by EskelCz doesn't seem useful with concave objects and as far as i can see they are positioned using a point that is not the mean nor the center.
Thanks in advance!


build: doxygen rule is fragile

I can run "make doxygen" and it fails, then run "make doxygen" again and it passes. The issue is we can fail after the product is generated, so a second run will think the product is already there. The three last lines might fail. In my particular case I was failing the test of the logfile (something wrong generated warning in the log file).
In tools/build/Makefile.targets
$(DOXYGEN_GENERATED): $(DOXYGEN_RESOURCES) $(shell find $(top_srcdir)src/ -name '*.h' -type f)
	$(Q)$(MKDIR) -p $(build_doxygen_targetdir)
	$(Q)$(RM) $(build_doxygen_logfile)
	$(Q)$(RM) -rf $(build_doxygen_targetdir)/*
	$(Q) cat $(top_srcdir)doc/doxygen/examples_head.dox >> $(build_doxygen_targetdir)/examples.dox
	$(Q) find $(top_srcdir)src/samples -type f -name *.c -printf "@example %h/%f\n" | sort >> $(build_doxygen_targetdir)/examples.dox
	$(Q) cat $(top_srcdir)doc/doxygen/examples_foot.dox >> $(build_doxygen_targetdir)/examples.dox
	$(Q)$(DOXYGEN) doc/doxygen/Doxyfile
	$(Q) test `du $(build_doxygen_logfile) | cut -f1` -eq 0 || (echo "Failed to build doc" && cat $(build_doxygen_logfile) && exit 1)
	$(Q) cd $(build_doxygen_targetdir)/html/; for i in `find . -name "_2src*.html"`; do mv $$i $${i#./_2}; done
	$(Q) cd $(build_doxygen_targetdir)/html/; find . -name "*.html" -exec sed -i -e 's/_2src/src/g' {} \;

@cmarcelo I believe this empty file generated in the end making sure every rule succeed should be enough.
ptal when you have some time.
Thanks


Schema: containerEnv's type should add support for "boolean"

          @samruddhikhandale maybe a bit nitpicky but I would like to be able to use JSON booleans in `containerEnv`, validation is failing because only strings are allowed.

          ``` json
          {
              "containerEnv": {
                  "DOTNET_RUNNING_IN_CONTAINER": true
              }
          }
          ```

Originally posted by @sliekens in https://github.com/devcontainers/features/issues/628#issuecomment-1664581701

It feels like this is an unnecessary complication, it leaves it up to implementing tools to decide how to render a JSON boolean (unless I suppose the spec takes a hard stance on it). Generally speaking, tools will either use the presence of a certain environment variable to indicate "true" or a set of specific values (such as "1", "true", "yes", "on").
If I'm reading the original issue correctly, all that matters is that it's present with any value which seems entirely consistent with leaving the spec alone?
Is there a significant UX benefit to changing the spec? My concern is that another tool wants a specific value to indicate truthiness and now you have containerEnv entries with both true and "true" which seems to be more confusing (in a case where true renders as, say, "1").

I originally raised this because I expected to be able to use JSON types in JSON configuration.
Sorry, but having to write this feels really weird:
{
  "containerEnv": {
    "FLAG": "true"
  }
}

versus
{
  "containerEnv": {
    "FLAG": true
  }
}

This is what you intuitively write when you are typing JSON code so it's an annoyance that it doesn't work (crashes dev container cli).
I think boolean true should just be stringified and render the string "true".

I think boolean true should just be stringified and render the string "true".

Yeah, sure. I mean, I don't have a strong objection to it. It's more that the enclosing type is clearly about environment variables and environment variables must have string values, so any tool that accepts anything other than strings must take a stance on exactly how to render non-string values. It seems to me that the least surprising thing is to require users to specify strings.
Also, it doesn't look like the CLI crashes. It exits with a validation error. I do think it could use a more friendly error message though.

I originally raised this because I expected to be able to use JSON types in JSON configuration.

It's configuration though, and not code. The implementation decides what's valid and what isn't, no matter what tool you're using that consumes JSON.
--
Ultimately, I'm just someone building a (non JS) implementation of the spec and not an actual spec maintainer. My opinion is always going to be biased towards ease of implementation and in every case where a particular field which currently supports multiple types can be reduced to a single type I'm going to prefer that choice because it makes my implementation simpler :)

I just checked again and VSCode does give a fair warning that containerEnv expects strings in devcontainer.json (but not in devcontainer-feature.json).  And you are right, a friendlier error message instead of a technical error would not have caused me frustration. So perhaps adding a better validation message without relaxing the validation is where we can find middle ground.


CLI: Allow for list of lists

nargs and append are exclusive right now

@wilke how can we help you?

Missing use case.


Increasing more length to the name of the group

Hello django @django
This pull request only have one change for increasing more length to the name of the group from 80 to 100
This change is for support one name more long
This changed fixed https://github.com/WeblateOrg/weblate/issues/1549

Hi, I didn't follow the justification for this change. What's special about 100 characters compared to 80 characters?

@timgraham
I need created group like github.com:vauxoo-dev/runbot_branch_remote_name_grp_feature2 (fast-travis-oca)@Administration
The length of the string is 93, I think what 100 is a good number.

We don't make schema changes like this without much thought and justification. See https://groups.google.com/d/topic/django-developers/v3wYKWWYFT4/discussion for a past discussion. Feel free to start a thread on that mailing list but you'll need a stronger rationale than "I think 100 is a good number." I don't see anything in a name like that (GitHub repo URL) which would limit it to 100 characters. Personally, strings like that doesn't look like a group name to me, as far as how I've usually seen Django used. Maybe you should consider a separate table to store this data.
Closing for now until there's consensus to make this change.


check for required secret/configmap/psp before creating the csi driver resources

Describe the bug
When we apply cr file without creating required secret/configmap/psp , then also driver pods are getting created
How to Reproduce?
Please list the steps to help development teams reproduce the behavior

apply cr file with correct details without creating required secrets

Expected behavior
Operator should check for availability of required secret/configmap/psp , then only create driver pods

@nitishkumar4 @amdabhad Is this still valid ?

Yes, we still don't have checks for the existence of these resources.


Visual bug in the Lapce terminal when using starship

Lapce Version
0.2.3
System information
macOS 13.0.1
Describe the bug
I'm using starship in my terminal config which is also automatically used in Lapce. In the Lapce terminal, there is a visual bug though in combination with starship: The first key typed in the prompt gets doubled which makes it look strange. I made mini video to visualize the issue here.
It's just a visual thing as the commands I enter work fine otherwise.

@VolatileMark This fixes the issue I described above. But it shows a follow-up issue. After every key press the zsh auto-suggestions appear as they should, but this seems to also trigger the prompt to move on.

Just an update to the problem. I could narrow it down to alacritty being incompatible with the zsh plugin zsh-autocomplete. Conditionally not loading this plugin for alacritty in my .zshrc fixes my problems.

When it's impossible to type or paste export TERM=alacritty into a terminal, it might aid to have the shell run through env. The respective Lapce settings would look like the following, e.g. for ZSH:
[terminal]
shell = "env TERM=alacritty /usr/bin/zsh"


Slightly improve ergonomics of writing Cargo tests

As discussed in https://github.com/rust-lang/cargo/pull/5196#discussion_r175110557
I've also employed this to some of the longer command-lines in our test suite :)

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+
Nice!

:pushpin: Commit 54d9968 has been approved by alexcrichton

:hourglass: Testing commit 54d9968c847554afdd68ec944176569db8fd3a17 with merge e1d0926b2e51c898f0d0e6c54027a714ba77c29b...

:broken_heart: Test failed - status-travis

@bors: retry
On Fri, Mar 16, 2018 at 12:32 PM, bors<EMAIL_ADDRESS>wrote:

💔 Test failed - status-travis
https://travis-ci.org/rust-lang/cargo/builds/354424128?utm_source=github_status&utm_medium=notification
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang/cargo/pull/5197#issuecomment-373788032, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95AdkLnNqWyw28VbANxZWSv9vh2C-ks5te_cdgaJpZM4SuIdc
.

:hourglass: Testing commit 54d9968c847554afdd68ec944176569db8fd3a17 with merge 93475ff514f2fb0a6dbca4aae4aa876a531d5734...

:broken_heart: Test failed - status-travis

@bors r=alexcrichton
Funny enough, the tests passed locally and here on travis, but not on bors, due to that regression with .multiple(true). Should be fixed now!

:pushpin: Commit e5971b9 has been approved by alexcrichton

:hourglass: Testing commit e5971b935ca01b010bce3a74ef050ae43caef0ff with merge 545a4a2c930916cc9c3dc1716fb7a33299e4062b...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 545a4a2c930916cc9c3dc1716fb7a33299e4062b to master...


Merging optimizer states from different pipeline parallel size to resume training

Suppose you start training with a pipeline parallel size of 4. We need to make it supports resuming training with a different pipeline parallel size, like 2, by merging optimizer states.