cutadapt trim first 10 base on second read

Hi Brad, I have one question is possible to trim the first 10 base of the second read in a varinat call run? I have set trim_reads: true and set the cutadapt option -U 10 in the resource section but I still not get the trimmed fastq.
do you have any suggestion ?
Thanks a lot

algorithm:
aligner: bwa
ensemble:
numpass: 2
mark_duplicates: true
remove_lcr: true
trim_reads: true
variantcaller:

vardict
mutect2
freebayes
analysis: variant2
description: 2680-2
files:
2680-2_CTTGTAAT_1.fastq.gz
2680-2_CTTGTAAT_2.fastq.gz
genome_build: hg19
metadata:
batch: 2680-2
phenotype: tumor
fc_date: '2017-06-06'
fc_name: FL
resources:
cutadapt:
options:

-U 10
-p

We're hoping to include trimming into variant calling prior to the next release, hopefully getting it finished over the next week. We've previously relied on bwa soft-clipping to remove adapters since it does a solid job but are seeing more cases where trimming upstream can be helpful and would like to incorporate this.
If you need it prior to being able to include in bcbio you could trim upstream and then feed into bcbio, which is the standard workaround right now. Thanks again for the discussion and encouraging us to support this.

Thanks a lot!
Alessandro

On Jun 9, 2017, at 11:53, Brad Chapman<EMAIL_ADDRESS>wrote:
We're hoping to include trimming into variant calling prior to the next release, hopefully getting it finished over the next week. We've previously relied on bwa soft-clipping to remove adapters since it does a solid job but are seeing more cases where trimming upstream can be helpful and would like to incorporate this.
If you need it prior to being able to include in bcbio you could trim upstream and then feed into bcbio, which is the standard workaround right now. Thanks again for the discussion and encouraging us to support this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/chapmanb/bcbio-nextgen/issues/1971#issuecomment-307427215, or mute the thread https://github.com/notifications/unsubscribe-auth/AGOdNto1iKslnIS1M85r0zN1kAsgvXvEks5sCWpkgaJpZM4N1Ymn.

Alessandro;
Thanks again for the suggestion. This is now available in the latest development version of bcbio, including passing custom options to cutadapt:
http://bcbio-nextgen.readthedocs.io/en/latest/contents/configuration.html#read-trimming
Looking at your example above, you should remove the -p specification in your YAML but otherwise it should work as desired. Please let us know if you run into any problems using this and thanks again for the suggestion.

Alessandro;
If you're looking to do defined end trimming as part of variant calling, the latest development version now has a faster way to do this using trim_ends:
http://bcbio-nextgen.readthedocs.io/en/latest/contents/configuration.html#read-trimming
So you'd supply:
trim_ends: [0, 0, 10, 0]

To trim from the 5' end of read 2. This is integrated with fastq preparation so much faster than cutadapt if you don't need adapter trimming. Hope this is useful.


[BUG] ObjectDisposedException when calling GetConfigurationSettingAsync / SetConfigurationSettingAsync

Library name and version
Azure.Data.AppConfiguration 1.3.0-beta.2
Describe the bug
We have a ConfigurationClient set up with a non default transport and are seeing calls to GetConfigurationSettingAsync / SetConfigurationSettingAsync throw ObjectDisposedException when using 1.3.0-beta.2 and 1.3.0-beta.1.
Note: The calls succeed when using 1.2.0.
Expected behavior
Calls to GetConfigurationSettingAsync / SetConfigurationSettingAsync succeed
Actual behavior
Calls to GetConfigurationSettingAsync / SetConfigurationSettingAsync throw ObjectDisposedException
Unhandled exception. System.ObjectDisposedException: Cannot access a closed Stream.
   at System.IO.MemoryStream.Read(Byte[] buffer, Int32 offset, Int32 count)
   at System.IO.MemoryStream.ReadAsync(Memory`1 buffer, CancellationToken cancellationToken)
--- End of stack trace from previous location ---
   at System.Text.Json.JsonDocument.ReadToEndAsync(Stream stream, CancellationToken cancellationToken)
   at System.Text.Json.JsonDocument.ParseAsyncCore(Stream utf8Json, JsonDocumentOptions options, CancellationToken cancellationToken)
   at Azure.Data.AppConfiguration.ConfigurationServiceSerializer.DeserializeSettingAsync(Stream content, CancellationToken cancellation)
   at Azure.Data.AppConfiguration.ConfigurationClient.CreateResponseAsync(Response response, CancellationToken cancellation)
   at Azure.Data.AppConfiguration.ConfigurationClient.GetConfigurationSettingAsync(String key, String label, Nullable`1 acceptDateTime, MatchConditions conditions, CancellationToken cancellationToken)
   at Azure.Data.AppConfiguration.ConfigurationClient.GetConfigurationSettingAsync(String key, String label, CancellationToken cancellationToken)
   at ConsoleApp1.Program.Main(String[] args) in C:\Users\ernestt\source\repos\ConsoleApp1\Program.cs:line 37
   at ConsoleApp1.Program.<Main>(String[] args)

Reproduction Steps
Run the following code, replacing <connection_string> with any well formed connection string.
using Azure.Core.Pipeline;
using Azure.Data.AppConfiguration;
using System.Net;
using System.Text;

namespace ConsoleApp1
{
    class TestHttpMessageHandler : HttpMessageHandler
    {
        protected override Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
        {
            return Task.FromResult(new HttpResponseMessage()
            {
                StatusCode = HttpStatusCode.OK,
                Content = new StringContent("{ \"key\": \"abc\", \"value\" :\"def\" }", Encoding.UTF8, "application/json")
            });
        }
    }

    internal class Program
    {
        static async Task Main(string[] args)
        {
            //
            // Use any well formed connection string
            string connectionString = "<connection_string>";

            var client = new ConfigurationClient(
                connectionString,
                new ConfigurationClientOptions
                {
                    Transport = new HttpClientTransport(new TestHttpMessageHandler())
                });

            var result = await client.GetConfigurationSettingAsync("abc");

            var result2 = await client.SetConfigurationSettingAsync("abc", "def");
        }
    }
}

Environment
Windows 11
.NET 6
Visual Studio 17.8 Preview 1.0

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Adding ARM - Core because of https://github.com/Azure/azure-sdk-for-net/issues/38505#issuecomment-1709207185

The bug has been fixed specific to AppConfig and will appear in our October release.  The general form is more nuanced and will require deeper investigation to plan a fix that works in all scenarios.   Shifting this over fully to Azure.Core.

@jsquire Does your fix address #38505 as well, or does that require the general form fix?

@snarkywolverine:  Unfortunately, it does not.   I was able to work around the specific implementation for AppConfig, but we'll still need to solve this in the general case to fix the resource manager packages.
That said, you should be able to work around this by returning a content stream that returns false for IsSeekable or which is not owned by the content type being returned.  That likely means that you'd need to avoid StringContent and implement a custom version, or use another means of returning response content.

This was fixed in https://github.com/Azure/azure-sdk-for-net/commit/c7dbcdebeba5e9a11f0c3aa6e90c26b83b657426#diff-8820ebe47c628d19247979220b62335b2fca905fce27d416c50a0560947f1713


I got a plot that don't look like an inverted cone. Is it Ok?

I got the below plot. Is it correct because I was expecting a shape like an inverted cone?

Sorry.. my question is not relevant to this

Sorry.. my question is not relevant to this


[WIP] Barbican as Key Manager Service for Kubernetes

What this PR does / why we need it:
This PR adds Feature to use Barbican as Key manager service for kubernetes
Which issue this PR fixes:
fixes #44
Special notes for your reviewer:
This Plugin cannot be used in production until[1] in barbican gets implemented.
As per Kubernetes documents[2], we need to return encrypted dek to kubernetes, It stores encrypted dek in etcd, only kek is stored in Key Manager Service. OpenStack Barbican currently does not support to return encrypted secret, It only returns secret id. This Plugin returns the secret id to Kubernetes rather than encrypted secret. It results in problem of storing encrypted data at two different locations and since there is no notification from kuberenetes about secret deletion, user has to manually delete secrets stored in barbican. Once feature[1] gets implemented barbican needs to store only single kek and encrypted secret will be stored in kubernetes etcd only.
[1] https://review.openstack.org/#/c/598389/
[2] https://kubernetes.io/docs/tasks/administer-cluster/kms-provider/
Release note:

NONE.

discussion from slack
dims   [9:11 AM] @adisky around? wanted to chat about the barbican kms (edited)
adisky [9:28 AM] @dims yes, please tell
dims   [9:36 AM] may be just try here… so if you take the aescbc provider, the key needs to be specified in the config right? both encrypt and decrypt are done in the go code itself without talking to a service. So what if we follow the same pattern, except we specify a uuid of a key in the config. Then get the key from barbican and do the rest in the go code. is that possible?
adisky [9:37 AM] @dims yes, it is possible if we do encryption decryption in our code
dims   [9:38 AM]So let’s start there, then we can make things more complex slowly as needed. sounds good? (sorry took longer than expected to get back to you)
adisky [9:40 AM] @dims sure, thanks for your inputs :) , in this way it is non blocking and does not depend on openstack specs
adisky [9:44 AM] @dims But there is one concern here, the key should never leave the KMS and in this case for doing enc dec we need to get the key from barbican
dims   [9:45 AM] @adisky we do this in cinder now. get the key from KMS and use it to encrypt volumes

/close
please reopen when needed (or ping me and i'll take care of it)


[Hub Generated] Review request for Microsoft.HybridCompute to add version preview/2021-01-28-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.  March
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.  March
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.

Given that we want to get the current private link scope's data assigned to a given machine, without needing to know anything more than just the machine resource id, and that there will only ever be at most one, how would you recommend we structure this api?

Re: the SDK Track2 Validation error. I'm not clear what its actually complaining about :(

/azp run

@markcowl I've addressed most of your comments and left a note on the extension section. I'm not sure what it's complaining about at this point, but please take a look as soon as you can.

@markcowl everything looks happier after my latest update

@dkirby017  See my comments on the unresolved comments above.  Underspecifying extensions is a long-standing problem, so if this is urgent, we can file an issue and defer until later.  I would hate to do this with settings/protectSettings, but am open to treating these the same way if your team will fix the problem in an upcoming api-version

@markcowl it is urgent that we get this new api in. It's blocking an upcoming preview of our private link support. I've filed https://msazure.visualstudio.com/One/_workitems/edit/9398639 to address your comments on the extensions and we'll get those in in the next update :)


Cursor placement

I'm not certain if you're in control of this but I've noticed when I click 'next page' the cursor jumps two blocks left to the center position making it a bit clunky to browse through the pages quickly.  If the cursor's position didn't change it would make it easier to scroll through pages.
Thx and sry to bother you over something petty. We do speed builds. Everything's in a hurry.

No, it's fine! It's not petty :P
Yes, I do know about the problem. No, I can't is it without rewriting the entire inventory system. I AM actually planning on doing that (even adding a category system) but I have other code I need to work on right now. The reason why it does this is because when it goes to the next page it closes the inventory, generates the next one, and then opens that. In between the two inventories your cursor resets back to the center of your screen because that's where it always is in-game.
Just a tip, if you do t move your cursor at all after click next/previous it won't update and it'll think you're still on whatever slot you were on before.

On Aug 4, 2016, at 3:19 AM, FatherWh0<EMAIL_ADDRESS>wrote:
I'm not certain if you're in control of this but I've noticed when I click 'next page' the cursor jumps two blocks left to the center position making it a bit clunky to spam click to a certain page quickly. If the cursor's position didn't change it would make it easier to scroll through pages.
Thx and sry to bother you over something petty.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes, too much work for too little an issue, lol. Thx for the time to respond.

No problem! I will probably re-write the whole inventory system soon, but I've been putting of my other work.
Bye!


fix: Ensures that the parsed value is within the bounds of int32

Fixes https://github.com/edgexfoundry/device-onvif-camera/security/code-scanning/2
To fix the problem, we need to ensure that the integer value parsed from the string is within the bounds of int32 before converting it. This can be done by using strconv.ParseInt with a bit size of 32, which directly parses the string into an int32 value, or by adding explicit bounds checks before the conversion.
In this case, we will use strconv.ParseInt with a bit size of 32 to parse the MessageLimit value. This ensures that the parsed value is within the bounds of int32.
Suggested fixes powered by Copilot Autofix. Review carefully before merging.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 32.74%. Comparing base (124a5f1) to head (22d23d7).

Files with missing lines
Patch %
Lines

internal/driver/subscriptionrequest.go
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #475      +/-   ##
==========================================
- Coverage   32.76%   32.74%   -0.02%
==========================================
  Files          21       21
  Lines        1923     1924       +1
==========================================
  Hits          630      630
- Misses       1217     1218       +1
  Partials       76       76

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


Double click should always zoom

When the TMS is active (and the detail modal is already highlighting a single image), double-clicking activates the detail modal and shows available images.

(This does not happen when the thumbnail is active.)

We need to capture the click event on the map to be able to select the appropriate cell.
Apparently adding a click event listener to a layer blocks the propagation of the double click to the map.
This appears to be a limitation of leaflet itself. Instead of checking if a double click is going to happen it just fires the click event.
This can be hacked with timeouts to check for the double clicks, but the easiest option would be to disable double click altogether and make the behavior consistent. The scroll zoom and the zoom buttons would still available.

@smit1678 @mojodna Would disabling double click zoom be ok for you?

@danielfdsilva If we disable double click, does the grid still get deselected if you have a cell already selected?

@smit1678 We don't have any interaction on double click right now, so nothing would really change.

If double click does anything that isn't zooming, I think it's going to be irritating. Personally, I hate scroll to zoom (but I recognize that I'm an outlier) and needing to move to the buttons is similarly tedious.


Use ReactDOM.findDOMNode

This component works with react 0.12.* which i don't think has React.findDOMNode. Will make the change as part of react upgrade, maybe

:+1:


Autocompletion is losing type information in variable initialization

VSCode Version: 1.6.1
OS Version: Windows 10, Build 14393.321

Much to my surprise, any attempt to assign constructor function to a variable doesn't lead to correct autocompletion if that variable used to construct new object.
Example:

What is actually surprising, that doing same operation in object initializer or constructor works:

If if works with object properties, it must work with scope variables as well.

I dived deep into the code and figured that this issue stems from TypeScript, not VSCode itself. The problem in variable identifiers not resolving correctly by getResolvedSymbol function in checker.ts. I think it is better to move issue to TypeScript repository.

@weaknespase thank you for opening this issue. Can you point me to the TypeScript issue you opened? I want to track it on VS Code's end.

@waderyan of course, here it is: https://github.com/Microsoft/TypeScript/issues/11857


Add handling for UUID types

Use the Base.Random.UUID type to parse UUIDs from Postgres.
I used unsafe_string because bytestring is deprecated. You'll need to merge a PR that fixes deprecations to replace those other uses of bytestring and get everything else working too. I suggest enabling femtocleaner on this repo to make that easy :)

Coverage decreased (-0.3%) to 75.225% when pulling 5bf0b285fd5a59685e77d034c242e71ba589e82e on swt30:add-uuid-parsing into a29b45f9d922948dc914d4cc3e34a7352fa29282 on JuliaDB:master.


Make it possible to link the time line of independant worksheets.

When linked worksheets are linked together, they should all be set to the same selected time range.
Changing the time range on one of the linked worksheet should be applied to the others.

Done


missing 3.9.2 tag on GitHub

Also the version on master should be 2.9.3-SNAPSHOT

2.9.3-SNAPSHOT

You mean 3.9.3-SNAPSHOT :-)

I pushed a v3.9.2 tag just now

Also the version on master should be 2.9.3-SNAPSHOT

I'd merge a PR that updated that appropriately, if it's still needed

see #256 for that version change pr


syscall: LoadDLL() bad error formatting on Windows

This is a tiny issue. I use Windows 7 x64. How to reproduce:
echo > corrupted.dll
Create main.go:
package main

import (
	"syscall"
	"fmt"
)

func main() {
	_, err := syscall.LoadDLL("./corrupted.dll")
	if err != nil {
		fmt.Println(err)
	}
}

It writes: Failed to load ./corrupted.dll: %1 is not a valid Win32 application.
Probably there are more similar errors in other places.

Windows returns you ERROR_BAD_EXE_FORMAT. The error message for it (https://msdn.microsoft.com/en-us/library/windows/desktop/ms681382(v=vs.85).aspx) is "%1 is not a valid Win32 application.". So that is what Go prints.
It is unfortunate, but I don't see how we can improve this situation.
Alex

Yes, sorry, I just wrote same program in C++ and got same output.


error in redirect url in login function

for a given aws_cognito_domain the login function returns an error in the html string: https://example.auth.eu-central-1.amazoncognito.com/error?error=redirect_mismatch&client_id=some_id

Can you provide some more information on how you have configured Cognito and what how your routes in Flask are setup?

In the mean time i found out, why the url was faulty. In my case, when the parts of the url e.g. the scope were in the wrong order it had thrown an error. So the information passed for the aws service with the variable full_url had to be in the first place (auth_url then client_id, response_type, redirect_uri) and afterwards i was able to append the state and the code_challenge etc. to not get an error message.
Additionally the host UI could not load correctly if after the aws_cognito_domain was not login? instead of /oauth2/authorize from the authorize_endpoint function in the config file.
E.g. https://auth.example.com/login?client_id=some_id&response_type=code&scope=email+openid&redirect_uri&state=somestring&nonce=somestring&codechallenge=somestring
I configured Cognito with required MFA, required attributes and a custom domain. As i got confused, i tried the example you provided in your repo, but was not able to get it work correctly.
Thanks already :)

According to the AWS Cognito docs, the oauth2/authorize endpoint provides a redirect to either the identity provider or the login endpoint and silently passes through the query parameters. As far as I can tell, this shouldn't be causing any problems, do you have any more logs about what was failing?
The order of the query parameters should not make any difference and it looks like all of the mandatory ones are being populated. I don't have access to a Cognito user pool for testing at the moment so I would be grateful for anymore information you can provide, thanks

also came across the same issue with local development. Having a bit of a problem debugging because there are no obvious indications of what could be wrong. After testing and a bit of debugging, there seems to be no communication to get the session token, and throws an error with code_verifier. Could definetly use some help to figure out what could be done to resolve this issue

Hi there, i configured the cognito credentials as stated in the example application, in a file called .env.example.
The file info looks like this
AWS_REGION= "us-east-2"
AWS_COGNITO_DOMAIN="https://domain.auth.us-east-2.amazoncognito.com"
AWS_COGNITO_USER_POOL_ID="insert cognito user pool id"
AWS_COGNITO_USER_POOL_CLIENT_ID="insert cognito pool client id"
AWS_COGNITO_USER_POOL_CLIENT_SECRET="insert cognito user pool client secret"
AWS_COGNITO_REDIRECT_URL="https://localhost:5000/postlogin"
AWS_COGNITO_LOGOUT_URL="https://localhost:5000/postlogout"
AWS_COGNITO_COOKIE_AGE_SECONDS=1800
BASE_URL="https://localhost:5000"
Both the redirect url. and logout url are on my app integration cognito , under callback urls. I am able to get the redirect to the login page of cognito, however when testing, once the user logs in, the user is redirected to a 500 error page. My plan is that the cognito redirects to a custom html template which is protected. I am testing on local host, however enabled SSL, so my links that are associated to my flask app are https and not http.

my app.py looks like this
from os import path, environ, urandom
from dotenv import load_dotenv
from flask import Flask,redirect, url_for, request,render_template,abort, flash, send_from_directory, jsonify, send_file,session
import json
import pandas as pd
import numpy as np
import datetime
import requests
from flask_cognito_lib import CognitoAuth
from flask_cognito_lib.decorators import(
auth_required,
cognito_login,
cognito_login_callback,
cognito_logout,
)
from flask_cognito_lib.exceptions import (
AuthorisationRequiredError,
CognitoGroupRequiredError,
)
class Config:
#Set Flask configuration vars from .env file.
# General Config
SECRET_KEY = environ.get("SECRET_KEY", urandom(32))
FLASK_APP = "TEST_APP"
FLASK_ENV = "TESTING"

# Cognito config
# AWS_COGNITO_DISABLED = True  # Can set to turn off auth (e.g. for local testing)
AWS_REGION = environ["AWS_REGION"]
AWS_COGNITO_USER_POOL_ID = environ["AWS_COGNITO_USER_POOL_ID"]
AWS_COGNITO_DOMAIN = environ["AWS_COGNITO_DOMAIN"]
AWS_COGNITO_USER_POOL_CLIENT_ID = environ["AWS_COGNITO_USER_POOL_CLIENT_ID"]
AWS_COGNITO_USER_POOL_CLIENT_SECRET = environ["AWS_COGNITO_USER_POOL_CLIENT_SECRET"]
AWS_COGNITO_REDIRECT_URL = environ["AWS_COGNITO_REDIRECT_URL"]
AWS_COGNITO_LOGOUT_URL = environ["AWS_COGNITO_LOGOUT_URL"]
AWS_COGNITO_COOKIE_AGE_SECONDS = environ["AWS_COGNITO_COOKIE_AGE_SECONDS"]

app = Flask(name)
app.config.from_object(Config)
auth = CognitoAuth(app)
Content Security Policy Header
csp_header = {
'default-src': ["'self'"],
'script-src': ["insert https", "insert https"],
# Add other directives as needed
}
@app.route("/login/")
@cognito_login
def login():
pass
@app.route("/postlogin", methods =('GET','POST'))
@cognito_login_callback
def postlogin():
return redirect(url_for("/login_success"))
@app.route("/login_success")
@auth_required()
def login_success():
return render_template("login_success.html")
@app.route("/func_one/", methods =('GET','POST'))
@auth_required()
def func_one():
.. other code ..
return render_template("func_one.html")
@app.route("/func_two/", methods =('GET','POST'))
@auth_required()
def func_two():
.. other code ..
return render_template("func_two.html")
@app.route("/func_three/", methods = ['POST','GET'])
@auth_required()
def func_three():
.. other code ..
return render_template("func_three.html")
@app.route("/func_four/", methods = ['POST','GET'])
@auth_required()
def func_four():
.. other code ..
return render_template("func_four.html")
@app.route("/func_five/", methods = ['POST','GET'])
@auth_required()
def func_five():
.. other code ..
return render_template("func_five.html")
@app.route("/func_six/", methods = ['GET','POST'])
@auth_required()
def func_six():
.. other code ..
return render_template("func_six.html")
@app.route("/func_seven/", methods = ['GET','POST'])
@auth_required()
def func_seven():
.. other code ..
return render_template("func_seven.html")
@app.route("/logout")
@cognito_logout
def logout():
pass
@app.route("/postlogout")
def postlogout():
return render_template('login.html')
@app.route("/messages/int:idx")
def message(idx):
messages = ['Message Zero', 'Message One', 'Message Two']
try:

    return render_template('message.html', message=messages[idx])
except IndexError:

    abort(404)

@app.errorhandler(404)
def page_not_found(error):
return render_template("404.html"),404
@app.route("/500/")
def error500():
abort(500)
@app.errorhandler(500)
def internal_error(error):
return render_template('500.html'),500
@app.after_request
def apply_csp(response):
response.headers['Content-Security-Policy'] = '; '.join([f"{key} {' '.join(value)}" for key, value in csp_header.items()])
return response
@app.errorhandler(AuthorisationRequiredError)
def auth_error_handler(err):
return redirect(url_for("login"))
@app.errorhandler(CognitoGroupRequiredError)
def missing_group_error_handler(err):
return jsonify("Group memberhship does not allow acces to this resource"),403
if name == "main":
app.secret_key ='super secret key'
app.debug = True
app.run(ssl_context=('somefile.pem','somefile.pem'))

I've run into same error while trying to setup and test login functionality with cognito, the problem for me was having http instead of https in cognito redirect.
I'm facing new problem afterwards, since after login I'm getting the error:

flask_cognito_lib.exceptions.CognitoError: Error getting public keys from Cognito]

and I'm not sure how to handle this one, is there additional setting or some param to setup?

I haven't been able to replicate this one, but then again I don't have easy access to Cognito at the moment.
To validate the signature, we use get_signing_key_from_jwt() for PyJWT. From the Cognito docs:

The JWKS URI contains public information about the private key that signed your user's token. You can find the JWKS URI for your user pool at https://cognito-idp.<Region>.amazonaws.com/<userPoolId>/.well-known/jwks.json.

So I guess first thing to verify is if you can connect and obtain the JWKs from that endpoint


Various docs need to be updated for addition of NAUTOBOT_ROOT and removal of BASE_STORAGE_DIR

Environment

Python version: n/a
Nautobot version: 1.0.0b1

Expected Behavior

NAUTOBOT_ROOT should be documented in either optional-settings.md or required-settings.md, especially its required interaction with the $NAUTOBOT_ROOT environment variable.
BASE_STORAGE_DIR should no longer be referenced in optional-settings.md or in docs/configuration/index.md

Observed Behavior

Neither of the settings documentation files mention NAUTOBOT_ROOT
References to BASE_STORAGE_DIR were not removed in #74.

I'm trying to follow the installation instructions that were just committed today, find that the nautobot-server init step is writing the new configuration file to /opt/nautobot/.nautobot/nautobot_config.py. The nautobot-server migrate follow-up step expects to find the configuration at /opt/nautobot/nautobot_config.py`, isn't happy about the discrepancy.
Is there an environment variable I can set to work around this?
Thanks!

Hi, can you confirm that you set the $NAUTOBOT_ROOT environment variable to /opt/nautobot/ before running nautobot-server init?

nautobot@ip-172-31-73-196:~$ printenv NAUTOBOT_ROOT /opt/nautobot nautobot@ip-172-31-73-196:~$ nautobot-server init Configuration file created at '/opt/nautobot/.nautobot/nautobot_config.py' nautobot@ip-172-31-73-196:~$

Oh, sorry! We haven't released v1.0.0b2 yet (aiming for today, but we'll see) so at the moment if you did a pip install nautobot you're still getting v1.0.0b1, which doesn't have the NAUTOBOT_ROOT changes made in #74. If you want the version that matches the docs (with the fixes therein) in the meantime you'll need to pip3 install git+https://github.com/nautobot/nautobot@develop to get the latest bleeding-edge version. Sorry for any confusion!

Excellent, thank you. I saw #74, figured it was related, didn't realize I had only half of the ingredients :)

@chrismarget For the moment if you'd like to continue, please try to install Nautobot directly from the develop branch.
Instead of:
pip3 install nautobot

Run:
pip3 install git+https://github.com/nautobot/nautobot@develop


Run nightly promotion build once per week for old release branches

For old release branches, we still want to run nightly promotion job infrequently, like once per week.
This is cherry-picked from https://github.com/gradle/gradle/pull/23717

@bot-gradle test and merge

I've triggered a build for you.


[BUG] Swarms Environment Variables and Template Swarms

Describe the bug
Was running  Legal-Swarm-Template/main.py in a colab notebook with python 3.10:
Traceback (most recent call last):
  File "/content/Legal-Swarm-Template/main.py", line 7, in <module>
    from swarms.agents.create_agents_from_yaml import (
  File "/usr/local/lib/python3.10/dist-packages/swarms/__init__.py", line 6, in <module>
    from swarms.telemetry.bootup import bootup  # noqa: E402, F403
  File "/usr/local/lib/python3.10/dist-packages/swarms/telemetry/bootup.py", line 7, in <module>
    from swarms.utils.disable_logging import disable_logging
  File "/usr/local/lib/python3.10/dist-packages/swarms/utils/__init__.py", line 2, in <module>
    from swarms.utils.data_to_text import (
  File "/usr/local/lib/python3.10/dist-packages/swarms/utils/data_to_text.py", line 5, in <module>
    from swarms.utils.pdf_to_text import pdf_to_text
  File "/usr/local/lib/python3.10/dist-packages/swarms/utils/pdf_to_text.py", line 2, in <module>
    from swarms.utils.try_except_wrapper import try_except_wrapper
  File "/usr/local/lib/python3.10/dist-packages/swarms/utils/try_except_wrapper.py", line 5, in <module>
    from swarms.utils.loguru_logger import logger
  File "/usr/local/lib/python3.10/dist-packages/swarms/utils/loguru_logger.py", line 8, in <module>
    os.path.join(WORKSPACE_DIR, "swarms.log"),
  File "/usr/lib/python3.10/posixpath.py", line 76, in join
    a = os.fspath(a)
TypeError: expected str, bytes or os.PathLike object, not NoneType

The error is occurring because WORKSPACE_DIR, which is used for logging is not set.
The .env.example fot Legal-Swarms-Template only includes the OpenAI and Groq api keys.
When someone uses the template, and not the swarms source install, they would not see the full list of environment variables assumed to be available in swarms, as shown in swarms/.env.example
Perhaps a more robust swarms install process, or configuration tool could be used to set the env vars in projects that use swarms.

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Bug resolved in the legal template, you have to declare WORKSPACE_DIR before loading env vars with dotenv.
This solution works across different swarms examples/cookbooks.


hpcprof takes long time and no response

After I used mpirun -np 3 hpcrun ....., hpcstruct, I used hpcprof-mpi -S ... -I ... [measure dir], however, I wait a long time but this command only give me following output:
 % hpcprof -S main.hpcstruct -I ../src/wrapper_mpi_new.cpp hpctoolkit-main-measurements
msg: STRUCTURE: ~/workspace/MultiVersion/DefaultImplFinder/cpp_wrapper_multi_process/build/main
msg: Line map : ~/workspace/spack/opt/spack/linux-ubuntu20.04-x86_64/gcc-9.3.0/hpctoolkit-2021.03.01-ucxspqiohaofwv5pno64q2exzvgveqq3/lib/hpctoolkit/libhpcrun.so
msg: Line map : ~/workspace/spack/opt/spack/linux-ubuntu20.04-x86_64/gcc-9.3.0/hpctoolkit-2021.03.01-ucxspqiohaofwv5pno64q2exzvgveqq3/lib/hpctoolkit/ext-libs/libmonitor.so.0.0.0

And there's nothing in main database dir, is there something wrong or I just need to wait for more time?

This repository has moved to GitLab, please continue this issue there.


[Sema] Do not attempt warn extraneous checked cast for placeholder types

As mentioned here those fixes shouldn't be recorded for placeholder types.

@swift-ci Please smoke test


Just want to say thank you

Hi,
I Just want to say Thank you for your "butterchurn". I use it in my personal music app, really awesome !

Thanks so much! Really cool to see people using butterchurn on their own. Super cool looking app too!!!
Thanks!

@LeFrach Hang on what's your player like? How do I get in touch?


New version: CeVIO.CeVIOAI version 9.1.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @Exorcism0666,

This PR's version number 9.1.1 has 3 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version <IP_ADDRESS>, which has 4 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 719)

@stephengillie Yes!


Attempting to free stack allocated channel_s object

Version 0.7.0.beta8
When building a release with gcc (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0 I see the following:
In function ‘fio_channel_free.part.26’,
    inlined from ‘fio_ch_set__insert_or_overwrite_.constprop’,
    inlined from ‘fio_filter_dup_lock_internal’ at lib/facil/fio.h:6127:10,
    inlined from ‘fio_filter_dup_lock’ at lib/facil/fio.c:5383:10,
    inlined from ‘fio_subscribe’ at lib/facil/fio.c:5455:8:
lib/facil/fio.c:5151:3: warning: attempt to free a non-heap object ‘ch’ [-Wfree-nonheap-object]
   free(ch);
   ^~~~~~~~
In function ‘fio_channel_free.part.26’,
    inlined from ‘fio_ch_set__insert_or_overwrite_.constprop’,
    inlined from ‘fio_filter_dup_lock_internal’ at lib/facil/fio.h:6127:10,
    inlined from ‘fio_channel_match_dup_lock.isra.67’ at lib/facil/fio.c:5416:14,
    inlined from ‘fio_subscribe’ at lib/facil/fio.c:5457:8:
lib/facil/fio.c:5151:3: warning: attempt to free a non-heap object ‘ch’ [-Wfree-nonheap-object]
   free(ch);
   ^~~~~~~~
In function ‘fio_channel_free.part.26’,
    inlined from ‘fio_ch_set__insert_or_overwrite_.constprop’,
    inlined from ‘fio_filter_dup_lock_internal’ at lib/facil/fio.h:6127:10,
    inlined from ‘fio_channel_dup_lock.isra.68’ at lib/facil/fio.c:5396:14,
    inlined from ‘fio_subscribe’ at lib/facil/fio.c:5459:8:
lib/facil/fio.c:5151:3: warning: attempt to free a non-heap object ‘ch’ [-Wfree-nonheap-object]
   free(ch);

Hi @adamansky ,
Thank you very much for opening this issue.
I can't replicate the issue at the moment, I'll need to upgrade my gcc and VM for that, which I'll try to do tonight.
In the meantime, I wonder - could this be a compiler error?
The stack memory is assigned a high reference count, preventing the fio_channel_free function from freeing stack allocated objects.
The result is that stack objects are either ignored or that they are "copied" to the heap (with a new reference count), but never freed.
In this case, the if (fio_atomic_sub(&ch->ref, 1)) statement is always true (always 7, which is luckier than always -1, but produces the same result).
Did I miss something?
Kindly,
Bo.

I installed GCC 7... which installed version 7.4.0 (gcc version 7.4.0 (Ubuntu 7.4.0-1ubuntu1~16.04~ppa1))...
...the warning doesn't seem to pop up. I can't reproduce.
Perhaps it's a GCC issue that was fixed?

Hi Boaz! I think compiler is quite objective here, for example
In function ‘fio_channel_free.part.26’,
    inlined from ‘fio_ch_set__insert_or_overwrite_.constprop’,
    inlined from ‘fio_filter_dup_lock_internal’ at lib/facil/fio.h:6127:10,
    inlined from ‘fio_channel_match_dup_lock.isra.67’ at lib/facil/fio.c:5416:14,
    inlined from ‘fio_subscribe’ at lib/facil/fio.c:5457:8:

Where ch is placed on stack in fio_channel_match_dup_lock() and then used in fio_channel_free() It seems the compiler is not smart enough to prove that fio_atomic_sub(&ch->ref, 1) will be always true in runtime. May be 7.4.0 is able to derive it :)
Yes I see your comment: /* avoid freeing stack memory */ in https://github.com/boazsegev/facil.io/blob/master/lib/facil/fio.c#L5395
So seems to be no problems here. Thank you!


Firefox not reaching 100% in progress (always stops between 94 and 97% or so)

It seems that Firefox does not fire the last progress event. I printed out the (data.loaded / data.total) * 100 ratio in every callback and it never reaches 100%.
Even in the last callbacks done/always it's never 100%.
Debug code (just paste it into firebug on your demo page and watch the console as you upload a file)
$('#fileupload').bind('fileuploaddone', function (e, data) { console.log('done', ((data.loaded/data.total) * 100)) });
$('#fileupload').bind('fileuploadfail', function (e, data) { console.log('fail', ((data.loaded/data.total) * 100)) });
$('#fileupload').bind('fileuploadalways', function (e, data) { console.log('always', ((data.loaded/data.total) * 100)) });
$('#fileupload').bind('fileuploadprogress', function (e, data) { console.log('progress', ((data.loaded/data.total) * 100)) });
$('#fileupload').bind('fileuploadprogressall', function (e, data) { console.log('progressall', ((data.loaded/data.total) * 100)) });

See Firefox Bug #642463.
If desired, you can workaround this bug by manually setting the progress bar to 100% when the load event (done event) fires.

yeah, that's what i did. but thanks for the link. so we can just wait for now :(


Improve handling of arrays with empty values

The primary issue here is that Ecto removes "" as empty, but doesn't filter [""] to [] as well. This PR changes that behaviour.

I agree that explicit is better. But, a list of values is just that - a list of values. If a value inside that list would be considered "empty" when not in a list, it should be considered empty if inside a list. By considering them to be different, I think we're sacrificing predictability.
Instead of doing it automatically, would you consider another API for the same functionality? Maybe an option can be passed in alongside empty_values, or another changeset function?

To clarify, this PR does not make [] an "empty value". It simply strips "empty values" from a list of values, like so ->
%{"topics" => ["", "", "", ""]} -> %{topics: []}

It is something that can be trivially handled before calling cast, so I don't see a strong reason to support it in Ecto. As mentioned, it is not a very common use case and sometimes it is simpler to handle it  upstream. Thanks.

Okay, thank you

Just to add a voice on the not-touching-it side, I really really would not like 'empty' values being stripped from an array because the empty value is a valid value in an array and it's very presence is a marker that something is there but that something is just null or "" or so.

@OvermindDL1 yeah, I understand, thanks for sharing.
I've thought about it some more, and agree with @josevalim that it's not a good idea to do automatically.


Error

Traceback (most recent call last):
File "main.py", line 22, in
File "clr_loader/init.py", line 40, in get_coreclr
File "clr_loader/hostfxr.py", line 13, in init
File "clr_loader/ffi/init.py", line 28, in load_hostfxr
RuntimeError: Could not find a suitable hostfxr library in /usr/local/share/dotnet
[18198] Failed to execute script 'main' due to unhandled exception!
Exception ignored in: <function DotnetCoreRuntime.del at 0x11390b940>
Traceback (most recent call last):
File "clr_loader/hostfxr.py", line 88, in del
File "clr_loader/hostfxr.py", line 83, in shutdown
AttributeError: 'DotnetCoreRuntime' object has no attribute '_handle'
i am using macos air m1

ARM is not currently supported.


Feature Suggestion: Make high water marks immediately available in partition consumers

Currently the high water mark of a partitionConsumer isn't filled in until it's processed at least one fetch response. This makes life hard for consumers e.g. reading compaction topics from the start that want to delay until they have reasonably "caught up" to the topic end, in the case when the topic is empty. You need to either detect this from the Client (which may not be available if you used e.g. NewConsumer or want your code to also work with mock consumers), or assume no HWM within some time span is an empty topic, or put a dummy first message in all such topics.
I think during chooseStartingOffset, the HWM could be filled in from the offsetNewest lookup. This would make it available by the time ConsumePartition returns.

Hello, I also encountered this issue and was surprised that it works this way, and just worked around it. But now, after reading your great description with a solution basically in there, I went ahead and created PR: https://github.com/Shopify/sarama/pull/2082

Hello, I also encountered this issue and was surprised that it works this way, and just worked around it. But now, after reading your great description with a solution basically in there, I went ahead and created PR: https://github.com/Shopify/sarama/pull/2082

Good point. To be honest, I'm not that deep into Kafka, or Sarama, so...not sure. But when I look at this:
func (client *client) getOffset(topic string, partitionID int32, time int64) (int64, error) {
	broker, err := client.Leader(topic, partitionID)
	if err != nil {
		return -1, err
	}

	request := &OffsetRequest{}
	if client.conf.Version.IsAtLeast(V0_10_1_0) {
		request.Version = 1
	}
	request.AddBlock(topic, partitionID, time, 1)

	response, err := broker.GetAvailableOffsets(request)

... and this:
type OffsetRequest struct {
	Version        int16
	IsolationLevel IsolationLevel
	replicaID      int32
	isReplicaIDSet bool
	blocks         map[string]map[int32]*offsetRequestBlock
}

func (r *OffsetRequest) encode(pe packetEncoder) error {
	if r.isReplicaIDSet {
		pe.putInt32(r.replicaID)
	} else {
		// default replica ID is always -1 for clients
		pe.putInt32(-1)
	}

	if r.Version >= 2 {
		pe.putBool(r.IsolationLevel == ReadCommitted)
	}

then I would say that all we need to do is use v2 (or newer, if any) when applicable. WDY?

The ReadCommitted isolation level is only relevant for transactional producers/consumers, I think, not the general idea of data "committed" by completing replication.  But in the end I'm fine with what got merged, we don't depend on absolutely monotonic HWMs especially in such an edge case. Thanks for doing the actual work! :)


Change some of the language to be more inclusive.

Adhere to the inclusive language requirements.

Thank you!

Thank you!


Fix windows build error

src/aqt/cassowary/Context.hpp(86):
warning C4127: conditional expression is constant
@jbo-ableton LGTMed verbally

Fixes https://github.com/AbletonAppDev/cassowary/issues/10


[feat]separate theme color for border and background

thanks!

Style Customization is under development. Maybe this weekend I will release a new version.

new version released! Now there are three ways for style customization!


chore: bump deps

Update k8s to 0.27.2
Pull Request

What? (description)
Why? (reasoning)
Acceptance
Please use the following checklist:

[ ] you linked an issue (if applicable)
[ ] you included tests (if applicable)
[ ] you ran conformance (make conformance)
[ ] you linted your code (make lint)
[ ] you linted your code (make unit)

See make help for a description of the available targets.

/m


Carousel should be less drag sensitive

It's really hard to scroll vertically with a full width carousel on a smartphone. Delta should be at least 8.

closed in favor of #4415


Anchor navigation

Blueprint
We need a blueprint for an anchor nav component that matches the features we want to have.
Documentation & research

Mobile design
Desktop design

Outline tasks

[x] anatomy
[x] color
[x] typography
[x] layout
[x] spacing
[x] animation/behavior
[x] variants
[x] states (hover, focus, active, focus-visible)
[x] a11y

Optional

[x] Competitive analysis
[x] research
[x] site audit
[x] usage audit
[x] inspirational work

@leeejune Engineering approves design work. Ticket opened for code implementation here: #10.


3.21.1 (ENG) + (GER)

Current State: 3.21.1 PTU 8889500
English done
German finished until 49143

Current State: 3.21.1 PTU 8892745

[x] English
[x] German

3.21.1 PTU 8892745 is the Live State of the global.ini

I'm not sure why, but now the tables are fixed on GitHub pages.

I am done. Ready for Merging!


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 2188 ms

Resolved: Bofrost.at performance has improved in abc4655 after 37 minutes.


CGG1 test added

white list check fixed: xiaomi_mac is binary data so lower() does not make sense here

And I think there is issue with project structure - tests use code from installed packages instead of source code

And I think there is issue with project structure - tests use code from installed packages instead of source code

Yes, I know, I still have to fix that.

Are you able to resolve the conflict, I only have the option to do it via the command line? I guess it is due to the changed location of the tests folder. Otherwise, I will add your change later manually.
Thanks for your contribution!


GCP Workload Identity - Secret Manager in different project than GKE cluster

Describe the solution you'd like
Currently, the Workload Identity implementation assumes that Google Secret Manager will be used in the same project where the GKE cluster is located. This limitation is not documented anywhere and I had to dig up the code to find it.
link to code
	idProvider := fmt.Sprintf("https://container.googleapis.com/v1/projects/%s/locations/%s/clusters/%s",
		spec.Provider.GCPSM.ProjectID,
		wi.ClusterLocation,
		wi.ClusterName)
	idPool := fmt.Sprintf("%s.svc.id.goog", spec.Provider.GCPSM.ProjectID)

This is not true in most Enterprise setup, using GCP Organisations. In our use-case GKE clusters are managed separately in a different GCP project than the projects where Google Secret Manager is used.
What is the added value?
The External Secrets service can be used in a more distributed GCP project setup which is widely used in Google Cloud Landing Zones.
Give us examples of the outcome
I propose the addition of spec.provider.gcpsm.auth.workloadIdentity.clusterProjectID field
kind: ClusterSecretStore
metadata:
  name: example
spec:
  provider:
    gcpsm:
      projectID: my-project
      auth:
        workloadIdentity:
          # the projectID where the GKE cluster resides
          clusterProjectID: example-project-id
          # name of the cluster region
          clusterLocation: europe-central2
          # name of the GKE cluster
          clusterName: example-workload-identity
          # reference the sa from above
          serviceAccountRef:
            name: team-a
            namespace: team-a

I would say the proposed change wouldn't break the current API either as we can introduce the new field (clusterProjectID) as optional and keep the current behaviour as is.

yes, true but think that in the end always using clusterProjectID instead of default to the projectID is better. It creates some duplication but makes it more clear

agree again that it is best to move forward as you sugested, i'll have a dash at it

@AtzeDeVries If you need any help on the topic or you don't have the time in the end feel free to ping me as this is a highly desired feature for us so we can help with the implementation.


Enhanced gitlab-asg module, added output to the nat module

Added output for the nat ec2 instance private ip address
Added load_balancers and public_ip parameters to the gitlab-asg  module

These updates have been merged, this is good to close.


ibus-engines.typing-booster-unwrapped: 2.1.2 -> 2.1.3

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ibus-typing-booster/versions.
meta.description for ibus-engines.typing-booster-unwrapped is: '"A typing booster engine for the IBus platform"'.
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3/bin/emoji-picker had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3/bin/.emoji-picker-wrapped had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 2.1.3 with grep in /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3
directory tree listing: https://gist.github.com/c6058cf558e9108812ae1c47314f29dc
du listing: https://gist.github.com/149167cb7a7fea5959cc87201978efcc

Outpath report (click to expand)

Outpath difference report
2 total rebuild paths
2 package rebuilds
2 x86_64-linux rebuilds
0 i686-linux rebuilds
0 x86_64-darwin rebuilds
0 aarch64-linux rebuilds
First ten rebuilds by attrpath
ibus-engines.typing-booster
ibus-engines.typing-booster-unwrapped

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A ibus-engines.typing-booster-unwrapped https://github.com/r-ryantm/nixpkgs/archive/c281799090f851628675a4d651418c96fea29bc4.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3
ls -la /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3/bin

cc @ma27 for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: ibus-engines.typing-booster-unwrapped
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: ibus-engines.typing-booster-unwrapped
Partial log (click to expand)
Wrapping program /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3/libexec/ibus-engine-typing-booster
shrinking RPATHs of ELF executables and libraries in /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3/libexec  /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3/bin
patching script interpreter paths in /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3
/nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3/libexec/.ibus-engine-typing-booster-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
/nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3/libexec/.ibus-setup-typing-booster-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
/nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3/bin/.emoji-picker-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3...
/nix/store/agql2k3s8dxa8mglhapy4088q4gi3zxp-ibus-typing-booster-2.1.3

Success on aarch64-linux (full log)
Attempted: ibus-engines.typing-booster-unwrapped
Partial log (click to expand)
Wrapping program /nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3/libexec/ibus-engine-typing-booster
shrinking RPATHs of ELF executables and libraries in /nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3/libexec  /nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3/bin
patching script interpreter paths in /nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3
/nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3/libexec/.ibus-engine-typing-booster-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
/nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3/libexec/.ibus-setup-typing-booster-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
/nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3/bin/.emoji-picker-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3...
/nix/store/gq5r93cl73qf6h6sbdl6ssxksgq7s1r5-ibus-typing-booster-2.1.3


Centralised collection of nautobot common commands

Environment

Python version: 3.8.10
Nautobot version: 1.1.2

Proposed Functionality
Throughout both the core nautobot documentation and nautobot plugin documentation, there are a series of helpful/common commands they are dispersed through that documentation.
Examples of commands or references in doco:
https://github.com/nautobot/nautobot-plugin-device-onboarding#final-configuration-steps
https://github.com/nautobot/nautobot-plugin-chatops/blob/develop/docs/chat_setup/chat_setup.md#server-configuration
A possible idea is to have a "core" page with the command ID, command, user role, brief description, and why you would run that command. An example is below:
Use Case

Command ID
User Level Access
Command
Description
Reason

CMD001
sudo
sudo systemctl status nautobot.service
Display the system control status of the nautobot.service
Used for validating the nautobot service status

CMD002
sudo
sudo systemctl status nautobot-worker.service
Display the system control status of the nautobot-worker.service
Used for validating the nautobot-worker service status

CMD003
sudo
sudo systemctl restart nautobot.service
Restart the nautobot.service
Used for restarting the nautobot service status. Commonly used when a change is made to the configuration.

CMD004
sudo
sudo systemctl restart nautobot-worker.service
Restart the nautobot-worker.service
Used for restarting the nautobot service status. Commonly used when a change is made to the configuration.

CMD005
Standard (nautobot user)
nautobot-server --version
Display the current version of nautobot
Used for displaying the current version of the nautobot-server, in case it's needed for troubleshooting purposes

Then, all the associated documentation could have something like below:
After doing x thing, you need to restart the nautobot-worker service. Please refer to CMD004 on the common commands page.
This would prevent repetition and also ensure that people can understand what the commands are used for and why.
Database Changes
None
External Dependencies
None

FYI @tim-fiola

In relation to the nautobot-server <x> commands, it should be noted that there is already some great documentation on this:
https://nautobot.readthedocs.io/en/stable/administration/nautobot-server/#available-commands
I wasn't aware of this and adding here for visibility


Docs do not explain purpose of serviceAccount.secrets field

The following documentation do not provide enough information about what the ServiceAccount.secrets field is for:

https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/
https://kubernetes.io/docs/admin/service-accounts-admin/
https://kubernetes.io/docs/concepts/configuration/secret/#service-accounts-automatically-create-and-attach-secrets-with-api-credentials

The only piece of documentation appears on API reference: https://github.com/kubernetes/kubernetes/blob/04c4c212266f3426fc770580b9d791fcef2531bb/pkg/api/types.go#L2785:

// Secrets is the list of secrets allowed to be used by pods running using this ServiceAccount

...but even this is confusing to me. It sounds like RBAC, but is it really? Having a bit of more info on the documentation would certainly help. I think this field is intended for some use case, though I don't think users can understand it easily right now.
/assign @bprashanth

Found a bit of information at https://github.com/kubernetes/kubernetes/blob/04c4c212266f3426fc770580b9d791fcef2531bb/plugin/pkg/admission/serviceaccount/admission.go#L309-L324.
My deductions:

this field is used by the ServiceAccount admission controller
If a user is not making use of this admission controller, this field is a no-op
the same functionality can be achieved with RBAC.

/cc @liggitt @deads2k if you can shed any light, or submit a small PR explaining this, I appreciate.

/cc @liggitt @deads2k if you can shed any light, or submit a small PR explaining this, I appreciate.

A service account can be used to provide a level of indirection when a system actor is performing an action on a user's behalf.  For instance, when a user creates an RC, the pod that finally gets created is created by the RC controller identity, not the user's identity.  If you want to be able to restrict things like PSP permissions, those permissions must be attached the SA, not the user, since the user information is gone at the time of pod creation.

the same functionality can be achieved with RBAC

The admission plugin uses the references in that field to determine which service account token secret to automount into pods.

@deads2k I guess your comment is a conceptual overview of service accounts (which is really helpful and I'm not sure if we explain it like this). But it doesn't explain what "secrets" field does, we lack that part of the documentation
@liggitt it seems like the token (which I suppose is .secrets[0]) is the only one that is auto-mounted. The rest of .secrets field is not auto-mounted at all (not implemented maybe).

@ahmetb :wave: Do you think there's enough information here to be actionable for docs contributors?

@zacharysarah Not sure if docs contributors would be able to provide the required content. So I'm looking at folks above to provide authoritative information, and preferably a PR. 😉

/priority important-longterm

Can we close this issue then?

Can we close this issue then?

@ahmetb as I understand it, this request remains open and valid. Please respond if you feel that's not the case.
/triage accepted

@ahmetb as I understand it, this request remains open and valid. Please respond if you feel that's not the case.
/triage accepted

/sig auth

https://kubernetes.io/docs/reference/kubernetes-api/authentication-resources/service-account-v1/#ServiceAccount now partly covers this.

/kind documentation

https://kubernetes.io/docs/concepts/security/service-accounts/ explains how that field is now a legacy detail.
On that basis, I'll close this issue.
/close


Docker build on master results in error

When I run docker build -t shields . on a clean checkout I get the following error:
> Failed to build
{ Error: (client) ./badge-examples.json
Module build failed: SyntaxError: Unexpected token F in JSON at position 0
    at JSON.parse (<anonymous>)
    at Object.module.exports (/usr/src/app/node_modules/json-loader/index.js:4:49)

I checked badge-examples.json in the newly generated Docker image layer, even parsed it with jq '' badge-examples.json, but it parses fine.
The full output is at: https://pastebin.com/6ctb53qw
Running npm run build on my local machine runs successfully.
I tried some things, like updating the Docker base image to node:10.11-alpine or downgrading the glob dependency (since json-loader is a transitive dependency on glob). All to no avail.
Any ideas?

@agboom I found the same thing.
Tried different node versions too with no success :(

I've got a workaround for the above issue that's working for me.
I also hopped into the failed Docker image layer and inspected the generated badge-examples.json file. The file content looked like this:
FALLBACK_FONT_PATH is deprecated. Please use FONT_PATH.
[{"category":{"id":"build","name":"Build"},"examples" ...

That first line doesn't look like valid json to me!
Searching the repo, it seems like that messaging is a deprecation warning that comes from here:
https://github.com/badges/shields/blob/master/lib/server-config.js#L78
So, I just commented out the console.log line in server-config.js and the docker build ... was able to complete successfully. Doubt it'll break anything as it's just a logging line, but 🤷♂ .
Hopefully one of the maintainers can figure out whatever is using FALLBACK_FONT_PATH and fix it there so the generated json file is valid again.

Thanks for helping track this down - the fix proposed in #2179 should solve this. If any of you want to confirm that solves the issue before we merge, that would be helpful.

The proposed fix works for me, I was able to successfully build a Docker image with the change.
Kicking myself for missing the ENV in the Dockerfile... Thanks @chris48s !

The proposed fix works for me, was able to successfully build a new Docker image.
Kicking myself for missing the ENV in the Dockerfile... Thanks @chris48s !

Me too, I build it with 8.12.0-alpine


Feature Request: Bring all timers to the front and tile and sort them from shortest time remaining

I would really appreciate an option where I can bring all the timers to the front by bringing any of them to the front of the windows, always having them tiled neatly, and sort them by title or by shortest time remaining.
Thank you.

Implemented in https://github.com/i2van/hourglass and much more.

@i2van Thank you. An option of sorting by title first, then time left, would be really what I need from an app like this. I use multiple simultaneous timers, and give them custom titles that help in categorization.

Tracked by https://github.com/i2van/hourglass/issues/22

@Wisdawn You can get PoC of sorting by title in notification area menu here. Is it what you need?


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2024-10-30T20:45:02+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-discovery_istio_release-1.24
4e52c84ad9a837a95cc14c3cabd48615286cf32e
link
true
/test integ-telemetry-discovery

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


Write documentation for /e-link/explore endpoint

Using the /e-link/explore endpoint a client can retrieve description of a resource from a given endpoint. The endpoint can be SPARQL or Linked Data Fragments endpoint.
The API accepts three parameters:

resource - a URI of the resource which should be described. Note that the URI should be UTF-8 encoded.
endpoint - a URL of the endpoint which should be used to retrieve info about the resource. Note that the URL should be UTF-8 encoded.
endpoint-type - the type of the endpoint. Possible values: sparql or ldf

@m1ci thanks
@josauder please add a new api endpoint to the full api documentation in e-link as described here.

Thanks. Some updates:

Please start the parameter descriptions with a capital letter
Please provide an example. So find a useful example, e.g. explore the resource for berlin in dbpedia


Spring Boot QuickStarter fails to deploy when provisioned

When someone deploys a new clean SpringBoot QS Jenkins reports:
Recording test results
No test report files were found. Configuration error?

The QS should work out of the box and If someone hasn't written any tests yet the following line should be comment out:
junit 'build/test-results/test/*.xml'

But I would say that is not the desirable solution.

Duplicate of https://github.com/opendevstack/ods-quickstarters/issues/50.
I'm closing the issue here since 1.2.x worked, but master of ods-project-quickstarters does not but will not be fixed as it won't be part of the 2.0 release.


Add E2E test for nodeSelector based on ResourceFlavor

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Ensure basic fungibility works E2E
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Builds on top of #894 (first commit)
Does this PR introduce a user-facing change?

NONE

/assign @trasc

Somehow it looks like the actual pod is not running:
e2e-hrqff            test-job-pxrw5                               0/1     Pending   0          50s

/hold
for #894 to merge first

/hold cancel

/lgtm


use-first-commit-message-for-new-prs overrides user set title when refreshing page

Description
When opening a PR and refreshing the page, Github will keep the previously entered title and body.
When use-first-commit-message-for-new-prs is enabled, refreshing the page will overwrite the title with the first commit.

 Without `use-first-commit-message-for-new-prs`
  With `use-first-commit-message-for-new-prs`

How to replicate the issue + URL
https://github.com/pgimalac/refined-github/compare/main...pgimalac:refined-github:pgimalac/fit-rendered-markdown?expand=1
I guess the above URL will only work if you have permissions on the source/target repo, you can use any URL for opening a PR.
Note that it doesn't seem to reproduce for cross-repo PRs.
Extension version
23.12.17
Browser(s) used
Chrome 120.0.6099.129

Possible solutions:

detect session:resume event

I think that's what's refilling the screen here

@fregante what do you think of just not replacing the title if the page was reloaded, using the logic described in https://stackoverflow.com/questions/5004978/check-if-page-gets-reloaded-or-refreshed-in-javascript/53307588#53307588 ?
Let me know if that would work for you, I can open a PR.

I think we still need to cover reloads for the feature to work normally, right?
If you don't touch the field, what happens when you reload the page? Does first commit stay in the title?

I assumed the first time you open the page the feature replaces the title with the first commit, and then when you refresh GIthub restores whatever was in the field, whether it's the first commit or the user wrote something.
I'll give it a try and open a PR if it works properly.

That may be right, it should be tested:

Load the page
Have Refined GitHub set the title
Disable Refined GitHub
Reload the page

I tested what you described and it works properly.
I implemented a fix and it also works as expected. I'll open a PR.


fix translation in extended app

Description
Currently on extended app, Template Extensibility projects are not able to use translation on overriden files . This PR attempted to fix it by adjusting translation scripts to extract messages from both extended app and base template, so the extended app will have full translations.

Types of Changes

[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

template-retail-app/app/utils/locale.js fetchTranslation, we need to adjust the dynamic import from relative import to absolute import so it will get the correct translation files.
package.json in extended app

How to Test-Drive This PR

npm ci
npm start in my-extended-app
Add a new translation in the extended app
Run npm run build-translations to extract compile all translations
Confirm that new added translations are present in the en-US.json, and its compiled file

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[ ] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

@alexvuong One requirement that this PR already fulfills is with regards to precedence. The change to a translated text in the overrides dir should take priority.
I just wanted to spell it out here what that looks like. Say, on the homepage, I change the button text from Get started → Get Started FOO, while using the same message id. The end result is that the translation file would have the FOO version.


add support for delimiters in join() statement

Fixes https://github.com/iovisor/bpftrace/issues/25.
Default is retained as space, single-length and multiple-length strings are acceptable.
[jkoch@jkoch-nflx build]$ sudo ./src/bpftrace -e 'kprobe:do_execve { join(arg1); join(arg1, "."); join(arg1, "___"); }'
Attaching 1 probe...
ls --color=auto
ls.--color=auto
ls___--color=auto
^C

tested, works, thanks!


(Feature) Update contribution page if account was switched in wallet

Closes #1201

Pull Request Test Coverage Report for Build 3381

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 18.865%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/contribute/index.js
0
6
0.0%

Totals

Change from base Build 3376:
-0.03%

Covered Lines:
893

Relevant Lines:
4047

💛  - Coveralls


new release?

GH report 48 commits since 1.1.0 was tagged, maybe it's time for a new release of basemap?

Totally +1 especially since 1.1.0 will break with matplotlib 2.3, see #382.

On this note, would it be possible to get a new release in Pypi also? The version there is still 1.0.7.
I just hit matplotlib issue https://github.com/matplotlib/basemap/issues/395 - basemap 1.0.7 apparently does not work with matplotlib 2.2.2, both of which I installed with pip.
I see this has been discussed elsewhere too, though: https://github.com/matplotlib/basemap/issues/251

Hi there,
Just ran into #395 as well. Wondering when a new version will be pushed to PyPI?

Installing basemap 1.2 with Conda actually downgraded my Matplotlib all the way to 1.5. Yikes.

No. I think my setup was just mildly "broken", I have a ton of packages installed and I'm deep in dependency hell anyway.

With the merge of #454, it would be especially nice to have a release of basemap that supports the latest proj4/pyproj.  basemap is starting to cause major compatibility problems with other packages on conda-forge, which have moved on to pyproj>=2 and proj4>=6.

There is a new release available (1.3.0). This release also brings some precompiled binary wheels to PyPI, and basemap is split in three parts in a similar way to what Anaconda does. With pip, now it should be as simple as:
python -m pip install basemap

and, in case that you also need the high-resolution datasets:
python -m pip install basemap-data-hires


Text Area Padding Fix

As shown in here: http://imgur.com/a/HzrJu
Haven't minified it, that's up to you.

Sorry, I can't accept this PR. It doesn't follow guidelines :(
Next time, please make a local copy and run npm install. Then run gulp. This will build the files. And edit the files in the src folder.
Still, thanks for your help, I will be sure to fix this with your solution :)


Document GOPATH is required for use.

If I create a Go app (using package main) somewhere outside of my $GOPATH, vendor init works, but vendor list and vendor add -status external both fail with the error:
Error: Package "/usr/local/google/home/gmlewis/src/scenarios/scenario1-2" not a go package or not in GOPATH.

Am I doing something wrong?
Thanks!

I'll add a note to the readme.
vendor operates by re-writing import paths. The go tool will not be able to find the re-written import paths outside of GOPATH. The vendor file must write the new local path relative to the GOPATH as well.
GOPATH is requried.


Pythonic Parallelization of WCS

We'll need to change step1_get_refimg_ctrlpts.py and step2_mkwcs.py to use Pool, so that we can throw arbitrary numbers of cores at this in EM2 (which as 200 sec FFIs --> 3x as miuch data).

Can probably also improve step1 speed by making these local querries of tic 8.2

fixed in version 1.3.0.  Interesting, not as much speed as one might think.  Experimentation found that .apply_async could get about 1000 seconds faster. Can get individual CCDs down to 200sec using 60 cores, but needing to do 16 of these is just as fast as doing 16 python processes with 4 CCDs each.
This is pushing into issues with our harddware and IO system, so keeping the flexible framework and marking as done.


cannot find domtomhtml

thanks for the cool project!
when running the test webppl --require webppl-dp --require . tests/tests.wppl, another person and me get "cannot find domtomhtml". How can we resolve this?

I'm having this same issue. Was it ever resolved? Thanks for any tips!


[v6.6.3] onSubmit function doesn't provide "values", "dispatch", and "props" as arguments

Bug
The onSubmit function passed by the parent, give me (event1, event2) as arguments, instead of (values, dispatch, props)
(When input values change, I can see updated values into the store)
Step to reproduce

The getting started example with redux-form v6.6.3
Observe arguments received in the onSubmitfunction

Versions
    "react": "^15.4.2",
    "react-dom": "^15.4.2",
    "react-redux": "^5.0.3",
    "react-router-dom": "^4.1.1",
    "react-router-redux": "^5.0.0-alpha.6",
    "redux": "^3.6.0",
    "redux-form": "^6.6.3",
    "redux-saga": "^0.14.7",
    "redux-thunk": "^2.2.0",

I simply followed the "getting started" example, triple checked issues opened and closed, and something escapes me!
Thx by advance

Hi! Thanks for getting started with redux-form and sorry no one replied sooner. I can certainly confirm this functionality is not broken, so I think you'll need to provide some code so we can better advise. Also, stack overflow is a great place for usage questions where you might get more help sooner.
I suggest taking a look at the simple form example, but I also added logging to the console in this sandbox to show you that those properties are being passed.
I'll close this out since it is a usage question which is a few months old, but feel free to reply if you still need clarification.

@danielrob thanks for you feedback!
My problem came because I was passing the submit function to the form as a handleSubmit or onSubmit prop, which I think overrides the redux-form prop.
When I discovered the redux-form lib I was too exited, and I certainly missed few lines on the doc ;)

@MaxInMoon a truly excellent reason ;).


Incorporated some bugfixes.

fix the definition of async cb param
fix for build warning
fixed some issues related to aead

uadk_aead.c: In function 'uadk_e_aes_gcm_init':
uadk_aead.c:371:1: warning: control reaches end of non-void function [-Wreturn-type]
371 | }
| ^


remove space that sometimes prevents tests from running

This is just a wild theory that the leading space in the executable causes the tests not to run in Jenkins (but they do in our official build.)
Nobody merge this until I've finished looking at the logs.

I think this fixes it:   https://github.com/Microsoft/visualfsharp/pull/2845


object_detection + MQTT - Expose number of detected objects

First of all, thanks for all the excellent work on Viseron, it works impressively well with CUDA.
The binary sensors in Home Assistant showing if a person car etc was detected are great, but it would be better if they could report a count of how many of that object is detected.
For my usecase, I want to be notified when a car enters my driveway, but also want to be notified if a second car enters (assuming the first car is parked for hours).
Would it be possible to expose counts of each identified label?

Great to hear you're liking Viseron!
Seems quite obvious now that you mention it, I will try to squeeze it in to the next release, should be fairly easy. I'll expose the count as an attribute.
I am working on face recognition right now so it might take a few days. License plate recognition will come soon aswell which I guess would fit your use case quite well.

This is now available in the current beta


Create default S3 bucket if not present

Summary
When writing output S3 artifacts to a bucket (e.g. default bucket), if the specified bucket doesn't exist, the execution fails. A message is shown in the Argo UI saying: failed to save outputs: timed out waiting for the condition.
I propose we add the option to enable the auto-creation of the required S3 bucket if it does not exist.
Motivation
In a newly bootstrapped environment, the S3 bucket might not exist initially.
Proposal
In the Save method in s3.go, check if the bucket exists and create it if it doesn't. (Could make it configurable via Config Map).
https://github.com/argoproj/argo/blob/cdc935ae76b3d7cc50a486695b40ff2f647b49bc/workflow/artifacts/s3/s3.go#L84

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

This is a great idea. Would you like to submit a PR?

I'm looking into it atm

I've raised 2 PRs for this:

(pkg) https://github.com/argoproj/pkg/pull/29
(argo) https://github.com/argoproj/argo/pull/4098

The argo PR depends on pkg being merged first


Refine Documentation

aferritto created the following on Apr 15:
Follow up from #98
#98 establishes a basic sphinx build setup which recursively documents the oneqa API.
We should refine the .rst files to extend this basic API documentation and provide a better user experience.
We will likely want to wait until there is more code (with documentation too) in master before working on this.
Since this will likely not be done until after issues that have not been created yet are completed, I am applying the blocked label.
Original issue: https://github.ibm.com/ai-foundation/PrimeQA/issues/125

@bhavani105 : is it complete or being actively worked on ? Please update the status/pipeline accordingly.

Closing this as its covered under https://github.com/primeqa/primeqa/issues/279


Associations - Auto Create/Update - gorm:insert_option

Your issue may already be reported! Please search on the issue track before creating one.
What version of Go are you using (go version)?
go version go1.13 windows/amd64
Which database and its version are you using?
MySQL 8.0
Please provide a complete runnable program to reproduce your issue.
user := User{
  Name:            "jinzhu",
  BillingAddress:  Address{Address1: "Billing Address - Address 1"},
  ShippingAddress: Address{Address1: "Shipping Address - Address 1"},
  Emails:          []Email{
    {Email<EMAIL_ADDRESS>    {Email<EMAIL_ADDRESS>  },
CreatedAt: time.Now(),
  Languages:       []Language{
    {Name: "ZH"},
    {Name: "EN"},
  },
}

db.
Set("gorm:insert_option", "ON DUPLICATE KEY UPDATE created_at=CURDATE()").
Create(&user).
//// BEGIN TRANSACTION;
//// INSERT INTO "addresses" (address1) VALUES ("Billing Address - Address 1") ON DUPLICATE KEY UPDATE created_at=CURDATE();
//// INSERT INTO "addresses" (address1) VALUES ("Shipping Address - Address 1") ON DUPLICATE KEY UPDATE created_at=CURDATE();
//// INSERT INTO "users" (name,billing_address_id,shipping_address_id) VALUES ("jinzhu", 1, 2) ON DUPLICATE KEY UPDATE created_at=CURDATE();
//// INSERT INTO "emails" (user_id,email) VALUES (111<EMAIL_ADDRESS>ON DUPLICATE KEY UPDATE created_at=CURDATE();
//// INSERT INTO "emails" (user_id,email) VALUES (111<EMAIL_ADDRESS>ON DUPLICATE KEY UPDATE created_at=CURDATE();
//// INSERT INTO "languages" ("name") VALUES ('ZH') ON DUPLICATE KEY UPDATE created_at=CURDATE();
//// INSERT INTO user_languages ("user_id","language_id") VALUES (111, 1) ON DUPLICATE KEY UPDATE created_at=CURDATE();
//// INSERT INTO "languages" ("name") VALUES ('EN') ON DUPLICATE KEY UPDATE created_at=CURDATE();
//// INSERT INTO user_languages ("user_id","language_id") VALUES (111, 2) ON DUPLICATE KEY UPDATE created_at=CURDATE();
//// COMMIT;

db.Save(&user)

How make different gorm:insert_option for each insert statement?

what u say,  don't understand what you mean

on each INSERT statement:
INSERT INTO "addresses" (address1) VALUES ("Shipping Address - Address 1") ON DUPLICATE KEY UPDATE created_at=CURDATE();
INSERT INTO "users" (name,billing_address_id,shipping_address_id) VALUES ("jinzhu", 1, 2) ON DUPLICATE KEY UPDATE created_at=CURDATE();
INSERT INTO "emails" (user_id,email) VALUES (111<EMAIL_ADDRESS>ON DUPLICATE KEY UPDATE created_at=CURDATE();

equal ON DUPLICATE KEY UPDATE statement.
I need different  ON DUPLICATE KEY UPDATE on each INSERT statement


Chapter 7 - Building a Multi-Page App with Vue Router

On page 207,  the code,
   routes: [
     { path: '/', component: HomePage, name: 'home' },
     { path: '/listing/:listing', component: ListingPage, name: 'listing' }
]

is meant to be be in resources/assets/js/router.js and not resources/assets/js/app.js

Well spotted, thanks.


STT doesn't work, Website doesn't work on Firefox anymore

Hey!
I was super excited to see that Whisper is now integrated into this remarkable app. But unfortunately the website doesn't work on Firefox Desktop (macOS) anymore. It just gets stuck at the three pulsating blue dots indicating a loading process. The same problem occurs when using Firefox for Android.
When using a Chromium-based browser, the website loads fine.
Also, I cannot get the speech recognition to work :( Neither on Chrome for Desktop (macOS) nor on Chrome for Android. When clicking the microphone button, the website asks for access to my microphone, which I grant it. Then the microphone turns red, indicating that it is recording, but nothing happens.
No text is being written into the text field, so I thought maybe it's just not visible, but just pressing send after speaking also doesn't work.

Can confirm. I get the following error in the console:
Uncaught ReferenceError: webkitSpeechRecognition is not defined
    <anonymous> https://chatwithgpt.netlify.app/static/js/main.63992e72.js:2
    <anonymous> https://chatwithgpt.netlify.app/static/js/main.63992e72.js:2
    <anonymous> https://chatwithgpt.netlify.app/static/js/main.63992e72.js:2

Can confirm the issue exists on Firefox for Windows 10 as well.

Firefox works now again!


feat: remote input subscribe on barrier mutation (#17612)

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
cherry pick https://github.com/risingwavelabs/risingwave/pull/17612

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Rubber stamp. As long as it can pass the compilation


Add an initial health endpoint

This will allow an external monitoring system to check if the
broker can be accessed.
This can be extended to report detailed health information.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Add AIR001: task variable name should be same as task_id arg

Summary

This PR adds the first Ruff Airflow rule: AIR001.
AIR001: An Airflow task's Python variable name should be the same as the task_id arg you give it. This follows https://github.com/BasPH/pylint-airflow C8300.
This PR also adds the groundwork for adding new Airflow-related rules!
Test Plan

Unit tests included in the PR.

@charliermarsh any chance you can help with the CI issues? Not too sure what’s going on, this works fine locally and the code looks consistent with what I see elsewhere.

Not sure what that failure was -- we replaced .ctx with .semantic_model(), but your branch was also passing for me locally (since that change was past your branch's upstream). Maybe some kind of weird Cargo caching failure on CI? Regardless, I merged in main + changed the reference, so hopefully it passes now.


add items images

What about minifying those images using Pillow? So you can automate the process.

А как насчет минимизации этих изображений с помощью Pillow? Таким образом, вы можете автоматизировать процесс.

I do not plan to minimize these images using Pillow, but if you know how to use it and want to utilize it, feel free to do so.

Hi @Rodion15 !
A already the last changes (from last commits, renaming files). Update your origin!
And let me know when you finish so I can merge your PR!

@mlg404 It's ready now. I have already added the item.json file, which includes the name, image path, price, and weight.


deck sync failed to propagate changes from a existing file into a new kong installation

When trying to propagate changes from a existing dump generated with deck to a new/empty kong 1.2 + postgres 9.6 deck sync failed with the following error:
creating consumer cl-consumer-test
creating service cl-backoffice-order-status
creating service webscraper_blumix
creating service super_cl_picking_orderhistory_preprod_service
creating service super_cl_bnf_control_orders
creating service super_cl_syg_rewards_service_dev
Error: 1 errors occurred:
while processing event: {Create} failed: 400 Bad Request {"message":"schema violation (custom_id: length must be at least 1)","name":"schema violation","fields":{
"custom_id":"length must be at least 1"},"code":2}
deck sync with --verbose 2
{"next":null,"data":[]}
creating consumer cl-consumer-test
PUT /consumers/1b31d0cb-1d1d-4e25-818e-9dbae3a3005e HTTP/1.1
Host: <IP_ADDRESS>:8001
User-Agent: Go-http-client/1.1
Content-Length: 90
Content-Type: application/json
Accept-Encoding: gzip
{"id":"1b31d0cb-1d1d-4e25-818e-9dbae3a3005e","custom_id":"","username":"cl-consumer-test"}
HTTP/1.1 400 Bad Request
Content-Length: 155
Access-Control-Allow-Origin: *
Connection: keep-alive
Content-Type: application/json; charset=utf-8
Date: Fri, 08 May 2020 23:33:55 GMT
Server: kong/1.2.0
{"message":"schema violation (custom_id: length must be at least 1)","name":"schema violation","fields":{"custom_id":"length must be at least 1"},"code":2}
deck diff after sync
Summary:
Created: 2639
Updated: 0
Deleted: 0
Steps to recreate:

deck dump from existing kong 1.2 with data
delete kong 1.2 container + postgres 9.6 container
create new kong 1.2 container + postgres 9.6 container
deck sync from dump file previously created

Aditional info:

docker toolbox
kong database postgres 9.6
decK v1.1.0 (acdb161)

Can you share the file generated (redact any sensitive info)?

It seems like you have custom_id: "" which is the root of the problem. Please remove that line and you should be good to go. I don't know why that was generated in the first place.
Can you do deck dump --verbose 2 on the first cluster and share the output?

there is one line generated in the dump file:

custom_id: ""
username: cl-consumer-test
keyauth_credentials:

key: rHhmUVJHdoctHMhsl48zD4bOenn6rRRh
acls:
group: cl-consumer-test

i changed - custom_id: "" line to - custom_id: "some" and it work
Summary:
Created: 2639
Updated: 0
Deleted: 0
$ deck diff --kong-addr http://<IP_ADDRESS>:8001
Warning: please note that changes in password of basic-auth credentials are not detected by decK!!
Summary:
Created: 0
Updated: 0
Deleted: 0
let me do deck dump --verbose 2 and i share with you

Harry,
i attach the result of the deck dump command and the dump file generated
Thanks for the help
Atte.
David González Cortez.
☎ +56992258037
El vie., 8 may. 2020 a las 19:51, Harry<EMAIL_ADDRESS>escribió:

It seems like you have custom_id: "" which is the root of the problem.
Please remove that line and you should be good to go. I don't know why that
was generated in the first place.
Can you do deck dump --verbose 2 on the first cluster and share the
output?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hbagdi/deck/issues/149#issuecomment-626067153, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIOEWIZUI2ZFP6BHY3AWMLDRQSLI7ANCNFSM4M4REOBA
.

The attachment won't carry over via email. Please attach it via Github UI.

ok
dump_log.TXT

I see.
Please remove the  "custom_id": "" and change it to null instead. That will fix your problem.
Kong shouldn't allow for empty strings in custom_id. If it does, that's a bug in Kong.

Closing this, please re-open if needed. Thanks!


support for isolated routes to solve multiple instances problem

This modification solves the issue of isolating routes and adding more swagger instances in Elysia using app.group()
fix #138
firstApp
	.get('/first-route', () => {
		return 'first route!'
	})
	.use(
		swagger({
			path: '/first-doc',
			routes: firstApp.routes
		})
	)

const secondApp = new Elysia({ prefix: '/second' })

secondApp
	.get('/second-route', () => {
		return 'second route!'
	})
	.use(
		swagger({
			path: '/second-doc',
			routes: secondApp.routes
		})
	)

const app = new Elysia({
	// aot: false
})
	.use(firstApp)
	.use(secondApp)
	.listen(3000)

this way we can isolate into groups and have two instances with different paths and separate the documentation without needing a lot of code

just an idea that helps in specific cases without changing the code too much and without breaking any part of the plugin


Fetch enough pages to fill a page's worth of items

We have a hack that fetches enough items to fill the first page's worth of items (30) if we're not getting enough items within the first few pages (6). This hack isn't reliable and doesn't handle similar cases further down the scroll (e.g. when there's an empty [] response in for further pages, or when too many items get filtered out later).
I'm adding a more extensive fix that should handle these cases consistently. The idea of the fix is to keep track of the "wanted" item count. When we start paginating, we bump the "wanted" item count by the page size (30). If we then don't receive enough items, we keep trying to paginate until those 30 items are filled. There are a few conditions where we reset that counter.
This mostly acts as a failsafe. In practice FlatList content size change triggering onEndReached (which was ported to web in https://github.com/bluesky-social/social-app/pull/4859) is accidentally sufficient in most cases because rendering a pagination spinner causes a content size change. However, that's not guaranteed to be reliable. So this is an attempt to be more comprehensive.
It also means that, for next pages, fetching the missing pages happens a bit earlier, similar to how we try to preemptively load the first 30 items without waiting for onEndReached in the existing hack.
Test Plan
This branch is based before https://github.com/bluesky-social/social-app/pull/4859 to show that it solves the problem independently from https://github.com/bluesky-social/social-app/pull/4859. However, it should also work when rebased to main.
You can try adding some console logs to verify the behavior:
https://github.com/user-attachments/assets/c2d2afdf-1ba9-4620-9ca4-06dcb2fe22a2

If there's ever some case that causes all pages to come back empty, then it's going to run infinitely

So I think if there's an appview bug like this, it would be an infinite loop regardless if the tail is in the viewport. But it's fair that this wouldn't be a repeated issue if you scroll up. I'll add a failsafe.