[LOOP-1702] Fix floating point error in max basal rate guardrail

Make sure the upper recommended bound aligns with a supported value
Refactor some floating point utilities out of FractionalQuantityPicker
Make MockPumpManager's various deliverable increments match the actual PumpManagers exactly

@rickpasetto I've added LoopKitUITests and included tests for different floating point values (notable, 0.3, 0.3.nextDown, and 0.3.nextUp) for your review. I'll add LoopKitUITests to the testing scheme in an upcoming LW PR (looks like a couple are queued up at the moment).

@mpangburn actually, I just grabbed this branch (updated it to dev), put a breakpoint in your test, and ran Product -> Test for the Tidepool Loop scheme, and the breakpoint wasn't hit.  I think it also needs to be added to the Tidepool Loop scheme.

@mpangburn Aha!  I think I figured out why. Minimum deployment target of LoopKitUITests was set to iOS 13.6.
If I change
IPHONEOS_DEPLOYMENT_TARGET = 13.6;

to
IPHONEOS_DEPLOYMENT_TARGET = 13.0;

in LoopKit.xcodeproj/project.pbxproj the test runs.  Maybe consider making this change?
If you’re making changes, a few really minor nits:

it might be a good idea to add the test to the Shared LoopKit scheme.  That way folks running the LoopKit tests only would run LoopKitUITests
It would be nice if the LoopKitUITests target came after LoopKitUI in the Targets list

@rickpasetto Done, thanks! Can you take one more look?

(I'm not really sure if it makes sense to move those guardrail functions into LoopKit...but I don't feel strongly...)

I think it makes more sense than LoopKitUI; which values are considered in range for a setting is not a concept intrinsically tied to UI.

By the way, did you want to remove the old GuardrailTests.swift file? It is still there, apparently, in LoopKitUITests..

Ah, yes, thanks. I deleted the target but missed the folder.


Extension adds significant execution overhead

This is on AWS Lambda, Python 3.9, with extension version 83 on ARM64.

According to the extension's description, it's supposed to ship logs asynchronously, but it appears to be doing so synchronously. Setting DD_SERVERLESS_FLUSH_STRATEGY=periodically,60000 only defers the latency -- most executions go down to around 50ms, but every minute or so, they spike to ~1000ms, which is not ideal.
Extension Enabled

Extension Disabled, with DD_FLUSH_TO_LOG=true

As you can see, the difference is an order of magnitude.
Is this expected/normal?

Hi @mrworkman - thanks for reaching out!
When using Lambda extensions, work performed by extensions after the lambda function handler exits and returns to the caller is still counted in the duration metric.
So what you're seeing both here and with DD_SERVERLESS_FLUSH_STRATEGY=periodically,60000 is expected, but happening after the function execution completes and not impacting the response time of your Lambda function.
This is why we provide the runtime_duration enhanced metric, although overall we always recommend tracking end to end latency (using something like RUM or api gateway quantile metrics, for example)
We document this in full here and there's a nice AWS blog post which also covers this.
I hope I've helped!

So, to add more to this, we discovered this issue where we have one lambda synchronously invoking others. So, the first one has to wait for the others to finish before it can proceed.
You can see an example in the following trace. One lambda invokes the other, and has to wait 776ms for the request to complete.

When removing the DD Extension, this drops considerably (in this case, to 94ms), and the overall request takes that much less time.

Okay great, thanks for the context  – that's super helpful!
I'm at re:Invent this week but have asked someone else to investigate this, although it may be fastest to open a bug ticket with us; you can email<EMAIL_ADDRESS>to kick that off. Feel free to include this report in the bug ticket.
Thanks!

For reference, the support ticket is 1453840.

Hey folks!
I just wanted to follow up in public quickly, as we're still figuring things out via support.
I'm having trouble reproducing this with the same parameters (py3.9-arm, 1024mb RAM, deployed to eu-west-1, pushing telemetry to datadog in US1, datadog extension v51 and datadog-lambda-python v85).
I've created two functions, both are identical and create some logs, and a custom metric. One function utilizes the default flush strategy, the other configures DD_SERVERLESS_FLUSH_STRATEGY=periodically,60000, as per this thread.
Then I created an invoker function which synchronously invokes each function (using API Gateway), and measures the end to end response time. To make this easier to reproduce, I both create these as a custom metric and emit them as structured logs.
Regardless of the flush strategy, the max RTT latency is about 50-60ms during a warm start:

Function code:
import json
import requests
import os
import time

from datadog_lambda.metric import lambda_metric

def hello(event, context):
    body = {
        "message": "function executed successfully!",
    }
    print('Creating a custom metric!')
    lambda_metric(
        "rtt_latency.custom_metric_created",
        1,
        tags=[]
    )
    print('we did it, time to return')
    response = {
        "statusCode": 200,
        "body": json.dumps(body)
    }

    return response

def invoke(event, context):
    base_url = os.getenv('BASE_URL')
    baseline_start = time.time_ns()
    requests.get(f'{base_url}/baseline')
    baseline_end = time.time_ns()
    requests.get(f'{base_url}/periodic')
    periodic_end = time.time_ns()

    baseline_duration = (baseline_end - baseline_start) // 1_000_000
    periodic_duration = (periodic_end - baseline_end) // 1_000_000
    print(f'{{baseline_duration: {baseline_duration}}}')
    print(f'{{periodic_duration: {periodic_duration}}}')
    lambda_metric(
        "rtt_latency.baseline",
        baseline_duration,
        tags=[]
    )
    lambda_metric(
        "rtt_latency.periodic",
        periodic_duration,
        tags=[]
    )

serverless.yml
service: py39-cross-region-demo
frameworkVersion: '3'

provider:
 name: aws
 runtime: python3.9
 architecture: arm64
 region: eu-west-1

custom:
 datadog:
   apiKey: ${env:DD_API_KEY}

functions:
 baselineDefaultFlush:
   handler: handler.hello
   events:
     - httpApi:
         method: get
         path: /baseline
 periodicFlush:
   handler: handler.hello
   environment:
     DD_SERVERLESS_FLUSH_STRATEGY: periodically,60000
   events:
     - httpApi:
         method: get
         path: /periodic
 invoker:
   handler: handler.invoke
   environment:
     BASE_URL: !GetAtt HttpApi.ApiEndpoint
   events:
     - schedule: rate(1 minute)

plugins:
 - serverless-plugin-datadog

My hypotheses are limited as I'd expect any tracing-based latency to also exist with the datadog lambda forwarder configuration (which you've mentioned seems to remediate this issue entirely).
I'm looking forward to our discussion later this week,
Thanks!

Hi @astuyve.
Thanks for catching up with me the other day!
In case you look here first, I sent you (and support) a basic project that reproduces—at least, for me—the issue, so please check your email 🙂

Hey everyone!
After an excellent bug report, reproduction case, and assistance provided by @mrworkman, we narrowed the cause down to setting the X-Amz-Log-Type to Tail as part of a direct lambda invocation request. This forces the Lambda runtime to wait for all logs to be written & flushed before returning the response. The docs for this are here.
A snippet of the change & results can be found here: https://github.com/mrworkman/invoker/pull/1
As always, please reach out with any questions.
Thanks!

For anyone landing here in search of more info, I ran into this specific issue when using the "test" functionality in the aws lambda console to invoke a function. Inspecting the service request in the web inspector, it does indeed have the header x-amz-log-type: Tail set.

@jedschneider Yup! The Lambda console does not stream logs, but rather waits for them to display them entirely in the console, so it triggers this behavior.


Error in qs.utils.consecutive_wins()

I tried the above function and got an error.

Fixed in 0.0.34


Cloud costs optimization solutions

This article is talking about Microsoft tools and Azure services that may help to optimize the costs.
Some options within the Cost Modeling article could also be applied here such as Low priority VM, Azure Spot Instance and either the newly Azure Dedicated Host could be leveraged in a cost optimization scenario.
Among other things I was also thinking about:

scaling (up/down, in/out)
auto-shutdown / auto-start
Engineering on SLA (for instance single instance VM on Premium storage vs multiple instance in Availability Set, geo replication when not needed...)
Right-sizing (from ops monitoring)
Right-region (Data management article)
What's your thought ?

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

ID: 54dd2146-8293-bb39-e90a-f83b5207c000
Version Independent ID: fdb76897-f950-4a7b-db74-35265c6ecfda
Content: Optimizing Cloud Costs
Content Source: docs/framework/cost/optimizing.md
Service: architecture-center
Sub-service: cloud-design-principles
GitHub Login: @david-stanford
Microsoft Alias: pnp

the Cost pillar content is being re written and your feedback is being incorporated.


binance.exceptions.BinanceAPIException: APIError(code=-1021)

When I using market order API, it will return the
raise BinanceAPIException(response)
binance.exceptions.BinanceAPIException: APIError(code=-1021): Timestamp for this request is outside of the recvWindow.
The related source code is:
order = client.order_market_sell(symbol=symbol, quantity=unit, recvWindow=10000)
But when I change the code by changing recvWindow=10000 to recvWindow=100000, the problem solved.
So, what's the main meaning of recvWindow? I set to 100000 means the network delay could be 100 seconds. but I think 10 seconds is enough for an average delay of most of the internet environment.

@arthurlirui make sure your local time is synced with an internet time server to avoid the Timestamp error. I wouldn't recommend changing the recvWindow to fix an issue caused by your time being out of sync.
You can read more about the recvWindow parameter at the official docs https://github.com/binance-exchange/binance-official-api-docs/blob/master/rest-api.md#timing-security


Summoner Toasts

Related Issues

#59

Closed by https://github.com/HausDAO/daohaus-monorepo/pull/306


下载新的版本，编译报错---mac下

crypto/saf/saf_ec.c:477:23: error: expected ';' after return statement
return SAR_IndataErr);
^
;
crypto/saf/saf_ec.c:479:31: error: use of undeclared identifier 'uiCertificate';
did you mean 'pucCertificate'?
if (uiCertificateLen <= 0 || uiCertificate > INT_MAX) {
^~~~~~~~~~~~~
pucCertificate
crypto/saf/saf_ec.c:461:17: note: 'pucCertificate' declared here
unsigned char *pucCertificate,
^
crypto/saf/saf_ec.c:479:45: warning: ordered comparison between pointer and
integer ('unsigned char *' and 'int')
if (uiCertificateLen <= 0 || uiCertificate > INT_MAX) {
~~~~~~~~~~~~~ ^ ~~~~~~~
crypto/saf/saf_ec.c:550:23: error: expected ';' after return statement
return SAR_IndataErr);
^
;
crypto/saf/saf_ec.c:552:31: error: use of undeclared identifier 'uiCertificate';
did you mean 'pucCertificate'?
if (uiCertificateLen <= 0 || uiCertificate > INT_MAX) {
^~~~~~~~~~~~~
pucCertificate
crypto/saf/saf_ec.c:461:17: note: 'pucCertificate' declared here
unsigned char *pucCertificate,
^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
5 warnings and 20 errors generated.
make[1]: *** [crypto/saf/saf_ec.o] Error 1
make: *** [all] Error 2
bash-3.2$

在linux下 也编译出错：
[root@MAASG GmSSL-master]# ./config --prefix=/usr/local
Operating system: i686-whatever-linux2
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LANG = "zh_CN:GB18030"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LANG = "zh_CN:GB18030"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
Perl v5.10.0 required--this is only v5.8.5, stopped at ./Configure line 12.
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LANG = "zh_CN:GB18030"
are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
Perl v5.10.0 required--this is only v5.8.5, stopped at ./Configure line 12.
This system (linux-x86) is not supported. See file INSTALL for details.

./config no-saf no-sdf no-skf no-sof no-zuc

@guanzhi 在windows下面编译最新的代码也出错：
环境：64位win10 version: 1607，msys2更新到最新，gcc version: 6.3.0,
错误：
D:/Development/msys2/mingw32/i686-w64-mingw32/include/minwindef.h:24:29: error: conflicting types for 'ULONG'
typedef unsigned __LONG32 ULONG;
^~~~~
In file included from include/openssl/gmapi.h:63:0,
from crypto/gmapi/gmapi_skf_ec.c:52:
include/openssl/skf.h:93:18: note: previous declaration of 'ULONG' was here
typedef UINT32   ULONG;
^~~~~
In file included from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windef.h:8:0,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windows.h:69,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/winsock2.h:23,
from ./e_os.h:190,
from crypto/gmapi/../ec/ec_lcl.h:33,
from crypto/gmapi/gmapi_skf_ec.c:54:
D:/Development/msys2/mingw32/i686-w64-mingw32/include/minwindef.h:131:15: error: conflicting types for 'BOOL'
typedef int BOOL;
^~~~
In file included from include/openssl/gmapi.h:64:0,
from crypto/gmapi/gmapi_skf_ec.c:52:
include/openssl/sof.h:56:14: note: previous declaration of 'BOOL' was here
typedef long BOOL;
^~~~
In file included from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windef.h:8:0,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windows.h:69,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/winsock2.h:23,
from ./e_os.h:190,
from crypto/gmapi/../ec/ec_lcl.h:33,
from crypto/gmapi/gmapi_skf_ec.c:54:
D:/Development/msys2/mingw32/i686-w64-mingw32/include/minwindef.h:141:29: error: conflicting types for 'DWORD'
typedef unsigned __LONG32 DWORD;
^~~~~
In file included from include/openssl/gmapi.h:63:0,
from crypto/gmapi/gmapi_skf_ec.c:52:
include/openssl/skf.h:96:18: note: previous declaration of 'DWORD' was here
typedef UINT32   DWORD;
^~~~~
In file included from D:/Development/msys2/mingw32/i686-w64-mingw32/include/minwindef.h:163:0,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windef.h:8,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windows.h:69,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/winsock2.h:23,
from ./e_os.h:190,
from crypto/gmapi/../ec/ec_lcl.h:33,
from crypto/gmapi/gmapi_skf_ec.c:54:
D:/Development/msys2/mingw32/i686-w64-mingw32/include/winnt.h:278:16: error: conflicting types for 'CHAR'
typedef char CHAR;
^~~~
In file included from include/openssl/gmapi.h:63:0,
from crypto/gmapi/gmapi_skf_ec.c:52:
include/openssl/skf.h:89:17: note: previous declaration of 'CHAR' was here
typedef UINT8   CHAR;
^~~~
In file included from D:/Development/msys2/mingw32/i686-w64-mingw32/include/minwindef.h:163:0,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windef.h:8,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windows.h:69,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/winsock2.h:23,
from ./e_os.h:190,
from crypto/gmapi/../ec/ec_lcl.h:33,
from crypto/gmapi/gmapi_skf_ec.c:54:
D:/Development/msys2/mingw32/i686-w64-mingw32/include/winnt.h:280:20: error: conflicting types for 'LONG'
typedef __LONG32 LONG;
^~~~
In file included from include/openssl/gmapi.h:63:0,
from crypto/gmapi/gmapi_skf_ec.c:52:
include/openssl/skf.h:92:17: note: previous declaration of 'LONG' was here
typedef INT32   LONG;
^~~~
In file included from D:/Development/msys2/mingw32/i686-w64-mingw32/include/minwindef.h:163:0,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windef.h:8,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/windows.h:69,
from D:/Development/msys2/mingw32/i686-w64-mingw32/include/winsock2.h:23,
from ./e_os.h:190,
from crypto/gmapi/../ec/ec_lcl.h:33,
from crypto/gmapi/gmapi_skf_ec.c:54:
D:/Development/msys2/mingw32/i686-w64-mingw32/include/winnt.h:329:24: error: conflicting types for 'LPSTR'
typedef CHAR *NPSTR,*LPSTR,*PSTR;
^~~~~
In file included from include/openssl/gmapi.h:63:0,
from crypto/gmapi/gmapi_skf_ec.c:52:
include/openssl/skf.h:98:18: note: previous declaration of 'LPSTR' was here
typedef CHAR *   LPSTR;
^~~~~
make[1]: *** [Makefile:4012: crypto/gmapi/gmapi_skf_ec.o] Error 1
make[1]: Leaving directory '/d/tobia/ClionProjects/GmSSL'
make: *** [Makefile:128：all] 错误 2


Clean up old examples in Example Provider

There are a lot of examples in the Example Provider that feel a little stale now we have more advanced kits. We should probably consolidate our newer examples and such so that they feel geared towards particular use-cases.

Thining we should migrate packages/boards to providing the examples for the Examples Provider in the Visual Editor.

still WIP https://exadev.github.io/projects/Breadboard/Boards

Fixed by virtue of making a Playground / Examples split


Error in bnxSort Step

On attempting to run runBNG, I get an error where bnxSort fails to complete properly. Log information does not provide much that would help me get to the root of this problem. Tools and scripts are both installed and have executable permissions set for all files, and example files seem to work as intended. It seems I can get it to run if I reduce the genome size substantially, however a 5GBase genome is what I am expecting. The command I am attempting to run is:
./runBNG denovo -t ~/tools -s ~/scripts -b ~/Storage/BioNano/Assembly/Datasets/Q/Detect\ Molecules/Molecules.bnx -T 1400 -j 28 -z 5000 -o ~/Storage/BioNano/Output
The complete output I get is as follows:
========================================== De novo assembly starts =============================================
## Start date: Thu Jul 13 13:15:22 EDT 2017
## The bnx file is: /fs/project/PAS1172/BioNano/Assembly/Datasets/Q/Detect Molecules/Molecules.bnx
## The digested reference is:
## The minimum molecule length is (Kb): 150
## The minimum label on a molecule is:
## The path of BNG scripts folder is: /users/PAS1172/osu0330/scripts
## The path of BNG tools folder is: /users/PAS1172/osu0330/tools
## The number of threads is: 1400
## The number of jobs is: 28
## The number of iterations is: 5
## False Positive Density (/100Kb) [FP]: 1.5
## False Negative Rate (%/100) [FN]: 0.15
## ScalingSD (Kb^1/2) [sd]: 0.0
## SiteSD (Kb) [sf]: 0.2
## RelativeSD [sr]: 0.03
## The genome size (Mb) is: 5000
## The output directory is: /users/PAS1172/osu0330/Output

  Prerun Tests:
	0 ERRORS
	2 WARNINGS

  WARNING: Disabling SV detection (-V, default ON) because no reference supplied, or invalid.
  WARNING: Disabling alignment of molecules vs reference (-m, default ON) because no reference supplied, or invalid.

  Pipeline Version: $Id: SVModule.py 5134 2016-07-07 00:51:33Z wandrews $

  RefAligner Version: 5122

  Pipeline start time: Thu Jul 13 13:15:27 2017

Reading molecule stats from /users/PAS1172/osu0330/Output/all.bnx:
Molecule Stats:
N mols: 1087029
Total len (Mb): 237551.802
Avg len (kb)  :    218.533
Mol N50 (kb)  :    214.465
Lab (/100kb)  :      6.815

Sorting /users/PAS1172/osu0330/Output/all.bnx into /users/PAS1172/osu0330/Output/all_sorted
 Starting Multi-Threaded Process:
  SortBNX
  Running 1 jobs with 1 threads
   START   1:                        SortBNX,  1Thr,   1R,   1T,   0F,   0Q
   STOP    1:                        SortBNX,  1Thr,   0R,   1T,   1F,   0Q   0h 0.00m
 Finished Multi-Threaded Process:
  SortBNX

ERROR: sortBNX failed. Check: /users/PAS1172/osu0330/Output/all.bnx

  Pipeline end time: Thu Jul 13 13:15:32 2017
  Elapsed time: 0.08m; 0.00h; 0.00d

Warning/Error messages:
warning : Could not open file \"/users/PAS1172/osu0330/Output/all_sorted.stdout\" tries=0 exception=[Errno 2] No such file or directory: '/users/PAS1172/osu0330/Output/all_sorted.stdout'\n
error : job has not completed, see stdout=\"/users/PAS1172/osu0330/Output/all_sorted.stdout\"
critical : ERROR: sortBNX failed. Check: /users/PAS1172/osu0330/Output/all.bnx

Warning/Error summary:
	1 warning(s)
	1 critical(s)
	1 error(s)

Pipeline has completed with errors

## End date: Thu Jul 13 13:15:32 EDT 2017
===============================================================================================================`

Hi @GWheelerEB,
Thank you for the issue you raised. I found that I missed a "$" in Line 1254. The "site" should be "$site". You may change the code in your 'runBNG' or download the fixed version. Let me know if it works.
Cheers,
Andy

Thanks a lot! It now seems to work as intended even when setting large expected genome size values.


Update index.d.ts

Typings hasn't been updated after
https://github.com/adaltas/node-csv-parse/commit/37027d285f492b7df01dfb86e9a6448531cc4ecb

good improvements

version 4.9.1 embed the correction, thank you for taking the time

Improvement that really need!


AB23420 Deploy "feat.paginate"-branch to test-environment

@elwinschmitz We want to keep deploying the paginate branch to test-vm so diderik/tijs can test on it. Are these the required changes to do that?

Yes, but we think we will be testing on the paginate branch more often than the master branch in the coming month. Therefore the switch makes sense. If we want to test often on both we may think about spinning up an extra slot + set of interfaces. But let's only go through this hassle if we find this approach to annoying. @elwinschmitz


Maintainers not shown in package details

In the package details dialog the "Maintainers" always seem to be empty:

Versions
Chocolatey: v0.10.3 Professional
ChocolateyGUI: v0.14.0-unstable0145

Weird. Looks like we're not getting maintainers back from the API.

Just been looking at this...
Looks like this call:

Is what is not populating the Owners/Maintainers property.  This is going down into Chocolatey.Lib.  Need to debug further.

I think I made an issue on the Chococlatey side of the fence back when I commented.

@RichiCoder1 this one? https://github.com/chocolatey/choco/issues/777

I believe that's it. There was other bits of info I wasn't getting.


ConcurrentModificationException

I'm using Lettuce 4.2.1-Final for some days without any issue, but today throw an exception
com.lambdaworks.redis.RedisException: java.util.ConcurrentModificationException
Sorry for no logs, I will try to reproduce the error.

Can't proceed without a stack trace.

Any update on this ticket?

No. The issue doesn't appear again. So, you can close this issue.


Include whitespace between semicolon and encoding

Updates the text/plain media type string to use a space between the semicolon and the encoding. While this is not strictly required, it more closely aligns with the media types in the content format registry.
https://www.iana.org/assignments/core-parameters/core-parameters.xhtml

Thx for contribution :)


Fixes #375

Makes cdxgen attempt a pip freeze for all virtual environments
@anthonyharrison, could you kindly test with this branch?

Hello @prabhu
Sorry for delay in testing.
I cloned the branch and created a docker image (cdxgen-cdxgen). It started the server but I wasn't getting any SBOMs generated, so I restarted the Docker image with the following command (I was within the directory of my Python project)
docker run --rm -v /tmp:/tmp -v $(pwd):/app:rw cdxgen-cdxgen -t python -o /tmp/cdxgen0307.json .
I tried this in the same repo (within and outside a Python Virtual environment)
I get an SBOM! but don't understand the components which are being reported. For instance, If I do pip list in my virtual environment I can see I have idna version 3.4 installed; however the SBOM is reporting 2.8 which isn't installed!
I don't see any of the modules included in the requirements.txt file either.

@anthonyharrison, apologies for not sharing the instructions to test.
For docker invocations, the last argument is /app instead of dot. The current directory is mounted as /app when you pass $(pwd):/app:rw so it needs to be referred as /app.
Alternatively, you can clone the repo, switch the branch, and then do
npm install
node bin/cdxgen.js -t python -o ...

Hi @prabhu
Ran the following command
`docker run --rm -v /tmp:/tmp -v $(pwd):/app:rw cdxgen-cdxgen -t python -o /tmp/cdxgen0307.json /app'
from within a Python virtual environment and outside.
The SBOMs are identical.
I get multiple versions of the module but get the direct dependencies (from the requirements.txt file) but none of the trasnsitive dependencies.
There are no dependencies included.
Looking at the generated SBOM, it is not clear why a GitHUB URL in the external reference is sometimes referred to as 'website' and in other cases it is 'vcs'.
`

Thanks, @anthonyharrison, for sharing the result. Does the result look different when run without docker directly via node.js? I will check about the github url.
Please note that the dependencies section is currently available only for a handful of languages. https://github.com/CycloneDX/cdxgen#dependency-tree

The docker image doesn't include the python3-virtualenv package. Will work on a fix for this.

@prabhu I did some testing using
TheAlgorithms\Python. Cdxgen is missing some transitive components and has other components that I don't know where they've come from. Attached are three lists - 1) packages reported by cdxgen but not in pip list 2) packages from pip list 3) packages common to both lists. A brief check of the packages that appear to be missing from cdxgen suggests they are transitive dependencies or dependencies of the transitive dependencies. For example, tensorflow is a direct dependency and it requires tensorflow-intel, which in turn requires absl-py - neither are listed by cdxgen.
pip but not cdxgen
cdxgen but not pip
both

Thanks, @cerrussell will take a look at this. I have created a separate ticket, #380, with instructions and suggested scenarios. Few more results like this might help with the fix.
https://github.com/CycloneDX/cdxgen/issues/380
cc: @anthonyharrison

@cerrussell, let's meet online later since I see different behavior for this repo where cdxgen results are better than pip. We have to confirm that the setup is as per #380


Feature Request: Js and Css document generator

I originally posted this in https://github.com/zewa666/generator-aurelia/issues/6, but per comments it has been moved to skeleton
Js doc generator:
I was thinking about using https://github.com/tj/dox, it just generates a json file based on jsdoc without generating an html app. Then we could make an aurelia app/template for displaying the data. It could be a simple display or a fancy app like YuiDoc, this way we can include a default template or the user can build there own based on the json file.
Css Doc Generator:
if you look at either the kss example http://primercss.io/scaffolding/ or the
hologram example http://trulia.github.io/hologram-example/base_css.html
It generates an html app that has all the components, buttons ect used in your project with the related css classes
It makes it easier for a consumer of your app to understand how your css is used, ie what class do I need to change to change the style for all primary buttons. Y can provide documentation in your css or sass describing what each main class is used for. This makes it easy for you to identify what class you need to change the background color or to change all main buttons. In addition the html generated style guide gives you a demo of what it currently looks like, so you can identify that element in your working app.
In aurelia's case when you create a new app every time you build it would create a doc folder with a generated css style guide. The guide would be configurable to include/exclude files/folders. I personally like the idea of just generating a data file in json like https://github.com/tj/dox does for javascript then have a default template to provide a view, that way if the user doesn't like the look and feel of the generated html they can just create their own view and bind it to the data.
I am still new to using style guides myself, but so far I like it, it really helps someone understand your app, instead of having to open the css file and wade through it. One thing about style guides is that you don't include everything in the styleguide comments, just the stuff that you want to expose to a consumer allowing them to skin your app, think of it as documentation describing how create a custom skin for your app(sort of).
If you need more info or more use case examples let me know.
Thanks,
Dan
Also I understand that the cli may be including a doc generator, this is just food for thought.

Think it's not so easy you describe. Some of us use LESS, SASS/SCSS or PostCSS.
PrimerCSS is just for SASS/SCSS. PrimerCSS has multiple issues by design, like grouping content on it file structure, what is not good solution. Namespaces would be better.
Cause of up coming PostCSS, LESS and SASS become minor relevant. To document CSS there actually do not exist an unique solution.

We have a new docs app releasing with the beta. We are generating api docs from our source for that. We don't really have any css in the core of Aurelia. We will consider various documentation options for that as we launch projects containing css.


Fix bugs when scaling controls

Changes:

Fix GroupBox reported control size being stuck at 20
Fix WindowBox reported control size being stuck at 24
WindowBox selection now has a minimum size of 48, as the control will not go shorter causing a mismatch between the desired height and actual height

@TornadoCookie Thanks for the review!


"Generate type" context menu option does nothing

Selecting the "Generate type" option from this context menu does not do anything.

Just keep in mind that the context menu shown in this issue is completely different from what I see with the configuration specified at #1119. The "Generate type" command is not present in the context menu (or I don't know how to access the context menu shown above).

It appears that this is due to the fact that Roslyn code actions can now be nested. In VS, "Generate type" is the parent of a submenu of several code action options.


fix(transaction): use legacy tx for cancel operations

Checklist

[X] I have read the coding guide
[ ] My change requires a documentation update and I have done it
[X] I have added tests to cover my changes.

Description

Use legacy tx instead of access control tx which does not work with the configured signer.

This change is

@ralph-pichler We plan to tag an RC today. Can this be merged safely?

@aloknerurkar yes


created configuration-driven proxy filter, which uses a handlebar-lik…

created configuration-driven proxy filter, which uses a handlebar-like approach to wiring up configuration values - like routes and endpoints - based on values coming out of configuration.
resolves #2165

Also, config filters have a weakness where they're not able to respond to configuration change notifications. We'll want to solve that before building something like this on them.

I agree on the problem of config filters adapting to changed, but I think the primary source for these values is environment variables that should be considered immutable during a process/container's lifetime.

I don't think we want to do it this way. MapForwarder seems like a better fit. #2165 (comment)
Also, config filters have a weakness where they're not able to respond to configuration change notifications. We'll want to solve that before building something like this on them.

Nope. We want full goodness of YARP which you don't get with the forwarder. This needs name substitution to be able to deal with running locally and then in the cloud - and wanting to map service names for each of them based on environment.

ping to @Tratcher and @samsp-msft - all changes you both suggested have been made. Let me know if there's anything additional needed here, and appreciate the reviews!

Triage: Looks like plans have changed, closing.


Improving geo-search: bounding box

Hey ! I was wondering if this discussion is still up to date https://github.com/meilisearch/product/discussions/431 ?
If so, may I implement this feature ? 🙂

Hello @mlemesle! Thanks a lot for your involvement, this is really appreciated! 😄 🎉
I asked our PM directly in the discussion! If we want to add it, I will open a dedicated issue with the API and behavior expectations! Indeed we need to do some works on the product side before starting the implementation (e.g. how the API should look like)
Thanks again for working with us!! ❤
Closing this in favor of https://github.com/meilisearch/product/discussions/431#discussioncomment-3586320

Thanks for allowing me to contribute!
I'll try to follow the discussion and catch the issue when it'll be prioritized then!
Thanks again!


Can't use constants as values

It seems values are limited to literal values, even though constant are also static values
Example
The following code
import std/strutils
import cliche

const val = 0

@["--a=1"].getOpt (a: val)

echo a

results in Error: undeclared identifier: 'a'

Constants as default values:
const x = 0
template z(): auto = x
@["--a=1"].getOpt (a: z())
echo a

This is something in Nim macro system itself, but just using a template it works Ok,
just make a template that returns the value and it will work.
🙂

I do not think this bug is fixable by any other means than this workaround.


Added 1 node of 1 user

Added:

@jdiem:matrix.org 1 node of this user

@ironoa Element handle of this user was verified


Enforce const evaluation in path! macro

By using a temporary const value, we can enforce that the path! macro always causes a compiler error for illegal values even if it is not used in a const context.  This makes it easier to write correct code.

Right, but I would prefer to not break compatibility with older Rust versions for this.  Or is there a benefit to the const block other than better readability?

Certainly not worth using in this case where the current solution works well. The only benefit would be the ability to use it in match statements but I don't think is ever useful.


Can't set keymaps on windows.

[x] I am on the latest Hyper.app version
[ ] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: windows10 family edition latest
Hyper.app version: 1.47stable
Link of a Gist with the contents of your .hyper.js:
Relevant information from devtools (CMD+ALT+I on Mac OS, CTRL+SHIFT+I elsewhere):
The issue is reproducible in vanilla Hyper.app:

Issue

I've uncommented the example, try to use ctrl+o to open devtools, but it's not working.
Configuration is like this:
  keymaps: {
    // Example
    'window:devtools': 'ctrl+o',
    // 'split-vertical': 'Ctrl+Shift+L'
  }

Can someone show me the right way to do this? Thanks!

I encounter the same issue. Additionally, the following configuration gives strange behaviour
...
  keymaps: {
    "tab:next": "ctrl+shift+p",
    "tab:prev": "ctrl+shift+o"
  }
...

I cannot switch tabs using ctrl+shift+p, ctrl+shift+o, OR the defaults of ctrl+tab and ctrl+shift+tab


[Laravel 9] Call to undefined function [...] array_get() / array_set()

Describe the bug
Hello! Thank you for the useful package. I tried to use it on Laravel 9.8.1, but I hit some problems. Neither Setting::get(key) nor Setting::set(key, value) seem to work when I have a multi-level array.
To Reproduce
Steps to reproduce the behavior:

Install the package (composer require ...)
Publish the configuration (php artisan vendor:publish ...)
Try to get/set any Setting value
Get error "Call to undefined function [...] array_get" or "Call to undefined function [...] array_set"

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
array_get / array_set helper functions have been deprecated since (I think) Laravel 6, so they aren't available any more. This is fixed doing the following:
In vendor\janiskelemen\laravel-setting\src\Setting.php:

Import Illuminate\Support\Arr:
use Illuminate\Support\Arr
Replace array_get in line 336 by:
$setting = Arr::get($setting, $subkey);
Replace array_set in line 351 by:
Arr::set($setting, $subkey, $new_value);

I don't do a PR as probably this would break support for older Laravel versions.

Other deprecated functions: array_forget, array_key_exists...

Hi @Petemir, thanks for pointing this out. I haven't been able to fully test it on Laravel 9.8.1, but I would really appreciate a PR. I will then tag it with a new major version with the info that older versions of Laravel are not supported by that release.

@Petemir Fixed with version 2.0.0

Sorry, got caught with easter break and correcting midterms. Thanks for the fix!


Add missing keyword entry to desktop file

Hello,
I have added a patch for the missing keyword entry in the gnome-pie.desktop file:
Description: Add missing Keyword entry to desktop file
Author: Jörg Frings-Fürst
Forwarded:
Last-Update: 2015-01-15
This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
Index: trunk/resources/gnome-pie.desktop
--- trunk.orig/resources/gnome-pie.desktop
+++ trunk/resources/gnome-pie.desktop
@@ -5,3 +5,4 @@ Comment=Cakes don't lie.
Exec=gnome-pie
Icon=gnome-pie
Categories=GNOME;Utility;
+Keywords=Utility;Menu;

Thank you for the patch! Committed in 2b20190ea01850cf24c45fc76450a49d12621acd.


Disable webjob schedules for staging environment

I am looking for a way to disable webjobs schedules for a particular environment. For example, in staging, I only want to manually trigger each webjob after I deploy. I want to disable the schedules, but in Production slot, I want the schedules turned on.
I use settings.job to specify the schedule for a manual triggered webjob. The only solution that I can think of is having two settings.job files and rename/transform them during build for staging and production environments. This will involve some scripting during the build process. I'm hoping there is something built in already that I didn't know about so I don't have to implement a workaround.
Ideally, what I would love to see is a setting that I can set such as WEBJOBS_DISABLE_SCHEDULE=1 to turn off the cron and have that sticky to Staging slot.

Fixed per #9a9cc16bb3adfd223535242d9b118ecb2da5f9f5. Deployment has started and is expected to complete by end of week.


update the placement for -A and remove regular exp

Update the placement for -A with following error

[INFO] Cert manager not requested in scope, deploying...
operandrequest.operator.ibm.com/ibm-cert-manager-operator created
[INFO] Waiting for deployment cert-manager-operator to be running ...
flags cannot be placed before plugin name: -A
flags cannot be placed before plugin name: -A
[INFO] RETRYING: Waiting for deployment cert-manager-operator to be running ... (20 left)

Remove regular expression on grep ^"cert-manager-operator"
If we are using grep ^${name}, it will explicitly look for deployment named cert-manager-operator.

➜  ✗ name=cert-manager-operator
➜  ✗ oc get deploy -A --no-headers --ignore-not-found | egrep '1/1' | grep ${name}
cs-control                                         ibm-cert-manager-operator                                   1/1   1     1     28m
ibm-cert-manager                                   ibm-cert-manager-operator                                   1/1   1     1     38d
➜  ✗ oc get deploy -A --no-headers --ignore-not-found | egrep '1/1' | grep ^${name}

/lgtm


Extend placeholder with a variable and hyphen break sass-lint

What version of Sass Lint are you using?
1.10.2
Please include any relevant parts of your configuration
Ubuntu 16.10
What did you do? Please include the actual source code causing the issue.
@extend %#{$foo}--modifier;
Put hyphen after the variable seens to break sass-lint
What actually happened? Please include any error messages given to you by Sass Lint.
error  Please check validity of the block starting from line #136  Fatal
If you're using a IDE plugin have you tried the CLI too?
yes

This is a problem with the AST we use. It looks to be fixed in the latest version of Gonzales-pe but we haven't had the time to update yet, we believe there are also a few regressions we need to check for.

This is fixed in the newly released version v1.11.0. Please let me know if you still have issues. Thanks!


Multi tenancy not supported?

Describe the bug
When adding the bank id provider multiple times following exception is thrown:
InvalidOperationException: The HttpClient factory already has a registered client with the type 'ActiveLogin.Authentication.BankId.Api.IBankIdApiClient'. Client types must be unique. Consider using inheritance to create multiple unique types with the same API surface.

What area is it related to
BankID
To Reproduce
Steps to reproduce the behavior:
Create a new empty NET Core application, install the ActiveLogin-package and call AddBankId twice.
Expected behavior
When calling AddBankId a isolated authentication provider is created for given schema. So we can add multiple customers with different certificates.
Screenshots
NuGet package version
ActiveLogin.Authentication.BankId.AspNetCore 2.0.1 (same behavior i 3.0.0)
Runtime version
NET Core 3.0 (same behavior i 3.1)
Desktop (please complete the following information):

Windows
Chrome

Additional context
Is multi tenancy something you had in mind when creating this package or was it never meant to support multiple customers with different certificates in the same application?

Hi Anton! Glad you are using Active Login :)
Multi tenancy is in fact supported, but using the technique you describe above. We have customers which we have helped to configure a multi tenancy setup.
As you say, the current architecture does not allow for multiple instances of .AddBankId(), we currently only register one BankIdClient in the IoC container and it's that client that will be used.
The solution we have for multi tenancy is the ability to, per request, select the appropriate client certificate to use for authenticating against the BankId API. This can be achieved using the .(UseClientCertificateResolver) extension. Using that, you'll be given all client certificates registered and can choose the correct one for the current tenant.
All other options should be configurable on a per schema basis (.AddSameDevice() etc), so you could use that.
Would the above solution for you? I'll create a separate issue to ensure we have some better docs for it and close this issue now.

Added an issue at #237 to update our docs regarding multi tenancy.

Hi Peter!
Thank you for your quick reply!
That's great! We will solve our multi tenancy feature using the UseClientCertificateResolver-extension and some kind of cookie to determinate where the end user is coming from.
But first we need to update to latest version (Active Login, Identity Server, NET Core) to get the extension but that wont be that big of a deal :)
Thank you for bringing us Active Login!
Regards

That's great! We will solve our multi tenancy feature using the UseClientCertificateResolver-extension and some kind of cookie to determinate where the end user is coming from.

Great! Yes, using a cookie and accessing that from IHttpContext accessor is one way we've solved this.

But first we need to update to latest version (Active Login, Identity Server, NET Core) to get the extension but that wont be that big of a deal :)

Should be an easy update. A list of breaking changes can be found in the release notes.

Thank you for bringing us Active Login!

You are welcome :)

Hi again @PeterOrneholm,
Is there some kind of caching of resolved certificates (using UseClientCertificateResolver). I have setup a multi tenancy scenario using the same scheme... and UseClientCertificateResolver only seems to be called once per application. Do we need to have unique schemes per customer or am I missing something?
Regards,
Anton

Would very much appreciate an example on how to correctly set this up, any code would do :) I can't get the UseClientCertificateResolver-callback to execute per customer, the callback is only called once and therefore same certificate is used for all customers.
I have tried use same and unique scheme for all customers. :)

Would very much appreciate an example on how to correctly set this up, any code would do :) I can't get the UseClientCertificateResolver-callback to execute per customer, the callback is only called once and therefore same certificate is used for all customers.

Ohh, you've found a bug! Thanks for reporting this. The project we are implementing multi tenancy for isn't live yet, and we simply haven't found this in our testing yet. It's really sad we don't have any unit test for this specific part, but the problem is that the callback is only triggered when making a real call to their API, so no easy way to fix that at the moment.
The problem is (what I can find out) is that ASP.NET caches the choice of certificate in the HttpMessageHandler pool. It seems to be cached by hostname, because usually you want to select cert based on hostname. But in our case we have multiple certificates for the same hostname.
Anyway, I'll make a hotfix to fix this issue, but until that is out, could you confirm that the below code would work? UseClientCertificateResolver is only an extension method to make it easier to configure, so you can try out the code below yourself before we make the hotfix. The fix is to disable caching if you enable UseClientCertificateResolver.
        public static IBankIdBuilder UseClientCertificateResolver(this IBankIdBuilder builder, Func<ServiceProvider, X509CertificateCollection, string, X509Certificate> configureClientCertificateResolver)
        {
            builder.ConfigureHttpClientHandler(httpClientHandler =>
            {
                var services = builder.AuthenticationBuilder.Services;
                var serviceProvider = services.BuildServiceProvider();

                httpClientHandler.PooledConnectionLifetime = TimeSpan.Zero;
                httpClientHandler.SslOptions.LocalCertificateSelectionCallback =
                    (sender, host, certificates, certificate, issuers) => configureClientCertificateResolver(serviceProvider, certificates, host);
            });

            return builder;
        }

It's not optimal, but the best/inly solution with current architecture. I've had a brief discussion with a colleague and for version 5 or similar we'd like to rearchitechture the solution to allow for a IBankIdClientFactory that you can override and provide different clients for different tenants.
If you confirm that the above works, I'll try to find time this week to release the hotfix.


Consume: Running the task with .NET Framework throws error

Using the task with .NET Framework throws the following error.
Could not load file or assembly 'Apache.NMS, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=82756feee3957618' or one of its dependencies. The system cannot find the file specified.

Fixed in version 1.0.1.


Draw & desing Game Logo

Make a Logo of the game with Affter Efects
References
Tritan Studios
Forge of Empires

New logo added to the project in WikiResources/LogoHostileKingdom.png


RuntimeError when trying to use 2-layer GCN in example with Tox21 dataset.

❓ Questions & Help
I'm new to deep learning so I'm not sure whether this is a bug or not. I'm getting RuntimeError: result type Floatcan't be cast to the desired output type Long when running the 2-layer GCN in the introduction with Tox21 dataset.
Here is my code:
import torch
import torch.nn.functional as F
from torch_geometric.nn import GCNConv
from torch_geometric.datasets import MoleculeNet, Planetoid'

#dataset = Planetoid(root='Planetoid', name='Cora')
dataset = MoleculeNet('MoleculeNet', "Tox21")

class Net(torch.nn.Module):
    def __init__(self):
        super(Net, self).__init__()
        self.conv1 = GCNConv(dataset.num_node_features, 16)
        self.conv2 = GCNConv(16, dataset.num_classes)

    def forward(self, data):
        x, edge_index = data.x, data.edge_index

        x = self.conv1(x, edge_index)
        x = F.relu(x)
        x = F.dropout(x, training=self.training)
        x = self.conv2(x, edge_index)

        return F.log_softmax(x, dim=1)

test = dataset[0]
net = Net()
out = net(test)

The RuntimeError is thrown at this line. The code above runs fine with Planetoid dataset though. Any suggestion for the code above?

Hi and sorry for the confusion! The MoleculeNet datasets come with discrete node features of type Long (following the OGB convention here). You first need to convert your input features into continuous embeddings, e.g, by the usage of an AtomEncoder:
class Net(torch.nn.Module):
    def __init__(self):
        super(Net, self).__init__()
        self.emb = AtomEncoder(16)
        self.conv1 = GCNConv(16, 16)
        self.conv2 = GCNConv(16, dataset.num_classes)

    def forward(self, data):
        x, edge_index = data.x, data.edge_index
        x = self.emb(x)

        x = self.conv1(x, edge_index)
        x = F.relu(x)
        x = F.dropout(x, training=self.training)
        x = self.conv2(x, edge_index)

        return F.log_softmax(x, dim=1)

Thank you. It worked!


When using multiple replicas to subscribe a message, multiple duplicate messages are received.

My Dapr CLI/Runtime is 0.8.0 in kubernetes.
When I used emqx as the mqtt broker to subscribe and publish messages, the following problems occurred:
I use the dapr mqtt component to subscribe and publish messages. I set the number of replicas of the receiver to 3. When the sender sends one message, only one copy should receive one message, but the reality is that all three copies have received the message. They received 4, 2, 2 identical messages respectively.
When I try to set the QOS of the mqtt component to 0, it will return to normal. The number of messages received by the three replicas is 0, 1, 0, but when the QOS is 1 or 2, they are all 4, 2, 2.
Due to the needs of the project, QOS can only be set to 2. I hope to know the reason and modification method of this problem in dapr.
Thanks！

Hi, this was fixed and will come out in 0.11 release.

@AI-Marvel, I'm closing this for now. If you're still encountering this after upgrading to 0.11, please re-open.

In the new version 0.11.2, I found that it is normal only when sending multiple copies for the first time. If the pod is deleted and restarted, the above problems will occur, unless the name of deployment in YAML file is changed.
In addition, when sending messages with multiple copies, the receiver often receives more messages than its own copies. For example:
Pod A has three copies, and each copy sends a piece of data to pod B, but actually B receives more than ten or more pieces of data.

I'm also facing the same issue. There are more messages received by the consumers when using MQTT with activemq. I see MQTT is the only supported protocol for connecting to activemq.


FIX: import crossencoder without torch installed and git push of creds

Codecov Report
Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 73.63%. Comparing base (1d27dde) to head (2d6007b).
Report is 2 commits behind head on main.

Files
Patch %
Lines

...emb/infinity_emb/transformer/crossencoder/torch.py
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #181      +/-   ##
==========================================
- Coverage   73.64%   73.63%   -0.02%
==========================================
  Files          32       32
  Lines        1791     1794       +3
==========================================
+ Hits         1319     1321       +2
- Misses        472      473       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[CWS] add missing SBOM config to system probe config

What does this PR do?
This PR adds definition for missing SBOM config that were resulting in those logs in the tests:
[2024-03-06 17:16:39.515] [WARN] github.com/DataDog/datadog-agent/pkg/config/model.(*safeConfig).GetDuration:342 failed to get configuration value for key "sbom.scan_queue.base_backoff": unable to cast <nil> of type <nil> to Duration
[2024-03-06 17:16:39.515] [WARN] github.com/DataDog/datadog-agent/pkg/config/model.(*safeConfig).GetDuration:342 failed to get configuration value for key "sbom.scan_queue.max_backoff": unable to cast <nil> of type <nil> to Duration

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


*: Drop verify-{commit,tag} workflows

These workflows have some shortcomings, as discussed in https://github.com/gittuf/gittuf/issues/384#issuecomment-2099153472.
verify-commit doesn't currently do enough to ensure the right policy is identified for when a commit is first introduced. verify-ref, with some enhancements, is better.
verify-tag implements a subset of verify-ref already because it recognizes tags are refs. Thus, verify-ref is again a better option.
Note: this makes #431 a bit cleaner.

cc @patzielinski @flandweber @neilnaveen

This makes much more sense since the verify commit cannot assure us that the ref is correctly verified, and the tags themselves are just special refs. We would have to add some code into verify-ref to handle tags for everything to work smoothly, or would we only want the basic verification we get from verify-ref? Thoughts?

I agree. I believe the most important benefit of this change is that it makes gittuf simpler to use.

This makes much more sense since the verify commit cannot assure us that the ref is correctly verified, and the tags themselves are just special refs. We would have to add some code into verify-ref to handle tags for everything to work smoothly, or would we only want the basic verification we get from verify-ref? Thoughts?

We already handle tags separately via verifyTagEntry, so I think we're covered actually. I also have a change (ab1fa83bc406a4394217f0047f699e6292f3a933) coming that updates verifyTagEntry to use the verifier bits to enable attestations.


add documentation node and parsing logic spec2doc

approach changed to Explicit::Documentation::Property that knows how to speak spec


⚠ Bofrost.at has degraded performance

In 9cf490d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1292 ms

Resolved: Bofrost.at performance has improved in ba1320d after 9 minutes.


[FEATURE REQUEST] New Format placeholder to create random code when insert into other document

Is your feature request related to a problem? Please describe.
I need to create a block reference when insert text into excalidraw drawing, You could refer something here: https://github.com/zsviczian/obsidian-excalidraw-plugin/releases/tag/1.2.0-alpha-1
Describe the solution you'd like
A {{random:6}} to create a 6 random alphabet or digital code: like 3YusT5 OR IGaspd.
Like  when i use {{value}} ^{{random:6}} to get my text and a random code
Additional context
NO

Hey!
Just wanted to give a response. This could be a great custom command.
For now, you could create a macro or inline script to use this code:
function uuidv4(length) {
  return 'xxxxxxxxxxxx4xxxyxxxxxxxxxxxxxxx'.replace(/[xy]/g, function(c) {
    var r = Math.random() * 16 | 0, v = c == 'x' ? r : (r & 0x3 | 0x8);
    return v.toString(16);
  }).substring(0, length);
}

Given a length, it returns a random string of characters of that length.
Modified from here.


Element-id attribute doesn't work.

Hi,
i am using ngPrint directive in my case print-element-id="id-of-element" doesn't works.
its print whole view instead of element view.

Hi,
Can you be more specific?
Where did you put the element-print-id?
Are you using the directive's CSS?

no i am not using directive css.
my code:
`

{{thing.name}}
×

 Print
`

Please use the ngPrint.css as part of your app or things won't work.


Update .gitignore

Add VSCode and mac related files
I saw that some mac related files are commited in the repo

Are there any mac-files that are already in the repo? These should then be removed.

.vscode and .DS_Store  should be deleted

I don't see any DS_STore file, and the already committed vs code files are fine as well.


Trying to figure out how to customize ttyd, but "yarnpkg run start" does not work for me

I'm trying to customize ttyd to show a terminal, in combination with a side-bar for displaying keyboard-shortcuts for a particular system.
I'm a node.js and yawn an-alphabet so please bear with me...
I tried to follow the instructions to fire up the development environment
## Development
1. Start ttyd: `ttyd bash`
2. Start the dev server: `yarn run start`

ttyd bash starts fine on 7681
Then I found that yarn on Debian is yarnpkg, and running that I get:
$ yarnpkg run start
yarn run v1.22.19
$ gulp clean
[14:39:27] Using gulpfile ~/Projects/ttyd/html/gulpfile.js
[14:39:27] Starting 'clean'...
[14:39:27] Finished 'clean' after 15 ms
$ webpack serve
[HPM] Proxy created: [ '/token', '/ws' ]  ->  http://localhost:7681
ℹ ｢wds｣: Project is running at http://localhost:9000/
ℹ ｢wds｣: webpack output is served from /
ℹ ｢wds｣: Content not from webpack is served from /home/pdj/Projects/ttyd/html/dist
ℹ ｢wdm｣: asset app.js 2.72 MiB [emitted] [big] (name: app)
asset favicon.png 1.62 KiB [emitted] [from: favicon.png] [copied]
asset index.html 298 bytes [emitted]
runtime modules 1.29 KiB 7 modules
modules by path ../node_modules/ 1.02 MiB 64 modules
modules by path ./ 31.2 KiB
  modules by path ./components/ 28.7 KiB
    modules by path ./components/modal/ 5.73 KiB 3 modules
    modules by path ./components/terminal/ 15.4 KiB 2 modules
    ./components/app.tsx 1.5 KiB [built] [code generated]
    ./components/zmodem/index.tsx 6.04 KiB [built] [code generated]
  modules by path ./style/*.scss 2.4 KiB
    ./style/index.scss 1.22 KiB [built] [code generated]
    ../node_modules/css-loader/dist/cjs.js!../node_modules/sass-loader/dist/cjs.js!./style/index.scss 1.17 KiB [built] [code generated]
  ./index.tsx 165 bytes [built] [code generated]
webpack 5.74.0 compiled successfully in 3855 ms
ℹ ｢wdm｣: Compiled successfully.

When I then connect the browser to http://localhost:9000/ I get:
[HPM] Error occurred while trying to proxy request /token from localhost:9000 to http://localhost:7681 (ECONNREFUSED) (https://nodejs.org/api/errors.html#errors_common_system_errors)
[HPM] Upgrading to WebSocket
[HPM] Error occurred while trying to proxy request /ws from localhost:9000 to http://localhost:7681 (ECONNREFUSED) (https://nodejs.org/api/errors.html#errors_common_system_errors)

The end goal is to get ttyd running a terminal with a side-bar containing static html that shows how to use the terminal.
I'm not even sure that I'm on the right track here, so I hope someone can find the time to push me in the right direction.
Thanks
~Per

ttyd bash starts fine on 7681
[HPM] Error occurred while trying to proxy request /token from localhost:9000 to http://localhost:7681 (ECONNREFUSED)

It's not fine, it you can't access http://localhost:7681


Add support for simple linear/logistic regression

You've done a great job with the more sophisticated algorithms: would it be possible, for completeness, to throw in linear/logistic regression? I imagine it would be fairly quick comparatively.

Hi @codeartsciencemechanicsassets,
Originally there actually was a linear regression and a logistic regression learner/model in the project, these were based on stochastic gradient decent. However, after the introduction of SharpLearning.Neural, I decided to remove them, since you can build a network that solves linear regression and logistics regression, exactly like the previous implementation.
You can see an example on how this is done here:
Logistic Regression:
// No hidden layers and SoftMax output layer corresponds to logistic regression classifier.
var net = new NeuralNet();
net.Add(new InputLayer(featureCount));
net.Add(new SoftMaxLayer(numberOfClasses));

var learner = new ClassificationNeuralNetLearner(net, iterations: 10);

Linear Regression:
// No hidden layers and SquaredError output layer corresponds to linear regression.
var net = new NeuralNet();
net.Add(new InputLayer(featureCount));
net.Add(new SquaredErrorRegressionLayer());

var learner = new RegressionNeuralNetLearner(net, iterations: 10);

The implementation of the neural network layers and learner really needs an overhaul, so it becomes easier to define loss functions and so forth. Currently, the actual loss is defined implicitly by the last layer. However, I am waiting for the some of the .net tensor libraries to mature a bit more, and then I will start the process of improving this project.
You should however be able to use linear/logistics regression with the above examples.
Best regards
Mads

Elegant and true! I wish I had some time to contribute to this library, because I've been writing Q-Learning and general RL packages in C#, my fear is that they would be too domain specific for a general library like yours [they are highly optimized for our environment]. Anyhow, keep up the excellent work.

It could be very interesting with some reinforcement learning methods. I need to research more into that topic before I can contribute with something general I think, but it is definitely on my list, and contributions are always welcome if you have the time at some point :-). It has also been discussed in issue #98.

Closing this issue, since an example has provided, and another issue is already open on reiforcement learning methods. Feel free to reopen if necessary.


Disable NavigationDrawer in Pushed ViewController

Hi, i just have a question
i have a RootViewController (MainViewController With navidrawer and Menu Button(FAB))
let MainViewController: MainViewController = {
            return UIStoryboard.viewController(identifier: "MainViewController") as! MainViewController
        }()

        let NaviLeftViewController: NaviLeftViewController = {
            return UIStoryboard.viewController(identifier: "NaviLeftViewController") as! NaviLeftViewController
        }()

        let ApptoolBar =  AppNavigationController(rootViewController: MainViewController)

        let NaviDrawer =  AppNavigationDrawerController(rootViewController: ApptoolBar, leftViewController: NaviLeftViewController, rightViewController: nil)

        let SnackBar   =    AppSnackbarController(rootViewController: NaviDrawer)

        let Menu       = AppMenuController(rootViewController: SnackBar)

        UIApplication.shared.keyWindow?.rootViewController = Menu

when i push using the navigation controller the naviDrawer and the menu are shown in the Pushed View Controller
self.navigationController?.pushViewController(OrderBidsView, animated: true)
i can hide them and show in both viewControllers , but is there a way to hide the navidrawer and the menu in pushed view controllers instead of showing and hiding them manually?

Hey :)
They are showing because you put them as the outermost view controller. When you do that, they will always be there ready for use, but you don't need to use them until necessary. It is actually better, because you can reuse the controllers, also you can use them as points to put code that is completely relevant to them. The other option is to use them as your sub controllers. Then you will need to create a new one for every view controller that needs it. This is also good when you need that type of organization. Their design is meant to be used in anyway that is correct for your needs and maintenance. I am working on an article for it: http://cosmicmind.com/danieldahan/application-architecture-with-material, so feel free to follow that over the next couple days, as I will be covering all of this :)
Please reopen or create a new issue if needed.

@danieldahan  @tarek903 Can you please guys take a look into this. This is something very urgent https://github.com/CosmicMind/Material/issues/733


API Need: Promise with full receipt

Currently promise support only covers the response Vec<u8> from callbacks.
This is useful, but is missing critical other information available via RPC: logs array, gas used, etc per-receipt.

Goal:
Create/expose full receipt via [PromiseResult](https://docs.rs/near-sdk/4.0.0-pre.7/near_sdk/enum.PromiseResult.html) or similar in a way that contracts can consume logs/gas for the remainder of their function stack.

Simple Use case:
Say I want to paginate a list of accounts and call the maximum executions within the cap 300Tgas.
If the contract were able to know how much gas was previously used overall (the top level txn gas), then the contract would know if it can continue paginating and avoid failure.
Including access to logs, allows for additional conditional data available to the contract as well - imagine NFT event data coming back within an NFT transfer within a DAO proposal finalization. This could ultimately reduce cross contract calls drastically.

Here's more context:
env::prepaid_gas() only pertains to the gas within the context of current function execution - it doesnt know anything about the total available gas for the TXN as a whole.
So going back to my example above, if you have a callback at any point in the stack, you have no idea how much gas was previously spent versus how much was (attached - used) accurately.
Quick example:
# Batch call with 300TGas
A. prepaid: 30TGas, Used Gas 0
   - 1. nested: prepaid 30TGas, used 10TGas
   - 2. nested: prepaid 15TGas, used 20TGas

I know these are fake numbers, but within each nested execution, there's no way to how much total gas is remaining so i know if I can keep calling more fns until full 300Tgas is used. Unless there's something missing to make that possible...
From the docs:
Used Gas: The gas that was already burnt during the contract execution (cannot exceed prepaid_gas)
source: used_gas
Prepaid Gas: The amount of gas attached to the call that can be used to pay for the gas fees.
source: prepaid_gas
In practice its more confusing than helpful.

I think the presented use-case can be solved in a more elegant way using recently introduced yeild-resume features of NEAR Protocol: https://docs.near.org/blog/yield-resume
I am closing this issue as it is not actionable from SDK side. The discussion should start on the protocol side first:

Chat: https://near.zulipchat.com/
NEPs: https://github.com/near/NEPs


move dependencies in peer and dev dependencies to remove vunerability

Proposed Changes

Potential Breaking Changes

Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

Merging #1188 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1188      +/-   ##
==========================================
+ Coverage   99.66%   99.67%   +0.01%
==========================================
  Files         351      350       -1
  Lines        8867     8860       -7
  Branches     2101     2099       -2
==========================================
- Hits         8837     8831       -6
+ Misses         30       29       -1

Impacted Files
Coverage Δ

.../src/components/flatSelect/FlatSelectConnected.tsx
100% <0%> (ø)
:arrow_up:

...r/src/components/flatSelect/FlatSelectSelectors.ts

...por/src/components/table-hoc/TableRowConnected.tsx
100% <0%> (+1.25%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 503d2e0...bee4b0b. Read the comment docs.


Update README.md

Divine Yuri's Custom City Panel
The image URLs for the new Districts bar and Religions bar tooltips were corrected (swapped).

The image URLs for the new Districts bar and Religions bar tooltips were corrected (swapped).


release concurrency lock for errors

Currently the concurrency limiter will only release the lock if an exception is thrown in the block request. This PR will catch Throwable instead to release the lock when an Error occurs

Interesting. Maybe the code should be changed to not use tap but just release in a finally` block? 🤔
Untested:
try {
  $callback();
} finally {
                       $this->release($slot, $id);
}


Edit: Advanced Input

Description
This PR adds an advanced Edit input that:

Allows users to manually adjust the selection of the Edit
Allows users to manually change the LLM that is used for an Edit (currently only Claude 2 and Claude Instant are supported)
Allow users to easily trigger the Document command, and select a relevant symbol to document
Allow users to easily trigger the Test command, and select a relevant symbol to document

Videos
Changing range and model:
https://github.com/sourcegraph/cody/assets/9516420/f8f0bdcb-5633-41a2-a0c9-10164a2ec5a9
Document & Test
https://github.com/sourcegraph/cody/assets/9516420/3462c94c-c803-4d21-a6ad-7ce853d112ec
Test plan
Create edits:

From no selection (generate code)

Modify range to switch to expanded selection (edit code)

From a selection (edit code)

Modify range to switch to cursor (generate code)

Modify the range of an edit:

To a symbol in the document
To the expanded selection
Back to a previous selection after changing it

Trigger the document command:

From a symbol in the document
From the selection

Trigger the unit test command:

From a symbol in the document
From the selection

@toolmantim @abeatrix I think this is ready for review now...
I am adding some integration tests to cover the different scenarios that the quick pick can end up in.
I am adding support for the "Model" selector in: https://github.com/sourcegraph/cody/pull/2951 After chatting with @abeatrix, we will need some extra work to ensure this works well with Enterprise and also shows the correct models.

Some additional screenshots for reference:
Document:

Test:

Range:

@abeatrix Thanks for catching these bugs!

when trying to edit code at cursor position, it would add an empty line after the generated code, but i think the line should be added above the newly added code?

I think this is because I was incorrectly calculating the cursor position - just using the end of the selection. I've updated it so it actually uses selection.active now.

Follow up from 1., adding an empty line above / below the current cursor won't work for edits that happened between existing code on the same line though 🤔 (don't think it should block this PR though)

Yeah I think we might need to follow up on some more fine tuning for generate... The formatting can also be weird if you are within some existing code, I think maybe we need to use the full line in that case. That might help here too?

smart selection no longer work on /doc and /test command when invoking from context menu or sidebar, is that intended?

Nope, should be fixed now though!


Save the files in Chapter with the complete path

The list of files in a Chapter Struct should have the full path.


DocuTune: Apply metadata properties for advocate contributors

DocuTune: Apply metadata properties for advocate contributors
Note the following:

DocuTune drives improved quality of technical content by finding and automatically correcting a broad set of issues in Markdown and YAML files. DocuTune performs these updates intelligently, only making corrections in the sections of a document where the correction is suitable.
DocuTune PRs often standardize spacing, formatting, and casing of Markdown elements to support the DocuTune onboarding process.
DocuTune PRs might contain other minor edits noticed during a manual review.
DocuTune PRs might include files whose Acrolinx score was below the target threshold prior to this PR.
Delimiters such as asterisks and backticks may be added to link descriptions or alt-text to denote UI elements or other string literals that should not be modified during automation.
Other formatting or indentation corrections might appear in DocuTune PRs to improve the accuracy of document parsing.

PointOfContact: @scottboc (Scott Bockheim)
CorrelationId: 70d29a1f-52c7-4693-b6b6-48364523765f

Docs Build status updates of commit 65e6337:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/type-of-apps-microsoft-365.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit efd267a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/type-of-apps-microsoft-365.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Dealing with missing modules

after a fresh install, I got missing "Module no found" errors for both 'tether' and 'history'.
I dealt with it in a quick/dirty way (I guess) by adding their full reference to index.js:
diff --git a/src/index.js b/src/index.js
index e164c4b..6aa9359 100644
--- a/src/index.js
+++ b/src/index.js
@@ -5,7 +5,7 @@ import 'bootstrap/dist/css/bootstrap.css';
import 'font-awesome/css/font-awesome.css';
import jquery from 'jquery';
window.$ = window.jQuery=jquery;
-window.Tether=require('tether');
+window.Tether=require('bootstrap/node_modules/tether/dist/js/tether');
require('bootstrap/dist/js/bootstrap');
/////////////////////////////////////////////
@@ -20,7 +20,7 @@ import { Router, Route, IndexRoute} from 'react-router';
// Please change to browserHistory if you have your own backend server.
// import {browserHistory as history} from 'react-router';
import { useRouterHistory } from 'react-router';
-import { createHashHistory } from 'history'
+import { createHashHistory } from 'react-router/node_modules/history';
const history = useRouterHistory(createHashHistory)();
I don't know why/if I'm the only one reporting this problem (something might be wrong with my setup). An other way to solve it would be to include tether and history modules in package.json dependencies.

What's your npm version?  Node >= 6 and npm >= 3 are recommended. see README.
npm3 would install dependencies in a flat way.
You may also want to delete node_modules and do a fresh npm install.

you're absolutely right, an old version of npm 1.* was hiding in my environment. Thanks for the fast feedback !


Distinguish edge tpu models from regular tflite models

Description
While visualising several edge tpu models using netron I found 'edgetpu-custom-op' in all of them. Do all edge tpu models and only edge tpu models contain 'edgetpu-custom-op'? Can we use this 'edgetpu-custom-op' to differentiate edgetpu models from regular tflite models?
(I tried to get ops using interpreter._get_ops_details() in python )
Click to expand!
Issue Type
Support
Operating System
Linux
Coral Device
USB Accelerator
Other Devices
No response
Programming Language
C++, Python 3.8
Relevant Log Output
No response

yes, you can use the edgetpu-custom-op to differentiate edgetpu models from regular tflite models: https://github.com/google-coral/edgetpu/issues/546#issuecomment-1050665727. Thanks!


Remove pip from pyproject.toml

Retaining pip as a dependency becomes problematic for some installations, especially for conda-based environments on Windows

Fixed via https://github.com/uclamii/model_tuner/pull/138


Fix memory allocation issues in binomial graph generators

This commit fixes issues with the allocation of memory in the binomial
graph generators. Previously there were 2 issues in the binomial graph
generator functions when it came to allocating memory. The first was a
temporary Vec was used to store all the node indices in the graph which
would eat num_nodes * sizeof(usize) bytes of memory. This Vec isn't
actually needed as the NodeIndex is just a usize integer wrapper and we
know the indices of the nodes. This commit removes this and just
iterates over the range of nodes instead of creating a Vec reducing the
amount of memory required. The second issue was that it was easily
possible to overflow the max Vec size (either inside the graph objects
or from the Vec previously created to store the node indices) by using
an order where 2**order > isize::MAX or if order > 64 causing an
overflow of a 64bit usize which would result in a panic. This fixes
this issue by checking the input of order and seeing if it will overflow
or exceed the max Vec size and raise an OverflowError.
Fixes #457

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

61 of 61 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.003%) to 98.286%

Totals

Change from base Build<PHONE_NUMBER>:
-0.003%

Covered Lines:
10323

Relevant Lines:
10503

💛  - Coveralls


[opencsg] Update to 1.6.0

Fixes #37446.

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Thanks for the update and removing the unused vcpkg-cmake-config port reference!


Support for ARM64

I'm trying out the Docker image on my Macbook (with ARM CPU), and I'm seeing the following error:

grafana-lgtm The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

I'm not seeing any traces are coming in, in Tempo, but I'm not sure if it's related (but I do not think so, since I've never had it working before :p).
Not sure it's already on the roadmap.
As a workaround I've added:
  platform: linux/amd64

to my compose.yaml file.

Thanks for reporting this! Would you be interested in contributing an ARM64 compose file? I think your workaround is the correct solution here.

The run command: https://github.com/grafana/docker-otel-lgtm/blob/main/run-lgtm.sh#L3C1-L3C77
Would be:
docker run --platform linux/amd64 -p 3000:3000 -p 4317:4317 -p 4318:4318 --rm -ti grafana/otel-lgtm

My docker-compose looks like this:
services:
  grafana-otel-lgtm:
    platform: linux/amd64
    image: grafana/otel-lgtm
    container_name: grafana-otel-lgtm
    ports:
      - "3000:3000"
      - "4317:4317"
      - "4318:4318"

It's part of my "default" services docker-compose.yaml file.
Btw, I've also managed to get data in, so thats all working :).

@TomKeur a PR contribution is highly welcome :smile:

I can make an contribution to https://github.com/grafana/docker-otel-lgtm/blob/main/run-lgtm.sh#L3
For example:
# Check if macOS is the OS.
if [ "$(uname)" == "Darwin" ]; then
    PLATFORM="--platform linux/amd64"
fi

docker run ${PLATFORM} -p 3000:3000 -p 4317:4317 -p 4318:4318 --rm -ti grafana/otel-lgtm

Let me know if that's welcome =).

I guess, adding this would help?
https://docs.docker.com/build/ci/github-actions/multi-platform/
And publish the Docker image with GitHub Actions instead of "local"?
I'm not really sure how the tagging works atm, but it should not be that hard to setup.


New Forge Release

Can you push a new forge release to make repos available?

Yes sure, sorry I was waiting to add some extra stuff but haven't got around to it yet, so I'll release the current state as-is today.....

@jhg03a sorry for the delay - I've pushed this to the forge as 0.2.0, including some small updates to the README.


⚠ gorzdrav.org has degraded performance

In 945052e, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 9086 ms

Resolved: gorzdrav.org performance has improved in 9a69b61.


Installation instructions for Ubuntu / Debian based systems

Sorry if this isn't the right place to put this...
Just came across this project, and I was shocked to find there was no install instructions for Ubuntu. I'm sort of new-ish to Linux, and I'm running Pop!_OS. I came to this installing page and noticed instructions for Nix, asdf, Arch, FreeBSD, OpenBSD, and Void, but nothing debian or ubuntu based. I tried installing with apt-get, flatpak, and snap, and all of those failed. I'm sure I can build from source, but it would be neat if there was a section at least labeled "Ubuntu/Debian" that mentions "Use asdf or build from source".
Can't wait to check it out! :raised_hands:

that mentions "Use asdf or build from source"

fwiw, you're on the right path there. most linux peeps use asdf to manage their gleam installation. :)


Build android failed on Windows

./gradlew app:assembleRelease outputs,
> Build command failed.
  Error while executing process C:\Users\xxx\AppData\Local\Android\Sdk\cmake\3.18.1\bin\ninja.exe with arguments {-C C:\Users\xxx\AndroidStudioProjects\KhronosGroupVulkan-Samples\build\android_gradle\app\buil
d-native\RelWithDebInfo\2h311d26\arm64-v8a spirv-cross vulkan_samples}
  ninja: Entering directory `C:\Users\xxx\AndroidStudioProjects\KhronosGroupVulkan-Samples\build\android_gradle\app\build-native\RelWithDebInfo\2h311d26\arm64-v8a'

  [0/2] Re-checking globbed directories...
  ninja: build stopped: .

  ninja: error: mkdir(third_party/CMakeFiles/native_app_glue.dir/C_/Users/xxx/AppData/Local/Android/Sdk/ndk/21.4.7075529/sources): No such file or directory

Hard to help with so little details. The error hints at a missing file in either a submodule or the NDK. So is the Android NKD properly installed, and did you clone the repo with submodules?

Hi @ylz-at, let us know if you can provide more details.  I'll close this shortly if not - we can re-open later if more info can be provided.  Thanks!

Oops, closed too soon.  But we'll likely close this in a week or 2 without further info.

@SaschaWillems Android SDK and NDK are correctly installed with AndroidStudio, and I can compile other android-native projects, such as https://github.com/SaschaWillems/Vulkan, just fine. And the repo is cloned with submodules.
I was trying to build a CI pipeline on windows to demonstrate this problem, but it takes hours to run...weird.
https://github.com/ylz-at/Vulkan-Samples/runs/7421535652?check_suite_focus=true

Well...path should be short enough on Windows.
similar issue, https://github.com/cocos2d/cocos2d-x/issues/19399

FWIW: https://www.howtogeek.com/266621/how-to-make-windows-10-accept-file-paths-over-260-characters/  Not that I recommend this as a real solution to ya know... anything....


ZSH_AUTOSUGGEST_HIGHLIGHT_STYLE is overwritten at startup

I'm used to separate my dotfiles based on their scope. E.g. I have a file .colors.zsh that defines all colors of my theme and makes sure that all components (prompt, LS_COLORS, completion, ...) share the same palette. Some plugins require these variables to be set prior to zplug load which is why I source that file early in my .zshrc.
Unfortunately most variables of zsh-autosuggestions, including ZSH_AUTOSUGGEST_HIGHLIGHT_STYLE, are overwritten at startup with their respective default values which makes pre-configuration impossible. This could be fixed by a simple check prior to setting the default value.

#335 #378

Oh, sorry about the duplicate ... I was probably too focused on this particular Variable when I looked through the existing issues.


Better source line and column numbers

Attempt to improve the graphical reporter source file line number and column number .
Currently oxlint renders the line number as the first line in the snippet column number is always 1 AFAICT. Example of current version on VS Code repo:

This means that when CTRL-clicking on the source-file:line:col "link" it takes me to the wrong place.
This PR is an attempt to improve things by getting closer to the source of the issue. Here is my version again on VS Code:

I'm a bit worried that none of the tests failed though - would that be expected?
Also the approach feels naïve and there may be several cases it won't work for, or won't work quite so well. Specifically it's just finding the first labelled-span, but I'm guessing in some cases that could be providing context, rather than the thing the user would want to change.
As before I'm no expert and any feedback is very much appreciated.
Also in my local testing, VS Code is still a bit unpredictable - even with the correct line + col it will always go to the correct line number but doesn't always go to the correct column 🤷 But that's not our issue 😄

I'm a bit worried that none of the tests failed though - would that be expected?

This is the forked graphical reporter, the testers are using the once from miette. Let me clean this up a little bit.

Ah that explains it, thanks @Boshen 🙏


What's going on with this repository?

I'm submitting a ...
[ ] bug report
[ ] feature request
[x ] question about the decisions made in the repository

I see a lot of issues and PRs were closed without any resolution. What will happen with this repo? Is it dead? Should it be transferred to someone else?

This repo is likely deprecated in favor of Angular CLI

yes please use the angular cli for now. I'm making my own version but wont be ready til later

sry about that the real reason was because i was waiting for ivy I'll be updating the repo in the next month


[Bug]: Spotify Error

Version
1.4.0
Provider type
Spotify
Environment
 System:
    OS: Windows 10 10.0.19045
    CPU: (4) x64 AMD A10-8700P Radeon R6, 10 Compute Cores 4C+6G
    Memory: 150.02 MB / 3.87 GB
  Binaries:
    Node: 22.2.0 - C:\Program Files\nodejs\node.EXE
    npm: 10.9.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Chromium (126.0.2592.61)
    Internet Explorer: 11.0.19041.4355

Describe the issue

App.tsx?t=1729318475817:48 TypeError: btoa is not a function
at encodeBase64 (encode.ts:7:12)

SyntaxError: Unexpected token '<', "<!doctype "... is not valid JSON

How to reproduce

Install the package
Use any Spotify Component
Then it is showing this error

Expected behavior

Don't show this error.
Return  us the user data and access token.

We have fixed the issue in the branch v-1.5.0. It will launched within 2 days.


Improved message API

cf socketio/socket.io-client-cpp#56

Good change, but can you replace these tabs with spaces? The indent seems incorrect.


Tons of Updates

See changelog

👍 looks good to me - just an update to the latest, right? I created this fork a while back to submit a PR - but keeping it up to date makes good sense too (if for no other reason, when we rely on git for a gem's source, we should do it out of the DT org and not someone else's who can delete it out from under us [which has happened]. Not that AMS is at risk of that, of course. Just on principle. :)

yeah, this is just an update. :-)
On my side project, I'm using AMS#master, so, I can vouch for it :-)

Does anyone use this repo? or do people use the rails-api one?

Closing, cause AMS is in disarray at the moment (ha).
I'd personally recommend http://jsonapi-rb.org/ (written by one of the AMS Owners).
I think AMS vNext is going to take a lot of inspiration from JSONAPI-rb.
But, I think JSONAPI-rb has more potential -- especially since it's broke up into smaller gems -- which makes writing native extensions much easier. With the rust-ruby integration via Helix, I think JSONAPI-rb could eventually be really fast. :-)
(Right now, JSONAPI-rb is only slightly faster than AMS)


No issues report index error

Recently @birdofpreyru  found the following bug:
root@4b8e9a2487db:/home# myth -xc abcc
Traceback (most recent call last):
  File "/usr/local/bin/myth", line 4, in <module>
    __import__('pkg_resources').run_script('mythril==0.17.12', 'myth')
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 742, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 1503, in run_script
    exec(code, namespace, namespace)
  File "/usr/local/lib/python3.6/dist-packages/mythril-0.17.12-py3.6.egg/EGG-INFO/scripts/myth", line 9, in <module>
    mythril.interfaces.cli.main()
  File "/usr/local/lib/python3.6/dist-packages/mythril-0.17.12-py3.6.egg/mythril/interfaces/cli.py", line 205, in main
    'text': report.as_text(),
  File "/usr/local/lib/python3.6/dist-packages/mythril-0.17.12-py3.6.egg/mythril/analysis/report.py", line 54, in as_text
    filename = list(self.issues.values())[0].filename
IndexError: list index out of range

This is caused by a problem in report.py


Add artifacts-dir param to flink runner

Please add a meaningful description for your change here
R: @robertwb @ibzib

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Please add a JIRA issue next time. This is a new feature which should be documented.


[FEATURE REQ] core-mgmt - move PollerFactory out of implementation package and into polling

Move PollerFactory out of implementation package and into polling

Depends on https://github.com/Azure/azure-sdk-for-java/issues/8266


Confusion about enum and titleMap in select

Problem
So I'm getting to understand this great library, though I am still running into some troubles using the select field.
Expected behaviour
Reading through the documentation and checking the code
if (stripNullType(schema.type) === 'string' && schema['enum'])

It is clear that I am required to use an enum, which requires some data.
Now there is also this part:
if (!f.titleMap) {
        f.titleMap = enumToTitleMap(schema['enum']);
}

Suggesting that a titleMap can be used, and when it does not exist, enum will become titleMap.
But since enum appears not to handle value:key objects, and only values, the incorrect value will be assigned to the model
If someone could please provide a code example for a select box using enum and titlemap I would be forever grateful.
Cheers
@json-schema-form/angular-schema-form-lead

@HeerRik there's a bunch of titleMap examples in the demo page: http://schemaform.io/examples/bootstrap-example.html
The schema is based on json-schema, that does not support titleMap as it is a data model definition syntax only. The form definition is where you can apply a titleMap. The schema enum must contain valid VALUES only, not keys.
Please re-open if you do not feel the question has been answered sufficiently.

First off, thanks for your reply
So if I create a string type with an enum in the schema, and then a select type with the titleMap in the form defenition, they should work together right? Right now I for some reason get a select box and a string box under 1 title. Do you need a fiddle or something to see or do you happen to know the problem?

@HeerRik yes I would need a gist, I suggest using the titleMap demo to replicate what you are trying to do and save to gist and then post that and re-open :)

@Anthropic , Can we reset the selected titleMap value based on some condition ?


Add requested models to RISC-V 64 benchmarks

Add MiniLML12H384Uncased and ClipTextSeqLen64PT to RISC-V 64 benchmarks.
Also enable 2-threads configuration on all benchmarks
#14623

@rednoah91 FYI, this change adds the models mentioned in #14623. At the same time I also enable 2-thread benchmark configurations

LGTM. Thanks @pzread.


Update German translations

Please, make sure you are targeting the develop branch!
More instructions about contributing with Gogs code can be found here:
https://github.com/gogits/gogs/wiki/Contributing-Code

@Unknwon I'm sorry, I've spent a couple of hours to improve the German translation - it is really a mess right now. But then I've found that you don't accept pull requests for locale files :(((.
Any suggestions what to do? I hope, all my work was for nothing now.

Thanks your PR!
However, https://github.com/gogits/gogs/wiki/Contributing-Code#those-we-do-not-accept

@Unknwon I know, but to me it seems obvious that the current system with crowdin does not work! The german translation is of very low quality. For example it's very inconsistent even in the salutation of the user or in translation of "issue". There's much more wrong.
Is there no way to somehow get my contribution merged?

Everyone thinks their translation is the best, as I don't know Germen at all, I can't do that just because you ask. I'm sorry.

@Unknwon I understand that. I don't expect this to be merged without some discussion. But I think every German will agree that it's not acceptable to mix the two possible translations for "you" in German (Du/Sie). It's either always "Du" or always "Sie". Right now it's a wild mix of both.
Another thing is the inconsistens translation of terms like "Issue", "Mirror", etc. They should always use the same translation. That's also not the case.
My suggestion here is an attempt to make this consistens all over the application. I think crowdin is not really the best tool to achieve a good translation. There should be one person in charge and maybe a catalog of common terms.

I'll see what I can do.

OK, I uploaded results to Crowdin, but under my name (with latest updated time), you can go https://crowdin.com/translate/gogs/1/enus-de to vote or just mention me on comment.

Hmm, ok. At least something. Still a lot of work. But thanks!

It's a good lesson about what could happen if you don't read the docs before doing something, :D

Yeah, sorry. I was so sure, that a simple PR is all I need. Another question: I see many phrases on crowdin with many upvotes. For example the translation for "Mirror". Someone translated it as "Spiegel", which sounds weird. The most votes are for "Mirror" (which is fine). So why is it still "Spiegel" in the locale file? Or how do you sync between crowdin and this repo?

@mikehaertl locale files in main repo is not up-to-date, that's why we ask you to do on Crowdin, I merge to main repo constantly.

@Unknwon I'm sorry, I don't understand. Why is it not up to date, if you merge constantly?

I don't merge for every single change, but every week.

@Unknwon Sorry, for bothering so much. But I still see many wrong translations, even if they have correct translations with many votes on crowin. For example:
https://github.com/gogits/gogs/blob/develop/conf/locale/locale_de-DE.ini#L790
The better German suggestion "Team-Repositories" has 4 upvotes, the bad "Team-Repositorys" has -1 votes. Both were proposed over a year ago. So why is the latter still in the main repo?

@mikehaertl because they did not get approved before last merge.

@Unknwon Ok, final question: When will this approval happen or who's responsible for this? If this process is documented somewhere, then please forgive me and just point me there.
BTW I forgot to mention that I love what I see from gogs so far!! :) It's looks so much better in many regards than this nasty resource hog §$"#!..lab stuff. It even motivated me to immediately contribute some improvements for localization. So thanks, for all your hard work and keep it up!

@Unknwon Ok, final question: When will this approval happen or who's responsible for this? If this process is documented somewhere, then please forgive me and just point me there.

I have mentioned, you take votes or mention on the comment.

@Unknwon But highest votes are not always used?  See above.

I don't know which you are talking without Crowdin link.
No activity recently, I could miss that one.

@Unknwon See here: https://crowdin.com/translate/gogs/1/enus-de#379
I think there are many more like this.

Same rule: you take votes or mention me on the comment. I'll check on them.

@Unknwon Hmm, Ok, I will continue to vote. But you already seem to have missed at least 3 votes on this one: It's +3:-2 votes. Still the one with -2 is approved. Maybe I still don't get it. Sorry.


Server crashed for unknown reason

Spark link
https://spark.lucko.me/moRU1jRGnk
Stack trace
https://paste.gg/p/anonymous/055ec0ef5a1f43e399ec8b17afc4d127
Actions to reproduce (if known)
No response
Purpur version
Current: git-Purpur-2016 (MC: 1.20.1)*
Previous: git-Purpur-2015 (MC: 1.20.1)

You are running the latest version

Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[ ] I ticked all the boxes without actually reading them
[X] My version of Minecraft is supported by Purpur.

Other
We're encoutering some random crashes, they appear to be occuren on an Player#asyncTeleport, but I'm not rly sure about that.
We're running this server behind a Velocuty proxy.
None of the unkowen Plugins like Prime*, doing world modifiing tasks, The only ones that I'm awere of are P2 and FAWE.

          @WiIIiam278 We have verified the issue on both paper and Purpur

Originally posted by @nicolube in https://github.com/WiIIiam278/HuskSync/issues/152#issuecomment-1653540878


Added group_vars containing the default service groups for all enviro…

…nments.
Added following tasks:

Create default service groups.
Remove non essential service groups.

LGTM

LGTM.