Choose encoding when export csv

Is there any plan to add this feature that I can choose encoding when export csv?
Because Excel uses the ansi to open the csv files, Chinese is mess code.

We should just probably export the csv encoded in utf-8. Isn't this the case? If so and excel cannot handle it, that's an excel issue. Btw what do you mean with ansi?

You're right, it's an excel issue. But the truth is, people not familiar with encoding will always use excel to open csv. So I think it's better to let them choose encoding(gbk) which I tell them to when exporting csv.

You can add utf8 file header manually, to tell software decode it with utf8
header = bytes(b'\xEF\xBB\xBF').decode('utf-8')
return Response(
header+payload,
status=200,
headers=generate_download_headers("csv"),
mimetype="application/csv")

PR welcomed!

We are having issues with csv default encoding (and default separator as well) too.
Would you be interested in a PR that allows to choose csv encoding and separator using the config.py ?
We were thinking about adding two variables in config.py (CSV_SEP and CSV_ENCODING) to be able to change the default way of writing CSV (values would be retrieved when calling to_csv method).
Does it sound right to you ?

I tried to adding the CSV_ENCODING in config.py like #3441, In Python3.5 it doesn't work，however in Python2.7 it works well.
It seems that Pandas IO tool "to_csv" ignores encoding argument in Python3.

Yes I noticed this too ... I am planning on issuing a new PR to fix this using flask Response encoding.

I modify config.py:
CSV_EXPORT = {
'encoding': 'gbk',
}
But it doesn't work.Using vim to view the downloaded file:
fileencoding=utf-8
Do I have an operation in error?

now if you solve this problem? i am in this trouble now. @chengwei1231

in /views/core.py, i change Response to  CsvResponse.
eg:
response = CsvResponse(csv, mimetype='text/csv')

@chuancyzhang Could you explain more detail? you mean def generate_json ??

For those who are still suffering from csv export encoding matters,
Try editting /views/core.py
Look for the term
'Responce(csv,'
You'll find it under def csv.
Add 'Csv' in front of the term to show like
'CsvResponse(csv,'
Save it and add below lines to your Superset_config.py:
CSV_EXPORT = {
'encoding': 'utf_8_sig',
}
The encoding should be changed accodingly to your requirement.
Then run superset init and restart.
Now you should be able to download properly encoded csvs.

It was useful for me to change default delimiter to ;
    CSV_EXPORT = {
      "encoding": "utf_8_sig",
      "sep": ";"
    }

It allows to import CSV to MS Excel without any additional actions

It was useful for me to change default delimiter to ;
    CSV_EXPORT = {
      "encoding": "utf_8_sig",
      "sep": ";"
    }

It allows to import CSV to MS Excel without any additional actions

Encoding option was really useful, thanks!


Standardize engine artifact names.

The engine artifacts are not following a naming convention forcing logic to generate the names differently according to the platforms. Changing the names require a complicated testing process and coordinating with changes to the flutter tool.
This change will happen in multiple steps:

Generate artifacts with gn and ninja using the same artifact names.
Validate the artifacts generated with gn and ninja with the framework tests and the flutter tool.
Make the gn generated artifacts the default ones
Standardize names.

Flutter tool is expecting sky_engine for android debug to be in the root folder of GCS artifacts but with GN rules is being generated in out/android_debug/zip_archives/android-arm/sky_engine.zip to place this file in the location expected by flutter tool we need an if clause to remove android-arm from the path if the config is android_debug.

shell/platform/android:gen_snapshot needs to be deprecated once engine_v2 migration is complete and shell/platform/android:new_gen_snapshot renamed to shell/platform/android:gen_snapshot

From triage: any updates on this?

This is work in progress: flutter.dev/go/gn-ninja-engine-artifacts

Closing as obsolete.

There's still a TODO on this in //sky/dist/BUILD.gn - can that be removed?

And in https://github.com/flutter/engine/blob/1dd75f5ac749698774e082d161ad12439ca528cd/build/archives/BUILD.gn#L38C63-L38C85 :)

Also (can be later bug), I notice that Win and Mac support "archive_gen_snapshot" target, but Linux does not (linux presumably does the same under another name).

I also many of these TODOs in https://github.com/flutter/engine/blob/12122eac2820d28a36c8f5f8c22a0efa6365832d/shell/platform/android/BUILD.gn, I'm re-opening this issue but @godofredoc feel free to file a separate one if that's your preference - this should really get done though.

I won't be able to work on this issue in the next few weeks. Removing myself from the assignees in case someone has spare cycles to work on it.


Add 7579 tutorial

@louis-md  @tanay1337

Since permissionless can only be used to deploy single signer Safes, can we callout in the tutorial that multisig 7579 is coming soon?
Question: How is a module installed if it is not yet installed? Is there a UI for that? Or is a module installed "counterfactually" ?
The link in the tutorial: awesome 7579 modules might be outdated. Can we rather use this erc7579 modules: https://erc7579.com/modules

Further proposed todos:

[ ] Add to resource hub
[ ] Add to erc7579.com
[ ] Add to bounty page


[5.4] Fix bug of Eloquent\Builder::implode() returns the builder self

Bug
SomeModel::implode('the_field');                        // returns a instance of Eloquent\Builder
$modelObject->someRelationship()->implode('the_field'); // returns a instance of Eloquent\Builder
(new Eloquent\Builder)->implode('the_field');           // returns a instance of relation class

Fixing
Add "implode" to Eloquent\Builder::$passthru

Model::getQuery()->implode('the_field') should work.

So please make SomeModel::implode('the_field') throws Method Not Found Exception, or the bug always be a bug


SOCIAL_AUTH_LINKEDIN_FIELD_OAUTH2_SELECTORS Not being used to populate user creation backend

I have set the SOCIAL_AUTH_LINKEDIN_FIELD_OAUTH2_SELECTORS field in my Django Settings. I know that it is being used becuase when I remove the field I get an error when accessing the backend. However when I use it and include the selectors I want it is not being merged into the default selectors listed.
The attached screenshot results in only the default selectors being printed to the screen:

That's because the setting name is SOCIAL_AUTH_LINKEDIN_OAUTH2_FIELD_SELECTORS.


Don't access fields of composite types directly

As Toivo argued recently, we shouldn't encourage users to access the fields of composite types directly. In particular, we shouldn't tell people to work with df.columns directly.
Instead, this should be part of the effective metadata for a DataFrame:

columns(df)
names(df)
types(df)
isnullable(df)

In particular, this is going to be important if we start allowing users to access columns as df.colName since one could have a column called columns and we can't let ambiguity slip into our interface.

Implemented in #729.


docs[vi]: add metadata, code generator, handle event & business logic

Codecov Report

Merging #70 (992e2dd) into main (309998e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #70   +/-   ##
=========================================
  Coverage     95.00%   95.00%
  Complexity      445      445
=========================================
  Files            96       96
  Lines          1723     1723
=========================================
  Hits           1637     1637
  Misses           86       86

Flag
Coverage Δ

phpunit
95.00% <ø> (ø)

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

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 982faf8...992e2dd. Read the comment docs.


Fix issue with dangling pointer when calling utime

The temporary string object returned by the function call is
deallocated at the end of the expression. Since c_str() returns a
pointer to the internal string representation, the data is no longer
valid once the string object is destroyed. Due to this utime returns
ENOENT and Grabber prints an error in the log when modifying
file creation time.
Fixes https://github.com/Bionus/imgbrd-grabber/issues/2203
I don't think this issue affects Windows, since toWcharT is directly called on path variable. Maybe someone can check in Windows.

Also should the tests for SetFileCreationDate and SetFileCreationDataUtf8 be enabled for unix?

LGTM, thanks!
As for the tests, I guess they can be enabled if they pass. I guess they were disabled for this very reason, but it was so long ago I don't really remember.


Roles cannot be assigned for access control when creating an API using REST API

Description:
Some roles cannot be assigned for access control when creating an API using REST API if the number of roles exceeds "MaxRoleNameListLength" in user-mgt.xml. Actually this issue is caused due to the roles which are created after exceeding the "MaxRoleNameListLength".
Steps to reproduce:

Create 6 additional roles using the management console (/carbon).
Set the "MaxRoleNameListLength" in user-mgt.xml to some lower value like 5.
Restart the server.
List the roles in carbon console. Then you will able to see the role you have added as sixth is not shown unless you explicitly search for it.
Create/Update an API using Publisher API (REST API) with the following values.

"accessControl":"RESTRICTED", "accessControlRoles":[ <6th role you added>]

In the meantime, try to create the same API using the publisher portal. You can see that it will be created without any issue.

Solution:
Bring the same implementation used in the publisher portal to REST API. Then it will no longer depend on the "MaxRoleNameListLength" parameter.
Affected Versions:
2.6
Suggested Labels:
Bug

Publisher API implementation:
https://github.com/wso2/carbon-apimgt/blob/v6.4.50/components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/utils/APIUtil.java#L4152-L4171
Publisher Portal Implemetation:
https://github.com/wso2/carbon-apimgt/blob/5ca239b360cb14357c717eedeb5da2713d11638b/components/apimgt/org.wso2.carbon.apimgt.hostobjects/src/main/java/org/wso2/carbon/apimgt/hostobjects/APIProviderHostObject.java#L4692


Define e2e execution logic in ols repo

Description
Defines a script that generates a config file for OLS, starts the server, and
then runs our e2e tests against the OLS server.

Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

@tisnik ptal
once this merges i'll update our current CI e2e job to invoke this script.

/lgtm


JSON update while using transform

I am having a problem in seeing updates in canvas_result.json_data while using transform on rect:
As an example, I am adding a few lines to app.py
If I add a few simple lines the top and left updates while using transform but somehow not width and height while scaling a rect
(this is the last part of app.py)
# Do something interesting with the image data and paths
if canvas_result.image_data is not None:
    st.image(canvas_result.image_data)
if canvas_result.json_data is not None:
    st.dataframe(pd.json_normalize(canvas_result.json_data["objects"]))
    # Just adding a json view of the first rect.
    dic = canvas_result.json_data["objects"][0]
    st.text('Top value in json =' + str(dic['top']))
    st.text('Left value in json =' + str(dic['left']))
    st.text('Height value in json =' + str(dic['height']))
    st.text('Width value in json =' + str(dic['width']))

Hello @ahexemer,
Under the hood, json_data is a FabricJS object exported in JSON with .toObject.
It seems for a scaled object, you will need to apply the scaleX/scaleY factor to the height/width to retrieve the correct values. FabricJS preserves the original dimensions and applies a ratio independently.
st.text(f"Scaled height: {dic['height'] * dic['scaleY']}")
st.text(f"Scaled height: {dic['width'] * dic['scaleX']}")

doesn't seem like rotating the rectangle adds more complexity :) .
Best,
Fanilo

Hello @ahexemer,
Under the hood, json_data is a FabricJS object exported in JSON with .toObject.
It seems for a scaled object, you will need to apply the scaleX/scaleY factor to the height/width to retrieve the correct values. FabricJS preserves the original dimensions and applies a ratio independently.
st.text(f"Scaled height: {dic['height'] * dic['scaleY']}")
st.text(f"Scaled height: {dic['width'] * dic['scaleX']}")

doesn't seem like rotating the rectangle adds more complexity :) .
Best,
Fanilo

Hi Fanilo,
Thanks for the reply. Yes, in the actual code I scale the value you are correct. I was more wondering about the updating of the width and height values in the JSON while applying a "transform" to a rectangular.
Cheers,
Alex

Hi Fanilo,
Thanks for the reply. Yes, in the actual code I scale the value you are correct. I was more wondering about the updating of the width and height values in the JSON while applying a "transform" to a rectangular.
Cheers,
Alex

Yup, unfortunately that's not on my side :) I directly export the FabricJS object to JSON and I don't mess up with it because I'm currently working on reloading canvas from this JSON state, so I guess if I edit the width/height in the JSON i may mess up future Fabric.js reloading :confused:
But yeah, thanks for bringing this up! Hopefully this can help other users. Feel free to close this up if you have got your answers.
Cheers,
Fanilo

Yup, unfortunately that's not on my side :) I directly export the FabricJS object to JSON and I don't mess up with it because I'm currently working on reloading canvas from this JSON state, so I guess if I edit the width/height in the JSON i may mess up future Fabric.js reloading :confused:
But yeah, thanks for bringing this up! Hopefully this can help other users. Feel free to close this up if you have got your answers.
Cheers,
Fanilo


Support #[kv(collection, flatten([prefix = "..."])]

Currently when trying to do something similar to:
use kv_derive::{prelude::*, IntoVec};

#[derive(IntoVec)]
pub struct Complex {
    x: String,
}

#[derive(IntoVec)]
pub struct Test {
    #[kv(flatten(prefix = "xs::"), collection)]
    xs: Vec<Complex>,
}

the code generation creates non-compiling code due to the requirement of CollectionProducer<P> to have P be Copy and Clone, but FlatteningProducer and PrefixedFlatteningProducer do not derive Copy and Clone. As these clones are trivial, this MR adds them.

That's correct @Wassasin, I seem to have overlooked it. Thank you!


Laravel API throttle issue with Sanctum on Vapor.

Laravel Version: 7.11.0
PHP Version: 7.4
Database Driver & Version: MySQL 5.8

Description:
I deployed a basic Laravel site with a "flash sale" functionality. Forntend is VueJS SPA and used Laravel Sanctum for auth. It is serverless using Vapor.
The site is no fancy just a basic form to enter into the sale (So just one DB+API interaction) and one API interaction to get the products.
The product expected to have 30k visits in 30min (But we ended up having 590k page views and 200k in 30min). Huge but thanks to Vapor/Lambda it went smooth.
Now the thing is I used the default throttle of 60:1 for the API considering we are not going to hit that for any user. But the thing is as soon as we started having a good number of traffic (not the full traffic but possibly 50-60 concurrent) we started facing 429 Too Many Requests issue. The fun part is we were facing it only when not logged in.
I have been developing in Laravel for more than 5 years and made dozens of projects in Laravel and never faced this issue. The only things that were new: Sanctum and Vapor. Though after lots of dives into the source code didn't get anything that could be causing this issue. Anyone here faced this here? Am I taking the throttle flow as wrong (i.e it is based on per user per URL basis)?
Steps To Reproduce:
I don't know if this is the correct way to reproduce it but still will try: a basic Laravel project with an API call and auth using Sanctum deployed via Laravel Vapor.
I don't know how to reproduce that real time concurrency as Apache benchmark won't help there.

I don't know if this is clear from the above but it was not just users but myself faced 429 Error even when I just reloaded the page once that was making only 1 API call.

When not authenticated, Laravel will throttle by domain and IP address. However, this can lock out a lot of users because many people may share the same IP address.

I can assure that when I was browsing it in not logged in state it was only
me. No one was browsing at same time from my IP.
On Sat, 30 May 2020 at 10:35 PM, Taylor Otwell<EMAIL_ADDRESS>wrote:

Closed #33023 https://github.com/laravel/framework/issues/33023.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/issues/33023#event-3389898540, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB5RCZWNABCKRJY5LW77IATRUE4GRANCNFSM4NOSZGSQ
.

--
Regards,
Abhishek
+91-8447999488

@taylorotwell I tried adding some logs. If I am not mistaken this code is used to generate the signature in ThrottleRequests Middleware:
protected function resolveRequestSignature($request)
{
    if ($user = $request->user()) {
        return sha1($user->getAuthIdentifier());
    }
    if ($route = $request->route()) {
        return sha1($route->getDomain().'|'.$request->ip());
    }
    throw new RuntimeException('Unable to generate the request signature. Route unavailable.');
}

I tried logging the $request->route()->getDomain() and $request->ip()
Here's the output in my local:

Here's from the Vapor panel on live site:

Isn't it looks like domain and ip both are null and hence signature is same for all not logged in users?
Let me know if I should furnish some more details.

Another screenshot of log of same line but on a forge deployed server.


Cancel subscription

Super helpful article, Suragch.
Should we cancel all those stream subscriptions in the dispose method of the PageManager?

I'm finally got around to updating the article and looking at old issues. Sorry for the delay.
That's a good question. I'm kind of assuming that calling _audioHandler.stop() and _player.dispose() takes care of those streams, but I don't know that for certain.


Remove of use_exclude_ctrl_bones (was removed in io_scene_dae by Serg…

…ey Lapin, commit 4210d5e4594fcf80bc121bccc916d411663d02bc) in godot_export_manager.py

Looks fine.


JEI Plugin is borked, and so is the creative tab

While trying to diagnose why my game is stalling when entering a world, I found this in the logs.
https://pastebin.com/Vb8LQ6N2
Looks like a weird cross-compat issue with copygirl's backpacks mod.
Edit: It appears this mod might be causing my stalling issue.
https://pastebin.com/VZ1qUk1T

Mod works fine with the hotfix. Closing.


update: install script

Summary
Update config/install.js to use the proper default values of the .env.example  when creating the .env file:

Changes: MONGO_URI port from 27017 to 27018
Remove: Question about GPT4 since fetching the models is now the proper default behavior

Change Type

[x] Bug fix (non-breaking change which fixes an issue)

Testing
I ran the script to ensure that the resulting .env was in line with the .env.example
Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] My changes do not introduce new warnings

The only issue with this is that mongo default port is 27017, the project is only configured for 27018 for docker, since this allows a local service to be accessible along with docker compose

@danny-avila I'm perfectly aware of that, this only sets it to 27018 in the .env, like in it is currently set in the .env.example. This allows access to the mongo container (when its ports are open) from outside docker.
Example: sometimes when I want to test something, I only shut down the librechat container and fire up the local install, which access the same db (docker) through the port 27018. Having it set to 27017 in the .env does not give that possibility.

I'm perfectly aware of that, this only sets it to 27018 in the .env, like in it is currently set in the .env.example. This allows to access the mongo container (when its ports are open) from outside docker.
Example: sometimes when I want to test something, I only shut down the librechat container and fire up the local install, which access the same db (docker) through the port 27018

the environment variable for MONGO_URI in the compose file overrides what is set in the .env file

But, it's not a bug, it's actually a feature! We changed the port in the .env.example 2 months ago to 27018 because users were confused as why they couldn't externally access the docker mongodb from 27017.
Setting it to 27107 doesn't really add any value. In fact, it just adds confusion for users who want to access the docker mongodb from outside docker since it's not the default "externally accessible port".

Having it set to 27018 has the benefit of showing clearly the default "externally accessible port" (when exposed) in our default setup
27018 also allows to access our docker db from the default in the .env.example

Right now, you can start a fresh LibreChat installation in just four straightforward commands without any further input:
cp .env.example .env
npm ci
npm run frontend
npm run backend
Reverting it back to the previous port definitely breaks what I consider a valuable feature and doesn't provide any useful enhancements to the project.

the environment variable for MONGO_URI in the compose file overrides what is set in the .env file

But, it's not a bug, it's actually a feature! We changed the port in the .env.example 2 months ago to 27018 because users were confused as why they couldn't externally access the docker mongodb from 27017.
Setting it to 27017 doesn't really add any value. In fact, it just adds confusion for users who want to access the docker mongodb from outside docker since it's not the default "externally accessible port".
* Having it set to `27018` has the benefit of showing clearly the default "externally accessible port" (when exposed) in our default setup

* `27018` also allows to access our docker db from the default in the .env.example

Right now, you can start a fresh LibreChat installation in just four straightforward commands without any further input: cp .env.example .env npm ci npm run frontend npm run backend
Reverting it back to the previous port definitely breaks what I consider a valuable feature and doesn't provide any useful enhancements to the project.

Ok, it just doesn't match my own use case but I can see the value behind it


fix: init project, bump version

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


I want to change the background color for Saturday and Sunday.

I would like to change the background color of Saturday and Sunday to blue or red when viewMode is Day. Is it possible?
I would like to change it as shown in the image below.

This request seemed similar.
I would be happy if it would be possible to reflect Saturdays and holidays if possible.
https://github.com/MaTeMaTuK/gantt-task-react/pull/122


"istioctl upgrade" requires the same settings used to install Istio, but doesn't take --set parameter

What I did:
Deleted istio-system namespace
./istio-1.4.5/bin/istioctl manifest apply --set profile=demo --set values.global.mtls.enabled=true
Waited a few minutes
./istio-1.5.0-beta.5/bin/istioctl x upgrade
./istio-1.5.0-beta.5/bin/istioctl version
client version: 1.5.0-beta.5
egressgateway version: 1.4.5
ingressgateway version: 1.5.0-beta.5
pilot version: 1.5.0-beta.5
data plane version: 1.5.0-beta.5 (2 proxies), 1.4.5 (1 proxies)

The egressgateway should be at the same version as the ingressgateway.

You need to pass the same settings to upgrade

@howardjohn I can't pass the settings to istioctl upgrade
./istio-1.5.0-beta.5/bin/istioctl upgrade --set profile=demo --set values.global.mtls.enabled=true
Error: unknown flag: --set

I don't know why but you can only use -f
On Tue, Mar 3, 2020 at 12:25 PM Ed Snible<EMAIL_ADDRESS>wrote:

@howardjohn https://github.com/howardjohn I can't pass the settings to istioctl
upgrade
./istio-1.5.0-beta.5/bin/istioctl upgrade --set profile=demo --set values.global.mtls.enabled=true
Error: unknown flag: --set
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/21778?email_source=notifications&email_token=AAEYGXJY55UF24V4LT3EWADRFVRURA5CNFSM4LARBB4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENVANMQ#issuecomment-594151090,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXJALKWF2IXVNNDEYKLRFVRURANCNFSM4LARBB4A
.

@howardjohn I feel the syntax SHOULD be the same, so I have added the enhancement tag to the Issue.

I think we should update istioctl upgrade to take also the --set command, /CC @elfinhe

Based on the feedback I read from this discussion, I think we need to add --set to istioctl upgrade.

@richardwxn I have made this an Enhancement and set the milestone for 1.6.  Let's track this for 1.6.

why upgrade can't  automatically recognize the current configuration to update :(

@maplebeats I also believe Istio should recognize the current configuration.  Please add a thumbs-up or celebration vote to https://github.com/istio/istio/issues/20000 , or comment there.

Verified this works in Master.


Greengrass component name case insensitivity

Describe the bug
Greengrass fails to save artifacts into packages directory when there is an existing component with the same, case-different name
To Reproduce
Deploy a component called Component
Deploy a second component called component that depends on an artifact (in our case, a config file pulled from s3)
Expected behavior
component to successfully retrieve S3 and save it, OR an error to be raised about the name collision
Actual behavior
Silent failure. The component comes up sometimes, and fails other times (with missing config file error). I think it succeeds on initial come-up, but fails to restart thereafter (but cannot confirm that). I can can confirm that no component folder is made in packages/artifacts, and the packages/artifacts/Component folder still exists but is empty.
deleting the packages/artifacts/Component folder and creating packages/artifacts/component fixed the issue
`
Environment

OS: macOS 11.4
JDK version: openjdk 17.0.2 2022-01-18
Nucleus version: 2.5.3

Additional context
mac
E.g. what is the impact of the bug?

Hi Travis,
Thank you for submitting your issue. Greengrass's configuration is entirely case insensitive, which includes component names. Therefore, your component names must be unique when compared in a case insensitive way. Additionally, MacOS's file system is case insensitive by default which will also cause confusion.

Thanks for the reply @MikeDombo.

your component names must be unique when compared in a case insensitive way

If thats the expectation, there seems to be a discrepancy between the Greengrass UI in AWS Console (which is case-sensitive and will show component and Component as two different components) and greengrass nucleus, which will treat them as the same

Thanks again for bringing this up. I'm passing it along to our cloud team to see if they can/will/want to enforce uniqueness case insensitively.

this would seem to be incredibly confusing to have meaningfully different components with the same name and only different capitalization

totally agree. :) This is not a use case that would be valuable, more just a case where catching the issue early and raising an error would be really helpful. We did not realize we had two components with the same name and different case (two engineers were working in parallel) and it took a while to connect our missing artifacts issue to the difference in case

Thanks once again for the feedback. I have additionally reached out to have our documentation updated to reflect this.
As I mentioned earlier, our cloud team will also be looking into enforcing this from the cloud side.
Closing this issue now as a wontfix in the Nucleus. Feel free to comment and reopen if necessary.


Max number of clipboards

A new option can be added to define a maximum number of clipboards (like in xfce4-clipman).

Thanks for your suggestion!
Well, I think there is already a "Storage" option to specify the days of clips to save. Two options which have similar usage may bother some users.


Bintray is deprecated!

The api now returns a 403 so we cant use conan with the exists urls ("https://api.bintray.com/conan/includeos/includeos") So what now?

This should be the new place https://center.conan.io

(https://conan.io/center/)

I may of got it working by updating homebrew but not tried yet

updating homebrew didn't work

Thanks, we are removing the dependency on conan. I still want to support conan files, but currently they're all broken. We are in the process of getting a nix build up and running.


Auto correction does not work for all keyboards

When typing in a message the badge for autocorrection does not get showed.
Although when typing a search string the badge gets showed
RW mentions here and here that the same behaviour exists on AnySoftKeyboard as well
My Environment
Android 5.1 with default keyboard (Moto G)
Here a Link to the depending chat in riot-android
Screenshots

WhatsApp how it is expected in Riot Android as well

Riot Android - how it get showed to me

So the question is, what is inputType in Riot in the first place? And does it make sense to explicitly set inputType=textAutoCorrect?
What does the Android dev guide say on it?

It has been fixed on the latest develop.


springboot+lettuce occurs a OutOfMemory Error

Bug Report
My project is using springboot 2.0.9.RELEASE, it connects to redis cluster by lettuce which is provided by spring-boot-starter-data-redis. it occured a OutOfMemoryError this morning, but I cannot reproduce it.

Current Behavior

Stack trace
// your stack trace here;

I dump the heap file, and open it with Eclise Memory Analyzer, this is reported by Eclipse Memory Analyzer.

and this is the configurations about redis:

Input Code

Input Code
// your code here;

Expected behavior/code

Environment

Lettuce version(s): [e.g. 5.0.0.RELEASE, 4.2.2.Final]
Redis version: [e.g. 4.0.9]

Possible Solution

Additional context

If you would like us to spend some time helping you to diagnose the problem, please spend some time describing it and, ideally, providing a minimal sample that reproduces the problem.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Unable to see email (SMTP) notification

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Hi There,
Greetings for the Day,

I configured uptime kuma using docker image and i got web UI also,
Next i configure my application url, so that i will get notification when status of my application changes.
I am able to get notification when i added notification for a single user like<EMAIL_ADDRESS>(this is my email addrss)
When i added group email address( there are 5 users ,email ID are available in this group mailid/alias<EMAIL_ADDRESS>and click on test, i got a pop sent successfully, but i am unable to any notifications/email in my email box.
Can you guide how to setup this.

📝 Error Message(s) or Log
I checked the logs, i am not getting anything.
🐻 Uptime-Kuma Version
YARN_VERSION=1.22.19   NODE_VERSION=16.20.2
💻 Operating System and Arch
ubuntu 20.04
🌐 Browser
Google chrome: Version 116.0.5845.187
🐋 Docker Version
Docker version 20.10.21
🟩 NodeJS Version
No response

Given that the first mail worked, I strongly susspect your mailinglist to be at fault.
Some hints:

Is the message maybe held for moderation?
Have you double checked that you are subscribed to the list
what do the logs of the mailinglist say


PICS ASSIGNMENT 1 TASK 4: added and modified questionnaire

Added an appropriate PROM Questionnaire
:recycle: Current situation & Problem
The current app contains a questionnaire that is not relevant
:gear: Release Notes
Added a new json file for the new PROM questionnaire and integrated the new questionnaire into the app
:books: Documentation
Please ensure that you properly document any additions in conformance to Spezi Documentation Guide.
Referred to the SpeziQuesionnaire documentation (https://swiftpackageindex.com/StanfordSpezi/SpeziQuestionnaire/documentation/speziquestionnaire) and (https://github.com/StanfordSpezi/SpeziQuestionnaire) for more details on how questionnaires could be displayed.
(https://swiftpackageindex.com/StanfordSpezi/SpeziScheduler/documentation/spezischeduler) and (https://github.com/StanfordSpezi/SpeziScheduler) for more details on how repeated tasks can be easily scheduled through SpeziScheduler.
:white_check_mark: Testing
No automated testing yet
:pencil: Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

Nice job with the updates @karansoin!


Add Configured[FiniteDuration]

This is one possible resolution to #33. I've gone this route for a
couple reasons:

It's not a breaking change like making CfgDuration wrap a FiniteDuration would be.
I don't think that there's a theoretical reason that an infinite duration shouldn't be allowed, even though there's currently not support for it in the parser.

@ceedubs looks good to me. Can you bump the feature version?


Get About box info dynamically

Right now it's hardcoded but what should happen is that:

The app name and version should come from the app config
The engine version comes from a ping to the /alive WS endpoint although this now returns no version e.g. <alive authentication="false" localfs="true" version="???"/> cc @bertfrees is there supposed to be version info returned?

Yes

@marisademeglio The version information is currently stored in a releaseDescriptor.xml file inside the etc folder of the Pipeline engine installation. Currently, in order to find it the org.daisy.pipeline.home property (or PIPELINE2_HOME env variable) should be set.

I'd rather get the version info from the /alive endpoint, access to pipeline config files is not guaranteed since we might have a remote pipeline instance someday. I don't mind hardcoding the version info for now.

/alive gets the information from the releaseDescriptor.xml file, but in order to find it it needs the org.daisy.pipeline.home property.


Introduce a KeyStoreManager to abstract management of root and non-root key storage

Do not merge, PR for review only.
This structure encapsulates what used to be "rootKeyStore" and
"privKeyStore". These are being moved out of NotaryRepository, so that
operations like listing keys, importing keys, and exporting keys aren't
tied to a NotaryRepository structure.
Initial draft for review. Will add import/export functionality next if
this looks good. Do not merge yet.
Signed-off-by: Aaron Lehmann<EMAIL_ADDRESS>

I shouldn't have opened this against master. There are other changes included that make the PR way too large. Closing, and will reopen against the branch I worked off.

Replaced by https://github.com/docker/notary/pull/55


Courser won't cover the whole screen

I am using the latest version of controllable (1.15.2) and I am using a wired Xbox 1S controller, but my courser will not move away from a quarter of the screen.

I've got the same issue with a PS4 controller.

semi solution: disabled Virtual Mouse in settings.

Are you using Optifine? If so, uninstall. Optifine is not 100% compatible with Forge.

I've got the same issue with macOS 10.15.4, wired Xbox One controller and Minecraft 1.15.2. The cursor UI only covers the top left quarter of the screen. The cursor itself seems to cover the entire screen but it looks like there is a mismatch. Screenshot: https://pasteboard.co/J985Vi9.png

semi solution: disabled Virtual Mouse in settings.

I tried this but the cursor will go back to the last spot it was when there's a new input with the joystick.

Closing this issue due to it being stale. Reopen a new issue if this is still a problem in the latest version.

Closing this issue due to it being stale. Reopen a new issue if this is still a problem in the latest version.


AttributeError: module 'smtplib' has no attribute 'SMTP_SSL'

Traceback (most recent call last):
File "", line 1, in
File "/usr/local/python3.6/lib/python3.6/site-packages/zmail/server.py", line 54, in send_mail
server.send_ssl(recipients, message, timeout)
File "/usr/local/python3.6/lib/python3.6/site-packages/zmail/server.py", line 169, in send_ssl
with smtplib.SMTP_SSL(self.host, self.port, local, timeout=timeout) as server:
AttributeError: module 'smtplib' has no attribute 'SMTP_SSL'

Did you solve this proble? I have meeting the same proble!

I got the same issue in windows. But it works fine on linux OS.


Trip widget improvements

Is your feature request related to a problem? Please describe.
Make the widgets on the trips graph show 25/75 10/90 percentiles rather than min/max.

Suggested by Chris Friend:
https://transitmatters.slack.com/archives/GSJ6F35DW/p1693954520220809
Describe the solution you'd like
Let me (@PatrickCleary) know if you are working on this. I have design thoughts. Some of which are in the Slack thread linked above.
Additional context

Does #864 resolve this or is there anything left we want to do before closing?


Async highlighting is broken in 8.1

Async highlight is not working after upgrade to 8.0 (or 8.1) for methods
    public async asyncResponse(): Promise<Object> {
        return {
            promiseResult: 'promise is ok'
        }
    }

also, class keyword now looks like storage type ts in this example:
export class IndexController extends AppController {
...

Please create an issue at https://github.com/Microsoft/TypeScript-TmLanguage/issues and link back here :rose:

I've added grammar docs https://github.com/TypeStrong/atom-typescript/blob/master/docs/grammar.md#cson-grammar to help everyone know what's going on :rose:


Single run

Feature as requested and specified in https://github.com/khenderick/zfs-snap-manager/issues/32

Thanks! Can you resolve the conflict?

Sure, can you advice? Guess I would need write access for that:

This should be possible by merging khenderick:master back to mmeidlinger:single_run

I'm closing this issue as I unfortunately don't have the time anymore to maintain ZFS Snapshot Manager. I would like to point you towards the awesome tool zrepl which has nearly all of ZFS Snapshot Manager's features and much more.
In any case, thanks for being a part of this community.


debug terminal infinite length

also fixed in https://github.com/Skewax/skewax/commit/714c5e53e16d6ddbb9e36e31cc9c4d13fe671cbb just forgot to mention


[fix] don't append CSS files to the end of html_static_path list

html_static_path is a list of paths that contain custom static files.  They are
copied to the output’s _static directory after the theme’s static files, so a
file named default.css will overwrite the theme’s default.css [1]
Without this patch a tabs.css can't be overwritten by the conf.py file:
html_static_path = [ 'static/tabs.css', ]

The /static folder from sphinx-tabs needs to be added in front of
html_static_path since the last item in the list will be written last to
/_static.
[1] https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-html_static_path
Signed-off-by: Markus Heiser<EMAIL_ADDRESS>

Thanks for this, I hadn't considered that someone would want to overwrite the CSS file(s) for a plugin!
Please could you add a line to our Sphinx docs to describe how a user would do this in their sphinx conf?

Please could you add a line to our Sphinx docs to describe how a user would do this in their sphinx conf?

IMO this is not needed, it is the common solution to overwrite static files (images, CSS, JS ..) and is described in the Spinx framework, see html_static_path. By example: this was the first I tried to implement a workaround for #152.


feature: forward egress traffic to external gateway

Gateway node(s) forwards egress traffic to external gateway based on policy routing. Supports both distributed gateway and centralized gateway modes. New properties are available in subnet spec: externalGateway, policyRoutingPriority & policyRoutingTableID.
Example:
apiVersion: kubeovn.io/v1
kind: Subnet
metadata:
  name: subnet1
spec:
  cidrBlock: <IP_ADDRESS>/24
  default: false
  gateway: <IP_ADDRESS>
  gatewayType: centralized
  gatewayNode: kube-ovn-worker
  natOutgoing: false
  private: false
  externalGateway: <IP_ADDRESS>
  policyRoutingPriority: 2000
  policyRoutingTableID: 2000
  namespaces:
    - ns1

@oilbeater please review


[stable/mongodb] Fix missing labels

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

/ok-to-test


Insane Korean translation update

I added one translation fix that were missing in the last translation update.

@eidals224 Thank you!


Remove mkdirp.

As per https://github.com/asfktz/autodll-webpack-plugin/pull/26#issuecomment-353509744.

Merged 🎉! Thank you @wtgtybhertgeghgtwtg for your contribution!


Missing transform Vector3 operator for Pose3

Hey there,
is there a reason why there is no Vector3 operator*(Vector3) implemented for Pose3? Wouldn't it be rather handy for transforming points between different link/model/joint frames?
Cheers,
Stefan

We've had some trouble with Pose3's operator*(Pose3) in the past and had to change its definition, see #60. That's to say that the meaning of these operators can get fuzzy. I'm personally in favor of just using functions that are well-defined rather than operators that can be misinterpreted.
Unless it's for a class that has a well-established meaning in mathematics, like Matrix4:
https://github.com/ignitionrobotics/ign-math/blob/6a3c4a32b18e6bd7956fecfe53ccfe14bfcd2907/include/ignition/math/Matrix4.hh#L673-L685

@stefanbuettner , after the comment above, do you still think the operator is needed?

Either the operator or a function which let's me transform a vector from the pose's frame to the pose's parent frame.
I would have expected the operator to be equivalent to
Vector3 Pose3::operator*(Vector3 const& p) {
    return Matrix4(*this) * p;
}

I find the addition and subtraction operators the confusing part. To me the Pose3 represents a transformation from one frame of reference to another frame of reference. Thus I want to concatenate poses (i.e. nesting reference frames) and transform points from one frame of reference to another.
This can also be described by 4x4 matrices. Although I would make it explicit via the Pose3 class.
Pose3d const table; // w.r.t the world
Pose3d const mug; // w.r.t the table
Vector3d const precomputed_grasp_point; // w.r.t. the mug

// I like both
Vector3d grasp_point = table * mug * precomputed_grasp_point;
Vector3d grasp_point = (table * mug).Transform(precomputed_grasp_point);

// Weird
Vector3d grasp_point = table.Concatenate(mug).Transform(precomputed_grasp_point);

Although having a named function for concatenating poses would be weird. But probably just because I'm used to the multiplication operator and see matrices there.

For what it's worth: I want to second this.
Typically you'd name the transformation variables according to their references, e.g.:
Pose3d const T__world__table; // Transforms coordinates from {table} coordinate frame to {world}
Pose3d const T__table__mug; // Transforms coordinates from {mug} to {table}
Vector3d const p_precomputed_grasp_point__mug; // grasp point coordinates expressed in {mug}

Vector3d p_grasp_point__world = T__world__table * T__table__mug * p_precomputed_grasp_point__mug;

Unless it's for a class that has a well-established meaning in mathematics, like Matrix4:

Imo. it is exactly the inverse. There is no mathematical meaning to multiplying a 4x4 matrix with a 3x1 vector. Unless it is implicitly assumed that the 4x4 matrix represents a homogeneous transformation matrix, and that the 3x1 vector represents homogeneous coordinates [px, px, pz, 1].
The way it is implemented does imply this assumption. But this is contradictory with the general constructor which allows you to set all coordinates without restrictions.
But even if the 4x4 matrix represents a homogeneous transformation matrix:
Both a homogeneous transformation matrix as well as a Pose3 represent identical concepts (relative position and orientation of coordinate systems) and hence should have identical semantics.

Since there is not much of a discussion in the last 3 years and many opinions, let’s just close it. I think it’s not that important.


[WIP] Making it easy to startup a fresh nodes cluster

This PR is trying to make it easy to start a new node cluster; it aims to capture the needed steps for bootstrapping a cluster for others to deploy test environments and experiment with the project as it moves towards mainnet 2.0.

[ ] Add workflow to generate the keystore so it won't happen in the dockerfile
[ ] Add multiple instances to garden file and figure out how to configure peer discovery
[ ] Add tests section in garden file
[ ] Add workflow to generate a genesis from parameters
[ ] Add prometheus and grafana dashboards

Hey @esomore. Thank you for your contribution.
I'd like to inform you that we're not going to merge your pull request, because of following reasons:

First of all, the node deployment should be separated from the core protocol code. We're going to support many cloud providers, as well as docker/k8s. The work is there and there.
Due to security policies, we cannot make a dependency on 3rd party like mozilla/sbt:latest.
The run script you provided is obsolete and already deprecated. We are using different configurations (env variables, file config), as well as opening jmxremote is a security leak. We can do that on our managed environments but it would be a serious issue once launched on mainnet.
In the roadmap there is a plan for docker support and orchestration, but we cannot do it right now, because the code is quickly evolving. The bouncy-castle related things has been changed today, the "run" script is also in progress, and we have no time for fixing it over and over during the first phase. We will provide the scripts and documentation when that part will be ready.

I really appreciate your effort, but I need to close this pull request. If you want to contribute to the project, we can find something out that can be done at this stage of the project.

Thank you for the fast response. I will take a look on the other reps you linked.


Restore generated chainweb openapi, consolidate chainweb-node and cha…

…in-stream into two topics instead of three, update config.yaml accordingly.
Accidentally deleted the generated API doc. Restore the imported version until I can take a different stab at providing this information.

@LindaOrtega Can you take a look and approve? I didn't realize that the generated Chainweb node API got left out of the party right as we were merging the new structure PR. This PR restores the missing file and some related changes to the config file and other topics in the section.


fix: converting slice or maps of specific types (non interface)

part of nitrictech/go-sdk#44
Once this is merged, I'll use this function utlis.NewStruct() in go-sdk.

Codecov Report

Merging #253 (17a7b7d) into develop (238dd44) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop     #253      +/-   ##
===========================================
- Coverage    54.12%   54.10%   -0.03%
===========================================
  Files           45       45
  Lines         4517     4517
===========================================
- Hits          2445     2444       -1
  Misses        1922     1922
- Partials       150      151       +1

Flag
Coverage Δ

unittests
54.10% <0.00%> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/adapters/grpc/document_grpc.go
0.00% <0.00%> (ø)

pkg/adapters/grpc/queue_grpc.go
0.00% <0.00%> (ø)

pkg/plugins/gateway/lambda/lambda.go
62.03% <0.00%> (-0.54%)
:arrow_down:

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 238dd44...17a7b7d. Read the comment docs.

@asalkeld looks good!
My one concern is using this code as a direct dependency in the go-sdk, as they both contain potentially conflicting versions of our generated SDK code (the client SDK has some minor differences that make it easier to mock, but less safe to use for writing a server).
Would it be worth creating a separate lib to manage this? I can see us creating additional proto related tools and utilities in future.

Here it is: github.com/nitrictech/protoutils


Add snapshot expiry to lxc info

jenkins: test this please

jenkins: test this please

Should be good to go


Split multiple sounds in different columns

Hi,
in the courses I'm using there are always two audio files, a male and a female version. Currently, I hear them both when I add the audio column, and I couldn't find a way to play only one of the audio files.
So, it would be cool, if you could either split the audios in two columns or point me to an HOW-TO for playing only one audio file.

By default columns are imported the way they were created by the author of the memrise course. In your case I think both audio files are contained in one column even on memrise. Splitting columns is beyond the scope of this add-on, I'm afraid. But there is an article about how to split fields in the anki knowledge base:
https://anki.tenderapp.com/kb/collection-management/splitting-a-field-into-multiple-fields

Hi, thanks for the fast response - that link was helpfull enough, I can split the course with that on my own!


Add support for density compenstation estimation with cufinufft

This is crucial support for many use cases. I have some temporary things added to ensure tests still run good.

Adding @paquiteau for another short reveiew for the changes in pyproject.toml and the test-ci.yml

cufinufft 2.3.1 is here!! Ill update the PR to pull it.


cozytouch

hello, what is the server adresse to connect at cozytouch atlantic ?

Hi @mdtdamien, use the following host: ha110-1.overkiz.com
Also make sure that the cozytouch login handler is selected.
Best regards

Thank you for your return.
I have try with https://ha110-1.overkiz.com and ha110-1.overkiz.com
and in debug, i have : Failed to initialize device.

The url must be specified without „https“: ha110-1.overkiz.com
The message you are seeing comes from the state node. Probably due to incomplete device configuration. Do you see any devices in the drop down list of the state node AFTER deploying the gateway node with the correct host and credentials?

hi there , I'm trying it with Thermor (cozytouch), is that the same server address ? it is not working for me. is that possible to print more debug options ?
thanks

Hi, I'm facing the same issue, no devices are listed in the drop down list.
Host : ha110-1.overkiz.com
Login handler : Cozytouch
Device list is showing : Internal gateway error

I think I found the issue, the dependencies you are using -> overkiz-api for the CozytouchLoginHandler is using a wrong clientId.
https://github.com/bbriatte/overkiz-api/blob/master/src/cozytouch-login-handler.ts
Using postman I'm only able to get the getAccessToken working using this -> Basic Q3RfMUpWeVRtSUxYOEllZkE3YVVOQmpGblpVYToyRWNORHpfZHkzNDJVSnFvMlo3cFNKTnZVdjBh

Any update? same issue here...


Fix the issue on self-hosting Timeplus

Sorry I realized there was an issue to run this CLI on self-hosting Timeplus Enterprise. The http header need to be set properly to transform the token.
To test it, maybe you can run the Timeplus Enterprise via a docker
docker run --name tpe2.5.11 -p 8000:8000 timeplus/timeplus-enterprise:2.5.11

Then access http://localhost:8000 and create the user with name admin and password changeme
Then you can run the CLI via
./bin/dev.js start timeplus --token admin:changeme --endpoint http://localhost:8000/local --stream jovetmpd12

Hi @sdairs , would you please review this PR? Thanks


Scaffold with elevation: 0 shows white line between appBar and body on Samsung devices

Steps to Reproduce

Run this on any samsung device https://github.com/ghenry/scaffold_elevation_0_shows_white_line/

Expected results:
Fluid primaryColor top to bottom
Actual results:

If I set   extendBodyBehindAppBar: true then I see a fluid blue on the above demo app, but in our own app we have:
          bottom: true,
          color: Colors.white,
          child: SlidingUpPanel(

If I slide up the panel a tiny bit the white line goes away as the parallax comes in but adding extendBodyBehindAppBar: true doesn't get rid of it. See below:
 |  |
Only happens on Samsung. Options? I've tried beta and master channels. The same.
Thanks!

[✓] Flutter (Channel stable, 1.20.3, on Linux, locale en_GB.UTF-8)
    • Flutter version 1.20.3 at /home/ghenry/development/flutter
    • Framework revision 216dee60c0 (10 days ago), 2020-09-01 12:24:47 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /home/ghenry/Android/Sdk
    • Platform android-30, build-tools 28.0.3
    • Java binary at: /home/ghenry/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/192.6392135/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Android Studio (version 3.6)
    • Android Studio at /home/ghenry/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/192.6392135
    • Flutter plugin version 37.1.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

Hi @ghenry
I cannot reproduce this on latest stable channel (1.20.3) using a Samsung S10+

Please provide the output of your flutter run -v and some information about the device you are facing this issue.
Thank you

Thanks for testing. I've seen this on my S10e, S7 and Galazy A51. Attached.
flutter_run_verbose_samsung_white_line.txt

A51 attached.
flutter_run_verbose_samsung_white_line_A51.txt

Actually, doesn't show on an S7. A
flutter_run_verbose_samsung_no_white_line_S7.txt
ttached.

Hi all,
I don't know what I've done, but in our own app, after adding in ScreenUtil().setWidth() etc. so all our sizing, the white line has gone. I'll look at the example app I did to demostrate the issue.
Thanks.

Ah, I know. I was changing my phone resolution to test screenutil on a device. Put it back to my usual, and it's still there. False alarm :-(

This is almost certainly antialiasing. When two rectangles of the same color abut over a subpixel boundary and the color below them is different, the color below them will shine through the antialiasing.
If you change the color behind them, does it go away?

They are both default blue though?
On Wed, 16 Sep 2020, 00:55 Ian Hickson<EMAIL_ADDRESS>wrote:

This is almost certainly antialiasing. When two rectangles of the same
color abut over a subpixel boundary and the color below them is different,
the color below them will shine through the antialiasing.
If you change the color behind them, does it go away?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/65631#issuecomment-693089058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG66GGQOB6MLZENQ4JRU3SF75HJANCNFSM4RHKMAGQ
.

@ghenry I think Hixie means to set the Scaffold's backgroundColor to the same color as the appBar

Should that not happen on all resolutions then or only really high ones?
On Wed, 16 Sep 2020, 00:55 Ian Hickson<EMAIL_ADDRESS>wrote:

This is almost certainly antialiasing. When two rectangles of the same
color abut over a subpixel boundary and the color below them is different,
the color below them will shine through the antialiasing.
If you change the color behind them, does it go away?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/65631#issuecomment-693089058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG66GGQOB6MLZENQ4JRU3SF75HJANCNFSM4RHKMAGQ
.

Yep, setting backgroundColor on the Scaffold resolves this. Should have thought of that.
Thanks.
Closing.


the filePushOrder option in not working as documented

Just because the inherited class has a different loading order(alphabetical), it becomes "TypeError: Cannot read property 'prototype' of undefined". Screen shot
I created a sample project to convey this.
Expected Behavior
case OK:
https://github.com/jakenjarvis/clasp-ts-sample/tree/master/IssueSample/Issue01/ProjectNormal
Actual Behavior
case NG:
https://github.com/jakenjarvis/clasp-ts-sample/tree/master/IssueSample/Issue01/ProjectAbnormal
TypeError: Cannot read property 'prototype' of undefined (line 14, file "AClass")
Steps to Reproduce the Problem

Swap the class names of AClass and BClass.
Oh wonder, it becomes TypeError.

Specifications

Windows 10 64bit
node v10.15.0
yarn 1.13.0
@google/clasp 1.6.3
@types/google-apps-script 0.0.36

The TypeScript to Apps Script conversion is done via the ts2gas module. Please file the issue there instead.

Thank you. I reported Issue to ts2gas.

@erickoledadevrel it seems to be in fact a @google/clasp issue.
You can read my reasoning here https://github.com/grant/ts2gas/issues/27
To summarise

the transpiled TypeScript code is correct (i.e. nothing to change in ts2gas)
the filePushOrder option in not working as documented (i.e. order of *.gs files do not comply)

In the name of @jakenjarvis , I'd like this issue to be re-opened.

@jakenjarvis if you have the opportunity to test the above PR which should fix the issue

It seems like the filePushOrder is broken in two places:

in files.ts, getProjectFiles(), the nonIgnoredFilePaths.sort(...) does not sort correctly
in files.ts, pushFiles(), the script.projects.updateContent sends the filesForAPI with applying the sort order from nonIgnoredFilePaths

We can close after clasp is published and the fix is verified.

Note: I've published<EMAIL_ADDRESS>

@kornthing can you please give a status about this issue?


Make types.Image Docker-independent, add docker.GenericImageFromSource

Following up on the discussions in #52 and #58 , this allows using types.Image for non-Docker types.ImageSource backends, instead moving the Docker-specific knowledge into docker.Image and the skopeo inspect implementation.
This will be useful for policy evaluation unit tests, by allowing testing against a dir:… types.Image without having to construct testing-only mocks and reimplement Image functionality like MatchesManifestDigest (#75) in the mocks.
As is, this is not a complete API cleanup (e.g. having the generic Image constructor as docker.GenericImageFromSource is clearly not the right thing to do) but it does move us closer to the desired end state, at least in the implementation if not in the external API.
See the individual commits for detailed comments.

:+1: fine to me for now, feel free to merge it!


Support Streaming ?

is there a way to receive the results realtime by streaming ?

currently , only the final transcript is received (is_final=True)

I want the fastest possible stt result. Any ideas to optimize ?
@RyanChimienti

i found a solution.
setting enforceRealtime = False makes the result much faster.
the result is fetched almost real-time.

Thanks for the reply.
Also, is there a way i can receive only FromCustomer audio input ??
Currently it is by default FromCustomer, ToCustomer 2 channels.
Is only receiving FromCustomer possible? Will this also improve latency?
Thank you @RyanChimienti


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 63 ms

Resolved: ClamAV performance has improved in f206f06.


[Automation] Generate Fluent Lite from storagepool#package-2021-08-01

[Automation] Generate Fluent Lite from storagepool#package-2021-08-01

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-resourcemanager-storagepool


Create a model for cmn-600

[x] Model the coherency protocol correctly: CoreTile, Sliced L3 with home nodes.
[x] Model the Tile interconnect as a pt2pt network between L3 routers, directories, memory controllers.
[x] Model the Tile interconnect as a mesh.

Still need to figure out where memory controllers and dma_controllers go (not logically, rather topologically)


chore: Ship datadog_flutter_plugin 2.7.0

What and why?
Changes related to shipping datadog_flutter_plugin 2.7.0
Review checklist

[ ] This pull request has appropriate unit and / or integration tests
[ ] This pull request references a Github or JIRA issue

Bypassing bitrise checks as something weird is going on with the iOS simulator there.


[Bug]: runtime bundle throws exception in Safari.  TypeError: undefined is not an object (evaluating e[n].call).

System Info
System:
OS: macOS 14.2.1,
CPU: Apple M2 pro
Memory: 16gb
Shell: 5.9 - /bin/zsh
Binaries:
Node: 20.10.0
npm: 10.8.2
Browsers:
Chrome 121.0.6167.160
Safari 18.0
npmPackages:
@rspack/cli: 1.0.8
@rspack/core: 1,0.8
@swc/core: 1.7.26
Details
When running production build in safari 18.0 our app crashes and shows only white screen and in the console an error shows "TypeError: undefined is not an object (evaluating e[n].call)" thrown from our rspack run-time bundle. If we downgrade to 0.7.5 our production build works just fine in safari.
Reproduce link
No response
Reproduce Steps

run production build.
Open app in safari 18 and runtime bundle should throw an TypeError.

I've seen this as well, but it is almost impossible to debug. Sometimes the built bundle does this, sometimes it doesn't, and it is very hard to nail down when.
What I can say is that we've had to pin rspack packages to 1.0.0-beta.1 which works consistently. Anything after that breaks, but not consistently.


Problem with muted players.

Where else have you looked for answers to this question?
I've checked the website and I can't seem to find a fix
Share your question with as much detail as possible?
If players are muted they cannot say yes/no in chat. This makes it so they can't start quests
If applicable, which class or method is this in reference to?
Quests, General

This is a massive problem for me as I don't want anyone to not be able to use this plugin.

Alright I found a fix for anyone with this problem just go into config and set ask-confirmation: false

Although I understand this is not ideal, it is beyond my control. The Bukkit Conversation API (which Quests uses for all prompts) simply does not see the player's input if they are muted.
The best solution is the one you've found - disable the prompt entirely with that config setting. Thanks for using Quests!


Using model an 8-camera dataset

Hello, can the dataset of 8 cameras be tested with the given model, and if so, which files do I need to modify

@fxyQAQ were you able to test it?


Remove caddy, use acme.sh as Let's Encrypt client

This PR addresses some of the concerns in #1282 to drop caddy.
Thanks to the comment by @jimmyye , I discovered https://github.com/Neilpang/acme.sh with which I had great success.
To re-cap, some of the features of acme.sh:

An ACME protocol client written purely in Shell (Unix shell) language.
Full ACME protocol implementation.
Simple, powerful and very easy to use. You only need 3 minutes to learn it.
Bash, dash and sh compatible.
Simplest shell script for Let's Encrypt free certificate client.
Purely written in Shell with no dependencies on python or the official Let's Encrypt client.
Just one script to issue, renew and install your certificates automatically.
DOES NOT require root/sudoer access.
Docker friendly
IPv6 support

I've tested the PR on a fresh AWS EC2 instance, and HTTPS worked with zero additional setup.
Please have a review @jayfk @pydanny @webyneter @luzfcb
This approach may not be perfect but it's definitely promising and cleaner than certbot.

Thanks for your effort @sfdye but for the reasons pointed out above, I'm going to close this PR.
We are going to accept a PR that comes without the usage of nginx-proxy. If you find the time to work on a solution that uses acme.sh only, feel free to open this PR again or create a new one.

@jayfk don't you think we should switch back to Nginx by no means?

I’d love to switch back to nginx, but with a sane implementation only (default https + no privileged docker container).
If someone wants to work on this, please do! I’m currently fighting for every free minute I can get...

@jayfk the problem is how to update the SSL certificates inside a no privileged docker container ?


try reuse Android tooling config from React Native

Try to use Android tooling config from React Native

Thanks for submitting this, @dulmandakh ! Can you explain more what this PR will change and what it affects?

It'll try to use tools and their versions that is used by React Native project or app, which landed with https://github.com/facebook/react-native/commit/0a3055d98a36e49746144e883edc7e20afec4fcb. So the advantage is you don't have to bump tools versions every time RN bumps them, also users don't have to download other versions of Android tools just to compile this module.
For example, RN master landed SDK 27 and uses build tools 27.0.3 with https://github.com/facebook/react-native/commit/044b399e6590d84065a9b186750f77bc9d851aac. If we don't have this configuration RN developers have to download SDK 27 for RN and SDK 26 for this module.

Should we default to 27 ? what do you think @dulmandakh ?
This PR is indeed needed and we will merge it :)

RN 0.57 use SDK 26, so it's safe to use it as default. SDK 27 landed in master, and will be a part of RN 0.58.

If you check there: https://github.com/react-native-community/react-native-releases/pull/40/files it says it will be 27 for 0.57, is this a typo in the changelog?

Ready to merge if @Titozzz gives approval.

By the way, I'm okay dropping support for 0.56 if it's causing issues, @Titozzz .


Extension cleanup

Using default parameters to remove duplicate methods
Changed variables to constants where possible
Infer types where possible

?

Sorry for the misunderstanding. However I could not get the diff file which is not understandable. Can you open new pull request?

You should be able to view the diff under "Files changed" (https://github.com/ridvank/UIImageViewWithWords/pull/3/files). Then, you can just reopen this PR and merge it :)

👍

Thanks for your effort btw:)


feat(progressbar): added non color-elements to ThemeDictionnaries

GitHub Issue: #
https://github.com/unoplatform/Uno.Themes/issues/994
PR Type
What kind of change does this PR introduce?

Feature

Description

Added height and max width to the themedictionnary
PR Checklist
Please check if your PR fulfills the following requirements:

Commits must be following the Conventional Commits specification
[X] Tested UWP
[ ] Tested iOS
[ ] Tested Android
[X] Tested WASM
[ ] Tested MacOS
[ ] Contains No breaking changes

If the pull request contains breaking changes, commit message must contain a detailed description of the action to take for the consumer of this library. As explained by the Conventional Commits specification

Other information

Internal Issue (If applicable):

https://github.com/unoplatform/Uno.Themes/issues/994

There's more change than needed because I put space instead of tabs in the previous pr, and put tabs back in with this PR.

@mergifyio backport legacy/2x


az ad app should be able to set oauth2AllowImplicitFlow=true

To provision or update an AD App Registration which allows the OAuth2 Implicit flow you must either use the PowerShell tools or open the portal to complete this configuration step.
It would be good to have a true cross platform means of doing this with the az ad app commands

I added yesterday through #5852


Run Security Audit job nightly only

Description
Let's run the security audit job daily rather than on every PR and on master. We'll still get alerted of security issues via an issue and will recover the CI status to mean whether or not the various test suite have passed or not.
@mzabaluev What do you think?

I agree with your assessment, let's go ahead with this then and maybe revisit this at a later time (sic)


LFR Generator fails randomly

The LFR generator draws random community sizes from a certain distribution (with the given parameters) and draws random node degrees from a certain distribution (again with given parameters). It then tries to assign nodes to communities. Each node u needs to have (1 - mu) * deg(u) neighbors in its own community. Depending on the random community sizes drawn and the random node degrees drawn and depending on the given parameters, it might be that this assignment is just completely impossible as e.g. the largest community is just too small to fit the node of the highest degree. How likely this is totally depends on the given parameters but it definitely happens relatively frequently with parameters reported in the literature.
The original implementation just merges communities if this happens until all nodes fit. Imho this is very bad as if you choose your parameters unwisely, you end up with completely different community sizes. That's why I did not implement this but just let the generator fail.
Unfortunately, this means that in code where you use the LFR generator, you might need to write loops to try calling the generator several times until it succeeds. I think it would be nice if this was not needed as, as discussed in #170, this is not so nice from a user's perspective.
The code has some sanity checks to test if finding a fitting distribution is plain impossible and fails earlier then. However, I think it is quite possible to supply parameters where finding a fitting distribution is highly unlikely. Therefore, I am not sure if just re-drawing distributions until a fitting distribution is drawn is a good idea. We might of course add more sanity checks to detect such edge cases. Another possibility would be to only re-draw a certain number of times, which would simply make a failure more unlikely (but it would of course still be possible). Due to the (very flexible) interface of the generator I'm also not sure where exactly the re-drawing should happen. However, a different, less flexible but more convenient interface could also be added while properly fixing #26.
Do you have any other suggestions what the desired behavior should be?

Is it possible to detect cases in which the generator fails with high probability (e.g. > 50% or whatever)? In that case we could just fail on those cases and loop on the others. However, given your description, that seems hard to do, so we might need a more complex solution.

I don't think it is easily possible to determine the failure probability. If it is likely to have a community in which the node of maximum degree fits, the generator should succeed in most cases. However, it might also succeed if having a node of such a high degree is not likely.  All of this of course also depends on the exponents of the power law distributions.
I would suggest to have a parameter for the number of retries and set it by default e.g. to 10. I would assume that this should make failure really unlikely for frequently used parameter sets but it would still prevent that the algorithm runs forever. This would require some refactoring though as I think this is not possible with the current interface and data structure.


implement HMR in core (finally)

Summary by CodeRabbit

New Features

Introduced a new custom element "lift-counter" that reacts to changes in the "count" attribute.
Enhanced Hot Module Replacement (HMR) functionality with new options for users to enable or disable it.

Bug Fixes

Updated test cases to accurately reflect changes in component behavior, ensuring correct assertions for the "solid" component.

Documentation

Enhanced verification of properties in test cases for improved clarity and accuracy.

@coderabbitai generate docstrings


Revise savePosition API

This ticket captures various follow-up things for the savePosition Protocol Engine command, originally specified in #8425 and implemented in #8637.
As we understand more about these items, we should feel free to split them up into individual tickets, for independent prioritization.

Semantics

[ ] As currently implemented in #8637, savePosition returns the pipette's current deck coordinates with respect to its "current critical point." The "current critical point" appears to change depending on the last labware visited; see https://github.com/Opentrons/opentrons/pull/8637#discussion_r742326600. So a client can't easily compare and do math on positions from two different labware objects, unless the client has deep knowledge of labware quirks and pipette geometry. Especially since the savePosition result doesn't tell the client which critical point it used.
This happened to be good enough for the initial Labware Position Check implementation, which exclusively compares positions from the same pipette, visiting the same labware. But it's pretty confusing for clients and probably not what we want.

[ ] Implementation detail: Protocol Engine and the underlying hardware API have apparently parallel notions of the "current critical point"? Again, see https://github.com/Opentrons/opentrons/pull/8637#discussion_r742326600. I don't understand what the difference is (if there is a difference), but for the client's benefit, if the client needs to know what "the critical point" is in order to correctly interpret the returned coordinates, we need to specify exactly what that means.

[ ] Specify what it means to get the current position of the pipette other than the one that was most-recently used. Does it return something null-ish? Does it return the actual deck coordinates of the pipette, which is presumably hanging in its home position far above the deck?

API style

[ ] Reconsider whether savePosition needs to be a command at all. The rationale was explained in #8425, and I agree with not making it a separate HTTP endpoint. But I think we missed a third option: including the full Protocol Engine state, including the pipettes' positions, as a built-in part of every command's result. This doesn't necessarily have to inflate our HTTP responses; we have options for giving clients finer-grained control of what's in the response, such as putting the details behind a separate endpoint.
[ ] Reconsider the name savePosition. Maybe snapshotPosition or getPosition?
[ ] Reconsider whether the positionId field is necessary, given the fact that every command already has an id field. See https://github.com/Opentrons/opentrons/pull/8637#discussion_r742315849.

Maybe we could relieve the client of the burden of knowing any details about pipette geometry and critical points by including everything in the response, like:
{
    // Position of some reference point on the pipette mount itself. (The bottom center?)
    "mount": {"x": "...", "y": "...", "z": "..."}

    // For 8-Channel pipettes, the position of the bottom of the back nozzle.
    // For Single-Channel pipettes, the position of the bottom of the only nozzle.
    "backNozzle": {"x": "...", "y": "...", "z": "..."}

    // For 8-Channel pipettes, the position of the bottom of the back tip.
    // For Single-Channel pipettes, the position of the bottom of the only tip.
    // Null if the pipette does not have any tips picked up.
    "backTip": {"x": "...", "y": "...", "z": "..."}

    // For 8-Channel pipettes, the position of the center of all nozzles.
    // For Single-Channel pipettes, equal to backNozzle.
    "nozzlesCenter": {"x": "...", "y": "...", "z": "..."}

    // For 8-Channel pipettes, the position of the center of the bottoms of all tips.
    // For Single-Channel pipettes, equal to backTip.
    // Null if the pipette does not have any tips picked up.
    "tipsCenter": {"x": "...", "y": "...", "z": "..."}
}

Closing this because v5.0.0 will ship with the current API. But this could be reopened as several smaller, more actionable tickets/RFCs for reconsideration.


Before submitting a SEP-8 revised transaction, display the revised transaction operations for user approval

What
Before submitting a SEP-8 revised transaction, display the revised transaction operations for user approval.
How to test
Use the jenkins preview link with /account?secretKey=SBQZPYFQNQD4B7MP7PC3NKCMS5G5LD6PVO7BUYC34M4BBFN6KCXSKSVD and send a small payment to GBID36ML6VVNPIF6SQATQW5QIBREQAVEHQIUMWDLQCSVIYX2IJGK4KPP.
Screenshot

Preview is available here:http://stellar-demo-wallet-pr157.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr157.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr157.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr157.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr157.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr157.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr157.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr157.previews.kube001.services.stellar-ops.com/


Use sc_network::NetworkStateInfo instead of implementing redundant traits

ref #4439

warning: unused variable: `message`
   --> primitives/runtime/src/lib.rs:383:3
    |
375 |       /// A custom error in a module
    |  _____-
376 | |     Module {
377 | |         /// Module index, matching the metadata module index
378 | |         index: u8,
...   |
382 | |         #[codec(skip)]
383 | |         message: Option<&'static str>,
    | |         ^^^^^^-
    | |_______________|
    |                 help: try ignoring the field: `message: _`
    |
    = note: `#[warn(unused_variables)]` on by default

this made the build fail

About the ci fix, since it's very simple and there should exist more specific solution to it, I'd like to get it merged in this pr and try fixing it in another way later.
The problem is rather subtle and urgent - it'll block everyone until it gets fixed.

@kigawas in regards to the CI failures: https://github.com/paritytech/substrate/pull/4469 has been merged within the last 3 hours with a green CI, thus I think current master should be fine now.

Good catch @mxinden, we should not bundle those changes in that PR unless they are necessary. @kigawas could you please revert Cargo.lock changes and resubmit that as a separate PR so that we can easily review them in isolation?

fixed


Backspace no longer works on v0.1.0

I recently updated my code from using a pinned 0.0.23 to 0.1.0 and noticed backspace no longer works in a text input field to delete single characters, but it can delete the enter block of text if selected. I see the same behavior running the test_window_impl example in v0.1.0's imgui-examples directory. In case it matters, this is an older (2013) macbook pro keyboard with a US layout.
Looking at the new support code, it may be related to the Winit support? The 0.0.23 version I used called a method called 'configure_keys()' which doesn't seem to be used in the newer examples, but the current imgui_winit_support docs seem to indicate that has been rolled into init()?. I appreciate any help, and the work you've done so far!

Interesting, I found it works if I hold 'ctrl' while pressing backspace now.

Looking at the new support code, it may be related to the Winit support? The 0.0.23 version I used called a method called 'configure_keys()' which doesn't seem to be used in the newer examples.

On further testing, it still works on my Windows system.

Perhaps a different issue, but bakcspaces in input_float don't appear to be working, and imgui appears to be getting my backspaces and deletes. Even if I remove it from the receivedchar, I still can't get it to register em

I'm still running into this (on windows), is anyone else?

I'm still running into this (on windows), is anyone else?


Fixed crash when using tracker.json from older version

Despite not being marked as nullable, JsonSerializer will still inject null without any warning or error. Now, it'll create a default instance for any missing parts, or throw if the items are missing (as those are pretty critical).

I'm going to approve this, but while these are definitely good changes, should we possibly try to catch some of these situations and display a more specific message to the user?
I know customizing the json has been an advertised feature, so I could see more issues coming up from bad json.

That's a good idea, but that sort of validation would probably require a PR of its own.


telegraf should pass in metadata annotations in the service.yaml

We use annotations to expose the service via a LB.  Would be helpful if telegraf passed in the annotations to the service.yaml like influxdb does.

As of today:

in aws eks this is automatically assigned by annotation
telegraf-ds makes no sense unless plugin exposes sth special

Sorry for not being clear - I was thinking more about the single instance plugins, like the socket listener plugin.  We are using on prem k8s, not AWS, so we have to set the annotation after deployment manually.


scripts: Add throttler details to collect script

Add details of the KSM throttler to the output of the data collection
script. Look for both Kata and Clear Containers KSM throttler details.
Fixes #739.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>(cherry picked from commit 6fc13e9548b6eea65175a47e01b937566bdc92f8)

The fact that the collect script didn't previously record throttler details was an oversight, so I'd argue this was a bug :)

Low impact, risk and effort backport, so I +1 the stable candidate label.

Odd ubuntu ci fail, feels unrelated. I'll nudge the .img one:
# NAME                           READY     STATUS        RESTARTS   AGE
# first-test-6c9475dc95-84xgf    1/1       Terminating   0          35s
# second-test-59c67cfccc-bbfmj   0/1       Terminating   0          24s
Makefile:44: recipe for target 'kubernetes' failed
make: *** [kubernetes] Error 1
Build step 'Execute shell' marked build as failure
Performing Post build task...
Match found for :.* : True
Logical operation result is TRUE
Running script  : #!/bin/bash

k8s tests failing? Unheard of! :)
/me kicks the CI's again...

Ping @kata-containers/runtime - more lovin' needed over here for the stable branch!


Upgrade local dev environment to OpenJDK10 for OSX

https://discourse.brew.sh/t/how-to-install-openjdk-with-brew/712/4
brew tap AdoptOpenJDK/openjdk
brew install adoptopenjdk-openjdk10

Later, it is necessary to upgrade documentation for OSX

java -version
openjdk version "10-internal"
OpenJDK Runtime Environment (build 10-internal+0-adhoc.jenkins.openjdk)
OpenJDK 64-Bit Server VM (build 10-internal+0-adhoc.jenkins.openjdk, mixed mode)


deamon doesn't start after crash - leveldb manifest corrupted

I am running 0.17.0 on Fedora rawhide. After a crash, the deamon fails to start (see message below).
What is the recommended procedure to recover from this ?
Thanks
level=error msg="Error opening memory series storage: leveldb: manifest corrupted (field 'comparer'): missing [file=MANIFEST-000445]" source="main.go:139"

Unless you know how to fix a corrupted leveldb, there is nothing you can do, unfortunately.
LevelDB corruptions upon crashes are rare but if they happen, they essentially mean you have to nuke your storage and start from scratch.
See #651 to track efforts of allowing hot backups.

Unless you know how to fix a corrupted leveldb


Run inference in 16bit mode on GPU

In generate.py if I change load_in_8bit = False if the device is "cuda" then I am getting the following error upon submitting the text -

Traceback (most recent call last):
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/gradio/routes.py", line 393, in run_predict
output = await app.get_blocks().process_api(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/gradio/blocks.py", line 1059, in process_api
result = await self.call_function(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/gradio/blocks.py", line 868, in call_function
prediction = await anyio.to_thread.run_sync(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/anyio/to_thread.py", line 31, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 937, in run_sync_in_worker_thread
return await future
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 867, in run
result = context.run(func, *args)
File "/home/kd/Desktop/proj/ai/hf/llm/alpaca-lora/generate.py", line 100, in evaluate
generation_output = model.generate(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/peft/peft_model.py", line 581, in generate
outputs = self.base_model.generate(**kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/transformers/generation/utils.py", line 1490, in generate
return self.beam_search(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/transformers/generation/utils.py", line 2749, in beam_search
outputs = self(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 765, in forward
outputs = self.model(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 614, in forward
layer_outputs = decoder_layer(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 309, in forward
hidden_states, self_attn_weights, present_key_value = self.self_attn(
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 209, in forward
query_states = self.q_proj(hidden_states).view(bsz, q_len, self.num_heads, self.head_dim).transpose(1, 2)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/peft/tuners/lora.py", line 358, in forward
result += self.lora_B(self.lora_A(self.lora_dropout(x))) * self.scaling
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/home/kd/anaconda3/envs/langchain/lib/python3.10/site-packages/torch/nn/modules/linear.py", line 114, in forward
return F.linear(input, self.weight, self.bias)
RuntimeError: expected scalar type Half but found Float

Should be fixed in latest commit, e04897baaec39280fac97f1ad2bf33059b0df643


Get rid of Joi

We don't want to use 2 validation libraries. Both Joi and Hapi frustrate me a lot.
Usage of Joi on frontend is painful: very big, not frontend-friendly. Official answer is "It was not designed for frontend", but I personally don't see why validation should be backend-only. It's one of the few truly environment-agnostic things. Joi for backend is kinda ok, but still: no changelogs, esoteric error formats, arrogant devs... Time to find something better. Will obsolete https://github.com/Paqmind/react-starter/issues/13

Crap. Just about all libraries are framework bound. jquery-validate, backbone-validate, angular-validate, ember-validate. There is no alternative to Joi right now. So lets enable it back.

Joi is totally broken:

still no custom validators (OMG)
no async validators
no injectable parsers
no proper validation messages
... something I forgot to mention

Will try to convert codebase to use this 3 libs:
https://github.com/gcanti/tcomb
https://github.com/gcanti/tcomb-validation
https://github.com/jquery/globalize

Done


boolean-positional-value-in-call (FBT003) - don't warn when function name starts with set_ and only takes one argument

functions like these should not report the error imo, since the meaning of the bool is self explanatory, which is often the case when the functin starts with set_:
def set_enabled(enabled: bool):
    ...

set_enabled(True) # self explanatory
set_enabled(enabled=True) # unnecessary/redundant

We may want to do this, it would also help with https://github.com/astral-sh/ruff/issues/9287 and https://github.com/astral-sh/ruff/issues/3247#issuecomment-1842529367.

Maybe also allow the use_ prefix? I encountered that with torch.use_deterministic_algorithms. The following feels a bit silly:
torch.use_deterministic_algorithms(mode=True)

I often use flag-holding context vars.
from contextvars import ContextVar

cv = ContextVar("cv", default=False)
cv.set(True)  # triggers FBT003

What a pity the fix for this doesn't apply to ContextVar.set.

it looks like the check expects the function name to start with "set" followed by either an underscore or an uppercase character, but not "set" on its own. it should probably be updated to allow that too.

It seems reasonable to expand it to cover that case.


option to group maldi spectra based on plate map; barebones mzml meta…

…data option

Group spectra from different spots into the same mzML file based on sample name in MALDI plate map. Option used for grouping data for downstream analysis (i.e. IDBac). Also added flag for barebones mzML metadata in case downstream analysis programs are not compatible with certain CV params (i.e. CVs for "timsTOF" or "psims").