Linux --version doesn't work

Hello,
The Linux version of Mattermost desktop respond to --version or -v with correct information, we receive "unknow".
Environment

Operating System: Debian 10
Mattermost Desktop App version: 4.6.1

Steps to reproduce
For tgz archive :
wget https://releases.mattermost.com/desktop/4.6.1/mattermost-desktop-4.6.1-linux-x64.tar.gz
tar xvfz mattermost-desktop-4.6.1-linux-x64.tar.gz
cd mattermost-desktop-4.6.1-linux-x64
./mattermost-desktop -v
unknow
./mattermost-desktop --version
unknow

For Appimage :
wget https://releases.mattermost.com/desktop/4.6.1/mattermost-desktop-4.6.1-linux-x86_64.AppImage
chmod +x mattermost-desktop-4.6.1-linux-x86_64.AppImage
./mattermost-desktop-4.6.1-linux-x86_64.AppImage  -v
unknown
./mattermost-desktop-4.6.1-linux-x86_64.AppImage --version
unknown

Expected behavior
./mattermost-desktop  -v
4.6.1
./mattermost-desktop --version
4.6.1

Actually I can't check the current version of desktop client and the script who look for new release need to download all the time.
Thanks

what was the last version that worked for you?
I've created a jira ticket for it

I don't know, before I have a script who download the latest stable from github. Now I would like to optimize this to download the archive only if version differ from latest release.

I'll take this one.


Textmate: fix Razor expression classification inside of HTML tags using injection

This fixes some problem created in dotnet/vscode-csharp#7051. In this new approach, we use the "injection" feature of the textmate grammar so that we don't have to change the grammar of html.

Noticed a changed when writing code in Blazor and just popping in to say awesome job, works great 👍

Thanks for confirming, and the kind words @GlacierFox
FYI @jordi1215 😄


⚠ ClamAV has degraded performance

In 906d2fb, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 282 ms

Resolved: ClamAV performance has improved in 710c8f9.


Blog Page for your community

Hey @SahityaRoy , Currently we do not have blogs , we will add once we discuss within our community members and then release a form for so.


Accept long data type as input for quantize - wmagent

wmagent branch version of #8461

 Fix #8462 - Accept long data type as input for quantize - wmagent


Selector.select removes duplicates

Hi,
I'd like to apologize in advance if there already is such an issue, but I didn't find one.
I've been using Jsoup for HTML parsing, specifically this page: http://darksouls.lljk.net/1.04/weapon/shortsword
and noticed that Selector.select() method removes duplicates, which, I believe is the wrong behavior. Actually, it doesn't remove them, it just adds them to LHS, which doesn't store duplicate values. Here's an example of what I'm talking about:
Elements reinforcementLevelRows = doc.getElementsByClass("level0");
reinforcementLevelRows.forEach(System.out::println);
Elements tds = reinforcementLevelRows.select("td");
tds.forEach(td -> System.out.println(td.text()));

and here's the result:

 +0
 78
 0
 0
 0
 20
 C
 51
 C
 51
 –
 –
 50.0
 10.0
 35.0
 35.0
 40.0
 40.0
 40.0
 38.0
 32

+0
78
0
20
C 51
–
50.0
10.0
35.0
40.0
38.0
32

As you can see, in the second case, after selecting of "td"-s, it just removed cells which have duplicate values. I can suggest replacing LHS with ArrayList or smth here:
Selector.java, Line 46:
 LinkedHashSet<Element> elements = new LinkedHashSet();

Can't reproduce. I'm using latest version 1.11.3 and executing your code correctly shows duplicated elements:
 +0
 78
 0
 0
 0
 20
 C
 51
 C
 51
 –
 –
 50.0
 10.0
 35.0
 35.0
 40.0
 40.0
 40.0
 38.0
 32

+0
78
0
0
0
20
C 51
C 51
–
–
50.0
10.0
35.0
35.0
40.0
40.0
40.0
38.0
32

Can't reproduce. I'm using latest version 1.11.3 and executing your code correctly shows duplicated elements:
 +0
 78
 0
 0
 0
 20
 C
 51
 C
 51
 –
 –
 50.0
 10.0
 35.0
 35.0
 40.0
 40.0
 40.0
 38.0
 32

+0
78
0
0
0
20
C 51
C 51
–
–
50.0
10.0
35.0
35.0
40.0
40.0
40.0
38.0
32

Oh, alright, looks like I'm using an outdated version (1.8.3), which for some reason popped out as the first link on google search for maven repo. I guess I'll just upgrade my dependency then. Thank you for a  quick response and have a nice day!


Merge develop branch to main

All the commits should be already reviewed via the various PRs agains develop.
A new release can be tagged after the merge.

Hi @dbast thanks for this, all looks good :)


Create Privacy Page and Story Page

Assigned on 3/20 however issue created 3/27

Make sure pages are responsive
Make sure pages have all technical requirements


Editing generic method type argument comment doesn't update quick fix list

Similar to #26911
I have code like

then I add the type parameter

Then I want to convert the inline function style to a block body using the quick fix.
I need to edit something else (adding a space) to get the quick fixes shown

It's not important to me to get this fixed if you think this issue is not serious enough to invest the time until proper generic method syntax becomes available.
Feel free to close.
Dart VM version: 1.19.0-edge.db48637ab39adfc79e981a60ab9e3d0b59671323 (Wed Jul 20 13:09:21 2016) on "linux_x64"

@leafpetersen Any word on how soon we can add generic method support?

Syntactic support in the VM should happen this quarter - I believe that is the last remaining platform.


[Feature Request] Support for JSON operator

unsupported feature: (->>) JSON operators not supported

We have this issue in Dolt so this is a dupe:
https://github.com/dolthub/dolt/issues/5662
This is equivalent to JSON_UNQUOTE(JSON_EXTRACT()) which will work. Please follow the Dolt issue.


Query by percentage rather than count for "Disordered regions"

From Alison Pidoux:

It’s great that there’s an option to search for disordered regions in proteins in the Advanced Search.  I wonder if it would be possible to modify it so that rather than specifying eg ‘number of DRs > 6’ (which can give back proteins that have several tiny DRs, but not a protein with only a few long regions) it could give the option of ‘DR is > xx% of protein’ or ‘DR of > xx amino acids’ ?  I don’t know if that’s easy or hard to ask for or if there’s some reason that number of DRs is a good idea.

My prototype is here: https://desktop.kmr.nz/query
We should work on the wording.

Maybe something like "Percentage of residues in disordered regions" would be clearer ?

All the changes are checked in, including a percentage query option for coiled-coils.  I'll close this issue in the morning if it all looks OK.


Navigator is not defined when using Server-side rendering

I am using the Controlled component in my GatsbyJS app.
import {Controlled as CodeMirror} from 'react-codemirror2';

import 'codemirror/lib/codemirror.css';
import 'codemirror/mode/css/css';

When I run gatsby build, I get the following error:
  16 | // Kludges for bugs and behavior differences that can't be feature
  17 | // detected are enabled based on userAgent etc sniffing.
> 18 | var userAgent = navigator.userAgent;
     | ^
  19 | var platform = navigator.platform;
  20 |
  21 | var gecko = /gecko\/\d/i.test(userAgent);

  WebpackError: navigator is not defined

  - codemirror.js:18
    ~/codemirror/lib/codemirror.js:18:1

  - codemirror.js:11 userAgent
    ~/codemirror/lib/codemirror.js:11:1

  - codemirror.js:14 Object.<anonymous>
    ~/codemirror/lib/codemirror.js:14:2

  - css.js:6 CodeMirror.defineMode.inline
    ~/codemirror/mode/css/css.js:6:1

  - css.js:11 Object.defineProperty.value
    ~/codemirror/mode/css/css.js:11:2

  - index.tsx:12 Object.assign.i
    src/components/pages/Editor/Textarea/index.tsx:12:1

  - index.tsx:10 Object.<anonymous>
    src/components/pages/Editor/index.tsx:10:1

I noticed that in the docs, it says, "react-codemirror2 will prevent rendering in absence of navigator."

Fixed with:
let CodeMirror = null;

try {
    CodeMirror = require('react-codemirror2').Controlled;
    require('codemirror/mode/css/css');
} catch(e) {
    console.error(e);
}

import 'codemirror/lib/codemirror.css';

The issue is not with React-codemirror2, rather codemirror/mode/**/*.js


Added Euclidean-Algorithm-Calculator

Fixes Issue🛠
Closes #1080
Description👨💻
This pull request adds a new feature to the website: a "Euclidean Algorithm Calculator." The calculator allows users to compute the Greatest Common Divisor (GCD) and the Least Common Multiple (LCM) of two integers entered.
This includes the following changes:

A user-friendly interface with the "two required numbers" input field and a calculate button.
A visually appealing dark-themed background for the webpage.

No additional dependencies are required for this change.
Type of change📄

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
This feature has been tested by:

Manually giving several inputs and verifying that the correct GCD and LCM are displayed.
Ensuring the gradient "calculate" button and the container render correctly on various screen sizes.
Checking the readability of text within the container against the dark-themed background.
Verifying that the site functions correctly across different browsers (Chrome, Edge).

Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] I have added a demonstration in the form of Screenshots.
[x] I am an Open Source Contributor

Screenshots/GIF📷

Sample Output 1 :-

Sample Output 2 :-

Hello Project Admin,
I've updated the new feature that was assigned to me, Can you merge my PR into yours @Rakesh9100
Also please add Levels to the Issue because then only the project will be considered for my points.
Thank you


Update README.md

Regular install don't work in google colab, so I made little change to fix it.

Exact error:
git clone https://github.com/ai-forever/sage.git
cd sage
pip install .
ERROR: Directory '.' is not installable. Neither 'setup.py' nor 'pyproject.toml' found.
// cd just don't work correct in colab now


Deliverable: data structures

Data structures for:

[x] packages
[x] reviews
[ ] reviewers
[x] editors

This is for this initially-funded stage an editors dashboard, so data on reviewers are out of scope. This issue is otherwise now completed.


Minor_correction

In file model/model.py at line number 73 spelling of implemented needs to be corrected. Those who are new to this community and want to contribute can work on this issue.

Hi, I wanted to start contributing here so I fixed this easy one first. #1154


Update Export Services docs to reflect new application functions message bus architecture

Overviews:
https://docs.edgexfoundry.org/Ch-Intro.html#export-services-layer
https://docs.edgexfoundry.org/Ch-ExportServices.html
Quickstart:
https://docs.edgexfoundry.org/Ch-QuickStart.html#connecting-an-application
API Walkthrough:
https://docs.edgexfoundry.org/Ch-WalkthroughExporting.html
API Reference:
https://docs.edgexfoundry.org/Ch-APIExportServicesClientRegistration.html

@mhall119 , the application functions are separate from the export distro so there shouldn't be any updates export distro docs.

@lenny-intel do you think the export-services layer of the architecture documentation needs to be changed? Or would it be enough to just add new documentation about the application functions architecture at the same level a the export-services docs?

@mhall119 core-data and export-distro architecture documentation needs to be updated for the new message bus abstraction, but not for app functions. Yes there needs to be new documentation about the application functions architecture at the same level a the export-services docs. @tsconn23 , you agree?

Yes there needs to be new documentation about the application functions architecture at the same level a the export-services docs

Agreed


CORE-2192: Can't generate SQL scripts from Offline Oracle databases. In ...

Fix for CORE-2192, which prevents me from using liquibase:updateSQL from the Maven plugin. OracleDatabase's setConnection is always calling getWrappedConnection for the DatabaseConnection, but OfflineConnection doesn't implement it.

Thanks!


Check whether assembly version and application version are in sync

There seems to be no easy way to automatically update the application version in the solution file whenever the product version is updated in the assembly info. So it might easily happen that those two versions get out of sync. Therfore it would be nice if there was a check at start-up to ensure that both versions are in sync.

Implemented by https://github.com/makmu/outlook-matters/commit/c4a5895caa9833ea08a44cbb9c086ddf57f096bb


Upgrade hadoop dependencies for hdfs plugin to mitigate CVEs.

Description
Hadoop 2.8.5 has been reported to have CVEs (https://bugzilla.redhat.com/show_bug.cgi?id=1883549). We need to upgrade this to 2.10.1. This also overwrites the change in #645 which updated the hadoop-minicluster to 3.3.0
This upgrade also requires two additional dependencies, woodstox-core and stax2-api. Add the dependencies along with the sha1s, licenses and notices.
Issues Resolved
Relates #646
Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.
Signed-off-by: Rabi Panda<EMAIL_ADDRESS>

start gradle check

start gradle check


Users and Auth: Unable to add user to project when using third-party auth

Setup

Rancher version: v2.10
Rancher UI Extensions:
Browser type & version: FF, Chrome

Describe the bug
Unable to add user to project when using third-party auth.

admission webhook "rancher.cattle.io.projects.management.cattle.io" denied the request: metadata.annotations: Invalid value: "field.cattle.io/creator-principal-name": annotation is immutable

To Reproduce

enable third-party auth (I used GitHub OAuth)
go to cluster dashboard
clusters > projects/namespaces
add a user to an existing project (I added GitHub user: yonasberhe23 to Default project)
click save

Result
User is unable to add a user to a project when using third-party auth.
Expected Result
User is able to add a user to a project when using third-party auth.

Screenshots
https://github.com/user-attachments/assets/1d4cd44b-1280-455d-af4e-ed2532a9b6c3

Additional context
I also noticed that the user is unable to remove a project member that was previously added by the admin user to the project.

Probably related to https://github.com/rancher/dashboard/issues/11852. To expand on error message, norman resource annotation cannot change if they contain cattle.io

Was only applicable to projects not created by the signed in user as the annotation did not change.
Annotation in question is immutable and should only be set on creation of project (rather than edit as well)

User can successfully add a user to a project when using third-party auth.
Tested in:

Rancher	v2.10-e2d0b42c91253243696d36f09e8e6e23cc6fb0db-head
Dashboard master 3c808b4


search on sys.prefix for the PROJ data directory

[x] Closes #387
[x] Tests added
[x] Fully documented, including history.rst for all changes and api/*.rst for new API

Couldn't reproduce the problem, but I assume this will be a helpful addition nonetheless and will hopefully solve the problem mentioned in the issue.

Thanks!


New animation proposal: squash

Inspired in the first animation principle, Squash & Stretch, follow a new animation proposal, just called Squash:
http://codepen.io/FelipeMartinin/pen/dYXabb
Pull request link:  https://github.com/daneden/animate.css/pull/467

If you are taking comments, it's very jarring and a single animation happens way to quick to know what happened.

Agreed, this is a little aggressive with a lot going on. How does this animation look applied to, say, a text-heavy form? Think about the use case first and try experimenting with something simpler :)

Closing this issue and relaying my feedback on the original PR.


Allow unlinking Github user profile from local app

I think this is all we can do here.
According to Github docs, when deauthorizing a token, they require both a client_id, AND a client_secret, which we decidedly don't have since we're using the device flow.
Octocrab also doesn't seem to have corresponding functionality. So we nuke the key and move on.

No, there's no revocation for GH tokens on self serve at the moment. Likely we'll want another endpoint to do that.


Fix CMake warnings + drop old style export of library

Fixed CMake warnings reported by Jean-Christophe Fillion-Robin (jcfr) at #21
Dropped old style export of Elastix library files.

We could rebase this branch on develop and see if it works on azure.

The CMake warnings are fixed and the old style export of the elastix library has been dropped by other pull requests:

Pull request #282 "COMP: Fix CMake 3.x warnings policy CMP0033 and CMP0007 (issue #21)"
Pull request #263 "ENH: Upgrade cmake_minimum_required from 2.8 to 3.10.2"

It is unclear whether the SuperElastix Linux link errors (https://github.com/SuperElastix/elastix/issues/21#issuecomment-405554289) would still occur with the latest elastix library version. But we have no clue and no resources to address them any time soon. This pull request (#46) is obsolete anyway now.


lncli: add command for publishtransaction

This pr adds a rpc command for PublishTransaction for the wallet sub-server. As is, this feature is only available though the rest interface. This pr was requested by #5438.
Command will look like: lncli wallet publishtx <hex-encoded_raw_transaction> --label="name" were the flag label is optional.

To test this feature I fetched raw transactions from https://github.com/btcsuite/btcwallet/blob/6ab9b615576f4a86c31bcaef9eddee060b36da00/wallet/wallet.go#L3482.

Added a comment in the release notes.


the error of the builded step of centernet-tensorrt

when i build the centernet-tensorrt, i got an error shown in the below.
/CenterNet-TensorRT/src/decode.cu(170): error: identifier "compAndSwap" is undefined
detected during instantiation of "void bitonicLocalSortIndices(T *, size_t *, int, size_t, size_t, int) [with T=float]"

in decode.cu(170), i change the compAndSwap to compAndSwapIndices, then i finish the build of centernet-tensorrt. i want to know this change is right for your code?

不好意思，我又遇到一个问题。当我在onnx->tensorrt engine过程中设置batch_size=32，在pose_infe.cpp中运行得到的binding维度打印结果还是batch=1，打印结果如下：
Create the engine from weights/multi_pose.trt successfully!
i=0 tensor's name:input dim:(1,3,512,512) size:786432
i=1 tensor's name:wh dim:(1,2,128,128) size:32768
i=2 tensor's name:hps dim:(1,34,128,128) size:557056
i=3 tensor's name:reg dim:(1,2,128,128) size:32768
i=4 tensor's name:hp_offset dim:(1,2,128,128) size:32768
i=5 tensor's name:hm dim:(1,1,128,128) size:16384
i=6 tensor's name:hm_hp dim:(1,17,128,128) size:278528
这个batch的错误会导致后面CUDA内存分配出现问题。请问您有什么好的想法吗？

不好意思，我又遇到一个问题。当我在onnx->tensorrt engine过程中设置batch_size=32，在pose_infe.cpp中运行得到的binding维度打印结果还是batch=1，打印结果如下：
Create the engine from weights/multi_pose.trt successfully!
i=0 tensor's name:input dim:(1,3,512,512) size:786432
i=1 tensor's name:wh dim:(1,2,128,128) size:32768
i=2 tensor's name:hps dim:(1,34,128,128) size:557056
i=3 tensor's name:reg dim:(1,2,128,128) size:32768
i=4 tensor's name:hp_offset dim:(1,2,128,128) size:32768
i=5 tensor's name:hm dim:(1,1,128,128) size:16384
i=6 tensor's name:hm_hp dim:(1,17,128,128) size:278528
这个batch的错误会导致后面CUDA内存分配出现问题。请问您有什么好的想法吗？

Currently , the tool onnx-tensorrt in this repo. does not support the dynamic shape, which means the batch size was determined when converting the onnx model. However, I do successfully implement a version that supports the dynamic shape, and I can give u some tips:

make sure that the version of TensorRT is 7.x .
modify the main.cpp in onnx-tensorrt , and u can refer to the TensorRT developer guide to see how to use the dynamic shape.
once successfully converted the onnx model with dynamic shape into the engine, u just need to bind the dimension shape in the runtime and enjoy the speedup !

in decode.cu(170), i change the compAndSwap to compAndSwapIndices, then i finish the build of centernet-tensorrt. i want to know this change is right for your code?

u are right. this is a little syntax bug.

不好意思，我又遇到一个问题。当我在onnx->tensorrt engine过程中设置batch_size=32，在pose_infe.cpp中运行得到的binding维度打印结果还是batch=1，打印结果如下：
Create the engine from weights/multi_pose.trt successfully!
i=0 tensor's name:input dim:(1,3,512,512) size:786432
i=1 tensor's name:wh dim:(1,2,128,128) size:32768
i=2 tensor's name:hps dim:(1,34,128,128) size:557056
i=3 tensor's name:reg dim:(1,2,128,128) size:32768
i=4 tensor's name:hp_offset dim:(1,2,128,128) size:32768
i=5 tensor's name:hm dim:(1,1,128,128) size:16384
i=6 tensor's name:hm_hp dim:(1,17,128,128) size:278528
这个batch的错误会导致后面CUDA内存分配出现问题。请问您有什么好的想法吗？

Currently , the tool onnx-tensorrt in this repo. does not support the dynamic shape, which means the batch size was determined when converting the onnx model. However, I do successfully implement a version that supports the dynamic shape, and I can give u some tips:

make sure that the version of TensorRT is 7.x .
modify the main.cpp in onnx-tensorrt , and u can refer to the TensorRT developer guide to see how to use the dynamic shape.
once successfully converted the onnx model with dynamic shape into the engine, u just need to bind the dimension shape in the runtime and enjoy the speedup !

i use the original method to create the engine and  the code "int maxBatchsize = context->getEngine().getMaxBatchSize();" is applied to get the batchsize. after doing the above, i use batchsize to replace the "dim.d[0]" to get the right size. however, when i  take 3 same images as input, the output of this application is error. the first image input is correct but the other 2 image are error. do you comfirm your code with multi-batch inputs?

不好意思，我又遇到一个问题。当我在onnx->tensorrt engine过程中设置batch_size=32，在pose_infe.cpp中运行得到的binding维度打印结果还是batch=1，打印结果如下：
Create the engine from weights/multi_pose.trt successfully!
i=0 tensor's name:input dim:(1,3,512,512) size:786432
i=1 tensor's name:wh dim:(1,2,128,128) size:32768
i=2 tensor's name:hps dim:(1,34,128,128) size:557056
i=3 tensor's name:reg dim:(1,2,128,128) size:32768
i=4 tensor's name:hp_offset dim:(1,2,128,128) size:32768
i=5 tensor's name:hm dim:(1,1,128,128) size:16384
i=6 tensor's name:hm_hp dim:(1,17,128,128) size:278528
这个batch的错误会导致后面CUDA内存分配出现问题。请问您有什么好的想法吗？

Currently , the tool onnx-tensorrt in this repo. does not support the dynamic shape, which means the batch size was determined when converting the onnx model. However, I do successfully implement a version that supports the dynamic shape, and I can give u some tips:

make sure that the version of TensorRT is 7.x .
modify the main.cpp in onnx-tensorrt , and u can refer to the TensorRT developer guide to see how to use the dynamic shape.
once successfully converted the onnx model with dynamic shape into the engine, u just need to bind the dimension shape in the runtime and enjoy the speedup !

i use the original method to create the engine and the code "int maxBatchsize = context->getEngine().getMaxBatchSize();" is applied to get the batchsize. after doing the above, i use batchsize to replace the "dim.d[0]" to get the right size. however, when i take 3 same images as input, the output of this application is error. the first image input is correct but the other 2 image are error. do you comfirm your code with multi-batch inputs?

Do u understand my reply?  u should manually modify the main.cpp in the onnx-tensorrt, and currently this version is not released.

i modify the main.cpp in the onnx-tensorrt, but i meet another error. [PluginV2Ext]: PluginV2Layer must be V2DynamicExt when there are runtime input dimensions. this error means i should change the DCNv2 plugin to inherit the PluginV2DynamicExt ? do you realize the plugin inheritig from PluginV2DynamicExt ?


Wednesday 19-07-2023 roll call

Roll Call!
Leave a comment with these two things:

A summary of your study using only emojis
Something you'd like to share, anything goes! (within respect)

emoji cheat sheet

Present!

Good morning 🌺

hi

good morning

good morning

Good Morning

good morning

Good morning !

Hi

Good morning


Is possible to generate any length hash of shake128 and shake256

In this link https://www.openssl.org/docs/man1.1.1/man3/EVP_shake256.html,

The SHAKE-128 and SHAKE-256 Extendable Output Functions (XOF) that can generate a variable hash length.
Specifically, EVP_shake128 provides an overall security of 128 bits, while EVP_shake256 provides that of 256 bits.

In FIPS-202

An extendable-output function (XOF) is a function on bit strings (also called messages) in which
the output can be extended to any desired length. The two SHA-3 XOFs are named SHAKE128
and SHAKE256. The suffixes “128” and “256” indicate the security strengths that these two
functions can generally support, in contrast to the suffixes for the hash functions, which indicate
the digest lengths. SHAKE128 and SHAKE256 are the first XOFs that NIST has standardized.

But it seems the digest length of EVP_shake128 is fixed to 128, and EVP_shake256 is fixed to 256.
How could I get a digest which length is longer than 128 with EVP_shake128 (or longer than 256 with EVP_shake256)?
Thank you.

Use the EVP_DigestFinalXOF() function

Thank you, @kroeckx, it works!


Is there a compilable version with Octets?

On the google docs page, https://dnp3.github.io/docs/guide/current/features/features/, it says that there is Octet support with DNP3. Is there a branch or version with the compilable demos and octet support?

Yes, the 2.x version supports Octet String (group 110/111).
For an outstation example, look here. It updates an Octet String.
On the master side, you will receive updates through the ISOEHandler here


fix(sqlite): do not ignore API misuse errors from sqlite3_finalize

Good idea. Panicking when the connection is incorrectly dropped means a programming error somewhere.


Error when adding new site (nitro apply)

When trying to mount a new folder / site I keep getting this error:
umount failed: The following errors occurred:
"=>" is not mounted
"=>" not found in database

I'm not sure where these arrow functions are coming from...
Using 1.0.0-beta.3

Hi @dhmax88, I have seen this a few times. Can you run the following commands and provide the output?

nitro context
nitro info

That should give me a good indicator of whats happening.

Hi @jasonmccallister
I destroyed and re-initialised the box, which seems to have resolved the issue for now. It seems to happen when the box has been stopped/paused and then restarted – so I'll report back when / if it happens again.
Current results of your commands:

nitro context

Using config: /Users/david/.nitro/nitro-dev.yaml
------
php: "7.4"
mounts:
  - source: ~/Sites/MUD/OMCT
    dest: /home/ubuntu/sites/local.omct.com
  - source: ~/Sites/DUET/MMC Ventures
    dest: /home/ubuntu/sites/local.mmc.com
databases:
  - engine: mysql
    version: "5.7"
    port: "3306"
sites:
  - hostname: local.omct.com
    webroot: /home/ubuntu/sites/local.omct.com/deploy/public
  - hostname: local.mmc.com
    webroot: /home/ubuntu/sites/local.mmc.com/deploy/public

nitro info

Name:           nitro-dev
State:          Running
IPv4:           <IP_ADDRESS>
Release:        Ubuntu 18.04.4 LTS
Image hash:     3b2e3aaaebf2 (Ubuntu 18.04 LTS)
Load:           0.08 0.26 0.28
Disk usage:     4.5G out of 38.6G
Memory usage:   660.9M out of 3.9G
Mounts:         /Users/david/Sites/MUD/OMCT          => /home/ubuntu/sites/local.omct.com
                    UID map: 501:default
                    GID map: 20:default
                /Users/david/Sites/DUET/MMC Ventures => /home/ubuntu/sites/local.mmc.com
                    UID map: 501:default
                    GID map: 20:default

@dhmax88 ahh ok, I have to give credit to @lukeholder for this one. There is an issue with multipass removing mounts during macOS sleep (https://github.com/canonical/multipass/issues/1531).
If this happens again, you can run nitro restart and it should resolve itself!

I don't think this issue is resolve.
I've tried nitro restart but still getting the error... output below:
Webroot for "local.omct.com" matches
Webroot for "local.mmc.com" matches
Webroot for "local.omct-next.com" does not match, found "/etc/nginx/sites-available/local.omct-next.com"
Removing mount /Users/david/Sites/DUET/MMC from nitro-dev
Mounting ~/Sites/DUET/MMC Ventures to nitro-dev
Mounting ~/Sites/MUD/OMCT-NEXT to nitro-dev
Adding site local.omct-next.com to nitro-dev
umount failed: The following errors occurred:
"=>" is not mounted
"=>" not found in database
Error: exit status 2

php: "7.4"
mounts:
  - source: ~/Sites/MUD/OMCT
    dest: /home/ubuntu/sites/local.omct.com
  - source: ~/Sites/DUET/MMC Ventures
    dest: /home/ubuntu/sites/local.mmc.com
  - source: ~/Sites/MUD/OMCT-NEXT
    dest: /home/ubuntu/sites/local.omct-next.com
databases:
  - engine: mysql
    version: "5.7"
    port: "3306"
sites:
  - hostname: local.omct.com
    webroot: /home/ubuntu/sites/local.omct.com/deploy/public
  - hostname: local.mmc.com
    webroot: /home/ubuntu/sites/local.mmc.com/deploy/public
  - hostname: local.omct-next.com
    webroot: /home/ubuntu/sites/local.omct-next.com/public

Name:           nitro-dev
State:          Running
IPv4:           <IP_ADDRESS>
Release:        Ubuntu 18.04.4 LTS
Image hash:     3b2e3aaaebf2 (Ubuntu 18.04 LTS)
Load:           0.85 0.52 0.21
Disk usage:     4.5G out of 38.6G
Memory usage:   355.6M out of 3.9G
Mounts:         /Users/david/Sites/MUD/OMCT          => /home/ubuntu/sites/local.omct.com
                    UID map: 501:default
                    GID map: 20:default
                /Users/david/Sites/DUET/MMC Ventures => /home/ubuntu/sites/local.mmc.com
                    UID map: 501:default
                    GID map: 20:default


Send the entire level object

A recent commit https://github.com/clappr/clappr/commit/aa4effd3b524475d2a161ceca682ca9b105c68dc changed the label of the levels from the height (e.g. 720p) to the bitrate in kbps.  This affects level selector plugins.  We should allow the plugins to decide how to format their own labels by sending everything (including height and bitrate) to the PLAYBACK_LEVELS_AVAILABLE event.

This makes sense to me!


Update Gingko to latest HEAD release and revert hack to handle SIGTERM in e2e tests

Changes to Ginkgo this pulls in:

Fixed an issue where a pending test within a focused context (or a focused test within a pending context) would skip all other tests.
Be more consistent about handling SIGTERM as well as SIGINT
maybe a few other minor bugfixes?

I should note that I tested locally:

all e2e tests ran successfully
I sent a SIGTERM to the test runner and it handled it correctly

LGTM.


can't login

hey its telling me
instagram call checkpoint for this action

I can't enter to my account and mesagges on IG:DM beacause there is an error ´´No configs given´´ and I don't know how I have to do


Siege Engines are described or known as numbers (At some lines)

Idk if you guys know that or some of you only do , Raiding siege engines are known as 3 digits in the AIC file , and I Think some AIVs can recruit a BattleRam for example (As a kind of harassment or raiding) , But for now the only found codes for the siege engines are
FireBallista : 358
Catapult : 190
So if you open the AIC file and go to the lines from Unknown115 to Unknown122 You will find that every AI has atleast one of these whether it be 190 or 358 , And even some AIs has them both ,
while you can clearly see at this line (		SiegeEngine1	= unit   )  that the siege engines are not written in codes or numbers !
Is there any probability of changing those numbers to units (text) or finding new codes for other siege engines?

Both properties are actually numbers. Alternative text input was introduced for known numbers. So you can actually use numbers or text for SiegeEngine1-8.
https://github.com/Sh0wdown/UnofficialCrusaderPatch/blob/master/UnofficialCrusaderPatch/AICharacters/PersonalityEnums/SiegeEngines.cs
For the harassment property the optional text input hasn't yet been implemented, but will be in the next version. Moreover if you compare the two enums in the link, the numbers vary, so you cannot just use the numbers of the SiegeEngine property for the Harrassment properties.

Yeah I see , They differ as you said .

This issue can be closed, as it will be solved in the next update, right?

@Sh0wdown Siege engines should now be written in after updating it with the updater, right?

I'm closing this issue.


cannot set 'is_timeout' attribute of immutable type 'TimeoutError'

python 3.10.3 运行会出现这个哦
Traceback (most recent call last):
File "", line 1027, in _find_and_load
File "", line 1006, in find_and_load_unlocked
File "", line 688, in load_unlocked
File "", line 883, in exec_module
File "", line 241, in call_with_frames_removed
File "D:\python\python3.10\lib\site-packages\funboost_init.py", line 11, in
from funboost.assist.user_custom_broker_register import register_custom_broker
File "D:\python\python3.10\lib\site-packages\funboost\assist\user_custom_broker_register.py", line 2, in
from funboost.publishers.base_publisher import AbstractPublisher
File "D:\python\python3.10\lib\site-packages\funboost\publishers\base_publisher.py", line 22, in
from funboost.concurrent_pool import CustomThreadPoolExecutor
File "D:\python\python3.10\lib\site-packages\funboost\concurrent_pool_init.py", line 14, in
from .custom_evenlet_pool_executor import CustomEventletPoolExecutor
File "D:\python\python3.10\lib\site-packages\funboost\concurrent_pool\custom_evenlet_pool_executor.py", line 7, in
from eventlet import greenpool, monkey_patch, patcher, Timeout
File "D:\python\python3.10\lib\site-packages\eventlet_init.py", line 17, in
from eventlet import convenience
File "D:\python\python3.10\lib\site-packages\eventlet\convenience.py", line 7, in
from eventlet.green import socket
File "D:\python\python3.10\lib\site-packages\eventlet\green\socket.py", line 4, in
import('eventlet.green.socket_nodns')
File "D:\python\python3.10\lib\site-packages\eventlet\green_socket_nodns.py", line 11, in
from eventlet import greenio
File "D:\python\python3.10\lib\site-packages\eventlet\greenio_init.py", line 3, in
from eventlet.greenio.base import *  # noqa
File "D:\python\python3.10\lib\site-packages\eventlet\greenio\base.py", line 32, in
socket_timeout = eventlet.timeout.wrap_is_timeout(socket.timeout)
File "D:\python\python3.10\lib\site-packages\eventlet\timeout.py", line 166, in wrap_is_timeout
base.is_timeout = property(lambda _: True)
TypeError: cannot set 'is_timeout' attribute of immutable type 'TimeoutError'

python3.10 时候，自己pip install eventlet 安装到更高的最新版本就好了。


Error when reproduce the result  :  No OpKernel was registered to support Op 'CudnnRNNParamsSize'

any tips to solve the error?
python 3.6 + tensorflow 1.4.1
GPU :  Nvidia  Tesla M40

I had the same issue. I fixed it by uninstalling tensorflow and then installating tensorflow-gpu==1.4.0

Fixed by last commit


[FLINK-28250][Connector/Kafka] Fix exactly-once Kafka Sink out of memory

What is the purpose of the change
The goal of this PR is to address a memory leak where the Kafka Producer does not get cleaned up after kafka commit is successful.
Brief change log
7304f9d: recycles producer after successful kafka commit by adding a finally block to the existing try/catch.
3d68e4e: adds unit test
c5fead3: removes finally block so that we don't recycle producer in the case of RetriableException
Verifying this change
Manually verified the change by running a 1 node cluster with 1 JobManagers and 1 TaskManagers, running the following job for several hours without any memory issues:
public class FlinkTest {
    public void main(String[] args) throws Exception {
        StreamExecutionEnvironment env = StreamExecutionEnvironment.getExecutionEnvironment();
        KafkaSource<String> source = KafkaSource.<String>builder()
            .setBootstrapServers("localhost:9092")
            .setTopics("input")
            .setGroupId("my-group" + System.currentTimeMillis())
            .setStartingOffsets(OffsetsInitializer.earliest())
            .setValueOnlyDeserializer(new SimpleStringSchema())
            .build();
        DataStream<String> sourceStream = env.fromSource(
            source,
            WatermarkStrategy.forMonotonousTimestamps(), "Kafka Source");
        KafkaRecordSerializationSchema<String> serializer = KafkaRecordSerializationSchema.builder()
            .setValueSerializationSchema(new SimpleStringSchema())
            .setTopic("output")
            .build();
        Properties sinkProps = new Properties();
        sinkProps.put("[transaction.timeout.ms](http://transaction.timeout.ms/)", 180000);
        KafkaSink<String> sink = KafkaSink.<String>builder()
            .setBootstrapServers("localhost:9092")
            .setKafkaProducerConfig(sinkProps)
            .setRecordSerializer(serializer)
            .setDeliverGuarantee(DeliveryGuarantee.EXACTLY_ONCE)
            .build();
        sourceStream.sinkTo(sink);
        env.enableCheckpointing(10000);
        env.getCheckpointConfig().setCheckpointTimeout(60000);
        env.getCheckpointConfig().setCheckpointingMode(CheckpointingMode.EXACTLY_ONCE);
        env.getCheckpointConfig().setMinPauseBetweenCheckpoints(500L);
        env.getCheckpointConfig().setTolerableCheckpointFailureNumber(1);
        env.execute("tester");
    }
}

Took heap dumps using jmap tool and noticed the taskmanager memory was stable. Link to the relevant mailing list thread: https://lists.apache.org/thread/c86cd8qyqb6qxy639hkzbozkwv2qxk84.
Heap dump analysis
Before patch using Flink 1.15.0 (note similar behavior observed with 1.15.1-r):
~30min:
~90min:
After patch using Flink 1.15.1-r:
~30min:
~90min:
This change was also noted in the JIRA ticket and validation was included in the ticket description.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@flinkbot run azure

Thanks for the patch @charles-tan ! LGTM. Could you squash all commits together? I'll merge the PR then.
Also we need to backport the patch to 1.15. Could you help to create the backport PR?

@PatrickRen thanks for the review! I've squashed the commits and created a backport patch for 1.15 here.


Spark is a trademark of the Apache Software Foundation

see: https://spark.apache.org/trademarks.html

We shouldn't let this slide too long to avoid any sort of animosity towards the project. The action items would be:

Choose a new name for the project (tough one).
Update any mentions in the code comments or in documentation/readme to referee to Spark properly (eg. Apache Spark(tm)).
In the future guarantee/enforce any mentions are done properly.

Is understandable we still want to maintain some sort of reference to the original project, so we can mention it by framework inspired by Apache Spark(tm) .  The profile of the project will be keep increasing as we do work and publish articles etc. so I wouldn't worry about it too much.

Good plan of action. From what I've read referring to trademarks by their name without the trademark logo ("tm") is sufficient. Apache have guidance for how to refer to their trademarks.

Yes, these issues should be considered early

I think "fleck" and "flake" nicely parallels the noun usage of spark since rusted iron flakes off producing flecks of rust.  Also, rust starts where some protective layer flakes or flecks off, which parallels the verb usage.  I think both imply "numerous" or "clusters" as well as or better than spark.  "Fleck: Rust framework inspired by Apache Spark(tm)"

Thanks for the suggestion @burdges . Fleck, Bolt, Streak, Light, Photon are the options for now. If we can reach a consensus on one of these, we can go ahead and rename the project.

Photon is not an option - it's a famous GUI subsystem from QNX and a registered trademark:

Photon microGUI® for QNX® Neutrino®

But this is not a competing/similar service to Photon. So does it still count as a trademark infringement?

I don't know, just pointing out there is a well-known trademark with the same name.

Fleck is very creative, but I also like Streak; those two are my favorite so far :)

@rajasekarv maybe is a good time to move forward with this before we start adding a lot of configuration, documentation stuff etc. we would have to rename later? Vega is my favorite so far, or nano would also do.

Going with Vega then. Will change it today

Awesome. This finally happened.
Just out of curious, after the renaming would the old repo URL on the internet pointing to native_spark still works? Will they be redirected to the new Vega repo automatically by Github?

Old repo URL will get redirected to this repo.


Revert "Updates toString calls affected by change in method signature"

Description
Reverts https://github.com/opensearch-project/security/pull/2452 in response to the reversion from core (https://github.com/opensearch-project/OpenSearch/pull/6431).
This retains the change in 2.x, but reverts for 2.6.
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (2.6@b2f2a6d). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 7cdda5c differs from pull request most recent head bdb3c9d. Consider uploading reports for the commit bdb3c9d to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@          Coverage Diff           @@
##             2.6    #2460   +/-   ##
======================================
  Coverage       ?   60.96%
  Complexity     ?     3246
======================================
  Files          ?      258
  Lines          ?    18129
  Branches       ?     3231
======================================
  Hits           ?    11052
  Misses         ?     5501
  Partials       ?     1576

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Merging this now as all checks, but MacOS have succeeded. MacOS is not required and has been removed on main.


Add option to automatically discover database names

This feature adds an option to dynamically retrieve the available databases / DSN's from a single DSN. This allows users to only configure a single DSN per database server and scrape metrics for all databases on the configured servers.

I feel like the failing build is not related to the changes made in this Pull Request, or am I missing something?

Coverage decreased (-3.5%) to 58.389% when pulling 52fb61b1421ac0fd51712d1f6972fac62212c435 on sqills:auto-discover-databases into 07dd31dea6e4fe97cbe8b82f79f5ef0b3cf7052d on wrouesnel:master.


Thermal Evaporation Base Causing Timeout

Please use the search functionality before reporting an issue. Also take a look at the closed issues!
Issue description:
I am Playing on my server and the Thermal Evaporation Base is causing me to timeout every time i place one of the blocks and i do not know how to fix this.
Steps to reproduce:

Place the Thermal Evaporation Base block
Done

Version (make sure you are on the latest version before reporting):
Forge 1.12.2 (<IP_ADDRESS>24)
Mekanism: 1.12.2-<IP_ADDRESS>6
Other relevant version: 1.12.2
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
[gist/pastebin/etc link here]

Duplicate of #5354


Worms do not get skeletonised, all trajectories get filtered in SKE_INIT step

Hi,
I did not manage to reach successfully pass the SKE_INIT (and further) steps in the analysis of my video. TRAJ_CREATE and TRAJ_JOIN work well and the results are gathered in plate_worms (I checked through my matlab script), while SKE_INIT step does not seem to allow any trajectories to reach skeletonisation: trajectories_data is empty. I tried many different sets of parameters but without success.
params:
test.json
sample video (I needed to convert it to mp4, which could have corrupted the resolution slightly)
https://github.com/user-attachments/assets/820ee59d-0925-4cf4-9c40-f9cb16521d48
Many thanks!

UPDATE: It seems that the issue was given by the line
"nn_filter_to_use": "pytorch_default",
replacing it to:
"use_nn_filter": false,
solved the issue. Would you mind giving a bit of explanation what this line does?

The neural network was trained to recognise single worms to reduce the chance of tracking non-worm objects like tracks in food or thin shadows.  But it was trained only using data from our lab.  If your imaging system makes different-looking videos (which is likely) then the neural network will falsely thing every object is a non-worm object and filter them out.  Thanks for the quick follow-up with the solution!


Cannot build Android after installing bugsnag - [cannot find symbol, report.getNotifier, Task :bugsnag-react-native:compileDebugJavaWithJavac FAILED]

Issue
After installing bugsnag, android builds no longer work.

Environment
Library versions:

├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>

cocoapods version (if any) (pod -v):

iOS/Android version(s): 27

simulator/emulator or physical device?: pixel emulator

debug mode or production?: debug

[ ] (iOS only) [BugsnagReactNative start] is present in the
application:didFinishLaunchingWithOptions: method in your AppDelegate
class?

[ ] (Android only) BugsnagReactNative.start(this) is present in the
onCreate method of your MainApplication class?

Error messages:
> Task :bugsnag-react-native:compileDebugJavaWithJavac FAILED
.../node_modules/bugsnag-react-native/android/src/main/java/com/bugsnag/BugsnagReactNative.java:384: error: cannot find symbol
        report.getNotifier().setName(NOTIFIER_NAME);
              ^
  symbol:   method getNotifier()
  location: variable report of type Report

.../node_modules/bugsnag-react-native/android/src/main/java/com/bugsnag/BugsnagReactNative.java:385: error: cannot find symbol
        report.getNotifier().setURL(NOTIFIER_URL);
              ^
  symbol:   method getNotifier()
  location: variable report of type Report

.../node_modules/bugsnag-react-native/android/src/main/java/com/bugsnag/BugsnagReactNative.java:386: error: cannot find symbol
        report.getNotifier().setVersion(String.format("%s (Android %s)",
              ^
  symbol:   method getNotifier()
  location: variable report of type Report
3 errors

i'm having this issue too after updated to version 2.11.0 and i'm using<EMAIL_ADDRESS>

Thanks for the report @mshibl - we're looking into it as a priority. Do you still see this behaviour after performing a clean build i.e. deleting the .gradle/ folder in your android directory, and deleting node_modules then running npm install? We've only been able to see this intermittently so far, and suspect that it's a dependency caching issue.
Would you also be able to attach the output of ./gradlew app:dependencies, where app is the name of your module, and the areas of your package-lock.json which reference "bugsnag"? This will help us see whether the correct dependencies are being used.

+1

Deleting ~/.gradle/caches and ~/.gradle/daemon didn't helped in my case ;/ Cache was cleared and new binaries were fetched but i still have this error.

@ahilles107 Would you also be able to attach the output of ./gradlew app:dependencies, where app is the name of your module, and the areas of your package-lock.json which reference "bugsnag"? This will help us see whether the correct dependencies are being used.

+1

+1

Deleting ~/.gradle/caches and ~/.gradle/daemon works for me.

for me too


ReplaceBlock don’t work with blockstates

OTG Plugin or Mod

[ ] OTG Mod for Forge
[x] OTG Plugin for Spigot

OTG Version

[ ] 1.12.2_v8_r1
[ ] 1.12.2_v7_DEV_r26
[ ] 1.12.2-v6
[ ] 1.12.2-v5
[ ] 1.11.2-v8
[ ] 1.11.2-v9
[ ] 1.10.2-v21
[x] 1.16.5-0.0.15

Forge or Spigot Versions
1.16.5
What other plugins are you running?
Only Essentials, WorldGuard, WorldEdit, Essentials-Spawn
Description
When the plugin loads my biomes, Replace Blocks containing blocks that have custom blockstates don't work due to a formatting error:
Example:
[01:38:37 ERROR]: [otg] [Configs] The value "(minecraft:gravel,minecraft:sand,50,63),(minecraft:sandstone,minecraft:note_block[instrument=banjo,note=1,powered=false],64,256),(minecraft:gravel,minecraft:note_block[instrument=banjo,note=2,powered=false],64,255),(minecraft:note_block[instrument=banjo,note=1,powered=false],minecraft:stone,63,72)" is not valid for the setting ReplacedBlocks in SandyDunes on line 116: Replace parts must be in the format (from,to) or (from,to,minHeight,maxHeight)
I hope that the plugin considers blockstates without generating errors and that my biomes don't have this kind of hitch.
Steps to reproduce

Sets a replaceblock to a biome, which uses blocks with custom blockstates.

for example: (minecraft:sandstone,minecraft:note_block[instrument=banjo,note=1,powered=false],64,256)

Restart the server with scrive logs for Configs
You will notice the format error in the console and that the replacedBlock will not work on the server

Hey @SpiderDeluxe , thanks for reporting this. I assume the comma's in the block data are causing the parsing issues, any OTG settings that use comma's may be affected. Will look at this for 0.0.16.

This may affect anything that uses materials and comma's:

ReplacedBlocks
BiomeConfig resources
SurfaceAndGroundControl
BO functions (Block() etc)

Hey @SpiderDeluxe , thanks for reporting this. I assume the comma's in the block data are causing the parsing issues, any OTG settings that use comma's may be affected. Will look at this for 0.0.16.

Thanks to you and the OTG team working on this great plugin, I look forward to new feedback

Will just keep this issue open until we fix the problem, thanks!

Should be fixed for 0.0.16, to be released on the discord asap.


when i use navigator ,null is not an object

[tid:com.facebook.react.RCTExceptionsManagerQueue] Unhandled JS Exception: null is not an object (evaluating 'internalInstance._currentElement')

Can you post some code where this is happening?

You can even host it on React Native Playground.

@facebook-github-bot stackoverflow

You can find my ask about DrawerLayoutAndroid on StackOverflow... If you can help me.
 http://stackoverflow.com/questions/36722237/show-my-drawerlayoutandroid-onpress-rightbutton-from-navigationbarroutemappe  Just posted here for more visibility, don"t want to create a new issues, just for a link... Thank's !

Awesome, left a comment, let's keep discussion up there


abaplint.json and defaults

Seems like set of rules is applied even if a limited set defined in abaplint.json.
I mean I have a abaplint.json with several explicit rules with, say, does not include avoid_use. And still avoid_use is applied. This is unexpected.
Would be nice to have this logic:

if no rules specified - use default set
if rules specified - use just them
probably it would be good to have a template set like in eslint and then re-define specific rules. So that I define in abaplint.json that I want to use "default set" but certain rules are redefined in addition.

what is the value of applyUnspecifiedRules in your example? https://github.com/larshp/abapGit/blob/master/abaplint.json#L7

you're right, I forgot about this feature. Sorry.
Closing.


Deprecate @arguments decorator.

Argument contract placeholder was implemented in #602
We could remove @arguments decorator which is already a duplicate functionality.

Superseded by #627


ref/use_rct_export_view_property

Refactored AdView to follow the React Native ViewManager framework:

use RCT_EXPORT_VIEW_PROPERTY in iOS for the properties
add a wrap-up function to destroy adView
update the Javascript AdView

Updated, please review again.  I'll run more testing to make sure no regression.

@alhiwatan - Let me know when ready for re-review, I see some comments not addressed.

Have updated the Android side.   Please run the review.

Updated, please review again, especially for Boolean of placement and customData.   Should we modify the iOS?   Javascript can't set a null value for properties, so if placement and customData are not set by Javascript, they should be nil.   The user has to an empty string to reset them.

Updated to have default values for the props.    Without the default values in iOS, both autoRefresh and adaptiveBannerEnabled will be set to false.

Have updated.  Please review.

Updated, please review again.

Again updated, please review.

Updated, hopefully, this is good to go.

Updated..., please review.

Updated and tested.  Please review.


Update pickle_core.py

#57

Currently the issue itself is not valid - in my opinion - and the suggested change is unexplained and undocumented.
Please feel free to reopen if this is ammended.


Usage of White Space

Hi guys,
in our projects we need a very efficient use of space for our power users. Lists, Cards, Headlines, etc should be very sensitive about whitespace, fontsizes and the use of margins and paddings.
Eg. for Cards we would prefer like 10px padding instead of 40px.
Will there be a "Efficient-Mode" that can be turned on. That would be awesome. A hard override needs to be testest for every component after every update. So not ideal ;-)
Thank you,
André

Feature: Spacing.
It will have two modes:

Normal: Default; No action needed, use of wide margins and paddings.
Compact: Reduced margins, paddings and separating spacing to have more information density on the same screen real state.

Normal
Compact

@bachofner
This solution will suffice your Uses Cases for EC?

@bachofner I'm collecting screen shots of Hybris interfaces on Invision. I can get you an account and added to our project or you can just throw them somewhere and I can grab them. Don't want to post them here of course. But really anything you have would be useful, some screens with lots of fields, edit modes, creation modes.
I want to create a library to help us in our research and discovery.

This request has been expanded gonna be addressed at #751


Is there an option to initialize all the register with random number?

For the register without Init values, the firrtl spec is to initialize them with random number like real circuits. Current treadle seems to initialize all registers with zeros, it this true? Is there an option to initialize all the register with random number?

@davidmlw not currently but I think this would be very easy to implement in treadle. Is there any particular strategy, I could put a random (correctly sized) value in every symbols, or could constrain the randomizing to just registers or other specific kinds of symbols. Perhaps there would be some separate rules for memories

@davidmlw Please take a look at PR #215 which attempts to do this kind of randomization. It would be great if you could do a quick review

Closing because I think #215 addresses this


Update next crawl time calculation to match expectations

Rationale
Previous calculation caused the crawler to visit folders every nine hours (on average). Every 12 hours +/- 10% seems sufficient.
https://www.labkey.org/home/Developer/issues/issues-details.view?issueId=48415

I the intention was that changes would be noticed within 12 hrs, but if this solves some problem great.

I the intention was that changes would be noticed within 12 hrs, but if this solves some problem great.

Average just makes it easier to understand if we're indexing more than expected (i.e., if there are 1,100 folder then we see 1,100 folders reindexed every 12 hours, on average).


chore: Improves the native filters UI/UX - iteration 1

SUMMARY
Improves the native filters UI/UX - iteration 1.

Sets a fixed height for the modal
Splits the configuration and scoping into different tabs
Inverts the order of the buttons of the confirmation message
Removes the x icon of the confirmation message
Fixes the tests to account for the async nature of the screen

PS: A future PR will handle the case of automatically selecting a tab/collapse when an error occurs.
@villebro @rusackas @junlincc
This work is part of the Native dashboard filter project
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
https://user-images.githubusercontent.com/70410625/118834003-49585e80-b898-11eb-855a-34a301d46c71.mov
https://user-images.githubusercontent.com/70410625/118834020-4c534f00-b898-11eb-900f-aeb15c8dafba.mov
TEST PLAN
1 - Execute all tests
2 - All tests should pass
ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Great work @michael-s-molina; Going to review shortly!

Thanks for the first step. @michael-s-molina  how many iteration we need to get to the progressive reveal action?

Thanks for the first step. @michael-s-molina how many iteration we need to get to the progressive reveal action?

@junlincc I'm planning to work on it in iteration 3. Iteration 2 is to create the collapse panels.


Create a how to play page explaining the rules and restrictions

Finished, but it could use a refactor


Join failed [Security]

Hi Experts,
In my test environment, a Silicon Labs BRD4180A as Commissioner, the firmware built from Simplicity Studio, and the version is
> version
SL-OPENTHREAD/<IP_ADDRESS>_GitHub-3b79cb0d0; EFR32; Jun 17 2022 19:01:51
Done

And a NRF52840 USB dongle as Joiner, the firmware built from https://github.com/openthread/ot-nrf528xx/commit/27a002fbc6abc36974a25666282705c4b73d723c
> version
OPENTHREAD/thread-reference-20200818-1510-g3b79cb0d0; NRF52840; Jun 17 2022 14:40:28
Done

The two devices are very close together, but the Joiner join failed at last.
Here is the failed log and sniffer data.
joining failed.zip
When I try the firmware built from ot-efr32 - https://github.com/openthread/ot-efr32/commit/31e9fc8606d9a68cd79868d2b3169992168a7398, It can Join Success.

Hi @lancersky the trace you attached in joining failed.zip doesn't seem too helpful, I see the following packet logs:

However, I did see reassembly timeout errors.  To resolve these, I would recommend you increase the baud rate on your OTBR.
To change baud rate, you should:

Add ?uart-baudrate=460800 at the end of the radio URL in /etc/default/otbr-agent
The URL should look like so: spinel+hdlc+uart:///dev/ttyACM0?uart-baudrate=460800

You should also rebuild your RCP image with the increased baud rate. Please see the following recommendation in our application note [AN1256: Using the Silicon Labs RCP with the OpenThread Border Router (https://www.silabs.com/documents/public/application-notes/an1256-using-sl-rcp-with-openthread-border-router.pdf):

Closing based on proposed baud rate change.


Add guidance on closing PRs

Objective

Bevy has a large number of open PRs in the backlog.
When there are problems with these PRs, contributors, maintainers and SMEs are all often left wondering what to do with them and are reluctant to close them fully.
Instead, PRs that are a bad idea end up sititng with Controversial status forever, while severely rotten PRs have Adopt-Me on them when it would be more appropriate to simply remake them.

Solution

Add clear guidance on how and when to close PRs.

Looks great! Two questions:

Should adopt-me PRs be left closed or open?
I've closed three or four issues (and I think a PR) unilaterally by posting a comment saying something like "this looks like it should be closed, I will close after 7 days if no one objects" and then doing that when no one says anything. Some guidance to org-members as to if that's ok or not would be appreciated.

Merging following a blessing from @cart here.


PUBSUB: unsubscribe without handler reference

Hi :-)
following up on an issue in interface-js-ipfs-core I'm creating this PR.
Also created PR in js-ipfs-http-client accordingly.
This would help in cases like mine where my implementation is not able to support references to a handler function.
I will add the final url in the README to the method documentation once I merge the final PR to interface-js-ipfs-core since its pointing nowhere now.
Edit:
Opened a PR on interface-js-ipfs-core

Per https://github.com/ipfs/interface-js-ipfs-core/pull/437#issuecomment-461403871, I think we should just overload unsubscribe.
cc @vasco-santos

Also, when you update, can you fix linting npm run lint?

@jacobheun thanks, I commented in interface-js-ipfs.
Will do the linting, thanks for the hint I forgot.

I think we should just overload unsubscribe.

Subscribe this point


incorrect documentation

following the documentation for Intune Shared PC configuration profile on your page does not reflect the actual options in Intune-Devices-Windows-Configuration profile-Settings catalog-Shared PC. In your documentation there is 18 settings wheras in the Intune console there is only 14 options. I am specificaly looking for Sleep settings.

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

ID: e3329b98-7e66-aa67-d27e-d14471cf63b3
Version Independent ID: e6c88589-b252-4e9c-6016-8b53154dce96
Content: SharedPC CSP - Windows Client Management
Content Source: windows/client-management/mdm/sharedpc-csp.md
Product: windows-client
Technology: itpro-manage
GitHub Login: @vinaypamnani-msft
Microsoft Alias: vinpa

@ipavlina thank you for the feedback. If the missing settings are not exposed in the settings catalog, you will have to create a separate custom policy or a different profile type that contains the configuration you need.


Compact menu: should remove DOM nodes after use

If you inspect the compact menu, I can see that all the top level menu items are in the dom even though I did not click:

Imho DOM nodes that are not visible by default should be created lazily and removed after being used.

Yea these are there as an artifact of the original menu. They are never shown in the compact case. The reason they are there normally and set to visibility hidden is to ensure we can do math on the dimensions of the full menu bar. Keeping them around made it easy to switch between menu modes but cleanup can be done.


[Feature Request] Add support for spot instances

Hi
Can you please add support to mix on-demand and spot instances, as there're already args like on_demand_base_capacity, on_demand_percentage_above_base_capacity , spot_instance_pools , spot_instance_remedy ... in https://www.terraform.io/docs/providers/alicloud/r/ess_scaling_group.html.
Thanks.

HI  @yongzhang Thanks for your feedback and the release 1.5.0 has supported them. Please check it.

@xiaozhu36 Thanks!


How to correctly specify URL to self-hosted node in AnchorRequest?

I'm experimenting with an ION node and using ion-tools to interact with it. I'm having trouble figuring out how to send an anchor request to it, though.
I can successfully resolve a DID using
let response = await ION.resolve('did:ion:test:EiClWZ1MnE8PHjH6y4e4nCKgtKnI1DK1foZiP61I86b6pw', {
    nodeEndpoint: 'http://localhost:3000/identifiers/',
});

But when I try to submit an anchor request I get an error. Here's what I'm trying to do:
const requestBody = did.generateRequest();
const request = new ION.AnchorRequest(requestBody, {
    challengeEndpoint: 'http://localhost:3000/api/v1.0/proof-of-work-challenge',
    solutionEndpoint: 'http://localhost:3000/api/v1.0/operations',
});
let response = await request.submit();

Basically I'm changing https://beta.ion.msidentity.com to http://localhost:3000.
The error I get is Error: Get challenge service not available. I can only assume the paths to the endpoints are incorrect. I have also tried http://localhost:3000/proof-of-work-challenge and http://localhost:3000/operations.
What is the correct path to the challenge and solution endpoints when running your own node?

Was trying to solve my own problem by looking through the code. I see where the operations path is configured in the router, but I don't see anything related to the challenge.

Hello folks!
I am trying to get started with this by running some examples with a local (dockerized) node.
If ion-tools is no longer supported, what should I use?
How do I sent did operations created by ion-tools to my local node?
Where can I find technical documentation to get started as a developer?
Thanks!


[OTHER] Incorrect Map

What would you like to share?
Good morning, I was going through Map tab and noticed an incorrect map of my country Bharat(India)
I request you to use a correct map.
Additional information
No response

Please can you share more information about what is incorrect and what it should be? Please use screenshots for both

This is the incorrect map

Here is the correct one

We do not make the map, it is provided to us by the tool we use leaflet. You will need to submit it to that project for any changes


Add support for multiple operators via several improvements

This PR makes the following changes:
scripts/gen_operator_csv.py (8fd92c1c88adc2cae821422e2c669554f658641e)

Fix typo, add guardrails around missing assets
If the manifests directory is missing, complain and quit. If there are no Deployments detected, complain and quit.

Makefile (4d99ecdc6fa2fc7b642363e4a9dddfdc04fc8e8e)

In all places substitute filesystem glob matching in $(SOURCE_DIR)/**/ to iterate over $(OPERATORS) and determine the name of the repository from the list.
Adds guardrails around certain variables that become part of the running environment due to eval $(make -C operatordir) env:

Tests are made to ensure the variables the make target cares about are found
Unsets variables so that subsequent loop iterations (and other make targets) aren't polluted with previous values

Makes more explicit the testing around failures: Instead of || (... && exit), check the previous command's exit value in an if-then-else-style control block.
Created an env target, which echos nothing, so that functions.mk's /process_template doesn't emit an error.
Most targets are now silent; they do not print the Bash that make is running. (To be more verbose try make SHELL="bash -x")
Consolidated multiple mkdir -p manifests into its own make target, manifestdir, with accompanying variable MANIFESTDIR

PR into this PR to update readme: https://github.com/lisa/osd-operators-registry/pull/1

Changes in this repo are good, so:
/approve
But the config-alertmanager-operator repo has problems, noting here:

inconsistent make target for build
missing make target for push
manifest filenames not catalogsource friendly (format is .<kind.lowerCase>.yaml)

Bottom line is merging this breaks generation of the catalog.  Need the operator to be updated before we can merge. So:
/hold

Filenames can be handled here.  I'll put in a quick PR for that.  See https://github.com/openshift/osd-operators-registry/issues/15

Filename renames here: https://github.com/openshift/osd-operators-registry/pull/16

This will conflict with #17.

/lgtm

/lgtm


[Event Chang Request] page 343 "Check Credit Limit" - OnShowWarningOnBeforeExitValue

Could you please include the sender on the following event?
//+Change
//-[IntegrationEvent(false, false)]
[IntegrationEvent(true, false)]
//-Change
local procedure OnShowWarningOnBeforeExitValue(var Customer: Record Customer; ExitValue: Integer; var Result: Boolean; var IsHandled: Boolean; var Heading: Text[250]; var SecondHeading: Text[250]; var NotificationID: Guid)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 46009.


net/http: unexpected behavour of client.Do() with cookie jar

What version of Go are you using (go version)?

$ go version go1.12.7 linux/amd64

Does this issue reproduce with the latest release?
Yes, stable 1.13.6 version has same problem too.
What operating system and processor architecture are you using (go env)?
Any platform with any enviroment
What did you do?
code is here: https://play.golang.org/p/M7q1GhoW1wV
So, for my opinion, the problem is in r.AddCookie() function, which don't check request header is nil and forcely call Headers methods. :thinking:
What did you expect to see?
My projects must not panicing for situations like this :+1:
i think, that request initializing with &http.Request{} instead http.NewRequest() is more convenient, cause http.NewRequest() does too many things. Declaration of request object looks more readable and simple.
What did you see instead?
They are panicing! :open_mouth:

If you initialize an http.Request without NewRequest, you must create the map yourself. This is Working As Intended.

i understand that now i need to call NewRequest(), but as i said, it is not obvious, and code becomes bloated. I wrote in detail why this is inconvenient and illogical, so I do not think that the problem is resolved by NewRequest().

The issue is not with adding three lines of checking code to client.Do. The issue is that it sets a precedent for net/http to do nil-checking any functions that interact with a request. In general for Go, if a struct has members that need to be initialized, the responsibility is either for the package to provide a factory function like NewRequest or for the user of the package to initialize its members themselves.

I agree with you, didn't think about it at this first, this situation is really not a concern of stdlib. well, what if we can add some function similar to newrequest which will not require me parse the same uri twice time?
i mean, this is really important for me, cause i think, that same code must be simple and concise, adding line with call request initializer and handling error feels a bit overheaded.


Slider vals

values for sliders
including time vals, which are now too long, need width
ipywidgets 5 (with work around)

ok, i think this is ready for discussion and review now

I have just tried that out, and it is beautiful.  Bug is fixed, slider value is a date value, everything works and it is lovely.

Coverage increased (+0.002%) to 99.481% when pulling 2ab974b19cc8fa2efb2528b14881a3d0c05bfc3e on marqh:slider_vals into adfc12b2f22c049521abc03d278b0baf4448e382 on SciTools:master.

Coverage increased (+0.002%) to 99.481% when pulling bd6e3c44a729e2e0bbcc8904fd95cb167c17a780 on marqh:slider_vals into adfc12b2f22c049521abc03d278b0baf4448e382 on SciTools:master.


Dyim42 patch add local address attribute

Add support to listen-address. This will support dhcp server address to listen on specific address.

I don't think this PR is required.  You should just be able to define node['dhcp']['parameters']['local-address'] in either a wrapper cookbook, role or as a node attribute directly on the node.  You don't need to add it as a default attribute to this cookbook in order to declare it.


Kmloverlay marker latlng on click

Hi,
I need to get latlng of kml marker on click but not able to do so. Any help appreciated.
map.addKmlOverlay({
  'animation': false,
  'preserveViewport': true,
   'url': "www/files/Dog_Walking_Areas.kml"
    },  function(kmlOverlay){
    kmlOverlay.on(plugin.google.maps.event.OVERLAY_CLICK, function(overlay, latLng) {
    console.log(latLng);//undefined
    if (overlay.type === "Marker") {
      overlay.showInfoWindow();

    }
  });
})

Unfortunately latLng is not given if you use on ios.

Thanks for reply,
Solution:
I used array of marker separated from Kml file. Then toggle marker on ovelay.


How can i set the timeout callback

As title, i want to call method of the item when it is removed because of timeout.

@STofone
It's not possible to set timeout callback in Theine(Python) currently. I'm not actively developing new features at the moment as I'm not using Python or Rust recently. The need to split the package into two languages due to Python's limitations also has been a bit frustrating to me. Tha't why my Go version of Theine is significantly more feature-rich. However, contributions are always welcome!


OnSegmentSelected event double firing

Whenever a segment is selected and the OnSegmentSelected or SegmentSelectedCommand are triggering twice. I have seen this behavior in my own app and when using the test app that is in source.
I put a breakpoint on this line and it was hit twice consistently when switching segments.

Thank you. Any idea what might cause this?
What version of Xamarin Forms are you using?

I am using Xamarin.Forms version <IP_ADDRESS>9999. No idea what could be causing it. I haven't looked too much into your implementation yet.

It looks like it is happening because setting .SelectSegment on the SegementedControl is triggering the event in addition to user interactions. Is there a a way to filter out programatic setting of .SelectedSegement?

Now, to be honest, I've not had time to look at it in more depth. Nevertheless, no I don't believe there is a way to tell the two situations apart, unless you handle it in your own code. From the controls perspective, there is no difference from a segment being set programmatically and a segment being set by the user. The same event is fired in both cases.

@1iveowl It seems that this was introduced by #50. What was the case when the selected segment event was not getting fired before this commit? I'm trying to understand why the need for that code. Is there a test case somewhere?
RaiseSelectionChanged is fired off by the native control renderer. Then the code that was added in #50 again fires off RaiseSelectionChanged.

Good catch.
This seem unnecessary:
var segmentedControl = (SegmentedControl)bindable;

I see this behavior as well. Will this be fixed soon?
Thanks for this awesome control!

@1iveowl Unless @shakehar speaks up with a reason to keep #50 do you want a PR to revert that change?

Agree. Let's revert the change and eliminate the double event.
I'm traveling at the moment and will welcome a PR.

Awesome. Thanks everyone! Can't wait for the merge/deploy


Parallel Initialization in ZooKeeperDynamicConfigSource

Creating each NodeCache in sequence was turning out to be slow with large numbers of configuration entries.
Updated to run per-node initialization on the RxJava IO scheduler, limiting the concurrency to 25-at-a-time.  Constructor of ZooKeeperDynamicConfigSource waits for all node initializations to complete.
Concurrency level is adjustable by inheritance, and overriding the appropriate protected method.

@whited looks good to me.  Who did you want this passed to for second pass?


[Question] was rust-gpu evaluated?

Hello!
Out of pure curiosity, was rust-gpu (https://github.com/EmbarkStudios/rust-gpu) instead of wgsl evaluated at some point?
I know it says in their README that the first focus is graphics shaders and compute being a second, hence I ask if you considered it and hit any sharp edges.
Again, pure curiosity!

Absolutely. I'd love to have something along the lines of rust-gpu, but it is sadly not mature enough. Another major concern was portability - WGSL is carefully designed to work well even on Vulkan 1.0, while rust-gpu requires more modern GPUs. This is a tradeoff of course, there are also advantages to being able to depend on modern features.
For evaluating shader languages, I strongly recommend implementing prefix sum. That should only be about a hundred lines of code (give or take) but shakes out a lot of the potential problems and also is a good chance to show off the abstraction capabilities of the language. In particular, I'd love to have a Rust trait to express a monoid, and a scan implementation generic on the monoid. For extra points, have that trait include methods for subgroup reduction and scan with a default implementation based on subgroup shuffle.
The currently most promising alternative to WGSL is slang. I've lightly prototyped prefix sum, but run into issues with default trait methods (called "extension" in slang). I should file issues there :)
BTW, there's nothing wrong with using issues to ask questions, but Zulip might be better for such discussions, in this case the #gpu stream.

I am sorry for opening an issue, it seemed inappropriate to me too, but there was no discussions tab and I am not on zulip (too many different messengers these days, but worry not, once xilem is ready I'll write the chosen one to rule them all! ;))
Thanks a bunch for taking time to answer and in such depth! Feel free to close and again: sorry for opening an issue when it's clearly not!


Add PWA

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md

I am:

[ ] Suggesting edits to an existing Topic page
[x] Curating a new Topic page

EDITING AN EXISTING TOPIC PAGE*
I'm suggesting these edits to an existing topic:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[ ] Content (and my changes are in index.md)

Please explain why these changes are necessary:
CURATING A NEW TOPIC PAGE

[x] I've formatted my changes as a new folder directory, named for the topic as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please explain why you think this Topic page should be curated:
PWAs are rapidly being added to many SPA frameworks out of the box or with starter kits.

Angular
Stencil
React
Vue

Image is CC0

Thanks so much for your contribution!  You rock, @abraham!

I'm with @kenchris. Maybe it would be better to remove the created_by field.

It would be nice to educate people better though

@kenchris Will happily accept PRs to update the created_by field.

@MikeMcQuaid Is the best way to just remove the created_by or could we refer to a link, like say wikipedia instead?

I think @kenchris raises a very valid point.
I have created a PR for this (#173) and explained my reasoning in (great) detail.
And let me also thank @abraham for taking the time to create this topic. 🎉 :octocat:

Awesome @TalAter!


Improve rating on https://securityheaders.com/?q=keedrop.de&followRedirects=on

Frustrating: the headers are configured in the Caddyfile but they don't appear in the response :cursing_face:

Done. It's A+ now.


feat: adds cert-utility.

closes #1869

Summary

currently, there is no standard method for creating cert chains for fulcio or tsa. the community has used an assortment of open source scripts/tools, but i thought it would be nice to have a small cloud agnostic go app to create/sign (via awskms, cloudkms, or azurekms) certificates. the smallstep crypto library is fairly comprehensive in its kms/cert capabilities.
@haydentherapper / @bobcallaway  gave the go ahead in proceeding w/ this work.
Release Note

Adds certificate utility to create and sign certificates via AWS KMS, Google Cloud KMS, or Azure Key Vault.

Documentation

Overview
This tool creates root, intermediate (optional), and leaf certificates for  Fulcio  (compliant to Fulcio's certificate requirements):

Two-level chain (root -> leaf)
Three-level chain (root -> intermediate -> leaf)

Requirements

Access to one of the supported KMS providers (AWS, Google Cloud, Azure)
Pre-existing KMS keys (the tool uses existing keys and does not create new ones)

Local Development
Clone and build the project locally:
# Clone the repository
git clone https://github.com/sigstore/fulcio

# Change to project directory
cd fulcio

# Build the binary
go build -o fulcio-certificate-maker ./cmd/certificate_maker

Usage
The tool can be configured using either command-line flags or environment variables.
Command-Line Interface
Available flags:

--kms-type: KMS provider type (awskms, cloudkms, azurekms)
--kms-region: KMS region (required for AWS KMS)
--root-key-id: KMS key identifier for root certificate
--leaf-key-id: KMS key identifier for leaf certificate
--kms-tenant-id: Azure KMS tenant ID
--kms-credentials-file: Path to credentials file (for Google Cloud KMS)
--root-template: Path to root certificate template
--leaf-template: Path to leaf certificate template
--root-cert: Output path for root certificate (default: root.pem)
--leaf-cert: Output path for leaf certificate (default: leaf.pem)
--intermediate-key-id: KMS key identifier for intermediate certificate
--intermediate-template: Path to intermediate certificate template
--intermediate-cert: Output path for intermediate certificate

Environment Variables

KMS_TYPE: KMS provider type ("awskms", "cloudkms", "azurekms")
KMS_REGION: Region (required for AWS KMS)
ROOT_KEY_ID: Key identifier for root certificate
KMS_INTERMEDIATE_KEY_ID: Key identifier for intermediate certificate
LEAF_KEY_ID: Key identifier for leaf certificate
KMS_TENANT_ID: Azure tenant ID
KMS_CREDENTIALS_FILE: Path to credentials file (for Google Cloud KMS)

Certificate Templates
The tool uses JSON templates to define certificate properties:

root-template.json: Defines root CA certificate properties
intermediate-template.json: Defines intermediate CA certificate properties (when using --intermediate-key-id)
leaf-template.json: Defines leaf certificate properties

Templates are located in pkg/certmaker/templates/.
Note: Templates use standard JSON format with code signing extensions.

Root CA -> Intermediate CA -> Leaf Certificate

any reason we should increase the number of intermediate certs to more than 1 like so?
Root CA -> Intermediate CA 1 -> Intermediate CA 2 -> Leaf Certificate

i think this is ready for 👀 now. just a couple of notes.

the following use-cases are now covered:

root ca -> leaf
root ca -> intermediate ca -> leaf

both awskms and azurekms have been tested / work, but cloudkms (e.g. gcp), has not been tested.

i think that about covers, i have some basic readmes above as well.

i think this is ready for 👀 now. just a couple of notes.

the following use-cases are now covered:

root ca -> leaf
root ca -> intermediate ca -> leaf

both awskms and azurekms have been tested / work, but cloudkms (e.g. gcp), has not been tested.

i think that about covers it, i have some basic readme/documentation above as well.
cc @haydentherapper

Thanks @ianhundere, I’ll take a look at this later this week.

Thanks @ianhundere, I’ll take a look at this later this week.

@haydentherapper resolved some lint errors / should be ready for 👀 now.

@haydentherapper / @haydentherapper
just a heads up that i tested gcpkms and it's working:
❯ ./fulcio-certificate-maker create --kms-type gcpkms --gcp-credentials-file ~/.config/gcloud/application_default_credentials.json --root-key-id projects/blah/locations/global/keyRings/sigstore/cryptoKeys/fulcio-root/cryptoKeyVersions/1 --leaf-key-id projects/blah/locations/global/keyRings/sigstore/cryptoKeys/fulcio-leaf/cryptoKeyVersions/1 --root-cert root.pem --leaf-cert leaf.pem
Your root certificate has been saved in root.pem.
Your leaf certificate has been saved in leaf.pem.

i also removed testify / updated tests to avoid pulling in further dependencies

this is ready for another 👀
cc @haydentherapper / @bobcallaway


Update Lock amounts based on slashing.

Context. We have pallets that have the ability to reduce the balance of an account's in a particular token. For simplicity, we assume a single-currency chain. This can be achieved through T::Currency::slash_reserved() or T::Currency::slash(), or anything similar.
And there are other pallets that make an assumption on a particular account having a specific amount of tokens, and throw a lock in there to ensure the balance does not go below that.
The problem. The problem is that this assumption can be easily torn apart by the former group of pallets, and if there are no communication between them, the latter group will make wrong assumptions. Note that sometimes both sides of the above scenario can even be the same pallet. For example, PhragmenElections sets a lock for a particular amount as the weight of a vote, but staking can slash that account in the meantime, potentially even killing the account entirely, whilst the vote will stay remain in PhragmenElection, with the same weight.
Potential solution. The straightforward approach to me is to allow different pallets to listen to account balance changes from other pallets. Currently, staking aggregates all slash amounts into one NegativeImbalance and reports them to the outer world. This needs to stay, but we need another similar event that is triggered per-slashed-account. For example:
enum ImbalanceType {
   Free,
   Reserved,
}
trait OnAccountImbalance<A, B> {
    fn on_account_imbalance(who: A, amount: B, type: ImbalanceType) {}
}

Any pallet that wants to listen to this event would implement the above. Staking, ElectionsPhragmen, and other pallets that have the ability to slash funds can then implement it.
Discussion. I hope someone will recommend an easier way to solve this. What I proposed can be a bit to expensive, and is quite a pain to weigh properly.
Similar: https://github.com/paritytech/substrate/issues/3088

An alternative to a hook would simply be a mechanism for people to report such discrepancies in the system and for the system itself to check there is an issue. I would consider this kind of extrinsic a "fisherman" task which ideally would reward the user for reporting the issue correctly, and charge the tx fee from them otherwise.

One of the problem is which pallet's reserved token should get slashed first?
Also most of the pallets assume their token reserved token will never get slashed / unreserved by other pallets otherwise the economic incentive will not hold. e.g. the deposit for proxy, identity, etc all provides incentive for people to purge unwanted data. but if the reserved token get slashed, the incentive will disappear.

One of the problem is which pallet's reserved token should get slashed first?
Also most of the pallets assume their token reserved token will never get slashed / unreserved by other pallets otherwise the economic incentive will not hold. e.g. the deposit for proxy, identity, etc all provides incentive for people to purge unwanted data. but if the reserved token get slashed, the incentive will disappear.

This can potentially be fixed by https://github.com/paritytech/substrate/issues/7223

for nomination pools we added a OnSlash hook which can probably be used to fix this as well.


Fix CMake enable/disable logic for Fortran API

See TriBITS manual for reference.

There is no test that demonstrate that this resolves the issue but here is how to check that it does the trick using our dev container:
[host]$ cd docker
[host]$ docker-compose up -d
[host]$ docker-compose exec dtk_dev bash
[container]# cd $TRILINOS_DIR/DataTransferKit
[container]# [ ! -d build ] && mkdir build
[container]# cd build
[container]# ../scripts/docker_cmake -D Trilinos_ENABLE_Fortran=OFF

Look toward the end of what was printed to the screen and search for:
Processing enabled package: DataTransferKit (Utils, Kokkos, Interface, BasicGeometryAdapters, IntrepidAdapters, Operators, C_API, STKMeshAdapters, MoabAdapters, LibmeshAdapters, ClassicDTKAdapters, Tests, Examples)

Make sure that Fortran_API is not listed any more.

So why did it not work before?

Variable Trilinos_ENABLE_DataTransferKitFortran_API was set to FALSE after the graph of dependencies was computed.

Thanks, could you please amend the commit to include this information? Or, create an issue with it that this commit fixes? I think it would be useful to know in the future.

Per @aprokop's comment, created issue #192 and amended commit message.

Codecov Report

Merging #191 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #191   +/-   ##
=======================================
  Coverage   95.71%   95.71%
=======================================
  Files         380      380
  Lines       28126    28126
=======================================
  Hits        26921    26921
  Misses       1205     1205

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 177f322...a1a4fcc. Read the comment docs.

@dalg24 please squash and merge yourself.


Feature/arrowcontracts

I've added contracts for Validated, Either and Option.
These contracts allow us to extract the field from these datatypes, but because of the issue with contracts not supporting generics yet, the extracted field will be Any?.
This is still a big improvement I think, and also means when contracts support generics (1.4?) we can easily update these matchers.
Current example:
val either = "foo".right()
either.shouldBeRight()
(either as Either.Right<String>).b shouldBe "foo"
(either as Either.Right<String>).b.length shouldBe 3

Example with this PR:
val either = "foo".right()
either.shouldBeRight()
either.b shouldBe "foo"
(either.b as String).length shouldBe 3

Example in the future:
val either = "foo".right()
either.shouldBeRight()
either.b shouldBe "foo"
either.b.length shouldBe 3

This partially addresses the feature request of #612

Great.
Let's merge this!
On Mon., Jun. 17, 2019, 12:16 Stephen Samuel<EMAIL_ADDRESS>wrote:

@sksamuel commented on this pull request.
In
kotlintest-assertions/kotlintest-assertions-arrow/src/main/kotlin/io/kotlintest/assertions/arrow/either/matchers.kt
https://github.com/kotlintest/kotlintest/pull/834#discussion_r294350402:

@@ -6,14 +6,20 @@ import io.kotlintest.Result
import io.kotlintest.matchers.beInstanceOf2
import io.kotlintest.should
import io.kotlintest.shouldNot
+import kotlin.contracts.ExperimentalContracts
+import kotlin.contracts.contract

+@UseExperimental(ExperimentalContracts::class)
+fun  Either<*, T>.shouldBeRight() {

contract {
returns() implies (this@shouldBeRight is Either.Right<*>)

Yes, so once contracts support generics we can just change Either.Right<*>
to Either.Right and it will be generic.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/kotlintest/kotlintest/pull/834?email_source=notifications&email_token=AAMBCI2XAH4WQ73EWYTMFJTP26TDPA5CNFSM4HYOI2NKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3XUO7I#discussion_r294350402,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMBCI4HDMG7P4DQMNTN3PLP26TDPANCNFSM4HYOI2NA
.

Please do if you're happy. We can apply this strategy to other parts too.
On Mon, 17 Jun 2019, 10:18 Leonardo Colman Lopes<EMAIL_ADDRESS>wrote:

Great.
Let's merge this!
On Mon., Jun. 17, 2019, 12:16 Stephen Samuel<EMAIL_ADDRESS>wrote:

@sksamuel commented on this pull request.
In

kotlintest-assertions/kotlintest-assertions-arrow/src/main/kotlin/io/kotlintest/assertions/arrow/either/matchers.kt

<https://github.com/kotlintest/kotlintest/pull/834#discussion_r294350402
:

@@ -6,14 +6,20 @@ import io.kotlintest.Result
import io.kotlintest.matchers.beInstanceOf2
import io.kotlintest.should
import io.kotlintest.shouldNot
+import kotlin.contracts.ExperimentalContracts
+import kotlin.contracts.contract

+@UseExperimental(ExperimentalContracts::class)
+fun  Either<*, T>.shouldBeRight() {

contract {
returns() implies (this@shouldBeRight is Either.Right<*>)

Yes, so once contracts support generics we can just change
Either.Right<*>
to Either.Right and it will be generic.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<
https://github.com/kotlintest/kotlintest/pull/834?email_source=notifications&email_token=AAMBCI2XAH4WQ73EWYTMFJTP26TDPA5CNFSM4HYOI2NKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3XUO7I#discussion_r294350402
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AAMBCI4HDMG7P4DQMNTN3PLP26TDPANCNFSM4HYOI2NA
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kotlintest/kotlintest/pull/834?email_source=notifications&email_token=AAFVSGR5LWEYJYPESMBRJJTP26TMVA5CNFSM4HYOI2NKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX3QHZY#issuecomment-502727655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFVSGQDIECOHK6Z6NSKKVDP26TMVANCNFSM4HYOI2NA
.


sendNotification function show WrongValueError when using SHA-384 and SHA-512

ISSUE
The sendNotification function gives an error - WrongValueError when attempting to use the asyncio,sendNotification function.
On debugging the issue, it seems to arise from the writeMibObject function, which errors when setting the MIb value.
LOGS
I will attach detailed logs some time later.
CURRENT Workaround
The issue does not occur with SHA-256 or lower. It seems to be a MIB issue.

The issue was with PYSNMP_USM_MIB. I noticed that it was a pysnmp specific mib and would return a ContstraintUnion value object between 8 and 32 for length of octets. This was breaking the functions that related to authenticating and committing MIB objects related to SHA-2. Increasing the constraint value fixed the issue. I will be creating a PR against this.