text:
Loan Object Interest rate issue.

Please change interest rate property to
private double _interestRate;
public double InterestRate
{
get { return _interestRate * 100;}
set { _interestRate = value; }
}
to calculate accurate percentage rates. This will make the interest rate in the testLoan object show up correctly. However, you will still have an issue with testLoan.InterestRate change this to be the same decimal format as the interest rate in the test loan object so
instead of 7.1 change it to 0.071. testLoan.InterestRate = 0.071;
Then when the application executes it will correctly show interest rate.
New customer value: Henry Clay
7.5
7.1

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

ID: b7bc5cfb-d247-62c8-2173-9403efe72b16
Version Independent ID: 3fab702d-66a7-3338-6c61-f535b29d86a7
Content: Walkthrough: Persisting an Object using C#
Content Source: docs/csharp/programming-guide/concepts/serialization/walkthrough-persisting-an-object-in-visual-studio.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for writing this issue @Esjihn
I agree that referring to an "interest rate" not using a percentage is confusing. I'm going to make a different suggestion.  If it were updated as above, the result would be:
loan.LoanAmount = 0.071;
Console.WriteLine(loan.LoanAmount); // displays "7.1".

What would you think of the following:

[ ] Change the property and field name to PercentageInterestRate and _percentageInterestRate for clarity.
[ ] Change the first initialization to use a percentage.

I've added this to our backlog, and added the "up for grabs" label. Would you like to submit a PR for these change?

Hello Bill,
The first initialization used the correct decimal amount of 0.075 and the
second one did not 7.1. Since there is no calculation for interest rate in
the property the only way you would get this result.
[image: image.png]
is to either make the TestLoan object second parameter 7.5 or to calculate
the percentage and change the second interest rate to be an actual
percentage "0.071". Either way is fine. Ive never submitted a "PR". I only
thought there was this feed back option and if something got changed I
would be added to contributor list.
Either way I appreciate your time and thanks for all your communication.
Best Regards,
Matthew D. Miller
Information Systems Programmer
Florida Senate
On Mon, Sep 16, 2019 at 6:08 PM Bill Wagner notifications@github.com
wrote:

Thanks for writing this issue @Esjihn https://github.com/Esjihn
I agree that referring to an "interest rate" not using a percentage is
confusing. I'm going to make a different suggestion. If it were updated as
above, the result would be:
loan.LoanAmount = 0.071;Console.WriteLine(loan.LoanAmount); // displays "7.1".
What would you think of the following:

Change the property and field name to PercentageInterestRate and
_percentageInterestRate for clarity.
Change the first initialization to use a percentage.

I've added this to our backlog, and added the "up for grabs" label. Would
you like to submit a PR for these change?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dotnet/docs/issues/14367?email_source=notifications&email_token=AIMLFHQ6EMN3NZ35HFHOG5LQJ77UZA5CNFSM4IVUE3HKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD62VINQ#issuecomment-531977270,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIMLFHR5L5VRTEQ5DT2AUVDQJ77UZANCNFSM4IVUE3HA
.

@Esjihn This is fine for feedback. We'll update this sample as part of our regular updates.
Our docs are all Open Source, so anyone can make contributions. Those contributions are Pull Requests. The Readme on our repository has pointers to get started, if you are interested.

Hello Bill,
Oh a pull request. Sorry for not putting the two together. Alright, thanks
for the heads up and I appreciate the update.
Best Regards,
Matthew
Information Systems Programmer
Florida Senate
On Tue, Sep 17, 2019 at 9:50 AM Bill Wagner notifications@github.com
wrote:

@Esjihn https://github.com/Esjihn This is fine for feedback. We'll
update this sample as part of our regular updates.
Our docs are all Open Source, so anyone can make contributions. Those
contributions are Pull Requests
https://help.github.com/articles/creating-a-pull-request. The Readme
https://github.com/dotnet/docs on our repository has pointers to get
started, if you are interested.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dotnet/docs/issues/14367?email_source=notifications&email_token=AIMLFHVKCQSHAMRA6GJHWRDQKDOEHA5CNFSM4IVUE3HKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD64S6NQ#issuecomment-532229942,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIMLFHRVN4CO7BA347IO443QKDOEHANCNFSM4IVUE3HA
.

I'll take this one on -- seems straight-forward enough. Will open a PR shortly.

authors: BillWagner Esjihn SeanKilleen


text:
AnalysisLevel default to opt-in on pre NET 5 projects confusing

EnableNETAnalyzers specifies that to "enable .NET code analysis for projects that target earlier versions of .NET":

Another way to enable .NET code analysis on projects that target .NET versions prior to .NET 5.0 is to set the AnalysisLevel property to latest.

Clicking on AnalysisLevel tells us that the default is:

The default value is latest.

So what is it? Either the default is latest, and thus code analysis is enabled for projects targeting earlier versions of .NET or the default is something else and thus code analysis is not enabled for projects targeting earlier versions of .NET.

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

ID: 59d4dafc-2217-9a7b-4590-27faa85164cd
Version Independent ID: 64b80e80-1c15-c293-466a-918de98e96f6
Content: MSBuild properties for Microsoft.NET.Sdk - .NET
Content Source: docs/core/project-sdk/msbuild-props.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

I think that AnalysisLevel doesn't come into play at all unless you specifically add it to projects that target .NET Core 3.1 or earlier. But for .NET 5+ projects, the default is latest. @mavasani Can you confirm?

Furthermore, setting AnalysisLevel to "latest" doesn't appear to enable the analyzer in .NET Framework projects. I originally had this in my project file:
    <AnalysisLevel>latest</AnalysisLevel>
    <AnalysisMode>AllEnabledByDefault</AnalysisMode>

With only a few warnings being displayed. After I added:
    <EnableNETAnalyzers>true</EnableNETAnalyzers>
...there was a huge increase. The page needs to be updated to clarify.

Furthermore, setting AnalysisLevel to "latest" doesn't appear to enable the analyzer in .NET Framework projects. I originally had this in my project file:
    <AnalysisLevel>latest</AnalysisLevel>
    <AnalysisMode>AllEnabledByDefault</AnalysisMode>

With only a few warnings being displayed. After I added:
 <EnableNETAnalyzers>true</EnableNETAnalyzers>
...there was a huge increase. The page needs to be updated to clarify.

The doc page says:

Another way to enable .NET code analysis on projects that target .NET versions prior to .NET 5.0 is to set the AnalysisLevel property to latest.

If that's not working, It could be a product bug, not a doc bug.

I think that AnalysisLevel doesn't come into play at all unless you specifically add it to projects that target .NET Core 3.1 or earlier. But for .NET 5+ projects, the default is latest. @mavasani Can you confirm?

That sounds correct.
https://github.com/dotnet/sdk/blob/2702c14c485d20f662800883ff7951235999beef/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.NET.Sdk.Analyzers.targets#L18-L22

source: gharchive/issue

authors: Bouke DiskCrasher Youssef1313 gewarren

repo: dotnet/docs

license_type: permissive

license_source: github-api


text:
Install a template from a MyGet feed?

I was wondering if it's possible to install a new template from a MyGet feed directly with something like "dotnet new -i MyTemplatePack -Source https://mygetfeed/index.json" or something like that.
I can manually download the template and then install it from the file system, but was hoping to avoid the step.
Thanks.  :)

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

ID: 4e7deb06-37bb-b659-7cf3-9c47a91e065e
Version Independent ID: d6e6f989-3956-6a0e-5e9c-fb3d1f9bd751
Content: Custom templates for dotnet new
Content Source: docs/core/tools/custom-templates.md
Service: unspecified
Product: dotnet-core
GitHub Login: @guardrex
Microsoft Alias: mairaw

@livarcocc @KathleenDollard can you comment on this?

I think the parameter name is --nuget-source.

Ah, you're right @livarcocc. I haven't documented the 2.1.300 SDK options yet, so this is new in 2.1.

authors: dahlsailrunner livarcocc mairaw


text:
[CLEANUP] Fix method name trimInput

Summary
The method 'trimInput' does not follow Microsoft naming conventions for method names. This will fix it using PascalCasing convention.

@KrzysztofCwalina @terrajobst the conventions only talk about public methods but it seems the .NET team also uses the same conventions for private methods as shown here: https://github.com/dotnet/corefx/blob/master/Documentation/coding-guidelines/coding-style.md
So, would you say that private methods should also use PascalCasing?

Any updates on this pull request? Thanks,

@mairaw is correct. The FDG specifically only prescribe naming conventions for public APIs. Naming of privates is considered code style.
However, I think we can still take this PR because camelCasing for private methods (while being used) isn't exactly mainstream. To my knowledge, none of our OSS code bases use that.

Thanks for the feedback @terrajobst. I'll approve this PR then.

source: gharchive/pull-request

authors: kahveci mairaw terrajobst


text:
chore: Run Markdownlint with GitHub Actions

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

Sorry, I keep selecting my fork as the target, then it creates it against this repo despite the selection


text:
Allow bundling of cross-platform native libraries

Summary

Stop including profiler files during RID publishing.
Move profiler inclusion to archive build.
Include both musl and glib based profilers in each other's archives.
Move archive build to separate stage since they depend on all build jobs to finish and publish artifacts into a unified artifact.

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

Release Notes Entry

/backport to release/7.0

/backport to release/6.x

repo: dotnet/dotnet-monitor

license: MIT


text:
when using .mbconfig with git, train time gets reset

I trained a regression model locally on my machine for 60 seconds and then saved and pushed changes to git
@luisquintanilla pulled the changes and opened the mbconfig - in the Train screen, the state of the last training run was saved and all there, but the train time in the textbox was set to 600 - we opened the .mbconfig json and saw the train time was still set to 60
Luis added a Web API from Consume step and saved everything
Luis used the ML.NET CLI to train the same .mbconfig and the train time was 600 seconds in the CLI - we opened the .mbconfig json again, and now the train time said 600 seconds

I suspect that we're use the calculated value instead of the value from the training config. We should use the training config value. When the training time value is 0 (like on first initialize) then we should run the calculation.

LGTM

authors: LittleLittleCloud beccamc briacht

repo: dotnet/machinelearning-modelbuilder


text:
ViewComponent tag helper with dictionary attribute doesn't work

@addTagHelper *, TestProject
@{
    var items = new List<string>() { "One", "Two", "Three" };
}
<vc:generic items-foo="items"></vc:generic>

public class GenericViewComponent : ViewComponent
{
    public string Invoke(Dictionary<string, List<string>> items)
    {
        return System.Text.Json.JsonSerializer.Serialize(items).Replace("\"", "");
    }
}

HTML output in .NET 5.0.408: {foo:[One,Two,Three]}
HTML output in .NET 6.0.407 and newer: null

Investigation
The relevant compiler-generated file diff (.NET 5 → 6):
 [Microsoft.AspNetCore.Razor.TagHelpers.HtmlTargetElementAttribute("vc:generic")]
 public class __Generated__GenericViewComponentTagHelper : Microsoft.AspNetCore.Razor.TagHelpers.TagHelper
 {
     private readonly global::Microsoft.AspNetCore.Mvc.IViewComponentHelper __helper = null;
     public __Generated__GenericViewComponentTagHelper(global::Microsoft.AspNetCore.Mvc.IViewComponentHelper helper)
     {
         __helper = helper;
     }
     [Microsoft.AspNetCore.Razor.TagHelpers.HtmlAttributeNotBoundAttribute, global::Microsoft.AspNetCore.Mvc.ViewFeatures.ViewContextAttribute]
     public global::Microsoft.AspNetCore.Mvc.Rendering.ViewContext ViewContext { get; set; }
     public System.Collections.Generic.Dictionary<System.String, System.Collections.Generic.List<System.String>> items { get; set; }
      = new System.Collections.Generic.Dictionary<System.String, System.Collections.Generic.List<System.String>>();
     public override async global::System.Threading.Tasks.Task ProcessAsync(Microsoft.AspNetCore.Razor.TagHelpers.TagHelperContext __context, Microsoft.AspNetCore.Razor.TagHelpers.TagHelperOutput __output)
     {
         (__helper as global::Microsoft.AspNetCore.Mvc.ViewFeatures.IViewContextAware)?.Contextualize(ViewContext);
-        var __helperContent = await __helper.InvokeAsync("Generic", new { items });
+        var __helperContent = await __helper.InvokeAsync("Generic", ProcessInvokeAsyncArgs(__context));
         __output.TagName = null;
         __output.Content.SetHtmlContent(__helperContent);
     }
+    private Dictionary<string, object> ProcessInvokeAsyncArgs(Microsoft.AspNetCore.Razor.TagHelpers.TagHelperContext __context)
+    {
+        Dictionary<string, object> args = new Dictionary<string, object>();
+        if (__context.AllAttributes.ContainsName("items"))
+        {
+            args[nameof(items)] = items;
+        }
+        return args;
+    }
 }

The compiler now tries to find attribute "items" which doesn't exist since both versions generate this line when calling the tag helper:
__tagHelperExecutionContext.AddTagHelperAttribute("items-foo", __GenericViewComponentTagHelper.items["foo"], global::Microsoft.AspNetCore.Razor.TagHelpers.HtmlAttributeValueStyle.DoubleQuotes);

Related
There's a (disabled) test for this in aspnetcore repo:

https://github.com/dotnet/aspnetcore/blob/d4be4957c24c7c8b745ade4cbaf290ad9cad1ad2/src/Mvc/test/Mvc.FunctionalTests/TagHelpersTest.cs#L61
https://github.com/dotnet/aspnetcore/blob/d4be4957c24c7c8b745ade4cbaf290ad9cad1ad2/src/Mvc/test/WebSites/TagHelpersWebSite/Views/Home/ViewComponentTagHelpers.cshtml#L10
https://github.com/dotnet/aspnetcore/issues/29597

Found this while working on https://github.com/dotnet/razor/pull/8520.
I haven't found this feature in the official docs so perhaps we don't have to support it 🤷

Triage: we'll close this as no one has reported it not working. We can reconsider if we find a customer need.

authors: chsienki jjonescz

repo: dotnet/razor


text:
update tests

Unify GENERATE_BASELINE handling to a single location for all tests
DebuggerBrowsable
Unify test diagnostic handling. We will now always verify the diagnostics reported by the C# compiler. At some point, we should consider doing the same for Razor diagnostics, but that can be done as a future step.

Note to razor team: Don't review. I'll publish when I'm ready.


text:
Revert "Suppress await warnings in async lambdas (#8346)"

Reverts #8346 as discussed in https://github.com/dotnet/razor/pull/8709#discussion_r1195069620.
Fixes warnings described in https://github.com/dotnet/aspnetcore/pull/48252#issuecomment-1548808052.
Closes https://github.com/dotnet/razor/pull/8709.
Plus adds tests to verify the current behavior (second commit).

@333fred @jaredpar @dotnet/razor-compiler for reviews, thanks

@jaredpar @dotnet/razor-compiler for a second review on this simple revert, thanks


text:
Migration: Opening a .xproj project triggers Error

Repro Steps

Copy \mlangfs1\public\Jinuj\TestCases\MigrationIssue1\AspNetCoreWebApp_NoAuth_1.zip
Unzip
Go to AspNetCoreWebApp_NoAuth_1\AspNetCoreWebApp_NoAuth_1\src\AspNetCoreWebApp_NoAuth_1
Double click the .xproj file

Expected
I should be prompted for migration
Actual

Note: I am prompted to migration UI, if I open corresponding .sln file
I have "Cross Platform and  Desktop Workload Installed.

	Product version details:
o   VS : VisualStudio/15.0.0-RC.3+26110.0.d15rel
o   CLI : 1.0.0-preview5-004460

repo: dotnet/roslyn-project-system


text:
Ability to run Angular cli & backends written in other languages

What should we add or change to make your life better?
Extend "tye run" to include Angular CLI projects & projects created from other "popular" runtimes like node and python.
Why is this important to you?
We are following a "polyglot" approach to our services. If "tye" can extend the "local process orchestrator" and "deployment" of these services to K8's environments to other run-times, "tye" can become the gold-standard for developer productivity.

I can run other Non dotnet projects as "Containers". I was able to solve the "Angular" issue by adding a "csproj" to the Angular cli project and  running it as part of "tye run". Closing this..

Closing

repo: dotnet/tye


text:
new project

URL
https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/create
Operating system
windows
More information about your system
palette commande
Description
quand je vais dans la palette de commande de vs code et que je met .net new project il me met error not found
dotnet --info
No response

And have you installed the .NET SDK and C# Dev Kit Extension?

repo: dotnet/website-feedback


text:
Removed CFP section with a temp Speakers section.

Here's the temp Speakers section for now until we get speakers selected and confirmed @thoward.

Looks like "stuff for whatever" commit is unintended? :)

Closing this one for another one...

authors: Adron thoward

license: mit

license_source: bigquery


text:
🛑 Biseri-jadrana.si is down

In 56410f5, Biseri-jadrana.si (https://www.biseri-jadrana.si/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Biseri-jadrana.si is back up in f15f000 after 5 minutes.

repo: dotsi/bitamiuptime


text:
🛑 Biseri-jadrana.si is down

In 92eb6f3, Biseri-jadrana.si (https://www.biseri-jadrana.si/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Biseri-jadrana.si is back up in 25a9875.


text:
Update sbt-tpolecat to 0.4.3

About this PR
📦 Updates io.github.davidgregory084:sbt-tpolecat from 0.4.2 to 0.4.3
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.davidgregory084", artifactId = "sbt-tpolecat" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.github.davidgregory084", artifactId = "sbt-tpolecat" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #125.

repo: doubleloop-io/applied-fp-workshop-scala


text:
Respect user's primary goal selection

Swap out primary goal for steps

Fixed in e14a6bb44 and 43940bcd

Fixed in e14a6bb44 and 43940bcd

repo: dougwaldron/readable-fitbit-clock


text:
fix validation specs for media assets

This fixes the specs for media assets regarding validations in the models.  With this change, all of our rspec tests are passing, and none are skipped!
This addresses ticket #8193.

:+1: Rebase?

@anarchivist rebased to develop

authors: AudreyAltman anarchivist

repo: dpla/primary-source-sets


text:
Support configuration for inner spacing in template literals

When writing template literals with inner spacing between braces and variable, the format job always removes them:
const text = 'Hello World!';
- const templateLiteral = `${ text }`;
+ const templateLiteral = `${text}`;

It should be possible to configure the inner spacings in template literals.
If there is already an option to change the behaviour I haven't found them.

I would greatly appreciate this being implemented!

Duplicate of #290

authors: g-plane patrickschroeter przemyslaw-zan

repo: dprint/dprint-plugin-typescript


text:
MethylDackel won't compile

Just cloned the latest repos
id: fd05cc154a8d833167767ce03212e7830061cd44
running make give me this:
$ make make -C libBigWig all make[1]: Entering directory '/home/ubuntu/MethylDackel/libBigWig' make[1]: *** No rule to make target 'all'.  Stop. make[1]: Leaving directory '/home/ubuntu/MethylDackel/libBigWig' Makefile:22: recipe for target 'libbw' failed make: *** [libbw] Error 2
gcc version is:
$ gcc --version gcc (Ubuntu 7.4.0-1ubuntu1~18.04.1) 7.4.0 Copyright (C) 2017 Free Software Foundation, Inc. This is free software; see the source for copying conditions.  There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Any help would be appreciated
thanks

Got it... just realized that libBigWig was a submodule... You should update your installation instruction to:
git clone --recurse-submodules https://github.com/dpryan79/MethylDackel.git

I recommend using the most recent release instead of the master branch. I'd planned on updating the instructions with the next release :)

Hmm, I just realized that I explicitly instruct people to do exactly what you did in the readme. Sorry about that! Yes, I really should update that ASAP.

Hi, just cloned from the  git clone --recurse-submodules https://github.com/dpryan79/MethylDackel.git
but running make stills gives me errors:
make -C libBigWig all
make[1]: Entering directory '/home/data/rrebollo/MethylDackel/libBigWig'
cc -I. -g -Wall -O3 -Wsign-compare  -c -o io.o io.c
cc -I. -g -Wall -O3 -Wsign-compare  -c -o bwValues.o bwValues.c
cc -I. -g -Wall -O3 -Wsign-compare  -c -o bwRead.o bwRead.c
cc -I. -g -Wall -O3 -Wsign-compare  -c -o bwStats.o bwStats.c
cc -I. -g -Wall -O3 -Wsign-compare  -c -o bwWrite.o bwWrite.c
ar -rcs libBigWig.a io.o bwValues.o bwRead.o bwStats.o bwWrite.o
ranlib libBigWig.a
cc -I. -g -Wall -O3 -Wsign-compare  -fpic -c -o io.pico io.c
cc -I. -g -Wall -O3 -Wsign-compare  -fpic -c -o bwValues.pico bwValues.c
cc -I. -g -Wall -O3 -Wsign-compare  -fpic -c -o bwRead.pico bwRead.c
cc -I. -g -Wall -O3 -Wsign-compare  -fpic -c -o bwStats.pico bwStats.c
cc -I. -g -Wall -O3 -Wsign-compare  -fpic -c -o bwWrite.pico bwWrite.c
cc -shared  -o libBigWig.so io.pico bwValues.pico bwRead.pico bwStats.pico bwWrite.pico  -lcurl -lm -lz
make[1]: Leaving directory '/home/data/rrebollo/MethylDackel/libBigWig'
echo '#define VERSION "0.4.0"' > version.h
cc -c -Wall -g -O3 -pthread  -IlibBigWig common.c -o common.o
common.c: In function 'getMappabilityValue':
common.c:211:5: error: 'for' loop initial declarations are only allowed in C99 or C11 mode
     for(int i = 0; i<config->chromCount; i++) //loop over chromosomes
     ^
common.c:211:5: note: use option -std=c99, -std=gnu99, -std=c11 or -std=gnu11 to compile your code
common.c:223:13: error: redefinition of 'i'
     for(int i = 0; i<end-start; i++)
             ^
common.c:211:13: note: previous definition of 'i' was here
     for(int i = 0; i<config->chromCount; i++) //loop over chromosomes
             ^
common.c:223:5: error: 'for' loop initial declarations are only allowed in C99 or C11 mode
     for(int i = 0; i<end-start; i++)
     ^
Makefile:19: recipe for target 'common.o' failed
make: *** [common.o] Error 1

Any ideas would be welcome, thank you!

Which compiler is cc on your system? Have you tried doing what the error message says and added -std=c11 to the flags? That is make CFLAGS="-std=c11 -Wall -g -O3 -pthread"?

Hi!
The cc on my system is:
cc (Debian 4.9.2-10+deb8u1) 4.9.2
Copyright (C) 2014 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

I changed the flags, but still got an error:
make -C libBigWig all
make[1]: Entering directory '/home/data/rrebollo/MethylDackel/libBigWig'
make[1]: Nothing to be done for 'all'.
make[1]: Leaving directory '/home/data/rrebollo/MethylDackel/libBigWig'
echo '#define VERSION "0.4.0"' > version.h
cc -c -std=c11 -Wall -g -O3 -pthread  -IlibBigWig common.c -o common.o
In file included from common.c:1:0:
MethylDackel.h:9:1: error: unknown type name 'pthread_mutex_t'
 pthread_mutex_t positionMutex;
 ^
MethylDackel.h:10:1: error: unknown type name 'pthread_mutex_t'
 pthread_mutex_t bwMutex;
 ^
MethylDackel.h:11:1: error: unknown type name 'pthread_mutex_t'
 pthread_mutex_t outputMutex;
 ^
common.c: In function 'parseBounds':
common.c:12:5: warning: implicit declaration of function 'strdup' [-Wimplicit-function-declaration]
     char *p, *s = strdup(s2), *end;
     ^
common.c:12:19: warning: initialization makes pointer from integer without a cast
     char *p, *s = strdup(s2), *end;
                   ^
Makefile:19: recipe for target 'common.o' failed
make: *** [common.o] Error 1

Thanks for the help

Apparently the version of gcc you're using has a different implementation of the -pthread option than most systems. Try adding #include <pthread.h> as the first line in MethylDackel.h.

Hello,
I am currently having a similar problem after issuing the  'make' command to compile MethylDackel (cloned from 'git clone --recurse-submodules https://github.com/dpryan79/MethylDackel.git')
make -C libBigWig all make[1]: Entering directory /scratch/emb19132/RRBS_prelim/MethylDackel/libBigWig'
make[1]: Nothing to be done for all'. make[1]: Leaving directory /scratch/emb19132/RRBS_prelim/MethylDackel/libBigWig'
echo '#define VERSION "0.4.0"' > version.h
ar -rcs libMethylDackel.a common.o bed.o svg.o pileup.o extract.o MBias.o mergeContext.o perRead.o
cc -Wall -g -O3 -pthread  -o MethylDackel common.o bed.o svg.o pileup.o extract.o MBias.o mergeContext.o perRead.o main.c libMethylDackel.a libBigWig/libBigWig.a -lm -lz -lpthread -lhts -lcurl
main.c:3:24: fatal error: htslib/hts.h: No such file or directory
#include "htslib/hts.h"
^
compilation terminated.
make: *** [MethylDackel] Error 1
`
Any suggestions on how to get around this issue? I have loaded updated versions of both gcc and htslib

@embertucci I suggest just installing via conda. However if you'd really like the development version then pass an appropriate CFLAGS to make:
make CFLAGS="-O3 -pthread -I/path/to/htslib/include" LIBS="-L/path/to/htslib/lib"

authors: RRebo dpryan79 embertucci mblanche


text:
How to avoid big % Dropped

Hello i want to scan all collection
I read that to improve accuracy, you need to increase the density. I increased the Desity parameter from 20 to 100 and the result is a higher % Dropped. (for example YOUTUBE 01 D2.pklz (79.70% dropped)) Some of the records became less recognizable than with standard Density = 20
How can I increase the output file and decrease the % Dropped to increase the overall recognition accuracy. I take as a basis the material I want to recognize low quality audio from tapes from the early 90's, so accuracy is important. It doesn't matter how much space the database of fingerprints will take up.

“% dropped” comes from buckets overflowing in the database. You can
increase the database size with the bucketsize flag; the default is 100.
The database in memory will grow in proportion, and at some point you’ll
run out of RAM.
DAn.
On Wed, Mar 6, 2024 at 07:27 lightcomc @.***> wrote:

Hello i want to scan all collection
I read that to improve accuracy, you need to increase the density. I
increased the Desity parameter from 20 to 100 and the result is a higher %
Dropped. (for example YOUTUBE 01 D2.pklz (79.70% dropped)) Some of the
records became less recognizable than with standard Density = 20
How can I increase the output file and decrease the % Dropped to increase
the overall recognition accuracy. I take as a basis the material I want to
recognize low quality audio from tapes from the early 90's, so accuracy is
important. It doesn't matter how much space the database of fingerprints
will take up.
—
Reply to this email directly, view it on GitHub
https://github.com/dpwe/audfprint/issues/94, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEGZUKBROI5XN6CAC73KYLYW4DRZAVCNFSM6AAAAABEI6VCB6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3TCMZYHE3TKMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you i'll check thatt parameter to find optimal size and memory comsumption in process, speed is not critical.

You can try to use my fork with some minor changes. Recognition speed is increased and memory consumption is reduced. - https://github.com/ZhymabekRoman/audfprint-enhanced

You can try to use my fork with some minor changes. Recognition speed is increased and memory consumption is reduced. - https://github.com/ZhymabekRoman/audfprint-enhanced

Are you doing any active commits anymore?

You can try to use my fork with some minor changes. Recognition speed is increased and memory consumption is reduced. - ZhymabekRoman/audfprint-enhanced

Are you doing any active commits anymore?

Yeap

authors: ZhymabekRoman dpwe godzfire lightcomc

repo: dpwe/audfprint


text:
⚠ 366.ru has degraded performance

In 99ef753, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 5109 ms

Resolved: 366.ru performance has improved in 0c22c78.


text:
New Theme - Textastic

I made some subtle changes to your Textmate scheme to allow it to work with Textastic, a text editor for the Apple ecosystem.

https://github.com/kjharcombe/textastic/tree/master

That's super cool @kjharcombe!
Feel free to transfer to the org now ;)

https://draculatheme.com/textastic
Awesome stuff @kjharcombe ;D

authors: kjharcombe zenorocha

repo: dracula/dracula-theme


text:
Increase room number

List of changes:

backend database reorganized from arrays to objects to improve flexibility
room number limit increased, coded by letters
room enter interface changed (focused on features like entering a specific room, finding empty or staying in private)
fixed bouncing links while modal opening

This is really good PR, especially the backend changes, thank you.
I wonder how you managed to navigate in my messy code so well. You have my appreciation 👍
It's a good idea to simplify the room selection. And to allow for a larger number of rooms.
I like the "find random empty room" feature.
Some problems, however:

I'd like to preserve the possibility of easily accessing the default main room (gid: 0) - there is no obvious way to do that now.
does the "enter room hash" modal makes sense? You'll usually just click the link someone sent you. And you can always just edit the URL in the address bar, right? So what is the reason behind it?
I don't like the icons, especially the seat-recline-extra - Not a big deal, however.
Is it obvious, that "private room" means offline room just for you? 🤔 Two people can have privacy in the custom room.
"empty room" is more like a command, not actually a room (cannot be selected) so it should have different design
it is not clear where the numbers 141167095653375/6 come from.
Could you use a named constant, for example, const MAX_HASHABLE_INT = 26**10 - 1 ?

My proposal on how to change the room "tabs"

static tab labeled either main room, global room or public room, gid: 0 - we can use mid-globe as icon for example
static tab labeled offline mode or incognito mode gid: -1 - it can have the mid-incognito icon (but I might change it to something nicer later)
static button new empty room as the last item, with icon mdi-plus-thick- it should have background: transparent; box-shadow: none; to distinguish it from the selectable tabs
dynamic list of custom rooms tabs (gid > 0), which were created/visited by the user

with this, you can easily return back to the room you just left
we can use some of the mdi-numeric- or mdi-alpha- icons for those (the number/letter in the icon will be either index or derived from the room hash)
not sure whether the list should be placed between main and incognito tabs, or between the incognito tab and the plus button
this list could be limited to some small number, so it won't grow indefinitely when clicking the plus button (either the plus will disappear when the limit is reached, or the oldest/left-most room will be removed)
the list might be preserved in localStorage, so it won't be lost on refresh - in that case, it would probably be nice to add an option to remove a room from the list

What do you think?
I can implement that by myself.
I changed the destination branch from master to dev.
That way I can merge this PR and implement the changes without deploying it (the frontend is hosted on GitHub Pages, automatically deployed from the master branch).
Also, for the rooms with gid > 255, the sound from other players does not actually play.
It is  probably because the gid and uid values are being sent with each action inside of Uint8Array - so the values are truncated to fit in 8 bits https://github.com/drahoslove/pianco/blob/master/js/io.js#L374
and are checked on arrival https://github.com/drahoslove/pianco/blob/master/js/io.js#L224-L227
This check is probably not necessary, since the messages are filtered on backend. -> Let me fix that.

Thank you for your feedback!
I agree, frontend is a tricky part here. Let's discuss!
Problems:

Well, of course we can leave it there, but I don't think users need it.. I thought we shouldn't focus on specific rooms.
I just think editing hash in modal is more a user-friendly way. I agree it duplicates functionality, but it isn't obvious that user can just edit the URL to change a room. Personally, I had no idea) Firstly, I wanted to put a room indicator, but with hash it would be space consuming. So I left the room change modal with an indicator instead.
I don't insist on them, we could choose something better
That's a good one! Well, it isn't obvious.. I just feel that "isolation" word has a negative connotation. "Solitude" is better but I've used "private". We have linguistic issues) That's why I choose "incognito" icon. Like no one can hear you. Maybe it's all nonsense, you tell me)
Totally agree!
My bad. I should've replaced by a constant, agree.

I like your proposals 1-3: 'main room', 'offline mode' and 'new empty room'.
4 proposal:

I think it is a good idea to save a room number people randomly chose by 'new empty room'. If they want to continue, they wouldn't have to choose a new one, share number. We saved it for them.
Well, wouldn't it be space consuming to use large numbers or hash? We will have like two saved rooms that way..
I personally like it being between main and incognito tabs
Agree. I'd prefer disabling button. It is a good interface!
We could open a modal onclick: 'enter' or 'delete' room. But it doesn't feel right.. Or we could add another button for removing saved rooms. It would show the list, and user will choose what room to remove. And it appears if user have saved rooms.
So, there is a question left about modal for editing hash. Do you still feel it excessive?

I don't mind if you implement changes. You definitely feel the app's style better :)

Great. I'm glad you mostly agree :)

Well, wouldn't it be space consuming to use large numbers or hash? We will have like two saved rooms that way. - Not sure you understand me - I meant to use a single icon - for example if the hash of the room starts with letter 'b', the icon could be mdi-circle-b, or something like that. I'll just try something and will see what works well.
I'd prefer disabling button. - yes, disabling is better than hiding 👍
the tab of custom room which is empty could have a little (×) button for closing, which could appear on mouse hover, like this:

(I know the hover feature It's not ideal for mobile devices, but I'd like to change the UI for those devices anyway, because currently it is too much cluttered on the smaller screens and you could easily accidentally switch the room when shifting octaves. So I'll probably hide all the tabs under a single button which would open a list of rooms in a modal window, for small devices.)

So, there is a question left about modal for editing hash. Do you still feel it excessive?
I mainly just still don't understand what's the use case for it.
Could you describe when would you need to enter the hash manually?

Either you create a new room, in which case you don't care about the letters, and you'll just copy and share the URL to whoever you want to share it with.  (It would make more sense to implement some share link button, which would copy the URL to your clipboard - that might be helpful for phones, where the URL copying is a bit more annoying)

Or you are the other person, who just received the URL from someone, in which case you just click the link.

Or you might not want to / be able to send the link, because you are with the person either on a phone call or meeting in person - in those situations you'll probably be dictating the whole address anyway.
What I could do to make that easier, is remove the #, so the URL would be just pian.co/xxx-xxxx-xxx - that's a bit easier to read and to type.
Btw, the dashes are for better readability? I think those could be removed too. And we could make the string a bit shorter as well, eg. 8 of base32 characters - that's still plenty of possible values to ensure privacy.

To me, the modal for entering the room name would make sense if you could enter arbitrary letters to name the room using words. (This might be an interesting idea for the future, but I'd rather keep it simple for now.)

Interesting idea about letter icons, but they could duplicate.. Or just use numbers like now? Yeah, looks like we have to try some options and see
This is a nice (×) button design! I love it) Agree about small devices
That's it! 'Share link' is an ultimate solution to do it user-friendly way, from my point of view)

Looks like we've come to an agreement :)

authors: DanZalov drahoslove

repo: drahoslove/pianco


text:
additional tests

Addresses #171. This brings in a bunch of tests that have tested the scalability of cbat. Note that they also introduce a dependency to run the test suite on gcc-8 for the loop unrolling pragma. This may be installed with:
sudo add-apt-repository -y ppa:ubuntu-toolchain-r/test
sudo apt-get -y update
sudo apt-get -y install gcc-8 g++-8

The tests involved include:

a hash function.
a solver for the n-queens problem. Currently set to n=4 for quick completion.
a 4 by 4 sudoku solver.
a comparison of of LSB calculations between two binaries. The first method uses debruijn sequences and the second method uses a for loop.

Okay I've removed the 128 bit code because it's too much for now. I've also added in a readme for debruijn that provides a brief summary of how it works. Finally, I've removed the gcc-8 dependency (should now work with any version of gcc!).

This is not ready to be merged. There are two outstanding issues (thanks @gltrost for pointing these out!):

nqueens has multiple solutions. All these solutions should be added as options so the test always passes.
Removing the gcc-8 dependency results, and using gcc-5 with a different set of pragmas instead results in the 8 and 16 bit debruijn loops being completely unrolled. However the 32 and 64 bit loops do not get unrolled completely.

I've unrolled the 32 and 64 bit loops manually for debruijn. The issue now is that there is an implicit loop in the 64 bit version with debruijn sequences. WP does not finish in a reasonable amount of time when manually unrolling the loop 64 times. GCC-8 does not have this problem. Since we're encountering performance problems with the 32bit version, I'm going to delete the 64bit version for now.

I'm getting SAT for Debruijin 64-bit. Am I doing something wrong here?

Oops I forgot to upstream my changes (they should be pushed now). I'm removing 64bit debruijn for the time being so we don't have a dependency on gcc-8. Note that this isn't ready to be merged; it needs to be rebased on top of #228

Okay, I've rebased onto master! Nqueens has two solutions (which are now both included).

I've added comments on the expected output to each run_wp* script!

Looks good to me! I think wp/resources/sample_binaries/hash_function/.run_wp.sh.swp was accidentally checked in, but other than that, I think it's all good! Does @gltrost have any more input?

Oops. Should be unstaged now :+1:

authors: DieracDelta fortunac


text:
feat: only and skip support

Fixes issue 5
WIP
Developments so far

Implemented only usage

Only a single suite or case can be only'ed, if more than 1 is, theen rhum throws an error
Will show all other tests as ignored

added skip support

same as above but as many test *'s can be skipped

Temporarily changed the output of rhum, see below

What's left

Complete and fix integration tests for only
add unit and integration tests for skip
add unit tests for only method
correct the output format, and adjust tests for this

Different types of output

Developments so far:

Implemented only usage

Only a single suite or case can be only'ed, if more than 1 is, theen rhum throws an error
Will show all. other tests as ignored

Temporarily changed the output of rhum which hasn't been best received, so it needs to be changed again - i'm proposing we ditch the current format but keep the test  bit in, so it's easier to write tests for

Decided to add skip support as well, because the only way only or skip will land, is if we ditch current output format, so may as well do it in one pr

can we rework this PR to be pointed to v2? or should we trash it and start over with a new branch, but taking the logic from this branch as a starting point?

Oh we can trash this PR, I was only keeping it open as a reference really until we landed your pr, if this PR is no use to you then feel free to close it and delete the branch 👍

i think it's still useful. we can keep it open as a reference for .only(). i like the edge cases so let's keep it open for now

Alright

Do we need only and skip properties for every test interface? It seems like it just overcomplicates things. For example, what happens if both only and skip are set to true?

Hey, yes because it means any test suite or case can be skipped, and i believe i added checks for if both are used but i can't remember, this PR was from a while ago :P Tbh, we're only keeping this PR up for reference purposes for the rhum v2 PR, none of this code will end up in master and when rhum v2 is in master, this pr will be closed and the code lost forever 😢

I know. I just realized that this getting this to work is probably not possible if we keep using deno.std/testing as a base, unless we find some way to change the message format for deno.std/testing. The only way I can see to get this work is just to write our own base for testing instead of working off of deno.std/testing.

So basically we would have to write our own "deno:cli/rt/40_testing.js".

it might be that we don't need this feature now that i think about it. i say this because rhum v2 CLI has the --filter-test-case and --filter-test-suite options

I think those options achieve the same functionality as skip, but I'm not sure those options achieve the same functionality as only. I'm pretty sure ebebbington's code would have worked if Deno's testing api had more customizable format options.

My pr was going to use a different test definition so we could use skip and only, but this meant the output would have changed in the shell for users
Where as now with Rhum v2, we have good output and can still ignore and only tests

For reference, this PR is only open to aid in eric's v2 PR, should any of the work i have done come in handy.
Once the v2 pr is merged, this will be closed without merging

not doing this anymore. v2 will be a test double framework

authors: MVEMCJSUNPE crookse ebebbington

repo: drashland/rhum


text:
🛑 admin.drawbeat.com is down

In 823f507, admin.drawbeat.com (http://admin.drawbeat.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: admin.drawbeat.com is back up in 2c56e11.

repo: drawbeat/upptime


text:
UI/UAT REVIEW - Cart [Empty Cart]

[ ] ukuran section "keranjang kosong" tidak full, tidak sama dengan desain

figma:

staging:

[ ] ada hyperlink untuk belanja lagi di staging, seharusnya hanya ada 1 button belanja lagi seperti di desain

figma:

staging:

[ ] button belanja sekarang tidak tepat di tengah

figma:

staging:

[ ] tidak ada section footer?

figma:

staging:

[ ] tampilan di mobile juga ada hyperlink untuk belanja lagi, dan tidak ada section footer

[ ] ketika button Belanja Lagi di click, redirect ke https://urban.bstage.me/[lng]/id/products

[ ] terjadi juga ketika hypelink Belanja Lagi di click, redirect ke https://urban.bstage.me/[lng]/id/products

done

repo: dreamcubeid/urban


text:
Implement mconcatdefault

It might be handy to have mconcatdefault defined!

Do you have a use case for this? mconcat already has a default definition, and I doubt that implementing mconcat via GHC.Generics would be any more performant (in fact, it'd likely be worse).

I don't have a use case for this at the moment, just crossed my mind when using this package.
One use case could be for the monoid instance for a product type containing a member which has a significantly faster mconcat. Without looking, I couldn't say whether any of the instances of GMonoid satisfy this property though...
I'm happy to close this if this isn't the case.

One use case could be for the monoid instance for a product type containing a member which has a significantly faster mconcat.

I'm not sure what you mean here. The default definition of mconcat is literally just foldr mappend mempty, so to my knowledge, we never recursively call mconcat.

Yeah, I've just looked: None of the instances of GMonoid have a faster mconcat, so this is almost certainly not a win.

repo: dreixel/generic-deriving


text:
One to One Video Conference Chrome Problem When show other user Video first time

In the project, we use free agora rtc for video and voice conversations of two people in video calls. In this section, when a user calls another user, the Chrome browser is too late to display the first video image of the other person. This problem is not in Firefox. Can you help with Chrome as a very urgent cause of this problem and how we can solve it?

Hi @caglarturkurka I suggest you contact the folks at agora.io for this one. Any browser compatibility issues are dependent on the Agora SDK. I hope you were able to resolve this!

authors: caglarturkurka drew-thompson

repo: drew-thompson/ngx-agora


text:
Suggestion: Generated providers

Currently when you generate a provider using the ionic@beta cli you get a provider that has http imported and injected and a simple promise based http request. I would suggest that our generated providers should instead use an observable based template based off of something like this:
import { Injectable }     from 'angular2/core';
import { Http, Response } from 'angular2/http';
import { Observable }     from 'rxjs/Observable';

@Injectable()
export class DataService {
  constructor (private http: Http) {}

  getData (): Observable<any[]> {
    return this.http.get(/*api endpoint*/)
                    .map(this.extractData)
                    .catch(this.handleError);
  }

  private extractData(res: Response) {
    if (res.status < 200 || res.status >= 300) {
      throw new Error('Bad response status: ' + res.status);
    }
    let body = res.json();
    return body.data || { };
  }

  private handleError (error: any) {
    // In a real world app, we might send the error to remote logging infrastructure
    let errMsg = error.message || 'Server error';
    console.error(errMsg); // log to console instead
    return Observable.throw(errMsg);
  }
}

This would put it more in line with what i think is considered "standard" for Angular 2 and I think gives a better starting point for devs to build their provider off of. Would love to hear what anyone else thinks of this! Is something like this too complicated for just a simple generated provider?

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.

repo: driftyco/ionic-cli


text:
"TypeErrpr: Cannot read property 'includes' of undefined".

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x ] 3.x
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
I was using the Ionic 3 beta CLI, and tried to install the ConnectivityService. It couldn't work in that version, so I performed a clean uninstall of ionic, and then, a new install aiming to ionic@latest. It gave a dependency error on app-scripts, so I used the following command:
npm install --save @ionic/app-scripts@^1.3.1
Then, I used again "npm install -g ionic@latest". After that, everything stopped working. I can't do anything with ionic anymore; I just keep having the same error with whatever I do: "TypeErrpr: Cannot read property 'includes' of undefined". It doesn't matter if I uninstall it and install it again; I just can't get it working.
Expected behavior:
The new version should have been properly installed and the provider, too.
Steps to reproduce:
(already told)

Any help regarding this? My project is stopped from this point and I can't get to test anything.

This issue was moved to driftyco/ionic-cli#2285

repo: driftyco/ionic


text:
bug: $resolve is undefined when used within an ionView

Type: bug
Ionic Version: 1.x
Platform: mobile browser
To reproduce create a component, add a resolve in the state and use that resolve for the binding, ie:
<my-component events="$resolve.events"></my-component>
```

Don't remember writing any of that :confused:
The bug i reported is at #8140


text:
Error starting project: npminstall error

Short description of the problem:
I've tried to start a simple project on the new version of Ionic 2: 2.0.0.rc0
But the following error appeared:
X:\Desenvolvimento\Ionic>ionic start test blank --v2
Creating Ionic app in folder X:\Desenvolvimento\Ionic\test based on blank project
Downloading: https://github.com/driftyco/ionic2-app-base/archive/master.zip
[=============================]  100%  0.0s
Downloading: https://github.com/driftyco/ionic2-starter-blank/archive/master.zip
[=============================]  100%  0.0s
Installing npm packages...
Error with start undefined
Error Initializing app: There was an error with the spawned command: npminstall
There was an error with the spawned command: npminstall
Caught exception:
 undefined

I've read other problems similar to mine, they said to update de npm to a 3.x or later version. I did it, but still the same error.
The node_modules folder is missing too.
When it does appear. I simply can't delete the project folder, because the Windows File System doesn't allow a huge directory names (Because inside the node_modules folder, there is a bunch of endless subfolders). And I can't use Linux right now.
Cordova CLI: 6.3.1
Gulp version:  CLI version 3.9.1
Gulp local:
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.1.0-beta.1
OS: Windows 10 Home x64
Node Version: v6.7.0

Have Git installed.

Thank you Verean. Worked for me

├── cordova@6.4.0
├── ionic@2.1.18
├── ios-deploy@1.9.0
├── ios-sim@5.0.13
├── npm@3.10.9
└── typescript@2.1.4
system version: MacOS 10.12.2 Sierra
git version 2.8.1
I aslo got this problem, my environment is above. All tool is newest. Git also installed. Can you tell me that how you solve this problem?
Thanks.

authors: Fconstant Verean sudotamm


text:
docs(ionicons): move to top level

Short description of what this resolves:
Updates the link to the Ionicons (according to commit: https://github.com/driftyco/ionic-site/commit/e75930d9324f4ab97d6e335ebca1ae009e60610d).
Changes proposed in this pull request:

Update of the Ionicons link in the ion-icon docs.

Ionic Version: 2.x
Fixes: #

thanks

authors: adamdbradley iignatov


text:
fix(chip): Icon alignment fix

Short description of what this resolves:
UI fixes for <ion-chip> component (icon in chip alignment).
Ionic Version: 2.x
Fixes: #5386

Thanks for the PR! I'm not able to see any issues with the icon alignment. Please see my comment here: https://github.com/driftyco/ionic/issues/5386#issuecomment-256179231

Thanks! I made a small change to use a variable in case people want to override this but I committed it here: https://github.com/driftyco/ionic/commit/ace727e627b56793464d7bdd77ffd5546de49220

authors: brandyscarney nakulgulati


text:
Target not found: ttf_dejavu

Not sure why or what's going on i'm following all the instructions and am using the florida mirror. Has anyone else had this problem?

Okay i've made some progress turns out all the font files are named ttf-dejavu without and underscore is there anyway to fix this? @elkangaroo

Thanks for your report, Noah. From your email I got that you are on an ARM chromebook.
What's the exact command you run for installing chroagh (crouton), including the targets and mirror?
I don't see any of the fonts with underscore in the code, I wonder why it wants to install 'em that way?!

I know it's driving me nuts I can't figure it out either. I spent some time trying to tinker with the files had no success. Anyway here is my installation command!
sudo sh -e installer/main.sh -r arch -t xfce -m 'http://fl.us.mirror.archlinuxarm.org/armv7h/$repo'

that looks correct so i'm not sure what it going on. If we can work together and fix this it would be awesome :D

I think I found something in target xorg (which is a requirement for your target xfce):
Can you please try to change line https://github.com/elkangaroo/chroagh/blob/master/targets/xorg#L194
install arch=ttf_dejavu into install arch=ttf-dejavu and see if that installs?

Okay all is good I have made the changes on my branch and guess what???? It installs which is fine and dandy. I can run sudo enter-chroot and that works! I run into problems when I try to run the desktop. On running sudo startxfce4 I get the following error
Entering /mnt/stateful_partition/crouton/chroots/arch...
chown: invalid user: 'messagebus:messagebus'
/usr/bin/startxfce4: Starting X server

/usr/lib/xorg-server/Xorg.wrap: Only console users are allowed to run the X server
/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: Connection refused
/usr/bin/xinit: server error
Unmounting /mnt/stateful_partition/crouton/chroots/arch...

Good to hear the change fixes the install on your chromebook :+1:
As for the error starting X (xorg-server): I got the same error on a x86_64 chromebook with a current arch  mirror and I did not find a solution yet.
As far as I know, the root cause are the changes in xorg-server 1.16, that why it should work if you find an old mirror with an xorg-server < 1.16, but I don't know if there are any for Arch ARM.
I already spend several days trying to fix this for current versions of xorg-server, but my knowledge there is limited and I did not get it to work yet :(

authors: NoahVlncrt elkangaroo

repo: drinkcat/chroagh


text:
[BUG]: ERROR: Transforming const to the configured target environment ("es5") is not supported

What version of drizzle-orm are you using?
0.27.0
What version of drizzle-kit are you using?
0.19.2
Describe the Bug
Created a new next13 project and setup drizzle/drizzle kit using the quick start https://orm.drizzle.team/kit-docs/quick. Tried to run the generate script - dirzzle-kit generate:pg and got the following error:
Error: Transform failed with 1 error:
<path-to-app>/src/db/schema.ts:3:7: ERROR: Transforming const to the configured target environment ("es5") is not supported yet

errors: [
    {
      detail: undefined,
      id: '',
      location: [Object],
      notes: [],
      pluginName: '',
      text: 'Transforming const to the configured target environment ("es5") is not supported yet'
    }
  ],

Downgrading to version 0.18.1 and 0.26.5 runs the generate command successfully not sure why this occurs with the latest version
Expected behavior
Run generate migrations successfully with drizzle kit version 0.19.2 and nextjs 13
Environment & setup
No response

Additionally, if you want to just fix that with patch-package then you can use this patch:
diff --git a/node_modules/drizzle-kit/index.cjs b/node_modules/drizzle-kit/index.cjs
index b75ca3b..2c41c65 100644
--- a/node_modules/drizzle-kit/index.cjs
+++ b/node_modules/drizzle-kit/index.cjs
@@ -10441,6 +10441,7 @@ var require_node2 = __commonJS({
         var configString = readFileSync4(configFilePath);
         var cleanedJson = StripBom(configString);
         var config = JSON5.parse(cleanedJson);
+        config.compilerOptions.target = "ESNext";
         var extendedConfig = config.extends;
         if (extendedConfig) {
           if (typeof extendedConfig === "string" && extendedConfig.indexOf(".json") === -1) {

See https://dev.to/zhnedyalkow/the-easiest-way-to-patch-your-npm-package-4ece

Actually @samhithk I think I made may have not got the patch correct. I thought it was working but now I am not sure - I'm still tinkering so I will post again if I get more info

Ok, I think I found a better way:
I created a file called tsconfig.drizzle.json with these contents:
{
  "extends": "./tsconfig",
  "compilerOptions": {
    "target": "ESNext",
  },
}

I then patched the node_modules/drizzle-kit/index.cjs by replacing all instances of tsconfig.json with tsconfig.drizzle.json.
I got this patch from using patch-package:
diff --git a/node_modules/drizzle-kit/index.cjs b/node_modules/drizzle-kit/index.cjs
index b75ca3b..5b115da 100644
--- a/node_modules/drizzle-kit/index.cjs
+++ b/node_modules/drizzle-kit/index.cjs
@@ -10402,7 +10402,7 @@ var require_node2 = __commonJS({
       }
       function resolveConfigPath(cwd, filename) {
         if (filename) {
-          var absolutePath = fs32.lstatSync(filename).isDirectory() ? path3.resolve(filename, "./tsconfig.json") : path3.resolve(cwd, filename);
+          var absolutePath = fs32.lstatSync(filename).isDirectory() ? path3.resolve(filename, "./tsconfig.drizzle.json") : path3.resolve(cwd, filename);
           return absolutePath;
         }
         if (fs32.statSync(cwd).isFile()) {
@@ -10415,7 +10415,7 @@ var require_node2 = __commonJS({
         if (existsSync4 === void 0) {
           existsSync4 = fs32.existsSync;
         }
-        var configPath = path3.join(directory, "./tsconfig.json");
+        var configPath = path3.join(directory, "./tsconfig.drizzle.json");
         if (existsSync4(configPath)) {
           return configPath;
         }
@@ -10711,7 +10711,7 @@ var require_node2 = __commonJS({
         if (!loadResult.tsConfigPath) {
           return {
             resultType: "failed",
-            message: "Couldn't find tsconfig.json"
+            message: "Couldn't find tsconfig.drizzle.json"
           };
         }
         if (!loadResult.baseUrl) {
@@ -10922,7 +10922,7 @@ var require_node2 = __commonJS({
     });
     var getOptions = (cwd) => {
       var _a, _b, _c, _d;
-      const { data, path: path3 } = joycon.loadSync(["tsconfig.json", "jsconfig.json"], cwd);
+      const { data, path: path3 } = joycon.loadSync(["tsconfig.drizzle.json", "jsconfig.json"], cwd);
       if (path3 && data) {
         return {
           jsxFactory: (_a = data.compilerOptions) == null ? void 0 : _a.jsxFactory,

Hi @saborrie this worked for me (changing temporarily to ESNext). Do you know if this is a known issue and is being fixed by someone?

@andresgutgon I'm not a contributor to Drizzle, hopefully this issue has been seen by the Drizzle team. Unfortunately Drizzle-kit is not open source at the moment so we can't submit pull request fixes even if we wanted to.

Potential related Issues / PRs:

Drizzle-Kit 0.19.0 Fails to Generate Migrations: This issue is similar to the one you're facing, where the drizzle-kit generate:pg command fails with an error. The linked issue discusses a similar error message and provides some insights into the cause and potential solutions. It might help you understand the underlying problem and find a workaround or fix for your issue.
SyntaxError: Cannot use import statement outside a module with drizzle-kit: This issue discusses a syntax error that occurs when using drizzle-kit. The error message and stack trace provided in the issue are similar to the ones you're experiencing. The linked issue might provide some insights into the cause of the error and potential solutions or workarounds. It could be helpful in troubleshooting and resolving your issue.

This message was generated by AI from https://www.triagefeedback.com

i attempted this fix on my nextjs project and encountered
import type {SchemaType} from '../../CodegenSchema.js';
^^^^^^

SyntaxError: Cannot use import statement outside a module

I couldn't find anyone talking about how to workaround this specific issue

es5 as a target is not supported, just try to use higher versions. es6 should work well

Next.js has es5 configured as its default and I assume not without reason. What is your suggestion?

Here's another solution without creating an extra tsconfig.json, just hard code the target option.
I'm using pnpm patch to patch this.
diff --git a/index.cjs b/index.cjs
index b913e366250329eb9a6684fe711af908a1f8d75e..3b8a20975cdb47b8fd972fcd059ad7beef680371 100755
--- a/index.cjs
+++ b/index.cjs
@@ -11148,7 +11148,7 @@ var require_node2 = __commonJS({
           sourcefile: filename,
           loader: getLoader(filename),
           sourcemap: "both",
-          target: options.target,
+          target: "es6",
           jsxFactory: options.jsxFactory,
           jsxFragment: options.jsxFragment,
           format,

es5 as a target is not supported, just try to use higher versions. es6 should work well

Thank you for the suggestion! I tried using es6 as the target and it worked well.

I just removed target in my Next.js project and it works fine. According to this comment it should be ok to remove it from tsconfig.

es5 as a target is not supported, just try to use higher versions. es6 should work well

I'd still like to see a fix on the Drizzle kit side - there's absolutely no reason why drizzle kit should be stuck using the same tsconfig as the src directory - drizzle kit shouldn't cause clashes like this.

What version of drizzle-orm are you using?
0.27.0
What version of drizzle-kit are you using?
0.19.2
Describe the Bug
Created a new next13 project and setup drizzle/drizzle kit using the quick start https://orm.drizzle.team/kit-docs/quick. Tried to run the generate script - dirzzle-kit generate:pg and got the following error:
Error: Transform failed with 1 error:
<path-to-app>/src/db/schema.ts:3:7: ERROR: Transforming const to the configured target environment ("es5") is not supported yet

errors: [
    {
      detail: undefined,
      id: '',
      location: [Object],
      notes: [],
      pluginName: '',
      text: 'Transforming const to the configured target environment ("es5") is not supported yet'
    }
  ],

Downgrading to version 0.18.1 and 0.26.5 runs the generate command successfully not sure why this occurs with the latest version
Expected behavior
Run generate migrations successfully with drizzle kit version 0.19.2 and nextjs 13
Environment & setup
No response

Change option compilerOptions on file tsconfig.json: "target": "es5" to "target": "esnext"

authors: AndriiSherman andresgutgon bernhardw cusxio jokull matthewwong525 mgarf rajanshresth saborrie samhithk vinatechpro

repo: drizzle-team/drizzle-orm


text:
Repository maintenance script has incomplete functionality

The repository maintenance script USElecData currently has incomplete functionality. The following are currently broken or not implemented:

[ ] The source option does not respect the --resource-all flag. It will currently only update the existing source data.
[ ] The build option does not respect the --rebuild-all flag. It currently will only update the current build with new source data.
[ ] The package option only exports .zip files. It should examine the file extension and export .tar.gz files or remove the option from the description.
[ ] The delete option is not implemented.

Allow setting year bounds in the init script

The source and build options will now pass --resource or --rebuild flags to subprocesses when those flags are specified. The loadCEMSData.R script will respect the --rebuild flag.

repo: drofphilosophe/USElecData


text:
Support default object expiration

This could be useful when combined with our existing flush logic
https://aws.amazon.com/about-aws/whats-new/2011/12/27/amazon-s3-announces-object-expiration/
For example imagine that we have a small node project with a below average amount of dependencies that results in a 1GB cache file 😈 . If the project is inactivate (or is deleted), builds are not running, and files might not get flushed. Expiration could be a useful fallback here.

Do you have some proper yaml schema for that in mind? With the latest Minio sdk this can be integrated.

Do you think the default branch should have a different expiration time?

I guess that's up to the drone user :)

Another way to support this for S3 without changing the plugin is to leverage default lifecycle rules at the bucket level.

authors: bradrydzewski donny-dont ewang tboerger


text:
[x]  fedor/test-bad-links-detection * [x]  push_changes_master * [x]  create_portfolio

[x]  fedor/test-bad-links-detection
[x] add_continuous_intergration
[x] merge_pull_request
[x]  create_portfolio
[x]  push_commits_upstream

[x]   fedor/test-bad-links-detection
[x]  add_continuous_intergration
[x]  merge_pull_request
[x]   create_portfolio
[x]   push_commits_upstream

repo: drone/drone


text:
Issue with vehicle.commands, we cannot access last element using cmds[-1]

Beacuse of changes in the CommandSequence class, especially in the line 2791 of the __init__.py file, we cannot access commands using negative indexes like [-1] for last element (it adds +1, which makes it 0). I am pretty sure that around one year ago it was working fine.
Here's a snippet of my old code:
        cmds = vehicle.commands
        cmds.download()
        cmds.wait_ready() # wait until download is complete.

        if cmds[-1].command != mavutil.mavlink.MAV_CMD_NAV_LOITER_UNLIM:

I think CommandSequence should support negative addressing or at least correct "Operations include ‘array style’ indexed access to the various contained waypoints." line it the docs :)

Well over a year ago the cmds were set so you couldn't over or underrun. This was to remove the zero'th hidden home position waypoint from the array - which was confusing to users.  The behaviour may change at some point to better support PX4 - which doesn't have the zero position home waypoint.
@peterbarker  Anything to add?

authors: Ekci hamishwillee

repo: dronekit/dronekit-python


text:
Standardizing commands

I've extracted an interface called IdentifiableResults to mark all the commands that uses the OutIdentifier property to return a value.
In a different commit I've modify the SetGlobal Command to not use the out boolean and work in the same way at the other commands that uses the OutIdentifier. Notice that the out boolean was set True and no one else was using it.
This is the first step to improve and standardize the commands

Merged

Rather, I'm not seeing an implementation of the IdentifiableResult interface, and that puzzles me. :/

authors: Salaboy ge0ffrey mrietveld

repo: droolsjbpm/drools


text:
[6.4.x] GUVNOR-2352 : Adding of manual dependency does not work correctly

Same as the one in master.
(cherry picked from commit 394f642e5555af3f4250a953aba5ee2758028296)
https://github.com/droolsjbpm/kie-wb-common/pull/327

+1 to merge.

6.4.x: f22a6be9cc967733b5e9a44d6c9e31a7688d3b40

authors: Rikkola csadilek manstis

repo: droolsjbpm/guvnor


text:
shared links

For some reason i can't get the shared links in a folder, it always returns null (or empty array)
here's the code i'm using
 DropboxClient client = GetDropboxClient(Profile);
            string sharename = "";
            if (string.IsNullOrEmpty(Folder))
            {
                Folder = "/";
            }
            try
            {
                ListSharedLinksResult results = await client.Sharing.ListSharedLinksAsync(Folder);
                sharename = string.Join(",", results.Links);
            }
            catch (Exception ex)
            {

            }

is it a known bug? or do i need to user another method?

The ListSharedLinksAsync method will return links that "allow access to the given path - direct links to the given path and links to parent folders of the given path".
You mentioned you're trying to get the shared links in the folder though, which ListSharedLinksAsync won't return. You'll need to call it and specify the path(s) for the specific item(s) that you're looking for.

repo: dropbox/dropbox-sdk-dotnet


text:
Add support for parametrized commands with parametrized configurations

Jackson is able to deserialize parametrized configuration classes, so we should support configured commands with them. To figure out the type, we need to check the raw type one of the type arguments and check if it's can be cast to the Configuration class.
Simply put, for the class
RunDbVacuum extends ConfiguredCommand<ReplicaConfig<Replica>>, Generics should be able to figure out that we need to deserialize the config as ReplicaConfig.
Resolves #2187

Changes Unknown when pulling 43939f48983e7264da66bbedc1a38eb62191d40d on support_parametrized_commands_with_parametrized_config into ** on release/1.2.x**.

@bearrito Please review!

I'm not able to build and test at the moment.
Your analysis of the problem is correct. My issue was identical to RunDbVacuum extends ConfiguredCommand<ReplicaConfig<Replica>>, Generics.  If this PR solves that problem then LGTM.

Thanks for the review!

authors: arteam bearrito coveralls

repo: dropwizard/dropwizard


text:
Code safety in ExponentiallyDecayingReservoir

While reading the implementation, these two things jumped out at me.
Getting a random double == 0.0 and overflowing a long counter are unlikely, but who knows.

I'm sold on the random fix, but not on the long overflow. At 1mil event/sec it'd take almost 300,000 years to get there.

Cherrypicked 172e44b as 258e80b25473184535df4f0c7ec9a80554b21f6a

authors: bwarminski ryantenney

repo: dropwizard/metrics


text:
Migrate to setuptools

Close #4

Reorganize the file hierarchy. The module live inside the src dir. Build files are located in the root dir. This should be mirrored into SPT install script (I will make dedicated a PR. see an example against my branch https://github.com/rjanvier/superpoint_transformer/blob/a07d92db07e1cdf8a228cb6d26814257bf6d7f85/install.sh#L106-L112)
Migrate to setuptools. Pgeof install script is now pep 517 "compliant". it could be installed by symply using pip install . on the root of the directory. In order to achieve this, the way Eigen lib include dir is passed to the build system was changed. We now use env variables because of deprecation of some config flags in pip (https://github.com/pypa/pip/issues/11859) and because setuptools does not implemente alternatives (https://github.com/pypa/setuptools/issues/2491) yet.
Use cibuildwheels to produce pre compiled binaries for Windows and Linux. The build is triggered on each PR/Push so it could be used a CI system too. The upload on PyPI is triggered when tagging a specific commit. You have to setup a PiPY account and to set secrets.PYPI_TOKEN variable on your github repo.

Seems you close this without merging it :)

Thanks for your edits, this seems good to me 👍

authors: drprojects rjanvier


text:
Add configuration flags for ddev pull

What happened (or feature request):
In #345 we introduced pantheon provider support. This was a big step forward, but there are ways we can improve it.
What you expected to happen:
We should introduce additional flags for ddev pull which allow more control over its behavior. The following flags should be added:

--ignore-db (does not download a new database archive and uses most recent version)
--ignore-files (does not download a new file archive and uses most recent version)
--skip-import (performs the download of files and/or db but does not perform the import)
--env=ENV (allows users to target a different environment than the one specified in config)

Related source links or issues:
https://github.com/drud/ddev/pull/345#issuecomment-313776460

This looks mostly actionable, but probably needs another set of eyes from a UX perspective before we commit to this approach. Also needs prioritization.

This would be a nice addition. Specifically the option to choose to skip downloading either the files, or the db. I'm using ddev with the Pantheon provider, and being able to update the DB with a fresh copy but not worry about the files directory would be nice. Especially for sites with files directories that are multiple Gigabytes of data.

@eojthebrave We completely agree that this will be a necessary feature in the future. We're in the process of finding additional team members to support this and other requests. I've updated the labels to note that this needs final definition and prioritization. Thanks!

Agreed - a way to only pull the DB is pretty necessary in the short-term, IMHO. I have sites with multiple-GB-sized files directories that I don't want pulled to my local (that's what Stage File Proxy is for).

We need to get this in by next release... it's valuable and I don't believe it's a heavy lift.

We'll want to do this for both Pantheon as well as drud-s3. I think this is one of those low effort, high value items.

It's just one effort, not separate things.

Just stopping by to say thanks. This --skip-files feature works great and easily replaces a handful of commands I was using to do this using a combination of terminus and ddev. And saves a lot of time/bandwidth for me. 🙏

Awesome, thanks! Always love to hear how you're using ddev and what your needs are.

And looking forward to a drupalize.me training on getting started with ddev and drupal, that would be awesome. We're happy to help with anything it would need. Would love to see what you'd do with that.

authors: beeradb eojthebrave rfay rickmanelius ultimike

repo: drud/ddev


text:
Unable to build 0.10.0-rc1 from source

I always build previous version from source and this worked fine. I can also build master. It's just 0.10.0-rc1 that doesn't work. I'm afraid I'm not familiar enough with building java projects to fix this myself :(
...
[INFO] io.druid:druid .................................... SUCCESS [5.516s]
[INFO] java-util ......................................... SUCCESS [8.701s]
[INFO] druid-api ......................................... SUCCESS [2.961s]
[INFO] druid-common ...................................... FAILURE [0.171s]
...
[ERROR] Failed to execute goal on project druid-common: Could not resolve dependencies
 for project io.druid:druid-common:jar:0.10.0-rc1: Could not find artifact
 io.druid:java-util:jar:tests:0.10.0-rc1 in central (http://repo.maven.apache.org/maven2
...

I already tried removing my .m2/ directory.
$ java -version
openjdk version "1.8.0_111"
OpenJDK Runtime Environment (build 1.8.0_111-8u111-b14-3~14.04.1-b14)
OpenJDK 64-Bit Server VM (build 25.111-b14, mixed mode)

$ mvn -version
Apache Maven 3.0.5
Maven home: /usr/share/maven
Java version: 1.8.0_111, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-openjdk-amd64/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "3.19.0-74-generic", arch: "amd64", family: "unix"

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 14.04.5 LTS
Release:        14.04
Codename:       trusty

Is this a build from the tag druid-0.10.0-rc1? Does it work if you do mvn install first?

Tag druid-0.10.0-rc1 yes. mvn install is the command I'm trying to run.
I now found that -DskipTests works but -Dmaven.test.skip=true give the error above.

authors: erikdubbelboer gianm

repo: druid-io/druid


text:
Message for not default value types

en:
  errors:
    max_size?:
      default: "baz"
      value:
        array: "foo"
        default: "bazz"

# validator
OFFER_SCHEMA = Dry::Validation.Schema do
  key(:name).required(max_size?: 50)
end

SCHEMA = Dry::Validation.Schema do
  key(:goods)
    .each { schema(OFFER_SCHEMA) }
   .required(:array?, max_size?: 3)

SCHEMA.call(goods: [{ name: '1' * 50 }, { name: '1' }]*2)
# => {:offer_goods=>["bazz"]}

This should work now but with the following syntax:
required(:goods).value(:array?, max_size?: 3) { each(OFFER_SCHEMA) }

authors: badlamer solnic

repo: dry-rb/dry-validation


text:
we respond within 1-2 working days

Is a duplicate issue of #7

authors: Nictanws kavxya


text:
Test CLA Workflow

update CONTRIBUTING.md
add a PR template

I have read the CLA Document and I hereby sign the CLA

repo: dsommerville-illumio/illumio-py


text:
StrongNameKeyBlob method not found using .NET 4.7.2

Create a project that targets both netcoreapp3.1 and net472, for example a test project that you want to run on both .net core and .net classic. Example here.
Add StrongNamer and a reference to an assembly that is not strong named (e.g. Selenium)
Build from VS 2019 or via msbuild.

Actual:
2>C:\Users\bogavril.nuget\packages\strongnamer\0.2.5\build\StrongNamer.targets(30,5): error MSB4018: The "StrongNamer.AddStrongName" task failed unexpectedly.
2>C:\Users\bogavril.nuget\packages\strongnamer\0.2.5\build\StrongNamer.targets(30,5): error MSB4018: System.MissingMethodException: Method not found: 'Void Mono.Cecil.WriterParameters.set_StrongNameKeyBlob(Byte[])'.
2>C:\Users\bogavril.nuget\packages\strongnamer\0.2.5\build\StrongNamer.targets(30,5): error MSB4018:    at StrongNamer.AddStrongName.ProcessAssembly(ITaskItem assemblyItem, Byte[] keyBytes, StrongNamerAssemblyResolver resolver)
2>C:\Users\bogavril.nuget\packages\strongnamer\0.2.5\build\StrongNamer.targets(30,5): error MSB4018:    at StrongNamer.AddStrongName.Execute()
2>C:\Users\bogavril.nuget\packages\strongnamer\0.2.5\build\StrongNamer.targets(30,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
2>C:\Users\bogavril.nuget\packages\strongnamer\0.2.5\build\StrongNamer.targets(30,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskBuilder.d__26.MoveNext()
Note: if I target just NetCore or NetFx it all works fine.

One of our developers experiences the same error with projects that target net472 only. Build from VS2019.

I'm seeing the same issue with .net 4.8 @AndrewRybka  @bgavrilMS
It looks like StrongNamer doesn't work with anything after .net 4.6.1
Anyone know how to fix this or a workaround because I still need a strong name for my files but I also need to target .net 4.8
Open to ideas

Well, the meta solution is for all dependencies to be strong named. Microsoft's guidance suggests this, even for open source projects. For example in MSAL we keep the full sn key in the public repo https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/blob/main/build/MSAL.snk ; Strong Naming is not a security feature, so this is doable.
So my suggestion is to identify those dependencies which are not strong named (including those that use that partial strong-naming stuff, which can be considered obsolete).

if internal, strong name it
if external, find a replacement or ask the maintainer to strong name

authors: AndrewRybka bgavrilMS rboy1

repo: dsplaisted/strongnamer


text:
[DEVOPS-1357] template compatibility with terraform 0.12

make it work with terraform 0.12

Coverage remained the same at 100.0% when pulling 5fcceed267f40e3a239bf43110571ddc22425d57 on vonq:DEVOPS-1357 into 518879dfe36542b8bb1fbd37281ba4a862a920e4 on dtan4:master.

Coverage remained the same at 100.0% when pulling 5fcceed267f40e3a239bf43110571ddc22425d57 on vonq:DEVOPS-1357 into 518879dfe36542b8bb1fbd37281ba4a862a920e4 on dtan4:master.


text:
pre-commit: PR115988

Link: https://github.com/llvm/llvm-project/pull/115988
Requested by: @dtcxzyw

/add-label reviewed
/close

repo: dtcxzyw/llvm-opt-benchmark


text:
optional arguments appearing in help menu

Is there a way to have the help menu print without the ["your arg..."] appearing after the command? Fantastic cli tool but this little feature doesn't fit with our application. Any ideas would be great.

Hey, thanks!
I happened to have it planned this week to introduce a vorpal.help method that will override the automated help. Would this work?

Yes that would probably work nicely, its non critical so I can push it out as an update. thanks

Sure, no problem.

Okay there's a new command, vorpal.help(fn), which lets you do a totally custom help menu.
To access the list of commands registered to vorpal, it's this.commands. You can play around with it from there.

Wiki: https://github.com/dthree/vorpal/wiki/API-|-vorpal#vorpalhelpfunction
Feel free to add clarity to them if you want.

authors: dthree matthewoconnor-development

repo: dthree/vorpal


text:
🛑 AutosUsadoe Next is down

In f8fa174, AutosUsadoe Next (https://autosusados.web.app/) was down:

HTTP code: 500
Response time: 1360 ms

Resolved: AutosUsadoe Next is back up in b85de14.


text:
Preserve cause-chain of errors propagated from rust -> c++ -> rust

use anyhow::{anyhow, Result};

#[cxx::bridge]
mod ffi {
    extern "C" {
        include!("repro.h");
        fn cthing() -> Result<()>;
    }

    extern "Rust" {
        fn rustthing() -> Result<()>;
    }
}

fn rustthing() -> Result<()> {
    Err(anyhow!("no such file or directory").context("failed to open: /nonexist"))
}

fn main() -> Result<()> {
    ffi::cthing()?;
    Ok(())
}

// repro.cc

void cthing() {
  rustthing();
}

This currently loses the cause chain and only prints:
Error: failed to open: /nonexist

Strongly related to this: in our projects we basically use Result and C++ exceptions to carry strings that are displayed to the user.  I really strongly prefer the anyhow formatter "{:#}" - but it looks like cxx-rs is hardcoded to use the default Display implementation.
Currently the inverse path C++ exception :arrow_right: Rust Result can be customized.
Maybe something as simple as
#[cxx::bridge(namespace = "demo")]
mod ffi {
    const ERROR_FORMAT = "{:#}";
}

Or, just switch to doing that by default?
Or, support a injecting a custom Rust function like
#[cxx::bridge(namespace = "demo")]
mod ffi {
    fn map_err(e: Error) -> String { format!("{:#}", e) }
}

?
(Conceptually this is a separate problem from actually preserving the cause chain, I can file a new issue if you prefer)

Strongly related to this: in our projects we basically use Result and C++ exceptions to carry strings that are displayed to the user.  I really strongly prefer the anyhow formatter "{:#}" - but it looks like cxx-rs is hardcoded to use the default Display implementation.
Currently the inverse path C++ exception :arrow_right: Rust Result can be customized.
Maybe something as simple as
#[cxx::bridge(namespace = "demo")]
mod ffi {
    const ERROR_FORMAT = "{:#}";
}

Or, just switch to doing that by default?
Or, support a injecting a custom Rust function like
#[cxx::bridge(namespace = "demo")]
mod ffi {
    fn map_err(e: Error) -> String { format!("{:#}", e) }
}

?
(Conceptually this is a separate problem from actually preserving the cause chain, I can file a new issue if you prefer)

authors: cgwalters dtolnay

repo: dtolnay/cxx


text:
Negation operator

When a broken crate is released, and isn't yanked, users don't have a future-proof way of specifying version requirements that don't pick the known-broken release.
If a crate foo 1.2.3 has an unexpected breaking change, its users can require something like foo = "1, <1.2.3", but this excludes all future releases of this crate, not just the one known to be broken. This exclusion becomes problematic, because even if 1.2.4 is later released with a fix, it won't be selected, and the range operator may cause dependency conflicts in Cargo.
If semver supported an "or" logical operator, users could specify 1 AND (<1.2.3 OR 1.2.4). But the easiest solution may be to support a negation operator that matches every version except the one specified. For example, it could look like: 1, !=1.2.3. This way the known-broken v1.2.3 won't be picked, but a future release may be.

This would need to go through a Cargo RFC in https://github.com/rust-lang/rfcs.

authors: dtolnay kornelski

repo: dtolnay/semver


text:
serde-yaml 0.5.0 is harder to test because keys are no longer alphabetical

Thank you for serde_yaml! We're using it almost everywhere in cage now, and it's great.
I tried upgrading to 0.5.0 this morning, however, and a lot of my YAML-related test suites started failing. These test suites often take the form of round-trip YAML parsing and serialization tests:
macro_rules! assert_roundtrip {
    ( $ty:ty, $yaml:expr ) => {
        {
            let yaml: &str = $yaml;
            let data: $ty = serde_yaml::from_str(&yaml).unwrap();
            let yaml2 = serde_yaml::to_string(&data).unwrap();
            assert_eq!(normalize_yaml(yaml), normalize_yaml(&yaml2));
        }
    }
}

#[test]
fn extends_can_be_roundtripped() {
    let yaml = r#"---
"file": "bar/docker-compose.yml"
"service": "foo"
"#;
    assert_roundtrip!(Extends, yaml);
}

These test cases fail because they no longer output YAML keys in an easily predictable order:
---- v2::extends_can_be_roundtripped stdout ----
	thread 'v2::extends_can_be_roundtripped' panicked at 'assertion failed: `(left == right)` (left: `"---\n\"file\": \"bar/docker-compose.yml\"\n\"service\": \"foo\""`, right: `"---\n\"service\": \"foo\"\n\"file\": \"bar/docker-compose.yml\""`)', src/v2/extends.in.rs:53
note: Run with `RUST_BACKTRACE=1` for a backtrace.

If keys are being output in unstable orders, this also means that we won't be able to usefully diff the YAML files output by cage export, which is helpful if we need to debug a configuration change to our production servers.
Would you expect the new order of serialized YAML keys to be predictable from run to run?
Thank you very much for maintaining serde_yaml, and for any advice you can provide!

I definitely should have mentioned this in the release notes - added now. The keys will be in the same order as they appear in the struct.
I think this is a better default than sorting everything. Often there is a particular order that makes sense in the source code, and the same order most likely makes sense in YAML too. It also matches the behavior of serde_json as well as the equivalent Go code.
Are you going to need a way to revert to the previous sorting behavior and if so, rather than sorting in serde_yaml how would you feel about a container attribute like #[serde(serialize_sorted)] to sort at compile time?
package main

import (
	"fmt"
	"gopkg.in/yaml.v2"
)

const y = `
"file": "bar/docker-compose.yml"
"service": "foo"
`

type Extends struct {
	Service string
	File    string
}

func main() {
	var data Extends
	err := yaml.Unmarshal([]byte(y), &data)
	if err != nil {
		panic(err)
	}

	y2, err := yaml.Marshal(data)
	if err != nil {
		panic(err)
	}

	// service: foo
	// file: bar/docker-compose.yml
	fmt.Print(string(y2))
}

Actually, source order is great. As long as the order is consistent from
run to run, we can maintain testability and diff-ability. There's
definitely no need to support alphabetic order.
Thank you very much!

authors: dtolnay emk

repo: dtolnay/serde-yaml


text:
TokenMarker is unexported

I believe that TokenMarker has been accidentally been left unexported. It's pub while other non-exported items are pub(crate).
pub enum TokenMarker {}

https://github.com/dtolnay/syn/blob/master/src/lookahead.rs#L157
I ran into this when trying to make part of my code more generic, and wanted to map my types to types and items in syn, like this.:
impl InputVariant for MyIdent {
    const TOKEN: fn(syn::lookahead::TokenMarker) -> proc_macro2::Ident = syn::Ident
}

Can you go into more detail what "map my types to types and items in syn" entails? How is the user of the API that you pasted expected to use that API?

For the purpose of making something that can be used from generic code, I think I would prefer to have peek be an associated function, instead of const TOKEN:
impl InputVariant for MyIdent {
    fn peek(input: ParseStream) -> bool {
        input.peek(syn::Ident)
    }
}

authors: dtolnay fredrik-hammar

repo: dtolnay/syn


text:
--fixHard not working

Device: iPhone 7 (iPhone9,3)
Host OS: MacOS Monterey 12.5.6 (21G531)
Host CPU: AMD Ryzen 5 2600
Device info: Main OS: iOS 15.6, Dualboot OS iOS 13.6, 32 GB edition
Logs after uploading Ramdisk:
[==================================================] 100.0%

cd ..
'[' -f /Users/enderman/.ssh/known_hosts ']'
'[' Darwin = Darwin ']'
sed -i.bak /localhost/d /Users/enderman/.ssh/known_hosts
sed -i.bak '/127.0.0.1/d' /Users/enderman/.ssh/known_hosts
'[' Darwin = Linux ']'
/Users/enderman/dualra1n/binaries/Darwin/iproxy 2222 22
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
echo '[] Waiting for the ramdisk to finish booting'
[] Waiting for the ramdisk to finish booting
i=1
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
sleep 1
i=2
'[' 2 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
sleep 1
i=3
'[' 3 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
sleep 1
i=4
'[' 4 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
sleep 1
i=5
'[' 5 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
Error connecting to device: No such file or directory
sleep 1
i=6
'[' 6 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
echo 8
8
echo '[] Testing for baseband presence'
[] Testing for baseband presence
++ remote_cmd '/usr/bin/mgask HasBaseband | grep -E '''true|false''''
++ sleep 1
++ /Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost '/usr/bin/mgask HasBaseband | grep -E '''true|false''''
++ sleep 1
'[' true = true ']'
[[ 0x8010 == \0\x\7\0\0 ]]
++ remote_cmd '/usr/bin/mgask HasBaseband | grep -E '''true|false''''
++ sleep 1
++ /Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost '/usr/bin/mgask HasBaseband | grep -E '''true|false''''
++ sleep 1
'[' true = false ']'
'[' '' = 1 ']'
echo 8
8
dataB=9
prebootB=10
echo 9
9
echo 10
10
remote_cmd /usr/bin/mount_filesystems
sleep 1
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost /usr/bin/mount_filesystems
seputil: Gigalocker file (/mnt7/EBC574DF-3F27-5203-B5EE-5738D70ECD86.gl) exists
seputil: Gigalocker initialization completed
sepi digest (48 bytes): 9f36a585b5ac2cba031410cf8b1a02cbd8204c704d58abc95a64ba001e017fe0aff349ea808254b4eabdd749ed7b0103
sepi nonce (20 bytes): 1345d18ac78524b3665b7df8a856eb7874ebb927
rsep digest (48 bytes): 2b8fc86948a20ff461d135742461cf1674df2ac691241e142140a17c0b88dfe8deb609e8fa99d590c753d0f6cbd11953
rsep nonce (20 bytes): 1345d18ac78524b3665b7df8a856eb7874ebb927
sleep 1
++ remote_cmd 'ls /mnt6/active'
has_active=/mnt6/active
'[' '!' /mnt6/active = /mnt6/active ']'
++ remote_cmd 'cat /mnt6/active'
active=44B120DB7761F83B5D0656726CF2701C91A08064098C25A31969E37F2863FAEB584617ED56406D2C9965368DB8890D66
mkdir -p boot/iPhone9,3
'[' '' = 1 ']'
remote_cp root@localhost:/mnt6/44B120DB7761F83B5D0656726CF2701C91A08064098C25A31969E37F2863FAEB584617ED56406D2C9965368DB8890D66/System/Library/Caches/apticket.der blobs/iPhone9,3-13.6.der
sleep 1
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine scp -r -o StrictHostKeyChecking=no -P2222 root@localhost:/mnt6/44B120DB7761F83B5D0656726CF2701C91A08064098C25A31969E37F2863FAEB584617ED56406D2C9965368DB8890D66/System/Library/Caches/apticket.der blobs/iPhone9,3-13.6.der
sleep 1
cp -av blobs/iPhone9,3-13.6.der work/IM4M
blobs/iPhone9,3-13.6.der -> work/IM4M
'[' '' = 1 ']'

Device: iPhone 7 (iPhone9,3) Host OS: MacOS Monterey 12.5.6 (21G531) Host CPU: AMD Ryzen 5 2600 Device info: Main OS: iOS 15.6, Dualboot OS iOS 13.6, 32 GB edition Logs after uploading Ramdisk:
[==================================================] 100.0%

cd ..
'[' -f /Users/enderman/.ssh/known_hosts ']'
'[' Darwin = Darwin ']'
sed -i.bak /localhost/d /Users/enderman/.ssh/known_hosts
sed -i.bak '/127.0.0.1/d' /Users/enderman/.ssh/known_hosts
'[' Darwin = Linux ']'
/Users/enderman/dualra1n/binaries/Darwin/iproxy 2222 22
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
echo '[] Waiting for the ramdisk to finish booting'
[] Waiting for the ramdisk to finish booting
i=1
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
sleep 1
i=2
'[' 2 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
sleep 1
i=3
'[' 3 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
sleep 1
i=4
'[' 4 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
sleep 1
i=5
'[' 5 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
Error connecting to device: No such file or directory
sleep 1
i=6
'[' 6 == 15 ']'
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost 'echo connected'
echo 8
8
echo '[] Testing for baseband presence'
[] Testing for baseband presence
++ remote_cmd '/usr/bin/mgask HasBaseband | grep -E '''true|false''''
++ sleep 1
++ /Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost '/usr/bin/mgask HasBaseband | grep -E '''true|false''''
++ sleep 1
'[' true = true ']'
[[ 0x8010 == \0\x\7\0\0 ]]
++ remote_cmd '/usr/bin/mgask HasBaseband | grep -E '''true|false''''
++ sleep 1
++ /Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost '/usr/bin/mgask HasBaseband | grep -E '''true|false''''
++ sleep 1
'[' true = false ']'
'[' '' = 1 ']'
echo 8
8
dataB=9
prebootB=10
echo 9
9
echo 10
10
remote_cmd /usr/bin/mount_filesystems
sleep 1
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine ssh -o StrictHostKeyChecking=no -p2222 root@localhost /usr/bin/mount_filesystems
seputil: Gigalocker file (/mnt7/EBC574DF-3F27-5203-B5EE-5738D70ECD86.gl) exists
seputil: Gigalocker initialization completed
sepi digest (48 bytes): 9f36a585b5ac2cba031410cf8b1a02cbd8204c704d58abc95a64ba001e017fe0aff349ea808254b4eabdd749ed7b0103
sepi nonce (20 bytes): 1345d18ac78524b3665b7df8a856eb7874ebb927
rsep digest (48 bytes): 2b8fc86948a20ff461d135742461cf1674df2ac691241e142140a17c0b88dfe8deb609e8fa99d590c753d0f6cbd11953
rsep nonce (20 bytes): 1345d18ac78524b3665b7df8a856eb7874ebb927
sleep 1
++ remote_cmd 'ls /mnt6/active'
has_active=/mnt6/active
'[' '!' /mnt6/active = /mnt6/active ']'
++ remote_cmd 'cat /mnt6/active'
active=44B120DB7761F83B5D0656726CF2701C91A08064098C25A31969E37F2863FAEB584617ED56406D2C9965368DB8890D66
mkdir -p boot/iPhone9,3
'[' '' = 1 ']'
remote_cp root@localhost:/mnt6/44B120DB7761F83B5D0656726CF2701C91A08064098C25A31969E37F2863FAEB584617ED56406D2C9965368DB8890D66/System/Library/Caches/apticket.der blobs/iPhone9,3-13.6.der
sleep 1
/Users/enderman/dualra1n/binaries/Darwin/sshpass -p alpine scp -r -o StrictHostKeyChecking=no -P2222 root@localhost:/mnt6/44B120DB7761F83B5D0656726CF2701C91A08064098C25A31969E37F2863FAEB584617ED56406D2C9965368DB8890D66/System/Library/Caches/apticket.der blobs/iPhone9,3-13.6.der
sleep 1
cp -av blobs/iPhone9,3-13.6.der work/IM4M
blobs/iPhone9,3-13.6.der -> work/IM4M
'[' '' = 1 ']'

lol you must use it with --dualboot 13.6 --fixHard and if you want to create the files boot again instead to dualboot again, you should add --dont-create-part

lol you must use it with --dualboot 13.6 --fixHard and if you want to create the files boot again instead to dualboot again, you should add --dont-create-part

@edwin170
[*] Fixing firmwares
mount_apfs: volume could not be mounted: Resource busy
mount_apfs: volume could not be mounted: Resource busy
mount_apfs: volume could not be mounted: Resource busy
seputil: Gigalocker file (/mnt7/EBC574DF-3F27-5203-B5EE-5738D70ECD86.gl) exists
AssertMacros:(value = 0x5),  file: seputil.c, line: 3974
seputil.c: Gigalocker initialization failed: 5
sepi digest (48 bytes): 9f36a585b5ac2cba031410cf8b1a02cbd8204c704d58abc95a64ba001e017fe0aff349ea808254b4eabdd749ed7b0103
sepi nonce (20 bytes): 1345d18ac78524b3665b7df8a856eb7874ebb927
rsep digest (48 bytes): 2b8fc86948a20ff461d135742461cf1674df2ac691241e142140a17c0b88dfe8deb609e8fa99d590c753d0f6cbd11953
rsep nonce (20 bytes): 1345d18ac78524b3665b7df8a856eb7874ebb927
seputil: load fw returned 0xe00002d5
seputil: failed to load /mnt6/44B120DB7761F83B5D0656726CF2701C91A08064098C25A31969E37F2863FAEB584617ED56406D2C9965368DB8890D66/usr/standalone/firmware/sep-firmware.img4 in  mode
mount_apfs: volume could not be mounted: Resource busy


text:
Make easier to accept TC

Task/Issue URL: https://app.asana.com/0/488551667048375/1205581341025832/f
Description
See task description
Steps to test this PR
See task description

Current dependencies on/for this PR:

develop

PR #3597   👈

This comment was auto-generated by Graphite.

repo: duckduckgo/Android


text:
Handle Samsung store links

Task/Issue URL: https://app.asana.com/0/488551667048375/1206018177037542/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3925   👈

This stack of pull requests is managed by Graphite.