text:
enh(#895): Custom refresh response token pointer

🔗 Linked issue
#895
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Added refreshResponseTokenPointer option to allow the token assignment (during the refresh call) to a different property respect to the signIn call.
This is useful in cases where the server returns the token in a property with different names between login and refresh.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have added tests (if possible).
[x] I have updated the documentation accordingly.

@zoey-kaiser Could you please re-review this?

repo: sidebase/nuxt-auth


text:
fix: add missing runtime dep of mraa to libjsonc

Signed-off-by: Felix Moessbauer felix.moessbauer@siemens.com

This lacks information on how to reproduce the issue. A plain kas-container build kas-iot2050-example.yml --target mraa is not enough, thus the package by itself is sufficiently described.

I wanted to keep it simple to not get into the rabbits hole of not having SOVERSION and not a split into -dev and lib- packages. The problem appears when you link against libmraa.so. Then, the transitive dependency to libjsonc is missing. Normally, dpkg-shlibdeps automatically adds these dependencies. But this seems not to work for libraries without SOVERSION. In the official image, this is not an issue as libjsonc is probably anyways installed. But I recently moved a large layer over to ISAR sbuild and there, a component could not be build because the linking failed, as only mraa was added as build dep, but not libjsonc. Luckily we catched it in the build...

OK, that explanation - or a short summary of if - definitely belongs to the commit at least as it is not obvious at all.

FWIW: libcryptsetup12:arm64 depends on libjson-c5 (>= 0.15).

I thought about my comment again and concluded that the reason is pretty strange (just made based on the observation). In fact, the reason is much simpler: We overwrite the DEBIAN_DEPENDS and by that also remove the ${shlibs:Depends}. I'll check meta-iot2050 for more of these bugs and fix it in this PR.

Actually, the default of DEBIAN_DEPENDS is empty in ISAR. I'm really wondering why the default does not contain shlib depends. That would not hurt in the normal case, but save us from issues like this one.

Yeah, maybe also an upstream Isar topic, indeed.

source: gharchive/pull-request

authors: fmoessbauer jan-kiszka

license: MIT

license_type: permissive

license_source: github-api


text:
Add KubeCon session link

I enjoyed watching the session and I'm looking fowared to use this tool

@OmerKahani Thanks for the contribution!

authors: OmerKahani marshtompsxd

repo: sieve-project/sieve


text:
Feat/sighupio/product management#440/update ingress nginx

Checks

verified if are there no changes between helm chart v4.11.2 and v4.11.3

Changes

update controller tag in base/kustomization.yaml
removed rule to access ingress-controller-leader configMap as it seems no longer in use
add appProtocol in ingress-nginx service
add protocol in each port of DeamonSet
add terminationGracePeriodSeconds, dnsPolicy and kubernetes.io/os: linux nodeSelector in DaemonSet
update nginx and dual-nginx version in the main README
update version in nginx module README
update the Furyfile.yaml example in the nginx module
update nginx MAINTENANCE.md with the last helm chart version
update version in dual-nginx module README
update the Furyfile.yaml example in the dual-nginx module

Hey! Probably we should have aligned before. Since I'm working on https://github.com/sighupio/product-management/issues/499 I am refactoring nginx, so this PR will probably change. I will keep you posted 😄

repo: sighupio/fury-kubernetes-ingress


text:
Upstream sync

git cherry-pick eeb6eb2..b33c465

git log:
commit b33c465d5e8bf42b70a1e2eef7b5251a1471f490
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Wed Oct 6 08:59:53 2021 +0100

    Delete performance pipeline, and manually update GitHub statuses (#1868)

    * Revert "Fix performance pipeline (#1851)"

    This reverts commit 438c66895a4a0ab143e6948980ec0e775237492e.

    * Revert "Add a separate performance pipeline for running Performance tests (#1842)"

    This reverts commit 91b80bb98053ec8b0c161873c520ab50fcb11c54.

    * Manually report statuses to GitHub

    This gives us pretty much the same "out of the box" experience as we would have if the app connection would work (but it doesn't, so this will have to do!)

    I'm not reporting statuses for _everything_ (e.g. code coverage, upload to s3) to reduce some clutter

commit 92780516ae6c64caa648cef25cfa8f54c81a2572
Author: Tony Redondo <tony.redondo@datadoghq.com>
Date:   Tue Oct 5 20:59:23 2021 +0200

    Add PInvoke Map rewriting in all platforms to support native library renaming (#1809)

    * Add PInvoke Map rewriting in all platforms to support native library renaming.

    * Fallback to the fixed name when we can't extract the path (Installed in the GAC).

    * Fix native filename per platform.

    * Adds the filename checker project. (Test is still missing for next commit)

    * Changes.

    * Fix GetCLRProfilerPath function.
    Adds the NativeProfilerChecks test.

    * Fix non Windows build

    * Revert unnecessary changes.

    * Add try/catch on the Checker path.
    Change GetCLRPRofilerPath to detect the runtime type.

    * try to fix tests.

    * Adds conditional for the check

    * setup the test project for linux.

    * commenting to do some checks.

    * Build passing. Uncomment, initialization block.

    * Add ToString to check and simplify catch scenario.

    * comments

    * change

    * change std::filepath with a normal rfind.

    * changing algorithm to avoid std::filesystem

    * Remove validation based on the process name, all samples are running through the dotnet.exe process

commit 2465fb3fd76d5144ae636838f2e950b860587cbe
Author: Zach Montoya <zach.montoya@datadoghq.com>
Date:   Tue Oct 5 09:47:54 2021 -0700

    Re-implement log4net logs injection (#1710)

    ## Changes
    This change modifies log4net logs injection to stop adding the Span information to the log4net MDC (which has demonstrated that it cannot safely be serialized across AppDomain boundaries) and to start adding the Span information directly to the log events. This requires automatic instrumentation so this information can be added as soon as possible after the LoggingEvent object is created, before other Appenders (i.e. sinks) consume the events and write them elsewhere.

    This change has a big limitation: **log4net logs injection now requires automatic instrumentation.**

    Additionally, the CustomLog4NetLogProvider type has been renamed to NoOpLog4NetLogProvider and modified so that the LibLog interface does not affect the log4net state.

    ## Testing
    The sample `test/test-applications/integrations/LogsInjection.Log4Net` has been updated so that the application crosses an AppDomain boundary after a trace has started, demonstrating that the previous implementation throws a SerializationException while the new implementation does not.

    ## Benefits
    The biggest benefit of this change is that we should no longer receive issues about log4net logs injection causing SerializationExceptions. This is because we no longer store the Datadog span information in the ambient context. Instead, we directly attach the Datadog span information to the log event at the time it's generated. A side benefit is that the properties will always exist on the log event objects, so we can now support some async scenarios where the customer previously could not grab the ambient log context.

    ## Performance Considerations
    I expected the new logs injection mechanism to be less performant because we must add Span information on each log statement, instead of once per scope activation. However, the benchmark results show that the overhead is largely unchanged. I guess this isn't too surprising as the new logs injection mechanism is just as performant as manually adding an Appender to the logging configuration (which is what New Relic does for log correlation).

    ## Follow-up
    If we need to make log4net logs injection operate with only manual instrumentation, we can publish an Appender package that customers can add via NuGet and configure themselves.

commit 4b3e84457df555c62108a346a3a35cee69449b0e
Author: Kevin Gosse <krix33@gmail.com>
Date:   Tue Oct 5 18:01:12 2021 +0200

    Kill the old Samples.Shared project (#1867)

commit 75b91c6da01b8c44711c8ed2c8933cc6733fb3a0
Author: Zach Montoya <zach.montoya@datadoghq.com>
Date:   Tue Oct 5 08:46:05 2021 -0700

    Update test package versions (#1862)

    Update the following NuGet packages under test:
    - MongoDB.Driver
    - Elasticsearch v7
    - Confluent.Kafka
    - MySql.Data
    - Microsoft.Data.SqlClient
    - System.Data.SqlClient
    - ServiceStack.Redis
    - Microsoft.Data.Sqlite
    - MSTest.TestAdapter / MSTest.TestFramework
    - Microsoft.Azure.Cosmos
    - Aerospike.Client
    - NLog
    - AWSSDK.SQS
    - GraphQL / GraphQL.SystemReactive v4

commit d974024c284e6a2655589663bd51228b51144a65
Author: Tony Redondo <tony.redondo@datadoghq.com>
Date:   Tue Oct 5 17:25:22 2021 +0200

    Update CI namespace types (#1864)

commit 52b378c5edbc6ab76ee142a614028c4cef76d829
Author: Kevin Gosse <krix33@gmail.com>
Date:   Tue Oct 5 14:27:01 2021 +0200

    Use Xunit.SkippableFact for inconclusive tests (#1858)

commit 3a9042b2ee2df2e237eb8cd55f174b62b84f8e6b
Author: Zach Montoya <zach.montoya@datadoghq.com>
Date:   Mon Oct 4 11:40:32 2021 -0700

    Additional updates to launchSettings.json for test applications (#1848)

commit 89a1592876561303f6edc5a59ef11887735d7f11
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Mon Oct 4 13:56:55 2021 +0100

    Add LifetimeManager for handling shutdown events (#1841)

    * Add LifetimeManager for handling shutdown

    We currently have duplicated handling of shutdown events in the Tracer and Security classes. Future work (e.g. Log submission and Telemetry) will like ly add further to this pattern.

    Rather than keep duplicating this pattern, adding a LifetimeManager to register all the callbacks should simplify things somewhat

    * Switch to a queue to preserve registration order

    * Replace CIVisibility lifetime hooks with LifetimeManager

    * Fix incorrect logger type reference

commit a507b787db139e2c2b2104caf02749800e23dcd6
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Mon Oct 4 13:00:35 2021 +0100

    Check the logs for errors automatically after each test run (#1776)

    Excludes "known" issues such as "Unable to resolve method MongoDB" and the "CallTargetNativeTest"s

commit d72d905149a9f0e36109aa0f9c69bf758908856a
Author: macrogreg <macrogreg@users.noreply.github.com>
Date:   Fri Oct 1 11:10:38 2021 -0700

    Add lookup for `LocalRootSpanId` property and tests for `SpanId` and `LocalRootSpanId`.  (#1839)

    A [recent PR](https://github.com/DataDog/dd-trace-dotnet/pull/1831) suggested to add direct access properties to Span to access the id and the id of the root span of the local non-reentrant sub-trace. That PR added private fields to `Span` to store those values, to avoid dereferencing multiple pointers at every lookup. However, it was not clear, whether that memory increase was justified, however small it was.

    To address that, _this_ PR provides and tests those properties without using additional memory. We are adding using tests, and will keep https://github.com/DataDog/dd-trace-dotnet/pull/1831 archived in case we need to get back to using private fields.

commit 438c66895a4a0ab143e6948980ec0e775237492e
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Fri Oct 1 17:41:03 2021 +0100

    Fix performance pipeline (#1851)

    * Remove additional performance pipeline triggers

    With the additional triggers, the performance pipeline was being triggered twice
    - Immediately when a push to a monitored branch
    - When the ci pipeline had finished building

    We only want the latter one

    * Remove ContinueOnError from benchmarks

    These failures don't block the ability to merge, so they should be counted as real failures

    * Disable triggers

    * Disable PR verification triggers in performance pipeline

    * Try fix throughput tests not able to access the correct commit

    * Propagate the original source commit and repo from the originating pipeline

        This is such a horrible mess, but appears to be necessary

    * Try fix variables

commit c38ccce959d112e730be13cbf6aa3e5b728dcbd9
Author: Tony Redondo <tony.redondo@datadoghq.com>
Date:   Fri Oct 1 13:51:17 2021 +0200

    Skip WriteTrace call when no there's no spans and improve filtering (#1843)

    * Skip WriteTrace call when no there's no spans and improve filtering

    * Skip the ArraySegment if doesn't have any span.

    * Update tracer/src/Datadog.Trace/Ci/Agent/CIAgentWriter.cs

    Co-authored-by: Andrew Lock <andrew.lock@datadoghq.com>

    * Simplify the CIAgentWriter algorithm.

    * Update CIAgentWriter.cs

    Co-authored-by: Andrew Lock <andrew.lock@datadoghq.com>

commit 91b80bb98053ec8b0c161873c520ab50fcb11c54
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Fri Oct 1 08:58:29 2021 +0100

    Add a separate performance pipeline for running Performance tests (#1842)

    * Add a separate performance pipeline for running Benchmarks and Throughput tests

    The advantage of this is that we can make the benchmark and throughput results optional. We still want to run them for every PR, but it means in cases where a rebase or a trivial change is required, we wouldn't need to wait for the performance tests to finish before merging.

    * Update benchmark comparison tool to use correct pipeline ID

    * Need to have a stage without any dependencies

    * depends_on isn't required, as shuoldn't be triggered until source completes

    * Make stages run in parallel

commit 00a737325c1c7d317f2097c5575cf562c6aff0f7
Author: Robert Pickering <robert.pickering@datadoghq.com>
Date:   Fri Oct 1 09:56:30 2021 +0200

    Report attack when response has ended (#1847)

    Otherwise wrong status code is reported

commit 8cab68b961c34d1e416835a71259164157b3565b
Author: Gregory LEOCADIE <gregory.leocadie@datadoghq.com>
Date:   Thu Sep 30 20:49:04 2021 +0200

    Managed loader cannot resolve profiler managed libraries when installed by the MSI installer (#1820)

    ### Context

    When installing the profiler and the tracer using the MSI installer, the profiler managed library cannot be resolved.

    To resolve the profiler managed library, the managed loader uses the `COR_PROFILER_PATH_XX` or `CORECLR_PROFILER_PATH_XX` to locate the profiler home directory.

    But, the MSI installer does not populate those environment variables, instead it registers the CLR profiler CLSID and the path in the registry.

    ### Fixes

    Changes proposed in this pull request:

    To resolve the tracer managed library, the managed loader relies on the `DD_DOTNET_TRACER_HOME` environment variable. (even if today the tracer does not use the managed loader).

    We can introduction a `DD_DOTNET_PROFILER_HOME` environment variable to replicate the same pattern used for the tracer.

commit 976bf737f468e0d213bc73a423e5610b9923000e
Author: Kevin Gosse <krix33@gmail.com>
Date:   Thu Sep 30 18:12:49 2021 +0200

    Skip segmentation faults in .NET Core 2.1 tests (#1835)

    * Skip segmentation faults in .NET Core 2.1 tests

commit e7b904a67cb4817205e8fafdee666a155fb9842a
Author: Kevin Gosse <krix33@gmail.com>
Date:   Thu Sep 30 17:32:20 2021 +0200

    Fix transient error in aspnetcore tests (#1836)

    SubmitRequest can throw instead of just returning a status code if the connection couldn't be established.

commit e3341b46eab872744a4569dd979cdb3866c5de33
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Thu Sep 30 15:24:38 2021 +0100

    Update Gitlab build image to use 5.0.401 SDK (#1834)

    * Shutdown .NET server after build

    Might help avoid some locked files

    * Mention about downloading native symbols when creating a release

    * Ignore some types from the code coverage report that frequently come up but aren't meaningful

    * Update build image in GitLab (to use 5.0.401 SDK)

commit 777c7da31f5a0526b4cd905871b927c640c8e1ed
Author: Kevin Gosse <krix33@gmail.com>
Date:   Thu Sep 30 16:18:50 2021 +0200

    Enable static analysis for ConfigureAwait (#1833)

    * Enable static analysis for ConfigureAwait
    * Fix warning IL3000

    error IL3000: 'System.Reflection.Assembly.Location' always returns an empty string for assemblies embedded in a single-file app. If the path to the app directory is needed, consider calling 'System.AppContext.BaseDirectory'.

commit f552f52713e7e5d44824ad7e0a8850e503b3a2bb
Author: Kevin Gosse <krix33@gmail.com>
Date:   Thu Sep 30 16:18:19 2021 +0200

    Use temporary folder for NServiceBus storage (#1840)

commit abf548f957c5fc94b50a68769b098ef20c158526
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Thu Sep 30 13:17:31 2021 +0100

    Small build improvements (#1813)

    * Stop copying profiler around for azure functions

    * Fix incorrect path in Directory.Build.props

    * Update launchSettings.json to use tracer home directory for profiler

    * Add shared helper project for samples, to reduce the need to copy-paste code everywhere

    * Delete copy pasted code and use the shared project instead

    * Apply suggestions from code review

    Co-authored-by: Zach Montoya <zach.montoya@datadoghq.com>

    * Apply suggestions from code review

    Co-authored-by: Zach Montoya <zach.montoya@datadoghq.com>

    Co-authored-by: Zach Montoya <zach.montoya@datadoghq.com>

commit d24fc0fc0cd7697ad464f49f0897a740ac65793f
Author: Tony Redondo <tony.redondo@datadoghq.com>
Date:   Thu Sep 30 09:51:31 2021 +0200

    CI Visibility Mode v1 (#1795)

    * Initial CIVisibility POC

    * Add a Custom Sampler.

    * Refactor

    * Changes

    * Major refactor and support for adding origin tag to all CITracer spans.

    * Fix tests

    * Change AgentWriter algorithm.

    * Add dd-trace parameter to enable CI Visibility Mode

    * Apply suggestions from code review

    Co-authored-by: Andrew Lock <andrew.lock@datadoghq.com>

    * Change CIVisibilitySettings to readonly.

    * Use Tracer.UnsafeSetTracerInstance in all cases.

    * Refactor GetServiceNameFromRepository

    * Adds edge cases to GetServiceNameFromRepository and fixes the Regex to handle those cases.

    * Changes.

    * Add CIVisibility Mode autodetection to dd-trace based on the command wrapped.

    * Apply PR suggestion.

    * Fix LockedTracerInstanceSwap test to expect the InvalidOperationException.

    Co-authored-by: Andrew Lock <andrew.lock@datadoghq.com>

commit c55f8087c6cc9c8125962f4faa2fc7054d8bee81
Author: Zach Montoya <zach.montoya@datadoghq.com>
Date:   Wed Sep 29 10:11:12 2021 -0700

    CallSite instrumentation - Add exceptions to active span during ASP.NET Web API 2 message handler exception (#1817)

    Follow-up to https://github.com/DataDog/dd-trace-dotnet/pull/1734

    See the original PR for the full motivation. The original PR only added a fix for CallTarget instrumentation, so this PR adds the fix when the instrumented application is using CallSite instrumentation.

commit 2711473d70aed49df9fd72b57e42a66cd1758080
Author: Colin Higgins <colin-higgins@users.noreply.github.com>
Date:   Wed Sep 29 10:58:25 2021 -0400

    Generate Dependabot File for Integrations (#1754)

commit 51de1eb12443acd001bd1cf6e25d39c0ff402858
Author: Tony Redondo <tony.redondo@datadoghq.com>
Date:   Wed Sep 29 16:00:21 2021 +0200

    Add Test Framework version to test spans (#1828)

commit d2637d4da21b706dc0e574394dcabcca5b1d7d03
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Wed Sep 29 14:48:22 2021 +0100

    Dont test all integration package versions on every PR (#1818)

    * Fix TestAllPackageVersions usage (was always set to true)

    Ensure we compile multi-api samples using default package version if TestAllPackageVersions is false

    * Don't test all package versions on master branch

    * Fix tests to handle running the "default" package version tests

commit c27fcdd3962b31cbf282a3c46ef2f755d90de56b
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Wed Sep 29 14:48:08 2021 +0100

    Parallelise Windows integration tests by framework (#1819)

    * Only build integration tests for a single framework

    Allows us to parallelize by framework

    * Parallelize integration tests by framework

    * Rename platform -> target platform

    Unfortunately, using the 'platform' key was causing the Nuke process itself to run as an x86 process, which was causing BadImageFormatException when loading the target frameworks for a project.

    Renaming the parameter to TargetPlatform instead solves the problem.

commit 6754e304ffd75c74150a8396554a4cc26dbcb338
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Wed Sep 29 12:32:44 2021 +0100

    Fix path in gitlab.bat after repository move (#1832)

commit 610a99303d5fb11c92a8ed541da65e1c3c892257
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Wed Sep 29 11:28:53 2021 +0100

    Post benchmark result comparison as a comment to the PR (#1811)

    * Extract PostCommentToPullRequest as separate function

    * Post benchmark results to the PR as a comment

    Based on the "results comparer" tool in the [dotnet/performance repo](https://github.com/dotnet/performance/blob/main/src/tools/ResultsComparer/README.md), using a default threshold and noise levels. We may well need to tweak these for our cases - we'll have to keep an eye on that.

    * Fix CSV reading

    Was using positional fields previously, but the CSV has a variable number of fields

    * Refactor benchmark comparison + add allocation comparison

    * Fix divide by zero error

    * Reduce sensitivity

    * Rename StatisticalTestThreshold to SignificantResultThreshold

    And update it to 10% - I misunderstood, we always use a 5% test for significance, but this rejects error results that show less than 10% difference

    * Update markdown generated

commit 790a2df7c285b9ee1cf35aa0f9edef9912fa86d1
Author: Kevin Gosse <krix33@gmail.com>
Date:   Wed Sep 29 11:01:32 2021 +0200

    Change more timeouts in MassTransit (#1829)

commit e01e6aba6b9563aa07c7d7bf63209f38a06b1a03
Author: Kevin Gosse <krix33@gmail.com>
Date:   Wed Sep 29 11:01:15 2021 +0200

    Add a sleep to minimize risks of segfaults on 2.1 (#1830)

commit ed0e465a7ca7fc7cec7f7523997c533104961742
Author: Andrew Lock <andrew.lock@datadoghq.com>
Date:   Tue Sep 28 18:05:44 2021 +0100

    [Version Bump] 1.28.8 (#1827)

    * Fix verification step (broken in tracer move)

    * classify area:test-apps issues as build/test related

    * [Version Bump] 1.28.8

    The following files were found to be modified (as expected)

    - [x] docs/CHANGELOG.md
    - [x] tracer/build/_build/Build.cs
    - [x] tracer/integrations.json
    - [x] tracer/samples/AutomaticTraceIdInjection/MicrosoftExtensionsExample/MicrosoftExtensionsExample.csproj
    - [x] tracer/samples/AutomaticTraceIdInjection/Log4NetExample/Log4NetExample.csproj
    - [x] tracer/samples/AutomaticTraceIdInjection/NLog40Example/NLog40Example.csproj
    - [x] tracer/samples/AutomaticTraceIdInjection/NLog45Example/NLog45Example.csproj
    - [x] tracer/samples/AutomaticTraceIdInjection/NLog46Example/NLog46Example.csproj
    - [x] tracer/samples/AutomaticTraceIdInjection/SerilogExample/SerilogExample.csproj
    - [x] tracer/samples/ConsoleApp/Alpine3.10.dockerfile
    - [x] tracer/samples/ConsoleApp/Alpine3.9.dockerfile
    - [x] tracer/samples/ConsoleApp/Debian.dockerfile
    - [x] tracer/samples/WindowsContainer/Dockerfile
    - [x] tracer/src/Datadog.Monitoring.Distribution/Datadog.Monitoring.Distribution.csproj
    - [x] tracer/src/Datadog.Trace.AspNet/Datadog.Trace.AspNet.csproj
    - [x] tracer/src/Datadog.Trace.ClrProfiler.Managed.Loader/Datadog.Trace.ClrProfiler.Managed.Loader.csproj
    - [x] tracer/src/Datadog.Trace.ClrProfiler.Managed.Loader/Startup.cs
    - [x] tracer/src/Datadog.Trace.ClrProfiler.Native/CMakeLists.txt
    - [x] tracer/src/Datadog.Trace.ClrProfiler.Native/dd_profiler_constants.h
    - [x] tracer/src/Datadog.Trace.ClrProfiler.Native/Resource.rc
    - [x] tracer/src/Datadog.Trace.ClrProfiler.Native/version.h
    - [x] tracer/src/Datadog.Trace.MSBuild/Datadog.Trace.MSBuild.csproj
    - [x] tracer/src/Datadog.Trace.OpenTracing/Datadog.Trace.OpenTracing.csproj
    - [x] tracer/src/Datadog.Trace.Tools.Runner/Datadog.Trace.Tools.Runner.Standalone.csproj
    - [x] tracer/src/Datadog.Trace.Tools.Runner/Datadog.Trace.Tools.Runner.Tool.csproj
    - [x] tracer/src/Datadog.Trace/Datadog.Trace.csproj
    - [x] tracer/src/Datadog.Trace/TracerConstants.cs
    - [x] tracer/src/WindowsInstaller/WindowsInstaller.wixproj
    - [x] tracer/test/test-applications/regression/AutomapperTest/Dockerfile

    @DataDog/apm-dotnet

commit 2e5d383da9672e945a6c451cf29abfabfdd14751
Author: Tony Redondo <tony.redondo@datadoghq.com>
Date:   Tue Sep 28 10:15:50 2021 +0200

    Adds support for .NET Framework 4.5 in the native loader. (#1825)

    * Adds support for .NET 4.5 in the native loader.

    * initialize mask variables.

Same on my box.
./tracer/build.cmd RunWindowsIntegrationTests -framework net5.0 -filter Log4NetTests.DoesNotInjectLogsWhenDisabled

is failing

The active test run was aborted. Reason: Test host process crashed : Unhandled exception. System.InvalidOperationException: There is no currently active test.
   at Xunit.Sdk.TestOutputHelper.GuardInitialized() in C:\Dev\xunit\xunit\src\xunit.execution\Sdk\Frameworks\TestOutputHelper.cs:line 51
   at Xunit.Sdk.TestOutputHelper.QueueTestOutput(String output) in C:\Dev\xunit\xunit\src\xunit.execution\Sdk\Frameworks\TestOutputHelper.cs:line 66
   at Datadog.Trace.ClrProfiler.IntegrationTests.AspNetCore.AspNetCoreMvcTestBase.AspNetCoreTestFixture.<>c__DisplayClass15_0.<EnsureServerStarted>b__0(Object sender, DataReceivedEventArgs args) in C:\Users\rpajak\repos\s
ignalfx-dotnet-tracing\tracer\test\Datadog.Trace.ClrProfiler.IntegrationTests\AspNetCore\AspNetCoreMvcTestBase.cs:line 178
   at System.Diagnostics.AsyncStreamReader.FlushMessageQueue(Boolean rethrowInNewThread)
--- End of stack trace from previous location where exception was thrown ---
   at System.Diagnostics.AsyncStreamReader.<>c.<FlushMessageQueue>b__18_0(Object edi)
   at System.Threading.QueueUserWorkItemCallback.<>c.<.cctor>b__6_0(QueueUserWorkItemCallback quwi)
   at System.Threading.ExecutionContext.RunForThreadPoolUnsafe[TState](ExecutionContext executionContext, Action`1 callback, TState& state)
   at System.Threading.QueueUserWorkItemCallback.Execute()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()

repo: signalfx/signalfx-dotnet-tracing


text:
Using Orion to Detect Anomaly in Personal Dataset

Orion version:
Python version: 3.7.3
Operating System: macOS 11.2.3

Description
I am trying to detect an anomalous feature in my personal dataset BOX1_TEMP.csv. I know that there is about a month's worth of data that is anomalous (in the 100s rather than the 20s). Can Orion detect this and give me the range of time where the data was anomalous? I have no idea if I am using the right primitive or pipeline. The data is not evenly spaced over time. I am also getting some errors about an index being out of bounds.
What I Did
Inputs
filename = "BOX1_TEMP.csv"
from orion.data import load_signal

data = load_signal(filename)

from orion.analysis import analyze

hyperparameters = {
    "mlprimitives.custom.timeseries_preprocessing.rolling_window_sequences#1":{
        'index':9518
    },
    'keras.Sequential.LSTMTimeSeriesRegressor#1': {
        'epochs': 5,
        'verbose': True
    }
}

pipeline = 'lstm_dynamic_threshold'

anomalies = analyze(pipeline, data, hyperparams=hyperparameters)
anomalies

Outputs
Using TensorFlow backend.
ERROR:mlblocks.mlpipeline:Exception caught fitting MLBlock keras.Sequential.LSTMTimeSeriesRegressor#1
Traceback (most recent call last):
  File "/Applications/anaconda3/lib/python3.7/site-packages/mlblocks/mlpipeline.py", line 549, in _fit_block
    block.fit(**fit_args)
  File "/Applications/anaconda3/lib/python3.7/site-packages/mlblocks/mlblock.py", line 302, in fit
    getattr(self.instance, self.fit_method)(**fit_kwargs)
  File "/Applications/anaconda3/lib/python3.7/site-packages/mlprimitives/adapters/keras.py", line 98, in fit
    self._augment_hyperparameters(X, kwargs)
  File "/Applications/anaconda3/lib/python3.7/site-packages/mlprimitives/adapters/keras.py", line 89, in _augment_hyperparameters
    shape = np.asarray(X)[0].shape
IndexError: index 0 is out of bounds for axis 0 with size 0
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-12-68bd1be1d045> in <module>
     13 pipeline = 'lstm_dynamic_threshold'
     14
---> 15 anomalies = analyze(pipeline, data, hyperparams=hyperparameters)
     16 anomalies

/Applications/anaconda3/lib/python3.7/site-packages/orion/analysis.py in analyze(pipeline, train, test, hyperparams)
     46         pipeline = _load_pipeline(pipeline, hyperparams)
     47
---> 48     events = _run_pipeline(pipeline, train, test)
     49
     50     return _build_events_df(events)

/Applications/anaconda3/lib/python3.7/site-packages/orion/analysis.py in _run_pipeline(pipeline, train, test)
     22 def _run_pipeline(pipeline, train, test):
     23     LOGGER.debug("Fitting the pipeline")
---> 24     pipeline.fit(train)
     25
     26     LOGGER.debug("Finding events")

/Applications/anaconda3/lib/python3.7/site-packages/mlblocks/mlpipeline.py in fit(self, X, y, output_, start_, **kwargs)
    647                     continue
    648
--> 649             self._fit_block(block, block_name, context)
    650
    651             if (block_name != self._last_block_name) or (block_name in output_blocks):

/Applications/anaconda3/lib/python3.7/site-packages/mlblocks/mlpipeline.py in _fit_block(self, block, block_name, context)
    547         try:
    548             fit_args = self._get_block_args(block_name, block.fit_args, context)
--> 549             block.fit(**fit_args)
    550         except Exception:
    551             if self.verbose:

/Applications/anaconda3/lib/python3.7/site-packages/mlblocks/mlblock.py in fit(self, **kwargs)
    300             fit_kwargs.update(kwargs)
    301             fit_kwargs = self._get_method_kwargs(fit_kwargs, self.fit_args)
--> 302             getattr(self.instance, self.fit_method)(**fit_kwargs)
    303
    304     def produce(self, **kwargs):

/Applications/anaconda3/lib/python3.7/site-packages/mlprimitives/adapters/keras.py in fit(self, X, y, **kwargs)
     96
     97     def fit(self, X, y, **kwargs):
---> 98         self._augment_hyperparameters(X, kwargs)
     99         self.model = self._build_model(**kwargs)
    100

/Applications/anaconda3/lib/python3.7/site-packages/mlprimitives/adapters/keras.py in _augment_hyperparameters(self, X, kwargs)
     87
     88     def _augment_hyperparameters(self, X, kwargs):
---> 89         shape = np.asarray(X)[0].shape
     90         length = shape[0]
     91         self._setdefault(kwargs, 'input_shape', shape)

IndexError: index 0 is out of bounds for axis 0 with size 0

Hi @jgubner, thanks for opening this issue! There are some elements needed to be taken care of, I will go through this step by step
Raw Data
You can definitely use your own data in Orion, we only require it to be in a ['timestamp', 'value'] format (which it is) but additionally, the timestamp column needs to be of integer type, preferably a unix timestamp.
Aggregation Levels
The first step in all our pipelines is to define the aggregation level you want, this makes the signal equi-spaced. For this primitive, you need to set the interval hyperparameter accordingly. For example, if your data has a frequency of 5 minutes then interval=300 would be most appropriate since we are dealing with second intervals.
hyperparameters = {
    "mlprimitives.custom.timeseries_preprocessing.time_segments_aggregate#1": {
        "interval": 300
    }
}

I understand your data has inconsistent spacing, it would be best if you have an estimate of what aggregation interval would work best because the remaining pipeline is under the assumption that each value and another are equally spaced.
Rolling Window Sequences
I'm a little bit confused here as to why you are setting index to a single value. The rolling_window_sequence primitive requires two inputs: X (ndarray), and index (ndarray) which are one-to-one corresponding. This is something you do NOT need to modify because it is implicitly taken care of within the pipeline. More on the primitive documentation here.
If you wish to change the window size of the primitive, say window_size=100 then you change that particular hyperparameter.
hyperparameters = {
    "mlprimitives.custom.timeseries_preprocessing. rolling_window_sequences#1": {
        "window_size": 100
    }
}

Putting a Pipeline Together
Per your example, you have selected the lstm_dynamic_threshold pipeline, which is a prediction based pipeline. After you have transformed your data into a format usable by Orion, we can put everything together
from orion import Orion

hyperparameters = {
    "mlprimitives.custom.timeseries_preprocessing.time_segments_aggregate#1": {
        "interval": 300
    },
    "mlprimitives.custom.timeseries_preprocessing. rolling_window_sequences#1": {
        "window_size": 100
    },
    "keras.Sequential.LSTMTimeSeriesRegressor#1": {
        "epochs": 5,
        "verbose": True
    }
}

orion = Orion(
    pipeline='lstm_dynamic_threshold',
    hyperparameters=hyperparameters
)

orion.fit(data)

Let me know if this makes sense and if you have any further questions

Hi @sarahmish, thank you for the information.
I tried what you suggested, but I am now getting a key error:
KeyError                                  Traceback (most recent call last)
<ipython-input-30-57be5ca3b78f> in <module>
     14 orion = Orion(
     15     pipeline='lstm_dynamic_threshold',
---> 16     hyperparameters=hyperparameters
     17 )
     18

/Applications/anaconda3/lib/python3.7/site-packages/orion/core.py in __init__(self, pipeline, hyperparameters)
     62         self._pipeline = pipeline or self.DEFAULT_PIPELINE
     63         self._hyperparameters = hyperparameters
---> 64         self._mlpipeline = self._get_mlpipeline()
     65         self._fitted = False
     66

/Applications/anaconda3/lib/python3.7/site-packages/orion/core.py in _get_mlpipeline(self)
     54         mlpipeline = MLPipeline(pipeline)
     55         if self._hyperparameters:
---> 56             mlpipeline.set_hyperparameters(self._hyperparameters)
     57
     58         return mlpipeline

/Applications/anaconda3/lib/python3.7/site-packages/mlblocks/mlpipeline.py in set_hyperparameters(self, hyperparameters)
    477         hyperparameters = self._sanitize(hyperparameters)
    478         for block_name, block_hyperparams in hyperparameters.items():
--> 479             self.blocks[block_name].set_hyperparameters(block_hyperparams)
    480
    481     def _get_block_args(self, block_name, block_args, context):

KeyError: 'mlprimitives.custom.timeseries_preprocessing.time_segments_aggregate#1'

source: gharchive/issue

authors: jgubner sarahmish

repo: signals-dev/Orion


text:
Error: Fail to load sigstore/cosign-installer/v2.0.0/action.yml

cosign-installer version: v2.0.0
When trying to use this version, the github action will sometimes fail on start up, and sometimes it'll pass the start up and proceed as expected.
Here's the code snippet of where I'm using the cosign installer:

Here is the error I'm getting:

Would this be an issue with v2.0.0? Would using the latest version fix these intermittent failures?
It's hard to know exactly how often it fails, but seems to be one in every 5 or so builds.

please use release v2.3.0
i will try to reproduce this issue, but i would say to use the latest version and if that persist the error we need to investigate deeper

We tried to use the latest version but still got the error

is it possible point me to your github action workflow?

our github action is on our company's github network. So I won't be able to point you directly to it.
The github actions are running on github hosted runners, with ubuntu-latest

Have you been able to re-produce the error on your end? We're still experiencing intermittent issues with this github action

@UrsusLvovich unfortunately not, we use this action across several projects and I never saw this error, and with the information you provide is kind of hard to debug.
things I would like to know:
1 - it is using a public GitHub runner? or private (this might have a possible impact)
2 - the action yaml definition (i know that you cannot share)
3 - which ubuntu version you are using
also please update to the latest action which is v2.4.0

We're using a github hosted runner
The action.yaml file is below
The runner is using ubuntu-latest

---
name: Deploy Images

on:
  push:
    branches:
      - "main"
      - "develop"
    tags:
      - "**"
jobs:
  push-images:
    name: Push Images
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-go@v2
        with:
          go-version: 1.17.9
      - name: Install Task
        run: |
          go install github.com/go-task/task/v3/cmd/task@latest
      - name: Build Images
        run: |
          task
      - name: |
        run: |
          host [REDACTED]
      - name: Login to Docker
        run: |
          echo ${{ secrets.HARBOR_REGISTRY_TOKEN }} | docker -D -ldebug login --username ${{ secrets.HARBOR_USERNAME }} [REDACTED] --password-stdin
      - name: Extract branch name
        shell: bash
        run: echo "##[set-output name=branch;]$(echo ${GITHUB_REF#refs/heads/})"
        id: extract_branch
      - name: Extract tag name
        if: contains(steps.extract_branch.outputs.branch, 'tags/release')
        run: echo "##[set-output name=tag;]$(echo ${{ steps.extract_branch.outputs.branch }} | cut -d '/' -f4)"
        id: extract_tag
      - name: Set tag name
        id: tag
        shell: bash
        run: |
          if [ "${{ steps.extract_tag.outputs.tag }}" != "" ]; then
            echo ::set-output name=name::${{ steps.extract_tag.outputs.tag }}
          else
            echo ::set-output name=name::${{ steps.extract_branch.outputs.branch }}
          fi
      - name: Install cosign
        uses: sigstore/cosign-installer@v2.3.0
      - name: Write signing key to disk
        run: 'echo "$KEY" > cosign.key'
        shell: bash
        env:
          KEY: ${{ secrets.COSIGN_PRIVATE_KEY }}
      - name: Tag, push and sign cds/pipeline
        env:
          COSIGN_PASSWORD: ${{ secrets.COSIGN_PASSWORD }}
        run: |
          docker tag cds/pipeline:latest [REDACTED]/cds/pipeline:${{ steps.tag.outputs.name }}
          docker push [REDACTED]/cds/pipeline:${{ steps.tag.outputs.name }}
          cosign sign --key cosign.key [REDACTED]/cds/pipeline:${{ steps.tag.outputs.name }}

thanks!
i've configure a self-runner (linux debian-11-bullseye-v20220519on GCP) using this repo https://github.com/actions/runner/
logs from the runner

√ Connected to GitHub

Current runner version: '2.293.0'
2022-06-16 09:54:00Z: Listening for Jobs
2022-06-16 09:54:10Z: Running job: build
2022-06-16 09:54:20Z: Job build completed with result: Succeeded

sample job
name: cosign with key
on:
  workflow_dispatch:

jobs:
  build:
    permissions:
      packages: write
      contents: read

    runs-on: self-hosted
    steps:
      - uses: actions/setup-go@v2
        with:
          go-version: '1.18'

      - name: Install cosign
        uses: sigstore/cosign-installer@v2.4.0

      - name: Install crane
        uses: imjasonh/setup-crane@v0.1

      - name: Build and sign a container image
        env:
          COSIGN_PASSWORD: ${{ secrets.COSIGN_PASSWORD }}
          COSIGN_PRIVATE_KEY: ${{ secrets.COSIGN_PRIVATE_KEY }}
        run: |
          cosign sign --key env://COSIGN_PRIVATE_KEY ghcr.io/cpanato/testing-ci-providers/test:latest

link for the execution: https://github.com/cpanato/testing-ci-providers/runs/6916203906?check_suite_focus=true
i've run that a few times and always worked

I have encountered the same error. Might this be related to the runner version  Current runner version: '2.283.3'?

not sure, is it possible to bump the runner version?

We bumped the runner version and it works fine now.

This issue can be closed. The problem was with our organization. Our organization has their own hosted github runners running all the jobs for the projects. But whatever configuration of the runner they had was causing some intermittent connection issues with some of the github modules. Within the last couple of days, they updated and changed the configuration on the hosted runners, so these issues SHOULD be resolved hopefully.

feel free to reopen or open a new one. thanks!

authors: UrsusLvovich cpanato usysrc

repo: sigstore/cosign-installer


text:
Support docker mirrors as default registry

Description
Currently, when no registry is provided into the Global Unique Name (GUN), such as vendor/image:version, cosign will default the registry do docker.io and try to upload/retrieve the signatures when using cosign verify vendor/image:version or cosign sign vendor/image:version.
This is very true when using the default "mirror" registry, but can be false when there are extra mirror registries configured on the docker-daemon.json.
Although it would be possible to manually call cosign sign MIRROR_REGISTRY/vendor/image:version, when using third-party tools, such as connaisseur, on a cluster where there are mirrors configured, connaisseur will automatically pass the GUN to cosign without the registry, since it was pulled from a mirror that was configured on docker-daemon.json, meaning that "I don't" have control over  the image/GUN provided to cosign, causing it to fail since the image will not exist on the docker.io registry, but rather lives on a private registry.
The scenario that causes us to not prepend with the registry is that we have several mirrors that the URL change depending on the cluster it lives (I have several EKS clusters running on different countries) and I can re-use the same manifest for all of them just by configuring the mirror to that specific region.

cc @imjasonh @jonjohnsonjr

authors: dlorenc williamokano

repo: sigstore/cosign


text:
Additional details on successful verification

Description
Today, when a manifest is successfully verified, other than a success message nothing else is shown. Owing to the fact that there is no verbosity flag (and the K8S_MANIFEST_SIGSTORE_LOG_LEVEL=trace env var does not show this), that means it isn't possible to show anything else. For example, what was the subject, who was the issuer, any annotations, cert details, etc? There needs to be some way to access more of this information especially when using the public Rekor instance and keyless signing.

Sorry for no response on this issue, but after several updates, k8s-manifest-sigstore supports verification with some cosign details like issuer, rekor url and so on (link).
This does not expose details in the console, but users can use this feature to specify certs that should be allowed.
Let me close this now.

authors: chipzoller hirokuni-kitahara


text:
貢獻者頁面詞條重複問題

Ref: #159 貢獻者頁面
有發現有人的詞條被重複計算，找時間再來修掉這個問題
以下示意圖，重複的獨角仙：

#162 做完了可是沒有 close XD

license: mit

license_source: bigquery


text:
On silex.sensiolabs.org/download the "fat" archive is mostly empty

This seems to be a recurrent problem, but the "fat" silex archive on silex.sensiolabs.org/download is empty (more or less).
I want to try Silex for a new project but I can't use composer on the server.
Any help ?

The silex archive to start the project is not something you would download on a server anyway, but locally (unless you work on the code directly on the server, but I really hope you don't).
And the same can be true for Composer. nothing prevents you from uploading your whole local project to the server rather than uploading only your own code and then running composer on the server. This means that the requirement would be to be able to use composer locally, not on the server.
Regarding the download itself, I cannot really help. this must probably be solved by @fabpot who has access to the Silex server.

I was not speaking of the production server :)
I've just realize that there's a windows installer for composer. I'll try to install it on my desktop.

repo: silexphp/Silex


text:
fix: prioritize prefix matches but still find late in string matches

This change is

components/language-chooser/common/find-language/searchForLanguage.ts line 46 at r1 (raw file):
Previously, andrew-polk wrote…
I got tripped up by the term location for a while thinking it had something to do with region as opposed to the location within the string.
Would it be helpful to rename locationIndepdendentResults to allFuzzyMatchResults or something similar? Possibly just fuzzyResults? Or even allResults?

Done. Makes sense, I didn't thank of that!

repo: sillsdev/EthnoLib


text:
Update Android build

This PR allows the app to be built with the latest version of Android Studio.
It fixes a dependency issue, so all that needs to be done is ./gradlew assemble (or run it from android studio)

Update the SDK compile version to 33 to fix dependency issues
Update the Build plugin as a dependency

Perfect, thanks!

repo: silverbulletmd/silverbullet


text:
BUG Security/logout doesn't log out properly if using basic auth

https://stackoverflow.com/questions/449788/http-authentication-logout-via-php
If a user logs in with basic auth, going to Security/logout won't log them out, as the Authorization header re-authenticates them.

Assuming this issue already existed in 3.x, I'm downgrading the criticality

@tractorcow Chris said you might've addressed this already?

No, I've only fixed basic-auth not logging users in at all.
Once that was fixed, the above issue surfaced.

Right. I think this can wait until 4.1.0

authors: chillu tractorcow

repo: silverstripe/silverstripe-framework


text:
SS4 DBDate::explodeDateString regex issue

I have a Date field that I am setting the default value of through populateDefaults() by
$this->Date = DBField::create_field(DBDate::class, time())->getISOFormat();

This throws an exception with error message [Emergency] Uncaught InvalidArgumentException: Invalid date: 'y-MM-dd'. Use y-MM-dd to prevent this error. which doesn't make much sense as it is the format I am using
Partial trace
SilverStripe\ORM\FieldType\DBDate->explodeDateString(y-MM-dd)
DBDate.php:530
SilverStripe\ORM\FieldType\DBDate->fixInputDate(y-MM-dd)
DBDate.php:69
SilverStripe\ORM\FieldType\DBDate->parseDate(y-MM-dd)
DBDate.php:39
SilverStripe\ORM\FieldType\DBDate->setValue(y-MM-dd, ChristopherDarling\Mysite\Models\NewsArticle, )
DataObject.php:2672
SilverStripe\ORM\DataObject->dbObject(Date)
FormScaffolder.php:97
SilverStripe\Forms\FormScaffolder->getFieldList()
DataObject.php:2065

getISOFormat() is literally returning the string 'y-MM-dd' rather than the date value in that format. see: https://github.com/silverstripe/silverstripe-framework/blob/c942ca5be6c6b526b62905e6ff6eb3942ae702c5/src/ORM/FieldType/DBDate.php#L206-L214
You'll need to do something like:
$date = DBField::create_field(DBDate::class, time());
$this->Date = $date->Format($date->getISOFormat());

Ahh.. that totally makes sense, thanks Dan - I've managed to get this working now

authors: christopherdarling dhensby


text:
FIX Save long decimal values into DBDecimal field correctly

When a decimal number is long and has more than the max number of 9s, it gets rounded to the nearest int.
The ctype_digit check was basically testing for not having the decimal point in it while converted to a string when it overflowed. In the PR I've used Stephen's suggestion from the issue of comparison using the epsilon constant.

Can you please create a corresponding issue for this? That will make it easier for us to put it through backlog refinement etc and therefore we're more likely to review it sooner.

There already was one which is why I started looking into it. Should've referenced it here, my bad — https://github.com/silverstripe/silverstripe-framework/issues/10592

authors: GuySartorelli michalkleiner


text:
Expanded bulk action checkbox hit zone

Increased checkbox click area so it's harder to accidentally click underlying link. Also lightened selected checkbox background colour for consistency (fixes #4342, fixes https://github.com/silverstripe/silverstripe-cms/issues/400, closes #4352)

Thanks Jono! Can I just confirm that you've cross-browser tested this a bit, incl. IE9?

Yep, good in IE8, IE9, and Chrome on Mac. It's pretty basic css, if it works the same between IE8 and Chrome it should be pretty safe everywhere! :grinning:

authors: chillu jonom


text:
GUID validation is incorrect

current \SilverStripe\SAML\Helpers\SAMLHelper::validGuid allows:

only uppercase "guid"

should be case insensitive

allows actually invalid values to be considered as valid

standard guid should be hex range [0-9a-f]
current implementation check [A-Z0-9] which allows non-guid to pass validation

expected result of validation: only valid guid format is passing

Can you please provide some basic examples of what you're finding correct and incorrect? Are you saying that we should only accept A-F range rather than A-Z?

"textual representation of UUID are represented as 32 hexadecimal (base-16) digits"
source: https://en.wikipedia.org/wiki/Universally_unique_identifier

sometimes (microsoft) surrounds guid with brackets { }

only [0-9A-F] range should be accepted as for example Z is not valid character of guid
example:
valid:  11111111-1111-1111-ffff-000000000000
invalid: 11111111-zzzz-1111-ffff-000000000000
created pull request #22

authors: JakubDolba robbieaverill

repo: silverstripe/silverstripe-saml


text:
ENH Add silverstripe/doorman

Issue https://github.com/silverstripe/.github/issues/350
~Needs https://github.com/silverstripe/supported-modules/pull/49 merged before CI will work~

Needs https://github.com/silverstripe/supported-modules/pull/49 merged before CI will work

This is #49... did you mean to put that note in https://github.com/silverstripe/doorman/pull/49 instead?

Have moved

authors: GuySartorelli emteknetnz

repo: silverstripe/supported-modules


text:
atom-material-syntax-dark variation

There is also a dark variation of the original Sublime theme that would be nice to have:

This would require an entire new Atom package. I'll probably resolve this in the UI package with a setting for a darker background.
I'll close this issue as it doesn't have a place in this repository, but I'll add the feature, don't worry!

Cool, looking forward to it!

authors: silvestreh theproducer

repo: silvestreh/atom-material-syntax


text:
[histogramnd] unit tests vs histogramdd failure (rare)

A couple tests from one test suite (the one where histogramnd is compared to histogramdd) can sometimes fail due to some precision error (when using np.all_close when comparing both results).
This suite was only added in order to test histogramnd with large data sets (everything is pretty much already covered by the other tests, i think, but with very small hand written data sets).
The failure rate is low (as far as I know), but still...
Possible solutions :

at the very least print the state of the random number generator (RNG) when there is a fail so we can reproduce the result (@kif's  suggestion)
remove those tests from the automatically executed tests (run them manualy when needed)
generate a data set that works, manualy validate the results, and then provide that data with the tests (or actually just the RNG state)

So actually the fail occurs when comparing the sum of the weighted histogram to the sum of the weights (so it's not something between histogramnd and histogramdd).
@vasole suggestion :

call weights.sum(np.float64) instead of weights.sum().
call cumul.sum(np.float64) instead of cumul.sum().

This solution seems to work (it passed on a data set that failed previously).

repo: silx-kit/silx


text:
[silx convert] Allow to use a filter id for compression

The command line argument is converted to int, if possible.

I think the accompanying documentation should be improved too. For instance by showing the usage with some typical filters id (GZIP, LZ4,..) and providing a link to the HDF5 group page where the filter ids are stored:
https://support.hdfgroup.org/services/filters.html

I was also thinking about adding mapping (id<->name) on hdf5plugins? At least filters exposed.

authors: vallsv vasole


text:
Fix #91 (spec_date_to_iso8601)

Fix for #91
Added unit tests for spec_date_to_iso8601.
Nominal tests only.
The zone keyword is not tested. I haven't added tests for the error cases either.
I checked that the new tests fail with the previous version of the function.
BTW the function only works if the provided spec dates are in english (Mon, Apr, Aug, Sun, ....). Are we sure this will always be the case?

I have investigated Jérôme's suggestion of using datetime. It could simplify the function, especially if we expect to add more date formats in the future. Initially I expected SPEC to write a single format.
>>> from datetime import datetime
>>> d = datetime.strptime('Thu Feb 11 09:54:35+0100 2016', '%a %b %d %H:%M:%S%z %Y')
>>> d.isoformat()
'2016-02-11T09:54:35+01:00'
>>>
>>> d = datetime.strptime('Sat 2015/03/14 03:53:50+0200', '%a %Y/%m/%d %H:%M:%S%z')
>>> d.isoformat()
'2015-03-14T03:53:50+02:00'

Des anyone have any objection to rewriting the entire function?

I think this works only if you set the locale.LC_TIME right.
e.g :
d = datetime.strptime('Mo Feb 11 09:54:35 2016', '%a %b %d %H:%M:%S %Y')

doesn't work if the computers locale is set to german (locale.setlocale(locale.LC_TIME, 'de_DE.UTF-8'))
It expects this :
d = datetime.strptime('Do Feb 11 09:54:35 2016', '%a %b %d %H:%M:%S %Y')

authors: PiRK dnaudet


text:
:murishite:

つらい, もうやだ, やめたいに対して無理してってリアクションするようにしました！！！

ありがとおお～～！！
とてもありがたい


text:
Cv option

Description

suppress shuffle and n_splits parameters
introduce the cv parameters
reduce number of valid methods by redundancy of cv
update documentation
update unit tests

Type of change
Please check options that are relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Not tested yet, but many unit tests are impacted and updated.
Checklist:

[x] I have read the contributing guidelines
[ ] I have updated the HISTORY.rst and AUTHORS.rst files
[ ] Linting passes successfully : flake8 . --exclude=doc
[ ] Typing passes successfully : mypy mapie examples --strict --config-file mypy.ini
[ ] Unit tests pass successfully : pytest -vs --doctest-modules mapie
[ ] Coverage is 100% : pytest -vs --doctest-modules --cov-branch --cov=mapie --pyargs mapie
[ ] Documentation builds successfully : cd doc; make clean; make html

Hello @gmartinonQM! Thanks for opening this PR.

In the file mapie/tests/test_estimators.py:

Line 14:1: F401 'sklearn.base.RegressorMixin' imported but unused
Line 193:36: F821 undefined name 'all_methods'
Line 202:36: F821 undefined name 'jackknife_methods'
Line 202:56: F821 undefined name 'cv_methods'
Line 211:36: F821 undefined name 'cv_methods'
Line 227:36: F821 undefined name 'all_methods'
Line 250:36: F821 undefined name 'all_methods'
Line 259:54: F821 undefined name 'expected_widths'
Line 260:69: F821 undefined name 'expected_coverages'

Do see the Hitchhiker's guide to code style

repo: simai-ml/MAPIE


text:
Proper use of Bounding Box

Currently the bounding box is not very much in "use" cause one can still draw outside the bounding box. In reality since voxelart is similar to pixel art and the bounding box should serve as a canvas (like in 2D). It would be nice if bounding box was the only "drawable" portion, not only will it be a good usability improvement, it would also optimize/improve most internal features eg the exporter and #210.

At most I would replace it with an infinite floor where you can draw voxels, I use box wall currently to take height to my project because i can not draw on Y axis (instead of clicking so much on a tower of voxels)

That will be a disaster if we implement the face tool, so i think it wrong

The best solution is to set a reasonable size as the default bounding box eg 126,  you can then expand it (currently supported)  to fit your needs.

Why face tool is a disaster? How can I expand the bouncing box?

In simple term the max size supported  by voxelshop is the max size of an Integer. If we made the ground that size, using the face to fill the ground like in MV will probably crash on most computers. There is an arrow button below the bounding box button,  clicking on the button will show the input for changing the bounding box size,  I wonder why such an important function was hidden behind a button.

this limit comes from Java?
Something that really worries me is VS performance, I tested those days Magica Voxel and I really can build complex voxel art more faster than in VS (more faster, but the result is less efficient for games)
In Voxel Shop a simple project of 400 voxels begins slowing down the program on my computer (i7-4720HQ, nvidia GTX 970M, SSD rom, 16 Gb ram DDR4)
I use PHPStorm commonly (based on Intellij IDEA, built in Java too)
Some years ago they have several performance problems but finally they solve this (I did not followed their development so I do not know how)
Intellij IDEA is open source so probably you can learn something from their repo (how they manage large files or how they manage program task or if they are using some libraries for performance)
Android Studio (by Google) is based on Intellij IDEA too

Yeah the limit is from java. Magicavoxel will be faster than voxelshop due to fact that most of it current limitations were due to performance consideration e.g it 126 max size and it was written in c++ (although that doesn't matter much). It's true that VS begins to get slow when working with large models and even Magicavoxel that am running via wine tends to perform better but i think that's due to multiple reasons and i wish simlu  would be able to look into this.

I'm a little lost with performance I do not know how to improve this part, it requires more deep knowledges, but sure with all our help @simlu will find the way to solve this

No no no, the issue does not come from Java. There are two parts here:
(1) Data structure
(2) Rendering
Why you are currently experiencing slowness is bc of rendering. I'm using a software renderer to be compatible easily with all os.
The next step then is to change the data structure. That is important for undo / redo operations and for really large objects...

Im planning to address rendering first.

Sounds nice @simlu

Nice to hear, but the limit i was talking about was Integer.MAX_VALUE.

Yeah, that's an implementation detail and not the reason why it's slow. In general you are right, but it's not the limiting factor here.

Nooooo,  am not saying that's why it's  slow, am just saying making the default ground size that amount will be unrealistic and will probably cause more issues.

There is an implicit limit ~pow(MAX_INT, 1/3)

Im not opposed to an infinite ground... Not sure how it would display.

Never saw that :D.. Infinite ground? Are you not bothered about performance?

Well, by infinite I meant ~pow(MAX_INT, 1/3)

not infinte, but if it follows the camera movement? like side-scrolling games

anyway, I do not know that I can change their size, with this option bounding box is more than done to go
A infinite floor does not needs attention at this moment

I'm reworking milestones atm around performance improvements. Closing this as it went way off of topic. Created a new issue here: https://github.com/simlu/voxelshop/issues/216

authors: funabab rubengc simlu

repo: simlu/voxelshop


text:
Export Options: Merged / Per Layer / Per Voxel

@RichysHub
Will post updates here

Basic work done in this commit: https://github.com/simlu/voxelshop/commit/436d83803a7fbcd4da3cf47856bbfbade821e98e

@RichysHub Can you confirm please that this works as expected now?
https://s3-us-west-2.amazonaws.com/voxelshop/build/develop/voxelshop-bin.zip
I'm particularly interested if quads are noticeably faster in Blender. If you could compare to your importer and see what changes we still need to make that would be great!
Work remains:

object center option should be per collada object center (not global center)

Looks to work great.
Import time seems to be comparable to my script, certainly the same order of magnitude.
The per object center is a single Blender operation, Set Origin->Origin to Geometry.
I'm not well versed enough to know the quads/tri tradeoffs.

Having the colors being texture based also allows for some effects that are harder to achieve with my importer.

Looks very pretty! How long does it take to render?
I've fixed the collada object center positioning in this commit: https://github.com/simlu/voxelshop/commit/a86d8aea07f9ac88b7994510739c685a14d1e2b2
Please test and see if the centers now work as expected!

Pre release is up here: https://github.com/simlu/voxelshop/releases/tag/1.8.17

authors: RichysHub simlu


text:
Primary key not inferred when using Dart tables with inheritance

What was I trying to do?
Setting a primary key on a Table by overriding the primaryKey property of the Table class.
What did I expect to happen?
I have the following code:
my_database.dart
part 'my_database.g.dart';

LazyDatabase _openConnection() {
  return LazyDatabase(() async {
    final dbFolder = await getApplicationDocumentsDirectory();
    final file = File(join(dbFolder.path, 'dev_db.sqlite'));
    print('SQLite File Path: ${file.path}');
    if (kDebugMode) {
      // Does not run in Profile or Release Mode
      if (file.existsSync()) {
        await file.delete();
      }
    }

    return VmDatabase(file, logStatements: true);
  });
}

@UseMoor(
  tables: [Projects],
  daos: [ProjectsDao],
)
class MyDatabase extends _$MyDatabase {
  MyDatabase() : super(_openConnection());

  @override
  int get schemaVersion => 1;

  @override
  MigrationStrategy get migration {
    return MigrationStrategy(
      beforeOpen: (OpeningDetails details) async {
        await customStatement('PRAGMA foreign_keys = ON');
      },
    );
  }
}

projects.dart
class Projects extends EntityWithUuid {
  TextColumn get name => text()();
  TextColumn get client => text().nullable()();
}

entity_with_uuid.dart
abstract class EntityWithUuid extends Table {
  TextColumn get id =>
      text().clientDefault(() => GetIt.instance.get<Uuid>().v4())();

  @override
  Set<Column> get primaryKey => {id};
}

I expected the primary key of the projects table to be the id column
What actually happens?
When the application starts, the following is logged in the console:
flutter: Moor: Sent CREATE TABLE IF NOT EXISTS projects (is_playground_entity INTEGER NOT NULL DEFAULT 0 CHECK (is_playground_entity in (0, 1)), id VARCHAR NOT NULL, name VARCHAR NOT NULL, client VARCHAR NULL, color INTEGER NOT NULL, total_worktime VARCHAR NULL, total_earned VARCHAR NULL, time_goal VARCHAR NULL, budget_goal VARCHAR NULL, goal_type INTEGER NOT NULL DEFAULT 0, PRIMARY KEY (id)); with args []

I noticed that even though I set Set<Column> get primaryKey => {id};, PRIMARY KEY is not set on the id attribute. I verified this using a SQLite browser as well.
In addition, I also got rid of using the EntityWithUuid abstract class you see above, just to make sure it wasn't my code that was causing the problem.
Eventually, I resorted to doing:
@override
  List<String> get customConstraints => [
        'PRIMARY KEY (id)',
      ];

on the table, which works.
Wanted to make sure there isn't any bugs with Set<Column> get primaryKey => {id};. Perhaps I'm doing something wrong in my code?

Thanks for the report. This is a bug in the generator that will be fixed in the next version.
Until that's released, another workaround (apart from overriding customConstraints) is duplicating the primaryKey getter in the child class:
class Projects extends EntityWithUuid {
  TextColumn get name => text()();
  TextColumn get client => text().nullable()();

  @override
  Set<Column> get primaryKey => {id};
}

Not ideal, but at least you don't have to touch generated code which can be very fragile.

Gotcha, thanks for another workaround example. Appreciate the quick response!


text:
Update sqlcipher to 4.5.0

I have just updated dependencies to SQLCipher 4.5.0 for Android, mac and iOS

Thanks!


text:
🛑 Victoris Webby is down

In a3112fd, Victoris Webby (https://$VW_SITE1/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Victoris Webby is back up in e2bf1fb.


text:
LOL WTF MAN???

i have rooted LG and webOS Dev Manager for install IPK directly to my device. IDneed compilate and other shit. where is *.ipk installer for this app???

Why bother? Just use the official PLEX app (until it stops working due to
no longer being updated).
On Sun, 26 Feb 2023, 7:21 am orabeidm, @.***> wrote:

i have rooted LG and webOS Dev Manager for install IPK directly to my
device. IDneed compilate and other shit. where is *.ipk installer for this
app???
—
Reply to this email directly, view it on GitHub
https://github.com/simonjhogan/PLAYz-Media-for-PLEX/issues/125, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRTC4DBTEBV3FH466O2CRLWZKHYVANCNFSM6AAAAAAVIDVTF4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

authors: orabeidm ryballs

repo: simonjhogan/PLAYz-Media-for-PLEX


text:
Update Torii dependency to 0.2.2 (latest release)

Updated Torii dependency to latest release (squashes a few ios bugs)

:+1:

authors: andrewobrien marcoow

repo: simplabs/ember-cli-simple-auth-torii


text:
Add slightly clearer install instructions

Make explicit the need to install the helpers when using ember-cli

That's already in the Addon install instructions that the README already links to.

Whilst that's true, so are the config instructions below it.  Having those there also led me to think that this was all that was required.  Perhaps they could be removed?

Yeah, good point.

authors: marcoow mfeckie

repo: simplabs/ember-simple-auth


text:
🛑 SimpleLogin MX2 is down

In dc81e20, SimpleLogin MX2 (https://mx2.simplelogin.co/live) was down:

HTTP code: 502
Response time: 391 ms

Resolved: SimpleLogin MX2 is back up in 23d4a71.

repo: simple-login/uptime


text:
Can't scroll source listings with the keyboard

Hi,
For some reason, it's not possible to scroll the source listings that simplecov-html creates with the keyboard in Chrome, even when you click inside the scrollable div.

I can confirm that this feature is missing. Specifically, neither up/down arrows nor pageup/pagedown keys work.
Also, the "Find" action works for source code, but not for line numbers, so there's no way to jump to a line number (e.g. when you just want to know if one specific line is covered)

authors: CyberShadow jaredbeck

repo: simplecov-ruby/simplecov-html


text:
NameError: name 'OctoInference' is not defined.

I have download the octo files according to Octo Inference Setup,why still no octo model found

by the way I am running command:
python simpler_env/main_inference.py --policy-model octo-small --ckpt-path None --robot widowx --policy-setup widowx_bridge   --control-freq 5 --sim-freq 500 --max-episode-steps 60   --env-name StackGreenCubeOnYellowCubeBakedTexInScene-v0 --scene-name bridge_table_1_v1 --rgb-overlay-path ManiSkill2_real2sim/data/real_inpainting/bridge_real_eval_1.png  --robot-init-x 0.147 0.147 1 --robot-init-y 0.028 0.028 1 --obj-variation-mode episode --obj-episode-range 0 24   --robot-init-rot-quat-center 0 0 0 1 --robot-init-rot-rpy-range 0 0 1 0 0 1 0 0 1 --additional-env-save-tags octo_init_rng_0

And the folder is like:
SimplerEnv-main
-octo
-docs
-examples
-octo
....
-simpler_env
-main_inference.py
...
-...

I think you forgot to cd octo/; pip install -e .

I think you forgot to cd octo/; pip install -e .
Please follow the instruciton in the Octo inference setup in README (it also requires installing jax)

I am sure I have done this all,the "pip install -e ." result is:
Successfully built octo
Installing collected packages: octo
Successfully installed octo-0.0.0
but still the error

repo: simpler-env/SimplerEnv


text:
Merge several forks' commits; add log level data (FKA siphon) service

I started with the branch that updates Faraday by @hollow and added a few features:

verbose logging of Faraday requests and responses via the APPNEXUS_API_DEBUG environment variable
added LogLevelDataService and LogLevelDataDownloadService classes to handle finding new log level data and downloading the files.  There was a several year old attempt to do this on this PR: https://github.com/simplifi/appnexusapi/pull/3 back when it was still called Siphon

:+1:

@apurvis Thanks for the PR!
We're going to just double check that our current code works with this, and then we should be good to merge. The ones of us who maintain that code are all going to a conference through the end of the week, so it might be early next week before it gets done.

there's no rush at all on our end; take your time.
wish this project had some VCR/webmock specs though!

any updates here? are you planning to merge?

@apurvis @rfroetscher @hollow Released as v0.0.6. Thanks again!

:+1:

authors: apurvis hollow joshuawscott rfroetscher

repo: simplifi/appnexusapi


text:
AS2.1.2无法进行编译了

截止到7月26日，AS和SDK都是最新的
下午的时候突然无法编译，提示自动生成的所有Bean找不到，不存在
我试过了clean，invalidate caches and restart，重启AS，重启电脑，删除build目录，删除gradle中的PowerfulAapter的配置，然后重新添加依赖，依然都是不行的。。

        classpath 'com.android.tools.build:gradle:2.1.0'
        // the latest version of the android-apt plugin
        classpath 'com.neenbedankt.gradle.plugins:android-apt:1.8'

你的android plugin 和 android-apt 版本对吗，我怀疑是版本问题

抱歉，问题找到了，是ButterKnife注解的时候，xml中我删除了某个控件，但是代码中注解依然存在忘记删除，导致编译失败，但是报错一直指向PowerfulRecyclerViewAdapter的注解DataBean


text:
Hash symbol in custom NormalEntry section

Describe the bug
Unable to create a hash character in a custom NormalEntry block. Tried both:  C# and escaping it:  C\#
To Reproduce
  custom_sections:
    - title: Skillz
      entry_type: NormalEntry
      entries:
        - name: "Programming Languages:"
          highlights:
            - "C\#"
            - "Special characters test: üğç"

Screenshots
Console Output:
CRITICAL | ScannerError('while scanning a double-quoted scalar', <ruamel.yaml.error.FileMark object at 0x000002089ECD4380>, "found unknown escape character '#'"

Thank you for finding this. In fact, I never coded something to escape special $\LaTeX$ characters. 08a4fcdcc09cacaff6fbbc20ea90187e1f2b6ad1 fixes the issue. Now, plain C# is fine. The new version of RenderCV with the fix will be online in ~20 minutes.

authors: Vinhcognito sinaatalay

repo: sinaatalay/rendercv


text:
Optional Keys crash Xcode 16

It seems that optional keys are not caught by
		if (defaultValue as? _DefaultsOptionalProtocol)?._defaults_isNil == true {
			return
		}

The good news is that this is caught in the tests...

XCode 15.4 defines ExpressibleByNilLiteral
Conforming Types
[Optional]
XCode 16 states:
[Optional]
Conforms when Wrapped conforms to Escapable.
meanwhile, Escapable seems to have no requirements, but not to be available without
-enable-experimental-feature NonescapableTypes
If I do enable that feature, then compilation fails with a circular reference as soon as I include a key

It seems to be the same as this issue: https://github.com/apple/swift/issues/74282

Brilliant - thank you.

fyi - not fixed on beta 2

Fixed in Beta 3 :)

repo: sindresorhus/Defaults


text:
The app does not show up in the menu bar - solution

The app does not show up in the menu bar
macOS hides menu bar apps when there is no space left in the menu bar. This is a common problem on MacBooks with a notch. Try quitting some menu bar apps to free up space. If this does not solve it, try quitting Bartender if you have it installed.

You can sort icons in MenuBar using Cmd button + mouse.
No need to use any extra soft like Bartender. Just move app to the right corner of Menubar :)

I don't think you are fully understanding what the text is saying. It's about what to do if you cannot see the menu bar item in the menu bar. It's not a recommendation to use Bartender.

authors: sindresorhus ukushu

repo: sindresorhus/System-Color-Picker


text:
docs: add awesome list template

Since this is not awesome list contributions, so I removed the pull request templates.
This pull request adding awesome list template for awesome. Which is to make it easier for people to contribute their awesome list.
Also I'm added ready-to-fork template here: https://github.com/bl0cknumber/awesome-template

Can this be closed in favor of https://github.com/jthegedus/awesome-list-template ?

I don't think that this super-simple template is useful to get started. You'll have to do a lot of manual work afterwards to include things like license, issue templates, linting etc.

I don't think that this super-simple template is useful to get started. You'll have to do a lot of manual work afterwards to include things like license, issue templates, linting etc.

It's the repository that include things like license, issue templates, linting, you just need to modify it by yourself https://github.com/bl0cknumber/awesome-template
I can't close this.

It's great that you are maintaining a template. But perhaps you can propose a link directly to the template repository rather than the intermediary .md file.
I find that https://github.com/jthegedus/awesome-list-template has more features and is also equipped with instructions of how to use it.

It's great that you are maintaining a template. But perhaps you can propose a link directly to the template repository rather than the intermediary .md file.
I find that https://github.com/jthegedus/awesome-list-template has more features and is also equipped with instructions of how to use it.

I think you are missing something, I provide the link to the intermediary .md file https://github.com/sindresorhus/awesome/pull/2159/files
I already take a look of the repository and it's looks promising with one more addition for awesome bot.
Instructions is really useless to add as we can mark the repository to Public template iirc, after all, I will close this PR because I don't have the time to update my repository for awesome bot.

repo: sindresorhus/awesome


text:
Support for data URIs

When using a data URI as a pageres source (data:text/html;base64,PGgxPkZPTzwvaDE+), pageres generates a white, empty image file.
Is it possible to get pageres to use data URIs? phatom has support for data uris afaik!

Probably just need to support data-URLs in our URL detection: https://github.com/sindresorhus/pageres-cli/blob/4301a0964f5e6f352299c44b4ad3f1d38d2c8db4/cli.js#L150
PR welcome :)

authors: JacksonGariety sindresorhus

repo: sindresorhus/pageres


text:
Diff: shifted indentation

Hi! It looks like some lines are sometimes weirdly indented:

Here's what it looks like without Refined GitHub:

And here's the URL to reproduce:
https://github.com/vatesfr/xen-orchestra/pull/3155/files#diff-37458fb9e51d193ef360403ba893e2e2R47
(Might be related to #2085)

I don't see any weird indentation, I think you're just noticing the missing + sign, which leaves the empty space behind

Notice how line 50 is shifted right compared to line 49 while they should be aligned. And here's how it looks like when you select it:

Indeed, maybe #2088 will fix it

Hopefully :)

I tried to run the fix manually (simply adding the extra CSS directly in the browser) and it fixes #2085 but it doesn't seem to work for this issue so there might be something more to it.

authors: bfred-it pdonias

repo: sindresorhus/refined-github


text:
text-decoration: line-through; not working for deleted branches.

Only colon is line-throughed here.

Link to PR in screenshot: https://github.com/sindresorhus/refined-github/pull/2233

Thanks for the report!
It’s a duplicate of https://github.com/sindresorhus/refined-github/issues/1985
I hope someone sends a PR soon ☺

Reopening because now this needs to be re-implemented as its own feature: cross-deleted-pr-branches
Most of the logic is in https://github.com/sindresorhus/refined-github/commit/cb8de12eb680475024c21d5586aa31a232493c73 (removed) but now we only need to use the part that adds the strike-through, not the linkifier.

The feature could also point the link to the repo root (instead of the 404 deleted branch) to make it somewhat useful.

Forked repo root or the primary repo?

The head branch could either be from the same repo or from a fork. I think it would only make sense to point to the fork’s default branch and do nothing if it’s the same repo.

authors: fregante gurrrung


text:
Watch button dropdown not on top of issue details.

Hey there, Whenever I hover over the watch button, there's a issue with the z-ordering of the panels. This does not happen without the extension on my side.
I'm running chrome v76.0.3809.100 on macOS Mojave 10.14.5

This is due to f8c2fab4cfd8f85f19552a897f1a6df49ea441d9

authors: fregante velddev


text:
envs option doesn't work from within overrides option

It seems that errors are still flagged when specifying envs options within the overrides option.
For example, with the below configuration inside package.json, I still see document is not defined and  window is not defined.  Our xo version is specified as "xo": "^0.17.0".
"xo": {
    "envs": [
        "node"
    ],
    "overrides": [
        {
            "files": "app/js/**/*.js",
            "envs": [
                "browser"
            ]
        }
    ]
}

Actually this seems to be the case for any overrides, not just envs.

How are you using XO? Directly or through an editor plugin?

Directly through cli, e.g. xo index.js.
I've created a simple example if it helps? https://github.com/davidpauljunior/xo-overrides-test

@davidpauljunior I have this working for me without an issue. Here's my config:
{
 "xo": {
    "space": true,
    "semicolon": false,
    "overrides": [
      {
        "files": "client/**/*.js",
        "envs": ["browser"]
      }
    ]
  }
}

node is applied as a default env so you shouldn't need to specify that at the top-level. Also try specifying the value as an array. Hope it works for you.

Looks like you're right @lsdriscoll, in my example repo I'd missed the enclosing array.  I'm sure I had these errors even with those, but as it's working now in my example repo, I'll close this.  Thanks for the comment.

authors: davidpauljunior lsdriscoll sindresorhus

repo: sindresorhus/xo


text:
no more release 11.11 ?

Fixed. Thanks

authors: hebus jfletouze

repo: sinequa/sba-angular


text:
Components / Export (fix): added auditing event to the export functionality

The export functionality wasn't audited anymore (I guess there was a change to the API?) so I added an auditing event for each type of export possible.
Would you like me to remove the old code for sending the audit of the export? (which doesn't work anymore, like I said)

I'm sorry, I didn't notice this pull request..!
For me, the best way to send audit events to the backend is still as it is currently done in the QueryExportWebService. So if the backend does not process correctly the audit events, this is a bug and I will open a ticket internally...

In fact there is an error in the QueryExportWebService... The audit event is not structured properly. So I don't think that this ever worked...

An update about this PR: Even after resolving the structure of the audit events sent by the front end, it seems that the events are still not recorded. There seems to be a problem in the backend, linked to this specific web service...

Is this something that you will request to be fixed or should we apply the code I was using in this PR?

It's already fixed and will be delivered in a hotfix of the 11.6.1 in the following days.
The code in this PR is fine too, but keep in mind that when you'll upgrade Sinequa, these events will be redundant with the standard ones.

authors: Guillaume-Developer ericleib


text:
POST /packages

Path: /packages
Method: POST
Auth: COORDINATOR
Payload:
{
  name: String,
  items: [{ item: Item, quantity: Number }],
  price: Number,
  vat: Number
}

Response:
{
  name: String,
  items: [{ item: Item, quantity: Number }],
  price: Number,
  vat: Number
}

Notes:
Creates a new package.
Status codes:

Code
Rule

400
Invalid payload

200
Always

Tests:

Missing field on payload
Should return a 400 status code.

Invalid field on payload
Should return a 400 status code.

Valid payload
Should return the new package.

Again, you create a package in the context of an event (template) or in a company participation.
One of those cases is already taken care of here: https://github.com/sinfo/deck2/issues/74


text:
[Mobile] Single embed in a text line always takes the full width

Because of the following code in TextLine class
    if (widget.line.hasEmbed && widget.line.childCount == 1) {
      // For video, it is always single child
      final embed = widget.line.children.single as Embed;
      return EmbedProxy(
        widget.embedBuilder(
          context,
          widget.controller,
          embed,
          widget.readOnly,
          null,
        ),
      );
    }

Single embed in a text line always takes the full width. This makes sense for image and video. But my use case is mentions that are implemented as custom embed type

On the screenshot above cursor is on the right edge because embed took the whole width.
If you type at least one more character then everything looks good again

Any ideas on how to workaround the issue? Or what's the best way to make this behaviour configurable? I am willing to contribute

Actually for image we allow it to not take the whole line.
I think the code you pasted is the right place. And you can see it is meant for video.
Well, in the past embed is either image or video.
I guess you want the new embed behavior to be more like image.
Just check how image works in the code following the section.

If you change the if condition to be video only, your embed will probably fall through to the rest of the code

@singerdmx In the code that I posted in the initial message I don't actually see a type check related to video although comment says so.
Plus in the debugger image flow falls into same if condition as video https://github.com/singerdmx/flutter-quill/blob/master/lib/src/widgets/text_line.dart#L135

If you change the if condition to be video only, your embed will probably fall through to the rest of the code
To avoid breaking changes I would keep logic for images and videos as it is now

What I am trying to say is:
For the if condition here (code you pasted), you can restrict it to only types of video and image. And you can follow how image works when it comes to the scenario which allows embed not to take up the whole line

Also in document load method we automatically add line break for video type. Just fyi

Thanks, @singerdmx! I'll try to come up with the solution. Closing this issue for now

I have the same problem,more info about this?

hi @huangsir0  i try to solve this problem by some trick, add a space before adding customEnbed, but when user delete the space the customEnbed will take whole line. so may be have better method to solve this issue and i browsed all the issues and didn't see a good solution
    _controller.document.changes.listen((event) {
      // due to customEmbedWidget takes 100% width of view, so add space in editor start
      String text = _controller.document
          .toDelta()
          .first
          .data
          .toString()
          .replaceAll('\n', '');
      if (!text.startsWith(' ') && text.length < 2) {
        _controller.document.insert(0, ' ');
        _controller.updateSelection(
          TextSelection.collapsed(offset: _controller.document.length),
          quill.ChangeSource.LOCAL,
        );
      }
    });

hi @huangsir0 i try to solve this problem by some trick, add a space before adding customEnbed, but when user delete the space the customEnbed will take whole line. so may be have better method to solve this issue and i browsed all the issues in repo and didn't see a good solution
    _controller.document.changes.listen((event) {
      // due to customEmbedWidget takes 100% width of view, so add space in editor start
      String text = _controller.document
          .toDelta()
          .first
          .data
          .toString()
          .replaceAll('\n', '');
      if (!text.startsWith(' ') && text.length < 2) {
        _controller.document.insert(0, ' ');
        _controller.updateSelection(
          TextSelection.collapsed(offset: _controller.document.length),
          quill.ChangeSource.LOCAL,
        );
      }
    });

you can use expanded of EmbedBuilder to resoule this issue, like this:

Thanks, https://github.com/singerdmx/flutter-quill/pull/1141#issue-1640497433 this PR solved this issue recently

repo: singerdmx/flutter-quill


text:
Add a footer to all the created blogs that the blog are made with love using JekLog

To increase the awareness of the platform we have to add a link to jeklog.com in the footer.

Feature added in the production environment.


text:
[r] [WIP] build against shared library libtiledbsoma

Issue and/or context:
This PR enables building against a system-level libtiledbsoma.  It is at this point a trial balloon as we likely need to add other fallback options in case the system library is not present.  But the current (draft) proposal works locally under Linux and should work in CI.
Tracking issues: #1277 , #1293
Changes:
Instruct build for use libfmt header-only, remove all special R tags in cmake build instructions, install to /usr/local (in CI) and call ldconfig in CI.  Then instruct pkg-config to look for both libtiledbsoma and libtiledb.   We have a fallback artifact for the latter, the former can be built from repo sources where but could not be built from a standalone tar.gz not containing the library sources.
Notes for Reviewer:
This is a draft at this point.

So there is apparently some more work to do at the cmake level as it assume there was a shared library of libtiledb to also install along with libtiledbsome:
-- Install configuration: "Release"
-- Installing: /usr/local/lib/libtiledb.so.2.15
CMake Error at src/cmake_install.cmake:46 (file):
  file INSTALL cannot copy file
  "/home/runner/work/TileDB-SOMA/TileDB-SOMA/build/externals/install/lib/libtiledb.so.2.15"
  to "/usr/local/lib/libtiledb.so.2.15": Success.
Call Stack (most recent call first):
  cmake_install.cmake:47 (include)

Codecov Report
Patch coverage has no change and project coverage change: +26.05 :tada:

Comparison is base (845de39) 64.98% compared to head (f6ed99e) 91.04%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1280       +/-   ##
===========================================
+ Coverage   64.98%   91.04%   +26.05%
===========================================
  Files          96       30       -66
  Lines        7689     2389     -5300
===========================================
- Hits         4997     2175     -2822
+ Misses       2692      214     -2478

Flag
Coverage Δ

python
91.04% <ø> (+0.53%)
:arrow_up:

r
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 77 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@eddelbuettel now I'm getting an R CI fail on Ubuntu.
Can you take a look?

#1302 took care of this.

authors: codecov-commenter eddelbuettel johnkerl

repo: single-cell-data/TileDB-SOMA


text:
[c++] Depend on core 2.17.0

Issue and/or context: For #866. The "real" PRs are #1519, #1511, and #1559 which are at the moment depending on core 2.17.0-rc0. On this PR, if CI passes, I'll attempt to first merge solely the core-dependency change -- and then the big three PRs will be able to be rebased on top of this once it's merged.
Changes:

Core 2.17.0
#1676 redux
TileDB-Py 0.23.0
TileDB-R 0.21.0

Notes for Reviewer:

Codecov Report
Patch has no changes to coverable lines.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:loudspeaker: Thoughts on this report? Let us know!.

Conda is for later in our release procedure:
https://github.com/single-cell-data/TileDB-SOMA/wiki/Branches-and-releases
Green CI matters for this PR.
This Python/R interop which has been solidly quietly green for months is now vexing me.

I'll merge this PR as it fixes some issues. But Python/R interop remains to be solved.

I'll debug the interop first thing tomorrow.

authors: codecov-commenter johnkerl


text:
BrowserPlatformLocation not found ( Angular 10 )

Hello, I really appreciate the time and effort you put into this library.
I'm trying to use your library with an existing angular 10 app and i get the following errors:

I fixed them by changing the line 3 in extra-providers.d.ts:

to this:

And by changing the line 3 in single-spa-angular.js

to this:

And it worked.
Can you please update the library to support angular 10

I don't know what is causing that issue and no longer spend time diagnosing and fixing issues for this repository (was too many hours spent on stuff I don't enjoy). There are a few others who jump in to comment on things who may be able to help here. One thing that might help is providing a repository demonstrating the issue.

Steps to reproduce ( with Angular CLI 10.0.5 ):
ng new app
ng add single-spa-angular
fix a small known issue by adding "main.single-spa.ts" to "files": [ .. in tsconfig.app.json
npm run serve:single-spa
and you get the errors.
Here's a repo
https://github.com/thehunckerma/single-spa-angular-issue

I will close this issue as angular 10 is not going to be supported.
for those who are looking for a solution i switched to Qiankun and it works perfectly.

I will close this issue as angular 10 is not going to be supported.

This is untrue - we'd like Angular 10 to be supported. We've had a few PRs improving support for it and hopefully will fully support it over time.

ah okey sorry 😅

Hello @thehunckerma

for those who are looking for a solution i switched to Qiankun and it works perfectly.
By curiosity I had a look at umijs/qiankun project . It is an interesting one that aim to industrialize the code of what we call sometimes a "portal" in Microfrontends (the core frontend that routes to the others) but it is using single-spa-angular actually. It is not a "concurrent".

Moreover it seems to be completely dependent on single-spa-angular regarding the compatibility to Angular 10 or not (which is a good architecture choice !)
Sorry for the intervention, I just wanted to clarify.
About your issue I will check your repo but I managed to make it work with Angular 10 in a personal project so I'm sure it's nothing ;)

Okay looking at the sinlge-spa-angular code https://github.com/single-spa/single-spa-angular/blob/master/src/src/extra-providers.ts
It makes me think that your are not using the latest version. You should try something like
ng add single-spa-angular@4.4.2

@celian-garcia Hello, I know qiankun is based on single spa that s why i wanted to use single spa in the first place but doing exactly what the documentation says didn t make it work
but thank you for the clarifications i will try it in my next project :D

Hey @thehunckerma
Was your issue resolved? Can this issue be closed?

Hello, yeah i guess so thanks :D

authors: arturovt celian-garcia joeldenning thehunckerma

repo: single-spa/single-spa-angular


text:
storage: prioritize L0 flush I/O

Sometimes when we are writing to MinIO, the write throughput is constrained, and we should prioritize some operations. If compute nodes cannot flush tables to object store efficiently, barrier latency will be affected. Therefore, we should have some strategies to prioritize L0 flush I/O, by, for example:

a separate prefix for L0 SSTs in object store
limit compaction throughput based on some metrics
...

"based on some metrics` seems not accurately

The most difficult problem is how to limit the throughput for several compactor nodes....

https://www.usenix.org/system/files/atc19-balmau.pdf
Preempting higher-level compactions seems to be another option.

authors: Little-Wallace lmatz skyzh

repo: singularity-data/risingwave


text:
'Tests\TestCase' not found in laravel-served/tests/Unit/DockerImageTest.php:13

If I run 'composer test' in root directory, I get the following error:
Class 'Tests\TestCase' not found in ~/laravel8/laravel-served/tests/Unit/DockerImageTest.php:13
This appears because the tests are being run outside of a Laravel install.
Switching to running tests within Laravel8 install
with the following
        "sinnbeck/laravel-served": "@dev"

and
 "repositories": [
        {
            "type": "path",
            "url": "~/laravel8/laravel-served",
            "options": {
                "symlink": true
            }
        }
    ]

in composer.json
./vendor/bin/phpunit ../laravel-served/tests/Unit
I get the following:

Tests\Unit\DockerImageTest::it_can_make_apache_docker_file
Error: Call to undefined method Tests\Unit\DockerImageTest::getExpectedContent()

and similar for other tests.
@sinnbeck can you let me know how you've got your package set up locally and what command you run to run your DockerImageTest class?

I didn't spot that you have Laravel installed as a dev dependency. Run composer install. Job done

Yeah I also set up some auto loading and installed the orchestra test package

authors: iammikek sinnbeck

repo: sinnbeck/laravel-served


text:
Missing LICENSE

Very interesting library and lots of work. Sadly not usable, as no LICENSE is given.
Maybe you could improve on that.

Released in public domain. See here.

authors: elgohr siongui

repo: siongui/godom

license: unlicense


text:
Implement GA component

Is your feature request related to a problem? Please describe.
We need to be able to monitor GET requests made to a certain endpoint using Google Analytics.
Describe the solution you'd like
Add optional NodeJs backend component (Docker) with redirect from api-gateway.
Describe alternatives you've considered
Additional context

https://developers.google.com/analytics/devguides/collection/protocol/v1/devguide

There is now an endpoint at ./api/v1/monitor/view that on call raise an event in GA.


text:
[Bulk Messaging] Limited to messaging 10-12 users in a minute ?!

I'm developing a notifier, and I need to message 500 users, for example.
By the Reddit API limitations, it means that I can do it in 9 minutes (60 messages in 1 minute), but after testing it, I concluded that only 10 - 12 users can be messaged in 1 minute, and then an exception is thrown.
Is there any limitations in bulk messaging users, provided by this wrapper ?
Can someone clarify this for me ?
Thanks :)

What exception is thrown when this happens?  I'm not aware of any limitations that would prevent ~60 messages/min from going through, provided that you're not performing any other queries that would go against the API speed limit.

What exception is thrown when this happens?  I'm not aware of any limitations that would prevent ~60 messages/min from going through, provided that you're not performing any other queries that would go against the API speed limit.

Rate Limit Exception is thrown.
I made sure that I'm not performing any queries related to the Reddit API

Rate Limit Exception is thrown.
I made sure that I'm not performing any queries related to the Reddit API

How old is the account you're using?  Does it have a decent amount of both comment and post karma?  Reddit has been known to impose tighter rate limits in certain circumstances, particularly with newer and less-established accounts.


text:
Error marshalling JSON when Panic() is called and JSONFormatter used

I'm using JSONFormatter and calling logrus.Panic("panic msg") which gives me this error:
Failed to obtain reader, Failed to marshal fields to JSON, json: unsupported type: logrus.exitFunc
If I add a Marshaler interface for logrus.extFunc that just returns "", it works, but this involves modifying logrus vendor code. This seems related to  #835 and #642.
I don't really care about handling type-functions as field values in general--I just want to be able to call Panic()--this seems like a bug, but I fully admit I could be missing something and doing something wrong.

Hello @deusnefum, I can't reproduce the issue by just instantiating a logger with a JSONFormatter and calling Panic.
Can you provide a minimum code sample which exhibits the issue ?

I was correct that I was missing something and doing something wrong. We have generic crash-recovery code and part of that code adds the error object as a field--I think my answer is to just remove the offending field.
	if x := recover(); x != nil {
		delete(x, "Loggger")  // "fix" the problem
		// recovering from a panic; x contains whatever was passed to panic()
		logrus.WithFields(logrus.Fields{
			"error": x,
		}).Info("panic occurred")

I'll close the issue, but is there a more generic solution? Anyway to add a JSON marshaller or tell logrus to skip fields it can't marshall?

authors: deusnefum dgsb

repo: sirupsen/logrus


text:
Problem with number of artists

Number of matched artists in version 1.2 of LastFM dataset is 10,180 (number of rows in the file) not 11,180 as listed in the Readme

Thanks Giulio. issue closed

authors: giulioverazzo paolo-tomeo

repo: sisinflab/LODrecsys-datasets


text:
Setting up project (missing codesystem,valueset files)

How can i get codesystem and valueset files for this project.
I read a closed issue reporting that you cannot provide due to licensing issues.
how about using valuesets from.
http://art-decor.org/decor/services/ValueSetIndex

Collect as many as you can from VSAC first. For the ones you can't get from VSAC or aren't provided in the repo/instructions, you have to hand-create them to fit the VSAC format.

repo: siteadmin/reference-ccda-validator