Fixed Contributing Guide link (top link) within  /contributing/README.md

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://github.com/github/docs/blob/main/contributing/README.md
What part(s) of the article would you like to see updated?
The top link to the contributing guide is broken.
Additional information
No response

I noticed this is within the contributing directory which has been depreciated, not sure it is needed but wanted to fix the link aways. If it is not needed this can be closed.


Gracefully handle abbreviated revisions during import

dep in general frowns on / doesn't support abbreviated revisions. However during import, and when reading external config on-the-fly, dep shouldn't just give up. Instead we should lookup the full revision for the short revision, and use the full revision in the imported lock, etc.
I'm hoping that we can handle this in SourceMgr.InferConstraint, so that all the importers get this for "free", though we probably need plumbing in gps to do the dirty work.
Notes:

We already have SourceMgr.RevisionPresentIn to check if the revision is valid, and for the git implementation the underlying command used actually prints the full revision, but nothing exists right now to bubble that up.
Currently InferConstraint doesn't actually hit the repository to figure out if a rev is valid, it just matches a pattern. Since we actually validate tags/branches, it may be time to retire the guesswork, like doing length checks, and just reply on the source manager entirely to check if a string is a valid revision (and get the full revision if it was abbreviated).

yeah, we'll definitely need some new plumbing.
i think trying to work it in via InferConstraint() is a wise place to start. i'd rather we not have to split it out into its own thing, but if we do, it's not the end of the world.

I've spoken with @carolynvs about this and she says I can give it a try.

I bequeath upon ye the sacride golden star of approval ⭐

😟 Hope I didn't cut the line by submitting #1027 to fix this issue. Sorry for not checking in here first!

I think we all realized before much work was done. Since we can't easily assign issues in GH, it's a distributed game of Hungry Hungry Hippos. 😝  Sorry @markwest1!

No problem! I'm sure there's more work out there for me!


Kroma discovery updates: challenged roots

Resolves L2B-4364

test fails with coingecko api: premature close, but tests succeed locally


chore: update ethers v6

This PR updates the etheres v5 package to v6

The old ethers v5 package was dependant on old version of ws that suffered from: https://security.snyk.io/vuln/SNYK-JS-WS-7266574
Docs on migration from v5 to v6: https://docs.ethers.org/v6/migrating/
Because v6 is using native BigInt, and the same is with this package now, JS ES2020 is required

Hey, any news on this PR? This would also allow the package to properly support the sepolia testnet.


WIP - Feature/account controller redux

Proposed changes
Lots to digest here.

Move business logic into a pkg level controller
Abstract away the data layer and use Go implicit interfaces to work with the data layer
Simplify the data layer and how queries are built and executed
Use GO error wrapping and create an error handler to decide exit codes
Remove redundant Account structs and use one struct for json, dynamodb, and parsing api queries
Normalization of the config service via Nathan's work

Currently migrated portions of the code
Accounts POST, GET
This that will be deprecated via this work.

pkg/db
pkg/api/response/account, lease, usage, and a lot of helpers

Remaining work:

Create an event engine similar to the data engine.  Let that pkg decide where things go and when they go.
Finishing moving the logic for create/delete account once the event engine is created
Migrate leases/usage

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (changes to code, which do not change application behavior)

Checklist

[ ] I have filled out this PR template
[ ] I have read the CONTRIBUTING doc
[ ] I have added automated tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (README.md, inline comments, etc.)
[ ] I have updated the CHANGELOG.md under a ## next release, with a short summary of my changes

Relevant Links

Further comments

Closing in replacement of #195


[Doc Improvement][Next step updated from See also][3951421]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/design/plan-to-monetize.md
100
0
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/design/plan-to-monetize.md
100
0
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/deploy-and-publish/appsource/prepare/monetize-overview.md
100
1
link
:white_check_mark:

msteams-platform/concepts/design/plan-to-monetize.md
100
0
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Add NotifyFS protocol

I'm sure this is going to evolve, but this is just a starting point to get some sort of protocol available for desktop.
r? @maxtaco

:+1:


[idea] Add more quickstarts which use a local db

Which of these best describes you?

[X] dbt Community member
[ ] Partner
[ ] dbt Labs employee
[ ] Other

What's your idea for new content?
As a user, I would like at least one quickstart guide for using dbt with a local database. Ideally more than one. All the existing quickstart guides involve using a cloud service, which means signing up and providing a credit card. This is suboptimal for various reasons.
Currently, work seems to be ongoing for a DuckDB-based quickstart [#5783]. (I'd help, but I can't actually see the vercel error, and it's not immediately obvious if this is a permission issue, a didn't-install-a-github-app issue, or an I'm-not-being-smart-right-now issue). This would definitely be a step in the right direction. Also, when I originally complained on slack:
https://getdbt.slack.com/archives/C2JRTGLLS/p1733789906454009
my attention was directed to https://github.com/dbt-labs/jaffle_shop_duckdb, which is definitely useful, but also something I didn't find by looking at the beginner docs. The quickstarts in question should be something a user would reasonably expect to find when looking at beginner documentation (see recommendation below).
Ideally, however, there should be more than one option if that's not too difficult. For example, since Postgres is an official adapter for dbt, it makes sense to have a quickstart for a local Postgres instance. Also, it should ideally be clearly signposted that these options work for a local instance and are not just another variation of cloud services.
Where would you recommend this content live on the docs.getdbt.com?
On https://docs.getdbt.com/docs/get-started-dbt
In one or more little boxes under "dbt Core" analogous to the ones under "dbt Cloud" like

Quickstart for dbt Core and local DuckDB
Quickstart for dbt Core and local Postgresql DB
...

hey @dradetsky , thanks for opening this issue up and those are all great points! you're right that we're currently working through the duckdb quickstart and we appreciate your offer to help! the quickstart pr is something we're still working through and can reference your issue as we make progress.
in terms of beginner documentation, we can for sure add cards to represent the core quickstarts, which will be helpful as we add the duckdb one in the future. i'm happy to create a pr to address that in the short term as we continue to assess your feedback for local db.


Typing argument as union of string | Component leads to glint error

Hi Team, could you please assist me with understanding :)
I've composed an example repo: https://github.com/danieljust/glint-question-1
What I do:
I try to pass an argument into component, this argument can be a string or component.
I noticed that if I try to type argument as component only, then everything works just fine.
But if I use union string | SomeComponent Glint starts screaming at me.
interface ComponentASignature {
  Element: HTMLDivElement;
  Args: {
    onlyComponent: typeof ComponentAsArgument;
    componentUnionString: string | typeof ComponentAsArgument;
  }
}

This actually doesn't work if I use union with any other type mentioned in
export type ContentValue =
  | string
  | number
  | boolean
  | null
  | undefined
  | void
  | SafeString
  | Node
  | ArglessCurlyComponent;

What I see:
in attached repo in app/components/main-component.hbs you may see the following error:

This is a limitation of how TypeScript handles overloaded function signatures: a function with a signature that accepts A and a signature that accepts B doesn't automatically accept A | B.
See this playground for a (very simplified) example: https://www.typescriptlang.org/play?#code/C4TwDgpgBAwg9gWzHAdhFwoF4oG8oCWAggkfEqusAFxTABOArtAL4DcAUBwCYQDGAGwCG9aADNGKPsAKooolL3rlkaDAAoUQhBFoBnBgRQBzAJS0AbnALdOvQSPGTpslPPRKVlDX0SqqtF5qwOZQVjacXPbColC+KAZQABIQAgJwgX7ewHb8MdDxiQAWqekAQiAActoQAPL0AGpCAsz6hiZQAD504BBwYsmlcJEKnlnB6gDkJWlwk6acoxDK41TqKbMLHEsrFBMz5VU19U0tEAtAA


Stackoverflow in Lazy Loading

Accessing navigation property inside of another property leads to StackOverflowException.
Inside of my Subproject Entity I need to access the No from the Project Entity which holds a collection of Subprojects. When I request the sub project from the db and want to access the FullSubprojectNo from the sub project the line if (Project != null) leads to a stack overflow exception.
When I deactivate LazyLoading everything is working as expected.
Exception details from VS 2017
System.StackOverflowException
  HResult=0x800703E9
  Source=<Cannot evaluate the exception source>
  StackTrace:
<Cannot evaluate the exception stack trace>

Steps to reproduce
Run the UnitTests
The first test GetProjectNo_LazyloadedSubprojectWithNewContext_ThrowsStackOverflow produces the StackOverflow exception.
The test GetProjectNo_LazyLoadedSubprojectWithSameContext_ReturnProjectNo succeeds when using a second instance of the Context
Full Gist
Further technical details
EF Core version: 2.1.1
EF Core InMemory Version: 2.1.1
EF Core Proxies Version: 2.1.1
Database Provider: InMemory
Operating system: Windows 10
IDE: Visual Studio 2017 15.8.1

Notes for triage:

Stack overflow is essentially a duplicate of #12451, since it happens when Load triggers DetectChanges
However, even with DetectChanges turned off, there is still an exception because a second query is starting before the first one ends

@ChristophvdF Have you consider making the computed property not-mapped? This is a more common pattern for this type of thing.

@ajcvickers Sadly that isn't possible since we need the property persisted in the database because there are other systems which need to access the value as well.

@ChristophvdF I meant something like this:
public class Project
{
    public int Id { get; set; }
    public string No { get; set; }
    public virtual ICollection<Subproject> Subprojects { get; set; }
}

public class Subproject
{
    public int Id { get; set; }
    public int ProjectId { get; set; }
    public virtual Project Project { get; set; }

    private string PersistedFullSubprojectNo { get; set; }

    [NotMapped]
    public string FullSubprojectNo
    {
        get
        {
            // requests the project number from the project entity, to ensure correct number.
            if (Project != null)
            {
                PersistedFullSubprojectNo = Project.No;
            }

            return PersistedFullSubprojectNo;
        }
        set => PersistedFullSubprojectNo = value;
    }

}

public class BloggingContext : DbContext
{
    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        => optionsBuilder
            .UseSqlServer(@"Server=(localdb)\mssqllocaldb;Database=Test;ConnectRetryCount=0")
            .UseLazyLoadingProxies();

    protected override void OnModelCreating(ModelBuilder builder)
    {
        builder.Entity<Subproject>().Property("PersistedFullSubprojectNo").HasColumnName("FullSubprojectNo");
    }

    public DbSet<Project> Projects { get; set; }
    public DbSet<Subproject> Subprojects { get; set; }
}

Although as I was writing this I realized you could just force EF to access the backing field directly for the same result:
public class Project
{
    public int Id { get; set; }
    public string No { get; set; }
    public virtual ICollection<Subproject> Subprojects { get; set; }
}

public class Subproject
{
    private string _fullSubprojectNo;
    public int Id { get; set; }
    public int ProjectId { get; set; }
    public virtual Project Project { get; set; }

    public string FullSubprojectNo
    {
        get
        {
            // requests the project number from the project entity, to ensure correct number.
            if (Project != null)
            {
                _fullSubprojectNo = Project.No;
            }

            return _fullSubprojectNo;
        }
        set => _fullSubprojectNo = value;
    }

}

public class BloggingContext : DbContext
{
    private static readonly LoggerFactory Logger
        = new LoggerFactory(new[] {new ConsoleLoggerProvider((_, __) => true, true)});

    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        => optionsBuilder
            .UseSqlServer(@"Server=(localdb)\mssqllocaldb;Database=Test;ConnectRetryCount=0")
            .UseLazyLoadingProxies();

    protected override void OnModelCreating(ModelBuilder builder)
    {
        builder.Entity<Subproject>()
            .Property(e => e.FullSubprojectNo)
            .UsePropertyAccessMode(PropertyAccessMode.Field);
    }

    public DbSet<Project> Projects { get; set; }
    public DbSet<Subproject> Subprojects { get; set; }
}

@ajcvickers I just tested your solution. In my small test project the changes worked, no exception and correct result.
I still can't really wrap my head around why the loop doesn't occur when set to use field only.


move gitlab API to v4

To fix: https://github.com/rancher/pipeline/issues/48
it's time to move to gitlab v4 API, as v3 is deprecated now.

LGTM


⚠ Bofrost.lu has degraded performance

In 43d921c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2231 ms

Resolved: Bofrost.lu performance has improved in d793a3a after 13 minutes.


RuntimeError: Jacobian mismatch for output 0 with respect to input 1

Does anyone have same issue?  How to fix it?

I have the same issue, anyone can help?

terrible! same problem. No solution?

this is not a critical issue.  anaybody see this, can ignore it. In my experiments, if you run the code many times, this issue arises in only some cases.


dynamic framerate

Is it possible to change framerate on running pipeline ?
In python, with other parameters  like exposure time, I use:
# pipeline create
pylon = Gst.ElementFactory.make("pylonsrc", "pylon1")

# pipeline runtime
cam = pylon.get_child_by_name("cam")
cam.set_property("ExposureTime", 4000)

there is camera parameter that allows to control the cameras speed.
https://docs.baslerweb.com/acquisition-frame-rate


Can't run pacmd from VS Code terminal

VSCode Version: 1.30.2
OS Version: Ubuntu 18.04.1

Steps to Reproduce:

Run pacmd list-cards in the VS Code terminal.

Does this issue occur when all extensions are disabled?: Yes

pacmd list-cards produces the output No PulseAudio daemon running, or not running as session daemon. when run in the VS Code terminal. From a normal terminal, it produces the usual output. The issue here of course isn't pacmd specifically, it's that the VS Code terminal doesn't behave like an ordinary terminal window, which it should.

I run into this issue and I think there's something to do with snap
With a snap package of vscode, I cannot access pulseaudio daemon, but with the vscode, installed with .deb package pactl and pacmd works fine

I've built minimal snap for testing and got the same issue:
name: min-snap
base: core18
version: '0.1'
summary: blah
description: blah-blah

grade: stable
confinement: classic

parts:
  dummy:
    plugin: nil

apps:
  sh:
    command: sh pactl stat

➜ /snap/bin/min-snap.sh -c "/usr/bin/pactl stat"
Connection failure: Connection refused
pa_context_connect() failed: Connection refused

➜ /snap/bin/min-snap.sh -c "/usr/bin/pacmd list-cards"
No PulseAudio daemon running, or not running as session daemon.

So it's maybe not a vscode issue at all

Please report this to the snap team.

I've built minimal snap for testing and got the same issue:
name: min-snap
base: core18
version: '0.1'
summary: blah
description: blah-blah

grade: stable
confinement: classic

parts:
  dummy:
    plugin: nil

apps:
  sh:
    command: sh

➜ /snap/bin/min-snap.sh -c "/usr/bin/pactl stat"
Connection failure: Connection refused
pa_context_connect() failed: Connection refused

➜ /snap/bin/min-snap.sh -c "/usr/bin/pacmd list-cards"
No PulseAudio daemon running, or not running as session daemon.

So it's maybe not a vscode issue at all

Reinstalling VS Code from .deb package instead of snap worked for me. Ubuntu 20.04.


[Sprint: 44] XD-2719: More Rabbit Job Cleanup

JIRA: https://jira.spring.io/browse/XD-2719
Partitioned jobs add another queue
xdbus.job:<jobName>.requests
Previously, this had an incorrect name (the job: part was omitted).
The .replies queue is auto-deleted.
Clean up this additional queue if it's present.

Rebased and merged as 2d608bda99d8b795deff9770593c6f03ec8750e6


Update project TOCs

Reviewers: here are links to see what this will look like in production:

WPF
Android
Forms
UWP
iOS

@mbranscomb I think the text 'working with addressing' for the Search>Find Address sample is awkward and should be replaced with 'working with addresses'. Do you think that can go in this PR?