text:
Attempt to fix Promote Transaction issue

Fixes #237
After more investigations we found that this issue occurs due to SqlClient's inability to handle "Transaction Aborted" callback in SqlDelegatedTransaction. When transaction promotion is triggered by WCF Service after first set of commands, the transaction is also requested to be aborted due to exception scenarios from SysTx.Transactions.
The driver receives callback when transaction gets aborted, but it sends this connection back to the pool, without updating its "Current Transaction" state, which gets picked up later by other new connection requests. In the second round of transaction scope, it does not respect new transaction scope created for the new connection request because there was no "currentTransaction" expected.
Hence the Debug.Assert failed all the time.
Moving on with this "old" transaction, which is aborted externally, we also notice warning from SQL Server as it sends "TransactionEnded" token followed by "Begin" to initiate new transaction request, Everytime! as explained here: https://github.com/dotnet/SqlClient/issues/237#issuecomment-569965788
Most of the times we get lucky, that the transaction was aborted by SQL Server, before we tried to use it again and it always gave us new transaction to work with. But in 1% scenarios, where the error happened, server got a little late to end this transaction and we end up enlisting our connection for 2nd Transaction Scope in the 1st transaction. But an abort is pending on server side, so as soon as it aborts the transaction, all further requests to server are free of the 1st "TransactionScope", 2nd TransactionScope had no links to this round of queries, and queries get executed with "Implicit" transactions, hence the "Commits".
Everything is linked to the fact that this connection was sent back to "GeneralPool" without cleaning it's transaction state where only connections WITHOUT any "Active" transaction reference must reside.
Since this connection has enlisted transaction that got promoted to DTC but was recently aborted externally, safest approach in my opinion is to Doom this connection as soon as we capture the aborted state of current transaction to block sending back to the pool.
I couldn't find any other clean way to reset this connection properly in order to send it back to General Pool. Would welcome suggestions if that can achieved!
Please give the below package a try, that contains fix:
Microsoft.Data.SqlClient.2.0.0-dev-437.nupkg.zip
cc @saurabh500 @David-Engel @scale-tone

This must have been a nightmare to trace.
When you say that "The driver receives callback when transaction gets aborted" do you mean that TransactionEnded is called? If so isn't it possible that from there you could call connection.CleanupConnectionOnTransactionCompletion here as is done in the Rollback implementation?
It might also be worth adding some checks in the pool return function for various pieces of dangling state and dooming the connection at the return to pool stage. If done this way any other errors from the inappropriate release could be forced safe without finding all possible caller combinations and we could have have it assert in tests.

Nice to see progress on this (think it's safe to say) critical issue, @cheenamalhotra . Still we don't seem to have full understanding of the mechanics.
If you go up in the stack, you'll see that another attempt to properly cleanup that connection before returning to the pool is being made:
            CleanupConnectionOnTransactionCompletion(transaction);

Do you have an idea, why that attempt isn't successful? If that line effectively does nothing, then why is it in place after all?
I can confirm that with the proposed DoomThisConnection()  line added the issue stops happening. But ObjectDisposedException still occurs. Why is that?
@Wraith2 , CleanupConnectionOnTransactionCompletion() doesn't help. With that line added the issue still occurs (can't explain, why).

@scale-tone
I looked up more into the CleanupConnectionOnTransactionCompletion method and it does not cleanup completely the "CurrentTransaction" associated with "SqlInternalConnectionTds" object instance. I was able to modify the code to do so, and could clean first connection's transaction reference to be transferred back to "General Pool" safely.
However, doing so does not work well in multi-threaded scenarios, and seems like there's more to this problem than we're looking at from face-value if we try to set things right here.
Alternatively, I'm also going to investigate making this PR more robust to avoid any possible thread-safety issues so we can have one solution to address this problem, while the other can still be investigated, where we can allow re-use of connection.

If that line effectively does nothing, then why is it in place after all?

This is effective to remove the connection from Transaction Pool, but we definitely need to groom it to do only necessary steps in this case.

But ObjectDisposedException still occurs.

I will come back on this, after I sanitize the code to do only required tasks on "Abort". It seems to be related in initial glance. Could you also share your stacktrace to confirm I'm also looking at the same error.

@scale-tone
I did manage to fix the problem with ObjectDisposedException as it was thrown due to race condition scenarios between Rollback and Abort. I did not find the need to modify code any further, and I also verified with DB Performance Pool Counters we don't have any leaks, the connections are getting cleaned up as expected.
However, there may be cases where connection takes time to connect and may lead to timeout if running too many parallel executions, increasing connection timeout is a viable solution in those scenarios.
The alternate solution of cleaning the connection state and sending it back to the General pool needs deep investigations as that leads to big design changes in the driver. I will get back with progress on that one, but also would request you verify this option as solution.
Uploading updated NuGet package here for testing:
Microsoft.Data.SqlClient.2.0.0-dev-pr495-1.zip

@cheenamalhotra Question about the approach: Is this approach of dooming the connection on ending transaction final?
There has to be a different approach considered, where if the transaction is not ended, then we wait for it to end, and possibly not return this connection to the pool till its related transaction has ended.
The DTC and potentially not DTC customers using SqlTransaction will hit a performance problem while retrieving connections from the pool. The change effectively disables pooling for all transacted connections.

Hi @saurabh500
I am investigating that option too, but wanted to make 1 solution a complete fix that can be considered as a solution if we don't find a working solution with alternate solution as it's more important to fix the problem than to worry about performance which is a very edge case scenario. Also this is not going to completely stop Pooling on DTC Transactions. We will only doom the connection if it has been Aborted externally.
In normal Commit/Rollback cases, we will continue to pool the connection as always.
The alternate solution is a can of worms which is more tricky due to the design of "Transaction Pool" that differs from "General Pool". We may have to do a lot of changes to achieve the right solution we're looking for. It's in progress, but is more time consuming too. If it gets too complex, we may have to take a call if it's worth the effort.

I see, I agree that the perf problem will hit in case of a DTC transaction abort. Sure.
Here's how I am looking at this situation, which makes performance very important in failure cases.
An application is using DTC to coordinate transactions between SQL Server and another system i.e. a WCF service in this case.
Let's say that the transactions start aborting due to WCF service throwing errors due to some issues (networking / bad configuration deployment / something else not working, failover testing). All the connection participating in those transactions will be doomed and new connections will need to be created the next time a sql interaction is needed. This will overload the connection pool causing timeouts in the connection pool in scenarios where some other part of the service is experiencing problems. So the problem will shift from WCF throwing exceptions to application receiving SqlException during transient errors in some other part of the application.
The connection pool is single threaded for each pool and it will have a hard time providing connections in case of a WCF service or another component participating in DTC, experiencing short durations of outages.
Couple this with the connection pool blocking period where the applications will start seeing a lot of timeouts while trying to get connections from the pool.
These edge case perf issues will cause outages for the applications and investigations about why a call to WCF failed will then become why is SqlClient timing out in case of failures in some other parts of the system even though the root cause of WCF outage was fixed.

@cheenamalhotra my comment above was not to dissuade fixing this issue in steps. It was something I wanted to list down before I lost my train of thought.
I do agree that this fix will at guarantee correctness which is high priority item in the light of this bug you have been working on.
By the end of completing this investigation, if you could open an issue to list a performance impact and possible solutions to address the edge case perf case, that would be nice.
At least folks hitting any issues can search the internet and come to github to find out what is the problem and the related discussion. And if this perf problem becomes a real issue later down the line, a developer looking at this issue will have context since investigating this problem as a perf issue later is going to be harder than debugging this issue itself.
Since you have rich context around this issue, I think we should definitely list down your findings in another issue as perf issue, and the potential approaches you have tried and why they failed.

Could you also share your stacktrace to confirm I'm also looking at the same error.

It is clearly stated in the bug:

Opening a new PR with same branch as CI doesn't seem to trigger here.


text:
Fix semantic error in THSTensor.cpp

While NULL is defined as 0 which is a valid default value for the float, the default value should be 0.0f
This fixes the compiler warning:
TorchSharp/src/Native/LibTorchSharp/THSTensor.cpp(389,5): error G13A67890: converting to non-pointer type ‘float’ from NULL [-Werror=conversion-null] [TorchSharp/src/Native/build.proj]
    389 |     CATCH_RETURN(float, NULL, (float)(tensor->data_ptr<c10::Half>())[i]);
        |     ^~~~~~~~~~~~

There are some other uses of NULL which generally are replaced with nullptr since c++11:
LibTorchSharp/THSNN.cpp:        if (outAsAnyModule != NULL)
LibTorchSharp/THSNN.cpp:        if (outAsAnyModule != NULL)
LibTorchSharp/THSModule.cpp:    if (outAsAnyModule != NULL)
LibTorchSharp/Utils.cpp:thread_local char * torch_last_err = NULL;
LibTorchSharp/THSTensor.cpp:        res = grad.defined() ? new torch::Tensor(grad) : NULL;
LibTorchSharp/THSTensor.cpp:            auto end = (indexEnds == NULL || indexEnds[i] == INT64_MIN) ? c10::optionalc10::SymInt() : c10::optionalc10::SymInt(indexEnds[i]);
LibTorchSharp/THSTensor.cpp:            auto step = (indexSteps == NULL || indexSteps[i] == INT64_MIN) ? c10::optionalc10::SymInt() : c10::optionalc10::SymInt(indexSteps[i]);
LibTorchSharp/THSTensor.cpp:    CATCH_RETURN(Scalar, NULL, new torch::Scalar(tensor->item()));
LibTorchSharp/THSJIT.cpp:        return NULL;
LibTorchSharp/Utils.h:#define CATCH_RETURN_NNModule(stmt) CATCH_RETURN_RES(NNModule, NULL, stmt)
LibTorchSharp/Utils.h:#define CATCH_RETURN_Tensor(stmt) CATCH_RETURN_RES(Tensor, NULL, stmt)
LibTorchSharp/Utils.h:// Return undefined tensors as NULL to C#
LibTorchSharp/Utils.h:        return NULL;
LibTorchSharp/Utils.h:    if (outAsAnyModule != NULL)
LibTorchSharp/Utils.h:    if (outAsAnyModule != NULL)
LibTorchSharp/THSLoss.cpp:        if (weight != NULL)
LibTorchSharp/THSLoss.cpp:        if (weight != NULL)
LibTorchSharp/THSLoss.cpp:        if (weight != NULL)
LibTorchSharp/THSLoss.cpp:        if (weight != NULL)
LibTorchSharp/THSLoss.cpp:        if (weight != NULL)
These might be good to clean up in this PR.

source: gharchive/pull-request

repo: dotnet/TorchSharp

license: MIT

license_type: permissive

license_source: github-api


text:
With* methods don't work with containers

I was testing a project converter to container locally and I realized that it doesn't work because we pass the localhost address instead of using the host address (host.docker.internal). Localhost is local to this container so this always blows up.
using Aspire.Hosting.Otlp;
using Aspire.Hosting.Redis;

var builder = DistributedApplication.CreateBuilder(args);

var pubsub = builder.AddRedisContainer("pubsub");

// Projects
//builder.AddProject<RedisPubSub.App.Projects.Publisher>()
//    .WithRedis(pubsub);

//builder.AddProject<RedisPubSub.App.Projects.Subscriber>()
//    .WithRedis(pubsub);

// Project published as containers
builder.AddContainer("publisher", "publisher")
    .WithRedis(pubsub)
    .WithOtlpExporter();

builder.AddContainer("subscriber", "subscriber")
    .WithRedis(pubsub)
    .WithOtlpExporter();

builder.Build().Run();

The easiest fix here would be to always use host.docker.internal instead of localhost for anything that runs in a container. Injecting different environment based on the consumer might be a bit more difficult...
cc @karolz-ms

This is going to be a pain. It appears that host.docker.internal is not enabled by default on Linux hosts. So we would need to add platform specific logic here.

related: https://github.com/moby/moby/pull/40007

@karolz-ms for DCP on Linux we might need to invoke with --add-host=host.docker.internal:host-gateway when spinning up a container with Docker.

@adityamandaleeka is @BrennanConroy planning to pick up app model work? David suggests this might be a good one to start with. Preview 2 cc is Tues 12/5

We believe DCP work is done.

I do not think so/don't recall us adding this option on DCP side

This wasn't the one @danmoseley , it was the command line args for containers 😄

argh -- thanks for the correction. don't mind me, just dropping in and causing chaos in the issues

Mitigation in place https://github.com/dotnet/aspire/pull/1203

This may end up being the long-term solution since we're going through proxies here. I'm going to close this as resolved for now and we can track another networking support in containers here

source: gharchive/issue

authors: danmoseley davidfowl karolz-ms mitchdenny

repo: dotnet/aspire


text:
Using own PrimaryEndpointName

Some builder extensions reference MySqlServerResource.PrimaryEndpointName instead of their own.
Microsoft Reviewers: Open in CodeFlow

I'm kinda new at contributing to projects like this.
Is splitting these small changes into multiple pr-s annoying or wanted?
#2657 #2656 #2655

Is splitting these small changes into multiple prs annoying or wanted?

wanted.


text:
Tag Helper is generating the empty link

I am trying creating link using tag helper but that is creating the empty link. I have included Tag helper reference as well. PFA the code file:
TagHelper.txt

Hi @Sourabh721992 could you please specify in more detail what you're doing and what you're trying to achieve? Could you share a minimal reproduction (source code) of the problem so we can help.

Hi,
I am using a tag anchor link to go from view, Index.cshtml to view, Details.cshtml. Both views are linked to HomeController in the solution. But what is happening is tag helper is creating empty link for me. I am attaching all the files.
Index.txt
Details.txt
_ViewImports.txt
HomeController.txt

Are you following any example or documentation?
https://docs.microsoft.com/en-us/aspnet/core/mvc/views/tag-helpers/intro?view=aspnetcore-5.0 might help you, there are also some samples here: https://github.com/dpaquette/TagHelperSamples
Unfortunately, this is beyond my knowledge. @Pilchie is this something you can help with?

Yes I have followed the documentation but I am not sure why the link is href is not getting generated.

repo: dotnet/core


text:
.NET Core 2.0 Preview 2 won't install in Ubuntu 16.04.2

Thanks for your interest in .NET Core.
Please note that this repo (dotnet/core) is not for filing product issues. Here are some repos where you can file an issue:

dotnet/cli - for CLI tools and questions
dotnet/corefx - for API issues and questions
dotnet/coreclr - for runtime issues
nuget/home - for NuGet questions and issues

I have follow both installation instructions in GitHub and the .NET Core Download page.  I'm getting the same error message:
Unpacking dotnet-sdk-2.0.0-preview2-006497 (2.0.0-preview2-006497-1) ...
dpkg: error processing archive /var/cache/apt/archives/dotnet-sdk-2.0.0-preview2-006497_2.0.0-preview2-006497-1_amd64.deb (--unpack):
 trying to overwrite '/usr/share/dotnet/store/x64/netcoreapp2.0/microsoft.identitymodel.protocols/2.1.3/lib/netstandard1.4/Microsoft.IdentityModel.Protocols.dll', which is also in package dotnet-dev-2.0.0-preview1-005977 2.0.0-preview1-005977-1
dpkg-deb: error: subprocess paste was killed by signal (Broken pipe)
Processing triggers for man-db (2.7.5-1) ...
Errors were encountered while processing:
 /var/cache/apt/archives/dotnet-sdk-2.0.0-preview2-006497_2.0.0-preview2-006497-1_amd64.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)

Any suggestions!

If you remove Preview1, perhaps it will get rid of the problem? dotnet-dev-2.0.0-preview1-005977
/cc @livarcocc it looks like we're installing the same thing to the same location and getting a conflict.

Thanks @Petermarcu!  I'll give it a try and see if uninstalling the previous version will clear the issue.
:)

Thanks @MaximoTrinidad let us know.

HURRAY!!! @Petermarcu
Someone need to add this additional step in the instruction:

remove previous version:

sudo apt remove dotnet-dev-2.0.0-preview1-005977

Then, install the latest one:

sudo apt-get install dotnet-sdk-2.0.0-preview2-006497

I got it loaded now.

Thanks for the tip!

Glad to hear it! I've copied this issue to the dotnet/cli repo because you definitely shouldnt have had to uninstall preview1. I'm going to close this issue here now. Thanks for reporting it!

@richlander @leecow can we get a note on the instructions that Preview1 of the sdk needs to be uninstalled?

Notes added to the download page, known issues doc and made the requirement more obvious on the Preview getting started page.

@Petermarcu!!  Awesome!  Thanks again.
:)

authors: MaximoTrinidad Petermarcu leecow


text:
[ILVerify] Implement verification of return instruction

I implemented the verification of the return instruction and added relevant unit tests.
This also prompted the implementation of the IsPermanentHome flag for ByRefs and the according checks / state initializations in other instructions.
I also added the trackObjCtor state flag, which will be used for tracking the this-pointer state and added references to the relevant PEVerify code parts. I will be implementing the actual tracking of the this-pointer in a separate PR.
The verification of the return instruction resulted in 7 existing tests failing mostly because they did not clear the evaluation stack before returning. I fixed those tests.

@ArztSamuel,
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

authors: ArztSamuel dnfclas

repo: dotnet/corert


text:
Not for a review - Kirupap fixing h2s 1

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Starting with a new re-forked repo

repo: dotnet/docs-desktop


text:
Usage section missing from API reference

I have noticed that usage sections don't seem to be rendering in API reference.
For example, the source for CompareInfo.LastIndexOf(string, string) contains <block subset="none" type="usage">, which is not shown on its API reference page.
On MSDN, this section seems to render as a "Notes to Callers" section.

The same seems to apply to overrides sections too. For DbDataAdapter, the "Notes to Inheritors" is present on MSDN and in source, but not on docs.ms.

@svick, thanks for opening this issue.
In addition to "Notes to Callers" and "Notes to Inheritors", "Notes to Implementers" sections (for example, at IComparable<T>.CompareTo(T) on MSDN are not rendering.
We've opened two internal bugs, #963599 and #963655, to track this.

Both the internal bugs reference above are completed, so I'm closing this.

authors: BillWagner rpetrusha svick

repo: dotnet/docs


text:
Are there cases where removing a parameter default value would be acceptable?

RE:

❌ DISALLOWED: Changing the default value of a property, field, or parameter

Consider the following scenario:
public void Foo(int a = 1, int b = 2) { ... }

Let's say we want to add a new overload of Foo that also accepts int c. The following change should (in my understanding) be both binary-compatible and source-compatible:
public void Foo(int a, int b) => Foo(a, b, 3);
public void Foo(int a = 1, int b = 2, int c = 3) { ... }

(1) Would this be allowed in Microsoft's .NET libraries/runtime?
(2) If it is true that this change does not break either binary or source compatibility, could we add a note about that (I'm happy to file a PR) to this rule for the benefit of those who reference this document to understand breaking changes for .NET generally?

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

ID: c4b482b0-6dd1-42de-5389-5b0b47823402
Version Independent ID: 79a4adc9-b2c9-68b7-669e-2bd8d2d44faf
Content: Types of breaking changes - .NET
Content Source: docs/core/compatibility/index.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

@stephentoub Could you answer this question or suggest someone that can?

cc: @terrajobst, @bartonjs

That change would be allowed, yes.  In a very technical sense it has removed a default from a parameter, but at the same time it didn't :smile:.
Relevant things from Framework Design Guidelines, 3rd Edition:

5.1 Member Overloading
...
DO NOT change the default value for a parameter once it has been publicly released.
DO move all default parameters to the new, longer overload when adding optional parameters to an existing method.

D.11.17 Changing a Parameter from Optional to Required
...
The .NET BCL team considers this an acceptable breaking change when done as part of “moving” default values to a new overload. Otherwise, it is an unacceptable breaking change.

So, the terminology used in the 3rd edition update for this pattern is "move" (vs remove).  But a clarifying sub-bullet wouldn't hurt.

@madelson
Another way to think about this: removing a default parameter is a source breaking change, not a runtime breaking change as the compiler has "burned" the default values into the call sites (the runtime has no notion of default values). By adding an overload with corresponding defaults you addressed the source breaking change which is why it's OK.

authors: bartonjs gewarren madelson stephentoub terrajobst


text:
Error CS0470

Error example please.

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

ID: e755f4f6-b578-d3c9-10b3-c917fe1f741a
Version Independent ID: bff14002-14f0-9f7d-ad78-c4557dc00e61
Content: Compiler Error CS0470
Content Source: docs/csharp/misc/cs0470.md
Product: dotnet-csharp
Technology: csharp-diagnostics
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@BurAndBY Can you add more details to what you're asking for here? This article does have an example that shows the error, and I want to make sure I address your concern properly.
Thanks.

@BurAndBY Can you add more details to what you're asking for here? This article does have an example that shows the error, and I want to make sure I address your concern properly.
Thanks.

Yes, it does not have an error.

@BurAndBY I am still confused. I tried the exact sample shown on this page, and it produces the same compile error shown in the page, on the same line. What am I missing?

@BurAndBY I am still confused. I tried the exact sample shown on this page, and it produces the same compile error shown in the page, on the same line. What am I missing?

OK, a misunderstanding probably happened. I meant something like "error CS1520: Class, struct, or interface method must have a return type"

authors: BillWagner BurAndBY


text:
Highlight the runnable nature of samples in API Ref

In the DateTime reference, highlight the runnable nature of the C# samples:

[ ] Change the intro wording to add "runnable" in the opening.
[ ] Consider adding a quick sample above the fold.

Note that the second task would be jarring without a VB interactive experience.
/cc @LadyNaggaga

This is really cool! 😄 Can't wait for this to become more widespread - would love to make use of it for the NUnit Docs at some point too.
I presume the eventual plan is for this to be rolled out to all the docs? Is there a central issue tracking that, or is it all being handled internally?

DateTime reference already has a boilerplate with the explanation of the Run button.

authors: BillWagner ChrisMaddock mairaw


text:
Migration guide for Microsoft.Extensions.Http.Polly

The Microsoft resilience documentation claims that Microsoft.Extensions.Http.Polly is superseded by Microsoft.Extensions.Http.Resilience. Is this true? Would be nice if there were a migration guide.

@timheuer is this something that the Upgrade Assistance is capable of migrating?

In theory yes! UA basically has project-based (e.g., move this whole thing from this version to this version) or feature-based (move this 'thing' in my app from X-Y -- like what we've been doing with Aspire).  This I think would be a feature-based thing.

authors: IEvangelist kmcclellan timheuer


text:
Add detail in HttpClient documentation

Add detail in HttpClient documentation
See PR #4330 for background information.
Issues with Existing Topics
The HttpClient documentation needs to be updated to address issues that are not discussed or differences between .NET implementations. These include:

The connection limit, on a per-implementation basis.
Issues related to using a single instance of HttpClient
HttpClient and the DNS refresh problem
Guidance on when to instantiate a new HttpClient instance

Suggest reviewers
@feherzsolt @davidsh @karelz

Can you set milestone and priority @rpetrusha? Thanks.

authors: mairaw rpetrusha


text:
mixin -> mix in

Fixes #15727

@Youssef1313 "mixin" looks to be correct:
https://en.wikipedia.org/wiki/Mixin
https://docs.scala-lang.org/tour/mixin-class-composition.html

@Youssef1313 "mixin" looks to be correct:
https://en.wikipedia.org/wiki/Mixin
https://docs.scala-lang.org/tour/mixin-class-composition.html

Thanks for pointing this. Can you have a look on the current change ?


text:
Add missing word and space in RC4 CLI doc

Summary
Within the Moving away from project.json paragraph, start a sentence with "The" and add a missing space.

Hi @scottaddie, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Thanks, @scottaddie, for submitting the correction!

authors: dnfclas rpetrusha scottaddie


text:
Update instructions for RHEL

Summary
Fix package asp.net core RHEL7 and add links to official docs
Fixes #16030
Contributes to #15948

I think @mairaw is your assigned reviewer.

authors: Thraka gewarren


text:
.NET Core/5+ naming

Fixes #22553

@tdykstra Looks like this introduced a couple warnings.

@tdykstra Looks like this introduced a couple warnings.

authors: gewarren tdykstra


text:
Update struct.md

Readonly static members are possible. I may have not exhausted the list of members excluded from being applicable to both readonly and static (those that can modify the structure instance), so further edits may be necessary.
Summary
Fixes unclear and misleading information regarding the usage of static readonly members.

Hi 👋 @chessboards
That statement on disallowing readonly should stay. It's meant to cover this example:
struct S
{
    // CS 8657: Can't apply "readonly" to a static method:
    public static readonly void M()
    {

    }
}

How did you interpret that statement? Would a different wording make that more clear?

I think the wording could be changed to make the distinction clearer between static members that should not interact with the structure instance or state (methods, events, props with setters), and those that have no state changing logic (therefore can be made readonly). The wording of members led me to include static fields and get-only properties as also being excluded, which they are not. https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/members

thanks again @chessboards
I've set this to auto merge as soon as the builds finish.  You should see it live tomorrow, on our regular publishing cycle.
Thanks again!

authors: BillWagner chessboards


text:
Fix incorrect endpoint

Summary
The incorrect endpoint is provided in step 2 of "Verify the service is working" - propose to either change the endpoint there, or update on line 13 in Program.cs.
Fixes #Issue_Number (if available)

Internal previews

📄 File
🔗 Preview link

docs/framework/wcf/how-to-host-and-run-a-basic-wcf-service.md
Tutorial: Host and run a basic Windows Communication Foundation service

closing and reopening for a fresh build

authors: BillWagner tonesoutherland


text:
Increase token expiration and retry until compatible token

Summary

Increase the token expiration for Helix runs. The test job allows Helix up to 2 hours since it can take some time to wait in the Helix queues before tests are run. Don't want the token to expire before the tests get a chance to run.
Update the get-delegation-sas yaml to retry getting a token if it contains a forward slash. Tokens with forward slashes seem to trip up Helix and cause it to not download the correlation payload. Similar workarounds have been employed elsewhere: https://github.com/dotnet/aspnetcore/blob/release/6.0/eng/common/templates-official/steps/get-delegation-sas.yml#L34

Validation build: https://dev.azure.com/dnceng/internal/_build/results?buildId=2528027&view=results

Release Notes Entry

/backport to main

/backport to release/8.0

/backport to release/8.0

repo: dotnet/dotnet-monitor


text:
Error launch New interactive NoteBook

Describe the bug
Please provide as much information as you can.
Please complete the following:
Which version of .NET Interactive are you using? (In a notebook, run the #!about magic command. ):

OS

[ ] Windows 11
[X] Windows 10
[ ] macOS
[ ] Linux (Please specify distro)
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Edge
[ ] Firefox
[ ] Safari

Frontend

[ ] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[X] Visual Studio Code
[ ] Visual Studio Code Insiders
[ ] Visual Studio
[ ] Other (please specify)

Screenshots
If applicable, add screenshots to help explain your problem.

I have latest as of today, VSCode, and latest dotnet-interactive.
There are two issues with same description but those are not the only reason issue happens, i did all mentioned in them.
https://github.com/dotnet/interactive/issues/1729     (this points to path to SDKs and some other minors things, still addressing that won't solve the issue)
https://github.com/dotnet/interactive/issues/1775    (this points to other techniques but doing them also won't solve this issue as i have latest everything (except that windows is not 11, its windows 10)

Could you provide the versions of the extension and VS Code you have installed?

Related, issue #2206 will track future work to make scenarios like this easier to diagnose.

authors: Shahid-Roofi-Khan brettfo jonsequitur

repo: dotnet/interactive


text:
[AutoML] Auto detection of extra header rows mixed into the dataset

I have dataset in text file with 20 columns, 1st column is the class name (string), other columns are features (floats)
Here are first lines of this file
Class	A1	A2	A3	A4	A5	A6	A7	A8	A9	A10	A11	A12	A13	A14	A15	A16	A17	A18	A19
CS	61.00000	0.16855	0.00000	1.77778	3.00000	0.25375	0.07984	0.00169	0.02250	0.01535	0.07984	0.01027	0.27415	6.00000	4.00000	0.37649	3552.00000	0	26.00000
CS	316.00000	0.14823	15.00000	1.77778	10.00000	0.02352	0.00440	0.20407	0.00357	0.00914	0.03585	0.14171	0.01674	21.00000	4.00000	0.14961	4235.00000	0	17.00000
CS	176.00000	0.00000	20.00000	1.77778	3.00000	0.01850	0.19659	0.00469	0.03895	0.00000	0.19659	0.59670	0.19659	10.00000	5.00000	0.23767	3850.00000	0	24.00000
CS	133.00000	0.00000	4.00000	1.33333	3.00000	0.00049	0.01214	0.22827	0.18777	0.18778	0.12627	0.00915	0.18777	11.00000	7.00000	0.32619	1880.00000	0	16.00000
CS	140.00000	0.00000	14.00000	1.33333	1.00000	0.01787	0.02860	0.48472	0.02860	0.59853	0.02860	1.06538	0.02860	9.00000	7.00000	0.02860	1876.00000	0	142.00000

and the full file data.txt
Let's execute AutoML

mlnet auto-train --task multiclass-classification --dataset "data.txt" --has-header --label-column-name Class --max-exploration-time 10

as a  results AutoML will generate ModelInput.cs file that starts like this
 public class ModelInput
    {
        [ColumnName("Class"), LoadColumn(0)]
        public string Class { get; set; }
        [ColumnName("A1"), LoadColumn(1)]
        public string A1 { get; set; }
        [ColumnName("A2"), LoadColumn(2)]
        public string A2 { get; set; }
        [ColumnName("A3"), LoadColumn(3)]
        public string A3 { get; set; }

all columns are recognized as string instead of float 😢
as a result data pipeline also incorrect (OneHotEncoding was applied to numeric columns)
            var dataProcessPipeline = mlContext.Transforms.Conversion.MapValueToKey("Class", "Class")
                .Append(mlContext.Transforms.Categorical.OneHotEncoding(new[]
                {
                    new InputOutputColumnPair("A3", "A3"), new InputOutputColumnPair("A4", "A4"),
                    new InputOutputColumnPair("A5", "A5"), new InputOutputColumnPair("A14", "A14"),
                    new InputOutputColumnPair("A15", "A15"), new InputOutputColumnPair("A18", "A18")
                }))
                .Append(mlContext.Transforms.Categorical.OneHotHashEncoding(new[]
                {
                    new InputOutputColumnPair("A1", "A1"), new InputOutputColumnPair("A2", "A2"),
                    new InputOutputColumnPair("A6", "A6"), new InputOutputColumnPair("A17", "A17"),
                    new InputOutputColumnPair("A19", "A19")
                }))
                .Append(mlContext.Transforms.Concatenate("Features",
                    new[] {"A3", "A4", "A5", "A14", "A15", "A18", "A1", "A2", "A6", "A17", "A19"}))
                .Append(mlContext.Transforms.NormalizeMinMax("Features", "Features"))
                .AppendCacheCheckpoint(mlContext);

Why in this case all columns recognized as strings?
Why in some columns OneHotHashEncoding was used instead of OneHotEncoding?

@gvashishtha: This issue would be better in the machineleaning repo, as it would be implemented in the AutoML code during the dataset analysis.
Work:
When AutoML takes a sample of the dataset to figure out the dataset columns and column purposes, if there is a header row, then check if subsequent rows match it.
I'd purpose doing dataset checks as a new step after InferSplit() and InferColumnTypes():
https://github.com/dotnet/machinelearning/blob/e5a19af589dfb1468cd99628e82f6b49fb125323/src/Microsoft.ML.AutoML/ColumnInference/ColumnInferenceApi.cs#L16-L18

authors: justinormont sergey-tihon

repo: dotnet/machinelearning-modelbuilder


text:
Microsoft.NET.Sdk.Razor.SourceGenerators can't find interface when building MAUI Hybrid app with generic component using the interface as type constraint

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Project fails to build Razor pages/components with Razor component when building a MAUI hybrid project for a net6.0-android target. (EDIT: Issue doesn't seem to be android target specific, android was just the first target built. I tried all targets and they all failed.)
The issue seems to be related to having an interface constraint on a generic Razor component's @typeparam.
Expected Behavior
Project builds successfully.
Steps To Reproduce
Reference project is attached below for reference.

Create a MAUI Blazor hybrid project.
Create an interface ITest. It can be blank.
Create an implementation of the interface class Test : ITest, it can also be blank.
Create a generic Razor component with @typeparam TItem where TItem : ITest and add a component parameter that uses TItem.
Use the generic component in another Razor page/component.
IDE will show no errors while editing, but will fail to build on the net6.0-android target.

android-interface-test.zip
The reference
Exceptions (if any)
The error I get at build time:
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0246	The type or namespace name 'IDataItem' could not be found (are you missing a using directive or an assembly reference?)	AndroidInterfaceTest (net6.0-android)	G:\projects\android-interface-test\AndroidInterfaceTest\AndroidInterfaceTest\Microsoft.NET.Sdk.Razor.SourceGenerators\Microsoft.NET.Sdk.Razor.SourceGenerators.RazorSourceGenerator\Pages_Index_razor.g.cs	125	Active

.NET Version
6.0.400
Anything else?
Microsoft Visual Studio Community 2022
Version 17.3.3
VisualStudio.17.Release/17.3.3+32825.248
Microsoft .NET Framework
Version 4.8.04084
Installed Version: Community
Visual C++ 2022   00482-90000-00000-AA565
Microsoft Visual C++ 2022
Add Multiple Projects To Solution V2   2.0.0
Visual Studio extension that allows adding multiple existing projects into the solution. Optionally creates the solution folder structure reflecting to the physical hierarchy on disk.
ADL Tools Service Provider   1.0
This package contains services used by Data Lake tools
ASA Service Provider   1.0
ASP.NET and Web Tools   17.3.376.3011
ASP.NET and Web Tools
Azure App Service Tools v3.0.0   17.3.376.3011
Azure App Service Tools v3.0.0
Azure Data Lake Tools for Visual Studio   2.6.5000.0
Microsoft Azure Data Lake Tools for Visual Studio
Azure Functions and Web Jobs Tools   17.3.376.3011
Azure Functions and Web Jobs Tools
Azure Stream Analytics Tools for Visual Studio   2.6.5000.0
Microsoft Azure Stream Analytics Tools for Visual Studio
C# Tools   4.3.0-3.22423.10+b35bb0baca1071d50cc4f8cf5a1a89e7cc112461
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
Extensibility Message Bus   1.2.6 (master@34d6af2)
Provides common messaging-based MEF services for loosely coupled Visual Studio extension components communication and integration.
Microsoft Azure Hive Query Language Service   2.6.5000.0
Language service for Hive query
Microsoft Azure Stream Analytics Language Service   2.6.5000.0
Language service for Azure Stream Analytics
Microsoft Azure Tools for Visual Studio   2.9
Support for Azure Cloud Services projects
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
Mono Debugging for Visual Studio   17.3.20 (3f4cb00)
Support for debugging Mono processes with Visual Studio.
NCrunch
Continuous Testing Tool for .NET
Copyright � 2010-2022 Remco Software Ltd
NuGet Package Manager   6.3.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/
Razor (ASP.NET Core)   17.0.0.2232702+e1d654e792aa2fe6646a6935bcca80ff0aff4387
Provides languages services for ASP.NET Core Razor.
SQL Server Data Tools   17.0.62207.04100
Microsoft SQL Server Data Tools
Syncfusion ASP.NET Core Project Templates   20.1.0.47
Provides project templates for creating an ASP.NET Core Web Application using Syncfusion ASP.NET Core components.
Syncfusion ASP.NET Core Project Templates   16.1.0.32
Provides project templates for creating an ASP.NET Core Web Application using Syncfusion ASP.NET Core components.
Syncfusion Xamarin Project Templates   20.2.0.43
A multi-project template for building iOS, Android, and Windows apps using Syncfusion Xamarin components.
Syncfusion Xamarin Toolbox   20.2.0.43
Syncfusion Essential Studio for Xamarin Toolbox Visual Studio extension.
SyncfusionCodeGeneratorMenuCommand Extension   16.1.0.32
SyncfusionCodeGeneratorMenuCommand Visual Studio Extension Detailed Info
SyncfusionMenu Extension   20.2.0.43
SyncfusionMenu Visual Studio Extension Detailed Info
Syntax Visualizer   1.0
An extension for visualizing Roslyn SyntaxTrees.
Test Adapter for Boost.Test   1.0
Enables Visual Studio's testing tools with unit tests written for Boost.Test.  The use terms and Third Party Notices are available in the extension installation directory.
Test Adapter for Google Test   1.0
Enables Visual Studio's testing tools with unit tests written for Google Test.  The use terms and Third Party Notices are available in the extension installation directory.
ToolWindowHostedEditor   1.0
Hosting json editor into a tool window
TypeScript Tools   17.0.10701.2001
TypeScript Tools for Microsoft Visual Studio
Visual Basic Tools   4.3.0-3.22423.10+b35bb0baca1071d50cc4f8cf5a1a89e7cc112461
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Visual F# Tools   17.1.0-beta.22363.4+1b94f89d4d1f41f20f9be73c76f4b229d4e49078
Microsoft Visual F# Tools
Visual Studio IntelliCode   2.2
AI-assisted development for Visual Studio.
Visual Studio Tools for Unity   17.3.0.0
Visual Studio Tools for Unity
VisualStudio.DeviceLog   1.0
Information about my package
VisualStudio.Mac   1.0
Mac Extension for Visual Studio
Web Compiler   1.14.8
Compiler for LESS, Sass and CoffeeScript files
WiX Toolset Visual Studio Extension   1.0.0.18
WiX Toolset Visual Studio Extension version 1.0.0.18
Copyright (c) .NET Foundation and contributors. All rights reserved.
Xamarin   17.3.0.296 (d17-3@a0ff24b)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.
Xamarin Designer   17.3.0.208 (remotes/origin/d17-3@e4f67afef)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.
Xamarin.Android SDK   13.0.0.0 (d17-3/030cd63)
Xamarin.Android Reference Assemblies and MSBuild support.
Mono: dffa5ab
Java.Interop: xamarin/java.interop/d17-3@7716ae53
SQLite: xamarin/sqlite/3.38.5@df4deab
Xamarin.Android Tools: xamarin/xamarin-android-tools/main@14076a6

Closing this as by-design. Feel free to comment further if you disagree. Thanks!

authors: bit-wrangler chsienki

repo: dotnet/razor-tooling


text:
Pass DisposalToken in tests

Random pet peeves on a Thursday evening.

@davidwengier: I went and pushed a commit to fix up the failing tests.

@davidwengier: I went and pushed a commit to fix up the failing tests. I hope that's ok!

I probably would have used It.IsAny<CancellationToken>(), like I did in the LiveShare tests, but it feels rude to override your generous contribution :) Thank you!

I just put it back like it was. :smile:

authors: DustinCampbell davidwengier

repo: dotnet/razor


text:
Remove HostProject descendants

This change removes the MiscFilesHostProject and FallbackHostProject descendants of HostProject.

Reviewing commit-by-commit is recommended.

A validation build has completed here: https://dev.azure.com/devdiv/DevDiv/_git/VS/pullrequest/598306
It looks like this PR causes a regression in one of the tests where Razor seems to be parsing more than before. I'm not sure what would cause that, but I'm going to dig in and see what I can find.

It looks like this PR causes a regression in one of the tests where Razor seems to be parsing more than before. I'm not sure what would cause that, but I'm going to dig in and see what I can find.

Actually, that same regression showed up in another VS insertion without this pull request. So, I'll go ahead and merge this and continue investigating.


text:
NETSDK1082 if installed on WinUI project targeting 19041 or higher

Bug
Despite our best efforts, bugs can slip into releases or corner cases forgotten about. We will try our best to remedy the situation
and/or provide workarounds. Note that certain (odd) behaviors are by design and as such are not considered bugs.

Which library version?

System.Reactive 5.0.0

What are the platform(s), environment(s) and related component version(s)?
WinUI 3 Preview 3 project

The winui template targets Windows 18362. When installing Rx, everything is OK.
However, if the target platform is set to net5.0-windows10.0.19041, msbuild will try to add a FrameworkReference to Microsoft.WindowsDesktop.App, add results in NETSDK1082 for missing arm64 rid.
It assumes Windows Desktop support always available on Windows, but it's not the case for ARM64.

This seems to be causing issues for WinUI 3 support for ReactiveUI. If we can get some progress on this issue that would be good.

ARM should be supported on net6.0. I don't think much is likely to change before then.

Even if ARM is supported, referencing WinForms is still a wrong action.

The -windows TFM brings in all of the Windows API. That's by design. You don't have to use them if you don't want. If you're targeting Windows, then you get the entire surface area.

Any updates on this issues or plans to resolve it?
We've encountered the same problem as in #1607 and deactivating Resharper isn't really and option either. Downgrading the target platform to 18362 fixes the issue for now, but isn't a long term solution.
Using WinUI 3 with System.Reactive and Resharper doesn't seem like a rare use case, so couldn't the framework depended bits (WPF/WinForms/WinUI3) be moved into separate packages, so that one only needs to reference what he actually needs?
Thanks in advance and a happy new year

Nope and I raised an issue their too, but even without the Resharper issue, we've to carry around ~20MB of dlls we don't need and that would also be true for other types of projects, e.g. a console applications, only because they choose to target windows only.

@clairernovotny Things are worse in WinUI, since WPF and WinForms are unnecessarily carried for 3 platforms, increasing ~60MB for each platform after installation, and ~20MB in the installer package.

The -windows TFM brings in all of the Windows API. That's by design. You don't have to use them if you don't want. If you're targeting Windows, then you get the entire surface area.

Strictly speaking, no. WinForms and WPF are opt-in by UseWinForms and UseWPF properties. Unlike .NET Framework, referencing unnecessary component does have cost since there's self-contained publishing.
Trimming is not an option here since it's not compatible with WinUI yet.

Adding to all of the above that I've just had to make the decision to avoid Rx.NET in one of my projects and implement a subset of observable types myself solely because of this issue. The behavior of pulling in WPF/WinForms for -windows TFMs simply makes Rx.NET an unacceptable dependency for a library project. It's a fairly tragic state of affairs.

@HowardvanRooijen recently took over ownership of this project, so hopefully he can put this on the board to be looked at. Agreed the issue hasn't been great in regards to implied WPF support and has caused issues for ReactiveUI as well.

Adding to all of the above that I've just had to make the decision to avoid Rx.NET in one of my projects and implement a subset of observable types myself solely because of this issue. The behavior of pulling in WPF/WinForms for -windows TFMs simply makes Rx.NET an unacceptable dependency for a library project. It's a fairly tragic state of affairs.

Keep an eye on https://github.com/dotnet/reactive/discussions/1868

People running into this issue could try adding this to their project files as a workaround:
  <PropertyGroup>
    <DisableTransitiveFrameworkReferences>true</DisableTransitiveFrameworkReferences>
  </PropertyGroup>

We think this is a viable workaround. It essentially undoes Rx's attempt to add the unwanted desktop framework reference. In our experiments this is sufficient to prevent the packaging bloat problems. However, if people are finding that this doesn't work, we really want to know because this workaround is provisionally part of our long term plan for fixing this.
Our view is that if a viable workaround is available, we don't need to do anything radical in the short term. We can gradually deprecate the use of the UI-framework-specific types that are baked into System.Reactive today. The long-term goal is that System.Reactive should contain nothing UI-framework-specific, but that it will take many years to get to that end state because to remove these types today would cause breaking changes for many current users of Rx. Deprecation needs to occur slowly.
However in the absence of a viable workaround, radical change may be unavoidable. In that case, we might need to do the thing we really want to avoid: changing the main Rx package, leaving System.Reactive as a mere type forwarding shim preserved for backwards compatibility reasons. We really don't want to do that, but if people don't have a workaround for this issue, we may have no choice.
So that's why it would be very helpful for us to know if this workaround doesn't work.

See here for discussion on DisableTransitiveFrameworkReferences: https://github.com/dotnet/reactive/discussions/2038#discussioncomment-7628554

authors: HowardvanRooijen alexrp clairernovotny glennawatson huoyaoyuan idg10 marcpiechura wmanning

repo: dotnet/reactive


text:
Any .settings designer crashes for net462 exe project

I'm still working on whittling down a repro. I'm just converting legacy to new csproj and fixing the disassociation of the .settings from the .designer.cs that comes with the manual conversion. I assume you just delete the .designer.cs and open the .settings and tell it to do codegen, because that works in brand new projects. However in the converted project, when you open the designer (either via the .settings file or via the project properties) the page shows only a Windows error icon and the message:

Value cannot be null.
Parameter name: key

Sometimes it shows

Could not resolve mscorlib for target framework '.NETFramework,Version=v4.6.2'.  This can happen if the target framework is not installed or if the framework moniker is incorrectly formatted.

With the message box:

The project system has encountered an error.
Value cannot be null.
Parameter name: key
A diagnostic log has been written to the following location: "%temp%\VsProjectFault_04bf065c-6a0b-478e-be15-5364f34d8b9b.failure.txt".

So here is VsProjectFault_04bf065c-6a0b-478e-be15-5364f34d8b9b.failure.txt:
=====================
3/20/2017 4:08:03 PM
Crippling
System.AggregateException: One or more errors occurred. ---> System.ArgumentNullException: Value cannot be null.
Parameter name: key
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Microsoft.VisualStudio.ProjectSystem.VS.References.DesignTimeAssemblyResolution.GetAllResolvedReferences()
   at Microsoft.VisualStudio.ProjectSystem.VS.References.DesignTimeAssemblyResolution.ResolveReferences(String[] originalNames, AssemblyName[] assemblyName, VsResolvedAssemblyPath[] assemblyPaths)
   at Microsoft.VisualStudio.ProjectSystem.VS.References.DesignTimeAssemblyResolution.ResolveAssemblyPathInTargetFx(String[] prgAssemblySpecs, UInt32 cAssembliesToResolve, VsResolvedAssemblyPath[] prgResolvedAssemblyPaths, UInt32& pcResolvedAssemblyPaths)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse.ResolveAssemblyPath(String assemblySpec)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse.ResolveAssembly(AssemblyName assemblyName)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse.GetSystemAssembly()
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse..ctor(IDesignTimeAssemblyLoader assemblyLoader, IVsDesignTimeAssemblyResolution projectAssemblyResolution, IVsSmartOpenScope dispenser)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse.GetUniverse(IDesignTimeAssemblyLoader assemblyLoader, IVsDesignTimeAssemblyResolution projectAssemblyResolution, IVsSmartOpenScope dispenser)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkProvider..ctor(IVsDesignTimeAssemblyResolution assemblyResolution, IDesignTimeAssemblyLoader assemblyLoader, TypeDescriptionProvider parentProvider, IVsSmartOpenScope openScope)
   at Microsoft.VisualStudio.ProjectUtilities.GetTargetFrameworkProvider(IServiceProvider provider, IDesignTimeAssemblyLoader assemblyLoader, IVsHierarchy hierarchy, Object instance)
   at Microsoft.VisualStudio.Design.VSTypeResolutionService.get_TypeProvider()
   at Microsoft.VisualStudio.Design.VSTypeResolutionService.VSLangProj._dispReferencesEvents.ReferenceAdded(Reference r)
   at VSLangProj._dispReferencesEvents_ReferenceAddedEventHandler.Invoke(Reference pReference)
   at Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.Automation.OAVSReferences.OnReferenceAdded(Reference reference)
   at Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.ReferencesHostBridge.ApplyAsync(ReferencesDelta value)
   at Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.ProjectHostBridge`3.<>c__DisplayClass36_1.<<InitializeCoreAsync>b__3>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.VisualStudio.Threading.JoinableTask.<JoinAsync>d__76.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.VisualStudio.Threading.JoinableTask`1.<JoinAsync>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.ProjectHostBridge`3.<>c__DisplayClass36_0.<<InitializeCoreAsync>b__2>d.MoveNext()
   --- End of inner exception stack trace ---
---> (Inner Exception #0) System.ArgumentNullException: Value cannot be null.
Parameter name: key
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Microsoft.VisualStudio.ProjectSystem.VS.References.DesignTimeAssemblyResolution.GetAllResolvedReferences()
   at Microsoft.VisualStudio.ProjectSystem.VS.References.DesignTimeAssemblyResolution.ResolveReferences(String[] originalNames, AssemblyName[] assemblyName, VsResolvedAssemblyPath[] assemblyPaths)
   at Microsoft.VisualStudio.ProjectSystem.VS.References.DesignTimeAssemblyResolution.ResolveAssemblyPathInTargetFx(String[] prgAssemblySpecs, UInt32 cAssembliesToResolve, VsResolvedAssemblyPath[] prgResolvedAssemblyPaths, UInt32& pcResolvedAssemblyPaths)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse.ResolveAssemblyPath(String assemblySpec)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse.ResolveAssembly(AssemblyName assemblyName)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse.GetSystemAssembly()
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse..ctor(IDesignTimeAssemblyLoader assemblyLoader, IVsDesignTimeAssemblyResolution projectAssemblyResolution, IVsSmartOpenScope dispenser)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkUniverse.GetUniverse(IDesignTimeAssemblyLoader assemblyLoader, IVsDesignTimeAssemblyResolution projectAssemblyResolution, IVsSmartOpenScope dispenser)
   at Microsoft.VisualStudio.Shell.Design.VsTargetFrameworkProvider..ctor(IVsDesignTimeAssemblyResolution assemblyResolution, IDesignTimeAssemblyLoader assemblyLoader, TypeDescriptionProvider parentProvider, IVsSmartOpenScope openScope)
   at Microsoft.VisualStudio.ProjectUtilities.GetTargetFrameworkProvider(IServiceProvider provider, IDesignTimeAssemblyLoader assemblyLoader, IVsHierarchy hierarchy, Object instance)
   at Microsoft.VisualStudio.Design.VSTypeResolutionService.get_TypeProvider()
   at Microsoft.VisualStudio.Design.VSTypeResolutionService.VSLangProj._dispReferencesEvents.ReferenceAdded(Reference r)
   at VSLangProj._dispReferencesEvents_ReferenceAddedEventHandler.Invoke(Reference pReference)
   at Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.Automation.OAVSReferences.OnReferenceAdded(Reference reference)
   at Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.ReferencesHostBridge.ApplyAsync(ReferencesDelta value)
   at Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.ProjectHostBridge`3.<>c__DisplayClass36_1.<<InitializeCoreAsync>b__3>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.VisualStudio.Threading.JoinableTask.<JoinAsync>d__76.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.VisualStudio.Threading.JoinableTask`1.<JoinAsync>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.ProjectHostBridge`3.<>c__DisplayClass36_0.<<InitializeCoreAsync>b__2>d.MoveNext()<---

===================

I'll try to simplify to a repro. None of my guesses have worked yet and when I delete a bunch of the project and copy the project to a new folder, the problem disappears.

I will say that https://developercommunity.visualstudio.com/content/problem/33158/cannot-use-the-main-or-any-right-click-menu-while.html becomes particularly onerous when reloading the solution over and over. I am not a bit happy with the lack of followup in the last four months.

Oh hey, guess what. The problem goes away when I remove this section:
  <!-- Workaround for https://github.com/dotnet/roslyn-project-system/issues/1739 -->
  <PropertyGroup>
    <AssemblySearchPaths>$(AssemblySearchPaths);{GAC}</AssemblySearchPaths>
  </PropertyGroup>

Oh hey, guess what. The problem goes away when I remove this section:
Edit:

No, it's something else. Hang on.

Did not expect that. My repro is currently looking like this:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net462</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <ProjectReference Include="..\Another\Project.csproj" />
  </ItemGroup>

  <ItemGroup>
    <None Update="Properties\Settings.settings">
      <Generator>SettingsSingleFileGenerator</Generator>
    </None>
  </ItemGroup>

</Project>

If the project reference is removed, the settings designer works. If the project reference is added, the designer starts crashing again.
On to simplifying the next project.

Can't repro with fewer than three projects so far :D Still working.

Well, I built up the repro from scratch and now I'm getting a new error message in place of the others:

Object reference not set to an instance of an object

Here's the complete repro:

Open the solution in https://github.com/jnm2/misc-codesamples/raw/master/Bug reports/.NET/Settings designer crash.zip
Open ConsoleAppWithSettings/Properties/Settings.settings in solution explorer or view the Settings section in ConsoleAppWithSettings properties.
See "Object reference not set to an instance of an object." No idea why that changed from "Could not resolve mscorlib for target framework '.NETFramework,Version=v4.6.2'. This can happen if the target framework is not installed or if the framework moniker is incorrectly formatted." or "Value cannot be null. Parameter name: key" like it says on my original project, but you can probably take it from here.

The csprojs:
ConsoleAppWithSettings.csproj
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net462</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <ProjectReference Include="..\Library1\Library1.csproj" />
  </ItemGroup>

  <ItemGroup>
    <None Update="Properties\Settings.settings">
      <Generator>SettingsSingleFileGenerator</Generator>
    </None>
  </ItemGroup>

</Project>

Library1.csproj
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net462</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <ProjectReference Include="..\Library2\Library2.csproj" />
  </ItemGroup>

</Project>

Library2.csproj
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net462</TargetFramework>
  </PropertyGroup>

</Project>

Thanks @jnm2 and for your efforts to get a repro. We're aware of this issue and we've fixed this. It will be available in the next update.

repo: dotnet/roslyn-project-system


text:
Linux runtimes under packs should be excluded from SDK diff tests

Linux runtimes under packs folder are needed for RID-specific self-contained apps, and only exist in source-built .NET product.
These folders should be excluded from SDK diff tests:
packs/Microsoft.AspNetCore.App.Runtime.<RID>/*
packs/Microsoft.NETCore.App.Runtime.<RID>/*

Fixed with https://github.com/dotnet/installer/pull/16692

repo: dotnet/source-build


text:
Release pipeline validation run fails PR creation when no new changes

The SB release pipeline has a mode that can be configured for PR validation. In the "Release" stage of the pipeline, it creates the Source Build PR to update the sdk repo to use the released SDK version (e.g. global.json update). When doing so, it uses a hardcoded release version to test with (example (internal link)). If that version, for example 9.0.100-rc.1, matches the version that already exists in the target release branch, then there will be no commits differences between the release branch and the PR branch.
This results in a failure in the step to create the PR:
pull request create failed: GraphQL: No commits between release/9.0.1xx and oct2024-source-build-1729871686 (createPullRequest)

Example build

Rather than failing, maybe we should warn when this happens? I don't think that we should let the step be green if no PR is created, but I also don't think that a failure is correct either.

authors: ellahathaway mthalman


text:
Adding dotnet3.1/dotnet-eng packages sources

In order to remove dependencies on sleet from release branches,
it is neccessary to preemptively add these sources so that mehanisms
will be able to restore these packages.
Sleet must be removed because of it's strong dependency on spcific
nuget.packaging versions, which are not stable between 3.0 and 3.1
causing method load exceptions.

This was obsoleted due to other NuGet feed changes.

authors: ChadNedzlek crummel


text:
Expose Project sdk format to be edited via extensions

Summary
Customers wishing to write an extension to be able to add to sdk format need the sdk in IProjectFile to be editable.
Examples
Specific example from a customer is wishing to have sdk formats specified like so in addition to the default Microsoft.NET.sdk:
<Project Sdk="XYZ.Build.Before;Microsoft.NET.Sdk;XYZ.Build.After">

Risks / unknowns
The IsSDK check in IProjectFile determines if a project is converted to sdk-style, the change to have custom sdk project settings should not break this functionality as it is quire prevalently used.
Need a way to extend IComponentIdentifier so we can determine what kind of component the custom sdk is.

Closed via #642

repo: dotnet/upgrade-assistant


text:
Text in quick info tooltip not wrapping

Environment data
dotnet --info output:
Host:
  Version:      8.0.0-rc.1.23419.4
  Architecture: x64
  Commit:       92959931a3
  RID:          win-x64

VS Code version: 1.82.2
C# Extension version: v2.2.10
OmniSharp log
Steps to reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected  behavior
When I press ctrl + space to read method/parameter descriptions, the text should wrap after a reaching the tooltip width.
Actual behavior
All text remains on a single line:

delete me

repo: dotnet/vscode-csharp


text:
long project startup time in new versions of C# extension (wait for number of projects to appear)

I'm using vs code + C# + "C# devkit" in last 4 months. After updating C# and C# dev kit extensions to their new versions, the project startup time dramatically increased. The vs code itself is ok but I have to wait a significant amount of time (around 3 min) for intellisence to start working.
on this time in vs code bottom bar I see a loading icon after projects as shown in this picture:

and after that loading goes, number of projects appears and intelisense works.
I revert my C# extension version from 2.7.25 to 2.4.4 and the long waiting time reduced to 5 seconds.

Interesting - devkit is generally responsible for project loading - but if downgrading the C# extension version worked it sounds like an issue on our side with processing the project info they send us.

I'm not sure but maybe this help: my solution contains a project from a folder outside of my workspace.

I wouldn't expect that to matter, but thanks for mentioning it.

updated to C# 2.14.8 and the long waiting time is back.
C# dev kit is 1.1.16

@inlineHamed can you let me know if you can still reproduce this on the latest (prerelease) version of the C# extension and C# DevKit?

authors: dibarbet inlineHamed jasonmalinowski


text:
Failing to change TargetFramework to TargetFrameworks when multi-targeting gives unclear error message

Type: Feature Request

in a C# project targeting a single TFM with "TargetFramework",
add a second TFM (without changing the TargetFramework element to TargetFrameworks)
receive the error message below that is not clear, or even pointing to the proper source file that contains the issue

Determining projects to restore...
C:\Program Files\dotnet\sdk\9.0.101\NuGet.targets(170,5): error : Sequence contains no matching element [D:\source\MySolution.sln]

Extension version: 2.55.29
VS Code version: Code 1.95.3 (f1a4fb101478ce6ec82fe9627c43efbf9e98c813, 2024-11-13T14:50:04.152Z)
OS version: Windows_NT x64 10.0.22631
Modes:

EXPECTED RESULT:
a more clear error message is displayed like:
MyProject.csproj(7,5):  error:  The TargetFramework property may only contain a single TFM.  Did you mean to use TargetFrameworks?

or better yet, either deprecate the TargetFramework (singular) property and push everyone to use TargetFrameworks, or add support for multiple TFMs in the TargetFramework property and deprecate TargetFrameworks (plural)


text:
patchwork/utf8 error

[RuntimeException]
Could not scan for classes inside "D:\phpstudy_pro\WWW\amreport\include\composer\vendor/patchwork/utf8/src/Normaliz
er.php" which does not appear to be a file nor a folder

Package patchwork/utf8 is abandoned, you should avoid using it. Use symfony/polyfill-mbstring or symfony/string instead.

Package patchwork/utf8 is abandoned, you should avoid using it. Use symfony/polyfill-mbstring or symfony/string instead.

repo: double-break/spapi-php


text:
Install steps aren't quite right

See https://s3.amazonaws.com/archive.travis-ci.org/jobs/155699875/log.txt

Missing test executables (which we ARE building)
Installing include/scrypt header files

Being addressed in upstream.

repo: doublethinkco/cpp-ethereum-cross

license_source: bigquery


text:
[Question] Is possible to disconnect?

Question
Hello,
I started to use your plugin and everything is going very well until now. i just want to wonder if is possible to disconnect from a device? I know there is no documentation for it, but i just want to know if it is possible
i'm trying to call:
EasyBluetooth.disconnect(device)

But i'm receiving the following error
EasyBluetoothClassic.disconnect got 3 arguments, expected 2

Hi, thanks for reporting.
I'll take a look soon.

I know that there is no documentation for disconnect inside your README file
But i'm wondering because it could be very usefull :)

Sorry, now I got the problem.
You can't pass arguments to the function disconnect.
https://github.com/douglasjunior/react-native-easybluetooth-classic/blob/8971d151920f4f4749863f271c957ca35c26d40d/android/src/main/java/io/github/douglasjunior/ReactNativeEasyBluetooth/classic/ClassicModule.java#L55-L59

May i open a pull request to put it on your documentation?

Sure, thanks!

:)

authors: douglasjunior renanborgez

license: mit


text:
Status not found on commit

Sometimes, after putting the new status, the implicit get will fail because it can't find the status on the commit. I think this could be resolved by having the put wait for a bit and check that the status actually appears. What's your opinion on this?

Hi - I have noticed this a couple times as well. I guess replication is sometimes a bit slow in their datacenters.
I like the suggestion of having put internally attempt to get its own status for a short period before returning. I'd probably avoid a hard-coded delay + internal get in favor of geting immediately and sleep + retry a few times if it fails.
Thoughts, or interested in making a PR?

The sleep and retry sounds like a good idea to me. I'll create a PR soonish.

I think I'm hitting this one now, distinct from #7. It's very strange because it seems to be repo-by-repo.

@jchesterpivotal, does it eventually work on repos? Does it sometimes work the first time, sometimes not? And it’s for sure unrelated to permissions or scopes?
Any ideas on how I can best help debugging this? I think I have stopped seeing this original error since the PR and retries was added, so don’t have a sample size to experiment on.

I don't think it's due to scopes or permissions -- the same bot user, with the same token, getting its write permissions through a team -- successfully works with other repos under that team. When I GET the status endpoint manually, I get this:
{
    "commit_url": "https://api.github.com/repos/projectriff/ci/commits/36a5595504cb67ff66db19211ae54b5a21ac350b",
    "repository": {
        "archive_url": "https://api.github.com/repos/projectriff/ci/{archive_format}{/ref}",
        "assignees_url": "https://api.github.com/repos/projectriff/ci/assignees{/user}",
        "blobs_url": "https://api.github.com/repos/projectriff/ci/git/blobs{/sha}",
        "branches_url": "https://api.github.com/repos/projectriff/ci/branches{/branch}",
        "collaborators_url": "https://api.github.com/repos/projectriff/ci/collaborators{/collaborator}",
        "comments_url": "https://api.github.com/repos/projectriff/ci/comments{/number}",
        "commits_url": "https://api.github.com/repos/projectriff/ci/commits{/sha}",
        "compare_url": "https://api.github.com/repos/projectriff/ci/compare/{base}...{head}",
        "contents_url": "https://api.github.com/repos/projectriff/ci/contents/{+path}",
        "contributors_url": "https://api.github.com/repos/projectriff/ci/contributors",
        "deployments_url": "https://api.github.com/repos/projectriff/ci/deployments",
        "description": null,
        "downloads_url": "https://api.github.com/repos/projectriff/ci/downloads",
        "events_url": "https://api.github.com/repos/projectriff/ci/events",
        "fork": false,
        "forks_url": "https://api.github.com/repos/projectriff/ci/forks",
        "full_name": "projectriff/ci",
        "git_commits_url": "https://api.github.com/repos/projectriff/ci/git/commits{/sha}",
        "git_refs_url": "https://api.github.com/repos/projectriff/ci/git/refs{/sha}",
        "git_tags_url": "https://api.github.com/repos/projectriff/ci/git/tags{/sha}",
        "hooks_url": "https://api.github.com/repos/projectriff/ci/hooks",
        "html_url": "https://github.com/projectriff/ci",
        "id": 106029851,
        "issue_comment_url": "https://api.github.com/repos/projectriff/ci/issues/comments{/number}",
        "issue_events_url": "https://api.github.com/repos/projectriff/ci/issues/events{/number}",
        "issues_url": "https://api.github.com/repos/projectriff/ci/issues{/number}",
        "keys_url": "https://api.github.com/repos/projectriff/ci/keys{/key_id}",
        "labels_url": "https://api.github.com/repos/projectriff/ci/labels{/name}",
        "languages_url": "https://api.github.com/repos/projectriff/ci/languages",
        "merges_url": "https://api.github.com/repos/projectriff/ci/merges",
        "milestones_url": "https://api.github.com/repos/projectriff/ci/milestones{/number}",
        "name": "ci",
        "notifications_url": "https://api.github.com/repos/projectriff/ci/notifications{?since,all,participating}",
        "owner": {
            "avatar_url": "https://avatars1.githubusercontent.com/u/33602961?v=4",
            "events_url": "https://api.github.com/users/projectriff/events{/privacy}",
            "followers_url": "https://api.github.com/users/projectriff/followers",
            "following_url": "https://api.github.com/users/projectriff/following{/other_user}",
            "gists_url": "https://api.github.com/users/projectriff/gists{/gist_id}",
            "gravatar_id": "",
            "html_url": "https://github.com/projectriff",
            "id": 33602961,
            "login": "projectriff",
            "organizations_url": "https://api.github.com/users/projectriff/orgs",
            "received_events_url": "https://api.github.com/users/projectriff/received_events",
            "repos_url": "https://api.github.com/users/projectriff/repos",
            "site_admin": false,
            "starred_url": "https://api.github.com/users/projectriff/starred{/owner}{/repo}",
            "subscriptions_url": "https://api.github.com/users/projectriff/subscriptions",
            "type": "Organization",
            "url": "https://api.github.com/users/projectriff"
        },
        "private": false,
        "pulls_url": "https://api.github.com/repos/projectriff/ci/pulls{/number}",
        "releases_url": "https://api.github.com/repos/projectriff/ci/releases{/id}",
        "stargazers_url": "https://api.github.com/repos/projectriff/ci/stargazers",
        "statuses_url": "https://api.github.com/repos/projectriff/ci/statuses/{sha}",
        "subscribers_url": "https://api.github.com/repos/projectriff/ci/subscribers",
        "subscription_url": "https://api.github.com/repos/projectriff/ci/subscription",
        "tags_url": "https://api.github.com/repos/projectriff/ci/tags",
        "teams_url": "https://api.github.com/repos/projectriff/ci/teams",
        "trees_url": "https://api.github.com/repos/projectriff/ci/git/trees{/sha}",
        "url": "https://api.github.com/repos/projectriff/ci"
    },
    "sha": "36a5595504cb67ff66db19211ae54b5a21ac350b",
    "state": "pending",
    "statuses": [],
    "total_count": 0,
    "url": "https://api.github.com/repos/projectriff/ci/commits/36a5595504cb67ff66db19211ae54b5a21ac350b/status"
}

One thing that interests me is that "state": "pending" is set. So the put appears to "work", but not get picked up by the auto-get that follows it.
An example of a failing build is here. I note that after the put, status shows as null.

I'm seeing this again. It appears to be unpredictable within a single job. I'm not sure how to debug it.
Eg

Found my mistake!
When I used state: failed instead of state: failure, I get the null thing.
So what's missing here is a "Jacques, you used the wrong word" warning.

Ah, interesting. Glad you were able to figure it out, but would like to avoid others having this issue as well.
Unless you beat me to a PR, I should be able to look into this and the other issues soon.

I'm happy to let you have the glory, now that I've worked out my end :)

I'm still seeing this issue intermittently.  Did a PR ever get merged to fix this?


text:
the amulets do not appear

Hello I have an Error and I dont know how solve, when I go into the game in the charms section, the 15 new notches that the mod includes appear, but when I go to the location where the charms should be to pick them up, they don't appear. I have all 3 mods needed to make it work. Sorry if something is not well written, I am Spanish

I'm sorry I missed this for so long. If you used one of your existing save files, the issue is that Transcendence only places its charms when you create a new save file. You can still get the charms on an existing file by using the debug mod's "give all charms" button, or by editing the save file directly.

same issue, new save, give all charms doesn't work also

@FireBornFox can you provide a ModLog.txt from a session where you had this issue? (You can find that file in the same directory as your saves.)

Closed due to inactivity.

authors: FireBornFox MiskoJoness dpinela

repo: dpinela/Transcendence

license: Unlicense


text:
🛑 File Service is down

In 31587ce, File Service (https://files.dplayzgames06.tk) was down:

HTTP code: 530
Response time: 1076 ms

Resolved: File Service is back up in b8e76ee after 26 minutes.

repo: dplayz/status


text:
🛑 Fastbin Service is down

In 8006b0f, Fastbin Service (https://paste.dpG06.top) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Fastbin Service is back up in 29b34ed after 3 hours, 1 minute.


text:
Output not flushing consistently

When a program is sending a lot of information, it doesn't always update the output text.

This is a consequence of programs not flushing their output consistently. I wonder if there is another fix however...

Looking into it further, programs in a hosted environment do not need to flush the output like they do in a freestanding environment, which is pretty much anything not run in the console. Therefore to fix this, Nodejs would have to emulate a TTY itself, so I am going to close this, because this issue cannot realistically be fixed, especially with just this package.

repo: dqsully/dqs-shell-commands


text:
🛑 366.ru is down

In 2809a84, 366.ru (https://366.ru) was down:

HTTP code: 0
Response time: 0 ms

Resolved: 366.ru is back up in ca3e0e3.


text:
Running Dragonfly 2 without manager

Feature request:
Is it possible to run Dragonfly 2 without the manager component?
Can you please provide a guide to install/configure Dragonfly 2 without the manager, MySQL, redis and UI?
While manager sounds useful it adds complexity to the overall architecture and all we're looking for in our team is to run dragonfly 2 as we were running the old dragonfly.

There should be no plans to remove the manager deployment, because the manager is more important for cluster stability. If you are interested, you can try to submit a pull request and use the manager as a plugin.

authors: gaius-qi ish-xyz


text:
Don't auto add test:decorators to test:run

Prior to Rails 4.2 if you ran rake test (which in turn ran rake test:run) it would only run tests in specific directories. So if you had a new type of file (such as decorators) you needed to append this rake task to include your new file type.
As of commit 3b12abb in Rails this is no longer neccessary. The old rake test:all which was more inclusive (ran all tests under the test/ directory) became rake test. The old rake test that only ran specific directories was removed.
This commit removes the special addition of running the decorator tests. Without this, running rake test will first test the decorators, then it will test all files (including the decorators).

Hello,
Any news about this PR?
Thanks you.

Thanks!

Thanks @eric1234 this had been bugging me.  Nice to see less rake overhead in my test run with draper 2.1.

authors: eric1234 jaredbeck jcasimir nmeylan

repo: drapergem/draper


text:
Perform naiive branch relaxing (and other fixes to the BIR passes)

There's a number of changes here:

Branch relaxing (see #195)
Hacky workaround for #179 (only affects Thumb, see the huge comment in Bir_passes.Shape.split_on_conditional)
Only "finalize" higher variables at the "implicit exit" node. If we have a block that does goto L_0x1234, then this is not an implicit exit and we can assume that the higher vars aren't needed "at exit" in any particular register.
Adds the lsl rt, [rn, rm, lsl #n] pattern to the current selector (results in much smaller code!)

This also adds the "init-mem" feature?

Seems like good stuff. Great work! I am worried that it feels like the patch_spaces being in bir_passes is ensnarling the codebase in itself, but I don't see an easy way around it. Everything is dependent on everything if you want it to be really good unfortunately.
Is point 3 above correct? I think this is maybe because of a weakness of the schema of the config file. Each possible exit point should have it's own notion in general of where hvars need to be put. In other words we need a write-to relation:  (addr, hvar, lowloc)

Yes, it seems that this would be an avenue for future work. In general when we have a patch like:
x = y + z;
if (x > w) {
  goto L_0x1234;
}
--y;

We could just assume that once we go to 0x1234 there is an entirely different set of data dependencies (but for our purposes we just assume there are none at all), since we're kind of "aborting" the normal control flow of the function being patched.
Meanwhile, the fallthrough to --y means that we're continuing back to the end of the patch point, which means the dependencies should be preserved.
It would be nice to have some granularity per-exit about what kind of data dependency is needed.

repo: draperlaboratory/VIBES


text:
[WIP] BREAKING: add firefox support

Fixes #24
Summary

To support firefox, the API has been changed slightly, this makes both chrome and firefox classes maintainable. Tests have been adjusted for this and it was pretty quick and easy to do luckily
API's for firefox and chrome client are exactly the same
Options can be passed to the build method, to customise the client
The API is now as follows:

import { ChromeClient, FirefoxClient } "https://deno.land/x/sinco/mod.ts"

const Sinco = await ChromeClient.build() // or `const Chrome = ...`
await Sinco.goTo("https://drash.land") // as usual

Added firefox support - (in progress)

TODO

All tests for all other methods of firefox api
add unit tests
Remove dead coe
Refactor
lint and fmt

so im debating removing the waitForAnchorChange, because if i go to chromestatus.com, and change the value of the input field we modify in our tests, the uri doesn't change, but the chrome client does pick it up
Alongside this, if i do the same with a firefox client, the client doesn't pick up any event for this

repo: drashland/sinco


text:
Other subkey generation method

First, many thanks for this useful guide that helped me a lot in order to configure my yubikeys.
There exist another method to generate subkeys and this is a more secure way: generating subkeys from the yubikey. With this method, the subkeys is every time on the yubikey from generation to storage which is more secure. The drawback comparing to the original method is that there is no way to backup subkey.
This issue's purpose is to discuss about how to contribute to this guide in order to provide this method. Should we give user the choice between the 2 methods? If not, the guide relies a lot on the subkeys backups so what will be the implications of giving just this new method?

This has been discussed in the past and a decision was made for portable keys for reliability/durability reasons. Certainly it's worth mentioning there exists the other method, and what its possible advantages are. Please create a PR for that.

authors: dangoncalves drduh

repo: drduh/YubiKey-Guide


text:
安装了不能用.vue一直说找不到模块

安装成功了, 在package.json中有存在的. 在node_modules中也存在文件夹,只是这个文件夹图标是快捷方式的图标.
然后我在main.js中使用这个就报错.
import RouteTransition from 'vue-route-transition'
Vue.use(RouteTransition)
提示这个模块不存在. 可能是和这个快捷方式图标有关.

我是用cnpm安装的, 然后我卸载了, 重新用npm来安装,提示我要安装
Missing write access to D:\workspace\vue\quan_h5\node_modules\vue-route-transition
npm WARN ajv-keywords@2.1.1 requires a peer of ajv@^5.0.0 but none is installed. You must install peer dependencies yourself.
然后我执行
npm install ajv@^6.0.0
再重新npm install vue-route-transition --save
发现安装成功了. 然后快捷方式图标也不存在了.
我再重启vue cli
启动时还是报错
This dependency was not found:

vue-route-transition in ./src/main.js
To install it, you can run: npm install --save vue-route-transition

安装后也找不到 这个模块。。后来  import RouteTransition from  "../node_modules/vue-route-transition/lib/vue_route_transition.min.js"    这样子就可以用了。。

重新安装, 用了1.0.7的,正常了.

但是使用上,还是有问题哦.

按他的基本使用 可以动画切换啊。我试过了。没出现这个问题。 我用的是 1.0.6

App.vue

我把这个
加在我当前项目中. 然后这里面能不能嵌入router-view 或是其它的组件呢?

用是可以用的。。 把替换成这个的。要用动画的用这个，一个页面的头部和footer要固定的用作者提供的那个结构。。就是 嵌套路由的都要单独写出来。渲染要跳到 顶层的 这个上了。。。有疑问的是 嵌套路由 是不是一定要单独写出来,单独写出来感觉路由不好看

repo: dreamback/vue-route-transition


text:
Postgres boolean bug

Fixed a bug in the postgres module where the type used is boolean. In postgres only:
true
'true'
'0'
However, 0 and 1 are not valid values for a boolean. Therefore, postgres returns an error everytime these function are called.

Your commit has a few backup files attached, but I see where you changed it. I'll merge your changes manually ;)

Could you test it? I think I merged the necessary changes, if I missed something just tell me. If you could send me your e-mail I'll put it in the contributors.

authors: dresende yagudaev

repo: dresende/node-orm


text:
protocol Scrivener.Paginater not implemented for #Ecto.Query

Hello, I had pagination working fine in my app using:

phoenix 1.1.4
phoenix_ecto 2.0
scrivener_html 1.1
scrivener 1.1.4

But now, after upgrading to:

phoenix 1.2.0
phoenix_ecto 3.0
scrivener_html 1.3
scrivener 2.0.0

It's failing with:
[error] #PID<0.4760.0> running MyApp.Endpoint terminated
Server: localhost:4000 (http)
Request: GET /s
** (exit) an exception was raised:
    ** (Protocol.UndefinedError) protocol Scrivener.Paginater not implemented for #Ecto.Query<from s in MyApp.Sitemap, where: s.user_id == ^1, order_by: [desc: s.inserted_at]>
        lib/scrivener/paginater.ex:1: Scrivener.Paginater.impl_for!/1
        lib/scrivener/paginater.ex:10: Scrivener.Paginater.paginate/2
        (MyApp) web/controllers/sitemap_controller.ex:28: MyApp.SitemapController.index/2
        (MyApp) web/controllers/sitemap_controller.ex:1: MyApp.SitemapController.action/2
        (MyApp) web/controllers/sitemap_controller.ex:1: MyApp.SitemapController.phoenix_controller_pipeline/2
        (MyApp) lib/MyApp/endpoint.ex:1: MyApp.Endpoint.instrument/4
        (MyApp) lib/phoenix/router.ex:261: MyApp.Router.dispatch/2
        (MyApp) web/router.ex:1: MyApp.Router.do_call/2
        (MyApp) lib/MyApp/endpoint.ex:1: MyApp.Endpoint.phoenix_pipeline/1
        (MyApp) lib/plug/debugger.ex:93: MyApp.Endpoint."call (overridable 3)"/2
        (MyApp) lib/MyApp/endpoint.ex:1: MyApp.Endpoint.call/2
        (plug) lib/plug/adapters/cowboy/handler.ex:15: Plug.Adapters.Cowboy.Handler.upgrade/4
        (cowboy) src/cowboy_protocol.erl:442: :cowboy_protocol.execute/4

I've already cleaned up and reinstalled the deps, not sure what is failing here. This is the code in the controller:
  def index(conn, params) do
    sitemaps = Sitemap
               |> where(user_id: ^(current_user(conn).id))
               |> order_by(desc: :inserted_at)
               |> Repo.paginate(params)

    render(conn, "index.html", sitemaps: sitemaps)
  end

Any idea?
Thanks!
(Comes from https://github.com/mgwidmann/scrivener_html/issues/30)

@jaimeiniesta Hi! With the upgrade of scrivener to 2.0, the ecto-specific functionality has moved to scrivener_ecto (see the note at the top of the README). Including that library should fix your issue (assuming scrivener_html works with scrivener 2.0).

That was it! It works fine now, thanks a lot.

It appears when using the new MyXQL driver, with the exact same error, just followed by
"...for type Ecto.Query (struct)
I had to switch back to mariaex to make it work again.

how to solve this error?

authors: MeerKatDev drewolson jaimeiniesta karankt

repo: drewolson/scrivener


text:
bug: installation failed with npm install -g ionic

From @elhamsarikhani on October 21, 2015 14:54
Type: bug
Platform: desktop browser
I'm trying to install ionic  using
npm install -g ionic
but I get this error:

Copied from original issue: driftyco/ionic#4547

From @MesutGulecen on October 21, 2015 16:23
you must  to  download node.js and cordova first.

From @MesutGulecen on October 22, 2015 7:37
You're welcome :)

From @elhamsarikhani on October 22, 2015 4:13
Thank you I installed node.js and now it's fine : ]

I am having the same exact issue, I could not resolve this until I downgraded NodeJS and NPM to 0.10.4.  I spent hours uninstall/reinstall NodeJS and NPM.
Did you manage to resolve this issue on latest NodeJS and NPM versions?  What did you do exactly?  I just downloaded NodeJS x64 (I tried x86, same thing), installed Cordova (works every time), and got the unexpected eof error every time.

+1

Guys installing the latest version of nodeJS solved my issue

Thanks for opening the issue. It seems that issue has been resolved. I am going to close this issue but please feel free to reopen another issue if this occurs again.

authors: TheMattRay elhamsarikhani hitmantb jthoms1 mhartington

repo: driftyco/ionic-cli


text:
Ionic Cloud fails to init the Push Plugin

Hi, i just recently have trouble with my phonegap-push-plugin, it won't init anymore. I dont think that it relates to the issue, but the problem appeared ever since I installed the deploy-plugin, although the deploy-plugin works.
The error that appears during initialization is the following:
I/chromium(15125): [INFO:CONSOLE(447)] "Uncaught TypeError: Object function () {
I/chromium(15125): } has no method 'init'", source: file:///android_asset/www/lib/ionic.cloud.min.js?ionicCachebuster=14929 (447)

the line it is referring to in the ionic.cloud.min.js is the following:
 return this.blockRegistration ? e.reject(new Error("Another registration is already in progress.")) : (this.blockRegistration = !0, this.emitter.once("device:ready", function() {
                        var i = t._getPushPlugin();
/* ThisLine */          i ? (t.plugin = i.init(t.options.pluginConfig), t.plugin.on("registration", function(i) { t.blockRegistration = !1, t.token = { token: i.registrationId }, t.token.registered = !0, e.resolve(t.token) }), t._callbackRegistration(), t.registered = !0) : e.reject(new Error("Push plugin not found! See logs.")) })), e.promise }, t.prototype.unregister = function() {
                    var t = this,
                        e = new n.DeferredPromise;

I ended up trying different dependencies and what have you. But I was able to replicate the error when i removed my platforms folder and used ionic state reset to restore it. The weird thing is, that when i remove the platforms folder and run ionic platform add android the push plugin works.
I'm using "@ionic/cloud": "0.9.0" and <plugin name="phonegap-plugin-push" spec="1.8.2">.
I'm not really familiar with cordova, thats why at this point i could really use some help. Thanks in advance :)

Hey I figured, out that removing the push plugin from the package.json solved the issue

repo: driftyco/ionic-cloud


text:
item-sliding: update the ion-item to be a button

This is dependent on the next release having this issue in it: https://github.com/driftyco/ionic2/issues/513

Oops, this is a duplicate of #20

repo: driftyco/ionic-conference-app


text:
how to exit app?

witch api function should I use?

What do you mean? You don't need to add close app functionality, as users on all mobile platforms know exactly how to do this using their OS.
http://ux.stackexchange.com/questions/7502/why-dont-mobile-apps-have-a-close-button

navigator.app.exitApp as JavaScript call

@graphefruit
is there any ionic way?

https://forum.ionicframework.com/t/how-to-close-application-using-button-ionic-2/48050/5
exitApp(){
     this.platform.exitApp();
  }

Terrible idea to close an app. Also this is not an issue and should be further discussed in the forums.

@dylanvdmerwe Sorry but why should it be an terrible idea?
Many android-apps supports the feature to close the app when pressing back on the "RootPage".
Some asks to press again within X-Seconds, other asks the users, and some just closes the app.
Its your UX-Design which handles this case.

@graphefruit it's just bad mobile UX. It goes against Apple's guidelines. And Android has both a back and home button for the user to press - no need for an exit interaction.
As @vintage posted above, read this thread: http://ux.stackexchange.com/questions/7502/why-dont-mobile-apps-have-a-close-button
Still not an ionic issue.

Closing. This is not an ionic issue.
As the issue template states, if you have question, please use the ionic forum or the slack group.

authors: dylanvdmerwe graphefruit mhartington vintage yozman

repo: driftyco/ionic


text:
ng-repeat refresh severe issue - replicable

Hello,
I'm running ionic for an Android app and I have a chat zone that needs to be refreshed every time a new message is received (yes, I'm doing $apply() on the async method). The problem is that after I receive the 6th message (this happens every time after the 6th message), the view is not refreshed properly (the divs are there but with white overlay). If I go back to another tab for instance (or call the side menu) and then come back the items are displayed normally (ignore the green boxes please):

Code (VIEW):

	{{msg.username}} {{msg.text}}

Code (CONTROLLER):
function onMessageArrived(message) {
  console.log("onMessageArrived:"+message.payloadString);
  var json = JSON.parse(message.payloadString);
  var msg = new Message();
  msg.avatar_url = json.avatar_url;
  msg.text = json.text;
  msg.username = json.username;
  msg.user = json.user;

  if(msg.user == userInfo.id) {
    msg.class = "me";
  }

  $scope.chatMessages.push(msg);

  $scope.$apply();
  $ionicScrollDelegate.scrollBottom();
};

Notes:

This issue does not happen on my machine web browser

Thanks!

Greetings @Whitebeard86!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

I've changed the element from "div" to "ion-item" and now it works correctly :
<ion-item class="chat-item" ng-class="msg.class" ng-repeat="msg in chatMessages track by $index | limitTo: 512">

	{{msg.username}} {{msg.text}}

</ion-item>

ng-repeat doesn't apply correctly when using divs?
Thanks

@Whitebeard86 What are you using to contain the divs/ion-items? Are you using an ion-list? I think ion-list expects/depends on you using ion-items to ng-repeat over

Greetings @Whitebeard86!
I've closed this issue because my sensors indicated it was old and inactive, and may have already been fixed in recent versions of Ionic. However, if you are still experiencing this issue, please feel free to reopen this issue by creating a new one, and include any examples and other necessary information, so that we can look into it further.
Thank you for allowing me to assist you.

authors: Ionitron Whitebeard86 jskrzypek


text:
Is this IONIC Bug? IONIC + MAPBOX

My ionic info command here
Cordova CLI: 6.0.0
Ionic Version: 1.2.4
Ionic CLI Version: 1.7.13
Ionic App Lib Version: 0.6.5
ios-deploy version: 1.8.4
ios-sim version: 5.0.6
OS: Mac OS X El Capitan
Node Version: v0.12.7
Xcode version: Xcode 7.2 Build version 7C68
And my IOS version 9.2
Note 1. I'm using option <data-tap-disabled="true"> ref
Note 2. I read #4538, and #4506 and others.
I'm using ionic + mapbox.js(v2.2.4)
When I command ionic serve(in browser) click event working nice(click event fired nice). It's not problem.
When I command ionic emulate ios -l -c (in emulator) and click the map at first it works fine. But random
time (after some click and move the map, expand map, contract map) click event are broken completly.
context(long tab event), and drag event working nice but only click(tab) event broken.
I think it's ionic tab,click bug  because all click(touch) event working nice on browser.
only problem on emulator.
Is this bug on ionic?
I Created a example Repository  here
(In this repo i want to show click event working on. so i added code console.log about click event. look carefully console on emulator. ionic emulate ios -l -c)

Hello! Is this still an issue with the latest version of ionic? Thanks!

Hi @yoohoogun114,
The repository you linked to isn't available any more.  Happy to check it out if you can get that squared away.  I'm going to go ahead and close this, but feel free to respond if you'd like to dive into it again.
Thanks,
Dan


text:
@font-face OpenType embedding

Generating CSS3114 errors on Windows phone
CSS3114: @font-face failed OpenType embedding permission check. Permission must be Installable.
noto-sans-regular.ttf
Steps to reproduce:

Set Config.mode to 'wp'
Run on Windows Phone

Lumia 950 XL
Windows 10 Mobile
Cordova CLI: 6.1.1
Ionic Framework Version: 2.0.0-beta.4
Ionic CLI Version: 2.0.0-beta.24
Ionic App Lib Version: 2.0.0-beta.14
OS: Windows 8.1
Node Version: v0.12.2

Other than run on windows phone, what other steps do I need to follow for this? Is this happening with ionicons?

Hey there! So it looks this issues has been inactive for some time. I'm going to close it. If there's still an issue, please submit a new issue with the steps to reproduce. Thanks!

authors: gcrockenberg mhartington