Gazebo fortress launch

I was able to follow all the steps provided for Humble + Fortress (Ubuntu 22.04) but not able to launch gazebo fortress.
I used the command "ros2 launch bcr_bot ign.launch.py" and I'm getting this error "file 'ign.launch.py' was not found in the share directory of package 'bcr_bot' which is at '/opt/ros/humble/share/bcr_bot' "
but I'm able to view in rviz using the command "ros2 launch bcr_bot rviz.launch.py"
please help.

for_gazebo is the package name. and also I'm able to use "ign gazebo" command and my gazebo fortress is working

@sudheer0404 the latest version of BCR bot is not yet released in the binaries.
Try a source build or to launch bcr_bot in the fortress you can run:
ros2 launch bcr_bot gz.launch.py

Hey @sudheer0404 closing this issue due to inactivity. Feel free to reopen if the problem persists.


[HOLD for payment 2024-06-28] [$250] Web - Categorizing - Offline indicator appears in Edit categories RHP on web layout

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when validating https://github.com/Expensify/App/pull/41344
Version Number: 1.4.79-3
Reproducible in staging?: y
Reproducible in production?: no, new feature
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:
Precondition:

Workspace has no categories.

Go to staging.new.expensify.com
Go to FAB > Track expense > Manual.
Select Categorize it from the actionable whisper.
Select the workspace with no categories.
Click Edit categories.
Go offline.

Expected Result:
Offline indicator will not appear in Edit categories RHP on web layout.
Actual Result:
Offline indicator appears in Edit categories RHP on web layout.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/91ab780b-a584-4abd-84f2-0005633a764d

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a222d1a32818ce76
Upwork Job ID:<PHONE_NUMBER>349278720
Last Price Increase: 2024-06-12
Automatic offers:

bernhardoj | Contributor | 102727195

Issue OwnerCurrent Issue Owner: @lschurr

@laurenreidexpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

This seems to be enabled in several workspace pages. I'm not sure why but it doesn't seem accidental. I'm going to remove the blocker label since this is not really a new issue and reach out about it.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The offline indicator is shown in edit categories RHP page.
What is the root cause of that problem?
The categories page enable the offline indicator in wide screen.
https://github.com/Expensify/App/blob/3c04b02d275b70649790598c0bbbc05fa18bfc63/src/pages/workspace/categories/WorkspaceCategoriesPage.tsx#L286-L290
By default, the offline indicator is only shown on a small screen because we don't want to show the offline indicator in RHP. On a large screen, the offline indicator is only shown on the report screen and any screen that is in the center of the page, like categories and any other workspace/settings page.
But when the categories page is used in RHP too, we don't want the offline indicator.
What changes do you think we should make in order to solve the problem?

Disable the offline indicator in wide screen if we are in RHP.
shouldShowOfflineIndicatorInWideScreen={!shouldUseNarrowLayout}

I searched for all usages of shouldShowOfflineIndicatorInWideScreen and only categories page that use useResponsiveLayout which indicates it's also being used in RHP.

Reviewing now.

Thanks for the proposal @bernhardoj.
Can you explain how adding the following:

shouldShowOfflineIndicatorInWideScreen={!shouldUseNarrowLayout}

is better than removing the shouldShowOfflineIndicatorInWideScreen prop from WorkspaceCategoriesPage altogether as suggested by @Nodebrute?

We still need the offline indicator for the full screen categories page

Still working through proposals

Thanks for the explanation @bernhardoj!
@bernhardoj's proposal looks good to me. The RCA is correct and the solution makes sense as well.
🎀 👀 🎀 C+ reviewed

Proposal LGTM, feel free to create the PR @bernhardoj.

PR is ready
cc: @allroundexperts

@lschurr adding you here for parental leave sub. no action required this BZ this week, PR is still in review, but adding you for payment in coming weeks when the fix makes it to prod. Thanks

Payment summary for tomorrow, 6/28:

Contributor: $250 @bernhardoj (to be paid in Upwork)
Contributor+: $250 @allroundexperts (please request in Newdot)

This is complete.

$250 approved for @allroundexperts


Ctrl+Alt+Shift+F does not work on windows 10

System information:

Operating system (distribution) and version: windows 10 pro 1903 (18362.207)
DBeaver version: 6.1.2
Additional extensions: None

Describe the problem you're observing:
Ctrl+Shift+Alt+F does not change to the Filter field but clicking the option in the keybinding list does go to the field.
Steps to reproduce, if exist:
Ran a select in the sqlite database that comes with dbeaver for testing.
Select * from Album
Ctrl+Alt+Shift+F and nothing happens.

I can't reproduce this (it perfectly switches between filter input and data grid).

Check Preferenes->General->Keys for any shortcut conflicts
Try to reset UI settings (https://github.com/dbeaver/dbeaver/wiki/Reset-UI-settings)

I can't reproduce this (it perfectly switches between filter input and data grid).

Did you try this on Windows 1903? Ever since I upgraded Windows 10 to this version this key combination stopped working in every program.

I tested again now with both the 1903 and 1809 and on 1809 it works but not on latest 1903 as mentioned by @JorisMolnar
Thank you for pointing this out.

Also tested now a second machine where it worked on 1809
Upgraded it to version 1903 and now Ctrl+Alt+Shift+F does not work.
So this is related to version diff of windows 10.

Maybe there is some other software installed shich handles ctrl+alt+shift+f globally?
Anyway, looks like we can't fix this from DBeaver side..

For the archives: I tripped on this bug with a software of mine today, I was still running Windows 10 1909 (not sure why Windows auto-update hadn't happened yet). I updated my Windows to 20H2 using the Update Assistant, and joyfully note: the bug seems to have been fixed by Microsoft: KeyDown-events for CTRL-ALT-SHIFT-F do occur again.


Add support for zsh terminals

Description
I have added some support for ZSH terminals following: (https://github.com/paulirish/git-open)
Changes

Add support for ZSH
Apply some markdown lint fixes

@mohammed-elhaouari thanks for the first contribution! ⭐
For clarity, this does work on ZSH given the current instruction. However, this is a way to add it as a ZSH "plugin"?

Yes @jasonmccreary. It is to add it as plugin to oh-my-zsh.
I didn't explore the other possibilities (zgen, zplug, antigen ...)

@jasonmccreary I agree with you on that.
Works like charm.


Adds info panel to the left hand side of dashboards

Some important stuff to review:

I added some new fields to the viz.json with information for the info bar.
I introduced a new dependency (moment.js) which I'm using to format a date.
We currently don't a model for visualization/dahsboards to pass to this new view, so I decided to go for a basic Backbone.Model.

Also, I think this view will need to be part of a DashboardCanvasView or similar in the near future.
@xavijam what do you think?

Come on, add a screenshot! :P

For me looks great! :+1:

You got it...

:dancer:


Support opening/closing issues

Mutations to open/close an issue.

Yeah, we can definitely add tests in follow-ups.


how to implement the RG approach?

We tried to use the code made publicly available by the authors of Zhang et al. (2023) but failed to execute them with the provided instructions. As a result, we implemented the RG approach by ourselves. Note that, unlike Zhang et al. (2023), we perform only 1-step retrieval augmented generation. Can you open source the implementation of Repocoder?

We currently do not have the plan to open source the code to run Zhang et al. (2023).


Invalid Signature - Email Verification AWS Nginx

Laravel Version: 9.2
PHP Version: 8.0
Database Driver & Version:

Description:
I am trying to verify email address, upon clicking on Verify Email Address button that i get an email, It gives me error " 403 | Invalid Signature "
I am using AWS Elastic Beanstalk with Nginx Server.
here is my nginx file.
`user                    nginx;
error_log               /var/log/nginx/error.log warn;
pid                     /var/run/nginx.pid;
worker_processes        auto;
worker_rlimit_nofile    32153;
events {
worker_connections  1024;
}
http {
include       /etc/nginx/mime.types;
default_type  application/octet-stream;
log_format  main  '$remote_addr - $remote_user [$time_local] "$request" '
                  '$status $body_bytes_sent "$http_referer" '
                  '"$http_user_agent" "$http_x_forwarded_for"';

include       conf.d/*.conf;

map $http_upgrade $connection_upgrade {
    default     "upgrade";
}

server {
    listen        80 default_server;
    access_log    /var/log/nginx/access.log main;

    client_header_timeout 60;
    client_body_timeout   60;
    keepalive_timeout     60;
    gzip                  off;
    gzip_comp_level       4;
    gzip_types text/plain text/css application/json application/javascript application/x-javascript text/xml application/xml application/xml+rss text/javascript;

    # Do not include the Elastic Beanstalk generated locations
    # include conf.d/elasticbeanstalk/*.conf;

    # Move Elastic Beanstalk healthd.conf content here
    if ($time_iso8601 ~ "^(\d{4})-(\d{2})-(\d{2})T(\d{2})") {
        set $year $1;
        set $month $2;
        set $day $3;
        set $hour $4;
    }

    access_log /var/log/nginx/healthd/application.log.$year-$month-$day-$hour healthd;

    # Move Elastic Beanstalk php.conf content here
    root /var/www/html/public;

    index index.php index.html index.htm;

    # This is an additional configuration
    location / {
        try_files $uri $uri/ /index.php?$query_string;
        gzip_static on;
    }

    location ~ \.(php|phar)(/.*)?$ {
        fastcgi_split_path_info ^(.+\.(?:php|phar))(/.*)$;

        fastcgi_intercept_errors on;
        fastcgi_index  index.php;

        fastcgi_param  QUERY_STRING       $query_string;
        fastcgi_param  REQUEST_METHOD     $request_method;
        fastcgi_param  CONTENT_TYPE       $content_type;
        fastcgi_param  CONTENT_LENGTH     $content_length;

        fastcgi_param  SCRIPT_NAME        $fastcgi_script_name;
        fastcgi_param  REQUEST_URI        $request_uri;
        fastcgi_param  DOCUMENT_URI       $document_uri;
        fastcgi_param  DOCUMENT_ROOT      $document_root;
        fastcgi_param  SERVER_PROTOCOL    $server_protocol;
        fastcgi_param  REQUEST_SCHEME     $scheme;
        fastcgi_param  HTTPS              $https if_not_empty;

        fastcgi_param  GATEWAY_INTERFACE  CGI/1.1;
        fastcgi_param  SERVER_SOFTWARE    nginx/$nginx_version;

        fastcgi_param  REMOTE_ADDR        $remote_addr;
        fastcgi_param  REMOTE_PORT        $remote_port;
        fastcgi_param  SERVER_ADDR        $server_addr;
        fastcgi_param  SERVER_PORT        $server_port;
        fastcgi_param  SERVER_NAME        $server_name;

        # PHP only, required if PHP was built with --enable-force-cgi-redirect
        fastcgi_param  REDIRECT_STATUS    200;

        fastcgi_param  SCRIPT_FILENAME  $document_root$fastcgi_script_name;
        fastcgi_param  PATH_INFO $fastcgi_path_info;
        fastcgi_pass   php-fpm;
    }
}

}
`
If you want to verify it by yourself, Kindly visit
https://alayaservice.com/register
Register for your self and an email will be sent to you.
Steps To Reproduce:

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

Thank you @driesvints  i have posted the issue on Larave Forum, here is the link.
https://laravel.io/forum/403-invalid-signature
Kindly help out if you have time please.
Thank you.

Add this in AppServiceProvider
if(config('app.env') === 'production') {
\URL::forceScheme('https');
}
And then change headers like below
Class TrustProxies extends Middleware
{
protected $proxies = '*';
protected $headers = Request::HEADER_X_FORWARDED_FOR | Request::HEADER_X_FORWARDED_HOST | Request::HEADER_X_FORWARDED_PORT | Request::HEADER_X_FORWARDED_PROTO;

}


Add unsafe attributes at yarte fixed bench

It can cause UB, so I have put it unsafe.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   82.59%   82.59%
=======================================
  Files          20       20
  Lines        1247     1247
=======================================
  Hits         1030     1030
  Misses        217      217

Continue to review full report at Codecov.

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

Thank you for your work! Benchmark results were updated.


[T2A3][T13-A3]Ong Qi Yong

Ready for review

You may skip an activity PR if the changes are almost the same as another one. Or, better still, find other places to change.


issues under opening balances

[x] after giving opening balance of ledgers,it must display it in the first listing screen of opening balances
[x] its working only for assets and liabilities
[x] remove account receivable from asset and account payable field from liabilities
[x] currency displayed under listing and edit page of opening balance must be of the organization's currency

[ ] make "is editable " enabled(ticked) in default under new account

@midhunkrishnanr07

ISSUE FIXED !


Outline Arm64 Steps

A few things come to mind to do this when testing and writing a quick guide.

Make sure base images are multi-platform. They should be.
How is local VS Code devcontiner affected? Codespaces? Deploy Action?
Easy part is adding the lines in template.yaml file. Arch, etc.

https://awstip.com/aws-sam-with-apple-m1-e61937e0328e

https://github.com/actions/runner-images/issues/5631

See the new guides and all the work wrapped up here. https://github.com/customink/lamby-site/pull/60


Add advice about IMPORT to documentation

The current ticket requests to update documentation of cloud-storage-extension to

mention alternative ways for data import
focus links to additional important documentation

Background
Besides using the cloud-storage-extension (CSE) users should consider direct import which usually is much faster and even supports PARALLELISM, e.g. when importing multiple files and import does not require a specific ordering.
Existing documentation
Currently the official documentation already contains instructions for Azure
IMPORT INTO table1 FROM CSV AT
CLOUD AZURE BLOBSTORAGE 'DefaultEndpointsProtocol=https;EndpointSuffix=core.windows.net'
USER '<AccountName>' IDENTIFIED BY '<AccountKey>' FILE '<container>/<blob>'

However the page contains instructions for generic http or https with option file_src and more information is available in other places

https://docs.exasol.com/db/latest/loading_data/load_data_google_cloud_storage_buckets.htm
https://github.com/exasol/connection-parameter-specification/blob/main/specification.md

Rationale
The current ticket is inspired by support ticket L3-561 and in addition to doc ticket DOC-2424 to improve documentation.

Hello @ckunki,
Thanks for the feedback!
The above issue is for CSV format. CSV import is not supported in cloud-storage-extension.
The cloud-storage-extension is for binary Parquet, Avro and ORC formats. In the user guide we have overview of the supported format and supported cloud storage systems.
I am not sure if we need to add any more information without confusing the users.


HW-5 js logic by IgoFromUA

added all the functionality as in the technical task

Hey @IgorFromUA you rock! Well done!


If I use generic array type instead of regular array it panics.

Describe the bug
To reduce boilerplate, I defined a generic array type instead of using a named type of array.
To Reproduce
First of all, I defined the following generic array wrapper:
package slices

type SliceWrapper[T any] []T

func (s SliceWrapper[T]) Append(v T) SliceWrapper[T] {
	return append(s, v)
}

func (s SliceWrapper[T]) IsEmpty() bool {
	return len(s) == 0
}

func (s SliceWrapper[T]) Len() int {
	return len(s)
}

func (s SliceWrapper[T]) First() T {
	if s.IsEmpty() {
		var t T
		return t
	}

	return s[0]
}

Previously, I defined and used the following response dto type.
...
type ShopsResponse []ShopResponse

type ListShopsResponse struct {
	Shops      ShopsResponse      `json:"shops"`
	Pagination PaginationResponse `json:"pagination"`
}

But if I apply a generic type like this:
...
type ShopsResponse slices.SliceWrapper[ShopResponse]

type ListShopsResponse struct {
	Shops      ShopsResponse      `json:"shops"`
	Pagination PaginationResponse `json:"pagination"`
}

It panics and fails.
2024/02/19 18:28:31 Generating domain.ShopsResponse
panic: interface conversion: ast.Expr is *ast.ArrayType, not *ast.StructType

goroutine 1 [running]:
github.com/swaggo/swag.(*PackagesDefinitions).parametrizeStruct(0x14003b16c18?, 0x140009c06c0, 0x14003b23290, {0x140041bd3e1, 0x21}, 0x30?)
        /Users/myyrakle/go/pkg/mod/github.com/swaggo/swag@v1.8.6/generics.go:145 +0xe60
github.com/swaggo/swag.(*PackagesDefinitions).findGenericTypeSpec(0x140000a06c0, {0x140041bd3e1, 0x21}, 0x140009c06c0?, 0x0?)
        /Users/myyrakle/go/pkg/mod/github.com/swaggo/swag@v1.8.6/packages.go:435 +0x12c
github.com/swaggo/swag.(*PackagesDefinitions).FindTypeSpec(0x140000a06c0, {0x140041bd3e1, 0x21}, 0x140009c06c0, 0x0)
        /Users/myyrakle/go/pkg/mod/github.com/swaggo/swag@v1.8.6/packages.go:392 +0x738
github.com/swaggo/swag.(*Parser).getTypeSchema(0x140001581c0, {0x140041bd3e1, 0x21}, 0x0?, 0x0)
        /Users/myyrakle/go/pkg/mod/github.com/swaggo/swag@v1.8.6/parser.go:891 +0x6c0
github.com/swaggo/swag.(*Parser).parseGenericTypeExpr(0x140001581c0, 0x14000166120?, {0x103563550?, 0x14001835920})
        /Users/myyrakle/go/pkg/mod/github.com/swaggo/swag@v1.8.6/generics.go:344 +0xf4
github.com/swaggo/swag.(*Parser).parseTypeExpr(0x140001660f0?, 0x1033b61c7?, {0x103563550?, 0x14001835920?}, 0x18?)
        /Users/myyrakle/go/pkg/mod/github.com/swaggo/swag@v1.8.6/parser.go:1192 +0x518

Expected behavior
In my opinion, since generics are removed from preprocessing, they should behave the same as they would without generics.
But it didn't work that way.
Of course, I know that the principle is different since this is generate before compilation.
Is support for these features possible?
Screenshots

Your swag version
v1.8.6
Your go version
go version go1.20.6 darwin/arm64
Desktop (please complete the following information):

OS: macOS
Browser: none
Version: ventura 13.3

Additional context

Try the latest version of swag.

Since I use the latest version, there is no panic, but it doesn't generate like an array.
I think this is probably intentional behavior from here on out. We will custom process the rest ourselves.
The issue is closed.


Feat: [199] hook up section privacy input to api, loading indicator, …

Description
hook up section privacy input (in section navigator bar)
Fixes #199
Test Instructions

log in
create a new site
click on site, go to first section
note the privacy select input in the section nav bar has placeholder text
change its value
refresh section. See that the value persists/is loaded from server (note the placeholder is disabled and unselectable)
move through other sections doing the same.

HArd to test error messages without editing code, but this is what it looks like when the api fails to save privacy.

what is the default state, public or private? I think the default state should be visible to the user

what is the default state, public or private? I think the default state should be visible to the user

Tahts a great question. There is a ticket for the create new site to have an input for section default state', but it would be also good for that to be visible after the fact somewhere, and probably configurable. Ill link Al to this.


glibc: 2.38-27 -> 2.38-44 and patch for glibc possible memory corruption in qsort()

Description of changes
Fixes CVE-2023-6246, CVE-2023-6779, and CVE-2023-6780
Advisories from Qualys:
https://www.qualys.com/2024/01/30/cve-2023-6246/syslog.txt
https://www.qualys.com/2024/01/30/qsort.txt
Upstream advisories:
https://sourceware.org/git/?p=glibc.git;a=blob;f=advisories/GLIBC-SA-2024-0001;h=28931c75ae018cc398344af80e6be6ab6e1874b7;hb=6cdc44214253a74e7140d75a7ebfc900820a5fa8 https://sourceware.org/git/?p=glibc.git;a=blob;f=advisories/GLIBC-SA-2024-0002;h=940bfcf2fcb760505ec59d84b94250aaf49c3be9;hb=6cdc44214253a74e7140d75a7ebfc900820a5fa8 https://sourceware.org/git/?p=glibc.git;a=blob;f=advisories/GLIBC-SA-2024-0003;h=b43a5150ab1b0cc4ef2cabf5bf792428389f5e37;hb=6cdc44214253a74e7140d75a7ebfc900820a5fa8

Things done

Built on platform(s)

[x] x86_64-linux (top of master)
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

https://www.qualys.com/2024/01/30/qsort.txt

2024-01-16: We backported commit b9390ba to all current and past stable
versions of the glibc, and sent this patch and a draft of our advisory
to the linux-distros@openwall (to piggyback on the glibc embargo for
CVE-2023-6246). They immediately acknowledged receipt of our email.

The related patch seems to be missing in https://sourceware.org/git/?p=glibc.git;a=log;h=refs/heads/release/2.38/master
I used the Debian patch in the meantime...

Successfully built pkgsCross.aarch64-multiplatform.vim & pkgsi686Linux.vim, nixos x86_64

Not surprised given that glibc doesn't seem to consider this a security issue given what I read in https://www.qualys.com/2024/01/30/qsort.txt: [...]

Yeah but at the same time the timeline in the Qualys advisory mention "We backported commit b9390ba to all current and past stable versions of the glibc" so I'm a bit confused.
The patch seems low risk enough to be included but if someone has a concern about it I can drop it from this PR and we can discuss about it in another PR.

That said @LeSuisse would you mind rebasing against staging-next? This is a vulnerability that is OK to fast-track I think.

Done, thanks.


Blank/null response handling for the player-name prompt

Expected behavior:
-the prompt should keep requesting the player name until valid entry.
Current behavior:
-a blank answer is stored as the player's name.
-if cancelled, Null is stored in the player's name.
Possible Solution:
-create a function that checks the answer and loops until the player enters a valid answer.

fixed.


Add support to update a single application within the same Tenant

Related to #250
Current state:
Currently, the AS3 model is Tenant based, with one Tenant being responsible for one or more Applications. We have the option to have AS3 separation of Tenant concerns, by the option to add one or more Tenants to the AS3 URI endpoint, as documented here.
 POSTING to the URI /mgmt/shared/appsvcs/declare/tenant1,tenant2

Possible improvement:
Add the option to also have a separation of concern on Application level
 POSTING to the URI /mgmt/shared/appsvcs/declare/tenant1/application1

Scenario:
From a DevOps perspective, I would like a clean separation of concern between Applications. This means:

Application code is stored in separate git repo's
Ansible provision scripts are stored in the same (or a different) repo, per Application
My service delivery (AS3) is stored per Application as well

Given the current capabilities of AS3, you have no choice other than to store AS3 blobs per Tenant.
Workarounds:
There are a few ways to get around this today:

Use 1 Application per Tenant only
Have a separate system (eg a CICD pipeline) to fetch split per Application AS3 deployments and recombine them into one per Tenant based AS3
Use PATCH instead of POST, which has different disadvantages (and or risks)

The second work around however, violates the principle that a CICD pipeline is responsible for 1 Application only. This might have both security and instability implications (like an application development team making mistakes in their Application specific blob).

While I agree that this could be useful, it will likely be a while before we make a larger change or update to the API like this one. If and when we do get to a point where we are able to work on an API update like this, we will certainly keep this feedback in mind. Especially since this is not the first time it has come up.
In the meantime I have added AUTOTOOL-1502 to our internal product backlog to begin tracking this feature.

Thanks for the update @dstokesf5 . It would avoid a whole lot of boilerplate verification and recomposition code for about everyone make the move to our ATC toolchain.

If you are still looking for this feature, please reach out to us at<EMAIL_ADDRESS>


⚠ Bofrost.fr has degraded performance

In c10f70e, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1936 ms

Resolved: Bofrost.fr performance has improved in 247efdb after 1 hour, 3 minutes.


VM Hardware: NICs and IPs management

For the VM's NIC hardware component where we show the NICs, implement the following:

[x] Add network to VM
[x] Change IP address of NIC
[x] Edit/add secondary IPs for the NIC
[x] Set nic to default
[x] Remove NIC from VM

VM can have more than one nic, in that we also want buttons to set the nic as default nic and also to remove NIC from the VM:

I've updated the screenshot and actions list, please check ^^ @RitchieVincent

👍  Luckily I haven't started this task yet, nearly finished #57 and then I'll move on to this one.

Fixed in #71


Label Vector Fonts don't' have the right rendering

Label text founded in style-func has weird break fonts
When i try to load my map with dynamic text in each vector, the font texts are appearing like a weird writing.
In each loading, the weird font appears in randomly vectors

Can i fix this using the style-func or would be other way?
Expected behavior
The dynamic vectors text comes with the normal config style and font

Would you be able to share a working example that exposes the problem you describe?

Had the same looking phenomenon. Check for overlapping labels.


Dark theme

I'd love to have a dark theme for nighttime use! 😎

@evanw Sorry it took me so long to respond!
This is really cool! The high-level approach looks right to me, though I think ideally I'd like to specify functional colors by role somewhere and do something like Color.foregroundText(isDarkMode) or something along those lines to make it hard to forget to do this when adding new bits of UI in various places, rather than having isDarkMode switches everywhere. I've also seen other styling tools pass down a "theme" object via React context to avoid needing to do this at all.
When I have some more time, I'd like to understand how designers manage their color palettes around use case rather than naming the color for theming, since I think that'll apply well in this case too.

Makes sense. Sounds like you'd like to take it from here. Is that correct? Let me know if you need anything from me. I'm also happy to help if it's useful.

Yep! When I have time to look into this and have things in a place I'm happy with, I'll probably send it to you to user test since it sounds like you're maybe one of the heaviest nighttime users 😂

Okay! This is live on https://www.speedscope.app/ now, and published to npm as<EMAIL_ADDRESS>

Shoot I didn't catch you. The problem mentioned in https://github.com/jlfwong/speedscope/pull/323#issuecomment-725978352 happens on https://www.speedscope.app/ for me too.

Welp. Tracking in #327


Add Support for Moving Finished Video Files

Using the variable $SEPARATERAWFINISH, finished video files can be moved into their own folder to be picked up by software like Handbrake or even Radarr/Sonarr, potentially.

Also, in regards to identifying disks, this is how automatic ripping machine does it:
Identify.py file
Also, under their "How it Works" section on the original blog post, this is a snippet of how it identifies:

When UDEV/systemd detects a disc insert as defined by /lib/udev/rules.d/51-automedia.rules it runs the wrapper which in turn runs /opt/arm/identify.sh which identifies the type of media inserted and then calls the appropriate scripts.  (if you ever need it this is a great command get get info on a disk):
udevadm info -q env -n /dev/sr0

Udevadm didn't work inside docker when I last tried this (hence the makeMKV method) .. Ill investigate again.
Also I like the changes for further automation.

this project uses udev rules to start the container, in case this is useful to this project for inspiration https://github.com/git-developer/marvyn/tree/develop/udev/rules.d
Enable autorip:
$ sudo ln -s /opt/docker/marvyn/udev/rules.d/disc-detection.rules /etc/udev/rules.d/disc-detection.rules
$ sudo udevadm control --reload


extend example for join!/try_join!

Hello :crab: , this PR makes the following change.

Update examples for join!/try_join! to make explicit that these macros are variadic.

closes #2120
Thank you for reviewing this PR :+1:

The initial commit failed in CI due to an issue external to this PR

Also, it may be good to add a comment explaining "variadic" after the comment "join!/try_join! is variadic".
e.g., "join! is variadic, so you can pass any number of futures."

Also, it may be good to add a comment explaining "variadic" after the comment "join!/try_join! is variadic".
e.g., "join! is variadic, so you can pass any number of futures."

Thank you for your feedback! I made commits to apply the suggested changes :smiley_cat:

Thanks!


feat:[CDS-96279]: improving documentation for Helm Chart Metadata fea…

…ture.
Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6630bf4754a7aabf45ad5f7c--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6630c511923ea7b44f245ce8--harness-developer.netlify.app


[Bug]: Importing Python module apache_beam.dataframe.convert raises AttributeError

What happened?
In Python, with Apache beam 2.48.0 and Pandas >=2.0.0 installed, running the following import statement raises an exception:
>>> import apache_beam.dataframe.convert
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/jzxu/env/lib/python3.11/site-packages/apache_beam/dataframe/convert.py", line 33, in <module>
    from apache_beam.dataframe import transforms
  File "/home/jzxu/env/lib/python3.11/site-packages/apache_beam/dataframe/transforms.py", line 33, in <module>
    from apache_beam.dataframe import frames  # pylint: disable=unused-import
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jzxu/env/lib/python3.11/site-packages/apache_beam/dataframe/frames.py", line 1231, in <module>
    class DeferredSeries(DeferredDataFrameOrSeries):
  File "/home/jzxu/env/lib/python3.11/site-packages/apache_beam/dataframe/frames.py", line 1338, in DeferredSeries
    @frame_base.populate_defaults(pd.Series)
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jzxu/env/lib/python3.11/site-packages/apache_beam/dataframe/frame_base.py", line 600, in wrap
    base_argspec = getfullargspec(unwrap(getattr(base_type, func.__name__)))
                                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: type object 'Series' has no attribute 'append'. Did you mean: '_append'?

This is because the function Series.append has been removed in Pandas 2.0.0:
https://pandas.pydata.org/docs/dev/whatsnew/v2.0.0.html
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[X] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

Thanks, this is a duplicate of https://github.com/apache/beam/issues/27221. Left a comment there.

closing as a duplicate, but this is something that should be fixed.


Grant to target not working

I click a token, then hit grant to target and it says I need to click a token that I just clicked

Targeting is not the same as selecting.


store.state is false on client refresh in v2.0 networkInterface default function

@Atinux thanks for merging my PR but it seems to not work properly:
defaultNetworkInterface.js
export default ({store, req, isServer} ) => {
  let token
  if (isServer) {
    token = getTokenFromCookie(req)
    token && store.commit('SET_AUTH_TOKEN', token)
  } else {
    token = store.state.authToken
  }
  console.log(isServer, token) // => token is always empty on client side
}

The network interface is missing the store states on isServer===false.
This brings me to a general question: how is Vuex supposed to work? If users commit a Vuex store state on server side, should the Vuex state be present on client side as well? I think the plugin/module behaviour might miss to persist the correct Vuex state at the moment

Hi @dohomi
With the new release of Nuxt.js, the store replace state has been moved after plugins calls, see https://github.com/whtsky/nuxt.js/blob/add2bcb89fdaa4b366694a04b2cc95c8a11cad33/lib/app/index.js#L147
What you can do on client side is to read window.__NUXT__.state.authToken

@Atinux Thanks works great!


Make self.fd Optional.

Right now, self.fd is sometimes in DiskDevice. This messes up mypy/pylint, because those tools work poorly with attributes that are added and removed dynamically. Hence, we make it an Optional field instead.
Necessary for my fix for #6294.

So hasattr for this is also checked in realize, and that will change that behavior. Is this okay?


Throttle password request attempts

As described in #3858 we should probably throttle the attempts to request the reset password link.
I was not convinced about having the throttle mechanism url based as @tabacitu suggested, because the limitation would be per ip and not per email as I think it should be in this case.
I digged a bit and find out that Laravel already provides the throttle mechanism from 6.x ++, I just set it up as 1 attempt per 60 seconds.
Let me know what you think @tabacitu !
Best,
Pedro

@tabacitu you make a valid point.
I think having multiple-ips at your disposal for this kind of attack is what makes it way more difficult to pull out! If you are a person with that many resources I don't really think any of the protections we offer would suffice, but yeah, I agree with you, having an extra layer of security is a good thing I think.
I just noticed we are trying to get to a "reasonable" number for the delay .. should we make it configurable ? Since it's setup in the Provider it's not that easy to override and I am pretty sure that people would want to override it for testing purposes or something.
I am updating this PR in a few! Will let you know!
Pedro

A new inspection was created.

A new inspection was created.


Prioritize custom view templates

Current behavior:
Custom templates are ignored by the tailwindcss-rails generator
Expected behavior:
View templates under "#{RAILS_ROOT}/lib/templates/erb/scaffold" should be prioritized by the generator.
https://github.com/rails/tailwindcss-rails/issues/164

@mojodojo101 Thanks for submitting this. CI is failing, can you look into it, please?

@mojodojo101 Thanks for submitting this. CI is failing, can you look into it, please?

Hmm, i guess Rails.root isnt initialized in those tests, would it be fine to simply check if its nil with an if statement or is that piece of code i wrote just rubbish?

Can you please remove the version bump and changelog entry from this PR to avoid conflicts?
Would it be possible to write a test to describe the behavior you're introducing?

I removed the Changelog and Version bump 😄 . I dont know much about writing tests/ integration tests so i might have to look into that once i have some time on my hands.
The tldr of the behavior introduced here is:

Check if there is a /lib/templates/erb/scaffold  folder in rails
If that is the case use the templates in that scaffold folder rather than the folder provided by tailwindcss-rails

Following the procedure that is described here
https://guides.rubyonrails.org/generators.html :
... Another common use of custom templates is overriding the default scaffold view templates. You can override any of these by creating the appropriate file (e.g. index.html.erb, show.html.erb, etc) in lib/templates/erb/scaffold.

At the moment it just checks if the folder is there, if we want file granularity, this commit wont do just fyi.

I just hit this issue. Is there any additional work required to make this mergeable?

I'm closing this in preference of #314 which is a simpler fix and has test coverage.


Fix GENEVE test case error message

Error message says "VXLAN". Possibly a cut&paste error.

Fixed in the https://github.com/gopacket/gopacket fork.


"docker ps" doesn't include non-running containers with --before filter

Description of problem:
ubuntu@docker:~$ docker ps -a
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS                    PORTS               NAMES
a3396c149690        busybox             "sh"                24 hours ago        Up 9 seconds                                  container3
e86c22769399        busybox             "sh"                24 hours ago        Exited (0) 23 hours ago                       container2
0e338f61bf62        busybox             "sh"                24 hours ago        Up 14 seconds                                 container1
ubuntu@docker:~$ docker ps --before=container3
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES
0e338f61bf62        busybox             "sh"                24 hours ago        Up 27 seconds                           container1
ubuntu@docker:~$ docker ps --since=container1
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS                    PORTS               NAMES
a3396c149690        busybox             "sh"                24 hours ago        Up 31 seconds                                 container3
e86c22769399        busybox             "sh"                24 hours ago        Exited (0) 23 hours ago                       container2

I think the before filter should include the non-running containers like since filter, also like the man doc (https://github.com/docker/docker/blob/master/man/docker-ps.1.md#options) says.
docker version
Client:
 Version:      1.9.0-rc5
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   9318004
 Built:        Tue Nov  3 05:59:27 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.0-rc5
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   9318004
 Built:        Tue Nov  3 05:59:27 UTC 2015
 OS/Arch:      linux/amd64

docker info :+1:
Client:
 Version:      1.9.0-rc5
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   9318004
 Built:        Tue Nov  3 05:59:27 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.0-rc5
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   9318004
 Built:        Tue Nov  3 05:59:27 UTC 2015
 OS/Arch:      linux/amd64
ubuntu@docker:~$ docker info
Containers: 3
Images: 2
Server Version: 1.9.0-rc5
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 8
 Dirperm1 Supported: false
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.13.0-43-generic
Operating System: Ubuntu 14.04.1 LTS
CPUs: 4
Total Memory: 15.67 GiB
Name: docker
ID: UT5D:M5ZY:IEXQ:6LCU:YX6J:ZMWM:BLNJ:MTXH:PRHO:ETVF:3FHH:6LC4
WARNING:

uname -a :+1:
Linux docker 3.13.0-43-generic #72-Ubuntu SMP Mon Dec 8 19:35:06 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux

Environment details (AWS, VirtualBox, physical, etc.):OpenStack Cloud instance
Actual Results: before filter doesn't include the non-running containers
Expected Results:  before filter includes the non-running containers

debug info for docker 1.8.2 version:
    mars@docker:~$ docker ps -a
CONTAINER ID        IMAGE                    COMMAND              CREATED             STATUS                      PORTS               NAMES
a493de0ead7e        wenchma/ubuntu:mariadb   "/bin/bash"          6 days ago          Up 5 hours                  3306/tcp            thirsty_lumiere
37c688837ad4        ubuntu:14.04             "/bin/bash"          10 weeks ago        Dead
35f84153ecbd        ubuntu:14.04             "/usr/sbin/sshd-D"   10 weeks ago        Created                                         furious_darwin
0a86cb72698a        wenchma/ubuntu:mariadb   "/bin/bash"          10 weeks ago        Exited (0) 10 weeks ago                         condescending_leakey
d31eca25d0a9        wenchma/ubuntu:mariadb   "/bin/bash"          3 months ago        Up 4 hours                  3306/tcp            grave_engelbart
79938f784a12        ubuntu                   "/bin/bash"          3 months ago        Exited (0) 3 months ago                         agitated_newton
04cb438ca8bb        training/webapp          "python app.py"      3 months ago        Exited (137) 3 months ago                       admiring_hypatia
f6d8b1019f5b        ubuntu                   "/bin/bash"          3 months ago        Exited (0) 3 months ago                         reverent_wozniak
f430e879030c        ubuntu:14.04             "/bin/bash"          3 months ago        Exited (0) 3 months ago                         grave_brown
mars@docker:~$ docker version
Client:
 Version:      1.8.2
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   0a8c2e3
 Built:        Thu Sep 10 19:19:00 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.8.2
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   0a8c2e3
 Built:        Thu Sep 10 19:19:00 UTC 2015
 OS/Arch:      linux/amd64
mars@docker:~$ docker ps --before=furious_darwin
CONTAINER ID        IMAGE                    COMMAND             CREATED             STATUS                      PORTS               NAMES
0a86cb72698a        wenchma/ubuntu:mariadb   "/bin/bash"         10 weeks ago        Exited (0) 10 weeks ago                         condescending_leakey
d31eca25d0a9        wenchma/ubuntu:mariadb   "/bin/bash"         3 months ago        Up 4 hours                  3306/tcp            grave_engelbart
79938f784a12        ubuntu                   "/bin/bash"         3 months ago        Exited (0) 3 months ago                         agitated_newton
04cb438ca8bb        training/webapp          "python app.py"     3 months ago        Exited (137) 3 months ago                       admiring_hypatia
f6d8b1019f5b        ubuntu                   "/bin/bash"         3 months ago        Exited (0) 3 months ago                         reverent_wozniak
f430e879030c        ubuntu:14.04             "/bin/bash"         3 months ago        Exited (0) 3 months ago                         grave_brown

@wenchma So the problem is that --before and --since do not act the same, one is showing only running ones, and the other all of them.
Now, I wonder if it's --before we should change or --since. My point is, by default docker shows only running one. You have to had -a to show them all. From a UX perspective, I would this a flag like --before or --since would act the same.
$ docker ps
# shows running ones
$ docker ps -a
# shows all of them
$ docker ps --before=mycontainer
# shows running ones started before mycontainer
$ docker ps -a --before=mycontainer
# shows all the container started before mycontainer
$ docker ps --since=mycontainer
# shows running ones started since mycontainer
$ docker ps -a --since=mycontainer
# shows all the container started since mycontainer

Right now to achieve the "Only show the running container since mycontainer", you have to do docker ps --since=mycontainer --filter=status=running.
So, I tend to agree something should be done to fix the difference of behavior between --since and --before, but not sure it's --before.
/ping @thaJeztah for opinions :stuck_out_tongue_closed_eyes:

I actually think (and mentioned that thought in other issues) we should mark --before and --since deprecated, and re-implement them as options for --filter. In that implementation we can change the behavior to what we think is most logical. (And keep the behavior for --before and --since unchanged until they are removed)

@thaJeztah ah yeah, that feels like a pretty good idea :+1:

And to answer "how should they behave"; a filter should only affect the thing you're filtering on; so --filter=since=something should show the same as docker ps, but filtered, unless I add a status filter

@wenchma would you like to implement this (by updating your current PR) ? :wink:

@vdemeester  @thaJeztah 's idea looks good, :+1:  I will implement it to update my current PR.

fixed this issue and completed the design, close it.


Name and copyright

We can add a copyright with dynamic year and we can add name "To-Do List" in   list for better understanding ,name is not written in page.

Can I fix this issue , if possible please assign me this , I will complete it within a day

@manish-bhukar r u working on this?

Please assign me, I can complete the task today.


Add compare tool to library-view

This tool lets you diff between Dats and publish/revert files between them.

Looks good. I'll test soon and report back.

If I need to scroll down in compare mode, I can't see, which direction "publish" or "revert" actuall is. .. So do I copy from A -> B or B -> A
I would like the tooltip to be more descriptive. eg: test -> new for [publish]  AND new -> test for [revert]

Version 17 and latest are differnt! It would be nice see the differences between 2 versions, to resurrect deleted content if needed.

The [swap] function is a very nice feature and extremely useful ... but ... the [swap] button should be more visible.

@pfrazee @taravancil Looks good 👍🏼 I think its a good idea to start here. Provide developers with something simple and wait to see where the missing use cases are. My only feedback is that the "publish" feature is a bit hidden.
I'm wondering if it will be immediately apparent to devs who aren't following this conversation that workspaces -> publish is now hidden behind compare -> select archive -> publish. If we are taking Github as an example, they have a call to action on the landing page of a repository in the form of a New Pull Request button. You don't have to go digging through navigation to know it's an option:

If we aren't going to provide a "draft" solution in the immediate future, it might be good to at least provide some hints to devs on how they might use "Compare" to replace the old Workspaces workflow. For example, when they land on an empty "Compare" page, we could use that as an opportunity to say teach the user how to make a makeshift draft:

Select an archive to compare with Test Website. You can review the changes and merge them together.
Hint: Want to draft some changes, but aren't ready for the world to see them yet? Make a copy of your site and use this page to "publish" the changes when you're ready!

I'm mostly viewing this from the perspective of a user who has never seen Github, but knows they don't want to "save their changes" yet. In their case, "Compare" might not be self-explanatory. This is a bit of a worst-case scenario, but hey we already have kids creating sites in Beaker so I don't think it's too unlikely. This was kind of my thought behind ultimately implementing a draft process, nevertheless it would be interesting to see if we can find a way to survive without.

Thanks for the feedback @pmario and @webdesserts.
@webdesserts I'm thinking that, for the moment, "Compare" will be a slightly more advanced feature. We can find ways to direct people to it, and we can document it, but my thinking is that automatic-publish is suitable for the average user. (The lack of drafts is reflective of our limited resources -- trying to get best possible without holding up 0.8 much longer.)
Ideally what we will figure out is a way for somebody to think, "How do I choose when things get published?" and then they can find that answer quickly.
In the meantime, we can spend some time trying this tool and thinking about it.

Let's sit on this before merging. There are still a number of unanswered questions that I don't have time to answer right now. Listing them here as TODOs for myself:
[ ] - The language around publishing and reverting isn't clear. What's being published to what? What gets reverted?
[ ] - In the dropdown for selecting the target, Untitled archives are blank. I think we should also included the shortened dat URL next to the title so dats are easier to identify.
[ ] - I don't love the arrow icon, let's replace that

We’re just getting back into the swing of things after some international
travel. I won’t give you a timeline, but yes, eventually there will be
progress.
On Wed, Jun 13, 2018 at 07:23 Mario Pietsch<EMAIL_ADDRESS>wrote:

Any progress expected on this front?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/beakerbrowser/beaker/pull/988#issuecomment-396918332,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHO8QYVNWcX04AOewNmlapRKnztXTDAJks5t8QQngaJpZM4T-tE4
.

We're going to merge this into draft-mode-rebased and update it to work with the new drafts system


[BUG] -  error when trying to upload a text or pdf file

Description
error when trying to upload a text or pdf file

Using reader <kotaemon.loaders.pdf_loader.PDFThumbnailReader object at 0x000001B9BD6CF580>
Page numbers: 72
Got 72 page thumbnails
Adding documents to doc store
Getting embeddings for 200 nodes
RetryError[<Future at 0x1b9bfb639d0 state=finished raised AuthenticationError>]
Traceback (most recent call last):
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\ktem\index\file\pipelines.py", line 724, in stream
file_id, docs = yield from pipeline.stream(
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\ktem\index\file\pipelines.py", line 588, in stream
yield from self.handle_docs(docs, file_id, file_path.name)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\ktem\index\file\pipelines.py", line 400, in handle_docs
yield from insert_chunks_to_vectorstore()
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\ktem\index\file\pipelines.py", line 385, in insert_chunks_to_vectorstore
self.handle_chunks_vectorstore(chunks, file_id)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\ktem\index\file\pipelines.py", line 427, in handle_chunks_vectorstore
self.vector_indexing.add_to_vectorstore(chunks)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\kotaemon\indices\vectorindex.py", line 92, in add_to_vectorstore
embeddings = self.embedding(docs)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\theflow\base.py", line 1097, in call
raise e from None
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\theflow\base.py", line 1088, in call
output = self.fl.exec(func, args, kwargs)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\theflow\backends\base.py", line 151, in exec
return run(*args, **kwargs)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\theflow\middleware.py", line 144, in call
raise e from None
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\theflow\middleware.py", line 141, in call
output = self.next_call(*args, **kwargs)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\theflow\middleware.py", line 117, in call
return self.next_call(*args, **kwargs)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\theflow\base.py", line 1017, in runx
return self.run(*args, **kwargs)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\kotaemon\embeddings\base.py", line 10, in run
return self.invoke(text, *args, **kwargs)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\kotaemon\embeddings\openai.py", line 104, in invoke
resp = self.openai_response(client, input=input, **kwargs).dict()
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\tenacity_init.py", line 289, in wrapped_f
return self(f, *args, **kw)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\tenacity_init_.py", line 379, in call
do = self.iter(retry_state=retry_state)
File "D:\ia\koeman\kotaemon-app\install_dir\env\lib\site-packages\tenacity_init_.py", line 326, in iter
raise retry_exc from fut.exception()
tenacity.RetryError: RetryError[<Future at 0x1b9bfb639d0 state=finished raised AuthenticationError>]
len(results)=3, len(file_list)=3

Reproduction steps
I try to upload files but I always get some error whether they are PDF files in English or Spanish

Screenshots
![DESCRIPTION](https://prnt.sc/KcUt1cfNJH8q)

Logs
No response
Browsers
Chrome, Other
OS
Windows
Additional information
No response

Can you provide more context ?
Do you use ollama (local version) or some API from azure, etc.?
Have you set it up correctly ?
For more information you can refer to our guideline here.
Hope it help

I have the same issue.
Setup: I am running the application inside Docker and accessing it via http://host.docker.internal/.
Vendor: I'm using OpenAI as the vendor, and I have not customized any settings – all values are set to the default.
Authentication: The API key is set correctly in the environment variables, but I’m still encountering the RetryError related to authentication.
Version: I’m not using Ollama (local version) or Azure API, just OpenAI’s default setup.
Chat functionality: The chat functionality works without any issues, but the error occurs when trying to generate embeddings via the GPT API.
I've already checked the documentation, but I’m unsure if there are additional setup steps when running inside Docker.
Any help on this would be appreciated. I’ll review the guidelines you linked as well. Thank you!

Hi @Lee-Ju-Yeong
In the Resources > Emebdding tab, we have the function for testing connection. Can you try it & see what's the output ?
Previously It's recommend to set up all credentials via the UI. (The .env will be used for the 1st time and later credentials will be stored on DB).
Hope it help


feat: support graceful shutdown (#2469)

Optimize the code to support graceful shutdown, and add configuration server.gracefulShutdownTimeout to allow users to customize the timeout, the default value is 5(seconds).

Codecov Report
Patch coverage: 91.48% and project coverage change: +0.02 :tada:

Comparison is base (cbbfd85) 79.98% compared to head (cdbcc13) 80.01%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2475      +/-   ##
==========================================
+ Coverage   79.98%   80.01%   +0.02%
==========================================
  Files         599      596       -3
  Lines       49785    49454     -331
==========================================
- Hits        39821    39571     -250
+ Misses       8007     7943      -64
+ Partials     1957     1940      -17

Flag
Coverage Δ

go-1.15-386
?

go-1.15-amd64
?

go-1.16-386
?

go-1.16-amd64
80.01% <91.48%> (+0.01%)
:arrow_up:

go-1.17-386
?

go-1.17-amd64
?

go-1.18-386
?

go-1.18-amd64
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

net/ghttp/ghttp.go
100.00% <ø> (ø)

net/ghttp/ghttp_server_graceful.go
59.55% <0.00%> (-1.03%)
:arrow_down:

net/ghttp/ghttp_server.go
61.94% <100.00%> (ø)

net/ghttp/ghttp_server_config.go
61.70% <100.00%> (+0.20%)
:arrow_up:

os/gcron/gcron_schedule_fix.go
81.81% <0.00%> (-9.10%)
:arrow_down:

os/gfsnotify/gfsnotify_watcher_loop.go
78.15% <0.00%> (-5.05%)
:arrow_down:

frame/gins/gins_database.go
53.78% <0.00%> (-3.37%)
:arrow_down:

net/gtcp/gtcp_pool.go
72.22% <0.00%> (-3.34%)
:arrow_down:

os/gfpool/gfpool_file.go
65.71% <0.00%> (-2.86%)
:arrow_down:

container/gmap/gmap_hash_str_any_map.go
97.00% <0.00%> (-1.00%)
:arrow_down:

... and 6 more

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

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


Improve performance of sasl authentication

Before this PR, the ImpersonationAuthenticator class could effectively
bottleneck the amount of concurrent channels that a worker could handle.
The <init> for the ImpersonationAuthenticator class required iterating over
all keys within an Alluxio configuration, not only doing key lookups, but also
performing regex matching. For some keys this could be especially expensive
(i.e, default supplier which calls Runtime.getRuntime().availableProcessors()).
On top of that, performing regex matching is also quite expensive for afew
hundred keys.
After taking a quick peek at the code it seems that it's entirely unnecessary to
instantiate the impersonation authenticator for every new handler that we
create. Instead, it can just be a field within the DefaultAuthenticationServer.
Then, it only ever needs to be instantiated once.
pr-link: Alluxio/alluxio#11293
change-id: cid-96438acee91197e2e9a380494848db481c458660

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9143/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9144/
Test PASSed.

The only code change for this cherry pick was a test fix, so I'm going to merge it. a2d72240d26dd39af9014f5990ba90cb7f09d419

alluxio-bot, merge this please


ToAnotherOrDefault

Create docs for this method
Right here by analogy with another methods

Hi.

solved in this PR


About symbols' visibility in shared library

System information

TensorFlow version (you are using):  v2.0.0
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
As we can see from tf_version_script.lds in v2.0.0, none of Eager* symbols have been set to global visibility. But in newest master codes, this has been updated.
In our usage scenario, we need transfer EagerTensor_Handle in our python apis to read or update the tensors. It's important that no copy happens in Tensor transfering. However, EagerTensor* like symbols depend on _pywrap_tensorflow_internal.so are Local visibility in current stable release.
Will this change the current api? How?
No. Only version script file: tf_version_script.lds
Who will benefit with this feature?
The one who need read or write Tensors outside tensorflow.
And more
Maybe it's not good to update symbols' stripping rule in v2.0.0 release (or other tags). But this have changed in master branch. When will you release this feature in next edition? Or is this a good way to interchange data between outerside and tf (with zero copy)?

It is an API change as new symbols get exposed.
For 2.0 and 2.1 it is unlikely that these new symbols will be exposed. However, you can get them from tf-nightly and from the TF2.2 release in a few months.

@mihaimaruseac Thanks for response. Looking forward to the new feature and TF2.2 release.


Corrected typo in skbio.diversity docs

Please complete the following checklist:

[x] I have read the guidelines in CONTRIBUTING.md.

[ ] I have documented all public-facing changes in CHANGELOG.md.

[ ] This pull request includes code, documentation, or other content derived from external source(s). If this is the case, ensure the external source's license is compatible with scikit-bio's license. Include the license in the licenses directory and add a comment in the code giving proper attribution. Ensure any other requirements set forth by the license and/or author are satisfied. It is your responsibility to disclose code, documentation, or other content derived from external source(s). If you have questions about whether something can be included in the project or how to give proper attribution, include those questions in your pull request and a reviewer will assist you.

[x] This pull request does not include code, documentation, or other content derived from external source(s).

Note: REVIEWING.md may also be helpful to see some of the things code reviewers will be verifying when reviewing your pull request.
Issue #1872
It's a one character change to a doc page. I'm not sure it warrants a change log entry.

Thanks, @jmeppley!

This is a pretty straightforward PR. Let me merge.


Huge Memory Consumtion

Created a dotnet 5 project with the nuget reference, the application consumed 70MiB RSS memory.
static async Task Main(string[] args)
{
    while (true)
    {
        Console.WriteLine(Guid.NewGuid().ToString().Unidecode());
        await Task.Delay(TimeSpan.FromSeconds(1));
    }
}

Thought it was quite a lot of memory for such a small library, created a new dotnet 5 project with a project
reference to the source code of the master branch, and the application consumed 200 MiB RSS memory???
Removed the partial (Diff below) and the RSS memory shrunk down to 5 MiB RSS.
diff --git a/src/Unidecoder.Characters.cs b/src/Unidecoder.Characters.cs
index dd8cdac..3b4d792 100644
--- a/src/Unidecoder.Characters.cs
+++ b/src/Unidecoder.Characters.cs
@@ -27,11 +27,11 @@ using System.Collections.Generic;

 namespace Unidecode.NET
 {
-    public static partial class Unidecoder
+    public static class Unidecoder2
     {
-        private static readonly Dictionary<int, string[]> characters;
+        public static readonly Dictionary<int, string[]> characters;

-        static Unidecoder()
+        static Unidecoder2()
         {
             characters = new Dictionary<int, string[]> {
                 {0 /*0 000*/, new[]{
@@ -605,4 +605,4 @@ namespace Unidecode.NET
             }
         }
     }
-
\ No newline at end of file
+
diff --git a/src/Unidecoder.cs b/src/Unidecoder.cs
index 558c725..1e1653e 100644
--- a/src/Unidecoder.cs
+++ b/src/Unidecoder.cs
@@ -6,7 +6,7 @@ namespace Unidecode.NET
     /// <summary>
     /// ASCII transliterations of Unicode text
     /// </summary>
-    public static partial class Unidecoder
+    public static class Unidecoder
     {
         /// <summary>
         /// Transliterate Unicode string to ASCII string.
@@ -42,7 +42,7 @@ namespace Unidecode.NET
                 {
                     var high = c >> 8;
                     var low = c & 0xff;
-                    if (characters.TryGetValue(high, out var transliterations))
+                    if (Unidecoder2.characters.TryGetValue(high, out var transliterations))
                     {
                         sb.Append(transliterations[low]);
                     }
@@ -71,7 +71,7 @@ namespace Unidecode.NET
             {
                 var high = c >> 8;
                 var low = c & 0xff;
-                result = characters.TryGetValue(high, out var transliterations) ? transliterations[low] : "";
+                result = Unidecoder2.characters.TryGetValue(high, out var transliterations) ? transliterations[low] : "";
             }

             return result;

My pull request (https://github.com/thecoderok/Unidecode.NET/pull/16) now contains also a fix for this memory usage problem.
this issue has been mentioned also here: https://github.com/dotnet/runtime/issues/54688
reading that thread I learned that the problem in the static initialization of the characters dictionary: it is a huge initialization and it generates some complex code. I applied one of the fixes suggested in that thread: i totally scrapped the Unidecode.Characters.cs source code and initialized the characters variable at runtime from data read from an embedded resource file.
I have modified the python script in order to generate that embedded resource file.
you can give it a try by pulling from this branch of mine: https://github.com/csm101/Unidecode.NET/tree/array_instead_of_dictionary
this fork is also 3 times faster in decoding (which was the first reason for which I forked it)


Error: Cannot truncate time period Period { begin: 453.57, end: 457.84 } at 453.57

Hi! I'm getting this error using the export function. I get it with both the binary release and installing via cargo install.
$ ~/.cargo/bin/substudy export review umaru-1.mp4 en-1.srt jp-1.srt
Error: Cannot truncate time period Period { begin: 453.57, end: 457.84 } at 453.57
$ ~/.cargo/bin/substudy export csv umaru-1.mp4 en-1.srt jp-1.srt
Error: Cannot truncate time period Period { begin: 453.57, end: 457.84 } at 453.57

The sub files were originally .ass files that I converted to .srt files using this tool: https://subtitletools.com/convert-to-srt-online
This is my system: Linux macbookpro-arch 4.14.11-1-ARCH #1 SMP PREEMPT Wed Jan 3 07:02:42 UTC 2018 x86_64 GNU/Linux
If it's relevant, here is my FFmpeg version:
ffmpeg version 3.4.1 Copyright (c) 2000-2017 the FFmpeg developers
built with gcc 7.2.1 (GCC) 20171128
configuration: --prefix=/usr --disable-debug --disable-static --disable-stripping --enable-avisynth --enable-avresample --enable-fontconfig --enable-gmp --enable-gnutls --enable-gpl --enable-ladspa --enable-libass --enable-libbluray --enable-libfreetype --enable-libfribidi --enable-libgsm --enable-libiec61883 --enable-libmodplug --enable-libmp3lame --enable-libopencore_amrnb --enable-libopencore_amrwb --enable-libopenjpeg --enable-libopus --enable-libpulse --enable-libsoxr --enable-libspeex --enable-libssh --enable-libtheora --enable-libv4l2 --enable-libvidstab --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxcb --enable-libxvid --enable-shared --enable-version3
libavutil      55. 78.100 / 55. 78.100
libavcodec     57.107.100 / 57.107.100
libavformat    57. 83.100 / 57. 83.100
libavdevice    57. 10.100 / 57. 10.100
libavfilter     6.107.100 /  6.107.100
libavresample   3.  7.  0 /  3.  7.  0
libswscale      4.  8.100 /  4.  8.100
libswresample   2.  9.100 /  2.  9.100
libpostproc    54.  7.100 / 54.  7.100

If it helps, here is (what I think is) the relevant sections in the .srt files:
106
00:07:30,520 --> 00:07:33,550
Ah, so cat videos are popular.

107
00:07:33,570 --> 00:07:37,840
Wins
1 Million Yen!

108
00:07:33,570 --> 00:07:37,840
The pet that ranks 1st
in number of views...

109
00:07:33,570 --> 00:07:37,840
Contest!

110
00:07:33,570 --> 00:07:37,840
Animal
Video

111
00:07:33,570 --> 00:07:37,840
Enter Now!

112
00:07:33,990 --> 00:07:37,850
Oh? Looks like there's a contest going
on for whoever can get the most views.

113
00:07:37,850 --> 00:07:39,890
The one million yen is mine!

There are several segments with the time period that it complains about.
It doesn't seem to be refering to any times in jp-1.srt but here is that general area:
125
00:07:31,520 --> 00:07:33,180
猫動画が人気あるのか

126
00:07:34,960 --> 00:07:37,620
今　再生数を競う大会が開かれてるな

127
00:07:38,140 --> 00:07:39,860
１００万はうまるのだよ

Fix: After manually merging the segments in en-1.srt that started at the same time, this issue went away and everything worked as expected.

A more informative error message would definitely be helpful.
I think, however, that the most useful approach would be for substudy to just save the file with a different name (with an added suffix, for example) and not fail at all
If substudy is meant for language learners to have an easy and streamlined way to extract audio segments, I think it is important not to make them manually edit subtitle files so there are no duplicated timestamps.
Having said that, thank you for making this tool, it is definitely a useful one

I'm running into this error now and I haven't been able to resolve it. In my case I did notice one place in one of my srt files where there was a duplicate start time. However, after I manually merged the two entries into one, I got another error message, referring to another timestamp. A better error message would be of great help :)

Thank you, everyone, for the reports!
People keep having problems with overlapping subtitles in SRT files. I need to come up with some sort of strategy for handling this.


feat(serving): Support custom image

What this PR does / why we need it:
Add your description
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @your-reviewer

Release note:

NONE

/assign @FogDong @zw0610


[main] chore: update third party notices

[main] chore: update third party notices

Going to merge this with [ci skip] in the title.


Pull latest version

https://github.com/gregjacobs/Autolinker.js/releases/tag/0.25.0

Fixed my problem by upgrading to Babel 6. See https://github.com/gregjacobs/Autolinker.js/issues/86#issuecomment-202915086

Premature celebration, production build still broken :(


CSS nesting first-selector not highlight

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.88.1
OS Version: Darwin arm64 23.4.0

Steps to Reproduce:

Duplicate of upstream issue about supporting nesting: https://github.com/microsoft/vscode-css/issues/9


program state now obeys visibility settings

Besides the changes requested by @AlexVlx , this PR causes TensorFlow to fail if using HIP_VISIBLE_DEVICES=1 (anything non-zero) due to HIP's default stream for the default context being for device 0 regardless of users' requested visibility.  I am investigating the new issue caused by this PR and will amend ASAP.

The visibility env vars are read once (when hip is initialized) in a call_once routine.
From: Alex Voicu<EMAIL_ADDRESS>Reply-To: ROCm-Developer-Tools/HIP<EMAIL_ADDRESS>Date: Monday, March 25, 2019 at 10:18 AM
To: ROCm-Developer-Tools/HIP<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [ROCm-Developer-Tools/HIP] program state now obeys visibility settings (#985)
@AlexVlx requested changes on this pull request.

In src/hip_hcc.cpphttps://github.com/ROCm-Developer-Tools/HIP/pull/985#discussion_r268694995:

         const auto agent = acc.get_hsa_agent();

         if (!agent || !acc.is_hsa_accelerator()) continue;

       // If device is not in visible devices list, ignore

       if (std::find(g_hip_visible_devices.begin(), g_hip_visible_devices.end(), (i - 1)) ==

Strictly from the standpoint of "I'd like to understand this code months / years later" the above would be highly preferable IMHO (even though it still pre-assumes that there's a bijection from index in the vector returned by get_all() and visibility info as reflected in g_hip_visible_devices, which induces implicit coupling at a distance).
Separately, it's unclear to me if the access to g_hip_visible_devices is safe - could a situation arise in which someone is adjusting visibility in a separate thread, concurrently with this invocation of visible_hsa_agents? Or is visibility constant per process?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ROCm-Developer-Tools/HIP/pull/985#pullrequestreview-218396840, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACYSAqVtdtvEQDI7VUi1YXv2Sq2T-w7Bks5vaOjNgaJpZM4cElyM.

The correct solution was not to have program state use visible devices, but to use the TLS current device.
@AlexVlx please review again.
@whchung please review.

This is wrong. Executables pre-populates for all GPUs in the process, once. With this change if you ever feel like using anything but the current agent it will not have a loaded executable i.e. this breaks quite a few things.

The current logic of pre-populating for all GPUs in the process is what is breaking Horovod.  I thought the code was doing a lazy init, and if the needed kernel wasn't found, it was rebuilding the in-memory kernels.  Besides using an env var, e.g., HIP_VISIBLE_DEVICES, we need to support this use case.  There must be some solution that works for all use cases.

There must be but this isn't it.

I hadn't been following development close enough.  It seems #929 got rid of the rebuild/reinit logic.  What was the reason for the rebuild logic removal?  If we are opposed to reviving it, what might be a possible alternative that would only load for 1 GPU at a time, on demand?

This PR is obsolete; replaced by #991.


"Hello World" flask app

The end goal is: 1. Spin an environment 2. Open your host browser 3. Go to: http://localhost:5000
The result should be: "Hello World"  in your webpage

Uploaded a PR for review.
@David Mdiniaradze
@Fakhri Said

@tomerguttman Ask David and Fakhri add a comment to this issue. As soon as they add a comment (any comment), I can add them to the Assignees list.


Add support for the phusion passenger PPA

Phusion provides a PPA for installing nginx + passenger. Especially on a slow VPS this is much faster than compiling using the gem, and the package gets updated by doing standard apt-get upgrades.
If that were an option to the passenger recipe it would be a great feature.

Resolved in #321.


Future proof

Resolves #25

Fixes how result proc in some tests was passed, which makes tests pass in ruby >=3.0
No actual application code needed updating
Pins some dependencies to versions required by ruby 3.2

@mark-cooper - was not sure if I was supposed to

get each gem on 3.2; or
get each gem on 3.0, then let you know so you can get Importer on 3.0, then do the same for 3.1, then 3.2

I went with getting this to 3.2 first, but it's easy to roll back if necessary

@kspurgin the way you've done it looks good to me at a glance. I'll take a proper look soon!


[Feature] Add session variable to toggle ANSI color in explain output

Why I'm doing:
Adds a session variable to toggle ANSI color in explain and analyze output
What I'm doing:
Fixes #52716
What type of PR is this:

[ ] BugFix
[X] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

 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.Khee-Chin Chua seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Kaisen, given this affects the output for both EXPLAIN and EXPLAIN ANALYZE, I think colorExplainOutput is a better name. What do you think?
Sent from Outlook for iOShttps://aka.ms/o0ukef

From: kangkaisen @.>
Sent: Saturday, November 23, 2024 10:11:32 AM
To: StarRocks/starrocks @.>
Cc: KC @.>; Assign @.>
Subject: Re: [StarRocks/starrocks] [Feature] Add session variable to toggle ANSI color in explain output (PR #52960)
@kangkaisen commented on this pull request.

In fe/fe-core/src/main/java/com/starrocks/qe/SessionVariable.javahttps://github.com/StarRocks/starrocks/pull/52960#discussion_r1855010399:

@@ -966,6 +967,10 @@ public static MaterializedViewRewriteMode parse(String str) {
@VariableMgr.VarAttr(name = ENABLE_PROFILE, alias = IS_REPORT_SUCCESS)
private boolean enableProfile = false;

// Toggle ANSI color in explain output
@VariableMgr.VarAttr(name = COLOR_EXPLAIN_OUTPUT)
private boolean colorExplainOutput = true;

⬇ Suggested change

private boolean colorExplainOutput = true;

private boolean colorExplainAnalyzeOutput = true;

—
Reply to this email directly, view it on GitHubhttps://github.com/StarRocks/starrocks/pull/52960#pullrequestreview-2456169807, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AACDOX7CUR5MCJTRPU5UKAL2B7P5JAVCNFSM6AAAAABR6ETBPKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDINJWGE3DSOBQG4.
You are receiving this because you were assigned.Message ID: @.***>

@Mergifyio rebase

@Mergifyio rebase


1 Low Risk exposure but MatchCount is 0

Avoid duplicates

[x] Bug is not mentioned in the FAQ
[x] Bug is not already reported in another issue

Describe the bug
CWA says there was a low risk exposure "1 Risiko-Begegnung mit niedrigem Risiko", but MatchCount is in every check null (in the last 14 days).
grep Match Downloads/ExposureChecks-2020-09-17\ 2.json | grep -v 0 | wc -l
0

Expected behaviour
The exposure was before the last 14 Days and the App says "no risk"
Steps to reproduce the issue

Checking the risk in CWA
Comparing the risk with MatchCount in IOS

Technical details

iOS Version: 14
Device: iPhone 11

Possible Fix
--
Additional context
--

@igerhold
is this the same as #1106 ?

Hi @Tho-Mat,
I am sorry, did not found that issue with the search. In my point of view, it is the same issue as described there.
Kind Regards


Quick way of validating current parser output

Execute all parsers and compare output to expectation matrix

I see how this is useful. I don't know yet where/when to use this script. If it's some kind of test, I'd use assert to fail if an expectation i  not met. Or at least fail with sys.exit(1) so that a caller knows it failed  (e.g. if we want to integrate it in the GitHub action workflow).
WDYT?

@metaodi Sure, we I can add the sys.exit(1) without issues. Actually I was wondering if it wouldn't make sense to switch to a more object orient design, creation an abstract class CantonParser and then have AGParser, VDParser, ZHParser etc that would inherit from them. We could then have the validation method in the parent class and only would need in each of the child classes to define the expected fields (similar to the matrix) ... but with the additional benefit of when running the parser we could immediately report that something is missing. Let me know if that's something you would be interested in.

For now I've updated with sys.exit(1) if any error is present. Let me know if you like the object orient approach and I'll start the refactoring

@dominikgehl I don't think this is a direction we currently want to go. While I see your point of less duplicate code and clearer structure, I think the simplicity of our current approach is a strength.
I think it's good, that a lot of people (even if they don't know python) are able to understand and contribute to the current code.

@metaodi Got it with regard to the refactoring. That's exactly why I asked before starting to work on it. Have a good evening !

@dominikgehl
Could you move this code, to a separate file maybe?
I wanted to keep the matrix just a code with expectations.
I think the code is useful, but it should be separate and has few bugs.
Change
         result = subprocess.run(['python', f'scrape_{canton}.py'], stdout=subprocess.PIPE)

to
         result = subprocess.run([f'./scrape_{canton.lower()}.py'], stdout=subprocess.PIPE)

There is no need for python. In fact it is incorrect. As we use python3.
Also the cantons are in lower case letters in the file name.
Thank you.

@baryluk Sure. I'll move the code and create a new pull request


Profile picture in status bar 403s

Seems to have been an issue with Google's servers.  Not able to replicate anymore

initial error rate ~95%
present commit without change ~0%

Seems to be a CORS issue that only occurs on certain networks?


Help stating wrong environment variables to configure Azure storage

Describe the bug
tusd -help states AZURE_ACCOUNT_NAME and AZURE_ACCOUNT_KEY must be configured, if using -azure-storage.
The variables read are AZURE_STORAGE_ACCOUNT, respectively AZURE_STORAGE_KEY.
Full output:
 -azure-storage string
    	Use Azure BlockBlob Storage with this container name as a storage backend (requires the AZURE_ACCOUNT_NAME and AZURE_ACCOUNT_KEY environment variable to be set)

Thank you for reporting!


this is experimental, right?

https://tangrams.github.io/tron/#14.785377687974684/40.8080/-74.8714

I've been so focused on roads, no haven't touched this yet. Have to figure out what to do here. More elevation patterns?


Could you publish data sets in advance?

Can't wait to test our algorithm on the real image dataset.

You can search for video data online and make it according to the methods mentioned in the paper. We will release the data later.


unanalyzable not clearly defined

I am trying ExplicitImports.jl with a package with a slightly complex module structure (the package is mostly implemented in modules loaded by extensions). When I run check_no_implicit_imports, I get UnanalyzableModuleException and a message telling me that I show allow my module to be unalayzable. However, what "unanalyzable" means is not really clear. Is it an issue I can solve on my end?
(The package I want to check is ClimaUtilities.jl)

often it means dynamic include statements, but I don't see any there. If you do print_explicit_imports it should show which modules aren't analyzable. Also try setting ENV["JULIA_DEBUG"] = "ExplicitImports" first.
I wasn't able to reproduce:
julia> using ExplicitImports

julia> ENV["JULIA_DEBUG"] = "ExplicitImports"
"ExplicitImports"

julia> check_no_implicit_imports(ClimaUtilities)

To reproduce, you also have to load the packages that trigger extension loading:
import ExplicitImports
ENV["JULIA_DEBUG"] = "ExplicitImports"

import NCDatasets
import ClimaComms
import Interpolations
import ClimaCoreTempestRemap

using ClimaUtilities
ExplicitImports.check_no_implicit_imports(ClimaUtilities)

results in
julia> ExplicitImports.check_no_implicit_imports(ClimaUtilities)
ERROR: UnanalyzableModuleException
Module `ClimaUtilitiesClimaCoreTempestRemapExt.TempestRegridderExt` was found to be unanalyzable. Include this module in the `allow_unanalyzable` keyword argument to allow it to be unanalyzable.

Stacktrace:
 [1] should_ignore!(::Nothing, mod::Module; ignore::Tuple{})
   @ ExplicitImports ~/.julia/packages/ExplicitImports/JSYBk/src/checks.jl:260
 [2] check_no_implicit_imports(mod::Module, file::String; skip::Tuple{Module, Module, Module}, ignore::Tuple{}, allow_unanalyzable::Tuple{})
   @ ExplicitImports ~/.julia/packages/ExplicitImports/JSYBk/src/checks.jl:231
 [3] check_no_implicit_imports
   @ ~/.julia/packages/ExplicitImports/JSYBk/src/checks.jl:223 [inlined]
 [4] check_no_implicit_imports(mod::Module)
   @ ExplicitImports ~/.julia/packages/ExplicitImports/JSYBk/src/checks.jl:223
 [5] top-level scope
   @ REPL[8]:1

As for printing imports:
julia> ExplicitImports.print_explicit_imports(ClimaUtilities)
  Module ClimaUtilitiesClimaCoreTempestRemapExt.TempestRegridderExt could not be accurately analyzed, likely due to dynamic include statements. You can pass
  strict=false to attempt to get (possibly inaccurate) results anyway.

  Module ClimaUtilities.OutputPathGenerator is not relying on any implicit imports.

  Module ClimaUtilities.Utils is not relying on any implicit imports.

  Module ClimaUtilitiesClimaCoreTempestRemapExt is not relying on any implicit imports.

  Module ClimaUtilitiesClimaCoreInterpolationsExt.InterpolationsRegridderExt could not be accurately analyzed, likely due to dynamic include statements. You
  can pass strict=false to attempt to get (possibly inaccurate) results anyway.

  Module ClimaUtilitiesClimaCoreExt.TimeVaryingInputs0DExt could not be accurately analyzed, likely due to dynamic include statements. You can pass
  strict=false to attempt to get (possibly inaccurate) results anyway.

  Module ClimaUtilitiesClimaCommsExt.MPIUtilsExt could not be accurately analyzed, likely due to dynamic include statements. You can pass strict=false to
  attempt to get (possibly inaccurate) results anyway.

  Module ClimaUtilities.FileReaders is not relying on any implicit imports.

  Module ClimaUtilitiesClimaCoreInterpolationsExt is not relying on any implicit imports.

  Module ClimaUtilitiesNCDatasetsExt is not relying on any implicit imports.

  Module ClimaUtilitiesClimaCoreNCDatasetsExt.DataHandlingExt could not be accurately analyzed, likely due to dynamic include statements. You can pass
  strict=false to attempt to get (possibly inaccurate) results anyway.

  Module ClimaUtilities.DataHandling is not relying on any implicit imports.

  Module ClimaUtilities.Regridders is not relying on any implicit imports.

  Module ClimaUtilities.TimeVaryingInputs is not relying on any implicit imports.

  Module ClimaUtilities.TimeManager is not relying on any implicit imports.

  Module ClimaUtilitiesClimaCoreNCDatasetsExt is not relying on any implicit imports.

  Module ClimaUtilitiesClimaCoreNCDatasetsExt.TimeVaryingInputsExt could not be accurately analyzed, likely due to dynamic include statements. You can pass
  strict=false to attempt to get (possibly inaccurate) results anyway.

  Module ClimaUtilities is not relying on any implicit imports.

  Module ClimaUtilities.DataStructures is not relying on any implicit imports.

  Module ClimaUtilities.ClimaArtifacts is not relying on any implicit imports.

  Module ClimaUtilitiesClimaCommsExt is not relying on any implicit imports.

  Module ClimaUtilitiesClimaCoreExt is not relying on any implicit imports.

  Module ClimaUtilitiesClimaCoreNCDatasetsExt.SpaceVaryingInputsExt could not be accurately analyzed, likely due to dynamic include statements. You can pass
  strict=false to attempt to get (possibly inaccurate) results anyway.

  Module ClimaUtilities.MPIUtils is not relying on any implicit imports.

  Module ClimaUtilities.SpaceVaryingInputs is not relying on any implicit imports.

  Module ClimaUtilitiesNCDatasetsExt.NCFileReaderExt could not be accurately analyzed, likely due to dynamic include statements. You can pass strict=false to
  attempt to get (possibly inaccurate) results anyway.

The common thread I think I see is that the modules that cannot be analyzed are modules that are included in the extensions, as in ClimaUtilitiesClimaCoreTempestRemapExt.jl
module ClimaUtilitiesClimaCoreTempestRemapExt

include("TempestRegridderExt.jl")

end

where TempestRegridderExt.jl is a module itself.

interesting, I have a test for extensions but it doesn't include submodules in the extension, so something must be going wrong there, thanks for the report


.Net: When turning a plugin into an OpenAI plugin, payload property is introduced

When using a normal plugin with native functions, I can write code that looks like the following:
double answer = await kernel.InvokeAsync<double>(
"MathPlugin", "Sqrt",
    new() {
        { "number1", 12 }
    }
);
Console.WriteLine($"The square root of 12 is {answer}.");

But when I turn that same plugin into a REST service, the arguments get wrapped in a payload property which makes it difficult to call. (I'm only assuming this is what it'll look like since I get an error saying that this isn't supported yet)
double answer = await kernel.InvokeAsync<double>(
"MathPlugin", "Sqrt",
    new() {
        { "payload" , new Dictionary<string, object> {
            { "number1", 12 }
        }}
    }
);
Console.WriteLine($"The square root of 12 is {answer}.");

This could cause issues with planners and function calling because it makes calling REST services more complicated for the LLM.
The payload property should be removed and the child properties flattened out as individual parameters for KernelArguments to fill out.
If this cannot be finished by v1.0, we should make the OpenAPI package experimental.

It's possible to pass the arguments as a flat list as with other plugins, the functionality is there. It just needs to be activated through the EnableDynamicPayload property. More details in the Dynamic payload... ADR
We should make this the default behavior for consistency with other plugins and use the 'payload' property only for RestApi plugins with text/plain context, as OpenApi metadata does not advertise parameters for those.

This is great news! Yes, we should make this the default behavior.

The OpenAPI functionality has been updated to have the dynamic behavior enabled by default.

What work needs to be done to make it so the package is not marked as experimental? @matthewbolanos @SergeyMenshykh @markwallace-microsoft


Fixes #188 - Implement loading animation on trust.mybit.io

Fixes https://github.com/MyBitFoundation/MyBit-Network.tech/issues/188

Hi @yossipik , this might take a bit to solve as we have to setup CI/CD on this repo, and connect the bounty to this repository as well, but bear with us!

Hi @jjperezaguinaga, thanks for the heads-up.
Actually, I did not manage to connect the bounty. How should I do that?

Hi @yossipik , this might take a bit to solve as we have to setup CI/CD on this repo, and connect the bounty to this repository as well, but bear with us!

@yossipik Unfortunately for the time being, you need to create pull-request within the https://github.com/MyBitFoundation/MyBit-Network.tech repository that's empty and write "Fixes #188" from there otherwise it won't be picked up by Open Bounty

@yossipik Could you add an empty commit to trigger our CI/CD setup?

@jjperezaguinaga Build failed but I don't think it is related to my code. Please advise.

@yossipik It's missing the configuration that was added to the branch recently. Just sent you a Pull Request, for more real-time answers feel free to drop by our Gitter room.


🪲 Bug Report - VMSS Upgrade Error parameter name 'skipMigrateNATPoolsToNATRules' missing

Describe the bug
Upgrading a VMSS LB hit a bug where a parameter has been renamed but references were not fixed, resulting in error:
Start-AzBasicLoadBalancerUpgrade: A parameter cannot be found that matches parameter name 'skipMigrateNATPoolsToNATRules'.
Details
Version: 2.4.1

Fixed in 2.4.2


Add better error reporting to pio import

catch parsing errors when importing in parallel from text/json or other file types. Often a single malformed line will be very difficult to find so log the malformed line before throwing an abort exception.  Consider exception counting for parsing, allow some small number before aborting, this will catch a blank line at the end and simple exceptions.

comment from @k4hoo
can add try catch {} block for readEvent to catch any error when convert the json string to event and print message of the error json string
https://github.com/PredictionIO/PredictionIO/blob/develop/tools/src/main/scala/io/prediction/tools/imprt/FileToEvents.scala#L88

Is anyone working on this issue? If not, I would like to send a PR

No, would love to get a PR. Set it up against the "develop" branch. We are nearing a 0.9.6 release.


LibreOfficeDev 5.2.0 Beta2

Current version provided by you is Beta1

Done

@probonopd We need somehow to include locales into LO appimages.

If Bintray increases the maximum allowable file size... ;-)


More space under table in LinkedCellEditor

Didn't we have more gap between buttons and table?

Yes, but I made it uniform with the other dialogs we have.
https://github.com/sibvisions/flutter_jvx/commit/1b84ef28924ba8f012288c89257cdc6fba7b27a3

TableEditDialog, the dialog to edit columns in the table directly via popup, also only has 4px to the buttons. Thought it should look uniform and didn't think it looks that bad. But yes, we had double at 8px.

? do you mean 8px?
8px is better than 4px

Okay, will do that. But what about the TableEditDialog? Is it okay for both of these dialogs to have different gaps and its intentional, or should I increase the gap there too?


drivers: can: rename SocketCAN and CAN controller data types

Rename the CAN controller API data types to not have the "z" prefix as this makes them appear as internal APIs. This requires renaming of the SocketCAN API data types to avoid naming clashes. To match the new naming, the SocketCAN data types are moved into the correct header namespace (and the corresponding tests are moved too).
I suggest reviewing this PR patch by patch to get the history correct for all the renames and reshuffling.

Cool! Do you think it would be possible to provide something to migrate an application to the new naming? Wondering if we could attach a Coccinelle patch to help convert the types.

A simple sed statement will do. I could add those to the commit log, if needed.


Support SpeechRecognizer for iOS 10

1-fix error of xcode8 beta 5
2-support SpeechRecognizer for iOS 10
3-check  value of NSPhotoLibraryUsageDescription in plist

@delba  please check this ?

Awesome! I'll find some time to review it during the week, thank you @dimohamdy :)

@delba  waiting your review and i will update it with the newest xcode version

@delba  any updates 😄

Awesome work @dimohamdy, thank you very much!!

Please correct me if I'm wrong: MediaLibrary is available since 9.3, right?
Also, for which authorizations do we have to add an usage description key in the Info.plist ?

@delba  it's my fault MediaLibrary from iOS 3 I will fix it and check callbackAsync and callback

No problem, I fixed it :)


Quality of life changes

adding navigation icon to config
fix array args of commands, at the moment throws error, change Select field to TagsInput its make more sense
add "Run action" label in modal to lang file
ability to disable developer_gate through config

hi @izica thanks for your PR, good job 👏


Relaxed density matrix for TDDFT excited states

Hi - is there a way currently to obtain the relaxed density matrix and/or Z matrix for TD-DFT excited states? I think I've found the relevant lines for obtaining Z from the TDRKS gradients (https://github.com/pyscf/pyscf/blob/master/pyscf/grad/tdrks.py lines 116-127) but I was wondering if there's a more straightforward way to obtain the relaxed density matrix. Thanks!

I'm not sure whether there is a widely accepted definition of "relaxed density matrix" in the post-HF methods. Therefore, I never implement an API to provide such density matrices, although you might find some functions in the package called respons_density_matrix or relaxed_density_matrix. If the Z vector is considered as the occ-virt block of the relaxed density matrix, as you found in the TDRKS implementation, you can extract the dm1 to define your density matrix from https://github.com/pyscf/pyscf/blob/df093594f27af8cebe546f08a38277f3e40bd63f/pyscf/grad/tdrks.py#L142 .