addAttributeKeyControl does not work for express dataobjects

The field field_set_controls in the table expressfieldsetcontrols is not set by the addAttributeKeyControl() method. Therefore the attributekeys are not visible inside the forms (because they're not associated with the fieldset).
A working solution is to change the concrete/src/Express/ObjectBuilder/FormBuilder.php... Change line 39ff from:
foreach($this->fieldsets as $fieldset) {
$builtSet = $fieldset->build($this->objectBuilder);
$builtSet->setPosition($position);
$builtSet->setForm($form);
$form->getFieldSets()->add($builtSet);
$position++;
}
to:
foreach($this->fieldsets as $fieldset) {
$builtSet = $fieldset->build($this->objectBuilder);
foreach ($fieldset->controls as $control) {
$control->setFieldSet($builtSet);
}
$builtSet->setPosition($position);
$builtSet->setForm($form);
$form->getFieldSets()->add($builtSet);
$position++;
}
Unfortunately one must access $fieldset->controls, so one must set $controls in concrete/src/Express/ObjectBuilder/FieldsetBuilder.php from protected to public. Please provide a more elegant solution to access the controls of a fieldset.

Issue has already been reported (issue #5389) and was fixed in the development branch.


Building Decentralized Backends Workshop

If you're submitting an event, please read the instructions below and fill in
the content. Otherwise just remove all the content of this issue and explain
in detail the problem you're experiencing.

####################
INSTRUCTIONS
####################
This file is a template to create new events.
Don't remove any text before the colons at the beginning of each line,
only edit what is after the colon. Example:
Don't remove the word nor colon on 'description:'

Every line starting with a hash symbol (#) is a comment. It will be ignored
and can be safely removed, including these instructions.
###############
###########
SEO
###########
The title of the page, displayed by the browser on the title of the window.
Ideally this is the same as the name of the event.
title: Building Scalable Decentralized Backends. Workshop.
Description for this event. This will be rendered as a  tag in the HTML,
and displayed on the /events page. Keep it short.
Linebreaks are ignored, but they must start with two spaces.
description: Developing DApps that are something more than just a set of smart contracts today requires a lot of effort and skill. Fluence Network creates a decentralized environment that makes DApps backend development frictionless and straightforward.
#####################
EVENT DETAILS
#####################
The name of the event you're creating.
Ideally this is the same as the title.
name: Building Scalable Decentralized Backends. Workshop.
There needs to be one hyphen before each paragraph.
Linebreaks are ignored, but they must start with two spaces.
Indentation is crucial:
Two spaces before the hyphen, four spaces before the text. No tabs allowed.
Add or remove paragraphs as needed, but remember the hyphen before each entry.
synopsis:
At the workshop, you will have a hands-on experience with Fluence Devnet — a decentralized network where you can deploy your own arbitrary code or set up a database.

The plan is simple:

Short intro on how Fluence approaches decentralization and what makes it fit for app development.
Hacking with the Devnet and a set of developer tools we've prepared

The venue is right next to EthCC conference.
Bring Your Own Laptop!
For any questions, please, reach out to the team in Discord: https://discord.gg/mt86peA

The date should be in the format year-month-day (ISO 8601).
Example: 2018-02-28
date: 2019-03-06
The date when the event ends. Can be left empty or set to the same day the
event starts.
endDate: 2019-03-06
Set the time in 24 hours format, surrounded by quotes.
Only the starting time!
Example: '18:00'
time: '18:30'
Time when the event ends. Can be left empty.
endTime: '21:00'
The URL where to akquire the tickets. Can be left empty.
tickets: https://www.eventbrite.com/e/building-scalable-decentralized-backends-workshop-tickets-57724001041
If the entrance is free, set zero (0) as the price, or leave it empty.
Don't write the currency symbol (Euro symbol will be used).
price: 0
The name of the venue where the event will be held. Can be left empty.
venue: Coinhouse
The address to link to a Google map. Please test the address on Google Maps.
Example: Skalitzer Strasse 85-86, 10997 Berlin, Germany
address: Rue du Caire 35, 75002 Paris, France
The category of the event. Valid options:
- conference
- event
- hackathon
- panel
- workshop
- party
Use only one, and don't capitalize.
category: workshop
#################
SPEAKERS
#################
There needs to be one hyphen before each entry.
Linebreaks are ignored, but they must start with two spaces.
Indentation is crucial:
Two spaces before the hyphen, four spaces before the text. No tabs allowed.
Add or remove speakers as needed, but remember the hyphen before each entry.
speakers:
# Required.
name: Dmitry Kurinskiy

# Can be left empty.
title: CTO & Co-Founder

# Can be left empty.
company: Fluence Labs

# The full URL, including http(s)://. Can be left empty.
companyURL: https://fluence.network

# Just the twitter handle, without Twitter's URL, nor the '@' symbol.
# Can be left empty.
twitter: Fluence_One

# The full URL, including http(s)://. Can be left empty.
website: https://github.com/alari

# The bio is a single line.
# Linebreaks are ignored, but they _must_ start with two spaces.
bio: Dmitry is a software engineer and technical leader, with solid expertise in complex engineering projects since 2004. Keen functional programming advocate.

DON'T MAKE CHANGES BELOW THIS LINE!

Its live! https://blockchainweek.fr/events/fluence/
Feel free to share (and mention @ComBlockWeek :)


aar R issue

When packaged as aar, R can not be found when import to project.

Seems that it's fixed. Using gradle instead of maven-andriod-plugin.


Test Issue created at Mon, 07 Jun 2021 09:22:11 GMT

Test issue body  Mon, 07 Jun 2021 09:22:11 GMT

Comment on issue at Mon, 07 Jun 2021 09:22:13 GMT


Sensitive data faqfix

faq fix

There are failing tests. Please check why (see the contribute guide for help), commit the fixes and when the tests pass, check also the actual content that it's ok, then accept.

These FAQ entries may also be helpful: https://github.com/CSCfi/csc-user-guide/blob/master/FAQ.md#my-pr-did-not-pass-the-tests-what-to-do and https://github.com/CSCfi/csc-user-guide/blob/master/FAQ.md#i-was-asked-to-review-a-pr-what-should-i-do

These FAQ entries may also be helpful: https://github.com/CSCfi/csc-user-guide/blob/master/FAQ.md#my-pr-did-not-pass-the-tests-what-to-do and https://github.com/CSCfi/csc-user-guide/blob/master/FAQ.md#i-was-asked-to-review-a-pr-what-should-i-do

These FAQ entries may also be helpful: https://github.com/CSCfi/csc-user-guide/blob/master/FAQ.md#my-pr-did-not-pass-the-tests-what-to-do and https://github.com/CSCfi/csc-user-guide/blob/master/FAQ.md#i-was-asked-to-review-a-pr-what-should-i-do

sorry. Was trying to answer.... I changed a spelling mistake in a paragraph title and did not remember to change the link to it, so the link got broken. I still need to make few more changes now to instructions we installed yesterday on a new feature in SD Desktop and will try again tomorrow.


Add ConfigurationException class

ConfigurationException is thrown when an error occurs while reading the config file. Allows us to print error messages to the user if the config is missing

[ ] Catch exception in frontend and print error if config is missing

I'm only getting the exception once and after that the stacktrace is:
java.lang.NoClassDefFoundError: Could not initialize class com.csselect.user.management.PlayerManagement
	com.csselect.API.APIFacadePlayer.login(APIFacadePlayer.java:44)
	com.csselect.API.httpAPI.Login.login(Login.java:84)
	com.csselect.API.httpAPI.Login.post(Login.java:36)
	com.csselect.API.httpAPI.Servlet.doPost(Servlet.java:102)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:660)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:741)

its because the configuaration is only created once and can therefore only throw the exception once

I'll look if I can catch the noclassdeffound exception where it's thrown and re-throw it as a ConfigurationException

I could do the same in the api, but that would be rather messy

I'll look into it because I could do it on a lower level so we don't accidentally catch other exceptions as well. (Assuming this is possible)

I looked into it and we can't get this to work by catching the exception when we create the configuration, as this kills the instantiation of the injector as well and references to the injector get saved even though instantiation didn't work out as planned. Maybe you could save in a static boolean whether the exception was thrown and check this on login as well? I don't know of a better solution as of now

in that case the static boolean should be in the injector

That's the problem, if I throw the Exception in the Injector it doesn't help, as classes like for example the playerManagement (which throws the noclassdeffound exception) save static final instances of the then malformed injector

I would propose that you just save whether the exception has been thrown and check against it before doing the login. If it wasn't thrown before you might run into the catch clause and set the boolean there so in the future when java doesn't throw the exception anymore as it already registered that configurations can't be constructed, we just avoid the problem via a simple if condition. Seems like the simplest solution for me atm

yes, but that is anything but good style.
I would say that no class should keep the injector, as that seems to be the root cause now

I'll refactor this and try again

I refactored the injector-dependent static fields and it's now working as planned on my end

can confirm, on my end as well


Angular: support for urls in Waltz without using the # anchor

This is to get around authentication proxies that may strip out all url characters after the #.

Duplicate of: #2868 ?

+1 This will be very useful

i think so yes.

#2868


Added principal diagnosis field value code selection for portfolio users

Allows customized code selections for Principal Diagnosis field values for portfolio users.

This only seems to apply to Principal Diagnosis. Would it be good to have for Diagnosis as well?


change docker image for airsonic role

What this PR does / why we need it:
This PR makes the airsonic docker image configurable.
Additionally, the default image is changed to https://hub.docker.com/r/linuxserver/airsonic because it also supports arm architectures.

Arm architectures are not something that I've said is supported, so I'm not keen on merging this without a more holistic approach across the project. I would like to support arm (and I've even hacked Ansible-NAS onto a Pi myself) - but it needs a bit more thought than just changing this one role.
Also, I'm not sure that allowing changing the whole docker image (rather than just the tag) is desirable, as it then says that any docker image is supported - which we can see from your PR here is not the case due to path and config differences.

just closing this for now


Victor VosMottor: Watch 12monthloansforbadcredit.com

Victor VosMottor requests the watch of the watch_keyword 12monthloansforbadcredit\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

12monthloansforbadcredit\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Mast in Charcoal HQ


Angular 4+ Support

Any chance this can be updated to work with Angular 4? Or an Angular 4 fork can be made?

Hi, I'm currently using it in Angular 4 projects. Did you have problems when trying to use it?

I have some errors when installing via npm because I have Angular 4.0.2 installed.
+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>`-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
I assumed it's because your package.json doesn't include Angular 4 in the semver ranges. Possibly change all angular peer dependencies ">=2.4.0" so it includes latest Angular 2 and future Angular versions.
I'm not sure why it's complaining about needing 2.2.3 instead of 2.4.0. Is there an older version on npm than github? Or does one of your other dependencies restrict the Angular version? Correct me if I'm misunderstanding something.

I see. Will try applying your suggestion as soon as I can. Anyway, in the mean time I think you can ignore the warnings and use the component normally. Let me know if this is not possible or causes troubles I'm not seeing right now.

Applying your suggestion fixed the warning messages. Thanks.

Cool, it should be fine if you updated angular to the latest 4.X version and it builds and passes all your tests. If angular happens to break your code in the far future, you could change it to a range instead. Example: (>=2.2.3 <5.0.0). I think that would work, but I'm no expert.


amplify auth add user to group Cognito trigger lambda does not have permissions to add/create group or add user to group

Before opening, please confirm:

[X] I have searched for duplicate or closed issues and discussions.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

JavaScript Framework
Angular
Amplify APIs
Authentication
Amplify Categories
auth, function
Environment information

  System:
    OS: Windows 10 10.0.19043
    CPU: (8) x64 Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz
    Memory: 6.96 GB / 15.60 GB
  Binaries:
    Node: 14.17.3 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.13 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1023.0), Chromium (93.0.961.52)
    Internet Explorer: 11.0.19041.1202
  npmPackages:
    @angular-devkit/build-angular: ~12.2.4 => 12.2.4
    @angular-eslint/builder: ~12.3.1 => 12.3.1
    @angular-eslint/eslint-plugin: ~12.3.1 => 12.3.1
    @angular-eslint/eslint-plugin-template: ~12.3.1 => 12.3.1
    @angular-eslint/template-parser: ~12.3.1 => 12.3.1
    @angular/cli: ~12.2.4 => 12.2.4
    @angular/common: ~12.2.4 => 12.2.4
    @angular/common/http:  undefined ()
    @angular/common/http/testing:  undefined ()
    @angular/common/testing:  undefined ()
    @angular/common/upgrade:  undefined ()
    @angular/compiler: ~12.2.4 => 12.2.4
    @angular/compiler-cli: ~12.2.4 => 12.2.4
    @angular/compiler/testing:  undefined ()
    @angular/core: ~12.2.4 => 12.2.4
    @angular/core/testing:  undefined ()
    @angular/forms: ~12.2.4 => 12.2.4
    @angular/language-service: ~12.2.4 => 12.2.4
    @angular/platform-browser: ~12.2.4 => 12.2.4
    @angular/platform-browser-dynamic: ~12.2.4 => 12.2.4
    @angular/platform-browser-dynamic/testing:  undefined ()
    @angular/platform-browser/animations:  undefined ()
    @angular/platform-browser/testing:  undefined ()
    @angular/router: ~12.2.4 => 12.2.4
    @angular/router/testing:  undefined ()
    @angular/router/upgrade:  undefined ()
    @aws-amplify/ui-angular: ^1.0.23 => 1.0.23
    @ionic/angular: ^5.7.0 => 5.7.0
    @ionic/angular-toolkit: ^4.0.0 => 4.0.0
    @types/jasmine: ~3.6.0 => 3.6.11
    @types/jasminewd2: ~2.0.3 => 2.0.10
    @types/node: ^12.11.1 => 12.20.21
    @typescript-eslint/eslint-plugin: 4.16.1 => 4.16.1
    @typescript-eslint/parser: 4.16.1 => 4.16.1
    aws-amplify: ^4.2.8 => 4.2.8
    eslint: ^7.6.0 => 7.32.0
    eslint-plugin-import: 2.22.1 => 2.22.1
    eslint-plugin-jsdoc: 30.7.6 => 30.7.6
    eslint-plugin-prefer-arrow: 1.2.2 => 1.2.2
    example-typescript:  1.0.0
    jasmine-core: ~3.8.0 => 3.8.0 (2.8.0)
    jasmine-spec-reporter: ~5.0.0 => 5.0.2
    karma: ~6.3.2 => 6.3.4
    karma-chrome-launcher: ~3.1.0 => 3.1.0
    karma-coverage: ~2.0.3 => 2.0.3
    karma-coverage-coffee-example:  1.0.0
    karma-coverage-istanbul-reporter: ~3.0.2 => 3.0.3
    karma-jasmine: ~4.0.0 => 4.0.1
    karma-jasmine-html-reporter: ^1.5.0 => 1.7.0
    lib:  0.0.1
    memo-parser:  0.2.1
    node-example:  1.0.0
    protractor: ~7.0.0 => 7.0.0
    protractor-example:  1.0.0
    rxjs: ~6.6.0 => 6.6.7 (7.3.0)
    rxjs/ajax:  undefined ()
    rxjs/fetch:  undefined ()
    rxjs/internal-compatibility:  undefined ()
    rxjs/operators:  undefined ()
    rxjs/testing:  undefined ()
    rxjs/webSocket:  undefined ()
    ts-node: ~8.3.0 => 8.3.0
    tslib: ^2.2.0 => 2.3.1 (2.3.0, 1.14.1, 2.1.0)
    typescript: ~4.2.4 => 4.2.4 (4.3.5)
    typescript-example:  1.0.0
    zone-mix:  undefined ()
    zone-node:  undefined ()
    zone-testing:  undefined ()
    zone.js: ~0.11.4 => 0.11.4
    zone.js/async-test:  undefined ()
    zone.js/async-test.min:  undefined ()
    zone.js/fake-async-test:  undefined ()
    zone.js/fake-async-test.min:  undefined ()
    zone.js/jasmine-patch:  undefined ()
    zone.js/jasmine-patch.min:  undefined ()
    zone.js/long-stack-trace-zone:  undefined ()
    zone.js/long-stack-trace-zone.min:  undefined ()
    zone.js/mocha-patch:  undefined ()
    zone.js/mocha-patch.min:  undefined ()
    zone.js/proxy:  undefined ()
    zone.js/proxy.min:  undefined ()
    zone.js/sync-test:  undefined ()
    zone.js/sync-test.min:  undefined ()
    zone.js/task-tracking:  undefined ()
    zone.js/task-tracking.min:  undefined ()
    zone.js/webapis-media-query:  undefined ()
    zone.js/webapis-media-query.min:  undefined ()
    zone.js/webapis-notification:  undefined ()
    zone.js/webapis-notification.min:  undefined ()
    zone.js/webapis-rtc-peer-connection:  undefined ()
    zone.js/webapis-rtc-peer-connection.min:  undefined ()
    zone.js/webapis-shadydom:  undefined ()
    zone.js/webapis-shadydom.min:  undefined ()
    zone.js/wtf:  undefined ()
    zone.js/wtf.min:  undefined ()
    zone.js/zone-bluebird:  undefined ()
    zone.js/zone-bluebird.min:  undefined ()
    zone.js/zone-error:  undefined ()
    zone.js/zone-error.min:  undefined ()
    zone.js/zone-legacy:  undefined ()
    zone.js/zone-legacy.min:  undefined ()
    zone.js/zone-patch-canvas:  undefined ()
    zone.js/zone-patch-canvas.min:  undefined ()
    zone.js/zone-patch-cordova:  undefined ()
    zone.js/zone-patch-cordova.min:  undefined ()
    zone.js/zone-patch-electron:  undefined ()
    zone.js/zone-patch-electron.min:  undefined ()
    zone.js/zone-patch-fetch:  undefined ()
    zone.js/zone-patch-fetch.min:  undefined ()
    zone.js/zone-patch-jsonp:  undefined ()
    zone.js/zone-patch-jsonp.min:  undefined ()
    zone.js/zone-patch-message-port:  undefined ()
    zone.js/zone-patch-message-port.min:  undefined ()
    zone.js/zone-patch-promise-test:  undefined ()
    zone.js/zone-patch-promise-test.min:  undefined ()
    zone.js/zone-patch-resize-observer:  undefined ()
    zone.js/zone-patch-resize-observer.min:  undefined ()
    zone.js/zone-patch-rxjs:  undefined ()
    zone.js/zone-patch-rxjs-fake-async:  undefined ()
    zone.js/zone-patch-rxjs-fake-async.min:  undefined ()
    zone.js/zone-patch-rxjs.min:  undefined ()
    zone.js/zone-patch-socket-io:  undefined ()
    zone.js/zone-patch-socket-io.min:  undefined ()
    zone.js/zone-patch-user-media:  undefined ()
    zone.js/zone-patch-user-media.min:  undefined ()
  npmGlobalPackages:
    @angular/cli: 12.2.3
    @aws-amplify/cli: 6.0.0
    ionic: 5.4.16
    typescript: 4.3.5
    yarn: 1.22.10

Describe the bug
Amplify CLI can generate an "add-to-group" lambda which allows a cognito post confirmation trigger to add a user to a cognito group automatically once confirmed. However, the cloudformation stack that Amplify creates doesn't give the lambda role enough permissions. It is missing the cognito-idp:CreateGroup, GetGroup and AdminAddUserToGroup
Expected behavior
I would expect that the Amplify CF stack creates a role with a policy that has the necessary permissions to get, create and add a user to a group in Cognito.
Reproduction steps
New Amplify project.
Added auth:
$ amplify add auth
Using service: Cognito, provided by: awscloudformation

 The current configured provider is Amazon Cognito.

 Do you want to use the default authentication and security configuration? Manual configuration
 Select the authentication/authorization services that you want to use: User Sign-Up, Sign-In, connected with AWS IAM controls (Enables per-user Storage features for images or other content, Analytics, and more)
 Please provide a friendly name for your resource that will be used to label this category in the project: OnSiteAuth
 Please enter a name for your identity pool. onsite0d7e8047_identitypool_0d7e8047
 Allow unauthenticated logins? (Provides scoped down permissions that you can control via AWS IAM) No
 Do you want to enable 3rd party authentication providers in your identity pool? No
 Please provide a name for your user pool: onsite0d7e8047_userpool_0d7e8047
 Warning: you will not be able to edit these selections.
 How do you want users to be able to sign in? Email or Phone Number
 Do you want to add User Pool Groups? Yes
? Provide a name for your user pool group: workers
? Do you want to add another User Pool Group Yes
? Provide a name for your user pool group: controllers
? Do you want to add another User Pool Group Yes
? Provide a name for your user pool group: superusers
? Do you want to add another User Pool Group Yes
? Provide a name for your user pool group: admins
? Do you want to add another User Pool Group No
√ Sort the user pool groups in order of preference · workers, controllers, superusers, admins
 Do you want to add an admin queries API? Yes
? Do you want to restrict access to the admin queries API to a specific Group Yes
? Select the group to restrict access with: admins
 Multifactor authentication (MFA) user login options: OPTIONAL (Individual users can use MFA)
 For user login, select the MFA types: SMS Text Message
 Please specify an SMS authentication message: Your authentication code is {####}
 Email based user registration/forgot password: Disabled (Uses SMS/TOTP as an alternative)
 Please specify an SMS verification message: Your verification code is {####}
 Do you want to override the default password policy for this User Pool? Yes
 Enter the minimum password length for this User Pool: 8
 Select the password character requirements for your userpool: Requires Lowercase, Requires Uppercase, Requires Numbers
 Warning: you will not be able to edit these selections.
 What attributes are required for signing up? Name (This attribute is not supported by Signinwithapple.), Phone Number (This attribute is not supported by Facebook, Login With Amazon, Signinwithapple.)
 Specify the app's refresh token expiration period (in days): 30
 Do you want to specify the user attributes this app can read and write? Yes
 Specify read attributes: Email, Name, Phone Number, Email Verified?, Phone Number Verified?
 Specify write attributes: Email
 Do you want to enable any of the following capabilities? Add User to Group
 Do you want to use an OAuth flow? No
? Do you want to configure Lambda Triggers for Cognito? Yes
? Which triggers do you want to enable for Cognito Post Confirmation
? What functionality do you want to use for Post Confirmation Add User To Group
? Enter the name of the group to which users will be added. workers
Successfully added resource OnSiteAuthPostConfirmation locally.

This automatically creates the following lambda code:
/* eslint-disable-line */ const aws = require('aws-sdk');

const cognitoidentityserviceprovider = new aws.CognitoIdentityServiceProvider({
  apiVersion: '2016-04-18',
});

exports.handler = async event => {
  const groupParams = {
    GroupName: process.env.GROUP,
    UserPoolId: event.userPoolId,
  };
  const addUserParams = {
    GroupName: process.env.GROUP,
    UserPoolId: event.userPoolId,
    Username: event.userName,
  };
  /**
   * Check if the group exists; if it doesn't, create it.
   */
  try {
    await cognitoidentityserviceprovider.getGroup(groupParams).promise();
  } catch (e) {
    await cognitoidentityserviceprovider.createGroup(groupParams).promise();
  }
  /**
   * Then, add the user to the group.
   */
  await cognitoidentityserviceprovider.adminAddUserToGroup(addUserParams).promise();

  return event;
};

Amplify push works fine.
When creating user and providing SMS code through Amplify UI components the app shows an error saying that the lambda does not have permissions for cognito-idp:CreateGroup.
I checked IAM and I can see the Role OnSiteAuthPostConfirmation-dev was created but does not have any permissions for creating groups. It works when I manually add an inline policy for GetGroup, CreateGroup and AdminAddUserToGroup.
However, why do I have to do it manually? Shouldn't Amplify CLI create the necessary CF stack and role and policy?
I can see the permissions here in parameters.json
"permissions": [
        "{\n  \"policyName\": \"AddToGroupCognito\",\n  \"trigger\": \"PostConfirmation\",\n  \"effect\": \"Allow\",\n  \"actions\": [\n    \"cognito-idp:AdminAddUserToGroup\",\n    \"cognito-idp:GetGroup\",\n    \"cognito-idp:CreateGroup\"\n  ],\n  \"resource\": {\n    \"paramType\": \"!GetAtt\",\n    \"keys\": [\n      \"UserPool\",\n      \"Arn\"\n    ]\n  }\n}"
    ],

But this never makes it into the CF stack anywhere. There is a property called permissions in the CF code, but it doesn't get used anywhere.
Code Snippet
// Put your code below this line.

Log output

// Put your logs below this line

aws-exports.js
No response
Manual configuration
No response
Additional configuration
No response
Mobile Device
No response
Mobile Operating System
No response
Mobile Browser
No response
Mobile Browser Version
No response
Additional information and screenshots
No response

Hey @oliverschenk as @chrisbonifacio mentioned, thanks for raising this 🙂 , this looks to be a duplicate of https://github.com/aws-amplify/amplify-cli/issues/7582 which also links back to https://github.com/aws-amplify/amplify-cli/issues/7576#issuecomment-865912228. Can you confirm this is what you are encountering?

Hi @josefaidt my apologies you are correct it is the same issue as #7582.


[Issue]: VX_AMD_MEDIA - Failure

Problem Description
Without FFMPEG on a test system, media sample/tests fail with ERROR: no video found in error.
Start 8: vx_amd_media_test

8: Test command: /opt/rocm/bin/runvx "-dump-profile" "file" "/opt/rocm/share/mivisionx/test/amd_media_tests/gdf/decoder_test.gdf"
8: Working Directory: /home/amd/nn-test/test
8: Test timeout computed to be: 1500
8: runvx 1.0.0
8: OK: using AMD OpenVX 1.3.0
8: OK: loaded 2 kernels from libvx_amd_media.so
8: ERROR: no video found in /opt/rocm/share/mivisionx/test/videos/AMD_driving_virtual_20.mp4
8: ERROR: failed with status = (-16:0xfffffff0:????) at /home/amd/nn-test/mivisionx/amd_openvx_extensions/amd_media/decoder.cpp#994
8: ERROR: vxVerifyGraph(graph) failed (-1:VX_FAILURE)
1/1 Test #8: vx_amd_media_test ................***Failed    0.14 sec

0% tests passed, 1 tests failed out of 1

Total Test time (real) =   0.14 sec

The following tests FAILED:
	  8 - vx_amd_media_test (Failed)

Operating System
Any
CPU
ALL
GPU
AMD Instinct MI300
Other
No response
ROCm Version
ROCm 6.0.0
ROCm Component
MIVisionX
Steps to Reproduce
runvx /opt/rocm/share/mivisionx/test/amd_media_tests/gdf/decoder_test.gdf
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

Support for FFMPEG V4+ added to TOT


Support for material3 bottom sheets

Hello! Is there any work going on to support using material3 bottom sheets?
I noticed accompanist hasn't updated to support it and from what I can tell v2 of this library also didn't mention it

Hi @cj1098
The bottom sheets used on navigation is the same that official libraries provide (not just accompanist - it was imported to official library too).
Material bottom sheets can still be used, just not as a navigation destination.
There is no work to be done here 🙂

@raamcosta Thanks for replying!
So you're saying if we move to material3's bottom sheets it's not possible to have them be destinations?
Previously when we declared our BottomSheetLayout we would say something like:
ModalBottomSheetLayout(             bottomSheetNavigator = bottomSheetNavigator,         )
Where we could pass in a bottomSheetNavigator that was provided via accompanist. In the m3 version it doesn't support a bottomSheetNavigator.
I do see that official libraries from androidx do support bottom sheets as navigation. So if we wanted to use m3 and still be able to navigate to bottom sheets we'd need to look to use one of those libraries?

Im saying those are two different kinds of bottom sheets. You can use both material and navigation bottom sheets.
Material ones cannot be destinations, they’re something you show or hide as part of another screen.
Navigation ones are something else, they’re not material 2 or 3, they are their own thing.

Gotcha. Thanks for the clarification!


Track who/when data source is updated

Description
To implement the layout presented in this design for the connection flow, we need to track the details of who connects or updates a data source connection and when this action occurs. In order to safeguard user IDs from being exposed to the connectors, we're introducing two additional fields to the data_sources table.
A follow-up PR will include a migration to backfill these new fields.

Risk

Deploy Plan
This PR requires a migration that needs to be run before it gets deployed in production.

Migration applied.

:fire:

Hi hi @flvndvd from Jan 25 :wave:
We really don't want APIError in lib/api/* (I know bad names). Will create a PR to refactor this a bit.


Multiple issues encouterred

First I would thank you for the effort made building this. It's an interresting initiative, however I would like to point the lack of documentation, not easy to understand what does what,etc.
Also would precise that I'm a java developer, no experience at all with dotnet, c#...
I managed to run the server locally on a Windows machine, using Visual Studio Code, added a settings.json file in the root folder providing all necessary variables. The android app also works and is set up.
I'm tunneling to my localhost so that I can connect using and android device.
But I'm running into many errors :

I the app, when launched,I'm getting status 202 meaning the request was accepted but it needs further processing because the status shows: "started" and the error dialog pops up.
In the server, sometimes it won't start with the following message error in the log:

A host error has occurred during startup operation 'd7e45bb4-43fc-4f9f-95b5-4d24c8dbf49f'.
Sygic.Corona.Api: Could not load file or assembly 'Microsoft.Extensions.DependencyInjection.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'. File was not found. System.Private.CoreLib: Could not load the specified file.
Value cannot be null.
Parameter name: provider

Tried everything, even installing Microsoft.Extensions.DependencyInjection.Abstractions using NuGet
When the above error message arises, I can't start server anymore,it will fail to that error again and again.
I know you may be busy with your work, but please assist

@Daziko same problem here. Would you kindly help set things up? What's the point of open sourcing a project then leaving people struggle to make it work without any documentation, guide etc? The server keeps returning status 202 whatever I do! I can't pass the welcome screen

I can't pass this screen on Android device. The server keeps returning status 202 while the device is waiting for status 200. Can you help troubleshoot this when you get a chance?
@tomaspetrasek

@Daziko same problem here. Would you kindly help set things up? What's the point of open sourcing a project then leaving people struggle trying to make it work without any documentation, guide etc? Open source projects don't look like this!!!
The server keeps returning status 202 whatever I do! I can't pass the welcome screen. Everything from both client and server is set up and running correctly (I can only assume that because no guidance is provided).

Hi this response certainly not come from "covid server". Maybe is from firebase?
I can only give you some general advice how to debug and solve issue.
So first run server locally and try to call it from postman (you can set up some brake points in order to see what's happening ). When everything is ok deploy to the azure and repeat those postman calls against deployed functions.
Second ensure you have setup application insights in your server. In application insights you can see  incoming requests and their particular response, exceptions etc. It is very helpful.
Next you can start the app and start to call server from app. If you see some issues or wrong request go to the application insights and check the logs or call locally via your tunnel and check if you hit particular part of code via brake points.

First I would thank you for the effort made building this. It's an interresting initiative, however I would like to point the lack of documentation, not easy to understand what does what,etc.
Also would precise that I'm a java developer, no experience at all with dotnet, c#...
I managed to run the server locally on a Windows machine, using Visual Studio Code, added a settings.json file in the root folder providing all necessary variables. The android app also works and is set up.
I'm tunneling to my localhost so that I can connect using and android device.
But I'm running into many errors :

I the app, when launched,I'm getting status 202 meaning the request was accepted but it needs further processing because the status shows: "started" and the error dialog pops up.
In the server, sometimes it won't start with the following message error in the log:

A host error has occurred during startup operation 'd7e45bb4-43fc-4f9f-95b5-4d24c8dbf49f'.
Sygic.Corona.Api: Could not load file or assembly 'Microsoft.Extensions.DependencyInjection.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'. File was not found. System.Private.CoreLib: Could not load the specified file.
Value cannot be null.
Parameter name: provider

Tried everything, even installing Microsoft.Extensions.DependencyInjection.Abstractions using NuGet
When the above error message arises, I can't start server anymore,it will fail to that error again and again.
I know you may be busy with your work, but please assist

I tried it on 2 different pc and this error never occurred. So you probably somehow screwed references (you can try to delete obj and bin folders and run dotnet restore). So I advice you to fresh clone of the protect setup local emulators for storage account and cosmos. Then try to run server locally and test it from postman.

Dobrý deň, ďakujem za odpoveď @Daziko
As I said above, I spent days trying to set up everything. The response is coming from Covid server as I only set apiUrl variable points to the server url (localhost tunneled with Ngrok). Firebase works correctly because I can receive the push token and that token is passed to the api call along with deviceId but the server returns 202 Accepted, same when trying with Postman.  I also run Azure Cosmos Emulator locally

But I got somehow confused with the fact that function url are different from the one called from mobile client:
Http functions exposed are like this:

But the app calls are different:

When trying to call from Postman by using the function url, I get error 500 each time. I would like to point that I was able to get log only this way. The client never hit any breakpoint set in the server

Here is the log with the error:

Oh and I would like to point that no database is being created in the zure Cosmos Emulator. But it is working as I could use their sample app which created the Database and the
I won't deny it, I'm not experienced with dotnet, azure functions nor C# so I may be doing things wrong, that's why I need to borrow your precious time to get some guidance. Setting up should net be such a tough process, I would've rather start a project from scratch, it would be easy.
Prosím pomôžte mi

But I got somehow confused with the fact that function url are different from the one called from mobile client:

You have to adjust url (doesn't matter if on server or client site) because we are using API proxy.
So in app for example for createProfile just change "profile" -> "CreateDeviceProfile".

Oh and I would like to point that no database is being created in the zure Cosmos Emulator.

You need co create DB, container and partition key (/ProfileId) manualy.
Or you can call this method https://github.com/CovidWorld/server/blob/a61aa004421ed076f1b4124e5df1f7ebd4e37af8/Sygic.Corona.Admin/Startup.cs#L79

But I got somehow confused with the fact that function url are different from the one called from mobile client:

You have to adjust url (doesn't matter if on server or client site) because we are using API proxy.
So in app for example for createProfile just change "profile" -> "CreateDeviceProfile".

Oh and I would like to point that no database is being created in the zure Cosmos Emulator.

You need co create DB, container and partition key (/ProfileId) manualy.
Or you can call this method https://github.com/CovidWorld/server/blob/a61aa004421ed076f1b4124e5df1f7ebd4e37af8/Sygic.Corona.Admin/Startup.cs#L79

Here is the log with the error:

Error say that in startup.cs (dependency injection composition root)
App cannot resolve mediatr. So you probably changed some namespace on Application project or you change nuget references. Here is the crucial line where you  can investigate
https://github.com/CovidWorld/server/blob/a61aa004421ed076f1b4124e5df1f7ebd4e37af8/Sygic.Corona.Profile/Startup.cs#L46

I'm not experienced with dotnet, azure functions nor C#

But definitely you need .net/azure guy in your team because unfortunately I don't have time and capacity to support lack of man power in your team.

Well, this is what makes me wonder if you really want people use your project or not! How would one GUESS all of these ?  No one asks you to "support the lack of man power in your team"  rather take one minute and provide a setup process! That was rude and useless! If you provided a documentation, no one would bother begging for your help!
Anyway thanks for the superficial support.

@Daziko Is this an open source project with such arrogance??? TAKE THIS DOWN ALREADY


Spring in hero modifire

Here i am trying some damping effect using hero modifire but when i am using the spring effect, the animation is too slow. we also can not fix the time because it is auto calculated. other modifires are working well in the project.
Ex Code: .spring(stiffness: 0.5, damping: 0.3)
This is very slow. Please fix this.

0.5 and 0.3 is too small. Hero feed these number into CASpringAnimation. So something like .spring(stiffness: 150, damping: 20) would be a good number.

Cleaning Old Issues. Feel free to reopen.


Feature/timeout

This PR implements timeout option for experiments. Timeout can be specified in seconds by adding a "timout" section in the fault json file. Without a specification, it will default to 1200 seconds. If a timeout is exceeded, execution will be stopped.

Closed as redesign was made and merged in #80


Just a brush-up

Mainly related to updating the CI system, but also eliminates some code that wasn't actually used in the tests.

Sorry, I haven't been in shape to make sure everything goes alright after a release and had computer trouble for a bit. LGTM, thanks!


Remove   DV Chain (OTC) - Cryptocurrency OTC desk that offers buying and selling of Monero from the list of merchants

What's the name of the merchant you are asking to remove?
DV Chain (OTC) - Cryptocurrency OTC desk that offers buying and selling of Monero
What's the URL of their website?
https://dvchain.co/
Why do you think it should be removed?

[x] They don't accept Monero anymore
[ ] They are likely scammers
[ ] They closed / Their services are unavailable

Useful links
https://dvchain.co/
Additional comments
Passed the registration, KYC and as it turned out the seller does not accept Monero.

Hi @Zoloqiko, there must have been a misunderstanding. DV Chain 100% supports monero and we encourage our counterparties to trade it more.  Please reach back out.

Thanks for the clarification @gsee. Monero is also listed in the FAQ ("What cryptocurrencies can I trade?").
Rejected.


[Feature Request] Have a non-blocking Table Upload

When uploading a table from client to remote server, depending on table size, the upload part can take some time... So it would be awesome not to have the dialog modal to be able to work while the upload processes. The other import dialogs can be blocking, but maybe the unpacking one could be non blocking too
Alternative is to have it blocked by default but with a button in the dialog 'Run in background' to close the dialog and let the upload operates in background until the opening of the Table Data for confirming names and table match

I have disabled the button for 3.9 since the action will let the main FX thread continue. So the operations running right after them have no results they can check and errors popping up.
Also, the progress should not vanish but end up in the jobs menu.


loss is "nan"

Actually, in your code, even the first training loss is nan. How could you "loss.backward()"? Besides, the output of your net is also "nan".

因为他他妈optimizer.zero_grad()这个玩意放错位置了，ctmd

Sorry for replying so late and for the error training pipeline.
I have already updated the training pipeline.

Hi, I have the same problem,the first output of the net is 'nan', how can I solve it?


CI with coverage reports

Codecov Report

:exclamation: No coverage uploaded for pull request base (redesign@974df1f). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##             redesign       #22   +/-   ##
============================================
  Coverage            ?   100.00%
============================================
  Files               ?         7
  Lines               ?       124
  Branches            ?         0
============================================
  Hits                ?       124
  Misses              ?         0
  Partials            ?         0

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

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


Issue with IDE configuration.

Hello,
I have a probem with Neovim, Google test included headers are not found.
I am using coc-clangd as a extension for clangd in Neovim, and it works fine with compile_commands.json file.
However, I found that *_test.cc files are not found within compile_commands.json file, I tried to manually add an entry for a _test.cc file and it worked.
Is it a known limitation or an expected behavior?

Hi.
It is a known limitation. Only targets explicitly listed in compilation_database target are included in compile_commands.json.
As a workaround, you can use external script, generate.py (AKA bazel-compdb), from https://github.com/grailbio/bazel-compilation-database. It does include test sources when executed with //... target.

Thanks a lot!
I will try to use this one.


Fix car docs diff bot

We really need to just look at the MD file. This code is really fragile and not tested

⚠ This PR makes changes to CARS.md ⚠
🔀 Column Changes

Make
Model
Supported Package
ACC
No ACC accel below
No ALC below
Steering Torque
Resume from stop
Harness
Video

Toyota
Corolla Cross Hybrid (Non-US only) 2020-22 ➡ CorollaCross Hybrid (Non-US only) 2020-22
All
openpilot
17 mph
0 mph

Toyota

📖 Detail Sentence Changes

Sentence for Toyota Corolla Cross Hybrid (Non-US only) 2020-22 changed!- openpilot upgrades your Toyota Corolla Cross Hybrid (Non-US only) with automated lane centering and adaptive cruise control while driving above 17 mph.
+ openpilot upgrades your Toyota CorollaCross Hybrid (Non-US only) with automated lane centering and adaptive cruise control while driving above 17 mph.


Improve code style

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ac59eda) 100.00% compared to head (ac5a15e) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #108   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          179       180    +1
  Branches        32        32
=========================================
+ Hits           179       180    +1

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


Create source datasets from CELLxGENE source studies at the time of the match on DOI.

Need
We need immediate feedback on the source datasets belonging to a source study when matching in CELLxGENE.
On add source study, when it matches by DOI immediately update the source datasets in the database (before the add source study API response).
If there is no match immediately by DOI but later on refresh the DOI matches, add the source datasets for the source study to the DB at the time the match is detected.

Complete!


make test-e2e fails on push

It does not seem right that only privileged people should be allowed to run the e2e test.
make -C controllers/nginx container
make[2]: Entering directory `/home/porridge/Pulpit/coding/go/src/k8s.io/ingress/controllers/nginx'
docker build -t gcr.io/google_containers/nginx-ingress-controller:0.8.4 rootfs
Sending build context to Docker daemon 27.92 MB
Step 1 : FROM gcr.io/google_containers/nginx-slim:0.11
 ---> 1f831b9bc034
Step 2 : RUN DEBIAN_FRONTEND=noninteractive apt-get update && apt-get install -y   diffutils   ssl-cert   --no-install-recommends   && rm -rf /var/lib/apt/lists/*   && make-ssl-cert generate-default-snakeoil --force-overwrite
 ---> Using cache
 ---> d503fdd37ab3
Step 3 : COPY . /
 ---> e8f230d9d3a0
Removing intermediate container 19fa9207856d
Step 4 : RUN curl -sSL -o /sbin/dumb-init https://github.com/Yelp/dumb-init/releases/download/v1.2.0/dumb-init_1.2.0_amd64 &&   chmod +x /sbin/dumb-init
 ---> Running in ea440bcbb2a1
 ---> e5adf07224f9
Removing intermediate container ea440bcbb2a1
Step 5 : ENTRYPOINT /sbin/dumb-init --
 ---> Running in bdc2f45c6af7
 ---> 6c2c67728394
Removing intermediate container bdc2f45c6af7
Step 6 : CMD /nginx-ingress-controller
 ---> Running in 352d60740191
 ---> 3968d1630603
Removing intermediate container 352d60740191
Successfully built 3968d1630603
make[2]: Leaving directory `/home/porridge/Pulpit/coding/go/src/k8s.io/ingress/controllers/nginx'
make -C controllers/nginx push
make[2]: Entering directory `/home/porridge/Pulpit/coding/go/src/k8s.io/ingress/controllers/nginx'
gcloud docker push gcr.io/google_containers/nginx-ingress-controller:0.8.4
WARNING: The '--' argument must be specified between gcloud specific args on the left and DOCKER_ARGS on the right. IMPORTANT: previously, commands allowed the omission of the --, and unparsed arguments were treated as implementation args. This usage is being deprecated and will be removed in March 2017.
This will be strictly enforced in March 2017. Use 'gcloud beta docker' to see new behavior.
Using 'push gcr.io/google_containers/nginx-ingress-controller:0.8.4' for DOCKER_ARGS.
The push refers to a repository [gcr.io/google_containers/nginx-ingress-controller]
c3a8219e9c1c: Pushing [==================================================>] 48.64 kB
cc537ee68cae: Pushing [==================================================>] 27.89 MB/27.89 MB
5911c4a66511: Pushing [==================================================>] 1.036 MB
6d33e15f21fc: Mounted from google_containers/nginx-slim
1d2899dd3441: Mounted from google_containers/nginx-slim
161ae3947447: Waiting
36e0302572e7: Waiting
5f70bf18a086: Waiting
85f69a0fae58: Waiting
denied: Access denied.
make[2]: *** [push] Error 1
make[2]: Leaving directory `/home/porridge/Pulpit/coding/go/src/k8s.io/ingress/controllers/nginx'
make[1]: *** [docker-push] Error 2
make[1]: Leaving directory `/home/porridge/Pulpit/coding/go/src/k8s.io/ingress'
2016/12/22 09:53:12 e2e.go:278: Step 'build-release' finished in 1m7.73947222s
2016/12/22 09:53:12 e2e.go:137: Something went wrong: error building: error building: error running build-release: exit status 2
exit status 1
make: *** [test-e2e] Error 1
porridge@beczulka:~/Desktop/coding/go/src/k8s.io/ingress$

OTOH, if I skip the push, then the kubelet does not see the image after I apply examples/deployment/nginx/rc.yaml to the local cluster:
kubelet   1m    10s     4       {kubelet <IP_ADDRESS>}       spec.containers{nginx-ingress-lb}       Warning Failed          Failed to pull image "gcr.io/google_containers/nginx-ingress-controller:0.8.4": image pull failed for gcr.io/google_containers/nginx-ingress-controller:0.8.4, this may be because there are no credentials on this request.  details: (Tag 0.8.4 not found in repository gcr.io/google_containers/nginx-ingress-controller)

Even though it is available locally:
porridge@beczulka:~/Desktop/coding/go/src/k8s.io/ingress$ docker images
REPOSITORY                                            TAG                           IMAGE ID            CREATED             SIZE
gcr.io/google_containers/nginx-ingress-controller     0.8.4                         3968d1630603        2 hours ago         148.1 MB

@aledbf I think you said the push should be unnecessary.. perhaps we need to pass some option to hypercube to make it look in the local image storage?

Silly me, it's enough to drop imagePullPolicy: Always.


README: 15.09 -> 16.03

The README still refers to 15.09 as the latest release. We should update that.

Let's hope Travis CI does not fail against this change :smile:

thanks :)


Better indicate preview tab

It would be clearer to indicate preview tab by adding some visual effects.
This allows user to easily distinguish from normal tabs.
e.g. A different background color, text color or adding a special symbol.

sublime does it using italic font for tab title

Visual Studio uses different background color and border.

sublime does it using italic font for tab title

Atom works the same now.

I notice that: Italic font, like sublime

This issue was moved to atom/tabs#180


[PE-D][Tester B] Vague referencing of Viewing modes

Your UG does not really tell the readers what is Person List View and what is Transaction List View, I had to use your app for a while before I understood it. I would suggest including a short description of this at the start of your UG so that the readers can better appreciate your feature explanations and it would be more intuitive for them :)
No screenshot included as this bug is about a lack of something and not an inherent incorrect thing.

Labels: severity.Medium type.DocumentationBug
original: hovanleong/ped#5

severity MEDIUM are you CRAZY


Click handlers not working on nightly

Currently having an issue with click handlers. They seem to be not working at all unless I set the click handler globals manually after calling setup. Any idea what could be causing this?
Clicking anywhere in the statuscol only moves the cursor to that line, and doesn't execute the click handler.
Setting manually works, but then I'm not able to use the builtin functions which seem quite awesome.
Thanks in advance!

You're on the latest commit? What does your config look like? It works fine for me.

I am, yes. Actually does work if I manually call setup from the cmdline, so it's likely an issue with my config - I did try with a clean config + defaults and was still having the same issue though, so I'll keep trying things and update here soon.

Tried your config and it worked perfectly, I guess I have some debugging to do haha. My bad!

If you post the config that doesn't work I can check if it should.

If you post the config that doesn't work I can check if it should.

It's a relatively heavy config with over 100 plugins, I don't want to make you go through all that. Thanks though! I'm using default settings for statuscol while I test, I think there's got to be another UI-related plugin interfering with it.
I'll update here once I figure it out in case anyone else runs into the same conflict :)

I don't want to make you go through all that. Thanks though! I'm using default settings for statuscol while I test

No I meant your statuscol.nvim config specifically haha. But if you use the default setup() then idk.

Loll yeah I understood that, I just meant that whatever the cause is, it is outside of my statuscol.nvim config so any testing would need to use everything and narrow it down from there, and I wouldn't have someone else do that.

@luukvbaal alright, figured it out - based on your config, I switched my config for segments from
{
    text = { "%s" },
    click = "v:lua.ScSa",
},
{
    text = { builtin.lnumfunc },
    click = "v:lua.ScLa"
},
{
    text = { builtin.foldfunc },
    click = "v:lua.ScFa",
},

--- etc

to
{
  sign = {
    name = { "Diagnostic", "GitSigns*" },
    maxwidth = 2,
  },
  click = "v:lua.ScSa"
},
{
  text = { builtin.lnumfunc, " " },
  condition = { true, builtin.not_empty },
  click = "v:lua.ScLa",
},
{
  text = { builtin.foldfunc },
  click = "v:lua.ScFa",
}

With that, all click handlers started working. The defaults use text = {} for the signs, which don't seem to work. I've made the change in my fork if you want me to make a PR that keeps the default look but uses sign = {} instead of text = {}.

Thanks, this was actually a regression from https://github.com/luukvbaal/statuscol.nvim/commit/d14706eb789dbd7a7e8507adbbd252ab39265278#diff-d8ee2642cd48ac900ad0c1d6a1406be2c27e84a18e67ac5c726add285b0eefb9L91 where I erroneously removed this call. I guess I forgot about %s usage while developing the custom sign segments.
I should have tested the default config when you said that that's what you used, sorry.
I'm not sure what exactly you want with your sign segments. Judging by your description you might want this?
...
{
  sign = { name = { "Diagnostic" }, maxwidth = 1, auto = true },
  click = "v:lua.ScSa"
},
{
  sign = { name = { ".*" }, maxwidth = 1, auto = true },
  click = "v:lua.ScSa"
},
...

This would create one column with only diagnostic signs and a column right next to it with all other signs.

This would create one column with only diagnostic signs and a column right next to it with all other signs.

Okay so I may have spoken too soon. It seems this has also regressed with the 'signcolumn' == "number" support.

I've been messing with the config to get closer to what I want, essentially I'm looking for one column with gitsigns, and one column with everything else. That one gets close, but with .* the second column duplicates the diagnostics.

Duplicating should not and cannot happen I don't think. Signs are only placed once.
Currently signs are indeed allocated to builtin.lnumfunc even though 'signcolumn' != "number", I'm working on a fix.
The following should work once fixed, signs that have a dedicated column will not be placed in the .* wildcard column.
...
{
  sign = { name = { ".*" }, maxwidth = 1, auto = true },
  click = "v:lua.ScSa"
},
{
  sign = { name = { "GitSign*" }, maxwidth = 1, auto = true },
  click = "v:lua.ScSa"
},
...

Perfect, thanks! Awesome plugin btw, and I appreciate the quick responses.
Also, with auto = true, if there are no signs will the column disappear? I currently have colwidth set so that the column will always stay the same size, just curious if that's needed or if I can just leave out auto to get the same result.

Thanks. Yes, auto is equivalent to neovims :set signscolumn=auto:x. It will adapt the column width to the maximum number of signs on a single line. If you disable auto the column width will be static and equal to maxwidth. The README needs an update 😝
On the latest commit your config should work with .* wildcard again for the second sign segment, although for a different reason. (the builtin.lnumfunc will not overwrite the second sign segment now, still need to fix that it cannot get any signs allocated when 'signcolumn' != "number)

Working perfectly for me!


Talk Support for laravel 5.5.*

can talk be installed on laravel 5.5 and above?
When i tried  composer require on laravel v5.5.34 this is what i get
Problem 1
- Conclusion: remove laravel/framework v5.5.34
- Conclusion: don't install laravel/framework v5.5.34
- nahid/talk v2.2.0 requires illuminate/support 5.0.* || 5.1.* || 5.2.* || 5.3.* || 5.4.* ->
- nahid/talk v2.2.1 requires illuminate/support 5.0.* || 5.1.* || 5.2.* || 5.3.* || 5.4.* ->

For me i currently use the master. Add this line to your composer.json
"nahid/talk": "dev-master",

please use latest version


Disable camera when timelapsing ?

Hello,
Is it possible to disable picam when is in timelapse mode ? (just power-on on picture) Because i need to use battery and when the timelapse mode is enabled picam is always running.. Have you any idea how to do that ? Because i really need to save power...)
Thanks so much.

The camera is normally being used to provide a continuous preview all the time even during time lapse. I assume you are using a large interval between pictures and are not interested in the preview. It may be easier just to use raspistill with a script.
If you want to use the web interface as well to set things up then it may be possible to trigger a script from the web interface that turns off the camera and then does periodic raspistill commands.
Turning the camera off and on is possible using the ru cmd (0 off, 1 on)

Hello,
I understand that is not the first goal of your application, but for timelapse it could be very intersting to use your scheduler, to have the possibility to record video/image. More, i think  the preview could be an option because no need to have a preview for timelapse no ?
You think if i start timelapse and send CMD to turn off the camera, rpicam will continue run in timelapse mode ?

Turning off camera will stop the timelapse. It totally removes the connection to the camera hardware.
It is not quite as simple as Preview On, Preview Off because to achieve what you want the camera would be needed to be turned on just before each time lapse exposure and then turned back off again. Raspimjpeg is also maintaining a video buffer which is in effect recording all the time so that data can be pre-captured prior to motion triggers.
It is not impossible to do, but needs quite a bit of thinking through how to achieve it.

Ok i understand, I would be glad that this function was implemented, i think lot of people use timelapse mode and this feature to save energy would be so interesting on small device like rpi (for battery usage and for consumption)
Thanks you, tell me know if you planned to implement that !

Hi Robert, are there any news on this function? I am using a battery powered pi and the 10000mAH battery pack just lasts for less then a day :(
Can you turn the preview on and off with the scheduler? The camera doesn't really work when it's dark so we could save some energy by turning it off on nighttime.
Thanks for your help
Torsten

Hi Robert, I just read the documentation for the scheduler and I think it can do exactly what I want. Can you quickly check my parameters? I created 4 periods with the following commands:
6:59 ru 0 that should start the camera right? no preview before 06:59 "power saving mode"
7:00 tl 0 tv 300 that should start the time-lapse, 1 picture every 30s
18:00 tl 1 that should stop the time-lapse
18:01 ru 1 that should stop the camera, no preview after 18:01 "power saving mode"
Thanks for your help
Torsten

ru 1 starts camera ru 0 stops camera so you need to flip those.
tl 1 starts timelapse tl 0 stops time lapse so you need to flip those.
tv 300 will be remembered when set. So you could just set it once manually in camera settings.
If you put multiple commands in scheduler then put a ; in between and don't leave spaces after the ;
Like tv 300;tl 1; would set interval and then start tht etime lapse going

Hi Robert,
thanks for you help. I changed it and will try afterwards.
Maybe change the scheduler documentation. I not quite clear which number is stop and start

tl	0/1	start/stop timelapse

Scheduler command reference updated to be clearer on number usage.

Hi Robert, I have version 6.0.37 installed on apache with the old installation.sh.
Unfortunately the start/stop camera function doesn't work anymore. It automatically restarts the camera every time I try to switch it off. Or does that happen automatically when you open the webpage? In the last version there was a picture saying no connection.
Thanks for your help
Torsten

I don't have much to do with old installation method which is maintained by miraaz. I maintain and use the install.sh method. So it is possible the installation is not as needed.
To check on the ru command, open a terminal window and with camera operating normally issue the command
echo -n 'ru 0' >/var/www/FIFO
include any subfolder used in the install.
The preview should freeze and all buttons dim out.
If you now issue a
echo -n 'ru 1' >/var/www/FIFO
then camera should restart and carry on normal operation.


chore/fail-safes-for-defs

I collected those along the last couple of weeks, all of them happened at least once.
bonus:
add highlight to clusterrolebindings

LGTM! 🚀


add more content to /services

fill the /services page of the site with some content

the "services" has been deprecated. We came up with a more simplistic one-pager solution, where there is no services listed.


Edit add-on help text to include type: module

Describe the bug:

There is a discrepancy between the Github installation docs and the ones located inside the add-on for HACS. The HACS version does not show type: module even though this is required creating confusion. Can this be added or is that coming directly from HACS?
Config YAML:
<!-- Paste your lovelace code for the card after this line -->

HA and/or Browser Error:

HA, Card, and Browser Versions:

1.0.4b9

Added with latest release

That was fast! Thanks.


Update linux-dsvm-walkthrough.md

Fixes a few glitches inside the doc.

The instructions to install azureml package was outdated.
Undefined variable ws
~/.azureml/settings.json file is needed for workspace initialization.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: penggu sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @penggu - Thanks for submitting this pull request. Could you take a moment to sign the CLA? Then, we can review the PR and your proposed updates. Thanks.

Hi! Will it help if I approve the changes?

Hi @shivanissambare - The pull request can't be accepted (it can't be reviewed, approved, or merged) until the contributor signs the Contributor License Agreement (CLA).

Hi @penggu - Thanks for signing the CLA! The change to the metadata author attribute is not correct. This value should be the author's GitHub user name, and it appears to be correct as-is: https://github.com/vijetajo. (For future reference, article metadata should be changed only by Microsoft-internal users, and from the private repo.) I've reversed that edit. Merging the PR.


Allow admin user/password through environment variables

Allows specifying the default admin user / password through the PI_ADMIN_USER / PI_ADMIN_PASSWORD (or PI_ADMIN_USER_FILE / PI_ADMIN_PASSWORD_FILE) environment variables, and update the documentation to strongly recommend changing the default password.
(The reference to the possibility to remove the default admin user has been removed, as it allows the possibility of an insecure setup if the database were to be recreated).
Fixes: #220

Sorry for the late reply.
Thank you for the contributions, but I found a better place fort his through the config. see e956199 and my comment in #220


Save interval not working

I am running Mosaicbert pretraining code from /examples/benchmarks/bert using the docker container. Everything works well, other than saving number of checkpoints.
My conf is as below :
save_interval: 1ep
save_num_checkpoints_to_keep: -1
save_folder: ./{run_name}/ckpt
save_overwrite: True

Even tried with giving batch interval and without specifying save interval as well(which should by default save every epoch). Whatever interval I specify, only 1 checkpoint is getting saved. I was wondering if any one else facing the same issue.

How many epochs does your run go for? Also, what is the save_filename? Is it unique per timestamp (to verify it isn't being overwritten each time)

Only 1 checkpoint is getting saved..in the format of 'ep3-ba1458-rank0.pt'. If I specify save interval 1ep only 1st epoch gets saved even if I have 30 epochs. And if save interval is 3ep, the 3rd epoch gets saved, rest are ignored

Hm... do you get any errors or traces?

There are no errors and everything working fine including plotting the loss etc.
I am using this git branch for mosaicbert pretraining. https://github.com/Skylion007/mosaicml-examples/tree/skylion007/add-fa2-to-bert/examples

Mind sharing what version of composer you are using?

0.17.2

That's a very old (>6 months old) version of composer. is there a reason you need to use that old of a version?

I have been using this particular docker image https://github.com/Skylion007/mosaicml-examples/tree/skylion007/add-fa2-to-bert/examples to use flashattention, triton etc. These set up require sepcific versions, if I update to latest composer,  triton fails.
But now I managed by updating to a slightly higher version like 0.19. And it works fine. But I wonder why composer failed only at checkpoint part for the previous 0.17.2 version

Going to close as it seems to work.
I'm not super sure what the bug was, but it's certainly possible there was an issue in older versions. Definitely recommend upgrading to latest :)


resolve .kube/config.lock: file exists flake

What type of PR is this?
/kind failing-test
What does does this PR do / why we need it:
Some PRs are failing with this error
Creating a new project: vhgeggwlqs
Running kubectl with args [kubectl create namespace vhgeggwlqs]
[kubectl] namespace/vhgeggwlqs created
Running kubectl with args [kubectl config set-context --current --namespace vhgeggwlqs]
[kubectl] error: open /home/travis/.kube/config.lock: file exists

Hence resolving this error
https://travis-ci.com/github/openshift/odo/jobs/440575550
Which issue(s) this PR fixes:
Fixes https://github.com/openshift/odo/issues/4190
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[X] I have read the test guidelines

How to test changes / Special notes to the reviewer:

[odo] [INFO] BUILD FAILURE
[odo] [INFO] ------------------------------------------------------------------------
[odo] [INFO] Total time:  25.791 s
[odo] [INFO] Finished at: 2020-11-16T10:44:14Z
[odo] [INFO] ------------------------------------------------------------------------
[odo] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project sprtest1: Execution default-compile of goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile failed: Plugin org.apache.maven.plugins:maven-compiler-plugin:3.1 or one of its dependencies could not be resolved: The following artifacts could not be resolved: org.apache.maven:maven-toolchain:jar:1.0, org.apache.maven.shared:maven-shared-incremental:jar:1.1: Could not transfer artifact org.apache.maven:maven-toolchain:jar:1.0 from/to central (https://repo.maven.apache.org/maven2): Transfer failed for https://repo.maven.apache.org/maven2/org/apache/maven/maven-toolchain/1.0/maven-toolchain-1.0.jar: Connection reset -> [Help 1]

maven error in travis

/approve

 ⚠  Registry DefaultDevfileRegistry is not set up properly with error: Unable to unmarshal the devfile index.json from https://raw.githubusercontent.com/odo-devfiles/registry/master/devfiles/index.json: invalid character ']' after top-level value, please check the registry URL and credential (refer `odo registry update --help`)
[odo]
[odo] I1116 11:59:29.299547   43035 preference.go:209] The path for preference file is /tmp/743953526/preference.yaml
[odo] Validation
[odo]  •  Checking devfile existence  ...
[odo]  ✗  Checking devfile existence [9649ns]
[odo] I1116 11:59:29.301392   43035 util.go:431] path devfile.yaml doesn't exist, skipping it
[odo]  ⚠  Devfile component type java-maven is not supported, please run `odo catalog list components` for a list of supported devfile component types
[odo] I1116 11:59:29.301454   43035 preference.go:209] The path for preference file is /tmp/743953526/preference.yaml
[odo] I1116 11:59:29.386294   43035 preference.go:209] The path for preference file is /tmp/743953526/preference.yaml
[odo]  ✗  component type "java-maven" not found

/retest

/retest


Options not working

According to  curl wttr.in/:help:
Options can be combined:

    /Paris?0pq
    /Paris?0pq&lang=fr
    /Paris_0pq.png          # in PNG the file mode are specified after _
    /Rome_0pq_lang=it.png   # long options are separated with underscore

When I run it, it returns no matches found.

I had the same problem, solved using \ before the symbols.
Like this:
/Paris\?0pq
  /Paris\?0pq\&lang=fr

Thanks @ilGallo. It works.
But I hope @chubin is going to fix this. Because it is not how URL query works.

The problem here is not the service (or that it handles URL wrongly), but zsh.
zsh tries to process the glob (it thinks that it is a glob because of the ? in the name),
and it can't find any file matching this glob (wttr.in/Paris?n).
If you run setopt +o nomatch first, it will work without any escaping

@chubin
Thanks, it works.


Bucklescript Support

It would be nice to have an easy boilerplate or generator to start a vue.js or nuxt.js project in bucklescript or reasonml.

any update on this?


Icons.keyboard_command_key_outlined missing from Icons class

Hi @visign3d, please provide the output of flutter doctor -v.
Thank you

`
[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.19043.1526], locale hu-HU)
• Flutter version 2.8.1 at C:\Flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 77d935af4d (9 weeks ago), 2021-12-16 08:37:33 -0800
• Engine revision 890a5fca2e
• Dart version 2.15.1
[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
• Android SDK at C:\Users\vikto\AppData\Local\Android\sdk
• Platform android-31, build-tools 32.0.0
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[√] Android Studio (version 2020.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
[√] VS Code (version 1.62.0)
• VS Code at C:\Users\vikto\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.27.0
[√] VS Code, 32-bit edition (version 1.32.3)
• VS Code at C:\Program Files (x86)\Microsoft VS Code
• Flutter extension version 3.27.0
[√] Connected device (3 available)
• sdk gphone x86 (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
• Chrome (web)            • chrome        • web-javascript • Google Chrome 98.0.4758.82
• Edge (web)              • edge          • web-javascript • Microsoft Edge 98.0.1108.43
• No issues found!
`

Hi @visign3d, you are on an older version of flutter.
These icons are available on stable 2.10+, but there is an issue on the stable channel with how these new icons are rendered. That has already been reported in https://github.com/flutter/flutter/issues/97767 and there's a request for it to hotfixed into the new stable version. Kindly follow up in https://github.com/flutter/flutter/issues/97767 for more details.
Closing from here as duplicate.
Thank you


Use slack to replace gitter.

Slack is more popular and easier to use in the phone. The gitter channel wouldn't shutdown, just remove it from our readme.

Coverage remained the same at 15.411% when pulling 608459b047266e7ed9a343d39c7919e0e299c789 on wu-sheng-patch-1 into 315c0e69ee4ac5748d0787f6e638798aa3639400 on master.


MONGOID-5706 Depend on mongoid-odm instead of mongoid

In order to allow mongoid to depend on railsmdb, we need to change mongoid to be a meta-gem, which in turn depends on mongoid-odm (see https://github.com/mongodb/mongoid/pull/5766). This updates the railsmdb gem so that it depends on the mongoid-odm gem as well, thus breaking the circular dependency.

(Note: failing specs because of the dependency on the mongoid-odm gem, which doesn't exist yet.)


handbrake: 1.3.1 -> 1.3.2

Motivation for this change
Starting to use GitHub CDN.
Every time there is any HandBrake release - it gets published on their website after a timeout of arbitrary days. And I want to be able to respond to notifications of release right away, and not postpone, stagger, keep checking wait website tarball and remember that I still need to update.
Also update to 1.3.2.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

What happens when you build from the checkout as opposed to the release tarball?

and add autoreconfHook to nativeBuildInputs.

Also, some parts of the preConfigure are not needed since:
https://github.com/HandBrake/HandBrake/issues/2690
https://github.com/HandBrake/HandBrake/pull/2712

@peterhoeg:

and add autoreconfHook to nativeBuildInputs.

Results:
source root is HandBrake-1.3.2
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file HandBrake-1.3.2/version.txt
patching sources
autoreconfPhase
autoreconf: 'configure.ac' or 'configure.in' is required
builder for '/nix/store/dvg6dlm5pwrxq8pfbljjqjfky6v5hsra-handbrake-1.3.2.drv' failed with exit code 1

@peterhoeg:

What happens when you build from the checkout as opposed to the release tarball?

Git checkout
I was discussing this issue with them upstream a couple of years ago. They have invented their-own versioning inside a project:
nix-shell . -A handbrake
unpackPhase
cd source
patchPhase
scripts/repo-info.sh    # source: https://github.com/HandBrake/HandBrake/blob/8e2a08267705f09e92f06289820ad6ebbfc9040e/scripts/repo-info.sh

Output:

URL=
HASH=ccb2e7234e01a2a07f537512045f7511583deaf9
SHORTHASH=ccb2e7234e0
REV=224188
BRANCH=handbrake-1.3.2
REMOTE=
DATE=2020-05-05 12:44:02 +0300

And this returned information results into probe: repo info...(fail) code 1
Results in build log:
source root is source
patching sources
configuring
...
probe: repo info...(fail) code 1
probe: version.txt...(fail)
ERROR: HandBrake is missing version information it needs to build properly.
Clone the official git repository at https://github.com/HandBrake/HandBrake.git
or download an official source archive from https://handbrake.fr
; configure stop.
compute: project data...
builder for '/nix/store/vwispj413x66pgh419awlgd6z9vj3nnv-handbrake-1.3.2.drv' failed with exit code 1

I got no description on what this situation is about, threads I've read resolve things replying "use website tarball".
Tarball package
Results in:
scripts/repo-info.sh

This also results into probe: repo info...(fail) code 1, but now of the other kind

fatal: not a git repository (or any of the parent directories): .git
fatal: not a git repository (or any of the parent directories): .git
Not a valid repository.

And that situation "works" in the tarball:
probe: repo info...(fail) code 1
probe: version.txt...(pass)
...

If to give the long answer.
As I understand that there does not exist the case to succesfully go there proselytize and rip-out their internal distribution & versioning - I do not worry about this too much.

Also, some parts of the preConfigure are not needed since:
HandBrake/HandBrake#2690
HandBrake/HandBrake#2712

@doronbehar
Seems done.
Also seems that upstream fixes around libxml2 not fixed some of the libxml2 cases we receiving..

BTW, I'm getting this when running it:
** Message: 14:08:09.629: Couldn't initialize gstreamer. Disabling live preview.

I think you can fix it with something similar to:
https://github.com/Anton-Latukha/nixpkgs/blob/f9a7d6c4ed32bb78d0df7d25dd1a3b72f658dbe4/pkgs/applications/audio/transcribe/default.nix#L46-L47
While some gst-plugins are located in the build inputs so wrapGAppsHook will pick them up.
But it's also possible that gappsWrapperArgs+=( is not needed...

BTW, I'm getting this when running it:
** Message: 14:08:09.629: Couldn't initialize gstreamer. Disabling live preview.

I specifically tested that live preview working. Since there is already wrapper in place, I would just add configuration for the wrapper.

@doronbehar
I specifically tested that live preview works.
And with https://github.com/Anton-Latukha/nixpkgs/blob/f9a7d6c4ed32bb78d0df7d25dd1a3b72f658dbe4/pkgs/applications/audio/transcribe/default.nix#L46-L47 ghb still throws that same warning. And just as before - in ghb live preview still works despite of that warning.

OK I guess then it's not that bad (I'm not using Handbrake actively, I only tested it once in the past). If someone will notice some missing functionality that may be related to this message, he'll put the effort to debug this.

@doronbehar
It makes in Preview the Live Preview button appear.
Like here: https://handbrake.fr/docs/en/latest/workflow/preview-settings.html
I also do not use HandBrake mutch these days, Preview is needed, I almost never run Live Preview.
Wrapper fix did not helped.
I may dig into it, construct --prefix LD_LIBRARY_PATH : "${libPath}" and assemble its libPath variable or make deeper assessment.

I do not understand why if one maintainer changes to the package, and already went through the review process - why other same-level maintained should "mandatory" review it.
The Mertens for example probably or too busy or do not care for it long time ago.
The review is done.

Yea this is how things are in this repo unfortunately. Sometimes a maintainer with a merge bit goes through the reviewed pulls sorted from old to new and they may encounter this PR at some point, but don't hold your breath for this to happen.

Sorry, I missed that it was ok to merge. Indeed, I don't use handbrake any more, so feel free to sign yourself as a maintainer

@wmertens I am also a maintainer of this package for 2-3 years already.
We chatted with you back in the day about the package.
I am talking that I am a maintainer, process requires review from the higher-order maintainer, like doronbehar.
But I am maintainer that passes review.
I am just puzzled why "process" requires review from same-level package maintainers.
People tend to not remove themself from maintainer fields and not care and not show up.
And so - there are packages that have 5-6 "maintainers" but no one shows up to maintain it. So why that package should have all 5-6 maintainers sign-off between each other every time? It is not like this is Linux Kernel development, and as I know they not do such multicross signoff all the time.

@doronbehar
Thanks to you a lot. You are a great guy.
How is auto update bot there.
I see that it still not pipelined to autotrack and automerge
Why? It seems really strange.
I think most people woul've preferred for minor updates to go without bothering them.

Thanks to you a lot. You are a great guy.

:green_heart:

It is not like this is Linux Kernel development, and as I know they do multistage reviews, but do not require such multicross peer review signoff all the time, because nothing would get done then.

How is auto update bot there.
I see that it still not pipelined to autotrack and automerge
Why? It seems really strange.
I think most people, and you, would've preferred for minor updates to go without bothering them.

Your phrasing is not that good but I think I understand you. See:

https://github.com/NixOS/rfcs/pull/50
https://discourse.nixos.org/t/new-rfc-50-merge-bot-for-maintainers/3795
https://discourse.nixos.org/t/steps-towards-even-more-pr-automation/5634/13
https://discourse.nixos.org/t/suggestion-to-help-maintainers-with-nixpkgs-prs-bloat/4794/20
https://github.com/NixOS/rfcs/pull/51
https://github.com/NixOS/rfcs/pull/30


Awards Doesn't Upload Control Award

Control Award is not Uploaded for FiM events

Auburn Hills Qualifier is one with a control Award winner


Phantom tests run on feature branches 👻

Very interestingly, the test workflow runs tests present on main that are on neither of the participating branches of a PR. This is why sometimes PRs fail checks when it is actually main that is at fault. I think this is true for other code issues as well (linting, syntax errors), not just tests, but this is just from what I remember.
For tests, I'm currently testing this behaviour in this draft PR.
Hmm, or do they?

I've looked at this again and I now think it may have been the product of the following:

Tests are run on the result of the merge between the base branch and the feature branch (so not just the feature branch).
If you set up a PR with main as the base branch and then change the base branch to something else, and if you then rerun the test action, it actually still runs against main + the feature branch. This makes it look like tests are run which are on neither of the new base branch or the feature branch.

This still doesn't explain why IDs started being sorted in reverse though...


Is atlas_densities/app/data/metadata/thalamus_metadata.json actually used?

I recently accidentally discovered the file atlas_densities/app/data/metadata/thalamus_metadata.json, and am trying to determine if I need to update it at the same time I am updating analogous files in other packages, since the regions in the query are out of date. However, as far as I can determine, this file is used in neither the atlas-densities code nor blue_brain_atlas_pipeline, unlike isocortex_metadata.json etc. Is this actually used? If so, what is it doing?

blue_brain_atlas_pipeline is not invoking that file directly. Unless it is implicitly consumed by some atlas-densities command, I can confirm that it is not used in blue_brain_atlas_pipeline.


[FEAT]: Upload Repo from a local folder.

What would you like to see?
GitHub has a limit for downloading repositories directly via the API. It would be useful if we could upload a folder directly from the computer containing our data. This way, a new folder with our documents would be automatically created within the data directory. Additionally, I often encounter the following error:
[collector] error: Error: Unable to fetch repository files: 403 {"message":"API rate limit exceeded for user ID . If you reach out to GitHub Support for help, please include the request ID and timestamp","documentation_url":"https://docs.github.com/rest/overview/rate-limits-for-the-rest-api","status":"403"} at /app/collector/node_modules/langchain/dist/document_loaders/web/github.cjs:
This is a feature request for the application, upload from local repo wil be very fast and no limit.

Moving to #923


initial component sheet

@keybase/react-hackers
the very beginnings of getting the component sheet up and running

:+1: % commented code

feel free to merge the commented code isn't a big deal


Adds auto-reload dev option

Automatically reload the bot whenever the source code changes if you use the --dev option. Also picks up changes to data files if they change during runtime.

Codecov Report

Merging #135 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #135   +/-   ##
=======================================
  Coverage   99.38%   99.38%
=======================================
  Files           1        1
  Lines         973      975    +2
  Branches      192      192
=======================================
+ Hits          967      969    +2
  Misses          2        2
  Partials        4        4

Impacted Files
Coverage Δ

src/turbot/__init__.py
99.38% <100.00%> (+<0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


[mesheryctl] Improve pattern list Error Message

Current Behavior
The mesheryctl pattern list error response currently displays as ;
➜  meshery git:(master) ✗ mesheryctl pattern list
Error: Get "http://localhost:9081/api/pattern": dial tcp [::1]:9081: connect: connection refused
➜  meshery git:(master) ✗

Expected Behavior
The mesheryctl pattern list error response should be “Unable to reach Meshery Server at ….. Please confirm that Meshery Server is available.”
Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Thank you. I'll be sure to do that.

Checking in... how are things progressing?

Struggling a bit with understanding and navigating the entire code base to a point where I figure out where my target error is coming from, but I'm spending some time tracking each function to determine it's output and just taking time to learn while at it.

Please leave this particular issue to me. I'll keep trying until I get It.

hey @leecalcote ,i found out that the codeline which returns the error (and needs to be changed) is this.
Would it be a good idea to replace it with something like :
res, err := client.Do(req)
if err != nil {
	return errors.New("Unable to reach Meshery Server at port 9081.
               Please make sure that meshery is running by using command 'mesheryctl system start' ")

@ayushthe1, yes, that's pretty close, however:

The port # should not be hardcoded, but taken from mesheryctl's central config/struct.
Always capitalize Meshery.
Not system start, but system status is how you verify if Meshery is running.
Don't say "Please", even though it is polite, it's inappropriate tone.
First suggest system check, which is what I was telling you previously, not system start

Hey @ayusha and @leecalcote
I actually already did this back in december and came up with this
res, err := client.Do(req)
		if err != nil {
    return fmt.Errorf("Unable to reach Meshery Server at %s. Please confirm that Meshery Server is available", mctlCfg.GetBaseMesheryURL())
}

however, I think it's something with my system. I'm unable to run everything needed to test it. That's why I haven't committed it yet.

I also was going to try defining a new error message as a constant and using it in the return statement like this;

const mesheryServerError = "Unable to reach Meshery Server at ….. Please confirm that Meshery Server is available"

if err != nil {
    return fmt.Errorf(mesheryServerError)
}

But again, I've been unable to test them out due to my laptop shutting when I try to run the needed softwares.

Thanks for the feedback @leecalcote . Is this a correct way .
res, err := client.Do(req)
		if err != nil {
			return errors.Errorf("Unable to reach MESHERY server at %s. Verify your environment's readiness for a MESHERY deployment by running `mesheryctl system check`. ", mctlCfg.GetBaseMesheryURL())

Only capitalize the “M” in Meshery. Don’t yell at users.

Only capitalize the “M” in Meshery. Don’t yell at users.

Ok ,I will do that . Should I open a pr with these changes?

Someone should do something.

I'm on it.

Done @leecalcote :+1: . Could you please review it.

I see that @Ebi27 has been assigned the issue, but didn't make a PR for a long time and the issue went stale.
Thus, @ayushthe1 takes over from there. I am considering the PR made by @ayushthe1 , hope you understand the situation here @Ebi27


Prevent Ours is the Fury double adding characters to challenge

Previously, when Ours is the Fury was played on a character that was already participating in the challenge, that character's strength would be added again. This PR fixes that by adding a check to challenge.addDefender.
Fixes #1273.

I think this would be better fixed in the card itself, check if it's already participating and don't call addDefender()


Pip install

@ScriptSlayer0
In case you haven't seen it, I wanted to let you know that the project has been published and linked to PyPi, which means it can be installed using pip. We will have to run checks after releases to make sure our edits don't mess up the pip release.

Good, I have seen that you were working on that.

Yeah, I finally got it working today. My last error was that it wasn't including the lib_bdb.so when building the whl, which meant pip never installed it. That took a second to fix


Store theme

How can we change the theme is their any theme available for cezerin2 store.

/cezerin2-store/theme folder.
You can edit theme files there.
There is no public available themes at this moment.
https://plasha.ru
Here is the example of cezerin2 theme.

Where do i get this theme used in plasha.ru

This is commercial template and not available for free download.


installing issue

go get -u -v github.com/dwisiswant0/cf-check  command not working.

Installing executables with 'go get' in module mode is deprecated. Try with go install -v<EMAIL_ADDRESS>

thanks its worked. please update this command on your github repo
On Thu, Feb 17, 2022 at 2:33 AM Dwi Siswanto @.***>
wrote:

Installing executables with 'go get' in module mode is deprecated. Try
with go install -v @.***
—
Reply to this email directly, view it on GitHub
https://github.com/dwisiswant0/cf-check/issues/7#issuecomment-1042240733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AROHDVK6MLS53ZVGMF2I7ADU3QCZJANCNFSM5ORUXGKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


RuntimeError: multi-target not supported for CrossEntropyLoss

After I checked your code, I think this error may because your command written in README is wrong.
Since CMU-MOSEI is a multi-label classification task, we need to use binary cross entropy loss, so the command for CMU-MOSEI should use bce loss instead of ce. And the command for IEMOCAP also has this problem.
BTW, why you also use binary cross entropy for IEMOCAP instead of normal cross entropy ?
=====Below is old message=====
Hello,
I'm trying to run your model on the CMU-MOSEI dataset. And I got the multi-target error.
It seems the emotion label of processed dataset is not one-hot but multi-target.
Do you have any idea to fix this problem ?

@Aria-K-Alethia
Hi, yes you are right. I'll update the README soon.
The reason I use BCE for IEMOCAP is to compare with previous papers, they report binary performance and some papers state explicitly that they treat IEMOCAP as a multi-label dataset.

@Aria-K-Alethia
Hi, yes you are right. I'll update the README soon.
The reason I use BCE for IEMOCAP is to compare with previous papers, they report binary performance and some papers state explicitly that they treat IEMOCAP as a multi-label dataset.

I see. Thank you for your help !


Update aria-and-ui-automation.md

minor typo

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Alias react to inferno-compat

It's what I proposed at https://github.com/infernojs/create-inferno-app/issues/26
We won't bloat the install by adding inferno-compat, but if one wants to user it then the one should just yarn add inferno-compat and it's good to use React libs.

We should add some documentation that this is the case to the README.md

Usage with React (through inferno-compat)
You can use React components with create-inferno-app. All you have is to do is install inferno-compat (npm install --save inferno-compat)  and React and ReactDOM will be already aliased to inferno-compat. After installing it you can import any React dependent library or use any code that imports React and that should work fine.

@degroote22 sounds good merging.


Feature request: Enable setting Variables in a Recipe

There's already a great docs section that talks about Setting Variables in a Recipe.
This is a feature that I frequently miss, and sometimes I want to avoid the suggested solution of using a single shell for the recipe using the she-bang line. (Typically, this is when I want the exit-code behavior of just, as using a she-bang line changes that functionality.)
It seems like it could be achieved by mimicking a shell's export behavior; that is, if the recipe line starts with export then add the resulting variable to the just process environment.
Since each line is within a shell, any current shell line with export NAME=variable is effectively a no-op, so this feature only extends, but does not modify, existing functionality.

Though not likely, this could be a breaking change for some recipes. Maybe a special function you could call at the end of the recipe could give back the normal just exit code behavior? Might that be a possibility @casey ?

I wouldn't do this as a breaking change, since that would be too disruptive, so this would have to have syntax that didn't conflict with normal shell syntax. Also, just is very static-analysis heavy, which I think is one of its strengths. I.e. it checks that all variable accesses are valid ahead of time, and resolves dependencies between them. So this feature would also have to have the same properties, i.e. not possible to misspell a variable.

Though not likely, this could be a breaking change for some recipes. Maybe a special function you could call at the end of the recipe could give back the normal just exit code behavior? Might that be a possibility @casey ?

I realized that the exit on non-zero codes can be recovered with #!/bin/sh -e.
With that realization, it's easy for work around not having this feature. This feature might still be nice, as I think setting variables within scripts, without needing to add the she-bang line, would be more intuitive for users seeing just for the first time. Specifically, I think it's surprising and somewhat counter-intuitive to have to change the execution context with a shebang line just to set variables.
@runeimp What scenario would be a breaking change? As I mentioned above, if the syntax was export NAME=value, such a line is a no-op in the shell-per-line context that just uses to execute recipes, so I don't see how that feature could break existing functionality. I'm certainly open to counter-examples, but I can't think of any. Using the export syntax has the advantage, in my opinion, of being "less surprising" to new users, since it's giving just the exact same variable behavior that shells follow. Namely, a line NAME=value is a no-op unless there's a command on the same line, whereas export NAME=value causes subsequent executions to have NAME in its environment.

@kurtbuilds hm, I'd thought of one immediately as I was reading the suggestion originally. But now I'm drawing a blank. Like I said it would be rare. But on the off chance it comes up it would likely be a complete nightmare to debug. And could effect recipes already in use that may be part of a CI/CD process. If I think of an, or remember my, example I'll update this issue. Sorry, my brain is done for the day I think. 😇

This feature might still be nice, as I think setting variables within scripts ...

Similar feature request was discussed in a previous issue.
https://github.com/casey/just/issues/1011
It would be more useful to set the value computed within the shell process.
k := ""
demo:
  v=$(program); just --rpc-client k := "$v";
  $a="{{k}}"; if [ $a = "ok" ]; then just --rpc-client run-code-in-parent-just-process; \
  else just --rpc-client issue-cmd-from-child-process; fi

If this feature just sets a fixed value (defined at the time of writing the justfile) when running to the specific line, the usage scenario is limited.

Also, just is very static-analysis heavy ... I.e. it checks that all variable accesses are valid ahead of time ...

The above can be written as v=$(program); {| k := "$v" |};, after doing static analysis, dynamically converted to the corresponding shell command when executed. For example, v=$(program); just --rpc-client k := "$v";
The parent just process receives the instruction and changes the corresponding variable.

A clean way to do this could be a recipe attribute with parameters that allows to declare environment variables should be passed on to subsequent recipes.
Something like:
[ env VAR1 VAR2 ]
recipe1:
    #!/bin/sh
    export VAR1="hello"
    export VAR2="world"

recipe2: recipe1
    #!/bin/sh
   echo $VAR1 $VAR2 # will output "hello world"

could work if  just inspects the environment after running recipe1 and makes (only) the variables declared in env  available before running recipe2 (there's probably some better name than "env" for this).
I'm using a shebang in recipe1 above, but the same thing may work for non-shebang recipes, as long as the environment is inspected after each line is executed.
This could also make things like this work
[env GREETING]
recipe:
    export GREETING="hello"
    echo "$GREETING world"